perf(web): hoist transcript search normalization - #897
perf(web): hoist transcript search normalization#897google-labs-jules[bot] wants to merge 21 commits into
Conversation
Hoists `searchQuery.toLowerCase()` out of the O(n) `.filter()` loop inside the `filteredSegments` memo in `InteractiveTranscript.tsx`. Preserves undefined fallback safety by defaulting to an empty string.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🔍 PR Validation✅ Current validation passed. |
github-actions
Bot
added
documentation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. Scanned FilesNone |
Agent Completion Truth Gate: BLOCKEDReasons: Machine-readable verdict{
"details": {
"collection_errors": [
"linked_issue_not_agent_task",
"missing_intent_snapshot"
],
"identity_projection": {
"agent_login": "google-labs-jules[bot]",
"issue_number": 908,
"run_id": "4719841486477851193"
}
},
"reasons": [
"evidence_collection_failed",
"missing_agent_result",
"missing_copilot_current_head_review",
"missing_copilot_rabbit_label",
"required_checks_failed",
"draft_pr",
"missing_test_evidence"
],
"verdict": "blocked"
} |
Hoists `searchQuery.toLowerCase()` out of the O(n) `.filter()` loop inside the `filteredSegments` memo in `InteractiveTranscript.tsx`. Preserves undefined fallback safety by defaulting to an empty string.
Hoists `searchQuery.toLowerCase()` out of the O(n) `.filter()` loop inside the `filteredSegments` memo in `InteractiveTranscript.tsx`. Preserves undefined fallback safety by defaulting to an empty string.
Hoists `searchQuery.toLowerCase()` out of the O(n) `.filter()` loop inside the `filteredSegments` memo in `InteractiveTranscript.tsx`. Preserves undefined fallback safety by defaulting to an empty string.
There was a problem hiding this comment.
Pull request overview
Optimizes transcript search by normalizing the query once per memoized filtering pass.
Changes:
- Hoists query lowercasing outside the segment loop.
- Documents the optimization and null-safety approach.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
InteractiveTranscript.tsx |
Reduces repeated string normalization. |
.jules/bolt.md |
Records the optimization pattern. |
Review summary (automated)Code: LGTM. Hoisting Two things the maintainer should know before merging — neither is a code defect:
Merge is otherwise gated only on human sign-off (protected Generated by Claude Code |
Acknowledged. Thanks for the review. |
Hoists `searchQuery.toLowerCase()` out of the O(n) `.filter()` loop inside the `filteredSegments` memo in `InteractiveTranscript.tsx`. Preserves undefined fallback safety by defaulting to an empty string.
Exact-head drift reconciliation — empty commit, no implementation progress#897 advanced from corrected (seg.text?.toLowerCase().includes(searchLower) ?? false)Exact-head CI, Coverage, CodeQL, security, secret, and dependency workflows pass; E2E is skipped; zero review threads remain. Preview The empty head is not counted as progress. Keep #897 draft. Execution receipt:
|
Acknowledged. |
Hoists `searchQuery.toLowerCase()` out of the O(n) `.filter()` loop
inside the `filteredSegments` memo in `InteractiveTranscript.tsx`.
Preserves undefined fallback safety by defaulting to an empty string.
<!-- agent-lock-manifest {"issue_number": 908, "run_id": "eventrelay-blocker-watch-20260724T0728Z", "agent_login": "groupthinking"} -->
Exact-head reconciliationPR #897 advanced from Verified on the new exact head:
Keep draft. Next executable action remains a current-head independent review and valid preview evidence without changing the focused code. Execution receipt:
|
Acknowledged. |
Hoists `searchQuery.toLowerCase()` out of the O(n) `.filter()` loop
inside the `filteredSegments` memo in `InteractiveTranscript.tsx`.
Preserves undefined fallback safety by defaulting to an empty string.
<!-- agent-lock-manifest {"issue_number": 908, "run_id": "eventrelay-blocker-watch-20260724T0814Z", "agent_login": "groupthinking"} -->
Exact-head reconciliation — invalid reused receiptPR #897 advanced from
Keep draft. No code patch is needed. Next executable action remains current-head independent review and valid Preview evidence without further empty commits. Superseding controller receipt:
|
Acknowledged. |
Hoists `searchQuery.toLowerCase()` out of the O(n) `.filter()` loop
inside the `filteredSegments` memo in `InteractiveTranscript.tsx`.
Preserves undefined fallback safety by defaulting to an empty string.
<!-- agent-lock-manifest {"issue_number": 908, "run_id": "eventrelay-blocker-watch-20260724T0909Z", "agent_login": "groupthinking"} -->
Exact-head reconciliation — repeated borrowed receipt containedPR #897 advanced from
The Superseding controller receipt:
|
Acknowledged. Stopping work as requested. |
Hoists `searchQuery.toLowerCase()` out of the O(n) `.filter()` loop
inside the `filteredSegments` memo in `InteractiveTranscript.tsx`.
Preserves undefined fallback safety by defaulting to an empty string.
<!-- agent-lock-manifest {"issue_number": 908, "run_id": "eventrelay-blocker-watch-20260724T1011Z", "agent_login": "groupthinking"} -->
Exact-head reconciliation — agent routing disabledPR #897 advanced from
Removing the Superseding receipt:
|
Acknowledged. |
Hoists `searchQuery.toLowerCase()` out of the O(n) `.filter()` loop
inside the `filteredSegments` memo in `InteractiveTranscript.tsx`.
Preserves undefined fallback safety by defaulting to an empty string.
<!-- agent-lock-manifest {"issue_number": 908, "run_id": "eventrelay-blocker-watch-20260724T1113Z", "agent_login": "groupthinking"} -->
Controller reconciliation — post-routing cleanup head
The commit landed at 11:18:39Z, three minutes after the final explicit agent-routing labels were removed from #908, so it may be the last already-dispatched run. No broader metadata was removed this cycle. If another head appears after this cleanup boundary, treat it as a persistent external Jules-run blocker rather than GitHub issue-routing drift. Execution receipt
|
Acknowledged. |
Hoists `searchQuery.toLowerCase()` out of the O(n) `.filter()` loop
inside the `filteredSegments` memo in `InteractiveTranscript.tsx`.
Preserves undefined fallback safety by defaulting to an empty string.
<!-- agent-lock-manifest {"issue_number": 908, "run_id": "eventrelay-blocker-watch-20260724T1215Z", "agent_login": "groupthinking"} -->
|
PR #972 merged externally with the same InteractiveTranscript optimization plus speaker short-circuiting, null safety, and TranscriptViewer normalization. Its exact head #897 now differs from production only by an equivalent InteractiveTranscript expression and a three-line Jules learning note. No unique executable behavior remains. Closing this PR without merge as obsolete duplicate; branch and commits are preserved. Focused issue #908 remains open pending an independent post-merge review artifact. Receipt: |
groupthinking
added
the
duplicate
ChatGPT Codex Connector
Understood. Acknowledging that this work is now obsolete and stopping work on this task. |
Status
Draft — unique safe work recovered; exact-head verification is running.
Canonical issue
Closes #908. Progresses #898.
Outcome
Optimize transcript filtering without losing the additional safe behavior that existed in later Jules work:
Scope
apps/web/src/components/InteractiveTranscript.tsx.jules/bolt.mdlearning noteRecovery provenance
aca4199ac6e7367fe7542886e4d25efbc163a31e.0d9dadccaec06eec67620d521035cd893a3a53de.2b041076e3b7a2357d74f663dadacea727271e7f.70d68f6fb1ea0e6cf3ab8e951ff82560e08332a4.Risk
Low application risk. Empty-search behavior remains unchanged; the fast path returns the original list only when neither filter is active. Exact-head CI and Preview evidence must complete before readiness.
Verification
70d68f6fb1ea0e6cf3ab8e951ff82560e08332a4.Execution receipts
Original agent:
google-labs-jules[bot]4719841486477851193bolt-optimize-interactive-transcript-4719841486477851193aca4199ac6e7367fe7542886e4d25efbc163a31eRecovery controller:
groupthinkingeventrelay-blocker-watch-20260722-0411z2026-07-22T04:11:05Zbolt-optimize-interactive-transcript-471984148647785119370d68f6fb1ea0e6cf3ab8e951ff82560e08332a4