Skip to content

fix(setup): keep post-edit explicitly opt-in - #137

Merged
tt-a1i merged 2 commits into
mainfrom
codex/issue-136-post-edit-opt-in
Aug 25, 2026
Merged

fix(setup): keep post-edit explicitly opt-in#137
tt-a1i merged 2 commits into
mainfrom
codex/issue-136-post-edit-opt-in

Conversation

@tt-a1i

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

Copy link
Copy Markdown
Collaborator

Summary

  • keep the existing empty post-edit command as the canonical default-off state
  • tell the episode-scoped setup tool to set a non-empty command only when the current user explicitly asks for Post-edit
  • preserve unrelated and existing setup values when the field is omitted
  • independently bound the displayed command and stderr/stdout in failure notifications
  • bound rejected execution errors before they reach the TUI
  • cover first-run off, explicit enable/disable, and unrelated-update preservation through the real typed setup tool

The runtime does not add a keyword parser: natural-language intent remains model-owned, while the model-visible field contract carries the opt-in rule.

Validation

  • focused setup/post-edit suite: 25 passed
  • bun run check
  • bun run test: 900 Node tests and 30 Vitest tests passed

Closes #136

@tt-a1i
tt-a1i merged commit 6e82039 into main Aug 25, 2026
2 checks passed
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.

setup: post-edit 默认关闭,只有用户明确 opt-in 才能启用

1 participant