fix: stdout with line-break and strings

source/mite-amend.js

@@ -219,7 +219,7 @@ async function main(timeEntryId, note) {
   return getTimeEntryData(timeEntryId)
     .then(getUpdatedTimeEntryData.bind(this, program.opts(), note))
     .then(entry => updateTimeEntry(entry.id, entry).then(() => entry))
-    .then(entry => process.stdout.write(`Successfully modified note of time entry (id: ${entry.id})`))
+    .then(entry => process.stdout.write(`Successfully modified note of time entry (id: ${entry.id})\n`))
     .catch(handleError);
 }
 

source/mite-config.js

@@ -44,9 +44,9 @@ Examples:
           throw new GeneralError(err.message);
         }
         if (value === undefined) {
-          process.stdout.write(`successfully set "${key}" to the default value`);
+          process.stdout.write(`successfully set "${key}" to the default value\n`);
         } else {
-          process.stdout.write(`successfully set "${key}" to "${value}"`);
+          process.stdout.write(`successfully set "${key}" to "${value}"\n`);
         }
         // make sure file is only write- and readable by the user
         const configFilename = config.stores.file.file;
@@ -66,11 +66,11 @@ Examples:
   Get the current value of the "account" config variable:
     mite config get account
   `)
-  .action((key) => process.stdout.write(config.get(key)));
+  .action((key) => process.stdout.write(JSON.stringify(config.get(key)) + '\n'));
 
 program.command('list')
   .description('list all currently defined config vars')
-  .action(() => process.stdout.write(config.get()));
+  .action(() => process.stdout.write(JSON.stringify(config.get(), null, 2) + '\n'));
 
 program.parse();
 

source/mite-customer-delete.js

@@ -35,7 +35,7 @@ function main(customerId) {
   }
   const mite = miteApi(config.get());
   return util.promisify(mite.deleteCustomer)(customerId)
-    .then(() => process.stdout.write(`Successfully deleted customer (id: ${customerId})`))
+    .then(() => process.stdout.write(`Successfully deleted customer (id: ${customerId})\n`))
     .catch(handleError);
 }
 

source/mite-customer-list.js

@@ -83,7 +83,7 @@ async function main() {
       const format = DataOutput.getFormatFromOptions(opts, config);
       const columns = commandOptions.columns.resolve(opts.columns, customersCommand.columns.options);
       const tableData = DataOutput.compileTableData(items, columns, format);
-      process.stdout.write(DataOutput.formatData(tableData, format, columns));
+      process.stdout.write(DataOutput.formatData(tableData, format, columns) + '\n');
     })
     .catch(handleError);
 } // main

source/mite-customer-new.js

@@ -53,7 +53,7 @@ function main(name, opts) {
       process.stdout.write(`Successfully created customer (id: ${customerId}).
 
 Please use web-interface to modify complicated service & hourly rates settings:
-https://${config.get('account')}.mite.yo.lk/customers/${customerId}/edit`);
+https://${config.get('account')}.mite.yo.lk/customers/${customerId}/edit\n`);
     });
 }
 

source/mite-customer-update.js

@@ -68,7 +68,7 @@ function main(customerId, opts) {
   };
 
   return util.promisify(mite.updateCustomer)(customerId, data)
-    .then(() => process.stdout.write(`Successfully updated customer (id: ${customerId})`))
+    .then(() => process.stdout.write(`Successfully updated customer (id: ${customerId})\n`))
     .catch(handleError);
 }
 

source/mite-delete.js

@@ -35,7 +35,7 @@ function main(timeEntryId) {
       const message = (err && err.message) ? err.message : err;
       handleError(new Error(`Error while deleted time entry (id: ${timeEntryId}) ${message}`));
     }
-    process.stdout.write(timeEntryId);
+    process.stdout.write(`${timeEntryId}\n`);
   });
 }
 

source/mite-list.js

