Skip to content

ci: run the transfer report job on Blacksmith - #16178

Merged
juliusmarminge merged 1 commit into
mainfrom
ci/transfer-report-on-blacksmith
Oct 5, 2026
Merged

juliusmarminge merged 1 commit into
mainfrom
ci/transfer-report-on-blacksmith

Conversation

@juliusmarminge

@juliusmarminge juliusmarminge commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

Problem

main CI went red on 194c73f (#16138) and b12b3f7 (#16167) even though every lint, typecheck, build and test job passed. In both runs, Check failed only because transfer-report: abandoned.

transfer-report was the only CI job on a GitHub-hosted runner (ubuntu-24.04). That queue is shared with the PR bot workflows (PR Vouch, PR Size, Thread Transfer Report, Cursor hygiene), and over 130 runs were queued at the time. In both runs the job sat with no runner and zero steps for about 15 minutes until GitHub abandoned it. Later runs passed the same job in seconds.

Fix

Run transfer-report on blacksmith-2vcpu-ubuntu-2404 like every other CI job, with a 5 minute timeout. The job only downloads one artifact.

Verification

This PR's own CI run exercises the job on the new runner.

Model: Claude Opus 5.5 (1M context). Harness: Claude Code in T3 Code.

🤖 Generated with Claude Code


Devin Review

The transfer-report job was the only CI job on a GitHub-hosted runner. When
that queue is backed up by the PR bot workflows, the job never gets a runner,
is abandoned after ~15 minutes, and fails Check on an otherwise green run.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@macroscopeapp

macroscopeapp Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 7b40f25

Macroscope's review found this PR approvable — This is a small CI-only change that moves an existing artifact-report job to the same Blacksmith runner family already used elsewhere and adds a bounded timeout. It does not affect product behavior, defaults, deployment, or static-analysis settings.

You can add or adjust custom eligibility rules. Learn more.

@coderabbitai

coderabbitai Bot commented Oct 5, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Team
  • Run ID: ae5bbe93-b840-4c3f-98af-c86e70049db3
📥 Commits

Reviewing files that changed from the base of the PR and between 5b613de and 7b40f25.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 6 remain after this review.


📝 Walkthrough

Walkthrough

The transfer-report CI job now uses a Blacksmith Ubuntu 24.04 runner and has a five-minute timeout.

Changes

Transfer report CI runner

Layer / File(s) Summary
Configure the transfer-report runner
.github/workflows/ci.yml
The job now uses blacksmith-2vcpu-ubuntu-2404 instead of ubuntu-24.04. It also has a five-minute timeout.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

Suggested reviewers: t3dotgg

Merge Risk: ⚪ Minimal · up to 7b40f

No concrete issue with the transfer-report runner change remains; merge after normal CI checks.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: moving the transfer-report job to Blacksmith.
Description check ✅ Passed The description explains the CI problem, the runner change, and the reported verification. It is mostly complete, but it does not include the template’s Scope and approval section or explain why this …
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@juliusmarminge
juliusmarminge merged commit 2bffcc5 into main Oct 5, 2026
24 of 27 checks passed
@juliusmarminge
juliusmarminge deleted the ci/transfer-report-on-blacksmith branch October 5, 2026 20:47
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Oct 6, 2026
## What's Changed
* chore(deps): upgrade Effect to stable 4.0.1 by @juliusmarminge in pingdotgg/t3code#16138
* fix(server): worktree threads survive a local branch named t3code by @juliusmarminge in pingdotgg/t3code#16167
* chore(refs): sync Effect and Alchemy references to 4.0.1 and beta.80 by @juliusmarminge in pingdotgg/t3code#16170
* fix(web): match subagent timestamp fonts to chat by @StiensWout in pingdotgg/t3code#16151
* fix(web): wrap full status text in composer hover details by @UtkarshUsername in pingdotgg/t3code#16158
* ci: run the transfer report job on Blacksmith by @juliusmarminge in pingdotgg/t3code#16178
* fix(server): Stop also stops delegated tasks and pull request watches by @t3dotgg in pingdotgg/t3code#16002
* feat: native /goal for Codex and Claude, with goal status in the UI by @t3dotgg in pingdotgg/t3code#15592
* fix(server): name the cause of a failed git command by @walid-baharwal in pingdotgg/t3code#8645
* fix(server): PR watch wakes the agent when a bot edits its review comment by @Gigioxx in pingdotgg/t3code#15415
* feat(source-control): omit agent credits from PR merge messages by @juliusmarminge in pingdotgg/t3code#16192
* fix(clients): dropped connections say why in the client trace by @t3dotgg in pingdotgg/t3code#16200
* fix(server): PR watch reports a required check that first appears already passed by @ScottN-PV in pingdotgg/t3code#15804
* fix: a failed DPoP key load and a fresh maintenance read are no longer cached by @juliusmarminge in pingdotgg/t3code#15500
* fix: tool screenshots show as images, not base64 text by @t3dotgg in pingdotgg/t3code#16199
* docs(mcp): thread tools reach threads in any project by @t3dotgg in pingdotgg/t3code#15947
* fix(threads): threads watching a PR stay in Working instead of bouncing to the inbox by @t3dotgg in pingdotgg/t3code#16204
* chore(deps): bump cursor sdk and astro to clear vulnerable transitives by @juliusmarminge in pingdotgg/t3code#16214
* fix(server): PR sync waits out a GitHub rate limit pause instead of failing every PR by @t3dotgg in pingdotgg/t3code#16203
* feat(server): scheduled tasks can run on a webhook by @juliusmarminge in pingdotgg/t3code#15085
* feat(relay): forward webhook requests to the environment's tunnel by @juliusmarminge in pingdotgg/t3code#15086
* feat(mobile): create and copy webhook automations by @juliusmarminge in pingdotgg/t3code#15087
* feat(web): create webhook automations and inspect their deliveries by @juliusmarminge in pingdotgg/t3code#15088
* feat(relay,server,web,mobile): opt-in to hold webhooks while offline by @juliusmarminge in pingdotgg/t3code#15487
* fix(server): PR watches stop burning GitHub's rate limit and giving up by @t3dotgg in pingdotgg/t3code#16208
* fix(server): delegation sees a fixed provider without the app open by @t3dotgg in pingdotgg/t3code#16219
* feat: new branches use the shorter t3/ prefix by @t3dotgg in pingdotgg/t3code#16220
* perf: cheaper shell refreshes, one copy of Codex streaming text, no MCP wait polling by @t3dotgg in pingdotgg/t3code#15033
* feat: agents can show HTML pages inline in threads by @t3dotgg in pingdotgg/t3code#15968
* chore(relay): match Alchemy to the PS-80 Postgres cluster by @juliusmarminge in pingdotgg/t3code#16228
* feat: agents ask the user for a secret through a private card by @juliusmarminge in pingdotgg/t3code#15907
* feat(web): see and stop pull request watches in the thread details card by @t3dotgg in pingdotgg/t3code#16235
* fix(server): ACP mode states with null descriptions are no longer dropped by @juliusmarminge in pingdotgg/t3code#16218
* fix(server): finished outbox rows and old PR cache files are pruned by @juliusmarminge in pingdotgg/t3code#16247
* fix(relay): releasing a tunnel that still has a connector no longer 500s by @juliusmarminge in pingdotgg/t3code#16250

## New Contributors
* @ScottN-PV made their first contribution in pingdotgg/t3code#15804

**Full Changelog**: pingdotgg/t3code@v0.0.46-nightly.20261005.2689...v0.0.46-nightly.20261005.2702

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.46-nightly.20261005.2702
github-actions Bot added a commit to davidvanderklay/t3code-flake that referenced this pull request Oct 6, 2026
## What's Changed
* chore(deps): upgrade Effect to stable 4.0.1 by @juliusmarminge in pingdotgg/t3code#16138
* fix(server): worktree threads survive a local branch named t3code by @juliusmarminge in pingdotgg/t3code#16167
* chore(refs): sync Effect and Alchemy references to 4.0.1 and beta.80 by @juliusmarminge in pingdotgg/t3code#16170
* fix(web): match subagent timestamp fonts to chat by @StiensWout in pingdotgg/t3code#16151
* fix(web): wrap full status text in composer hover details by @UtkarshUsername in pingdotgg/t3code#16158
* ci: run the transfer report job on Blacksmith by @juliusmarminge in pingdotgg/t3code#16178
* fix(server): Stop also stops delegated tasks and pull request watches by @t3dotgg in pingdotgg/t3code#16002
* feat: native /goal for Codex and Claude, with goal status in the UI by @t3dotgg in pingdotgg/t3code#15592
* fix(server): name the cause of a failed git command by @walid-baharwal in pingdotgg/t3code#8645
* fix(server): PR watch wakes the agent when a bot edits its review comment by @Gigioxx in pingdotgg/t3code#15415
* feat(source-control): omit agent credits from PR merge messages by @juliusmarminge in pingdotgg/t3code#16192
* fix(clients): dropped connections say why in the client trace by @t3dotgg in pingdotgg/t3code#16200
* fix(server): PR watch reports a required check that first appears already passed by @ScottN-PV in pingdotgg/t3code#15804
* fix: a failed DPoP key load and a fresh maintenance read are no longer cached by @juliusmarminge in pingdotgg/t3code#15500
* fix: tool screenshots show as images, not base64 text by @t3dotgg in pingdotgg/t3code#16199
* docs(mcp): thread tools reach threads in any project by @t3dotgg in pingdotgg/t3code#15947
* fix(threads): threads watching a PR stay in Working instead of bouncing to the inbox by @t3dotgg in pingdotgg/t3code#16204
* chore(deps): bump cursor sdk and astro to clear vulnerable transitives by @juliusmarminge in pingdotgg/t3code#16214
* fix(server): PR sync waits out a GitHub rate limit pause instead of failing every PR by @t3dotgg in pingdotgg/t3code#16203
* feat(server): scheduled tasks can run on a webhook by @juliusmarminge in pingdotgg/t3code#15085
* feat(relay): forward webhook requests to the environment's tunnel by @juliusmarminge in pingdotgg/t3code#15086
* feat(mobile): create and copy webhook automations by @juliusmarminge in pingdotgg/t3code#15087
* feat(web): create webhook automations and inspect their deliveries by @juliusmarminge in pingdotgg/t3code#15088
* feat(relay,server,web,mobile): opt-in to hold webhooks while offline by @juliusmarminge in pingdotgg/t3code#15487
* fix(server): PR watches stop burning GitHub's rate limit and giving up by @t3dotgg in pingdotgg/t3code#16208
* fix(server): delegation sees a fixed provider without the app open by @t3dotgg in pingdotgg/t3code#16219
* feat: new branches use the shorter t3/ prefix by @t3dotgg in pingdotgg/t3code#16220
* perf: cheaper shell refreshes, one copy of Codex streaming text, no MCP wait polling by @t3dotgg in pingdotgg/t3code#15033
* feat: agents can show HTML pages inline in threads by @t3dotgg in pingdotgg/t3code#15968
* chore(relay): match Alchemy to the PS-80 Postgres cluster by @juliusmarminge in pingdotgg/t3code#16228
* feat: agents ask the user for a secret through a private card by @juliusmarminge in pingdotgg/t3code#15907
* feat(web): see and stop pull request watches in the thread details card by @t3dotgg in pingdotgg/t3code#16235
* fix(server): ACP mode states with null descriptions are no longer dropped by @juliusmarminge in pingdotgg/t3code#16218
* fix(server): finished outbox rows and old PR cache files are pruned by @juliusmarminge in pingdotgg/t3code#16247
* fix(relay): releasing a tunnel that still has a connector no longer 500s by @juliusmarminge in pingdotgg/t3code#16250

## New Contributors
* @ScottN-PV made their first contribution in pingdotgg/t3code#15804

**Full Changelog**: pingdotgg/t3code@v0.0.46-nightly.20261005.2689...v0.0.46-nightly.20261005.2702

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.46-nightly.20261005.2702
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