Repository navigation
Correct instruction scopes, API guidance, and safety boundaries 🤖🤖🤖 - #6332
aaronpowell merged 2 commits into
Conversation
🔒 PR Risk Scan ResultsScanned 10 changed file(s).
|
|
🟡 Contributor Reputation Check: MEDIUM risk
Maintainers: please review this contributor before merging. |
🚦 Submission status: 👀 Ready for reviewRisk tier: Why this tier
Automated checks
Review
Commands
Updated for |
There was a problem hiding this comment.
🟡 Changes recommended
The test-location rule still mandates tests/ despite supporting other suite directories, potentially placing specs outside Playwright’s configured discovery paths.
1 open issue
What changed in this PR
Refreshes ten GitHub Copilot instruction resources identified in #6277 to improve version compatibility, technical accuracy, and safety.
Changes:
- Favor installed versions and existing project configuration over automatic upgrades.
- Correct TanStack Router options and refine automatic instruction scopes.
- Update infrastructure guidance, implementation boundaries, and unsupported example comments.
| File | Description |
|---|---|
instructions/tanstack-start-shadcn-tailwind.instructions.md |
Correct route UI options and clarify version-aware configuration. |
instructions/taming-copilot.instructions.md |
Clarify instruction priority, implementation authorization, and validation boundaries. |
instructions/self-explanatory-code-commenting.instructions.md |
Replace unsupported service claims in example comments. |
instructions/ruby-on-rails.instructions.md |
Add Rails version context and preserve existing application choices. |
instructions/power-platform-connector.instructions.md |
Limit automatic scope to standard connector definition files. |
instructions/playwright-typescript.instructions.md |
Narrow automatic scope and reference configured test discovery. |
instructions/nextjs-tailwind.instructions.md |
Add version-specific Next.js and Tailwind guidance. |
instructions/generate-modern-terraform-code-for-azure.instructions.md |
Respect version constraints and clarify subscription selection and safe validation. |
instructions/cmake-vcpkg.instructions.md |
Focus scope on build configuration and preserve project conventions. |
instructions/bicep-code-best-practices.instructions.md |
Cover parameter files, versioned modules, and project-specific lint checks. |
Review effort: Max (auto)
aaronpowell
left a comment
There was a problem hiding this comment.
Thanks for refreshing these instructions. The changes add concrete version-aware guidance—for example, the Next.js 15+ request API behavior and the AzureRM 4.x subscription requirement—while narrowing automatic scopes and preserving project-specific configurations. The PR reports successful scope checks and build validation, and the generated README check passes.

Description
Refs #6277.
Update the ten concrete priorities identified by the staleness report:
errorComponent/pendingComponentexamples, and narrow Playwright, CMake and Power Platform automatic scopes. Playwright location, naming and execution guidance follows the configured suite instead of imposing atests/directory or project name.Existing-version compatibility and project-specific configuration remain part of the guidance. The linked primary documentation is recorded in the affected files.
Validation
npm run build; the prepared integration website build also completed.git diff --checkpassed.Framework applications, cloud deployments and connectors were not executed. This addresses the report's ten explicit priorities; its wider age-based review queue remains ongoing maintenance.
AI-assisted implementation and validation.