fix: enhance file removal functionality and add coverage tests

Author: winterdrive

.github/workflows/validate.yml

@@ -55,8 +55,8 @@ jobs:
       - name: Type check
         run: tsc -p ./
 
-      - name: Run Unit and Property tests
-        run: npm test
+      - name: Run Unit and Property tests with coverage
+        run: npm run test:coverage
 
       - name: Package extension
         run: npx @vscode/vsce package --out virtual-tabs.vsix

.gitignore

@@ -36,3 +36,4 @@ test-resources/
 # Testing
 /coverage
 test-ui-output.txt
+docs/assets/fix_rounded_corners.py

CHANGELOG.md

@@ -11,6 +11,12 @@ All notable changes to the "VirtualTabs" extension will be documented in this fi
 ### Fixed
 
 - Directory drag-and-drop now skips hidden directories whose names start with `.`, while still including dotfiles such as `.gitignore` and `.editorconfig`.
+- Removing selected files from a group now works when the group stores workspace-relative file paths after config reload.
+
+### Tests and CI
+
+- Added focused unit coverage for file-entry matching across URI, absolute path, and workspace-relative path storage forms.
+- Added `npm run test:coverage` and updated PR validation to run Jest coverage for the file-entry matching helper before packaging.
 
 ## [0.5.5] - 2026-05-13
 

DEVELOPMENT.md

@@ -137,17 +137,23 @@ VirtualTabs uses three automated test layers:
 | Layer | Command | Purpose |
 | :--- | :--- | :--- |
 | TypeScript + Jest unit tests | `npm run test` | Compiles the extension and runs Jest unit tests. |
+| Jest coverage gate | `npm run test:coverage` | Runs Jest with coverage enabled for focused unit-tested core helpers and enforces configured thresholds. |
 | Property tests | `npm run test:properties` | Exercises config scope discovery, path routing, and tree aggregation invariants with generated inputs. |
 | VS Code UI/E2E tests | `npm run test:ui` | Launches a real VS Code instance with `vscode-extension-tester` and verifies Activity Bar, sidebar, and multi-root scope behavior. |
 
 ### Run the Full Local Test Set
 
 ```bash
 npm run test
+npm run test:coverage
 npm run test:properties
 npm run test:ui
 ```
 
+### Coverage Gate
+
+`npm run test:coverage` uses Jest coverage and is part of the PR validation workflow. Coverage is intentionally scoped in `jest.config.js`; when adding files to `collectCoverageFrom`, add focused unit tests in the same PR so the gate remains meaningful instead of reflecting unrelated legacy or UI-heavy code.
+
 ### UI/E2E Test Setup
 
 The UI test script compiles the UI test files and then runs them through `extest`:

jest.config.js

@@ -7,5 +7,21 @@ module.exports = {
     modulePathIgnorePatterns: ['<rootDir>/.vscode-test/'],
     transform: {
         '^.+\\.tsx?$': ['ts-jest', { tsconfig: 'tsconfig.test.json' }]
+    },
+    moduleNameMapper: {
+        '^(\\.{1,2}/.*)\\.js$': '$1'
+    },
+    collectCoverageFrom: [
+        'src/core/FileEntryMatcher.ts',
+        'src/core/GroupFileRemoval.ts',
+        'src/core/GroupFileTargets.ts'
+    ],
+    coverageThreshold: {
+        global: {
+            statements: 90,
+            branches: 80,
+            functions: 90,
+            lines: 90
+        }
     }
 };

package-lock.json

@@ -2,7 +2,7 @@
     "name": "virtual-tabs",
     "displayName": "VirtualTabs - Virtual File Directories & AI context management",
     "description": "%extension.description%",
