Skip to content

feat(go): add approval chain parity#3242

Open
carloshvp wants to merge 1 commit into
microsoft:mainfrom
carloshvp:go-approval-chain-parity
Open

feat(go): add approval chain parity#3242
carloshvp wants to merge 1 commit into
microsoft:mainfrom
carloshvp:go-approval-chain-parity

Conversation

@carloshvp

Copy link
Copy Markdown
Contributor

Summary

  • add Go SDK approval-chain protocol objects, action binding digests, and a fail-closed coordinator for require_approval
  • add a versioned webhook approval transport that validates request/action binding echoes and requires verified approver identity for allow responses
  • wire AgentMeshClient to optionally route require_approval decisions through an approval coordinator
  • document the new Go approval-chain surface

Scope

This is the Go SDK slice for #3083. TypeScript already has a separate parity PR in flight, so this PR keeps the language scope to agent-governance-golang/.

Validation

  • /tmp/codex-go1.25.11/go/bin/gofmt -w agent-governance-golang/packages/agentmesh/approval.go agent-governance-golang/packages/agentmesh/approval_test.go agent-governance-golang/packages/agentmesh/client.go agent-governance-golang/packages/agentmesh/types.go
  • /tmp/codex-go1.25.11/go/bin/go test ./... from agent-governance-golang/

Signed-off-by: Carlos Hernandez <carloshvp@gmail.com>
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

PR Review Summary

Check Status Details
🔍 Code Review ⚠️ Missing No current-run comment
🛡️ Security Scan ⚠️ Missing No current-run comment
🔄 Breaking Changes ⚠️ Missing No current-run comment
📝 Docs Sync ⚠️ Missing No current-run comment
🧪 Test Coverage ⚠️ Missing No current-run comment

Verdict: ⚠️ AI review incomplete; ready for human review

AI review comments are untrusted advisory output. The summary reports workflow-generated completion status only, not model-authored pass/fail claims.

@github-actions github-actions Bot added documentation Improvements or additions to documentation size/XL Extra large PR (500+ lines) labels Jul 1, 2026
@carloshvp carloshvp marked this pull request as ready for review July 1, 2026 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size/XL Extra large PR (500+ lines)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant