Skip to content

chore: remove ralph plugin - #711

Merged
baleen37 merged 1 commit into
mainfrom
nimble-harbor-kernighan
Jun 26, 2026
Merged

chore: remove ralph plugin#711
baleen37 merged 1 commit into
mainfrom
nimble-harbor-kernighan

Conversation

@baleen37

@baleen37 baleen37 commented Jun 26, 2026

Copy link
Copy Markdown
Owner

Summary

plugins/ralph/ (Ralph Loop) 플러그인을 저장소에서 완전히 제거합니다.

Changes

  • plugins/ralph/ 디렉터리 전체 삭제 (plugin.json 2종, hooks, ralph/ralph-cancel 스킬)
  • ralph 전용 테스트 삭제: tests/ralph_hooks_json.bats, tests/ralph_persist.bats
  • .claude-plugin/marketplace.json, .agents/plugins/marketplace.json에서 ralph 항목 제거
  • plugins/me/skills/setup/settings.json에서 ralph@bstack 등록 제거
  • README.md 디렉터리 트리 및 스킬 표에서 ralph 항목 제거
  • .pre-commit-config.yaml exclude 패턴에서 .ralph/ 제거 (tests/만 유지)
  • tests/codex_marketplace_json.bats 플러그인 카운트 5 → 4 갱신

docs/CHANGELOG의 ralph 과거 기록은 이력 보존을 위해 유지했습니다.

Tests

  • bats tests/ 62개 전부 통과
  • marketplace/settings JSON 유효성 확인
  • README markdownlint 통과

Summary by CodeRabbit

  • Bug Fixes

    • Removed the Ralph plugin from marketplace and setup configurations, and updated documentation to match the current plugin set.
    • Adjusted marketplace checks to reflect the reduced plugin count.
  • Chores

    • Cleaned up related plugin metadata, hooks, and test coverage tied to the removed plugin.
    • Updated whitespace handling rules for test files.

@baleen37
baleen37 enabled auto-merge (squash) June 26, 2026 08:38
@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3e35db87-2586-4e98-bb01-b49260414537

📥 Commits

Reviewing files that changed from the base of the PR and between 0058c5b and db0ffaf.

📒 Files selected for processing (14)
  • .agents/plugins/marketplace.json
  • .claude-plugin/marketplace.json
  • .pre-commit-config.yaml
  • README.md
  • plugins/me/skills/setup/settings.json
  • plugins/ralph/.claude-plugin/plugin.json
  • plugins/ralph/.codex-plugin/plugin.json
  • plugins/ralph/hooks/hooks.json
  • plugins/ralph/hooks/ralph-persist.ts
  • plugins/ralph/skills/ralph-cancel/SKILL.md
  • plugins/ralph/skills/ralph/SKILL.md
  • tests/codex_marketplace_json.bats
  • tests/ralph_hooks_json.bats
  • tests/ralph_persist.bats

📝 Walkthrough

Walkthrough

The PR removes the Ralph plugin from marketplace registration, deletes its hook and skill files, updates related tests and setup configuration, and cleans up README and pre-commit references.

Changes

Ralph plugin removal

Layer / File(s) Summary
Marketplace entries
.agents/plugins/marketplace.json, .claude-plugin/marketplace.json, tests/codex_marketplace_json.bats
Removes the ralph plugin from both marketplace manifests and updates the marketplace plugin count assertion from 5 to 4.
Plugin artifacts and tests
plugins/ralph/hooks/hooks.json, plugins/ralph/hooks/ralph-persist.ts, plugins/ralph/skills/ralph/SKILL.md, plugins/ralph/skills/ralph-cancel/SKILL.md, tests/ralph_hooks_json.bats, tests/ralph_persist.bats
Deletes the Ralph hook manifest, hook script, skill docs, and the tests that exercised them.
Downstream references
plugins/me/skills/setup/settings.json, .pre-commit-config.yaml, README.md
Removes ralph@bstack from the setup plugin list, narrows the trailing-whitespace exclusion to tests/, and drops Ralph references from the README structure and skills sections.

Estimated review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Poem

I hopped through manifests, light as a breeze,
Then tucked Ralph’s trail beneath the trees.
Four plugins left, the burrow sings,
With tidy hooks and quieter things.
Thump thump! 🐰

✨ 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 nimble-harbor-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.

@baleen37
baleen37 merged commit 7d88f9b into main Jun 26, 2026
4 of 5 checks passed
@baleen37
baleen37 deleted the nimble-harbor-kernighan branch June 26, 2026 08:39
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