@phren/cli 0.0.6 → 0.0.7

package/mcp/dist/mcp-session.js

@@ -294,7 +294,7 @@ export function getSessionArtifacts(phrenPath, sessionId, project) {
     const shortId = sessionId.slice(0, 8);
     try {
         const projectDirs = getProjectDirs(phrenPath);
-        const targetProjects = project ? [project] : projectDirs;
+        const targetProjects = project ? [project] : projectDirs.map((d) => path.basename(d));
         for (const proj of targetProjects) {
             // Findings with matching sessionId
             const findingsResult = readFindings(phrenPath, proj);
@@ -364,15 +364,16 @@ export function computeSessionDiff(phrenPath, project, lastSessionEnd) {
                 superseded++;
         }
     }
-    // Count completed tasks since last session
-    const tasksPath = path.join(projectDir, "TASKS.md");
+    // Count tasks completed since last session by checking Done items with recent activity dates
     let tasksCompleted = 0;
-    if (fs.existsSync(tasksPath)) {
-        const taskContent = fs.readFileSync(tasksPath, "utf8");
-        const doneMatch = taskContent.match(/## Done[\s\S]*/);
-        if (doneMatch) {
-            const doneLines = doneMatch[0].split("\n").filter(l => l.startsWith("- "));
-            tasksCompleted = doneLines.length;
+    const tasksResult = readTasks(phrenPath, project);
+    if (tasksResult.ok) {
+        for (const item of tasksResult.data.items.Done) {
+            // lastActivity is updated on completion; fall back to createdAt
+            const itemDate = item.lastActivity?.slice(0, 10) ?? item.createdAt?.slice(0, 10);
+            if (itemDate && itemDate >= cutoff) {
+                tasksCompleted++;
+            }
         }
     }
     return { newFindings, superseded, tasksCompleted };
@@ -443,12 +444,12 @@ export function register(server, ctx) {
             try {
                 const tasks = readTasks(phrenPath, activeProject);
                 if (tasks.ok) {
-                    const queueItems = tasks.data.items.Queue
+                    const activeItems = tasks.data.items.Active
                         .filter((entry) => isMemoryScopeVisible(normalizeMemoryScope(entry.scope), activeScope))
                         .slice(0, 5)
                         .map((entry) => `- [ ] ${entry.line}`);
-                    if (queueItems.length > 0) {
-                        parts.push(`## Active task (${activeProject})\n${queueItems.join("\n")}`);
+                    if (activeItems.length > 0) {
+                        parts.push(`## Active task (${activeProject})\n${activeItems.join("\n")}`);
                     }
                 }
             }

package/mcp/dist/mcp-skills.js

@@ -120,9 +120,12 @@ export function register(server, ctx) {
             }
             fs.mkdirSync(destDir, { recursive: true });
             const existing = findLocalSkill(phrenPath, scope, safeName);
-            const dest = existing
+            const dest = path.resolve(existing
                 ? existing.path
-                : path.join(destDir, `${safeName}.md`);
+                : path.join(destDir, `${safeName}.md`));
+            if (!dest.startsWith(phrenPath + path.sep) && dest !== phrenPath) {
+                return mcpResponse({ ok: false, error: "Skill path escapes phren store." });
+            }
             const existed = Boolean(existing) || fs.existsSync(dest);
             fs.writeFileSync(dest, content);
             updateFileInIndex(dest);

package/mcp/dist/mcp-tasks.js

@@ -159,9 +159,12 @@ export function register(server, ctx) {
                     data: { project, includedSections: view.includedSections, totalItems: view.totalItems, summary: true },
                 });
             }
+            const sectionCounts = view.includedSections
+                .map((s) => `${s}: ${view.doc.items[s].length}/${doc.items[s].length}`)
+                .join(", ");
             const paginationNote = view.truncated
-                ? `\n\n_Showing ${offset ?? 0}–${(offset ?? 0) + view.totalItems} of ${view.totalUnpaged} items. Use offset/limit to page._`
-                : (offset ? `\n\n_Page offset: ${offset}. ${view.totalItems} items returned._` : "");
+                ? `\n\n_${sectionCounts} (offset ${offset ?? 0}). Use offset/limit to page._`
+                : (offset ? `\n\n_Page offset: ${offset}. ${sectionCounts}._` : "");
             return mcpResponse({
                 ok: true,
                 message: `## ${project}\n${taskMarkdown(view.doc)}${paginationNote}`,
@@ -231,7 +234,7 @@ export function register(server, ctx) {
             const { added, errors } = result.data;
             if (added.length > 0)
                 refreshTaskIndex(updateFileInIndex, phrenPath, project);
-            return mcpResponse({ ok: added.length > 0, message: `Added ${added.length} of ${items.length} tasks to ${project}`, data: { project, added, errors } });
+            return mcpResponse({ ok: added.length > 0, ...(added.length === 0 ? { error: `No tasks added: ${errors.join("; ")}` } : {}), message: `Added ${added.length} of ${items.length} tasks to ${project}`, data: { project, added, errors } });
         });
     });
     server.registerTool("complete_task", {
@@ -303,7 +306,7 @@ export function register(server, ctx) {
             }
             if (completed.length > 0)
                 refreshTaskIndex(updateFileInIndex, phrenPath, project);
-            return mcpResponse({ ok: completed.length > 0, message: `Completed ${completed.length}/${items.length} items`, data: { project, completed, errors } });
+            return mcpResponse({ ok: completed.length > 0, ...(completed.length === 0 ? { error: `No tasks completed: ${errors.join("; ")}` } : {}), message: `Completed ${completed.length}/${items.length} items`, data: { project, completed, errors } });
         });
     });
     server.registerTool("remove_task", {