Skip to content

chore: update fullsend shim workflow#2052

Merged
rh-hemartin merged 1 commit into
mainfrom
fullsend/onboard
Jun 9, 2026
Merged

chore: update fullsend shim workflow#2052
rh-hemartin merged 1 commit into
mainfrom
fullsend/onboard

Conversation

@fullsend-ai-fullsend

Copy link
Copy Markdown
Contributor

This PR updates the fullsend shim workflow to match the current template in the .fullsend config repo.

The shim content has drifted from the template — this brings it back in sync.

Update the shim workflow to match the current template
in the .fullsend config repo.
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Site preview

Preview: https://d0dcdad5-site.fullsend-ai.workers.dev

Commit: 4a92eaacb3e4213c8681d7d1e419fa3d6ae830f2

@fullsend-ai-review

fullsend-ai-review Bot commented Jun 9, 2026

Copy link
Copy Markdown

🤖 Finished Review · ❌ Failure · Started 6:48 AM UTC · Completed 6:48 AM UTC
Commit: 9bd7234 · View workflow run →

@rh-hemartin rh-hemartin enabled auto-merge June 9, 2026 08:36
@rh-hemartin rh-hemartin added this pull request to the merge queue Jun 9, 2026
Merged via the queue into main with commit 77771b4 Jun 9, 2026
9 checks passed
@rh-hemartin rh-hemartin deleted the fullsend/onboard branch June 9, 2026 08:39
@fullsend-ai-retro

fullsend-ai-retro Bot commented Jun 9, 2026

Copy link
Copy Markdown

🤖 Finished Retro · ✅ Success · Started 8:42 AM UTC · Completed 8:47 AM UTC
Commit: ba204cb · View workflow run →

@fullsend-ai-retro

Copy link
Copy Markdown

Retro: PR #2052chore: update fullsend shim workflow

This was a trivial 2-line automated onboard PR by fullsend-ai-fullsend[bot] that added a YAML document marker and managed-content comment to .github/workflows/fullsend.yaml. The review agent failed due to authentication — Claude Code CLI reported apiKeySource: "none" and exited immediately with authentication_failed. The harness retried once with the identical result. A human (rh-hemartin) manually approved and merged ~2 hours later.

Existing issues that cover most of the waste here

  • #1371 — Skip review dispatch for bot-authored PRs (would have prevented the review run entirely)
  • #1630 — Skip retro dispatch for bot-authored PRs with review-only agent interaction (would have prevented this retro)
  • #1032 — Pre-flight Vertex AI auth check (would catch auth issues before spinning up the sandbox)

One new proposal filed

The harness should classify agent exit reasons and not retry on authentication failures, which are infrastructure issues that won't self-resolve within the same run.

Proposals filed

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