fix(compress): skip summary requirement in validateArgs when delegation is active

Author: loss-and-quick

lib/compress/message-utils.ts

@@ -27,7 +27,7 @@ class SoftIssue extends Error {
     }
 }
 
-export function validateArgs(args: CompressMessageToolArgs): void {
+export function validateArgs(args: CompressMessageToolArgs, delegated = false): void {
     if (typeof args.topic !== "string" || args.topic.trim().length === 0) {
         throw new Error("topic is required and must be a non-empty string")
     }
@@ -48,7 +48,7 @@ export function validateArgs(args: CompressMessageToolArgs): void {
             throw new Error(`${prefix}.topic is required and must be a non-empty string`)
         }
 
-        if (typeof entry?.summary !== "string" || entry.summary.trim().length === 0) {
+        if (!delegated && (typeof entry?.summary !== "string" || entry.summary.trim().length === 0)) {
             throw new Error(`${prefix}.summary is required and must be a non-empty string`)
         }
     }

lib/compress/message.ts

@@ -51,7 +51,7 @@ export function createCompressMessageTool(ctx: ToolContext): ReturnType<typeof t
         args: buildSchema(),
         async execute(args, toolCtx) {
             const input = args as CompressMessageToolArgs
-            validateArgs(input)
+            validateArgs(input, delegated)
             const callId =
                 typeof (toolCtx as unknown as { callID?: unknown }).callID === "string"
                     ? (toolCtx as unknown as { callID: string }).callID

lib/compress/range-utils.ts

@@ -11,7 +11,7 @@ import type {
 
 const BLOCK_PLACEHOLDER_REGEX = /\(b(\d+)\)|\{block_(\d+)\}/gi
 
-export function validateArgs(args: CompressRangeToolArgs): void {
+export function validateArgs(args: CompressRangeToolArgs, delegated = false): void {
     if (typeof args.topic !== "string" || args.topic.trim().length === 0) {
         throw new Error("topic is required and must be a non-empty string")
     }
@@ -32,7 +32,7 @@ export function validateArgs(args: CompressRangeToolArgs): void {
             throw new Error(`${prefix}.endId is required and must be a non-empty string`)
         }
 
-        if (typeof entry?.summary !== "string" || entry.summary.trim().length === 0) {
+        if (!delegated && (typeof entry?.summary !== "string" || entry.summary.trim().length === 0)) {
             throw new Error(`${prefix}.summary is required and must be a non-empty string`)
         }
     }

lib/compress/range.ts

@@ -66,7 +66,7 @@ export function createCompressRangeTool(ctx: ToolContext): ReturnType<typeof too
         args: buildSchema(),
         async execute(args, toolCtx) {
             const input = args as CompressRangeToolArgs
-            validateArgs(input)
+            validateArgs(input, delegated)
             const callId =
                 typeof (toolCtx as unknown as { callID?: unknown }).callID === "string"
                     ? (toolCtx as unknown as { callID: string }).callID