@izara_project/izara-core-generate-service-code 1.0.97 → 1.0.98

package/src/plan/domainRegistry.js

@@ -52,12 +52,22 @@ import {
   attributeTreeObjectSchemaManagedPaths,
   attributeTreeRelationshipSchemaManagedPaths
 } from '../emit/schemas/attributeTree/emitAttributeTreeSchemas.js';
+import {
+  emitAttributeTreeTargetValueStandard,
+  attributeTreeTargetValueStandardManagedPaths
+} from '../emit/schemas/attributeTree/emitAttributeTreeTargetValueStandard.js';
 import {
   emitLedgerObjectSchemas,
   emitLedgerRelationshipSchemas,
   ledgerObjectSchemaManagedPaths,
   ledgerRelationshipSchemaManagedPaths
 } from '../emit/schemas/ledger/emitLedgerSchemas.js';
+import {
+  emitPropertyValueObjectSchemas,
+  emitPropertyValueRelationshipSchemas,
+  propertyValueObjectSchemaManagedPaths,
+  propertyValueRelationshipSchemaManagedPaths
+} from '../emit/schemas/propertyValue/emitPropertyValueSchemas.js';
 import {
   emitErrorSchemas,
   errorSchemaManagedPaths
@@ -71,6 +81,7 @@ export async function resolveEmissionGroups(context) {
     flows,
     notifications,
     errors,
+    directives = [],
     objectGraphServiceTags,
     relationshipGraphServiceTags,
     logger
@@ -85,7 +96,9 @@ export async function resolveEmissionGroups(context) {
     resource,
     coreFunction,
     systemText,
-    errorSchemas
+    errorSchemas,
+    ledgerObjectArtifacts,
+    ledgerRelationshipArtifacts
   ] = await Promise.all([
     emitCrudApplicationArtifacts({
       objects,
@@ -125,6 +138,14 @@ export async function resolveEmissionGroups(context) {
       project,
       errors,
       objectGraphServiceTags
+    }),
+    emitLedgerObjectSchemas({
+      objects,
+      serviceTag: project.serviceTag
+    }),
+    emitLedgerRelationshipSchemas({
+      objects,
+      serviceTag: project.serviceTag
     })
   ]);
 
@@ -185,6 +206,11 @@ export async function resolveEmissionGroups(context) {
       managedPaths: rbacFlowSchemaManagedPaths(objects),
       artifacts: rbacFlowArtifacts
     },
+    {
+      scope: 'attribute-tree-target-value-standard',
+      managedPaths: attributeTreeTargetValueStandardManagedPaths(directives),
+      artifacts: emitAttributeTreeTargetValueStandard({ directives })
+    },
     {
       scope: 'attribute-tree-object-schemas',
       managedPaths: attributeTreeObjectSchemaManagedPaths(objects),
@@ -204,15 +230,25 @@ export async function resolveEmissionGroups(context) {
     {
       scope: 'ledger-object-schemas',
       managedPaths: ledgerObjectSchemaManagedPaths(objects),
-      artifacts: emitLedgerObjectSchemas({
+      artifacts: ledgerObjectArtifacts
+    },
+    {
+      scope: 'ledger-relationship-schemas',
+      managedPaths: ledgerRelationshipSchemaManagedPaths(objects),
+      artifacts: ledgerRelationshipArtifacts
+    },
+    {
+      scope: 'property-value-object-schemas',
+      managedPaths: propertyValueObjectSchemaManagedPaths(objects),
+      artifacts: emitPropertyValueObjectSchemas({
         objects,
         serviceTag: project.serviceTag
       })
     },
     {
-      scope: 'ledger-relationship-schemas',
-      managedPaths: ledgerRelationshipSchemaManagedPaths(objects),
-      artifacts: emitLedgerRelationshipSchemas({
+      scope: 'property-value-relationship-schemas',
+      managedPaths: propertyValueRelationshipSchemaManagedPaths(objects),
+      artifacts: emitPropertyValueRelationshipSchemas({
         objects,
         serviceTag: project.serviceTag
       })

package/task.md

@@ -0,0 +1,132 @@
+# Task Checklist: Attribute Tree Per-Repo Code Generation Refactoring
+
+> **Canonical Architecture Plan**: [[10.Systems/GenerateServiceCode/30-Generation/Task Roadmap - Attribute Tree Per-Repo Generation Refactoring|Task Roadmap in Obsidian]]  
+> **Selected Design**: **Option C (`GenerateDirectives/`)** + **Style 1 (`generatedCode/<domain>/`)**  
+> **Repository**: `IzaraCore-SupportServices-GenerateServiceCode-Seagame`
+
+---
+
+## 🎯 Target Architecture: Style 1 (`generatedCode/<domain>/`)
+
+### Target Service (e.g. `SellOfferTermValueStandard`)
+```text
+app/src/schemas/
+├── GenerateDirectives/
+│   └── sellOfferTermValueStandard.js                       <-- [Directive] Developer creates this
+│
+├── ObjectSchemas/
+│   └── generatedCode/
+│       ├── attributeTree/
+│       │   └── sellOfferTermValueStandard.js               <-- [Emitted by TASK-02]
+│       └── propertyValue/
+│           ├── sellOfferTermValueStandardPropertyLabel.js   <-- [Emitted by PropertyValue addOn]
+│           └── sellOfferTermValueStandardPropertyValue.js   <-- [Emitted by PropertyValue addOn]
+│
+└── RelationshipSchemas/
+    └── generatedCode/
+        └── propertyValue/
+            ├── hasSellOfferTermValueStandardPropertyValue.js
+            ├── disabledSellOfferTermValueStandardPropertyValue.js
+            └── isSellOfferTermValueStandardPropertyLabel.js
+```
+
+### Source Service (Self-Repo)
+```text
+app/src/schemas/
+├── ObjectSchemas/
+│   ├── sellOffer.js                                        <-- Source ObjectSchema
+│   └── generatedCode/
+│       └── attributeTree/
+│           ├── sellOfferTermLink.js
+│           ├── sellOfferTermValue.js
+│           └── sellOfferTermLabel.js
+│
+└── RelationshipSchemas/
+    └── generatedCode/
+        └── attributeTree/
+            ├── hasSellOfferTermLink.js
+            ├── disabledSellOfferTermLink.js
+            ├── isSellOfferTerm.js
+            └── isSellOfferTermLabel.js
+```
+
+---
+
+## 📋 Implementation Checklist
+
+- [x] **TASK-01: Recursive Schema Loading & Directives Loader**
+  - **File**: `src/compile/loadSchemas.js`
+  - **Action**:
+    1. Update `loadLocalSchemaDirectory(project, directoryName, options)` to recursively find all `.js` files within subdirectories (supporting `ObjectSchemas/generatedCode/**` and `RelationshipSchemas/generatedCode/**`).
+    2. Export `loadGenerateDirectives(project, options)` calling `loadLocalSchemaDirectory(project, 'GenerateDirectives', options)`.
+  - **Acceptance Criteria**: `loadLocalSchemaDirectory(project, 'ObjectSchemas')` discovers and validates schemas nested inside `generatedCode/attributeTree/` and `generatedCode/propertyValue/`. Returns `[]` when `GenerateDirectives` folder is missing.
+
+- [x] **TASK-02: Target ValueStandard Schema Emitter**
+  - **File**: `src/emit/schemas/attributeTree/emitAttributeTreeTargetValueStandard.js` (NEW)
+  - **Action**:
+    1. Filter `directives` for `directive.type === 'attributeTreeTarget'`.
+    2. Emit `{tag}ValueStandard.js` to `app/src/schemas/ObjectSchemas/generatedCode/attributeTree/${tag}ValueStandard.js`.
+    3. Include preservation hook `//(<addOnDataStructure${tag}ValueStandard>)` with commented `propertyValue` addOn template.
+    4. Export `attributeTreeTargetValueStandardManagedPaths(directives)`.
+  - **Acceptance Criteria**: Emitted file adheres strictly to Izara backend syntax rules and matches Style 1 output path.
+
+- [x] **TASK-03: Align AttributeTree Self-Repo Emitter to Style 1**
+  - **File**: `src/emit/schemas/attributeTree/emitAttributeTreeSchemas.js`
+  - **Action**:
+    1. Update ObjectSchema emission paths from `generatedAttributeTree/` to `generatedCode/attributeTree/`.
+    2. Update RelationshipSchema emission paths from `generatedAttributeTree/` to `generatedCode/attributeTree/`.
+    3. Update `attributeTreeObjectSchemaManagedPaths` and `attributeTreeRelationshipSchemaManagedPaths`.
+  - **Acceptance Criteria**: Self-repo attributeTree schemas output cleanly into `generatedCode/attributeTree/`.
+
+- [x] **TASK-04: Align PropertyValue Emitter to Style 1**
+  - **File**: `src/emit/schemas/propertyValue/emitPropertyValueSchemas.js`
+  - **Action**:
+    1. Update ObjectSchema emission paths from `generatedPropertyValue/` to `generatedCode/propertyValue/`.
+    2. Update RelationshipSchema emission paths from `generatedPropertyValue/` to `generatedCode/propertyValue/`.
+    3. Update `propertyValueObjectSchemaManagedPaths` and `propertyValueRelationshipSchemaManagedPaths`.
+  - **Acceptance Criteria**: PropertyValue schemas output cleanly into `generatedCode/propertyValue/`.
+
+- [x] **TASK-05: Register Managed Scopes in Domain Registry**
+  - **File**: `src/plan/domainRegistry.js`
+  - **Action**:
+    1. Import `emitAttributeTreeTargetValueStandard` and `attributeTreeTargetValueStandardManagedPaths`.
+    2. Register new scope `attribute-tree-target-value-standard`.
+    3. Pass `directives` into `resolveEmissionGroups(context)`.
+  - **Acceptance Criteria**: All 5 dynamic schema scopes registered and tracked in `.generate-service-code-v2-managed.json`.
+
+- [x] **TASK-06: Decouple Generation Pipeline in `src/generate.js`**
+  - **File**: `src/generate.js`
+  - **Action**:
+    1. Load directives: `const directives = await loadGenerateDirectives(project, schemaLoadOptions);`.
+    2. Pass `directives` to `resolveEmissionGroups`.
+    3. Delete lines 104-111 (`targetFiles = await _emitTargetServiceAttributeTreeSchemas(...)`).
+    4. Delete lines 129-186 (`_emitTargetServiceAttributeTreeSchemas` function definition).
+  - **Acceptance Criteria**: Zero cross-repo filesystem writes. Generator operates strictly within `workspaceOutputRoot`.
+
+- [x] **TASK-07: Delete Legacy Cross-Repo Files**
+  - **Files to Delete**:
+    - `src/emit/schemas/attributeTree/resolveTargetServicePath.js`
+    - `src/emit/schemas/attributeTree/emitAttributeTreeTargetSchemas.js`
+  - **Action**: Remove dead files and any unused imports.
+  - **Acceptance Criteria**: `git status` shows clean file deletions.
+
+- [x] **TASK-08: Unit & Integration Test Verification**
+  - **Files**:
+    - `test/unit/schemas/attributeTreeDirectives.test.js` (NEW)
+    - `test/unit/schemas/attributeTreeSchemas.test.js` (Update paths)
+    - `test/unit/schemas/propertyValueSchemas.test.js` (Update paths)
+  - **Verification Command**: `npm test`
+  - **Acceptance Criteria**: All 198+ tests pass with zero regression.
+
+- [x] **TASK-09: End-to-End Cascading Verification**
+  - **Action**: Verify two-phase generation locally:
+    1. Target service with directive -> generates `sellOfferTermValueStandard.js`.
+    2. Target service uncomments `propertyValue` -> generates PropertyLabel, PropertyValue, and 3 Relationship schemas.
+  - **Acceptance Criteria**: Zero cross-repo dependency required.
+
+- [x] **TASK-10: Synchronize Vault & Taiga Documentation**
+  - **Vault Files**:
+    - `10.Systems/GenerateServiceCode/20-Schema/Dynamic Schema.md`
+    - `40.Vault/Operations/timeline-log.md`
+    - `40.Vault/Memory/active-context.md`
+  - **Taiga Wiki**: Update `generatecode` wiki page.

package/test/integration/publicApi.test.js

@@ -208,7 +208,7 @@ test('generate commits the system WebSocket completion path for own-topic flows'
   assert.match(webSocketComplete, /\(\<createIdentifiersFlow\>\)/);
   assert.match(webSocketComplete, /\(\<beforePostMessage\>\)/);
   assert.match(webSocketComplete, /postToConnection\(_izContext,/);
-  const webSocketHandler = await readFile(
+    const webSocketHandler = await readFile(
     join(
       rootPath,
       'app/src/generatedCode/SystemFlowSchemas/WebSocketMain/source/WebSocketComplete_HdrSqs.js'
@@ -217,6 +217,49 @@ test('generate commits the system WebSocket completion path for own-topic flows'
   );
   assert.match(webSocketHandler, /record\.body\.TopicArn/);
   assert.match(webSocketHandler, /addCallingFlowToPassOnProperties/);
+
+  const registerComplete = await readFile(
+    join(
+      rootPath,
+      'app/src/generatedCode/SystemFlowSchemas/Register/source/RegisterComplete_Main.js'
+    ),
+    'utf8'
+  );
+  assert.match(registerComplete, /getFlowSchema\.getFlowSchemaS3WithCache/);
+  assert.doesNotMatch(registerComplete, /getObjectSchema\.getFlowSchemaS3WithCache/);
+
+  const registerMain = await readFile(
+    join(
+      rootPath,
+      'app/src/generatedCode/SystemFlowSchemas/Register/source/Register_Main.js'
+    ),
+    'utf8'
+  );
+  assert.match(registerMain, /getFlowSchema\.getFlowSchemaS3WithCache/);
+  assert.doesNotMatch(registerMain, /getObjectSchema\.getFlowSchemaS3WithCache/);
+
+  assert.match(webSocketComplete, /getFlowSchema\.getFlowSchemaS3WithCache/);
+  assert.doesNotMatch(webSocketComplete, /getObjectSchema\.getFlowSchemaS3WithCache/);
+
+  const trackHdrWbs = await readFile(
+    join(
+      rootPath,
+      'app/src/generatedCode/FlowSchemas/Track/source/Track_HdrWbs.js'
+    ),
+    'utf8'
+  );
+  assert.match(trackHdrWbs, /getFlowSchema\.getFlowSchemaS3WithCache/);
+  assert.doesNotMatch(trackHdrWbs, /getObjectSchema\.getFlowSchemaS3WithCache/);
+
+  const trackWbsMain = await readFile(
+    join(
+      rootPath,
+      'app/src/generatedCode/FlowSchemas/Track/source/Track_WbsMain.js'
+    ),
+    'utf8'
+  );
+  assert.match(trackWbsMain, /getFlowSchema\.getFlowSchemaS3WithCache/);
+  assert.doesNotMatch(trackWbsMain, /getObjectSchema\.getFlowSchemaS3WithCache/);
 });
 
 test('generate commits the trigger-cache suite with its queues when triggerFlowTypes is set', async () => {
@@ -245,6 +288,16 @@ test('generate commits the trigger-cache suite with its queues when triggerFlowT
     ),
     /CartCheckTriggerFlowsHdrSqs/
   );
+
+  const cartCheckTrigger = await readFile(
+    join(
+      rootPath,
+      'app/src/generatedCode/FlowSchemas/Cart/source/CartCheckTriggerFlows_Main.js'
+    ),
+    'utf8'
+  );
+  assert.match(cartCheckTrigger, /getFlowSchema\.getFlowSchemaS3WithCache/);
+  assert.doesNotMatch(cartCheckTrigger, /getObjectSchema\.getFlowSchemaS3WithCache/);
 });
 
 test('generate removes the system WebSocket path when the last own-topic flow is removed', async () => {

package/test/unit/application/customFlowApplicationArtifacts.test.js

@@ -232,6 +232,15 @@ test('emits handlers, function YAML, and IAM roles for supported custom flow eve
       join(rootPath, `app/src/generatedCode/FlowSchemas/Upload/source/${path}`)
     );
   }
+  const uploadCreateObject = await readFile(
+    join(
+      rootPath,
+      'app/src/generatedCode/FlowSchemas/Upload/source/UploadCreateObject_Main.js'
+    ),
+    'utf8'
+  );
+  assert.match(uploadCreateObject, /getFlowSchema\.getFlowSchemaS3WithCache/);
+  assert.doesNotMatch(uploadCreateObject, /getObjectSchema\.getFlowSchemaS3WithCache/);
   for (const path of [
     'UploadReservedLimit_Main.js',
     'UploadCreatePreSignUrl_Main.js',

package/test/unit/application/relationshipApplicationArtifacts.test.js

@@ -154,6 +154,36 @@ test('emits valid hook tags without trailing periods and with correct callingFlo
   assert.doesNotMatch(changeMain, /\/\/\(<requestParamCreateRel>\)/);
 });
 
+test('imports getRelSchema and uses getRelationshipSchemaWithCache from getRelSchema in all relationship Main files', async () => {
+  const rootPath = await createProject(true);
+  await generateRelationshipApplicationArtifacts(rootPath, {
+    serviceTag: 'orders'
+  });
+
+  const actions = ['Create', 'Update', 'Delete', 'Get', 'Change', 'Move'];
+  for (const action of actions) {
+    const mainContent = await readFile(
+      join(
+        rootPath,
+        `app/src/generatedCode/FlowRelationships/${action}Relationship/source/${action}Relationship_Main.js`
+      ),
+      'utf8'
+    );
+    assert.match(
+      mainContent,
+      /import\s*\{[^}]*getRelSchema[^}]*\}\s*from\s*'@izara_project\/izara-core-library-service-schemas'/
+    );
+    assert.match(
+      mainContent,
+      /const\s*\{[^}]*getRelationshipSchemaWithCache[^}]*\}\s*=\s*getRelSchema/
+    );
+    assert.doesNotMatch(
+      mainContent,
+      /const\s*\{[^}]*getRelationshipSchemaWithCache[^}]*\}\s*=\s*getObjectSchema/
+    );
+  }
+});
+
 test('removes managed relationship artifacts after relationships are removed', async () => {
   const rootPath = await createProject(true);
   await generateRelationshipApplicationArtifacts(rootPath, {

package/test/unit/infrastructure/resourceArtifacts.test.js

@@ -167,6 +167,7 @@ test('emits queues, topics, tables, and IAM roles from schema data', async () =>
   assert.match(procFindDataRole, /dynamodb:Query/);
   assert.match(procFindDataRole, /sns:Publish/);
   assert.match(procFindDataRole, /FindData_Out/);
+  assert.match(procFindDataRole, /ProcessLogical_Out/);
   assert.doesNotMatch(
     iam,
     /s3:GetObjectVersion|sqs:DeleteMessageBatch|sqs:GetQueueUrl/

package/test/unit/output/mergeHooks.test.js

@@ -0,0 +1,268 @@
+import assert from 'node:assert/strict';
+import { mkdir, mkdtemp, rm, writeFile } from 'node:fs/promises';
+import os from 'node:os';
+import path from 'node:path';
+import test from 'node:test';
+
+import { mergeHooks } from '../../../src/output/mergeHooks.js';
+
+async function writeTestFile(tmpDir, relativePath, content) {
+  const fullPath = path.join(tmpDir, relativePath);
+  await mkdir(path.dirname(fullPath), { recursive: true });
+  await writeFile(fullPath, content, 'utf8');
+}
+
+test('merges single hook in JS file when source content has custom code', async () => {
+  const tmpDir = await mkdtemp(path.join(os.tmpdir(), 'mergeHooks-test-'));
+  try {
+    const filePath = 'app/src/sample.js';
+    await writeTestFile(
+      tmpDir,
+      filePath,
+      '//(<customHook>)\nconst custom = true;\n//(</customHook>)\n'
+    );
+
+    const artifacts = [
+      {
+        path: filePath,
+        content: '//(<customHook>)\n//(</customHook>)\n'
+      }
+    ];
+
+    const result = await mergeHooks(tmpDir, artifacts);
+    assert.equal(
+      result[0].content,
+      '//(<customHook>)\nconst custom = true;\n//(</customHook>)\n'
+    );
+  } finally {
+    await rm(tmpDir, { recursive: true, force: true });
+  }
+});
+
+test('merges multiple occurrences of the same hook tag without losing occurrences', async () => {
+  const tmpDir = await mkdtemp(path.join(os.tmpdir(), 'mergeHooks-test-'));
+  try {
+    const filePath = 'app/src/Flow_Main.js';
+    const diskContent = [
+      'function test() {',
+      '  //(<identifiers>)',
+      '  firstIdentifier: true',
+      '  //(</identifiers>)',
+      '',
+      '  //(<identifiers>)',
+      '  secondIdentifier: "customValue"',
+      '  //(</identifiers>)',
+      '}'
+    ].join('\n');
+    await writeTestFile(tmpDir, filePath, diskContent);
+
+    const templateContent = [
+      'function test() {',
+      '  //(<identifiers>)',
+      '  //(</identifiers>)',
+      '',
+      '  //(<identifiers>)',
+      '  //(</identifiers>)',
+      '}'
+    ].join('\n');
+
+    const artifacts = [
+      {
+        path: filePath,
+        content: templateContent
+      }
+    ];
+
+    const result = await mergeHooks(tmpDir, artifacts);
+    assert.equal(result[0].content, diskContent);
+  } finally {
+    await rm(tmpDir, { recursive: true, force: true });
+  }
+});
+
+test('preserves second hook when the first occurrence of a duplicate hook is empty on disk', async () => {
+  const tmpDir = await mkdtemp(path.join(os.tmpdir(), 'mergeHooks-test-'));
+  try {
+    const filePath = 'app/src/Flow_Main.js';
+    const diskContent = [
+      'function test() {',
+      '  //(<identifiers>)',
+      '  //(</identifiers>)',
+      '',
+      '  //(<identifiers>)',
+      '  orderId: requestParams.orderId',
+      '  //(</identifiers>)',
+      '}'
+    ].join('\n');
+    await writeTestFile(tmpDir, filePath, diskContent);
+
+    const templateContent = [
+      'function test() {',
+      '  //(<identifiers>)',
+      '  //(</identifiers>)',
+      '',
+      '  //(<identifiers>)',
+      '  //(</identifiers>)',
+      '}'
+    ].join('\n');
+
+    const artifacts = [
+      {
+        path: filePath,
+        content: templateContent
+      }
+    ];
+
+    const result = await mergeHooks(tmpDir, artifacts);
+    assert.equal(result[0].content, diskContent);
+  } finally {
+    await rm(tmpDir, { recursive: true, force: true });
+  }
+});
+
+test('merges hooks with relaxed whitespace around comment slashes and tags', async () => {
+  const tmpDir = await mkdtemp(path.join(os.tmpdir(), 'mergeHooks-test-'));
+  try {
+    const filePath = 'app/src/sample.js';
+    await writeTestFile(
+      tmpDir,
+      filePath,
+      '// (<spacedHook>)\nconst spaced = 1;\n// (</spacedHook>)\n'
+    );
+
+    const artifacts = [
+      {
+        path: filePath,
+        content: '//(<spacedHook>)\n//(</spacedHook>)\n'
+      }
+    ];
+
+    const result = await mergeHooks(tmpDir, artifacts);
+    assert.equal(
+      result[0].content,
+      '// (<spacedHook>)\nconst spaced = 1;\n// (</spacedHook>)\n'
+    );
+  } finally {
+    await rm(tmpDir, { recursive: true, force: true });
+  }
+});
+
+test('merges hooks with hyphens and dots in tag names', async () => {
+  const tmpDir = await mkdtemp(path.join(os.tmpdir(), 'mergeHooks-test-'));
+  try {
+    const filePath = 'app/src/sample.js';
+    await writeTestFile(
+      tmpDir,
+      filePath,
+      '//(<custom-hook.name>)\nconst customName = "valid";\n//(</custom-hook.name>)\n'
+    );
+
+    const artifacts = [
+      {
+        path: filePath,
+        content: '//(<custom-hook.name>)\n//(</custom-hook.name>)\n'
+      }
+    ];
+
+    const result = await mergeHooks(tmpDir, artifacts);
+    assert.equal(
+      result[0].content,
+      '//(<custom-hook.name>)\nconst customName = "valid";\n//(</custom-hook.name>)\n'
+    );
+  } finally {
+    await rm(tmpDir, { recursive: true, force: true });
+  }
+});
+
+test('merges hooks in YAML files with relaxed whitespace and hyphens', async () => {
+  const tmpDir = await mkdtemp(path.join(os.tmpdir(), 'mergeHooks-test-'));
+  try {
+    const filePath = 'app/sls_yaml/test.yml';
+    const diskContent = [
+      '# <#custom-filter-patterns#>',
+      'filterPatterns:',
+      '  - body: custom',
+      '# <#/custom-filter-patterns#>'
+    ].join('\n');
+    await writeTestFile(tmpDir, filePath, diskContent);
+
+    const templateContent = [
+      '#<#custom-filter-patterns#>',
+      '#<#/custom-filter-patterns#>'
+    ].join('\n');
+
+    const artifacts = [
+      {
+        path: filePath,
+        content: templateContent
+      }
+    ];
+
+    const result = await mergeHooks(tmpDir, artifacts);
+    assert.equal(result[0].content, diskContent);
+  } finally {
+    await rm(tmpDir, { recursive: true, force: true });
+  }
+});
+
+test('merges hooks in .mjs and .cjs files', async () => {
+  const tmpDir = await mkdtemp(path.join(os.tmpdir(), 'mergeHooks-test-'));
+  try {
+    const mjsPath = 'app/src/sample.mjs';
+    const cjsPath = 'app/src/sample.cjs';
+    await writeTestFile(
+      tmpDir,
+      mjsPath,
+      '//(<mjsHook>)\nexport default 1;\n//(</mjsHook>)\n'
+    );
+    await writeTestFile(
+      tmpDir,
+      cjsPath,
+      '//(<cjsHook>)\nmodule.exports = 2;\n//(</cjsHook>)\n'
+    );
+
+    const artifacts = [
+      {
+        path: mjsPath,
+        content: '//(<mjsHook>)\n//(</mjsHook>)\n'
+      },
+      {
+        path: cjsPath,
+        content: '//(<cjsHook>)\n//(</cjsHook>)\n'
+      }
+    ];
+
+    const result = await mergeHooks(tmpDir, artifacts);
+    assert.equal(
+      result[0].content,
+      '//(<mjsHook>)\nexport default 1;\n//(</mjsHook>)\n'
+    );
+    assert.equal(
+      result[1].content,
+      '//(<cjsHook>)\nmodule.exports = 2;\n//(</cjsHook>)\n'
+    );
+  } finally {
+    await rm(tmpDir, { recursive: true, force: true });
+  }
+});
+
+test('respects writeWhenFileExist false flag by returning disk content unchanged', async () => {
+  const tmpDir = await mkdtemp(path.join(os.tmpdir(), 'mergeHooks-test-'));
+  try {
+    const filePath = 'app/src/immutable.js';
+    await writeTestFile(tmpDir, filePath, '// existing disk content\n');
+
+    const artifacts = [
+      {
+        path: filePath,
+        content: '// new template content\n',
+        writeWhenFileExist: false
+      }
+    ];
+
+    const result = await mergeHooks(tmpDir, artifacts);
+    assert.equal(result[0].content, '// existing disk content\n');
+  } finally {
+    await rm(tmpDir, { recursive: true, force: true });
+  }
+});