You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Found during the v0.11.2 release review (five-persona pass over v0.11.1..main). None of these block the release; each is deferred because it needs investigation or design work beyond a same-day fix.
Flaky packages/tui/test/cli/tui/dialog-altimate-base.test.tsx — fails roughly 1 in 3–4 local runs with a rendering timeout / blank-frame capture, worse when run alongside sibling files. It covers the migration consent dialog, and app.tsx documents this exact area as a multi-signal startup race (kv.ready, promptHistory.loaded(), local.model.ready, dialog.replace() losing a race), so treat it as a possible canary for a real race under load, not just test infra. One focused pass on the file's wait conditions. (CTO)
Cross-repo consent trust boundary — the TUI's "registered → migrate silently" branch trusts "registered on this machine" as proof a human accepted the disclosure; src/altimate/free/host.ts itself says the server-side check is a text-version agreement, not proof of consent, and the other registering host (VS Code extension, vscode-altimate-mcp-server) lives outside this repo. Document/verify that every registering host shows the disclosure before calling POST /altimate/base/register. Docs for the per-machine semantics were added in v0.11.2 (providers.md). (Chaos Gremlin, privacy/compliance)
workspace-sidebar.tsx renders the server-controlled workspace name without stripControlChars — pre-existing; the CLI path sanitizes rigorously, the opentui path does not. Confirm opentui's cell renderer cannot emit raw escape bytes from text children, or sanitize for symmetry. (CTO, Gremlin)
isSafeHttpUrl duplicated between cli/cmd/link.ts and plugin/tui/altimate/workspace.tsx, kept in sync by comment only. Share it. (CTO)
Decline durability under double corruption — a decline is stored in kv.json and model.json; if both are lost on an already-registered machine, the user is silently re-migrated. Consider a third signal or a re-prompt when both stores are freshly empty on a machine with existing sessions. (Gremlin)
Hard quit right after clicking No can race the async writeJsonAtomic for the decline (matches the file's existing fire-and-forget pattern). (End User)
First-run text loss for a genuinely unconfigured user — the prompt gate still discards the typed prompt when it opens the welcome picker (only the kv-hydration race was fixed). Consider stashing and restoring the draft. (End User)
mergeStartupHistory has no direct unit test (only indirect full-mount coverage). (Tech Lead)
Comment density / inline review history in packages/tui/src/context/local.tsx (~32% comments, 38 marker blocks citing external review-round ids). Extract the migration state machine's invariants into a design doc before the next migration of this shape. (Tech Lead)
altimate-code link always emits OSC 8 bytes on any TTY, gating only the underline on the allowlist; verify Apple Terminal (excluded by name) renders cleanly. (End User)
Related: #1303 (follow-ups from the #1302 review), #1257 (dbt-tools E2E cached-uv flake, hit again on the v0.11.2 head run 34581122233).
Found during the v0.11.2 release review (five-persona pass over v0.11.1..main). None of these block the release; each is deferred because it needs investigation or design work beyond a same-day fix.
packages/tui/test/cli/tui/dialog-altimate-base.test.tsx— fails roughly 1 in 3–4 local runs with a rendering timeout / blank-frame capture, worse when run alongside sibling files. It covers the migration consent dialog, and app.tsx documents this exact area as a multi-signal startup race (kv.ready,promptHistory.loaded(),local.model.ready,dialog.replace()losing a race), so treat it as a possible canary for a real race under load, not just test infra. One focused pass on the file's wait conditions. (CTO)src/altimate/free/host.tsitself says the server-side check is a text-version agreement, not proof of consent, and the other registering host (VS Code extension,vscode-altimate-mcp-server) lives outside this repo. Document/verify that every registering host shows the disclosure before callingPOST /altimate/base/register. Docs for the per-machine semantics were added in v0.11.2 (providers.md). (Chaos Gremlin, privacy/compliance)workspace-sidebar.tsxrenders the server-controlled workspace name withoutstripControlChars— pre-existing; the CLI path sanitizes rigorously, the opentui path does not. Confirm opentui's cell renderer cannot emit raw escape bytes from text children, or sanitize for symmetry. (CTO, Gremlin)isSafeHttpUrlduplicated betweencli/cmd/link.tsandplugin/tui/altimate/workspace.tsx, kept in sync by comment only. Share it. (CTO)writeJsonAtomicfor the decline (matches the file's existing fire-and-forget pattern). (End User)mergeStartupHistoryhas no direct unit test (only indirect full-mount coverage). (Tech Lead)packages/tui/src/context/local.tsx(~32% comments, 38 marker blocks citing external review-round ids). Extract the migration state machine's invariants into a design doc before the next migration of this shape. (Tech Lead)altimate-code linkalways emits OSC 8 bytes on any TTY, gating only the underline on the allowlist; verify Apple Terminal (excluded by name) renders cleanly. (End User)Related: #1303 (follow-ups from the #1302 review), #1257 (dbt-tools E2E cached-uv flake, hit again on the v0.11.2 head run 34581122233).