Repository navigation
Conversation
Add the selected review and workflow skills, checksum all maintained assets, and include the shared documentation-audit checker fix. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
Batch execution and cleanup defects, unsafe command expansion, and release guidance conflicts remain unresolved.
13 open issues
What changed in this PR
Adds five assistant skills for repository maintenance and review without changing the production log-copy script.
Changes:
- Adds skill guidance, references, and batch templates.
- Adds a documentation-drift checker and focused regression coverage.
- Updates checksums for maintained skill assets.
| File | Description |
|---|---|
| CHECKSUMS.txt | Records skill asset hashes. |
| .github/skills/pester-should-migration/SKILL.md | Introduces assertion migration guidance. |
| .github/skills/pester-should-migration/references/assertion-map.md | Maps classic assertions to v6. |
| .github/skills/github-release/SKILL.md | Defines release preparation workflow. |
| .github/skills/github-release/references/semver-rules.md | Explains version selection. |
| .github/skills/github-release/references/commit-classification.md | Guides changelog classification. |
| .github/skills/github-actions-hardening/SKILL.md | Introduces workflow security review. |
| .github/skills/github-actions-hardening/references/triggers-and-privilege.md | Explains workflow trust boundaries. |
| .github/skills/github-actions-hardening/references/supply-chain.md | Covers action and artifact integrity. |
| .github/skills/github-actions-hardening/references/report-format.md | Provides security report structure. |
| .github/skills/github-actions-hardening/references/permissions-and-tokens.md | Documents least-privilege permissions. |
| .github/skills/github-actions-hardening/references/injection.md | Explains injection prevention. |
| .github/skills/docs-sync-audit/tests/test_docs_drift.py | Covers audit false-positive scenarios. |
| .github/skills/docs-sync-audit/SKILL.md | Defines documentation audit procedure. |
| .github/skills/docs-sync-audit/scripts/docs_drift.py | Implements documentation checks. |
| .github/skills/batch-files/SKILL.md | Introduces batch scripting guidance. |
| .github/skills/batch-files/references/windows-subsystem-on-linux.md | Collects WSL references. |
| .github/skills/batch-files/references/tools-and-resources.md | Lists Windows development tools. |
| .github/skills/batch-files/references/msys2.md | Collects MSYS2 guidance. |
| .github/skills/batch-files/references/cygwin.md | Documents Cygwin integration. |
| .github/skills/batch-files/references/batch-files-and-functions.md | Catalogs batch scripting techniques. |
| .github/skills/batch-files/assets/task.txt | Adds automation task template. |
| .github/skills/batch-files/assets/library.txt | Adds reusable function template. |
| .github/skills/batch-files/assets/executable.txt | Adds command-line tool template. |
Review effort: Balanced
Comment on lines
+457
to
+458
| if /i "%~1"=="--target" set "_TARGET=%~2" & shift | ||
| if /i "%~1"=="--help" call :usage & exit /b 0 |
Comment on lines
+51
to
+53
| if "%_parOneMyTool%"=="/?" call :_showHelpMyTool & goto _removeBatchVariablesMyTool | ||
| if /i "%_parOneMyTool%"=="-h" call :_showHelpMyTool & goto _removeBatchVariablesMyTool | ||
| if /i "%_parOneMyTool%"=="--help" call :_showHelpMyTool & goto _removeBatchVariablesMyTool |
Comment on lines
+90
to
+110
| echo [VERBOSE] Input: %_parOneMyTool% | ||
| echo [VERBOSE] Output: %_parTwoMyTool% | ||
| ) | ||
|
|
||
| REM Validate input file exists. | ||
| if NOT EXIST "%_parOneMyTool%" ( | ||
| echo ERROR: Input file not found: %_parOneMyTool% 1>&2 | ||
| goto :eof | ||
| ) | ||
|
|
||
| call :_runMyTool | ||
| goto :eof | ||
|
|
||
| :_runMyTool | ||
| REM =================================================================== | ||
| REM TODO: Replace this section with your tool's logic. | ||
| REM =================================================================== | ||
| echo Processing: %_parOneMyTool% | ||
|
|
||
| if NOT "%_checkParTwoMyTool%"=="--" ( | ||
| echo Output to: %_parTwoMyTool% |
Comment on lines
+36
to
+44
| if "%_funcMyLib%"=="/?" call :_showHelpMyLib & goto _removeBatchVariablesMyLib | ||
| if /i "%_funcMyLib%"=="-h" call :_showHelpMyLib & goto _removeBatchVariablesMyLib | ||
| if /i "%_funcMyLib%"=="--help" call :_showHelpMyLib & goto _removeBatchVariablesMyLib | ||
|
|
||
| if /i "%_funcMyLib%"=="trimWhitespace" call :_trimWhitespaceMyLib & goto _removeBatchVariablesMyLib | ||
| if /i "%_funcMyLib%"=="toLower" call :_toLowerMyLib & goto _removeBatchVariablesMyLib | ||
| if /i "%_funcMyLib%"=="getTimestamp" call :_getTimestampMyLib & goto _removeBatchVariablesMyLib | ||
| if /i "%_funcMyLib%"=="logMessage" call :_logMessageMyLib & goto _removeBatchVariablesMyLib | ||
| if /i "%_funcMyLib%"=="padRight" call :_padRightMyLib & goto _removeBatchVariablesMyLib |
| if "!_dtMyLib:~3,1!"==" " set "_dtMyLib=!_dtMyLib:~4!" | ||
| set "_tmMyLib=%TIME: =0%" | ||
| set "_tsMyLib=!_dtMyLib:~6,4!-!_dtMyLib:~0,2!-!_dtMyLib:~3,2! !_tmMyLib:~0,2!:!_tmMyLib:~3,2!:!_tmMyLib:~6,2!" | ||
| echo [!_tsMyLib!] [%_argOneMyLib%] %_argTwoMyLib% |
| set _checkParTwoMyTask= | ||
| REM Append new variables above this line. | ||
|
|
||
| if EXIST "%_tmpDirMyTask%" rmdir /S /Q "%_tmpDirMyTask%" >nul 2>nul |
| f"documents `make {target}`, which is not a target in the Makefile.", | ||
| source="Makefile") | ||
| for whole, inner in CMD_SCRIPT.findall(line): | ||
| candidate = (inner or whole).lstrip("./") |
Comment on lines
+73
to
+79
| permissions: | ||
| pull-requests: write | ||
| jobs: | ||
| comment: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/download-artifact@<sha> # data only, not executed |
Comment on lines
+312
to
+314
| git add CHANGELOG.md | ||
| git commit -m "chore: release vX.Y.Z" | ||
| git push origin release/vX.Y.Z |
Comment on lines
+402
to
+403
| > Then go to GitHub Releases and publish the release from that tag. You can copy the | ||
| > changelog section directly into the release notes. |
This branch has not been deployed
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
github-actions-hardening,pester-should-migration,docs-sync-audit,github-release, andbatch-files.CHECKSUMS.txtfor all maintained skill assets.Validation
git diff --checkpassed.