@skyelight/mcp 0.1.2 → 0.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyelight/mcp",
-  "version": "0.1.2",
+  "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

@@ -232,6 +232,25 @@ export function renderWorkspaces(result) {
   return lines.join("\n");
 }
 
+/**
+ * Coarse relative time.
+ *
+ * Deliberately vague past a day: an agent deciding whether to look at a
+ * project needs "today" or "last month", and a precise timestamp invites it
+ * to quote a date it has no reason to trust to the minute.
+ */
+function ago(ts) {
+  const mins = Math.max(0, Math.round((Date.now() - ts) / 60000));
+  if (mins < 1) return "just now";
+  if (mins < 60) return `${mins}m ago`;
+  const hours = Math.round(mins / 60);
+  if (hours < 24) return `${hours}h ago`;
+  const days = Math.round(hours / 24);
+  if (days < 30) return `${days}d ago`;
+  const months = Math.round(days / 30);
+  return months < 12 ? `${months}mo ago` : `${Math.round(months / 12)}y ago`;
+}
+
 export function renderProjects(result) {
   const { projects, total } = result;
   if (total === 0) {
@@ -244,7 +263,12 @@ export function renderProjects(result) {
   for (const p of projects) {
     // The workspace is only worth naming when there is more than one in
     // play; repeating it down a single-workspace list is noise.
-    lines.push(`  ${p.name}${spans ? ` (${p.workspace.name})` : ""}`);
+    // The activity date is why this list is ordered the way it is, and it
+    // was previously only in the structured payload — so an agent reading the
+    // prose could not tell a busy project from a dormant one, and one reading
+    // the raw field got `updatedAt`, which meant "renamed recently".
+    const when = p.lastActivityAt ? ` · ${ago(p.lastActivityAt)}` : "";
+    lines.push(`  ${p.name}${spans ? ` (${p.workspace.name})` : ""}${when}`);
     lines.push(`    ${p.id}${p.repo ? ` — ${p.repo}` : ""}`);
   }
   lines.push("");
@@ -386,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
@@ -393,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
@@ -419,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) {
@@ -428,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}`,
         );
       }
     }