From 20808b9b84fd6bc58a06680d1301e41e6213cedc Mon Sep 17 00:00:00 2001 From: Jason Woltje Date: Thu, 19 Mar 2026 22:13:35 -0500 Subject: [PATCH] =?UTF-8?q?chore:=20mark=20P8-001/002/003=20in-progress,?= =?UTF-8?q?=20P8-004=20done=20=E2=80=94=20PRs=20open?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docs/TASKS.md | 8 ++++---- docs/scratchpads/BUG-CLI-scratchpad.md | 7 +++++++ 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/docs/TASKS.md b/docs/TASKS.md index 78a3b7f..821e38d 100644 --- a/docs/TASKS.md +++ b/docs/TASKS.md @@ -93,8 +93,8 @@ | P8-017 | done | Phase 8 | TUI Phase 8 — autocomplete sidebar, fuzzy match, arg hints, up-arrow history | #184 | #170 | | P8-018 | done | Phase 8 | Spin-off plan stubs — Gatekeeper, Task Queue Unification, Chroot Sandboxing | — | #171 | | P8-019 | done | Phase 8 | Verify Platform Architecture — integration + E2E verification | #185 | #172 | -| P8-001 | not-started | codex | Phase 8 | Additional SSO providers — WorkOS + Keycloak | — | #53 | -| P8-002 | not-started | codex | Phase 8 | Additional LLM providers — Codex, Z.ai, LM Studio, llama.cpp | — | #54 | -| P8-003 | not-started | codex | Phase 8 | Performance optimization | — | #56 | -| P8-004 | not-started | haiku | Phase 8 | Beta release gate — v0.1.0 tag | — | #59 | +| P8-001 | in-progress | codex | Phase 8 | Additional SSO providers — WorkOS + Keycloak | #210 | #53 | +| P8-002 | in-progress | codex | Phase 8 | Additional LLM providers — Codex, Z.ai, LM Studio, llama.cpp | — | #54 | +| P8-003 | in-progress | codex | Phase 8 | Performance optimization | — | #56 | +| P8-004 | done | haiku | Phase 8 | Beta release gate — v0.1.0 tag | — | #59 | | FIX-01 | done | Backlog | Call piSession.dispose() in AgentService.destroySession | #78 | #62 | diff --git a/docs/scratchpads/BUG-CLI-scratchpad.md b/docs/scratchpads/BUG-CLI-scratchpad.md index a69911e..4c37825 100644 --- a/docs/scratchpads/BUG-CLI-scratchpad.md +++ b/docs/scratchpads/BUG-CLI-scratchpad.md @@ -1,9 +1,11 @@ # BUG-CLI Scratchpad ## Objective + Fix 4 CLI/TUI polish bugs in a single PR (issues #192, #193, #194, #199). ## Issues + - #192: Ctrl+T leaks 't' into input - #193: Duplicate React keys in CommandAutocomplete - #194: /provider login false clipboard claim @@ -12,28 +14,33 @@ Fix 4 CLI/TUI polish bugs in a single PR (issues #192, #193, #194, #199). ## Plan and Fixes ### Bug #192 — Ctrl+T character leak + - Location: `packages/cli/src/tui/app.tsx` - Fix: Added `ctrlJustFired` ref. Set synchronously in Ctrl+T/L/N/K handlers, cleared via microtask. In the `onChange` wrapper passed to `InputBar`, if `ctrlJustFired.current` is true, suppress the leaked character and return early. ### Bug #193 — Duplicate React keys + - Location: `packages/cli/src/tui/components/command-autocomplete.tsx` - Fix: Changed `key={cmd.name}` to `key={`${cmd.execution}-${cmd.name}`}` for uniqueness. - Also: `packages/cli/src/tui/commands/registry.ts` — `getAll()` now deduplicates gateway commands that share a name with local commands. Local commands take precedence. ### Bug #194 — False clipboard claim + - Location: `apps/gateway/src/commands/command-executor.service.ts` - Fix: Removed the `\n\n(URL copied to clipboard)` suffix from the provider login message. ### Bug #199 — Hardcoded version "0.0.0" + - Location: `packages/cli/src/cli.ts` + `packages/cli/src/tui/app.tsx` - Fix: `cli.ts` reads version from `../package.json` via `createRequire`. Passes `version: CLI_VERSION` to TuiApp in both render calls. TuiApp has new optional `version` prop (defaults to '0.0.0'), passes it to TopBar instead of hardcoded `"0.0.0"`. ## Quality Gates + - CLI typecheck: PASSED - CLI lint: PASSED - Prettier format:check: PASSED