Repository navigation
Conversation
Ran a controlled benchmark (same task per overlapping skill, scored against answer keys, outputs verified with `cargo test --offline`). Applied the verdict: - Remove rtk-tdd: 11/12 in the TDD benchmark (used synthetic fixtures, the anti-pattern cli-testing.md warns against) vs tdd-rust's 12/12 at lower cost. TDD is now consolidated on tdd-rust. - Rename ship -> rtk-release: the repo `ship` skill was shadowed by gstack's `ship` in the skill list and was unreachable. Renamed the dir, added an explicit `name:`, and updated the `/ship` invocation hint to `/rtk-release`. - Consolidate worktree commands: they existed twice (root and tech/) with diverged content. Kept the richer root versions (they document --check/--fast, .worktreeinclude, manual force-remove), deleted the tech/ duplicates, and moved remove-worktree into root so the whole worktree family lives in one place. tech/ now holds only its unique commands (audit-codebase, codereview). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
|
Contributor
|
This pull request has been automatically marked as stale due to 90 days of inactivity. If this pull request is still relevant, please leave any comment (for example, "bump"), and we'll keep it open. Your contribution is very much appreciated — we're sorry we haven't been able to review it yet. |
Contributor
|
Closing this pull request after a prolonged period of inactivity. If this is still relevant, please ask for it to be reopened. Thank you! |
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
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.
What
Cleanup of repo-local
.claude/skills and commands, driven by a controlled benchmark of overlapping/duplicated skills.Method
Ran each overlapping skill against the same planted task, scored outputs against answer keys, and verified every produced artifact with
cargo test --offline(fixtures present,lazy_staticused, tests passing). Key finding: on these tasks, detection accuracy was at ceiling — skills differed in cost, judgment conservatism, and report structure, not correctness.Changes
rtk-tdd— scored 11/12 in the TDD benchmark because it used synthetic fixtures (the anti-patterncli-testing.mdexplicitly warns against), vstdd-rust's 12/12 at lower token cost. TDD is now consolidated ontdd-rust.ship→rtk-release— the reposhipskill was shadowed by gstack'sshipin the skill list and was effectively unreachable. Renamed the directory (history preserved), added an explicitname:field, and updated the/shipinvocation hint to/rtk-release..claude/commands/root andtech/) with diverged content. Kept the richer root versions (they document--check/--fast,.worktreeinclude, manual force-remove), deleted thetech/duplicates, and movedremove-worktreeinto root so the whole worktree family lives in one place.tech/now holds only its unique commands (audit-codebase,codereview).Not included (deliberately)
pr-reviewvspr-triage— benchmark showed their behavior converges (pr-triage drifted into per-PR deep review), but which to keep is a workflow preference, so left for maintainer judgment.investigate— flagged as redundant withsystematic-debugging, but it's a global gstack skill (out of this repo's scope; regenerated on gstack upgrade).Verification
All benchmark arm outputs compiled and passed
cargo test --offline. No source code changed — this touches only.claude/skill/command definitions. Dangling-reference check passed (no strayrtk-tddor/tech:worktreereferences).🤖 Generated with Claude Code