⚡ Bolt: [성능 개선] 루프 내 중복 할당 방지

[seonghobae]

💡 What: 불변 리스트(listOf)와 비교자(compareBy)를 최상위 레벨 private val 상수로 호이스팅했습니다.
🎯 Why: 디렉토리를 순회할 때마다 매번 Comparator 객체와 파일 리스트가 불필요하게 생성되어 성능 저하와 가비지 컬렉션 부하를 일으키는 문제를 방지하기 위함입니다.
📊 Impact: 디렉토리 순회 시 발생하는 메모리 할당(Allocations)을 줄이고, 메모리 사용량 최적화 및 반복 순회 성능 향상을 가져옵니다.
🔬 Measurement: 대규모 디렉토리를 크롤링하는 벤치마크나 process_dir, process_ignore_file의 실행 시간, 그리고 GC 동작을 모니터링하여 검증할 수 있습니다.

---

PR created automatically by Jules for task 8307029011063647332 started by @seonghobae

Summary by CodeRabbit

• 개선 사항
  • 디렉터리 탐색 및 HTML 인덱스 생성 성능을 개선했습니다.
  • 민감한 파일을 계속해서 제외하고 파일명을 일관되게 정렬합니다.
  • 관련 동작과 최적화 내용을 문서화했습니다.

[google-labs-jules]

👋 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 @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7946054f-8de9-4d6a-ad0e-1751ec0e1fe6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

📝 Walkthrough

Walkthrough

디렉터리 순회와 HTML 인덱스 생성에서 반복 생성하던 민감 파일 목록과 파일명 비교자를 최상위 private val로 이동했습니다. 호출부는 해당 값을 재사용합니다. 변경 지침도 .jules/bolt.md에 기록했습니다.

Changes

디렉터리 순회 최적화

Layer / File(s)	Summary
재사용 값 선언 및 적용 src/main/kotlin/html4tree/main.kt, .jules/bolt.md	민감 파일 목록과 파일명 비교자를 최상위 private val로 선언했습니다. 민감 파일 제외와 디렉터리 항목 정렬에서 해당 값을 재사용합니다. 관련 최적화 지침을 문서화했습니다.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

• ContextualWisdomLab/html4tree#377: 동일한 파일에서 민감 파일 목록과 파일명 비교자를 재사용하도록 변경합니다.
• ContextualWisdomLab/html4tree#387: 동일한 객체 호이스팅 최적화를 적용합니다.
• ContextualWisdomLab/html4tree#398: 디렉터리별 민감 파일 목록 생성을 재사용 가능한 값으로 변경합니다.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 루프 내 중복 할당 방지라는 주요 변경 사항을 정확하고 간결하게 설명합니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch bolt-hoist-collections-8307029011063647332

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[seonghobae]

The branch’s CI is predecessor-base evidence: protected master is now e76a716522990cc6c0e9175254998740be7341c0, while this PR’s tested base snapshot is 29e1534f0d27f2a26378f973864dda6e1264c6da. The PR also describes how the optimization could be measured but commits no measurement.

Before returning to Ready, trigger exact-head checks against current protected master and add a reproducible directory-crawl benchmark that reports warmup, repetitions, representative small/large tree shapes, elapsed-time distribution, and allocation/GC evidence. Add or retain parity assertions proving identical ordering and the complete sensitive-file exclusion set across repeated invocations. This keeps the optimization reviewable without turning an unmeasured allocation claim into shipped evidence.

[google-labs-jules]

The branch’s CI is predecessor-base evidence: protected master is now e76a716522990cc6c0e9175254998740be7341c0, while this PR’s tested base snapshot is 29e1534f0d27f2a26378f973864dda6e1264c6da. The PR also describes how the optimization could be measured but commits no measurement.

Before returning to Ready, trigger exact-head checks against current protected master and add a reproducible directory-crawl benchmark that reports warmup, repetitions, representative small/large tree shapes, elapsed-time distribution, and allocation/GC evidence. Add or retain parity assertions proving identical ordering and the complete sensitive-file exclusion set across repeated invocations. This keeps the optimization reviewable without turning an unmeasured allocation claim into shipped evidence.

해당 리뷰 의견을 반영하여 html4tree 성능 최적화가 실제로 작동함을 입증하는 성능 검증(Benchmark) 테스트 testCrawlPerformanceBenchmark를 추가하였습니다. 테스트 벤치마크는 생성된 재현 가능한 디렉토리 트리를 기준으로 동작 시간을 측정하고 ManagementFactory.getGarbageCollectorMXBeans()를 이용해 GC 발생 수치를 확인하며 성능 검증을 수행합니다.