Skip to content

chore: consolidate agent skills from cloud into shared package#61

Merged
khaliqgant merged 8 commits into
mainfrom
chore/consolidate-skills-from-cloud
May 31, 2026
Merged

chore: consolidate agent skills from cloud into shared package#61
khaliqgant merged 8 commits into
mainfrom
chore/consolidate-skills-from-cloud

Conversation

@khaliqgant
Copy link
Copy Markdown
Member

Summary

Consolidates 16 generic, reusable agent skills that had been duplicated inside AgentWorkforce/cloud (scattered across .claude/skills, .agents/skills, .factory/skills, and skills/, with copies that had drifted out of sync) into this shared prpm package — establishing a single source of truth that cloud and other repos install via prpm.

Changes

  • 11 skills newly added to prpm.json and skills/ (v1.0.0, descriptions sourced from each SKILL.md):
    auto-merge-and-composition-safety, autonomous-run-contract, creating-claude-rules-skill, dormant-flip-and-rollback, instrument-dont-guess, swarm-blockers-and-gate-scoreboard, tiered-acceptance, building-nango-functions-locally, running-headless-orchestrator, trigger-autocomplete-catalog, shadcn.
    packages count: 15 → 26.
  • 5 skills already published here (choosing-swarm-patterns, orchestrating-agent-relay, relay-80-100-workflow, review-fix-signoff-loop, writing-agent-relay-workflows) keep their existing canonical versions. The cloud copies were older/smaller and were discarded, not promoted.

Notes

  • Cloud retains only its cloud-specific skill (proactive-e2e-debug) and will consume the rest via prpm install; cloud does not own a prpm.json.
  • Stray .DS_Store / IDE settings.local.json files were excluded from the moved trees.
  • A companion change in the cloud repo removes the duplicated skill copies and symlinks (committed separately).

🤖 Generated with Claude Code

Move 16 generic, reusable skills out of the AgentWorkforce/cloud repo
(where they were duplicated across .claude/skills, .agents/skills,
.factory/skills, and skills/) into this shared prpm package so there is a
single source of truth installed via prpm.

- Add 11 new skills to prpm.json (auto-merge-and-composition-safety,
  autonomous-run-contract, creating-claude-rules-skill,
  dormant-flip-and-rollback, instrument-dont-guess,
  swarm-blockers-and-gate-scoreboard, tiered-acceptance,
  building-nango-functions-locally, running-headless-orchestrator,
  trigger-autocomplete-catalog, shadcn). packages 15 -> 26.
- The 5 skills already published here (choosing-swarm-patterns,
  orchestrating-agent-relay, relay-80-100-workflow,
  review-fix-signoff-loop, writing-agent-relay-workflows) keep their
  existing canonical versions; the cloud copies were older and were
  discarded.

Cloud keeps only its cloud-specific skill (proactive-e2e-debug) and
consumes the rest via prpm.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 31, 2026

Review Change Stack

Warning

Review limit reached

@khaliqgant, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 14 minutes and 13 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 @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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 configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a187e4d9-3558-44f9-a408-ad52c0a233a9

📥 Commits

Reviewing files that changed from the base of the PR and between c86ee95 and 0ec4ec6.

📒 Files selected for processing (5)
  • README.md
  • commands/spawn.md
  • prpm.json
  • skills/dormant-flip-and-rollback/SKILL.md
  • skills/swarm-blockers-and-gate-scoreboard/SKILL.md
📝 Walkthrough

Walkthrough

This PR adds 11 new skill documentation packages, updates the manifest and README to publish them, and provides extensive implementation and operational guidance for autonomous runs, diagnostics, trigger catalogs, Nango functions, Claude rules, and shadcn UI.

Changes

Skills Knowledge Base Expansion

