@open-mercato/cli 0.7.0 → 0.7.1-develop.7103.1.41ff100d93

package/src/lib/generators/__tests__/generated-registry-imports.test.ts

@@ -0,0 +1,128 @@
+import fs from 'node:fs'
+import path from 'node:path'
+import os from 'node:os'
+import type { PackageResolver, ModuleEntry } from '../../resolver'
+import { generateModuleRegistry, generateModuleRegistryApp, generateModuleRegistryCli } from '../module-registry'
+
+/**
+ * The generator's output, not its helpers.
+ *
+ * A worker declaring `metadata.onJobAbandoned` makes the registry emit
+ * `createLazyModuleWorkerAbandonHook(...)`, and the import for it used to be opted into per render
+ * site. Forgetting it at one of the six sites produced a file referencing an undeclared name, which
+ * type-checks nowhere and surfaces only at `build:app` — every unit test stayed green because no
+ * fixture declared such a worker.
+ *
+ * These assertions are driven off the files the generators actually wrote, so a renderer added later
+ * is covered without anyone remembering to extend this test.
+ */
+
+let tmpDir: string
+
+function touchFile(filePath: string, content = ''): void {
+  fs.mkdirSync(path.dirname(filePath), { recursive: true })
+  fs.writeFileSync(filePath, content)
+}
+
+function createMockResolver(dir: string, enabled: ModuleEntry[]): PackageResolver {
+  const outputDir = path.join(dir, 'output', 'generated')
+  fs.mkdirSync(outputDir, { recursive: true })
+  return {
+    isMonorepo: () => true,
+    getRootDir: () => dir,
+    getAppDir: () => path.join(dir, 'app'),
+    getOutputDir: () => outputDir,
+    getModulesConfigPath: () => path.join(dir, 'app', 'src', 'modules.ts'),
+    discoverPackages: () => [],
+    loadEnabledModules: () => enabled,
+    getModulePaths: (entry: ModuleEntry) => ({
+      appBase: path.join(dir, 'app', 'src', 'modules', entry.id),
+      pkgBase: path.join(dir, 'packages', 'core', 'src', 'modules', entry.id),
+    }),
+    getModuleImportBase: (entry: ModuleEntry) => ({
+      appBase: `@/modules/${entry.id}`,
+      pkgBase: `@open-mercato/core/modules/${entry.id}`,
+    }),
+    getPackageOutputDir: () => outputDir,
+    getPackageRoot: () => path.join(dir, 'packages', 'core'),
+  }
+}
+
+function scaffoldWorkerModule(dir: string, modId: string, workerSource: string): void {
+  touchFile(path.join(dir, 'packages', 'core', 'src', 'modules', modId, 'workers', 'sync.ts'), workerSource)
+}
+
+function generatedFiles(dir: string): Array<{ name: string; content: string }> {
+  const outputDir = path.join(dir, 'output', 'generated')
+  if (!fs.existsSync(outputDir)) return []
+  return fs.readdirSync(outputDir)
+    .filter((name) => name.endsWith('.ts'))
+    .map((name) => ({ name, content: fs.readFileSync(path.join(outputDir, name), 'utf8') }))
+}
+
+const ABANDON_HOOK_FACTORY = 'createLazyModuleWorkerAbandonHook'
+
+const WORKER_WITH_HOOK = `
+import { failAbandonedRun } from '../lib/abandoned-run'
+export const metadata = {
+  queue: 'demo-queue',
+  id: 'demo:worker',
+  concurrency: 2,
+  onJobAbandoned: failAbandonedRun,
+}
+export default async function handle() {}
+`
+
+const WORKER_WITHOUT_HOOK = `
+export const metadata = {
+  queue: 'demo-queue',
+  id: 'demo:worker',
+  concurrency: 2,
+}
+export default async function handle() {}
+`
+
+async function generateAll(resolver: PackageResolver): Promise<void> {
+  await generateModuleRegistry({ resolver, quiet: true })
+  await generateModuleRegistryApp({ resolver, quiet: true })
+  await generateModuleRegistryCli({ resolver, quiet: true })
+}
+
+beforeEach(() => {
+  tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'generated-registry-imports-'))
+})
+
+afterEach(() => {
+  fs.rmSync(tmpDir, { recursive: true, force: true })
+})
+
+describe('generated registries — emitted identifiers are imported', () => {
+  it('imports the abandon-hook helper in every file that calls it', async () => {
+    scaffoldWorkerModule(tmpDir, 'demo', WORKER_WITH_HOOK)
+    const resolver = createMockResolver(tmpDir, [{ id: 'demo', from: 'pkg' } as ModuleEntry])
+
+    await generateAll(resolver)
+
+    const files = generatedFiles(tmpDir)
+    const callers = files.filter(({ content }) => content.includes(`${ABANDON_HOOK_FACTORY}(`))
+    // The fixture must actually exercise the path, or this test passes vacuously — which is exactly
+    // how the missing import survived every existing suite.
+    expect(callers.length).toBeGreaterThan(0)
+
+    for (const { name, content } of callers) {
+      expect(`${name}: ${content.includes(`import { `) && new RegExp(`import \\{[^}]*\\b${ABANDON_HOOK_FACTORY}\\b[^}]*\\} from`, 's').test(content)}`)
+        .toBe(`${name}: true`)
+    }
+  })
+
+  it('leaves output untouched for a module set whose workers declare no hook', async () => {
+    scaffoldWorkerModule(tmpDir, 'demo', WORKER_WITHOUT_HOOK)
+    const resolver = createMockResolver(tmpDir, [{ id: 'demo', from: 'pkg' } as ModuleEntry])
+
+    await generateAll(resolver)
+
+    for (const { name, content } of generatedFiles(tmpDir)) {
+      expect(`${name}: ${content.includes(ABANDON_HOOK_FACTORY)}`).toBe(`${name}: false`)
+    }
+  })
+})

