fix(lint): resolve linting errors in create-app

involvex · involvex · commit fa1dbd8ba357 · 2025-09-20T19:10:30.000+02:00
diff --git a/.github/workflows/ci-release.yml b/.github/workflows/ci-release.yml
@@ -52,8 +52,6 @@ jobs:
 
       - run: npm test
 
-
-
   publish:
     runs-on: ubuntu-latest
     if: startsWith(github.ref, 'refs/tags/v') # Only run on version tags
diff --git a/.github/workflows/njsscan.yml b/.github/workflows/njsscan.yml
@@ -1,3 +1,5 @@
+# @format
+
 # This workflow uses actions that are not certified by GitHub.
 # They are provided by a third-party and are governed by
 # separate terms of service, privacy policy, and support
@@ -10,10 +12,10 @@ name: njsscan sarif
 
 on:
   push:
-    branches: [ "main" ]
+    branches: ['main']
   pull_request:
     # The branches below must be a subset of the branches above
-    branches: [ "main" ]
+    branches: ['main']
   schedule:
     - cron: '18 11 * * 4'
 
@@ -29,14 +31,14 @@ jobs:
     runs-on: ubuntu-latest
     name: njsscan code scanning
     steps:
-    - name: Checkout the code
-      uses: actions/checkout@v4
-    - name: nodejsscan scan
-      id: njsscan
-      uses: ajinabraham/njsscan-action@7237412fdd36af517e2745077cedbf9d6900d711
-      with:
-        args: '. --sarif --output results.sarif || true'
-    - name: Upload njsscan report
-      uses: github/codeql-action/upload-sarif@v3
-      with:
-        sarif_file: results.sarif
+      - name: Checkout the code
+        uses: actions/checkout@v4
+      - name: nodejsscan scan
+        id: njsscan
+        uses: ajinabraham/njsscan-action@7237412fdd36af517e2745077cedbf9d6900d711
+        with:
+          args: '. --sarif --output results.sarif || true'
+      - name: Upload njsscan report
+        uses: github/codeql-action/upload-sarif@v3
+        with:
+          sarif_file: results.sarif
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,17 +1,15 @@
-## [3.5.1](https://github.com/involvex/create-wizard/compare/v3.5.0...v3.5.1) (2025-09-20)
+<!-- @format -->
 
+## [3.5.1](https://github.com/involvex/create-wizard/compare/v3.5.0...v3.5.1) (2025-09-20)
 
 ### Bug Fixes
 