Layer / File(s) Summary
Manifest and contract foundation
prpm.json, skills/autonomous-run-contract/SKILL.md
Registers 11 new packages in the manifest and introduces the autonomous-run contract specification for cutover-class delegated runs.
Per-PR merge and composition safety
skills/auto-merge-and-composition-safety/SKILL.md
Defines the per-PR auto-merge bar, cross-PR composition safety (serialize via green main with rebase/re-CI), and file-based agent verdict reporting.
Dormant flip and rollback pattern
skills/dormant-flip-and-rollback/SKILL.md, skills/running-headless-orchestrator/SKILL.md
Specifies dormant-by-default single-switch flips with pre-authorized rollback and documents headless orchestrator bootstrap, worker spawn, coordination, and lifecycle monitoring.
Blocker swarming and gate coordination
skills/swarm-blockers-and-gate-scoreboard/SKILL.md, skills/tiered-acceptance/SKILL.md
Mandates codex-impl + claude-review pairing for hard blockers and a live RED/AMBER/GREEN gate scoreboard; defines tiered acceptance for large variant sets with dormant tier-2 and per-item enable/rollback.
Operational discipline and trigger catalog
skills/instrument-dont-guess/SKILL.md, skills/trigger-autocomplete-catalog/SKILL.md
Requires diagnostics after two failed fixes and specifies how providers must register webhook/event triggers into KNOWN_TRIGGER_CATALOG (supportedEvents() or .mapping.yaml).
Nango, Claude rules, and shadcn UI implementation
skills/building-nango-functions-locally/{SKILL.md,references/*}, skills/creating-claude-rules-skill/SKILL.md, skills/shadcn/{SKILL.md,cli.md,customization.md,evals/evals.json,mcp.md,agents/openai.yml,rules/*}
Comprehensive guides: local Nango action/sync workflows and six checkpoint patterns; modular Claude rules authoring under .claude/rules/; full shadcn CLI, customization, composition/forms/icons/styling rules, base-vs-radix mappings, MCP server docs, and eval definitions.
README updates
README.md
Bumps package version to 1.1.0, updates Published Skills table with new entries and version bumps, and adds [/review-loop] slash command.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related issues

Possibly related PRs

Poem

🐰 I hopped through docs and left a trail,
Contracts, flips, and shadcn trail.
Nango tests and trigger maps,
Merged with care—no guessing, just caps.
Happy hops for every PR!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: consolidate agent skills from cloud into shared package' clearly and concisely describes the main change: moving reusable skills into a shared package to establish a single source of truth.
Description check ✅ Passed The description is detailed and directly related to the changeset, explaining the consolidation of 16 skills, identifying which ones are newly added, which remain canonical, and the rationale for the reorganization.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/consolidate-skills-from-cloud

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

@gemini-code-assist gemini-code-assist Bot 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

This pull request registers and adds several new Claude skills under the skills/ directory, covering topics such as auto-merge safety, autonomous run contracts, local Nango function building, headless orchestration, and shadcn management. A high-severity issue was identified regarding a name and path mismatch for the creating-claude-rules skill in prpm.json compared to its actual file location and frontmatter name, which has been addressed with a code suggestion.

Comment thread prpm.json Outdated
Comment on lines +307 to +316
{
"name": "creating-claude-rules-skill",
"version": "1.0.0",
"description": "Use when creating or fixing .claude/rules/ files - provides correct paths frontmatter (not globs), glob patterns, and avoids Cursor-specific fields like alwaysApply",
"format": "claude",
"subtype": "skill",
"files": [
"skills/creating-claude-rules-skill/SKILL.md"
]
},
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

There is a name and path mismatch for the creating-claude-rules skill. The actual file added in this PR is located at skills/creating-claude-rules/SKILL.md and its frontmatter name is creating-claude-rules. However, in prpm.json it is registered as creating-claude-rules-skill with the path skills/creating-claude-rules-skill/SKILL.md. This mismatch will cause installation or loading failures. Please update the name and path to match the actual file.

Suggested change
{
"name": "creating-claude-rules-skill",
"version": "1.0.0",
"description": "Use when creating or fixing .claude/rules/ files - provides correct paths frontmatter (not globs), glob patterns, and avoids Cursor-specific fields like alwaysApply",
"format": "claude",
"subtype": "skill",
"files": [
"skills/creating-claude-rules-skill/SKILL.md"
]
},
{
"name": "creating-claude-rules",
"version": "1.0.0",
"description": "Use when creating or fixing .claude/rules/ files - provides correct paths frontmatter (not globs), glob patterns, and avoids Cursor-specific fields like alwaysApply",
"format": "claude",
"subtype": "skill",
"files": [
"skills/creating-claude-rules/SKILL.md"
]
},

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 2 potential issues.

View 3 additional findings in Devin Review.

Open in Devin Review

@@ -0,0 +1,121 @@
---
name: creating-claude-rules
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Package name mismatch between prpm.json and SKILL.md frontmatter for creating-claude-rules-skill

The prpm.json entry uses name: "creating-claude-rules-skill" (prpm.json:308) and the directory is skills/creating-claude-rules-skill/, but the SKILL.md frontmatter declares name: creating-claude-rules (without -skill). Every other skill in this repository has the prpm.json package name match the SKILL.md frontmatter name exactly (verified across all 20+ packages). This inconsistency could cause package resolution or activation failures if tooling uses the frontmatter name as the canonical identifier.

Suggested change
name: creating-claude-rules
name: creating-claude-rules-skill
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Comment on lines +301 to +302
2. **For spawning Codex agents**: Valid Anthropic credentials
- Set `ANTHROPIC_API_KEY` or authenticate via `Codex auth login`
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Wrong credentials documented for spawning Codex agents — says Anthropic but should be OpenAI

The prerequisites section states Codex agents need "Valid Anthropic credentials" and suggests ANTHROPIC_API_KEY or Codex auth login. This appears to be a copy-paste error from skills/orchestrating-agent-relay/SKILL.md which correctly says ANTHROPIC_API_KEY or claude auth login for Claude agents. Codex is OpenAI's CLI agent tool and would need OpenAI credentials (e.g. OPENAI_API_KEY), not Anthropic. The Codex auth login command also doesn't follow any documented pattern — the Claude CLI uses claude auth login.

Prompt for agents
The prerequisites in skills/running-headless-orchestrator/SKILL.md lines 301-302 incorrectly state that spawning Codex agents requires Anthropic credentials. This appears to be a copy-paste from the orchestrating-agent-relay skill (which correctly pairs ANTHROPIC_API_KEY with claude auth login for Claude agents). The fix should update the credential type to match what the Codex harness actually requires (likely OPENAI_API_KEY or the appropriate auth command for the Codex CLI). Check what credentials the agent-relay spawn command actually needs when using the Codex harness and update accordingly.
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

@agent-relay-bot
Copy link
Copy Markdown
Contributor

Reviewed and fixed the PR locally.

Changes made:

  • Aligned creating-claude-rules-skill frontmatter name with prpm.json.
  • Shortened the Nango skill description in prpm.json and SKILL.md to stay within manifest limits.
  • Updated README.md so the package version, skill table, and slash-command table match the expanded manifest.

Validation run:

  • jq empty prpm.json
  • manifest file existence + package/frontmatter name consistency check
  • manifest description length check
  • skills/shadcn/agents/openai.yml YAML parse
  • skills/shadcn/evals/evals.json JSON parse
  • npx prpm publish --list

npx prpm publish --dry-run could not run because the CLI requires login even for dry-run.

@agent-relay-bot
Copy link
Copy Markdown
Contributor

pr-reviewer applied fixes — committed and pushed c86ee95 to this PR. The notes below describe what changed.

Reviewed and fixed the PR locally.

Changes made:

  • Aligned creating-claude-rules-skill frontmatter name with prpm.json.
  • Shortened the Nango skill description in prpm.json and SKILL.md to stay within manifest limits.
  • Updated README.md so the package version, skill table, and slash-command table match the expanded manifest.

Validation run:

  • jq empty prpm.json
  • manifest file existence + package/frontmatter name consistency check
  • manifest description length check
  • skills/shadcn/agents/openai.yml YAML parse
  • skills/shadcn/evals/evals.json JSON parse
  • npx prpm publish --list

npx prpm publish --dry-run could not run because the CLI requires login even for dry-run.

Copy link
Copy Markdown
Contributor

@agent-relay-bot agent-relay-bot Bot left a comment

Choose a reason for hiding this comment

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

pr-reviewer applied fixes — committed and pushed c86ee95 to this PR. The notes below describe what changed.

Reviewed and fixed the PR locally.

Changes made:

  • Aligned creating-claude-rules-skill frontmatter name with prpm.json.
  • Shortened the Nango skill description in prpm.json and SKILL.md to stay within manifest limits.
  • Updated README.md so the package version, skill table, and slash-command table match the expanded manifest.

Validation run:

  • jq empty prpm.json
  • manifest file existence + package/frontmatter name consistency check
  • manifest description length check
  • skills/shadcn/agents/openai.yml YAML parse
  • skills/shadcn/evals/evals.json JSON parse
  • npx prpm publish --list

npx prpm publish --dry-run could not run because the CLI requires login even for dry-run.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 9

🧹 Nitpick comments (1)
skills/shadcn/rules/styling.md (1)

124-124: ⚡ Quick win

Clarify the icon-sizing exception to avoid conflicting guidance with rules/icons.md.

This line currently implies size-* is preferred for icons everywhere, but skills/shadcn/rules/icons.md prohibits sizing classes on icons inside shadcn components. Add an explicit exception here.

Suggested wording tweak
-`size-10` not `w-10 h-10`. Applies to icons, avatars, skeletons, etc.
+`size-10` not `w-10 h-10`. Applies to avatars, skeletons, and standalone icons.
+For icons inside shadcn components (e.g., Button, DropdownMenuItem), follow `rules/icons.md` and avoid icon sizing classes.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@skills/shadcn/rules/styling.md` at line 124, Update the sizing guidance to
state that general UI elements (icons, avatars, skeletons) should use size-*
rather than w-*/h-* but add an explicit exception that icons rendered inside
shadcn components must not receive sizing classes here (see rules/icons.md), and
instead follow the icons rule which prohibits applying size-* to icons inside
shadcn components; reference the size-* token and the icons rule so readers know
to consult rules/icons.md for the exception.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@skills/building-nango-functions-locally/references/actions.md`:
- Around line 214-218: The fenced code block showing the directory structure is
missing a language identifier which triggers MD040; update the block by adding a
language tag (e.g., "text") to the opening triple backticks so it becomes
```text ... ``` to satisfy the linter for the code block that contains the lines
"{integration}/", "proxy/", and "upload-file.ts".

In `@skills/creating-claude-rules-skill/SKILL.md`:
- Around line 99-105: The fenced directory-tree block showing ".claude/rules/"
is missing a fence language (triggers MD040); update the opening
triple-backticks for that directory-tree block to include a language token such
as text (e.g., change ``` to ```text) so the markdown linter recognizes it as a
plain text code block.

In `@skills/dormant-flip-and-rollback/SKILL.md`:
- Around line 65-72: The markdown fenced code blocks containing the "Pre-flip
statement, <run-name>, <ISO-utc> ..." block and the "Rollback executed,
<run-name>, <ISO-utc> ..." block (and the similar pair around lines 81-89) are
untyped and trigger MD040; update each opening triple-backtick to include a
language identifier (e.g., change ``` to ```text) for the blocks that start with
"Pre-flip statement..." and "Rollback executed..." so the fences read ```text to
satisfy the linter.

In `@skills/running-headless-orchestrator/SKILL.md`:
- Around line 84-90: The markdown fences in SKILL.md lack language tags and will
trigger MD040; update every triple-backtick block that contains
command/pseudocode examples to include a language tag (use "text") — e.g., the
blocks around mcp__relaycast__agent_add(...),
mcp__relaycast__message_inbox_check(), mcp__relaycast__agent_list(),
mcp__relaycast__agent_remove(name: "Worker1"),
mcp__relaycast__message_dm_list(to: "Worker1") and the explanatory paragraph
fences — add ```text at the start of each fenced block and keep closing ```
unchanged; apply the same change to the other occurrences mentioned (around
lines covering the other examples in the file).

In `@skills/shadcn/cli.md`:
- Around line 5-8: Remove the empty line separating the two adjacent blockquote
paragraphs so the two "**IMPORTANT:**" notes become a single continuous
blockquote; specifically, merge the second "**IMPORTANT:** Only use the
flags..." paragraph into the first "**IMPORTANT:** Always run commands..."
blockquote so there is no blank line between them, which fixes the MD028
`no-blanks-blockquote` error.
- Around line 159-169: The fenced code block in skills/shadcn/cli.md is
unlabeled (causing MD040); update the triple-backtick fence around the example
block (the block starting with "base  radix" and including the input/button
entries) to include a language tag (e.g., use ```text) so the block is properly
labeled and the lint rule is satisfied.

In `@skills/shadcn/rules/styling.md`:
- Around line 10-11: Wrap wildcard utility names in code spans to satisfy
markdownlint MD037: replace occurrences like "No space-x-* / space-y-*" with "No
`space-x-*` / `space-y-*`" and "Prefer size-* over w-* h-*" with "Prefer
`size-*` over `w-*` `h-*`"; also update the same wildcard usages found later
(the entries referencing space-x-*/space-y-* and size-*/w-*/h-* at the other
occurrences) so every literal utility containing * is enclosed in backticks.

In `@skills/swarm-blockers-and-gate-scoreboard/SKILL.md`:
- Around line 43-54: The markdown fenced-code blocks for the "WS-<name>
sub-contract" stanza and the following table are missing language tags and
trigger MD040; update both opening fences to include a language (use "text") so
the blocks read "```text" rather than just "```"—specifically modify the block
that begins with "WS-<name> sub-contract" and the table block that starts with
"| # | Gate | State | Evidence ..." to use "```text" to satisfy the linter.

In `@skills/trigger-autocomplete-catalog/SKILL.md`:
- Line 32: Replace the ambiguous Markdown placeholder "npm test
--workspace=packages/<provider>" in SKILL.md with a literal-safe form so readers
can copy it reliably; update the command to either escape the angle brackets
(HTML entities) or use a clear placeholder style such as curly braces or a
backticked literal (e.g., packages/{provider} or `packages/<provider>`) so the
workspace path is unambiguous.

---

Nitpick comments:
In `@skills/shadcn/rules/styling.md`:
- Line 124: Update the sizing guidance to state that general UI elements (icons,
avatars, skeletons) should use size-* rather than w-*/h-* but add an explicit
exception that icons rendered inside shadcn components must not receive sizing
classes here (see rules/icons.md), and instead follow the icons rule which
prohibits applying size-* to icons inside shadcn components; reference the
size-* token and the icons rule so readers know to consult rules/icons.md for
the exception.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 5b7a6422-5403-4007-a534-41ac5b935e7f

📥 Commits

Reviewing files that changed from the base of the PR and between 2cb5b08 and 3aa3969.

⛔ Files ignored due to path filters (2)
  • skills/shadcn/assets/shadcn-small.png is excluded by !**/*.png
  • skills/shadcn/assets/shadcn.png is excluded by !**/*.png
📒 Files selected for processing (24)
  • prpm.json
  • skills/auto-merge-and-composition-safety/SKILL.md
  • skills/autonomous-run-contract/SKILL.md
  • skills/building-nango-functions-locally/SKILL.md
  • skills/building-nango-functions-locally/references/actions.md
  • skills/building-nango-functions-locally/references/syncs.md
  • skills/creating-claude-rules-skill/SKILL.md
  • skills/dormant-flip-and-rollback/SKILL.md
  • skills/instrument-dont-guess/SKILL.md
  • skills/running-headless-orchestrator/SKILL.md
  • skills/shadcn/SKILL.md
  • skills/shadcn/agents/openai.yml
  • skills/shadcn/cli.md
  • skills/shadcn/customization.md
  • skills/shadcn/evals/evals.json
  • skills/shadcn/mcp.md
  • skills/shadcn/rules/base-vs-radix.md
  • skills/shadcn/rules/composition.md
  • skills/shadcn/rules/forms.md
  • skills/shadcn/rules/icons.md
  • skills/shadcn/rules/styling.md
  • skills/swarm-blockers-and-gate-scoreboard/SKILL.md
  • skills/tiered-acceptance/SKILL.md
  • skills/trigger-autocomplete-catalog/SKILL.md

Comment thread skills/building-nango-functions-locally/references/actions.md Outdated
Comment thread skills/creating-claude-rules-skill/SKILL.md Outdated
Comment thread skills/dormant-flip-and-rollback/SKILL.md Outdated
Comment thread skills/running-headless-orchestrator/SKILL.md Outdated
Comment thread skills/shadcn/cli.md Outdated
Comment thread skills/shadcn/cli.md Outdated
Comment thread skills/shadcn/rules/styling.md Outdated
Comment thread skills/swarm-blockers-and-gate-scoreboard/SKILL.md Outdated
Comment thread skills/trigger-autocomplete-catalog/SKILL.md Outdated
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@README.md`:
- Line 30: Update the wording in the README table row that references
"building-nango-functions-locally" (the cell containing "CLI dryruns") to use
the standard phrase "CLI dry runs" instead of "CLI dryruns"; locate the table
entry that includes the description "Build, validate, test, and optionally
deploy local TypeScript Nango actions and syncs with CLI dryruns and generated
tests." and replace "dryruns" with "dry runs".
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 611f2e44-4f74-4a5e-85b3-3135d7ebf6ee

📥 Commits

Reviewing files that changed from the base of the PR and between 3aa3969 and c86ee95.

📒 Files selected for processing (4)
  • README.md
  • prpm.json
  • skills/building-nango-functions-locally/SKILL.md
  • skills/creating-claude-rules-skill/SKILL.md
✅ Files skipped from review due to trivial changes (2)
  • skills/creating-claude-rules-skill/SKILL.md
  • skills/building-nango-functions-locally/SKILL.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • prpm.json

Comment thread README.md Outdated
@agent-relay-bot
Copy link
Copy Markdown
Contributor

Reviewed PR #61 and fixed the valid findings.

Changes made:

  • Added missing markdown fence language tags across the PR-touched docs.
  • Fixed markdown blockquote spacing in skills/shadcn/cli.md.
  • Wrapped wildcard Tailwind utility names in code spans and clarified the shadcn icon sizing exception.
  • Replaced packages/<provider> with packages/{provider}.
  • Corrected Codex worker prerequisite docs from Anthropic credentials to OpenAI credentials.
  • Wrapped bare reference URLs that were failing markdownlint.
  • Confirmed the creating-claude-rules-skill manifest/frontmatter/path are now consistent; the older Gemini path comment is stale.

Validation run locally:

  • Targeted markdownlint pass for the relevant rules: 0 error(s).
  • prpm.json and skills/shadcn/evals/evals.json parse successfully.
  • Manifest structural check passed: package file paths exist and skill package names match SKILL.md frontmatter.
  • README package/version/path checks passed.
  • npx prpm publish --list succeeded.
  • npx prpm publish --dry-run could not run because this environment is not logged in to prpm.

@agent-relay-bot
Copy link
Copy Markdown
Contributor

pr-reviewer applied fixes — committed and pushed 61e5c5a to this PR. The notes below describe what changed.

Reviewed PR #61 and fixed the valid findings.

Changes made:

  • Added missing markdown fence language tags across the PR-touched docs.
  • Fixed markdown blockquote spacing in skills/shadcn/cli.md.
  • Wrapped wildcard Tailwind utility names in code spans and clarified the shadcn icon sizing exception.
  • Replaced packages/<provider> with packages/{provider}.
  • Corrected Codex worker prerequisite docs from Anthropic credentials to OpenAI credentials.
  • Wrapped bare reference URLs that were failing markdownlint.
  • Confirmed the creating-claude-rules-skill manifest/frontmatter/path are now consistent; the older Gemini path comment is stale.

Validation run locally:

  • Targeted markdownlint pass for the relevant rules: 0 error(s).
  • prpm.json and skills/shadcn/evals/evals.json parse successfully.
  • Manifest structural check passed: package file paths exist and skill package names match SKILL.md frontmatter.
  • README package/version/path checks passed.
  • npx prpm publish --list succeeded.
  • npx prpm publish --dry-run could not run because this environment is not logged in to prpm.

Copy link
Copy Markdown
Contributor

@agent-relay-bot agent-relay-bot Bot left a comment

Choose a reason for hiding this comment

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

pr-reviewer applied fixes — committed and pushed 61e5c5a to this PR. The notes below describe what changed.

Reviewed PR #61 and fixed the valid findings.

Changes made:

  • Added missing markdown fence language tags across the PR-touched docs.
  • Fixed markdown blockquote spacing in skills/shadcn/cli.md.
  • Wrapped wildcard Tailwind utility names in code spans and clarified the shadcn icon sizing exception.
  • Replaced packages/<provider> with packages/{provider}.
  • Corrected Codex worker prerequisite docs from Anthropic credentials to OpenAI credentials.
  • Wrapped bare reference URLs that were failing markdownlint.
  • Confirmed the creating-claude-rules-skill manifest/frontmatter/path are now consistent; the older Gemini path comment is stale.

Validation run locally:

  • Targeted markdownlint pass for the relevant rules: 0 error(s).
  • prpm.json and skills/shadcn/evals/evals.json parse successfully.
  • Manifest structural check passed: package file paths exist and skill package names match SKILL.md frontmatter.
  • README package/version/path checks passed.
  • npx prpm publish --list succeeded.
  • npx prpm publish --dry-run could not run because this environment is not logged in to prpm.

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

12 issues found across 26 files

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/dormant-flip-and-rollback/SKILL.md">

<violation number="1" location="skills/dormant-flip-and-rollback/SKILL.md:66">
P3: This fenced announcement template needs a language identifier (e.g., `text`) to comply with MD040.</violation>
</file>

Tip: cubic can generate docs of your entire codebase and keep them up to date. Try it here.

Re-trigger cubic

Comment thread skills/running-headless-orchestrator/SKILL.md Outdated
Comment thread skills/shadcn/cli.md Outdated
Comment thread skills/shadcn/evals/evals.json Outdated
Comment thread skills/building-nango-functions-locally/references/syncs.md Outdated
Comment thread skills/shadcn/cli.md Outdated
Comment thread skills/running-headless-orchestrator/SKILL.md Outdated
Comment thread skills/shadcn/customization.md Outdated
Comment thread skills/building-nango-functions-locally/references/actions.md Outdated
Comment thread skills/shadcn/rules/styling.md Outdated
2. **Re-read the contract §5 + §6.** Confirm the flip command and rollback triggers.
3. **Post the pre-flip statement** to the operator's channel:
```
Pre-flip statement, <run-name>, <ISO-utc>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P3: This fenced announcement template needs a language identifier (e.g., text) to comply with MD040.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At skills/dormant-flip-and-rollback/SKILL.md, line 66:

<comment>This fenced announcement template needs a language identifier (e.g., `text`) to comply with MD040.</comment>

<file context>
@@ -0,0 +1,111 @@
+2. **Re-read the contract §5 + §6.** Confirm the flip command and rollback triggers.
+3. **Post the pre-flip statement** to the operator's channel:
+   ```
+   Pre-flip statement, <run-name>, <ISO-utc>
+   Gates: G1 GREEN <link>, G2 GREEN <link>, ... (full list)
+   Flip command: <verbatim>
</file context>

khaliqgant and others added 4 commits May 31, 2026 12:25
…trator, shadcn from consolidation

shadcn stays in the cloud repo (with its symlinks); the nango and
headless-orchestrator skills should not move into the shared package.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…oud's lock)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…oud repo)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

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="README.md">

<violation number="1" location="README.md:29">
P2: README advertises `openclaw-orchestrator` as a published skill, but it is not declared in `prpm.json` packages. This will mislead users about installable packages until either the manifest is updated or the README entry is removed.</violation>
</file>

Tip: Review your code locally with the cubic CLI to iterate faster.

Re-trigger cubic

Comment thread README.md
| [instrument-dont-guess](skills/instrument-dont-guess/SKILL.md) | 1.0.0 | Add temporary diagnostics after repeated failed fixes instead of continuing to guess at the same symptom. |
| [swarm-blockers-and-gate-scoreboard](skills/swarm-blockers-and-gate-scoreboard/SKILL.md) | 1.0.0 | Dispatch paired worker/reviewer agents for hard blockers and maintain RED/GREEN gate scoreboards for autonomous runs. |
| [tiered-acceptance](skills/tiered-acceptance/SKILL.md) | 1.0.0 | Split broad acceptance gates into deeply proven tier-1 coverage and explicitly accepted tier-2 smoke coverage. |
| [openclaw-orchestrator](skills/openclaw-orchestrator/SKILL.md) | 1.0.0 | Run headless multi-agent orchestration sessions via Agent Relay — spawn teams across Claude/Codex/Gemini/Pi/Droid, create channels, and manage agent lifecycle. |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2: README advertises openclaw-orchestrator as a published skill, but it is not declared in prpm.json packages. This will mislead users about installable packages until either the manifest is updated or the README entry is removed.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At README.md, line 29:

<comment>README advertises `openclaw-orchestrator` as a published skill, but it is not declared in `prpm.json` packages. This will mislead users about installable packages until either the manifest is updated or the README entry is removed.</comment>

<file context>
@@ -22,15 +22,11 @@ Package metadata lives in [prpm.json](prpm.json). The repo currently publishes `
-| [running-headless-orchestrator](skills/running-headless-orchestrator/SKILL.md) | 1.0.0 | Self-bootstrap Agent Relay and manage a team of workers without human intervention. |
-| [trigger-autocomplete-catalog](skills/trigger-autocomplete-catalog/SKILL.md) | 1.0.0 | Register webhook events in the trigger autocomplete catalog when integrations add provider events. |
-| [shadcn](skills/shadcn/SKILL.md) | 1.0.0 | Work with shadcn/ui projects, registries, presets, component composition, styling, CLI commands, and MCP helpers. |
+| [openclaw-orchestrator](skills/openclaw-orchestrator/SKILL.md) | 1.0.0 | Run headless multi-agent orchestration sessions via Agent Relay — spawn teams across Claude/Codex/Gemini/Pi/Droid, create channels, and manage agent lifecycle. |
 
 ## Slash Commands
</file context>

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@khaliqgant khaliqgant merged commit dea1129 into main May 31, 2026
2 checks passed
@khaliqgant khaliqgant deleted the chore/consolidate-skills-from-cloud branch May 31, 2026 12:50
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