dsh-loci 3.0.1 → 3.2.0

package/CHANGELOG.md

@@ -4,6 +4,77 @@ All notable changes to this project are documented here. The format follows
 [Keep a Changelog](https://keepachangelog.com/en/1.1.0/); the project follows
 [Semantic Versioning](https://semver.org/spec/v2.0.0.html) from 1.0.0.
 
+## [3.2.0] — 2026-11-15
+
+The sidebar was sending people to an empty room. Found in the live store, not in a test.
+
+### Fixed
+
+- **"N item(s) waiting for you in Review" counted rows Review hides.** The card read
+  `stats.pendingCandidates`, which counted every candidate and disputed row in scope; the
+  Review tab lists only the kinds that can reach the prompt (`rule`, `semantic`,
+  `procedural`). Episodic session summaries are candidates *by design* — they are history,
+  not claims — so on a normal day the entire count was composed of rows Review deliberately
+  excludes.
+
+  Live evidence, after one afternoon of use: **3 pending rows, all `episodic`**, and the
+  sidebar offering to review them somewhere it would not. The count is now `0` and Review
+  shows `0`.
+
+  `REVIEWABLE_KINDS` is one exported list. `countPending`, `stats.pendingCandidates`,
+  `stats.disputed` and the panel's request all use it, so the badge and the list cannot drift
+  apart again. `byStatus` stays complete — that is a data view, not a to-do.
+
+- **`stats.pendingCandidates` ignored `valid_to`** while `countPending` honoured it, so a
+  candidate closed by maintenance stayed in the badge. Both now exclude expired rows.
+
+### Why this matters beyond the cosmetic
+
+The card was telling the user to do something that could not be done, which reads as "the
+tool is broken" rather than "there is nothing to do". A number with a door is a promise.
+
+### Not fixed, recorded
+
+Episodic summaries still accumulate as candidates and are only removed by the 30-day
+candidate TTL. They are visible in the Feed tab, which is where history belongs, but nothing
+says so. That is part of R24 (the panel explains almost nothing about itself).
+
+## [3.1.0] — 2026-11-15
+
+Four things from looking at the live sidebar and panel.
+
+### Changed
+
+- **The sidebar icon now has the same weight as the menu it sits in.** Native
+  `sidebar.panellist` entries receive `{ size, active }`; this one used `size` correctly but
+  drew inside 4–20 of a 24 viewBox with a small off-centre dot, so beside the shipped icons
+  it read thinner and smaller. Redrawn as a symmetric **layers** glyph spanning 3–21 at
+  stroke 1.8 — the optical weight of its neighbours, and a shape that says layered memory.
+
+- **The panel is a readable column, not edge-to-edge text.** The scroll container's
+  horizontal padding became `max(16px, calc((100% - 860px) / 2))`, so rows sit in a centred
+  measure on a wide window without touching any tab component.
+
+- **"N item(s) waiting for you in Review" is now a way in.** It was a sentence telling a
+  person to go somewhere else and find the tab. It is a button: it opens the panel already
+  on Review. The card and the panel mount separately, so the request travels through module
+  scope and is consumed once.
+
+- **The card says what it knows, not only how much.** Commit `4065c98` (2026-11-13) recorded
+  the brief for this surface: *"not a dashboard … read the card and think 'it got that, it
+  recorded that, we agreed on that'"*, with one short paragraph of what the agent
+  understands in this project and pending work as one quiet line, never a number. 3.0.0
+  replaced that paragraph with counters. The counters stay — they were asked for and they
+  are useful — and one line of content goes back under them: a `Latest` row showing the most
+  recently trusted entry, from `GET /api/loci/knowledge?limit=1`.
+
+### Not changed
+
+- `docs/PRIOR-ART.md` still records that the card should be scoped to **this project's read
+  set** rather than every scope. It currently is not: `/stats` and `/knowledge` called
+  without parameters enumerate the whole store, and the sidebar card has no access to the
+  session's cwd. Scoping the card is a real open item, not something to assert.
+
 ## [3.0.1] — 2026-11-15
 
 The 3.0.0 card was correct markup with no stylesheet, and the screenshot showed it.

package/client.js

@@ -27,6 +27,9 @@ window.__ModuleLoader__.load({
     const API = '/api/loci';
     const ENTRY_ID = 'dsh-loci';
     const PANEL_ID = 'loci';
+    // The card and the panel mount separately, so a request to open a specific tab has
+    // to travel through module scope. The panel consumes it once, then clears it.
+    let requestedTab;
 
     const EN = {
       'meta.title': 'Loci memory palace',
@@ -44,6 +47,7 @@ window.__ModuleLoader__.load({
       'card.opened': 'Opened',
       'card.dead': 'Dead',
       'card.pendingHint': '{n} item(s) waiting for you in Review.',
+      'card.latest': 'Latest',
       'card.updated': 'updated',
       'card.pendingSentence': '{n} still need your review',
       'card.pinned': 'pinned',
@@ -194,6 +198,11 @@ window.__ModuleLoader__.load({
 .loci-card-foot-grow { flex: 1 1 auto; }
 .loci-card-empty { font-size: 9.5px; color: var(--dsw-alias-label-secondary); }
 .loci-card-hint { font-size: 9px; color: var(--dsw-alias-label-secondary); }
+.loci-card-known { display: flex; gap: 5px; align-items: baseline; min-width: 0; }
+.loci-card-known-tag { flex: 0 0 auto; font-size: 8px; letter-spacing: .05em; text-transform: uppercase; color: var(--dsw-alias-label-secondary); }
+.loci-card-known-text { min-width: 0; font-size: 9.5px; color: var(--dsw-alias-label-primary); overflow: hidden; display: -webkit-box; -webkit-line-clamp: 2; -webkit-box-orient: vertical; }
+.loci-card-hint-action { align-self: flex-start; margin: 0; padding: 2px 6px; border: 1px solid var(--dsw-alias-border-l1); border-radius: 6px; background: var(--dsw-alias-bg-layer-2); color: var(--dsw-alias-label-primary); font: inherit; font-size: 9px; line-height: 1.2; cursor: pointer; text-align: left; }
+.loci-card-hint-action:hover { border-color: var(--dsw-alias-brand-primary); color: var(--dsw-alias-brand-primary); }
 .loci-dot { width: 6px; height: 6px; border-radius: 50%; background: var(--dsw-alias-brand-primary); flex: 0 0 auto; }
 
 .loci-bar { display: flex; align-items: center; gap: 12px; padding: 10px 16px; border-bottom: 1px solid var(--dsw-alias-border-l1); }
@@ -203,7 +212,7 @@ window.__ModuleLoader__.load({
 .loci-tab:hover { background: var(--dsw-alias-bg-layer-2); color: var(--dsw-alias-label-primary); }
 .loci-tab.is-active { background: var(--dsw-alias-bg-layer-2); color: var(--dsw-alias-label-primary); font-weight: 600; }
 .loci-grow { flex: 1 1 auto; }
-.loci-body { flex: 1 1 auto; min-height: 0; overflow: auto; padding: 12px 16px 24px; }
+.loci-body { flex: 1 1 auto; min-height: 0; overflow: auto; padding: 14px max(16px, calc((100% - 860px) / 2)) 28px; }
 .loci-btn { border: 1px solid var(--dsw-alias-border-l1); background: var(--dsw-alias-bg-layer-1); color: var(--dsw-alias-label-primary); border-radius: 6px; padding: 3px 9px; font: inherit; cursor: pointer; }
 .loci-btn:hover { border-color: var(--dsw-alias-border-l2); background: var(--dsw-alias-bg-layer-2); }
 .loci-btn:disabled { opacity: .5; cursor: default; }
@@ -438,6 +447,7 @@ window.__ModuleLoader__.load({
       // read as bare text inside the sidebar rather than as a card — and the audit's own
       // "no dashboard" objection was about refusing to interrupt, not about hiding data.
       const stats = useJson('/stats', { intervalMs: 30_000 });
+      const knowledge = useJson('/knowledge?limit=1', { intervalMs: 30_000 });
       const config = useJson('/config');
       const update = useJson('/update');
       const [checking, setChecking] = useState(false);
@@ -468,6 +478,10 @@ window.__ModuleLoader__.load({
       const injected = usage.reduce((total, row) => total + Number(row.injections ?? 0), 0);
       const opened = usage.reduce((total, row) => total + Number(row.reads ?? 0), 0);
       const dead = usage.filter((row) => Number(row.injections ?? 0) > 0 && Number(row.reads ?? 0) === 0).length;
+      // What the agent currently understands, newest first. Numbers answer "how much";
+      // this answers "what".
+      const top = knowledge.data?.items?.[0];
+      const latest = typeof top?.title === 'string' && top.title.length > 0 ? top.title : null;
 
       if (!wide) {
         return h(
@@ -548,6 +562,14 @@ window.__ModuleLoader__.load({
                   metric(t('card.pending'), pending, pending > 0 ? 'is-warn' : ''),
                   metric(t('card.disputed'), disputed, disputed > 0 ? 'is-error' : ''),
                 ),
+        data !== undefined && latest !== null
+          ? h(
+              'div',
+              { className: 'loci-card-known' },
+              h('span', { className: 'loci-card-known-tag' }, t('card.latest')),
+              h('span', { className: 'loci-card-known-text' }, latest),
+            )
+          : null,
         data !== undefined && entries > 0 && budget > 0
           ? h(
               'div',
@@ -573,7 +595,21 @@ window.__ModuleLoader__.load({
               h('span', { className: 'loci-card-foot-grow' }),
             )
           : null,
-        pending > 0 ? h('div', { className: 'loci-card-hint' }, t('card.pendingHint').replace('{n}', String(pending))) : null,
+        pending > 0
+          ? h(
+              'button',
+              {
+                type: 'button',
+                className: 'loci-card-hint loci-card-hint-action',
+                onClick: (event) => {
+                  event.stopPropagation();
+                  requestedTab = 'review';
+                  open?.();
+                },
+              },
+              t('card.pendingHint').replace('{n}', String(pending)),
+            )
+          : null,
       );
     }
     // ---------------------------------------------------------------- panel
@@ -919,6 +955,7 @@ window.__ModuleLoader__.load({
       // Only the kinds that could actually reach the prompt are reviewable. An
       // episodic session summary is history: it is never injected, so parking it
       // here forever is how the queue became a wall nobody reads.
+      // The same three kinds the store counts as pending, so the badge and the list agree.
       const reviewable = 'kinds=rule,semantic,procedural';
       const candidates = useJson(`/list?status=candidate&limit=50&${reviewable}`);
       const disputed = useJson(`/list?status=disputed&limit=50&${reviewable}`);
@@ -1136,7 +1173,12 @@ window.__ModuleLoader__.load({
     }
 
     function MemoryPanel({ t }) {
-      const [tab, setTab] = useState('feed');
+      const [tab, setTab] = useState(() => {
+        // "N item(s) waiting for you in Review" in the card is a way in, not a remark.
+        const wanted = requestedTab;
+        requestedTab = undefined;
+        return wanted ?? 'feed';
+      });
       const [selected, setSelected] = useState(undefined);
       const [nonce, setNonce] = useState(0);
       const tabs = useMemo(
@@ -1180,6 +1222,10 @@ window.__ModuleLoader__.load({
     }
 
     function PanelIcon({ size, active }) {
+      // Three stacked sheets: the product is a layered index, and the glyph has to read at
+      // the same optical weight as the shipped icons beside it. The previous drawing lived
+      // in 4–20 of the viewBox with a small off-centre dot, which made it look thinner and
+      // smaller than its neighbours in the sidebar.
       return h(
         'svg',
         {
@@ -1188,10 +1234,28 @@ window.__ModuleLoader__.load({
           viewBox: '0 0 24 24',
           fill: 'none',
           'aria-hidden': true,
-          style: { opacity: active ? 1 : 0.75, display: 'block' },
+          style: { opacity: active ? 1 : 0.72, display: 'block' },
         },
-        h('path', { d: 'M4 6.5h16M4 12h16M4 17.5h10', stroke: 'currentColor', strokeWidth: 1.7, strokeLinecap: 'round' }),
-        h('circle', { cx: 18.5, cy: 17.5, r: 2.2, stroke: 'currentColor', strokeWidth: 1.5 }),
+        h('path', {
+          d: 'M12 3 21 7.5 12 12 3 7.5 12 3Z',
+          stroke: 'currentColor',
+          strokeWidth: 1.8,
+          strokeLinejoin: 'round',
+        }),
+        h('path', {
+          d: 'M3.6 12.2 12 16.4l8.4-4.2',
+          stroke: 'currentColor',
+          strokeWidth: 1.8,
+          strokeLinecap: 'round',
+          strokeLinejoin: 'round',
+        }),
+        h('path', {
+          d: 'M3.6 16.8 12 21l8.4-4.2',
+          stroke: 'currentColor',
+          strokeWidth: 1.8,
+          strokeLinecap: 'round',
+          strokeLinejoin: 'round',
+        }),
       );
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-loci",
-  "version": "3.0.1",
+  "version": "3.2.0",
   "description": "Layered long-term memory for DeepSeek Harness: the prompt sees only a stable index, full text is one lookup away. Bi-temporal facts, strict scope isolation, verified-only injection.",
   "type": "module",
   "license": "MIT",

package/src/host/index.js

@@ -23,7 +23,7 @@ import { registerCommands } from './commands.js';
 import { runDoctor } from './doctor.js';
 
 /** Version stamped into the database and diagnostics. */
-export const VERSION = '3.0.1';
+export const VERSION = '3.2.0';
 
 /** Optional collaborators; without them the plugin still serves tools. */
 export const inject = ['systemPrompt', 'tools'];

package/src/host/store/store.js

@@ -118,6 +118,13 @@ function tierOf(kind, pinned) {
 /** Statuses that may be injected or returned by default. */
 export const INJECTABLE_STATUSES = Object.freeze(['verified']);
 
+/**
+ * The kinds a person can review. Everything else — episodic summaries, working notes — is
+ * either history or scratch, and Review hides it. One list, used by the counter and the UI,
+ * so the number and the list cannot drift apart again.
+ */
+export const REVIEWABLE_KINDS = Object.freeze(['rule', 'semantic', 'procedural']);
+
 /** The phrase a caller must echo before the whole store is forgotten. */
 export const CONFIRM_ALL = 'forget everything';
 
@@ -877,18 +884,25 @@ export class Store {
    * @param scopeKeys - explicit scope keys (never a wildcard).
    * @returns the count of candidate and disputed entries.
    */
-  countPending(scopeKeys, now = this.now()) {
+  countPending(scopeKeys, options = {}) {
     if (scopeKeys.length === 0) return 0;
+    const now = options.now ?? this.now();
+    // Only rows a person can actually act on. Episodic summaries are candidates by design —
+    // they are history, not claims, and Review hides them — so counting them made the
+    // sidebar announce work with nowhere to do it.
+    const kinds = Array.isArray(options.kinds) && options.kinds.length > 0 ? options.kinds : REVIEWABLE_KINDS;
     const marks = scopeKeys.map(() => '?').join(', ');
+    const kindMarks = kinds.map(() => '?').join(', ');
     return Number(
       this.db
         .prepare(
           `SELECT COUNT(*) AS n FROM memory m
            WHERE m.status IN ('candidate', 'disputed') AND m.deleted_at IS NULL
              AND (m.valid_to IS NULL OR m.valid_to > ?)
+             AND m.kind IN (${kindMarks})
              AND m.scope_key IN (${marks})`,
         )
-        .get(now, ...scopeKeys)?.n ?? 0,
+        .get(now, ...kinds, ...scopeKeys)?.n ?? 0,
     );
   }
 
@@ -1095,18 +1109,28 @@ export class Store {
       .all(...keys)) {
       byScope[row.scope] = Number(row.n);
     }
+    // "Pending" and "disputed" are questions for a person, so they count only the kinds a
+    // person can act on. Episodic summaries are candidates by design and Review hides them;
+    // counting them here made the sidebar announce work with nowhere to do it. The byStatus
+    // view below stays complete, because that is data rather than a to-do.
+    const kindMarks = REVIEWABLE_KINDS.map(() => '?').join(', ');
     const pending = Number(
       this.db
         .prepare(
           `SELECT COUNT(*) AS n FROM memory m JOIN memory_head h ON h.ord = m.ord
-           WHERE h.status = 'candidate' AND m.deleted_at IS NULL${marks}`,
+           WHERE h.status = 'candidate' AND m.deleted_at IS NULL AND m.kind IN (${kindMarks})
+             AND (m.valid_to IS NULL OR m.valid_to > ?)${marks}`,
         )
-        .get(...keys)?.n ?? 0,
+        .get(...REVIEWABLE_KINDS, this.now(), ...keys)?.n ?? 0,
     );
     const disputed = Number(
       this.db
-        .prepare(`SELECT COUNT(*) AS n FROM memory m JOIN memory_head h ON h.ord = m.ord WHERE h.status = 'disputed'${marks}`)
-        .get(...keys)?.n ?? 0,
+        .prepare(
+          `SELECT COUNT(*) AS n FROM memory m JOIN memory_head h ON h.ord = m.ord
+           WHERE h.status = 'disputed' AND m.kind IN (${kindMarks})
+             AND (m.valid_to IS NULL OR m.valid_to > ?)${marks}`,
+        )
+        .get(...REVIEWABLE_KINDS, this.now(), ...keys)?.n ?? 0,
     );
     const lastInjection = this.db
       .prepare('SELECT ts, session_id, scope_key, section_bytes, entries_listed, entries_hidden, source FROM request_stat ORDER BY id DESC LIMIT 1')

package/src/mcp/store-facade.js

@@ -36,7 +36,7 @@ export function openFacadeStore(options = {}) {
     memoryRoot: root,
     paths,
     config,
-    pluginVersion: '3.0.1',
+    pluginVersion: '3.2.0',
     readOnly: options.readOnly === true,
   });
   return { store, paths };