Skip to content

fix(tui): hide initial instruction sync#36891

Merged
kitlangton merged 1 commit into
v2from
first-context-event
Jul 14, 2026
Merged

fix(tui): hide initial instruction sync#36891
kitlangton merged 1 commit into
v2from
first-context-event

Conversation

@opencode-agent

Copy link
Copy Markdown
Contributor

Summary

  • mark the first durable instruction value sync as baseline-only live metadata
  • keep the TUI from rendering that baseline sync as an "Instructions updated" message
  • continue rendering genuine mid-session instruction changes

Root cause

The value-delta instruction sync introduced in #36254 persists the initial baseline through the same session.instructions.updated event used for later chronological changes. The model request correctly places those initial values in its system instructions, but the TUI treated every live sync event as a chronological update.

Tests

  • bun test test/session-runner.test.ts (packages/core)
  • bun test test/cli/tui/data.test.tsx (packages/tui)
  • bun typecheck (packages/core)
  • bun typecheck (packages/tui)

@kitlangton
kitlangton merged commit cd9be63 into v2 Jul 14, 2026
8 of 9 checks passed
@kitlangton
kitlangton deleted the first-context-event branch July 14, 2026 16:42
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