From 98bcc78f010d5956fc6850db9a0c9ce8c1a5cbb3 Mon Sep 17 00:00:00 2001 From: seonghobae <8172694+seonghobae@users.noreply.github.com> Date: Wed, 29 Jul 2026 20:48:24 +0000 Subject: [PATCH] =?UTF-8?q?=E2=9A=A1=20Bolt:=20=EB=94=94=EB=A0=89=ED=86=A0?= =?UTF-8?q?=EB=A6=AC=20=EC=B2=98=EB=A6=AC=20=EB=A3=A8=ED=94=84=EC=9D=98=20?= =?UTF-8?q?=EC=A0=95=EC=A0=81=20=EC=97=90=EC=85=8B=20=ED=95=B4=EC=8B=B1=20?= =?UTF-8?q?=EB=B0=8F=20=EC=9E=AC=ED=95=A0=EB=8B=B9=20=EB=B0=A9=EC=A7=80?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 디렉토리 반복문(process_dir)에서 매번 할당되던 대용량 문자열과 SHA-256 해시 연산을 private object StaticAssets 로 추출하여 성능을 최적화했습니다. const val 및 @JvmField 를 사용하여 리플렉션/getter로 인한 테스트 커버리지 하락을 방지했습니다. --- .jules/bolt.md | 3 +++ src/main/kotlin/html4tree/main.kt | 42 ++++++++++++++++++------------- 2 files changed, 27 insertions(+), 18 deletions(-) diff --git a/.jules/bolt.md b/.jules/bolt.md index 19b4c613..db4032d3 100644 --- a/.jules/bolt.md +++ b/.jules/bolt.md @@ -43,3 +43,6 @@ ## 2025-01-24 - 단일 readAttributes 호출로 파일 속성 조회 최적화 **학습:** `isDirectory`, `!it.isDirectory()`, `isSymbolicLink` 3개의 개별적인 파일 시스템 I/O 호출을 수행하면 성능 저하가 큽니다. 이를 단일 `Files.readAttributes` 호출로 변경하여 메타데이터를 한 번에 조회함으로써 I/O 오버헤드를 대폭 줄일 수 있음을 확인했습니다. **조치:** 디렉토리 순회 시 파일의 여러 속성을 확인할 때는 개별적인 stat 호출보다 `Files.readAttributes`를 사용하여 필요한 모든 속성을 한 번에 가져오는 방식을 우선적으로 고려해야 합니다. +## 2024-07-29 - 디렉토리 처리 루프의 정적 에셋 재할당 방지 +**Learning:** 디렉토리 처리 루프(`process_dir`) 내에서 고정된 문자열 할당과 비용이 많이 드는 SHA-256 해시 계산이 반복적으로 발생하여 CPU 및 메모리 낭비가 발생했습니다. 또한 Kotlin에서 정적 변수를 추출할 때 컴파일러가 생성하는 암시적 getter로 인해 JaCoCo 테스트 커버리지가 떨어지는 문제가 있습니다. +**Action:** 대규모 정적 문자열과 결정론적 계산 결과를 `private object`로 추출하고, 100% 테스트 커버리지를 유지하기 위해 `const val` 및 `@JvmField`를 사용하도록 수정합니다. diff --git a/src/main/kotlin/html4tree/main.kt b/src/main/kotlin/html4tree/main.kt index b4558624..1b59fee0 100644 --- a/src/main/kotlin/html4tree/main.kt +++ b/src/main/kotlin/html4tree/main.kt @@ -240,11 +240,12 @@ fun write_index_file(curr_dir: File, content: String) { } } -fun process_dir(curr_dir: File, excludeSet: Set? = null, dirFiles: Array? = null){ - - val exclude: Set = excludeSet ?: process_ignore_file(curr_dir) - - val cssContent = """ +private object StaticAssets { + // ⚡ Bolt Performance Optimization: Extract static assets + // Prevents redundant string allocations and expensive SHA-256 hash recalculations + // on every process_dir loop iteration. + // Using const val and @JvmField avoids JaCoCo coverage drops from implicit getters. + const val cssContent = """ body { font-family: system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", sans-serif; line-height: 1.5; @@ -310,13 +311,26 @@ fun process_dir(curr_dir: File, excludeSet: Set? = null, dirFiles: Array } """ - val styleHash = "sha256-" + Base64.getEncoder().encodeToString(MessageDigest.getInstance("SHA-256").digest(cssContent.toByteArray(Charsets.UTF_8))) + @JvmField val styleHash = "sha256-" + Base64.getEncoder().encodeToString(MessageDigest.getInstance("SHA-256").digest(cssContent.toByteArray(Charsets.UTF_8))) - val css = """ + const val css = """ """ + const val index_bottom=""" + + + + + +""" +} + +fun process_dir(curr_dir: File, excludeSet: Set? = null, dirFiles: Array? = null){ + + val exclude: Set = excludeSet ?: process_ignore_file(curr_dir) + val index_top = """ @@ -324,11 +338,11 @@ ${cssContent} - + ${curr_dir.getName().escapeHtml()} - ${css} + ${StaticAssets.css}
@@ -377,16 +391,8 @@ ${cssContent} return l.toString(); } - val index_bottom=""" - - -
- - -""" - try { - write_index_file(curr_dir, index_top+index_middle()+index_bottom) + write_index_file(curr_dir, index_top+index_middle()+StaticAssets.index_bottom) } catch (e: Exception) { // 보안 향상: 디렉토리에 쓰기 권한이 없거나 파일 시스템 오류가 발생했을 때 // 전체 크롤링(프로세스)이 중단되는 DoS를 방지합니다. (Fail Securely)