Skip to content

[L2-07] Add slash skills and explicit @ context to Ask #41

Description

@dkritarth

Outcome

Let user discover supported prompt skills and add papers/files intentionally to Ask context.

Dependencies

Blocked by #40.

Scope

Slash picker for allowlisted skills; @ picker for valid local papers/files; typed context model reaching ACP contextFiles; visible removable chips; permission/path validation. No arbitrary shell commands.

Acceptance criteria

  • / opens keyboard-accessible picker with supported skills and descriptions.
  • @ searches permitted papers/files and adds visible context chip.
  • Selected skill changes prompt through defined template/contract.
  • Context reaches ACP as validated allowlisted paths.
  • Duplicate, missing, deleted, cross-workspace, and unsafe paths are rejected.
  • User can review/remove context before send.
  • Persisted chat records enough context provenance to understand prior answer.

Live verification

Invoke multiple skills, add two papers, remove one, send through Codex, inspect sanitized ACP request, test unsafe/deleted/cross-workspace path, keyboard flow, and restart/history behavior.

Required evidence

Focused PR uses Closes #41. Provide failing test/baseline, focused and full tests, typecheck, build, diff check, exact live Electron steps, screenshots where applicable, console state, restart/persistence result, and limitations. Reviewer independently exercises live behavior.

Activity

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

    area:acpACP adapter, agent session, or grounding workarea:electronElectron main, preload, IPC, or live app workarea:uxUser-facing workflow or interfacephase:2-libraryPhase 2: durable research organizationstatus:blockedDo not start; dependency or phase gate remains open

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions