Largest PR Test by enif-lee · Pull Request #25 · enif-lee/pr-plus

enif-lee · 2026-08-15T06:02:16Z

Why this PR exists

Largest-PR fixture for pr+ load testing: Conversation first paint, Diff virtualization, and ~100 inline review threads scattered across the tree.

Open from the pulls list (?prp_page=conversation&prp_number=25) or the native Files tab. Safe to leave open — not intended to merge.

Shape

Surface	Approx.
Files	658
Additions	~195k
Deletions	~2.5k
Inline review threads	~105
Files with threads	68

Base is demo/d ← head refactor/phase-9-large-sot-split (82f1cda). Threads are tagged [perf-fixture] so they can be grepped or removed later.

Suggested exercises

List → modal handoff — title, labels, and this milestone should sketch in before core fetch.
Conversation — scroll the description, task list, mermaid, and timeline.
Diff — jump files, hold j/k, open several [perf-fixture] threads, expand a resolved/unresolved mix.
Leave and reopen from /pulls twice — title/body should still first-paint from cache.

flowchart LR
  List["/pulls sketch"] --> Modal["pr+ modal"]
  Modal --> Conv[Conversation]
  Modal --> Diff[Diff + threads]
  Diff --> Virtual[Virtual rows]
  Diff --> Threads["~105 inline threads"]

Checklist

Large file tree (host / modal / fetch / tests)
Scattered inline comments ([perf-fixture])
Labels + milestone for header / aside chrome
Optional: add a few replies under a thread to stress lazy comments
Optional: resolve a handful and confirm deferred bodies

Sample snippet (ignore — fixture only)

function chooseOpenFirstPaintDetail(opts) {
  // memory (rich) → IDB title/body → list sketch
  return { detail, source, keepCache };
}

Demo only. Review comments are synthetic load, not a real review of Phase 9.

Allows APPROVE / REQUEST_CHANGES on demo PRs without self-review, so list badges can show Approved and Changes requested states.

Include a live PR list capture showing tree indents, review badges, and branch chips so the product overview is easier to understand at a glance.

Keep only the screenshot under the overview section.

Translate product overview, features, install, PAT notes, and development docs while keeping logo and demo screenshot placement.

Align manifest, package, and README badge for the latest store package.

Document data handling, optional PAT storage, host/storage justifications, and link the policy from the README for store submission.

Commit the logo and demo screenshot under .jpeg extensions so GitHub renders the README images correctly.

Open a centered React modal from the PR list without full navigation, expand to near-fullscreen Diff with file tree and virtualized rows, support Ctrl/Cmd+F across off-screen content, and wire conversation, reviews, checks, and line comments through the background API bridge.

Query refinements that keep hitIndex at 0 still scroll to the new match via resolveQuerySearchState; next/prev wraps a single hit the same way. Host reuses the React root on prop updates so layout, scroll, and search state survive detail refresh, and the modal bundle maps createRoot correctly for production IIFE builds.

Rebuild the modal with shadcn-style components and GitHub color-mode theming, conversation overview with sanitized markdown, nested file tree, syntax-highlighted virtualized diffs, default-collapsed large/generated files via gitattributes signals, and inline review comments with next/prev navigation helpers.

Treat linguist-generated=true as enabled, load collapse helper in the service worker, and re-annotate files in the modal so fetched attributes override weak SW defaults. Cover non-hardcoded *.pb.go paths in pure and fetchPrDetail tests.

…uting Ship a production-ready PR modal overhaul on the existing virtualized diff foundation: - Refactor modal into TypeScript modules (views, common components, zustand store, pure libs) with esbuild IIFE bundle and real typecheck. - Markdown Write/Preview composers (no B/I toolbar), paste/drop attachments via Contents API, hljs snippets, anchored searchable selects, header subscribe, conditional checks, merge-box check detail, icon edit/delete, thread collapse, and optimistic reply merge that survives host rehydrate. - Deep-link URI (pr+page / pr+number / pr+position) with history.replaceState; restore order sessionStorage then URI; clear residual pr+ and spaced keys on close; reset comment focus per PR open. - Host/content restore path, upload bridge, and expanded unit/fixture tests including URI, attach merge, and memo render counts.

Add an icon control in the PR detail header to switch between the centered modal shell and a right-docked side sheet. Preference is stored in localStorage (prp:shell) and reapplied on the next open; conversation, diff, and other features stay the same under both shells.

