Skip to content

docs(adr): reconcile accepted and superseded statuses - #3891

Merged
MohammadHaroonAbuomar merged 1 commit into
microsoft:mainfrom
carloshvp:docs-audit-adr-statuses
Sep 13, 2026
Merged

MohammadHaroonAbuomar merged 1 commit into
microsoft:mainfrom
carloshvp:docs-audit-adr-statuses

Conversation

@carloshvp

@carloshvp Carlos Hernandez (carloshvp) commented Sep 6, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • reconcile the ADR index with implementation state across the repository
  • mark ADR-0007, ADR-0026, and ADR-0031 as accepted
  • move ADR-0032 to Accepted because its file already records that status
  • move ADR-0030 to Accepted alongside the file correction in docs(adr): mark ADR-0030 accepted #3890
  • classify ADR-0011 and ADR-0014 as superseded after the v4 policy-language removal
  • add implementation links to the promoted ADRs

PR #3890 owns ADR-0030's file status. This PR owns the complete index
reconciliation, so the two changes can merge in either order.

Audit evidence

ADR Corrected status Evidence
0007 accepted External JWKS provider in #2380 and handshake integration in #3094
0011 superseded The ADR stub points to the native ACS verdict and PolicyEvaluation replacement
0014 superseded The ADR stub points to ACS extends and the one-way v4 migration path
0026 accepted Runnable Foundry AI Gateway PDP reference implementation in #2536
0030 accepted Python implementation in #3015, #3076, #3096, #3097, #3100, and #3106; file status in #3890
0031 accepted Python and Rust evidence-backend implementation in #3014
0032 accepted The ADR file already says accepted; implementation landed in #3098 and #3099

The audit retained ADR-0005, ADR-0006, ADR-0008, ADR-0010, ADR-0027,
ADR-0028, and ADR-0029 as proposed. Their defined implementation is absent,
partial, or still tracked. Examples include the open ADR-0010 handshake work
in #3162, the open Studio epic in #2729, and the repository's continued MCP
2025-11-25 implementation rather than ADR-0027's proposed 2026-07-28
dual-stack design.

Validation

  • python3 scripts/docs/check_links.py
  • python3 scripts/docs/check_frontmatter.py --strict on all changed pages
  • changed-line cspell check
  • ADR index uniqueness and file-status checks
  • git diff --check

@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 size/M Medium PR (< 200 lines) labels Sep 6, 2026
@github-actions

github-actions Bot commented Sep 6, 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.

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

@MohammadHaroonAbuomar MohammadHaroonAbuomar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved after group integration review (tests, gates and adversarial pass on the combined change).

@MohammadHaroonAbuomar
MohammadHaroonAbuomar merged commit 545a2df into microsoft:main Sep 13, 2026
86 checks passed
Karim Mehalebi (karimad) pushed a commit to karimad/agent-governance-toolkit that referenced this pull request Sep 14, 2026
Signed-off-by: Carlos Hernandez-Vaquero <carloshvp@gmail.com>
Yuvraj Singh (yuvrajsingh2428) pushed a commit to yuvrajsingh2428/agent-governance-toolkit that referenced this pull request Oct 1, 2026
Signed-off-by: Carlos Hernandez-Vaquero <carloshvp@gmail.com>
Signed-off-by: yuvrajsingh2428 <offcyuvi2428@gmail.com>
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/M Medium PR (< 200 lines)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants