fix: static prompts, move agents out of tasks

Author: solaris007

package.json

@@ -42,7 +42,8 @@
     "timeout": 900000,
     "nodeVersion": 22,
     "static": [
-      "src/tasks/agents/brand-profile/prompts",
+      "src/agents/brand-profile/prompts/system.prompt",
+      "src/agents/brand-profile/prompts/user.prompt",
       "static/cls1.md",
       "static/cls2.md",
       "static/inp1.md",

src/tasks/agents/base.js renamed to src/agents/base.js

@@ -12,7 +12,7 @@
 import { ok, badRequest } from '@adobe/spacecat-shared-http-utils';
 import { hasText, isNonEmptyObject } from '@adobe/spacecat-shared-utils';
 
-import { getAgent } from '../agents/registry.js';
+import { getAgent } from '../../agents/registry.js';
 
 /**
  * Message shape:

src/tasks/agents/brand-profile/index.js renamed to src/agents/brand-profile/index.js

@@ -18,7 +18,7 @@ import path from 'path';
 import os from 'os';
 import fs from 'fs';
 
-import { readPromptFile, renderTemplate } from '../../../src/tasks/agents/base.js';
+import { readPromptFile, renderTemplate } from '../../src/agents/base.js';
 
 describe('agents/base utilities', () => {
   describe('renderTemplate', () => {

src/tasks/agents/brand-profile/prompts/system.prompt renamed to src/agents/brand-profile/prompts/system.prompt

@@ -49,11 +49,11 @@ describe('agents/brand-profile', () => {
     });
     const createFrom = sandbox.stub().returns({ fetchChatCompletion });
 
-    const mod = await esmock('../../../../src/tasks/agents/brand-profile/index.js', {
+    const mod = await esmock('../../../src/agents/brand-profile/index.js', {
       '@adobe/spacecat-shared-gpt-client': {
         AzureOpenAIClient: { createFrom },
       },
-      '../../../../src/tasks/agents/base.js': {
+      '../../../src/agents/base.js': {
         readPromptFile: sandbox.stub()
           .onFirstCall()
           .returns('SYS_PROMPT')
@@ -78,8 +78,8 @@ describe('agents/brand-profile', () => {
   });
 
   it('run() throws on invalid baseURL', async () => {
-    const mod = await esmock('../../../../src/tasks/agents/brand-profile/index.js', {
-      '../../../../src/tasks/agents/base.js': {
+    const mod = await esmock('../../../src/agents/brand-profile/index.js', {
+      '../../../src/agents/base.js': {
         readPromptFile: sandbox.stub(),
         renderTemplate: sandbox.stub(),
       },
@@ -94,11 +94,11 @@ describe('agents/brand-profile', () => {
     });
     const createFrom = sandbox.stub().returns({ fetchChatCompletion });
 
-    const mod = await esmock('../../../../src/tasks/agents/brand-profile/index.js', {
+    const mod = await esmock('../../../src/agents/brand-profile/index.js', {
       '@adobe/spacecat-shared-gpt-client': {
         AzureOpenAIClient: { createFrom },
       },
-      '../../../../src/tasks/agents/base.js': {
+      '../../../src/agents/base.js': {
         readPromptFile: sandbox.stub()
           .onFirstCall()
           .returns('SYS_PROMPT')
@@ -119,11 +119,11 @@ describe('agents/brand-profile', () => {
     });
     const createFrom = sandbox.stub().returns({ fetchChatCompletion });
 
-    const mod = await esmock('../../../../src/tasks/agents/brand-profile/index.js', {
+    const mod = await esmock('../../../src/agents/brand-profile/index.js', {
       '@adobe/spacecat-shared-gpt-client': {
         AzureOpenAIClient: { createFrom },
       },
-      '../../../../src/tasks/agents/base.js': {
+      '../../../src/agents/base.js': {
         readPromptFile: sandbox.stub()
           .onFirstCall()
           .returns('SYS_PROMPT')
@@ -142,7 +142,7 @@ describe('agents/brand-profile', () => {
   });
 
   it('persist() returns early for invalid siteId', async () => {
-    const mod = await esmock('../../../../src/tasks/agents/brand-profile/index.js', {});
+    const mod = await esmock('../../../src/agents/brand-profile/index.js', {});
 
     await mod.default.persist(
       { context: { siteId: 'invalid' } },
@@ -153,7 +153,7 @@ describe('agents/brand-profile', () => {
   });
 
   it('persist() returns early for empty result', async () => {
-    const mod = await esmock('../../../../src/tasks/agents/brand-profile/index.js', {});
+    const mod = await esmock('../../../src/agents/brand-profile/index.js', {});
     await mod.default.persist(
       { context: { siteId: '123e4567-e89b-12d3-a456-426614174000' } },
       context,
@@ -166,7 +166,7 @@ describe('agents/brand-profile', () => {
     const findById = sandbox.stub().resolves(null);
     context.dataAccess.Site = { findById };
 
-    const mod = await esmock('../../../../src/tasks/agents/brand-profile/index.js', {});
+    const mod = await esmock('../../../src/agents/brand-profile/index.js', {});
     await mod.default.persist(
       { context: { siteId: '123e4567-e89b-12d3-a456-426614174000' } },
       context,
@@ -195,7 +195,7 @@ describe('agents/brand-profile', () => {
     context.dataAccess.Site = { findById };
 
     const toDynamoItem = sandbox.stub().callsFake((c) => c);
-    const mod = await esmock('../../../../src/tasks/agents/brand-profile/index.js', {
+    const mod = await esmock('../../../src/agents/brand-profile/index.js', {
       '@adobe/spacecat-shared-data-access/src/models/site/config.js': {
         Config: { toDynamoItem },
       },
@@ -230,7 +230,7 @@ describe('agents/brand-profile', () => {
     context.dataAccess.Site = { findById };
 
     const toDynamoItem = sandbox.stub().callsFake((c) => c);
-    const mod = await esmock('../../../../src/tasks/agents/brand-profile/index.js', {
+    const mod = await esmock('../../../src/agents/brand-profile/index.js', {
       '@adobe/spacecat-shared-data-access/src/models/site/config.js': {
         Config: { toDynamoItem },
       },
@@ -267,7 +267,7 @@ describe('agents/brand-profile', () => {
     context.dataAccess.Site = { findById };
 
     const toDynamoItem = sandbox.stub().callsFake((c) => c);
-    const mod = await esmock('../../../../src/tasks/agents/brand-profile/index.js', {
+    const mod = await esmock('../../../src/agents/brand-profile/index.js', {
       '@adobe/spacecat-shared-data-access/src/models/site/config.js': {
         Config: { toDynamoItem },
       },

src/tasks/agents/brand-profile/prompts/user.prompt renamed to src/agents/brand-profile/prompts/user.prompt

@@ -37,8 +37,8 @@ describe('agents/registry', () => {
       persist: sandbox.stub().resolves(),
     };
 
-    const registryModule = await esmock('../../../src/tasks/agents/registry.js', {
-      '../../../src/tasks/agents/brand-profile/index.js': {
+    const registryModule = await esmock('../../src/agents/registry.js', {
+      '../../src/agents/brand-profile/index.js': {
         default: mockAgent,
       },
     });
@@ -50,8 +50,8 @@ describe('agents/registry', () => {
   });
 
   it('returns null for unknown agent id', async () => {
-    const registryModule = await esmock('../../../src/tasks/agents/registry.js', {
-      '../../../src/tasks/agents/brand-profile/index.js': {
+    const registryModule = await esmock('../../src/agents/registry.js', {
+      '../../src/agents/brand-profile/index.js': {
         default: {},
       },
     });