@omnifyjp/ts 5.8.16 → 5.8.17

package/dist/php/resource-generator.js

@@ -46,7 +46,7 @@ function generateBaseResource(name, schema, reader, config) {
         if (isHiddenByDefault(type) || hidden)
             continue;
         if (type === 'Association') {
-            addAssociationFields(propName, prop, fields, resourceNamespace, modelNamespace, reader);
+            addAssociationFields(propName, prop, fields, resourceNamespace, modelNamespace, reader, config);
             continue;
         }
         if (expandedProperties[propName]) {
@@ -193,27 +193,38 @@ class ${editable.className} extends ${baseAlias}
 `;
     return userFile(`${editable.path}/${editable.fileName}`, content);
 }
-function addAssociationFields(propName, prop, fields, resourceNamespace, modelNamespace, reader) {
+function addAssociationFields(propName, prop, fields, resourceNamespace, modelNamespace, reader, config) {
     const relation = prop['relation'] ?? '';
     const target = prop['target'] ?? '';
     const methodName = toCamelCase(propName);
-    // Resolve resource namespace for the target (null if package without resource ns)
-    // Issue #98 v5.8.12: nest the resource FQCN by the TARGET schema's
-    // group folder so the inline `new \<ns>\<Name>Resource(...)` ref
-    // matches where the target Resource file actually lives. Pre-fix
-    // every cross-resource ref emitted FLAT (`use App\Omnify\Resources\PaymentProviderResource`)
-    // while the file lived at `App\Omnify\Resources\Payments\PaymentProviderResource` —
-    // PHP autoload failed at runtime on every base resource that
-    // referenced a sibling resource (whenLoaded relation).
-    const rawTargetResNs = target ? reader.resolveResourceNamespace(target, resourceNamespace) : resourceNamespace;
-    let targetResNs = rawTargetResNs;
-    if (rawTargetResNs && rawTargetResNs === resourceNamespace) {
-        // Project-owned target: nest by the target's group so the FQCN
-        // matches the grouped base resource. Package targets keep their
-        // own namespace.
+    // Resolve resource namespace for the target.
+    // Issue #98 v5.8.12 / #99 v5.8.17: cross-resource refs match where
+    // the target Resource file actually lives. Same pattern as the
+    // v5.8.10/11 model + service / controller fixes — point at the
+    // EDITABLE so app code consistently resolves the same class:
+    //   1. Package target → keep the package's resource namespace.
+    //   2. Project target on modular structure → use the resource
+    //      namespace as-is (modular has its own per-schema isolation).
+    //   3. `userEditableGroupByFolder: true` → nest editable namespace
+    //      by group so the FQCN matches the grouped editable file.
+    //   4. Default (flat editable) → use the editable namespace as-is.
+    //      v4 projects with `resource.namespace: App\\Http\\Resources`
+    //      single-tier flat layout (#99) hit this path and stay flat.
+    const packageResNs = target ? reader.resolveResourceNamespace(target, resourceNamespace) : resourceNamespace;
+    let targetResNs = packageResNs;
+    if (packageResNs && packageResNs === resourceNamespace) {
+        // Project-owned target: switch to editable namespace + apply
+        // optional group nesting. Modular structure aligns base + editable
+        // on the same namespace so this is a no-op.
+        const editableNs = config.resources.userEditableNamespace || resourceNamespace;
         const targetSchema = target ? reader.getSchema(target) : undefined;
-        if (targetSchema?.group) {
-            targetResNs = `${rawTargetResNs}\\${toPascalCase(targetSchema.group)}`;
+        if (config.structure !== 'modular'
+            && config.resources.userEditableGroupByFolder
+            && targetSchema?.group) {
+            targetResNs = `${editableNs}\\${toPascalCase(targetSchema.group)}`;
+        }
+        else {
+            targetResNs = editableNs;
         }
     }
     // If target is a package schema without a resource namespace, use simple whenLoaded

package/package.json

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