feat(workflows): add 065 post-auth mount session workflow#124
feat(workflows): add 065 post-auth mount session workflow#124khaliqgant merged 1 commit intomainfrom
Conversation
Add the supervisor-pattern workflow that orchestrates the cross-repo post-auth mount-session implementation across Cloud (#498) and Relayfile, covering contract lock, parallel impl, evidence capture, self/peer review, and PR creation. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ 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: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| 'BRANCH=$(git rev-parse --abbrev-ref HEAD)', | ||
| 'git push -u origin "$BRANCH"', | ||
| 'BODY=$(mktemp)', | ||
| 'cat > "$BODY" <<EOF', |
There was a problem hiding this comment.
🔴 Unquoted heredoc <<EOF causes backtick content to be executed as shell commands, garbling PR bodies
Both push-and-open-relayfile-pr (line 875) and push-and-open-cloud-pr (line 912) use <<EOF (unquoted delimiter) to write PR body content. In bash, an unquoted heredoc delimiter causes variable expansion and command substitution. The PR body text contains markdown backtick-fenced terms like `mountWorkspace`, `ensureMountedWorkspace`, `relayfile-mount`, `MountedWorkspaceHandle`, `POST /api/v1/workspaces/:workspaceId/relayfile/mount-session`, `expiresAt`, etc. Bash interprets these backtick pairs as command substitutions, attempts to execute the enclosed text as commands (which fail with "command not found"), and replaces each backtick expression with empty output. The resulting PR bodies will have missing/garbled text (e.g., - add the Relayfile SDK post-auth / flow instead of the intended content). The fix is to quote the heredoc delimiter: <<'EOF'.
| 'cat > "$BODY" <<EOF', | |
| 'cat > "$BODY" <<\'EOF\'', |
Was this helpful? React with 👍 or 👎 to provide feedback.
| 'BRANCH=$(git rev-parse --abbrev-ref HEAD)', | ||
| 'git push -u origin "$BRANCH"', | ||
| 'BODY=$(mktemp)', | ||
| 'cat > "$BODY" <<EOF', |
There was a problem hiding this comment.
🔴 Unquoted heredoc <<EOF in cloud PR body causes same backtick command substitution issue
Same root cause as the relayfile PR body: push-and-open-cloud-pr uses <<EOF (unquoted) on line 912, causing backtick content like `POST /api/v1/workspaces/:workspaceId/relayfile/mount-session` and `expiresAt` to be interpreted as shell command substitutions. This garbles the cloud PR body text created via gh pr create --body-file.
| 'cat > "$BODY" <<EOF', | |
| 'cat > "$BODY" <<\'EOF\'', |
Was this helpful? React with 👍 or 👎 to provide feedback.
1 participant
Summary
workflows/065-post-auth-mount-session.ts, the supervisor-pattern agent-relay workflow that drives the cross-repo post-auth mount-session implementation across Cloud (AgentWorkforce/cloud#498) and Relayfile.gh.Test plan
agent-relay run workflows/065-post-auth-mount-session.tsexecutes the workflow end-to-end against a fresh worktreeworkflows/tsconfig🤖 Generated with Claude Code