Refresh 6 stale instruction files (#2133)#2139
Conversation
🔒 PR Risk Scan ResultsScanned 7 changed file(s).
✅ No matching risk patterns were detected in changed files.
|
Targeted refresh: - blazor: C# 13 to C# 14. Drop the Visual Studio Enterprise mandate so contributors on VS Code or Rider aren't blocked by a paid SKU. Swap VS-only profiling for dotnet-trace and dotnet-counters. - copilot-thought-logging: narrow applyTo from '**' to '**/Copilot-Processing.md'. Replace 9 Windows backslash paths with POSIX './Copilot-Processing.md' so the workflow works on macOS and Linux. - genaiscript: drop the "avoid exception handlers or error checking" line. Replace it with: handle errors at I/O and external API boundaries, let unexpected exceptions surface. - memory-bank: add the required 'description' frontmatter field (was a validation failure). Narrow applyTo from '**' to 'memory-bank/**'. Add an opt-in note so contributors know auxiliary files land in the workspace root. Minor modernization: - azure-functions-typescript: Node.js v20 to v22 LTS. - localization: relative '../../issues' disclaimer link to absolute https://github.com/github/awesome-copilot/issues so it resolves regardless of the localized doc's path. docs/README.instructions.md regenerated by 'npm run build' to pick up the new memory-bank description.
a0bf675 to
45edc81
Compare
| @@ -1,5 +1,5 @@ | |||
| --- | |||
| applyTo: '**' | |||
| applyTo: '**/Copilot-Processing.md' | |||
There was a problem hiding this comment.
This is incorrect as it will make the instruction only apply when the agent is modifying the Copilot-Processing.md file, which is actually what the output of the instruction is trying to do.
aaronpowell flagged that narrowing applyTo from '**' to '**/Copilot-Processing.md' inverts the instruction. The instruction tells Copilot to CREATE Copilot-Processing.md when handling any user request, so it must apply globally, not only when that file is already open. Restore applyTo to '**'. Keep the POSIX path fixes (backslash to './Copilot-Processing.md') and the other 5 file fixes in this PR unchanged.
|
🟡 Contributor Reputation Check: MEDIUM risk
Maintainers: please review this contributor before merging. |
|
Thanks @aaronpowell. Reverted applyTo back to '**' in a8a2b45 since the instruction's job is to create Copilot-Processing.md from any user request, not to fire only when that file is already open. The 5 other file edits in the PR (blazor C# 14 + cross-IDE, copilot-thought-logging POSIX paths, genaiscript error-handling, memory-bank description field, azure-functions-typescript Node 22, localization absolute issues link) are unchanged. Ready for another look. |
Picks up the 6 instruction files the Resource Staleness Report deep-reviewed and flagged with concrete problems. 4 "Targeted refresh" + 2 "Minor modernization". The remaining 716 stale files are bulk batch-commit noise per the report itself, so they stay out.
Changes
Targeted refresh
instructions/blazor.instructions.mddotnet-trace/dotnet-counters, which work cross-platform.instructions/copilot-thought-logging.instructions.mdapplyTofrom**(every file in the repo) to**/Copilot-Processing.md.\Copilot-Processing.mdpaths with POSIX./Copilot-Processing.mdso the workflow runs on macOS and Linux.instructions/genaiscript.instructions.mdinstructions/memory-bank.instructions.mddescriptionfrontmatter (it was missing, which is a validation failure perAGENTS.md).applyTofrom**tomemory-bank/**so it only fires in repos that actually use the pattern.Minor modernization
instructions/azure-functions-typescript.instructions.mdinstructions/localization.instructions.md../../issueslink in the disclaimer to absolutehttps://github.com/github/awesome-copilot/issuesso it resolves regardless of where the localized doc sits.Auto-generated
docs/README.instructions.mdgot regenerated bynpm run buildto pick up the newmemory-bankdescription. CI requires this stay in sync.Verification
npm install && npm run buildexits 0. Generatedmarketplace.jsonwith 96 plugins.grep -n '\\Copilot-Processing' instructions/copilot-thought-logging.instructions.mdreturns 0 matches.Out of scope
bicep-code-best-practices,cmake-vcpkg,nextjs-tailwind,generate-modern-terraform-code-for-azure. The report explicitly says "no urgent change needed" on these.Closes #2133 partial.