@sascha384/tic 5.15.1 → 5.17.0

package/dist/components/WorkItemList.js

@@ -20,6 +20,7 @@ import { OverlayPanel } from './OverlayPanel.js';
 import { DetailPanel, countWrappedLines } from './DetailPanel.js';
 import { undoStore } from '../stores/undoStore.js';
 import { editorStore } from '../stores/editorStore.js';
+import { formStackStore } from '../stores/formStackStore.js';
 import { CommandBar } from './CommandBar.js';
 import { isSoftDeleteBackend } from '../backends/types.js';
 import { filterStore, useFilterStore } from '../stores/filterStore.js';
@@ -36,7 +37,7 @@ function buildWorkItemColumns(capabilities, collapsedIds, selectionFg) {
         width: 4, // overridden dynamically via useMemo
         required: true,
         sortable: true,
-        render: (ti, selected) => (_jsx(Text, { color: selected ? selectionFg : undefined, bold: selected, dimColor: ti.isCrossType && !selected, children: ti.item.id })),
+        render: (ti, selected) => (_jsx(Text, { color: selected ? selectionFg : undefined, bold: selected, dimColor: ti.isCrossType && !selected, children: ti.item.id ?? '\u00B7' })),
     });
     // Title column (flex)
     columns.push({
@@ -48,7 +49,7 @@ function buildWorkItemColumns(capabilities, collapsedIds, selectionFg) {
         render: (ti, selected) => {
             const { item, prefix, isCrossType, hasChildren } = ti;
             const collapseIndicator = hasChildren
-                ? collapsedIds.has(item.id)
+                ? collapsedIds.has(item.rowId)
                     ? '\u25B6 '
                     : '\u25BC '
                 : '  ';
@@ -123,11 +124,26 @@ function buildWorkItemColumns(capabilities, collapsedIds, selectionFg) {
     }
     return columns;
 }
-export function getTargetIds(markedIds, cursorItem) {
+/** Get display IDs (strings) for target items. Used for backend API calls. */
+export function getTargetIds(markedIds, cursorItem, allItems) {
+    if (markedIds.size > 0) {
+        // Look up display IDs from rowIds
+        const ids = [];
+        for (const rowId of markedIds) {
+            const item = allItems.find((i) => i.rowId === rowId);
+            if (item?.id)
+                ids.push(item.id);
+        }
+        return ids;
+    }
+    return cursorItem?.id ? [cursorItem.id] : [];
+}
+/** Get rowIds for target items. Used for undo entries and store operations. */
+export function getTargetRowIds(markedIds, cursorItem) {
     if (markedIds.size > 0) {
         return [...markedIds];
     }
-    return cursorItem ? [cursorItem.id] : [];
+    return cursorItem ? [cursorItem.rowId] : [];
 }
 export function WorkItemList() {
     const { accent, success, error: errorColor, warning: warningColor, marked, mutedDim, selectionBg, } = useThemeStore((s) => s.colors);
@@ -249,11 +265,16 @@ export function WorkItemList() {
         }
     }, [savedViews, defaultView]);
     const queue = useBackendDataStore((s) => s.queue);
-    const queueWrite = async (action, itemId) => {
+    /** Look up the rowId for a display ID. Returns -1 if not found. */
+    const rowIdOf = (displayId) => {
+        const item = allItems.find((i) => i.id === displayId);
+        return item?.rowId ?? -1;
+    };
+    const queueWrite = async (action, itemRowId) => {
         if (queue) {
             await queue.append({
                 action,
-                itemId,
+                itemRowId,
                 timestamp: new Date().toISOString(),
             });
             syncManager?.pushPending().catch((err) => {
@@ -271,7 +292,7 @@ export function WorkItemList() {
             type: 'update',
             label,
             itemSnapshots: snapshots,
-            syncItemIds: [...targetIds],
+            syncItemRowIds: snapshots.map((s) => s.rowId),
             syncAction: 'update',
         });
     };
@@ -306,14 +327,16 @@ export function WorkItemList() {
         capabilities.relationships,
         sortStack,
     ]);
-    const parentSuggestions = useMemo(() => allItems.map((item) => `${item.id} - ${item.title}`), [allItems]);
+    const parentSuggestions = useMemo(() => allItems
+        .filter((item) => item.id !== null)
+        .map((item) => `${item.id} - ${item.title}`), [allItems]);
     // Collapse state: set of item IDs that are collapsed (collapsed by default)
     // Track explicitly expanded items (inverse of collapsed).
     // All parents are collapsed by default; expanding removes from this set.
     // expandedIds comes from listViewStore (imported above)
     // Derive collapsedIds: all parents minus explicitly expanded ones
     const collapsedIds = useMemo(() => {
-        const parentIds = new Set(fullTree.filter((t) => t.hasChildren).map((t) => t.item.id));
+        const parentIds = new Set(fullTree.filter((t) => t.hasChildren).map((t) => t.item.rowId));
         const collapsed = new Set();
         for (const id of parentIds) {
             if (!expandedIds.has(id)) {
@@ -331,7 +354,7 @@ export function WorkItemList() {
                 continue;
             skipDepth = null;
             result.push(t);
-            if (collapsedIds.has(t.item.id)) {
+            if (collapsedIds.has(t.item.rowId)) {
                 skipDepth = t.depth;
             }
         }
@@ -387,14 +410,15 @@ export function WorkItemList() {
         }
         if (key.return) {
             const item = treeItems[cursor]?.item;
-            if (item) {
+            if (item && item.id) {
+                const displayId = item.id;
                 editorStore.getState().init(item.description ?? '', {
                     returnScreen: 'list',
                     onSave: (content) => {
                         const state = backendDataStore.getState();
                         if (state.backend) {
                             void state.backend
-                                .cachedUpdateWorkItem(item.id, { description: content })
+                                .cachedUpdateWorkItem(displayId, { description: content })
                                 .then(() => backendDataStore.getState().refresh());
                         }
                     },
@@ -450,7 +474,7 @@ export function WorkItemList() {
             setCursor(newCursor);
             const start = Math.min(anchor, newCursor);
             const end = Math.max(anchor, newCursor);
-            setMarkedIds(new Set(treeItems.slice(start, end + 1).map((t) => t.item.id)));
+            setMarkedIds(new Set(treeItems.slice(start, end + 1).map((t) => t.item.rowId)));
             clearWarning();
         }
         if (matchesCommand('list-navigate', input, key)) {
@@ -486,18 +510,18 @@ export function WorkItemList() {
             const current = treeItems[cursor];
             if (current &&
                 current.hasChildren &&
-                collapsedIds.has(current.item.id)) {
-                toggleExpanded(current.item.id);
+                collapsedIds.has(current.item.rowId)) {
+                toggleExpanded(current.item.rowId);
             }
         }
         if (matchesCommand('list-collapse', input, key) && treeItems.length > 0) {
             const current = treeItems[cursor];
             if (current) {
-                if (current.hasChildren && !collapsedIds.has(current.item.id)) {
-                    toggleExpanded(current.item.id);
+                if (current.hasChildren && !collapsedIds.has(current.item.rowId)) {
+                    toggleExpanded(current.item.rowId);
                 }
                 else if (current.depth > 0 && current.item.parent) {
-                    const parentIdx = treeItems.findIndex((t) => t.item.id === current.item.parent);
+                    const parentIdx = treeItems.findIndex((t) => t.item.rowId === current.item.parent);
                     if (parentIdx >= 0)
                         setCursor(parentIdx);
                 }
@@ -505,7 +529,7 @@ export function WorkItemList() {
         }
         if (matchesCommand('edit', input, key) && treeItems.length > 0) {
             setFormMode('item');
-            selectWorkItem(treeItems[cursor].item.id);
+            selectWorkItem(treeItems[cursor].item.rowId);
             navigate('form');
         }
         if (matchesCommand('quit', input, key))
@@ -513,7 +537,7 @@ export function WorkItemList() {
         if (matchesCommand('set-iteration', input, key) &&
             capabilities.iterations &&
             treeItems.length > 0) {
-            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
             if (targetIds.length > 0) {
                 openOverlay({ type: 'iteration-picker', targetIds });
             }
@@ -547,8 +571,20 @@ export function WorkItemList() {
                 navigate('form');
             }
         }
+        if (matchesCommand('create-child', input, key) &&
+            treeItems[cursor] &&
+            capabilities.fields.parent) {
+            formStackStore.getState().clear();
+            navigationStore
+                .getState()
+                .setCreateChildParentId(treeItems[cursor].item.rowId);
+            setFormMode('item');
+            setActiveTemplate(null);
+            selectWorkItem(null);
+            navigate('form');
+        }
         if (matchesCommand('delete', input, key) && treeItems.length > 0) {
-            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
             if (targetIds.length > 0) {
                 openOverlay({ type: 'delete-confirm', targetIds });
             }
@@ -561,38 +597,42 @@ export function WorkItemList() {
                 if (entry.type === 'delete') {
                     if (isSoftDeleteBackend(backend)) {
                         for (const snap of entry.itemSnapshots) {
-                            await backend.restoreWorkItem(snap.id);
+                            if (snap.id)
+                                await backend.restoreWorkItem(snap.id);
                         }
                     }
                     if (queue) {
-                        await queue.removeByIds(entry.syncItemIds, 'delete');
+                        await queue.removeByRowIds(entry.syncItemRowIds, 'delete');
                     }
                     refreshData();
                     setToast(entry.itemSnapshots.length === 1
-                        ? `Restored #${entry.itemSnapshots[0].id}`
+                        ? `Restored #${entry.itemSnapshots[0].id ?? entry.itemSnapshots[0].rowId}`
                         : `Restored ${entry.itemSnapshots.length} items`);
                 }
                 else if (entry.type === 'create') {
-                    for (const id of entry.createdIds ?? []) {
-                        await backend.cachedDeleteWorkItem(id);
+                    for (const rowId of entry.createdRowIds ?? []) {
+                        const item = allItems.find((i) => i.rowId === rowId);
+                        if (item?.id)
+                            await backend.cachedDeleteWorkItem(item.id);
                     }
                     if (queue) {
-                        await queue.removeByIds(entry.syncItemIds, 'create');
+                        await queue.removeByRowIds(entry.syncItemRowIds, 'create');
                     }
                     refreshData();
-                    setToast((entry.createdIds?.length ?? 0) === 1
-                        ? `Undid create #${entry.createdIds?.[0]}`
-                        : `Undid create of ${entry.createdIds?.length} items`);
+                    setToast((entry.createdRowIds?.length ?? 0) === 1
+                        ? `Undid create #${entry.createdRowIds?.[0]}`
+                        : `Undid create of ${entry.createdRowIds?.length} items`);
                 }
                 else if (entry.type === 'update') {
                     for (const snap of entry.itemSnapshots) {
-                        await backend.cachedUpdateWorkItem(snap.id, snap);
+                        if (snap.id)
+                            await backend.cachedUpdateWorkItem(snap.id, snap);
                     }
                     if (queue) {
-                        await queue.removeByIds(entry.syncItemIds, 'update');
+                        await queue.removeByRowIds(entry.syncItemRowIds, 'update');
                     }
                     for (const snap of entry.itemSnapshots) {
-                        await queueWrite('update', snap.id);
+                        await queueWrite('update', snap.rowId);
                     }
                     refreshData();
                     setToast(`Undid ${entry.label}`);
@@ -605,39 +645,44 @@ export function WorkItemList() {
         }
         if (matchesCommand('open', input, key) && treeItems.length > 0) {
             setFormMode('item');
-            selectWorkItem(treeItems[cursor].item.id);
+            selectWorkItem(treeItems[cursor].item.rowId);
             navigate('form');
         }
         if (matchesCommand('branch', input, key) &&
             gitAvailable &&
             treeItems.length > 0) {
             const item = treeItems[cursor].item;
-            const comments = item.comments;
-            try {
-                const itemUrl = backend?.getItemUrl(item.id) || '';
-                // Suspend terminal for interactive child process
-                process.stdin.setRawMode?.(false);
-                const result = beginImplementation(item, comments, { branchMode, branchCommand, copyToClipboard }, process.cwd(), { itemUrl });
-                // Restore terminal after interactive shell
-                process.stdin.setRawMode?.(true);
-                console.clear();
-                let msg = result.resumed
-                    ? `Resumed work on #${item.id}`
-                    : `Started work on #${item.id}`;
-                if (result.commandFailed) {
-                    msg += ' (branch command failed, fell back to shell)';
-                }
-                setWarning(msg);
+            if (!item.id) {
+                setWarning('Cannot create branch for item without display ID');
             }
-            catch (e) {
-                process.stdin.setRawMode?.(true);
-                console.clear();
-                setWarning(e instanceof Error ? e.message : 'Failed to start implementation');
+            else {
+                const comments = item.comments;
+                try {
+                    const itemUrl = backend?.getItemUrl(item.id) || '';
+                    // Suspend terminal for interactive child process
+                    process.stdin.setRawMode?.(false);
+                    const result = beginImplementation(item, comments, { branchMode, branchCommand, copyToClipboard }, process.cwd(), { itemUrl });
+                    // Restore terminal after interactive shell
+                    process.stdin.setRawMode?.(true);
+                    console.clear();
+                    let msg = result.resumed
+                        ? `Resumed work on #${item.id}`
+                        : `Started work on #${item.id}`;
+                    if (result.commandFailed) {
+                        msg += ' (branch command failed, fell back to shell)';
+                    }
+                    setWarning(msg);
+                }
+                catch (e) {
+                    process.stdin.setRawMode?.(true);
+                    console.clear();
+                    setWarning(e instanceof Error ? e.message : 'Failed to start implementation');
+                }
+                void backendDataStore
+                    .getState()
+                    .reloadItem(item.id)
+                    .catch(() => { });
             }
-            void backendDataStore
-                .getState()
-                .reloadItem(item.id)
-                .catch(() => { });
         }
         if (matchesCommand('status', input, key)) {
             navigate('status');
@@ -656,7 +701,7 @@ export function WorkItemList() {
             setToast('Filters cleared');
         }
         if (matchesCommand('list-status', input, key) && treeItems.length > 0) {
-            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
             if (targetIds.length > 0) {
                 openOverlay({ type: 'status-picker', targetIds });
             }
@@ -677,7 +722,7 @@ export function WorkItemList() {
             prCapabilities.create &&
             treeItems.length > 0) {
             const item = treeItems[cursor]?.item;
-            if (item) {
+            if (item && item.id) {
                 const cwd = process.cwd();
                 const currentBranch = getCurrentBranch(cwd);
                 const expectedBranch = `tic/${slugify(item.id, item.title)}`;
@@ -687,7 +732,7 @@ export function WorkItemList() {
                     .createPullRequest({
                     title: item.title,
                     sourceBranch,
-                    linkedItems: [item.id],
+                    linkedItems: [item.rowId],
                 })
                     .then((pr) => {
                     setToast(`PR #${String(pr.number)} created`);
@@ -718,12 +763,12 @@ export function WorkItemList() {
         }
         if (matchesCommand('mark', input, key) && treeItems.length > 0) {
             setRangeAnchor(null);
-            const itemId = treeItems[cursor].item.id;
-            toggleMarked(itemId);
+            const itemRowId = treeItems[cursor].item.rowId;
+            toggleMarked(itemRowId);
         }
         if (matchesCommand('clear-marks', input, key) && treeItems.length > 0) {
             setRangeAnchor(null);
-            const visibleIds = treeItems.map((t) => t.item.id);
+            const visibleIds = treeItems.map((t) => t.item.rowId);
             const allMarked = visibleIds.every((id) => markedIds.has(id));
             if (allMarked) {
                 clearMarked();
@@ -738,7 +783,7 @@ export function WorkItemList() {
         if (matchesCommand('set-priority', input, key) &&
             capabilities.fields.priority &&
             treeItems.length > 0) {
-            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
             if (targetIds.length > 0) {
                 openOverlay({ type: 'priority-picker', targetIds });
             }
@@ -746,7 +791,7 @@ export function WorkItemList() {
         if (matchesCommand('list-parent', input, key) &&
             capabilities.fields.parent &&
             treeItems.length > 0) {
-            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
             if (targetIds.length > 0) {
                 openOverlay({ type: 'parent-input', targetIds });
             }
@@ -754,7 +799,7 @@ export function WorkItemList() {
         if (matchesCommand('set-assignee', input, key) &&
             capabilities.fields.assignee &&
             treeItems.length > 0) {
-            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
             if (targetIds.length > 0) {
                 openOverlay({ type: 'assignee-input', targetIds });
             }
@@ -762,7 +807,7 @@ export function WorkItemList() {
         if (matchesCommand('set-labels', input, key) &&
             capabilities.fields.labels &&
             treeItems.length > 0) {
-            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
             if (targetIds.length > 0) {
                 openOverlay({ type: 'labels-input', targetIds });
             }
@@ -770,7 +815,7 @@ export function WorkItemList() {
         if (matchesCommand('set-type', input, key) &&
             capabilities.customTypes &&
             treeItems.length > 0) {
-            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+            const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
             if (targetIds.length > 0) {
                 openOverlay({ type: 'type-picker', targetIds });
             }
@@ -949,20 +994,20 @@ export function WorkItemList() {
                 if (treeItems[cursor]) {
                     navigationStore
                         .getState()
-                        .setCreateChildParentId(treeItems[cursor].item.id);
+                        .setCreateChildParentId(treeItems[cursor].item.rowId);
                     selectWorkItem(null);
                     navigate('form');
                 }
                 break;
             case 'edit':
                 if (treeItems[cursor]) {
-                    selectWorkItem(treeItems[cursor].item.id);
+                    selectWorkItem(treeItems[cursor].item.rowId);
                     navigate('form');
                 }
                 break;
             case 'delete':
                 if (treeItems.length > 0) {
-                    const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+                    const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
                     if (targetIds.length > 0) {
                         openOverlay({ type: 'delete-confirm', targetIds });
                     }
@@ -971,37 +1016,44 @@ export function WorkItemList() {
             case 'open':
                 if (treeItems[cursor]) {
                     setFormMode('item');
-                    selectWorkItem(treeItems[cursor].item.id);
+                    selectWorkItem(treeItems[cursor].item.rowId);
                     navigate('form');
                 }
                 break;
             case 'branch':
                 if (treeItems[cursor]) {
                     const item = treeItems[cursor].item;
-                    const comments = item.comments;
-                    try {
-                        const itemUrl = backend?.getItemUrl(item.id) || '';
-                        process.stdin.setRawMode?.(false);
-                        const result = beginImplementation(item, comments, { branchMode, branchCommand, copyToClipboard }, process.cwd(), { itemUrl });
-                        process.stdin.setRawMode?.(true);
-                        console.clear();
-                        let msg = result.resumed
-                            ? `Resumed work on #${item.id}`
-                            : `Started work on #${item.id}`;
-                        if (result.commandFailed) {
-                            msg += ' (branch command failed, fell back to shell)';
-                        }
-                        setWarning(msg);
+                    if (!item.id) {
+                        setWarning('Cannot create branch for item without display ID');
                     }
-                    catch (e) {
-                        process.stdin.setRawMode?.(true);
-                        console.clear();
-                        setWarning(e instanceof Error ? e.message : 'Failed to start implementation');
+                    else {
+                        const comments = item.comments;
+                        try {
+                            const itemUrl = backend?.getItemUrl(item.id) || '';
+                            process.stdin.setRawMode?.(false);
+                            const result = beginImplementation(item, comments, { branchMode, branchCommand, copyToClipboard }, process.cwd(), { itemUrl });
+                            process.stdin.setRawMode?.(true);
+                            console.clear();
+                            let msg = result.resumed
+                                ? `Resumed work on #${item.id}`
+                                : `Started work on #${item.id}`;
+                            if (result.commandFailed) {
+                                msg += ' (branch command failed, fell back to shell)';
+                            }
+                            setWarning(msg);
+                        }
+                        catch (e) {
+                            process.stdin.setRawMode?.(true);
+                            console.clear();
+                            setWarning(e instanceof Error
+                                ? e.message
+                                : 'Failed to start implementation');
+                        }
+                        void backendDataStore
+                            .getState()
+                            .reloadItem(item.id)
+                            .catch(() => { });
                     }
-                    void backendDataStore
-                        .getState()
-                        .reloadItem(item.id)
-                        .catch(() => { });
                 }
                 break;
             case 'sync':
@@ -1016,7 +1068,7 @@ export function WorkItemList() {
                 break;
             case 'set-iteration':
                 if (treeItems.length > 0) {
-                    const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+                    const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
                     if (targetIds.length > 0) {
                         openOverlay({ type: 'iteration-picker', targetIds });
                     }
@@ -1036,8 +1088,8 @@ export function WorkItemList() {
                 break;
             case 'mark':
                 if (treeItems[cursor]) {
-                    const itemId = treeItems[cursor].item.id;
-                    toggleMarked(itemId);
+                    const itemRowId = treeItems[cursor].item.rowId;
+                    toggleMarked(itemRowId);
                 }
                 break;
             case 'clear-marks':
@@ -1045,7 +1097,7 @@ export function WorkItemList() {
                 break;
             case 'set-priority':
                 {
-                    const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+                    const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
                     if (targetIds.length > 0) {
                         openOverlay({ type: 'priority-picker', targetIds });
                     }
@@ -1053,7 +1105,7 @@ export function WorkItemList() {
                 break;
             case 'set-assignee':
                 {
-                    const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+                    const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
                     if (targetIds.length > 0) {
                         openOverlay({ type: 'assignee-input', targetIds });
                     }
@@ -1061,7 +1113,7 @@ export function WorkItemList() {
                 break;
             case 'set-labels':
                 {
-                    const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+                    const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
                     if (targetIds.length > 0) {
                         openOverlay({ type: 'labels-input', targetIds });
                     }
@@ -1069,7 +1121,7 @@ export function WorkItemList() {
                 break;
             case 'set-type':
                 {
-                    const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+                    const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
                     if (targetIds.length > 0) {
                         openOverlay({ type: 'type-picker', targetIds });
                     }
@@ -1112,7 +1164,7 @@ export function WorkItemList() {
         }
     };
     const handleBulkAction = (action) => {
-        const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item);
+        const targetIds = getTargetIds(markedIds, treeItems[cursor]?.item, allItems);
         if (targetIds.length === 0)
             return;
         switch (action) {
@@ -1147,7 +1199,7 @@ export function WorkItemList() {
         : '';
     const visibleTreeItems = useMemo(() => treeItems.slice(viewport.start, viewport.end), [treeItems, viewport.start, viewport.end]);
     const workItemColumns = useMemo(() => {
-        const maxIdLen = visibleTreeItems.reduce((max, { item }) => Math.max(max, item.id.length), 2);
+        const maxIdLen = visibleTreeItems.reduce((max, { item }) => Math.max(max, (item.id ?? '\u00B7').length), 2);
         const cols = buildWorkItemColumns(capabilities, collapsedIds, autoFg(selectionBg));
         cols[0].width = maxIdLen + 2;
         return cols;
@@ -1161,7 +1213,7 @@ export function WorkItemList() {
                                         ? formatIterationDates(it.startDate, it.endDate)
                                         : null;
                                     return dates ? `  ${dates}` : '';
-                                })()] }), _jsx(Text, { dimColor: mutedDim, children: ` (${filterCount > 0 ? `${items.length}/${unfilteredCount}` : items.length} item${unfilteredCount === 1 ? '' : 's'})` }), markedCount > 0 && (_jsxs(Text, { color: marked, children: [` ● ${markedCount}`, markedDistribution.above > 0 && ` ↑${markedDistribution.above}`, markedDistribution.below > 0 && ` ↓${markedDistribution.below}`] })), filterCount > 0 && (_jsx(Text, { color: warningColor, children: ` [${filterCount} filter${filterCount === 1 ? '' : 's'}${activeViewName ? `: ${activeViewName}` : ''}]` }))] }) }), _jsx(TableLayout, { items: visibleTreeItems, columns: workItemColumns, cursor: viewport.visibleCursor, terminalWidth: terminalWidth, getKey: (ti) => `${ti.item.id}-${ti.item.type}`, isMarked: (ti) => markedIds.has(ti.item.id), sortStack: sortStack }), treeItems.length === 0 && !loading && initError && (_jsxs(Box, { marginTop: 1, flexDirection: "column", children: [_jsx(Text, { color: errorColor, children: "Failed to connect to backend:" }), _jsx(Box, { marginLeft: 2, children: _jsx(Text, { color: errorColor, children: initError }) }), _jsx(Text, { dimColor: mutedDim, children: "Press , for settings or q to quit." })] })), treeItems.length === 0 && !loading && !initError && (_jsx(Box, { marginTop: 1, children: _jsxs(Text, { dimColor: mutedDim, children: ["No ", activeType ?? 'item', "s in this iteration. Press c to create, / to search all."] }) })), loading && treeItems.length === 0 && (_jsx(Box, { marginTop: 1, children: _jsx(Text, { dimColor: mutedDim, children: "Loading..." }) })), showDetailPanel && treeItems.length > 0 && treeItems[cursor] && (_jsx(Box, { marginTop: 1, children: _jsx(DetailPanel, { item: treeItems[cursor].item, terminalWidth: terminalWidth, showFullDescription: showFullDescription, descriptionScrollOffset: descriptionScrollOffset, maxDescriptionHeight: maxDescriptionHeight }) })), _jsx(Box, { marginTop: 1, children: showFullDescription ? (_jsxs(Box, { children: [_jsx(Text, { dimColor: mutedDim, children: "\u2191\u2193 scroll space/esc close" }), positionText && _jsxs(Text, { dimColor: mutedDim, children: [" ", positionText] })] })) : activeOverlay?.type === 'command-bar' ? (_jsx(CommandBar, { commands: paletteCommands, onCommand: handleCommandSelect, onCancel: closeOverlay })) : activeOverlay?.type === 'bulk-menu' ? ((() => {
+                                })()] }), _jsx(Text, { dimColor: mutedDim, children: ` (${filterCount > 0 ? `${items.length}/${unfilteredCount}` : items.length} item${unfilteredCount === 1 ? '' : 's'})` }), markedCount > 0 && (_jsxs(Text, { color: marked, children: [` ● ${markedCount}`, markedDistribution.above > 0 && ` ↑${markedDistribution.above}`, markedDistribution.below > 0 && ` ↓${markedDistribution.below}`] })), filterCount > 0 && (_jsx(Text, { color: warningColor, children: ` [${filterCount} filter${filterCount === 1 ? '' : 's'}${activeViewName ? `: ${activeViewName}` : ''}]` }))] }) }), _jsx(TableLayout, { items: visibleTreeItems, columns: workItemColumns, cursor: viewport.visibleCursor, terminalWidth: terminalWidth, getKey: (ti) => `${ti.item.rowId}-${ti.item.type}`, isMarked: (ti) => markedIds.has(ti.item.rowId), sortStack: sortStack }), treeItems.length === 0 && !loading && initError && (_jsxs(Box, { marginTop: 1, flexDirection: "column", children: [_jsx(Text, { color: errorColor, children: "Failed to connect to backend:" }), _jsx(Box, { marginLeft: 2, children: _jsx(Text, { color: errorColor, children: initError }) }), _jsx(Text, { dimColor: mutedDim, children: "Press , for settings or q to quit." })] })), treeItems.length === 0 && !loading && !initError && (_jsx(Box, { marginTop: 1, children: _jsxs(Text, { dimColor: mutedDim, children: ["No ", activeType ?? 'item', "s in this iteration. Press c to create, / to search all."] }) })), loading && treeItems.length === 0 && (_jsx(Box, { marginTop: 1, children: _jsx(Text, { dimColor: mutedDim, children: "Loading..." }) })), showDetailPanel && treeItems.length > 0 && treeItems[cursor] && (_jsx(Box, { marginTop: 1, children: _jsx(DetailPanel, { item: treeItems[cursor].item, terminalWidth: terminalWidth, showFullDescription: showFullDescription, descriptionScrollOffset: descriptionScrollOffset, maxDescriptionHeight: maxDescriptionHeight }) })), _jsx(Box, { marginTop: 1, children: showFullDescription ? (_jsxs(Box, { children: [_jsx(Text, { dimColor: mutedDim, children: "\u2191\u2193 scroll space/esc close" }), positionText && _jsxs(Text, { dimColor: mutedDim, children: [" ", positionText] })] })) : activeOverlay?.type === 'command-bar' ? (_jsx(CommandBar, { commands: paletteCommands, onCommand: handleCommandSelect, onCancel: closeOverlay })) : activeOverlay?.type === 'bulk-menu' ? ((() => {
                     const bulkItems = [];
                     bulkItems.push({
                         id: 'status',
@@ -1239,7 +1291,7 @@ export function WorkItemList() {
                                 await backend.cachedUpdateWorkItem(id, {
                                     status: item.value,
                                 });
-                                await queueWrite('update', id);
+                                await queueWrite('update', rowIdOf(id));
                             }
                             for (const id of targetIds) {
                                 await backendDataStore.getState().reloadItem(id);
@@ -1267,7 +1319,7 @@ export function WorkItemList() {
                                 await backend.cachedUpdateWorkItem(id, {
                                     type: item.value,
                                 });
-                                await queueWrite('update', id);
+                                await queueWrite('update', rowIdOf(id));
                             }
                             for (const id of targetIds) {
                                 await backendDataStore.getState().reloadItem(id);
@@ -1295,7 +1347,7 @@ export function WorkItemList() {
                             pushUpdateUndo(targetIds, 'priority change');
                             for (const id of targetIds) {
                                 await backend.cachedUpdateWorkItem(id, { priority });
-                                await queueWrite('update', id);
+                                await queueWrite('update', rowIdOf(id));
                             }
                             for (const id of targetIds) {
                                 await backendDataStore.getState().reloadItem(id);
@@ -1337,16 +1389,21 @@ export function WorkItemList() {
                             return;
                         void (async () => {
                             const raw = item.value.trim();
-                            const newParent = raw.includes(' - ')
+                            const parentDisplayId = raw.includes(' - ')
                                 ? raw.split(' - ')[0].trim()
                                 : raw;
+                            const newParent = rowIdOf(parentDisplayId);
+                            if (newParent === -1) {
+                                setWarning(`Parent "${parentDisplayId}" not found`);
+                                return;
+                            }
                             try {
                                 pushUpdateUndo(targetIds, 'parent change');
                                 for (const id of targetIds) {
                                     await backend.cachedUpdateWorkItem(id, {
                                         parent: newParent,
                                     });
-                                    await queueWrite('update', id);
+                                    await queueWrite('update', rowIdOf(id));
                                 }
                                 clearWarning();
                             }
@@ -1367,18 +1424,24 @@ export function WorkItemList() {
                             return;
                         void (async () => {
                             const raw = text.trim();
-                            const newParent = raw === ''
-                                ? null
-                                : raw.includes(' - ')
+                            let newParent = null;
+                            if (raw !== '') {
+                                const parentDisplayId = raw.includes(' - ')
                                     ? raw.split(' - ')[0].trim()
                                     : raw;
+                                newParent = rowIdOf(parentDisplayId);
+                                if (newParent === -1) {
+                                    setWarning(`Parent "${parentDisplayId}" not found`);
+                                    return;
+                                }
+                            }
                             try {
                                 pushUpdateUndo(targetIds, 'parent change');
                                 for (const id of targetIds) {
                                     await backend.cachedUpdateWorkItem(id, {
                                         parent: newParent,
                                     });
-                                    await queueWrite('update', id);
+                                    await queueWrite('update', rowIdOf(id));
                                 }
                                 clearWarning();
                             }
@@ -1404,7 +1467,7 @@ export function WorkItemList() {
                                 await backend.cachedUpdateWorkItem(id, {
                                     assignee: item.value.trim(),
                                 });
-                                await queueWrite('update', id);
+                                await queueWrite('update', rowIdOf(id));
                             }
                             for (const id of targetIds) {
                                 await backendDataStore.getState().reloadItem(id);
@@ -1428,7 +1491,7 @@ export function WorkItemList() {
                                 await backend.cachedUpdateWorkItem(id, {
                                     assignee: text.trim(),
                                 });
-                                await queueWrite('update', id);
+                                await queueWrite('update', rowIdOf(id));
                             }
                             for (const id of targetIds) {
                                 await backendDataStore.getState().reloadItem(id);
@@ -1455,7 +1518,7 @@ export function WorkItemList() {
                             const labels = selected.map((i) => i.value);
                             for (const id of targetIds) {
                                 await backend.cachedUpdateWorkItem(id, { labels });
-                                await queueWrite('update', id);
+                                await queueWrite('update', rowIdOf(id));
                             }
                             for (const id of targetIds) {
                                 await backendDataStore.getState().reloadItem(id);
@@ -1481,7 +1544,7 @@ export function WorkItemList() {
                                 .filter(Boolean);
                             for (const id of targetIds) {
                                 await backend.cachedUpdateWorkItem(id, { labels });
-                                await queueWrite('update', id);
+                                await queueWrite('update', rowIdOf(id));
                             }
                             for (const id of targetIds) {
                                 await backendDataStore.getState().reloadItem(id);
@@ -1651,7 +1714,7 @@ export function WorkItemList() {
                                 await backend.cachedUpdateWorkItem(id, {
                                     iteration: item.value,
                                 });
-                                await queueWrite('update', id);
+                                await queueWrite('update', rowIdOf(id));
                             }
                             for (const id of targetIds) {
                                 await backendDataStore.getState().reloadItem(id);
@@ -1684,7 +1747,7 @@ export function WorkItemList() {
                                     else {
                                         await backend.cachedDeleteWorkItem(id);
                                     }
-                                    await queueWrite('delete', id);
+                                    await queueWrite('delete', rowIdOf(id));
                                 }
                                 if (softDelete) {
                                     const evicted = undoStore.getState().pushUndo({
@@ -1693,18 +1756,19 @@ export function WorkItemList() {
                                             ? `deleted #${targetIds[0]}`
                                             : `deleted ${targetIds.length} items`,
                                         itemSnapshots: snapshots,
-                                        syncItemIds: [...targetIds],
+                                        syncItemRowIds: snapshots.map((s) => s.rowId),
                                         syncAction: 'delete',
                                     });
                                     if (evicted?.type === 'delete') {
                                         for (const snap of evicted.itemSnapshots) {
-                                            await backend.permanentlyDeleteWorkItem(snap.id);
+                                            if (snap.id)
+                                                await backend.permanentlyDeleteWorkItem(snap.id);
                                         }
                                     }
                                 }
                                 closeOverlay();
                                 for (const id of targetIds) {
-                                    removeDeletedItem(id);
+                                    removeDeletedItem(rowIdOf(id));
                                 }
                                 setCursor(Math.max(0, cursor - 1));
                                 for (const id of targetIds) {