@omnifyjp/omnify 5.8.10 → 5.8.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@omnifyjp/omnify",
-  "version": "5.8.10",
+  "version": "5.8.11",
   "description": "Schema-driven code generation for Laravel, TypeScript, and SQL",
   "license": "MIT",
   "repository": {
@@ -36,10 +36,10 @@
     "zod": "^3.24.0"
   },
   "optionalDependencies": {
-    "@omnifyjp/omnify-darwin-arm64": "5.8.10",
-    "@omnifyjp/omnify-darwin-x64": "5.8.10",
-    "@omnifyjp/omnify-linux-x64": "5.8.10",
-    "@omnifyjp/omnify-linux-arm64": "5.8.10",
-    "@omnifyjp/omnify-win32-x64": "5.8.10"
+    "@omnifyjp/omnify-darwin-arm64": "5.8.11",
+    "@omnifyjp/omnify-darwin-x64": "5.8.11",
+    "@omnifyjp/omnify-linux-x64": "5.8.11",
+    "@omnifyjp/omnify-linux-arm64": "5.8.11",
+    "@omnifyjp/omnify-win32-x64": "5.8.11"
   }
 }

package/ts-dist/php/controller-generator.js

@@ -42,7 +42,13 @@ function generateBaseController(name, schema, reader, config) {
     const baseNs = config.structure === 'modular'
         ? resolveModularBaseNamespace(config, name, 'Controllers', config.controllers.baseNamespace)
         : groupNestBase(config.controllers.baseNamespace);
-    const modelNs = groupNestBase(config.models.namespace);
+    // Issue #98 v5.8.11: controller base methods type-hint the
+    // USER-EDITABLE Model — same reasoning as the service-generator
+    // signature fix. Without this, the controller's `show` /
+    // `update` / `destroy` route bindings resolve as base instances
+    // and every project-owned service / policy that takes the
+    // editable type fails with TypeError.
+    const modelNs = groupNestEditable(config.models.userEditableNamespace, config.models);
     const serviceBaseNs = config.structure === 'modular'
         ? resolveModularBaseNamespace(config, name, 'Services', config.services.baseNamespace)
         : groupNestBase(config.services.baseNamespace);

package/ts-dist/php/service-generator.js

@@ -510,15 +510,24 @@ function generateBaseService(name, schema, reader, config) {
     const baseNs = config.structure === 'modular'
         ? resolveModularBaseNamespace(config, name, 'Services', config.services.baseNamespace)
         : nestByGroup({ path: '', namespace: config.services.baseNamespace }, schema.group).namespace;
-    // Issue #98 v5.8.4: cross-layer Model imports must include the
-    // schema's group folder so they resolve to where Models actually
-    // live now. Pre-fix the service base emitted
-    // `use App\Omnify\Models\Banner;` while the model file was at
-    // `app/Omnify/Models/Cms/Banner.php` — autoload failed at runtime
-    // for every consumer (TypeError + missing-file include).
+    // Issue #98 v5.8.11: service base methods type-hint the USER-EDITABLE
+    // Model class (`App\Models\Banner`), NOT the grouped base
+    // (`App\Omnify\Models\Cms\Banner`). PHP signature contravariance:
+    // when the user-editable child service overrides a base method,
+    // PHP requires the override's parameter type to be the SAME or
+    // WIDER than the parent's. The user-editable class is what app code
+    // type-hints against everywhere else (factory $model since v5.8.9,
+    // relations since v5.8.10, controllers, tests). Without this fix,
+    // every override of `find / create / update / delete` on every
+    // editable service blew up with a Fatal compile error.
+    // Modular structure: per-Schema isolation — base + editable share ns.
+    // Pivot schemas don't have services (kind:object only), so the only
+    // shape adjustment is `userEditableGroupByFolder` group nesting.
     const modelNs = config.structure === 'modular'
         ? config.models.namespace
-        : nestByGroup({ path: '', namespace: config.models.namespace }, schema.group).namespace;
+        : (config.models.userEditableGroupByFolder
+            ? nestByGroup({ path: '', namespace: config.models.userEditableNamespace }, schema.group).namespace
+            : config.models.userEditableNamespace);
     // Build imports
     const imports = [
         `use ${modelNs}\\${modelName};`,