Skip to content

fix(persist): complete the storage boundary; respect binary content - #69

Merged
wolfy-j merged 1 commit into
masterfrom
fix/persist-boundary-completeness
Aug 9, 2026
Merged

fix(persist): complete the storage boundary; respect binary content#69
wolfy-j merged 1 commit into
masterfrom
fix/persist-boundary-completeness

Conversation

@wolfy-j

@wolfy-j wolfy-j commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Audit follow-ups to #67:

  • The UTF-8 boundary now covers every persist write: CREATE/UPDATE data content and metadata, node config and metadata, dataflow metadata, iteration-terminal replacement, commit payloads and metadata (ops, commit, commit_repo).
  • Content declared application/octet-stream passes byte-identical — sanitizing binary is corruption, so the boundary is content-type aware (ensure_storable).
  • A terminal table result that fails json.encode (cycles) persists as its failure text instead of poisoning the completion batch; the test uses a genuinely non-encodable value.
  • The execution-identity contract definition had zero consumers and zero bindings since the app-side retirement; removed with its harness stubs.

Tests: 968 sqlite green.

…nary

Every textual write — content, metadata, config, commit payloads — passes
the encoding boundary; declared octet-stream content passes byte-identical.
A terminal table result that cannot encode persists as its failure text.
The unconsumed execution-identity contract leaves with its stubs.
@wolfy-j
wolfy-j merged commit 9f85f18 into master Aug 9, 2026
11 of 12 checks passed
@wolfy-j
wolfy-j deleted the fix/persist-boundary-completeness branch August 9, 2026 00:49
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