Skip to content

fix(web): show composer status for Cursor - #1250

Merged
heavygee merged 1 commit into
tiann:mainfrom
swear01:agent/cursor-composer-status
Jul 31, 2026
Merged

fix(web): show composer status for Cursor#1250
heavygee merged 1 commit into
tiann:mainfrom
swear01:agent/cursor-composer-status

Conversation

@swear01

@swear01 swear01 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • show the existing composer status bar for Cursor sessions
  • reuse the shared online, thinking, permission, and background-task indicators
  • keep context usage conditional, so Cursor shows it only when ACP supplies usage data

Root cause

The composer hid the entire status bar for Cursor because Cursor native ACP usually does not emit usage_update. That also hid status information HAPI already knows independently of usage.

Impact

Cursor sessions now get the same available operational status as other agent flavors without inventing context or quota data. Enterprise-only Cursor quota APIs remain out of scope.

Related

Validation

  • bun typecheck
  • bun run test — CLI 1543, Hub 673, Web 1613, Shared 142 passed; 3 Hub integration tests skipped

AI disclosure

Code and PR text were prepared with OpenAI Codex.

@swear01
swear01 marked this pull request as ready for review July 30, 2026 07:09

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Findings

  • None.

Summary
Review mode: initial
No diff-introduced correctness, security, regression, data-loss, performance, or maintainability issues found. Residual risk: the Cursor visibility change has no component-level regression test; the patch only removes the previous helper test.

Testing

  • Not run (automation); static review only. Suggested coverage: render HappyComposer for agentFlavor="cursor" without usage data, assert operational status is visible, and assert no context label is rendered.

HAPI Bot

@heavygee heavygee added the low-impact Focused / low-blast-radius change; estate lane B promote candidate (PR merge policy) label Jul 31, 2026
@heavygee
heavygee merged commit 259d69c into tiann:main Jul 31, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

low-impact Focused / low-blast-radius change; estate lane B promote candidate (PR merge policy)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants