feat(me): add user-invocable to commit-push-pr skill - #6
Conversation
Allow commit-push-pr skill to be invoked as a slash command. Added user-invocable: true to frontmatter for Claude Code 2.1.0+. Co-Authored-By: Claude <noreply@anthropic.com>
📝 WalkthroughWalkthroughA metadata flag Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 📜 Recent review detailsConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🧰 Additional context used📓 Path-based instructions (1)plugins/*/skills/**/SKILL.md📄 CodeRabbit inference engine (CLAUDE.md)
Files:
🔇 Additional comments (2)
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. Comment |
# [1.3.0](v1.2.0...v1.3.0) (2026-01-14) ### Features * **me:** add user-invocable to commit-push-pr skill frontmatter ([#6](#6)) ([e48043e](e48043e))
|
🎉 This PR is included in version 1.3.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
…is missing The update-checker.sh script was exiting early when MARKETPLACE_FILE was not set or didn't exist, without creating the last-check timestamp file. This caused test #6 to fail in CI. The fix moves the timestamp update before the marketplace check, so the timestamp is always created (unless --check-only is used), regardless of whether a marketplace file exists. Co-Authored-By: Claude <noreply@anthropic.com>
* fix(ci): test release workflow fix Add debug logging to semantic-release step to troubleshoot release workflow issues. This test commit will verify the fix works correctly. Co-Authored-By: Claude <noreply@anthropic.com> * fix(auto-updater): ensure timestamp is created even when marketplace is missing The update-checker.sh script was exiting early when MARKETPLACE_FILE was not set or didn't exist, without creating the last-check timestamp file. This caused test #6 to fail in CI. The fix moves the timestamp update before the marketplace check, so the timestamp is always created (unless --check-only is used), regardless of whether a marketplace file exists. Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
Added debug output to verify CONFIG_DIR and timestamp file creation in CI environment. Co-Authored-By: Claude <noreply@anthropic.com>
* chore: trigger new release with auto-updater fix This empty commit triggers a new release to properly include the auto-updater fix from the previous release. Co-Authored-By: Claude <noreply@anthropic.com> * fix(auto-updater): add enhanced debug logging for timestamp creation Added verification after timestamp creation to detect race conditions or environment-specific issues in CI. Co-Authored-By: Claude <noreply@anthropic.com> * test(auto-updater): add debug logging to test #6 Added debug output to verify CONFIG_DIR and timestamp file creation in CI environment. Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
Changed TEMP_DIR from using $$ (PID) to $BATS_TEST_NAME to ensure each test has its own isolated temp directory. This fixes test #6 failing in CI due to shared state between tests. Root cause: When BATS runs tests in the same process, $$ returns the same PID for all tests, causing them to share the same TEMP_DIR. This leads to race conditions where one test's teardown affects another test's state. Co-Authored-By: Claude <noreply@anthropic.com>
Changed TEMP_DIR from using $$ (PID) to $BATS_TEST_NAME to ensure each test has its own isolated temp directory. This fixes test #6 failing in CI due to shared state between tests. Root cause: When BATS runs tests in the same process, $$ returns the same PID for all tests, causing them to share the same TEMP_DIR. This leads to race conditions where one test's teardown affects another test's state. Co-Authored-By: Claude <noreply@anthropic.com>
* chore: trigger new release with auto-updater fix This empty commit triggers a new release to properly include the auto-updater fix from the previous release. Co-Authored-By: Claude <noreply@anthropic.com> * fix(auto-updater): use unique temp directory per test Changed TEMP_DIR from using $$ (PID) to $BATS_TEST_NAME to ensure each test has its own isolated temp directory. This fixes test #6 failing in CI due to shared state between tests. Root cause: When BATS runs tests in the same process, $$ returns the same PID for all tests, causing them to share the same TEMP_DIR. This leads to race conditions where one test's teardown affects another test's state. Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
Summary
user-invocable: trueto commit-push-pr skill frontmatterTest plan
🤖 Generated with Claude Code
Co-Authored-By: Claude noreply@anthropic.com
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.