Skip to content

Widen the codex filing recognizer's create terminator to accept newline - #731

Merged
clkao merged 1 commit into
mainfrom
spacedock-ensign/filing-recognizer-newline-terminator
Aug 18, 2026
Merged

Widen the codex filing recognizer's create terminator to accept newline#731
clkao merged 1 commit into
mainfrom
spacedock-ensign/filing-recognizer-newline-terminator

Conversation

@clkao

@clkao clkao commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

A compliant agent that filed an entity with a newline-terminated command was graded as if it never filed at all. This stops the codex-live filing journey from reddening correct behavior.

What changed

  • Widen codexFilingCreateCount's create terminator to accept a newline.
  • Widen commandFilesViaNew's direct-invocation terminator to accept a newline.
  • Pin both classes with the failing run's exact 722-byte public event.
  • Add a malformed-verb negative row against those same run bytes.

Evidence

  • go test ./internal/ensigncycle/... -race -count=1: 44/44 filing subtests passed, including 22 red-expectation rows.
  • Adversarial matrix: 27/27 shapes graded correctly; no newly-accepted shape that a space, tab, quote, or end-of-string terminator did not already accept.

ca9

commandFilesViaNew's direct-invocation terminator (shared_filing_test.go:53)
and codexFilingCreateCount's create counter (:183) both stopped at
[ \t';|&] or end-of-string, so a compliant create followed by a
verification command on the next line (run 32105482382) graded as
"0 atomic creates" — punishing an FO that verified its own filing.

Add the exact run-32105482382 public bytes (codex-exec.jsonl line 14,
artifact 9313785785) as a fixture constant plus two seven-rung-matrix
rows: the real create+newline+verify transaction now passes, and the
same bytes with the create verb corrupted stay red, pinning both
terminator classes with one positive and one negative row.
@clkao
clkao merged commit 34dca09 into main Aug 18, 2026
5 of 8 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.

1 participant