@skyelight/mcp 0.1.1 → 0.1.3

package/package.json

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

package/src/tools.js

@@ -172,7 +172,7 @@ export function toolDefinitions(opts = {}) {
       description:
         "Move an item to open, deferred or resolved. Use it to close the loop after you have " +
         "fixed something and said so \u2014 post_update first, so the person who reported it " +
-        "reads what changed, then set_status. \"deferred\" is for work that is real but not now.",
+        'reads what changed, then set_status. "deferred" is for work that is real but not now.',
       inputSchema: {
         type: "object",
         properties: {
@@ -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("");
@@ -398,6 +422,41 @@ export function renderItem(item) {
         ? `Written by ${where}, in ${from.join(" on ")}.`
         : `Written by ${where}.`,
     );
+    // 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
+    // the person was actually looking at.
+    if (item.sourceUrl) lines.push(item.sourceUrl);
+  }
+
+  // The stamps read in both directions.
+  //
+  // Separated from the one-line `Written by` above because they answer a
+  // different question. That line says where the pinned element came from;
+  // this says which component it lives in, and which of its parts a request
+  // is likely to be about. A person points at a row and means the text
+  // inside it far more often than they mean the row.
+  const path = item.sourcePath;
+  if (path && (path.above.length || path.below.length)) {
+    if (path.above.length) {
+      lines.push("");
+      lines.push("Nested inside");
+      // 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}>`);
+      }
+    }
+    if (path.below.length) {
+      lines.push("");
+      lines.push("Contains");
+      for (const step of path.below) {
+        const indent = "  ".repeat((step.depth ?? 0) + 1);
+        const text = step.text ? `  "${step.text}"` : "";
+        lines.push(
+          `${indent}<${step.tag}> ${step.file}:${step.line ?? "?"}${text}`,
+        );
+      }
+    }
   }
 
   return lines.join("\n");