package/src/lib/generators/__tests__/metadata-extraction.test.ts

@@ -4,6 +4,7 @@ import os from 'node:os'
 import {
   extractNamedObjectLiteralExport,
   hasNamedExport,
+  namedObjectLiteralDeclaresProperty,
   resolveNamedObjectExport,
 } from '../module-registry'
 
@@ -218,3 +219,46 @@ describe('hasNamedExport', () => {
     expect(hasNamedExport(file, 'metadata')).toBe(false)
   })
 })
+
+describe('namedObjectLiteralDeclaresProperty', () => {
+  // A worker declaring `onJobAbandoned: someImportedFn` is the case that broke: the value resolvers
+  // cannot evaluate an imported identifier, so "is it declared" has to be answered from the syntax.
+  // Getting this wrong is silent — the generator simply never emits the hook.
+  it('sees a property whose value is an unresolvable identifier', () => {
+    const file = writeSource('worker.ts', `
+      import { failAbandonedRun } from '../lib/abandoned-run'
+      export const metadata = {
+        queue: 'data-sync-import',
+        concurrency: 5,
+        onJobAbandoned: failAbandonedRun,
+      }
+    `)
+    expect(namedObjectLiteralDeclaresProperty(file, 'metadata', 'onJobAbandoned')).toBe(true)
+    expect(extractNamedObjectLiteralExport(file, 'metadata')?.onJobAbandoned).toBeUndefined()
+  })
+
+  it('sees a property declared as an inline function', () => {
+    const file = writeSource('worker.ts', `
+      export const metadata = {
+        queue: 'q',
+        onJobAbandoned: async () => {},
+      }
+    `)
+    expect(namedObjectLiteralDeclaresProperty(file, 'metadata', 'onJobAbandoned')).toBe(true)
+  })
+
+  it('reports absence for a worker that declares no hook', () => {
+    const file = writeSource('worker.ts', `
+      export const metadata = { queue: 'q', concurrency: 1 }
+    `)
+    expect(namedObjectLiteralDeclaresProperty(file, 'metadata', 'onJobAbandoned')).toBe(false)
+  })
+
+  it('reports absence for a metadata export that is not an object literal', () => {
+    const file = writeSource('worker.ts', `
+      const base = buildMetadata()
+      export const metadata = base
+    `)
+    expect(namedObjectLiteralDeclaresProperty(file, 'metadata', 'onJobAbandoned')).toBe(false)
+  })
+})

package/src/lib/generators/__tests__/module-facts.bc-guard.test.ts

