Skip to content

Add repository-wide CODEOWNERS for agent-skills maintainers - #108

Merged
bgrgicak merged 2 commits into
trunkfrom
copilot/add-codeowners-file
Sep 25, 2026
Merged

bgrgicak merged 2 commits into
trunkfrom
copilot/add-codeowners-file

Conversation

Copilot AI commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

This adds a single repository-wide CODEOWNERS rule so changes anywhere in the repo request review from @WordPress/agent-skills-maintainers. It mirrors the existing CODEOWNERS pattern used for narrower ownership, but applies it uniformly across the full tree.

  • Scope

    • Add .github/CODEOWNERS for repo-level ownership.
  • Ownership rule

    • Assign @WordPress/agent-skills-maintainers as the default code owner for all paths.
  • Effect

    • Centralize review routing for all repository changes under the maintainer team.
* @WordPress/agent-skills-maintainers

Co-authored-by: jeffpaul <2818133+jeffpaul@users.noreply.github.com>
Copilot AI changed the title Add CODEOWNERS and designate @WordPress/agent-skills-maintainers as the code owners of the repository Add repository-wide CODEOWNERS for agent-skills maintainers Sep 22, 2026
Copilot AI requested a review from jeffpaul September 22, 2026 13:19
@jeffpaul
jeffpaul requested a review from a team September 22, 2026 13:20
@bgrgicak

Copy link
Copy Markdown
Contributor

Thanks @jeffpaul! This should help with us reviewing PRs more

@jeffpaul
jeffpaul marked this pull request as ready for review September 24, 2026 13:41
@bgrgicak
bgrgicak merged commit f1bac1f into trunk Sep 25, 2026
3 checks passed
@bgrgicak
bgrgicak deleted the copilot/add-codeowners-file branch September 25, 2026 04:30
henryperkins added a commit to henryperkins/agent-skills that referenced this pull request Sep 27, 2026
… security

Merges the six commits WordPress/agent-skills trunk was ahead by:

- ac16175 Add wp-env skill for local WordPress development (WordPress#38)
- ad58e37 Add plugin security guidance for escaping and AJAX (WordPress#63)
- f2f6af2 Fix Plugin/Theme header detection in PHP docblocks (WordPress#103)
- e9c58d5 Adding reference to webMCP at Playground Skill (WordPress#105)
- 9f4a603 chore: refresh upstream indices (WordPress#92)
- f1bac1f Add repository-wide CODEOWNERS for agent-skills maintainers (WordPress#108)

Conflict resolutions:

shared/references/wordpress-core-versions.json
  Kept ours. api.wordpress.org/core/version-check/1.7 reports 7.1.2
  (checked 2026-09-27); upstream's WordPress#92 snapshot stops at 7.1.1. The
  Gutenberg index merged cleanly: both sides already carry v24.0.0, and
  v24.1.0 is still at rc.1.

skills/wp-project-triage/scripts/detect_wp_project.mjs
  Kept ours. The fork already fixed WordPress#103's bug with `^[\s*#]*`, which
  matches every header line upstream's `^\s*\*?\s*` does, plus `#`-led
  lines. The detect_plugins.mjs half of WordPress#103 merged cleanly; the fork had
  not fixed that file. Both detectors now report docblock, bare, and
  non-plugin fixtures correctly.

skills/wp-playground/SKILL.md
  Kept the fork's frontmatter (Use-when opener, 6.9 floor, GPL license)
  and folded WordPress#105's "WebMCP site tools" into the description. Took WordPress#105's
  browser-workflow routing bullet in the fork's markdown-link style; it
  also drops the "Blueprint Editor" claim, which website.md never covered.

Clean merges: website.md now routes existing-site work to the new
webmcp.md, mcp.md, and sites-api.md references (the WebMCP tool names
match packages/playground/mcp in WordPress/wordpress-playground);
security.md gains escaping contexts and AJAX handler rules. wp-env
arrives exactly as upstream wrote it and is conformed and corrected in
the next commit. CODEOWNERS names @WordPress/agent-skills-maintainers,
which does not resolve in this fork; kept for tree parity with upstream.

The harness is not expected to pass at this commit: wp-env is not yet in
the skill inventory and skills/ changed without a version bump. The
follow-up commits close both.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
henryperkins added a commit to henryperkins/agent-skills that referenced this pull request Sep 27, 2026
This fork is maintained on its own and does not send changes upstream, so
two pieces of WordPress organization automation have no effect here:

- .github/CODEOWNERS, added by upstream WordPress#108, assigns every path to
  @WordPress/agent-skills-maintainers, a team that does not resolve in
  henryperkins/agent-skills.
- .github/workflows/props-bot.yml runs WordPress's contributor-credit bot
  on pull_request_target and issue_comment events.

Nothing else in the tree referenced either file. If a later upstream merge
edits one of them, resolve the modify/delete conflict by keeping the
deletion. The 1.12.0 release notes, not yet published, now describe the
removal.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
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.

3 participants