Skip to content

feat(engine): transport response_presentation across session paths (#1331) - #1632

Closed
jacktrnr wants to merge 1 commit into
mainfrom
1331-response-presentation-transport
Closed

jacktrnr wants to merge 1 commit into
mainfrom
1331-response-presentation-transport

Conversation

@jacktrnr

Copy link
Copy Markdown
Contributor

Implements #1331 (part of #1330 — the response-presentation / "concise mode" epic). TDD, tracer-bullet by tracer-bullet.

Progress

  • Shared enum — ResponsePresentation (concise | standard | detailed) in @opencode-ai/schema, with a rejecting contract test.
  • v1 admission — optional field on PromptInput, ShellInput, CommandInput; HTTP payloads derive from these, so admission propagates.
  • v2 wire contract + core session.prompt threading
  • Persist on the admitted session_input (queue/replay parity, AC2)
  • v2 server handler + protocol payload forward
  • Regenerate JS SDK + client generated artifacts (AC3)
  • Full compatibility matrix (test(app): verify response-presentation compatibility matrix #1334 is the dedicated verifier)

Protected regressions (green)

Closes #1331

…admission (#1331)

Add the shared response_presentation enum (concise|standard|detailed) to
@opencode-ai/schema and admit it (optional) on the v1 session request
contracts — PromptInput, ShellInput, CommandInput — so it validates at the
trusted request boundary and rejects unknown values. Payloads derive from
these fields, so the HTTP admission propagates automatically.

Transport-only: no provider interpretation. v2 wire/persistence, server
handler, and SDK regeneration follow in the same slice.

Part of #1331 (parent #1330)
@coderabbitai

coderabbitai Bot commented Sep 28, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jacktrnr

Copy link
Copy Markdown
Contributor Author

Superseded by the concise-mode skill (PR #1633). We chose a toggleable output-shaping skill over the response-presentation engine epic — it delivers 'toggle concise on/off' at a fraction of the cost with no fork expansion. The v1 enum work here isn't needed for that route. Closing unmerged; branch left for reference.

@jacktrnr jacktrnr closed this Sep 28, 2026
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.

feat(engine): transport response presentation across session paths

1 participant