[ptr727]

Promotes develop to main, carrying one merged pull request.

Closes #1522

What this promotes

• #1533 corrects WORKFLOW.md D2.2's explanation of why validate-release exits inside its step on a smoke build rather than skipping at job level. It named github-release as a dependent that does not opt out of a skipped needs: entry, where github-release's if: carries !failure() && !cancelled() and so runs past a skipped gate. D2.2 now names dotnet-publish, build-nuget, and build-pypi, whose if: carries no status-check function, so the implicit success() would skip their smoke builds. It also drops the sentence naming if: always() as the only opt-out, which D7.4 and build-docker's own guard contradict, and states only the direction that holds. The workflow-ci-contract Skill's generated include carries the same text. It settles the last open item of WORKFLOW.md D3.4 Promises a PyPI Ordering Nothing Implements, and D2.2 Names a Job That Opts Out #1522, item 1 of Eight more WORKFLOW.md section 4 claims the workflows falsify or a reader cannot act on #1200, and item 28 of tracker Tracker: WORKFLOW.md > 4. Behavioral Contract: Expected Outcomes #1369.

Adopter impact

Documentation only. No workflow, action, or script changes, so no fleet repository's build or publish behavior moves.

Left open deliberately

• #1200: only its first item is settled here.
• Whole-unit review of section 4 raised further statements this change did not write, each filed rather than fixed: WORKFLOW.md Sections 3, 4 and 6 Carry Statements the Tree Contradicts #1523, WORKFLOW.md Sections 3 and 4 Carry Further Statements the Tree Contradicts #1524, WORKFLOW.md Sections 4 and 6 Carry Further Statements the Tree Contradicts, Found Settling #1522 #1527, WORKFLOW.md D2 Does Not Describe the Release Gate's Branch-Name Refusal #1531, and Tracker: WORKFLOW.md > 4. Behavioral Contract: Expected Outcomes #1369 item 55 for section 4, and Tracker: docs/reusable-workflows.md #1360 for docs/reusable-workflows.md "Permissions".

Verification

• Four whole-unit review rounds over WORKFLOW.md section 4 and its Skill include, recorded in reports/canonical-review.json, the last finding nothing in D2.2, and a local strict-review pass over the whole diff. The wording went past the two-round budget with the maintainer's explicit go-ahead, since each "skips unless" form drew the same finding.
• Copilot reviewed Name the Build Jobs That Do Not Opt Out of the Smoke Skip in D2.2 #1533 at full coverage, 6 of 6 files, with no findings.
• Prose gate, repo gate, spec validation, generated-skill check, and the Docker linters are green.

🤖 Generated with Claude Code

https://claude.ai/code/session_01QNQoguANRoFqeCJL7Kczc8

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: e60eecb6-2eb1-43ad-903c-e15ecbc628cb

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

🟢 Approval recommended

The documentation correction is consistent with the repository's workflows and the associated generated and review-metadata updates are coherent with the source change.

Pull request overview

This promotion updates the Section 4 behavioral contract documentation to correctly name which build jobs would be skipped on smoke runs if validate-release were job-skipped, and carries the corresponding Skill reference and canonical review metadata updates into main.

Changes:

• Update WORKFLOW.md D2.2 to name dotnet-publish, build-nuget, and build-pypi as the dependents that would be skipped on smoke if validate-release were job-skipped.
• Apply the same D2.2 wording update to the workflow CI contract Skill source (.agents/skills/...) and its generated distributions.
• Refresh reports/canonical-review.json and the fleet-skills source digest to match the updated unit content.

File summaries

File	Description
WORKFLOW.md	Corrects D2.2 prose to name the actual validate-release dependents that would skip on smoke if the gate were job-skipped.
reports/canonical-review.json	Updates recorded unit digests, hub commit, timestamps, and findings counts for the relevant reviewed units.
.agents/skills/workflow-ci-contract/references/d-guarantees.md	Updates the Skill source copy of the behavioral contract to match the corrected D2.2 wording.
.github/skills/workflow-ci-contract/references/d-guarantees.md	Regenerates the GitHub-distributed Skill reference content to match the source update.
.claude-plugin/fleet-skills/skills/workflow-ci-contract/references/d-guarantees.md	Regenerates the Claude plugin Skill reference content to match the source update.
.claude-plugin/fleet-skills/.source-digests/workflow-ci-contract	Updates the recorded source digest for the regenerated workflow-ci-contract Skill distribution.

Review details

• Files reviewed: 6/6 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.