这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@mindctrl
Copy link
Contributor

Adds the wp_ prefix to code and docs.

Fixes #80.

Copilot AI review requested due to automatic review settings October 23, 2025 19:32
@github-actions
Copy link

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: mindctrl <mindctrl@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

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 adds the wp_ prefix to hook names throughout the codebase to align with WordPress naming conventions. The change ensures consistency by prefixing all abilities API hooks with wp_.

Key changes:

  • Updated all hook references from abilities_api_init to wp_abilities_api_init
  • Updated all hook references from abilities_api_categories_init to wp_abilities_api_categories_init
  • Applied changes consistently across production code, tests, and documentation

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/Unit/Core/McpAdapterConfigTest.php Updated hook check to use prefixed hook name
tests/TestCase.php Updated all hook registrations and checks to use prefixed names
tests/Fixtures/DummyAbility.php Updated hook registrations, checks, and documentation to use prefixed names
includes/Core/McpAdapter.php Updated hook registrations to use prefixed names
docs/guides/default-server.md Updated troubleshooting documentation to reference prefixed hook names
docs/getting-started/installation.md Updated example code to use prefixed hook name
docs/getting-started/basic-examples.md Updated all example code to use prefixed hook names
docs/getting-started/README.md Updated quick start example and troubleshooting to use prefixed hook names
README.md Updated example code to use prefixed hook name

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Collaborator

@jonathanbossenger jonathanbossenger left a comment

Choose a reason for hiding this comment

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

LGTM

@jonathanbossenger jonathanbossenger merged commit f2f2718 into WordPress:trunk Oct 29, 2025
10 of 17 checks passed
@mindctrl mindctrl deleted the fix/issue-80 branch October 29, 2025 13:49
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.

Update plugin to use wp_ prefixed hook names

2 participants