create-next-shadcn-kit 0.1.2 → 0.3.0

package/README.md

@@ -57,7 +57,7 @@ npx create-next-shadcn-kit my-app --pnpm
 
 ## Requirements
 
-- Node.js **20+** (Next.js 16 requires it)
+- Node.js **22+** (Next.js 16 plus the pnpm/lint-staged toolchain require it)
 - Network access (to fetch `create-next-app` and `shadcn`)
 - Works with **npm, pnpm, yarn, bun**. All are tested end-to-end.
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "create-next-shadcn-kit",
-    "version": "0.1.2",
+    "version": "0.3.0",
     "description": "Create a Next.js app with shadcn/ui pre-integrated — zero config.",
     "keywords": [
         "nextjs",
@@ -36,10 +36,11 @@
         "LICENSE"
     ],
     "engines": {
-        "node": ">=20.0.0"
+        "node": ">=22.0.0"
     },
     "scripts": {
         "start": "node bin/index.js",
+        "test": "node --test",
         "test:local": "node bin/index.js test-app --yes"
     },
     "dependencies": {

package/src/index.js

@@ -45,10 +45,22 @@ function devCommand(pm) {
     return `${pm} dev`;
 }
 
+// Minimum Node supported by the generated project's toolchain. Husky's
+// lint-staged pulls in listr2 (Node >=22.13) and pnpm 11's binary needs
+// node:sqlite (Node >=22.13), so the practical floor is Node 22.
+export const MIN_NODE_MAJOR = 22;
+
+export function isSupportedNode(version) {
+    const major = Number(String(version).replace(/^v/, "").split(".")[0]);
+    return Number.isFinite(major) && major >= MIN_NODE_MAJOR;
+}
+
 function assertNodeVersion() {
-    const major = Number(process.versions.node.split(".")[0]);
-    if (major < 20) {
-        throw new Error(`Node.js 20+ is required (Next.js 16 requires it). You are running ${process.versions.node}.`);
+    if (!isSupportedNode(process.versions.node)) {
+        throw new Error(
+            `Node.js ${MIN_NODE_MAJOR}+ is required (Next.js 16 and the pnpm/lint-staged toolchain need it). ` +
+                `You are running ${process.versions.node}.`
+        );
     }
 }
 

package/src/prompts.js

@@ -28,7 +28,20 @@ const onCancel = () => {
     process.exit(1);
 };
 
+function assertValidName(name) {
+    if (!isValidProjectName(name)) {
+        throw new Error(
+            `Invalid project name: "${name}". Use letters, numbers, dashes, dots, ` +
+                `tildes or underscores (optionally an npm scope) — no path separators.`
+        );
+    }
+}
+
 export async function promptConfig(args) {
+    // A name passed on the CLI bypasses the interactive prompt's validation,
+    // so guard it here before it ever reaches path.resolve()/spawn().
+    if (args.projectName !== undefined) assertValidName(args.projectName);
+
     if (args.yes) {
         return {
             projectName: args.projectName || "my-app",

package/src/scaffold.js

@@ -7,6 +7,14 @@ export async function scaffold(config) {
     const cwd = process.cwd();
     const projectPath = path.resolve(cwd, config.projectName);
 
+    // pnpm 10+/11 defaults strictDepBuilds=true, so unapproved dependency build
+    // scripts (e.g. sharp, unrs-resolver pulled in by shadcn) abort the install
+    // in a non-interactive run. Downgrade it to a warning for all child pnpm
+    // processes; respect an explicit override if the user already set one.
+    if (config.packageManager === "pnpm" && process.env.pnpm_config_strict_dep_builds === undefined) {
+        process.env.pnpm_config_strict_dep_builds = "false";
+    }
+
     if (fs.existsSync(projectPath) && fs.readdirSync(projectPath).length > 0) {
         throw new Error(`Directory "${config.projectName}" already exists and is not empty.`);
     }
@@ -66,6 +74,15 @@ function storeDirFor(projectPath, config) {
     return config.srcDir ? path.join(projectPath, "src", "store") : path.join(projectPath, "store");
 }
 
+// Turn a tsconfig-style import alias pattern (e.g. "@/*", "~/*") into the
+// prefix used in import statements ("@", "~"). Falls back to "@" if empty.
+export function aliasPrefixFor(importAlias) {
+    const prefix = String(importAlias || "@/*")
+        .replace(/\*$/, "")
+        .replace(/\/$/, "");
+    return prefix || "@";
+}
+
 async function setupRedux(projectPath, config) {
     console.log();
     console.log(pc.cyan("◆") + " Setting up Redux Toolkit...");
@@ -170,11 +187,12 @@ export const useAppSelector: TypedUseSelectorHook<RootState> = useSelector;
     }
 
     const appDir = appDirFor(projectPath, config);
+    const storeImport = `${aliasPrefixFor(config.importAlias)}/store`;
     const providers = ts
         ? `"use client";
 
 import { Provider } from "react-redux";
-import { store } from "@/store";
+import { store } from "${storeImport}";
 
 export function Providers({ children }: { children: React.ReactNode }) {
   return <Provider store={store}>{children}</Provider>;
@@ -183,7 +201,7 @@ export function Providers({ children }: { children: React.ReactNode }) {
         : `"use client";
 
 import { Provider } from "react-redux";
-import { store } from "@/store";
+import { store } from "${storeImport}";
 
 export function Providers({ children }) {
   return <Provider store={store}>{children}</Provider>;
@@ -294,6 +312,14 @@ async function setupHusky(projectPath, config) {
     delete pkg["lint-staged"];
     fs.writeFileSync(pkgPath, JSON.stringify(pkg, null, 2) + "\n");
 
+    // Husky needs this project to be its own git repository. create-next-app
+    // skips git init when nested inside an existing repo, and rolls it back if it
+    // can't make the initial commit (e.g. no git identity configured). Ensure one
+    // so the hook setup below doesn't fail.
+    if (!fs.existsSync(path.join(projectPath, ".git"))) {
+        await run("git", ["init"], { cwd: projectPath });
+    }
+
     const runner = runnerFor(config.packageManager);
     await run(runner.cmd, [...runner.args, "husky", "install"], {
         cwd: projectPath,

package/src/utils.js

@@ -1,5 +1,10 @@
 import { spawn } from "node:child_process";
 
+// Flags that take a value via the space form (e.g. `--state redux`). Every
+// other `--flag` is treated as a boolean so it never swallows the following
+// positional argument (e.g. `--pnpm my-app` keeps "my-app" as the project name).
+const VALUE_FLAGS = new Set(["state"]);
+
 export function parseArgs(argv) {
     const positional = [];
     const flags = {};
@@ -13,7 +18,7 @@ export function parseArgs(argv) {
             } else {
                 const key = arg.slice(2);
                 const next = argv[i + 1];
-                if (next && !next.startsWith("-")) {
+                if (VALUE_FLAGS.has(key) && next !== undefined && !next.startsWith("-")) {
                     flags[key] = next;
                     i++;
                 } else {