fix(test): list the published FILE subpaths ahead of the bare @objectstack/core / @objectstack/types alias entries (#9892)

* fix(test): list the published FILE subpaths ahead of the bare `@objectstack/core` / `@objectstack/types` alias entries

A string `find` in a vitest alias table matches by PREFIX, so a bare entry with a
FILE replacement swallows the package's published subpaths: `@objectstack/core/logger`
resolved to `…/core/src/index.ts/logger` and `@objectstack/types/node` to
`…/types/src/index.ts/node` — ENOTDIR at run time, in a config that reads as correct,
reported against whichever module performed the import rather than against the table.

Measured on main: 11 (config, subpath) pairs across 8 configs — 7 for `./logger`,
4 for `./node`. All of them latent: `check:test-source-alias` rule 5 only judges a
specifier a test graph actually reaches, and none of these had been reached yet.

Also adds the reachability-independent half to that gate: every subpath a workspace
package publishes in its own `exports` map is now resolved through every config's
alias table, so the next such pair fails at the table instead of on whoever first
writes the import.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XqDQYVU5smx29ts9pAErja

* test(tooling): report the published-subpath population on the gate's green line

The #9674 rule added a second scanned population (every subpath a workspace
package publishes in its own `exports` map) and a census guard for the exact
zero, but the OK line printed only the two older readings. A population that
silently shrinks — 43 to 1, say, from an `exports` read that regresses to the
first key — clears the zero guard and reads as a spotless repo, which is the
#4690 failure this file's census convention exists to refuse. The number a
human actually reads is on the green line, so it belongs there.

Self-test pins it as a positive NUMBER, not just non-zero: returning it under
a different name from `check()` passes every failure assertion and prints
`undefined published subpath(s)`.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XqDQYVU5smx29ts9pAErja

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: os-steve

packages/drivers/driver-memory/vitest.config.ts

