pi-ui-extend 0.1.27 → 0.1.28

package/dist/app/screen/file-link-opener.js

@@ -11,6 +11,8 @@ export function setFileLinkOpenerTestDeps(overrides) {
     };
 }
 export function openFileLink(link) {
+    if (isWebUrl(link.url))
+        return openPathWithSystemViewer(link.url);
     const filePath = link.filePath ?? filePathFromUrl(link.url);
     if (!filePath)
         return false;
@@ -19,6 +21,9 @@ export function openFileLink(link) {
         return true;
     return openPathWithSystemViewer(filePath);
 }
+function isWebUrl(url) {
+    return url.startsWith("http://") || url.startsWith("https://");
+}
 function filePathFromUrl(url) {
     if (!url.startsWith("file://"))
         return undefined;

package/dist/app/screen/file-links.js

@@ -3,10 +3,11 @@ import { homedir } from "node:os";
 import { isAbsolute, resolve } from "node:path";
 import { fileURLToPath, pathToFileURL } from "node:url";
 const FILE_PATH_CANDIDATE = /(?<![\p{L}\p{N}_:])((?:file:\/\/\/|~[\\/]|\.{1,2}[\\/]|[A-Za-z]:[\\/]|[\\/]|[A-Za-z0-9_.@-]+[\\/])[^\s"'`<>]*)/gu;
+const WEB_URL_CANDIDATE = /https?:\/\/[^\s"'`<>]+/gu;
 const TRAILING_PUNCTUATION = new Set([".", ",", ";", ")", "]", "}"]);
 export function detectFileLinks(text, cwd) {
     const links = [];
-    if (!text.includes("/") && !text.includes("\\"))
+    if (!text.includes("/") && !text.includes("\\") && !text.includes("http://") && !text.includes("https://"))
         return links;
     for (const match of text.matchAll(FILE_PATH_CANDIDATE)) {
         const raw = match[1];
@@ -28,6 +29,17 @@ export function detectFileLinks(text, cwd) {
             column: location.column,
         });
     }
+    for (const match of text.matchAll(WEB_URL_CANDIDATE)) {
+        const raw = match[0];
+        const candidate = trimTrailingPunctuation(raw);
+        if (!candidate)
+            continue;
+        links.push({
+            start: match.index,
+            end: match.index + candidate.length,
+            url: candidate,
+        });
+    }
     return mergeOverlappingLinks(links);
 }
 export function hyperlink(text, url) {

package/dist/app/screen/mouse-controller.d.ts

@@ -139,6 +139,7 @@ export declare class AppMouseController {
     private inputClickFlashRegionForEvent;
     private imageTargetAt;
     private fileLinkAt;
+    private fileLinkTargetAt;
     private statusTargetAt;
     private handleImageClick;
     private handleFileLinkClick;

package/dist/app/screen/mouse-controller.js

@@ -60,6 +60,8 @@ export class AppMouseController {
         this.showClickFlashOnPress(event);
         if (event.button === 0 && !event.released && this.handleInputBorderStatusClick(event))
             return;
+        if (event.button === 0 && !event.released && this.fileLinkAt(event))
+            return;
         if (this.handleMouseSelection(event))
             return;
         if (this.withClickFlash(event, () => this.handleImageClick(event)))
@@ -234,9 +236,9 @@ export class AppMouseController {
         const imageTarget = this.imageTargetAt(event);
         if (imageTarget)
             return { y: event.y, startColumn: imageTarget.start + 1, endColumn: imageTarget.end + 1 };
-        const link = this.fileLinkAt(event);
-        if (link)
-            return { y: event.y, startColumn: link.start + 1, endColumn: link.end + 1 };
+        const linkTarget = this.fileLinkTargetAt(event);
+        if (linkTarget)
+            return { y: event.y, startColumn: linkTarget.startColumn, endColumn: linkTarget.endColumn };
         const tabTarget = this.tabLineTargetAt(event);
         if (tabTarget)
             return { y: tabTarget.row, startColumn: tabTarget.startColumn, endColumn: tabTarget.endColumn };
@@ -290,10 +292,19 @@ export class AppMouseController {
         return targets?.find((candidate) => event.x >= candidate.start + 1 && event.x <= candidate.end);
     }
     fileLinkAt(event) {
+        return this.fileLinkTargetAt(event)?.link;
+    }
+    fileLinkTargetAt(event) {
         const text = this.renderedRowTexts.get(event.y);
         if (!text)
             return undefined;
-        return detectFileLinks(text, this.host.cwd()).find((candidate) => event.x >= candidate.start + 1 && event.x <= candidate.end);
+        for (const link of detectFileLinks(text, this.host.cwd())) {
+            const startColumn = stringDisplayWidth(text.slice(0, link.start)) + 1;
+            const endColumn = startColumn + stringDisplayWidth(text.slice(link.start, link.end));
+            if (event.x >= startColumn && event.x < endColumn)
+                return { link, startColumn, endColumn };
+        }
+        return undefined;
     }
     statusTargetAt(event) {
         const target = [
@@ -323,7 +334,7 @@ export class AppMouseController {
         };
     }
     handleImageClick(event) {
-        if (event.button !== 0 || !event.released)
+        if (!isPrimaryButtonRelease(event))
             return false;
         const imageTarget = this.imageTargetAt(event);
         if (!imageTarget)
@@ -341,7 +352,7 @@ export class AppMouseController {
     }
     handleFileLinkClick(event) {
         const modifiedPress = isModifiedPrimaryButton(event.button) && !event.released;
-        const plainRelease = event.button === 0 && event.released;
+        const plainRelease = isPrimaryButtonRelease(event);
         if (!modifiedPress && !plainRelease)
             return false;
         const link = this.fileLinkAt(event);
@@ -1053,6 +1064,9 @@ function isModifiedPrimaryButton(button) {
     const modifierBits = button & (8 | 16);
     return primaryButton && modifierBits !== 0;
 }
+function isPrimaryButtonRelease(event) {
+    return event.released && (event.button === 0 || (event.button & 3) === 3);
+}
 function editorLayoutRows(terminalRows, tabPanelRows) {
     return Math.max(1, terminalRows - tabPanelRows);
 }

package/extensions/session-title/index.ts

@@ -70,6 +70,10 @@ export function firstUserMessageText(ctx: ExtensionContext): string | undefined
 	return undefined;
 }
 
+function hasExistingUserMessage(ctx: ExtensionContext): boolean {
+	return firstUserMessageText(ctx) !== undefined;
+}
+
 export function fallbackSessionTitleFromInput(input: string, maxTitleChars: number): string | undefined {
 	const normalized = input
 		.replace(/[\t\r\n]+/gu, " ")
@@ -371,24 +375,6 @@ export default function sessionTitle(pi: ExtensionAPI) {
 		})();
 	}
 
-	function primeTitleGenerationFromExistingSession(ctx: ExtensionContext, currentConfig: SessionTitleConfig): void {
-		if (currentSessionName(ctx)) return;
-
-		const input = firstUserMessageText(ctx);
-		if (!input) return;
-		if (!currentConfig.enabled) {
-			applyFallbackSessionTitle(ctx, currentConfig, input);
-			return;
-		}
-
-		pendingGeneration = {
-			sessionId: ctx.sessionManager.getSessionId(),
-			input: truncateInput(input, currentConfig.maxInputChars),
-			attempts: 0,
-		};
-		startTitleGeneration(ctx, currentConfig);
-	}
-
 	function isSameSessionPath(left: string | undefined, right: string | undefined): boolean {
 		if (!left || !right) return false;
 		if (left === right) return true;
@@ -447,7 +433,6 @@ export default function sessionTitle(pi: ExtensionAPI) {
 		await prepareForkTitleState(event, ctx);
 		refreshSessionUi(ctx, { force: true });
 		scheduleSessionUiRefresh(ctx);
-		if (!forkTitleState) primeTitleGenerationFromExistingSession(ctx, config);
 	});
 
 	pi.on("session_shutdown", async () => {
@@ -480,6 +465,10 @@ export default function sessionTitle(pi: ExtensionAPI) {
 		sessionId = currentSessionId;
 		const currentName = currentSessionName(ctx);
 		const activeForkTitleState = forkTitleState?.sessionId === currentSessionId ? forkTitleState : undefined;
+		if (!activeForkTitleState && hasExistingUserMessage(ctx)) {
+			forkTitleState = undefined;
+			return { action: "continue" as const };
+		}
 		if (currentName && (!activeForkTitleState || currentName !== activeForkTitleState.inheritedSessionName)) {
 			forkTitleState = undefined;
 			return { action: "continue" as const };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-ui-extend",
-  "version": "0.1.27",
+  "version": "0.1.28",
   "private": false,
   "type": "module",
   "bin": {