feat: update buildTitle functions to include URL context for dynamic titles

Author: lollipop-onl

src/entrypoints/content/defineUnfurler.ts

@@ -2,7 +2,7 @@ import { replaceUrlInTextNodes } from "./replaceUrlInTextNodes";
 
 type UnfurlerObject<T> = {
 	parseUrl: (url: URL) => T | null | undefined;
-	buildTitle: (params: T) => string | Promise<string>;
+	buildTitle: (params: T, url: URL) => string | Promise<string>;
 };
 
 export const defineUnfurler = <T>(unfurler: UnfurlerObject<T>) => {
@@ -11,6 +11,6 @@ export const defineUnfurler = <T>(unfurler: UnfurlerObject<T>) => {
 		const parsed = unfurler.parseUrl(url);
 		if (!parsed) return;
 
-		replaceUrlInTextNodes(el, await unfurler.buildTitle(parsed));
+		replaceUrlInTextNodes(el, await unfurler.buildTitle(parsed, url));
 	};
 };

src/entrypoints/content/unfurlers.ts

@@ -11,7 +11,12 @@ const PULL_REQUEST_NUMBER_REGEX = "(?<number>[0-9]+)" as const;
 export const issueUnfurler = defineUnfurler({
 	parseUrl: (url) =>
 		regex(`^/view/${ISSUE_KEY_REGEX}$`).exec(url.pathname)?.groups,
-	buildTitle: (params) => `Issue: ${params.issueKey}`,
+	buildTitle: (params, url) => {
+		if (url.hash.startsWith("#comment-")) {
+			return `Comment on ${params.issueKey}`;
+		}
+		return `Issue: ${params.issueKey}`;
+	},
 });
 
 export const wikiUnfurler = defineUnfurler({
@@ -33,6 +38,10 @@ export const pullRequestUnfurler = defineUnfurler({
 		regex(
 			`^/git/${PROJECT_KEY_REGEX}/${REPOSITORY_REGEX}/pullRequests/${PULL_REQUEST_NUMBER_REGEX}$`,
 		).exec(url.pathname)?.groups,
-	buildTitle: (params) =>
-		`Pull Request: ${params.projectKey}/${params.repository}#${params.number}`,
+	buildTitle: (params, url) => {
+		if (url.hash.startsWith("#comment-")) {
+			return `Comment on Pull Request #${params.number}`;
+		}
+		return `Pull Request #${params.number}`;
+	},
 });