@@ -149,22 +149,46 @@ describe('module-facts BC resolve guard (T2)', () => {
     // wms, staff, integrations and checkout were therefore invisible to every fact
     // consumer — `integrations` published no contributions at all. Reading them costs
     // ~28KB, which is the fix working, not drift.
-    // The additive EUDR module contributes its real routes, ACL, events,
-    // entities, and extension surfaces without changing the extraction shape.
+    // The additive EUDR and Documents modules contribute their real routes, ACL,
+    // events, entities, and extension surfaces without changing the extraction
+    // shape.
     //
     // JSON cap raised a fifth time by the devices/push-notifications stack: the
     // `devices` and `push_notifications` modules plus the `channel-fcm`,
     // `channel-apns` and `channel-expo` provider packages add their own facts,
     // provenance entries and override targets to every render. The
-    // `warranty_claims` module (see above) lands alongside it, so the cap
-    // absorbs both additions.
-    expect(Buffer.byteLength(completeJson)).toBeLessThan(4_000_000)
-    expect(Buffer.byteLength(completeJson) - Buffer.byteLength(legacyJson)).toBeLessThan(1_800_000)
+    // `warranty_claims` module (see above) and the additive Documents module
+    // land alongside it.
+    //
+    // Raised again here for a reason unlike every raise above: those all tracked
+    // a schema or extraction change, this one tracks ordinary repo growth. The
+    // additive `channel_discord` module costs ~15.9KB JSON and ~9.2KB markdown
+    // (`channel_gmail` costs 6,886 / `channel_imap` 6,798; the difference is a
+    // gateway worker, a CLI command, a signed route and a subscriber, spread
+    // proportionally across overrideTargets, extensionSurfaces, factSources and
+    // ownedContracts — no duplicated payloads).
+    //
+    // The numbers below are MEASURED ON THE MERGED TREE, not transcribed from
+    // either side of the merge: neither side's cap covers the union. A cap that
+    // sits a fraction of a percent above the current measurement has stopped
+    // detecting blow-ups and started rejecting the next additive module of any
+    // size, whichever PR happens to add it — so keep bounded headroom over the
+    // measured size. A real blow-up here is multiplicative (a duplicated
+    // provenance payload, a contribution body copied per resolution), not one
+    // provider's worth of references.
+    expect(Buffer.byteLength(completeJson)).toBeLessThan(4_700_000)
+    // Measured on the merged tree (`yarn jest module-facts.bc-guard`):
+    //   completeJson 4,120,374 · legacy delta 1,795,214 · markdown 1,558,590 ·
+    //   directory markdown 1,709,511.
+    // The delta cap is raised alongside the JSON one for the same reason: at
+    // 1,800,000 it sat 0.27% above the measurement, which is not a guard.
+    expect(Buffer.byteLength(completeJson) - Buffer.byteLength(legacyJson)).toBeLessThan(2_050_000)
     // Markdown cap raised with the source-link contract: entities, events, ACL
     // features, DI tokens, search entities, notifications, UMES hosts and UMES
     // contributions all render a resolved Source cell, and contribution
-    // resolutions render as their own source-linked section.
-    expect(markdownBytes).toBeLessThan(1_750_000)
+    // resolutions render as their own source-linked section — plus the additive
+    // `channel_discord` module's own render, per the growth note above.
+    expect(markdownBytes).toBeLessThan(1_800_000)
     expect(directoryMarkdownBytes).toBeLessThan(2_050_000)
   })
 

package/src/lib/generators/__tests__/module-facts.customers.fixture.test.ts

@@ -28,7 +28,10 @@ describe('module-facts customers fixture (T1 anti-drift guard)', () => {
   })
 
   it('locks customers events, acl, search, and notification surfaces', () => {
-    expect(facts.events).toHaveLength(49)
+    // 50 since #5114 added `customers.person.company_assignment.detached` — the profile-only
+    // sibling of the person_company_link CRUD events. Bump deliberately: this guard exists to
+    // make an event-surface change visible, not to be silenced.
+    expect(facts.events).toHaveLength(50)
     expect(facts.aclFeatures).toHaveLength(21)
     expect(facts.searchEntities).toEqual([
       'customers:customer_person_profile',

package/src/lib/generators/__tests__/module-facts.extension-hosts.test.ts

@@ -103,6 +103,6 @@ describe('module-facts extension host extraction', () => {
     ])
 
     const sales = extractModuleFacts({ moduleId: 'sales', coreSrcRoot })
-    expect(sales.hostTokens.tableIds).toEqual(['sales.orders', 'sales.quotes'])
+    expect(sales.hostTokens.tableIds).toEqual(['sales.orders', 'sales.payments', 'sales.quotes'])
   })
 })

package/src/lib/generators/__tests__/module-subset.test.ts

@@ -186,65 +186,65 @@ describe('generateModuleRegistry with module subsets', () => {
     expect(output).toContain('export default modules')
   })
 
