Skip to content

Set up comprehensive Copilot coding agent instructions#1650

Merged
TwitchBronBron merged 2 commits intomasterfrom
copilot/setup-copilot-instructions
Mar 19, 2026
Merged

Set up comprehensive Copilot coding agent instructions#1650
TwitchBronBron merged 2 commits intomasterfrom
copilot/setup-copilot-instructions

Conversation

Copy link
Contributor

Copilot AI commented Mar 19, 2026

Expands .github/copilot-instructions.md from a minimal stub into actionable instructions that reflect the actual conventions in this codebase.

Changes

  • Testing: Explicitly documents npm run test:nocover as the preferred test command and warns against invoking mocha directly with custom arguments (.mocharc.cjs must be respected)
  • Code style: Documents enforced ESLint rules that differ from common defaults — no object shorthand (object-shorthand: never), single quotes, curly braces required, one var per declaration, for...of preferred, no floating promises; notes that strictNullChecks and prefer-const are off
  • Imports: import type for type-only imports; external packages before internal modules
  • Testing conventions: expect imported from chai-config.spec (not chai directly), sinon sandbox via testHelpers.spec, undent/trim for multiline BrightScript strings, helper functions (expectZeroDiagnostics, etc.)
  • Architecture: Expanded key directories (src/bscPlugin/, src/types/, src/validators/) and common patterns (AstEditor for reversible AST edits, DiagnosticMessages for structured error codes, parser error recovery)
Original prompt

This section details on the original issue you should resolve

<issue_title>✨ Set up Copilot instructions</issue_title>
<issue_description>Configure instructions for this repository as documented in Best practices for Copilot coding agent in your repository.

</issue_description>

<agent_instructions>look through all of my code to decide my preferred styles. running tests is done with npm run test:nocover. (do not try doing things like mocha --files ....blabla, that won't work).

</agent_instructions>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…esting guidelines

Co-authored-by: TwitchBronBron <2544493+TwitchBronBron@users.noreply.github.com>
Copilot AI changed the title [WIP] Configure Copilot instructions for repository Set up comprehensive Copilot coding agent instructions Mar 19, 2026
Copilot AI requested a review from TwitchBronBron March 19, 2026 19:19
@TwitchBronBron TwitchBronBron marked this pull request as ready for review March 19, 2026 19:39
@TwitchBronBron TwitchBronBron enabled auto-merge (squash) March 19, 2026 19:39
@TwitchBronBron TwitchBronBron merged commit 3de4f0c into master Mar 19, 2026
7 checks passed
@TwitchBronBron TwitchBronBron deleted the copilot/setup-copilot-instructions branch March 19, 2026 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

✨ Set up Copilot instructions

2 participants