From 8d57ca0e3c793d0c42e5badbf9ee326e0a1c76dd Mon Sep 17 00:00:00 2001 From: seonghobae <8172694+seonghobae@users.noreply.github.com> Date: Mon, 20 Jul 2026 20:57:47 +0000 Subject: [PATCH] =?UTF-8?q?=E2=9A=A1=20Bolt:=20=EC=A0=95=EC=A0=81=20?= =?UTF-8?q?=EB=AC=B8=EC=9E=90=EC=97=B4=20=EB=B0=8F=20=EC=95=94=ED=98=B8?= =?UTF-8?q?=ED=99=94=20=EC=97=B0=EC=82=B0=20=EC=B5=9C=EC=83=81=EC=9C=84=20?= =?UTF-8?q?=ED=94=84=EB=A1=9C=ED=8D=BC=ED=8B=B0=20=EC=B6=94=EC=B6=9C=20?= =?UTF-8?q?=EC=B5=9C=EC=A0=81=ED=99=94?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - `process_dir` 루프 내부에서 불필요하게 반복 생성되던 `cssContent`, `css` 정적 문자열을 파일 최상단 프로퍼티로 추출 - 재귀 호출마다 반복되던 무거운 SHA-256 해싱 및 Base64 인코딩 연산(`styleHash`)을 파일 로드 시 1회만 계산되도록 최적화 - 변경된 최상위 프로퍼티에 대한 100% JaCoCo 테스트 커버리지 유지 보장 테스트 추가 --- .jules/bolt.md | 3 +++ src/main/kotlin/html4tree/main.kt | 14 +++++++------- src/test/kotlin/html4tree/MainTest.kt | 9 +++++++++ 3 files changed, 19 insertions(+), 7 deletions(-) diff --git a/.jules/bolt.md b/.jules/bolt.md index 19b4c61..bb23b66 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`를 사용하여 필요한 모든 속성을 한 번에 가져오는 방식을 우선적으로 고려해야 합니다. +## 2025-01-24 - 디렉토리 순회 시 무거운 문자열 생성 및 암호화 연산 분리 +**학습:** `process_dir` 안에서 매번 정적 문자열 `cssContent`를 선언하고, SHA-256 해시를 계산하는 것은 모든 폴더를 방문할 때마다 발생하여 디렉토리 크롤링 성능을 크게 떨어트립니다. +**조치:** 무거운 초기화 비용이 드는 정적 문자열과 암호화 연산은 파일의 최상위 프로퍼티로 선언하여 애플리케이션 시작 시 한 번만 계산되도록 합니다. JaCoCo에서 이 최상위 프로퍼티들의 Getter에 대한 커버리지를 100%로 유지하기 위해 명시적인 테스트를 추가합니다. diff --git a/src/main/kotlin/html4tree/main.kt b/src/main/kotlin/html4tree/main.kt index b455862..5f0809b 100644 --- a/src/main/kotlin/html4tree/main.kt +++ b/src/main/kotlin/html4tree/main.kt @@ -240,11 +240,7 @@ 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 = """ +val cssContent = """ body { font-family: system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", sans-serif; line-height: 1.5; @@ -310,13 +306,17 @@ 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))) +val styleHash = "sha256-" + Base64.getEncoder().encodeToString(MessageDigest.getInstance("SHA-256").digest(cssContent.toByteArray(Charsets.UTF_8))) - val css = """ +val css = """ """ +fun process_dir(curr_dir: File, excludeSet: Set? = null, dirFiles: Array? = null){ + + val exclude: Set = excludeSet ?: process_ignore_file(curr_dir) + val index_top = """ diff --git a/src/test/kotlin/html4tree/MainTest.kt b/src/test/kotlin/html4tree/MainTest.kt index 1349471..4e8718b 100644 --- a/src/test/kotlin/html4tree/MainTest.kt +++ b/src/test/kotlin/html4tree/MainTest.kt @@ -676,6 +676,15 @@ class MainTest { assertFalse(excluded.contains("test.txt1001")) } + @Test + fun testTopLevelCssPropertiesCoverage() { + // These assertions ensure the implicit getters for top-level properties + // (cssContent, styleHash, css) are covered by JaCoCo. + assertTrue(cssContent.contains("body {")) + assertTrue(styleHash.startsWith("sha256-")) + assertTrue(css.contains("