openwriter 0.35.1 → 0.36.0

package/dist/server/state.js

@@ -1596,7 +1596,27 @@ export function setPendingCacheEntry(filename, count) {
         pendingDocCache.delete(filename);
     }
 }
-/** Populate the pending cache from a full disk scan. Called once on startup. */
+/** Pending count for one doc: sidecar entries + staged title rename (the
+ *  sidecar is authoritative since the overlay migration; legacy in-frontmatter
+ *  `pending:` is the fallback for docs that predate it).
+ *  adr: adr/pending-overlay-model.md */
+function pendingCountForDoc(docId, legacyPending) {
+    let count = 0;
+    if (docId) {
+        count += loadOverlay(docId).length;
+        if (loadPendingMetadata(docId))
+            count += 1;
+    }
+    if (count === 0 && legacyPending && Object.keys(legacyPending).length > 0) {
+        count = Object.keys(legacyPending).length;
+    }
+    return count;
+}
+/** Populate the pending cache from a full disk scan. Called once on startup.
+ *  Reads the `_pending/{docId}.json` sidecars (the authoritative store) —
+ *  scanning only legacy frontmatter here made every restart look like a
+ *  profile-wide accept-all, because the sidebar/review pending list came back
+ *  empty even though all sidecars survived on disk. */
 function populatePendingCache() {
     pendingDocCache.clear();
     try {
@@ -1605,9 +1625,9 @@ function populatePendingCache() {
             try {
                 const raw = readFileSync(join(getDataDir(), f), 'utf-8');
                 const { data } = matter(raw);
-                if (data.pending && Object.keys(data.pending).length > 0) {
-                    pendingDocCache.set(f, Object.keys(data.pending).length);
-                }
+                const count = pendingCountForDoc(data.docId, data.pending);
+                if (count > 0)
+                    pendingDocCache.set(f, count);
             }
             catch { /* skip unreadable files */ }
         }
@@ -1620,9 +1640,9 @@ function populatePendingCache() {
                 continue;
             const raw = readFileSync(extPath, 'utf-8');
             const { data } = matter(raw);
-            if (data.pending && Object.keys(data.pending).length > 0) {
-                pendingDocCache.set(extPath, Object.keys(data.pending).length);
-            }
+            const count = pendingCountForDoc(data.docId, data.pending);
+            if (count > 0)
+                pendingDocCache.set(extPath, count);
         }
         catch { /* skip unreadable files */ }
     }

package/dist/server/title-resolve.js

@@ -0,0 +1,87 @@
+/**
+ * Listing-time title resolution for documents that haven't been opened yet.
+ *
+ * Markdown files dropped into the profile directory externally (no
+ * frontmatter) used to render as "Untitled" in the sidebar and in MCP
+ * list_documents until first opened — opening injects the frontmatter title
+ * that listing relied on exclusively. This module fixes the class: title
+ * resolution at read time falls back through
+ *
+ *   frontmatter title → workspace JSON entry title → first h1 in body → filename stem
+ *
+ * This is deliberately fallback-only — no lazy frontmatter injection on the
+ * listing path. Listing is a read; writing to every titleless file on index
+ * would mutate user-dropped files behind their back (surprising for git and
+ * external editors, and racy against an editor mid-write). Frontmatter is
+ * still injected on first open/save, exactly as before — at which point the
+ * resolved title is what gets persisted.
+ */
+import { readFileSync, readdirSync } from 'fs';
+import { join, basename } from 'path';
+import { getWorkspacesDir, TEMP_PREFIX } from './helpers.js';
+function isRealTitle(t) {
+    return typeof t === 'string' && t.trim() !== '' && t !== 'Untitled';
+}
+/** First ATX h1 in the body, with trailing closing hashes stripped. */
+export function firstH1(content) {
+    const m = content.match(/^#[ \t]+(.+?)[ \t]*#*[ \t]*$/m);
+    return m ? m[1].trim() || null : null;
+}
+export function resolveListingTitle(src) {
+    if (isRealTitle(src.fmTitle))
+        return src.fmTitle;
+    if (isRealTitle(src.workspaceTitle))
+        return src.workspaceTitle;
+    if (src.content) {
+        const h1 = firstH1(src.content);
+        if (h1)
+            return h1;
+    }
+    if (src.filename) {
+        const stem = basename(src.filename).replace(/\.md$/i, '');
+        // Temp files are genuinely new/unnamed docs — their generated stem is
+        // not a title; keep them "Untitled" so auto-titling still owns them.
+        if (stem && !basename(src.filename).startsWith(TEMP_PREFIX))
+            return stem;
+    }
+    return 'Untitled';
+}
+/**
+ * Map of doc file identifier → title from workspace JSON entries.
+ * Read-only raw scan of _workspaces/*.json — intentionally does NOT go
+ * through readWorkspace(), which performs migrations and writes back.
+ * First entry wins when a doc appears in multiple workspaces.
+ */
+export function getWorkspaceTitleMap() {
+    const map = new Map();
+    let files = [];
+    try {
+        files = readdirSync(getWorkspacesDir()).filter((f) => f.endsWith('.json') && !f.startsWith('_'));
+    }
+    catch {
+        return map;
+    }
+    for (const f of files) {
+        try {
+            const ws = JSON.parse(readFileSync(join(getWorkspacesDir(), f), 'utf-8'));
+            walkNodes(Array.isArray(ws.root) ? ws.root : [], map);
+        }
+        catch { /* skip unreadable workspace */ }
+    }
+    return map;
+}
+function walkNodes(nodes, map) {
+    for (const node of nodes) {
+        if (!node || typeof node !== 'object')
+            continue;
+        if (node.type === 'doc' && typeof node.file === 'string') {
+            if (isRealTitle(node.title) && !map.has(node.file))
+                map.set(node.file, node.title);
+            if (Array.isArray(node.children))
+                walkNodes(node.children, map);
+        }
+        else if (node.type === 'container' && Array.isArray(node.items)) {
+            walkNodes(node.items, map);
+        }
+    }
+}

package/dist/server/workspaces.js

@@ -10,6 +10,7 @@ import matter from 'gray-matter';
 import trash from 'trash';
 import { getWorkspacesDir, ensureWorkspacesDir, sanitizeFilename, resolveDocPath, isExternalDoc } from './helpers.js';
 import { markdownToTiptap, tiptapToMarkdown } from './markdown.js';
+import { resolveListingTitle } from './title-resolve.js';
 function getOrderFile() { return join(getWorkspacesDir(), '_order.json'); }
 import { isV1, migrateV1toV2 } from './workspace-types.js';
 import { addDocToContainer, addContainer as addContainerToTree, removeNode, moveNode, reorderNode, findContainer, collectAllFiles, countDocs, findDocNode } from './workspace-tree.js';
@@ -588,10 +589,15 @@ export function collectFilesInContainer(wsFile, containerId) {
 export function getWorkspaceStructure(filename) {
     return getWorkspace(filename);
 }
-/** Read the frontmatter title for a doc file. Falls back to filename without extension. */
+/** Read the title for a doc file: frontmatter → first h1 in body → filename stem. */
 export function getDocTitle(filename) {
-    const fm = readDocFrontmatter(filename);
-    if (fm?.title && fm.title !== 'Untitled')
-        return fm.title;
+    try {
+        const filePath = resolveDocPath(filename);
+        if (existsSync(filePath)) {
+            const { data, content } = matter(readFileSync(filePath, 'utf-8'));
+            return resolveListingTitle({ fmTitle: data.title, content, filename });
+        }
+    }
+    catch { /* fall through to stem */ }
     return filename.replace(/\.md$/, '');
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openwriter",
-  "version": "0.35.1",
+  "version": "0.36.0",
   "description": "The open-source writing surface for AI agents. Markdown-native editor with pending change review — your agent writes, you accept or reject.",
   "type": "module",
   "license": "MIT",