-* **ci:** add .gitattributes to prevent line ending issues ([eed9869](https://github.com/involvex/create-wizard/commit/eed9869de11011a67c4d25cef2af2afd2f6356e4))
-* **cli:** resolve multiple issues in create-app wizard ([013ee97](https://github.com/involvex/create-wizard/commit/013ee97cf971a00f4c37b0dfda4df48f12af63a1))
-
+- **ci:** add .gitattributes to prevent line ending issues ([eed9869](https://github.com/involvex/create-wizard/commit/eed9869de11011a67c4d25cef2af2afd2f6356e4))
+- **cli:** resolve multiple issues in create-app wizard ([013ee97](https://github.com/involvex/create-wizard/commit/013ee97cf971a00f4c37b0dfda4df48f12af63a1))
 
 ### Features
 
-* **cli:** overhaul UI and add project name argument ([7f1031e](https://github.com/involvex/create-wizard/commit/7f1031ebfa1c643f1cfcd9f4f7b11c5ab19e902c))
-
-
+- **cli:** overhaul UI and add project name argument ([7f1031e](https://github.com/involvex/create-wizard/commit/7f1031ebfa1c643f1cfcd9f4f7b11c5ab19e902c))
 
 <!-- @format -->
 
diff --git a/dist/create-wizard.js b/dist/create-wizard.js
@@ -7638,6 +7638,7 @@ var Ce = async (t, n2) => {
 
 // scripts/create-app.js
 import debug2 from "debug";
+import { createRequire } from "module";
 
 // scripts/create-plugin.js
 import { join } from "path";
@@ -7897,14 +7898,7 @@ export default [
           {
             templates: () => fe({
               message: "Select .gitignore templates:",
-              options: [
-                "node",
-                "visualstudiocode",
-                "windows",
-                "macos",
-                "linux",
-                "jetbrain"
-              ],
+              options: ["node", "visualstudiocode", "windows", "macos", "linux", "jetbrain"],
               initialValue: ["node", "visualstudiocode"]
             })
           },
@@ -10872,16 +10866,41 @@ describe('Example Test', () => {
 function updatePackageJsonScripts(newScripts) {
   const packageJsonPath = join3(process.cwd(), "package.json");
   if (fs.existsSync(packageJsonPath)) {
-    const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, "utf-8"));
-    packageJson.scripts = { ...packageJson.scripts, ...newScripts };
-    fs.writeFileSync(packageJsonPath, JSON.stringify(packageJson, null, 2));
+    const packageJson2 = JSON.parse(fs.readFileSync(packageJsonPath, "utf-8"));
+    packageJson2.scripts = { ...packageJson2.scripts, ...newScripts };
+    fs.writeFileSync(packageJsonPath, JSON.stringify(packageJson2, null, 2));
   } else {
     console.error("package.json not found. Please run npm init first.");
   }
 }
 
 // scripts/create-app.js
+var require2 = createRequire(import.meta.url);
+var packageJson = require2("../package.json");
 var args = process.argv.slice(2);
+if (args.includes("-h") || args.includes("--help")) {
+  console.log(`
+    Usage: create-wizard [projectName] [options]
+
+    Initializes a new project using an interactive wizard.
+
+    Arguments:
+      projectName         The name of the project to create. If provided, the wizard will skip the project name prompt.
+
+    Options:
+      -v, --version         Output the current version and exit.
+      -h, --help            Display this help message and exit.
+      --plugin              Configure and install a new plugin (e.g., Prettier, ESLint).
+      --create-test         Set up a new test framework for the project.
+      --license             Generate a new LICENSE file.
+      --debug=TRUE          Enable debug logging.
+  `);
+  process.exit(0);
+}
+if (args.includes("-v") || args.includes("--version")) {
+  console.log(packageJson.version);
+  process.exit(0);
+}
 var debugFlag = args.find((arg) => arg.startsWith("--debug="));
 var isDebugEnabled = debugFlag === "--debug=TRUE" || process.env.DEBUG === "TRUE";
 if (isDebugEnabled) {
@@ -11122,21 +11141,21 @@ async function main4(deps) {
     }
   }
   const packageJsonPath = join4(projectDir, "package.json");
-  let packageJson = JSON.parse(fs2.readFileSync(packageJsonPath, "utf-8"));
+  let packageJson2 = JSON.parse(fs2.readFileSync(packageJsonPath, "utf-8"));
   const userDependencies = answers.dependencies.reduce((acc, dep) => {
     acc[dep.name] = dep.version;
     return acc;
   }, {});
-  packageJson.dependencies = {
-    ...packageJson.dependencies,
+  packageJson2.dependencies = {
+    ...packageJson2.dependencies,
     ...userDependencies
   };
-  packageJson.devDependencies = {
-    ...packageJson.devDependencies
+  packageJson2.devDependencies = {
+    ...packageJson2.devDependencies
   };
   if (answers.includeTypeScript) {
-    packageJson.devDependencies = {
-      ...packageJson.devDependencies,
+    packageJson2.devDependencies = {
+      ...packageJson2.devDependencies,
       typescript: "^5.3.3"
     };
     fs2.writeFileSync(
@@ -11158,13 +11177,13 @@ async function main4(deps) {
     );
   }
   if (answers.includeEslint) {
-    packageJson.devDependencies = {
-      ...packageJson.devDependencies,
+    packageJson2.devDependencies = {
+      ...packageJson2.devDependencies,
       eslint: "^8.56.0",
       globals: "^15.2.0"
     };
     if (answers.includePrettier) {
-      packageJson.devDependencies["eslint-config-prettier"] = "^9.1.0";
+      packageJson2.devDependencies["eslint-config-prettier"] = "^9.1.0";
     }
     const isESM = [
       "vite",
@@ -11176,10 +11195,10 @@ async function main4(deps) {
       "docusaurus-site"
     ].includes(answers.template);
     if (isESM) {
-      packageJson.type = "module";
+      packageJson2.type = "module";
     }
     let eslintConfigContent;
-    if (packageJson.type === "module") {
+    if (packageJson2.type === "module") {
       eslintConfigContent = `import js from "@eslint/js";
 import globals from "globals";
 ${answers.includePrettier ? "import prettierConfig from 'eslint-config-prettier';\n" : ""}
@@ -11201,15 +11220,15 @@ module.exports = [
 `;
     }
     fs2.writeFileSync(join4(projectDir, "eslint.config.js"), eslintConfigContent);
-    packageJson.scripts = {
-      ...packageJson.scripts,
+    packageJson2.scripts = {
+      ...packageJson2.scripts,
       lint: "eslint .",
       "lint:fix": "eslint . --fix"
     };
   }
   if (answers.includePrettier) {
-    packageJson.devDependencies = {
-      ...packageJson.devDependencies,
+    packageJson2.devDependencies = {
+      ...packageJson2.devDependencies,
       prettier: "^3.2.5"
     };
     fs2.writeFileSync(
@@ -11224,8 +11243,8 @@ module.exports = [
         2
       )
     );
-    packageJson.scripts = {
-      ...packageJson.scripts,
+    packageJson2.scripts = {
+      ...packageJson2.scripts,
       "format:check": "prettier --check .",
       format: "prettier --write ."
     };
@@ -11320,7 +11339,7 @@ test-job:
 `
     );
   }
-  fs2.writeFileSync(packageJsonPath, JSON.stringify(packageJson, null, 2));
+  fs2.writeFileSync(packageJsonPath, JSON.stringify(packageJson2, null, 2));
   if (answers.includeDebugConfig) {
     const vscodeDir = join4(projectDir, ".vscode");
     fs2.mkdirSync(vscodeDir, { recursive: true });
@@ -11344,10 +11363,10 @@ test-job:
       )
     );
   }
-  const allDependencies = Object.entries(packageJson.dependencies || {}).map(
+  const allDependencies = Object.entries(packageJson2.dependencies || {}).map(
     ([name, version]) => `${name}@${version}`
   );
-  const allDevDependencies = Object.entries(packageJson.devDependencies || {}).map(
+  const allDevDependencies = Object.entries(packageJson2.devDependencies || {}).map(
     ([name, version]) => `${name}@${version}`
   );
   if (allDependencies.length > 0) {
diff --git a/package.json b/package.json
@@ -40,14 +40,14 @@
     "lint:fix": "npx eslint . --config eslint.config.js --fix",
     "check": "npm run lint:fix && npm run format",
     "coverage": "vitest run --coverage",
-    "prebuild": "npm run check", 
+    "prebuild": "npm run check",
     "build": "npx esbuild scripts/create-app.js --bundle --platform=node --target=node22 --format=esm --external:yoctocolors-cjs --external:mute-stream --external:safer-buffer --external:iconv-lite --external:cross-spawn --external:graceful-fs --external:fs-extra --external:inquirer --external:cli-progress --external:debug --outfile=dist/create-wizard.js --banner:js=\"#!/usr/bin/env node\"",
     "start": "node scripts/create-app.js",
     "docs:start": "cd docs && npm start",
     "docs:build": "cd docs && npm run build",
     "docs:serve": "cd docs && npm run serve",
     "template:create": "node scripts/create-template.js",
-    "prerelease": "npm run check", 
+    "prerelease": "npm run check",
     "release": "node scripts/release.js",
     "debug": "cross-env DEBUG=TRUE && node scripts/create-app.js --debug=TRUE ",
     "review": "node scripts/ai-review.js"
diff --git a/scripts/ai-review.js b/scripts/ai-review.js
@@ -1,39 +1,48 @@
-import ModelClient, { isUnexpected } from "@azure-rest/ai-inference";
-import { AzureKeyCredential } from "@azure/core-auth";
-import env from 'dotenv';
-env.config();
+/** @format */
 
-const token = env.GH_TOKEN || process.env["GH_TOKEN"];
-const endpoint = "https://models.github.ai/inference";
-const model = "openai/gpt-4.1";
+import ModelClient, { isUnexpected } from '@azure-rest/ai-inference'
+import { AzureKeyCredential } from '@azure/core-auth'
+import env from 'dotenv'
+env.config()
 
-export async function main() {
+const token = env.GH_TOKEN || process.env['GH_TOKEN']
+const endpoint = 'https://models.github.ai/inference'
+const model = 'openai/gpt-4.1'
 
-  const client = ModelClient(
-    endpoint,
-    new AzureKeyCredential(token),
-  );
+export async function main() {
+  const client = ModelClient(endpoint, new AzureKeyCredential(token))
 
-  const response = await client.path("/chat/completions").post({
+  const response = await client.path('/chat/completions').post({
     body: {
       messages: [
-        { role:"system", content: "you are a helpful code review assistant" },
-        { role:"user", content: "summarize the project " + '../package.json' + './create-app.js' + './create-plugin.js' + './create-test-setup.js' + '../CHANGELOG.md' + '../README.md' + '../docs/docs/installation.md' + '../docs/docs/usage.md' }
+        { role: 'system', content: 'you are a helpful code review assistant' },
+        {
+          role: 'user',
+          content:
+            'summarize the project ' +
+            '../package.json' +
+            './create-app.js' +
+            './create-plugin.js' +
+            './create-test-setup.js' +
+            '../CHANGELOG.md' +
+            '../README.md' +
+            '../docs/docs/installation.md' +
+            '../docs/docs/usage.md',
+        },
       ],
       temperature: 1,
       top_p: 1,
-      model: model
-    }
-  });
+      model: model,
+    },
+  })
 
   if (isUnexpected(response)) {
-    throw response.body.error;
+    throw response.body.error
   }
 
-  console.log(response.body.choices[0].message.content);
+  console.log(response.body.choices[0].message.content)
 }
 
-main().catch((err) => {
-  console.error("The sample encountered an error:", err);
-});
-
+main().catch(err => {
+  console.error('The sample encountered an error:', err)
+})
diff --git a/scripts/create-app.js b/scripts/create-app.js
@@ -140,7 +140,7 @@ export async function main(deps) {
     }
   }
 
-  p.intro('Let\'s create a new project!')
+  p.intro("Let's create a new project!")
 
   const cliArgs = process.argv.slice(2).filter(arg => !arg.startsWith('--'))
   const projectNameFromArg = cliArgs[0]
@@ -650,4 +650,4 @@ if (process.argv[1] === fileURLToPath(import.meta.url)) {
     // If no specific flags are provided by the user, run main
     main({})
   }
-}
+}
diff --git a/scripts/create-plugin.js b/scripts/create-plugin.js
@@ -235,7 +235,8 @@ export default [
                 options: ['CommonJS', 'ESNext', 'NodeNext'],
                 initialValue: 'CommonJS',
               }),
-            strict: () => p.confirm({ message: 'Enable strict type-checking?', initialValue: true }),
+            strict: () =>
+              p.confirm({ message: 'Enable strict type-checking?', initialValue: true }),
             esModuleInterop: () =>
               p.confirm({ message: 'Enable esModuleInterop?', initialValue: true }),
           },
@@ -301,14 +302,7 @@ export default [
             templates: () =>
               p.multiselect({
                 message: 'Select .gitignore templates:',
-                options: [
-                  'node',
-                  'visualstudiocode',
-                  'windows',
-                  'macos',
-                  'linux',
-                  'jetbrain',
-                ],
+                options: ['node', 'visualstudiocode', 'windows', 'macos', 'linux', 'jetbrain'],
                 initialValue: ['node', 'visualstudiocode'],
               }),
           },
@@ -348,4 +342,4 @@ export default [
     p.log.success('Dependencies installed successfully.')
   }
   p.outro('Done!')
-}
+}
diff --git a/scripts/release.js b/scripts/release.js
diff --git a/tests/create-app.test.js b/tests/create-app.test.js
diff --git a/tests/create-plugin.test.js b/tests/create-plugin.test.js

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,7 @@ export async function main(deps) {`
`140`	`140`	`}`
`141`	`141`	`}`
`142`	`142`
`143`		`- p.intro('Let\'s create a new project!')`
	`143`	`+ p.intro("Let's create a new project!")`
`144`	`144`
`145`	`145`	`const cliArgs = process.argv.slice(2).filter(arg => !arg.startsWith('--'))`
`146`	`146`	`const projectNameFromArg = cliArgs[0]`
`@@ -650,4 +650,4 @@ if (process.argv[1] === fileURLToPath(import.meta.url)) {`
`650`	`650`	`// If no specific flags are provided by the user, run main`
`651`	`651`	`main({})`
`652`	`652`	`}`
`653`		`-}`
	`653`	`+}`