open/linkedin-studio
1
0
Fork 0
Code Issues Pull requests Releases Activity

fix(linkedin-studio): N18 Del 1 — opprydding velter aldri et vellykket figur-render [skip-docs]

Browse source 
Funnet av N18-roundtrip-røyken: render-suiten var grønn 1 av 4 kjøringer, ikke
"grønn siden N13" som STATE påsto. Rotårsaken er produksjonskode, ikke en test-flake.

runChromeScreenshot.finish() sender SIGKILL til forelder-Chrome og resolver
umiddelbart uten å vente på at prosessen dør. SIGKILL på forelderen dreper ikke
dens renderer/GPU-barn, som fortsetter å skrive inn i --user-data-dir under
workDir mens finally-blokken traverserer samme katalog. rmSync tømmer den, nye
filer dukker opp, og fjerning av selve katalogen feiler med ENOTEMPTY. Ren
timing-race — derav de ~50 %.

Brukerkonsekvensen var den verste sorten: PNG-en skrives til outPath UTENFOR
workDir og er allerede ferdig når dette skjer, så oppryddingen veltet et render
som faktisk lyktes.

Fix: cleanupWorkDir() — eksplisitt retry-løkke (testbar med injisert rm) pluss
fs sine egne maxRetries/retryDelay, og den kaster aldri. En temp-katalog som blir
liggende er en bagatell mot å miste et ferdig bilde; OS rydder tmpdir.

TDD: rødt bevist først (manglende eksport), så 3 tester på kontrakten — svelger
ENOTEMPTY og forsøker på nytt, fjerner katalogen i normaltilfellet, gir seg stille
når den allerede er borte.

render 60/0 -> 63/0, grønn 5 av 5 kjøringer (var 1 av 4). test-runner 270/0.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QxvWAjte7vPcF79QeSRvRJ
This commit is contained in:
Kjell Tore Guttormsen 2026-07-26 17:19:01 +02:00
commit e0f964ad60
2 changed files with 44 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

24
render/__tests__/build-figur.test.mjs
View file

@ -18,6 +18,7 @@ import {
resolveChrome,
parseArgs,
renderFigure,
cleanupWorkDir,
} from '../build-figur.mjs';
const __dirname = path.dirname(fileURLToPath(import.meta.url));
@ -282,3 +283,26 @@ describe('renderFigure — ende-til-ende via headless Chrome', { skip: !chrome.a
assert.equal(dim.height, 1350);
});
});
describe('cleanupWorkDir — opprydding velter aldri et vellykket render', () => {
test('svelger ENOTEMPTY fra en katalog Chrome fortsatt skriver til', () => {
const err = Object.assign(new Error('ENOTEMPTY, Directory not empty'), { code: 'ENOTEMPTY' });
let calls = 0;
const rm = () => { calls += 1; throw err; };
assert.doesNotThrow(() => cleanupWorkDir('/tmp/build-figur-fake', rm));
assert.ok(calls > 1, `skal forsoeke paa nytt foer den gir opp (kall: ${calls})`);
});
test('fjerner katalogen i normaltilfellet', () => {
const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'cleanup-ok-'));
fs.writeFileSync(path.join(dir, 'figur.html'), '<html></html>', 'utf8');
cleanupWorkDir(dir);
assert.equal(fs.existsSync(dir), false);
});
test('gir seg uten aa kaste naar katalogen allerede er borte', () => {
const dir = fs.mkdtempSync(path.join(os.tmpdir(), 'cleanup-gone-'));
fs.rmSync(dir, { recursive: true, force: true });
assert.doesNotThrow(() => cleanupWorkDir(dir));
});
});

21
render/build-figur.mjs
View file

@ -254,6 +254,25 @@ function runChromeScreenshot(chromePath, args, timeoutMs = RENDER_TIMEOUT_MS) {
});
}
// Chrome kan ha gjenlevende barneprosesser (renderer/GPU) som fortsatt skriver til
// --user-data-dir når vi rydder: SIGKILL på forelderen dreper ikke barna, så
// katalogen kan fylles på nytt midt i traverseringen og gi ENOTEMPTY. PNG-en ligger
// utenfor workDir og er allerede skrevet på dette punktet, så en temp-katalog som
// blir liggende er en bagatell mot å velte et vellykket render.
const CLEANUP_ATTEMPTS = 3;
export function cleanupWorkDir(dir, rm = fs.rmSync) {
for (let attempt = 0; attempt < CLEANUP_ATTEMPTS; attempt += 1) {
try {
rm(dir, { recursive: true, force: true, maxRetries: 5, retryDelay: 50 });
return;
} catch {
// Kan være rester fra Chrome; prøv igjen.
}
}
// Ga opp — OS rydder tmpdir. Denne funksjonen kaster aldri.
}
export async function renderFigure(opts) {
const { input, target = "article", out = null, dataDir, chrome = null } = opts;
const inPath = path.isAbsolute(input) ? input : path.join(process.cwd(), input);
@ -308,7 +327,7 @@ export async function renderFigure(opts) {
throw new Error(`Chrome skrev ikke ${outPath} (${run.reason}).\n${tail}`);
}
} finally {
fs.rmSync(workDir, { recursive: true, force: true });
cleanupWorkDir(workDir);
}
return { out: outPath, width: dims.width, height: dims.height, warnings, tokensSource };