Rename Close PR to Close, remove the Commands button (palette remains available via keyboard), and replace coarse loading blocks with finer conversation/diff/header skeletons so stats and action chrome no longer shift layout while loading.

Replace the Refreshing… status strip with a 2px accent progress overlay on the modal top edge so soft revalidation no longer shifts layout.

Reset font-family on inline thread and comment rows so review text uses the same proportional UI font as conversation view, not the mono code font inherited from the virtualized diff list. File paths stay monospace.

Allow collapsing the Diff file tree to a thin expand rail and dragging a vertical handle to resize navigator width (clamped). Persist collapsed state and width in localStorage (prp:file-nav) for subsequent opens.

When the shell preference is side sheet, conversation stays a right-docked panel but Diff expands to full viewport like the modal Diff layout so the file tree and virtualized list have full width.

Remove dim/blur overlay when shell is side sheet so the underlying GitHub list stays fully visible; only the sheet panel remains opaque.

Keep overlay hosts transparent for both shells, drop blur that washed to white, and use a light dark-alpha dim only in modal shell so the page stays visible. Side sheet backdrop remains fully transparent.

Set conversation side-sheet min width to at least 800px (viewport-capped on smaller windows) and max to 1200px via clamp, so the panel stays usable.

Match modal dimming (light alpha veil, no blur) for side-sheet mode so the underlying page is slightly darkened under the transparent overlay.

Increase conversation side-sheet minimum from 800px by ~150px for more usable conversation/aside layout.

Play sheet slide-out or modal scale-out before unmount, and fade the dim backdrop, so closing matches open transitions for both shells.

…fsets Measure the virtual list pane with ResizeObserver instead of a fixed 520px height, use prefix offsets for comment-heavy variable rows, and raise comment row estimates so the scroll spacer no longer clips bottom content.

Render real profile photos via github.com/{login}.png (and API avatar_url when present) with initials fallback on error, across conversation threads, diff review threads, and people meta lists.

Move overflow scrolling to an outer wrapper so the vertical rail spans the full commit list height, and add per-item segments so 4+ commits stay linked.

Render GitHub profile images in Add reviewer/assignee options and color swatches in Add label options, using option meta from builders.

Show mergeability, draft, mergeable_state, and checks as prominent badges in the merge box, with stronger box styling for ok/warn/muted states.

Add Diff commit/range filtering via GitHub compare, link git refs and SHAs, replace window.prompt with inline editors/pickers, and fix MV3 message-channel closes with Promise-based handlers, keep-alive, and retries. Ignore packaged .crx artifacts.

enif-lee

probe line/side

enif-lee · 2026-08-15T06:19:11Z

+    } catch {
+      /* ignore */
+    }
+    void refreshPrefs();


[perf-probe] line/side L10

enif-lee

probe position

enif-lee · 2026-08-15T06:19:12Z

+    } catch {
+      /* ignore */
+    }
+    void refreshPrefs();


[perf-probe] position 10

enif-lee

probe high-line

enif-lee · 2026-08-15T06:19:24Z

+              .catch((err) => {
+                creditAllThreadStages();
+                try {
+                  for (const id of [HOST_ID, embedHostId()]) {


enif-lee

probe mid-line

enif-lee · 2026-08-15T06:19:26Z

+                    : []),
+                ],
+              };
+              const cleanedCacheRc = filterFn


enif-lee

probe dts

enif-lee · 2026-08-15T06:19:27Z

+  code?: string;
+  pullNumber?: number;
+  graphqlErrors?: any;
+}


enif-lee

probe css

enif-lee · 2026-08-15T06:19:28Z

+  width: var(--prp-shell-w, 92vw);
+  height: var(--prp-shell-h, 90vh);
+  max-width: 100vw;
+  max-height: 100vh;


enif-lee

probe html

enif-lee · 2026-08-15T06:19:29Z

@@ -1,30 +1,81 @@
 <!DOCTYPE html>
-<html lang="ko">
+<html lang="en">
 <head>
  <meta charset="utf-8" />
  <title>pr+</title>


enif-lee

Perf fixture batch t20 (20 comments). Prefix [perf-fixture].

enif-lee · 2026-08-15T06:19:47Z

