Skip to content

fix(ui): make child session pages read-only - #123

Merged
tt-a1i merged 1 commit into
mainfrom
codex/readonly-child-session-page
Aug 24, 2026
Merged

fix(ui): make child session pages read-only#123
tt-a1i merged 1 commit into
mainfrom
codex/readonly-child-session-page

Conversation

@tt-a1i

@tt-a1i tt-a1i commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • remove the message editor from full-screen child session pages
  • keep Direct Subagent and Workflow child pages consistently read-only
  • preserve scrolling, follow-to-end, back navigation, and abort controls
  • keep parent-model steering through the existing subagent_send tool

Root cause

The shared child page supported an optional editor and Direct Subagent supplied its send callback. Accidental text entered while observing a child was therefore forwarded into the active child session.

Validation

  • regression test first reproduced the visible send affordance
  • focused child-page tests: 11 passed, 0 failed
  • bun run check
  • bun run test (Node 889 passed; Vitest 30 passed)
  • git diff --check

The installed stable runtime remains unchanged, so no manual TUI smoke is claimed from the modified checkout.

@tt-a1i
tt-a1i merged commit b5e9bb5 into main Aug 24, 2026
2 checks passed
@tt-a1i
tt-a1i deleted the codex/readonly-child-session-page branch August 24, 2026 14:01
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.

1 participant