Skip to content

Conversation

@marlonicus
Copy link
Contributor

Summary
This PR introduces Copilot custom instructions and fills in missing public documentation for key components.


🧩 Copilot Custom Instructions

  • Added comprehensive Copilot instructions to improve AI-assisted workflows — such as documentation writing, API consumption, testing, and onboarding.

  • Instructions are split into two files:

    • copilot-instructions.md – full internal instructions used by VS Code.
    • copilot-instructions-public-api.md – a slimmed-down version focused on the public API (for use in Copilot Spaces).
  • Added validation scripts to ensure instructions stay up to date:

    • /scripts/check-copilot-instructions-sync.sh
    • /scripts/check-public-api-instructions-sync.sh
  • Introduced two Copilot prompts for maintenance:

    • /fix-instructions – runs desync checks, updates instructions from existing docs, and flags missing documentation.
    • /create-documentation – generates new public documentation from referenced source files in collaboration with the user.

📘 Public Documentation Additions

  • Added a complete reference for the Unwrap component — usage, performance, best practices, and comparison with Mount.
  • Added a complete reference for the Times component — usage guidance and performance notes.

- Create comprehensive .github/copilot-instructions.md covering:
  - Project overview and React Facet core concepts
  - Repository structure and import patterns
  - All hooks, components, and APIs with usage examples
  - Common patterns, anti-patterns, and best practices
  - Testing patterns and quick reference guide
  - Maintenance guidelines to prevent documentation drift

- Add automated validation script (scripts/check-copilot-instructions-sync.sh):
  - Validates hooks mentioned in docs exist in code
  - Checks all code hooks are documented
  - Verifies components, packages, and import paths
  - Comprehensive inline documentation

- Add CI workflow (.github/workflows/validate-copilot-instructions.yml):
  - Runs on all PRs and pushes to main
  - Prevents documentation drift before merge
  - Ensures instructions stay current with codebase

This enables GitHub Copilot to provide context-aware code suggestions
specific to React Facet patterns and conventions.

Add initial copilot-instructions.md doc

Instruction updates.

Add info on NO_VALUE behaviour in memo/map/setters

Update instructions.
Copy link
Contributor

@Bahamoe Bahamoe left a comment

Choose a reason for hiding this comment

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

Nice work with this 🚀.

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.

2 participants