@@ -274,7 +274,7 @@ function main(period) {
       ;
     const format = DataOutput.getFormatFromOptions(opts, config);
     const report = getReport(items, columns, format, format === FORMAT.TABLE);
-    process.stdout.write(report);
+    process.stdout.write(`${report}\n`);
   });
 } // main
 

source/mite-lock.js

@@ -46,7 +46,7 @@ function main(timeEntryId) {
     ...(typeof opts.force === 'boolean' && { force: opts.force })
   };
   return util.promisify(mite.updateTimeEntry)(timeEntryId, data)
-    .then(() => process.stdout.write(`Successfully locked time entry (id: ${timeEntryId})`))
+    .then(() => process.stdout.write(`Successfully locked time entry (id: ${timeEntryId})\n`))
     .catch(handleError);
 }
 

source/mite-new.js

@@ -145,18 +145,18 @@ function checkResults(options, query, type) {
 
   switch (searchResults.length) {
     case 0:
-      process.stdout.write(`No ${type}s found that match "${query}".`);
+      process.stdout.write(`No ${type}s found that match "${query}".\n`);
       break;
     case 1:
       return searchResults;
     default:
-      process.stdout.write(`Found multiple ${type}s matching  "${query}":`);
-      searchResults.forEach(current => process.stdout.write(`- ${current.name}`));
+      process.stdout.write(`Found multiple ${type}s matching  "${query}":\n`);
+      searchResults.forEach(current => process.stdout.write(`- ${current.name}\n`));
       break;
   }
   process.stdout.write(
     `Use the exact name of an existing ${type}s. List available ${type}s `+
-    `using "mite ${type}s"`
+    `using "mite ${type}s"\n`
   );
   process.exit(1);
 }
@@ -263,11 +263,11 @@ async function main(note, project, service, minutes, date) {
     if (startTracker) {
       return miteTracker.start(timeEntryId)
         .then(() => {
-          process.stdout.write(timeEntryId);
+          process.stdout.write(`${timeEntryId}\n`);
           process.exit(0);
         });
     }
-    process.stdout.write(timeEntryId);
+    process.stdout.write(`${timeEntryId}\n`);
     process.exit(0);
   })
   .catch(handleError);

source/mite-open.js

@@ -48,7 +48,7 @@ function main(timeEntryId) {
   })
   .then((entry) => {
     let url = `https://${config.get('account')}.mite.yo.lk/`;
-    process.stdout.write('No time entry id given, opening the organisation’s account');
+    process.stdout.write('No time entry id given, opening the organisation’s account\n');
     if (entry) {
       url += 'daily/#' + (entry.date_at).replace('-', '/') + '?open=time_entry_' + entry.id;
     }

source/mite-project-delete.js

@@ -35,7 +35,7 @@ function main(projectId) {
   }
   const mite = miteApi(config.get());
   return util.promisify(mite.deleteProject)(projectId)
-    .then(() => process.stdout.write(`Successfully deleted project (id: ${projectId})`))
+    .then(() => process.stdout.write(`Successfully deleted project (id: ${projectId})\n`))
     .catch(handleError);
 }
 

source/mite-project-list.js

@@ -123,7 +123,7 @@ async function main() {
       const format = DataOutput.getFormatFromOptions(opts, config);
       const columns = commandOptions.columns.resolve(opts.columns, projectsCommand.columns.options);
       const tableData = DataOutput.compileTableData(items, columns, format);
-      process.stdout.write(DataOutput.formatData(tableData, format, columns));
+      process.stdout.write(DataOutput.formatData(tableData, format, columns) + '\n');
     })
     .catch(handleError);
 } // main

source/mite-project-new.js

@@ -66,7 +66,7 @@ function main(name, opts) {
       process.stdout.write(`Successfully created project (id: ${projectId}).
 
 Please use web-interface to modify complicated service & hourly rates settings:
-https://${config.get('account')}.mite.yo.lk/reports/projects/${projectId}`);
+https://${config.get('account')}.mite.yo.lk/reports/projects/${projectId}\n`);
     });
 }
 

