@yawlabs/tailscale-mcp 0.8.6 → 0.8.8

package/README.md

@@ -3,9 +3,9 @@
 [![npm version](https://img.shields.io/npm/v/@yawlabs/tailscale-mcp)](https://www.npmjs.com/package/@yawlabs/tailscale-mcp)
 [![License: MIT](https://img.shields.io/badge/License-MIT-blue.svg)](https://opensource.org/licenses/MIT)
 [![GitHub stars](https://img.shields.io/github/stars/YawLabs/tailscale-mcp)](https://github.com/YawLabs/tailscale-mcp/stargazers)
-[![CI](https://github.com/YawLabs/tailscale-mcp/actions/workflows/ci.yml/badge.svg)](https://github.com/YawLabs/tailscale-mcp/actions/workflows/ci.yml) [![Release](https://github.com/YawLabs/tailscale-mcp/actions/workflows/release.yml/badge.svg)](https://github.com/YawLabs/tailscale-mcp/actions/workflows/release.yml) [![Integration](https://github.com/YawLabs/tailscale-mcp/actions/workflows/integration.yml/badge.svg)](https://github.com/YawLabs/tailscale-mcp/actions/workflows/integration.yml)
+[![CI](https://github.com/YawLabs/tailscale-mcp/actions/workflows/ci.yml/badge.svg)](https://github.com/YawLabs/tailscale-mcp/actions/workflows/ci.yml) [![Release](https://github.com/YawLabs/tailscale-mcp/actions/workflows/release.yml/badge.svg)](https://github.com/YawLabs/tailscale-mcp/actions/workflows/release.yml)
 
-**Ask your agent questions about your tailnet and have it act on the answers.** 99 tools + 4 resources covering the full [Tailscale v2 API](https://tailscale.com/api). Backed by 735 tests and a nightly integration run against a real tailnet.
+**Ask your agent questions about your tailnet and have it act on the answers.** 99 tools + 4 resources covering the full [Tailscale v2 API](https://tailscale.com/api). Backed by 736 unit tests and an opt-in live-tailnet integration suite.
 
 Built and maintained by [Yaw Labs](https://yaw.sh).
 
@@ -33,9 +33,9 @@ Reasonable question. Both have their place. Where this MCP is better:
 
 - **Full admin API coverage.** The `tailscale` CLI is scoped to the node it runs on. Admin concerns — ACLs, users, invites, webhooks, log streaming, workload identity, OAuth clients, posture — live in the v2 HTTP API. You'd be shelling out to `curl` anyway.
 - **Typed tool surface, not string parsing.** Every tool has a Zod-validated input schema and a structured response. No brittle `tailscale status --json | jq` pipelines that break when the schema evolves.
-- **Cross-client, no user rewriting.** A Claude Code skill is tied to Claude Code. An MCP server works in Claude Code, Claude Desktop, Cursor, Windsurf, VS Code, and anything else that speaks MCP. Version bumps ship through `npx` — users don't re-author their skill when Tailscale adds an endpoint.
+- **Cross-client, no user rewriting.** A Claude Code skill only loads in Claude Code. An MCP server works in Claude Code, Claude Desktop, Cursor, Windsurf, VS Code, and anything else that speaks MCP. Version bumps ship through `npx` — users don't re-author their skill when Tailscale adds an endpoint.
 - **Safe-by-default writes.** Every tool declares `readOnlyHint` / `destructiveHint` / `idempotentHint` so clients can skip confirmation on reads and require it on mutations. A skill that shells out to the CLI can't express that.
-- **Real tests.** 735 unit tests + an integration suite hitting a live tailnet on every tag. Most skills are short markdown prompts without their own test layer — if the vendor changes output format, nothing catches it for you.
+- **Real tests.** 736 unit tests covering every tool's input validation, API shape, and error handling. Plus an opt-in live-tailnet integration suite (`RUN_INTEGRATION_TESTS=1` + a tailnet API key) for shape-drift detection. Most skills are short markdown prompts without their own test layer — if the vendor changes output format, nothing catches it for you.
 
 If you already have a skill that covers your 10% of Tailscale workflows, great — keep it. The MCP is for the other 90%.
 
@@ -43,10 +43,10 @@ If you already have a skill that covers your 10% of Tailscale workflows, great
 
 Fair critique from Reddit: a new repo claiming "actively maintained" with no visible tests is worth exactly zero trust. Here's what's actually verifiable:
 
-- **735 tests** (179 suites, `node --test`) covering every tool's input validation, API shape, and error handling. Run `npm test` to see them pass locally.
+- **736 tests** (179 suites, `node --test`) covering every tool's input validation, API shape, and error handling. Run `npm test` to see them pass locally.
 - **3 CI workflows** on GitHub Actions:
   - [`ci.yml`](.github/workflows/ci.yml) — lint + typecheck + build + unit tests on every push and PR.
-  - [`integration.yml`](.github/workflows/integration.yml) — runs the full tool surface against a real tailnet.
+  - [`integration.yml`](.github/workflows/integration.yml) — read-only live-API smoke tests against a real tailnet. Wired up with three triggers (nightly schedule, every tag push via `release.yml`, manual dispatch); skips gracefully when no test-tailnet secret is configured, so forks aren't blocked.
   - [`release.yml`](.github/workflows/release.yml) — publishes to npm from a signed tag.
 - **Dependabot alerts** surface on this repo and get fixed, not ignored.
 - **Every tool verified against the live API.** If it's in the tool list, it calls a real endpoint that exists in the current v2 API. No placeholder 404 tools.
@@ -91,7 +91,7 @@ Windows:
 }
 ```
 
-> **Why the extra step on Windows?** Since Node 20, `child_process.spawn` cannot directly execute `.cmd` files (that's what `npx` is on Windows). Wrapping with `cmd /c` is the standard workaround.
+> **Why the extra step on Windows?** On Windows, `npx` is a `.cmd` file, and Node 20+ refuses to spawn `.cmd` files directly. Wrapping with `cmd /c` is the standard workaround.
 
 **3. Restart and approve**
 
@@ -158,7 +158,7 @@ Set to `1` or `true` to drop every tool without `readOnlyHint: true`. Stacks wit
 The server logs the active filter to stderr on startup:
 
 ```
-@yawlabs/tailscale-mcp v0.8.x ready (19 tools, profile=core, readonly)
+@yawlabs/tailscale-mcp v0.8.8 ready (19 tools, profile=core, readonly)
 ```
 
 If you don't set any filter, startup prints a tip pointing you at the profiles.
@@ -453,7 +453,7 @@ This shows a read-only banner in the Tailscale Admin Console pointing to your re
 
 ## Contributing
 
-Contributions welcome. Please [open an issue](https://github.com/YawLabs/tailscale-mcp/issues) to discuss before a PR for anything beyond a typo fix.
+Contributions welcome. See [CONTRIBUTING.md](CONTRIBUTING.md) for the PR workflow and AI-agent guidelines. Please [open an issue](https://github.com/YawLabs/tailscale-mcp/issues) to discuss before a PR for anything beyond a typo fix.
 
 ```bash
 git clone https://github.com/YawLabs/tailscale-mcp.git
@@ -462,11 +462,15 @@ npm install
 npm run lint       # Biome check
 npm run lint:fix   # Auto-fix
 npm run build      # tsc + esbuild bundle
-npm test           # node --test (735 tests)
+npm test           # node --test (736 tests)
 ```
 
 For integration testing against your own tailnet: set `TAILSCALE_API_KEY` and run `node dist/index.js`.
 
+## Security
+
+Found a vulnerability? See [SECURITY.md](SECURITY.md) — please use GitHub's private vulnerability reporting, not a public issue.
+
 ## License
 
 MIT

package/dist/index.js

@@ -31376,7 +31376,7 @@ var keyTools = [
   },
   {
     name: "tailscale_update_key",
-    description: "Update an existing key's description or configuration. For OAuth clients and federated identities, scopes, tags, and identity fields can also be updated. Auth keys only support description updates.",
+    description: "Update an existing key. Supported fields depend on the key type: all key types accept 'description'; OAuth clients and federated identities additionally accept 'scopes' and 'tags'; federated identities additionally accept 'issuer', 'subject', 'audience', and 'customClaimRules'. For auth keys, pass only 'description' \u2014 the Tailscale API will reject other fields.",
     annotations: {
       title: "Update key",
       readOnlyHint: false,
@@ -32061,16 +32061,24 @@ var tailnetTools = [
       security: external_exports3.object({ email: external_exports3.string() }).optional().describe("Security contact email")
     }),
     handler: async (input) => {
-      const results = {};
+      const applied = {};
+      const failed = {};
       for (const contactType of ["account", "support", "security"]) {
         const value = input[contactType];
-        if (value !== void 0) {
-          const res = await apiPatch(`/tailnet/${getTailnet()}/contacts/${encPath(contactType)}`, value);
-          if (!res.ok) return res;
-          results[contactType] = res.data;
-        }
+        if (value === void 0) continue;
+        const res = await apiPatch(`/tailnet/${getTailnet()}/contacts/${encPath(contactType)}`, value);
+        if (res.ok) applied[contactType] = res.data;
+        else failed[contactType] = res.error ?? `HTTP ${res.status}`;
+      }
+      const hasFailed = Object.keys(failed).length > 0;
+      const hasApplied = Object.keys(applied).length > 0;
+      if (hasFailed && !hasApplied) {
+        return { ok: false, status: 500, error: `Contact update failed: ${JSON.stringify(failed)}` };
+      }
+      if (hasFailed) {
+        return { ok: true, status: 207, data: { applied, failed } };
       }
-      return { ok: true, status: 200, data: results };
+      return { ok: true, status: 200, data: applied };
     }
   },
   {
@@ -32393,7 +32401,7 @@ var workloadIdentityTools = [
   },
   {
     name: "tailscale_get_workload_identity",
-    description: "Get details for a specific workload identity provider.",
+    description: "Get details for a specific federated workload identity provider, including issuer URL, audience, and the subject patterns it accepts for OIDC token exchange.",
     annotations: {
       title: "Get workload identity",
       readOnlyHint: true,
@@ -32479,7 +32487,7 @@ var workloadIdentityTools = [
 ];
 
 // src/index.ts
-var version2 = true ? "0.8.6" : (await null).createRequire(import.meta.url)("../package.json").version;
+var version2 = true ? "0.8.8" : (await null).createRequire(import.meta.url)("../package.json").version;
 var subcommand = process.argv[2];
 if (subcommand === "deploy-acl") {
   const filePath = process.argv[3];
@@ -32584,9 +32592,13 @@ server.resource(
     ]);
     const data = {
       tailnet: getTailnet(),
-      deviceCount: devicesRes.ok ? devicesRes.data?.devices?.length ?? 0 : "error",
-      settings: settingsRes.ok ? settingsRes.data : void 0
+      deviceCount: devicesRes.ok ? devicesRes.data?.devices?.length ?? 0 : null,
+      settings: settingsRes.ok ? settingsRes.data : null
     };
+    const errors = {};
+    if (!devicesRes.ok) errors.devices = devicesRes.error ?? `HTTP ${devicesRes.status}`;
+    if (!settingsRes.ok) errors.settings = settingsRes.error ?? `HTTP ${settingsRes.status}`;
+    if (Object.keys(errors).length > 0) data.errors = errors;
     return { contents: [{ uri: uri.href, text: JSON.stringify(data, null, 2), mimeType: "application/json" }] };
   }
 );
@@ -32596,7 +32608,7 @@ server.resource(
   { description: "List of all devices in the tailnet with their status", mimeType: "application/json" },
   async (uri) => {
     const res = await apiGet(`/tailnet/${getTailnet()}/devices`);
-    const text = res.ok ? JSON.stringify(res.data, null, 2) : JSON.stringify({ error: res.error });
+    const text = res.ok ? JSON.stringify(res.data, null, 2) : JSON.stringify({ error: res.error ?? `HTTP ${res.status}` }, null, 2);
     return { contents: [{ uri: uri.href, text, mimeType: "application/json" }] };
   }
 );
@@ -32606,7 +32618,8 @@ server.resource(
   { description: "Current ACL policy (HuJSON with comments preserved)", mimeType: "application/hujson" },
   async (uri) => {
     const res = await apiGet(`/tailnet/${getTailnet()}/acl`, { acceptRaw: true, accept: "application/hujson" });
-    const text = res.ok ? res.rawBody ?? "" : `Error: ${res.error}`;
+    const text = res.ok ? res.rawBody ?? "" : `// Error: ${res.error ?? `HTTP ${res.status}`}
+`;
     return { contents: [{ uri: uri.href, text, mimeType: "application/hujson" }] };
   }
 );
@@ -32625,11 +32638,17 @@ server.resource(
       apiGet(`/tailnet/${getTailnet()}/dns/preferences`)
     ]);
     const data = {
-      nameservers: nameservers.ok ? nameservers.data : void 0,
-      searchPaths: searchPaths.ok ? searchPaths.data : void 0,
-      splitDns: splitDns.ok ? splitDns.data : void 0,
-      preferences: preferences.ok ? preferences.data : void 0
+      nameservers: nameservers.ok ? nameservers.data : null,
+      searchPaths: searchPaths.ok ? searchPaths.data : null,
+      splitDns: splitDns.ok ? splitDns.data : null,
+      preferences: preferences.ok ? preferences.data : null
     };
+    const errors = {};
+    if (!nameservers.ok) errors.nameservers = nameservers.error ?? `HTTP ${nameservers.status}`;
+    if (!searchPaths.ok) errors.searchPaths = searchPaths.error ?? `HTTP ${searchPaths.status}`;
+    if (!splitDns.ok) errors.splitDns = splitDns.error ?? `HTTP ${splitDns.status}`;
+    if (!preferences.ok) errors.preferences = preferences.error ?? `HTTP ${preferences.status}`;
+    if (Object.keys(errors).length > 0) data.errors = errors;
     return { contents: [{ uri: uri.href, text: JSON.stringify(data, null, 2), mimeType: "application/json" }] };
   }
 );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yawlabs/tailscale-mcp",
-  "version": "0.8.6",
+  "version": "0.8.8",
   "description": "Tailscale MCP server for managing your tailnet from AI assistants",
   "license": "MIT",
   "author": "YawLabs <contact@yaw.sh>",