@kitschpatrol/prettier-config 5.0.0 → 5.0.1

package/bin/cli.js

@@ -627,7 +627,7 @@ var source_default = chalk;
 import { cosmiconfig } from "cosmiconfig";
 import { execa } from "execa";
 import fse2 from "fs-extra";
-import fs2 from "node:fs";
+import fs3 from "node:fs";
 import path3 from "node:path";
 import { PassThrough } from "node:stream";
 import { fileURLToPath as fileURLToPath3 } from "node:url";
@@ -5541,7 +5541,7 @@ var Yargs = YargsFactory(esm_default);
 var yargs_default = Yargs;
 
 // ../../package.json
-var version = "5.0.0";
+var version = "5.0.1";
 
 // ../../src/execa-utils.ts
 function isErrorExecaError(error) {
@@ -5733,6 +5733,29 @@ function getCwdOverride(option) {
   return process.cwd();
 }
 
+// ../../src/prettier-utils.ts
+import fs2 from "node:fs/promises";
+async function formatTextAndSaveFile(filePath, content) {
+  try {
+    const { default: prettier } = await import("prettier");
+    const prettierConfig = await prettier.resolveConfig(filePath);
+    const formattedContent = await prettier.format(content, {
+      filepath: filePath,
+      ...prettierConfig
+    });
+    await fs2.writeFile(filePath, formattedContent, "utf8");
+  } catch {
+    console.warn(`Skipped formatting ${filePath} since Prettier is not installed.`);
+  }
+}
+async function formatFileInPlace(filePath) {
+  try {
+    const content = await fs2.readFile(filePath, "utf8");
+    await formatTextAndSaveFile(filePath, content);
+  } catch {
+  }
+}
+
 // ../../src/stream-utils.ts
 import { Transform } from "node:stream";
 function createStreamTransform(logPrefix, logColor) {
@@ -5778,7 +5801,11 @@ async function executeFunctionCommand(logStream, positionalArguments, optionFlag
     targetStream = subStream;
   }
   if (verbose) {
-    targetStream.write(source_default.bold(`Running: "${command2.name}()"`));
+    targetStream.write(
+      source_default.bold(
+        `Running: "${command2.name}() with Positional arguments: ${String(positionalArguments)} and Option flags: ${String(optionFlags)}"`
+      )
+    );
   }
   try {
     exitCode = await command2.execute(targetStream, positionalArguments, optionFlags);
@@ -5896,9 +5923,6 @@ async function copyAndMergeInitFiles(logStream, location, configFile, configPack
   const source = path3.join(path3.dirname(sourcePackage), "init");
   const destination = path3.dirname(destinationPackage);
   const hasConfigLocationOption = (location === "file" || location === "package") && configFile !== void 0 && configPackageJson !== void 0;
-  logStream.write(`Adding initial configuration files from:
-"${source}" \u2192 "${destination}"
-`);
   try {
     if (hasConfigLocationOption) {
       const configKey = Object.keys(configPackageJson)[0];
@@ -5910,7 +5934,8 @@ Package config key "${configKey}" \u2192 "${destination}" (Because --location is
 `
         );
         const mergedPackageJson = merge(destinationPackageJson, configPackageJson);
-        fse2.writeJSONSync(destinationPackage, mergedPackageJson, { spaces: 2 });
+        fse2.writeJSONSync(destinationPackage, mergedPackageJson, { spaces: "	" });
+        await formatFileInPlace(destinationPackage);
       } else {
         const destinationPackageJson = fse2.readJsonSync(destinationPackage);
         if (Object.keys(destinationPackageJson).includes(configKey)) {
@@ -5920,14 +5945,28 @@ Package config key "${configKey}" in "${destination}" (Because --location is set
 `
           );
           delete destinationPackageJson[configKey];
-          fse2.writeJSONSync(destinationPackage, destinationPackageJson, { spaces: 2 });
+          fse2.writeJSONSync(destinationPackage, destinationPackageJson, { spaces: "	" });
+          await formatFileInPlace(destinationPackage);
         }
       }
     }
+    const sourceExists = await fse2.pathExists(source);
+    if (!sourceExists) {
+      return 0;
+    }
+    const sourceFiles = await fse2.readdir(source);
+    if (sourceFiles.length === 0) {
+      logStream.write(`Source directory "${source}" is empty.
+`);
+      return 0;
+    }
+    logStream.write(`Adding initial configuration files from:
+"${source}" \u2192 "${destination}"
+`);
     await fse2.copy(source, destination, {
-      filter(source2, destination2) {
-        const isFile = fs2.statSync(source2).isFile();
-        const destinationExists = fs2.existsSync(destination2);
+      async filter(source2, destination2) {
+        const isFile = fs3.statSync(source2).isFile();
+        const destinationExists = fs3.existsSync(destination2);
         if (isFile) {
           if (hasConfigLocationOption && location === "package" && source2.includes(configFile)) {
             if (destinationExists) {
@@ -5946,25 +5985,28 @@ Package config key "${configKey}" in "${destination}" (Because --location is set
             }
             return false;
           }
-          if (destinationExists && destination2.includes(".vscode/") && path3.extname(destination2) === ".json") {
+          if (destinationExists && (destination2.includes(".vscode/") || destination2.includes("package.json")) && path3.extname(destination2) === ".json") {
             logStream.write(`Merging: 
 "${source2}" \u2192 "${destination2}"
 `);
             const sourceJson = fse2.readJSONSync(source2);
             const destinationJson = fse2.readJSONSync(destination2);
             const mergedJson = merge(destinationJson, sourceJson);
-            fse2.writeJSONSync(destination2, mergedJson, { spaces: 2 });
+            fse2.writeJSONSync(destination2, mergedJson, { spaces: "	" });
+            await formatFileInPlace(destination2);
             return false;
           }
           if (destinationExists) {
             logStream.write(`Overwriting: 
 "${source2}" \u2192 "${destination2}"
 `);
+            await formatFileInPlace(destination2);
             return true;
           }
           logStream.write(`Copying: 
 "${source2}" \u2192 "${destination2}"
 `);
+          await formatFileInPlace(destination2);
           return true;
         }
         return true;
@@ -6004,11 +6046,12 @@ async function buildCommands(commandDefinition2) {
       // Command: init.locationOptionFlag ? 'init [--location]' : 'init',
       describe: init.description ?? `Initialize by copying starter config files to your project root${init.locationOptionFlag ? " or to your package.json file." : "."}`,
       async handler(argv) {
+        const location = init.locationOptionFlag ? argv.location : void 0;
         const copyAndMergeInitFilesCommand = {
-          async execute(logStream2) {
+          async execute(logStream2, _, optionFlags) {
             return copyAndMergeInitFiles(
               logStream2,
-              init.locationOptionFlag ? argv.location : void 0,
+              optionFlags.at(1),
               init.configFile,
               init.configPackageJson
             );
@@ -6018,7 +6061,7 @@ async function buildCommands(commandDefinition2) {
         const exitCode = await executeCommands(
           logStream,
           [],
-          [],
+          location === void 0 ? [] : ["--location", location],
           [copyAndMergeInitFilesCommand, ...init.commands ?? []]
         );
         process.exit(exitCode);

package/dist/index.js

@@ -1,6 +1,17 @@
 import { deepmerge } from 'deepmerge-ts';
 import { homedir } from 'node:os';
+import { sortOrder as sortPackageJsonSortOrder } from 'sort-package-json';
 // export { commandDefinition } from './command.js'
+/**
+ * Merge custom keys into the `sort-package-json` `order` array. Where
+ * duplicated, delete existing and prioritize new keys.
+ */
+function customizeSortOrder(keys, newKeys) {
+    // If new keys are in keys, remove them
+    const filteredKeys = keys.filter((key) => !newKeys.includes(key));
+    // Append new keys to the end
+    return [...filteredKeys, ...newKeys];
+}
 const sharedPrettierConfig = {
     bracketSpacing: true,
     overrides: [
@@ -37,6 +48,22 @@ const sharedPrettierConfig = {
                 plugins: ['prettier-plugin-sh'],
             },
         },
+        // Make this match eslint 'json-package/order-properties'
+        // https://github.com/matzkoh/prettier-plugin-packagejson/issues/188
+        // This must stay in sync with packages/eslint-config/src/configs/json.ts
+        {
+            files: 'package.json',
+            options: {
+                packageSortOrder: customizeSortOrder(sortPackageJsonSortOrder, [
+                    'cspell',
+                    'knip',
+                    'mdat',
+                    'prettier',
+                    'remarkConfig',
+                    'stylelint',
+                ]),
+            },
+        },
     ],
     plugins: [
         '@prettier/plugin-php',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kitschpatrol/prettier-config",
-  "version": "5.0.0",
+  "version": "5.0.1",
   "description": "Prettier configuration for @kitschpatrol/shared-config.",
   "keywords": [
     "shared-config",
@@ -55,7 +55,8 @@
     "prettier-plugin-sql": "^0.18.1",
     "prettier-plugin-svelte": "^3.3.3",
     "prettier-plugin-tailwindcss": "^0.6.11",
-    "prettier-plugin-toml": "^2.0.1"
+    "prettier-plugin-toml": "^2.0.1",
+    "sort-package-json": "^2.14.0"
   },
   "engines": {
     "node": ">=22.0.0",