-  it('preserves exact legacy empty output format for the refactored registry files', async () => {
+  it('pins the exact emitted empty output format for the refactored registry files', async () => {
     const resolver = createMockResolver(tmpDir, [])
     const result = await generateModuleRegistry({ resolver, quiet: true })
 
     expect(result.errors).toEqual([])
     expect(readGenerated(tmpDir, 'modules.generated.ts')).toBe(dedent(`
       // AUTO-GENERATED by mercato generate registry
-      import { createLazyModuleSubscriber, createLazyModuleWorker, type Module } from '@open-mercato/shared/modules/registry'
+      import { createLazyModuleSubscriber, createLazyModuleWorker, type Module } from "@open-mercato/shared/modules/registry";
 
-      export const modules: Module[] = [
-        
-      ]
-      export const modulesInfo = modules.map(m => ({ id: m.id, ...(m.info || {}) }))
-      export default modules
+      export const modules: Module[] = [];
+
+      export default modules;
+
+      export const modulesInfo = modules.map((m) => ({
+          id: m.id,
+          ...m.info || {}
+        }));
     `))
     expect(readGenerated(tmpDir, 'modules.runtime.generated.ts')).toBe(dedent(`
       // AUTO-GENERATED by mercato generate registry
-      import { createElement } from 'react'
-      import { createLazyModuleSubscriber, createLazyModuleWorker, type Module } from '@open-mercato/shared/modules/registry'
+      import { createElement } from "react";
+      import { createLazyModuleSubscriber, createLazyModuleWorker, type Module } from "@open-mercato/shared/modules/registry";
 
-      export const modules: Module[] = [
-        
-      ]
-      export const modulesInfo = modules.map(m => ({ id: m.id, ...(m.info || {}) }))
-      export default modules
+      export const modules: Module[] = [];
+
+      export default modules;
+
+      export const modulesInfo = modules.map((m) => ({
+          id: m.id,
+          ...m.info || {}
+        }));
     `))
     expect(readGenerated(tmpDir, 'frontend-routes.generated.ts')).toBe(dedent(`
       // AUTO-GENERATED by mercato generate registry
-      import { resolvePageRouteMetadata, type FrontendRouteManifestEntry } from '@open-mercato/shared/modules/registry'
+      import { resolvePageRouteMetadata, type FrontendRouteManifestEntry } from "@open-mercato/shared/modules/registry";
 
-      export const frontendRoutes: FrontendRouteManifestEntry[] = [
-        
-      ]
+      export const frontendRoutes: FrontendRouteManifestEntry[] = [];
 
-      export default frontendRoutes
+      export default frontendRoutes;
     `))
     expect(readGenerated(tmpDir, 'backend-routes.generated.ts')).toBe(dedent(`
       // AUTO-GENERATED by mercato generate registry
-      import { resolvePageRouteMetadata, type BackendRouteManifestEntry } from '@open-mercato/shared/modules/registry'
+      import { resolvePageRouteMetadata, type BackendRouteManifestEntry } from "@open-mercato/shared/modules/registry";
 
-      export const backendRoutes: BackendRouteManifestEntry[] = [
-        
-      ]
+      export const backendRoutes: BackendRouteManifestEntry[] = [];
 
-      export default backendRoutes
+      export default backendRoutes;
     `))
     expect(readGenerated(tmpDir, 'api-routes.generated.ts')).toBe(dedent(`
       // AUTO-GENERATED by mercato generate registry
-      import type { ApiRouteManifestEntry } from '@open-mercato/shared/modules/registry'
+      import type { ApiRouteManifestEntry } from "@open-mercato/shared/modules/registry";
 
-      export const apiRoutes: ApiRouteManifestEntry[] = [
-        
-      ]
+      export const apiRoutes: ApiRouteManifestEntry[] = [];
 
-      export default apiRoutes
+      export default apiRoutes;
     `))
   })
 
-  it('preserves exact legacy populated output format for the refactored registry files', async () => {
+  it('pins the exact emitted populated output format for the refactored registry files', async () => {
     touchFile(
       path.join(tmpDir, 'packages', 'core', 'src', 'modules', 'orders', 'index.ts'),
       "export const metadata = { id: 'orders', label: 'Orders' }\n",
@@ -267,75 +267,99 @@ describe('generateModuleRegistry with module subsets', () => {
 
     expect(result.errors).toEqual([])
     const modulesGenerated = readGenerated(tmpDir, 'modules.generated.ts')
-    expect(modulesGenerated).toContain(`import { createElement } from 'react'`)
+    expect(modulesGenerated).toContain(`import { createElement } from "react"`)
     expect(modulesGenerated).toContain(`import * as I0_orders from "@open-mercato/core/modules/orders/index"`)
     expect(modulesGenerated).not.toContain(`import C1_orders_index`)
     expect(modulesGenerated).not.toContain(`import B5_orders_index`)
-    expect(modulesGenerated).toContain(`frontendRoutes: [{ pattern: "/"`)
-    expect(modulesGenerated).toContain(`backendRoutes: [{ pattern: "/backend/orders"`)
+    expect(modulesGenerated).toMatch(/frontendRoutes:\s*\[\s*\{ pattern: "\/"/)
+    expect(modulesGenerated).toMatch(/backendRoutes:\s*\[\s*\{ pattern: "\/backend\/orders"/)
     expect(modulesGenerated).toContain(`Component: async (props: any) => { const mod = await import("@open-mercato/core/modules/orders/frontend/page")`)
     expect(modulesGenerated).toContain(`Component: async (props: any) => { const mod = await import("@open-mercato/core/modules/orders/backend/page")`)
-    expect(modulesGenerated).toContain(`apis: [{ path: ((R9_orders_orders as any).metadata?.path ?? "/orders/orders"), metadata: (R9_orders_orders as any).metadata, handlers: R9_orders_orders as any }]`)
+    expect(modulesGenerated).toMatch(/apis:\s*\[\s*\{ path: \(\(R9_orders_orders as any\)\.metadata\?\.path/)
+    expect(modulesGenerated).toContain(`{ path: ((R9_orders_orders as any).metadata?.path ?? "/orders/orders"), metadata: (R9_orders_orders as any).metadata, handlers: R9_orders_orders as any }`)
     expect(readGenerated(tmpDir, 'modules.runtime.generated.ts')).toBe(dedent(`
       // AUTO-GENERATED by mercato generate registry
-      import { createElement } from 'react'
-      import { createLazyModuleSubscriber, createLazyModuleWorker, type Module } from '@open-mercato/shared/modules/registry'
-      import * as I0_orders from "@open-mercato/core/modules/orders/index"
-      import * as MRuntime3_orders_index from "@open-mercato/core/modules/orders/frontend/page"
-      import * as BMRuntime7_orders_index from "@open-mercato/core/modules/orders/backend/page"
+      import { createElement } from "react";
+      import { createLazyModuleSubscriber, createLazyModuleWorker, type Module } from "@open-mercato/shared/modules/registry";
+      import * as I0_orders from "@open-mercato/core/modules/orders/index";
+      import * as MRuntime3_orders_index from "@open-mercato/core/modules/orders/frontend/page";
+      import * as BMRuntime7_orders_index from "@open-mercato/core/modules/orders/backend/page";
 
       export const modules: Module[] = [
-        {
+          {
             id: "orders",
             info: I0_orders.metadata,
-            frontendRoutes: [{ pattern: "/", requireAuth: ((((MRuntime3_orders_index as any).metadata) as any))?.requireAuth, requireRoles: ((((MRuntime3_orders_index as any).metadata) as any))?.requireRoles, requireFeatures: ((((MRuntime3_orders_index as any).metadata) as any))?.requireFeatures, requireCustomerAuth: ((((MRuntime3_orders_index as any).metadata) as any))?.requireCustomerAuth, requireCustomerFeatures: ((((MRuntime3_orders_index as any).metadata) as any))?.requireCustomerFeatures, nav: ((((MRuntime3_orders_index as any).metadata) as any))?.nav, title: ((((MRuntime3_orders_index as any).metadata) as any))?.pageTitle ?? ((((MRuntime3_orders_index as any).metadata) as any))?.title, titleKey: ((((MRuntime3_orders_index as any).metadata) as any))?.pageTitleKey ?? ((((MRuntime3_orders_index as any).metadata) as any))?.titleKey, group: ((((MRuntime3_orders_index as any).metadata) as any))?.pageGroup ?? ((((MRuntime3_orders_index as any).metadata) as any))?.group, groupKey: ((((MRuntime3_orders_index as any).metadata) as any))?.pageGroupKey ?? ((((MRuntime3_orders_index as any).metadata) as any))?.groupKey, icon: ((((MRuntime3_orders_index as any).metadata) as any))?.icon, order: ((((MRuntime3_orders_index as any).metadata) as any))?.pageOrder ?? ((((MRuntime3_orders_index as any).metadata) as any))?.order, priority: ((((MRuntime3_orders_index as any).metadata) as any))?.pagePriority ?? ((((MRuntime3_orders_index as any).metadata) as any))?.priority, navHidden: ((((MRuntime3_orders_index as any).metadata) as any))?.navHidden, visible: ((((MRuntime3_orders_index as any).metadata) as any))?.visible, enabled: ((((MRuntime3_orders_index as any).metadata) as any))?.enabled, breadcrumb: ((((MRuntime3_orders_index as any).metadata) as any))?.breadcrumb, pageContext: ((((MRuntime3_orders_index as any).metadata) as any))?.pageContext, placement: ((((MRuntime3_orders_index as any).metadata) as any))?.placement, Component: async (props: any) => { const mod = await import("@open-mercato/core/modules/orders/frontend/page"); const Component = (mod.default ?? mod) as any; return createElement(Component, props) } }],
-            backendRoutes: [{ pattern: "/backend/orders", requireAuth: ((((BMRuntime7_orders_index as any).metadata) as any))?.requireAuth, requireRoles: ((((BMRuntime7_orders_index as any).metadata) as any))?.requireRoles, requireFeatures: ((((BMRuntime7_orders_index as any).metadata) as any))?.requireFeatures, requireCustomerAuth: ((((BMRuntime7_orders_index as any).metadata) as any))?.requireCustomerAuth, requireCustomerFeatures: ((((BMRuntime7_orders_index as any).metadata) as any))?.requireCustomerFeatures, nav: ((((BMRuntime7_orders_index as any).metadata) as any))?.nav, title: ((((BMRuntime7_orders_index as any).metadata) as any))?.pageTitle ?? ((((BMRuntime7_orders_index as any).metadata) as any))?.title, titleKey: ((((BMRuntime7_orders_index as any).metadata) as any))?.pageTitleKey ?? ((((BMRuntime7_orders_index as any).metadata) as any))?.titleKey, group: ((((BMRuntime7_orders_index as any).metadata) as any))?.pageGroup ?? ((((BMRuntime7_orders_index as any).metadata) as any))?.group, groupKey: ((((BMRuntime7_orders_index as any).metadata) as any))?.pageGroupKey ?? ((((BMRuntime7_orders_index as any).metadata) as any))?.groupKey, icon: ((((BMRuntime7_orders_index as any).metadata) as any))?.icon, order: ((((BMRuntime7_orders_index as any).metadata) as any))?.pageOrder ?? ((((BMRuntime7_orders_index as any).metadata) as any))?.order, priority: ((((BMRuntime7_orders_index as any).metadata) as any))?.pagePriority ?? ((((BMRuntime7_orders_index as any).metadata) as any))?.priority, navHidden: ((((BMRuntime7_orders_index as any).metadata) as any))?.navHidden, visible: ((((BMRuntime7_orders_index as any).metadata) as any))?.visible, enabled: ((((BMRuntime7_orders_index as any).metadata) as any))?.enabled, breadcrumb: ((((BMRuntime7_orders_index as any).metadata) as any))?.breadcrumb, pageContext: ((((BMRuntime7_orders_index as any).metadata) as any))?.pageContext, placement: ((((BMRuntime7_orders_index as any).metadata) as any))?.placement, Component: async (props: any) => { const mod = await import("@open-mercato/core/modules/orders/backend/page"); const Component = (mod.default ?? mod) as any; return createElement(Component, props) } }],
-            apis: [{ path: "/orders/orders", metadata: undefined, handlers: { GET: async (req: Request, ctx?: any) => { const mod = await import("@open-mercato/core/modules/orders/api/orders/route"); return (mod as any).GET(req, ctx) } } }],
-            
-            
-            
-            
-            customFieldSets: [],
-            
-            
-            
-            
-            
-            
+            frontendRoutes: [
+              { pattern: "/", requireAuth: ((((MRuntime3_orders_index as any).metadata) as any))?.requireAuth, requireRoles: ((((MRuntime3_orders_index as any).metadata) as any))?.requireRoles, requireFeatures: ((((MRuntime3_orders_index as any).metadata) as any))?.requireFeatures, requireCustomerAuth: ((((MRuntime3_orders_index as any).metadata) as any))?.requireCustomerAuth, requireCustomerFeatures: ((((MRuntime3_orders_index as any).metadata) as any))?.requireCustomerFeatures, nav: ((((MRuntime3_orders_index as any).metadata) as any))?.nav, title: ((((MRuntime3_orders_index as any).metadata) as any))?.pageTitle ?? ((((MRuntime3_orders_index as any).metadata) as any))?.title, titleKey: ((((MRuntime3_orders_index as any).metadata) as any))?.pageTitleKey ?? ((((MRuntime3_orders_index as any).metadata) as any))?.titleKey, group: ((((MRuntime3_orders_index as any).metadata) as any))?.pageGroup ?? ((((MRuntime3_orders_index as any).metadata) as any))?.group, groupKey: ((((MRuntime3_orders_index as any).metadata) as any))?.pageGroupKey ?? ((((MRuntime3_orders_index as any).metadata) as any))?.groupKey, icon: ((((MRuntime3_orders_index as any).metadata) as any))?.icon, order: ((((MRuntime3_orders_index as any).metadata) as any))?.pageOrder ?? ((((MRuntime3_orders_index as any).metadata) as any))?.order, priority: ((((MRuntime3_orders_index as any).metadata) as any))?.pagePriority ?? ((((MRuntime3_orders_index as any).metadata) as any))?.priority, navHidden: ((((MRuntime3_orders_index as any).metadata) as any))?.navHidden, visible: ((((MRuntime3_orders_index as any).metadata) as any))?.visible, enabled: ((((MRuntime3_orders_index as any).metadata) as any))?.enabled, breadcrumb: ((((MRuntime3_orders_index as any).metadata) as any))?.breadcrumb, pageContext: ((((MRuntime3_orders_index as any).metadata) as any))?.pageContext, placement: ((((MRuntime3_orders_index as any).metadata) as any))?.placement, Component: async (props: any) => { const mod = await import("@open-mercato/core/modules/orders/frontend/page"); const Component = (mod.default ?? mod) as any; return createElement(Component, props) } }
+            ],
+            backendRoutes: [
+              { pattern: "/backend/orders", requireAuth: ((((BMRuntime7_orders_index as any).metadata) as any))?.requireAuth, requireRoles: ((((BMRuntime7_orders_index as any).metadata) as any))?.requireRoles, requireFeatures: ((((BMRuntime7_orders_index as any).metadata) as any))?.requireFeatures, requireCustomerAuth: ((((BMRuntime7_orders_index as any).metadata) as any))?.requireCustomerAuth, requireCustomerFeatures: ((((BMRuntime7_orders_index as any).metadata) as any))?.requireCustomerFeatures, nav: ((((BMRuntime7_orders_index as any).metadata) as any))?.nav, title: ((((BMRuntime7_orders_index as any).metadata) as any))?.pageTitle ?? ((((BMRuntime7_orders_index as any).metadata) as any))?.title, titleKey: ((((BMRuntime7_orders_index as any).metadata) as any))?.pageTitleKey ?? ((((BMRuntime7_orders_index as any).metadata) as any))?.titleKey, group: ((((BMRuntime7_orders_index as any).metadata) as any))?.pageGroup ?? ((((BMRuntime7_orders_index as any).metadata) as any))?.group, groupKey: ((((BMRuntime7_orders_index as any).metadata) as any))?.pageGroupKey ?? ((((BMRuntime7_orders_index as any).metadata) as any))?.groupKey, icon: ((((BMRuntime7_orders_index as any).metadata) as any))?.icon, order: ((((BMRuntime7_orders_index as any).metadata) as any))?.pageOrder ?? ((((BMRuntime7_orders_index as any).metadata) as any))?.order, priority: ((((BMRuntime7_orders_index as any).metadata) as any))?.pagePriority ?? ((((BMRuntime7_orders_index as any).metadata) as any))?.priority, navHidden: ((((BMRuntime7_orders_index as any).metadata) as any))?.navHidden, visible: ((((BMRuntime7_orders_index as any).metadata) as any))?.visible, enabled: ((((BMRuntime7_orders_index as any).metadata) as any))?.enabled, breadcrumb: ((((BMRuntime7_orders_index as any).metadata) as any))?.breadcrumb, pageContext: ((((BMRuntime7_orders_index as any).metadata) as any))?.pageContext, placement: ((((BMRuntime7_orders_index as any).metadata) as any))?.placement, Component: async (props: any) => { const mod = await import("@open-mercato/core/modules/orders/backend/page"); const Component = (mod.default ?? mod) as any; return createElement(Component, props) } }
+            ],
+            apis: [
+              { path: "/orders/orders", metadata: undefined, handlers: { GET: async (req: Request, ctx?: any) => { const mod = await import("@open-mercato/core/modules/orders/api/orders/route"); return (mod as any).GET(req, ctx) } } }
+            ],
+            customFieldSets: []
           }
-      ]
-      export const modulesInfo = modules.map(m => ({ id: m.id, ...(m.info || {}) }))
-      export default modules
+        ];
+
+      export default modules;
+
+      export const modulesInfo = modules.map((m) => ({
+          id: m.id,
+          ...m.info || {}
+        }));
     `))
     expect(readGenerated(tmpDir, 'frontend-routes.generated.ts')).toBe(dedent(`
       // AUTO-GENERATED by mercato generate registry
-      import { resolvePageRouteMetadata, type FrontendRouteManifestEntry } from '@open-mercato/shared/modules/registry'
+      import { resolvePageRouteMetadata, type FrontendRouteManifestEntry } from "@open-mercato/shared/modules/registry";
 
       export const frontendRoutes: FrontendRouteManifestEntry[] = [
-        { moduleId: "orders", ...resolvePageRouteMetadata("/", (undefined as any)), load: async () => { const mod = await import("@open-mercato/core/modules/orders/frontend/page"); return (mod.default ?? mod) as any } }
-      ]
+          {
+            moduleId: "orders",
+            ...resolvePageRouteMetadata("/", (undefined as any)),
+            load: async () => {
+              const mod = await import("@open-mercato/core/modules/orders/frontend/page");
+              return ((mod.default ?? mod) as any);
+            }
+          }
+        ];
 
-      export default frontendRoutes
+      export default frontendRoutes;
     `))
     expect(readGenerated(tmpDir, 'backend-routes.generated.ts')).toBe(dedent(`
       // AUTO-GENERATED by mercato generate registry
-      import { resolvePageRouteMetadata, type BackendRouteManifestEntry } from '@open-mercato/shared/modules/registry'
+      import { resolvePageRouteMetadata, type BackendRouteManifestEntry } from "@open-mercato/shared/modules/registry";
 
       export const backendRoutes: BackendRouteManifestEntry[] = [
-        { moduleId: "orders", ...resolvePageRouteMetadata("/backend/orders", (undefined as any)), load: async () => { const mod = await import("@open-mercato/core/modules/orders/backend/page"); return (mod.default ?? mod) as any } }
-      ]
+          {
+            moduleId: "orders",
+            ...resolvePageRouteMetadata("/backend/orders", (undefined as any)),
+            load: async () => {
+              const mod = await import("@open-mercato/core/modules/orders/backend/page");
+              return ((mod.default ?? mod) as any);
+            }
+          }
+        ];
 
-      export default backendRoutes
+      export default backendRoutes;
     `))
     expect(readGenerated(tmpDir, 'api-routes.generated.ts')).toBe(dedent(`
       // AUTO-GENERATED by mercato generate registry
-      import type { ApiRouteManifestEntry } from '@open-mercato/shared/modules/registry'
+      import type { ApiRouteManifestEntry } from "@open-mercato/shared/modules/registry";
 
       export const apiRoutes: ApiRouteManifestEntry[] = [
-        { moduleId: "orders", kind: "route-file", path: "/orders/orders", methods: ["GET"], load: async () => import("@open-mercato/core/modules/orders/api/orders/route") }
-      ]
+          {
+            moduleId: "orders",
+            kind: "route-file",
+            path: "/orders/orders",
+            methods: [
+              "GET"
+            ],
+            load: async () => import("@open-mercato/core/modules/orders/api/orders/route")
+          }
+        ];
 
-      export default apiRoutes
+      export default apiRoutes;
     `))
   })
 
@@ -575,7 +599,7 @@ export { POST, PUT, DELETE }
     expect(result.errors).toEqual([])
     const apiRoutes = readGenerated(tmpDir, 'api-routes.generated.ts')!
     expect(apiRoutes).toContain('path: "/customers/people"')
-    expect(apiRoutes).toContain('methods: ["GET", "POST", "PUT", "DELETE"]')
+    expect(apiRoutes).toMatch(/methods:\s*\[[\s\S]*?"GET",[\s\S]*?"POST",[\s\S]*?"PUT",[\s\S]*?"DELETE"[\s\S]*?\]/)
   })
 
   it('partitions API and backend loader graphs while preserving legacy manifests', async () => {
@@ -660,7 +684,7 @@ export const { metadata, GET, POST, PUT, DELETE } = crud
     expect(result.errors).toEqual([])
     const apiRoutes = readGenerated(tmpDir, 'api-routes.generated.ts')!
     expect(apiRoutes).toContain('path: "/example/todos"')
-    expect(apiRoutes).toContain('methods: ["GET", "POST", "PUT", "DELETE"]')
+    expect(apiRoutes).toMatch(/methods:\s*\[[\s\S]*?"GET",[\s\S]*?"POST",[\s\S]*?"PUT",[\s\S]*?"DELETE"[\s\S]*?\]/)
   })
 
   it('uses metadata.path in api route manifests when source import fails during generation', async () => {
@@ -1047,7 +1071,7 @@ export function GET() {
     )
 
     expect(output).toContain('@open-mercato/core/modules/customers/i18n/en.json')
-    expect(output).toContain(`'en':`)
+    expect(output).toMatch(/['"]en['"]:/)
   })
 })
 

package/src/lib/generators/__tests__/output-snapshots.test.ts

@@ -832,7 +832,7 @@ describe('generator output compatibility', () => {
     expect(content).toContain(`import { backendRouteFacades } from "./backend-route-shards.generated"`)
     expect(content).toContain(`import { frontendRoutes } from "./frontend-routes.generated"`)
     expect(content).toContain(
-      `import { registerBackendRouteManifests, registerFrontendRouteManifests } from '@open-mercato/shared/modules/registry'`,
+      `import { registerBackendRouteManifests, registerFrontendRouteManifests } from "@open-mercato/shared/modules/registry"`,
     )
     expect(content).toMatch(/export function runBootstrapRegistrations\(\): void \{[\s\S]*registerBackendRouteManifests\(backendRouteFacades\)[\s\S]*\}/)
     expect(content).toMatch(/export function runBootstrapRegistrations\(\): void \{[\s\S]*registerFrontendRouteManifests\(frontendRoutes\)[\s\S]*\}/)