@omnifyjp/ts 6.3.7 → 6.3.9

package/dist/cli.js

@@ -322,7 +322,17 @@ program
         // Laravel Pint when it's installed in the project. Same preset
         // the project's CI already uses, so codegen output stops failing
         // `pint --test` and devs no longer need a manual sweep after
-        // every `omnify generate`. Silent no-op when Pint isn't found.
+        // every `omnify generate`.
+        //
+        // NOT a silent no-op when Pint is missing any more. It used to be, and that cost a
+        // consumer several investigation sessions: their CI regenerated without Pint on PATH,
+        // wrote 927 files differing from the committed (Pint-formatted) tree, and the only
+        // evidence was one ABSENT log line. It was read as a Linux/Darwin build divergence and
+        // chased as far as "pick a canonical platform" before anyone ran the two commands that
+        // settle it (godx-jp/godx-corebooks#108, reproduced exactly: hide `vendor/bin/pint`,
+        // regenerate, 927 files; restore it, 0).
+        //
+        // The opt-out (`OMNIFY_NO_PINT=1`) stays quiet — that one is a choice, not a surprise.
         if (writtenPaths.length > 0) {
             const pintResult = runPintFormat(writtenPaths);
             if (pintResult.ran) {
@@ -337,6 +347,11 @@ program
                     }
                 }
             }
+            else if (pintResult.skipped === 'pint-not-found') {
+                console.warn(`  [omnify] Laravel Pint not found — ${writtenPaths.length} PHP file(s) written UNFORMATTED.`);
+                console.warn('  If this project commits generated PHP, the output will differ from a tree formatted by Pint.');
+                console.warn('  Run `composer install` before `omnify generate`, or set OMNIFY_NO_PINT=1 to silence this deliberately.');
+            }
         }
         // Orphan cleanup: when a schema is removed, renamed, flipped from
         // kind:object → kind:pivot, or opted out via `options.service: false`,

package/dist/php/model-generator.d.ts

@@ -4,5 +4,6 @@
 import { SchemaReader } from './schema-reader.js';
 import type { GeneratedFile, PhpConfig } from './types.js';
 /** Generate base model and user model for all project-owned object schemas,
- *  plus user models for package schemas (extending the package model). */
+ *  plus, for each package schema the project extends, a user model that
+ *  extends the package model and carries the extend's columns. */
 export declare function generateModels(reader: SchemaReader, config: PhpConfig): GeneratedFile[];

package/dist/php/model-generator.js

@@ -8,24 +8,34 @@ import { buildRelation } from './relation-builder.js';
 import { baseFile, userFile, shadowingPropertyConflicts, resolveModularBasePath, resolveModularBaseNamespace, resolveSharedBaseNamespace, resolveGlobalTraitNamespace, resolveGlobalEnumNamespace, resolveBaseClass, resolveEditableClass, nestByGroup, nestEditableByGroup, } from './types.js';
 import { enumClassName } from './enum-generator.js';
 /** Generate base model and user model for all project-owned object schemas,
- *  plus user models for package schemas (extending the package model). */
+ *  plus, for each package schema the project extends, a user model that
+ *  extends the package model and carries the extend's columns. */
 export function generateModels(reader, config) {
     const files = [];
     // Project-owned schemas: base model + user model
     for (const [name, schema] of Object.entries(reader.getProjectObjectSchemas())) {
         files.push(...generateForSchema(name, schema, reader, config));
     }
-    // Package schemas: user model only (extends package model), plus a trait
-    // carrying anything a project-level `kind: extend` added to them (#164).
+    // Package schemas the project extends with `kind: extend`: a trait carrying
+    // the added columns (#164) plus a user model in the project namespace that
+    // `use`s it — the trait needs a class to live in, and the package's own
+    // model cannot be that class once it is installed under `vendor/`.
+    //
+    // A package schema the project does NOT extend gets nothing: the package
+    // model is the model, and relations already point at it. Every such
+    // schema used to get an empty `extends Package<Name>` stub anyway, and a
+    // workspace of three packages saw sixteen of them on every fresh checkout
+    // after deleting them (#182). A project that wants a local subclass for one
+    // of these writes the five lines itself, or extends the schema.
     for (const [name, schema] of Object.entries(reader.getPackageObjectSchemas())) {
         const pkgNs = reader.resolveModelNamespace(name, config.models.namespace);
-        if (pkgNs !== config.models.namespace) {
-            const extended = extendedPropertyNames(schema);
-            if (extended.length > 0) {
-                files.push(generateExtendAttributesTrait(name, schema, extended, reader, config));
-            }
-            files.push(generatePackageUserModel(name, pkgNs, config, extended.length > 0, schema.kind === 'pivot'));
-        }
+        if (pkgNs === config.models.namespace)
+            continue;
+        const extended = extendedPropertyNames(schema);
+        if (extended.length === 0)
+            continue;
+        files.push(generateExtendAttributesTrait(name, schema, extended, reader, config));
+        files.push(generatePackageUserModel(name, pkgNs, config, schema.kind === 'pivot'));
     }
     return files;
 }
