@rtorcato/repo-tooling 3.4.0 → 3.5.0

package/dist/base/checks.js

@@ -1,6 +1,29 @@
 import path from 'node:path';
 import fs from 'fs-extra';
 import { BADGE_START, hasPublicOnlyBadges } from '../cli/generators/badges.js';
+import { detectNestedLanguages } from '../cli/utils/detect-language.js';
+/**
+ * Root-only detection is the decision (#317), not an oversight — but it used to
+ * be a *silent* one: a Swift repo with a TypeScript docs app audited as Swift
+ * and never mentioned the half it skipped. This says so.
+ *
+ * Always `ok`. Nesting a second language is a legitimate repo shape, so failing
+ * on it would break every monorepo's CI to report a limitation of ours.
+ */
+export async function checkNestedLanguages(dir, rootLanguage) {
+    const check = 'Monorepo';
+    const nested = await detectNestedLanguages(dir, rootLanguage);
+    if (nested.length === 0) {
+        return { check, status: 'ok', detail: 'single-language repo' };
+    }
+    const summary = nested.map((n) => `${n.dir} (${n.language})`).join(', ');
+    return {
+        check,
+        status: 'ok',
+        detail: `auditing the root only — ${summary} ${nested.length === 1 ? 'is' : 'are'} not audited`,
+        hint: 'Run doctor with `--directory <path>` to audit a nested project on its own',
+    };
+}
 export async function checkFile(dir, spec) {
     for (const candidate of spec.candidates) {
         const filepath = path.join(dir, candidate);

package/dist/cli/commands/doctor.js

@@ -12,7 +12,7 @@ import { checkGitHubSettings } from '../../base/github-settings.js';
 import { checkGitIdentity } from '../../base/git-identity.js';
 import { LOCKFILE_VERSION, readLockfile } from '../utils/lockfile.js';
 import { declinedInLock, getFixTargetForCheck } from './fix-targets.js';
-import { checkAiSetup, checkCodeowners, checkCodeQL, checkCommunityHealth, checkCoverageUpload, checkDependabot, checkEditorConfig, checkFile, checkGitHooks, checkGitHubActions, checkGitLabCI, checkPrePushHook, checkReadmeBadges, COMMITLINT_FILE_CHECK, } from '../../base/checks.js';
+import { checkAiSetup, checkCodeowners, checkCodeQL, checkCommunityHealth, checkCoverageUpload, checkDependabot, checkEditorConfig, checkFile, checkGitHooks, checkGitHubActions, checkGitLabCI, checkNestedLanguages, checkPrePushHook, checkReadmeBadges, COMMITLINT_FILE_CHECK, } from '../../base/checks.js';
 import { allDeps, checkAreTheTypesWrong, checkConfigSchemaVersions, checkDocsSite, checkEnginesNode, checkGitDependencies, checkKnip, checkLintStaged, checkNodeVersionConsistency, checkNodeVersionPin, checkPackageJson, checkPublint, checkSemanticRelease, checkSizeLimit, checkTailwind, checkPnpmWorkspace, checkTreeshakeSetup, checkTurborepo, checkTypedoc, checkVerifyScript, checkVscodeExtensions, evaluateNodeVersion, FILE_CHECKS, findDocsAppDir, jsBadgeAudience, jsGitHooksProfile, readPackageJson, } from '../../languages/js/checks.js';
 export { evaluateNodeVersion };
 const PACKAGE = '@rtorcato/repo-tooling';
@@ -145,6 +145,7 @@ function demoteDeclined(results, lock) {
 async function runBaseChecks(dir, lock, opts) {
     const results = [];
     results.push(checkLockfile(lock));
+    results.push(await checkNestedLanguages(dir, opts.language));
     results.push(await checkGitIdentity(dir));
     results.push(await checkEditorConfig(dir));
     results.push(await checkFile(dir, COMMITLINT_FILE_CHECK));
@@ -191,6 +192,7 @@ export async function runDoctor(dir) {
                 hooks: null,
                 badges: { audience: 'public', fixTarget: null },
                 presetWorkflow: null,
+                language,
             })),
         ];
         return demoteDeclined(results, lock);
@@ -211,6 +213,7 @@ export async function runDoctor(dir) {
                 // package.json name/repository, which a Swift repo hasn't got.
                 badges: { audience: 'public', fixTarget: null },
                 presetWorkflow: renderSwiftWorkflow(await readSwiftPackage(targetDir)),
+                language,
             })),
             ...(await runSwiftChecks(targetDir)),
         ];
@@ -261,6 +264,7 @@ export async function runDoctor(dir) {
         hooks: jsGitHooksProfile(pkg),
         badges: { audience: jsBadgeAudience(pkg), fixTarget: 'badges' },
         presetWorkflow: renderGitHubWorkflow(githubJobs(inferProjectConfig(pkg))),
+        language,
     })));
     return demoteDeclined(results, lock);
 }

package/dist/cli/utils/detect-language.js

@@ -2,8 +2,9 @@ import path from 'node:path';
 import fs from 'fs-extra';
 /**
  * Marker files that identify a repo's language, checked in order — first match
- * wins. One language per repo root; multi-language monorepos are out of scope
- * for v1 (see #139). A dir with no marker → 'unknown' (base checks only).
+ * wins. One language per repo root; multi-language monorepos stay out of scope
+ * (#139, reaffirmed in #317 — the audit is root-only, but says so via
+ * `detectNestedLanguages`). A dir with no marker → 'unknown' (base checks only).
  */
 const MARKERS = [
     ['js', ['package.json']],
@@ -20,3 +21,44 @@ export async function detectLanguage(dir) {
     }
     return 'unknown';
 }
+/** Never worth descending into, and expensive when we do. */
+const SKIP_DIRS = new Set(['node_modules', 'dist', 'build', 'target', 'vendor', '.build']);
+/**
+ * Sub-project dirs whose language marker disagrees with the root's, as
+ * `{ dir, language }` sorted by dir. Depth-2 scan — enough for the shapes that
+ * exist (`apps/docs`, `packages/*`, a top-level `docs/`) and cheap enough to run
+ * on every doctor.
+ *
+ * Deliberately *not* driven by the workspace manifest: the case #317 is about is
+ * a Swift or Perl root with a JS docs app, and such a repo has no
+ * `pnpm-workspace.yaml` to read. Walking the tree finds it either way, and needs
+ * no glob dependency.
+ */
+export async function detectNestedLanguages(dir, rootLanguage) {
+    const found = [];
+    const scan = async (relative, depth) => {
+        let entries;
+        try {
+            entries = await fs.readdir(path.join(dir, relative));
+        }
+        catch {
+            return; // unreadable dir is the caller's problem, not the audit's
+        }
+        for (const entry of entries) {
+            if (entry.startsWith('.') || SKIP_DIRS.has(entry))
+                continue;
+            const child = relative ? `${relative}/${entry}` : entry;
+            if (!(await fs.stat(path.join(dir, child)).catch(() => null))?.isDirectory())
+                continue;
+            const language = await detectLanguage(path.join(dir, child));
+            if (language !== 'unknown' && language !== rootLanguage) {
+                found.push({ dir: child, language });
+                continue; // a package's own subdirs belong to it, not to the root
+            }
+            if (depth > 1)
+                await scan(child, depth - 1);
+        }
+    };
+    await scan('', 2);
+    return found.sort((a, b) => a.dir.localeCompare(b.dir));
+}

package/package.json

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