@synmux/claude-commit 1.0.0 → 1.0.1

package/README.md

@@ -151,6 +151,7 @@ keys are valid at every level:
   "interactive": true,
   "interactiveCount": 3,
   "interactiveTemperature": 1,
+  "spinner": "bouncingBall",
   "models": {
     "summary": "sonnet",
     "final": "sonnet"
@@ -162,6 +163,13 @@ keys are valid at every level:
 }
 ```
 
+`spinner` chooses the progress animation: any name from the
+[cli-spinners](https://github.com/sindresorhus/cli-spinners) set bundled with
+[ora](https://github.com/sindresorhus/ora) (`"dots"`, `"moon"`, `"pong"`,
+`"bouncingBall"`, ...). Unknown names are ignored and the default
+`bouncingBall` is used. `--no-spinner` disables the animated spinner, but final
+status lines still print.
+
 `maxChunkTokens` is a cap, not a promise: at run time it is clamped to the
 summary model's context window minus a fixed reserve (1M-window models such as
 current Sonnet/Opus keep the full budget; Haiku, older pinned model ids, and

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@synmux/claude-commit",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Generate git commit messages with Claude, using your Claude Code subscription.",
   "main": "index.ts",
   "module": "index.ts",
@@ -24,6 +24,7 @@
     "interactive": false,
     "interactiveCount": 3,
     "interactiveTemperature": 1,
+    "spinner": "bouncingBall",
     "models": {
       "summary": "sonnet",
       "final": "sonnet"
@@ -45,7 +46,9 @@
   "dependencies": {
     "@anthropic-ai/claude-agent-sdk": "^0.3.218",
     "@opentui/core": "^0.4.5",
-    "commander": "^15.0.0"
+    "cli-spinners": "^3.4.0",
+    "commander": "^15.0.0",
+    "ora": "^9.4.1"
   },
   "repository": {
     "type": "git",

package/src/cli.ts

@@ -256,7 +256,7 @@ async function runNonInteractive(
   abortController: AbortController,
 ): Promise<number> {
   const useSpinner = opts.spinner !== false && process.stderr.isTTY;
-  const spinner = new Spinner(useSpinner);
+  const spinner = new Spinner(useSpinner, config.spinner);
 
   spinner.start("Reading diff");
   let result;

package/src/config.ts

@@ -10,6 +10,7 @@
 import { dirname, isAbsolute, join, resolve } from "node:path";
 import { homedir } from "node:os";
 import { ClaudeCommitError } from "./errors";
+import { DEFAULT_SPINNER, isSpinnerName } from "./ui/spinner";
 import type { Config, ModelConfig, PartialConfig } from "./types";
 
 export const DEFAULT_CONFIG: Config = {
@@ -21,6 +22,7 @@ export const DEFAULT_CONFIG: Config = {
   interactive: false,
   interactiveCount: 3,
   interactiveTemperature: 1,
+  spinner: DEFAULT_SPINNER,
   models: {
     summary: "sonnet",
     final: "sonnet",
@@ -116,6 +118,9 @@ export function sanitizePartial(raw: unknown): PartialConfig {
       Math.max(0, obj.interactiveTemperature),
     );
   }
+  if (typeof obj.spinner === "string" && isSpinnerName(obj.spinner)) {
+    out.spinner = obj.spinner;
+  }
   if (typeof obj.maxChunkTokens === "number" && obj.maxChunkTokens > 0) {
     out.maxChunkTokens = Math.floor(obj.maxChunkTokens);
   }

package/src/types.ts

@@ -40,6 +40,12 @@ export interface Config {
    * at its default. Only applied in interactive mode.
    */
   interactiveTemperature: number | null;
+  /**
+   * Name of the progress spinner animation: any spinner from the cli-spinners
+   * set bundled with ora (e.g. `"dots"`, `"moon"`, `"bouncingBall"`). Unknown
+   * names are ignored and the default is used instead.
+   */
+  spinner: string;
   /** Models for each pipeline stage. */
   models: ModelConfig;
   /**

package/src/ui/interactive.ts

@@ -30,7 +30,7 @@ export async function runInteractive(
   opts: InteractiveOptions,
 ): Promise<number> {
   const count = Math.max(1, config.interactiveCount);
-  const spinner = new Spinner(process.stderr.isTTY);
+  const spinner = new Spinner(process.stderr.isTTY, config.spinner);
   spinner.start(`Generating ${count} option${count === 1 ? "" : "s"}`);
   let result;
   try {

package/src/ui/spinner.ts

@@ -1,48 +1,71 @@
 /**
- * A minimal, dependency-free progress spinner for non-interactive runs.
+ * The progress spinner for non-interactive runs, rendered by `ora`.
  *
- * It renders to stderr (so stdout stays clean for piping) and only animates when
- * stderr is a TTY; otherwise `start`/`update` are silent no-ops. This keeps the
- * read-only progress indicator out of the way of `cco --dry-run | git commit -F -`.
+ * It writes to stderr (so stdout stays clean for piping) and only animates when
+ * explicitly enabled: callers gate on `process.stderr.isTTY` and `--no-spinner`,
+ * and ora's own TTY/CI detection is bypassed so that decision lives in one
+ * place. When disabled, `start`/`update` are silent no-ops but final status
+ * lines (`succeed`/`fail`) still print. This keeps the progress indicator out
+ * of the way of `cco --dry-run | git commit -F -`.
+ *
+ * The animation is chosen by the `spinner` config key: any name from the
+ * cli-spinners set bundled with ora, defaulting to {@link DEFAULT_SPINNER}.
  */
+import ora, { type Ora } from "ora";
+import spinners, { type Spinner as SpinnerAnimation } from "cli-spinners";
 import { color } from "./colors";
 
-const FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
-const INTERVAL_MS = 80;
+/** The spinner used when none (or an unknown one) is configured. */
+export const DEFAULT_SPINNER = "bouncingBall";
+
+/** Whether `name` is one of the cli-spinners animations bundled with ora. */
+export function isSpinnerName(name: string): boolean {
+  return Object.hasOwn(spinners, name);
+}
+
+/**
+ * Look up a spinner animation by name, falling back to {@link DEFAULT_SPINNER}
+ * for unknown names (ora itself throws on those, and a cosmetic option must
+ * never be able to break a commit).
+ */
+export function resolveSpinner(name: string): SpinnerAnimation {
+  const known = (spinners as Record<string, SpinnerAnimation | undefined>)[
+    name
+  ];
+  return known ?? spinners[DEFAULT_SPINNER];
+}
 
 export class Spinner {
-  private timer: ReturnType<typeof setInterval> | null = null;
-  private frame = 0;
-  private label = "";
+  private instance: Ora | null = null;
   private readonly enabled: boolean;
+  private readonly animation: SpinnerAnimation;
 
-  constructor(enabled = process.stderr.isTTY) {
+  constructor(enabled = process.stderr.isTTY, spinnerName = DEFAULT_SPINNER) {
     this.enabled = Boolean(enabled);
+    this.animation = resolveSpinner(spinnerName);
   }
 
   start(label: string): void {
-    this.label = label;
     if (!this.enabled) return;
-    this.stopTimer();
-    process.stderr.write("\x1b[?25l"); // hide cursor
-    this.render();
-    this.timer = setInterval(() => {
-      this.frame = (this.frame + 1) % FRAMES.length;
-      this.render();
-    }, INTERVAL_MS);
+    this.instance?.stop();
+    this.instance = ora({
+      text: label,
+      spinner: this.animation,
+      stream: process.stderr,
+      // The caller already decided (TTY check + --no-spinner); don't let ora's
+      // own TTY/CI detection silently disagree.
+      isEnabled: true,
+    }).start();
   }
 
   update(label: string): void {
-    this.label = label;
-    if (this.enabled && this.timer) this.render();
+    if (this.instance) this.instance.text = label;
   }
 
   /** Stop and clear the spinner line, optionally printing a final status line. */
   stop(finalLine?: string): void {
-    this.stopTimer();
-    if (this.enabled) {
-      process.stderr.write("\r\x1b[2K\x1b[?25h"); // clear line, show cursor
-    }
+    this.instance?.stop();
+    this.instance = null;
     if (finalLine !== undefined) process.stderr.write(finalLine + "\n");
   }
 
@@ -53,19 +76,4 @@ export class Spinner {
   fail(label: string): void {
     this.stop(`${color("31", "✖")} ${label}`);
   }
-
-  private render(): void {
-    // `\r\x1b[2K` (cursor return + clear line) only runs when enabled (a TTY);
-    // the frame color additionally respects NO_COLOR via the helper.
-    process.stderr.write(
-      `\r\x1b[2K${color("36", FRAMES[this.frame]!)} ${this.label}`,
-    );
-  }
-
-  private stopTimer(): void {
-    if (this.timer) {
-      clearInterval(this.timer);
-      this.timer = null;
-    }
-  }
 }