Skip to content

Copilot instructions update#1598

Open
BurningAXE wants to merge 1 commit intomainfrom
copilot-instructions-update
Open

Copilot instructions update#1598
BurningAXE wants to merge 1 commit intomainfrom
copilot-instructions-update

Conversation

@BurningAXE
Copy link
Contributor

Notable changes

  • Used Opus 4.6 to analyze the project and generate instructions for itself
  • I only did a few minor corrections and additions
  • Then prompted it to add info about our event system, too

Additional work checklist

  • Effect on other features and security has been considered
  • Design document marked as "In development" (if applicable)
  • External (Gitbook) and internal (Confluence) Documentation is up to date (or ticket created)
  • Test cases in Testiny are up to date (or ticket created)
  • Other teams notified about the changes (if applicable)

@cla-bot cla-bot bot added the ... label Feb 20, 2026
@BurningAXE BurningAXE marked this pull request as ready for review February 20, 2026 12:04
@BurningAXE BurningAXE requested review from a team, TristramN, alex-vt, alexandr-simprints, luhmirin-s, meladRaouf and ybourgery and removed request for a team February 20, 2026 12:04
Copy link
Contributor

@luhmirin-s luhmirin-s left a comment

Choose a reason for hiding this comment

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

IIRC, the general instructions file is prepended to each prompt that we do in the repository, therefore it should be as short and generic as possible. IMO, there is a significant amount of redundant information and implementation details that could be cut-out from this version.

Also I am not convinced that every prompt needs a detailed explanation of the event system. Can it be in a separate prompt file and linked only when actually necessary?

@BurningAXE BurningAXE force-pushed the copilot-instructions-update branch from 6a9b32d to 9d2cb8c Compare February 23, 2026 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants