@projitive/mcp 2.0.3 → 2.1.0

package/output/source/common/response.js

@@ -1,76 +1,70 @@
-import fs from "node:fs";
-import path from "node:path";
-const MESSAGE_TEMPLATE_ENV = "PROJITIVE_MESSAGE_TEMPLATE_PATH";
-const CONTENT_TEMPLATE_TOKEN = "{{content}}";
+import fs from 'node:fs';
+import path from 'node:path';
+const MESSAGE_TEMPLATE_ENV = 'PROJITIVE_MESSAGE_TEMPLATE_PATH';
 function baseToolTemplateMarkdown() {
     return [
-        "# {{tool_name}}",
-        "",
-        "## Summary",
-        "{{summary}}",
-        "",
-        "## Evidence",
-        "{{evidence}}",
-        "",
-        "## Agent Guidance",
-        "{{guidance}}",
-        "",
-        "## Next Call",
-        "{{next_call}}",
-        "",
-        "## Raw Response",
-        "{{content}}",
-    ].join("\n");
-}
-function contextGuideTemplateExtra() {
-    return [
-        "",
-        "## Common Tool Guides To Read First",
-        "- ./CLAUDE.md",
-        "- ./AGENTS.md",
-        "- ./.github/copilot-instructions.md",
-        "- ./.cursorrules",
-        "- ./.github/instructions/*",
-        "- ./.cursor/rules/*",
-    ];
-}
-function idleDiscoveryTemplateExtra() {
-    return [
-        "",
-        "## Idle Discovery Checklist (When No Actionable Task)",
-        "- Scan backlog comments: TODO / FIXME / HACK / XXX.",
-        "- Check lint gaps and create executable fix tasks.",
-        "- Check test quality gaps (missing tests, flaky tests, low-value coverage).",
-        "- Learn current project architecture and consolidate/update design docs in designs/.",
-        "- Re-run {{tool_name}} after creating 1-3 focused TODO tasks.",
-    ];
-}
-function commitReminderTemplateExtra() {
-    return [
-        "",
-        "## Commit Reminder",
-        "- After this update, create a commit to keep progress auditable.",
-        "- Recommended format: type(scope): summary",
-        "- Example: feat(task): complete TASK-0007 validation flow",
-        "- Footer suggestion: Refs: TASK-0007, ROADMAP-0002",
-    ];
+        '# {{tool_name}}',
+        '',
+        '## Summary',
+        '{{summary}}',
+        '',
+        '## Evidence',
+        '{{evidence}}',
+        '',
+        '## Agent Guidance',
+        '{{guidance}}',
+        '',
+        '## Lint Suggestions',
+        '{{suggestions}}',
+        '',
+        '## Next Call',
+        '{{next_call}}',
+    ].join('\n');
 }
 export function getDefaultToolTemplateMarkdown(toolName) {
-    const base = baseToolTemplateMarkdown().split("\n");
-    if (toolName === "taskNext") {
-        return [...base, ...idleDiscoveryTemplateExtra()].join("\n");
-    }
-    if (toolName === "projectContext" || toolName === "taskContext" || toolName === "roadmapContext") {
-        return [...base, ...contextGuideTemplateExtra()].join("\n");
-    }
-    if (toolName === "taskUpdate" || toolName === "roadmapUpdate") {
-        return [...base, ...commitReminderTemplateExtra()].join("\n");
-    }
-    return base.join("\n");
+    const base = baseToolTemplateMarkdown();
+    if (toolName === 'taskNext') {
+        return [
+            base,
+            '',
+            '## Idle Discovery Checklist (When No Actionable Task)',
+            '- Scan backlog comments: TODO / FIXME / HACK / XXX.',
+            '- Check lint gaps and create executable fix tasks.',
+            '- Check test quality gaps (missing tests, flaky tests, low-value coverage).',
+            '- Learn current project architecture and consolidate/update design docs in designs/.',
+            '- Review and update architecture docs under designs/core/ (architecture.md, style-guide.md) if missing or outdated.',
+            '- Re-run {{tool_name}} after creating 1-3 focused TODO tasks.',
+        ].join('\n');
+    }
+    if (toolName === 'projectContext' || toolName === 'taskContext' || toolName === 'roadmapContext') {
+        return [
+            base,
+            '',
+            '## Common Tool Guides To Read First',
+            '- ./CLAUDE.md',
+            '- ./AGENTS.md',
+            '- ./.github/copilot-instructions.md',
+            '- ./.cursorrules',
+            '- ./.github/instructions/*',
+            '- ./.cursor/rules/*',
+        ].join('\n');
+    }
+    if (toolName === 'taskUpdate' || toolName === 'roadmapUpdate') {
+        return [
+            base,
+            '',
+            '## Commit Reminder',
+            '- After this update, create a commit to keep progress auditable.',
+            '- Recommended format: type(scope): summary',
+            '- Example: feat(task): complete TASK-0007 validation flow',
+            '- Footer suggestion: Refs: TASK-0007, ROADMAP-0002',
+        ].join('\n');
+    }
+    return base;
 }
 function loadTemplateFile(templatePath) {
     try {
-        const content = fs.readFileSync(templatePath, "utf-8").trim();
+        const content = fs.readFileSync(templatePath, 'utf-8').trim();
         return content.length > 0 ? content : undefined;
     }
     catch {
@@ -84,13 +78,13 @@ function ensureTemplateFile(templatePath, toolName) {
     }
     fs.mkdirSync(path.dirname(templatePath), { recursive: true });
     const generated = getDefaultToolTemplateMarkdown(toolName);
-    fs.writeFileSync(templatePath, `${generated}\n`, "utf-8");
+    fs.writeFileSync(templatePath, `${generated}\n`, 'utf-8');
     return generated;
 }
 function resolveTemplateTarget(toolName) {
     const configuredPath = process.env[MESSAGE_TEMPLATE_ENV]?.trim();
     if (!configuredPath) {
-        return path.resolve(process.cwd(), ".projitive", "templates", "tools", `${toolName}.md`);
+        return path.resolve(process.cwd(), '.projitive', 'templates', 'tools', `${toolName}.md`);
     }
     const absolutePath = path.resolve(configuredPath);
     try {
@@ -102,7 +96,7 @@ function resolveTemplateTarget(toolName) {
     }
     catch {
         const ext = path.extname(absolutePath).toLowerCase();
-        if (ext === ".md") {
+        if (ext === '.md') {
             return absolutePath;
         }
         return path.join(absolutePath, `${toolName}.md`);
@@ -114,17 +108,17 @@ function loadMessageTemplate(toolName) {
 }
 export function asText(markdown) {
     return {
-        content: [{ type: "text", text: markdown }],
+        content: [{ type: 'text', text: markdown }],
     };
 }
 function withFallback(lines) {
-    return lines.length > 0 ? lines : ["- (none)"];
+    return lines.length > 0 ? lines : ['- (none)'];
 }
 function shouldKeepRawLine(trimmed) {
     if (trimmed.length === 0) {
         return true;
     }
-    if (trimmed.startsWith("#") || trimmed.startsWith(">") || trimmed.startsWith("```")) {
+    if (trimmed.startsWith('#') || trimmed.startsWith('>') || trimmed.startsWith('```')) {
         return true;
     }
     if (/^[-*+]\s/.test(trimmed)) {
@@ -149,37 +143,37 @@ export function section(title, lines) {
     return { title, lines: normalizeLines(lines) };
 }
 export function summarySection(lines) {
-    return section("Summary", lines);
+    return section('Summary', lines);
 }
 export function evidenceSection(lines) {
-    return section("Evidence", lines);
+    return section('Evidence', lines);
 }
 export function guidanceSection(lines) {
-    return section("Agent Guidance", lines);
+    return section('Agent Guidance', lines);
 }
 export function lintSection(lines) {
-    return section("Lint Suggestions", lines);
+    return section('Lint Suggestions', lines);
 }
 export function nextCallSection(nextCall) {
-    return section("Next Call", nextCall ? [nextCall] : []);
+    return section('Next Call', nextCall ? [nextCall] : []);
 }
 function toSectionText(section) {
     if (!section) {
-        return "- (none)";
+        return '- (none)';
     }
-    return withFallback(section.lines).join("\n");
+    return withFallback(section.lines).join('\n');
 }
 function resolveSection(payload, title) {
     return payload.sections.find((item) => item.title === title);
 }
-function buildToolTemplateVariables(payload, classicMarkdown) {
+function buildToolTemplateVariables(payload) {
     return {
         tool_name: payload.toolName,
-        content: classicMarkdown,
-        summary: toSectionText(resolveSection(payload, "Summary")),
-        evidence: toSectionText(resolveSection(payload, "Evidence")),
-        guidance: toSectionText(resolveSection(payload, "Agent Guidance")),
-        next_call: toSectionText(resolveSection(payload, "Next Call")),
+        summary: toSectionText(resolveSection(payload, 'Summary')),
+        evidence: toSectionText(resolveSection(payload, 'Evidence')),
+        guidance: toSectionText(resolveSection(payload, 'Agent Guidance')),
+        suggestions: toSectionText(resolveSection(payload, 'Lint Suggestions')),
+        next_call: toSectionText(resolveSection(payload, 'Next Call')),
     };
 }
 function applyTemplateVariables(template, variables) {
@@ -187,33 +181,23 @@ function applyTemplateVariables(template, variables) {
     for (const [key, value] of Object.entries(variables)) {
         rendered = rendered.split(`{{${key}}}`).join(value);
     }
-    if (!rendered.includes(variables.content) && !template.includes(CONTENT_TEMPLATE_TOKEN)) {
-        rendered = `${rendered}\n\n${variables.content}`;
-    }
     return rendered.trimEnd();
 }
 export function renderToolResponseMarkdown(payload) {
-    const body = payload.sections.flatMap((section) => [
-        `## ${section.title}`,
-        ...withFallback(section.lines),
-        "",
-    ]);
-    const classicMarkdown = [
-        `# ${payload.toolName}`,
-        "",
-        ...body,
-    ].join("\n").trimEnd();
     const template = loadMessageTemplate(payload.toolName);
-    const variables = buildToolTemplateVariables(payload, classicMarkdown);
+    const variables = buildToolTemplateVariables(payload);
     return applyTemplateVariables(template, variables);
 }
 export function renderErrorMarkdown(toolName, cause, nextSteps, retryExample) {
-    return renderToolResponseMarkdown({
-        toolName,
-        sections: [
-            section("Error", [`cause: ${cause}`]),
-            section("Next Step", nextSteps),
-            section("Retry Example", [retryExample ?? "(none)"]),
-        ],
-    });
+    const sections = [
+        section('Error', [`cause: ${cause}`]),
+        section('Next Step', nextSteps),
+        section('Retry Example', [retryExample ?? '(none)']),
+    ];
+    const body = sections.flatMap((sec) => [
+        `## ${sec.title}`,
+        ...withFallback(sec.lines),
+        '',
+    ]);
+    return [`# ${toolName}`, '', ...body].join('\n').trimEnd();
 }

package/output/source/common/response.test.js

@@ -1,50 +1,50 @@
-import { describe, expect, it } from "vitest";
+import { describe, expect, it } from 'vitest';
 import { asText, nextCallSection, renderErrorMarkdown, renderToolResponseMarkdown, summarySection, } from './response.js';
-describe("response helpers", () => {
-    it("wraps markdown text as MCP text content", () => {
-        const result = asText("# hello");
-        expect(result.content).toEqual([{ type: "text", text: "# hello" }]);
+describe('response helpers', () => {
+    it('wraps markdown text as MCP text content', () => {
+        const result = asText('# hello');
+        expect(result.content).toEqual([{ type: 'text', text: '# hello' }]);
     });
-    it("renders error markdown sections", () => {
-        const markdown = renderErrorMarkdown("taskContext", "bad id", ["retry"], "taskContext(...)");
-        expect(markdown).toContain("# taskContext");
-        expect(markdown).toContain("## Error");
-        expect(markdown).toContain("- cause: bad id");
-        expect(markdown).toContain("- retry");
-        expect(markdown).toContain("## Retry Example");
+    it('renders error markdown sections', () => {
+        const markdown = renderErrorMarkdown('taskContext', 'bad id', ['retry'], 'taskContext(...)');
+        expect(markdown).toContain('# taskContext');
+        expect(markdown).toContain('## Error');
+        expect(markdown).toContain('- cause: bad id');
+        expect(markdown).toContain('- retry');
+        expect(markdown).toContain('## Retry Example');
     });
-    it("renders standard tool response sections with fallback", () => {
+    it('renders standard tool response sections with fallback', () => {
         const markdown = renderToolResponseMarkdown({
-            toolName: "taskList",
+            toolName: 'taskList',
             sections: [
-                { title: "Summary", lines: ["- governanceDir: /tmp/.projitive"] },
-                { title: "Evidence", lines: [] },
+                { title: 'Summary', lines: ['- governanceDir: /tmp/.projitive'] },
+                { title: 'Evidence', lines: [] },
             ],
         });
-        expect(markdown).toContain("# taskList");
-        expect(markdown).toContain("## Summary");
-        expect(markdown).toContain("## Evidence");
-        expect(markdown).toContain("- (none)");
+        expect(markdown).toContain('# taskList');
+        expect(markdown).toContain('## Summary');
+        expect(markdown).toContain('## Evidence');
+        expect(markdown).toContain('- (none)');
     });
-    it("auto-prefixes plain lines in section helpers", () => {
+    it('auto-prefixes plain lines in section helpers', () => {
         const markdown = renderToolResponseMarkdown({
-            toolName: "taskList",
+            toolName: 'taskList',
             sections: [
-                summarySection(["governanceDir: /tmp/.projitive"]),
+                summarySection(['governanceDir: /tmp/.projitive']),
             ],
         });
-        expect(markdown).toContain("- governanceDir: /tmp/.projitive");
+        expect(markdown).toContain('- governanceDir: /tmp/.projitive');
     });
-    it("nextCallSection accepts optional call and falls back when missing", () => {
+    it('nextCallSection accepts optional call and falls back when missing', () => {
         const withCall = renderToolResponseMarkdown({
-            toolName: "taskList",
-            sections: [nextCallSection("taskContext(projectPath=\"/tmp\", taskId=\"TASK-0001\")")],
+            toolName: 'taskList',
+            sections: [nextCallSection('taskContext(projectPath="/tmp", taskId="TASK-0001")')],
         });
-        expect(withCall).toContain("- taskContext(projectPath=\"/tmp\", taskId=\"TASK-0001\")");
+        expect(withCall).toContain('- taskContext(projectPath="/tmp", taskId="TASK-0001")');
         const withoutCall = renderToolResponseMarkdown({
-            toolName: "taskList",
+            toolName: 'taskList',
             sections: [nextCallSection(undefined)],
         });
-        expect(withoutCall).toContain("- (none)");
+        expect(withoutCall).toContain('- (none)');
     });
 });

package/output/source/common/store.js

@@ -1,5 +1,5 @@
-import fs from "node:fs/promises";
-import path from "node:path";
+import fs from 'node:fs/promises';
+import path from 'node:path';
 const STORE_SCHEMA_VERSION = 3;
 const storeCache = new Map();
 const storeLocks = new Map();
@@ -8,13 +8,13 @@ function defaultViewState(name) {
         name,
         dirty: true,
         lastSourceVersion: 0,
-        lastBuiltAt: "",
+        lastBuiltAt: '',
         recordVersion: 1,
     };
 }
 function defaultStore() {
     return {
-        schema: "projitive-json-store",
+        schema: 'projitive-json-store',
         tasks: [],
         roadmaps: [],
         meta: {
@@ -23,8 +23,8 @@ function defaultStore() {
             store_schema_version: STORE_SCHEMA_VERSION,
         },
         view_state: {
-            tasks_markdown: defaultViewState("tasks_markdown"),
-            roadmaps_markdown: defaultViewState("roadmaps_markdown"),
+            tasks_markdown: defaultViewState('tasks_markdown'),
+            roadmaps_markdown: defaultViewState('roadmaps_markdown'),
         },
         migration_history: [],
     };
@@ -33,24 +33,24 @@ function nowIso() {
     return new Date().toISOString();
 }
 function normalizeTaskStatus(status) {
-    if (status === "IN_PROGRESS" || status === "BLOCKED" || status === "DONE") {
+    if (status === 'IN_PROGRESS' || status === 'BLOCKED' || status === 'DONE') {
         return status;
     }
-    return "TODO";
+    return 'TODO';
 }
 function normalizeRoadmapStatus(status) {
-    return status === "done" ? "done" : "active";
+    return status === 'done' ? 'done' : 'active';
 }
 function normalizeStore(input) {
     const base = defaultStore();
     const meta = input.meta ?? {};
     const tasks = Array.isArray(input.tasks)
         ? input.tasks.map((task) => ({
-            id: String(task.id ?? ""),
-            title: String(task.title ?? ""),
-            status: normalizeTaskStatus(String(task.status ?? "TODO")),
-            owner: String(task.owner ?? ""),
-            summary: String(task.summary ?? ""),
+            id: String(task.id ?? ''),
+            title: String(task.title ?? ''),
+            status: normalizeTaskStatus(String(task.status ?? 'TODO')),
+            owner: String(task.owner ?? ''),
+            summary: String(task.summary ?? ''),
             updatedAt: String(task.updatedAt ?? nowIso()),
             links: Array.isArray(task.links) ? task.links.map((item) => String(item)) : [],
             roadmapRefs: Array.isArray(task.roadmapRefs) ? task.roadmapRefs.map((item) => String(item)) : [],
@@ -61,10 +61,10 @@ function normalizeStore(input) {
         : [];
     const roadmaps = Array.isArray(input.roadmaps)
         ? input.roadmaps.map((milestone) => ({
-            id: String(milestone.id ?? ""),
-            title: String(milestone.title ?? ""),
-            status: normalizeRoadmapStatus(String(milestone.status ?? "active")),
-            time: typeof milestone.time === "string" && milestone.time.length > 0 ? milestone.time : undefined,
+            id: String(milestone.id ?? ''),
+            title: String(milestone.title ?? ''),
+            status: normalizeRoadmapStatus(String(milestone.status ?? 'active')),
+            time: typeof milestone.time === 'string' && milestone.time.length > 0 ? milestone.time : undefined,
             updatedAt: String(milestone.updatedAt ?? nowIso()),
             recordVersion: Number.isFinite(Number(milestone.recordVersion)) ? Number(milestone.recordVersion) : 1,
         }))
@@ -72,7 +72,7 @@ function normalizeStore(input) {
     const tasksView = input.view_state?.tasks_markdown;
     const roadmapsView = input.view_state?.roadmaps_markdown;
     return {
-        schema: "projitive-json-store",
+        schema: 'projitive-json-store',
         tasks,
         roadmaps,
         meta: {
@@ -83,22 +83,22 @@ function normalizeStore(input) {
         view_state: {
             tasks_markdown: {
                 ...base.view_state.tasks_markdown,
-                dirty: typeof tasksView?.dirty === "boolean" ? tasksView.dirty : base.view_state.tasks_markdown.dirty,
+                dirty: typeof tasksView?.dirty === 'boolean' ? tasksView.dirty : base.view_state.tasks_markdown.dirty,
                 lastSourceVersion: Number.isFinite(Number(tasksView?.lastSourceVersion))
                     ? Number(tasksView?.lastSourceVersion)
                     : base.view_state.tasks_markdown.lastSourceVersion,
-                lastBuiltAt: typeof tasksView?.lastBuiltAt === "string" ? tasksView.lastBuiltAt : base.view_state.tasks_markdown.lastBuiltAt,
+                lastBuiltAt: typeof tasksView?.lastBuiltAt === 'string' ? tasksView.lastBuiltAt : base.view_state.tasks_markdown.lastBuiltAt,
                 recordVersion: Number.isFinite(Number(tasksView?.recordVersion))
                     ? Number(tasksView?.recordVersion)
                     : base.view_state.tasks_markdown.recordVersion,
             },
             roadmaps_markdown: {
                 ...base.view_state.roadmaps_markdown,
-                dirty: typeof roadmapsView?.dirty === "boolean" ? roadmapsView.dirty : base.view_state.roadmaps_markdown.dirty,
+                dirty: typeof roadmapsView?.dirty === 'boolean' ? roadmapsView.dirty : base.view_state.roadmaps_markdown.dirty,
                 lastSourceVersion: Number.isFinite(Number(roadmapsView?.lastSourceVersion))
                     ? Number(roadmapsView?.lastSourceVersion)
                     : base.view_state.roadmaps_markdown.lastSourceVersion,
-                lastBuiltAt: typeof roadmapsView?.lastBuiltAt === "string" ? roadmapsView.lastBuiltAt : base.view_state.roadmaps_markdown.lastBuiltAt,
+                lastBuiltAt: typeof roadmapsView?.lastBuiltAt === 'string' ? roadmapsView.lastBuiltAt : base.view_state.roadmaps_markdown.lastBuiltAt,
                 recordVersion: Number.isFinite(Number(roadmapsView?.recordVersion))
                     ? Number(roadmapsView?.recordVersion)
                     : base.view_state.roadmaps_markdown.recordVersion,
@@ -111,7 +111,7 @@ async function persistStore(dbPath, store) {
     await fs.mkdir(path.dirname(dbPath), { recursive: true });
     const tempPath = `${dbPath}.tmp-${process.pid}-${Date.now()}`;
     const body = `${JSON.stringify(store, null, 2)}\n`;
-    await fs.writeFile(tempPath, body, "utf8");
+    await fs.writeFile(tempPath, body, 'utf8');
     await fs.rename(tempPath, dbPath);
 }
 async function loadStoreFromDisk(dbPath) {
@@ -119,7 +119,7 @@ async function loadStoreFromDisk(dbPath) {
     if (!file || file.length === 0) {
         return { store: defaultStore(), shouldPersist: true };
     }
-    const text = file.toString("utf8").trim();
+    const text = file.toString('utf8').trim();
     if (text.length === 0) {
         return { store: defaultStore(), shouldPersist: true };
     }
@@ -164,7 +164,7 @@ async function withStoreLock(dbPath, action) {
     }
 }
 function bumpVersionAndDirtyView(store, kind) {
-    if (kind === "tasks") {
+    if (kind === 'tasks') {
         store.meta.tasks_version += 1;
         const view = store.view_state.tasks_markdown;
         view.dirty = true;
@@ -205,9 +205,9 @@ function safeTime(value) {
     return Number.isFinite(t) ? t : 0;
 }
 function normalizeStatusForSort(status) {
-    if (status === "IN_PROGRESS")
+    if (status === 'IN_PROGRESS')
         return 2;
-    if (status === "TODO")
+    if (status === 'TODO')
         return 1;
     return 0;
 }
@@ -216,7 +216,7 @@ export async function ensureStore(dbPath) {
 }
 export async function getStoreVersion(dbPath, kind) {
     const store = await openStore(dbPath);
-    return kind === "tasks" ? store.meta.tasks_version : store.meta.roadmaps_version;
+    return kind === 'tasks' ? store.meta.tasks_version : store.meta.roadmaps_version;
 }
 export async function getMarkdownViewState(dbPath, viewName) {
     const store = await openStore(dbPath);
@@ -253,20 +253,20 @@ export async function loadTasksFromStore(dbPath) {
 }
 export async function loadTaskStatusStatsFromStore(dbPath) {
     const tasks = await loadTasksFromStore(dbPath);
-    const todo = tasks.filter((task) => task.status === "TODO").length;
-    const inProgress = tasks.filter((task) => task.status === "IN_PROGRESS").length;
-    const blocked = tasks.filter((task) => task.status === "BLOCKED").length;
-    const done = tasks.filter((task) => task.status === "DONE").length;
+    const todo = tasks.filter((task) => task.status === 'TODO').length;
+    const inProgress = tasks.filter((task) => task.status === 'IN_PROGRESS').length;
+    const blocked = tasks.filter((task) => task.status === 'BLOCKED').length;
+    const done = tasks.filter((task) => task.status === 'DONE').length;
     const total = tasks.length;
     const latestUpdatedAt = tasks
         .map((task) => task.updatedAt)
-        .sort((a, b) => safeTime(b) - safeTime(a))[0] ?? "";
+        .sort((a, b) => safeTime(b) - safeTime(a))[0] ?? '';
     return { todo, inProgress, blocked, done, total, latestUpdatedAt };
 }
 export async function loadActionableTasksFromStore(dbPath, limit) {
     const tasks = await loadTasksFromStore(dbPath);
     const sorted = tasks
-        .filter((task) => task.status === "IN_PROGRESS" || task.status === "TODO")
+        .filter((task) => task.status === 'IN_PROGRESS' || task.status === 'TODO')
         .sort((a, b) => {
         const ap = normalizeStatusForSort(a.status);
         const bp = normalizeStatusForSort(b.status);
@@ -279,7 +279,7 @@ export async function loadActionableTasksFromStore(dbPath, limit) {
         }
         return b.id.localeCompare(a.id);
     });
-    if (typeof limit === "number" && Number.isFinite(limit) && limit > 0) {
+    if (typeof limit === 'number' && Number.isFinite(limit) && limit > 0) {
         return sorted.slice(0, Math.floor(limit));
     }
     return sorted;
@@ -307,7 +307,7 @@ export async function upsertTaskInStore(dbPath, task) {
                 recordVersion: 1,
             });
         }
-        bumpVersionAndDirtyView(store, "tasks");
+        bumpVersionAndDirtyView(store, 'tasks');
         await persistStore(dbPath, store);
     });
 }
@@ -321,7 +321,7 @@ export async function replaceTasksInStore(dbPath, tasks) {
             roadmapRefs: [...(task.roadmapRefs ?? [])],
             recordVersion: 1,
         }));
-        bumpVersionAndDirtyView(store, "tasks");
+        bumpVersionAndDirtyView(store, 'tasks');
         await persistStore(dbPath, store);
     });
 }
@@ -349,7 +349,7 @@ export async function replaceRoadmapsInStore(dbPath, milestones) {
             status: normalizeRoadmapStatus(milestone.status),
             recordVersion: 1,
         }));
-        bumpVersionAndDirtyView(store, "roadmaps");
+        bumpVersionAndDirtyView(store, 'roadmaps');
         await persistStore(dbPath, store);
     });
 }
@@ -372,7 +372,7 @@ export async function upsertRoadmapInStore(dbPath, milestone) {
                 recordVersion: 1,
             });
         }
-        bumpVersionAndDirtyView(store, "roadmaps");
+        bumpVersionAndDirtyView(store, 'roadmaps');
         await persistStore(dbPath, store);
     });
 }