Skip to content

Typed context chips: @threads, @PRs, @skills, terminal selections, several quotes with comments #1526

Closed

Description

@currentbits

Today @ inserts a plain @path string for files and folders (src/mention.ts:36). ⌘⇧C makes a single reply chip (useCiteShortcut.ts:10, composer/ReplyChip.tsx) with no comment. The terminal has no send-to-composer action.

Proposal (M):

  • The @ menu gains Threads, PRs, Skills (and Files as before). Each inserts a typed chip that expands at send time: a thread digest, a PR title/body/diffstat, skill instructions.
  • Select text in the Terminal pane → "Add to prompt" makes an excerpt chip with the command and line range.
  • Quote chips: allow several per draft, each with an optional comment ("this part is wrong because…"); clicking jumps back to the source. Reuse the review-comment chip plumbing (composer/ReviewCommentChips.tsx).
  • Chips survive restart through the existing draft persistence.

Covers #1185 (terminal → composer) and #1216 (quick-action output into the composer).

Activity

  1. added and removed on Oct 9, 2026
  2. currentbits commented on Oct 11, 2026

    @currentbits
    OwnerAuthor

    Starting #1526. Existing plumbing we will reuse: @ still inserts @path text (src/mention.ts); prompt snippets already sit in the same menu; review comments already persist and support several chips with optional notes (src/composerSession.ts, ReviewCommentChips.tsx); the single Reply chip is React state only and does not survive restart; formatThreadDigest already lives in electron/ask.js; terminal already has hasSelection/getSelection. Asking how the @ menu should list Threads / PRs / Skills before writing the design.

  3. currentbits commented on Oct 11, 2026

    @currentbits
    OwnerAuthor

    UX lock: one mixed @ list. Files stay @path text; Threads / PRs / Skills become chips. Design next: one ContextChip bag persisted next to review comments; expand at send; terminal gets Add to prompt only.

  4. currentbits commented on Oct 11, 2026

    @currentbits
    OwnerAuthor

    Design approved (mixed @ list, one ContextChip bag, expand at send). Writing the spec and starting implementation: chip model + persist first, then mention menu, quotes, terminal Add to prompt, skills.read.

  5. currentbits commented on Oct 11, 2026

    @currentbits
    OwnerAuthor

    Spec written (docs/superpowers/specs/2026-10-11-typed-context-chips-design.md) and implementation started.

    In this slice: ContextChip model + formatters; persist in composerSession; mixed @ list inserts thread/PR/skill chips; several quote chips via Reply/⌘⇧C (optional note, click-to-source, survive restart); send expands quotes and clears chips.

    Still to do: live PR/skill mention sources, skills.read + contextChipLoader (thread digest / PR detail), terminal Add to prompt.

  6. currentbits commented on Oct 11, 2026

    @currentbits
    OwnerAuthor

    First slice is up: #1601

    Shipped: mixed @ list, persisted ContextChips (thread/PR/skill/quote), several quote chips with optional notes, expand-at-send.

    Still out of this PR: live PR/skill mention sources, skills.read IPC, send-time loaders in App, Terminal Add to prompt.

  7. added and removed on Oct 11, 2026
  8. currentbits commented on Oct 11, 2026

    @currentbits
    OwnerAuthor

    Landed from Solenta thread "Typed context chips: @threads, @prs, @skills, terminal selec". Closed on merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions