Skip to content

feat: add governance action context#83

Merged
haasonsaas merged 1 commit intomainfrom
codex/governance-action-context
Apr 15, 2026
Merged

feat: add governance action context#83
haasonsaas merged 1 commit intomainfrom
codex/governance-action-context

Conversation

@haasonsaas
Copy link
Copy Markdown
Contributor

Summary

  • add governance.v1.ActionContext for magnitude-aware action evaluation and classification
  • attach structured context to both EvaluateActionRequest and ClassifyRiskRequest
  • update generated Go, TS, and Python artifacts plus governance contract fixture coverage

Testing

  • buf lint
  • buf generate --path proto/governance/v1/governance.proto --timeout 10m
  • go test ./... -count=1
  • npm ci && npm run check:package
  • python -m build (blocked locally: build module not installed in this environment)

Part of #25

@cursor
Copy link
Copy Markdown

cursor bot commented Apr 15, 2026

PR Summary

Medium Risk
This is a wire-level proto change adding new request fields, so it requires coordinated client/server regeneration and rollout to avoid compatibility issues across services.

Overview
Adds a new structured governance.v1.ActionContext message (record count, currency amount/code, environment, data classification) and includes it on EvaluateActionRequest and ClassifyRiskRequest.

Regenerates Go/Python/TS protobuf artifacts and updates the governance evaluate_action_request.json fixture plus contract_test.go assertions to cover the new context fields.

Reviewed by Cursor Bugbot for commit 6aff1ec. Bugbot is set up for automated code reviews on this repo. Configure here.

@haasonsaas haasonsaas merged commit 3382ec5 into main Apr 15, 2026
9 checks passed
@haasonsaas haasonsaas deleted the codex/governance-action-context branch April 15, 2026 18:14
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