Skip to content

chore: manage Changelog skill through Composer - #45

Merged
coisa merged 1 commit into
mainfrom
codex/llm-skills
Oct 20, 2026
Merged

coisa merged 1 commit into
mainfrom
codex/llm-skills

Conversation

@coisa

@coisa coisa commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Manage the Changelog contributor skill through Composer instead of committing generated copies. Preserve the existing @v0 caller, full package/version title template and publication settings.

Add development-only llm/skills:^1.15 and its specific plugin allowlist entry. Native skills:init / skills:add commands created skills.json, sourcing only fast-forward-changelog from php-fast-forward/changelog on GitHub within ^0.3.0. Composer dependency discovery, vendor sources and aliases are disabled. composer install restores the local skill; composer skills:show inspects it and composer skills:update --from=github refreshes it within that range.

Untrack only the Changelog skill's generated payload and former native receipt. Keep the six published payload files locally, ignore generated .agents/ files, and exclude development setup from Composer archives. Other existing tracked .agents assets remain unchanged. Application runtime requirements and other plugin permissions are preserved; no lockfile is introduced.

This follow-up starts from the current main after merged PR #42 and version PR #44. Existing publication configuration remains unchanged.

Validation: native manager synchronization and skills:show passed; all six installed file hashes match published Changelog 0.3.0; Composer strict metadata validation and the complete dependency-graph dry run passed with llm/skills 1.15.0; actionlint, fragment validation and a committed comparison for this migration passed. The plugin commands ran with a workspace-isolated Composer home/cache; no host-global plugin state was modified. Full application tests are delegated to CI because runtime code is unchanged.

Head: f6bb2fc0f5d3a79a559ff2da8d80c6e6647fc578. CI/reviews are being refreshed for this head. No merge, tag creation, GitHub Release publication or existing Release-body change was performed.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-20T16:19:58.781166Z f6bb2fc PR opened
🔒 Security Review ✅ Completed 2026-10-20T16:27:31.348285Z f6bb2fc PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 6ded43e5-4954-46f5-94f6-44b2794b55a3
📥 Commits

Reviewing files that changed from the base of the PR and between 42192e9 and f6bb2fc.

📒 Files selected for processing: 13
  • .agents/skills/fast-forward-changelog/.fast-forward-install.json
  • .agents/skills/fast-forward-changelog/LICENSE
  • .agents/skills/fast-forward-changelog/SKILL.md
  • .agents/skills/fast-forward-changelog/examples/contribution.php
  • .agents/skills/fast-forward-changelog/references/automation.md
  • .agents/skills/fast-forward-changelog/references/cli.md
  • .agents/skills/fast-forward-changelog/references/php-api.md
  • .changelog/composer-managed-changelog-skill.md
  • .gitattributes
  • .gitignore
  • composer.json
  • docs/changelog.md
  • skills.json
💤 Files with no reviewable changes: 7
  • .agents/skills/fast-forward-changelog/.fast-forward-install.json
  • .agents/skills/fast-forward-changelog/LICENSE
  • .agents/skills/fast-forward-changelog/SKILL.md
  • .agents/skills/fast-forward-changelog/references/php-api.md
  • .agents/skills/fast-forward-changelog/examples/contribution.php
  • .agents/skills/fast-forward-changelog/references/automation.md
  • .agents/skills/fast-forward-changelog/references/cli.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Summary

Summary by CodeRabbit

  • New Features

    • Added development-only Composer-based installation and synchronization for the changelog contributor skill.
    • Added configuration for sourcing the skill from its reviewed GitHub release line.
    • Excluded generated skill files from source archives and version control.
  • Documentation

    • Updated changelog guidance with Composer installation, skill inspection, and refresh workflows.
    • Added a changelog entry describing the move to managed skill installation.
  • Removals

    • Removed the previously bundled changelog skill files and supporting examples and references.

The repository adds Composer configuration for installing the Changelog skill from GitHub. It removes the checked-in skill copy and updates the changelog entry and setup documentation to describe the Composer-managed workflow.

Changes: Changelog skill management

Layer / File(s) Summary
Configure Composer-managed skill installation
skills.json, composer.json, .gitignore, .gitattributes
Adds the llm/skills development dependency and plugin permission. Configures the php-fast-forward/changelog source and .agents/skills target. Ignores the generated target and excludes skills.json from exported archives.
Remove checked-in skill and update setup guidance
.agents/skills/fast-forward-changelog/*, .changelog/composer-managed-changelog-skill.md, docs/changelog.md
Removes the checked-in skill and its supporting files. Updates the changelog entry and setup documentation with the Composer installation, inspection, and refresh workflow.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~12 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to f6bb2

The Composer-managed Changelog skill workflow is consistent and presents no established merge-blocking risk.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title clearly and concisely describes the main change: managing the Changelog skill through Composer.
Description check Passed The description directly explains the Composer migration, configuration, preserved behavior, removed generated files, and validation results.
Docstring Coverage Passed 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…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
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. (6 skipped: 6 unsupported.)

✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 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.

❤️ Share

A rabbit reads the skills file bright,
Then hops through Composer’s setup right.
The old checked-in pages leave the den,
GitHub brings the skill back again.
Fresh notes rustle in the changelog,
While carrots wait beside the log.

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

Changelog

✅ Contribution check passed.

Head: f6bb2fc0f5d3a79a559ff2da8d80c6e6647fc578

Base: 42192e9fea16b95c298345fda32dd23af715cdf2

Contribution: 1 new valid fragment(s). Policy: ordinary.

  • composer-managed-changelog-skill.md — changed, patch

    Restore the Changelog contributor skill through the development-only Composer skill manager and its reviewed GitHub source instead of versioning generated skill copies.

@coisa
coisa merged commit 991363d into main Oct 20, 2026
39 checks passed
@coisa
coisa deleted the codex/llm-skills branch October 20, 2026 16:38
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.

1 participant