Skip to content

spec: editable prompt suggestions (#9842)#10232

Open
lonexreb wants to merge 1 commit intowarpdotdev:masterfrom
lonexreb:spec/9842-editable-prompt-suggestions
Open

spec: editable prompt suggestions (#9842)#10232
lonexreb wants to merge 1 commit intowarpdotdev:masterfrom
lonexreb:spec/9842-editable-prompt-suggestions

Conversation

@lonexreb
Copy link
Copy Markdown
Contributor

@lonexreb lonexreb commented May 6, 2026

Spec for #9842. New insert-as-draft affordance (edit icon + modifier-click) on each prompt suggestion. Inserts the text into the agent input without sending; user edits and submits manually. Existing send-on-click behavior preserved.

Closes (spec-only) #9842.

@cla-bot cla-bot Bot added the cla-signed label May 6, 2026
@github-actions github-actions Bot added the external-contributor Indicates that a PR has been opened by someone outside the Warp team. label May 6, 2026
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented May 6, 2026

@lonexreb

I'm starting a first review of this spec-only pull request.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Copy link
Copy Markdown
Contributor

@oz-for-oss oz-for-oss Bot left a comment

Choose a reason for hiding this comment

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

Overview

This spec adds an insert-as-draft path for prompt suggestions while preserving the existing send-on-click behavior.

Concerns

  • The affordance contract does not unambiguously require both the edit icon and modifier-click path, even though the PR body describes both.
  • Existing draft handling is left for a future TECH decision, so implementation and acceptance tests can pick different behaviors.
  • The telemetry category mapping needs a named source in the existing model to keep event payloads consistent and private.

Verdict

Found: 0 critical, 2 important, 1 suggestions

Request changes

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Comment thread specs/GH9842/SPEC.md
Comment on lines +22 to +25
- **New: insert-as-draft.** Discoverable via either:
- A small "edit" icon next to the suggestion (preferred, low
friction).
- Modifier-click on the suggestion (Cmd-click / Ctrl-click).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ [IMPORTANT] State whether both the edit icon and modifier-click are required; this currently permits implementing only one path while the PR body promises both.

Comment thread specs/GH9842/SPEC.md
Comment on lines +26 to +30
- B2. Insert-as-draft replaces the current agent-input contents
with the suggestion text. If the input already has user text,
show a one-time confirmation tooltip ("Replace existing
draft?") OR insert at the current caret position — TECH spec
picks one.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ [IMPORTANT] Choose the existing-draft behavior in this spec instead of deferring it to TECH so implementation and acceptance tests have one deterministic contract.

Comment thread specs/GH9842/SPEC.md
Comment on lines +37 to +40
- B6. Telemetry: emit a single `prompt_suggestion_inserted_as_draft`
event when the user takes the new affordance. No payload beyond
the suggestion category (e.g., "zero_state", "follow_up") —
never the suggestion text or the user's edited result.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

💡 [SUGGESTION] Define how suggestion category maps to existing PromptSuggestion fields so implementers do not invent inconsistent categories or use text-derived values.

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

Labels

cla-signed external-contributor Indicates that a PR has been opened by someone outside the Warp team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant