@skyelight/mcp 0.2.0 → 0.3.0

package/README.md

@@ -4,7 +4,7 @@ An MCP server that lets a coding agent read the feedback people left on your
 running app — the thread, the page, and the element they were pointing at —
 and report back on it when the work is done.
 
-Works with anything that speaks MCP over stdio: Claude Code, Cursor, Codex.
+Works with anything that speaks MCP over stdio: Claude Code, Cursor, Grok, Codex.
 
 ## Setup
 
@@ -22,8 +22,8 @@ Finds your coding agent, registers the remote server with it, and leaves the
 sign-in to OAuth — nothing is pasted and no key is stored on disk. It shows
 what it will change and waits; `--yes` skips that once you have read it.
 
-It knows Claude Code, Cursor, Windsurf and Codex. Name one with `--client
-cursor` when more than one is installed, and point at another deployment with
+It knows Claude Code, Cursor, Grok, Codex and Windsurf. Name one with
+`--client cursor` when more than one is installed, and point at another deployment with
 `SKYELIGHT_URL=https://…`. The deployment tells the installer its own MCP URL
 and OAuth client over `/mcp/install-config`, so this package holds no
 environment-specific constants.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyelight/mcp",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "MCP server for Skyelight \u2014 read feedback items from your project as an agent",
   "type": "module",
   "bin": {

package/src/install.js

@@ -30,7 +30,8 @@ const home = () => homedir();
  *
  * Ordered by how likely a given machine is to have one, because the first
  * match wins when nothing is named. That is a guess, and `--client` exists
- * for when it guesses wrong.
+ * for when it guesses wrong. The same order the settings card lists them in,
+ * so the two never disagree about which agent is the obvious one.
  */
 export const CLIENTS = [
   {
@@ -67,12 +68,40 @@ export const CLIENTS = [
     detect: () => existsSync(join(home(), ".cursor")),
   },
   {
-    id: "windsurf",
-    label: "Windsurf",
-    file: () => join(home(), ".codeium", "windsurf", "mcp_config.json"),
-    format: "json",
-    entry: (url) => ({ serverUrl: url }),
-    detect: () => existsSync(join(home(), ".codeium", "windsurf")),
+    /**
+     * Grok's own entry, which it needs despite appearing to work without one.
+     *
+     * It reads `~/.claude.json` as a compatibility source, so on a machine
+     * that also has Claude Code it inherits that registration — client id,
+     * callback port and all — and looks configured when nothing here ran.
+     * On a machine without Claude Code there is nothing to inherit.
+     *
+     * Two differences from Codex, both from Grok's own config reference:
+     * `oauth_client_id` is a flat key rather than a nested table, and there
+     * is no callback setting at all — Grok picks the loopback address, which
+     * is why both spellings of it are registered with Clerk.
+     *
+     * `oauth_scopes` is spelled out because Grok sends no `scope` parameter
+     * otherwise, and a token minted without `offline_access` carries no
+     * refresh: the session works, then quietly stops.
+     */
+    id: "grok",
+    label: "Grok",
+    file: () => join(home(), ".grok", "config.toml"),
+    format: "toml",
+    entry: (url, clientId) =>
+      [
+        `[mcp_servers.${SERVER_NAME}]`,
+        `url = "${url}"`,
+        "enabled = true",
+        ...(clientId
+          ? [
+              `oauth_client_id = "${clientId}"`,
+              'oauth_scopes = ["profile", "email", "offline_access"]',
+            ]
+          : []),
+      ].join("\n"),
+    detect: () => existsSync(join(home(), ".grok")),
   },
   {
     id: "codex",
@@ -96,6 +125,14 @@ export const CLIENTS = [
       ].join("\n"),
     detect: () => existsSync(join(home(), ".codex")),
   },
+  {
+    id: "windsurf",
+    label: "Windsurf",
+    file: () => join(home(), ".codeium", "windsurf", "mcp_config.json"),
+    format: "json",
+    entry: (url) => ({ serverUrl: url }),
+    detect: () => existsSync(join(home(), ".codeium", "windsurf")),
+  },
 ];
 
 /** On PATH, without running the thing. */

package/src/tools.js

@@ -393,6 +393,18 @@ export function renderItem(item) {
   if (item.anchor.selectedText) {
     lines.push(`  with "${item.anchor.selectedText}" selected`);
   }
+  // Which row, when the element is one of many the same component rendered.
+  // The author's own React key, so it reads as something from their codebase
+  // — "the row with key plan:pro" locates a record, which is a different and
+  // better question than where on the page it was drawn.
+  //
+  // `skyId` is deliberately not here. It is an opaque hash: it anchors the
+  // pin and tells a reader nothing they can act on, and `source` below
+  // already names the file. It stays in the structured response for anything
+  // resolving anchors, out of the prose for anything reading one.
+  if (item.anchor.skyKey) {
+    lines.push(`  the item with key ${item.anchor.skyKey}`);
+  }
   lines.push(`  selector: ${item.anchor.selector}`);
   lines.push(
     `  seen at ${item.anchor.viewport.width}x${item.anchor.viewport.height}`,