From 46ad90329249040574611a61c5b17886fe4295be Mon Sep 17 00:00:00 2001 From: Claude Date: Tue, 8 Sep 2026 15:47:00 +0000 Subject: [PATCH] docs(platform-objects): correct the vitest config's trailing effect claim MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The trailing comment opened "No `test` block" and closed "this file's only effect the alias above". Both were true as written (#7934/#8314): the file was this package's first vitest config and carried the alias and nothing else, as its own commit message records — "It carries the alias and nothing else: no `test` block". Both were falsified in place when the console-intercept disarm landed ABOVE them and left them standing (#10374/#13522). Measured on 44c849c7d6: the file opens with a `test` block whose one key is `disableConsoleIntercept: true`, so there IS a test block and the alias is NOT the only effect. The load-bearing half of that sentence is kept intact and now stands on its own: discovery is still untouched (zero `include`/`exclude`/`dir` keys in the file), and narrowing `include` here would still silently drop cases while the gate this file answers went green. The retired wording is paraphrased rather than quoted, so a census grep for the old phrasing does not land back on this file. Config-only, publishes nothing: the file is absent from the package tarball and its text reaches zero dist files, while a control src JSDoc reaches six. Co-Authored-By: Claude Opus 5 Claude-Session: https://claude.ai/code/session_01ADLdAs2pVcH17h9tZKWMBg --- packages/platform-objects/vitest.config.ts | 23 +++++++++++++++++----- 1 file changed, 18 insertions(+), 5 deletions(-) diff --git a/packages/platform-objects/vitest.config.ts b/packages/platform-objects/vitest.config.ts index 44dbdea706..a873d9349b 100644 --- a/packages/platform-objects/vitest.config.ts +++ b/packages/platform-objects/vitest.config.ts @@ -62,10 +62,23 @@ export default defineConfig({ }, ], }, - // No `test` block: this package had no vitest config until now, so its suite - // ran on vitest's defaults. Leaving discovery untouched keeps this file's only - // effect the alias above — the sweep's population floors (76 object files / 51 + // Discovery is untouched — but this file is NOT "the alias and nothing else". + // The `test` block above carries exactly one key, and it is a run-exit fix + // (`disableConsoleIntercept`, #10374), not a discovery key: no `include`, + // `exclude` or `dir` is set anywhere here, so this package's suite is still + // discovered on vitest's defaults, as it was before this file existed. + // + // Narrowing `include` here would silently drop cases while the gate this file + // answers went green: the sweep's population floors (76 object files / 51 // in-scope / 9 packages) are measured by walking the filesystem, but the rest - // of the package's suite is discovered by vitest, and narrowing `include` here - // would silently drop cases while the gate this file answers went green. + // of the package's suite is discovered by vitest. + // + // The two claims this paragraph replaces — that a `test` block was absent + // here, and that the alias was this file's sole effect — were true as written + // (#7934/#8314), and were falsified in place when the disarm landed ABOVE them + // and left them standing (#10374/#13522): the same silent staleness the alias + // rationale above records against its own suite count. Deleting or rewriting + // this config now costs the disarm as well as the alias (#16189). Paraphrased + // rather than quoted, so a census grep for the retired wording does not land + // back on this file. });