@steve31415/baselib 3.36.1 → 3.36.2

package/dist/test-owners.js

@@ -9,32 +9,65 @@
 //
 // Scope is `.ts` only; `.tsx` is out of scope by decision (docs/TESTING.md,
 // "The src -> test mapping"). `.d.ts` files are declarations, not modules.
+//
+// Files git ignores are not part of the repo and are skipped in both
+// directions. A long-lived deploy checkout can hold leftovers of a build step
+// that no longer exists — games' clone kept a generated `src/build-version.ts`
+// from 2026-07-15, gitignored once its generator was deleted, and verify
+// failed on it in every deploy while passing in every fresh worktree
+// (2026-10-12, Coder task 37e13f8e). Untracked files that are NOT ignored still
+// count, so a new module is checked before its first `git add`. Outside a git
+// work tree (or without git) nothing is filtered.
+import { execFile } from 'node:child_process';
 import { readdir } from 'node:fs/promises';
 import { join } from 'node:path';
 const MODULE_EXT = '.ts';
 const TEST_EXT = '.test.ts';
-/** Files under `dir` (recursively) ending in `ext`, as paths relative to
- *  `dir` with forward slashes. A missing directory yields no files. */
-async function listFiles(dir, ext) {
+/** The subset of `paths` (relative to `root`) that git ignores. Tracked
+ *  files are never reported, even when a pattern matches them. Empty when
+ *  `root` is not in a git work tree or git is unavailable. */
+function gitIgnored(root, paths) {
+    if (paths.length === 0)
+        return Promise.resolve(new Set());
+    return new Promise((resolve, reject) => {
+        const child = execFile('git', ['check-ignore', '-z', '--stdin'], { cwd: root, maxBuffer: 64 * 1024 * 1024 }, (err, stdout) => {
+            // Exit 1 means "none ignored"; 128 (not a repo) or a missing git
+            // binary means there is nothing to filter against.
+            const code = err?.code;
+            if (err && code !== 1 && code !== 128 && code !== 'ENOENT') {
+                reject(err);
+                return;
+            }
+            resolve(new Set(err ? [] : stdout.split('\0').filter((p) => p !== '')));
+        });
+        child.stdin?.on('error', () => { }); // git exited before reading: the callback reports it
+        child.stdin?.end(paths.join('\0') + '\0');
+    });
+}
+/** Files under `root/dir` (recursively) ending in `ext` and not ignored by
+ *  git, as paths relative to `dir` with forward slashes. A missing directory
+ *  yields no files. */
+async function listFiles(root, dir, ext) {
     let entries;
     try {
-        entries = await readdir(dir, { recursive: true });
+        entries = await readdir(join(root, dir), { recursive: true });
     }
     catch (err) {
         if (err.code === 'ENOENT')
             return [];
         throw err;
     }
-    return entries
+    const files = entries
         .map((e) => e.split('\\').join('/'))
-        .filter((e) => e.endsWith(ext) && !e.endsWith('.d.ts'))
-        .sort();
+        .filter((e) => e.endsWith(ext) && !e.endsWith('.d.ts'));
+    const ignored = await gitIgnored(root, files.map((f) => `${dir}/${f}`));
+    return files.filter((f) => !ignored.has(`${dir}/${f}`)).sort();
 }
 export async function checkTestOwners(opts) {
     const srcDir = opts.srcDir ?? 'src';
     const testDir = opts.testDir ?? 'test';
-    const modules = (await listFiles(join(opts.root, srcDir), MODULE_EXT)).filter((m) => !m.endsWith(TEST_EXT));
-    const tests = new Set(await listFiles(join(opts.root, testDir), TEST_EXT));
+    const modules = (await listFiles(opts.root, srcDir, MODULE_EXT)).filter((m) => !m.endsWith(TEST_EXT));
+    const tests = new Set(await listFiles(opts.root, testDir, TEST_EXT));
     const missing = [];
     for (const mod of modules) {
         const expected = `${mod.slice(0, -MODULE_EXT.length)}${TEST_EXT}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@steve31415/baselib",
-  "version": "3.36.1",
+  "version": "3.36.2",
   "description": "Plasticine new-world shared platform library: logging, auth, service-to-service auth, db, HTTP, sync, app updates",
   "type": "module",
   "license": "MIT",