fix: improve shortUrl to handle pptr: and encoded URLs

- Decode percent-encoded URLs before extracting basename so that
  pptr:evaluate;file%3A%2F%2F... resolves to the inner filename.
- Recursively unwrap parenthesised inner URLs for the
  pptr:evaluateHandle;performEvaluation (file://...) form.
- Fix timeout values: 5s for concise, 10s for detailed (was both 10s).
- Use shortUrl for uiSourceCode paths too, falling back to displayName.

Author: masamaru0513

src/formatters/ConsoleFormatter.ts

@@ -522,9 +522,10 @@ function extractLocationFromStack(
   }
   if (frame.uiSourceCode) {
     const uiLocation = frame.uiSourceCode.uiLocation(frame.line, frame.column);
+    const url = uiLocation.uiSourceCode.url();
     return {
-      url: uiLocation.uiSourceCode.url(),
-      displayName: uiLocation.uiSourceCode.displayName(),
+      url,
+      displayName: shortUrl(url) || uiLocation.uiSourceCode.displayName(),
       // UILocation stores 0-based line/column indices.
       lineNumber: uiLocation.lineNumber + 1,
       columnNumber: (uiLocation.columnNumber ?? 0) + 1,
@@ -593,7 +594,7 @@ async function withResolveTimeout<T>(
   promise: Promise<T>,
   fetchDetailedData: boolean | undefined,
 ): Promise<T | undefined> {
-  const timeoutMs = fetchDetailedData ? 10_000 : 10_000;
+  const timeoutMs = fetchDetailedData ? 10_000 : 5_000;
   let timeoutId: ReturnType<typeof setTimeout> | undefined;
   const timeoutPromise = new Promise<undefined>(resolve => {
     timeoutId = setTimeout(() => resolve(undefined), timeoutMs);
@@ -610,21 +611,53 @@ async function withResolveTimeout<T>(
 /**
  * Extracts a short, human-readable name from a URL by returning the last path
  * segment. Used as a fallback when no source-mapped UI location is available.
+ *
+ * Handles a few special cases that come up in practice:
+ * - puppeteer-injected scripts use a URL of the form
+ *   `pptr:evaluateHandle;performEvaluation (file:///path/to/file.js:104:34)`
+ *   or the URL-encoded variant
+ *   `pptr:evaluate;file%3A%2F%2F%2Fpath%2Fto%2Ffile.js%3A104%3A18`,
+ *   from which we extract the inner file's basename.
+ * - URLs that already have a trailing `:line:column` suffix are stripped
+ *   before extracting the basename.
  */
 function shortUrl(url: string): string {
   if (!url) {
     return '';
   }
+  // Decode percent-encoded characters so embedded paths are extractable.
+  let decoded = url;
   try {
-    const parsed = new URL(url);
-    const segments = parsed.pathname.split('/').filter(s => s.length > 0);
-    if (segments.length > 0) {
-      return segments[segments.length - 1];
+    decoded = decodeURIComponent(url);
+  } catch {
+    // Leave url as-is when it is not validly percent-encoded.
+  }
+  // Recursively unwrap parenthesised inner URLs (puppeteer-style).
+  const parenMatch = decoded.match(/\(([^()]+)\)/);
+  if (parenMatch) {
+    const inner = shortUrl(parenMatch[1]);
+    if (inner) {
+      return inner;
+    }
+  }
+  // Strip trailing :line:col or :line so URL parsing succeeds.
+  const cleaned = decoded.replace(/:\d+(?::\d+)?$/, '');
+  try {
+    const parsed = new URL(cleaned);
+    const pathSegments = parsed.pathname.split('/').filter(s => s.length > 0);
+    if (pathSegments.length > 0) {
+      return pathSegments[pathSegments.length - 1];
+    }
+    // Some custom schemes (e.g. `pptr:`) leave the path in the opaque part.
+    // Fall back to the last `/`-separated segment of the trimmed URL.
+    const lastSlash = cleaned.lastIndexOf('/');
+    if (lastSlash >= 0 && lastSlash < cleaned.length - 1) {
+      return cleaned.slice(lastSlash + 1);
     }
-    return parsed.host || url;
+    return parsed.host || cleaned;
   } catch {
     // Not a valid URL (e.g. VM-backed inline scripts).
-    const lastSlash = url.lastIndexOf('/');
-    return lastSlash >= 0 ? url.slice(lastSlash + 1) : url;
+    const lastSlash = cleaned.lastIndexOf('/');
+    return lastSlash >= 0 ? cleaned.slice(lastSlash + 1) : cleaned;
   }
 }

tests/McpResponse.test.js.snapshot

@@ -184,7 +184,7 @@ exports[`McpResponse > adds color scheme emulation setting when it is set 2`] =
 exports[`McpResponse > adds console messages when the setting is true 1`] = `
 ## Console messages
 Showing 1-1 of 1 (Page 1 of 1).
-msgid=1 [log] Hello from the test (1 args) pptr:evaluate;file%3A%2F%2F%2FUsers%2Fmasahiro.kokubo%2FDocuments%2FGitHub%2Fchrome-devtools-mcp%2F…:2:25
+msgid=1 [log] Hello from the test (1 args) McpResponse.test.js:2:25
 `;
 
 exports[`McpResponse > adds console messages when the setting is true 2`] = `
@@ -206,7 +206,7 @@ exports[`McpResponse > adds console messages when the setting is true 2`] = `
       "id": 1,
       "location": {
         "url": "pptr:evaluate;<file-path>",
-        "displayName": "pptr:evaluate;<file-path>",
+        "displayName": "McpResponse.test.js",
         "lineNumber": 2,
         "columnNumber": 25
       }