Optimize transcript no-filter search path - #1723
Optimize transcript no-filter search path#1723groupthinking with Copilot wants to merge 2 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
🔍 PR Validation |
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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 |
|
Backlog triage: leaving open. Transcript search allocation (#908) is CONFLICTING vs rewritten |
Canonical issue
Linked automatically.
Outcome
Transcript filtering now avoids allocation when no search or speaker filter is active, while preserving null/empty-query behavior and one-time query normalization.
Scope
filterSegmentsRisk
Verification
Current head:
56c6c06707c41e9e50d39150f09c782cb60e4008npm test --workspace apps/web -- src/lib/__tests__/transcript-search.test.tsProduction evidence
Not applicable yet; this is a pure transcript-filter helper change. Preview/runtime evidence should attach to the current head if repository governance requires web deployment proof.
Agent handoff