Skip to content

refactor(ui): slash commands + @-reference tests#83

Merged
Pterjudin merged 1 commit into
mainfrom
feat/phase1-slash-command-and-at-tests
Jul 23, 2026
Merged

refactor(ui): slash commands + @-reference tests#83
Pterjudin merged 1 commit into
mainfrom
feat/phase1-slash-command-and-at-tests

Conversation

@Pterjudin

Copy link
Copy Markdown

Summary

  • Merge follow-up to refactor(ui): extract @-reference resolution from SidebarChat #82: extract slash-command handling → composer/handleSlashCommand.ts
  • Move resolveAtReferences to common/ (testable + correct import depth for bundle)
  • Add pure extractAtReferenceTokens parser in common/atReferenceTokens.ts (fixes @sym:Name token capture)
  • Unit tests: atReferenceTokens.test.ts, resolveAtReferences.test.ts (@workspace → Folder selection)
  • SidebarChat.tsx now ~574 lines

Test plan

  • npm run buildreact
  • npm run test-phase0-qa — 103/103 passing
  • CDP: @workspace resolver in renderer adds Folder selection
  • CDP: staging chips render after Add File to Chat

Made with Cursor

Move resolveAtReferences to common/, extract handleSlashCommand, add extractAtReferenceTokens with tests, and fix sym: token parsing.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cursor

cursor Bot commented Jul 23, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@Pterjudin
Pterjudin merged commit f8ba110 into main Jul 23, 2026
14 of 26 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.

1 participant