[issues/594] Refactor QuickPick item shape: single source of truth via nested info objects

.github/actions/run-integration-tests/action.yml

@@ -31,7 +31,7 @@ runs:
       run: |
         set -o pipefail
         xvfb-run --auto-servernum --server-args="-screen 0 1280x960x24" \
-          pnpm test:release:automated --exclude-label requires-extensions --exclude-label cursor 2>&1 | tee /tmp/test-report-automated.txt
+          pnpm test:release:automated 2>&1 | tee /tmp/test-report-automated.txt
 
     - name: Extract integration test stats
       id: stats

packages/rangelink-vscode-extension/package.json

@@ -51,7 +51,7 @@
     "test:coverage": "jest --coverage --coverageReporters=text --coverageReporters=text-summary --coverageReporters=html",
     "test:fast": "jest --coverage --testPathIgnorePatterns '<rootDir>/src/__integration-tests__/' '\\.integration\\.test\\.ts$'",
     "test:release": "./scripts/run-integration-tests.sh",
-    "test:release:automated": "./scripts/run-integration-tests.sh --automated",
+    "test:release:automated": "./scripts/run-integration-tests.sh --automated --exclude-label requires-extensions --exclude-label cursor",
     "test:release:grep": "./scripts/run-integration-tests.sh --grep",
     "test:release:prepare": "pnpm compile && rm -rf out/__integration-tests__ && tsc -p tsconfig.integration.json && node scripts/setup-integration-test-settings.js",
     "test:release:with-extensions": "./scripts/run-integration-tests.sh --with-extensions",

packages/rangelink-vscode-extension/src/LogCapture.ts

@@ -1,15 +1,9 @@
 import type * as vscode from 'vscode';
 
+import { ENV_RANGELINK_CAPTURE_LOGS } from './constants';
 import { RangeLinkExtensionError } from './errors/RangeLinkExtensionError';
 import { RangeLinkExtensionErrorCodes } from './errors/RangeLinkExtensionErrorCodes';
 
-/**
- * Environment variable that enables in-memory log capture.
- * Set by the integration test runner before launching VS Code.
- * When absent or not 'true', LogCapture is a transparent proxy with zero overhead.
- */
-const ENV_CAPTURE_LOGS = 'RANGELINK_CAPTURE_LOGS';
-
 /**
  * Wraps an OutputChannel to optionally capture log lines in memory.
  *
@@ -27,7 +21,7 @@ export class LogCapture {
   private readonly captureEnabled: boolean;
 
   constructor(private readonly outputChannel: vscode.OutputChannel) {
-    this.captureEnabled = process.env[ENV_CAPTURE_LOGS] === 'true';
+    this.captureEnabled = process.env[ENV_RANGELINK_CAPTURE_LOGS] === 'true';
   }
 
   /**
@@ -91,7 +85,7 @@ export class LogCapture {
     if (!this.captureEnabled) {
       throw new RangeLinkExtensionError({
         code: RangeLinkExtensionErrorCodes.LOG_CAPTURE_DISABLED,
-        message: `LogCapture.${methodName}() called without RANGELINK_CAPTURE_LOGS=true — this method is for integration tests only`,
+        message: `LogCapture.${methodName}() called without ${ENV_RANGELINK_CAPTURE_LOGS}=true — this method is for integration tests only`,
         functionName: `LogCapture.${methodName}`,
       });
     }

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

@@ -83,11 +83,7 @@ describe('BindToTextEditorCommand', () => {
         outcome: 'bound',
         bindInfo: { destinationName: 'app.ts', destinationKind: 'text-editor' },
       });
-      expect(mockDestinationManager.bind).toHaveBeenCalledWith({
-        kind: 'text-editor',
-        uri: eligibleFile.uri,
-        viewColumn: 1,
-      });
+      expect(mockDestinationManager.bind).toHaveBeenCalledWith(eligibleFile.bindOptions);
       expect(mockLogger.debug).toHaveBeenCalledWith(
         { fn: 'BindToTextEditorCommand.executeWithPicker', filename: 'app.ts' },
         'Single file, auto-binding',
@@ -134,11 +130,7 @@ describe('BindToTextEditorCommand', () => {
         outcome: 'bound',
         bindInfo: { destinationName: 'utils.ts', destinationKind: 'text-editor' },
       });
-      expect(mockDestinationManager.bind).toHaveBeenCalledWith({
-        kind: 'text-editor',
-        uri: eligibleFile2.uri,
-        viewColumn: 2,
-      });
+      expect(mockDestinationManager.bind).toHaveBeenCalledWith(eligibleFile2.bindOptions);
       expect(mockLogger.debug).toHaveBeenCalledWith(
         { fn: 'BindToTextEditorCommand.executeWithPicker', fileCount: 2 },
         'Starting bind to text editor command',