@vertesia/workflow 0.52.0 → 0.55.0

package/src/dsl/validation.test.ts

@@ -18,7 +18,7 @@ describe('workflow validation', () => {
         expect(errors.length).toBe(1);
     })
 
-    test('activities shpuld be an array', () => {
+    test('activities should be an array', () => {
         const workflow: any = {
             name: "test",
             activities: {}
@@ -81,7 +81,7 @@ describe('workflow validation', () => {
         expect(errors.length).toBe(0);
     })
 
-    test('import unknown imorted var through expressions', () => {
+    test('import unknown imported var through expressions', () => {
         const workflow: any = {
             name: "test",
             vars: { "foo": true },

package/src/dsl/vars.test.ts

@@ -125,7 +125,7 @@ describe('Workflow vars', () => {
         expect(resolved.prompt_data.message).toBe("hello");
         expect(resolved.data2).toBeTypeOf("object");
         expect(resolved.data2.message).toBe("hello");
-        expect(resolved.unnown).toBeUndefined();
+        expect(resolved.unknown).toBeUndefined();
     });
 
 

package/src/dsl/vars.ts

@@ -20,9 +20,9 @@ export function splitPath(path: string) {
 
 /**
  * Get the property named by "name" of the given object
- * If an array is idnexed using a string key then a map is done and an array with the content of the properties with that name are returned
+ * If an array is indexed using a string key then a map is done and an array with the content of the properties with that name are returned
  * Ex: docs.text => will return an array of text properties of the docs array
- * @param object the obejct
+ * @param object the object
  * @param name the name of the property.
  * @returns the property value
  */
@@ -128,7 +128,7 @@ export class Vars {
     map: Record<string, any>;
     /**
      * This property is used when resolving params. It contains the list of references that should not be resolved to their value
-     * but instead they need to return the string representation of the expression to eb able to regenrate another Vars instance with the same expression
+     * but instead they need to return the string representation of the expression to be able to regenerate another Vars instance with the same expression
      */
     //TODO this feature is no more used - it was replaced by importVars so we can now delete `preserveRefs`
     preserveRefs: Set<string> | undefined;
@@ -154,7 +154,7 @@ export class Vars {
     }
 
     /**
-     * Set a literal value (canboot set a ref)
+     * Set a literal value (cannot set a ref)
      * To add refs use `append()`
      * @param name
      * @param value
@@ -291,7 +291,7 @@ export class Vars {
     }
 
     getUnknownReferences(obj: any) {
-        const visitor = new UnknownRefrencesVisitor(this);
+        const visitor = new UnknownReferencesVisitor(this);
         new ObjectWalker().walk(obj, visitor);
         return visitor.result;
     }
@@ -311,7 +311,7 @@ function addImportVar(varPath: string, asName: string | undefined, vars: Vars, r
 }
 
 
-class UnknownRefrencesVisitor implements ObjectVisitor {
+class UnknownReferencesVisitor implements ObjectVisitor {
 
     result: { name: string, expression: string }[] = [];
 

package/src/dsl/workflow-exec-child.test.ts

@@ -1,6 +1,7 @@
-import { ContentEventName, DSLActivityExecutionPayload, DSLWorkflowExecutionPayload, DSLWorkflowStep } from '@vertesia/common';
+import * as protos from '@temporalio/proto';
 import { TestWorkflowEnvironment } from '@temporalio/testing';
 import { Worker } from '@temporalio/worker';
+import { ContentEventName, DSLActivityExecutionPayload, DSLWorkflowExecutionPayload, DSLWorkflowStep } from '@vertesia/common';
 import { afterAll, beforeAll, describe, expect, test } from 'vitest';
 import { dslWorkflow } from './dsl-workflow.js';
 import { setupActivity } from './setup/ActivityContext.js';
@@ -86,12 +87,21 @@ const steps2: DSLWorkflowStep[] = [
 // ========== test env setup ==========
 
 
-describe('DSL Workflow with chld workflows', () => {
+describe('DSL Workflow with child workflows', () => {
 
     let testEnv: TestWorkflowEnvironment;
 
     beforeAll(async () => {
         testEnv = await TestWorkflowEnvironment.createLocal();
+        const { connection } = testEnv;
+        await connection.operatorService.addSearchAttributes({
+            namespace: 'default',
+            searchAttributes: {
+                AccountId: protos.temporal.api.enums.v1.IndexedValueType.INDEXED_VALUE_TYPE_KEYWORD,
+                DocumentId: protos.temporal.api.enums.v1.IndexedValueType.INDEXED_VALUE_TYPE_KEYWORD,
+                ProjectId: protos.temporal.api.enums.v1.IndexedValueType.INDEXED_VALUE_TYPE_KEYWORD,
+            },
+        });
     });
 
     afterAll(async () => {
@@ -122,7 +132,7 @@ describe('DSL Workflow with chld workflows', () => {
             auth_token: 'test',
             config: {
                 studio_url: process.env.CP_STUDIO_URL || "http://localhost:8081",
-                store_url: process.env.CP_STODRE_URL || "http://localhost:8082",
+                store_url: process.env.CP_STORE_URL || "http://localhost:8082",
             },
             workflow: {
                 steps: steps1,
@@ -167,7 +177,7 @@ describe('DSL Workflow with chld workflows', () => {
             auth_token: 'test',
             config: {
                 studio_url: process.env.CP_STUDIO_URL || "http://localhost:8081",
-                store_url: process.env.CP_STODRE_URL || "http://localhost:8082",
+                store_url: process.env.CP_STORE_URL || "http://localhost:8082",
             },
             workflow: {
                 steps: steps2,

package/src/dsl/workflow-fetch.test.ts

@@ -110,7 +110,7 @@ describe('DSL Workflow', () => {
             auth_token: 'test',
             config: {
                 studio_url: process.env.CP_STUDIO_URL || "http://localhost:8081",
-                store_url: process.env.CP_STODRE_URL || "http://localhost:8082",
+                store_url: process.env.CP_STORE_URL || "http://localhost:8082",
             },
             workflow: {
                 activities,

package/src/dsl/workflow-import.test.ts

@@ -64,7 +64,7 @@ describe('DSL Workflow import vars', () => {
             auth_token: 'test',
             config: {
                 studio_url: process.env.CP_STUDIO_URL || "http://localhost:8081",
-                store_url: process.env.CP_STODRE_URL || "http://localhost:8082",
+                store_url: process.env.CP_STORE_URL || "http://localhost:8082",
             },
             workflow: {
                 activities,

package/src/dsl/workflow.test.ts

@@ -1,6 +1,7 @@
-import { ContentEventName, DSLActivityExecutionPayload, DSLActivitySpec, DSLWorkflowExecutionPayload } from '@vertesia/common';
+import * as protos from '@temporalio/proto';
 import { TestWorkflowEnvironment } from '@temporalio/testing';
 import { Worker } from '@temporalio/worker';
+import { ContentEventName, DSLActivityExecutionPayload, DSLActivitySpec, DSLWorkflowExecutionPayload } from '@vertesia/common';
 import { afterAll, beforeAll, describe, expect, it } from 'vitest';
 import { dslWorkflow } from './dsl-workflow.js';
 import { setupActivity } from "./setup/ActivityContext.js";
@@ -52,6 +53,15 @@ describe('DSL Workflow', () => {
 
     beforeAll(async () => {
         testEnv = await TestWorkflowEnvironment.createLocal();
+        const { connection } = testEnv;
+        await connection.operatorService.addSearchAttributes({
+            namespace: 'default',
+            searchAttributes: {
+                AccountId: protos.temporal.api.enums.v1.IndexedValueType.INDEXED_VALUE_TYPE_KEYWORD,
+                DocumentId: protos.temporal.api.enums.v1.IndexedValueType.INDEXED_VALUE_TYPE_KEYWORD,
+                ProjectId: protos.temporal.api.enums.v1.IndexedValueType.INDEXED_VALUE_TYPE_KEYWORD,
+            },
+        });
     });
 
     afterAll(async () => {
@@ -82,7 +92,7 @@ describe('DSL Workflow', () => {
             auth_token: 'test',
             config: {
                 studio_url: process.env.CP_STUDIO_URL || "http://localhost:8081",
-                store_url: process.env.CP_STODRE_URL || "http://localhost:8082",
+                store_url: process.env.CP_STORE_URL || "http://localhost:8082",
             },
             workflow: {
                 activities,

package/src/index.ts

@@ -3,7 +3,7 @@
  * Modify the `./activities/index.ts` if you want to modify activity exports
  * 2. Workflows are exported through the '/workflows' named export in package.json.
  * Modify the `./workflows.ts` file if you want to modify workflow exports
- * 3. Here we export the API to be used to validate workflows and the types reuired to create workflow TS definitions.
+ * 3. Here we export the API to be used to validate workflows and the types required to create workflow TS definitions.
  */
 
 //TODO remove this - it is only for backward compat - iot is used from old workflows

package/src/iterative-generation/activities/extractToc.ts

@@ -40,7 +40,7 @@ export async function it_gen_extractToc(payload: WorkflowExecutionPayload): Prom
     await buildAndPublishMemoryPack(client, `${vars.memory}/output`, async () => {
         return {
             toc,
-            lastProcessdPart: undefined, // the part index (a number array)
+            lastProcessedPart: undefined, // the part index (a number array)
             previouslyGenerated: ""
         } as OutputMemoryMeta
     });

package/src/iterative-generation/activities/generatePart.ts

@@ -32,7 +32,7 @@ export async function it_gen_generatePart(payload: WorkflowExecutionPayload, pat
 
     const content = await loadGeneratedContent(outMemory);
 
-    let previously_generated = getPreviouslyGeneratedContent(content, !part, vars.rememberance_strategy);
+    let previously_generated = getPreviouslyGeneratedContent(content, !part, vars.remembrance_strategy);
 
     if (!part) { // a new section
         content.push({
@@ -58,7 +58,7 @@ export async function it_gen_generatePart(payload: WorkflowExecutionPayload, pat
 
     const result = r.result as string;
     content[content.length - 1].content += result;
-    meta.lastProcessdPart = path;
+    meta.lastProcessedPart = path;
     await buildAndPublishMemoryPack(client, `${memory}/output`, async ({ copyText }) => {
         copyText(JSON.stringify(content, null, 2), "content.json");
         return meta;

package/src/iterative-generation/activities/generateToc.ts

@@ -89,7 +89,7 @@ export async function it_gen_generateToc(payload: WorkflowExecutionPayload): Pro
     await buildAndPublishMemoryPack(client, `${vars.memory}/output`, async () => {
         return {
             toc,
-            lastProcessdPart: undefined, // the part index (a number array)
+            lastProcessedPart: undefined, // the part index (a number array)
             previouslyGenerated: ""
         } as OutputMemoryMeta
     });

package/src/iterative-generation/iterativeGenerationWorkflow.ts

@@ -30,7 +30,7 @@ export async function iterativeGenerationWorkflow(payload: WorkflowExecutionPayl
     }
 
     // extractToc tries to extract the toc from the input memory pack (toc.json or toc.yaml)
-    // the generateToc activity is retiurning the toc hierarchy.
+    // the generateToc activity is returning the toc hierarchy.
     // It doesn't include extra TOC details like description etc.
     // To minimize the payload size only the hierarchy and the section/part names are returned
     let toc = await it_gen_extractToc(payload);

package/src/iterative-generation/types.ts

@@ -13,7 +13,7 @@ export interface IterativeGenerationPayload {
     // the main interaction will only be used to prepare the iteration (to generate the TOC)
     // otherwise it will be used for the iterative generation too.
     interaction: string;
-    // if defined this will be used for the iterative interaction which will genrate parts.
+    // if defined this will be used for the iterative interaction which will generate parts.
     // otherwise the main interaction will be used for iterative generation.
     iterative_interaction?: string;
     // the environment to use
@@ -34,10 +34,10 @@ export interface IterativeGenerationPayload {
      * If not set to "none" the previously generated content will be passed to the iteration.
      * If not set at all defaults to "section".
      * If "section" is used only the section content previously generated will be passed to the next iteration.
-     * If "document" is used the whole previopusly generated document content will be passed to the next iteration.
+     * If "document" is used the whole previously generated document content will be passed to the next iteration.
      * Defaults to section.
      */
-    rememberance_strategy?: "document" | "section" | "none";
+    remembrance_strategy?: "document" | "section" | "none";
     /**
      * If present will save sections in files using the pattern
      * The pattern must include a placeholder for the section id: %id.
@@ -88,7 +88,7 @@ export interface TocIndex {
 export interface OutputMemoryMeta {
     toc: Toc;
     previouslyGenerated: string;
-    lastProcessdPart?: number[] | undefined;
+    lastProcessedPart?: number[] | undefined;
 }
 
 export interface Section {

package/src/iterative-generation/utils.ts

@@ -79,19 +79,19 @@ export function getPreviousPathIndex(toc: Toc, pathIndex: number[]): number[] |
 
 
 export function expectMemoryIsConsistent(meta: OutputMemoryMeta, pathIndex: number[]) {
-    const metaLastProcessedPart = meta.lastProcessdPart;
+    const metaLastProcessedPart = meta.lastProcessedPart;
     if (!metaLastProcessedPart) {
         if (pathIndex.length > 1 && pathIndex[0] !== 0) {
-            throw ApplicationFailure.nonRetryable('Memory last processed part is not consitent with the workflow.', 'MemoryPackNotConsistent', { currentIndex: pathIndex, expectedPreviousIndex: [pathIndex[0], pathIndex[1] - 1], previousIndex: metaLastProcessedPart });
+            throw ApplicationFailure.nonRetryable('Memory last processed part is not consistent with the workflow.', 'MemoryPackNotConsistent', { currentIndex: pathIndex, expectedPreviousIndex: [pathIndex[0], pathIndex[1] - 1], previousIndex: metaLastProcessedPart });
         } else {
             return;
         }
     }
     const prevPathIndex = getPreviousPathIndex(meta.toc, pathIndex);
     if (!prevPathIndex) {
-        throw ApplicationFailure.nonRetryable('Memory last processed part is not consitent with the workflow', 'MemoryPackNotConsistent', { currentIndex: pathIndex, expectedPreviousIndex: prevPathIndex, previousIndex: metaLastProcessedPart });
+        throw ApplicationFailure.nonRetryable('Memory last processed part is not consistent with the workflow', 'MemoryPackNotConsistent', { currentIndex: pathIndex, expectedPreviousIndex: prevPathIndex, previousIndex: metaLastProcessedPart });
     } else if (!isSamePartIndex(prevPathIndex, metaLastProcessedPart)) {
-        throw ApplicationFailure.nonRetryable('Memory last processed part is not consitent with the workflow', 'MemoryPackNotConsistent', { currentIndex: pathIndex, expectedPreviousIndex: prevPathIndex, previousIndex: meta.lastProcessdPart });
+        throw ApplicationFailure.nonRetryable('Memory last processed part is not consistent with the workflow', 'MemoryPackNotConsistent', { currentIndex: pathIndex, expectedPreviousIndex: prevPathIndex, previousIndex: meta.lastProcessedPart });
     }
 }