From 0ea65eece027743607bfccd499bb0d1118a0c3a3 Mon Sep 17 00:00:00 2001
From: seonghobae <8172694+seonghobae@users.noreply.github.com>
Date: Sat, 18 Jul 2026 03:13:11 +0000
Subject: [PATCH] =?UTF-8?q?=E2=9A=A1=20Bolt:=20[=EC=84=B1=EB=8A=A5=20?=
=?UTF-8?q?=EA=B0=9C=EC=84=A0]=20CSS=20=EB=B0=8F=20=ED=95=B4=EC=8B=9C=20?=
=?UTF-8?q?=EA=B3=84=EC=82=B0=20=EC=BA=90=EC=8B=B1=20=EC=B5=9C=EC=A0=81?=
=?UTF-8?q?=ED=99=94?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
- 정적인 CSS 문자열 선언과 값비싼 SHA-256 해시 계산을 `process_dir` 함수에서 분리하여 파일 레벨 `private val` 상수로 이동.
- 이 최적화를 통해 각각의 디렉토리를 순회할 때마다 불필요하게 발생하는 문자열 포맷팅 및 해시 생성 오버헤드를 완벽하게 제거함.
- `private val` 선언을 통해 Kotlin 컴파일러의 Public Getter 생성을 방지하여 Jacoco의 Line Coverage 하락을 막음.
---
.jules/bolt.md | 4 +
src/main/kotlin/html4tree/main.kt | 146 +++++++++++++++---------------
2 files changed, 77 insertions(+), 73 deletions(-)
diff --git a/.jules/bolt.md b/.jules/bolt.md
index 19b4c61..44f6bc6 100644
--- a/.jules/bolt.md
+++ b/.jules/bolt.md
@@ -43,3 +43,7 @@
## 2025-01-24 - 단일 readAttributes 호출로 파일 속성 조회 최적화
**학습:** `isDirectory`, `!it.isDirectory()`, `isSymbolicLink` 3개의 개별적인 파일 시스템 I/O 호출을 수행하면 성능 저하가 큽니다. 이를 단일 `Files.readAttributes` 호출로 변경하여 메타데이터를 한 번에 조회함으로써 I/O 오버헤드를 대폭 줄일 수 있음을 확인했습니다.
**조치:** 디렉토리 순회 시 파일의 여러 속성을 확인할 때는 개별적인 stat 호출보다 `Files.readAttributes`를 사용하여 필요한 모든 속성을 한 번에 가져오는 방식을 우선적으로 고려해야 합니다.
+
+## 2026-07-18 - [Optimization] Cache Static Strings and Hashes using File-level private val
+**Learning:** Moving static configurations (like CSS strings or Hash computations) out of repeatedly called methods (`process_dir`) and into file-level properties prevents redundant string templating and expensive crypto calculations. Using `private val` instead of `val` is critical in Kotlin as it prevents the generation of public getters, which otherwise leads to drops in code coverage if those auto-generated methods aren't explicitly tested.
+**Action:** When extracting static variables for performance caching in Kotlin, always declare them as `private val` to ensure both maximum performance and untouched Jacoco code coverage metrics.
diff --git a/src/main/kotlin/html4tree/main.kt b/src/main/kotlin/html4tree/main.kt
index b455862..b2d2aa9 100644
--- a/src/main/kotlin/html4tree/main.kt
+++ b/src/main/kotlin/html4tree/main.kt
@@ -13,6 +13,79 @@ import com.github.ajalt.clikt.parameters.options.default
import com.github.ajalt.clikt.parameters.arguments.argument
import com.github.ajalt.clikt.parameters.types.int
+private val cssContent = """
+ body {
+ font-family: system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", sans-serif;
+ line-height: 1.5;
+ padding: 1rem;
+ color: #1f2328;
+ }
+ main {
+ max-width: 800px;
+ margin: 0 auto;
+ }
+ ul {
+ list-style-type: none;
+ padding-left: 0;
+ }
+ a.dir-link {
+ display: flex;
+ align-items: flex-start;
+ gap: 0.5rem;
+ width: 100%;
+ overflow-wrap: anywhere;
+ box-sizing: border-box;
+ }
+ .icon {
+ flex-shrink: 0;
+ width: 1.25rem;
+ text-align: center;
+ }
+ a {
+ padding: 0.5rem;
+ text-decoration: none;
+ color: #0969da;
+ border-radius: 4px;
+ transition: background-color 0.2s ease, outline-color 0.2s ease;
+ }
+ a:hover, a:focus-visible {
+ background-color: #f6f8fa;
+ text-decoration: underline;
+ outline: 2px solid #0969da;
+ outline-offset: -2px;
+ }
+ @media (prefers-reduced-motion: reduce) {
+ a {
+ transition: none;
+ }
+ }
+ @media (prefers-color-scheme: dark) {
+ body {
+ background-color: #0d1117;
+ color: #c9d1d9;
+ }
+ a {
+ color: #58a6ff;
+ }
+ a:hover, a:focus-visible {
+ background-color: #161b22;
+ outline-color: #58a6ff;
+ }
+ }
+ .empty-dir {
+ padding: 0.5rem;
+ opacity: 0.7;
+ font-style: italic;
+ }
+ """
+
+private val styleHash = "sha256-" + Base64.getEncoder().encodeToString(MessageDigest.getInstance("SHA-256").digest(cssContent.toByteArray(Charsets.UTF_8)))
+
+private val css = """
+
+ """
+
class Html4tree : CliktCommand() {
val maxLevel:Int by option(help="Number of levels deep for which to generate an index.html file", hidden = false).int().default(-1)
val topDir: String by argument(help="Top directory to crawl")
@@ -244,79 +317,6 @@ fun process_dir(curr_dir: File, excludeSet: Set? = null, dirFiles: Array
val exclude: Set = excludeSet ?: process_ignore_file(curr_dir)
- val cssContent = """
- body {
- font-family: system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", sans-serif;
- line-height: 1.5;
- padding: 1rem;
- color: #1f2328;
- }
- main {
- max-width: 800px;
- margin: 0 auto;
- }
- ul {
- list-style-type: none;
- padding-left: 0;
- }
- a.dir-link {
- display: flex;
- align-items: flex-start;
- gap: 0.5rem;
- width: 100%;
- overflow-wrap: anywhere;
- box-sizing: border-box;
- }
- .icon {
- flex-shrink: 0;
- width: 1.25rem;
- text-align: center;
- }
- a {
- padding: 0.5rem;
- text-decoration: none;
- color: #0969da;
- border-radius: 4px;
- transition: background-color 0.2s ease, outline-color 0.2s ease;
- }
- a:hover, a:focus-visible {
- background-color: #f6f8fa;
- text-decoration: underline;
- outline: 2px solid #0969da;
- outline-offset: -2px;
- }
- @media (prefers-reduced-motion: reduce) {
- a {
- transition: none;
- }
- }
- @media (prefers-color-scheme: dark) {
- body {
- background-color: #0d1117;
- color: #c9d1d9;
- }
- a {
- color: #58a6ff;
- }
- a:hover, a:focus-visible {
- background-color: #161b22;
- outline-color: #58a6ff;
- }
- }
- .empty-dir {
- padding: 0.5rem;
- opacity: 0.7;
- font-style: italic;
- }
- """
-
- val styleHash = "sha256-" + Base64.getEncoder().encodeToString(MessageDigest.getInstance("SHA-256").digest(cssContent.toByteArray(Charsets.UTF_8)))
-
- val css = """
-
- """
-
val index_top = """