Skip to content

PROD-318 feat(hooks): started-stage CONSTRAIN adapter seam - #11

Open
salamisandwich77 wants to merge 12 commits into
feat/PROD-250-sandbox-core-contractfrom
feat/PROD-318-behavioral-constrain
Open

PROD-318 feat(hooks): started-stage CONSTRAIN adapter seam#11
salamisandwich77 wants to merge 12 commits into
feat/PROD-250-sandbox-core-contractfrom
feat/PROD-318-behavioral-constrain

Conversation

@salamisandwich77

@salamisandwich77 salamisandwich77 commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

Adapter hooks for behavioral CONSTRAIN.

  • handle_constrain / handle_constrain_sync adapter callbacks (no-op defaults)
  • Started-stage CONSTRAIN routed through the adapter instead of discarded
  • Sandbox evidence keys: openbox.sandbox.provider, dispatch_id, egress decisions, violation counts
  • Sandbox span parented to the trigger span

Tests: 642 passed.

@salamisandwich77 salamisandwich77 changed the title PROD-318 feat(hooks): dispatch started-stage CONSTRAIN through the adapter PROD-318 feat(hooks): started-stage CONSTRAIN adapter seam Aug 19, 2026
@salamisandwich77
salamisandwich77 changed the base branch from main to feat/PROD-250-sandbox-core-contract August 26, 2026 07:54
@salamisandwich77
salamisandwich77 marked this pull request as ready for review August 26, 2026 07:55
The dispatcher already built ExecutionMetadata with egress_decisions,
violation_count, and violation_categories, and the span mapper read them,
but the dataclass never declared the fields. Every sandbox execution path
reached ExecutionMetadata(...) with unexpected keyword arguments and raised
TypeError, so the egress span attributes could never be produced.

Add the three fields with safe defaults and thread the evidence type through.
The retry_plan field was renamed to patch, and a later commit adapted the
parsing test to that vocabulary. The stale test asserted a field that no
longer exists; the patch_parsing test covers the current wire shape.
@salamisandwich77
salamisandwich77 force-pushed the feat/PROD-318-behavioral-constrain branch from 0d0f918 to 44db804 Compare August 27, 2026 17:03
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