+      // steal ⌥I into Diff/Conversation composers.
+      const finishReviewOpen = Boolean(
+        typeof document !== 'undefined' &&
+          document.querySelector('[data-prp-finish-review="1"]')


[perf-fixture 3/100] Question: still correct after host-data-first?

src/modal/hooks/usePrModalHotkeys.ts:607

enif-lee · 2026-08-15T06:19:47Z

+        ? liveDiffThreadFocused
+        : liveConvAnchor.startsWith('review-comment:');
+      // Code-body selection only — thread/file carets must not force file fold
+      // (otherwise ← / ⌥F close the file while a thread is focused via ↑↓).


[perf-fixture 4/100] Perf-load thread.

file: src/modal/hooks/usePrModalHotkeys.ts

line: 1213

enif-lee · 2026-08-15T06:19:47Z

+  const mentionCandidates = b.mentionCandidates;
+  const milestoneAddRef = b.milestoneAddRef;
+  const modalSize = b.modalSize;
+  const mut = b.mut;


[perf-fixture 5/100] Virtualization note.

src/modal/hooks/usePrModalSessionRoute.ts:574

enif-lee · 2026-08-15T06:19:47Z

+    /**
+     * Keep re-requesting conversation nav until focused.
+     * Virtual list may miss the first paint when the row is off-window / not
+     * yet in the progressive feed — also kick timeline/thread pagination.


[perf-fixture 6/100] Slightly longer body so threads are not all one-liners. Used to exercise Diff inline-thread rendering under load.

enif-lee · 2026-08-15T06:19:47Z

+                    creditRefreshThreadLadder();
+                  }
+                  return r;
+                });


[perf-fixture 7/100] Scatter comment on src/host/modules/props-build.ts:505.

enif-lee · 2026-08-15T06:19:48Z

+  const runDiffContextThreadAction = b.runDiffContextThreadAction;
+  const runPaletteCommand = b.runPaletteCommand;
+  const sampleDiffNav = b.sampleDiffNav;
+  const scheduleDiffReviewFilter = b.scheduleDiffReviewFilter;


[perf-fixture 18/100] Slightly longer body so threads are not all one-liners. Used to exercise Diff inline-thread rendering under load.

enif-lee · 2026-08-15T06:19:48Z

+  }
+  to {
+    opacity: 0.92;
+    transform: translateX(32px);


[perf-fixture 19/100] Scatter comment on src/modal/views/chrome/ShellLayout.css:350.

enif-lee · 2026-08-15T06:19:48Z

+    0 0 0 1.5px var(--prp-bg),
+    0 0 0 2.5px var(--prp-danger);
+}
+.prp-header__meta-stack__item--check-success {


[perf-fixture 20/100] Nit: second look at src/modal/views/chrome/ShellLayout.css L699.

enif-lee · 2026-08-15T06:19:48Z

+  if (hostHasPending) {
+    for (const c of nextRc) {
+      if (c?.pending && c.id != null) deletedReviewIds.delete(String(c.id));
+    }


[perf-fixture 21/100] Question: still correct after host-data-first?

src/modal/lib/composer-attach.ts:338

enif-lee · 2026-08-15T06:19:48Z

+    ? next.reviewThreads
+    : Array.isArray(prev.reviewThreads)
+      ? prev.reviewThreads
+      : next.reviewThreads;


[perf-fixture 22/100] Perf-load thread.

file: src/modal/lib/composer-attach.ts

line: 675

enif-lee

Perf fixture batch r1-20 (20 comments). Prefix [perf-fixture].

enif-lee · 2026-08-15T06:20:06Z

+  isContextThreadReplyFocused,
+  isContextThreadCommentActive,
+  PRP_CONTEXT_THREAD_TAB_LEAVE,
+  scrollChildToMaximizeInScroller,


[perf-fixture 23/100] Virtualization note.

src/modal/hooks/useContextThreadActions.ts:333

enif-lee · 2026-08-15T06:20:06Z

+  const scrollDiffCaretIntoView = b.scrollDiffCaretIntoView;
+  const scrollDiffPage = b.scrollDiffPage;
+  const scrollDiffThreadUnitIntoView = b.scrollDiffThreadUnitIntoView;
+  const scrollFileNavRowIntoView = b.scrollFileNavRowIntoView;


[perf-fixture 24/100] Slightly longer body so threads are not all one-liners. Used to exercise Diff inline-thread rendering under load.

enif-lee · 2026-08-15T06:20:06Z

+          Comment
+        </button>
+      ) : null}
+    </div>


[perf-fixture 25/100] Scatter comment on src/modal/views/diff/VirtualDiffRows.tsx:328.

enif-lee · 2026-08-15T06:20:06Z

+  const qForRow = isSearchMatch ? searchQuery : '';
+
+  // Leaf store subscription: only this row re-renders when its key changes.
+  // Middles stay "middle" under multi extend → no re-render. Override for tests.


[perf-fixture 26/100] Nit: second look at src/modal/views/diff/VirtualDiffRows.tsx L655.

enif-lee · 2026-08-15T06:20:06Z

+    // Yield between chunks (skip after last)
+    if (end < n) {
+      await yieldToMain();
+    }


[perf-fixture 27/100] Question: still correct after host-data-first?

src/modal/lib/search-index.ts:323

enif-lee · 2026-08-15T06:20:07Z

+      conversationCommentFocused: opts.conversationCommentFocused,
+      focusedReviewThread: opts.focusedReviewThread,
+      diffThreadFocused,
+      hasLineSelection,


[perf-fixture 38/100] Nit: second look at src/modal/lib/shortcut-policy-actions.ts L501.

enif-lee · 2026-08-15T06:20:07Z

+          : pathTarget.page || null;
+      const position =
+        r.position != null && String(r.position).trim()
+          ? String(r.position).trim()


[perf-fixture 39/100] Question: still correct after host-data-first?

src/host/modules/host-core-timeline-a.ts:201

enif-lee · 2026-08-15T06:20:07Z

+            tag === 'SCRIPT' ||
+            tag === 'STYLE' ||
+            tag === 'LINK' ||
+            tag === 'TEMPLATE' ||


[perf-fixture 40/100] Perf-load thread.

file: src/host/modules/host-core-timeline-a.ts

line: 401

enif-lee · 2026-08-15T06:20:07Z

+    // Real click opens SelectMenu + focuses the control.
+    return clickControl(hit.el);
+  }
+


[perf-fixture 41/100] Virtualization note.

src/host/modules/list-row-lifecycle.ts:187

enif-lee · 2026-08-15T06:20:07Z

+    try {
+      const d = document.getElementById('command-palette-pjax-container');
+      return Boolean(d?.open);
+    } catch {


[perf-fixture 42/100] Slightly longer body so threads are not all one-liners. Used to exercise Diff inline-thread rendering under load.

enif-lee

Perf fixture batch r21-40 (20 comments). Prefix [perf-fixture].

enif-lee · 2026-08-15T06:20:11Z

+      } else {
+        return Math.max(ROW_HEIGHT, Math.ceil(m));
+      }
+    }


[perf-fixture 43/100] Scatter comment on src/modal/components/common/utils.tsx:173.

enif-lee · 2026-08-15T06:20:11Z

+ */
+export function highlightCode(
+  code: unknown,
+  filePath?: unknown,


[perf-fixture 44/100] Nit: second look at src/modal/components/common/utils.tsx L345.

enif-lee · 2026-08-15T06:20:11Z

+
+export const useModalStore = create<ModalUiState>((set, get) => ({
+  layoutMode: LAYOUT_CENTERED as LayoutMode,
+  diffMode: 'unified',


[perf-fixture 45/100] Question: still correct after host-data-first?

src/modal/store/modal-store.ts:139

enif-lee · 2026-08-15T06:20:11Z

+          );
+        }
+      }
+    } catch {


[perf-fixture 46/100] Perf-load thread.

file: src/modal/store/modal-store.ts

line: 277

enif-lee · 2026-08-15T06:20:11Z

+  | 'checks'
+  | 'development';
+
+export interface HostLoadStage {


[perf-fixture 47/100] Virtualization note.

src/host/host-context.d.ts:20

enif-lee · 2026-08-15T06:20:12Z

+  let linkedIssues = [];
+  const developmentIssues: any[] = [];
+  const projects: any[] = [];
+  try {


[perf-fixture 58/100] Perf-load thread.

file: src/fetch/pr-detail.ts

line: 507

enif-lee · 2026-08-15T06:20:12Z

+    try {
+      const j = await res.json();
+      if (j?.message) detail = j.message;
+      // Surface field-level validation (common on 422 replies / review comments)


[perf-fixture 59/100] Virtualization note.

src/fetch/http.ts:244

enif-lee · 2026-08-15T06:20:12Z

+      (typeof performance !== 'undefined' && performance.now
+        ? performance.now()
+        : Date.now()) - t0;
+    recordGraphqlCostEntry({


[perf-fixture 60/100] Slightly longer body so threads are not all one-liners. Used to exercise Diff inline-thread rendering under load.

enif-lee · 2026-08-15T06:20:12Z

+      token,
+      { method: 'PATCH', body: { body: nextBody } }
+    );
+  } catch (err: any) {


[perf-fixture 61/100] Scatter comment on src/fetch/mutations-meta.ts:242.

enif-lee · 2026-08-15T06:20:12Z

+    token,
+    { method: 'POST', body }
+  );
+  return {


[perf-fixture 62/100] Nit: second look at src/fetch/mutations-meta.ts L483.

enif-lee

Perf fixture batch r41-50 (10 comments). Prefix [perf-fixture].

enif-lee · 2026-08-15T06:21:17Z

+      }
+    }
+  }
+}`;


[perf-fixture 63/100] Question: still correct after host-data-first?

src/fetch/timeline-items.ts:220

enif-lee · 2026-08-15T06:21:17Z

+        value: {
+          ...base,
+          event: 'merged',
+          commitId: node.commit?.oid || null,


[perf-fixture 64/100] Perf-load thread.

file: src/fetch/timeline-items.ts

line: 439

enif-lee · 2026-08-15T06:21:17Z

+      body.appendChild(
+        el(
+          'p',
+          'prp-onboarding__hint prp-onboarding__hint--pulse',


[perf-fixture 65/100] Virtualization note.

src/onboarding-ui.ts:803

enif-lee · 2026-08-15T06:21:17Z

+  inset: 0;
+  /* Soft dim — match modal shell (no heavy wash / blur that reads as opaque) */
+  background: rgba(1, 4, 9, 0.28);
+  -webkit-backdrop-filter: none;


[perf-fixture 66/100] Slightly longer body so threads are not all one-liners. Used to exercise Diff inline-thread rendering under load.

enif-lee · 2026-08-15T06:21:17Z

+    const name = document.createElement('span');
+    name.className = 'host-name';
+    name.textContent = row.host;
+    name.title = row.host;


[perf-fixture 67/100] Scatter comment on src/popup.ts:500.

enif-lee · 2026-08-15T06:21:17Z

+    ...patchObj,
+  });
+  // Pin uiLanguage if the patch set it (guards against any normalize slip)
+  if (Object.prototype.hasOwnProperty.call(patchObj, 'uiLanguage')) {


[perf-fixture 68/100] Nit: second look at src/storage.ts L421.

enif-lee · 2026-08-15T06:21:17Z

+  async fetchPrTimelineItemsPage(owner: any, repo: any, number: any, opts: any = {}) {
+    // Prefer TIMELINE_EVENTS + graphql mode (long-stable SW route). Also try
+    // dedicated TIMELINE_ITEMS type. Never treat REST {events} as a GraphQL page.
+    const payload = {


[perf-fixture 70/100] Perf-load thread.

file: src/content-bridge/bridge-fetch-part-b.ts

line: 351

enif-lee · 2026-08-15T06:21:17Z

+/**
+ * Normalize key for Option/Shift combos (macOS remaps key → glyphs).
+ * Prefer KeyboardEvent.code for physical J/K/digits when alt or shift held.
+ */


[perf-fixture 71/100] Virtualization note.

src/pulls-palette-actions.ts:349

enif-lee · 2026-08-15T06:21:17Z

+      err.status = res?.status;
+      throw err;
+    }
+    return res.result;


[perf-fixture 72/100] Slightly longer body so threads are not all one-liners. Used to exercise Diff inline-thread rendering under load.

enif-lee

Perf fixture batch r51-60 (10 comments). Prefix [perf-fixture].

enif-lee · 2026-08-15T06:21:20Z

+    id: 'peer-rd',
+    filterId: 'rd',
+    action: 'applyFilter',
+    title: 'Ready (non-draft)',


[perf-fixture 73/100] Scatter comment on src/pulls-palette-core.ts:334.

enif-lee · 2026-08-15T06:21:21Z

+        }
+        const page = await PRTreeFetch.fetchReviewThreadsPage(
+          message.owner,
+          message.repo,


[perf-fixture 74/100] Nit: second look at src/background/sw-handle-a.ts L332.

enif-lee · 2026-08-15T06:21:21Z

+    return name ? { name, color: 'ededed', description: '' } : null;
+  }
+  const name = String(raw.name || raw.label || '').trim();
+  if (!name) return null;


[perf-fixture 75/100] Question: still correct after host-data-first?

src/dom.ts:621

enif-lee · 2026-08-15T06:21:21Z

+      font-size: 11px;
+    }
+    .host-list button {
+      flex-shrink: 0;


[perf-fixture 76/100] Perf-load thread.

file: src/popup.html

line: 369

enif-lee · 2026-08-15T06:21:21Z

+  } catch {
+    /* ignore */
+  }
+  return false;


[perf-fixture 77/100] Virtualization note.

src/pr-list-focus.ts:320

enif-lee · 2026-08-15T06:21:21Z

+          else void enableFeatures();
+          return false;
+        }
+        if (message?.type === 'PR_TREE_RATE_LIMIT_CHANGED') {


[perf-fixture 78/100] Slightly longer body so threads are not all one-liners. Used to exercise Diff inline-thread rendering under load.

enif-lee · 2026-08-15T06:21:21Z

+      if (area?.get && area?.set) {
+        const prevRaw = await new Promise((resolve) => {
+          area.get(['extensionPrefs'], (result: any) => {
+            resolve(result?.extensionPrefs);


[perf-fixture 79/100] Scatter comment on src/content-bridge/bridge-prefs.ts:243.

enif-lee · 2026-08-15T06:21:21Z

+   */
+  function stripTrailingSlashes(url: any) {
+    return String(url || '').trim().replace(/\/+$/, '');
+  }


[perf-fixture 80/100] Nit: second look at src/github-endpoints.ts L228.

enif-lee · 2026-08-15T06:21:21Z

+      await ensureRateLimitMem();
+    } catch {
+      /* proceed if storage cold */
+    }


[perf-fixture 81/100] Question: still correct after host-data-first?

src/background/sw-rate-limit.ts:207

enif-lee · 2026-08-15T06:21:21Z

+        toggleButton.setMode(enabled ? 'tree' : 'original');
+      } catch {
+        /* ignore */
+      }


[perf-fixture 82/100] Perf-load thread.

file: src/content-bootstrap.ts

line: 255

enif-lee

Perf fixture batch r61-70 (10 comments). Prefix [perf-fixture].

enif-lee · 2026-08-15T06:21:23Z

+    keys.push(s[0]);
+    s = s.slice(1);
+  }
+  if (s) {


[perf-fixture 83/100] Virtualization note.

src/onboarding-diff-demo.ts:145

enif-lee · 2026-08-15T06:21:23Z

+    'a[href*="/pull/"], a[id^="issue_"], .js-issue-row, [data-hovercard-type="pull_request"]'
+  );
+  // Unique-ish: count rows preferred
+  const rows = doc.querySelectorAll(


[perf-fixture 84/100] Slightly longer body so threads are not all one-liners. Used to exercise Diff inline-thread rendering under load.

enif-lee · 2026-08-15T06:21:23Z

+      });
+    }
+  } catch {
+    /* not registered yet — fine */


[perf-fixture 85/100] Scatter comment on src/background/sw-enterprise.ts:105.

enif-lee · 2026-08-15T06:21:24Z

+  const want = String(slot || '').toLowerCase();
+  if (!want) return false;
+  const code = String(opts.code || '');
+  if (/^[1-9]$/.test(want)) {


[perf-fixture 86/100] Nit: second look at src/onboarding-hotkeys.ts L91.

enif-lee · 2026-08-15T06:21:24Z

+  EDIT_ISSUE_COMMENT: 'PR_TREE_EDIT_ISSUE_COMMENT',
+  EDIT_REVIEW_COMMENT: 'PR_TREE_EDIT_REVIEW_COMMENT',
+  REQUEST_REVIEWERS: 'PR_TREE_REQUEST_REVIEWERS',
+  REMOVE_REVIEWERS: 'PR_TREE_REMOVE_REVIEWERS',


[perf-fixture 87/100] Question: still correct after host-data-first?

src/sw-messages.ts:57

enif-lee · 2026-08-15T06:21:24Z

@@ -62,9 +62,9 @@ function buildPrTree(prs) {
    childrenByParentHead.get(parent.headRef).push(pr);
  }

-  function buildNode(pr) {
+  function buildNode(pr: any) {
    const childPrs = childrenByParentHead.get(pr.headRef) || [];


[perf-fixture 88/100] Perf-load thread.

file: src/tree.ts

line: 66

enif-lee · 2026-08-15T06:21:24Z

+  'src/modal/pure/detail-cache.js',
+  'src/modal/pure/detail-merge.js',
+  'src/modal/pure/detail-store.js',
+  'src/modal/pure/load-progress.js',


[perf-fixture 89/100] Virtualization note.

src/content-scripts-list.ts:22

enif-lee · 2026-08-15T06:21:24Z

+  PRTree?: any;
+  PRTreeDOM?: any;
+  PRTreeFetch?: any;
+  PRTreeStorage?: any;


[perf-fixture 90/100] Slightly longer body so threads are not all one-liners. Used to exercise Diff inline-thread rendering under load.

enif-lee · 2026-08-15T06:21:24Z

@@ -0,0 +1,5 @@
+
+declare var PRTreeFetch: any;
+declare var PRGithubEndpoints: any;


[perf-fixture 91/100] Scatter comment on src/background-shims.d.ts:3.

enif-lee · 2026-08-15T06:21:24Z

@@ -0,0 +1,5 @@
+/** Barrel onboarding */
+export * from './onboarding-hotkeys';
+export * from './onboarding-diff-demo';


[perf-fixture 92/100] Nit: second look at src/onboarding.ts L3.

enif-lee

Perf fixture batch r71-78 (8 comments). Prefix [perf-fixture].

enif-lee · 2026-08-15T06:21:27Z

@@ -0,0 +1,4 @@
+declare var module: any;
+declare var globalThis: any;
+declare var chrome: any;


[perf-fixture 93/100] Question: still correct after host-data-first?

src/entries-shim.d.ts:3

enif-lee · 2026-08-15T06:21:27Z

@@ -0,0 +1,3 @@
+/** Barrel */
+export * from './pulls-palette-core';


[perf-fixture 94/100] Perf-load thread.

file: src/pulls-palette.ts

line: 2

enif-lee · 2026-08-15T06:21:27Z

+      const merge =
+        host.querySelector('[data-search-anchor="merge"]') ||
+        host.querySelector('.prp-merge-box-focus-host');
+      const comment =


[perf-fixture 95/100] Virtualization note.

tests/e2e/lib/harness.mjs:910

enif-lee · 2026-08-15T06:21:27Z

+      for (const it of items) {
+        const t = (
+          it.querySelector('.prp-sselect-item__label')?.textContent ||
+          it.textContent ||


[perf-fixture 96/100] Slightly longer body so threads are not all one-liners. Used to exercise Diff inline-thread rendering under load.

enif-lee · 2026-08-15T06:21:27Z

+      `file header selection missing: ${JSON.stringify({ sel, hdr })}`
+    );
+    // Leave hover so dock is not hover-revealed without Opt
+    evalInPage(`


[perf-fixture 97/100] Scatter comment on tests/e2e/features/selection.mjs:697.

enif-lee · 2026-08-15T06:21:27Z

+    );
+    expect(cleaned.map((c: any) => Number(c.id))).toEqual([42]);
+    expect(cleaned.some((c: any) => c.pending)).toBe(false);
+    expect(detailHasViewerPending(network)).toBe(false);


[perf-fixture 98/100] Nit: second look at tests/stale-local-review-drop.rstest.ts L401.

enif-lee · 2026-08-15T06:21:27Z

+    ).toBe(true);
+  });
+
+  test('existing PENDING + asPending never POSTs REST /comments (even if create races 422)', async () => {


[perf-fixture 99/100] Question: still correct after host-data-first?

tests/pending-add-comment-422.rstest.ts:353

enif-lee · 2026-08-15T06:21:27Z

+    expect(shellN).toBeGreaterThan(100);
+    expect(cvN).toBeGreaterThan(100);
+  });
+


[perf-fixture 100/100] Perf-load thread.

file: tests/architecture-gates.rstest.ts

line: 288

enif-lee and others added 30 commits July 21, 2026 18:15

Add workflow to submit demo PR reviews via Actions.

657a7cf

Allows APPROVE / REQUEST_CHANGES on demo PRs without self-review, so list badges can show Approved and Changes requested states.

Add stack demo screenshot to README.

72446a7

Include a live PR list capture showing tree indents, review badges, and branch chips so the product overview is easier to understand at a glance.

Remove demo stack structure text from README.

3601f5d

Keep only the screenshot under the overview section.

Rewrite README in English.

282dab6

Translate product overview, features, install, PAT notes, and development docs while keeping logo and demo screenshot placement.

Bump extension version to 1.0.3.

f808ae8

Align manifest, package, and README badge for the latest store package.

Add public privacy policy for Chrome Web Store.

4aa2a0b

Document data handling, optional PAT storage, host/storage justifications, and link the policy from the README for store submission.

Fix README image paths after renaming assets to .jpeg.

e808707

Commit the logo and demo screenshot under .jpeg extensions so GitHub renders the README images correctly.

fix(modal): show thin blue top loading bar while refreshing

ca0137a

Replace the Refreshing… status strip with a 2px accent progress overlay on the modal top edge so soft revalidation no longer shifts layout.

fix(modal): use UI font for diff review comments

4643f45

Reset font-family on inline thread and comment rows so review text uses the same proportional UI font as conversation view, not the mono code font inherited from the virtualized diff list. File paths stay monospace.

feat(modal): collapsible and resizable Diff files navigator

7aebe8c

Allow collapsing the Diff file tree to a thin expand rail and dragging a vertical handle to resize navigator width (clamped). Persist collapsed state and width in localStorage (prp:file-nav) for subsequent opens.

fix(modal): expand side-sheet shell to fullscreen for Diff

3150cb5

When the shell preference is side sheet, conversation stays a right-docked panel but Diff expands to full viewport like the modal Diff layout so the file tree and virtualized list have full width.

fix(modal): use transparent backdrop in side-sheet shell

fe9a406

Remove dim/blur overlay when shell is side sheet so the underlying GitHub list stays fully visible; only the sheet panel remains opaque.

fix(modal): soft transparent dim for modal backdrop

1eea680

Keep overlay hosts transparent for both shells, drop blur that washed to white, and use a light dark-alpha dim only in modal shell so the page stays visible. Side sheet backdrop remains fully transparent.

fix(modal): bound side-sheet width between 800px and 1200px

c25701d

Set conversation side-sheet min width to at least 800px (viewport-capped on smaller windows) and max to 1200px via clamp, so the panel stays usable.

