@agentforge/tools 0.15.8 → 0.15.9

package/dist/index.js

@@ -807,8 +807,8 @@ function createSendSlackMessageTool(getSlackClient, logger22) {
         icon_emoji: config.botIcon
       });
       logger22.info("send-slack-message result", {
-        channel: result.channel,
-        timestamp: result.ts,
+        ...result.channel ? { channel: result.channel } : {},
+        ...result.ts ? { timestamp: result.ts } : {},
         messageLength: message.length,
         success: true
       });
@@ -853,8 +853,8 @@ function createNotifySlackTool(getSlackClient, logger22) {
       icon_emoji: config.botIcon
     });
     logger22.info("notify-slack result", {
-      channel: result.channel,
-      timestamp: result.ts,
+      ...result.channel ? { channel: result.channel } : {},
+      ...result.ts ? { timestamp: result.ts } : {},
       mentions: mentions.length
     });
     return {
@@ -1388,7 +1388,10 @@ function createArchiveConfluencePageTool(getAuth, getAuthHeader2, logger22) {
     page_id: z.string().describe("The ID of the page to archive"),
     reason: z.string().optional().describe("Optional reason for archiving (for audit trail)")
   })).implement(async ({ page_id, reason }) => {
-    logger22.info("archive-confluence-page called", { page_id, reason });
+    logger22.info("archive-confluence-page called", {
+      page_id,
+      ...reason ? { reason } : {}
+    });
     try {
       const { ATLASSIAN_SITE_URL } = getAuth();
       const getResponse = await axios16.get(
@@ -2133,7 +2136,7 @@ var Neo4jConnectionPool = class {
       const errorMessage = `Failed to connect to Neo4j: ${error instanceof Error ? error.message : "Unknown error"}`;
       logger3.error("Neo4j connectivity verification failed", {
         error: error instanceof Error ? error.message : "Unknown error",
-        uri: this.config?.uri
+        ...this.config?.uri ? { uri: this.config.uri } : {}
       });
       throw new Error(errorMessage);
     }
@@ -3158,7 +3161,7 @@ function createNeo4jQueryTool() {
     logger5.debug("Executing Neo4j query", {
       cypherPreview: input.cypher.substring(0, 100),
       parameterCount: Object.keys(input.parameters || {}).length,
-      database: input.database
+      ...input.database ? { database: input.database } : {}
     });
     try {
       const session = neo4jPool.getSession(input.database);
@@ -3530,8 +3533,8 @@ function createNeo4jVectorSearchWithEmbeddingTool() {
     logger6.debug("Performing vector search with embedding", {
       queryTextLength: input.queryText.length,
       indexName: input.indexName,
-      limit: input.limit,
-      model: input.model
+      ...input.limit !== void 0 ? { limit: input.limit } : {},
+      ...input.model ? { model: input.model } : {}
     });
     try {
       const embeddingResult = await embeddingManager.generateEmbedding(input.queryText, input.model);
@@ -4358,9 +4361,9 @@ var ConnectionManager = class extends EventEmitter {
     logger7.debug("Creating PostgreSQL connection pool", {
       vendor: this.vendor,
       poolConfig: {
-        max: connectionConfig.max,
-        idleTimeoutMillis: connectionConfig.idleTimeoutMillis,
-        connectionTimeoutMillis: connectionConfig.connectionTimeoutMillis
+        ...connectionConfig.max !== void 0 ? { max: connectionConfig.max } : {},
+        ...connectionConfig.idleTimeoutMillis !== void 0 ? { idleTimeoutMillis: connectionConfig.idleTimeoutMillis } : {},
+        ...connectionConfig.connectionTimeoutMillis !== void 0 ? { connectionTimeoutMillis: connectionConfig.connectionTimeoutMillis } : {}
       }
     });
     this.client = new Pool(connectionConfig);
@@ -4395,9 +4398,9 @@ var ConnectionManager = class extends EventEmitter {
       logger7.debug("Creating MySQL connection pool", {
         vendor: this.vendor,
         poolConfig: {
-          connectionLimit: connectionConfig.connectionLimit,
-          acquireTimeout: connectionConfig.acquireTimeout,
-          idleTimeout: connectionConfig.idleTimeout
+          ...connectionConfig.connectionLimit !== void 0 ? { connectionLimit: connectionConfig.connectionLimit } : {},
+          ...connectionConfig.acquireTimeout !== void 0 ? { acquireTimeout: connectionConfig.acquireTimeout } : {},
+          ...connectionConfig.idleTimeout !== void 0 ? { idleTimeout: connectionConfig.idleTimeout } : {}
         }
       });
     }
@@ -4422,7 +4425,11 @@ var ConnectionManager = class extends EventEmitter {
       validatePoolConfig(this.config.connection.pool);
       logger7.debug("SQLite pool configuration provided but not applied (SQLite uses single connection)", {
         vendor: this.vendor,
-        poolConfig: this.config.connection.pool
+        poolConfig: {
+          ...this.config.connection.pool.max !== void 0 ? { max: this.config.connection.pool.max } : {},
+          ...this.config.connection.pool.idleTimeoutMillis !== void 0 ? { idleTimeoutMillis: this.config.connection.pool.idleTimeoutMillis } : {},
+          ...this.config.connection.pool.acquireTimeoutMillis !== void 0 ? { acquireTimeoutMillis: this.config.connection.pool.acquireTimeoutMillis } : {}
+        }
       });
     }
     logger7.debug("Creating SQLite connection", {
@@ -5843,8 +5850,8 @@ async function withTransaction(manager, operation, options) {
   logger11.debug("Starting transaction", {
     transactionId,
     vendor,
-    isolationLevel: resolvedOptions.isolationLevel,
-    timeoutMs: resolvedOptions.timeoutMs
+    ...resolvedOptions.isolationLevel ? { isolationLevel: resolvedOptions.isolationLevel } : {},
+    ...resolvedOptions.timeoutMs !== void 0 ? { timeoutMs: resolvedOptions.timeoutMs } : {}
   });
   return manager.executeInConnection(async (executeQuery2) => {
     const transaction = new ManagedTransaction({
@@ -7165,8 +7172,8 @@ async function executeSelect(manager, input, context) {
           vendor: input.vendor,
           table: input.table,
           chunkSize: streamOptions.chunkSize ?? DEFAULT_CHUNK_SIZE,
-          maxRows: streamOptions.maxRows,
-          sampleSize: streamOptions.sampleSize
+          ...streamOptions.maxRows !== void 0 ? { maxRows: streamOptions.maxRows } : {},
+          ...streamOptions.sampleSize !== void 0 ? { sampleSize: streamOptions.sampleSize } : {}
         });
       }
       const streamResult = await executeStreamingSelect(executor, streamInput, streamOptions);
@@ -8124,7 +8131,7 @@ async function executeUpdate(manager, input, context) {
     vendor: input.vendor,
     table: input.table,
     hasWhere: !!input.where?.length,
-    allowFullTableUpdate: input.allowFullTableUpdate,
+    ...input.allowFullTableUpdate !== void 0 ? { allowFullTableUpdate: input.allowFullTableUpdate } : {},
     hasOptimisticLock: !!input.optimisticLock,
     operationCount: input.operations?.length ?? 0,
     batchModeEnabled: !!input.batch?.enabled
@@ -8630,8 +8637,8 @@ async function executeDelete(manager, input, context) {
     vendor: input.vendor,
     table: input.table,
     hasWhere: !!input.where?.length,
-    allowFullTableDelete: input.allowFullTableDelete,
-    cascade: input.cascade,
+    ...input.allowFullTableDelete !== void 0 ? { allowFullTableDelete: input.allowFullTableDelete } : {},
+    ...input.cascade !== void 0 ? { cascade: input.cascade } : {},
     softDelete: !!input.softDelete,
     operationCount: input.operations?.length ?? 0,
     batchModeEnabled: !!input.batch?.enabled
@@ -8651,7 +8658,7 @@ async function executeDelete(manager, input, context) {
       rowCount: result.rowCount,
       executionTime,
       softDelete: result.softDeleted,
-      cascade: input.cascade,
+      ...input.cascade !== void 0 ? { cascade: input.cascade } : {},
       batchMode: !!result.batch,
       partialSuccess: result.batch?.partialSuccess ?? false
     });
@@ -8666,7 +8673,7 @@ async function executeDelete(manager, input, context) {
       table: input.table,
       error: error instanceof Error ? error.message : String(error),
       executionTime,
-      cascade: input.cascade,
+      ...input.cascade !== void 0 ? { cascade: input.cascade } : {},
       softDelete: !!input.softDelete
     });
     const constraintMessage = getDeleteConstraintViolationMessage(error, input.cascade ?? false);
@@ -10037,14 +10044,26 @@ function createAskHumanTool() {
       suggestions: validatedInput.suggestions,
       status: "pending"
     };
-    logger21.debug("About to call interrupt()", { humanRequest });
+    logger21.debug("About to call interrupt()", {
+      humanRequest: {
+        id: humanRequest.id,
+        question: humanRequest.question,
+        priority: humanRequest.priority,
+        createdAt: humanRequest.createdAt,
+        timeout: humanRequest.timeout,
+        status: humanRequest.status,
+        ...humanRequest.context ? { hasContext: true } : {},
+        ...humanRequest.defaultResponse ? { defaultResponse: humanRequest.defaultResponse } : {},
+        ...humanRequest.suggestions ? { suggestions: humanRequest.suggestions } : {}
+      }
+    });
     let response;
     try {
       response = interrupt(humanRequest);
       logger21.debug("interrupt() returned successfully", { response, responseType: typeof response });
     } catch (error) {
       logger21.debug("interrupt() threw error (expected for GraphInterrupt)", {
-        errorType: error?.constructor?.name,
+        ...error && typeof error === "object" && "constructor" in error ? { errorType: error.constructor?.name } : {},
         error: error instanceof Error ? error.message : String(error)
       });
       throw error;