@regardio/dev 1.11.0 → 1.11.1

package/dist/bin/lint-package.js

@@ -18,21 +18,24 @@ else {
     bin = 'npx sort-package-json';
 }
 const args = process.argv.slice(2);
-const files = args.length > 0 ? args : ['package.json'];
+const fixMode = args.includes('--fix');
+const files = args.filter((arg) => arg !== '--fix');
+const targets = files.length > 0 ? files : ['package.json'];
 try {
-    execSync(`${bin} ${files.join(' ')}`, { stdio: 'inherit' });
+    const checkFlag = fixMode ? '' : '--check';
+    execSync(`${bin} ${checkFlag} ${targets.join(' ')}`.trim(), { stdio: 'inherit' });
 }
 catch {
     process.exit(1);
 }
-function fixExportsOrder(filePath) {
+function fixExportsOrder(filePath, fix) {
     const fullPath = resolve(process.cwd(), filePath);
     if (!existsSync(fullPath))
-        return;
+        return false;
     const content = readFileSync(fullPath, 'utf-8');
     const pkg = JSON.parse(content);
     if (!pkg.exports || typeof pkg.exports !== 'object')
-        return;
+        return false;
     let modified = false;
     function reorderConditions(obj) {
         if (typeof obj !== 'object' || obj === null)
@@ -65,10 +68,19 @@ function fixExportsOrder(filePath) {
         return result;
     }
     pkg.exports = reorderConditions(pkg.exports);
-    if (modified) {
+    if (modified && fix) {
         writeFileSync(fullPath, `${JSON.stringify(pkg, null, 2)}\n`);
     }
+    return modified;
 }
-for (const file of files) {
-    fixExportsOrder(file);
+let hasExportsIssues = false;
+for (const file of targets) {
+    const needsFix = fixExportsOrder(file, fixMode);
+    if (needsFix && !fixMode) {
+        console.error(`${file}: exports condition order is incorrect (types must come before default)`);
+        hasExportsIssues = true;
+    }
+}
+if (hasExportsIssues) {
+    process.exit(1);
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://www.schemastore.org/package.json",
   "name": "@regardio/dev",
-  "version": "1.11.0",
+  "version": "1.11.1",
   "private": false,
   "description": "Regardio developer tooling for testing, linting, and build workflows",
   "keywords": [
@@ -111,7 +111,7 @@
     "npm-run-all": "4.1.5",
     "postcss": "8.5.6",
     "rimraf": "6.1.2",
-    "sort-package-json": "3.1.0",
+    "sort-package-json": "3.6.0",
     "tsx": "4.21.0",
     "typescript": "5.9.3",
     "vite": "7.3.1",

package/src/bin/lint-package.ts

@@ -26,11 +26,14 @@ if (existsSync(sortPkgBin)) {
 
 // Get args passed to this script
 const args = process.argv.slice(2);
-const files = args.length > 0 ? args : ['package.json'];
+const fixMode = args.includes('--fix');
+const files = args.filter((arg) => arg !== '--fix');
+const targets = files.length > 0 ? files : ['package.json'];
 
 // Run sort-package-json
 try {
-  execSync(`${bin} ${files.join(' ')}`, { stdio: 'inherit' });
+  const checkFlag = fixMode ? '' : '--check';
+  execSync(`${bin} ${checkFlag} ${targets.join(' ')}`.trim(), { stdio: 'inherit' });
 } catch {
   process.exit(1);
 }
@@ -38,15 +41,16 @@ try {
 /**
  * Fix exports condition order: types must come before default for TypeScript.
  * See: https://www.typescriptlang.org/docs/handbook/esm-node.html
+ * Returns true if the file needs changes.
  */
-function fixExportsOrder(filePath: string): void {
+function fixExportsOrder(filePath: string, fix: boolean): boolean {
   const fullPath = resolve(process.cwd(), filePath);
-  if (!existsSync(fullPath)) return;
+  if (!existsSync(fullPath)) return false;
 
   const content = readFileSync(fullPath, 'utf-8');
   const pkg = JSON.parse(content) as Record<string, unknown>;
 
-  if (!pkg.exports || typeof pkg.exports !== 'object') return;
+  if (!pkg.exports || typeof pkg.exports !== 'object') return false;
 
   let modified = false;
 
@@ -88,12 +92,23 @@ function fixExportsOrder(filePath: string): void {
 
   pkg.exports = reorderConditions(pkg.exports as Record<string, unknown>);
 
-  if (modified) {
+  if (modified && fix) {
     writeFileSync(fullPath, `${JSON.stringify(pkg, null, 2)}\n`);
   }
+
+  return modified;
 }
 
 // Fix exports order in each file
-for (const file of files) {
-  fixExportsOrder(file);
+let hasExportsIssues = false;
+for (const file of targets) {
+  const needsFix = fixExportsOrder(file, fixMode);
+  if (needsFix && !fixMode) {
+    console.error(`${file}: exports condition order is incorrect (types must come before default)`);
+    hasExportsIssues = true;
+  }
+}
+
+if (hasExportsIssues) {
+  process.exit(1);
 }