Skip to content

docs: trim Copilot instructions to current, non-derivable guidance - #3680

Merged
afonsojramos merged 1 commit into
mainfrom
docs/trim-copilot-instructions
Nov 11, 2026
Merged

afonsojramos merged 1 commit into
mainfrom
docs/trim-copilot-instructions

Conversation

@afonsojramos

Copy link
Copy Markdown
Member

Reduces .github/copilot-instructions.md to guidance an agent cannot read from the repository: setup, the main commands, the checks to run before finishing, two gotchas, and the project philosophy. It points to ARCHITECTURE.md and CONTRIBUTING.md for the rest.

Removed

  • Stale or wrong claims:
    • Node >=24; the repository requires Node 26.
    • A separate tsc --noEmit step; vp run check type-checks.
    • Pre-existing lint warnings; vp run check reports none.
    • Timezone snapshot drift; vitest.config.mts forces TZ=UTC.
    • The codegen output path graphql/generated/graphql.ts; types are generated per forge.
  • The file map, which ARCHITECTURE.md covers.
  • The technology version list, which had drifted from package.json.
  • Timing estimates and the local packaging commands.

Added

  • GitLab in the list of supported forges.
  • vp run i18n:check for changes to user-visible strings.

This change was pushed to #3679 after that PR merged, so it never reached main.

@afonsojramos
afonsojramos requested a review from setchy as a code owner November 11, 2026 15:15
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Nov 11, 2026
@sonarqubecloud

Copy link
Copy Markdown

@afonsojramos
afonsojramos merged commit f54c985 into main Nov 11, 2026
19 checks passed
@afonsojramos
afonsojramos deleted the docs/trim-copilot-instructions branch November 11, 2026 15:27
@github-actions github-actions Bot added this to the Release 8.2.0 milestone Nov 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Development

Successfully merging this pull request may close these issues.

1 participant