From 49ef910e52cd18df65e5aa423d91916a319f0d98 Mon Sep 17 00:00:00 2001 From: sudacode Date: Sun, 5 Apr 2026 10:56:14 -0700 Subject: [PATCH] docs: update task record for review follow-up --- ...Fix-Linux-AppImage-child-process-libffmpeg-resolution.md | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/backlog/tasks/task-279 - Fix-Linux-AppImage-child-process-libffmpeg-resolution.md b/backlog/tasks/task-279 - Fix-Linux-AppImage-child-process-libffmpeg-resolution.md index e4481d48..99255af4 100644 --- a/backlog/tasks/task-279 - Fix-Linux-AppImage-child-process-libffmpeg-resolution.md +++ b/backlog/tasks/task-279 - Fix-Linux-AppImage-child-process-libffmpeg-resolution.md @@ -1,11 +1,11 @@ --- id: TASK-279 title: Fix Linux AppImage child-process libffmpeg resolution -status: In Progress +status: Done assignee: - '@codex' created_date: '2026-04-05 17:17' -updated_date: '2026-04-05 17:55' +updated_date: '2026-04-05 17:56' labels: [] dependencies: [] references: @@ -57,4 +57,6 @@ Added a shared electron-builder `afterPack` hook at `scripts/electron-builder-af Regression coverage now checks both the packaging config and the hook behavior: `src/release-workflow.test.ts` asserts the hook stays wired into release config, and `scripts/electron-builder-after-pack.test.ts` verifies Linux copies `libffmpeg.so` into `usr/lib` while non-Linux and missing-library cases no-op safely. The new script test is included in `test:fast`, and a changelog fragment was added under `changes/fix-appimage-libffmpeg-path.md`. Verification passed with `bun test scripts/electron-builder-after-pack.test.ts src/release-workflow.test.ts`, `bun run typecheck`, `bun run test:fast`, `bun run test:env`, `bun run build`, and `bun run test:smoke:dist`. + +Follow-up review fix on PR #45: Linux packaging now throws when `libffmpeg.so` is missing from the packaged app root, preventing silent shipment of a broken AppImage. Focused regression coverage was updated so the missing-library case rejects and `afterPack` propagates the failure.