@gokiteam/goki-dev 0.2.1 → 0.2.3

package/src/mcp/tools/firestore.js

@@ -5,6 +5,7 @@ export function registerFirestoreTools (server, apiClient) {
     'firestore_list_projects',
     'List configured Firestore emulator projects',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/firestore/projects/list')
@@ -21,6 +22,7 @@ export function registerFirestoreTools (server, apiClient) {
     {
       projectId: z.string().optional().describe('Firestore project ID')
     },
+    { readOnlyHint: true },
     async ({ projectId }) => {
       try {
         const data = await apiClient.post('/v1/firestore/collections/list', { projectId })
@@ -39,6 +41,7 @@ export function registerFirestoreTools (server, apiClient) {
       collectionPath: z.string().describe('Collection path'),
       page: z.object({ limit: z.number().optional(), offset: z.number().optional() }).optional().describe('Pagination options with limit and offset')
     },
+    { readOnlyHint: true },
     async ({ projectId, collectionPath, page }) => {
       try {
         const data = await apiClient.post('/v1/firestore/documents/list', { projectId, collectionPath, page })
@@ -57,6 +60,7 @@ export function registerFirestoreTools (server, apiClient) {
       collectionPath: z.string().describe('Collection path'),
       documentId: z.string().describe('Document ID')
     },
+    { readOnlyHint: true },
     async ({ projectId, collectionPath, documentId }) => {
       try {
         const data = await apiClient.post('/v1/firestore/documents/get', { projectId, collectionPath, documentId })
@@ -107,12 +111,13 @@ export function registerFirestoreTools (server, apiClient) {
 
   server.tool(
     'firestore_delete_document',
-    'Delete a Firestore document',
+    '[DESTRUCTIVE] Delete a Firestore document permanently. Ask the user for confirmation before calling this tool.',
     {
       projectId: z.string().optional().describe('Firestore project ID'),
       collectionPath: z.string().describe('Collection path'),
       documentId: z.string().describe('Document ID')
     },
+    { destructiveHint: true },
     async ({ projectId, collectionPath, documentId }) => {
       try {
         const data = await apiClient.post('/v1/firestore/documents/delete', { projectId, collectionPath, documentId })
@@ -123,6 +128,46 @@ export function registerFirestoreTools (server, apiClient) {
     }
   )
 
+  server.tool(
+    'firestore_set_document',
+    'Set (create or replace) a Firestore document by ID',
+    {
+      projectId: z.string().optional().describe('Firestore project ID'),
+      collectionPath: z.string().describe('Collection path'),
+      documentId: z.string().describe('Document ID'),
+      fields: z.record(z.any()).describe('Document fields in Firestore format')
+    },
+    async ({ projectId, collectionPath, documentId, fields }) => {
+      try {
+        const data = await apiClient.post('/v1/firestore/documents/set', { projectId, collectionPath, documentId, fields })
+        return { content: [{ type: 'text', text: JSON.stringify(data, null, 2) }] }
+      } catch (error) {
+        return { content: [{ type: 'text', text: `Error: ${error.message}` }], isError: true }
+      }
+    }
+  )
+
+  server.tool(
+    'firestore_create_batch',
+    'Create multiple Firestore documents in a collection at once',
+    {
+      projectId: z.string().optional().describe('Firestore project ID'),
+      collectionPath: z.string().describe('Collection path'),
+      documents: z.array(z.object({
+        documentId: z.string().optional().describe('Document ID (auto-generated if omitted)'),
+        fields: z.record(z.any()).describe('Document fields in Firestore format')
+      })).describe('Array of documents to create')
+    },
+    async ({ projectId, collectionPath, documents }) => {
+      try {
+        const data = await apiClient.post('/v1/firestore/documents/create-batch', { projectId, collectionPath, documents })
+        return { content: [{ type: 'text', text: JSON.stringify(data, null, 2) }] }
+      } catch (error) {
+        return { content: [{ type: 'text', text: `Error: ${error.message}` }], isError: true }
+      }
+    }
+  )
+
   server.tool(
     'firestore_query',
     'Execute a query against a Firestore collection with optional where/orderBy/limit',
@@ -133,6 +178,7 @@ export function registerFirestoreTools (server, apiClient) {
       orderBy: z.array(z.any()).optional().describe('Array of orderBy clauses'),
       limit: z.number().optional().describe('Maximum number of results')
     },
+    { readOnlyHint: true },
     async ({ projectId, collectionPath, where, orderBy, limit }) => {
       try {
         const data = await apiClient.post('/v1/firestore/query/execute', { projectId, collectionPath, where, orderBy, limit })
@@ -145,11 +191,12 @@ export function registerFirestoreTools (server, apiClient) {
 
   server.tool(
     'firestore_clear_collection',
-    'Delete all documents in a Firestore collection',
+    '[DESTRUCTIVE] Delete ALL documents in a Firestore collection. This cannot be undone. Ask the user for confirmation before calling this tool.',
     {
       projectId: z.string().optional().describe('Firestore project ID'),
       collectionPath: z.string().describe('Collection path')
     },
+    { destructiveHint: true },
     async ({ projectId, collectionPath }) => {
       try {
         const data = await apiClient.post('/v1/firestore/collection/clear', { projectId, collectionPath })

package/src/mcp/tools/functions.js

@@ -45,6 +45,7 @@ export function registerFunctionsTools (server, apiClient) {
       limit: z.number().optional().describe('Maximum number of results'),
       offset: z.number().optional().describe('Offset for pagination')
     },
+    { readOnlyHint: true },
     async ({ triggerType, status, limit, offset }) => {
       try {
         const body = {}
@@ -69,6 +70,7 @@ export function registerFunctionsTools (server, apiClient) {
       id: z.string().optional().describe('Function ID'),
       name: z.string().optional().describe('Function name')
     },
+    { readOnlyHint: true },
     async ({ id, name }) => {
       try {
         const body = {}
@@ -118,11 +120,12 @@ export function registerFunctionsTools (server, apiClient) {
 
   server.tool(
     'functions_delete',
-    'Delete a Cloud Function from the dev-tools emulator',
+    '[DESTRUCTIVE] Delete a Cloud Function from the dev-tools emulator. Ask the user for confirmation before calling this tool.',
     {
       id: z.string().optional().describe('Function ID'),
       name: z.string().optional().describe('Function name')
     },
+    { destructiveHint: true },
     async ({ id, name }) => {
       try {
         const body = {}
@@ -158,11 +161,12 @@ export function registerFunctionsTools (server, apiClient) {
 
   server.tool(
     'functions_stop',
-    'Stop a running Cloud Function',
+    '[DESTRUCTIVE] Stop a running Cloud Function. Ask the user for confirmation before calling this tool.',
     {
       id: z.string().optional().describe('Function ID'),
       name: z.string().optional().describe('Function name')
     },
+    { destructiveHint: true },
     async ({ id, name }) => {
       try {
         const body = {}
@@ -178,11 +182,12 @@ export function registerFunctionsTools (server, apiClient) {
 
   server.tool(
     'functions_restart',
-    'Restart a Cloud Function (stop then start)',
+    '[DESTRUCTIVE] Restart a Cloud Function (stop then start). Ask the user for confirmation before calling this tool.',
     {
       id: z.string().optional().describe('Function ID'),
       name: z.string().optional().describe('Function name')
     },
+    { destructiveHint: true },
     async ({ id, name }) => {
       try {
         const body = {}
@@ -228,6 +233,7 @@ export function registerFunctionsTools (server, apiClient) {
       limit: z.number().optional().describe('Maximum number of results (default: 50)'),
       offset: z.number().optional().describe('Offset for pagination')
     },
+    { readOnlyHint: true },
     async ({ functionId, functionName, triggerType, limit, offset }) => {
       try {
         const body = {}
@@ -248,6 +254,7 @@ export function registerFunctionsTools (server, apiClient) {
     'functions_stats',
     'Get aggregate statistics for Cloud Functions (counts by trigger type, status, invocations)',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/functions/stats')
@@ -264,6 +271,7 @@ export function registerFunctionsTools (server, apiClient) {
     {
       name: z.string().describe('Function name')
     },
+    { readOnlyHint: true },
     async ({ name }) => {
       try {
         const data = await apiClient.post('/v1/functions/dependencies/read', { name })
@@ -301,6 +309,7 @@ export function registerFunctionsTools (server, apiClient) {
       id: z.string().optional().describe('Function ID'),
       name: z.string().optional().describe('Function name')
     },
+    { readOnlyHint: true },
     async ({ id, name }) => {
       try {
         const body = {}
@@ -322,6 +331,7 @@ export function registerFunctionsTools (server, apiClient) {
       name: z.string().optional().describe('Function name'),
       filePath: z.string().describe('Relative file path within the function source directory')
     },
+    { readOnlyHint: true },
     async ({ id, name, filePath }) => {
       try {
         const body = { filePath }
@@ -359,12 +369,13 @@ export function registerFunctionsTools (server, apiClient) {
 
   server.tool(
     'functions_files_delete',
-    'Delete a file from a Cloud Function source directory',
+    '[DESTRUCTIVE] Delete a file from a Cloud Function source directory. Ask the user for confirmation before calling this tool.',
     {
       id: z.string().optional().describe('Function ID'),
       name: z.string().optional().describe('Function name'),
       filePath: z.string().describe('Relative file path within the function source directory')
     },
+    { destructiveHint: true },
     async ({ id, name, filePath }) => {
       try {
         const body = { filePath }

package/src/mcp/tools/httpTraffic.js

@@ -9,6 +9,7 @@ export function registerHttpTrafficTools (server, apiClient) {
       limit: z.number().optional().describe('Maximum number of records to return'),
       offset: z.number().optional().describe('Number of records to skip for pagination')
     },
+    { readOnlyHint: true },
     async ({ filter, limit, offset }) => {
       try {
         const body = {}
@@ -29,6 +30,7 @@ export function registerHttpTrafficTools (server, apiClient) {
     {
       id: z.string().describe('ID of the HTTP traffic entry to retrieve')
     },
+    { readOnlyHint: true },
     async ({ id }) => {
       try {
         const data = await apiClient.post('/v1/http-traffic/details', { id })
@@ -46,6 +48,7 @@ export function registerHttpTrafficTools (server, apiClient) {
       filter: z.record(z.any()).describe('Required filter criteria (method, targetHost, sourceService, statusCode, traceId, pathContains)'),
       timeout: z.number().optional().describe('Timeout in ms (default: 5000, max: 30000)')
     },
+    { readOnlyHint: true },
     async ({ filter, timeout }) => {
       try {
         const body = { filter }
@@ -60,8 +63,9 @@ export function registerHttpTrafficTools (server, apiClient) {
 
   server.tool(
     'http_traffic_clear',
-    'Clear all recorded HTTP traffic',
+    '[DESTRUCTIVE] Clear all recorded HTTP traffic. Ask the user for confirmation before calling this tool.',
     {},
+    { destructiveHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/http-traffic/clear')
@@ -76,6 +80,7 @@ export function registerHttpTrafficTools (server, apiClient) {
     'http_traffic_stats',
     'Get HTTP traffic statistics grouped by host, method, and error rates',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/http-traffic/stats')

package/src/mcp/tools/logging.js

@@ -5,6 +5,7 @@ export function registerLoggingTools (server, apiClient) {
     'logging_list_services',
     'List all services that have sent log entries to the dev-tools logging emulator',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/logging/services/list')
@@ -29,6 +30,7 @@ export function registerLoggingTools (server, apiClient) {
       limit: z.number().optional().describe('Max entries to return (default: 100, max: 1000)'),
       offset: z.number().optional().describe('Skip this many entries for pagination (default: 0)')
     },
+    { readOnlyHint: true },
     async ({ serviceName, severity, textContains, traceId, sinceTimestamp, untilTimestamp, compact, limit, offset }) => {
       try {
         const filter = {}
@@ -61,6 +63,7 @@ export function registerLoggingTools (server, apiClient) {
     {
       ids: z.array(z.string()).describe('Array of log entry IDs to retrieve (from the _id field in compact list results)')
     },
+    { readOnlyHint: true },
     async ({ ids }) => {
       try {
         const data = await apiClient.post('/v1/logging/entries/get', { ids })
@@ -73,8 +76,9 @@ export function registerLoggingTools (server, apiClient) {
 
   server.tool(
     'logging_clear',
-    'Clear all log entries from the dev-tools logging emulator',
+    '[DESTRUCTIVE] Clear all log entries from the dev-tools logging emulator. Ask the user for confirmation before calling this tool.',
     {},
+    { destructiveHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/logging/entries/clear')
@@ -91,6 +95,7 @@ export function registerLoggingTools (server, apiClient) {
     {
       filter: z.record(z.any()).optional().describe('Optional filter criteria to scope the export')
     },
+    { readOnlyHint: true },
     async ({ filter }) => {
       try {
         const body = {}
@@ -109,6 +114,7 @@ export function registerLoggingTools (server, apiClient) {
     {
       traceId: z.string().describe('The trace ID to look up')
     },
+    { readOnlyHint: true },
     async ({ traceId }) => {
       try {
         const data = await apiClient.post(`/v1/logging/trace/${traceId}`)
@@ -126,6 +132,7 @@ export function registerLoggingTools (server, apiClient) {
       filter: z.record(z.any()).describe('Filter criteria the log entry must match'),
       timeout: z.number().optional().describe('Maximum seconds to wait before timing out')
     },
+    { readOnlyHint: true },
     async ({ filter, timeout }) => {
       try {
         const body = { filter }
@@ -145,6 +152,7 @@ export function registerLoggingTools (server, apiClient) {
       traceId: z.string().optional().describe('Optional trace ID to scope the assertion to'),
       sinceTimestamp: z.string().optional().describe('Optional ISO timestamp — only check logs after this time')
     },
+    { readOnlyHint: true },
     async ({ traceId, sinceTimestamp }) => {
       try {
         const body = {}
@@ -162,6 +170,7 @@ export function registerLoggingTools (server, apiClient) {
     'logging_get_queue_stats',
     'Get Bull queue statistics including job counts and health status for the blackbox logs consumer',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/logging/queue-stats', {})

package/src/mcp/tools/mqtt.js

@@ -5,6 +5,7 @@ export function registerMqttTools (server, apiClient) {
     'mqtt_list_clients',
     'List all connected MQTT clients with their connection info',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/mqtt/clients/list')
@@ -22,6 +23,7 @@ export function registerMqttTools (server, apiClient) {
       filter: z.record(z.any()).optional().describe('Optional filter criteria for MQTT messages'),
       page: z.object({ limit: z.number().optional(), offset: z.number().optional() }).optional().describe('Optional pagination with limit and offset')
     },
+    { readOnlyHint: true },
     async ({ filter, page }) => {
       try {
         const body = {}

package/src/mcp/tools/postgres.js

@@ -5,6 +5,7 @@ export function registerPostgresTools (server, apiClient) {
     'postgres_test_connection',
     'Test the PostgreSQL database connection',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/postgres/connection/test')
@@ -19,6 +20,7 @@ export function registerPostgresTools (server, apiClient) {
     'postgres_list_databases',
     'List all PostgreSQL databases',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/postgres/databases/list')
@@ -33,6 +35,7 @@ export function registerPostgresTools (server, apiClient) {
     'postgres_list_schemas',
     'List all schemas in the current database',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/postgres/schemas/list')
@@ -49,6 +52,7 @@ export function registerPostgresTools (server, apiClient) {
     {
       schema: z.string().optional().describe('Schema name (defaults to public)')
     },
+    { readOnlyHint: true },
     async ({ schema }) => {
       try {
         const data = await apiClient.post('/v1/postgres/tables/list', { schema })
@@ -66,6 +70,7 @@ export function registerPostgresTools (server, apiClient) {
       table: z.string().describe('Table name'),
       schema: z.string().optional().describe('Schema name (defaults to public)')
     },
+    { readOnlyHint: true },
     async ({ table, schema }) => {
       try {
         const data = await apiClient.post('/v1/postgres/columns/list', { table, schema })
@@ -83,6 +88,7 @@ export function registerPostgresTools (server, apiClient) {
       table: z.string().describe('Table name'),
       page: z.object({ limit: z.number().optional(), offset: z.number().optional() }).optional().describe('Pagination options with limit and offset')
     },
+    { readOnlyHint: true },
     async ({ table, page }) => {
       try {
         const data = await apiClient.post('/v1/postgres/rows/list', { table, page })
@@ -95,11 +101,12 @@ export function registerPostgresTools (server, apiClient) {
 
   server.tool(
     'postgres_execute_query',
-    'Execute a raw SQL query against PostgreSQL',
+    '[DESTRUCTIVE] Execute a raw SQL query against PostgreSQL. Ask the user for confirmation before calling this tool.',
     {
       query: z.string().describe('SQL query to execute'),
       params: z.array(z.any()).optional().describe('Query parameters for parameterized queries')
     },
+    { destructiveHint: true },
     async ({ query, params }) => {
       try {
         const data = await apiClient.post('/v1/postgres/query/execute', { query, params })
@@ -118,6 +125,7 @@ export function registerPostgresTools (server, apiClient) {
       condition: z.record(z.any()).optional().describe('Condition to match against query results'),
       timeout: z.number().optional().describe('Timeout in milliseconds')
     },
+    { readOnlyHint: true },
     async ({ query, condition, timeout }) => {
       try {
         const data = await apiClient.post('/v1/postgres/rows/wait-for', { query, condition, timeout })

package/src/mcp/tools/pubsub.js

@@ -5,6 +5,7 @@ export function registerPubsubTools (server, apiClient) {
     'pubsub_list_topics',
     'List all Pub/Sub topics in the dev-tools emulator',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/pubsub/topics/list')
@@ -33,10 +34,11 @@ export function registerPubsubTools (server, apiClient) {
 
   server.tool(
     'pubsub_delete_topic',
-    'Delete a Pub/Sub topic from the dev-tools emulator',
+    '[DESTRUCTIVE] Delete a Pub/Sub topic from the dev-tools emulator. Ask the user for confirmation before calling this tool.',
     {
       topicName: z.string().describe('Name of the topic to delete')
     },
+    { destructiveHint: true },
     async ({ topicName }) => {
       try {
         const data = await apiClient.post('/v1/pubsub/topics/delete', { topicName })
@@ -54,6 +56,7 @@ export function registerPubsubTools (server, apiClient) {
       query: z.string().describe('Search query to match against topic names'),
       limit: z.number().optional().describe('Maximum number of suggestions to return')
     },
+    { readOnlyHint: true },
     async ({ query, limit }) => {
       try {
         const body = { query }
@@ -70,6 +73,7 @@ export function registerPubsubTools (server, apiClient) {
     'pubsub_list_subscriptions',
     'List all Pub/Sub subscriptions in the dev-tools emulator',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/pubsub/subscriptions/list')
@@ -99,10 +103,11 @@ export function registerPubsubTools (server, apiClient) {
 
   server.tool(
     'pubsub_delete_subscription',
-    'Delete a Pub/Sub subscription from the dev-tools emulator',
+    '[DESTRUCTIVE] Delete a Pub/Sub subscription from the dev-tools emulator. Ask the user for confirmation before calling this tool.',
     {
       subscriptionName: z.string().describe('Name of the subscription to delete')
     },
+    { destructiveHint: true },
     async ({ subscriptionName }) => {
       try {
         const data = await apiClient.post('/v1/pubsub/subscriptions/delete', { subscriptionName })
@@ -140,6 +145,7 @@ export function registerPubsubTools (server, apiClient) {
       subscriptionName: z.string().describe('Subscription to pull messages from'),
       maxMessages: z.number().optional().describe('Maximum number of messages to pull')
     },
+    { readOnlyHint: true },
     async ({ subscriptionName, maxMessages }) => {
       try {
         const body = { subscriptionName }
@@ -160,6 +166,7 @@ export function registerPubsubTools (server, apiClient) {
       searchIn: z.array(z.string()).optional().describe('Optional list of fields to search in (e.g. ["data", "attributes", "topic"])'),
       filter: z.record(z.any()).optional().describe('Optional additional filter criteria')
     },
+    { readOnlyHint: true },
     async ({ query, searchIn, filter }) => {
       try {
         const body = { query }
@@ -180,6 +187,7 @@ export function registerPubsubTools (server, apiClient) {
       filter: z.record(z.any()).optional().describe('Optional filter criteria (e.g. by topic, time range)'),
       page: z.object({ limit: z.number().optional(), offset: z.number().optional() }).optional().describe('Optional pagination with limit and offset')
     },
+    { readOnlyHint: true },
     async ({ filter, page }) => {
       try {
         const body = {}
@@ -195,11 +203,12 @@ export function registerPubsubTools (server, apiClient) {
 
   server.tool(
     'pubsub_clear_history',
-    'Clear Pub/Sub message history. Optionally keep a number of recent messages or clear only a specific topic.',
+    '[DESTRUCTIVE] Clear Pub/Sub message history. Optionally keep a number of recent messages or clear only a specific topic. Ask the user for confirmation before calling this tool.',
     {
       keepCount: z.number().optional().describe('Number of recent messages to keep'),
       topic: z.string().optional().describe('Only clear history for this topic')
     },
+    { destructiveHint: true },
     async ({ keepCount, topic }) => {
       try {
         const body = {}
@@ -217,6 +226,7 @@ export function registerPubsubTools (server, apiClient) {
     'pubsub_history_stats',
     'Get statistics about Pub/Sub message history including counts per topic and total volume',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/pubsub/history/stats')
@@ -234,6 +244,7 @@ export function registerPubsubTools (server, apiClient) {
       filter: z.record(z.any()).describe('Filter criteria with jsonPath and/or predicate to match incoming messages'),
       timeout: z.number().optional().describe('Maximum seconds to wait before timing out')
     },
+    { readOnlyHint: true },
     async ({ filter, timeout }) => {
       try {
         const body = { filter }
@@ -252,6 +263,7 @@ export function registerPubsubTools (server, apiClient) {
     {
       filter: z.record(z.any()).describe('Filter criteria: { topic (string), since (ISO timestamp), dataContains (string), attributesMatch (object) }')
     },
+    { readOnlyHint: true },
     async ({ filter }) => {
       try {
         const data = await apiClient.post('/v1/pubsub/messages/assert-published', { filter })
@@ -285,6 +297,7 @@ export function registerPubsubTools (server, apiClient) {
     {
       projectName: z.string().optional().describe('Optional project name to filter by')
     },
+    { readOnlyHint: true },
     async ({ projectName }) => {
       try {
         const body = {}
@@ -299,11 +312,12 @@ export function registerPubsubTools (server, apiClient) {
 
   server.tool(
     'pubsub_registry_unregister',
-    'Unregister topics for a project from the persistent Pub/Sub registry',
+    '[DESTRUCTIVE] Unregister topics for a project from the persistent Pub/Sub registry. Ask the user for confirmation before calling this tool.',
     {
       projectName: z.string().describe('Project name to unregister topics from'),
       topicNames: z.array(z.string()).describe('Array of topic name strings to unregister')
     },
+    { destructiveHint: true },
     async ({ projectName, topicNames }) => {
       try {
         const data = await apiClient.post('/v1/pubsub/registry/unregister', { projectName, topicNames })

package/src/mcp/tools/redis.js

@@ -5,6 +5,7 @@ export function registerRedisTools (server, apiClient) {
     'redis_test_connection',
     'Test the Redis connection',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/redis/connection/test')
@@ -19,6 +20,7 @@ export function registerRedisTools (server, apiClient) {
     'redis_info',
     'Get Redis server info including memory usage and db size',
     {},
+    { readOnlyHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/redis/info')
@@ -37,6 +39,7 @@ export function registerRedisTools (server, apiClient) {
       cursor: z.string().optional().describe('Cursor for pagination'),
       count: z.number().optional().describe('Approximate number of keys to return per scan')
     },
+    { readOnlyHint: true },
     async ({ pattern, cursor, count }) => {
       try {
         const data = await apiClient.post('/v1/redis/keys/scan', { pattern, cursor, count })
@@ -53,6 +56,7 @@ export function registerRedisTools (server, apiClient) {
     {
       key: z.string().describe('Redis key name')
     },
+    { readOnlyHint: true },
     async ({ key }) => {
       try {
         const data = await apiClient.post('/v1/redis/keys/get', { key })
@@ -69,6 +73,7 @@ export function registerRedisTools (server, apiClient) {
     {
       key: z.string().describe('Redis key name')
     },
+    { readOnlyHint: true },
     async ({ key }) => {
       try {
         const data = await apiClient.post('/v1/redis/keys/ttl', { key })
@@ -81,10 +86,11 @@ export function registerRedisTools (server, apiClient) {
 
   server.tool(
     'redis_delete_key',
-    'Delete a specific Redis key',
+    '[DESTRUCTIVE] Delete a specific Redis key. Ask the user for confirmation before calling this tool.',
     {
       key: z.string().describe('Redis key name')
     },
+    { destructiveHint: true },
     async ({ key }) => {
       try {
         const data = await apiClient.post('/v1/redis/keys/delete', { key })
@@ -115,8 +121,9 @@ export function registerRedisTools (server, apiClient) {
 
   server.tool(
     'redis_delete_all',
-    'Delete all Redis keys (FLUSHDB)',
+    '[DESTRUCTIVE] Delete all Redis keys (FLUSHDB). Ask the user for confirmation before calling this tool.',
     {},
+    { destructiveHint: true },
     async () => {
       try {
         const data = await apiClient.post('/v1/redis/keys/delete-all')
@@ -134,6 +141,7 @@ export function registerRedisTools (server, apiClient) {
       condition: z.record(z.any()).describe('Condition to match against the key value'),
       timeout: z.number().optional().describe('Timeout in milliseconds')
     },
+    { readOnlyHint: true },
     async ({ condition, timeout }) => {
       try {
         const data = await apiClient.post('/v1/redis/keys/wait-for', { condition, timeout })