fix: double-click word selection and pending request URL display

PortableSheep · Copilot · PortableSheep · commit e28188c579b6 · 2026-02-25T21:06:35.000-06:00
B1: Skip accidental-selection prevention for double/triple clicks
(event.detail &gt;= 2) so CodeMirror's native word/line selection works.

B4: Eagerly hydrate the request URL when building activeRequestInfo
so the pending request modal shows the resolved URL instead of
{{placeholder}} syntax. Falls back to responseData.processedUrl
once the request completes.

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/frontend/src/app/app.component.ts b/frontend/src/app/app.component.ts
@@ -78,6 +78,8 @@ import {
 import { buildStopActiveRunTickPatch } from './logic/active-run/active-run-stop.logic';
 import { decideActiveRunTickActions } from './logic/active-run/active-run-tick.logic';
 import { buildActiveRequestInfo, buildInitialLoadRunUiState } from './logic/request/active-request.logic';
+import { getCombinedVariablesForFile, getActiveEnvNameForFile } from './components/request-manager/env-vars';
+import { hydrateText } from './services/http/hydration';
 import { consumeQueuedRequest } from './logic/request/request-queue.logic';
 import { buildPendingRequestResetPatch } from './logic/request/pending-request-reset.logic';
 import { buildCancelActiveRequestErrorPatch, decideCancelActiveRequest } from './logic/request/cancel-active-request.logic';
@@ -537,6 +539,18 @@ export class AppComponent implements OnInit, OnDestroy {
     this.activeRequestInfo = buildActiveRequestInfo(activeFile.id, requestIndex, request, now);
     this.isCancellingActiveRequest = false;
 
+    // Eagerly hydrate the URL so the pending modal shows the resolved URL
+    const variables = getCombinedVariablesForFile(activeFile, this.currentEnvSignal());
+    const envName = getActiveEnvNameForFile(activeFile, this.currentEnvSignal());
+    const capturedId = this.activeRequestInfo.id;
+    hydrateText(request.url, variables, envName, (text, env) => this.secretService.replaceSecrets(text, env))
+      .then(resolved => {
+        if (this.activeRequestInfo?.id === capturedId) {
+          this.activeRequestInfo = { ...this.activeRequestInfo, processedUrl: resolved };
+        }
+      })
+      .catch(() => {});
+
     const loadRun = buildInitialLoadRunUiState(this.loadUsersSeriesMaxPoints, this.loadRpsSeriesMaxPoints);
     this.activeRunProgress = loadRun.activeRunProgress;
 
@@ -891,7 +905,8 @@ export class AppComponent implements OnInit, OnDestroy {
   getActiveRequestPreview(): string {
     const request = this.getActiveRequestDetails();
     const processedUrl = this.activeRequestInfo
-      ? this.currentFile.responseData?.[this.activeRequestInfo.requestIndex]?.processedUrl
+      ? (this.currentFile.responseData?.[this.activeRequestInfo.requestIndex]?.processedUrl
+         ?? this.activeRequestInfo.processedUrl)
       : undefined;
     return buildActiveRequestPreview(request, processedUrl);
   }
@@ -903,7 +918,8 @@ export class AppComponent implements OnInit, OnDestroy {
   getActiveRequestMeta(): string {
     const request = this.getActiveRequestDetails();
     const processedUrl = this.activeRequestInfo
-      ? this.currentFile.responseData?.[this.activeRequestInfo.requestIndex]?.processedUrl
+      ? (this.currentFile.responseData?.[this.activeRequestInfo.requestIndex]?.processedUrl
+         ?? this.activeRequestInfo.processedUrl)
       : undefined;
     return buildActiveRequestMeta({
       activeRequestInfo: this.activeRequestInfo,
diff --git a/frontend/src/app/components/editor/editor.component.ts b/frontend/src/app/components/editor/editor.component.ts
@@ -348,7 +348,9 @@ export class EditorComponent implements AfterViewInit, OnDestroy {
               }
             }
 
-            if (!event.shiftKey && !event.ctrlKey && !event.metaKey && !event.altKey) {
+            // Skip accidental-selection prevention for double/triple clicks
+            // (event.detail >= 2) so word/line selection works naturally.
+            if (!event.shiftKey && !event.ctrlKey && !event.metaKey && !event.altKey && event.detail < 2) {
               const scrollTop = view.scrollDOM.scrollTop;
               const hadSelection = !view.state.selection.main.empty;
               const downX = event.clientX;
diff --git a/frontend/src/app/logic/request/active-request.logic.ts b/frontend/src/app/logic/request/active-request.logic.ts
@@ -10,6 +10,7 @@ export type ActiveRequestInfo = {
   canCancel: boolean;
   type: 'single' | 'chain' | 'load';
   startedAt: number;
+  processedUrl?: string;
 };
 
 export type LoadRunUiState = {

Original file line number	Diff line number	Diff line change
`@@ -348,7 +348,9 @@ export class EditorComponent implements AfterViewInit, OnDestroy {`
`348`	`348`	`}`
`349`	`349`	`}`
`350`	`350`
`351`		`- if (!event.shiftKey && !event.ctrlKey && !event.metaKey && !event.altKey) {`
	`351`	`+ // Skip accidental-selection prevention for double/triple clicks`
	`352`	`+ // (event.detail >= 2) so word/line selection works naturally.`
	`353`	`+ if (!event.shiftKey && !event.ctrlKey && !event.metaKey && !event.altKey && event.detail < 2) {`
`352`	`354`	`const scrollTop = view.scrollDOM.scrollTop;`
`353`	`355`	`const hadSelection = !view.state.selection.main.empty;`
`354`	`356`	`const downX = event.clientX;`