Repository navigation
chore(signup): setup skills — desktop 1.1.9, sessions 0.2.8 - #172
Conversation
|
@codex review |
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. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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 |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
|
Codex Review: Didn't find any major issues. 🎉 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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. |
![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 across 4 files
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
prpm update moves two pins (skills#147, #150); npm run skills:generate regenerates web/lib/generated/setup-skills.ts and verify:setup-skills passes. - desktop 1.1.9: a bounded sign-in poll (stops after three unreadable status reads, at once when sign-in is no longer in progress, 5 s default interval), a shell-state note, and fleet subscriptions that use the installed CLI (one find_relay_cli for sections 2 and 6: user-level npm, mise or nvm, then PATH, never the .deb launcher) instead of npx. - sessions 0.2.8: the optional CLI checks run only for agent-relay 13.x, without clearing the CLI path the desktop skill uses. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Session-Id: 26d00c25-77c9-401f-b9f0-51ce6c9deda6
a19cde5 to
dcc56d0
Compare
khaliqgant
changed the title
|
@codex review |
|
Codex Review: Didn't find any major issues. 🚀 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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 2 reviewed files. 2 files intentionally excluded from review.
Reply with feedback, questions, or to request a fix.
View guided diff | Turn on auto-fix | Re-trigger cubic
1 participant
Follow-up to #167. Picks up AgentWorkforce/skills#147 and #150 in the vendored setup skills that render /signup/agent/teams. Rebuilt on current main.
@agent-relay/setting-up-agent-relay-desktop@agent-relay/setting-up-agent-relay-sessionssigned_out/not_signed_in. Default interval is 5 s.npx. Onefind_relay_cliserves sections 2 and 6: user-level npm, then mise/nvm, then PATH, never the.deblauncher.Verification (local):
npm run skills:generate;npm run verify:setup-skillspasses (desktop@1.1.9, sessions@0.2.8).prpm install --frozen-lockfilechanges nothing.webvitest: 44 files / 517 tests pass.🤖 Generated with Claude Code
Note
Low Risk
Documentation and vendored skill lockfile updates only; no application runtime or auth logic changes in this repo.
Overview
Bumps vendored Agent Relay setup skills used on
/signup/agent/teams:@agent-relay/setting-up-agent-relay-desktop1.1.7 → 1.1.9 and@agent-relay/setting-up-agent-relay-sessions0.2.6 → 0.2.8, with matchingprpm.lockentries.Desktop 1.1.9 tightens agent-driven setup scripts: documents that shell variables do not persist across tool calls (re-run socket discovery; don’t re-send sign-in just to restore
sign_in); adds a guard before statuscurl; bounds sign-in polling (fail fast onsigned_out/not_signed_in, stop after three unreadable states, 5s default interval whensign_inis missing); centralizes CLI discovery infind_relay_cli(sections 2 and 6) so fleet GitHub subscribe prefers an installed user-level CLI overnpx.Sessions 0.2.8 runs optional preflight
agent-relayCLI checks only for 13.x, while still keepingrelay_cliset for the desktop skill’s own version handling.Reviewed by Cursor Bugbot for commit dcc56d0. Bugbot is set up for automated code reviews on this repo. Configure here.
Agent Relay sessions
claudesession26d00c25· opened viagh pr create· last active 2026-10-08