From 0a46bb34143073ecf3c99e6e5b09754360a01d75 Mon Sep 17 00:00:00 2001 From: Chinesezjc Date: Tue, 18 Aug 2026 17:44:26 +0800 Subject: [PATCH] style(code-runtime-python): keep the teardown v8-ignore under the line limit The directive carried its whole justification inline at 203 characters, past the 140 the @stylistic/max-len rule allows (imports and template-literal messages are exempt; a line comment is not). The reasoning moves to the lines above and the directive keeps a short pointer, since a v8 ignore must stay on one line. --- packages/code-runtime/code-runtime-python/src/index.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/packages/code-runtime/code-runtime-python/src/index.ts b/packages/code-runtime/code-runtime-python/src/index.ts index 7f0bac4dd0..c8f0e15647 100644 --- a/packages/code-runtime/code-runtime-python/src/index.ts +++ b/packages/code-runtime/code-runtime-python/src/index.ts @@ -1428,7 +1428,10 @@ export class PythonCodeRuntime extends CodeRuntime { // falls through, which is also the behavior on platforms with no // `/proc` to read. const nowStarted = readProcessStart(child.pid) - /* v8 ignore next -- the refusal arm needs a real pid recycled into a new group leader between spawn and teardown, which no test can schedule; `readProcessStart` is covered directly instead. */ + // The refusal arm needs a real pid recycled into a new group leader + // between spawn and teardown, which no test can schedule; the reader + // itself is covered directly by the process-identity test. + /* v8 ignore next -- unreachable without real pid reuse; see above. */ if (leaderStarted !== undefined && nowStarted !== undefined && nowStarted !== leaderStarted) return process.kill(-child.pid, sig) } catch {