Skip to content

fix(tui): hide editor context from transcript#36264

Merged
kitlangton merged 2 commits into
v2from
hide-editor-context
Jul 10, 2026
Merged

fix(tui): hide editor context from transcript#36264
kitlangton merged 2 commits into
v2from
hide-editor-context

Conversation

@kitlangton

@kitlangton kitlangton commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • admit Zed editor context as a separate descriptionless synthetic message with resume: false
  • send the user-authored text unchanged as the following visible prompt
  • document the project rule to get user buy-in before adding complexity for speculative, vanishingly unlikely races or security edge cases

Root cause

The V2 prompt migration flattened the synthetic editor-context part and user-authored text into one user message. The V2 renderer therefore displayed the internal <system-reminder> verbatim.

The existing synthetic-message projection already hides descriptionless synthetic context, so preserving that boundary fixes the transcript without adding display metadata or special rendering paths.

Verification

  • bun typecheck in packages/tui
  • bun run test in packages/tui (223 passed, 1 skipped)
  • repository-wide push-hook typecheck (31 packages)

@kitlangton
kitlangton merged commit 41d0a9f into v2 Jul 10, 2026
9 checks passed
@kitlangton
kitlangton deleted the hide-editor-context branch July 10, 2026 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant