@rtorcato/repo-tooling 3.2.2 → 3.2.4

package/dist/cli/generators/linting.js

@@ -40,7 +40,17 @@ export async function generateBiomeConfig(targetDir) {
 export async function generateESLintConfig(config, targetDir) {
     const eslintConfigPath = path.join(targetDir, 'eslint.config.mjs');
     const configType = config.linting.eslintConfig || 'base';
-    const eslintConfig = `import { default as config } from '@rtorcato/repo-tooling/eslint/${configType}'
+    // eslint/nextjs is a fragment: it registers the @next/next plugin and its
+    // rules but sets no parser, so on its own it can't even parse .tsx ("Parsing
+    // error: Unexpected token <"). It has to be layered onto the base config,
+    // which is what brings typescript-eslint.
+    const eslintConfig = configType === 'nextjs'
+        ? `import base from '@rtorcato/repo-tooling/eslint/base'
+import nextjs from '@rtorcato/repo-tooling/eslint/nextjs'
+
+export default [...base, ...nextjs]
+`
+        : `import { default as config } from '@rtorcato/repo-tooling/eslint/${configType}'
 
 export default config
 `;

package/dist/cli/generators/package-json.js

@@ -237,6 +237,24 @@ function getDependencies(config) {
     if (config.linting.tool === 'eslint' || config.linting.tool === 'both') {
         deps['eslint'] = '^9.0.0';
         deps['prettier'] = '^3.0.0';
+        // Everything tooling/eslint/base.mjs imports. All optional peers of
+        // repo-tooling, so pnpm skips them and `eslint .` dies on the config
+        // import — the scaffold shipped a lint script that could never run.
+        deps['@eslint/js'] = '^9.0.0';
+        deps['@typescript-eslint/eslint-plugin'] = '^8.0.0';
+        deps['typescript-eslint'] = '^8.0.0';
+        deps['eslint-plugin-import'] = '^2.0.0';
+        deps['eslint-plugin-jest'] = '^29.0.0';
+    }
+    // The shipped prettier preset lists this in `plugins`, and the nextjs eslint
+    // preset imports the Next plugin. Both are only *optional* peers of
+    // repo-tooling, so pnpm skips them and the scaffold's first format/lint dies
+    // with ERR_MODULE_NOT_FOUND.
+    if (config.formatting.tool === 'prettier') {
+        deps['@ianvs/prettier-plugin-sort-imports'] = '^4.7.0';
+    }
+    if (config.linting.eslintConfig === 'nextjs') {
+        deps['@next/eslint-plugin-next'] = '^16.2.11';
     }
     // Testing frameworks
     if (config.testing.framework === 'vitest') {
@@ -266,6 +284,10 @@ function getDependencies(config) {
     }
     else if (config.bundler === 'esbuild') {
         deps['esbuild'] = '^0.25.0';
+        // The generated build.mjs imports this directly. It's only an *optional*
+        // peer of repo-tooling, so pnpm won't pull it in — without it here every
+        // esbuild scaffold fails its first `pnpm build` with ERR_MODULE_NOT_FOUND.
+        deps['esbuild-node-externals'] = '^1.18.0';
     }
     else if (config.bundler === 'rollup') {
         deps['rollup'] = '^4.0.0';
@@ -278,6 +300,11 @@ function getDependencies(config) {
     }
     else if (config.bundler === 'vite') {
         deps['vite'] = '^6.0.0';
+        // generateViteConfig emits `import react from '@vitejs/plugin-react'` for
+        // react-app, so the plugin has to be installed or `vite build` can't even
+        // load the config. Keep this condition in step with that one.
+        if (config.projectType === 'react-app')
+            deps['@vitejs/plugin-react'] = '^5.0.0';
     }
     // Tailwind CSS v4 — CSS-first, wired via the PostCSS plugin.
     if (config.tailwind) {

package/dist/cli/generators/testing.js

@@ -16,10 +16,21 @@ export async function generateTestingConfigs(config, targetDir) {
 }
 export async function generateVitestConfig(config, targetDir) {
     const vitestConfigPath = path.join(targetDir, 'vitest.config.ts');
+    // Vite reads `jsx` from tsconfig, and the next preset sets it to "preserve" so
+    // Next's own compiler handles the transform. Vitest has no such compiler, so
+    // .tsx tests fail to parse ("Unexpected token <") unless we opt back in here.
+    // `oxc` (not `esbuild`) is the Vite 8 spelling — vitest ^4 resolves Vite 8,
+    // and Vite 8 silently ignores the old `esbuild` key.
+    const jsxOverride = config.projectType === 'nextjs-app'
+        ? `  // tsconfig sets jsx: "preserve" for Next's compiler; vitest needs a
+  // transform of its own or .tsx tests won't parse.
+  oxc: { jsx: { runtime: 'automatic' } },
+`
+        : '';
     const vitestConfig = `import { defineConfig } from 'vitest/config'
 
 export default defineConfig({
-  test: {
+${jsxOverride}  test: {
     globals: true,
     environment: '${config.testing.environment === 'browser' ? 'jsdom' : 'node'}',
     setupFiles: ['./vitest.setup.ts'],

package/dist/cli/generators/tsconfig.js

@@ -19,6 +19,12 @@ export async function generateTSConfig(config, targetDir) {
         tsconfig.compilerOptions.outDir = './dist';
         tsconfig.compilerOptions.rootDir = './src';
     }
+    // web-app is the only browser target that lands on the `base` preset, whose
+    // lib is ES2022 — so `document`/`window` don't typecheck. The react and next
+    // presets already add the DOM libs themselves.
+    if (config.projectType === 'web-app' && preset === 'base') {
+        tsconfig.compilerOptions.lib = ['ES2022', 'DOM', 'DOM.Iterable'];
+    }
     if (config.projectType === 'nextjs-app') {
         tsconfig.include = ['next-env.d.ts', 'src', 'app', 'pages', 'components', 'reset.d.ts'];
         tsconfig.exclude.push('.next');

package/dist/cli/generators/typedoc.js

@@ -12,7 +12,7 @@ jobs:
     steps:
       - uses: actions/checkout@v7
       - uses: pnpm/action-setup@v6
-      - uses: actions/setup-node@v6
+      - uses: actions/setup-node@v7
         with:
           node-version: 22
           cache: pnpm

package/dist/languages/js/ci.js

@@ -15,7 +15,7 @@ const SETUP_STEPS = `      - name: 📦 Checkout repository
         uses: actions/checkout@v7
 
       - name: 📦 Setup Node.js
-        uses: actions/setup-node@v6
+        uses: actions/setup-node@v7
         with:
           node-version-file: .nvmrc
 
@@ -23,7 +23,7 @@ const SETUP_STEPS = `      - name: 📦 Checkout repository
         uses: pnpm/action-setup@v6
 
       - name: 📦 Restore dependencies cache
-        uses: actions/cache@v5
+        uses: actions/cache@v6
         with:
           path: |
             ~/.pnpm-store
@@ -38,7 +38,7 @@ const DEPENDENCIES_JOB = {
         uses: actions/checkout@v7
 
       - name: 📦 Setup Node.js
-        uses: actions/setup-node@v6
+        uses: actions/setup-node@v7
         with:
           node-version-file: .nvmrc
 
@@ -50,7 +50,7 @@ const DEPENDENCIES_JOB = {
         run: echo "key=\${{ runner.os }}-pnpm-\${{ hashFiles('**/pnpm-lock.yaml') }}" >> $GITHUB_OUTPUT
 
       - name: 📦 Cache dependencies
-        uses: actions/cache@v5
+        uses: actions/cache@v6
         with:
           path: |
             ~/.pnpm-store
@@ -164,7 +164,7 @@ ${attw}${publint}
           token: \${{ secrets.RELEASE_TOKEN || secrets.GITHUB_TOKEN }}
 
       - name: 📦 Setup Node.js
-        uses: actions/setup-node@v6
+        uses: actions/setup-node@v7
         with:
           node-version-file: .nvmrc
           registry-url: 'https://registry.npmjs.org'
@@ -173,7 +173,7 @@ ${attw}${publint}
         uses: pnpm/action-setup@v6
 
       - name: 📦 Restore dependencies cache
-        uses: actions/cache@v5
+        uses: actions/cache@v6
         with:
           path: |
             ~/.pnpm-store

package/dist/languages/swift/ci.js

@@ -45,7 +45,7 @@ const XCODE_SETUP = `      - name: 📦 Checkout repository
  * to rebuild; key on Package.resolved so the cache invalidates with the graph.
  */
 const SPM_CACHE = `      - name: 📦 Cache SwiftPM
-        uses: actions/cache@v5
+        uses: actions/cache@v6
         with:
           path: |
             .build
@@ -118,7 +118,7 @@ ${pkg.platforms.map((p) => `          - ${p}`).join('\n')}`,
             steps: `${XCODE_SETUP}
 
       - name: 📦 Cache DerivedData
-        uses: actions/cache@v5
+        uses: actions/cache@v6
         with:
           path: ~/Library/Developer/Xcode/DerivedData
           key: \${{ runner.os }}-derived-\${{ matrix.platform }}-\${{ hashFiles('**/Package.resolved') }}

package/package.json

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

package/tooling/commitlint/commitlint.mjs

@@ -1,6 +1,16 @@
 export default {
 	extends: ["@commitlint/config-conventional"],
-	ignores: [(commit) => commit.includes("[skip ci]")],
+	ignores: [
+		(commit) => commit.includes("[skip ci]"),
+		// Bot commit bodies are machine-written blocks (dependabot's
+		// `- dependency-name: …` YAML) that never wrap at 72, so every bot PR
+		// failed body-max-line-length. commitlint can't relax a single rule per
+		// commit, so skip bot commits wholesale — squash-merge uses the PR
+		// title, which is still linted on the merge commit.
+		// ponytail: matches the trailer, not a bare "[bot]", so a human commit
+		// that merely mentions a bot is still linted.
+		(commit) => /^Signed-off-by: .*\[bot\]/m.test(commit),
+	],
 	rules: {
 		// Enforce strict type validation
 		"type-enum": [

package/tooling/eslint/base.mjs

@@ -43,6 +43,14 @@ export default tseslint.config(
 		ignores: [
 			//
 			'**/*.config.js',
+			// Root-level TS config files are outside the tsconfig `include` of every
+			// app preset, and this config lints type-aware (parserOptions.project),
+			// so linting them fails with "file was not found in any of the provided
+			// project(s)". The .js forms were already ignored; these are the same
+			// thing (the scaffold emits vitest.config.ts / vitest.setup.ts).
+			'**/*.config.ts',
+			'**/*.config.mts',
+			'**/*.setup.ts',
 			'.turbo',
 			'dist',
 			'build',

package/tooling/github-actions/workflows/cloudflare-pages.yml

@@ -18,10 +18,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v7
 
       - name: Setup Node.js
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v7
         with:
           node-version-file: .nvmrc
 

package/tooling/github-actions/workflows/docker-publish.yml

@@ -17,7 +17,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v7
 
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v3

package/tooling/github-actions/workflows/preview-deployments.yml

@@ -20,10 +20,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v7
 
       - name: Setup Node.js
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v7
         with:
           node-version-file: .nvmrc
 

package/tooling/github-actions/workflows/vercel-deploy.yml

@@ -20,10 +20,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v7
 
       - name: Setup Node.js
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v7
         with:
           node-version-file: .nvmrc
 

package/tooling/typescript/tsconfig.base.json

@@ -9,8 +9,12 @@
     // "moduleResolution": "node", // Prefer "bundler" if you're using Vite, esbuild, etc.
     // "target": "esnext", // 
     "lib": ["ES2022"], // Specify library files to be included in the compilation
-    "rootDir": "src",
-    "outDir": "dist",
+    // ${configDir} for the same reason as paths/tsBuildInfoFile below: a bare
+    // "src" resolves against THIS file, i.e. inside node_modules, so every
+    // consumer that didn't redeclare rootDir got TS6059 "not under rootDir" the
+    // moment a file outside the preset's own directory entered the program.
+    "rootDir": "${configDir}/src",
+    "outDir": "${configDir}/dist",
 
     
     // 🔍 JavaScript Support

package/tooling/typescript/tsconfig.next.json

@@ -2,12 +2,19 @@
   "extends": "./tsconfig.base.json",
   "compilerOptions": {
     "jsx": "preserve", // Let Next.js handle JSX transformation
+    // The base pins rootDir to src/, but `include` below spans app/, pages/ and
+    // components/ — all siblings of src/ — so tsc raised TS6059 on any Next repo
+    // using the app router. These presets are noEmit, so rootDir buys nothing
+    // here beyond that trap; widen it to the project root.
+    "rootDir": "${configDir}",
     "lib": ["DOM", "DOM.Iterable", "ESNext"], // Include DOM types for browser APIs
     "module": "esnext", // Next.js expects ESNext modules
     "moduleResolution": "bundler", // Modern module resolution for Next.js
     "allowJs": true, // Allow JavaScript files
     "esModuleInterop": true, // Interop for CommonJS/ESM
-    "types": ["react", "tailwindcss", "vitest"],
+    // No "tailwindcss" — see tsconfig.react.json; it broke TS2688 for every
+    // project without Tailwind installed.
+    "types": ["react", "vitest"],
     "plugins": [ { "name": "next" } ], // Next.js plugin for TypeScript
     // ${configDir} anchors paths to the consuming project (drops deprecated baseUrl)
     "paths": {

package/tooling/typescript/tsconfig.react.json

@@ -2,8 +2,14 @@
   "extends": "./tsconfig.base.json",
   "compilerOptions": {
     "jsx": "react-jsx", // Use the new JSX transform for React 17+
+    // Same TS6059 trap as the next preset: `include` below covers index.d.ts and
+    // types/ alongside src/, which a src-rooted rootDir can't contain.
+    "rootDir": "${configDir}",
     "lib": ["DOM", "DOM.Iterable", "ESNext"],// Include DOM types for browser APIs
-    "types": ["react", "react-dom", "tailwindcss", "vitest"],
+    // No "tailwindcss": listing it here made tsc fail with TS2688 on every React
+    // project that doesn't install Tailwind. Tailwind v4 is CSS-first and ships
+    // no ambient types a project needs, so nothing is lost by dropping it.
+    "types": ["react", "react-dom", "vitest"],
     // ${configDir} anchors paths to the consuming project (drops deprecated baseUrl)
     "paths": {
       "~/*": ["${configDir}/src/*"],