fix(modal): apply soft dim backdrop in side-sheet shell

0e800aa

Match modal dimming (light alpha veil, no blur) for side-sheet mode so the underlying page is slightly darkened under the transparent overlay.

fix(modal): raise side-sheet min width to 950px

683f86f

Increase conversation side-sheet minimum from 800px by ~150px for more usable conversation/aside layout.

fix(modal): animate panel and backdrop on close

6699f64

Play sheet slide-out or modal scale-out before unmount, and fade the dim backdrop, so closing matches open transitions for both shells.

feat(modal): show GitHub profile images for user avatars

02e9cec

Render real profile photos via github.com/{login}.png (and API avatar_url when present) with initials fallback on error, across conversation threads, diff review threads, and people meta lists.

fix(modal): connect commit timeline dots for long lists

3b0c680

Move overflow scrolling to an outer wrapper so the vertical rail spans the full commit list height, and add per-item segments so 4+ commits stay linked.

feat(modal): show avatars and label colors in searchable pickers

5ddf76d

Render GitHub profile images in Add reviewer/assignee options and color swatches in Add label options, using option meta from builders.

feat(modal): emphasize merge box status with badges

5040364

Show mergeability, draft, mergeable_state, and checks as prominent badges in the merge box, with stronger box styling for ok/warn/muted states.

