@omnifyjp/ts 6.3.6 → 6.3.8

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/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/dist/php/type-mapper.js

@@ -128,9 +128,21 @@ export function toStoreRules(property, tableName, fkRuleType = 'integer') {
         case 'Date':
         case 'DateTime':
         case 'Timestamp':
-        case 'Time':
             rules.push('date');
             break;
+        case 'Time':
+            // #181: NOT `date`. Laravel's `date` rule runs `strtotime()` and then
+            // rejects anything that did not carry a date, so every time-only value a
+            // `time` column exists to hold — "12:00", "12:00:00", "23:59:59" — fails
+            // it, and only `null` gets through. A form with a time field it is
+            // impossible to fill, and nothing red anywhere: the migration, the model,
+            // the cast and the TypeScript were all correct.
+            //
+            // Both formats, because a client that reads a value and sends it back
+            // must be accepted: MySQL returns `H:i:s`, while an `<input type="time">`
+            // submits `H:i`.
+            rules.push('date_format:H:i,H:i:s');
+            break;
         case 'Json':
             rules.push('array');
             break;

package/package.json

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