feat(0.21.6): registry — Phase 4f orcahand (final Tier-2 migration)#59
Merged
Conversation
1 NEW entry (orcahand). Resolves the orcahand dedup — both source repos archived. Companion to broomva/skills PR #9 (merge df4fb13) + 2 redirect-stubs. Registry 63 → 64. Migration arc complete: 47 skills in monorepo.
|
Warning Review limit reached
More reviews will be available in 50 minutes and 38 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
4 tasks
broomva
added a commit
that referenced
this pull request
May 26, 2026
…attern (Phase 6b) (#60) * feat(0.21.7): bstack skills graduate — crystallize Tier-2 migration pattern (Phase 6b) New `bstack skills graduate <name>` subcommand automating the skill-graduation pattern that ran 8 times manually on 2026-05-25/26 (Phases 2-4f). P16 crystallization — rule-of-three exceeded ~8x. ## What it does Automates: clone source + monorepo → copy canonical content (excl. .git, dot-mirror dirs, LICENSE, skills-lock.json) → append README Tier-2 row → commit + push + PR on monorepo → (--stub) redirect-stub PR on source → (--merge) merge → cleanup. Registry update is printed (copy-paste), not auto-applied (needs coordinated version bump review). ## Files - NEW scripts/skill-graduate.sh (~230 lines) — the engine; env-overridable gh/git/tmpdir/dry-run hooks for hermetic testing - NEW tests/skill-graduate.test.sh — 9 offline tests (arg-parse, dry-run, rename detection, stub-gh/git execution with copy+exclude verification, no-SKILL.md error path). All 9 pass. - CHANGED bin/bstack-skills — graduate) dispatch + usage - CHANGED bin/bstack — top-level usage entry - VERSION 0.21.6 → 0.21.7 ## Validation (P11) - 9/9 hermetic tests pass (offline, stub gh/git) - dry-run verified against real skill names (rename detection: omnivoice-skill → omnivoice) - top-level dispatch `bstack skills graduate --help` routes correctly - invalid-name + missing-name + unknown-option all exit 2; no-SKILL.md exits 1 ## Provenance 8 manual instances: broomva/skills PRs #2-#9 + broomva/bstack PRs #53-#59. The next graduation is one command instead of a full manual cycle. * fix(p20-round-1): address cross-review blockers on skill-graduate CLI P20 cross-review (Strata B) returned NEEDS-WORK (7/10) with 2 blockers + 2 nits + a test-coverage gap. All addressed: 1. [BLOCKER] Word-splitting in copy loop — `for item in $(ls -A)` broke on spaced/globbed filenames (reproduced by reviewer; aborts under set -e mid-copy). Replaced with null-delimited `find -print0 | while read -d ''`. 2. [BLOCKER] Header lie — step 3 claimed "Append a row to the monorepo README Tier-2 table" but no such code existed. Removed the false claim; README row is now correctly documented as a printed manual follow-up (alongside the registry entry), since category placement needs judgment. 3. [idempotency] Re-run safety — added a pre-flight `gh pr list --head` check that bails cleanly if an open PR for the branch already exists, instead of failing opaquely inside the push/PR subshell on the 2nd run. 4. [merge masking] `--merge` no longer swallows genuine merge failures — captures the dual-attempt result and WARNs (red CI / conflicts / gate) instead of `|| true` blackholing. 5. [test gap] T8 now inspects the COPIED TREE (asserts references/ + scripts/ present, .claude/ + LICENSE + skills-lock.json absent) via the new BSTACK_GRADUATE_NO_CLEANUP test hook — previously only checked exit code. Added T10: spaced/globbed-filename regression test (the exact gap that let blocker #1 through a green suite). 9 → 10 tests, all pass. Validation: 10/10 hermetic tests pass. T10 fails against the pre-fix loop (confirmed the regression guard is real). * fix(p20-round-2): surface stub-merge failures too (consistency with monorepo path) Residual nit from P20 round-2 re-review (verdict PASS 9/10): the source redirect-stub merge still had `|| true`, asymmetric with the monorepo path's fix. Now warns on genuine merge failure instead of swallowing. Cosmetic — low-stakes (redirect stub, not canonical content) — but closes the asymmetry the reviewer flagged. 10/10 tests still pass.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
1 NEW entry (orcahand). Resolves orcahand dedup — both source repos archived. Companion to broomva/skills PR #9 (merge df4fb13) + 2 stubs. Migration arc complete: 47 skills in monorepo.