Repository navigation
chore(dev): manage agent skills through Composer - #119
Conversation
Signed-off-by: Felipe Sayão Lobato Abreu <github@mentordosnerds.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Code Coverage Report: Only Changed Files listed
Minimum allowed coverage is Head: a907ec9. Tested commit: c0bc15f. PHP CI run. |
Changelog✅ Contribution check passed. Head: Base: Contribution: 1 new valid fragment(s). Policy: ordinary.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 69aadab1a5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Signed-off-by: Felipe Sayão Lobato Abreu <github@mentordosnerds.com>
Signed-off-by: Felipe Sayão Lobato Abreu <github@mentordosnerds.com>
Replace the committed Changelog skill with Composer-managed development instructions. Add
llm/skills:^1.15torequire-dev, explicitly allow its plugin and registerphp-fast-forward/changelogthroughcomposer skills:add. The generatedskills.jsonselects the upstream^0.3.0source; normal developmentcomposer installrecreates the skill under ignored.agents/skills.Remove the old payload and receipt, exclude
skills.jsonfrom Composer archives, and retire the CI/fixer exceptions needed only by the versioned skill. Update contribution/release guidance and add one patch fragment. Runtime dependencies, PHP compatibility, the bundledcoisa-error-handlerconsumer skill and release-workflow configuration stay unchanged.Explicitly trust the installed Composer packages
fast-forward/containerandfast-forward/configas skill donors, including transitive Config. The instructions therefore match the resolved package versions; no duplicate GitHub source or wildcard trust is added. A normalcomposer installwas verified to synchronize all three skills.Validation: Composer manifest validation; full Composer check (380 tests / 1776 assertions, formatter, PHPCS and maximum configured PHPStan); an actual clean-clone
composer installwith automatic skill synchronization and clean Git status; a verified 85-file Composer archive excluding agent instructions and development configuration. No Composer lockfile or generated skill is committed.Repeated
composer installpassed the post-install hook; all three installed skills were verified, with Container/Config bytes matching their resolved Composer donors. The guide also corrects the reviewed synchronization controls:--no-scriptsalone does not disable plugin listeners; use--no-pluginsorauto-sync: falsefor that purpose. PHP CI, documentation and Changelog validation passed fora907ec9947c17f7a187cb5e1223de0361fcf84fd, with 22 successful checks/statuses and no failures.