Surface chrome webstore error details

source/cli.js

@@ -1,11 +1,11 @@
 #!/usr/bin/env node
 
 import nodePath from 'node:path';
-import process from 'node:process';
 import meow from 'meow';
 import createConfig from './config.js';
 import { upload, publish, fetchToken } from './wrapper.js';
 import { isUploadSuccess, handlePublishStatus } from './util.js';
+import { handleError } from './error-handler.js';
 
 const cli = meow(`
     Usage
@@ -121,24 +121,6 @@ async function doPublish() {
     handlePublishStatus(response);
 }
 
-function errorHandler(error) {
-    process.exitCode = 1;
-
-    // Handle CWSError from chrome-webstore-upload package
-    if (error?.name === 'CWSError') {
-        console.error(`❌ ${error.message}`);
-        console.error('Does the dev console require changes?');
-        console.error('https://chrome.google.com/webstore/devconsole');
-        console.error('');
-        console.error('Did you follow the guide to generate the keys?');
-        console.error('https://github.com/fregante/chrome-webstore-upload-keys');
-        return;
-    }
-
-    // Fallback for unknown error types
-    console.log(error);
-}
-
 async function init() {
     if (isUpload && autoPublish) {
         await doAutoPublish();
@@ -152,5 +134,5 @@ async function init() {
 try {
     await init();
 } catch (error) {
-    errorHandler(error);
+    handleError(error);
 }

source/error-handler.js

@@ -0,0 +1,81 @@
+import process from 'node:process';
+
+export function extractDetailLines(details) {
+    const detailItems = Array.isArray(details)
+        ? details
+        : (details ? [details] : []);
+    const lines = [];
+
+    for (const detail of detailItems) {
+        if (typeof detail === 'string') {
+            const message = detail.trim();
+
+            if (message) {
+                lines.push(message);
+            }
+
+            continue;
+        }
+
+        if (!detail || typeof detail !== 'object') {
+            continue;
+        }
+
+        if (Array.isArray(detail.fieldViolations)) {
+            for (const violation of detail.fieldViolations) {
+                if (!violation || typeof violation !== 'object') {
+                    continue;
+                }
+
+                const field = typeof violation.field === 'string' ? violation.field.trim() : '';
+                const description = typeof violation.description === 'string' ? violation.description.trim() : '';
+                const reason = typeof violation.reason === 'string' ? violation.reason.trim() : '';
+                const message = [description, reason].filter(Boolean).join(' ');
+
+                if (!message) {
+                    continue;
+                }
+
+                lines.push(field ? `${field}: ${message}` : message);
+            }
+
+            continue;
+        }
+
+        if (typeof detail.message === 'string') {
+            lines.push(detail.message);
+            continue;
+        }
+
+        if (typeof detail.reason === 'string') {
+            lines.push(detail.reason);
+        }
+    }
+
+    return [...new Set(lines)];
+}
+
+export function handleError(error) {
+    process.exitCode = 1;
+
+    if (error?.name === 'CWSError') {
+        console.error(`❌ ${error.message}`);
+
+        const detailLines = extractDetailLines(error.details);
+        if (detailLines.length > 0) {
+            console.error('Details:');
+            for (const line of detailLines) {
+                console.error(`- ${line}`);
+            }
+        }
+
+        console.error('Does the dev console require changes?');
+        console.error('https://chrome.google.com/webstore/devconsole');
+        console.error('');
+        console.error('Did you follow the guide to generate the keys?');
+        console.error('https://github.com/fregante/chrome-webstore-upload-keys');
+        return;
+    }
+
+    console.log(error);
+}

source/error-handler.test.js

@@ -1,6 +1,7 @@
 import process from 'node:process';
 import test from 'ava';
 import { stubConsoleLog } from '../test/helpers/stubs.js';
+import { extractDetailLines, handleError } from './error-handler.js';
 
 test('CWSError displays message with ❌ prefix and helpful links', t => {
     const logs = [];
@@ -26,15 +27,7 @@ test('CWSError displays message with ❌ prefix and helpful links', t => {
 
     process.exitCode = 0;
 
-    // Simulate error handling
-    if (error?.name === 'CWSError') {
-        console.error(`❌ ${error.message}`);
-        console.error('Does the dev console require changes?');
-        console.error('https://chrome.google.com/webstore/devconsole');
-        console.error('');
-        console.error('Did you follow the guide to generate the keys?');
-        console.error('https://github.com/fregante/chrome-webstore-upload-keys');
-    }
+    handleError(error);
 
     restoreLog();
     restoreError();
@@ -46,6 +39,9 @@ test('CWSError displays message with ❌ prefix and helpful links', t => {
     t.is(errors[3], '');
     t.is(errors[4], 'Did you follow the guide to generate the keys?');
     t.is(errors[5], 'https://github.com/fregante/chrome-webstore-upload-keys');
+
+    t.is(process.exitCode, 1);
+    process.exitCode = 0;
 });
 
 test('CWSError with publish-related message shows both links', t => {
@@ -68,15 +64,7 @@ test('CWSError with publish-related message shows both links', t => {
 
     process.exitCode = 0;
 
-    // Simulate error handling
-    if (error?.name === 'CWSError') {
-        console.error(`❌ ${error.message}`);
-        console.error('Does the dev console require changes?');
-        console.error('https://chrome.google.com/webstore/devconsole');
-        console.error('');
-        console.error('Did you follow the guide to generate the keys?');
-        console.error('https://github.com/fregante/chrome-webstore-upload-keys');
-    }
+    handleError(error);
 
     restoreError();
 
@@ -87,6 +75,9 @@ test('CWSError with publish-related message shows both links', t => {
     t.is(errors[3], '');
     t.is(errors[4], 'Did you follow the guide to generate the keys?');
     t.is(errors[5], 'https://github.com/fregante/chrome-webstore-upload-keys');
+
+    t.is(process.exitCode, 1);
+    process.exitCode = 0;
 });
 
 test('CWSError with any message shows both links', t => {
@@ -109,15 +100,7 @@ test('CWSError with any message shows both links', t => {
 
     process.exitCode = 0;
 
-    // Simulate error handling
-    if (error?.name === 'CWSError') {
-        console.error(`❌ ${error.message}`);
-        console.error('Does the dev console require changes?');
-        console.error('https://chrome.google.com/webstore/devconsole');
-        console.error('');
-        console.error('Did you follow the guide to generate the keys?');
-        console.error('https://github.com/fregante/chrome-webstore-upload-keys');
-    }
+    handleError(error);
 
     restoreError();
 
@@ -128,5 +111,77 @@ test('CWSError with any message shows both links', t => {
     t.is(errors[3], '');
     t.is(errors[4], 'Did you follow the guide to generate the keys?');
     t.is(errors[5], 'https://github.com/fregante/chrome-webstore-upload-keys');
+
+    t.is(process.exitCode, 1);
+    process.exitCode = 0;
 });
 
+test('extractDetailLines returns BadRequest field violations', t => {
+    const details = [
+        {
+            '@type': 'type.googleapis.com/google.rpc.BadRequest',
+            fieldViolations: [
+                {
+                    field: 'media',
+                    description: 'The manifest has an invalid version: 0.0.0.',
+                    reason: 'PKG_MANIFEST_PARSE_ERROR',
+                },
+            ],
+        },
+    ];
+
+    t.deepEqual(extractDetailLines(details), [
+        'media: The manifest has an invalid version: 0.0.0. PKG_MANIFEST_PARSE_ERROR',
+    ]);
+});
+
+test('extractDetailLines returns localized messages and reasons', t => {
+    const details = [
+        {
+            '@type': 'type.googleapis.com/google.rpc.LocalizedMessage',
+            message: 'The uploaded package was invalid.',
+        },
+        {
+            '@type': 'type.googleapis.com/google.rpc.ErrorInfo',
+            reason: 'INVALID_PACKAGE',
+        },
+    ];
+
+    t.deepEqual(extractDetailLines(details), [
+        'The uploaded package was invalid.',
+        'INVALID_PACKAGE',
+    ]);
+});
+
+test('extractDetailLines ignores empty entries', t => {
+    const details = [
+        null,
+        {
+            '@type': 'type.googleapis.com/google.rpc.BadRequest',
+            fieldViolations: [
+                {
+                    field: 'media',
+                    description: '   ',
+                },
+            ],
+        },
+        {
+            '@type': 'type.googleapis.com/google.rpc.ErrorInfo',
+        },
+    ];
+
+    t.deepEqual(extractDetailLines(details), []);
+});
+
+test('extractDetailLines includes non-empty string details', t => {
+    const details = [
+        '  Additional detail line  ',
+        '',
+        'Second line',
+    ];
+
+    t.deepEqual(extractDetailLines(details), [
+        'Additional detail line',
+        'Second line',
+    ]);
+});