Skip to content

feat: GOVERNANCE_DENIED error, flatten comply_test_controller, context/ext, require signal_id#2194

Merged
bokelley merged 2 commits into
mainfrom
bokelley/governance-denied-error
Apr 15, 2026
Merged

feat: GOVERNANCE_DENIED error, flatten comply_test_controller, context/ext, require signal_id#2194
bokelley merged 2 commits into
mainfrom
bokelley/governance-denied-error

Conversation

@bokelley

Copy link
Copy Markdown
Contributor

Summary

Also fixes get-content-standards-response.json variant 0 missing ext.

Closes #2185, closes #2189, closes #2190, closes #2192

Test plan

  • Transport error mapping tests pass (83/83)
  • Extension field tests pass (20/20)
  • Unit tests pass (587/587 via pre-commit hook)
  • TypeScript typecheck passes
  • Schema build succeeds
  • Mintlify docs validation passes (no broken links)
  • CI passes

🤖 Generated with Claude Code

bokelley and others added 2 commits April 15, 2026 21:53
…t/ext on all schemas, require signal_id (#2185, #2189, #2190, #2192)

- Add GOVERNANCE_DENIED to standard error codes (correctable recovery)
- Flatten comply_test_controller from oneOf union to flat object with
  scenario discriminant and if/then conditional validation
- Add missing context and ext fields to all request/response schemas
  (governance, collection, property, sponsored-intelligence, content-standards)
- Make signal_id required on get-signals-response signal items
- Fix get-content-standards-response variant 0 missing ext

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Doc JSON examples need signal_id now that it's required on the
get-signals-response schema.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@bokelley bokelley merged commit b674082 into main Apr 15, 2026
16 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

1 participant