Prepare public-first contribution documentation - #894
Draft
Brandon Miller (brandom-msft) wants to merge 3 commits into
Draft
Prepare public-first contribution documentation#894Brandon Miller (brandom-msft) wants to merge 3 commits into
Brandon Miller (brandom-msft) wants to merge 3 commits into
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Brandon Miller (brandom-msft)
temporarily deployed
to
L4-validation
August 7, 2026 22:39 — with
GitHub Actions
Inactive
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Brandon Miller (brandom-msft)
temporarily deployed
to
L4-validation
August 7, 2026 22:47 — with
GitHub Actions
Inactive
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Brandon Miller (brandom-msft)
temporarily deployed
to
L4-validation
August 7, 2026 22:52 — with
GitHub Actions
Inactive
foundry-samples-repo-sync Bot
pushed a commit
that referenced
this pull request
Aug 8, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 002c35aa-a47b-4b64-91ff-9577c98a4af6
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
README.mdandCONTRIBUTING.mdmainbehaviorCurrent verified baseline — 2026-08-07
mainisc188cbb30626e99b08be903998b683d8c5fb5dfd; its README and contributing guide still direct sample contributions to private staging.trustedcontext from GitHub Actions App integration 15368 with 0 approving reviews.disabled_manually, with zero queued or in-progress runs.public-pr-commentandpromote-to-publicdo not exist.microsoft-foundry/foundry-samples#895is OPEN/DRAFT/CLEAN at provisional head94fe10eb, distinct from private living-documentationmicrosoft-foundry/foundry-samples-pr#895. Its current code and tests correct the earlier97564019blockers, but the artifacts remain absent from publicmain; there is no independent review, merge, P4 wiring, or pin. The classifier remains TARGET and inert, and its release gate remains unchecked..github/public-transition-manifest.jsonand PUBLIC matcher at.github/scripts/public_transition_manifest.py, using glob + classification vocabulary. P4 consumes that matcher and owns readiness facts. Privatepublic_pr_transition.pyis orchestration only. Public artifacts must land before or atomically with consumers, which fail closed until present. No cadence schedule exists.Delegated targets are derived rather than statically inventoried in the manifest: Hosted Agents rules use
tracked-sample-yaml-roots; Bicep rules use routing-onlyrule-rootmetadata and do not produce dashboard targets. Paths and counts are readiness evidence, not manifest authority.The canonical CLI is
python .github/scripts/public_transition_manifest.py classify --manifest .github/public-transition-manifest.json --paths-file <file> --format json. Matching uses ordered first-matchp5-posix-v1rules with stable IDs, singularglobandclassificationfields, classification enuminternal-only | private-workflow-dependent | public-ready, unmatched defaultunresolved, andcheck_class/dispositionplus conditional delegation.patternsandcategoryare not contract vocabulary.Consumers may develop against test-only fixtures but remain fail-closed; after public artifacts land, P4 consumes a public checkout and private orchestration fetches verified public-main blobs. Active private sync continues to block routing.
The current documentation head is provisional, not release-final.
Caution
DO NOT MERGE. Keep this living PR draft and retain both transition notices until all four merge gates are proven against live state. Direct-public behavior remains TARGET and non-current.
Merge gates
disabled_manuallyand no run is active or queued while private sync workflow 233646160 remains active.disabled_manually. While private sync remains active, prove command rejection and zero public mutation. Deny internal-only, mixed-classification, and unresolved inputs, and require the exact dependent checks for the requested SHA. The intended output is a public pull request only, never an automatic merge; the inert proof must not perform that public mutation.disabled_manuallyand no run is active or queued.trustedbehavior is verified.During the allowed pre-first-public-merge rollback window, re-enabling either legacy workflow blocks promotion automatically.
The 34 open
mirror/public-*pull requests in the private repository remain inert; zero have auto-merge armed, and no backlog pull request was mutated.Contributor shorthand: private publication sync is retired and public-first routing is operational.
Before merge, remove both Transition draft notices, re-review the final copy against live behavior, and confirm the diff remains limited to
README.mdandCONTRIBUTING.md.Coordinated documentation release
Public PR 894 and private PR 895 release as one controlled slice only after both legacy workflows are retired with no active or queued runs, the public production manifest
.github/public-transition-manifest.jsonand matcher.github/scripts/public_transition_manifest.pyare approved and proven with glob + classification behavior, P4 readiness-fact integration is proven, privatepublic_pr_transition.pyis proven orchestration-only, public artifacts land before or atomically with fail-closed consumers, and marked-comment exact-SHA promotion is proven to open an attributed public pull request and never auto-merge. Record both final documentation SHAs before release. If the private half cannot land after the public half, revert the public documentation immediately; do not leave CURRENT and TARGET guidance split across repositories.Claim boundaries
This draft says that, after all four gates pass:
This draft intentionally does not claim fleet health, active cadence, automated refresh, full-repository validation coverage, Rust validation, exercised L4, implemented manifest behavior, private-status reading, quarantine, strike counters, team routing, centralized backlog draining, or bulk mirror-PR closure. It does not recommend forced full exports, marks repair, or replay.
The pinned view in #892 remains the two-row C#/Python presentation surface; missing results are Never run. This unchanged two-row pilot is an integration prerequisite, not final dashboard coverage. After P4.1 lands a finalized producer schema, a separate renderer/config integration slice must consume it, and issue 892 must visibly render real results for the complete derived target set. P4.1 and the full-set dashboard are sample-root-only. The current readiness snapshot is 72 sample rows: 68 public-ready plus 4 Hosted Agents delegated. Bicep
rule-rootremains canonical metadata for P5/private routing and ADO 5502656, contributes zero P4.1/dashboard rows, and must not create synthetic language, L3, or L4 results. Counts and paths remain re-derived evidence, not manifest authority. PR 893 at a79a438 implements only the dashboard renderer, two-sample configuration, and presentation tests in an open PR. The accepted ownership split assigns manifest consumption, validation orchestration, normalized JSON artifacts, issue-state publishing, schedule, and quarantine to the separate workflow implementation; none of those capabilities is present in PR 893 or operational. Workflow implementation has started but has no reviewed code yet. Any workflow rerun, merge, dispatch, schedule, or quarantine activation remains separately gated. There is no Rust or full-repository validation and no exercised L4 coverage.Validation
git diff --checkREADME.mdandCONTRIBUTING.mdmain, ruleset 6131793, workflows 233646160 and 289478915, absent promotion entry points, issue 892, and PR 893