Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 6, 2025

Updates .github/copilot-instructions.md to reflect the current repository state, aligning with AGENTS.md and CLAUDE.md.

Changes

  • Project metadata: Component count corrected to 49, added Pest as testing framework
  • Repository structure: Added types/, AGENTS.md, CLAUDE.md references
  • Key components: Reorganized into categorized sections (Development Tools, Database & Models, Caching & Storage, HTTP & API, Notifications & Mail, Search, Security, Infrastructure)
  • Scripts & workflows: Added all bin/ scripts and missing workflows (cnb.yaml, update-changelog.yml)
  • Testing: Updated to Pest-specific patterns with --group examples
  • Contributing: Added Conventional Commits format
  • Coroutine safety: New section with Hyperf\Context\Context API usage
  • ConfigProvider template: Enhanced with aspects and publish keys

Example test commands added

# Run tests for a specific component
vendor/bin/pest --group=telescope

# Run a specific test file
vendor/bin/pest tests/HttpClient/PendingRequestTest.php
Original prompt

根据项目最新代码更新.github/copilot-instructions.md


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI self-assigned this Dec 6, 2025
@coderabbitai
Copy link

coderabbitai bot commented Dec 6, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Co-authored-by: huangdijia <8337659+huangdijia@users.noreply.github.com>
Copilot AI changed the title [WIP] Update Copilot instructions in project documentation docs: update copilot-instructions.md with latest project information Dec 6, 2025
Copilot AI requested a review from huangdijia December 6, 2025 02:30
@huangdijia huangdijia marked this pull request as ready for review December 6, 2025 02:39
Copilot AI review requested due to automatic review settings December 6, 2025 02:39
@huangdijia huangdijia merged commit 001a2e8 into main Dec 6, 2025
7 checks passed
@huangdijia huangdijia deleted the copilot/update-copilot-instructions branch December 6, 2025 02:39
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the GitHub Copilot instructions file (.github/copilot-instructions.md) to accurately reflect the current state of the FriendsOfHyperf components monorepo. The changes align the documentation with information from AGENTS.md and CLAUDE.md, providing more comprehensive guidance for AI-assisted development.

Key Changes:

  • Corrected component count from "50+" to 49, matching the actual component directories
  • Added Pest as the official testing framework with group-based test organization
  • Reorganized component listing into logical categories (Development Tools, Database & Models, Caching & Storage, HTTP & API, Notifications & Mail, Search, Security, Infrastructure)
  • Enhanced repository structure documentation to include types/, AGENTS.md, and CLAUDE.md
  • Added comprehensive testing conventions with Pest-specific examples
  • Introduced coroutine safety guidelines with Context API usage patterns
  • Expanded build command documentation with all composer scripts
  • Added Conventional Commits format requirements


- **Repository**: friendsofhyperf/components
- **Type**: Monorepo containing 50+ Hyperf components
- **Type**: Monorepo containing 49 Hyperf components
Copy link

Copilot AI Dec 6, 2025

Choose a reason for hiding this comment

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

The component count has been updated to 49 in this file, but CLAUDE.md still references "50+ components" on line 7. For consistency across documentation files, consider updating CLAUDE.md to match this correction, or verify the actual component count.

Copilot uses AI. Check for mistakes.
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