Repository navigation
chore: align automatic Fast Forward skill sources - #16
Conversation
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Configure skill sources .changelog/consistent-skill-sources.md, skills.json |
skills.json uses the 1.15.0 schema, trusts fast-forward/* Composer packages with trusted replacement enabled, and adds resources/skills as a directory source. The Changelog source ref changes to ^0.4.0, and its explicit skills list is removed. The changelog entry records the local skill source, donor trust pattern, and stable Changelog release range. |
Priority:
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Other
Possibly related issues
- php-fast-forward/dev-tools issue 195: The configuration uses
resources/skills, trustsfast-forward/*packages, and resolves the Changelog skill from Composer metadata. - php-fast-forward/dev-tools issue 384: The configuration uses trusted Composer sources and a versioned skills schema.
- php-fast-forward/dev-tools issue 240: The configuration adds local skill discovery and sets a stable Changelog release range.
- php-fast-forward/code-standards issue 1: The configuration uses the existing skills setup for local and Composer-based skill sources.
Merge Risk: 
Minimal · up to d075c
-
No actionable merge-blocking risk was identified in the reviewed configuration changes.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
A rabbit checks the source list twice
A local skill hops into view
Trusted donors join the trail
Changelog follows 0.4.x
The schema keeps its numbers neat
And off the rabbit bounds with cheer
Comment @coderabbitai help to get the list of available commands.
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. |
Changelog✅ Contribution check passed. Head: Base: Contribution: 1 new valid fragment(s). Policy: ordinary.
|
Use the same skill-source configuration as the Fast Forward libraries:
fast-forward/*replaces the explicit Composer trust list,auto-sync: trueremains enabled, and the package's own payload is sourced fromresources/skillswithout a package override. Discover Changelog skills from its Composer metadata without a redundant allowlist and resolve stable0.4.xreleases through^0.4.0. Pin the schema to the verified installer release and add a patch fragment.Validation
composer validate --strictpassed.composer skills:updatesynchronized the local consumer skill and Changelog successfully; fragment and whitespace checks passed. Bothphp tests/generate-fixture.phpandphp tests/verify-consumer-skill.phppassed with phpDocumentor 3.10.0; the prepared PHAR's signature was verified.No merge or release is included.