Repository navigation
feat(signup): render agent signup guides from the vendored setup skills - #159
Conversation
…skills - /signup/agent/teams and /signup/agent/flows keep only the signup header (policy, progress, web input) and include the AgentWorkforce/skills setup skills verbatim from a pinned commit (web/scripts/sync-setup-skills.mjs, --check fails on drift). - Flows parts: Cloud account, Flows, and writing-relayflows (v2) for the custom-flow path; the web-input example asks prebuilt or custom instead of listing code-review, which is not in the catalog. - Name the flow Software Garden in the /flows examples, the onboarding handoff, the builder copy, the local START-HERE and the cookbook. The id stays software-factory. - TODO in flow-workflows.ts: align WORKFLOWS step lists with the cloud deploy wizard once it reports its final step names. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 49 seconds. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configuration
⛔ Files ignored due to path filters (2)
📒 Files selected for processing (11)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
⛔ Files ignored due to path filters (1)
📒 Files selected for processing (17)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThis pull request adds vendored Agent Relay setup skills and uses them to generate signup guidance. It documents Desktop, Sessions, and Flows setup, updates signup progress checks, and renames Software Factory references to Software Garden. ChangesAgent Relay setup guides and signup
Priority: ➖ Normal Estimated code review effort: 4 (Complex) | ~60 minutes Change: Feature Merge Risk: ⚪ Minimal · up to No demonstrated issue blocks merging; the vendored pin and signup behavior remain subject to normal validation. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 21.43% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 14 functions across 10 files. (7 skipped: 7 unsupported.) ✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 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. A rabbit reads the setup guide by moonlit garden light. Comment |
The vendored writing-relayflows may name it only to warn against the v1 engine; no part before it mentions it. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7d69310d21
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Preview deployed!
This is a Cloudflare Workers preview version of this PR's build. |
Picks up the review fixes in skills#145 (kept sharing mode, socket timeouts, base-relative URLs); tests follow the new text. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
All reported issues were addressed
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
All reported issues were addressed across 18 files
Requires human review: Auto-approval blocked because this review re-detected 5 unresolved issues already reported by Cubic.
View guided diff | Turn on auto-fix | Re-trigger cubic
- Flows progress steps map to the Flows skill's sections 3-5 (connect, activate, verify), so completion follows the listener check. - Local Teams guides rewrite Part 2's desktop commands for the prebuilt Dev app (download, app name, bundle id, socket pointer); Linux has no local artifact. - sync-setup-skills --check fails on, and sync removes, vendored copies SOURCE.json no longer lists. - Teams step copy qualifies the new-sessions default; the v2 test also rejects side-effect and dynamic imports of @relayflows/core. - Re-pin to AgentWorkforce/skills@d2e5a3e. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ions-handoff # Conflicts: # web/lib/agent-signup.ts
…al stack An HTTPS preview or staging host keeps the release app; only the plain-HTTP local stack serves the Dev DMG. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
All reported issues were addressed across 2 files (changes from recent commits).
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
…tack The Dev-app rewrite and the local note now share one check, so an HTTP staging or self-hosted guide keeps the release app. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
… sync - prpm.lock pins the @agent-relay setup skills; prpm install writes them to .claude/skills/<name>/SKILL.md, as in the relay repo. - web/scripts/generate-setup-skills.mjs reads the lock and the installed files and generates web/lib/generated/setup-skills.ts (prebuild and precf:build run it); --check fails when the module drifts. - CI reinstalls with prpm install --frozen-lockfile and fails if that changes prpm.lock or .claude/skills, then runs the check. - Removes web/scripts/sync-setup-skills.mjs and web/content/setup-skills. Blocked on publish: signing-in-to-agent-relay-cloud and setting-up-agent-relay-flows are not on the registry yet, and desktop 1.1.5 / sessions 0.2.5 are unpublished. Once AgentWorkforce/skills#145 merges and they publish, run prpm install for the two new packages, prpm update, and npm run skills:generate. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 4 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 36d728d. Configure here.
prpm install signing-in-to-agent-relay-cloud 0.1.0 and setting-up-agent-relay-flows 0.1.0; prpm update moves desktop to 1.1.5 and sessions to 0.2.5 (AgentWorkforce/skills@7d727ee). Regenerated the module. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
All reported issues were addressed across 14 files (changes from recent commits).
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 435d290369
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
All reported issues were addressed across 21 files
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
…drift check - The local-stack rewrite also renames the quoted "Agent Relay" display name (open -a, tell application) to "Agent Relay Dev". - stripFrontmatter accepts CRLF line endings. - CI diffs .claude/skills strictly and ignores only the lock's timestamp. - prepreview regenerates the module too. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
1 participant
Summary
/signup/agent/teamsand/signup/agent/flowskeep only the signup header (policy, User-Agent, progress, web input). After it, they include the AgentWorkforce/skills setup skills verbatim, installed with prpm:prpm.lockpins the@agent-relaypackages,prpm installwrites them to.claude/skills/<name>/SKILL.md(as in the relay repo), andweb/scripts/generate-setup-skills.mjsgeneratesweb/lib/generated/setup-skills.tsfrom them. The generator also runs inprebuildandprecf:build.prpm install --frozen-lockfilemust not changeprpm.lockor.claude/skills, andnpm run verify:setup-skillsmust pass.writing-relayflows(v2) for the custom path. Progress steps map to the Flows skill's sections 3-5. The web-input example asks "prebuilt or custom" (nocode-review).software-factory.Blocked: draft until the skills are published
signing-in-to-agent-relay-cloudandsetting-up-agent-relay-flowsare not on the prpm registry yet, and desktop 1.1.5 / sessions 0.2.5 (from AgentWorkforce/skills#145) are unpublished. So CI fails at "Verify setup skills are current". After #145 merges and a human publishes:I checked this locally by simulating the five installed files from the #145 branch: the generator,
--check, and every signup test pass.Open
web/lib/flow-workflows.ts: match WORKFLOWS run steps to cloud's run-step preview (flow-preview-overview.ts) once cloud reports the names.🤖 Generated with Claude Code
Note
Medium Risk
Changes the authoritative agent onboarding path and Teams completion criteria; CI is blocked until unpublished prpm packages are available, but runtime risk is mainly guide drift rather than core API behavior.
Overview
Agent-driven Teams and Flows signup no longer ship a single monolithic guide in
agent-signup.ts. They keep a signup-specific header (policy, progress API, web-input for Flows) and append the canonical@agent-relaysetup skills verbatim, versioned viaprpm.lockand installed under.claude/skills/.A new
generate-setup-skills.mjspipeline strips skill frontmatter and emitsweb/lib/generated/setup-skills.tsso Cloudflare Workers can serve the same text without a filesystem;prebuild/precf:buildand CI (prpm install --frozen-lockfile,verify:setup-skills) enforce that committed skills, lockfile, and generated module stay aligned.Teams onboarding now walks cloud sign-in → desktop setup → session handoff (progress steps and copy updated; completion requires a live round trip, not just app install). Flows uses sign-in → flows deploy API guide →
writing-relayflowsfor custom flows. Local localhost stacks rewrite desktop download/socket paths to the Agent Relay Dev app.Product copy renames the recommended flow to Software Garden (catalog id remains
software-factory) across onboarding, examples, and docs.Reviewed by Cursor Bugbot for commit 298b908. Bugbot is set up for automated code reviews on this repo. Configure here.
Agent Relay sessions
claudesessiona295461e· opened viagh pr create· last active 2026-10-08