@@ -10,7 +10,18 @@ export default defineConfig({
   },
   resolve: {
     alias: [
+      // Subpath BEFORE the bare package: `@objectstack/core` is a PREFIX match with a
+      // FILE replacement, so without this entry it swallows the published `./logger`
+      // subpath and resolves it to `…/core/src/index.ts/logger` — ENOTDIR at run time.
+      // Pinned by the published-subpath rule in `scripts/check-test-source-alias.mjs`,
+      // which derives the population from `@objectstack/core`'s own `exports` map.
+      { find: /^@objectstack\/core\/logger$/, replacement: path.resolve(__dirname, '../../core/src/logger.ts') },
       { find: '@objectstack/core', replacement: path.resolve(__dirname, '../../core/src/index.ts') },
+      // Subpath BEFORE the bare package, same prefix-match reason: `./node` is a
+      // published subpath served by a FILE (`types/src/node.ts` — the node-only slice
+      // the root export deliberately excludes), so the bare entry would resolve it to
+      // `…/types/src/index.ts/node`. Same published-subpath rule pins it.
+      { find: /^@objectstack\/types\/node$/, replacement: path.resolve(__dirname, '../../types/src/node.ts') },
       // [ADR-0105 D1] `resolveTenancyPosture()`, read by the #6915 tenancy guard.
       { find: '@objectstack/types', replacement: path.resolve(__dirname, '../../types/src/index.ts') },
       // #9457: ONE anchored rule for every `@objectstack/spec` namespace, in place

packages/drivers/driver-sql/vitest.config.ts

@@ -37,6 +37,12 @@ export default defineConfig({
         replacement: path.join(path.resolve(__dirname, '../..'), 'spec/src/$1/index.ts'),
       },
       { find: /^@objectstack\/spec$/, replacement: path.resolve(__dirname, '../../spec/src/index.ts') },
+      // Subpath BEFORE the bare package: `@objectstack/core` is a PREFIX match with a
+      // FILE replacement, so without this entry it swallows the published `./logger`
+      // subpath and resolves it to `…/core/src/index.ts/logger` — ENOTDIR at run time.
+      // Pinned by the published-subpath rule in `scripts/check-test-source-alias.mjs`,
+      // which derives the population from `@objectstack/core`'s own `exports` map.
+      { find: /^@objectstack\/core\/logger$/, replacement: path.resolve(__dirname, '../../core/src/logger.ts') },
       { find: '@objectstack/core', replacement: path.resolve(__dirname, '../../core/src/index.ts') },
     ],
   },

packages/metadata/vitest.config.ts

@@ -6,6 +6,12 @@ import path from 'node:path';
 export default defineConfig({
   resolve: {
     alias: [
+      // Subpath BEFORE the bare package: `@objectstack/core` is a PREFIX match with a
+      // FILE replacement, so without this entry it swallows the published `./logger`
+      // subpath and resolves it to `…/core/src/index.ts/logger` — ENOTDIR at run time.
+      // Pinned by the published-subpath rule in `scripts/check-test-source-alias.mjs`,
+      // which derives the population from `@objectstack/core`'s own `exports` map.
+      { find: /^@objectstack\/core\/logger$/, replacement: path.resolve(__dirname, '../core/src/logger.ts') },
       { find: '@objectstack/core', replacement: path.resolve(__dirname, '../core/src/index.ts') },
       // #9457: ONE anchored rule for every `@objectstack/spec` namespace, in place
       // of the hand-maintained list of the subpaths this package's tests happened
@@ -37,6 +43,11 @@ export default defineConfig({
         replacement: path.join(path.resolve(__dirname, '..'), 'spec/src/$1/index.ts'),
       },
       { find: /^@objectstack\/spec$/, replacement: path.resolve(__dirname, '../spec/src/index.ts') },
+      // Subpath BEFORE the bare package, same prefix-match reason: `./node` is a
+      // published subpath served by a FILE (`types/src/node.ts` — the node-only slice
+      // the root export deliberately excludes), so the bare entry would resolve it to
+      // `…/types/src/index.ts/node`. Same published-subpath rule pins it.
+      { find: /^@objectstack\/types\/node$/, replacement: path.resolve(__dirname, '../types/src/node.ts') },
       { find: '@objectstack/types', replacement: path.resolve(__dirname, '../types/src/index.ts') },
     ],
   },

packages/plugins/knowledge-memory/vitest.config.ts

@@ -10,6 +10,12 @@ export default defineConfig({
   },
   resolve: {
     alias: [
+      // Subpath BEFORE the bare package: `@objectstack/core` is a PREFIX match with a
+      // FILE replacement, so without this entry it swallows the published `./logger`
+      // subpath and resolves it to `…/core/src/index.ts/logger` — ENOTDIR at run time.
+      // Pinned by the published-subpath rule in `scripts/check-test-source-alias.mjs`,
+      // which derives the population from `@objectstack/core`'s own `exports` map.
+      { find: /^@objectstack\/core\/logger$/, replacement: path.resolve(__dirname, '../../core/src/logger.ts') },
       { find: '@objectstack/core', replacement: path.resolve(__dirname, '../../core/src/index.ts') },
       {
         find: '@objectstack/service-knowledge',

packages/plugins/knowledge-ragflow/vitest.config.ts

@@ -10,6 +10,12 @@ export default defineConfig({
   },
   resolve: {
     alias: [
+      // Subpath BEFORE the bare package: `@objectstack/core` is a PREFIX match with a
+      // FILE replacement, so without this entry it swallows the published `./logger`
+      // subpath and resolves it to `…/core/src/index.ts/logger` — ENOTDIR at run time.
+      // Pinned by the published-subpath rule in `scripts/check-test-source-alias.mjs`,
+      // which derives the population from `@objectstack/core`'s own `exports` map.
+      { find: /^@objectstack\/core\/logger$/, replacement: path.resolve(__dirname, '../../core/src/logger.ts') },
       { find: '@objectstack/core', replacement: path.resolve(__dirname, '../../core/src/index.ts') },
       {
         find: '@objectstack/service-knowledge',

packages/plugins/plugin-dev/vitest.config.ts

@@ -15,7 +15,18 @@ export default defineConfig({
   },
   resolve: {
     alias: [
+      // Subpath BEFORE the bare package: `@objectstack/core` is a PREFIX match with a
+      // FILE replacement, so without this entry it swallows the published `./logger`
+      // subpath and resolves it to `…/core/src/index.ts/logger` — ENOTDIR at run time.
+      // Pinned by the published-subpath rule in `scripts/check-test-source-alias.mjs`,
+      // which derives the population from `@objectstack/core`'s own `exports` map.
+      { find: /^@objectstack\/core\/logger$/, replacement: path.resolve(__dirname, '../../core/src/logger.ts') },
       { find: '@objectstack/core', replacement: path.resolve(__dirname, '../../core/src/index.ts') },
+      // Subpath BEFORE the bare package, same prefix-match reason: `./node` is a
+      // published subpath served by a FILE (`types/src/node.ts` — the node-only slice
+      // the root export deliberately excludes), so the bare entry would resolve it to
+      // `…/types/src/index.ts/node`. Same published-subpath rule pins it.
+      { find: /^@objectstack\/types\/node$/, replacement: path.resolve(__dirname, '../../types/src/node.ts') },
       { find: '@objectstack/types', replacement: path.resolve(__dirname, '../../types/src/index.ts') },
       // #9457: ONE anchored rule for every `@objectstack/spec` namespace, in place
       // of the hand-maintained list of the subpaths this package's tests happened

packages/plugins/plugin-hono-server/vitest.config.ts

@@ -10,6 +10,12 @@ export default defineConfig({
   },
   resolve: {
     alias: [
+      // Subpath BEFORE the bare package: `@objectstack/core` is a PREFIX match with a
+      // FILE replacement, so without this entry it swallows the published `./logger`
+      // subpath and resolves it to `…/core/src/index.ts/logger` — ENOTDIR at run time.
+      // Pinned by the published-subpath rule in `scripts/check-test-source-alias.mjs`,
+      // which derives the population from `@objectstack/core`'s own `exports` map.
+      { find: /^@objectstack\/core\/logger$/, replacement: path.resolve(__dirname, '../../core/src/logger.ts') },
       { find: '@objectstack/core', replacement: path.resolve(__dirname, '../../core/src/index.ts') },
       { find: '@objectstack/observability', replacement: path.resolve(__dirname, '../../observability/src/index.ts') },
       // #9457: ONE anchored rule for every `@objectstack/spec` namespace, in place

packages/runtime/vitest.config.ts

@@ -92,6 +92,11 @@ export default defineConfig({
         replacement: path.join(path.resolve(__dirname, '..'), 'spec/src/$1/index.ts'),
       },
       { find: /^@objectstack\/spec$/, replacement: path.resolve(__dirname, '../spec/src/index.ts') },
+      // Subpath BEFORE the bare package, same prefix-match reason: `./node` is a
+      // published subpath served by a FILE (`types/src/node.ts` — the node-only slice
+      // the root export deliberately excludes), so the bare entry would resolve it to
+      // `…/types/src/index.ts/node`. Same published-subpath rule pins it.
+      { find: /^@objectstack\/types\/node$/, replacement: path.resolve(__dirname, '../types/src/node.ts') },
       { find: '@objectstack/types', replacement: path.resolve(__dirname, '../types/src/index.ts') },
       // Dev-only: app-plugin.jobs.test.ts drives the REAL CronJobAdapter, so
       // the #4567 regression (croner rejecting the expression envelope) is