Skip to content

Commit a6ee02e

Browse files
authored
fix(review): remove eager media tool bootstrap (#318)
1 parent 30163df commit a6ee02e

4 files changed

Lines changed: 11 additions & 28 deletions

File tree

.github/actions/setup-media-proof-tools/action.yml

Lines changed: 0 additions & 24 deletions
This file was deleted.

.github/workflows/sweep.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -389,8 +389,6 @@ jobs:
389389
}
390390
react "eyes"
391391
392-
- uses: ./.github/actions/setup-media-proof-tools
393-
394392
- uses: actions/cache@v5
395393
with:
396394
path: ${{ steps.target.outputs.target_checkout_dir }}-cache.git
@@ -1219,8 +1217,6 @@ jobs:
12191217
with:
12201218
login-status: "true"
12211219

1222-
- uses: ./clawsweeper/.github/actions/setup-media-proof-tools
1223-
12241220
- uses: actions/cache@v5
12251221
with:
12261222
path: ${{ needs.plan.outputs.target_checkout_dir }}-cache.git

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ checkpoint, and status-only commits are intentionally omitted.
1313

1414
### Fixed
1515

16+
- Removed unconditional ffmpeg provisioning from review startup so optional media proof cannot block exact-review leases; unavailable media tools remain per-item evidence failures.
1617
- Prevented contributor-branch repairs and changelog-free repair artifacts from adding release-owned changelog entries, keeping contributor credit and release-note context in PR bodies or commit history instead.
1718
- Added an explicit trusted ephemeral-runner fallback for repair planning when the host cannot start Codex's Linux read-only sandbox.
1819
- Replaced runner-side exact-review capacity waiting and self-retries with a durable 32-slot Worker queue that coalesces item deliveries, leases executors before checkout, and reclaims abandoned leases.

test/sweep-workflow.test.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
import assert from "node:assert/strict";
2+
import fs from "node:fs";
3+
import path from "node:path";
4+
import test from "node:test";
5+
6+
test("sweep keeps optional media tooling out of review startup", () => {
7+
const workflow = fs.readFileSync(path.join(process.cwd(), ".github/workflows/sweep.yml"), "utf8");
8+
9+
assert.doesNotMatch(workflow, /setup-media-proof-tools/);
10+
});

0 commit comments

Comments
 (0)