source/mite-project-update.js

@@ -74,7 +74,7 @@ function main(projectId) {
   };
 
   return util.promisify(mite.updateProject)(projectId, data)
-    .then(() => process.stdout.write(`Successfully updated project (id: ${projectId})`))
+    .then(() => process.stdout.write(`Successfully updated project (id: ${projectId})\n`))
     .catch(handleError);
 }
 

source/mite-service-delete.js

@@ -32,7 +32,7 @@ function main(serviceId) {
   }
   const mite = miteApi(config.get());
   return util.promisify(mite.deleteService)(serviceId)
-    .then(() => process.stdout.write(`Successfully deleted service (id: ${serviceId})`))
+    .then(() => process.stdout.write(`Successfully deleted service (id: ${serviceId})\n`))
     .catch(handleError);
 }
 

source/mite-service-list.js

@@ -74,7 +74,7 @@ async function main() {
       const format = DataOutput.getFormatFromOptions(opts, config);
       const columns = commandOptions.columns.resolve(opts.columns, servicesCommand.columns.options);
       const tableData = DataOutput.compileTableData(items, columns, format);
-      process.stdout.write(DataOutput.formatData(tableData, format, columns));
+      process.stdout.write(DataOutput.formatData(tableData, format, columns) + '\n');
     })
     .catch(handleError);
 } // main

source/mite-service-update.js

@@ -74,7 +74,7 @@ function main(serviceId) {
     ...(typeof opts.updateEntries === 'boolean' && { update_hourly_rate_on_time_entries: true }),
   };
   return util.promisify(mite.updateProject)(serviceId, data)
-    .then(() => process.stdout.write(`Successfully updated service (id: ${serviceId})`))
+    .then(() => process.stdout.write(`Successfully updated service (id: ${serviceId})\n`))
     .catch(handleError);
 }
 

source/mite-start.js

@@ -45,7 +45,7 @@ async function main(timeEntryId) {
   }
   mite.tracker.start(timeEntryId)
     // output the id of the entry which was started for piping
-    .then(() => process.stdout.write(timeEntryId))
+    .then(() => process.stdout.write(`${timeEntryId}\n`))
     .catch(handleError);
 }
 

source/mite-stop.js

@@ -10,13 +10,13 @@ const { handleError, GeneralError } = require('./lib/errors');
 function main() {
   const miteTracker = require('./lib/mite-tracker')(config.get());
   return miteTracker.get()
-    .then(id => {
-      if (!id) {
+    .then(timeEntryId => {
+      if (!timeEntryId) {
         throw new GeneralError('No running time entry found.');
       }
-      return miteTracker.stop(id);
+      return miteTracker.stop(timeEntryId);
     })
-    .then((id) => process.stdout.write(id))
+    .then((timeEntryId) => process.stdout.write(`${timeEntryId}\n`))
     .catch(handleError);
 }
 

source/mite-unlock.js

@@ -46,7 +46,7 @@ function main (timeEntryId) {
     ...(typeof opts.force === 'boolean' && { force: opts.force })
   };
   return util.promisify(mite.updateTimeEntry)(timeEntryId, data)
-    .then(() => process.stdout.write(timeEntryId))
+    .then(() => process.stdout.write(`${timeEntryId}\n`))
     .catch(handleError);
 }
 

source/mite-users.js

@@ -127,7 +127,7 @@ async function main() {
       const format = DataOutput.getFormatFromOptions(opts, config);
       const columns = commandOptions.columns.resolve(opts.columns, usersCommand.columns.options);
       const tableData = DataOutput.compileTableData(items, columns, format);
-      process.stdout.write(DataOutput.formatData(tableData, format, columns));
+      process.stdout.write(DataOutput.formatData(tableData, format, columns) + '\n');
     })
     .catch(handleError);
 } // main