fix: keyed-text-bounds gate skips git-ignored tmp/, scaffold test moves under node_modules (#17421)

check:keyed-text-bounds walked packages/cli/tmp/, a git-ignored directory,
so a scaffold test killed before its afterAll cleanup ran left behind an
os-init-shaped .object.ts the gate cannot parse, redding the gate for a
reason unrelated to the author's diff.

- Add 'tmp' to SKIP_DIRS, the walk's existing filter point, alongside the
  other build/tooling-litter directories it already skips. The walk stays
  repo-wide minus SKIP_DIRS; only the skip list grew by one entry.
- Move init-scaffold-authoring-rules.test.ts's scaffold temp root from this
  package's tmp/ to under its own node_modules/, matching the placement
  generate-scaffold-validates.test.ts already documents for the same
  @objectstack/spec resolution reason. node_modules is already skipped by
  the gate regardless of the SKIP_DIRS change above.

The gate's refusal for an unrecognised declaration shape on a keyed column
is unchanged -- this only narrows which files are walked.


Claude-Session: https://claude.ai/code/session_012GKcPZbMoGq7WPzKLfRBTU

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

Author: baozhoutao

packages/cli/test/init-scaffold-authoring-rules.test.ts

@@ -30,11 +30,16 @@
  * (`writeTemplateSrcFiles`) and checked through the command's own self-test
  * (`validateScaffold`), so neither half can drift from what `init` really does.
  *
- * Temp projects are created under this package's git-ignored `tmp/` (not
- * `os.tmpdir()`) because the rendered config imports `@objectstack/spec`,
- * which only resolves where Node can walk up into this package's
- * `node_modules`. Keeping them out of `test/` also keeps generated `.ts` away
- * from any glob that collects sources.
+ * Temp projects are created under this package's own `node_modules` (not
+ * `os.tmpdir()`, and not this package's `tmp/`) because the rendered config
+ * imports `@objectstack/spec`, which only resolves where Node can walk up
+ * into this package's `node_modules` -- the same reasoning
+ * `generate-scaffold-validates.test.ts` documents for its own materialized
+ * scaffolds. `node_modules` is git-ignored and already excluded from every
+ * source-walking gate, so a run killed before `afterAll` runs (a timeout, the
+ * container's foreground cap, `ctrl-c`) leaves inert litter behind instead of
+ * files a repo-wide scan can trip over. Keeping them out of `test/` also
+ * keeps generated `.ts` away from any glob that collects sources.
  */
 
 import { describe, it, expect, afterAll } from 'vitest';
@@ -46,7 +51,7 @@ import { TEMPLATES, sanitizeNamespace, writeTemplateSrcFiles } from '../src/comm
 import { validateScaffold, SCAFFOLD_RULE_COMMAND } from '../src/utils/scaffold-validate.js';
 
 const HERE = path.dirname(fileURLToPath(import.meta.url));
-const TMP_ROOT = path.resolve(HERE, '../tmp');
+const TMP_ROOT = path.resolve(HERE, '../node_modules/.init-scaffold-authoring-rules');
 const PROJECT_NAME = 'my-app';
 
 const roots: string[] = [];

scripts/check-keyed-text-bounds.mjs

@@ -359,6 +359,7 @@ const MIN_KEYED_TEXT_COLUMNS = 140;
 
 const SKIP_DIRS = new Set([
   '.git', 'node_modules', 'dist', 'build', 'coverage', '.turbo', '.next', '.cache', '.changeset',
+  'tmp',
 ]);
 
 /** Where shipped object declarations are discovered. */