docs: correct the AI Copilot and settings tab references - #429
Merged
Conversation
The AI Copilot section of AGENTS.md still described includes/ai-copilot/tools-registry.php and desktop_mode_register_ai_tool(), both removed in 0.9.4 when the Copilot's tools moved to the WordPress Abilities API. Rewrite it against the current surface and drop the dead desktop_mode_ai_tool_registered action from the observability list. The built-in settings tab orders were also stale in four places: ai=20 no longer exists and themes=12 was never added. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed changes
AGENTS.mdagainst the Abilities API. It still documentedincludes/ai-copilot/tools-registry.phpanddesktop_mode_register_ai_tool(), both removed in 0.9.4.desktop_mode_ai_tool_registeredfrom the listed observability actions. The other nine filters and four actions are live and unchanged.AGENTS.md,docs/javascript-reference.md,includes/settings-tabs.php, andsrc/desktop.ts.ai=20no longer exists andthemes=12was missing.Why are these changes being made?
AGENTS.mdis what agents and contributors read before touching a surface, anddocs/is the public contract for plugin authors. Both pointed at a registration API that was deleted, so anyone following them would register a tool the Copilot never sees.docs/migration-ai-connectors.mdalready documented the move correctly, the other references never caught up.The tab list had the same problem in the public
registerSettingsTabcontract and in two docblocks, so a plugin author picking anordervalue was working from a list that no longer matches the shell.Testing instructions
Docs and comments only, no runtime change. To verify the claims:
grep -rn "desktop_mode_register_ai_tool\|tools-registry" includes/ src/returns nothing.includes/ai-copilot/abilities.php: abilities register onwp_abilities_api_init(line 371), thedesktop-modecategory onwp_abilities_api_categories_init(line 46), anddesktop_mode_ai_search_ability_names()selects onmeta.annotations.readonly.src/settings/panel.ts: appearance=10, themes=12, apps-icons=22, features=25, effects=27, help=40 (admin-only), About pinned withNumber.MAX_SAFE_INTEGER.npm run build,npm run typecheck, andnpm run lint. All pass, andassets/js/shows no diff.