git-coco 0.40.0 → 0.40.1

package/dist/index.esm.mjs

@@ -53,7 +53,7 @@ import { pathToFileURL } from 'url';
 /**
  * Current build version from package.json
  */
-const BUILD_VERSION = "0.40.0";
+const BUILD_VERSION = "0.40.1";
 
 const isInteractive = (config) => {
     return config?.mode === 'interactive' || !!config?.interactive;
@@ -2127,11 +2127,21 @@ function formatAuthenticationError(error, logger) {
     logger.verbose(`\nOriginal error: ${error.message}`, { color: 'gray' });
 }
 /**
- * Formats a generic error
+ * Formats a generic error.
+ *
+ * The error message prints unconditionally (was previously gated behind
+ * `--verbose`, which left users staring at a "Failed to execute command"
+ * line with no actionable detail when something crashed). The full stack
+ * trace stays under `logger.verbose` so plain output stays focused on the
+ * one-line cause; users running into something they can't diagnose can opt
+ * in with `--verbose` for the trace.
  */
 function formatGenericError(error, logger) {
     logger.log('\nFailed to execute command', { color: 'yellow' });
-    logger.verbose(`\nError: "${error.message}"`, { color: 'red' });
+    logger.log(`\nError: ${error.message}`, { color: 'red' });
+    if (error.stack) {
+        logger.verbose(`\n${error.stack}`, { color: 'gray' });
+    }
 }
 function commandExecutor(handler) {
     return async (argv) => {
@@ -17932,6 +17942,18 @@ function getLogInkInputEvents(state, inputValue, key = {}, context = {}) {
     if (key.rightArrow && state.focus === 'sidebar') {
         return [action({ type: 'nextSidebarTab' })];
     }
+    // ←/→ on the inspector switch between the [Inspector] / [Actions]
+    // tabs, mirroring the sidebar's left/right tab semantics. `[` and
+    // `]` still work as keyboard alternatives, but the visible hint in
+    // the inspector chrome shows ←/→ because the bracketed `[/]`
+    // notation reads as "press the / key" — which is the global filter
+    // trigger and was making users think the binding was busted.
+    if (key.leftArrow && state.focus === 'detail') {
+        return [action({ type: 'setInspectorTab', value: 'inspector' })];
+    }
+    if (key.rightArrow && state.focus === 'detail') {
+        return [action({ type: 'setInspectorTab', value: 'actions' })];
+    }
     // ←/→ on the status surface jump between the staged / unstaged /
     // untracked groups — the horizontal axis is "between groups", the
     // vertical axis (↑/↓ below) is "within the active group's files".
@@ -26778,22 +26800,39 @@ function renderHistoryInspector(h, components, state, context, _contextStatus, d
         h(Text, { key: 'detail-spacer-3' }, ''),
         h(Text, { key: 'detail-files-title' }, 'Changed files:'),
     ];
+    // Single-cursor invariant: the file list owns the cursor when the
+    // inspector tab is active; the actions list owns it when the actions
+    // tab is active. Pass `focused` only for the matching tab so users
+    // never see two simultaneous selection highlights inside the panel.
+    const fileListFocused = focused && state.inspectorTab === 'inspector';
     const fileListMaxRows = Math.max(4, Math.min(detail.files.length, 10));
-    const fileListNodes = renderCommitFileList(h, Text, detail.files, state.selectedFileIndex, focused, fileListMaxRows, width, theme);
-    // Tabbed mode (#806 follow-up — short terminals): render only the
-    // active inspector tab with a `[Inspector] Actions` header so the
-    // user knows what they're seeing and how to switch (`[/]` while
-    // focus is on the inspector). Tall terminals stack both sections
-    // as before.
+    const fileListNodes = renderCommitFileList(h, Text, detail.files, state.selectedFileIndex, fileListFocused, fileListMaxRows, width, theme);
+    // Tab indicator. Renders in BOTH tabbed (short-terminal) mode and
+    // tall-stacked mode so the user can always see which tab the cursor
+    // owns and learn the `[/]` toggle. Without this on tall terminals,
+    // the actions list looked like a static cheat-sheet — there was no
+    // visible signal that the cursor could move into it.
+    //
+    // Spacing between tab labels comes from the labels' own padding
+    // (the active label is bracketed `[Inspector]` while the inactive
+    // one is space-padded ` Inspector `, so adjacency reads cleanly).
+    // Earlier revisions stuck a raw `' '` between the Text children to
+    // pad them visually — that crashes Ink at first paint with
+    // "Text string ' ' must be rendered inside <Text> component"
+    // because Box only accepts component children, never bare strings.
+    const activeTab = state.inspectorTab;
+    const tabHeader = h(Box, { key: 'inspector-tabs', flexDirection: 'row' }, h(Text, {
+        bold: activeTab === 'inspector',
+        dimColor: activeTab !== 'inspector',
+    }, activeTab === 'inspector' ? '[Inspector]' : ' Inspector '), h(Text, {
+        bold: activeTab === 'actions',
+        dimColor: activeTab !== 'actions',
+    }, activeTab === 'actions' ? '[Actions]' : ' Actions '), ...(focused
+        ? [h(Text, { key: 'inspector-tabs-hint', dimColor: true }, '  · ←/→ switch')]
+        : []));
+    // Tabbed mode (short terminals): render only the active tab's
+    // content under the tab header.
     if (tabbed) {
-        const activeTab = state.inspectorTab;
-        const tabHeader = h(Box, { key: 'inspector-tabs', flexDirection: 'row' }, h(Text, {
-            bold: activeTab === 'inspector',
-            dimColor: activeTab !== 'inspector',
-        }, activeTab === 'inspector' ? '[Inspector]' : ' Inspector '), ' ', h(Text, {
-            bold: activeTab === 'actions',
-            dimColor: activeTab !== 'actions',
-        }, activeTab === 'actions' ? '[Actions]' : ' Actions '));
         return h(Box, {
             borderColor: focusBorderColor(theme, focused),
             borderStyle: theme.borderStyle,
@@ -26807,13 +26846,16 @@ function renderHistoryInspector(h, components, state, context, _contextStatus, d
                 cursorActive: focused,
             })));
     }
+    // Tall mode: stack both sections so the user can read everything at
+    // once, but show the tab header so the active section (and the
+    // `[/]` switch affordance) is visible.
     return h(Box, {
         borderColor: focusBorderColor(theme, focused),
         borderStyle: theme.borderStyle,
         flexDirection: 'column',
         width,
         paddingX: 1,
-    }, h(Text, { bold: true }, panelTitle('Inspector', focused)), ...headerNodes, ...fileListNodes, ...renderInspectorActionsSection(h, Text, 'history-commit', width, theme, {
+    }, h(Text, { bold: true }, panelTitle('Inspector', focused)), tabHeader, h(Text, { key: 'inspector-tabs-spacer' }, ''), ...headerNodes, ...fileListNodes, ...renderInspectorActionsSection(h, Text, 'history-commit', width, theme, {
         cursorIndex: state.inspectorActionIndex,
         cursorActive: focused && state.inspectorTab === 'actions',
     }));

package/dist/index.js

@@ -78,7 +78,7 @@ var readline__namespace = /*#__PURE__*/_interopNamespaceDefault(readline);
 /**
  * Current build version from package.json
  */
-const BUILD_VERSION = "0.40.0";
+const BUILD_VERSION = "0.40.1";
 
 const isInteractive = (config) => {
     return config?.mode === 'interactive' || !!config?.interactive;
@@ -2152,11 +2152,21 @@ function formatAuthenticationError(error, logger) {
     logger.verbose(`\nOriginal error: ${error.message}`, { color: 'gray' });
 }
 /**
- * Formats a generic error
+ * Formats a generic error.
+ *
+ * The error message prints unconditionally (was previously gated behind
+ * `--verbose`, which left users staring at a "Failed to execute command"
+ * line with no actionable detail when something crashed). The full stack
+ * trace stays under `logger.verbose` so plain output stays focused on the
+ * one-line cause; users running into something they can't diagnose can opt
+ * in with `--verbose` for the trace.
  */
 function formatGenericError(error, logger) {
     logger.log('\nFailed to execute command', { color: 'yellow' });
-    logger.verbose(`\nError: "${error.message}"`, { color: 'red' });
+    logger.log(`\nError: ${error.message}`, { color: 'red' });
+    if (error.stack) {
+        logger.verbose(`\n${error.stack}`, { color: 'gray' });
+    }
 }
 function commandExecutor(handler) {
     return async (argv) => {
@@ -17957,6 +17967,18 @@ function getLogInkInputEvents(state, inputValue, key = {}, context = {}) {
     if (key.rightArrow && state.focus === 'sidebar') {
         return [action({ type: 'nextSidebarTab' })];
     }
+    // ←/→ on the inspector switch between the [Inspector] / [Actions]
+    // tabs, mirroring the sidebar's left/right tab semantics. `[` and
+    // `]` still work as keyboard alternatives, but the visible hint in
+    // the inspector chrome shows ←/→ because the bracketed `[/]`
+    // notation reads as "press the / key" — which is the global filter
+    // trigger and was making users think the binding was busted.
+    if (key.leftArrow && state.focus === 'detail') {
+        return [action({ type: 'setInspectorTab', value: 'inspector' })];
+    }
+    if (key.rightArrow && state.focus === 'detail') {
+        return [action({ type: 'setInspectorTab', value: 'actions' })];
+    }
     // ←/→ on the status surface jump between the staged / unstaged /
     // untracked groups — the horizontal axis is "between groups", the
     // vertical axis (↑/↓ below) is "within the active group's files".
@@ -26803,22 +26825,39 @@ function renderHistoryInspector(h, components, state, context, _contextStatus, d
         h(Text, { key: 'detail-spacer-3' }, ''),
         h(Text, { key: 'detail-files-title' }, 'Changed files:'),
     ];
+    // Single-cursor invariant: the file list owns the cursor when the
+    // inspector tab is active; the actions list owns it when the actions
+    // tab is active. Pass `focused` only for the matching tab so users
+    // never see two simultaneous selection highlights inside the panel.
+    const fileListFocused = focused && state.inspectorTab === 'inspector';
     const fileListMaxRows = Math.max(4, Math.min(detail.files.length, 10));
-    const fileListNodes = renderCommitFileList(h, Text, detail.files, state.selectedFileIndex, focused, fileListMaxRows, width, theme);
-    // Tabbed mode (#806 follow-up — short terminals): render only the
-    // active inspector tab with a `[Inspector] Actions` header so the
-    // user knows what they're seeing and how to switch (`[/]` while
-    // focus is on the inspector). Tall terminals stack both sections
-    // as before.
+    const fileListNodes = renderCommitFileList(h, Text, detail.files, state.selectedFileIndex, fileListFocused, fileListMaxRows, width, theme);
+    // Tab indicator. Renders in BOTH tabbed (short-terminal) mode and
+    // tall-stacked mode so the user can always see which tab the cursor
+    // owns and learn the `[/]` toggle. Without this on tall terminals,
+    // the actions list looked like a static cheat-sheet — there was no
+    // visible signal that the cursor could move into it.
+    //
+    // Spacing between tab labels comes from the labels' own padding
+    // (the active label is bracketed `[Inspector]` while the inactive
+    // one is space-padded ` Inspector `, so adjacency reads cleanly).
+    // Earlier revisions stuck a raw `' '` between the Text children to
+    // pad them visually — that crashes Ink at first paint with
+    // "Text string ' ' must be rendered inside <Text> component"
+    // because Box only accepts component children, never bare strings.
+    const activeTab = state.inspectorTab;
+    const tabHeader = h(Box, { key: 'inspector-tabs', flexDirection: 'row' }, h(Text, {
+        bold: activeTab === 'inspector',
+        dimColor: activeTab !== 'inspector',
+    }, activeTab === 'inspector' ? '[Inspector]' : ' Inspector '), h(Text, {
+        bold: activeTab === 'actions',
+        dimColor: activeTab !== 'actions',
+    }, activeTab === 'actions' ? '[Actions]' : ' Actions '), ...(focused
+        ? [h(Text, { key: 'inspector-tabs-hint', dimColor: true }, '  · ←/→ switch')]
+        : []));
+    // Tabbed mode (short terminals): render only the active tab's
+    // content under the tab header.
     if (tabbed) {
-        const activeTab = state.inspectorTab;
-        const tabHeader = h(Box, { key: 'inspector-tabs', flexDirection: 'row' }, h(Text, {
-            bold: activeTab === 'inspector',
-            dimColor: activeTab !== 'inspector',
-        }, activeTab === 'inspector' ? '[Inspector]' : ' Inspector '), ' ', h(Text, {
-            bold: activeTab === 'actions',
-            dimColor: activeTab !== 'actions',
-        }, activeTab === 'actions' ? '[Actions]' : ' Actions '));
         return h(Box, {
             borderColor: focusBorderColor(theme, focused),
             borderStyle: theme.borderStyle,
@@ -26832,13 +26871,16 @@ function renderHistoryInspector(h, components, state, context, _contextStatus, d
                 cursorActive: focused,
             })));
     }
+    // Tall mode: stack both sections so the user can read everything at
+    // once, but show the tab header so the active section (and the
+    // `[/]` switch affordance) is visible.
     return h(Box, {
         borderColor: focusBorderColor(theme, focused),
         borderStyle: theme.borderStyle,
         flexDirection: 'column',
         width,
         paddingX: 1,
-    }, h(Text, { bold: true }, panelTitle('Inspector', focused)), ...headerNodes, ...fileListNodes, ...renderInspectorActionsSection(h, Text, 'history-commit', width, theme, {
+    }, h(Text, { bold: true }, panelTitle('Inspector', focused)), tabHeader, h(Text, { key: 'inspector-tabs-spacer' }, ''), ...headerNodes, ...fileListNodes, ...renderInspectorActionsSection(h, Text, 'history-commit', width, theme, {
         cursorIndex: state.inspectorActionIndex,
         cursorActive: focused && state.inspectorTab === 'actions',
     }));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "git-coco",
-  "version": "0.40.0",
+  "version": "0.40.1",
   "description": "zero-effort git commits with coco.",
   "author": "gfargo <ghfargo@gmail.com>",
   "license": "MIT",
@@ -72,7 +72,7 @@
     "eslint": "^8.54.0",
     "ink-testing-library": "4.0.0",
     "jest": "^30.0.5",
-    "release-it": "^19.0.4",
+    "release-it": "^20.0.1",
     "rollup": "^4.50.0",
     "rollup-plugin-dts": "^6.1.0",
     "rollup-plugin-executable": "^1.6.3",
@@ -96,7 +96,7 @@
     "@langchain/openai": "^1.4.5",
     "ajv": "^8.18.0",
     "chalk": "4.1.2",
-    "diff": "8.0.4",
+    "diff": "9.0.0",
     "ini": "6.0.0",
     "ink": "7.0.1",
     "minimatch": "^10.2.4",