From 73269f41d294c8b27c47212b25111b27bfa65c56 Mon Sep 17 00:00:00 2001
From: seonghobae <8172694+seonghobae@users.noreply.github.com>
Date: Sun, 16 Aug 2026 17:35:02 +0000
Subject: [PATCH] =?UTF-8?q?=E2=9A=A1=20Bolt:=20[=EA=B2=80=EC=83=89=20?=
 =?UTF-8?q?=EC=84=B1=EB=8A=A5=20=EC=B5=9C=EC=A0=81=ED=99=94]?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

💡 What: transcript_search.py에서 집합의 불필요한 방어적 복사 제거 및 중복된 텍스트 속성 조회를 단일 변수로 최적화.
🎯 Why: O(N) 방어적 복사와 중복 속성 조회가 반복문 내에서 불필요한 연산을 유발하여 검색 및 인덱싱 성능 저하를 초래함.
📊 Impact: 인덱싱 속도 및 다중 검색 시 성능 향상, 메모리 할당 최소화.
🔬 Measurement: pytest 및 프로파일링을 통해 검색 속도 개선 여부 확인 가능.
---
 .jules/bolt.md       | 3 +++
 transcript_search.py | 7 ++++---
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/.jules/bolt.md b/.jules/bolt.md
index 341c7c91..2a713095 100644
--- a/.jules/bolt.md
+++ b/.jules/bolt.md
@@ -67,3 +67,6 @@
 ## 2025-02-12 - [Fast Path Execution in Directory Traversal and Log Parsing]
 **Learning:** Checking for string existence (`if "silence_" not in stderr`) before invoking regex matchers provides significant speed improvements when parsing large blocks of text. Similarly, moving expensive I/O operations like `os.path.realpath` inside conditional blocks prevents redundant disk access when configuration (like path exclusions) isn't utilized.
 **Action:** When working on large text processing or disk operations, verify if early exit conditions or conditional execution can bypass the expensive system or library calls.
+## 2024-08-16 - [불필요한 방어적 복사 제거]
+**학습:** [루프 내에서 비트 연산자(&)를 사용하여 집합 교집합을 수행할 때, 초기 집합의 방어적 복사본(예: set(postings))을 만들지 않아도 됩니다. & 연산자는 본질적으로 새로운 집합을 반환하므로, 초기 O(N) 복사는 중복되며 성능 병목 현상을 일으킵니다.]
+**실행:** [반복문에서 집합 교집합을 처리할 때 불필요한 set() 변환을 피하고 바로 할당하여 사용합니다.]
diff --git a/transcript_search.py b/transcript_search.py
index 1660c1f2..3914e5d1 100644
--- a/transcript_search.py
+++ b/transcript_search.py
@@ -193,12 +193,13 @@ def add(self, recording_id: str, segments: Iterable[Any]) -> int:
         """
         added = 0
         for segment in segments:
+            text = str(_read_attr(segment, "text"))
             entry = _Entry(
                 recording_id=recording_id,
                 start=float(_read_attr(segment, "start")),
                 end=float(_read_attr(segment, "end")),
-                text=str(_read_attr(segment, "text")),
-                counts=Counter(tokenize(str(_read_attr(segment, "text")))),
+                text=text,
+                counts=Counter(tokenize(text)),
             )
             position = len(self._entries)
             self._entries.append(entry)
@@ -242,7 +243,7 @@ def search(self, query: str) -> list[Match]:
             if not postings:
                 return []
             candidates = (
-                set(postings) if candidates is None else candidates & postings
+                postings if candidates is None else candidates & postings
             )
             if not candidates:
                 return []