Skip to content

feat(tui): add capability keyword shimmer - #416

Merged
tt-a1i merged 1 commit into
openpi-dev:mainfrom
testikun:codex/issue-301-shimmer
Sep 7, 2026
Merged

feat(tui): add capability keyword shimmer#416
tt-a1i merged 1 commit into
openpi-dev:mainfrom
testikun:codex/issue-301-shimmer

Conversation

@testikun

@testikun testikun commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Problem

Closes #301. Capability keywords were highlighted with one static color, so the pre-submit affordance had no shimmer feedback.

Value

Authorized subagent/workflow keywords now receive a subtle animated lavender shimmer while preserving text, layout, authorization, and model context.

Approach

  • Add a pure phase-driven truecolor renderer with static 256-color fallback.
  • Advance the phase with a session-scoped 120ms timer and clean it up on shutdown.
  • Keep highlighting gated by the existing shared capability classifier.
  • Add focused renderer and fallback tests.

Validation

  • Node 24 focused test: 6 passed.
  • Biome lint: passed.
  • TypeScript: passed.
  • git diff --check: passed.
  • Full bun run check / bun run test: unavailable in this environment because Bun is not installed.

Impact

No persisted state, model-visible context, tool surface, or capability semantics changed.

@testikun
testikun force-pushed the codex/issue-301-shimmer branch from 9f1bcd8 to 7fa1f8c Compare September 6, 2026 07:45

@tt-a1i tt-a1i left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the exact head against current main, along both repository standards and issue #301 requirements. No P2 or higher issue confirmed. The change adds lavender animation to already-authorized capability keywords without changing text, classifier decisions, tools or model context. Focused renderer tests passed 6/6; current required CI is green and Git merge-tree reports a clean integration. Nonblocking: the 256-color fallback is static but still requests renders every 120 ms; avoiding those redundant ticks would improve efficiency. Product recommendation: optional polish, lower priority than Web observability. No native TUI visual acceptance or merge performed.

@tt-a1i
tt-a1i merged commit 4e767e4 into openpi-dev:main Sep 7, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(tui): add Claude-style purple shimmer for capability keywords

2 participants