@rtorcato/repo-tooling 3.10.0 → 3.10.1

package/dist/cli/generators/agent-rules.js

@@ -1,7 +1,9 @@
+import { glob } from 'node:fs/promises';
 import path from 'node:path';
 import fs from 'fs-extra';
 import { copyPreset, getPackageRoot } from '../utils/copy-preset.js';
 import { LEGACY_TOOL_NAME } from '../utils/lockfile.js';
+import { parseWorkspacePackages } from './misc.js';
 import { installSkillsInstallDocs } from './skills-install.js';
 /**
  * All agent rule files are generated from one source of truth — the shipped
@@ -41,6 +43,62 @@ function asObject(value) {
         ? value
         : null;
 }
+/**
+ * True when `rel` resolves to something at or under `targetDir`. The entries
+ * this guards end up in `worktree.symlinkDirectories`, which Claude Code reads
+ * as symlink targets — an escaping path would point a worktree symlink outside
+ * the repo, so a workspace glob is never trusted to stay inside on its own.
+ */
+function isInside(targetDir, rel) {
+    const outward = path.relative(path.resolve(targetDir), path.resolve(targetDir, rel));
+    return outward !== '' && !outward.startsWith('..') && !path.isAbsolute(outward);
+}
+/**
+ * The repo's own workspace globs, from `pnpm-workspace.yaml` and/or
+ * package.json `workspaces` (both array and `{ packages: [] }` forms). Both
+ * are read because a pnpm repo can keep its globs in either file.
+ *
+ * Negations (`!apps/x`) are dropped, and so is anything absolute or containing
+ * a `..` segment: a `packages: - '../shared-lib'` entry reads as plausible in a
+ * monorepo but would write an out-of-repo symlink target into settings.json.
+ */
+async function workspaceGlobs(targetDir) {
+    const yamlFile = path.join(targetDir, 'pnpm-workspace.yaml');
+    const yaml = (await fs.pathExists(yamlFile)) ? await fs.readFile(yamlFile, 'utf-8') : '';
+    const pkg = asObject(await fs.readJson(path.join(targetDir, 'package.json')).catch(() => null));
+    const field = Array.isArray(pkg?.workspaces)
+        ? pkg.workspaces
+        : asObject(pkg?.workspaces)?.packages;
+    const declared = Array.isArray(field) ? field : [];
+    return [...parseWorkspacePackages(yaml), ...declared].filter((g) => typeof g === 'string' &&
+        !g.startsWith('!') &&
+        !path.isAbsolute(g) &&
+        !g.split(/[\\/]/).includes('..'));
+}
+/**
+ * The nested `<workspace>/node_modules` to symlink alongside the root one
+ * (#406). Without them a workspace-scoped command in a fresh worktree fails on
+ * a missing binary — `pnpm --filter <pkg> build` can't find its bin shims —
+ * which reads as a broken toolchain rather than a missing symlink.
+ *
+ * Derived from the repo's own workspace globs, never a hardcoded layout: this
+ * is a public CLI and every consumer nests its packages differently.
+ * Workspaces with no `node_modules` in the main checkout are skipped, because
+ * an entry pointing at nothing warns on every worktree creation.
+ */
+export async function workspaceSymlinkDirs(targetDir) {
+    const globs = await workspaceGlobs(targetDir);
+    if (globs.length === 0)
+        return [];
+    const found = [];
+    for await (const match of glob(globs.map((g) => `${g}/node_modules`), { cwd: targetDir })) {
+        // Belt and braces: the glob result is what actually gets written, so it is
+        // re-checked for containment even though the input globs were filtered.
+        if (isInside(targetDir, match))
+            found.push(match.split(path.sep).join('/'));
+    }
+    return found.sort();
+}
 /** Parsed `.claude/settings.json`, or null when absent, malformed, or not an object. */
 export async function readClaudeSettings(targetDir) {
     try {
@@ -74,10 +132,8 @@ export async function installClaudeSettings(targetDir) {
     if (!settings)
         return null;
     const existing = worktreeSymlinkDirs(settings);
-    const symlinkDirectories = [
-        ...existing,
-        ...WORKTREE_SYMLINK_DIRS.filter((d) => !existing.includes(d)),
-    ];
+    const wanted = [...WORKTREE_SYMLINK_DIRS, ...(await workspaceSymlinkDirs(targetDir))];
+    const symlinkDirectories = [...existing, ...wanted.filter((d) => !existing.includes(d))];
     const worktree = { ...asObject(settings.worktree), symlinkDirectories };
     await fs.outputJson(file, { ...settings, worktree }, { spaces: 2 });
     return CLAUDE_SETTINGS_FILE;

package/dist/cli/generators/misc.js

@@ -256,7 +256,7 @@ export async function ensurePackageManager(targetDir, version = detectPnpmVersio
  * YAML dependency. Matches `- 'apps/*'` / `- "apps/*"` / `- apps/*` entries
  * under the `packages:` key, stopping at the next top-level key.
  */
-function parseWorkspacePackages(yaml) {
+export function parseWorkspacePackages(yaml) {
     const globs = [];
     let inPackages = false;
     for (const line of yaml.split('\n')) {

package/dist/languages/js/checks.js

@@ -1,7 +1,7 @@
 import path from 'node:path';
 import fs from 'fs-extra';
 import { hookHasUncommented } from '../../base/checks.js';
-import { CLAUDE_SETTINGS_FILE, readClaudeSettings, worktreeSymlinkDirs, } from '../../cli/generators/agent-rules.js';
+import { CLAUDE_SETTINGS_FILE, readClaudeSettings, workspaceSymlinkDirs, worktreeSymlinkDirs, } from '../../cli/generators/agent-rules.js';
 import { WORKSPACE_FILE, dependsOnEsbuild, familyGlob, missingPnpmSettings, } from '../../cli/generators/pnpm-workspace.js';
 const PACKAGE = '@rtorcato/repo-tooling';
 const NODE_MIN_MAJOR = 22;
@@ -980,13 +980,20 @@ export async function checkClaudeWorktreeSettings(dir) {
             hint: `Repair the JSON in ${CLAUDE_SETTINGS_FILE} by hand — \`fix ai\` refuses to overwrite it`,
         };
     }
-    if (worktreeSymlinkDirs(settings).includes('node_modules')) {
-        return { check, status: 'ok', detail: 'worktree.symlinkDirectories carries node_modules' };
+    const recorded = worktreeSymlinkDirs(settings);
+    const wanted = ['node_modules', ...(await workspaceSymlinkDirs(dir))];
+    const missing = wanted.filter((d) => !recorded.includes(d));
+    if (missing.length === 0) {
+        return {
+            check,
+            status: 'ok',
+            detail: `worktree.symlinkDirectories carries ${wanted.join(', ')}`,
+        };
     }
     return {
         check,
         status: 'optional-missing',
-        detail: `${CLAUDE_SETTINGS_FILE} has no worktree.symlinkDirectories entry for node_modules`,
+        detail: `${CLAUDE_SETTINGS_FILE} has no worktree.symlinkDirectories entry for ${missing.join(', ')}`,
         hint,
     };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@rtorcato/repo-tooling",
-	"version": "3.10.0",
+	"version": "3.10.1",
 	"description": "One CLI to scaffold, audit and fix your repo's whole toolchain — linting, tests, commits, releases & CI.",
 	"type": "module",
 	"keywords": [