@jshookmcp/jshook 0.2.1 → 0.2.2

package/dist/src/server/domains/graphql/handlers.impl.core.runtime.replay.js

@@ -1,5 +1,6 @@
-import { GraphQLRuntimeLimit, INTROSPECTION_QUERY, } from '../../domains/graphql/handlers.impl.core.runtime.shared.js';
+import { GRAPHQL_MAX_PREVIEW_CHARS, GRAPHQL_MAX_SCHEMA_CHARS, GRAPHQL_MAX_QUERY_CHARS, GRAPHQL_MAX_GRAPH_NODES, GRAPHQL_MAX_GRAPH_EDGES, INTROSPECTION_QUERY, } from '../../domains/graphql/handlers.impl.core.runtime.shared.js';
 import { GraphQLHandlersBase } from '../../domains/graphql/handlers.base.js';
+import { evaluateWithTimeout } from '../../../modules/collector/PageController.js';
 export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
     async handleCallGraphAnalyze(args) {
         try {
@@ -17,7 +18,7 @@ export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
                 }
             }
             const page = await this.collector.getActivePage();
-            const rawResult = await page.evaluate(({ maxDepth: depth, filterPattern: filter }) => {
+            const rawResult = await evaluateWithTimeout(page, ({ maxDepth: depth, filterPattern: filter }) => {
                 const globalScope = window;
                 const edgeMap = new Map();
                 const nodeMap = new Map();
@@ -185,16 +186,16 @@ export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
                 filterPattern,
             });
             const result = rawResult;
-            const nodesTruncated = result.nodes.length > GraphQLRuntimeLimit.MAX_GRAPH_NODES;
-            const edgesTruncated = result.edges.length > GraphQLRuntimeLimit.MAX_GRAPH_EDGES;
+            const nodesTruncated = result.nodes.length > GRAPHQL_MAX_GRAPH_NODES;
+            const edgesTruncated = result.edges.length > GRAPHQL_MAX_GRAPH_EDGES;
             return this.toResponse({
                 success: true,
-                nodes: result.nodes.slice(0, GraphQLRuntimeLimit.MAX_GRAPH_NODES),
-                edges: result.edges.slice(0, GraphQLRuntimeLimit.MAX_GRAPH_EDGES),
+                nodes: result.nodes.slice(0, GRAPHQL_MAX_GRAPH_NODES),
+                edges: result.edges.slice(0, GRAPHQL_MAX_GRAPH_EDGES),
                 stats: {
                     ...result.stats,
-                    nodesReturned: Math.min(result.nodes.length, GraphQLRuntimeLimit.MAX_GRAPH_NODES),
-                    edgesReturned: Math.min(result.edges.length, GraphQLRuntimeLimit.MAX_GRAPH_EDGES),
+                    nodesReturned: Math.min(result.nodes.length, GRAPHQL_MAX_GRAPH_NODES),
+                    edgesReturned: Math.min(result.edges.length, GRAPHQL_MAX_GRAPH_EDGES),
                     nodesTruncated,
                     edgesTruncated,
                 },
@@ -252,7 +253,7 @@ export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
                 matchType: rule.matchType,
                 createdAt: rule.createdAt,
             });
-            const replacementPreview = this.createPreview(replacement, GraphQLRuntimeLimit.MAX_PREVIEW_CHARS);
+            const replacementPreview = this.createPreview(replacement, GRAPHQL_MAX_PREVIEW_CHARS);
             return this.toResponse({
                 success: true,
                 message: 'Script replacement rule registered and interception enabled',
@@ -286,21 +287,31 @@ export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
             }
             const headers = this.normalizeHeaders(args.headers);
             const page = await this.collector.getActivePage();
-            const browserResult = (await page.evaluate(async (input) => {
+            const browserResult = (await evaluateWithTimeout(page, async (input) => {
                 const requestHeaders = {
                     'content-type': 'application/json',
                     ...input.headers,
                 };
                 try {
-                    const response = await fetch(input.endpoint, {
-                        method: 'POST',
-                        headers: requestHeaders,
-                        body: JSON.stringify({
-                            query: input.query,
-                            operationName: 'IntrospectionQuery',
-                        }),
-                    });
-                    const responseText = await response.text();
+                    const ac = new AbortController();
+                    const t = setTimeout(() => ac.abort(), 10000);
+                    let responseText;
+                    let response;
+                    try {
+                        response = await fetch(input.endpoint, {
+                            method: 'POST',
+                            headers: requestHeaders,
+                            body: JSON.stringify({
+                                query: input.query,
+                                operationName: 'IntrospectionQuery',
+                            }),
+                            signal: ac.signal,
+                        });
+                        responseText = await response.text();
+                    }
+                    finally {
+                        clearTimeout(t);
+                    }
                     let responseJson = null;
                     try {
                         responseJson = JSON.parse(responseText);
@@ -343,14 +354,16 @@ export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
                     status: browserResult.status,
                     statusText: browserResult.statusText,
                     error: browserResult.error ?? 'Introspection request failed',
-                    responsePreview: this.createPreview(browserResult.responseText || '', GraphQLRuntimeLimit.MAX_PREVIEW_CHARS),
+                    responsePreview: this.createPreview(browserResult.responseText || '', GRAPHQL_MAX_PREVIEW_CHARS),
                 });
             }
             const jsonRecord = browserResult.responseJson && typeof browserResult.responseJson === 'object'
                 ? browserResult.responseJson
                 : null;
-            const schemaPayload = jsonRecord && 'data' in jsonRecord ? jsonRecord.data : browserResult.responseJson ?? browserResult.responseText;
-            const schemaPreview = this.serializeForPreview(schemaPayload, GraphQLRuntimeLimit.MAX_SCHEMA_CHARS);
+            const schemaPayload = jsonRecord && 'data' in jsonRecord
+                ? jsonRecord.data
+                : (browserResult.responseJson ?? browserResult.responseText);
+            const schemaPreview = this.serializeForPreview(schemaPayload, GRAPHQL_MAX_SCHEMA_CHARS);
             const payload = {
                 success: browserResult.ok,
                 endpoint,
@@ -380,7 +393,7 @@ export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
         try {
             const limit = this.getNumberArg(args, 'limit', 50, 1, 200);
             const page = await this.collector.getActivePage();
-            const extraction = (await page.evaluate((maxItems) => {
+            const extraction = (await evaluateWithTimeout(page, (maxItems) => {
                 const globalScope = window;
                 const extracted = [];
                 let scannedRecords = 0;
@@ -446,7 +459,9 @@ export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
                             return null;
                         }
                     }
-                    if (trimmed.startsWith('query ') || trimmed.startsWith('mutation ') || trimmed.startsWith('subscription ')) {
+                    if (trimmed.startsWith('query ') ||
+                        trimmed.startsWith('mutation ') ||
+                        trimmed.startsWith('subscription ')) {
                         return { query: trimmed };
                     }
                     return null;
@@ -483,16 +498,22 @@ export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
                     const url = typeof record.url === 'string' ? record.url : '';
                     const method = typeof record.method === 'string' ? record.method : 'POST';
                     const timestamp = typeof record.timestamp === 'number' ? record.timestamp : null;
-                    const headers = (record.headers && typeof record.headers === 'object' && !Array.isArray(record.headers)
+                    const headers = (record.headers &&
+                        typeof record.headers === 'object' &&
+                        !Array.isArray(record.headers)
                         ? record.headers
                         : null) ??
-                        (record.requestHeaders && typeof record.requestHeaders === 'object' && !Array.isArray(record.requestHeaders)
+                        (record.requestHeaders &&
+                            typeof record.requestHeaders === 'object' &&
+                            !Array.isArray(record.requestHeaders)
                             ? record.requestHeaders
                             : null) ??
                         {};
                     const contentType = getHeader(headers, 'content-type').toLowerCase();
                     const bodyCandidates = [record.body, record.postData];
-                    if (record.options && typeof record.options === 'object' && !Array.isArray(record.options)) {
+                    if (record.options &&
+                        typeof record.options === 'object' &&
+                        !Array.isArray(record.options)) {
                         const optionsRecord = record.options;
                         bodyCandidates.push(optionsRecord.body);
                     }
@@ -559,8 +580,8 @@ export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
                 };
             }, limit));
             const queries = extraction.extracted.map((item, index) => {
-                const queryPreview = this.createPreview(item.query, GraphQLRuntimeLimit.MAX_QUERY_CHARS);
-                const variablesPreview = this.serializeForPreview(item.variables, GraphQLRuntimeLimit.MAX_PREVIEW_CHARS);
+                const queryPreview = this.createPreview(item.query, GRAPHQL_MAX_QUERY_CHARS);
+                const variablesPreview = this.serializeForPreview(item.variables, GRAPHQL_MAX_PREVIEW_CHARS);
                 const normalized = {
                     index,
                     source: item.source,
@@ -619,22 +640,32 @@ export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
             const operationName = operationNameRaw && operationNameRaw.trim().length > 0 ? operationNameRaw.trim() : null;
             const headers = this.normalizeHeaders(args.headers);
             const page = await this.collector.getActivePage();
-            const browserResult = (await page.evaluate(async (input) => {
+            const browserResult = (await evaluateWithTimeout(page, async (input) => {
                 const requestHeaders = {
                     'content-type': 'application/json',
                     ...input.headers,
                 };
                 try {
-                    const response = await fetch(input.endpoint, {
-                        method: 'POST',
-                        headers: requestHeaders,
-                        body: JSON.stringify({
-                            query: input.query,
-                            variables: input.variables,
-                            operationName: input.operationName,
-                        }),
-                    });
-                    const responseText = await response.text();
+                    const ac = new AbortController();
+                    const t = setTimeout(() => ac.abort(), 10000);
+                    let responseText;
+                    let response;
+                    try {
+                        response = await fetch(input.endpoint, {
+                            method: 'POST',
+                            headers: requestHeaders,
+                            body: JSON.stringify({
+                                query: input.query,
+                                variables: input.variables,
+                                operationName: input.operationName,
+                            }),
+                            signal: ac.signal,
+                        });
+                        responseText = await response.text();
+                    }
+                    finally {
+                        clearTimeout(t);
+                    }
                     let responseJson = null;
                     try {
                         responseJson = JSON.parse(responseText);
@@ -681,7 +712,7 @@ export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
                 responseHeaders: browserResult.responseHeaders ?? {},
             };
             if (browserResult.responseJson !== null) {
-                const responsePreview = this.serializeForPreview(browserResult.responseJson, GraphQLRuntimeLimit.MAX_SCHEMA_CHARS);
+                const responsePreview = this.serializeForPreview(browserResult.responseJson, GRAPHQL_MAX_SCHEMA_CHARS);
                 payload.responseLength = responsePreview.totalLength;
                 payload.responsePreview = responsePreview.preview;
                 payload.responseTruncated = responsePreview.truncated;
@@ -690,7 +721,7 @@ export class GraphQLToolHandlersRuntime extends GraphQLHandlersBase {
                 }
             }
             else {
-                const textPreview = this.createPreview(browserResult.responseText, GraphQLRuntimeLimit.MAX_SCHEMA_CHARS);
+                const textPreview = this.createPreview(browserResult.responseText, GRAPHQL_MAX_SCHEMA_CHARS);
                 payload.responseLength = textPreview.totalLength;
                 payload.responsePreview = textPreview.preview;
                 payload.responseTruncated = textPreview.truncated;

package/dist/src/server/domains/graphql/handlers.impl.core.runtime.script-replace.d.ts

@@ -1 +1 @@
-export { GraphQLToolHandlersRuntime as GraphQLToolHandlersScriptReplace, } from '../../domains/graphql/handlers.impl.core.runtime.replay.js';
+export { GraphQLToolHandlersRuntime as GraphQLToolHandlersScriptReplace } from '../../domains/graphql/handlers.impl.core.runtime.replay.js';

package/dist/src/server/domains/graphql/handlers.impl.core.runtime.script-replace.js

@@ -1 +1 @@
-export { GraphQLToolHandlersRuntime as GraphQLToolHandlersScriptReplace, } from '../../domains/graphql/handlers.impl.core.runtime.replay.js';
+export { GraphQLToolHandlersRuntime as GraphQLToolHandlersScriptReplace } from '../../domains/graphql/handlers.impl.core.runtime.replay.js';

package/dist/src/server/domains/graphql/handlers.impl.core.runtime.shared.d.ts

@@ -53,16 +53,5 @@ export interface ExtractedGraphQLQuery {
     timestamp: number | null;
     contentType: string;
 }
-export declare const enum GraphQLRuntimeLimit {
-    MAX_PREVIEW_CHARS = 4000,
-    MAX_SCHEMA_CHARS = 120000,
-    MAX_QUERY_CHARS = 12000,
-    MAX_GRAPH_NODES = 2000,
-    MAX_GRAPH_EDGES = 5000
-}
-export declare const GRAPHQL_MAX_PREVIEW_CHARS = GraphQLRuntimeLimit.MAX_PREVIEW_CHARS;
-export declare const GRAPHQL_MAX_SCHEMA_CHARS = GraphQLRuntimeLimit.MAX_SCHEMA_CHARS;
-export declare const GRAPHQL_MAX_QUERY_CHARS = GraphQLRuntimeLimit.MAX_QUERY_CHARS;
-export declare const GRAPHQL_MAX_GRAPH_NODES = GraphQLRuntimeLimit.MAX_GRAPH_NODES;
-export declare const GRAPHQL_MAX_GRAPH_EDGES = GraphQLRuntimeLimit.MAX_GRAPH_EDGES;
+export { GRAPHQL_MAX_PREVIEW_CHARS, GRAPHQL_MAX_SCHEMA_CHARS, GRAPHQL_MAX_QUERY_CHARS, GRAPHQL_MAX_GRAPH_NODES, GRAPHQL_MAX_GRAPH_EDGES, } from '../../../constants.js';
 export declare const INTROSPECTION_QUERY: string;

package/dist/src/server/domains/graphql/handlers.impl.core.runtime.shared.js

@@ -1,91 +1,79 @@
-export var GraphQLRuntimeLimit;
-(function (GraphQLRuntimeLimit) {
-    GraphQLRuntimeLimit[GraphQLRuntimeLimit["MAX_PREVIEW_CHARS"] = 4000] = "MAX_PREVIEW_CHARS";
-    GraphQLRuntimeLimit[GraphQLRuntimeLimit["MAX_SCHEMA_CHARS"] = 120000] = "MAX_SCHEMA_CHARS";
-    GraphQLRuntimeLimit[GraphQLRuntimeLimit["MAX_QUERY_CHARS"] = 12000] = "MAX_QUERY_CHARS";
-    GraphQLRuntimeLimit[GraphQLRuntimeLimit["MAX_GRAPH_NODES"] = 2000] = "MAX_GRAPH_NODES";
-    GraphQLRuntimeLimit[GraphQLRuntimeLimit["MAX_GRAPH_EDGES"] = 5000] = "MAX_GRAPH_EDGES";
-})(GraphQLRuntimeLimit || (GraphQLRuntimeLimit = {}));
-export const GRAPHQL_MAX_PREVIEW_CHARS = GraphQLRuntimeLimit.MAX_PREVIEW_CHARS;
-export const GRAPHQL_MAX_SCHEMA_CHARS = GraphQLRuntimeLimit.MAX_SCHEMA_CHARS;
-export const GRAPHQL_MAX_QUERY_CHARS = GraphQLRuntimeLimit.MAX_QUERY_CHARS;
-export const GRAPHQL_MAX_GRAPH_NODES = GraphQLRuntimeLimit.MAX_GRAPH_NODES;
-export const GRAPHQL_MAX_GRAPH_EDGES = GraphQLRuntimeLimit.MAX_GRAPH_EDGES;
-export const INTROSPECTION_QUERY = `
-query IntrospectionQuery {
-  __schema {
-    queryType { name }
-    mutationType { name }
-    subscriptionType { name }
-    types { ...FullType }
-    directives {
-      name
-      description
-      locations
-      args(includeDeprecated: true) { ...InputValue }
-    }
-  }
-}
-fragment FullType on __Type {
-  kind
-  name
-  description
-  fields(includeDeprecated: true) {
-    name
-    description
-    args(includeDeprecated: true) { ...InputValue }
-    type { ...TypeRef }
-    isDeprecated
-    deprecationReason
-  }
-  inputFields(includeDeprecated: true) { ...InputValue }
-  interfaces { ...TypeRef }
-  enumValues(includeDeprecated: true) {
-    name
-    description
-    isDeprecated
-    deprecationReason
-  }
-  possibleTypes { ...TypeRef }
-}
-fragment InputValue on __InputValue {
-  name
-  description
-  type { ...TypeRef }
-  defaultValue
-  isDeprecated
-  deprecationReason
-}
-fragment TypeRef on __Type {
-  kind
-  name
-  ofType {
-    kind
-    name
-    ofType {
-      kind
-      name
-      ofType {
-        kind
-        name
-        ofType {
-          kind
-          name
-          ofType {
-            kind
-            name
-            ofType {
-              kind
-              name
-              ofType {
-                kind
-                name
-              }
-            }
-          }
-        }
-      }
-    }
-  }
-}
+export { GRAPHQL_MAX_PREVIEW_CHARS, GRAPHQL_MAX_SCHEMA_CHARS, GRAPHQL_MAX_QUERY_CHARS, GRAPHQL_MAX_GRAPH_NODES, GRAPHQL_MAX_GRAPH_EDGES, } from '../../../constants.js';
+export const INTROSPECTION_QUERY = `
+query IntrospectionQuery {
+  __schema {
+    queryType { name }
+    mutationType { name }
+    subscriptionType { name }
+    types { ...FullType }
+    directives {
+      name
+      description
+      locations
+      args(includeDeprecated: true) { ...InputValue }
+    }
+  }
+}
+fragment FullType on __Type {
+  kind
+  name
+  description
+  fields(includeDeprecated: true) {
+    name
+    description
+    args(includeDeprecated: true) { ...InputValue }
+    type { ...TypeRef }
+    isDeprecated
+    deprecationReason
+  }
+  inputFields(includeDeprecated: true) { ...InputValue }
+  interfaces { ...TypeRef }
+  enumValues(includeDeprecated: true) {
+    name
+    description
+    isDeprecated
+    deprecationReason
+  }
+  possibleTypes { ...TypeRef }
+}
+fragment InputValue on __InputValue {
+  name
+  description
+  type { ...TypeRef }
+  defaultValue
+  isDeprecated
+  deprecationReason
+}
+fragment TypeRef on __Type {
+  kind
+  name
+  ofType {
+    kind
+    name
+    ofType {
+      kind
+      name
+      ofType {
+        kind
+        name
+        ofType {
+          kind
+          name
+          ofType {
+            kind
+            name
+            ofType {
+              kind
+              name
+              ofType {
+                kind
+                name
+              }
+            }
+          }
+        }
+      }
+    }
+  }
+}
 `.trim();

package/dist/src/server/domains/graphql/manifest.js

@@ -16,15 +16,33 @@ function ensure(ctx) {
     return ctx.graphqlHandlers;
 }
 const manifest = {
-    kind: 'domain-manifest', version: 1,
-    domain: DOMAIN, depKey: DEP_KEY,
+    kind: 'domain-manifest',
+    version: 1,
+    domain: DOMAIN,
+    depKey: DEP_KEY,
     profiles: ['workflow', 'full'],
     ensure,
     registrations: [
-        { tool: t('call_graph_analyze'), domain: DOMAIN, bind: b((h, a) => h.handleCallGraphAnalyze(a)) },
-        { tool: t('script_replace_persist'), domain: DOMAIN, bind: b((h, a) => h.handleScriptReplacePersist(a)) },
-        { tool: t('graphql_introspect'), domain: DOMAIN, bind: b((h, a) => h.handleGraphqlIntrospect(a)) },
-        { tool: t('graphql_extract_queries'), domain: DOMAIN, bind: b((h, a) => h.handleGraphqlExtractQueries(a)) },
+        {
+            tool: t('call_graph_analyze'),
+            domain: DOMAIN,
+            bind: b((h, a) => h.handleCallGraphAnalyze(a)),
+        },
+        {
+            tool: t('script_replace_persist'),
+            domain: DOMAIN,
+            bind: b((h, a) => h.handleScriptReplacePersist(a)),
+        },
+        {
+            tool: t('graphql_introspect'),
+            domain: DOMAIN,
+            bind: b((h, a) => h.handleGraphqlIntrospect(a)),
+        },
+        {
+            tool: t('graphql_extract_queries'),
+            domain: DOMAIN,
+            bind: b((h, a) => h.handleGraphqlExtractQueries(a)),
+        },
         { tool: t('graphql_replay'), domain: DOMAIN, bind: b((h, a) => h.handleGraphqlReplay(a)) },
     ],
 };

package/dist/src/server/domains/hooks/ai-handlers.js

@@ -1,4 +1,5 @@
 import { AIHookGenerator } from '../../domains/shared/modules.js';
+import { evaluateWithTimeout, evaluateOnNewDocumentWithTimeout } from '../../../modules/collector/PageController.js';
 import { logger } from '../../../utils/logger.js';
 import { argString, argStringRequired, argBool } from '../../domains/shared/parse-args.js';
 export class AIHookToolHandlers {
@@ -79,11 +80,11 @@ export class AIHookToolHandlers {
             const method = argString(args, 'method', 'evaluate');
             const page = await this.pageController.getPage();
             if (method === 'evaluateOnNewDocument') {
-                await page.evaluateOnNewDocument(code);
+                await evaluateOnNewDocumentWithTimeout(page, code);
                 logger.info(`Hook injected (evaluateOnNewDocument): ${hookId}`);
             }
             else {
-                await page.evaluate(code);
+                await evaluateWithTimeout(page, code);
                 logger.info(`Hook injected (evaluate): ${hookId}`);
             }
             this.injectedHooks.set(hookId, {
@@ -123,7 +124,7 @@ export class AIHookToolHandlers {
         try {
             const hookId = argStringRequired(args, 'hookId');
             const page = await this.pageController.getPage();
-            const hookData = await page.evaluate((id) => {
+            const hookData = await evaluateWithTimeout(page, (id) => {
                 if (!window.__aiHooks || !window.__aiHooks[id]) {
                     return null;
                 }
@@ -177,7 +178,7 @@ export class AIHookToolHandlers {
     async handleAIHookList(_args) {
         try {
             const page = await this.pageController.getPage();
-            const allHooks = await page.evaluate(() => {
+            const allHooks = await evaluateWithTimeout(page, () => {
                 if (!window.__aiHookMetadata) {
                     return [];
                 }
@@ -220,7 +221,7 @@ export class AIHookToolHandlers {
             const hookId = argString(args, 'hookId');
             const page = await this.pageController.getPage();
             if (hookId) {
-                await page.evaluate((id) => {
+                await evaluateWithTimeout(page, (id) => {
                     if (window.__aiHooks && window.__aiHooks[id]) {
                         window.__aiHooks[id] = [];
                     }
@@ -238,7 +239,7 @@ export class AIHookToolHandlers {
                 };
             }
             else {
-                await page.evaluate(() => {
+                await evaluateWithTimeout(page, () => {
                     if (window.__aiHooks) {
                         for (const key in window.__aiHooks) {
                             window.__aiHooks[key] = [];
@@ -278,7 +279,7 @@ export class AIHookToolHandlers {
             const hookId = argStringRequired(args, 'hookId');
             const enabled = argBool(args, 'enabled');
             const page = await this.pageController.getPage();
-            await page.evaluate((id, enable) => {
+            await evaluateWithTimeout(page, (id, enable) => {
                 if (window.__aiHookMetadata && window.__aiHookMetadata[id]) {
                     window.__aiHookMetadata[id].enabled = enable;
                 }
@@ -317,7 +318,7 @@ export class AIHookToolHandlers {
             const hookId = argString(args, 'hookId');
             const format = argString(args, 'format', 'json');
             const page = await this.pageController.getPage();
-            const exportData = await page.evaluate((id) => {
+            const exportData = await evaluateWithTimeout(page, (id) => {
                 if (id) {
                     return {
                         hookId: id,

package/dist/src/server/domains/hooks/definitions.js

@@ -121,6 +121,12 @@ export const aiHookTools = [
             },
             required: ['description', 'target', 'behavior'],
         },
+        annotations: {
+            readOnlyHint: false,
+            destructiveHint: false,
+            idempotentHint: false,
+            openWorldHint: false,
+        },
     },
     {
         name: 'ai_hook_inject',
@@ -145,6 +151,12 @@ export const aiHookTools = [
             },
             required: ['hookId', 'code'],
         },
+        annotations: {
+            readOnlyHint: false,
+            destructiveHint: false,
+            idempotentHint: false,
+            openWorldHint: false,
+        },
     },
     {
         name: 'ai_hook_get_data',
@@ -159,6 +171,12 @@ export const aiHookTools = [
             },
             required: ['hookId'],
         },
+        annotations: {
+            readOnlyHint: false,
+            destructiveHint: false,
+            idempotentHint: false,
+            openWorldHint: false,
+        },
     },
     {
         name: 'ai_hook_list',
@@ -167,6 +185,12 @@ export const aiHookTools = [
             type: 'object',
             properties: {},
         },
+        annotations: {
+            readOnlyHint: false,
+            destructiveHint: false,
+            idempotentHint: false,
+            openWorldHint: false,
+        },
     },
     {
         name: 'ai_hook_clear',
@@ -180,6 +204,12 @@ export const aiHookTools = [
                 },
             },
         },
+        annotations: {
+            readOnlyHint: false,
+            destructiveHint: false,
+            idempotentHint: false,
+            openWorldHint: false,
+        },
     },
     {
         name: 'ai_hook_toggle',
@@ -198,6 +228,12 @@ export const aiHookTools = [
             },
             required: ['hookId', 'enabled'],
         },
+        annotations: {
+            readOnlyHint: false,
+            destructiveHint: false,
+            idempotentHint: false,
+            openWorldHint: false,
+        },
     },
     {
         name: 'ai_hook_export',
@@ -217,6 +253,12 @@ export const aiHookTools = [
                 },
             },
         },
+        annotations: {
+            readOnlyHint: false,
+            destructiveHint: false,
+            idempotentHint: false,
+            openWorldHint: false,
+        },
     },
 ];
 export const hookPresetTools = [
@@ -239,9 +281,18 @@ export const hookPresetTools = [
                     type: 'object',
                     description: 'Inline custom template. body should contain the hook body inserted into the standard buildHookCode wrapper. Use {{STACK_CODE}} and {{LOG_FN}} placeholders when needed.',
                     properties: {
-                        id: { type: 'string', description: 'Stable preset id, for example deobfuscation-sinks' },
-                        description: { type: 'string', description: 'Human-readable description for listPresets output.' },
-                        body: { type: 'string', description: 'Hook body snippet inserted into the preset wrapper.' },
+                        id: {
+                            type: 'string',
+                            description: 'Stable preset id, for example deobfuscation-sinks',
+                        },
+                        description: {
+                            type: 'string',
+                            description: 'Human-readable description for listPresets output.',
+                        },
+                        body: {
+                            type: 'string',
+                            description: 'Hook body snippet inserted into the preset wrapper.',
+                        },
                     },
                     required: ['id', 'body'],
                 },
@@ -282,5 +333,11 @@ export const hookPresetTools = [
             },
             required: [],
         },
+        annotations: {
+            readOnlyHint: false,
+            destructiveHint: false,
+            idempotentHint: false,
+            openWorldHint: false,
+        },
     },
 ];