@thinremote/thinr-cli 1.3.4 → 1.4.0

package/lib/mcp/registry.js

@@ -1,46 +1,12 @@
 // @ts-check
-import { tools as devicesTools } from './tools-devices.js';
-import { tools as filesystemTools } from './tools-filesystem.js';
 import { tools as execTools } from './tools-exec.js';
-import { tools as resourcesTools } from './tools-resources.js';
-import { tools as propertiesTools } from './tools-properties.js';
-import { tools as scriptsTools } from './tools-scripts.js';
-import { tools as productsTools } from './tools-products.js';
-import { tools as playbookTools } from './tools-playbook.js';
-import { tools as profilesTools } from './tools-profiles.js';
-import { tools as installTools } from './tools-install.js';
-import { tools as bucketsTools } from './tools-buckets.js';
-import { tools as logsTools } from './tools-logs.js';
-import { tools as productProfileTools } from './tools-profile.js';
-import { tools as tokensTools } from './tools-tokens.js';
-import { tools as alarmsTools } from './tools-alarms.js';
 
-export const tools = [
-    ...devicesTools,
-    ...filesystemTools,
-    ...execTools,
-    ...resourcesTools,
-    ...propertiesTools,
-    ...scriptsTools,
-    ...productsTools,
-    ...playbookTools,
-    ...profilesTools,
-    ...installTools,
-    ...bucketsTools,
-    ...logsTools,
-    ...productProfileTools,
-    ...tokensTools,
-    ...alarmsTools,
-];
-
-// Inject the optional `profile` parameter on every tool so callers can
-// target any configured environment per-call without changing the saved
-// default. `thinr_profiles` is the lookup itself, so it's exempt.
-for (const t of tools) {
-    if (t.name === 'thinr_profiles') continue;
-    t.inputSchema.properties = t.inputSchema.properties || {};
-    t.inputSchema.properties.profile = {
-        type: 'string',
-        description: 'CLI profile to use for this call (see thinr_profiles). Omit for the default.',
-    };
-}
+/**
+ * ThinRemote's own MCP tools: only `device_exec`, the command on one device
+ * or a fleet. Everything else ThinRemote adds is served by the generic
+ * `resource_*` tools of @thinger-io/cli-core: its resource types (a
+ * device's files and scripts; a product's scripts, metrics, log sources
+ * and playbooks) and its actions on devices, in resource-types.js, and
+ * what it adds to the server's types, in resource-hooks.js.
+ */
+export const tools = [...execTools];

package/lib/mcp/resource-hooks.js

@@ -0,0 +1,53 @@
+// @ts-check
+import { deleteStorage } from '@thinger-io/cli-core/storage';
+import { toolName } from '@thinger-io/cli-core/mcp/naming';
+import { getAPI } from './helpers.js';
+
+/** Where the rest of what an agent offers is, keyed by what it is. */
+const agentMap = (device) => ({
+    monitoring: `live: ${toolName('device_resource_call')} resource "monitoring"; history: ${toolName('bucket_data_read')} bucket "monitoring" device "${device}" (fields cpu.usage, memory.usage, disk.root.usage, network.rx_rate, load.1m…), kept only when the device's product has a monitoring bucket (ThinRemote products do): empty otherwise`,
+    files: `${toolName('resource_list')} / _get / _create / _update / _delete type "device_fs" parent "${device}"`,
+    scripts: `type "device_script" parent "${device}"`,
+    commands: `${toolName('device_exec')} device "${device}"`,
+    agent_update: `${toolName('resource_call')} type "device" action "agent_update" id "${device}"`,
+});
+
+/**
+ * What ThinRemote adds to the generic `resource_*` tools of cli-core, which
+ * run these after their own hooks:
+ *
+ * - Reading a device also asks its agent, when it is connected, for the
+ *   system it runs on (hostname, OS, architecture…) and the base path of
+ *   its filesystem: what the server record cannot know. Best effort: a slow
+ *   or old agent leaves the record as it is. It also says where the rest of
+ *   what the agent offers is (monitoring, files, scripts, commands), so a
+ *   model that starts from the device finds it.
+ * - Deleting a product also deletes its file storage, which holds the
+ *   product's scripts and only ThinRemote products have.
+ *
+ * @type {Record<string, import('@thinger-io/cli-core/mcp/resource-hooks').ResourceHooks>}
+ */
+export const resourceHooks = {
+    device: {
+        async afterGet(record, ctx) {
+            if (!record?.connection?.active) return record;
+            const { api } = getAPI(ctx.id, ctx.user);
+            const agent = {};
+            try {
+                agent.system_info = await api.getResource('system');
+            } catch {}
+            try {
+                agent.fs_base_path = (await api.info('/')).path || '/';
+            } catch {}
+            if (!Object.keys(agent).length) return record;
+            return { ...record, agent: { ...agent, where: agentMap(ctx.id) } };
+        },
+    },
+    product: {
+        async afterDelete(ctx) {
+            if (await deleteStorage(ctx.id, ctx.user)) {
+                return `Also deleted its file storage "${ctx.id}" (product scripts).`;
+            }
+        },
+    },
+};