Skip to content

feat(me): allow model invocation for lifecycle skills - #654

Merged
baleen37 merged 2 commits into
mainfrom
sunny-mango-kernighan
May 13, 2026
Merged

feat(me): allow model invocation for lifecycle skills#654
baleen37 merged 2 commits into
mainfrom
sunny-mango-kernighan

Conversation

@baleen37

@baleen37 baleen37 commented May 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • Remove disable-model-invocation: true from build, test, verify, review, and ship skills so the model can auto-invoke them when relevant
  • Also drop the matching explicit-only rule (item 5) from verify SKILL.md

Changes

  • plugins/me/skills/build/SKILL.md
  • plugins/me/skills/test/SKILL.md
  • plugins/me/skills/verify/SKILL.md (frontmatter + body rule)
  • plugins/me/skills/review/SKILL.md
  • plugins/me/skills/ship/SKILL.md

Test plan

  • Confirm skills appear in available-skills list and can be invoked by the model
  • Confirm /build, /test, /verify, /review, /ship still work as explicit slash commands

Summary by CodeRabbit

  • Chores

    • Enabled automatic invocation for multiple workflow skills by removing their explicit-disable setting.
    • Adjusted verify workflow rules to allow automatic triggering (renumbered rules accordingly).
  • Tests

    • Updated tests to stop enforcing the now-removed disable setting in skill metadata.

Review Change Stack

Remove disable-model-invocation flag from build, test, verify, review,
and ship skills so the model can auto-invoke them when appropriate
contexts arise, instead of requiring explicit slash-command invocation.
@baleen37
baleen37 enabled auto-merge (squash) May 13, 2026 07:55
@coderabbitai

coderabbitai Bot commented May 13, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 34eda4e6-edac-4655-8cde-e2681d2e9364

📥 Commits

Reviewing files that changed from the base of the PR and between 724853a and 888da3c.

📒 Files selected for processing (1)
  • tests/me/me-specific.bats

📝 Walkthrough

Walkthrough

The disable-model-invocation: true flag is removed from the YAML frontmatter of five lifecycle skill files: build, review, ship, test, and verify. The verify skill's Rules section is updated to remove the explicit-only rule and renumber remaining rules accordingly.

Changes

Lifecycle Skills Model Invocation

Layer / File(s) Summary
Disable model-invocation flag across lifecycle skills
plugins/me/skills/build/SKILL.md, plugins/me/skills/review/SKILL.md, plugins/me/skills/ship/SKILL.md, plugins/me/skills/test/SKILL.md, plugins/me/skills/verify/SKILL.md, tests/me/me-specific.bats
Removed disable-model-invocation: true from the frontmatter of the five lifecycle skill SKILL.md files. Updated plugins/me/skills/verify/SKILL.md Rules list to remove the explicit-only rule and renumber remaining rules. Updated tests/me/me-specific.bats to stop asserting the removed flag while keeping delimiter/name/description checks.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • baleen37/bstack#635: Both PRs modify plugins/me/skills/ship/SKILL.md—this PR removes disable-model-invocation: true while that PR reworks /ship behavior.
  • baleen37/bstack#650: Touches the same lifecycle skill frontmatter flags; that PR adds the flags which this PR removes.
  • baleen37/bstack#640: Also edits “me” skill definitions and tests/me/me-specific.bats, overlapping with these frontmatter/test changes.

Poem

🐰 I nudged the flags and set them free,
Five little skills can now run with glee.
Verify's rule lost its solo song,
The workflows hum and hop along! 🥕

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat(me): allow model invocation for lifecycle skills' accurately and specifically describes the main change: removing disable-model-invocation flags from five lifecycle skills to enable model auto-invocation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sunny-mango-kernighan

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

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

The previous assertion forced lifecycle skills to remain explicit-only,
which contradicts the change that allows model invocation. Keep the
name/description checks; let model-invocation policy be per-skill.
@baleen37
baleen37 merged commit 0988d2d into main May 13, 2026
4 of 5 checks passed
@baleen37
baleen37 deleted the sunny-mango-kernighan branch May 13, 2026 07:58
baleen-release-app Bot pushed a commit that referenced this pull request May 13, 2026
# [17.10.0](v17.9.1...v17.10.0) (2026-05-13)

### Features

* **me:** allow model invocation for lifecycle skills ([#654](#654)) ([0988d2d](0988d2d))
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