chore: skills repo cleanup — remove stale content, fix references, delete template#9
Conversation
Brand guidelines are internal and shouldn't be in the public skills repo. The archived recoupable/brand-guidelines repo was already private. Made-with: Cursor
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 29 minutes and 9 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, 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 have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe comprehensive brand guidelines documentation file ( 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. ✨ 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 |
Made-with: Cursor
All API keys are personal. RECOUP_ACCOUNT_ID is a sandbox env var, not an org key indicator. Made-with: Cursor
sidneyswift
changed the title
Makes the "already set up" check prominent so agents see it before reading the steps. Made-with: Cursor
Tell the agent to run `ls orgs/` and check the result, not just assume it knows whether setup already happened. Made-with: Cursor
Made-with: Cursor
Made-with: Cursor
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
1 issue found across 1 file (changes from recent commits).
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="skills/setup-sandbox/SKILL.md">
<violation number="1" location="skills/setup-sandbox/SKILL.md:8">
P2: This change introduces a cross-skill dependency (`setup-sandbox` now directs users to `artist-workspace`), but repository rules require each skill to be self-contained.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review, or fix all with cubic.
|
|
||
| # Setup Sandbox | ||
|
|
||
| **Before running, check:** `ls orgs/` from the sandbox root. If the directory exists and contains org folders, the sandbox is already set up — stop here and read the `artist-workspace` skill instead. Only continue if `orgs/` doesn't exist or is empty. |
There was a problem hiding this comment.
P2: This change introduces a cross-skill dependency (setup-sandbox now directs users to artist-workspace), but repository rules require each skill to be self-contained.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At skills/setup-sandbox/SKILL.md, line 8:
<comment>This change introduces a cross-skill dependency (`setup-sandbox` now directs users to `artist-workspace`), but repository rules require each skill to be self-contained.</comment>
<file context>
@@ -5,7 +5,7 @@ description: Set up the initial file system for a new sandbox. Fetches the accou
# Setup Sandbox
-**Before running, check:** `ls orgs/` from the sandbox root. If the directory exists and contains org folders, the sandbox is already set up — stop here. Only continue if `orgs/` doesn't exist or is empty.
+**Before running, check:** `ls orgs/` from the sandbox root. If the directory exists and contains org folders, the sandbox is already set up — stop here and read the `artist-workspace` skill instead. Only continue if `orgs/` doesn't exist or is empty.
Create the folder structure for the connected account's organizations and artists.
</file context>
- No longer limited to short-form video — covers images, video, captions, lipsync, editorial, and more - Removed "primitives" concept — explains capabilities directly - Shows both CLI and API access for every capability - Links to developers.recoupable.com for full API docs Made-with: Cursor
sidneyswift
changed the title
2 participants
Summary
brand-guidelinesskill — internal content that shouldn't be in the public repocontent-strategy.mdreference fromstreaming-growthskillsetup-sandbox— all API keys are personal,RECOUP_ACCOUNT_IDis a sandbox env varartist-workspacehandoff tosetup-sandboxskill (before and after setup)content-creationskill — broadened from short-form video to all content types, removed "primitives" concept, shows both CLI and API accesstemplate/directory — it was being picked up bynpx skills addas a junk skill called "my-skill-name"README.md,AGENTS.md, andcontributing.mdto remove template referencesContext
Part of a broader skills registry cleanup. We also archived 6 old standalone skill repos on GitHub (
setup-sandbox,artist-workspace,release-management,chartmetric,songwriting,brand-guidelines) that were causing skills.sh to show stale duplicate sources.Next step
After this merges, update
tasks/src/sandboxes/ensureSetupSandbox.tsto install all skills vianpx skills add recoupable/skillsinstead of installing 3 standalone repos individually.