Drop D1.6's Universal-Sounding Only-X-Carries-It Claim

[ptr727]

Summary

• Copilot review on PR Scope WORKFLOW.md D1.6's secrets.json Pointer to Applicable Repos #906/Promote Develop to Main #907 caught that "only a repo of that type and shape carries the typeMechanisms entry" reads as a universal claim, but this hub repo's own spec/secrets.json also carries typeMechanisms despite not being registry-typed csharp/python (it is source-only/docs per registry/repos.json).
• Reworded to a permissive absence claim ("may carry no entry") scoped to where D1.6 does not apply, true regardless of why a given repo's own copy does or does not carry the key.

Verification

• scripts/prose_lint.py --diff origin/develop: 0 issues
• markdownlint-cli2 WORKFLOW.md: 0 issues

Related to #728.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Documentation
  • Clarified that spec/secrets.json may omit typeMechanisms when Codecov coverage requirements do not apply.
  • Updated guidance for repositories without applicable C# or Python test coverage requirements.

[qodo-code-review]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Tip of the day

💡 Did you know, you can commit Qodo's fix in one click with committable suggestions (GitHub & GitLab)

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: d88964dd-a162-4a69-a016-9c3cc6106aa1

📥 Commits

Reviewing files that changed from the base of the PR and between 538b471 and ea27cc0.

📒 Files selected for processing (1)

• WORKFLOW.md

Included review availability: Your plan provides up to 10 included reviews per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

The workflow documentation now permits repositories without Codecov requirements to omit typeMechanisms from spec/secrets.json.

Changes

Codecov configuration

Layer / File(s)	Summary
Update Codecov requirement WORKFLOW.md	The guidance permits non-applicable repositories to omit typeMechanisms from spec/secrets.json.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to ea27c

This is a localized documentation wording correction with no runtime or product behavior change, and no actionable merge-blocking risk remains after normal checks and review.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change: removing D1.6's universal-sounding claim about which repositories carry typeMechanisms.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix-workflow-d1-6-secrets-pointer-r2

---

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

[qodo-code-review]

PR Summary by Qodo

Clarify D1.6 wording around optional typeMechanisms in spec/secrets.json

📝 Documentation 🕐 Less than 10 minutes

AI Description

• Rewords D1.6 to avoid a universal claim about which repos carry typeMechanisms.
• Clarifies that absence of typeMechanisms is acceptable where the D1.6 guarantee does not apply.

High-Level Assessment

The chosen approach (tightening the statement from a universal claim to a scoped permissive claim) is the most reliable documentation fix: it preserves the D1.6 guarantee for C#/Python repos while avoiding false assertions about other repo types.

Files changed (1) +1 / -1

Documentation (1) +1 / -1

WORKFLOW.mdReword D1.6 to avoid a universal claim about 'typeMechanisms' +1/-1

Reword D1.6 to avoid a universal claim about 'typeMechanisms'

• Updates the D1.6 Codecov/coverage policy text to state that repos outside the guarantee scope may omit 'typeMechanisms' in 'spec/secrets.json', and that such absence is not drift. This resolves an over-broad statement that implied only certain repo types could carry the key.

WORKFLOW.md

[Copilot]

🟢 Approval recommended

The change is a narrow documentation rewording that improves correctness without altering any behavioral or tooling requirements.

Pull request overview

Updates WORKFLOW.md to remove a universal-sounding claim about which repositories carry typeMechanisms in their adapted spec/secrets.json, replacing it with a permissive statement that remains true even when non-C# / non-Python repos (or non-applicable profiles) omit the key.

Changes:

• Reword D1.6 to state that when the Codecov coverage guarantee does not apply, a repo's adapted spec/secrets.json may omit typeMechanisms and this is not drift.

File summaries

File	Description
WORKFLOW.md	Rewords D1.6 to avoid an over-broad claim about typeMechanisms presence in per-repo adapted spec/secrets.json.

Review details

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

---

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