docs: update docs to reflect updated changes#2167
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2167 +/- ##
==========================================
+ Coverage 81.24% 81.32% +0.07%
==========================================
Files 127 117 -10
Lines 18831 18755 -76
Branches 12 0 -12
==========================================
- Hits 15300 15252 -48
+ Misses 3528 3503 -25
+ Partials 3 0 -3
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Eval Execution✅ Status: Passed
No changed AI artifacts required evaluation. |
katriendg
left a comment
There was a problem hiding this comment.
Thank you, valuable updates!
Left one inline comment and adding a note here:
The Related Issues line:
Fixes #2152 #2153 #2156 #2162 #2150 #2151 #2143 #2142 #2163 #2144
GitHub applies a closing keyword only to the issue number immediately after it. With a single Fixes followed by a space-separated list, only #2152 will auto-close on merge — the other nine become plain references and stay open. Either repeat the keyword per issue (Fixes #2152, Fixes #2153, Fixes #2156, …) or plan to close the remaining nine manually after merge.
| | Excluded patterns | `*.Tests.ps1` | | ||
|
|
||
| Coverage directories include `linting/`, `security/`, `lib/`, `extension/`, `plugins/`, `collections/`, and `tests/`. | ||
| Coverage directories include `linting/`, `security/`, `lib/`, `extension/`, `collections/`, `evals/`, and `tests/`. |
There was a problem hiding this comment.
The updated sentence reads:
Coverage directories include
linting/,security/,lib/,extension/,collections/,evals/, andtests/.
But the source of truth, scripts/tests/pester.config.ps1 (line 85), is:
$coverageDirs = @('linting', 'security', 'lib', 'extension', 'collections', 'tests')Removing the stale plugins/ entry is correct, but evals/ was added and is not in $coverageDirs — so scripts/evals/*.ps1 are not actually in the coverage scope. Suggested correction:
Coverage directories include
linting/,security/,lib/,extension/,collections/, andtests/, plus skill scripts under.github/skills/(resolved separately further down in the config).
Note: the evals/ additions to the directory-tree diagrams are fine — scripts/evals/ and scripts/tests/evals/ both exist. Only the coverage-directories sentence is wrong.
Pull Request
Description
Updates documentation of:
github/workflows/*.yml/.yamland.github/actions/**/*.yml/.yaml(composite actions), matching the scanner change introduced in fix(ci): add composite action for PS module install with cache and retry #2134.Test-DependencyPinning.ps1validates already-==-pinned packages for staleness rather than flagging range specifiers.pyproject.tomlare acceptable for uv-managed skills because uv.lock records exact resolved versions and integrity hashes, achieving supply-chain security equivalent to == pinning.copilot-instructions.md/rpi-*skill command surface alongside the existing/task-*prompt commands — a new Skill Command column in the Quick Reference table plus a note covering/rpi-quickfor the full flow, consistent with the dual-mode approach in the sibling RPI docs. Refreshedms.date.plugins/from the coverage directories list, matching the directories actually analyzed by code coverage.enforces the 80% target through its **Coverage Threshold Check** step, which sources both the measured percentage (CoveragePercent) and the target (CoverageTarget) fromlogs/pester-summary.json. Clarified that the summary is authored byInvoke-PesterTests.ps1, the target originates inpester.config.ps1` (single source of truth), and the job fails below target unless soft-fail mode is enabled.CoveragePercentandCoverageTargetfields written tologs/pester-summary.jsonwhen code coverage is enabled.ms.dateto2026-06-25.6.scripts/tests/README.md
pester-summary.jsonfeature bullet now notes that CoveragePercent (measured percentage, rounded to two decimals) and CoverageTarget (configured threshold frompester.config.ps1) are included when code coverage is enabled.-Tag/-IncludeTagand-ExcludeTagwith descriptions matching the script's comment-based help (including the default @('Integration','Slow') and the replace-not-append behavior).documentation.agent.md,replacing the stale unlinked Doc Ops label. The table was reformatted to keep column alignment.Related Issue(s)
Fixes #2152 #2153 #2156 #2162 #2150 #2151 #2143 #2142 #2163 #2144
Type of Change
Select all that apply:
Code & Documentation:
Infrastructure & Configuration:
AI Artifacts:
prompt-builderagent and addressed all feedback.github/instructions/*.instructions.md).github/prompts/*.prompt.md).github/agents/*.agent.md).github/skills/*/SKILL.md)evals/)Other:
.ps1,.sh,.py)Sample Prompts (for AI Artifact Contributions)
User Request:
Execution Flow:
Output Artifacts:
Success Indicators:
For detailed contribution requirements, see:
Testing
Checklist
Required Checks
AI Artifact Contributions
/prompt-analyzeto review contributionprompt-builderreviewRequired Automated Checks
The following validation commands must pass before merging:
npm run lint:mdnpm run spell-checknpm run lint:frontmatternpm run validate:skillsnpm run lint:md-linksnpm run lint:psnpm run eval:lint:schemanpm run plugin:generatenpm run docs:testSecurity Considerations
Additional Notes