Skip to content

fix: standardize remaining clarify prompts#100

Merged
rlippmann merged 3 commits intomainfrom
codex/standardize-prompts
Apr 22, 2026
Merged

fix: standardize remaining clarify prompts#100
rlippmann merged 3 commits intomainfrom
codex/standardize-prompts

Conversation

@rlippmann
Copy link
Copy Markdown
Owner

What changed

  • Standardized the remaining clarify prompt_to_user strings in the directive spec for Section 9 cases 1-6 and 10.
  • Aligned engine clarify outputs to those canonical strings without changing directive parsing, state transitions, or pending-clarification behavior.
  • Updated exact-string tests for engine and REPL prompt rendering to match the standardized text.
  • Added focused REPL assertions for missing canonical clarify pass-through cases.
  • Added a new soft-regression behavioral examples test file for examples 03/04/05/06, using branch/call-level assertions rather than transcript snapshots.

Why this change was needed

  • Cases 1-6 and 10 were previously implementation-defined in the spec, which left prompt text non-canonical.
  • Standardizing and asserting these prompts closes contract gaps between spec, engine output, and REPL/user-facing rendering.
  • The added example behavior tests provide lightweight host-pattern regression protection without duplicating the full core conformance suite.

@rlippmann rlippmann merged commit 6e0a6ff into main Apr 22, 2026
12 checks passed
@rlippmann rlippmann deleted the codex/standardize-prompts branch April 22, 2026 06:23
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