FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update copilot-instructions.md with architecture, conventions, and release process by andyleejordan · Pull Request #5468 · PowerShell/vscode-powershell · GitHub

Update copilot-instructions.md with architecture, conventions, and release process - #5468

Merged
Andy Jordan (andyleejordan) merged 1 commit into
mainfrom
update-copilot-instructions
Apr 15, 2026
Merged

Andy Jordan (andyleejordan) merged 1 commit into
mainfrom
update-copilot-instructions

Conversation

Copy link
Copy Markdown
Member

Summary

Expands .github/copilot-instructions.md from just the NPM update section to comprehensive guidance for AI agents working in this repo.

What's added

  • Overview: LSP client/server relationship with PSES
  • Build, Lint, and Test: All commands with install step; note to always lint/format after changes
  • Architecture: Extension activation flow, SessionManager lifecycle, LanguageClientConsumer pattern, feature modules, external API, bundled modules with cross-repo PSES notes
  • Key Conventions: VS Code extension best practices, ILogger usage, settings access pattern, TypeScript style, import style, file headers
  • Versioning and Release Process: Even/odd minor scheme, updateVersion.ps1 workflow, release steps
  • Cross-repo work: Multi-root workspace reference for PSES collaboration

The existing NPM packages section is preserved and extended with the @types/vscode/@types/node version tracking note.

Copilot AI review requested due to automatic review settings April 10, 2026 20:21
Andy Jordan (andyleejordan) requested a review from a team as a code owner April 10, 2026 20:21

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Pull request overview

Expands .github/copilot-instructions.md into a repo-specific playbook for AI agents/contributors, covering extension architecture, build/lint/test workflow, key conventions, and the even/odd versioning + release process used by vscode-powershell.

Changes:

  • Add an overview of the extension’s LSP client role and the high-level activation/session architecture.
  • Document the local dev workflow (install/compile/lint/format/test) and key TypeScript/logging/settings conventions.
  • Add detailed versioning rules and release steps, including cross-repo (PSES) coordination notes.

Comment thread .github/copilot-instructions.md Outdated
Comment thread .github/copilot-instructions.md Outdated
Comment thread .github/copilot-instructions.md Outdated
- Rewrite .github/copilot-instructions.md with build commands, architecture
  overview, PSES/cross-repo context, and key conventions
- Add .github/skills/release/SKILL.md for on-demand release guidance
- Add .github/skills/update-npm-packages/SKILL.md for on-demand NPM update guidance
- Simplify docs/development.md: tighten prose, fix ADO push target (HEAD:upstream),
  add version prefix rules (v in tags/changelog, not in package.json)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Andy Jordan (andyleejordan) merged commit ecb8145 into main Apr 15, 2026
8 checks passed
Andy Jordan (andyleejordan) deleted the update-copilot-instructions branch April 15, 2026 23:26
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

Labels

Area-Documentation Issue-Enhancement A feature request (enhancement).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL