dsh-checklists 0.3.3 → 0.4.0

package/README.md

@@ -28,6 +28,7 @@ If the agent forgets to mark something, you say so in chat and it fixes it.
 | `checklist_edit` | Changes an item's text (not its state) |
 | `checklist_list` | Full tree with ids and states, for when the digest is truncated |
 | `checklist_delete` | Deletes a list (by id or exact title) |
+| `checklist_remove_item` | Removes one item and everything nested under it |
 | `checklist_rename` | Renames a list; titles stay unique within the session |
 | `checklist_template_save` | Saves a list as a reusable template for future sessions |
 
@@ -49,7 +50,7 @@ From a local build or tarball:
 npm install
 npm run build
 npm pack
-dsh plugin --profile dev add ./dsh-checklists-0.3.2.tgz
+dsh plugin --profile dev add ./dsh-checklists-0.4.0.tgz
 dsh --profile dev --dump-config        # the tool-checklist row should appear
 ```
 

package/lib/model.js

@@ -147,6 +147,27 @@ export function findItemById(lists, id) {
     }
     return undefined;
 }
+/**
+ * Remove one item (and its whole subtree) by exact id.
+ *
+ * Mutates the array in place, so callers must pass a document they own — the
+ * store's `update` already hands out a clone.
+ *
+ * @param items - sibling list to search, at any depth.
+ * @param id - exact item id.
+ * @returns the removed node, or `undefined` when no item has that id.
+ */
+export function removeItemById(items, id) {
+    const index = items.findIndex((entry) => entry.id === id);
+    if (index >= 0)
+        return items.splice(index, 1)[0];
+    for (const item of items) {
+        const removed = removeItemById(item.children, id);
+        if (removed)
+            return removed;
+    }
+    return undefined;
+}
 /**
  * Validate a checklist rename before applying it.
  *

package/lib/tools.js

@@ -5,7 +5,7 @@
  * the single source of truth and the sidebar (phase 2) is a read-only mirror.
  */
 import { defineTool } from '@deepseek-ai/dsh-tools';
-import { buildItems, checkRename, countItems, findList, flatten, ITEM_STATES, isItemState, mintId, nextItem, resolveRef, toLines, } from './model.js';
+import { buildItems, checkRename, countItems, findList, flatten, ITEM_STATES, isItemState, mintId, nextItem, removeItemById, resolveRef, toLines, } from './model.js';
 /** Ships with the plugin so a first run has something to instantiate. */
 const DEFAULT_TEMPLATES = {
     release: [
@@ -542,6 +542,78 @@ export function registerChecklistTools(ctx, store, config = {}) {
             });
         },
     })));
+    ctx.effect(() => ctx.tools.register(defineTool({
+        name: 'checklist_remove_item',
+        description: 'Remove one item — and everything nested under it — from a checklist in the CURRENT session. '
+            + 'Match it by id or exact text. Use it to drop work that no longer applies; '
+            + 'to change the item instead, use checklist_edit.',
+        parameters: {
+            listId: {
+                type: 'string',
+                required: true,
+                description: 'Checklist id, or its exact title.',
+            },
+            item: {
+                type: 'string',
+                required: true,
+                description: 'Item id or exact item text.',
+            },
+        },
+        output: {
+            schema: {
+                type: 'object',
+                additionalProperties: false,
+                properties: {
+                    listId: { type: 'string', required: true },
+                    itemId: { type: 'string', required: true },
+                    text: { type: 'string', required: true },
+                    removed: { type: 'integer', required: true },
+                },
+            },
+            render: (_args, value) => {
+                const result = value;
+                const extra = result.removed > 1 ? ` (+${result.removed - 1} nested)` : '';
+                return [{ type: 'text', text: `Removed "${result.text}"${extra}.` }];
+            },
+        },
+        timeoutMs: 10_000,
+        async execute(args, exec) {
+            const input = args;
+            const sessionId = sessionOf(exec);
+            const listRef = text(input.listId).trim();
+            const itemRef = text(input.item).trim();
+            if (listRef.length === 0)
+                throw new Error('checklist_remove_item: listId must not be empty');
+            if (itemRef.length === 0)
+                throw new Error('checklist_remove_item: item must not be empty');
+            return store.update(sessionId, (doc) => {
+                const list = findList(doc.lists, listRef);
+                if (!list) {
+                    const available = doc.lists.map((entry) => `${entry.id} (${entry.title})`).join(', ');
+                    throw new Error(`checklist_remove_item: no checklist "${listRef}" in this session. `
+                        + `Available: ${available.length > 0 ? available : 'none'}`);
+                }
+                const found = resolveRef([list], itemRef);
+                if (found.kind === 'ambiguous') {
+                    const options = found.candidates.map((entry) => `${entry.item.id} (${entry.item.text})`).join(', ');
+                    throw new Error(`checklist_remove_item: "${itemRef}" is ambiguous; use an id: ${options}`);
+                }
+                if (found.kind === 'miss') {
+                    throw new Error(`checklist_remove_item: no item "${itemRef}" in "${list.title}". Call checklist_list for ids.`);
+                }
+                const removed = removeItemById(list.items, found.hit.item.id);
+                if (!removed) {
+                    throw new Error(`checklist_remove_item: item "${itemRef}" disappeared from "${list.title}"`);
+                }
+                return {
+                    listId: list.id,
+                    itemId: removed.id,
+                    text: removed.text,
+                    removed: countItems([removed]).total,
+                };
+            });
+        },
+    })));
     ctx.effect(() => ctx.tools.register(defineTool({
         name: 'checklist_delete',
         description: 'Delete one checklist from the CURRENT session. The list and its items are gone; there is no archive. '

package/lib/types/model.d.ts

@@ -97,6 +97,17 @@ export declare function findItemById(lists: readonly Checklist[], id: string): {
     list: Checklist;
     item: ChecklistItem;
 } | undefined;
+/**
+ * Remove one item (and its whole subtree) by exact id.
+ *
+ * Mutates the array in place, so callers must pass a document they own — the
+ * store's `update` already hands out a clone.
+ *
+ * @param items - sibling list to search, at any depth.
+ * @param id - exact item id.
+ * @returns the removed node, or `undefined` when no item has that id.
+ */
+export declare function removeItemById(items: ChecklistItem[], id: string): ChecklistItem | undefined;
 export type RenameCheck = {
     kind: 'ok';
     list: Checklist;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-checklists",
-  "version": "0.3.3",
+  "version": "0.4.0",
   "description": "Session-scoped checklists for DeepSeek Harness: the agent tracks work as a tree, the model always sees a compact digest, and a read-only sidebar tab mirrors it.",
   "type": "module",
   "main": "lib/index.js",