Skip to content

feat: add skills/ folder with setup-sandbox submodule#15

Open
sweetmantech wants to merge 1 commit intomainfrom
sweetmantech/myc-4228-mono-skill-setup-sandbox-new-skills-folder-with-all-skills
Open

feat: add skills/ folder with setup-sandbox submodule#15
sweetmantech wants to merge 1 commit intomainfrom
sweetmantech/myc-4228-mono-skill-setup-sandbox-new-skills-folder-with-all-skills

Conversation

@sweetmantech
Copy link
Copy Markdown
Contributor

@sweetmantech sweetmantech commented Feb 12, 2026

Summary

  • Adds skills/setup-sandbox as a git submodule pointing to recoupable/skill-setup-sandbox
  • Establishes the skills/ directory as the home for all skill submodules in the monorepo

Test plan

  • Submodule clones correctly via git submodule update --init skills/setup-sandbox
  • Verify SKILL.md is accessible at skills/setup-sandbox/SKILL.md

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Integrated new sandbox setup module into the repository infrastructure.

Adds the setup-sandbox skill repo (recoupable/skill-setup-sandbox) as a git
submodule at skills/setup-sandbox. This establishes the skills/ folder as
the home for all skill submodules.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 12, 2026

📝 Walkthrough

Walkthrough

A Git submodule is added to the repository to integrate the external skill-setup-sandbox repository at skills/setup-sandbox, with a specific commit reference tracked in the submodule configuration.

Changes

Cohort / File(s) Summary
Git Submodule Addition
.gitmodules, skills/setup-sandbox
Adds a new submodule entry for skills/setup-sandbox pointing to https://github.com/recoupable/skill-setup-sandbox.git with commit reference bea2505.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A new skill hops into the fold,
Through submodule pathways, brave and bold,
Git tracks it all with careful care,
Setup-sandbox now lives in there!
The warren grows, with skills to share! 🌱

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main change: adding a skills/ folder with the setup-sandbox submodule, which matches the .gitmodules and submodule reference changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sweetmantech/myc-4228-mono-skill-setup-sandbox-new-skills-folder-with-all-skills

No actionable comments were generated in the recent review. 🎉


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.

Copy link
Copy Markdown

@recoup-coding-agent recoup-coding-agent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review: Add skills/ folder with setup-sandbox submodule

Summary

Adds skills/setup-sandbox as a git submodule pointing to recoupable/skill-setup-sandbox. Establishes the skills/ directory convention in the mono repo for skill submodules.

Assessment

✅ Minimal, correct submodule addition.
.gitmodules follows the existing pattern for other submodules.
✅ Pinned to a specific commit (bea2505).

🔵 Nit

The test plan has an unchecked item: [ ] Verify SKILL.md is accessible at skills/setup-sandbox/SKILL.md. Confirm this before merge to ensure the submodule ref is valid.

Verdict: approve

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.

2 participants