@omnifyjp/omnify 5.8.21 → 5.8.23

package/package.json

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

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

@@ -58,9 +58,14 @@ export function generatePolicies(reader, config) {
 // Per-schema generation
 // ============================================================================
 function generateForSchema(name, schema, reader, config) {
+    // Issue #101 v5.8.23: only emit the policy BASE. The user-editable
+    // stub was pure scaffolding (extends base, no body) for every
+    // schema — typically 80+ noise files in a project. When the dev
+    // wants to customize a permission check, they hand-create the
+    // wrapper at `userEditablePath`; Laravel's policy auto-discovery
+    // picks it up. Until then, the base alone is enough.
     return [
         generateBasePolicyClass(name, schema, reader, config),
-        generateUserPolicyClass(name, schema, config),
     ];
 }
 function generateBasePolicyClass(name, schema, reader, config) {

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

@@ -33,11 +33,14 @@ export function generateRequests(reader, config) {
     return files;
 }
 function generateForSchema(name, schema, reader, config) {
+    // Issue #101 v5.8.23: only emit the request BASE classes. The
+    // user-editable stubs were pure scaffolding (extends base, no body)
+    // — typically 121 noise files in a project. Controllers + DI inject
+    // the BASE FQN directly. When custom validation IS needed, the dev
+    // creates the editable file by hand under `userEditablePath`.
     return [
         generateBaseRequest(name, schema, reader, config, 'Store'),
         generateBaseRequest(name, schema, reader, config, 'Update'),
-        generateUserRequest(name, schema, config, 'Store'),
-        generateUserRequest(name, schema, config, 'Update'),
     ];
 }
 function generateBaseRequest(name, schema, reader, config, action) {

package/ts-dist/php/resource-generator.d.ts

@@ -3,5 +3,11 @@
  */
 import { SchemaReader } from './schema-reader.js';
 import type { GeneratedFile, PhpConfig } from './types.js';
-/** Generate Resource classes for all project-owned visible object schemas. */
+/** Generate Resource classes for all project-owned visible object schemas.
+ *
+ * Issue #101 v5.8.23: only the base is emitted. Empty editable stubs
+ * (every project's 30+ thin `class XResource extends Base {}`) are
+ * pure scaffolding noise — devs hand-create wrappers only when custom
+ * serialization logic IS needed.
+ */
 export declare function generateResources(reader: SchemaReader, config: PhpConfig): GeneratedFile[];

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

@@ -4,14 +4,19 @@
 import { toPascalCase, toSnakeCase, toCamelCase } from './naming-helper.js';
 import { isHiddenByDefault, toResourceExpression } from './type-mapper.js';
 import { baseFile, userFile, resolveModularBasePath, resolveModularBaseNamespace, resolveBaseClass, resolveEditableClass, nestByGroup, nestEditableByGroup } from './types.js';
-/** Generate Resource classes for all project-owned visible object schemas. */
+/** Generate Resource classes for all project-owned visible object schemas.
+ *
+ * Issue #101 v5.8.23: only the base is emitted. Empty editable stubs
+ * (every project's 30+ thin `class XResource extends Base {}`) are
+ * pure scaffolding noise — devs hand-create wrappers only when custom
+ * serialization logic IS needed.
+ */
 export function generateResources(reader, config) {
     const files = [];
     for (const [name, schema] of Object.entries(reader.getProjectVisibleObjectSchemas())) {
         if (schema.kind === 'extend')
             continue;
         files.push(generateBaseResource(name, schema, reader, config));
-        files.push(generateUserResource(name, schema, config));
     }
     return files;
 }

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

@@ -200,9 +200,21 @@ function warnLegacyServiceKeys(name, schema) {
     }
 }
 function generateForSchema(name, schema, reader, config) {
+    // Issue #101 v5.8.22: only emit the service BASE. The user-editable
+    // service stub is pure scaffolding (extends base, no body) for every
+    // schema in every project — 290+ noise files in a typical project.
+    // Consumers inject the BASE FQN directly:
+    //
+    //     public function __construct(
+    //         private \App\Omnify\Services\<Group>\<Name>ServiceBase \$service,
+    //     ) {}
+    //
+    // When custom logic IS needed, the dev creates the editable file by
+    // hand under the `userEditablePath` (`app/Services/<Name>Service.php`)
+    // — Laravel autoloads it like any other class. Omnify won't recreate
+    // it on subsequent generates because we no longer emit it at all.
     return [
         generateBaseService(name, schema, reader, config),
-        generateUserService(name, schema, config),
     ];
 }
 /**

package/ts-dist/php/sibling-skip.d.ts

@@ -38,5 +38,15 @@ export declare function findAppRootAncestor(filePath: string): string | null;
  * Scan `rootDir` recursively for a file whose basename matches
  * `baseName`, excluding `excludePath` itself. Returns the first
  * match's full path or null.
+ *
+ * Issue #99 v5.8.19 follow-up: skip the auto-gen `Omnify/` subtree
+ * (its files are codegen output, not project-owned siblings) so the
+ * editable layer at `app/Models/<Name>.php` doesn't get blocked by
+ * the sibling base at `app/Omnify/Models/<Group>/<Name>.php` under
+ * canonical+flatBase layout. Project-owned siblings under
+ * `app/Services/Omnify/<Name>Service.php` (v4 wrapper layout) DO still
+ * trigger a skip because they live OUTSIDE the auto-gen zone — the
+ * skip rule is "any directory whose name starts with `Omnify` is
+ * codegen-owned", which matches `Omnify/`, `OmnifyBase/`, etc.
  */
 export declare function findSiblingWithSameBasename(rootDir: string, baseName: string, excludePath: string): string | null;

package/ts-dist/php/sibling-skip.js

@@ -52,6 +52,16 @@ export function findAppRootAncestor(filePath) {
  * Scan `rootDir` recursively for a file whose basename matches
  * `baseName`, excluding `excludePath` itself. Returns the first
  * match's full path or null.
+ *
+ * Issue #99 v5.8.19 follow-up: skip the auto-gen `Omnify/` subtree
+ * (its files are codegen output, not project-owned siblings) so the
+ * editable layer at `app/Models/<Name>.php` doesn't get blocked by
+ * the sibling base at `app/Omnify/Models/<Group>/<Name>.php` under
+ * canonical+flatBase layout. Project-owned siblings under
+ * `app/Services/Omnify/<Name>Service.php` (v4 wrapper layout) DO still
+ * trigger a skip because they live OUTSIDE the auto-gen zone — the
+ * skip rule is "any directory whose name starts with `Omnify` is
+ * codegen-owned", which matches `Omnify/`, `OmnifyBase/`, etc.
  */
 export function findSiblingWithSameBasename(rootDir, baseName, excludePath) {
     let scanned = 0;
@@ -82,6 +92,15 @@ export function findSiblingWithSameBasename(rootDir, baseName, excludePath) {
                 continue;
             }
             if (stats.isDirectory()) {
+                // Skip the auto-gen `app/Omnify/` zone — its files are codegen
+                // outputs, not user-owned siblings. Detect via "directory is
+                // named `Omnify` AND its parent is named `app`": this catches
+                // the canonical `app/Omnify/` zone but does NOT skip
+                // `app/Services/Omnify/` (a v4 wrapper layout where `Omnify`
+                // is project-owned and should still trigger sibling-skip per
+                // issue #99 v5.8.19).
+                if (entry === 'Omnify' && basename(dir) === 'app')
+                    continue;
                 stack.push(fullPath);
             }
             else if (entry === baseName && fullPath !== excludePath) {