@omnifyjp/omnify 5.8.4 → 5.8.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@omnifyjp/omnify",
-  "version": "5.8.4",
+  "version": "5.8.5",
   "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.4",
-    "@omnifyjp/omnify-darwin-x64": "5.8.4",
-    "@omnifyjp/omnify-linux-x64": "5.8.4",
-    "@omnifyjp/omnify-linux-arm64": "5.8.4",
-    "@omnifyjp/omnify-win32-x64": "5.8.4"
+    "@omnifyjp/omnify-darwin-arm64": "5.8.5",
+    "@omnifyjp/omnify-darwin-x64": "5.8.5",
+    "@omnifyjp/omnify-linux-x64": "5.8.5",
+    "@omnifyjp/omnify-linux-arm64": "5.8.5",
+    "@omnifyjp/omnify-win32-x64": "5.8.5"
   }
 }

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

@@ -111,7 +111,7 @@ function generateBaseModel(name, schema, reader, config) {
     const hidden = buildHidden(properties, expandedProperties, propertyOrder);
     const appends = buildAppends(expandedProperties);
     const casts = buildCasts(properties, expandedProperties, propertyOrder, reader, config);
-    const relations = buildRelations(name, properties, propertyOrder, modelNamespace, reader);
+    const relations = buildRelations(name, properties, propertyOrder, modelNamespace, reader, config);
     const accessors = buildAccessors(expandedProperties);
     const fileAccessors = hasFiles ? buildFileAccessors(properties, propertyOrder, modelNamespace) : '';
     const auditSection = buildAuditSection(options, reader, modelNamespace);
@@ -578,7 +578,42 @@ function buildCasts(properties, expandedProperties, propertyOrder, reader, confi
         return '';
     return casts.map(c => `            ${c}`).join('\n') + '\n';
 }
-function buildRelations(schemaName, properties, propertyOrder, modelNamespace, reader) {
+/**
+ * Issue #98 v5.8.6: resolve the FQCN namespace of a related Model
+ * for inline `belongsTo / hasMany / belongsToMany / morphTo` references.
+ * The namespace must match the TARGET's actual file location, not the
+ * source schema's. Three knobs:
+ *   1. Package schemas — return the package's namespace verbatim
+ *      (already handled by reader.resolveModelNamespace).
+ *   2. Pivot schemas — always nest under `Pivot/` regardless of any
+ *      group / userEditableGroupByFolder setting.
+ *   3. Group nesting — only when `userEditableGroupByFolder: true`
+ *      (matches where the editable lives so the relation resolves to
+ *      the user's customizable class). Default off → flat FQCN
+ *      matches the Laravel-canonical flat editable layout.
+ *
+ * Pre-fix every relation emitted `\${modelNamespace}\${target}::class`
+ * (flat FQCN) regardless of where the target's file actually lived.
+ * For grouped projects this dispatched to a non-existent class at
+ * runtime — `ErrorException: include(... no such file)`.
+ */
+function resolveRelatedModelNamespace(target, modelNamespace, config, reader) {
+    const baseNs = reader.resolveModelNamespace(target, modelNamespace);
+    // Package schemas — leave as-is.
+    if (baseNs !== modelNamespace)
+        return baseNs;
+    const targetSchema = reader.getSchema(target);
+    if (!targetSchema)
+        return baseNs;
+    if (targetSchema.kind === 'pivot') {
+        return `${baseNs}\\Pivot`;
+    }
+    if (config.models.userEditableGroupByFolder && targetSchema.group) {
+        return `${baseNs}\\${toPascalCase(targetSchema.group)}`;
+    }
+    return baseNs;
+}
+function buildRelations(schemaName, properties, propertyOrder, modelNamespace, reader, config) {
     const methods = [];
     const sourceTableName = reader.getTableName(schemaName);
     // Track method names already declared on this model so reverse-relation
@@ -591,9 +626,11 @@ function buildRelations(schemaName, properties, propertyOrder, modelNamespace, r
         const prop = properties[propName];
         if (!prop || prop['type'] !== 'Association')
             continue;
-        // Resolve namespace per-target (package schemas use their own namespace)
+        // Resolve namespace per-target (package schemas use their own namespace).
+        // Issue #98 v5.8.6: also nest by target's group / Pivot/ so the
+        // FQCN matches the target file's actual location.
         const target = prop['target'] ?? '';
-        const ns = target ? reader.resolveModelNamespace(target, modelNamespace) : modelNamespace;
+        const ns = target ? resolveRelatedModelNamespace(target, modelNamespace, config, reader) : modelNamespace;
         const targetTableName = target ? reader.getTableName(target) : '';
         // For ManyToMany, look up the explicit `kind: pivot` schema (the
         // sorted PascalCase concat of source + target — auto-created on
@@ -623,7 +660,7 @@ function buildRelations(schemaName, properties, propertyOrder, modelNamespace, r
     // Auto-generate reverse HasMany/HasOne for any child schema that declares a
     // ManyToOne/OneToOne pointing at this schema, unless the user has already
     // declared a relation method with the same name. Issue #39.
-    const inverseRelations = buildInverseRelations(schemaName, modelNamespace, reader, declaredMethods, explicitInverses);
+    const inverseRelations = buildInverseRelations(schemaName, modelNamespace, reader, declaredMethods, explicitInverses, config);
     if (inverseRelations) {
         methods.push(inverseRelations);
     }
@@ -666,7 +703,7 @@ const LARAVEL_RESERVED_RELATION_NAMES = new Set([
  * a unique method name, or add `mappedBy: <customName>` to the child schema
  * so the derived method name changes.
  */
-function buildInverseRelations(parentName, modelNamespace, reader, declaredMethods, explicitInverses) {
+function buildInverseRelations(parentName, modelNamespace, reader, declaredMethods, explicitInverses, config) {
     const methods = [];
     const allSchemas = reader.getSchemas();
     // Stable iteration order — sort child names so output is deterministic
@@ -714,7 +751,11 @@ function buildInverseRelations(parentName, modelNamespace, reader, declaredMetho
             // Here we're emitting the *parent* accessor, so the child's ManyToOne
             // (or owning OneToOne) is what we react to.
             const fkColumn = `${toSnakeCase(childPropName)}_id`;
-            const childNs = reader.resolveModelNamespace(childName, modelNamespace);
+            // Issue #98 v5.8.6: nest the inverse-side FQCN by the CHILD's
+            // group / Pivot/ so the relation resolves to the child's actual
+            // file location (which lives in `<childGroup>/<Child>.php` or
+            // `Pivot/<Child>.php`, not flat).
+            const childNs = resolveRelatedModelNamespace(childName, modelNamespace, config, reader);
             const fqcn = `\\${childNs}\\${childName}`;
             const returnType = isOneToOne ? 'HasOne' : 'HasMany';
             const eloquentMethod = isOneToOne ? 'hasOne' : 'hasMany';