-    "version": "0.5.7",
+    "version": "0.5.8",
     "publisher": "winterdrive",
     "icon": "docs/assets/virtualtabs_icon_128.png",
     "categories": [
@@ -683,6 +683,7 @@
     },
     "scripts": {
         "test": "tsc -p ./ && jest --runInBand",
+        "test:coverage": "jest --runInBand --coverage",
         "test:properties": "jest --runInBand src/test/properties",
         "test:ui:setup": "extest setup-tests",
         "test:ui": "tsc -p tsconfig.test.ui.json && extest setup-and-run \"out/test/ui/**/*.test.js\" -e \".vscode-test/extensions\" -r \"test-resources/multi-root/virtual-tabs.code-workspace\" -c 1.96.0",

package.json

@@ -9,6 +9,7 @@ import { executeWithConfirmation } from './util';
 import { SkillGenerator } from './mcp/SkillGenerator';
 import { McpConfigPanel } from './mcp/McpConfigPanel';
 import { SendToManager } from './sendTo';
+import { groupItemsByGroupIdx } from './core/GroupFileTargets';
 
 // Global clipboard for VirtualTabs items
 let globalClipboardItems: (TempFileItem | TempFolderItem)[] = [];
@@ -633,15 +634,7 @@ export function registerCommands(
     context.subscriptions.push(vscode.commands.registerCommand('virtualTabs.removeSelectedFilesFromGroup', (item?: TempFileItem) => {
         const filesToRemove = resolveTargetItems(item, provider);
         if (filesToRemove.length === 0) return;
-        const filesByGroup = new Map<number, TempFileItem[]>();
-        for (const fileItem of filesToRemove) {
-            const current = filesByGroup.get(fileItem.groupIdx);
-            if (current) {
-                current.push(fileItem);
-            } else {
-                filesByGroup.set(fileItem.groupIdx, [fileItem]);
-            }
-        }
+        const filesByGroup = groupItemsByGroupIdx(filesToRemove);
 
         for (const [groupIdx, groupFiles] of filesByGroup) {
             provider.removeFilesFromGroup(groupIdx, groupFiles);
@@ -855,16 +848,9 @@ export function registerCommands(
 
 
         const executeRemove = () => {
-            const filesByGroup = new Map<number, TempFileItem[]>();
-            for (const fileItem of filesToRemove) {
-                if (!(fileItem instanceof TempFileItem)) continue;
-                const current = filesByGroup.get(fileItem.groupIdx);
-                if (current) {
-                    current.push(fileItem);
-                } else {
-                    filesByGroup.set(fileItem.groupIdx, [fileItem]);
-                }
-            }
+            const filesByGroup = groupItemsByGroupIdx(
+                filesToRemove.filter((fileItem): fileItem is TempFileItem => fileItem instanceof TempFileItem)
+            );
 
             for (const [groupIdx, groupFiles] of filesByGroup) {
                 provider.removeFilesFromGroup(groupIdx, groupFiles);

src/commands.ts

@@ -0,0 +1,38 @@
+import { TempGroup } from '../types';
+import { matchesStoredFileEntry } from './FileEntryMatcher';
+
+export interface FileRemovalTarget {
+    uri: string;
+    fsPath: string;
+}
+
+export function removeStoredFileEntriesFromGroup(
+    group: Pick<TempGroup, 'files' | 'bookmarks'>,
+    targets: FileRemovalTarget[],
+    scopeRoot?: string
+): boolean {
+    if (!group.files || targets.length === 0) {
+        return false;
+    }
+
+    const originalLength = group.files.length;
+    group.files = group.files.filter(storedEntry =>
+        !targets.some(target => matchesStoredFileEntry(storedEntry, target.uri, target.fsPath, scopeRoot))
+    );
+
+    if (group.bookmarks) {
+        for (const bookmarkKey of Object.keys(group.bookmarks)) {
+            const shouldDelete = targets.some(target =>
+                matchesStoredFileEntry(bookmarkKey, target.uri, target.fsPath, scopeRoot)
+            );
+            if (shouldDelete) {
+                delete group.bookmarks[bookmarkKey];
+            }
+        }
+        if (Object.keys(group.bookmarks).length === 0) {
+            delete group.bookmarks;
+        }
+    }
+
+    return group.files.length !== originalLength;
+}

src/core/GroupFileRemoval.ts

@@ -0,0 +1,18 @@
+export interface GroupIndexedItem {
+    groupIdx: number;
+}
+
+export function groupItemsByGroupIdx<T extends GroupIndexedItem>(items: T[]): Map<number, T[]> {
+    const itemsByGroup = new Map<number, T[]>();
+
+    for (const item of items) {
+        const current = itemsByGroup.get(item.groupIdx);
+        if (current) {
+            current.push(item);
+        } else {
+            itemsByGroup.set(item.groupIdx, [item]);
+        }
+    }
+
+    return itemsByGroup;
+}