gemiterm 2.0.0

package/install.sh

@@ -0,0 +1,275 @@
+#!/usr/bin/env bash
+# GemiTerm v2.0.0 installer for Linux/WSL
+#
+# Usage:
+#   curl -fsSL https://github.com/expert-vision-software/GemiTerm/releases/latest/download/install.sh | bash
+#   GEMITERM_TAG=v2.0.0-rc.1 bash install.sh
+#   GEMITERM_INSTALL_DIR=~/.local/bin bash install.sh
+#   bash install.sh --uninstall
+#   bash install.sh --whatif
+#
+# v1.4.1 → v2.0.0 upgrades replace the binary in place; the config dir at
+# ~/gemiterm/ (POSIX) is NEVER deleted by this installer.
+
+set -euo pipefail
+
+REPO="expert-vision-software/GemiTerm"
+INSTALL_DIR="${GEMITERM_INSTALL_DIR:-$HOME/.local/bin}"
+BIN_PATH="$INSTALL_DIR/gemiterm"
+CONFIG_DIR="$HOME/gemiterm"
+ENV_SNIPPET="$CONFIG_DIR/env.sh"
+TAG="${GEMITERM_TAG:-latest}"
+WHAT_IF=false
+
+remove_source_line() {
+    local rc_file="$1"
+    if [[ -f "$rc_file" ]]; then
+        local tmp
+        tmp=$(mktemp)
+        grep -v 'gemiterm/env.sh' "$rc_file" > "$tmp" || true
+        mv "$tmp" "$rc_file"
+    fi
+}
+
+# Parse flags
+if [[ "${1:-}" = "--whatif" || "${1:-}" = "--dry-run" ]]; then
+    WHAT_IF=true
+    shift
+fi
+
+# --- Detect Python v1.4.1 (task 8.2) ---
+if command -v pip >/dev/null 2>&1 && pip show gemiterm >/dev/null 2>&1; then
+    echo "WARNING: Python v1.4.1 detected. Your config data at ~/gemiterm/ is safe and will be preserved. Please run 'pip uninstall gemiterm' before re-running this installer to complete the v2.0.0 installation."
+    exit 1
+fi
+
+# --- Uninstall (task 3.3) ---
+if [[ "${1:-}" = "--uninstall" ]]; then
+    echo "Removing GemiTerm..."
+
+    if [[ -f "$BIN_PATH" ]]; then
+        if $WHAT_IF; then echo "[WhatIf] Would delete $BIN_PATH"; else rm -f "$BIN_PATH"; echo "  Removed $BIN_PATH"; fi
+    else
+        echo "  $BIN_PATH not found (already removed)"
+    fi
+
+    if [[ -f "$ENV_SNIPPET" ]]; then
+        if $WHAT_IF; then echo "[WhatIf] Would delete $ENV_SNIPPET"; else rm -f "$ENV_SNIPPET"; echo "  Removed $ENV_SNIPPET"; fi
+    fi
+
+    if $WHAT_IF; then
+        echo "[WhatIf] Would remove gemiterm/env.sh source line from $HOME/.bashrc"
+    else
+        remove_source_line "$HOME/.bashrc"
+    fi
+
+    if [[ -f "$HOME/.zshrc" ]]; then
+        if $WHAT_IF; then
+            echo "[WhatIf] Would remove gemiterm/env.sh source line from $HOME/.zshrc"
+        else
+            remove_source_line "$HOME/.zshrc"
+        fi
+    fi
+
+    echo "GemiTerm uninstall review complete. No changes were made."
+    exit 0
+fi
+
+# --- Upgrade detection (task 3.4) ---
+if [[ -x "$BIN_PATH" ]]; then
+    echo "Detected existing install at $BIN_PATH; upgrading in place."
+fi
+
+# --- v1.4.1 → v2.0.0 config migration (task 11.1) ---
+# v1.4.1 wrote config to $HOME/.config/gemiterm/ on POSIX. v2.0.0 reads from
+# $HOME/gemiterm/. If the old path exists and the new one does not, copy the
+# tree forward. The v1.4.1 directory is left in place as a safety net.
+V14_CONFIG_DIR="$HOME/.config/gemiterm"
+if [[ -d "$V14_CONFIG_DIR" && ! -d "$CONFIG_DIR" ]]; then
+    echo "Detected v1.4.1 config at $V14_CONFIG_DIR; migrating to $CONFIG_DIR"
+    if $WHAT_IF; then
+        echo "[WhatIf] Would create directory $CONFIG_DIR"
+        echo "[WhatIf] Would copy $V14_CONFIG_DIR/ to $CONFIG_DIR/"
+    else
+        mkdir -p "$CONFIG_DIR"
+        cp -R "$V14_CONFIG_DIR/." "$CONFIG_DIR/"
+        echo "v1.4.1 config copied to $CONFIG_DIR. The original at $V14_CONFIG_DIR is left in place as a backup."
+    fi
+fi
+
+# Determine package-manager executable (prefer bunx over npx)
+if command -v bun >/dev/null 2>&1; then
+    PACKAGE_MANAGER_X="bunx"
+elif command -v npm >/dev/null 2>&1; then
+    PACKAGE_MANAGER_X="npx"
+else
+    PACKAGE_MANAGER_X="bunx"   # will be bootstrapped below
+fi
+
+# --- Package-manager prompt (task 11.2) ---
+# If bun or npm is on PATH, recommend installing via the package manager
+# (npm: `npm i -g gemiterm`; bun: `bun i -g gemiterm`) instead of this
+# binary drop. The prompt is skipped when stdin is not a TTY (e.g. the
+# `curl | bash` one-liner flow runs unattended and must not block).
+if { command -v bun >/dev/null 2>&1 || command -v npm >/dev/null 2>&1; } && [[ -t 0 ]]; then
+    echo ""
+    echo "It is recommended to install via bun or npm package manager."
+    echo "Are you sure you want to continue with binary install? [y/N]"
+    read -r REPLY
+    case "$REPLY" in
+        [yY]|[yY][eE][sS]) ;;
+        *)
+            echo "Aborted. Install via: $(command -v bun >/dev/null 2>&1 && echo 'bun i -g gemiterm' || echo 'npm i -g gemiterm')"
+            exit 0
+            ;;
+    esac
+fi
+
+# --- Resolve release (task 3.5) ---
+API_URL="https://api.github.com/repos/$REPO/releases/latest"
+if [[ "$TAG" != "latest" ]]; then
+    API_URL="https://api.github.com/repos/$REPO/releases/tags/$TAG"
+fi
+
+# --- Network check (task 3.11) ---
+if ! curl -fsSI -o /dev/null "$API_URL" 2>/dev/null; then
+    echo "Cannot reach GitHub releases. Check your network connection or use the 'build from source' instructions in docs/INSTALL.md."
+    exit 1
+fi
+
+RELEASE_JSON=$(curl -fsSL "$API_URL")
+
+ASSET_URL=$(echo "$RELEASE_JSON" | grep -oE '"browser_download_url":"[^"]*"' \
+    | sed 's/"browser_download_url":"//;s/"$//' \
+    | grep -E '/gemiterm$' \
+    | head -n 1)
+
+if [[ -z "$ASSET_URL" ]]; then
+    echo "No gemiterm binary asset found in release."
+    echo "Build from source:"
+    echo "  git clone https://github.com/$REPO.git && cd GemiTerm && bun install && bun run build"
+    exit 1
+fi
+
+VERSION=$(echo "$RELEASE_JSON" | grep -oE '"tag_name":"[^"]*"' | head -n 1 | sed 's/"tag_name":"//;s/"$//')
+
+# --- Download (task 3.6) ---
+if $WHAT_IF; then
+    echo "[WhatIf] Would create directory $INSTALL_DIR"
+else
+    mkdir -p "$INSTALL_DIR"
+fi
+
+if $WHAT_IF; then
+    echo "[WhatIf] Would download $ASSET_URL to $BIN_PATH"
+    echo "[WhatIf] Would set executable bit on $BIN_PATH"
+else
+    if ! curl -fSL -o "$BIN_PATH.new" "$ASSET_URL"; then
+        rm -f "$BIN_PATH.new"
+        echo "Download failed."
+        exit 1
+    fi
+    mv "$BIN_PATH.new" "$BIN_PATH"
+    chmod +x "$BIN_PATH"
+fi
+
+# --- Bootstrap package manager if needed (task 9.2 + 9.3) ---
+if [[ "$PACKAGE_MANAGER_X" = "bunx" ]] && ! command -v bun >/dev/null 2>&1; then
+    if $WHAT_IF; then
+        echo "[WhatIf] Would install Bun from https://bun.sh/install"
+    else
+        echo "Installing Bun..."
+        if ! curl -fsSL https://bun.sh/install | bash; then
+            if command -v npm >/dev/null 2>&1; then
+                echo "Bun installation failed, falling back to npx."
+                PACKAGE_MANAGER_X="npx"
+            else
+                echo "Bun installation failed and npm is not available. Install Bun from https://bun.sh or npm from https://nodejs.org and re-run this installer."
+                exit 1
+            fi
+        fi
+        if [[ "$PACKAGE_MANAGER_X" = "bunx" ]]; then
+            export PATH="$HOME/.bun/bin:$PATH"
+            if ! command -v bun >/dev/null 2>&1; then
+                if command -v npm >/dev/null 2>&1; then
+                    echo "Bun installation failed, falling back to npx."
+                    PACKAGE_MANAGER_X="npx"
+                else
+                    echo "Bun installation failed. Install Bun manually from https://bun.sh and re-run this installer."
+                    exit 1
+                fi
+            fi
+        fi
+    fi
+fi
+
+# --- Install Chromium (task 3.7) ---
+if $WHAT_IF; then
+    echo "[WhatIf] Would run: $PACKAGE_MANAGER_X @playwright/cli install chromium"
+else
+    echo "Installing Chromium browser for Playwright..."
+    if ! $PACKAGE_MANAGER_X @playwright/cli install chromium; then
+        echo "Chromium installation failed. Re-run the installer after fixing the issue, or run '${PACKAGE_MANAGER_X} @playwright/cli install chromium' manually."
+        exit 1
+    fi
+fi
+
+# --- Verify Chromium (task 3.8) ---
+if $WHAT_IF; then
+    echo "[WhatIf] Would verify Chromium installation in $HOME/.cache/ms-playwright"
+else
+    CHROME_PATH=$(find "$HOME/.cache/ms-playwright" -path '*/chromium-*/chrome-linux/chrome' -type f -executable 2>/dev/null | head -n 1)
+    if [[ -z "$CHROME_PATH" ]]; then
+        CHROME_PATH=$(find "$HOME/.cache/ms-playwright" -path '*/chromium-*' -name chrome -type f -executable 2>/dev/null | head -n 1)
+    fi
+
+    if [[ -n "$CHROME_PATH" ]]; then
+        echo "Chromium verified at $CHROME_PATH"
+    else
+        echo "Chromium installation verification failed. Re-run the installer after fixing the network, or run '${PACKAGE_MANAGER_X} @playwright/cli install chromium' manually."
+        exit 1
+    fi
+fi
+
+# --- PATH augmentation (task 3.9) ---
+if $WHAT_IF; then
+    echo "[WhatIf] Would create directory $CONFIG_DIR"
+    echo "[WhatIf] Would write $ENV_SNIPPET"
+else
+    mkdir -p "$CONFIG_DIR"
+    cat > "$ENV_SNIPPET" <<'ENVEOF'
+export PATH="$HOME/.local/bin:$PATH"
+ENVEOF
+fi
+
+SOURCE_LINE='[[ -f "$HOME/gemiterm/env.sh" ]] && source "$HOME/gemiterm/env.sh"'
+
+if [[ -f "$HOME/.bashrc" ]] && ! grep -qiF 'gemiterm/env.sh' "$HOME/.bashrc"; then
+    if $WHAT_IF; then
+        echo "[WhatIf] Would append gemiterm/env.sh source line to $HOME/.bashrc"
+    else
+        echo "$SOURCE_LINE" >> "$HOME/.bashrc"
+    fi
+fi
+
+if [[ -f "$HOME/.zshrc" ]] && ! grep -qiF 'gemiterm/env.sh' "$HOME/.zshrc"; then
+    if $WHAT_IF; then
+        echo "[WhatIf] Would append gemiterm/env.sh source line to $HOME/.zshrc"
+    else
+        echo "$SOURCE_LINE" >> "$HOME/.zshrc"
+    fi
+fi
+
+if ! $WHAT_IF; then
+    export PATH="$INSTALL_DIR:$PATH"
+fi
+
+# --- Success (task 3.10) ---
+if $WHAT_IF; then
+    echo ""
+    echo "WhatIf review complete. No changes were made."
+    echo "GemiTerm ${VERSION:-unknown} would be installed to $BIN_PATH using ${PACKAGE_MANAGER_X}.Chromium."
+    echo "Run without --whatif to apply."
+else
+    echo "GemiTerm ${VERSION:-unknown} installed to $BIN_PATH. Run 'gemiterm status' to verify, then 'gemiterm auth' to authenticate. Restart your shell (or 'source ~/gemiterm/env.sh') to pick up the new PATH."
+fi

