@skyelight/mcp 0.1.3 → 0.1.4

package/package.json

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

package/src/tools.js

@@ -410,6 +410,9 @@ export function renderItem(item) {
     const where = item.source.line
       ? `${item.source.file}, line ${item.source.line}`
       : item.source.file;
+    // Named before located: "the DoseRow component" is what someone would say,
+    // and it stays true after the line has moved.
+    const rendered = item.sourcePath?.target?.component;
     // The build too, when the page carried one. Which commit was on the
     // screen is the difference between a live bug and a stale pin — and the
     // branch is what makes that commit findable when the page was a preview
@@ -417,10 +420,11 @@ export function renderItem(item) {
     const from = [];
     if (item.source.build) from.push(`build ${item.source.build}`);
     if (item.source.branch) from.push(`branch ${item.source.branch}`);
+    const by = rendered ? `the ${rendered} component, ${where}` : where;
     lines.push(
       from.length
-        ? `Written by ${where}, in ${from.join(" on ")}.`
-        : `Written by ${where}.`,
+        ? `Written by ${by}, in ${from.join(" on ")}.`
+        : `Written by ${by}.`,
     );
     // The permalink, when the project names a repository. Pinned to the
     // commit the page carried rather than to a branch, so it opens the code
@@ -443,7 +447,11 @@ export function renderItem(item) {
       // Outermost first, so a repeated file reads as one component and a
       // change of file reads as a boundary between two.
       for (const step of path.above) {
-        lines.push(`  ${step.file}:${step.line ?? "?"}  <${step.tag}>`);
+        // The component first when there is one: it is what a person would
+        // call this part of the page, and the one identifier that survives
+        // the lines below it moving.
+        const who = step.component ? `${step.component}  ` : "";
+        lines.push(`  ${who}${step.file}:${step.line ?? "?"}  <${step.tag}>`);
       }
     }
     if (path.below.length) {
@@ -452,8 +460,9 @@ export function renderItem(item) {
       for (const step of path.below) {
         const indent = "  ".repeat((step.depth ?? 0) + 1);
         const text = step.text ? `  "${step.text}"` : "";
+        const who = step.component ? ` ${step.component}` : "";
         lines.push(
-          `${indent}<${step.tag}> ${step.file}:${step.line ?? "?"}${text}`,
+          `${indent}<${step.tag}>${who} ${step.file}:${step.line ?? "?"}${text}`,
         );
       }
     }