@@ -1056,8 +1066,9 @@ function buildAccessors(expandedProperties) {
     }
     return methods.join('\n');
 }
-/** Generate a user model for a package schema that extends the package's model class. */
-function generatePackageUserModel(name, packageNamespace, config, hasExtendAttributes = false, isPivot = false) {
+/** Generate a user model for a package schema the project extends: it
+ *  extends the package's model class and `use`s the extend-attributes trait. */
+function generatePackageUserModel(name, packageNamespace, config, isPivot = false) {
     const modelName = toPascalCase(name);
     const modelNamespace = config.models.namespace;
     // #178: a `kind: pivot` schema's editable model is emitted under the
@@ -1074,12 +1085,8 @@ function generatePackageUserModel(name, packageNamespace, config, hasExtendAttri
     // predates its extend keeps the file it has, and `generateModels` warns with
     // the one line to add — overwriting somebody's model to add an import is the
     // worse of the two.
-    const traitImport = hasExtendAttributes
-        ? `use ${modelNamespace}\\Concerns\\${traitName};\n`
-        : '';
-    const traitUse = hasExtendAttributes
-        ? `    use ${traitName};\n\n`
-        : '';
+    const traitImport = `use ${modelNamespace}\\Concerns\\${traitName};\n`;
+    const traitUse = `    use ${traitName};\n\n`;
     const content = `<?php
 
 namespace ${modelNamespace};
@@ -1097,16 +1104,14 @@ class ${modelName} extends Package${modelName}
 ${traitUse}    // Add your custom methods here
 }
 `;
-    return userFile(`${config.models.path}/${modelName}.php`, content, hasExtendAttributes
-        ? {
-            needle: `use ${traitName};`,
-            hint: `this project's \`kind: extend\` adds columns to the package-owned `
-                + `${modelName} schema, but this model predates the trait that carries `
-                + `them, so those columns are still not fillable or cast. Add `
-                + `\`use ${config.models.namespace}\\Concerns\\${traitName};\` and `
-                + `\`use ${traitName};\` inside the class (#164).`,
-        }
-        : undefined);
+    return userFile(`${config.models.path}/${modelName}.php`, content, {
+        needle: `use ${traitName};`,
+        hint: `this project's \`kind: extend\` adds columns to the package-owned `
+            + `${modelName} schema, but this model predates the trait that carries `
+            + `them, so those columns are still not fillable or cast. Add `
+            + `\`use ${config.models.namespace}\\Concerns\\${traitName};\` and `
+            + `\`use ${traitName};\` inside the class (#164).`,
+    });
 }
 /**
  * Detect whether a schema uses nested set (tree structure).

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

@@ -38,6 +38,12 @@ export interface PintFormatResult {
     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

package/dist/php/pint-format.js

@@ -63,21 +63,21 @@ export function findPintBinary(anyWrittenPhpPath) {
  */
 export function runPintFormat(writtenPhpPaths) {
     if (writtenPhpPaths.length === 0) {
-        return { ran: false, pintPath: null, formattedCount: 0, exitStatus: null };
+        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 };
+        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 };
+        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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@omnifyjp/ts",
-  "version": "6.3.7",
+  "version": "6.3.9",
   "description": "TypeScript model type generator from Omnify schemas.json",
   "type": "module",
   "main": "dist/index.js",