package/openspec/MAESTRO_MIGRATION.md

@@ -0,0 +1,180 @@
+# Maestro → OpenSpec Migration Status
+
+This document maps every Maestro plan task to its current completion status
+and the corresponding OpenSpec artifact (main spec or active change). It
+serves as the canonical "task list with checkmarks" for the migration.
+
+**Source of truth:**
+- Maestro plans: `C:\dev\projects\github\webgemini-cli\.maestro\playbooks\2026-06-06-GemiTerm-Bun-Port\Phase-{01..06}-*.md`
+- OpenSpec main specs (completed work): `openspec/specs/`
+- OpenSpec changes (remaining work): `openspec/changes/`
+- Test baseline: `bun test` → **432 pass, 0 fail, 754 expect() calls** (10.5 s)
+
+**Status legend:**
+- ✅ **DONE** — capability exists in code, covered by passing tests, no obvious gaps vs the Maestro description
+- 🟡 **PARTIAL** — capability is mostly there but with documented deviations (e.g. a Maestro-spec'd behavior was not implemented; an intentional improvement replaced a Maestro item)
+- ❌ **NOT DONE** — Maestro task has no implementation; an OpenSpec change captures the remaining work
+- ⚠️ **N/A** — Maestro task is not applicable to the Bun rewrite (e.g. Python-only tooling)
+- ⏭️ **DEFERRED** — out of scope for the v2.0.0 release; tracked in a follow-up
+
+---
+
+## Phase 01: Project Setup & Working Prototype
+
+| # | Maestro task | Status | OpenSpec artifact | Evidence / notes |
+|---|---|---|---|---|
+| 1.1 | Initialize Bun project (`bun init`, name `gemiterm`, `type: module`, `private`) | ✅ DONE | `openspec/specs/cli/spec.md` | `package.json:2-6`; `e363609` (git) |
+| 1.2 | Create `src/` directory structure | ✅ DONE | `openspec/specs/cli/spec.md` | All 5 dirs present; `3877b6c` |
+| 1.3 | Create `tsconfig.json` | ✅ DONE | (covered by cli spec) | `tsconfig.json` matches Maestro shape exactly; 0 type errors |
+| 1.4 | Cross-platform path utility `src/infrastructure/path-utils.ts` | ✅ DONE (partial) | `openspec/specs/path-utils/spec.md`; missing helpers in `cross-platform-build-and-ci` | Core 4 functions present + extras. `isWindows`/`isWSL`/`isLinux`/`normalizePath`/`getPlatformName` are missing — `cross-platform-build-and-ci` adds them as re-exports. |
+| 1.5 | `src/infrastructure/config.ts` | ✅ DONE | `openspec/specs/configuration/spec.md` | All 7 functions present; 13 unit tests |
+| 1.6 | `src/core/types.ts` | ✅ DONE | `openspec/specs/domain-model/spec.md` | All 6 interfaces present |
+| 1.7 | `src/core/errors.ts` | ✅ DONE | `openspec/specs/domain-model/spec.md` | All 6 error classes; 13 unit tests |
+| 1.8 | `src/cli/index.ts` main entry | ✅ DONE | `openspec/specs/cli/spec.md` | shebang, global flags, dispatch all working; 3 smoke tests |
+| 1.9 | `src/cli/commands/help.ts` | ✅ DONE | `openspec/specs/cli/spec.md` | `showHelp(registry)`; covered by every `--help` integration test |
+| 1.10 | `package.json` scripts | ✅ DONE | (covered by `cli` spec + `cross-platform-build-and-ci`) | `start`, `dev`, `typecheck`, `test`, `test:unit`, `test:integration`, `test:parity`, `test:smoke`, `test:all`, `build`, `build:linux`, `build:windows` present. **`build*` are broken in Bun 1.3.x** — fixed by `cross-platform-build-and-ci`. `lint`/`format` (ruff) are N/A. |
+| 1.11 | `bun add commander chalk` and `bun add -d @types/bun` | ✅ DONE (with cleanup) | `cross-platform-build-and-ci` removes unused `commander` | `chalk` is used; `commander` is installed but **never imported** — cleanup task in change 1 |
+| 1.12 | Verify `bun run src/cli/index.ts --help` | ✅ DONE | `openspec/specs/testing/spec.md` | `tests/smoke/smoke.test.ts:31-37` |
+| 1.13 | Verify ruff linting | ⚠️ N/A | — | Maestro itself notes ruff is a Python tool not applicable to TS |
+| 1.14 | Verify `tsc --noEmit` | ✅ DONE | (covered by `cli` spec) | `bun run typecheck` → 0 errors |
+
+---
+
+## Phase 02: Core Architecture (Mediator Pattern + SOLID)
+
+| # | Maestro task | Status | OpenSpec artifact | Evidence / notes |
+|---|---|---|---|---|
+| 2.1 | `src/core/mediator.ts` | ✅ DONE | `openspec/specs/mediator/spec.md` | 9 unit tests; `4dd344d` |
+| 2.2 | `src/core/command-handlers.ts` | ✅ DONE | (internal — referenced by `commands` and `auth` specs) | All 7 command types + handlers; covered by `tests/core/query-handlers.test.ts` + integration tests |
+| 2.3 | `src/core/query-handlers.ts` | ✅ DONE | (internal — referenced by `commands` and `auth` specs) | All 5 query types + handlers; 9 unit tests |
+| 2.4 | `src/infrastructure/storage.ts` | ✅ DONE | `openspec/specs/storage/spec.md` | `CookieStorage` + `ProfileManager`; 21 unit tests |
+| 2.5 | `src/infrastructure/logger.ts` | ✅ DONE | `openspec/specs/logger/spec.md` | stderr-only timestamped logger; 7 unit tests |
+| 2.6 | `src/infrastructure/validators.ts` | ✅ DONE | `openspec/specs/validators/spec.md` | All 3 validators; 10 unit tests (file lives in `tests/infrastructure/`, not `tests/unit/` — same coverage) |
+| 2.7 | `src/infrastructure/formatters.ts` | ✅ DONE | `openspec/specs/formatters/spec.md` | All 4 formatters; 22 unit tests across 2 files |
+| 2.8 | `src/services/gemini-client-wrapper.ts` | ✅ DONE | `openspec/specs/conversations/spec.md` | 6 methods + `isAuthenticated()`; HTTP routes to `gemini.google.com/app/api/...` |
+| 2.9 | `src/services/profile-service.ts` | ✅ DONE | `openspec/specs/profiles/spec.md` | All 5 methods; 8 unit tests |
+| 2.10 | `src/cli/command-registry.ts` | ✅ DONE (deviation) | `openspec/specs/commands/spec.md` | Explicit `registerAllCommands()` (not auto-discovery); 6 unit tests; `84c24cf` formalized this |
+| 2.11 | `src/cli/index.ts` uses mediator + registry | ✅ DONE | `openspec/specs/cli/spec.md` | dispatch + unknown-command handling; `e784eef` |
+| 2.12 | Verify architecture compiles | ✅ DONE | (covered by `cli` + `testing` specs) | `tests/smoke/smoke.test.ts:31-37`; `c236027`, `0ec3682` |
+| 2.13 | `ruff check src/` + `tsc --noEmit` | ⚠️ N/A + ✅ | — | ruff N/A; `tsc --noEmit` clean |
+
+---
+
+## Phase 03: Auth Flow Implementation
+
+> **⚠️ DELICATE — recent commits `4bc4de8` and `0ec3682` show active iteration. Do not modify playwright-cli-driver / cookie-monitor / auth-service lightly.**
+
+| # | Maestro task | Status | OpenSpec artifact | Evidence / notes |
+|---|---|---|---|---|
+| 3.1 | `src/services/playwright-cli-driver.ts` | ✅ DONE | `openspec/specs/auth/spec.md` | All 8 methods + `withSession`; auto-detect between `playwright-cli` and `bunx @playwright/cli`; 15 unit tests |
+| 3.2 | `src/services/cookie-monitor.ts` | ✅ DONE (improvement) | `openspec/specs/auth/spec.md` | All 4 methods; `POLL_INTERVAL_MS=2_000`; `DEFAULT_TIMEOUT_MS=300_000`; timeout `.unref()`-ed. **Login detection uses sign-out link probe** (intentional improvement, `e4d1fef`), not Maestro's textarea+URL check. |
+| 3.3 | `src/services/auth-service.ts` | 🟡 PARTIAL | `openspec/specs/auth/spec.md` | All 8 methods present; 13 unit tests. **"Press Enter to launch browser..." is a one-shot UI message, not a blocking prompt** — Maestro specified blocking on Enter. Intentional change; documented as current behavior. |
+| 3.4 | `src/services/cookie-storage-service.ts` | ✅ DONE (deviation) | `openspec/specs/auth/spec.md` | All 4 methods; 7-day freshness window; 9 unit tests. **Composes `CookieStorage` via DI** (not extends), functionally equivalent |
+| 3.5 | `src/services/profile-auth-manager.ts` | ✅ DONE (with bug) | `openspec/specs/auth/spec.md`; bug fixed in `command-spec-conformance` | `ensureAuthenticated`, `getActiveProfiles`, `findProfileForConversation` all present. **`findProfileForConversation` ignores its `conversationId` argument** — returns first active profile. Bug is fixed in `command-spec-conformance` change. 8 unit tests document the BUG (to be updated). |
+| 3.6 | `src/cli/commands/auth-command.ts` | ✅ DONE | `openspec/specs/commands/spec.md` | Profile selection menu with all 5 options; 12 CLI tests + 7 integration tests. Registered as `auth` (Maestro's note said `login` — Maestro was slightly off) |
+| 3.7 | `src/cli/commands/status-command.ts` | ✅ DONE | `openspec/specs/commands/spec.md` | Config dir + 4-column profile table; exits 2 if no profiles; 3 CLI tests + 9 integration tests |
+| 3.8 | `src/cli/commands/profile-command.ts` | ✅ DONE | `openspec/specs/commands/spec.md` | All 5 actions + help; 17 integration tests |
+| 3.9 | `src/services/install-browser-service.ts` | ✅ DONE | `openspec/specs/install-browser/spec.md` (current surface); cross-platform refactor in `cross-platform-build-and-ci` | All methods + Windows/Linux/WSL browser detection; 5 unit tests. The refactor in `cross-platform-build-and-ci` extracts the WSL detection to the shared `platform-detect` module — **behavior-preserving**, the 5 tests are the regression gate |
+| 3.10 | `src/cli/commands/install-browser-command.ts` | ✅ DONE | `openspec/specs/install-browser/spec.md` | Wraps `InstallBrowserService.install()`; 3 CLI tests |
+| 3.11 | Verify auth flow end-to-end | ✅ DONE | `openspec/specs/commands/spec.md` + `auth` spec | 12 CLI tests + 7 integration tests + 13 service tests |
+| 3.12 | `ruff check src/` + `tsc --noEmit` | ⚠️ N/A + ✅ | — | ruff N/A; `tsc --noEmit` clean |
+
+---
+
+## Phase 04: Command Implementation
+
+| # | Maestro task | Status | OpenSpec artifact | Evidence / notes |
+|---|---|---|---|---|
+| 4.1 | `src/cli/commands/list-command.ts` | 🟡 PARTIAL | `openspec/specs/commands/spec.md`; fix in `command-spec-conformance` | All 10 flags parsed; 11 CLI tests + 21 integration tests. **Profile column missing when `--all-profiles`** — fixed in `command-spec-conformance` |
+| 4.2 | `src/cli/commands/fetch-command.ts` | ✅ DONE | `openspec/specs/commands/spec.md` | 18 integration tests |
+| 4.3 | `src/cli/commands/continue-command.ts` | 🟡 PARTIAL | `openspec/specs/commands/spec.md`; fix in `command-spec-conformance` | Non-interactive + interactive REPL with `/exit`. **Does NOT call `findProfileForConversation`** — fixed in `command-spec-conformance`. 5 CLI tests |
+| 4.4 | `src/cli/commands/new-command.ts` | ✅ DONE | `openspec/specs/commands/spec.md` | Non-interactive + interactive REPL with `--profile` |
+| 4.5 | `src/cli/commands/delete-command.ts` | 🟡 PARTIAL | `openspec/specs/commands/spec.md`; fix in `command-spec-conformance` | Confirmation prompt + `--force`. **Does NOT call `findProfileForConversation`** — fixed in `command-spec-conformance`. 7 CLI tests |
+| 4.6 | `src/cli/commands/export-command.ts` | ✅ DONE | `openspec/specs/commands/spec.md` | Default filename + JSON/markdown + output dir; 12 integration tests |
+| 4.7 | `src/cli/commands/export-all-command.ts` | ✅ DONE | `openspec/specs/commands/spec.md` | All flags; index.md with links; progress + failure tracking; 9 CLI tests |
+| 4.8 | Update `command-registry.ts` to register all 11 commands | ✅ DONE | `openspec/specs/commands/spec.md` | All 11 commands registered in `registerAllCommands()`; `84c24cf` |
+| 4.9 | Update `src/cli/index.ts` for global flags + unknown commands | ✅ DONE | `openspec/specs/cli/spec.md` | `--verbose`/`-v`, `--help`/`-h`, `--version`, unknown command → "Did you mean one of: …?"; `a3e1d8d` |
+| 4.10 | Verify all commands with `--help` | ✅ DONE | `openspec/specs/commands/spec.md` + `testing` spec | 600+ integration assertions exercise `--help` on every command |
+| 4.11 | `ruff check src/` + `tsc --noEmit` | ⚠️ N/A + ✅ | — | ruff N/A; `tsc --noEmit` clean |
+
+---
+
+## Phase 05: CI/CD & Build
+
+| # | Maestro task | Status | OpenSpec artifact | Evidence / notes |
+|---|---|---|---|---|
+| 5.1 | `.github/workflows/test.yml` | ❌ NOT DONE | `cross-platform-build-and-ci` change (`release-pipeline` spec) | The directory exists but is empty |
+| 5.2 | `.github/workflows/build.yml` | ❌ NOT DONE | `cross-platform-build-and-ci` change (`release-pipeline` spec) | Not present |
+| 5.3 | `.github/workflows/release.yml` | ❌ NOT DONE | `cross-platform-build-and-ci` change (`release-pipeline` spec) | Not present |
+| 5.4 | `package.json` build scripts | 🟡 PARTIAL (broken) | `cross-platform-build-and-ci` change (`release-pipeline` spec) | Scripts present but `bun run build` fails: `error: cannot use --compile with --outdir` (Bun 1.3.x). Fix uses `--outfile` instead |
+| 5.5 | `scripts/clean-build.sh` + `scripts/clean-build.ps1` | ❌ NOT DONE | `cross-platform-build-and-ci` change (`release-pipeline` spec) | `scripts/` directory does not exist |
+| 5.6 | `src/infrastructure/path-utils.ts` updates (isWindows, isWSL, normalizePath, getPlatformName) | ❌ NOT DONE | `cross-platform-build-and-ci` change (`platform-detection` spec) | 0 matches for any of these in `src/`. WSL detection duplicated inline in `install-browser-service.ts:143-171` |
+| 5.7 | `src/infrastructure/platform-detect.ts` | ❌ NOT DONE | `cross-platform-build-and-ci` change (`platform-detection` spec) | File does not exist |
+| 5.8 | `src/services/install-browser-service.ts` cross-platform | ✅ DONE | (current state covered by `install-browser` spec) | All platform-specific finders present; 5 unit tests |
+| 5.9 | `scripts/install-browser.ps1` | ❌ NOT DONE | `cross-platform-build-and-ci` change (`release-pipeline` spec) | Not present |
+| 5.10 | `scripts/install-browser.sh` | ❌ NOT DONE | `cross-platform-build-and-ci` change (`release-pipeline` spec) | Not present |
+| 5.11 | `src/cli/index.ts` shebang | ✅ DONE | `openspec/specs/cli/spec.md` | `#!/usr/bin/env bun` (line 1) |
+| 5.12 | Verify build works locally | ❌ NOT DONE | `cross-platform-build-and-ci` change (`release-pipeline` spec) | Build is broken (5.4) |
+| 5.13 | `ruff check src/` + `tsc --noEmit` | ⚠️ N/A + ✅ | — | ruff N/A; `tsc --noEmit` clean |
+
+---
+
+## Phase 06: Testing
+
+| # | Maestro task | Status | OpenSpec artifact | Evidence / notes |
+|---|---|---|---|---|
+| 6.1 | `tests/setup.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 7 global helpers; `ad0df81` |
+| 6.2 | `tests/fixtures/auth-fixtures.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 4 factories + 3 constants |
+| 6.3 | `tests/fixtures/chat-fixtures.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 3 factories |
+| 6.4 | `tests/unit/path-utils.test.ts` | 🟡 PARTIAL | `openspec/specs/testing/spec.md`; gap closed by `cross-platform-build-and-ci` | 19 tests; **missing tests for `isWindows`/`isLinux`/`isWSL`** (functions don't exist yet — gap closed when `platform-detection` change lands) |
+| 6.5 | `tests/unit/config.test.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 13 tests; `3446e3a` |
+| 6.6 | `tests/unit/validators.test.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 10 tests (file is in `tests/infrastructure/`, not `tests/unit/`) |
+| 6.7 | `tests/unit/formatters.test.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 14 unit + 8 infrastructure tests covering all 4 formatters; `698ff3d` |
+| 6.8 | `tests/unit/mediator.test.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 9 tests; `3ac6a41` |
+| 6.9 | `tests/unit/errors.test.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 13 tests; `3ac6a41` |
+| 6.10 | `tests/integration/commands/auth.test.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 7 tests; `1505d3e` |
+| 6.11 | `tests/integration/commands/list.test.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 21 tests; `87fe60b` + `9bdbe2f` |
+| 6.12 | `tests/integration/commands/fetch.test.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 18 tests; `96a586d` |
+| 6.13 | `tests/integration/commands/export.test.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 12 tests; `6619148` |
+| 6.14 | `tests/integration/commands/profile.test.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 17 tests; `81ccbe3` |
+| 6.15 | `tests/integration/commands/status.test.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 9 tests; `e4cc17a` |
+| 6.16 | `tests/parity/compare-outputs.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | Full framework with normalization; `451e54d` + `4cae594` |
+| 6.17 | `tests/parity/test-commands-parity.{sh,ps1}` | ✅ DONE | `openspec/specs/testing/spec.md` | Both shells present |
+| 6.18 | `tests/smoke/smoke.test.ts` | ✅ DONE | `openspec/specs/testing/spec.md` | 3 tests; `d3a4bb4` |
+| 6.19 | `package.json` test scripts | ✅ DONE | `openspec/specs/testing/spec.md` | All 6 scripts; `8bb251a` |
+| 6.20 | `bun test` → 0 failures | ✅ DONE | (verified at migration start; 432/432) | Re-verified at end of migration |
+| 6.21 | Run parity tests | 🟡 PARTIAL | `openspec/specs/testing/spec.md` | Parity infra exists and passes; **results against Python CLI not yet committed** (requires v1.4.1 Python CLI on PATH) |
+| 6.22 | `ruff check src/` + `tsc --noEmit` | ⚠️ N/A + ✅ | — | ruff N/A; `tsc --noEmit` clean |
+
+---
+
+## Cross-Phase Gaps (not in any Maestro task)
+
+| # | Item | Status | OpenSpec artifact | Evidence / notes |
+|---|---|---|---|---|
+| C.1 | `install.ps1` / `install.sh` for v2.0.0 (replaces deleted v1.4.1 installer) | ❌ NOT DONE | `v2-install-migration` change (`v2-installer` spec) | The v1.4.1 `install.ps1` was deleted in commit `4bdefa8` "remove all python implementation". For v1.4.1 → v2.0.0 seamless upgrade, a new install.ps1 + install.sh are required |
+| C.2 | `docs/INSTALL.md` end-user install guide | ❌ NOT DONE | `v2-install-migration` change (`v2-installer` spec) | Not present |
+| C.3 | `README.md` "Upgrading from v1.4.1" callout | ❌ NOT DONE | `v2-install-migration` change (`v2-installer` spec) | README still references old install flow |
+| C.4 | `.gitignore` stray `{` line + orphan `{` file | ❌ NOT DONE | `cross-platform-build-and-ci` change (`platform-detection` / cleanup tasks) | `.gitignore:23` is bare `{`; a 0-byte file named `{` is at repo root |
+| C.5 | Unused `commander` dependency | ❌ NOT DONE | `cross-platform-build-and-ci` change (cleanup tasks) | `commander@^15.0.0` in `package.json:20` and `bun.lock:33`; 0 imports in `src/` |
+| C.6 | `src/commands/` empty placeholder | ❌ NOT DONE | `cross-platform-build-and-ci` change (cleanup tasks) | Contains only `.gitkeep`; real commands live in `src/cli/commands/` |
+| C.7 | `bun run build` is broken in Bun 1.3.x | ❌ NOT DONE | `cross-platform-build-and-ci` change (`release-pipeline` spec) | `error: cannot use --compile with --outdir` |
+
+---
+
+## Summary
+
+- **DONE:** 53 tasks ✅ + 5 N/A ⚠️
+- **PARTIAL:** 6 tasks 🟡 (all have follow-up changes; see column 4)
+- **NOT DONE:** 16 tasks ❌ (all have follow-up changes; see column 4)
+
+| OpenSpec change | Maestro/cross-phase items | Implementation tasks | Status |
+|---|---|---|---|
+| `cross-platform-build-and-ci` | 5.1–5.3, 5.4 (build fix), 5.5, 5.6, 5.7, 5.9, 5.10, 5.12, C.4, C.5, C.6, C.7 | 36 tasks across 12 groups | ready to apply |
+| `v2-install-migration` | C.1, C.2, C.3 | 36 tasks across 8 groups | ready to apply |
+| `command-spec-conformance` | 3.5 (bug fix), 4.1 (Profile column), 4.3 (continue profile lookup), 4.5 (delete profile lookup) | ~40 tasks across 12 groups | ready to apply |
+| 15 main specs (auth, cli, commands, …) | 53 ✅ tasks documented as the canonical current state | — | all validate |
+
+**Test baseline:** 432 pass, 0 fail, 754 expect() calls, 10.5 s. No source code was modified during the migration; the OpenSpec artifacts are pure documentation that describe the work to be done in follow-up changes.
+
+**To start the implementation of a change:** `cd <repo> && /opsx-apply <change-name>` (or ask your AI assistant to apply the change).

package/openspec/changes/archive/2026-06-08-command-spec-conformance/.openspec.yaml

@@ -0,0 +1,2 @@
+schema: spec-driven
+created: 2026-06-08