pi-note 0.20.0 → 0.21.0

package/dist-mcp/mcp/server.js

@@ -1,13 +1,14 @@
 import { createHash, randomUUID } from 'node:crypto';
 import { promises as fs } from 'node:fs';
 import { homedir } from 'node:os';
-import { join } from 'node:path';
+import { isAbsolute, join } from 'node:path';
 import { execFile } from 'node:child_process';
 import { promisify } from 'node:util';
-import { readFileSync } from 'node:fs';
+import { readFileSync, watchFile, unwatchFile } from 'node:fs';
+import { getAgentDir } from "../config-path.js";
 import { Server } from '@modelcontextprotocol/sdk/server/index.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
-import { CallToolRequestSchema, ListToolsRequestSchema, ListResourcesRequestSchema, ReadResourceRequestSchema } from '@modelcontextprotocol/sdk/types.js';
+import { CallToolRequestSchema, ListToolsRequestSchema, ListResourcesRequestSchema, ReadResourceRequestSchema, McpError } from '@modelcontextprotocol/sdk/types.js';
 import { z } from 'zod';
 import { defaultFooterField, defaultHandoffPrompt, defaultNewSessionPrompt, effectiveFooterFields, parseFooterConfig, parseHandoffPrompt, parseNewSessionPrompt, readFooterDocument, saveFooterConfig } from "../footer-config.js";
 const URI = 'pinote://ui';
@@ -26,7 +27,7 @@ const Action = z.object({ method: z.literal('native-ui/action'), params: z.objec
         uri: z.literal(URI), revision: z.string().min(1).max(256), action: z.string().min(1).max(256),
         value: z.union([z.string().max(20_000), z.number().finite(), z.boolean(), z.null()]).optional(), _meta: z.record(z.unknown()).optional(),
     }) });
-const server = new Server({ name: 'pinote', version: '0.20.0' }, { capabilities: { tools: {}, resources: {}, experimental: { 'native-ui-v1': {} } } });
+const server = new Server({ name: 'pinote', version: '0.21.0' }, { capabilities: { tools: {}, resources: {}, experimental: { 'native-ui-v1': {} } } });
 // Serialize complete read/check/write cycles even when a host sends parallel calls.
 let queue = Promise.resolve();
 function serial(run) {
@@ -73,8 +74,8 @@ async function note(args, signal) {
     const options = { signal, timeout: 10_000, maxBuffer: 4 * 1024 * 1024 };
     const version = (await exec(noteCommand, ['--version'], options)).stdout.trim();
     const match = /^pinote (\d+)\.(\d+)\.(\d+)$/.exec(version);
-    if (!match || Number(match[1]) === 0 && Number(match[2]) < 4)
-        throw new Error('Pinote MCP requires note 0.4.0+');
+    if (!match || Number(match[1]) === 0 && Number(match[2]) < 12)
+        throw new Error('Pinote MCP requires note 0.12.0+');
     return (await exec(noteCommand, ['--no-notify', ...args], options)).stdout;
 }
 async function selected(state, signal) {
@@ -115,7 +116,25 @@ async function snapshot(state, signal) {
     const revision = createHash('sha256').update(JSON.stringify({ state, tasks, task, raw: config.raw })).digest('hex');
     return { state, tasks, task, config, revision };
 }
-function view(data) {
+function scopedActions(meta) {
+    return z.object({ actionRevisions: z.literal(true) }).safeParse(meta?.['native-ui/features']).success;
+}
+function actionRevision(data, control) {
+    const { id, revision: _revision, ...semantics } = control;
+    let dependency = null;
+    if (id === 'done')
+        dependency = data.task && { id: data.task.id, updated_at: data.task.updated_at };
+    else if (id.startsWith('select:'))
+        dependency = data.tasks.find((task) => task.id === Number(id.slice(7)));
+    else if (!['preview', 'continue', 'clear', 'search', 'add', 'proposal:accept', 'proposal:dismiss'].includes(id) && !id.startsWith('screen:') && !id.startsWith('field:')) {
+        dependency = { raw: data.config.raw, ...(['add-field', 'remove-field', 'label', 'format', 'width', 'link'].includes(id) ? { fields: fields(data.config.config, data.task?.agent_notes) } : {}) };
+    }
+    return createHash('sha256').update(JSON.stringify({ action: id, semantics, state: data.state, dependency })).digest('hex');
+}
+function staleAction() {
+    throw new McpError(-33003, 'View changed; refresh before acting', { 'native-ui/action': { status: 'not-applied', reason: 'stale-view' } });
+}
+function view(data, scoped = false) {
     const { tasks, task, config, revision } = data;
     const state = Context.parse({ conversationId: 'outgoing-state', state: { ...data.state, selectedId: task?.id ?? null } }).state;
     const controls = [];
@@ -207,6 +226,9 @@ function view(data) {
         body = `${control.label} exceeds the editor limit; edit it outside this UI.\n${body}`;
         return false;
     });
+    if (scoped)
+        for (const control of editable)
+            control.revision = actionRevision(data, control);
     const result = { version: 1, title, body: body.slice(0, 20_000), revision, state: boundedState(state), retainOnNew: true, controls: editable, footer };
     // JSON escaping and expanded footer formats can exceed the host budget even
     // with bounded row counts. Preserve navigation/filter controls ahead of rows.
@@ -231,16 +253,16 @@ server.setRequestHandler(ReadResourceRequestSchema, (request, extra) => serial(a
     if (request.params.uri !== URI)
         throw new Error('Unknown resource');
     const data = await snapshot(context(request.params._meta).state, extra.signal);
-    return { contents: [{ uri: URI, mimeType: MIME, text: JSON.stringify(view(data)) }] };
+    return { contents: [{ uri: URI, mimeType: MIME, text: JSON.stringify(view(data, scopedActions(request.params._meta))) }] };
 }));
 server.setRequestHandler(Action, (request, extra) => serial(async () => {
     const state = context(request.params._meta).state;
     const before = await snapshot(state, extra.signal);
-    if (before.revision !== request.params.revision)
-        throw new Error('View changed; refresh before acting');
+    const scoped = scopedActions(request.params._meta);
     const { action, value } = request.params;
-    if (!view(before).controls.some((c) => c.id === action))
-        throw new Error('Action is not available in this view');
+    const control = view(before, scoped).controls.find((item) => item.id === action);
+    if (!control || (control.revision ?? before.revision) !== request.params.revision)
+        staleAction();
     const next = { ...before.state };
     let draft, preview;
     if (action === 'proposal:accept' || action === 'proposal:dismiss') {
@@ -251,7 +273,7 @@ server.setRequestHandler(Action, (request, extra) => serial(async () => {
         next.proposal = null;
         if (action === 'proposal:accept') {
             const task = JSON.parse(await note(['agent', 'add', `--text=${proposal.text}`, ...(proposal.tag ? [`--tag=${proposal.tag}`] : [])], extra.signal));
-            await note(['start', String(task.id)], extra.signal);
+            await note(['agent', 'start', String(task.id), '--expected-updated-at', task.updated_at], extra.signal);
             next.selectedId = task.id;
         }
     }
@@ -271,11 +293,15 @@ server.setRequestHandler(Action, (request, extra) => serial(async () => {
         preview = (before.task.text + '\n\n' + JSON.stringify(before.task.agent_notes, null, 2)).slice(0, 20_000);
     else if (action.startsWith('select:')) {
         const id = Number(action.slice(7));
-        if (!before.tasks.some((t) => t.id === id))
-            throw new Error('Task no longer selectable');
+        const candidate = before.tasks.find((task) => task.id === id);
+        if (!candidate)
+            staleAction();
+        const target = JSON.parse(await note(['agent', 'get', String(id)], extra.signal));
+        if (target.text !== candidate.text || target.state !== candidate.state || target.tag !== candidate.tag)
+            staleAction();
         if (next.proposal)
             await consume(next.proposal.id);
-        await note(['start', String(id)], extra.signal);
+        await note(['agent', 'start', String(id), '--expected-updated-at', target.updated_at], extra.signal);
         next.selectedId = id;
         next.proposal = null;
     }
@@ -284,7 +310,7 @@ server.setRequestHandler(Action, (request, extra) => serial(async () => {
             await consume(next.proposal.id);
         next.proposal = null;
         const task = JSON.parse(await note(['agent', 'add', `--text=${z.string().min(1).parse(value)}`], extra.signal));
-        await note(['start', String(task.id)], extra.signal);
+        await note(['agent', 'start', String(task.id), '--expected-updated-at', task.updated_at], extra.signal);
         next.selectedId = task.id;
     }
     else if (action === 'done') {
@@ -325,14 +351,14 @@ server.setRequestHandler(Action, (request, extra) => serial(async () => {
         extra.signal.throwIfAborted();
         saveFooterConfig(cfg, before.config.raw, handoff, newSession, policy);
     }
-    const result = { view: view(await snapshot(next, extra.signal)), ...(draft ? { draft } : {}), ...(preview ? { preview } : {}) };
+    const result = { view: view(await snapshot(next, extra.signal), scoped), ...(draft ? { draft } : {}), ...(preview ? { preview } : {}) };
     await server.notification({ method: 'notifications/native-ui/changed', params: { uri: URI } });
     return result;
 }));
 const empty = { type: 'object', properties: {}, additionalProperties: false };
 server.setRequestHandler(ListToolsRequestSchema, async () => ({ tools: [
-        { name: 'propose', description: 'Suggest a task without creating it. Requires no selected task. Show a short action-oriented title, optional details after a blank line, and reuse a saved tag. The user clicks + to create/start/select or dismiss. Replaces an unaccepted suggestion; continue working without asking again. Never creates a task before a user action.', inputSchema: { type: 'object', properties: { text: { type: 'string', minLength: 1, maxLength: 4000 }, tag: { type: 'string', minLength: 1, maxLength: 64 } }, required: ['text'], additionalProperties: false } },
-        { name: 'get_current', description: 'Read the task selected through the native Pinote UI for this conversation. Returns null if unselected.', inputSchema: empty, annotations: { readOnlyHint: true } },
+        { name: 'propose', description: 'Before multi-step implementation or configuration migrations, suggest a task if get_current returns null; skip informational questions and trivial edits. Show a short action-oriented title, optional details after a blank line, and reuse a saved tag from tags. The user clicks + to create/start/select or dismiss. Replaces an unaccepted suggestion; continue working without waiting or asking again. Never creates a task before a user action.', inputSchema: { type: 'object', properties: { text: { type: 'string', minLength: 1, maxLength: 4000 }, tag: { type: 'string', minLength: 1, maxLength: 64 } }, required: ['text'], additionalProperties: false } },
+        { name: 'get_current', description: 'Read the task selected through the native Pinote UI for this conversation before multi-step implementation or configuration migrations. Returns null if unselected; then use tags and propose to suggest a task without creating one. Skip informational questions and trivial edits; never replace an existing selection.', inputSchema: empty, annotations: { readOnlyHint: true } },
         { name: 'fields', description: 'Read global footer field definitions before setting task handoff values. Does not edit settings.', inputSchema: empty, annotations: { readOnlyHint: true } },
         { name: 'tags', description: 'List saved Pinote tags.', inputSchema: empty, annotations: { readOnlyHint: true } },
         { name: 'update_current', description: 'Merge Markdown handoff fields on the selected task. Read get_current first. Keep handoff notes concise; PR is one GitHub PR URL.', inputSchema: { type: 'object', properties: { expected_updated_at: { type: 'string' }, set: { type: 'object', additionalProperties: { type: 'string' } }, remove: { type: 'array', items: { type: 'string' } } }, required: ['expected_updated_at'], additionalProperties: false } },
@@ -374,4 +400,40 @@ server.setRequestHandler(CallToolRequestSchema, (request, extra) => serial(async
         return { isError: true, content: [{ type: 'text', text: String(error) }] };
     }
 }));
+// Every MCP process observes shared storage, including writes by CLI/GTK/other
+// sessions. Stat polling survives missing files and atomic config replacement.
+server.oninitialized = () => {
+    server.oninitialized = undefined;
+    const dataHome = process.env.XDG_DATA_HOME;
+    const database = join(dataHome && isAbsolute(dataHome) ? dataHome : join(homedir(), '.local/share'), 'pinote/notes.db');
+    const paths = [database, `${database}-wal`, join(getAgentDir(), 'pi-note.json'), receiptDir];
+    let timer;
+    let closed = false;
+    const changed = () => {
+        if (closed || timer)
+            return;
+        timer = setTimeout(() => {
+            timer = undefined;
+            void server.notification({ method: 'notifications/native-ui/changed', params: { uri: URI } }).catch((error) => {
+                if (!closed)
+                    console.error('Pinote UI update failed:', error);
+            });
+        }, 100);
+        timer.unref();
+    };
+    const listeners = paths.map((path) => {
+        const listener = (current, previous) => {
+            if (current.mtimeMs !== previous.mtimeMs || current.ctimeMs !== previous.ctimeMs || current.size !== previous.size || current.ino !== previous.ino || current.nlink !== previous.nlink)
+                changed();
+        };
+        watchFile(path, { persistent: false, interval: 1000 }, listener);
+        return listener;
+    });
+    server.onclose = () => {
+        closed = true;
+        if (timer)
+            clearTimeout(timer);
+        paths.forEach((path, index) => unwatchFile(path, listeners[index]));
+    };
+};
 await server.connect(new StdioServerTransport());

package/mcp/README.md

@@ -10,7 +10,7 @@ the built-in Pi MCP resource renderer. Use gateway 0.12.0+ with its Pi adapter.
 
 ## Setup
 
-Requires Node.js 22.19+, Python 3.11+, and `note` 0.4.0+. From the Pinote checkout:
+Requires Node.js 22.19+, Python 3.11+, and `note` 0.12.0+. From the Pinote checkout:
 
 ```bash
 uv sync --locked
@@ -77,13 +77,25 @@ have saved the session for resume persistence. Existing extension selections are
 not imported. Changing the gateway config path or service name starts a separate
 selection namespace.
 
+## Task-offer guidance
+
+Before multi-step implementation or configuration migrations, agents should call
+`get_current`. If it returns null, use `tags` and `propose` to suggest a short task
+with a saved tag, then continue without waiting. Skip informational questions and
+trivial edits. Never replace an existing selection or create/select without consent.
+
+The tool descriptions reinforce this workflow after discovery. For proactive
+discovery, include the same guidance in the client's always-loaded agent rules;
+tool descriptions alone cannot make an agent discover the service.
+
 ## Initial-version limits
 
 This is a working alternative, **not full parity with the old extension**. It does
 not retag/edit existing tasks or install/upgrade the CLI. Completion intentionally
-does not reset conversations. Task-offer policy and new-session prompt can
-be edited for compatibility with the existing extension, but this host does not
-yet implement their automation. Continue requests a separately confirmed draft;
+does not reset conversations. The legacy `taskOfferPolicy` and new-session prompt
+remain editable for native-extension compatibility; the MCP host does not apply
+them automatically. Agent-side task offers follow the client rules and tool
+descriptions above. Continue requests a separately confirmed draft;
 it never overwrites existing editor text or submits a prompt. Previews are display-only transcript entries (never model context), showing plain
 text/Markdown source limited to 20,000 characters; the task list shows at most 200
 filtered matches (fewer if the serialized display budget is reached). Use the CLI for full text and unsupported task operations.
@@ -94,9 +106,21 @@ to reject historical proposals. Do not delete them while old sessions may resume
 They are replay protection, not a second selection database or an authorization
 boundary against local processes.
 
-Changes through this service notify the host. External CLI/GTK/config changes need
-Ctrl+R or reopening the app. Failed actions are never retried automatically: a
-write might have committed before an interruption. Read fresh state before retrying.
+Each MCP process observes the shared database (including its WAL), settings, and
+proposal receipts with coalesced, one-second stat checks. External CLI/GTK/other-session
+changes refresh connected hosts; Ctrl+R or reopening the app remains a fallback.
+Watchers stop on disconnect and never invoke the CLI or change task data.
+
+Gateway 0.16.0+ negotiates per-control revisions: unrelated task-list changes no
+longer invalidate preview/navigation, task creation, settings saves, or completion
+of an unchanged selected task. Completion checks the displayed task revision;
+selection starts its target with an atomic revision check. Settings writes check
+the original file and resolved field definitions, including task-defined `Bar`
+fields. Older hosts retain full-view revision checks without extra control fields.
+
+Known stale-view rejections occur before writes and report **Action not applied**
+on supporting hosts. Failed actions are never retried automatically: other errors
+can follow a committed or partial write. Read fresh state before retrying.
 Trusted services/agents have ordinary OS permissions; UI-only routing is not a
 secrecy or authorization boundary.
 

package/mcp/server.ts

@@ -1,13 +1,14 @@
 import { createHash, randomUUID } from 'node:crypto';
 import { promises as fs } from 'node:fs';
 import { homedir } from 'node:os';
-import { join } from 'node:path';
+import { isAbsolute, join } from 'node:path';
 import { execFile } from 'node:child_process';
 import { promisify } from 'node:util';
-import { readFileSync } from 'node:fs';
+import { readFileSync, watchFile, unwatchFile } from 'node:fs';
+import { getAgentDir } from '../config-path.ts';
 import { Server } from '@modelcontextprotocol/sdk/server/index.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
-import { CallToolRequestSchema, ListToolsRequestSchema, ListResourcesRequestSchema, ReadResourceRequestSchema } from '@modelcontextprotocol/sdk/types.js';
+import { CallToolRequestSchema, ListToolsRequestSchema, ListResourcesRequestSchema, ReadResourceRequestSchema, McpError } from '@modelcontextprotocol/sdk/types.js';
 import { z } from 'zod';
 import { defaultFooterField, defaultHandoffPrompt, defaultNewSessionPrompt, effectiveFooterFields, parseFooterConfig, parseHandoffPrompt, parseNewSessionPrompt, readFooterDocument, saveFooterConfig } from '../footer-config.ts';
 
@@ -26,12 +27,12 @@ const Context = z.object({ conversationId: z.string().min(1).max(256), contextId
 type State = z.infer<typeof Context>['state'];
 type Task = { id: number; text: string; state: string; tag: string | null; updated_at: string; agent_notes: Record<string, string> };
 type Scalar = string | number | boolean | null;
-type Control = { id: string; label: string; kind: 'action' | 'text' | 'number' | 'toggle' | 'select'; value?: Scalar; icon?: string; description?: string; multiline?: boolean; confirm?: string; options?: { label: string; value: Scalar }[] };
+type Control = { id: string; label: string; revision?: string; kind: 'action' | 'text' | 'number' | 'toggle' | 'select'; value?: Scalar; icon?: string; description?: string; multiline?: boolean; confirm?: string; options?: { label: string; value: Scalar }[] };
 const Action = z.object({ method: z.literal('native-ui/action'), params: z.object({
   uri: z.literal(URI), revision: z.string().min(1).max(256), action: z.string().min(1).max(256),
   value: z.union([z.string().max(20_000), z.number().finite(), z.boolean(), z.null()]).optional(), _meta: z.record(z.unknown()).optional(),
 }) });
-const server = new Server({ name: 'pinote', version: '0.20.0' }, { capabilities: { tools: {}, resources: {}, experimental: { 'native-ui-v1': {} } } });
+const server = new Server({ name: 'pinote', version: '0.21.0' }, { capabilities: { tools: {}, resources: {}, experimental: { 'native-ui-v1': {} } } });
 // Serialize complete read/check/write cycles even when a host sends parallel calls.
 let queue = Promise.resolve();
 function serial<T>(run: () => Promise<T>): Promise<T> {
@@ -63,7 +64,7 @@ async function note(args: string[], signal: AbortSignal): Promise<string> {
   const options = { signal, timeout: 10_000, maxBuffer: 4 * 1024 * 1024 };
   const version = (await exec(noteCommand, ['--version'], options)).stdout.trim();
   const match = /^pinote (\d+)\.(\d+)\.(\d+)$/.exec(version);
-  if (!match || Number(match[1]) === 0 && Number(match[2]) < 4) throw new Error('Pinote MCP requires note 0.4.0+');
+  if (!match || Number(match[1]) === 0 && Number(match[2]) < 12) throw new Error('Pinote MCP requires note 0.12.0+');
   return (await exec(noteCommand, ['--no-notify', ...args], options)).stdout;
 }
 async function selected(state: State, signal: AbortSignal): Promise<Task | null> {
@@ -98,7 +99,23 @@ async function snapshot(state: State, signal: AbortSignal) {
   return { state, tasks, task, config, revision };
 }
 type Snapshot = Awaited<ReturnType<typeof snapshot>>;
-function view(data: Snapshot) {
+function scopedActions(meta: Record<string, unknown> | undefined): boolean {
+  return z.object({ actionRevisions: z.literal(true) }).safeParse(meta?.['native-ui/features']).success;
+}
+function actionRevision(data: Snapshot, control: Control): string {
+  const { id, revision: _revision, ...semantics } = control;
+  let dependency: unknown = null;
+  if (id === 'done') dependency = data.task && { id: data.task.id, updated_at: data.task.updated_at };
+  else if (id.startsWith('select:')) dependency = data.tasks.find((task) => task.id === Number(id.slice(7)));
+  else if (!['preview', 'continue', 'clear', 'search', 'add', 'proposal:accept', 'proposal:dismiss'].includes(id) && !id.startsWith('screen:') && !id.startsWith('field:')) {
+    dependency = { raw: data.config.raw, ...(['add-field', 'remove-field', 'label', 'format', 'width', 'link'].includes(id) ? { fields: fields(data.config.config, data.task?.agent_notes) } : {}) };
+  }
+  return createHash('sha256').update(JSON.stringify({ action: id, semantics, state: data.state, dependency })).digest('hex');
+}
+function staleAction(): never {
+  throw new McpError(-33003, 'View changed; refresh before acting', { 'native-ui/action': { status: 'not-applied', reason: 'stale-view' } });
+}
+function view(data: Snapshot, scoped = false) {
   const { tasks, task, config, revision } = data;
   const state = Context.parse({ conversationId: 'outgoing-state', state: { ...data.state, selectedId: task?.id ?? null } }).state;
   const controls: Control[] = [];
@@ -173,6 +190,7 @@ function view(data: Snapshot) {
     body = `${control.label} exceeds the editor limit; edit it outside this UI.\n${body}`;
     return false;
   });
+  if (scoped) for (const control of editable) control.revision = actionRevision(data, control);
   const result = { version: 1, title, body: body.slice(0, 20_000), revision, state: boundedState(state), retainOnNew: true, controls: editable, footer };
   // JSON escaping and expanded footer formats can exceed the host budget even
   // with bounded row counts. Preserve navigation/filter controls ahead of rows.
@@ -190,14 +208,15 @@ server.setRequestHandler(ListResourcesRequestSchema, async () => ({ resources: [
 server.setRequestHandler(ReadResourceRequestSchema, (request, extra) => serial(async () => {
   if (request.params.uri !== URI) throw new Error('Unknown resource');
   const data = await snapshot(context(request.params._meta).state, extra.signal);
-  return { contents: [{ uri: URI, mimeType: MIME, text: JSON.stringify(view(data)) }] };
+  return { contents: [{ uri: URI, mimeType: MIME, text: JSON.stringify(view(data, scopedActions(request.params._meta))) }] };
 }));
 server.setRequestHandler(Action, (request, extra) => serial(async () => {
   const state = context(request.params._meta).state;
   const before = await snapshot(state, extra.signal);
-  if (before.revision !== request.params.revision) throw new Error('View changed; refresh before acting');
+  const scoped = scopedActions(request.params._meta);
   const { action, value } = request.params;
-  if (!view(before).controls.some((c) => c.id === action)) throw new Error('Action is not available in this view');
+  const control = view(before, scoped).controls.find((item) => item.id === action);
+  if (!control || (control.revision ?? before.revision) !== request.params.revision) staleAction();
   const next: State = { ...before.state };
   let draft: string | undefined, preview: string | undefined;
   if (action === 'proposal:accept' || action === 'proposal:dismiss') {
@@ -207,7 +226,7 @@ server.setRequestHandler(Action, (request, extra) => serial(async () => {
     next.proposal = null;
     if (action === 'proposal:accept') {
       const task: Task = JSON.parse(await note(['agent', 'add', `--text=${proposal.text}`, ...(proposal.tag ? [`--tag=${proposal.tag}`] : [])], extra.signal));
-      await note(['start', String(task.id)], extra.signal); next.selectedId = task.id;
+      await note(['agent', 'start', String(task.id), '--expected-updated-at', task.updated_at], extra.signal); next.selectedId = task.id;
     }
   } else if (action.startsWith('screen:')) next.screen = action.slice(7) as State['screen'];
   else if (action.startsWith('field:')) { next.screen = 'field'; next.field = action.slice(6); }
@@ -217,14 +236,17 @@ server.setRequestHandler(Action, (request, extra) => serial(async () => {
   else if (action === 'preview') preview = (before.task!.text + '\n\n' + JSON.stringify(before.task!.agent_notes, null, 2)).slice(0, 20_000);
   else if (action.startsWith('select:')) {
     const id = Number(action.slice(7));
-    if (!before.tasks.some((t) => t.id === id)) throw new Error('Task no longer selectable');
+    const candidate = before.tasks.find((task) => task.id === id);
+    if (!candidate) staleAction();
+    const target: Task = JSON.parse(await note(['agent', 'get', String(id)], extra.signal));
+    if (target.text !== candidate.text || target.state !== candidate.state || target.tag !== candidate.tag) staleAction();
     if (next.proposal) await consume(next.proposal.id);
-    await note(['start', String(id)], extra.signal); next.selectedId = id; next.proposal = null;
+    await note(['agent', 'start', String(id), '--expected-updated-at', target.updated_at], extra.signal); next.selectedId = id; next.proposal = null;
   } else if (action === 'add') {
     if (next.proposal) await consume(next.proposal.id);
     next.proposal = null;
     const task: Task = JSON.parse(await note(['agent', 'add', `--text=${z.string().min(1).parse(value)}`], extra.signal));
-    await note(['start', String(task.id)], extra.signal); next.selectedId = task.id;
+    await note(['agent', 'start', String(task.id), '--expected-updated-at', task.updated_at], extra.signal); next.selectedId = task.id;
   } else if (action === 'done') {
     await note(['agent', 'done', String(before.task!.id), '--expected-updated-at', before.task!.updated_at], extra.signal);
     next.selectedId = null;
@@ -250,14 +272,14 @@ server.setRequestHandler(Action, (request, extra) => serial(async () => {
     extra.signal.throwIfAborted();
     saveFooterConfig(cfg, before.config.raw, handoff, newSession, policy);
   }
-  const result = { view: view(await snapshot(next, extra.signal)), ...(draft ? { draft } : {}), ...(preview ? { preview } : {}) };
+  const result = { view: view(await snapshot(next, extra.signal), scoped), ...(draft ? { draft } : {}), ...(preview ? { preview } : {}) };
   await server.notification({ method: 'notifications/native-ui/changed', params: { uri: URI } });
   return result;
 }));
 const empty = { type: 'object' as const, properties: {}, additionalProperties: false };
 server.setRequestHandler(ListToolsRequestSchema, async () => ({ tools: [
-  { name: 'propose', description: 'Suggest a task without creating it. Requires no selected task. Show a short action-oriented title, optional details after a blank line, and reuse a saved tag. The user clicks + to create/start/select or dismiss. Replaces an unaccepted suggestion; continue working without asking again. Never creates a task before a user action.', inputSchema: { type: 'object' as const, properties: { text: { type: 'string', minLength: 1, maxLength: 4000 }, tag: { type: 'string', minLength: 1, maxLength: 64 } }, required: ['text'], additionalProperties: false } },
-  { name: 'get_current', description: 'Read the task selected through the native Pinote UI for this conversation. Returns null if unselected.', inputSchema: empty, annotations: { readOnlyHint: true } },
+  { name: 'propose', description: 'Before multi-step implementation or configuration migrations, suggest a task if get_current returns null; skip informational questions and trivial edits. Show a short action-oriented title, optional details after a blank line, and reuse a saved tag from tags. The user clicks + to create/start/select or dismiss. Replaces an unaccepted suggestion; continue working without waiting or asking again. Never creates a task before a user action.', inputSchema: { type: 'object' as const, properties: { text: { type: 'string', minLength: 1, maxLength: 4000 }, tag: { type: 'string', minLength: 1, maxLength: 64 } }, required: ['text'], additionalProperties: false } },
+  { name: 'get_current', description: 'Read the task selected through the native Pinote UI for this conversation before multi-step implementation or configuration migrations. Returns null if unselected; then use tags and propose to suggest a task without creating one. Skip informational questions and trivial edits; never replace an existing selection.', inputSchema: empty, annotations: { readOnlyHint: true } },
   { name: 'fields', description: 'Read global footer field definitions before setting task handoff values. Does not edit settings.', inputSchema: empty, annotations: { readOnlyHint: true } },
   { name: 'tags', description: 'List saved Pinote tags.', inputSchema: empty, annotations: { readOnlyHint: true } },
   { name: 'update_current', description: 'Merge Markdown handoff fields on the selected task. Read get_current first. Keep handoff notes concise; PR is one GitHub PR URL.', inputSchema: { type: 'object' as const, properties: { expected_updated_at: { type: 'string' }, set: { type: 'object', additionalProperties: { type: 'string' } }, remove: { type: 'array', items: { type: 'string' } } }, required: ['expected_updated_at'], additionalProperties: false } },
@@ -286,4 +308,36 @@ server.setRequestHandler(CallToolRequestSchema, (request, extra) => serial(async
     return { content: [{ type: 'text', text: JSON.stringify(result) }], ...(outgoing ? { _meta: { 'native-ui/state': outgoing } } : {}) };
   } catch (error) { return { isError: true, content: [{ type: 'text', text: String(error) }] }; }
 }));
+// Every MCP process observes shared storage, including writes by CLI/GTK/other
+// sessions. Stat polling survives missing files and atomic config replacement.
+server.oninitialized = () => {
+  server.oninitialized = undefined;
+  const dataHome = process.env.XDG_DATA_HOME;
+  const database = join(dataHome && isAbsolute(dataHome) ? dataHome : join(homedir(), '.local/share'), 'pinote/notes.db');
+  const paths = [database, `${database}-wal`, join(getAgentDir(), 'pi-note.json'), receiptDir];
+  let timer: ReturnType<typeof setTimeout> | undefined;
+  let closed = false;
+  const changed = () => {
+    if (closed || timer) return;
+    timer = setTimeout(() => {
+      timer = undefined;
+      void server.notification({ method: 'notifications/native-ui/changed', params: { uri: URI } }).catch((error) => {
+        if (!closed) console.error('Pinote UI update failed:', error);
+      });
+    }, 100);
+    timer.unref();
+  };
+  const listeners = paths.map((path) => {
+    const listener = (current: import('node:fs').Stats, previous: import('node:fs').Stats) => {
+      if (current.mtimeMs !== previous.mtimeMs || current.ctimeMs !== previous.ctimeMs || current.size !== previous.size || current.ino !== previous.ino || current.nlink !== previous.nlink) changed();
+    };
+    watchFile(path, { persistent: false, interval: 1000 }, listener);
+    return listener;
+  });
+  server.onclose = () => {
+    closed = true;
+    if (timer) clearTimeout(timer);
+    paths.forEach((path, index) => unwatchFile(path, listeners[index]));
+  };
+};
 await server.connect(new StdioServerTransport());

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-note",
-  "version": "0.20.0",
+  "version": "0.21.0",
   "description": "Persistent pinote tasks, Markdown handoffs, and task status for Pi",
   "license": "MIT",
   "type": "module",