Skip to content

feat(sidecar): bind decisions to policy-load provenance - #3909

Open
Naveen (1aifanatic) wants to merge 1 commit into
microsoft:mainfrom
1aifanatic:feat/sidecar-policy-provenance
Open

feat(sidecar): bind decisions to policy-load provenance#3909
Naveen (1aifanatic) wants to merge 1 commit into
microsoft:mainfrom
1aifanatic:feat/sidecar-policy-provenance

Conversation

@1aifanatic

Copy link
Copy Markdown
Contributor

Problem and change

Closes #3562. Implements the policy-load provenance proposal by Ali Toygar Abak (@halvrenofviryel).

A sidecar decision currently identifies at most its winning policy, not the policy set available for evaluation or any files rejected during loading. Build a fresh engine and immutable load manifest, publish them together, and return the captured generation's policy_set_id and policy_set_status with each decision.

The content-addressed manifest includes discovered YAML/JSON filenames, raw-byte hashes and load outcomes, including failures. Readiness/reload diagnostics expose file counts and status; the existing policy listing exposes the current manifest. INFO logs retain each completed manifest for deployments to archive. No raw policy content or exception messages enter the manifest. Identical manifests reuse the same ID. This does not change policy decisions, partial-loading permissiveness, or readiness HTTP status, and does not claim atomic filesystem discovery or signed attestation.

Validation

  • Sidecar, policy-schema, and rate-limit tests: 58 passed.
  • Regressions cover mixed YAML/JSON success/failure, failed-content changes, missing vs empty directory, unreadable files, and reload during an in-flight decision.
  • Targeted Ruff lint/format and git diff --check: passed.
  • python scripts/docs/check_links.py: 0 new broken links.
  • python scripts/docs/check_frontmatter.py --strict: 0 findings.
  • Full agent-mesh suite on WSL/Python 3.11 with local sibling sources: 3,716 passed, 123 skipped. Optional integrations requiring unavailable services/backends account for skipped coverage; no claim is made about those paths.

AI assistance

Codex prepared the implementation, tests, and this description under contributor direction. The contributor approved submission after receiving the exact commit diffs, review checklist, tradeoffs, and validation results.

Type and scope

New feature and documentation update in agent-mesh.

Timeline

None.

Signed-off-by: Naveen Chatlapalli <naveen.aifanatic@gmail.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@github-actions github-actions Bot added documentation Improvements or additions to documentation tests agent-mesh agent-mesh package size/L Large PR (< 500 lines) labels Sep 9, 2026
@github-actions

github-actions Bot commented Sep 9, 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 commented Sep 9, 2026

Copy link
Copy Markdown

🟡 Contributor Check: MEDIUM

Check Result
Profile MEDIUM
Credential LOW
Overall MEDIUM

Automated check by AGT Contributor Check.

@github-actions github-actions Bot added the needs-review:MEDIUM Contributor check flagged MEDIUM risk label Sep 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-mesh agent-mesh package documentation Improvements or additions to documentation needs-review:MEDIUM Contributor check flagged MEDIUM risk size/L Large PR (< 500 lines) tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Expose policy-load generation and completeness on sidecar evaluation responses

1 participant