@omnifyjp/ts 5.8.8 → 5.8.9

package/dist/php/factory-generator.js

@@ -3,7 +3,7 @@
  */
 import { toPascalCase } from './naming-helper.js';
 import { toFaker, compoundFaker, associationFaker } from './faker-mapper.js';
-import { userFile, nestByGroup } from './types.js';
+import { userFile } from './types.js';
 /** Generate Factory classes for all project-owned visible object schemas. */
 export function generateFactories(reader, config) {
     const files = [];
@@ -15,16 +15,51 @@ export function generateFactories(reader, config) {
     }
     return files;
 }
+/**
+ * Issue #98 v5.8.9: resolve the EDITABLE Model namespace for a given
+ * schema. Factory `$model` + association FK references must land on the
+ * user-editable class (e.g. `App\\Models\\Banner`) — that's what
+ * services / controllers / tests / seeders type-hint against, and what
+ * the project owns. Pre-fix every factory pointed at the BASE
+ * (e.g. `App\\Omnify\\Models\\Cms\\Banner`); calling
+ * `Banner::factory()->create()` returned a base instance and every
+ * downstream signature failed with TypeError.
+ *
+ * Pivot schemas always nest under `Pivot/` regardless of any flag.
+ * Group nesting only applies when the layer's `userEditableGroupByFolder`
+ * is true (matches where the editable file actually lives).
+ */
+function resolveEditableModelNamespace(schemaName, config, reader) {
+    const targetSchema = reader.getSchema(schemaName);
+    // Modular structure: editable shares per-Schema modular path.
+    if (config.structure === 'modular') {
+        return config.models.namespace;
+    }
+    const editableNs = config.models.userEditableNamespace;
+    // Package schemas: use the package's model namespace verbatim
+    // (we don't generate editable stubs for package schemas).
+    const baseFromReader = reader.resolveModelNamespace(schemaName, config.models.namespace);
+    if (baseFromReader !== config.models.namespace) {
+        return baseFromReader;
+    }
+    if (!targetSchema)
+        return editableNs;
+    if (targetSchema.kind === 'pivot') {
+        return `${editableNs}\\Pivot`;
+    }
+    if (config.models.userEditableGroupByFolder && targetSchema.group) {
+        return `${editableNs}\\${toPascalCase(targetSchema.group)}`;
+    }
+    return editableNs;
+}
 function generateFactory(name, schema, reader, schemas, config) {
     const modelName = toPascalCase(name);
-    // Issue #98 v5.8.4: legacy structure groups Models by folder, so the
-    // factory's `use ${modelNamespace}\\${modelName}` import must include
-    // the schema's group too — otherwise the factory references a flat
-    // path that no longer holds the model file. Modular structure
-    // already isolates per-schema so it skips the nesting.
-    const modelNamespace = config.structure === 'modular'
-        ? config.models.namespace
-        : nestByGroup({ path: '', namespace: config.models.namespace }, schema.group).namespace;
+    // Issue #98 v5.8.9: factories type the `$model` reference + associated
+    // FK references against the USER-EDITABLE class (`App\Models\Banner`)
+    // not the base (`App\Omnify\Models\Cms\Banner`). Service / controller /
+    // test signatures all type-hint against the editable; pointing
+    // factories at the base produced TypeErrors at every callsite.
+    const modelNamespace = resolveEditableModelNamespace(name, config, reader);
     const factoryNamespace = config.factories.namespace;
     const properties = (schema.properties ?? {});
     const expandedProperties = reader.getExpandedProperties(name);
@@ -38,9 +73,10 @@ function generateFactory(name, schema, reader, schemas, config) {
         const type = prop['type'] ?? 'String';
         // Handle associations (foreign keys)
         if (type === 'Association') {
-            // Resolve namespace per-target (package schemas use their own model namespace)
+            // Resolve namespace per-target — also points at the EDITABLE
+            // (matches what `<Target>::factory()` resolves to via Eloquent).
             const target = prop['target'] ?? '';
-            const targetNs = target ? reader.resolveModelNamespace(target, modelNamespace) : modelNamespace;
+            const targetNs = target ? resolveEditableModelNamespace(target, config, reader) : modelNamespace;
             const result = associationFaker(propName, prop, targetNs, name);
             if (result) {
                 attributes.push(result.fake);

package/package.json

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