enif-lee changed the base branch from demo/e to demo/d August 15, 2026 06:03

enif-lee added the test2 label Aug 15, 2026

enif-lee commented Aug 15, 2026

View reviewed changes

Comment thread src/host/modules/open-modal-run.ts

: []),

],

};

const cleanedCacheRc = filterFn

enif-lee Aug 15, 2026

Copy link
Copy Markdown

Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mid 617

enif-lee commented Aug 15, 2026

View reviewed changes

Comment thread src/extension-globals.d.ts

code?: string;

pullNumber?: number;

graphqlErrors?: any;

}

enif-lee Aug 15, 2026

Copy link
Copy Markdown

Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dts

enif-lee commented Aug 15, 2026

View reviewed changes

enif-lee added documentation Improvements or additions to documentation enhancement New feature or request demo Demo / screenshot fixture perf Performance / load fixture large-diff Very large Diff for load testing and removed test2 labels Aug 15, 2026

enif-lee added this to the Largest PR milestone Aug 15, 2026

enif-lee added test2 perf Performance / load fixture large-diff Very large Diff for load testing and removed documentation Improvements or additions to documentation enhancement New feature or request perf Performance / load fixture large-diff Very large Diff for load testing labels Aug 15, 2026

		@@ -0,0 +1,5 @@

		declare var PRTreeFetch: any;
		declare var PRGithubEndpoints: any;

		@@ -0,0 +1,3 @@
		/** Barrel */
		export * from './pulls-palette-core';

Conversation

enif-lee commented Aug 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Why this PR exists

Shape

Suggested exercises

Checklist

Uh oh!

enif-lee left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

enif-lee left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

enif-lee left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

enif-lee left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

enif-lee left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

enif-lee left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

enif-lee left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

enif-lee left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

enif-lee left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

enif-lee commented Aug 15, 2026 •

edited

Loading