@omnifyjp/ts 7.0.0 → 7.0.2

package/dist/cli.js

@@ -30,7 +30,7 @@ import { parse as parseYaml } from 'yaml';
 import { generateTypeScript } from './generator.js';
 import { generatePhp, derivePhpConfig } from './php/index.js';
 import { evaluateShadowingConflicts } from './php/shadowing-conflicts.js';
-import { hasEditableSubclassForBase, pruneDisabledScaffoldLayerBases, pruneOrphanServiceFilesIfEnabled } from './php/orphan-cleanup.js';
+import { collectReferencedClasses, hasEditableSubclassForBase, pruneDisabledScaffoldLayerBases, pruneOrphanServiceFilesIfEnabled } from './php/orphan-cleanup.js';
 import { findSiblingWithSameBasename, findAppRootAncestor } from './php/sibling-skip.js';
 import { resolveInput, sniffInputKind } from './input-resolver.js';
 function resolveFromConfig(configPath) {
@@ -228,9 +228,12 @@ program
         console.log(`\n[Laravel] Generating PHP files`);
         // #255: a disabled scaffold layer still regenerates the bases that
         // hand-written subclasses extend, so they track the schema.
+        // Read before anything is written, so the answer reflects the project's
+        // own code, not this run's output.
         const phpConfigForGen = derivePhpConfig(laravelOverrides);
+        const referencedClasses = collectReferencedClasses(configDir, phpConfigForGen);
         const phpFiles = generatePhp(input, laravelOverrides, {
-            keepDisabledBase: (file) => hasEditableSubclassForBase(configDir, phpConfigForGen, file),
+            keepDisabledBase: (file) => hasEditableSubclassForBase(phpConfigForGen, referencedClasses, file),
         });
         let phpCreated = 0;
         let phpOverwritten = 0;
@@ -339,7 +342,7 @@ program
         // pollutes Composer autoload. Prune base files automatically; warn
         // about editable `*Service.php` (may contain user code).
         const phpConfig = derivePhpConfig(laravelOverrides);
-        for (const summary of pruneDisabledScaffoldLayerBases(configDir, phpConfig, phpFiles)) {
+        for (const summary of pruneDisabledScaffoldLayerBases(configDir, phpConfig, phpFiles, referencedClasses)) {
             if (summary.prunedCount > 0) {
                 console.log(`  pruned ${summary.prunedCount} stale ${summary.label}(s) — layer disabled`);
             }

package/dist/php/factory-generator.js

@@ -178,7 +178,7 @@ class FileFactory extends Factory
     public function definition(): array
     {
         $extension = fake()->randomElement(['jpg', 'png', 'pdf', 'docx']);
-        $originalName = fake()->word() . '.' . $extension;
+        $originalName = fake()->word().'.'.$extension;
         $mimeTypes = [
             'jpg' => 'image/jpeg',
             'png' => 'image/png',
@@ -189,7 +189,7 @@ class FileFactory extends Factory
         return [
             'collection' => 'default',
             'disk' => 'public',
-            'path' => 'files/' . now()->format('Y/m') . '/' . Str::uuid() . '.' . $extension,
+            'path' => 'files/'.now()->format('Y/m').'/'.Str::uuid().'.'.$extension,
             'original_name' => $originalName,
             'mime_type' => $mimeTypes[$extension] ?? 'application/octet-stream',
             'size' => fake()->numberBetween(1024, 10485760),

package/dist/php/faker-mapper.js

@@ -57,7 +57,7 @@ export function compoundFaker(propName, expansion, sourceType) {
                 `'${snakeName}_branch_code' => ${jaFaker}->regexify('[0-9]{3}'),`,
                 `'${snakeName}_account_type' => ${jaFaker}->randomElement(['1', '2', '4']),`,
                 `'${snakeName}_account_number' => ${jaFaker}->regexify('[0-9]{7}'),`,
-                `'${snakeName}_account_holder' => ${jaFaker}->lastKanaName() . ' ' . ${jaFaker}->firstKanaName(),`,
+                `'${snakeName}_account_holder' => ${jaFaker}->lastKanaName().' '.${jaFaker}->firstKanaName(),`,
             ];
         default:
             return genericCompoundFaker(snakeName, expansion);
@@ -96,7 +96,7 @@ function stringFaker(propName, snakeName, property) {
     if (lower.includes('url') || lower.includes('website'))
         return `'${snakeName}' => fake()->url(),`;
     if (lower.includes('path') || lower.includes('file'))
-        return `'${snakeName}' => 'uploads/' . fake()->uuid() . '.jpg',`;
+        return `'${snakeName}' => 'uploads/'.fake()->uuid().'.jpg',`;
     if (propName === 'name' || propName === 'title')
         return `'${snakeName}' => fake()->sentence(3),`;
     if (lower.includes('name'))
@@ -183,7 +183,7 @@ function japaneseAddressFaker(snakeName, expansion) {
     else {
         lines.push(`'${snakeName}_prefecture' => fake()->randomElement(['hokkaido', 'aomori', 'iwate', 'miyagi', 'akita', 'yamagata', 'fukushima', 'ibaraki', 'tochigi', 'gunma', 'saitama', 'chiba', 'tokyo', 'kanagawa', 'niigata', 'toyama', 'ishikawa', 'fukui', 'yamanashi', 'nagano', 'gifu', 'shizuoka', 'aichi', 'mie', 'shiga', 'kyoto', 'osaka', 'hyogo', 'nara', 'wakayama', 'tottori', 'shimane', 'okayama', 'hiroshima', 'yamaguchi', 'tokushima', 'kagawa', 'ehime', 'kochi', 'fukuoka', 'saga', 'nagasaki', 'kumamoto', 'oita', 'miyazaki', 'kagoshima', 'okinawa']),`);
     }
-    lines.push(`'${snakeName}_address1' => ${jaFaker}->city() . ${jaFaker}->ward(),`);
+    lines.push(`'${snakeName}_address1' => ${jaFaker}->city().${jaFaker}->ward(),`);
     lines.push(`'${snakeName}_address2' => ${jaFaker}->streetAddress(),`);
     lines.push(`'${snakeName}_address3' => ${jaFaker}->optional()->secondaryAddress(),`);
     return lines;

package/dist/php/file-cleanup-generator.js

@@ -68,12 +68,16 @@ class CleanupFiles extends Command
 
         if ($phase === null || $phase === '1') {
             $r = $this->phase1SoftDelete($dryRun);
-            if ($r !== self::SUCCESS) $result = $r;
+            if ($r !== self::SUCCESS) {
+                $result = $r;
+            }
         }
 
         if ($phase === null || $phase === '2') {
             $r = $this->phase2Purge($dryRun);
-            if ($r !== self::SUCCESS) $result = $r;
+            if ($r !== self::SUCCESS) {
+                $result = $r;
+            }
         }
 
         return $result;

package/dist/php/orphan-cleanup.d.ts

@@ -45,13 +45,24 @@ export interface DisabledLayerPruneSummary {
  * When a scaffold layer is disabled (issue #252), remove stale auto-generated
  * BASE files that have no matching user-editable subclass on disk.
  */
-export declare function pruneDisabledScaffoldLayerBases(configDir: string, config: PhpConfig, generated: readonly GeneratedFile[]): DisabledLayerPruneSummary[];
+export declare function pruneDisabledScaffoldLayerBases(configDir: string, config: PhpConfig, generated: readonly GeneratedFile[], referenced?: ReadonlySet<string>): DisabledLayerPruneSummary[];
 /**
- * Issue #255: does a hand-written subclass exist on disk for this generated
- * base of a disabled scaffold layer? Same lookup the prune above uses to keep
- * a base, so a base is regenerated exactly when it would otherwise be kept.
+ * Issue #255: is this generated base of a disabled scaffold layer still used by
+ * hand-written code? Same question the prune above asks before deleting a base,
+ * so a base is regenerated exactly when it would otherwise be kept.
  */
-export declare function hasEditableSubclassForBase(configDir: string, config: PhpConfig, file: GeneratedFile): boolean;
+export declare function hasEditableSubclassForBase(config: PhpConfig, referenced: ReadonlySet<string>, file: GeneratedFile): boolean;
+/**
+ * Every class hand-written PHP under the Laravel root names in a `use` import
+ * or a fully-qualified `extends` (#260).
+ *
+ * "Is this base still used?" used to be answered by looking for a file named
+ * after it in the layer's configured directories. Real projects keep their
+ * subclasses wherever they like — `app/Policies/`, `app/Services/Promotion/` —
+ * so a used base looked unused and was deleted under running code. The import
+ * is what PHP resolves, so the import is what we check.
+ */
+export declare function collectReferencedClasses(configDir: string, config: PhpConfig): Set<string>;
 /**
  * Delete orphan `*ServiceBase.php` files; flag orphan editable services
  * for warning. Pure-ish: takes the generated file list (= source of

package/dist/php/orphan-cleanup.js

@@ -19,7 +19,7 @@
  * warning is printed so the user can decide whether to clean up by hand.
  */
 import { existsSync, readFileSync, readdirSync, statSync, unlinkSync } from 'node:fs';
-import { join, resolve, basename, relative, sep } from 'node:path';
+import { join, resolve, basename } from 'node:path';
 /** Matches the header emitted by `generateEditableService` in service-generator.ts */
 export const OMNIFY_EDITABLE_SERVICE_MARKER = 'SAFE TO EDIT - This file is never overwritten by Omnify.';
 const EXTENDS_SERVICE_BASE_RE = /extends\s+[\w\\]+ServiceBase\b/;
@@ -65,80 +65,7 @@ function collectModularLayerDirs(configDir, config, category) {
     }
     return dirs;
 }
-const BASE_SUBDIRS_SKIP = new Set(['Base', 'OmnifyBase']);
-function findEditableSubclassFile(roots, fileName, excludeAbsPath) {
-    for (const root of roots) {
-        const found = findFileAbsolutePath(root, fileName);
-        if (found && found !== excludeAbsPath)
-            return true;
-    }
-    return false;
-}
-function findFileAbsolutePath(dir, fileName) {
-    if (!existsSync(dir))
-        return null;
-    let info;
-    try {
-        info = statSync(dir);
-    }
-    catch {
-        return null;
-    }
-    if (!info.isDirectory()) {
-        return basename(dir) === fileName ? resolve(dir) : null;
-    }
-    for (const entry of readdirSync(dir)) {
-        if (BASE_SUBDIRS_SKIP.has(entry))
-            continue;
-        const full = join(dir, entry);
-        try {
-            info = statSync(full);
-        }
-        catch {
-            continue;
-        }
-        if (info.isDirectory()) {
-            const nested = findFileAbsolutePath(full, fileName);
-            if (nested)
-                return nested;
-        }
-        else if (entry === fileName) {
-            return resolve(full);
-        }
-    }
-    return null;
-}
-function collectEditableSearchRoots(configDir, config, layer, modularCategory, baseAbsPath) {
-    const roots = [];
-    const add = (p) => {
-        const abs = resolve(configDir, p);
-        if (!roots.includes(abs))
-            roots.push(abs);
-    };
-    add(layer.userEditablePath);
-    if (layer.path !== layer.userEditablePath) {
-        add(layer.path);
-    }
-    if (config.structure === 'modular') {
-        const modulesAbs = resolve(configDir, config.modules.path);
-        const rel = relative(modulesAbs, baseAbsPath);
-        if (rel && !rel.startsWith('..') && !rel.startsWith(`..${sep}`)) {
-            const segments = rel.split(/[/\\]/);
-            if (segments.length >= 2 && segments[1] === modularCategory) {
-                const schemaFolder = segments[0];
-                add(join(layer.userEditablePath, schemaFolder));
-                add(join(layer.path, schemaFolder));
-            }
-        }
-    }
-    return roots;
-}
-function baseToEditableFileName(baseFileName, baseSuffix, editableSuffix) {
-    if (!baseFileName.endsWith(baseSuffix))
-        return null;
-    return `${baseFileName.slice(0, -baseSuffix.length)}${editableSuffix}`;
-}
-function pruneDisabledLayerBasesInRoots(configDir, config, spec, roots, expectedAbs) {
+function pruneDisabledLayerBasesInRoots(spec, roots, expectedAbs, referenced) {
     let pruned = 0;
     let kept = 0;
     for (const root of roots) {
@@ -150,13 +77,17 @@ function pruneDisabledLayerBasesInRoots(configDir, config, spec, roots, expected
             const fullPath = join(root, entry);
             if (expectedAbs.has(fullPath))
                 continue;
-            const editableName = spec.toEditableFileName(entry);
-            if (editableName) {
-                const editableRoots = collectEditableSearchRoots(configDir, config, spec.layerConfig, spec.modularCategory, fullPath);
-                if (findEditableSubclassFile(editableRoots, editableName, fullPath)) {
-                    kept += 1;
-                    continue;
-                }
+            let content = '';
+            try {
+                content = readFileSync(fullPath, 'utf-8');
+            }
+            catch {
+                continue;
+            }
+            const fqcn = declaredClassName(content);
+            if (fqcn && referenced.has(fqcn)) {
+                kept += 1;
+                continue;
             }
             try {
                 unlinkSync(fullPath);
@@ -173,7 +104,7 @@ function pruneDisabledLayerBasesInRoots(configDir, config, spec, roots, expected
  * When a scaffold layer is disabled (issue #252), remove stale auto-generated
  * BASE files that have no matching user-editable subclass on disk.
  */
-export function pruneDisabledScaffoldLayerBases(configDir, config, generated) {
+export function pruneDisabledScaffoldLayerBases(configDir, config, generated, referenced = collectReferencedClasses(configDir, config)) {
     const expectedAbs = new Set();
     for (const f of generated) {
         expectedAbs.add(resolve(configDir, f.path));
@@ -186,7 +117,7 @@ export function pruneDisabledScaffoldLayerBases(configDir, config, generated) {
             ...collectModularLayerDirs(configDir, config, spec.modularCategory),
             resolve(configDir, spec.legacyBasePath),
         ];
-        const { pruned, kept } = pruneDisabledLayerBasesInRoots(configDir, config, spec, roots, expectedAbs);
+        const { pruned, kept } = pruneDisabledLayerBasesInRoots(spec, roots, expectedAbs, referenced);
         if (pruned > 0 || kept > 0) {
             summaries.push({
                 layer: spec.layer,
@@ -199,22 +130,94 @@ export function pruneDisabledScaffoldLayerBases(configDir, config, generated) {
     return summaries;
 }
 /**
- * Issue #255: does a hand-written subclass exist on disk for this generated
- * base of a disabled scaffold layer? Same lookup the prune above uses to keep
- * a base, so a base is regenerated exactly when it would otherwise be kept.
+ * Issue #255: is this generated base of a disabled scaffold layer still used by
+ * hand-written code? Same question the prune above asks before deleting a base,
+ * so a base is regenerated exactly when it would otherwise be kept.
  */
-export function hasEditableSubclassForBase(configDir, config, file) {
+export function hasEditableSubclassForBase(config, referenced, file) {
     const name = basename(file.path);
-    for (const spec of scaffoldLayerSpecs(config)) {
-        if (spec.enabled || !spec.matchesBaseFile(name))
+    const isDisabledScaffoldBase = scaffoldLayerSpecs(config).some((spec) => !spec.enabled && spec.matchesBaseFile(name));
+    if (!isDisabledScaffoldBase)
+        return false;
+    const fqcn = declaredClassName(file.content);
+    return fqcn !== null && referenced.has(fqcn);
+}
+/**
+ * Every class hand-written PHP under the Laravel root names in a `use` import
+ * or a fully-qualified `extends` (#260).
+ *
+ * "Is this base still used?" used to be answered by looking for a file named
+ * after it in the layer's configured directories. Real projects keep their
+ * subclasses wherever they like — `app/Policies/`, `app/Services/Promotion/` —
+ * so a used base looked unused and was deleted under running code. The import
+ * is what PHP resolves, so the import is what we check.
+ */
+export function collectReferencedClasses(configDir, config) {
+    // Paths resolve against configDir, which for a workspace run is a cache
+    // directory; the Laravel app itself sits at rootPath.
+    const laravelRoot = resolve(configDir, config.rootPath);
+    const referenced = new Set();
+    for (const dir of REFERENCE_SCAN_DIRS) {
+        walkPhpFiles(resolve(laravelRoot, dir), (content) => {
+            if (GENERATED_MARKER.test(content))
+                return;
+            for (const m of content.matchAll(/^use\s+\\?([A-Za-z_][\w\\]*)(?:\s+as\s+\w+)?\s*;/gm)) {
+                referenced.add(m[1]);
+            }
+            for (const m of content.matchAll(/\bextends\s+\\([A-Za-z_][\w\\]*)/g)) {
+                referenced.add(m[1]);
+            }
+            // An unqualified parent with no import resolves in the file's own namespace.
+            const ns = content.match(/^namespace\s+([^;\s]+)\s*;/m)?.[1];
+            for (const m of content.matchAll(/\bextends\s+([A-Za-z_]\w*)\b/g)) {
+                referenced.add(ns ? `${ns}\\${m[1]}` : m[1]);
+            }
+        });
+    }
+    return referenced;
+}
+const REFERENCE_SCAN_DIRS = ['app', 'routes', 'tests', 'database', 'config'];
+const GENERATED_MARKER = /auto-generated by Omnify/i;
+const SKIP_DIRS = new Set(['vendor', 'node_modules', 'storage', '.git']);
+function walkPhpFiles(dir, visit) {
+    let entries;
+    try {
+        entries = readdirSync(dir);
+    }
+    catch {
+        return;
+    }
+    for (const entry of entries) {
+        if (SKIP_DIRS.has(entry))
             continue;
-        const editableName = spec.toEditableFileName(name);
-        if (!editableName)
-            return false;
-        const roots = collectEditableSearchRoots(configDir, config, spec.layerConfig, spec.modularCategory, resolve(configDir, file.path));
-        return findEditableSubclassFile(roots, editableName, resolve(configDir, file.path));
+        const full = join(dir, entry);
+        let info;
+        try {
+            info = statSync(full);
+        }
+        catch {
+            continue;
+        }
+        if (info.isDirectory()) {
+            walkPhpFiles(full, visit);
+        }
+        else if (entry.endsWith('.php')) {
+            try {
+                visit(readFileSync(full, 'utf-8'));
+            }
+            catch {
+                // unreadable file: nothing to learn from it
+            }
+        }
     }
-    return false;
+}
+/** `Namespace\Class` a PHP file declares, or null. */
+function declaredClassName(content) {
+    const ns = content.match(/^namespace\s+([^;\s]+)\s*;/m);
+    const cls = content.match(/^(?:(?:abstract|final|readonly)\s+)*(?:class|interface|trait|enum)\s+(\w+)/m);
+    if (!cls)
+        return null;
+    return ns ? `${ns[1]}\\${cls[1]}` : cls[1];
 }
 const SCAFFOLD_LAYER_BASE_SUFFIX = {
     request: 'RequestBase.php',
@@ -223,32 +226,17 @@ const SCAFFOLD_LAYER_BASE_SUFFIX = {
     controller: 'ControllerBase.php',
     service: 'ServiceBase.php',
 };
-const SCAFFOLD_LAYER_EDITABLE_SUFFIX = {
-    request: 'Request.php',
-    resource: 'Resource.php',
-    policy: 'Policy.php',
-    controller: 'Controller.php',
-    service: 'Service.php',
-};
 /**
  * Legacy `flatBase` puts regenerated bases directly at `layer.basePath` with
  * the same filename as the user-editable stub (different directory). That
  * directory is generated-only: resolveBaseClass() always targets basePath and
  * only per-schema base classes are written there (issue #96).
  */
-function disabledLayerFileMatchers(config, layerConfig, baseSuffix, editableSuffix) {
-    const modularMatches = (name) => name.endsWith(baseSuffix);
-    const modularToEditable = (name) => baseToEditableFileName(name, baseSuffix, editableSuffix);
-    if (config.structure === 'modular') {
-        return { matchesBaseFile: modularMatches, toEditableFileName: modularToEditable };
-    }
-    if (layerConfig.flatBase) {
-        return {
-            matchesBaseFile: (name) => name.endsWith('.php'),
-            toEditableFileName: (name) => (name.endsWith('.php') ? name : null),
-        };
+function disabledLayerFileMatchers(config, layerConfig, baseSuffix) {
+    if (config.structure !== 'modular' && layerConfig.flatBase) {
+        return { matchesBaseFile: (name) => name.endsWith('.php') };
     }
-    return { matchesBaseFile: modularMatches, toEditableFileName: modularToEditable };
+    return { matchesBaseFile: (name) => name.endsWith(baseSuffix) };
 }
 function scaffoldLayerSpecs(config) {
     const layers = [
@@ -295,8 +283,7 @@ function scaffoldLayerSpecs(config) {
     ];
     return layers.map((entry) => {
         const baseSuffix = SCAFFOLD_LAYER_BASE_SUFFIX[entry.layer];
-        const editableSuffix = SCAFFOLD_LAYER_EDITABLE_SUFFIX[entry.layer];
-        const matchers = disabledLayerFileMatchers(config, entry.layerConfig, baseSuffix, editableSuffix);
+        const matchers = disabledLayerFileMatchers(config, entry.layerConfig, baseSuffix);
         return { ...entry, ...matchers };
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@omnifyjp/ts",
-  "version": "7.0.0",
+  "version": "7.0.2",
   "description": "TypeScript model type generator from Omnify schemas.json",
   "type": "module",
   "main": "dist/index.js",
@@ -9,7 +9,7 @@
     "omnify-ts": "dist/cli.js"
   },
   "scripts": {
-    "build": "tsc",
+    "build": "node -e \"require('fs').rmSync('dist', { recursive: true, force: true })\" && tsc",
     "dev": "tsc --watch",
     "generate-l12": "node dist/cli.js --config ../../examples/l12/omnify.yaml",
     "test": "vitest run"

package/dist/php/pint-format.d.ts

@@ -1,57 +0,0 @@
-/**
- * Auto-format generated PHP files via Laravel Pint. Issue #100 v5.8.20.
- *
- * Reporter scenario (godx-jp/godx-kintai 2026-04-26 audit): every
- * `omnify generate --force` produces 150+ PHP files that fail Laravel
- * Pint's standard preset (`vendor/bin/pint --test`):
- *
- *   - ordered_imports — `use` statements not alphabetised
- *   - fully_qualified_strict_types — exception types referenced via FQN
- *   - class_attributes_separation — missing blank lines between members
- *   - function_declaration — function-keyword spacing
- *   - unary_operator_spaces — `! \$x` vs `!\$x`
- *   - no_unused_imports — codegen leaves the occasional dangling import
- *   - single_line_empty_body — empty class bodies on multiple lines
- *   - self_static_accessor — `self::` vs `static::` on enums
- *   - blank_line_before_statement — enum `case` separators
- *
- * Auto-detect: when `<rootPath>/vendor/bin/pint` exists, run it on the
- * files we just wrote. Same Pint preset the project's CI already uses,
- * so no risk of formatter drift between codegen and CI checks.
- *
- * Opt-out: future config field; for now `chmod -x vendor/bin/pint` or
- * deleting it disables the auto-run.
- */
-/**
- * Resolve the `vendor/bin/pint` path for the project. Walks up from
- * the first written file until we find a sibling `vendor/bin/pint`,
- * stopping at filesystem root or after 12 levels (defensive against
- * pathological inputs). Returns the absolute path or null.
- */
-export declare function findPintBinary(anyWrittenPhpPath: string): string | null;
-export interface PintFormatResult {
-    ran: boolean;
-    pintPath: string | null;
-    /** Files passed to pint. */
-    formattedCount: number;
-    /** Pint's exit status if it ran. 0 = success. */
-    exitStatus: number | null;
-    /** Captured stderr when pint failed (status != 0). */
-    errorMessage?: string;
-    /**
-     * Why the run did not happen, when `ran` is false. The caller needs the three apart: only
-     * `pint-not-found` is worth telling a human about, and it is the one that silently changes what
-     * gets written to disk (godx-jp/godx-corebooks#108 — 927 files, reproduced).
-     */
-    skipped?: 'no-php-files' | 'opted-out' | 'pint-not-found';
-}
-/**
- * Run Laravel Pint on the given PHP files (absolute paths). No-op if
- * Pint isn't installed in the project. Pint is invoked with `--quiet`
- * so it stays out of the way unless something breaks.
- *
- * Pint takes a list of files (or directories) and formats them in
- * place. It uses the project's `pint.json` config if present, else the
- * Laravel preset.
- */
-export declare function runPintFormat(writtenPhpPaths: string[]): PintFormatResult;

package/dist/php/pint-format.js

@@ -1,102 +0,0 @@
-/**
- * Auto-format generated PHP files via Laravel Pint. Issue #100 v5.8.20.
- *
- * Reporter scenario (godx-jp/godx-kintai 2026-04-26 audit): every
- * `omnify generate --force` produces 150+ PHP files that fail Laravel
- * Pint's standard preset (`vendor/bin/pint --test`):
- *
- *   - ordered_imports — `use` statements not alphabetised
- *   - fully_qualified_strict_types — exception types referenced via FQN
- *   - class_attributes_separation — missing blank lines between members
- *   - function_declaration — function-keyword spacing
- *   - unary_operator_spaces — `! \$x` vs `!\$x`
- *   - no_unused_imports — codegen leaves the occasional dangling import
- *   - single_line_empty_body — empty class bodies on multiple lines
- *   - self_static_accessor — `self::` vs `static::` on enums
- *   - blank_line_before_statement — enum `case` separators
- *
- * Auto-detect: when `<rootPath>/vendor/bin/pint` exists, run it on the
- * files we just wrote. Same Pint preset the project's CI already uses,
- * so no risk of formatter drift between codegen and CI checks.
- *
- * Opt-out: future config field; for now `chmod -x vendor/bin/pint` or
- * deleting it disables the auto-run.
- */
-import { existsSync, statSync } from 'node:fs';
-import { spawnSync } from 'node:child_process';
-import { dirname, join } from 'node:path';
-/**
- * Resolve the `vendor/bin/pint` path for the project. Walks up from
- * the first written file until we find a sibling `vendor/bin/pint`,
- * stopping at filesystem root or after 12 levels (defensive against
- * pathological inputs). Returns the absolute path or null.
- */
-export function findPintBinary(anyWrittenPhpPath) {
-    let cur = dirname(anyWrittenPhpPath);
-    for (let i = 0; i < 12; i++) {
-        const candidate = join(cur, 'vendor', 'bin', 'pint');
-        if (existsSync(candidate)) {
-            try {
-                const st = statSync(candidate);
-                if (st.isFile())
-                    return candidate;
-            }
-            catch {
-                // ignore stat errors and keep climbing
-            }
-        }
-        const parent = dirname(cur);
-        if (parent === cur)
-            break;
-        cur = parent;
-    }
-    return null;
-}
-/**
- * Run Laravel Pint on the given PHP files (absolute paths). No-op if
- * Pint isn't installed in the project. Pint is invoked with `--quiet`
- * so it stays out of the way unless something breaks.
- *
- * Pint takes a list of files (or directories) and formats them in
- * place. It uses the project's `pint.json` config if present, else the
- * Laravel preset.
- */
-export function runPintFormat(writtenPhpPaths) {
-    if (writtenPhpPaths.length === 0) {
-        return { ran: false, pintPath: null, formattedCount: 0, exitStatus: null, skipped: 'no-php-files' };
-    }
-    // Opt-out (#174). The auto-run hides what the generators actually emit: a
-    // template that Pint rewrites looks fine here and produces churn in every
-    // project where Pint is absent or does not run. `OMNIFY_NO_PINT=1` is how
-    // the repo's own fixture check measures the raw output.
-    if (process.env.OMNIFY_NO_PINT === '1') {
-        return { ran: false, pintPath: null, formattedCount: 0, exitStatus: null, skipped: 'opted-out' };
-    }
-    // Resolution walks up from a written file. Trying only the first one makes
-    // it depend on emission order — a single file outside the Laravel root and
-    // the whole run goes unformatted, silently. Try each until one resolves.
-    const pintPath = writtenPhpPaths.reduce((found, path) => found ?? findPintBinary(path), null);
-    if (!pintPath) {
-        return { ran: false, pintPath: null, formattedCount: 0, exitStatus: null, skipped: 'pint-not-found' };
-    }
-    // Pint's working dir should be the Laravel project root (where
-    // composer.json + pint.json live). The pint binary is at
-    // `<projectRoot>/vendor/bin/pint`, so projectRoot is two parents up.
-    const projectRoot = dirname(dirname(dirname(pintPath)));
-    // Pass each file as its own arg. Pint accepts multiple file/dir
-    // arguments; capping the batch is unnecessary up to a few hundred
-    // files (well under the OS arg limit).
-    const result = spawnSync(pintPath, ['--quiet', ...writtenPhpPaths], {
-        cwd: projectRoot,
-        stdio: ['ignore', 'pipe', 'pipe'],
-        encoding: 'utf-8',
-    });
-    const exitStatus = result.status ?? null;
-    return {
-        ran: true,
-        pintPath,
-        formattedCount: writtenPhpPaths.length,
-        exitStatus,
-        errorMessage: exitStatus !== 0 ? (result.stderr || result.stdout || undefined) : undefined,
-    };
-}

package/dist/prettier-format.d.ts

@@ -1,23 +0,0 @@
-/**
- * Auto-format generated TypeScript files via the consuming project's Prettier.
- * Issue omnify-jp/omnify-go#211 (mirrors php/pint-format.ts).
- */
-/**
- * Resolve `node_modules/.bin/prettier` by walking up from a written `.ts` file.
- */
-export declare function findPrettierBinary(anyWrittenTsPath: string): string | null;
-/** Directory containing the `node_modules` that owns this Prettier binary. */
-export declare function prettierProjectRoot(prettierPath: string): string;
-export interface PrettierFormatResult {
-    ran: boolean;
-    prettierPath: string | null;
-    formattedCount: number;
-    exitStatus: number | null;
-    errorMessage?: string;
-    skipped?: 'no-ts-files' | 'opted-out' | 'prettier-not-found';
-}
-/**
- * Run the project's Prettier on the given TypeScript files (absolute paths).
- * Silent no-op when Prettier is not installed; never throws.
- */
-export declare function runPrettierFormat(writtenTsPaths: string[]): PrettierFormatResult;

package/dist/prettier-format.js

@@ -1,102 +0,0 @@
-/**
- * Auto-format generated TypeScript files via the consuming project's Prettier.
- * Issue omnify-jp/omnify-go#211 (mirrors php/pint-format.ts).
- */
-import { existsSync, statSync } from 'node:fs';
-import { spawnSync } from 'node:child_process';
-import { dirname, join } from 'node:path';
-/** Max file paths per Prettier invocation (stay under typical ARG_MAX). */
-const PRETTIER_BATCH_SIZE = 100;
-/**
- * Resolve `node_modules/.bin/prettier` by walking up from a written `.ts` file.
- */
-export function findPrettierBinary(anyWrittenTsPath) {
-    let cur = dirname(anyWrittenTsPath);
-    for (let i = 0; i < 12; i++) {
-        const candidate = join(cur, 'node_modules', '.bin', 'prettier');
-        if (existsSync(candidate)) {
-            try {
-                const st = statSync(candidate);
-                if (st.isFile())
-                    return candidate;
-            }
-            catch {
-                // ignore stat errors and keep climbing
-            }
-        }
-        const parent = dirname(cur);
-        if (parent === cur)
-            break;
-        cur = parent;
-    }
-    return null;
-}
-/** Directory containing the `node_modules` that owns this Prettier binary. */
-export function prettierProjectRoot(prettierPath) {
-    return dirname(dirname(dirname(prettierPath)));
-}
-function runPrettierBatch(prettierPath, projectRoot, files) {
-    const result = spawnSync(prettierPath, ['--write', ...files], {
-        cwd: projectRoot,
-        stdio: ['ignore', 'pipe', 'pipe'],
-        encoding: 'utf-8',
-    });
-    const exitStatus = result.status ?? null;
-    return {
-        exitStatus,
-        errorMessage: exitStatus !== 0 ? (result.stderr || result.stdout || undefined) : undefined,
-    };
-}
-/**
- * Run the project's Prettier on the given TypeScript files (absolute paths).
- * Silent no-op when Prettier is not installed; never throws.
- */
-export function runPrettierFormat(writtenTsPaths) {
-    if (writtenTsPaths.length === 0) {
-        return {
-            ran: false,
-            prettierPath: null,
-            formattedCount: 0,
-            exitStatus: null,
-            skipped: 'no-ts-files',
-        };
-    }
-    if (process.env.OMNIFY_NO_PRETTIER === '1') {
-        return {
-            ran: false,
-            prettierPath: null,
-            formattedCount: 0,
-            exitStatus: null,
-            skipped: 'opted-out',
-        };
-    }
-    const prettierPath = writtenTsPaths.reduce((found, path) => found ?? findPrettierBinary(path), null);
-    if (!prettierPath) {
-        return {
-            ran: false,
-            prettierPath: null,
-            formattedCount: 0,
-            exitStatus: null,
-            skipped: 'prettier-not-found',
-        };
-    }
-    const projectRoot = prettierProjectRoot(prettierPath);
-    let exitStatus = 0;
-    let errorMessage;
-    for (let i = 0; i < writtenTsPaths.length; i += PRETTIER_BATCH_SIZE) {
-        const batch = writtenTsPaths.slice(i, i + PRETTIER_BATCH_SIZE);
-        const batchResult = runPrettierBatch(prettierPath, projectRoot, batch);
-        if (batchResult.exitStatus !== 0) {
-            exitStatus = batchResult.exitStatus;
-            errorMessage = batchResult.errorMessage;
-            break;
-        }
-    }
-    return {
-        ran: true,
-        prettierPath,
-        formattedCount: writtenTsPaths.length,
-        exitStatus,
-        errorMessage,
-    };
-}