Normalize error messages

Author: yuanchuan

bin/handler.js

@@ -35,13 +35,13 @@ export async function handleRender(source, options) {
 export async function handleParse(source) {
     let { content, error } = await read(source);
     if (error) {
-        console.log(error.message);
+        console.warn(error.message);
         process.exit(1);
     } else {
         try {
             console.log(JSON.stringify(parse(content), null, 2));
         }  catch (e) {
-            console.log(e.message);
+            console.error(e.message);
             process.exit(1);
         }
     }
@@ -61,14 +61,14 @@ export async function handlePreview(source, options) {
 export async function handleGenerateSVG(source) {
     let { content, error } = await read(source);
     if (error) {
-        console.log(error.message);
+        console.error(error.message);
         process.exit(1);
     } else {
         content = content.trim();
         if (/^svg\s*\{/i.test(content) || !content.length) {
             console.log(generateSVG(content));
         } else {
-            console.log('Not a valid SVG format');
+            console.warn('warn: invalid SVG format');
         }
     }
 }
@@ -83,13 +83,17 @@ export async function handleGenerateShape(source) {
     }
 }
 
-export function handleSetConfig(field, value) {
-   config[field] = value;
-   if (value === '') {
-       delete config[field];
-   }
-   fs.writeFileSync(configPath, JSON.stringify(config, null, 2));
-   console.log('OK')
+export async function handleSetConfig(field, value) {
+    config[field] = value;
+    if (value === '') {
+        delete config[field];
+    }
+    try {
+        fs.writeFileSync(configPath, JSON.stringify(config, null, 2));
+        console.log('ok');
+    } catch (e) {
+        console.error('error: failed to write config file');
+    }
 }
 
 export function handleDisplayConfig(field) {
@@ -109,9 +113,9 @@ async function read(path) {
     let content = '';
     let error = null;
     if (path === undefined) {
-        console.log('No source file specified, reading from stdin...');
+        console.log('No source file specified, reading from stdin:');
         let key = os.platform() === 'win32' ? 'CTRL+Z' : 'CTRL+D';
-        console.log(`Press ${key} to finish input.\n`);
+        console.log(`(Press ${key} to finish input.)\n`);
         try {
             content = await readFromStdin();
         } catch (e) {

src/preview/client.html

@@ -52,7 +52,7 @@
             try {
                 input = JSON.parse(e.data);
             } catch(e) {
-                console.error(e);
+                console.error(e.message);
             }
             let { data, type } = input;
             switch (type) {

src/preview/server.js

@@ -59,16 +59,16 @@ wsServer.on('connection', (ws, req) => {
         data: read(sourceFile)
     }));
 
-    ws.on('error', err => {
-        console.error(err);
+    ws.on('error', e => {
+        console.error(e.message);
     });
 
     ws.on('message', (e) => {
         let input = {};
         try {
             input = JSON.parse(e);
         } catch(e) {
-            console.error(e);
+            console.error(e.message);
         }
         if (input.type === 'ping') {
             clearTimeout(timers[query.get('instance')]);

src/static.js

@@ -46,8 +46,8 @@ export function getCssDoodleLib() {
         if (fs.existsSync(libPath)) {
             return fs.readFileSync(libPath, 'utf8');
         } else {
-            console.warn(`[warn] css-doodle not found: ${libPath}. Use default css-doodle instead.`);
-            console.info('[info] Please check it with `config` command.\n');
+            console.warn(`warn: css-doodle not found: ${libPath}. Use default css-doodle instead.`);
+            console.info('Please check it with `config` command.\n');
         }
     }
     return read('../node_modules/css-doodle/css-doodle.min.js');
@@ -62,8 +62,8 @@ export function getBrowserPath() {
         if (fs.existsSync(browserPath)) {
             return browserPath;
         } else {
-            console.warn(`[warn] Browser not found: ${browserPath}. Use default browser instead.`);
-            console.info('[info] Please check it with `config` command.\n');
+            console.warn(`warn: browser not found: ${browserPath}. Use default browser instead.`);
+            console.info('Please check it with `config` command.\n');
         }
     }
     return '';