Skip to content

feat(worker): add bounded protocol framing - #29

Merged
jmgilman merged 1 commit into
masterfrom
feat/worker-protocol-framing
Aug 25, 2026
Merged

jmgilman merged 1 commit into
masterfrom
feat/worker-protocol-framing

Conversation

@jmgilman

Copy link
Copy Markdown
Contributor

Summary

  • add the private internal/worker frame protocol with uint32 big-endian length prefixes and allocation-before-cap checks
  • strictly decode concrete frame types and preserve exact int64 versus finite float64 JSON values
  • validate manifests, execution limits, payload-cap arithmetic, and parent/child state transitions
  • cover framing, numeric, bound, and lifecycle invariants with the Increment 3 test matrix

Verification

  • mise exec -- golangci-lint run --config .golangci.yml ./internal/worker/... --show-stats=false
  • mise exec -- go test ./internal/... -count=1 -timeout=60s
  • mise exec -- go test ./internal/worker -run '^(TestFrame|TestProtocol|TestValueCodec|TestFrameLimits)' -count=1 -timeout=60s
  • persistent bounded review: zero merge blockers

Scope

Part of the worker-isolation remediation for #12. This increment adds no process launcher or public protocol and does not close the issue; deterministic child supervision follows in Increment 4.

@jmgilman
jmgilman merged commit ca0a490 into master Aug 25, 2026
4 checks passed
This was referenced Sep 11, 2026
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