[issues/163] Add Bookmarks section to RangeLink status bar menu (#193)

* Add new bookmark-focused command and strings

* Selecting a bookmark from the quickpick copies it to clipboard and pastes to destination manager

* Bookmarks are now added to the quickpick menu

* Plumbing for new changes

* Adjust already-existing tests to new structure

* Introduce `createMockBookmarkStore()`

* Add tests for menu building when there are and are not bookmarks

* New tests

Author: couimet

packages/rangelink-vscode-extension/src/__tests__/extension.test.ts

@@ -3273,7 +3273,7 @@ describe('Extension lifecycle', () => {
 
     // Verify all commands are registered
     // Note: Command registration is IDE-agnostic; runtime availability differs by environment
-    expect(mockCommands.registerCommand).toHaveBeenCalledTimes(17);
+    expect(mockCommands.registerCommand).toHaveBeenCalledTimes(19);
     expect(mockContext.subscriptions.length).toBeGreaterThan(0);
     expect(vscode.window.createOutputChannel).toHaveBeenCalledWith('RangeLink');
 
@@ -3284,6 +3284,8 @@ describe('Extension lifecycle', () => {
       'rangelink.bindToGitHubCopilotChat',
       'rangelink.bindToTerminal',
       'rangelink.bindToTextEditor',
+      'rangelink.bookmark.add',
+      'rangelink.bookmark.manage',
       'rangelink.copyLinkOnlyWithAbsolutePath',
       'rangelink.copyLinkOnlyWithRelativePath',
       'rangelink.copyLinkWithAbsolutePath',

packages/rangelink-vscode-extension/src/__tests__/helpers/createMockBookmarksStore.ts

@@ -0,0 +1,38 @@
+/**
+ * Mock BookmarksStore for testing
+ *
+ * Provides factory function to create mock bookmarks stores with sensible defaults.
+ */
+
+import { Result } from 'rangelink-core-ts';
+
+import type { Bookmark, BookmarksStore } from '../../bookmarks';
+
+/**
+ * Options for creating a mock bookmarks store
+ */
+export interface MockBookmarksStoreOptions {
+  /** Initial bookmarks to return from getAll (default: []) */
+  bookmarks?: Bookmark[];
+}
+
+/**
+ * Create a mock BookmarksStore for testing
+ *
+ * @param options - Optional configuration for the mock
+ * @returns Mock BookmarksStore with jest functions
+ */
+export const createMockBookmarksStore = (
+  options: MockBookmarksStoreOptions = {},
+): jest.Mocked<BookmarksStore> => {
+  const bookmarks = options.bookmarks ?? [];
+
+  return {
+    getAll: jest.fn(() => bookmarks),
+    getById: jest.fn((id: string) => bookmarks.find((b) => b.id === id)),
+    add: jest.fn().mockResolvedValue(Result.ok({})),
+    update: jest.fn().mockResolvedValue(Result.ok({})),
+    remove: jest.fn().mockResolvedValue(Result.ok(undefined)),
+    recordAccess: jest.fn().mockResolvedValue(Result.ok(undefined)),
+  } as unknown as jest.Mocked<BookmarksStore>;
+};

packages/rangelink-vscode-extension/src/__tests__/helpers/index.ts

@@ -5,6 +5,7 @@
  */
 
 export * from './createMockAsRelativePath';
+export * from './createMockBookmarksStore';
 export * from './createMockCancellationToken';
 export * from './createMockClaudeCodeComposableDestination';
 export * from './createMockClaudeCodeDestination';