@rtorcato/js-tooling 2.5.0 → 2.5.1

package/dist/cli/commands/fix.js

@@ -124,9 +124,10 @@ const FIXERS = [
     },
     {
         target: 'husky',
-        description: 'Set up Husky + lint-staged (deep-merges existing lint-staged field)',
+        description: 'Set up Husky + lint-staged',
         appliesTo: ['Husky', 'lint-staged'],
         outputs: ['.husky/pre-commit', 'package.json (lint-staged field)'],
+        riskLevel: 'safe-merge',
         canFixDrift: true,
         async run({ targetDir, pkg }) {
             const pkgPath = path.join(targetDir, 'package.json');
@@ -209,9 +210,10 @@ const FIXERS = [
     },
     {
         target: 'engines',
-        description: 'Add engines.node to package.json (never overwrites)',
+        description: 'Add engines.node to package.json',
         appliesTo: ['engines.node'],
         outputs: ['package.json (engines.node field)'],
+        riskLevel: 'safe-merge',
         canFixDrift: true,
         async run({ targetDir }) {
             const result = await ensureEnginesNode(targetDir);
@@ -234,6 +236,7 @@ const FIXERS = [
         description: 'Add @rtorcato/js-tooling to devDependencies',
         appliesTo: ['package.json'],
         outputs: ['package.json (devDependencies)'],
+        riskLevel: 'safe-merge',
         canFixDrift: true,
         async run({ targetDir, pkg }) {
             const pkgPath = path.join(targetDir, 'package.json');
@@ -282,15 +285,29 @@ async function applyFixer(fixer, result, targetDir, pkg, dryRun, silent) {
     }
     return { filesWritten, dryRun: false };
 }
+function promptMessageFor(fixer, result) {
+    const risk = fixer.riskLevel ?? 'destructive';
+    if (risk === 'safe-merge') {
+        return { message: `${fixer.description} (existing fields preserved)?`, default: true };
+    }
+    if (risk === 'safe-add') {
+        return { message: `${fixer.description}?`, default: true };
+    }
+    // destructive
+    if (result.status === 'drift') {
+        return {
+            message: `⚠️  ${fixer.description} — overwrite existing file? user customizations will be lost`,
+            default: false,
+        };
+    }
+    return { message: `Apply ${fixer.description}?`, default: true };
+}
 async function confirmApply(fixer, result, assumeYes) {
     if (assumeYes)
         return true;
-    const isDrift = result.status === 'drift';
-    const message = isDrift
-        ? `⚠️  ${fixer.description} — overwrite existing file? user customizations will be lost`
-        : `Apply ${fixer.description}?`;
+    const { message, default: defaultValue } = promptMessageFor(fixer, result);
     const { confirm } = await inquirer.prompt([
-        { type: 'confirm', name: 'confirm', message, default: !isDrift },
+        { type: 'confirm', name: 'confirm', message, default: defaultValue },
     ]);
     return confirm === true;
 }

package/dist/cli/generators/build.js

@@ -53,32 +53,17 @@ console.log('Build completed!')
 `;
     await fs.writeFile(esbuildConfigPath, esbuildConfig);
 }
-async function generateViteConfig(config, targetDir) {
+export async function generateViteConfig(config, targetDir) {
     const viteConfigPath = path.join(targetDir, 'vite.config.ts');
-    let viteConfig = `import { defineConfig } from 'vite'
-`;
-    if (config.projectType === 'react-app') {
-        viteConfig += `import react from '@vitejs/plugin-react'
-`;
-    }
-    viteConfig += `
-export default defineConfig({
-  plugins: [${config.projectType === 'react-app' ? 'react()' : ''}],
-  resolve: {
-    alias: {
-      '@': '/src',
-      '~': '/src'
-    }
-  },
-  build: {
-    outDir: 'dist',
-    sourcemap: true,
-  },
-  server: {
-    port: 3000,
-    open: true
-  }
-})
+    // React apps need the plugin; we layer it on top of the shipped preset.
+    const viteConfig = config.projectType === 'react-app'
+        ? `import preset from '@rtorcato/js-tooling/vite'
+import react from '@vitejs/plugin-react'
+import { defineConfig, mergeConfig } from 'vite'
+
+export default mergeConfig(preset, defineConfig({ plugins: [react()] }))
+`
+        : `export { default } from '@rtorcato/js-tooling/vite'
 `;
     await fs.writeFile(viteConfigPath, viteConfig);
 }

package/dist/cli/generators/testing.js

@@ -38,41 +38,9 @@ async function generateJestConfig(config, targetDir) {
 `;
     await fs.writeFile(jestConfigPath, jestConfig);
 }
-async function generatePlaywrightConfig(targetDir) {
+export async function generatePlaywrightConfig(targetDir) {
     const playwrightConfigPath = path.join(targetDir, 'playwright.config.ts');
-    const playwrightConfig = `import { defineConfig } from '@playwright/test'
-
-export default defineConfig({
-  testDir: './tests/e2e',
-  fullyParallel: true,
-  forbidOnly: !!process.env.CI,
-  retries: process.env.CI ? 2 : 0,
-  workers: process.env.CI ? 1 : undefined,
-  reporter: 'html',
-  use: {
-    baseURL: 'http://localhost:3000',
-    trace: 'on-first-retry',
-  },
-  projects: [
-    {
-      name: 'chromium',
-      use: { ...devices['Desktop Chrome'] },
-    },
-    {
-      name: 'firefox',
-      use: { ...devices['Desktop Firefox'] },
-    },
-    {
-      name: 'webkit',
-      use: { ...devices['Desktop Safari'] },
-    },
-  ],
-  webServer: {
-    command: 'npm run dev',
-    url: 'http://localhost:3000',
-    reuseExistingServer: !process.env.CI,
-  },
-})
+    const playwrightConfig = `export { default } from '@rtorcato/js-tooling/playwright'
 `;
     await fs.writeFile(playwrightConfigPath, playwrightConfig);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@rtorcato/js-tooling",
-	"version": "2.5.0",
+	"version": "2.5.1",
 	"description": "JavaScript and TypeScript tooling for Node.js, React, Next.js, and Vitest.",
 	"type": "module",
 	"keywords": [
@@ -47,10 +47,14 @@
 		"tooling/eslint/*.d.mts",
 		"tooling/eslint/types.d.ts",
 		"tooling/jest-presets/**",
+		"tooling/playwright/playwright.config.mjs",
+		"tooling/playwright/playwright.config.d.mts",
 		"tooling/prettier/index.mjs",
 		"tooling/prettier/index.d.mts",
 		"tooling/typescript/*.json",
 		"tooling/typescript/reset.d.ts",
+		"tooling/vite/vite.config.mjs",
+		"tooling/vite/vite.config.d.mts",
 		"tooling/vitest/vitest.config.mjs",
 		"tooling/vitest/vitest.config.d.mts",
 		"tooling/vitest/vitest.config.react.mjs",
@@ -88,6 +92,10 @@
 			"types": "./tooling/jest-presets/node/jest-preset.d.mts",
 			"import": "./tooling/jest-presets/node/jest-preset.mjs"
 		},
+		"./playwright": {
+			"types": "./tooling/playwright/playwright.config.d.mts",
+			"import": "./tooling/playwright/playwright.config.mjs"
+		},
 		"./prettier": {
 			"types": "./tooling/prettier/index.d.mts",
 			"import": "./tooling/prettier/index.mjs"
@@ -99,6 +107,10 @@
 		"./typescript/react": "./tooling/typescript/tsconfig.react.json",
 		"./typescript/test": "./tooling/typescript/tsconfig.test.json",
 		"./typescript/reset": "./tooling/typescript/reset.d.ts",
+		"./vite": {
+			"types": "./tooling/vite/vite.config.d.mts",
+			"import": "./tooling/vite/vite.config.mjs"
+		},
 		"./vitest/config": {
 			"types": "./tooling/vitest/vitest.config.d.mts",
 			"import": "./tooling/vitest/vitest.config.mjs"

package/tooling/commitlint/commitlint.mjs

@@ -20,8 +20,9 @@ export default {
 				"test",
 			],
 		],
-		// Enforce length limits (strict)
-		"header-max-length": [2, "always", 50],
+		// Enforce length limits: 72 chars matches the conventional-commits
+		// recommendation and git's default email format
+		"header-max-length": [2, "always", 72],
 		"body-max-line-length": [2, "always", 72],
 		"footer-max-line-length": [2, "always", 72],
 		// Enforce case rules (allow common patterns)

package/tooling/playwright/playwright.config.d.mts

@@ -0,0 +1,4 @@
+import type { PlaywrightTestConfig } from '@playwright/test'
+
+declare const config: PlaywrightTestConfig
+export default config

package/tooling/playwright/playwright.config.mjs

@@ -0,0 +1,19 @@
+import { defineConfig, devices } from '@playwright/test'
+
+export default defineConfig({
+	testDir: './tests/e2e',
+	fullyParallel: true,
+	forbidOnly: !!process.env.CI,
+	retries: process.env.CI ? 2 : 0,
+	workers: process.env.CI ? 1 : undefined,
+	reporter: 'html',
+	use: {
+		baseURL: process.env.PLAYWRIGHT_BASE_URL ?? 'http://localhost:3000',
+		trace: 'on-first-retry',
+	},
+	projects: [
+		{ name: 'chromium', use: { ...devices['Desktop Chrome'] } },
+		{ name: 'firefox', use: { ...devices['Desktop Firefox'] } },
+		{ name: 'webkit', use: { ...devices['Desktop Safari'] } },
+	],
+})

package/tooling/vite/vite.config.d.mts

@@ -0,0 +1,4 @@
+import type { UserConfig } from 'vite'
+
+declare const config: UserConfig
+export default config

package/tooling/vite/vite.config.mjs

@@ -0,0 +1,18 @@
+import { defineConfig } from 'vite'
+
+export default defineConfig({
+	resolve: {
+		alias: {
+			'@': '/src',
+			'~': '/src',
+		},
+	},
+	build: {
+		outDir: 'dist',
+		sourcemap: true,
+	},
+	server: {
+		port: 3000,
+		open: true,
+	},
+})