Update clients/typescript/contextforgeClient.ts

VirtualAgentics · coderabbitai[bot] · web-flow · commit b8c237d87406 · 2025-10-24T11:20:57.000+02:00
Co-authored-by: coderabbitai[bot] &lt;136622811+coderabbitai[bot]@users.noreply.github.com&gt;
diff --git a/clients/typescript/contextforgeClient.ts b/clients/typescript/contextforgeClient.ts
@@ -524,10 +524,7 @@ export class ContextForgeClient {
     if (!phase || !['planning', 'execution', 'review'].includes(phase)) {
       throw new Error('phase must be one of: planning, execution, review');
     }
-    if (summary !== undefined && typeof summary !== 'string') {
-      throw new Error('summary must be a string when provided');
-    }
-    if (summary !== undefined && summary.trim() === '') {
+    if (summary !== undefined && (typeof summary !== 'string' || summary.trim() === '')) {
       throw new Error('summary must be a non-empty string when provided');
     }
 

Original file line number	Diff line number	Diff line change
`@@ -524,10 +524,7 @@ export class ContextForgeClient {`
`524`	`524`	`if (!phase \|\| !['planning', 'execution', 'review'].includes(phase)) {`
`525`	`525`	`throw new Error('phase must be one of: planning, execution, review');`
`526`	`526`	`}`
`527`		`- if (summary !== undefined && typeof summary !== 'string') {`
`528`		`- throw new Error('summary must be a string when provided');`
`529`		`- }`
`530`		`- if (summary !== undefined && summary.trim() === '') {`
	`527`	`+ if (summary !== undefined && (typeof summary !== 'string' \|\| summary.trim() === '')) {`
`531`	`528`	`throw new Error('summary must be a non-empty string when provided');`
`532`	`529`	`}`
`533`	`530`