From b9769369efe243eabccc9a91515aa850aeb555f5 Mon Sep 17 00:00:00 2001 From: joaovictor91123 Date: Wed, 22 Jul 2026 18:29:39 +0400 Subject: [PATCH] fix(extension): remove dead matchPullRequestTarget from content.js matchPullRequestTarget duplicated matchGitHubPageTarget's logic (minus the kind field) and was stranded by #7487's issue-page match removal. Confirmed via grep it's referenced nowhere else, including production code and the __loopoverContentInternals test-hook object. Closes #8023 --- apps/loopover-extension/content.js | 7 ------- 1 file changed, 7 deletions(-) diff --git a/apps/loopover-extension/content.js b/apps/loopover-extension/content.js index 29f94aed4a..f5e58b5306 100644 --- a/apps/loopover-extension/content.js +++ b/apps/loopover-extension/content.js @@ -13,12 +13,6 @@ function matchGitHubPageTarget(pathname) { return { kind: "pull_request", owner, repo, pullNumber: Number(number) }; } -function matchPullRequestTarget(pathname) { - const target = matchGitHubPageTarget(pathname); - if (!target) return null; - return { owner: target.owner, repo: target.repo, pullNumber: target.pullNumber }; -} - function mountOverlay(target) { if (document.querySelector("[data-loopover-pr-context]")) return; const container = document.createElement("aside"); @@ -192,7 +186,6 @@ function renderActions(body, actions) { if (globalThis.__LOOPOVER_EXTENSION_TEST__) { globalThis.__loopoverContentInternals = { matchGitHubPageTarget, - matchPullRequestTarget, createOverlayLoader, renderPullContext, renderSection,