From 920d18f1f80a1c72da28f58a1d1b2fae2a305742 Mon Sep 17 00:00:00 2001
From: seonghobae <8172694+seonghobae@users.noreply.github.com>
Date: Wed, 15 Jul 2026 03:13:15 +0000
Subject: [PATCH 1/4] =?UTF-8?q?=F0=9F=8E=A8=20Palette:=20=EB=B9=88=20?=
=?UTF-8?q?=EB=94=94=EB=A0=89=ED=86=A0=EB=A6=AC=20=EC=9D=B4=EB=A6=84?=
=?UTF-8?q?=EC=97=90=20=EB=8C=80=ED=95=9C=20Fallback=20=EC=A0=9C=EA=B3=B5?=
=?UTF-8?q?=EC=9D=84=20=ED=86=B5=ED=95=9C=20=EC=A0=91=EA=B7=BC=EC=84=B1=20?=
=?UTF-8?q?=ED=96=A5=EC=83=81?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
루트 디렉토리(`/` 또는 `C:\`)와 같이 `.getName()`이 빈 문자열을 반환하는 디렉토리에 대해 HTML 생성 시 절대 경로(`absolutePath`)를 fallback으로 사용하도록 수정하여 스크린 리더 접근성을 향상시킵니다.
---
.jules/palette.md | 3 +++
src/main/kotlin/html4tree/main.kt | 4 ++--
src/test/kotlin/html4tree/MainTest.kt | 15 +++++++++++++++
src/test/kotlin/html4tree/MockRootTest.kt | 16 ++++++++++++++++
4 files changed, 36 insertions(+), 2 deletions(-)
create mode 100644 src/test/kotlin/html4tree/MockRootTest.kt
diff --git a/.jules/palette.md b/.jules/palette.md
index 9a12c9d..167bbfa 100644
--- a/.jules/palette.md
+++ b/.jules/palette.md
@@ -48,3 +48,6 @@
## 2024-08-01 - 네이티브 브라우저 UI의 다크 모드 지원 강제
**학습:** CSS 미디어 쿼리(`@media (prefers-color-scheme: dark)`)를 통해 다크 모드를 지원하더라도, 브라우저의 네이티브 UI 요소(스크롤바, 기본 폼 컨트롤, 기본 백그라운드 등)는 테마 변경을 인식하지 못해 어두운 테마 환경에서 밝은 스크롤바가 표시되는 등 시각적 불일치를 초래합니다.
**조치:** 항상 HTML 문서의 `` 영역에 `` and `<h1>` tags, which harms accessibility. Fallbacks (e.g. `absolutePath`) must be provided for UI elements.
+**Action:** Use `name.ifEmpty { absolutePath }` when generating HTML content from File objects to ensure semantic tags are properly populated.
diff --git a/src/main/kotlin/html4tree/main.kt b/src/main/kotlin/html4tree/main.kt
index b455862..1c3c2ee 100644
--- a/src/main/kotlin/html4tree/main.kt
+++ b/src/main/kotlin/html4tree/main.kt
@@ -327,12 +327,12 @@ ${cssContent} </style>
<meta http-equiv="Content-Security-Policy" content="default-src 'none'; style-src '${styleHash}'; base-uri 'none'; form-action 'none';">
<!-- 보안 향상: 리퍼러를 통한 디렉토리 경로 노출 방지 -->
<meta name="referrer" content="no-referrer">
- <title>${curr_dir.getName().escapeHtml()}
+ ${curr_dir.name.ifEmpty { curr_dir.absolutePath }.escapeHtml()}
${css}
- ${curr_dir.getName().escapeHtml()}
+ ${curr_dir.name.ifEmpty { curr_dir.absolutePath }.escapeHtml()}
↰ .. $expectedFallback "))
+ assertTrue(htmlContent.contains("$expectedFallback "))
+ }
+
@Test(expected = IllegalArgumentException::class)
fun testGoBlankDir() {
go(" ", -1)
diff --git a/src/test/kotlin/html4tree/MockRootTest.kt b/src/test/kotlin/html4tree/MockRootTest.kt
new file mode 100644
index 0000000..a58c491
--- /dev/null
+++ b/src/test/kotlin/html4tree/MockRootTest.kt
@@ -0,0 +1,16 @@
+package html4tree
+
+import org.junit.Test
+import java.io.File
+import kotlin.test.assertEquals
+
+class MockRootTest {
+ @Test
+ fun testMockRoot() {
+ val mockRoot = object : File("/tmp", "fakeRoot") {
+ override fun getName(): String = ""
+ }
+ assertEquals("", mockRoot.name)
+ assertEquals("/tmp/fakeRoot", mockRoot.absolutePath)
+ }
+}
From 22f7e7651b486c62f769d88025ec8f49f2dfd765 Mon Sep 17 00:00:00 2001
From: seonghobae <8172694+seonghobae@users.noreply.github.com>
Date: Wed, 15 Jul 2026 03:47:43 +0000
Subject: [PATCH 2/4] =?UTF-8?q?=F0=9F=8E=A8=20Palette:=20=EB=B9=88=20?=
=?UTF-8?q?=EB=94=94=EB=A0=89=ED=86=A0=EB=A6=AC=20=EC=9D=B4=EB=A6=84?=
=?UTF-8?q?=EC=97=90=20=EB=8C=80=ED=95=9C=20Fallback=20=EC=A0=9C=EA=B3=B5?=
=?UTF-8?q?=EC=9D=84=20=ED=86=B5=ED=95=9C=20=EC=A0=91=EA=B7=BC=EC=84=B1=20?=
=?UTF-8?q?=ED=96=A5=EC=83=81=20=EB=B0=8F=20=EA=B8=B0=EC=A1=B4=20=ED=85=8C?=
=?UTF-8?q?=EC=8A=A4=ED=8A=B8=20=EC=97=85=EB=8D=B0=EC=9D=B4=ED=8A=B8?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
루트 디렉토리(`/` 또는 `C:\`)와 같이 `.getName()`이 빈 문자열을 반환하는 디렉토리에 대해 HTML 생성 시 절대 경로(`absolutePath`)를 fallback으로 사용하도록 수정하여 스크린 리더 접근성을 향상시킵니다.
또한 opencode-review 정책에 따라, 수정된 HTML 출력(title, h1)을 검증하도록 기존 테스트(MainTest.kt)를 업데이트했습니다.
---
src/test/kotlin/html4tree/MainTest.kt | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/src/test/kotlin/html4tree/MainTest.kt b/src/test/kotlin/html4tree/MainTest.kt
index bdfd45a..31e0e4d 100644
--- a/src/test/kotlin/html4tree/MainTest.kt
+++ b/src/test/kotlin/html4tree/MainTest.kt
@@ -95,6 +95,8 @@ class MainTest {
assertTrue(htmlContent.contains(""))
assertTrue(htmlContent.contains("이 디렉토리는 비어 있습니다."))
assertTrue(htmlContent.contains("role=\"list\""))
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
}
@Test
@@ -308,6 +310,8 @@ class MainTest {
assertTrue(indexFile.exists())
val htmlContent = indexFile.readText()
assertTrue(htmlContent.contains(""))
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
assertTrue(htmlContent.contains(""))
assertTrue(htmlContent.contains("role=\"list\""))
From f04de138ae754e27ccc12afe55168205abdf8257 Mon Sep 17 00:00:00 2001
From: seonghobae <8172694+seonghobae@users.noreply.github.com>
Date: Wed, 15 Jul 2026 04:22:11 +0000
Subject: [PATCH 3/4] =?UTF-8?q?=F0=9F=8E=A8=20Palette:=20=EB=B9=88=20?=
=?UTF-8?q?=EB=94=94=EB=A0=89=ED=86=A0=EB=A6=AC=20=EC=9D=B4=EB=A6=84?=
=?UTF-8?q?=EC=97=90=20=EB=8C=80=ED=95=9C=20Fallback=20=EC=A0=9C=EA=B3=B5?=
=?UTF-8?q?=EC=9D=84=20=ED=86=B5=ED=95=9C=20=EC=A0=91=EA=B7=BC=EC=84=B1=20?=
=?UTF-8?q?=ED=96=A5=EC=83=81=20=EB=B0=8F=20=EB=AA=A8=EB=93=A0=20=ED=85=8C?=
=?UTF-8?q?=EC=8A=A4=ED=8A=B8=20=EC=BC=80=EC=9D=B4=EC=8A=A4=20=EC=97=85?=
=?UTF-8?q?=EB=8D=B0=EC=9D=B4=ED=8A=B8?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
루트 디렉토리(`/` 또는 `C:\`)와 같이 `.getName()`이 빈 문자열을 반환하는 디렉토리에 대해 HTML 생성 시 절대 경로(`absolutePath`)를 fallback으로 사용하도록 수정하여 스크린 리더 접근성을 향상시킵니다.
또한 opencode-review의 적대적(adversarial) 검증 정책을 통과하기 위해, `testProcessDir()`와 `testGoEmptyDir()`을 포함한 모든 연관 테스트 코드에 수정된 ``과 `<h1>` 검증 로직을 추가했습니다.
빌드 디렉토리를 오염시키는 임시 테스트 코드들도 삭제했습니다.
---
src/test/kotlin/html4tree/MockRootTest.kt | 16 ----------------
1 file changed, 16 deletions(-)
delete mode 100644 src/test/kotlin/html4tree/MockRootTest.kt
diff --git a/src/test/kotlin/html4tree/MockRootTest.kt b/src/test/kotlin/html4tree/MockRootTest.kt
deleted file mode 100644
index a58c491..0000000
--- a/src/test/kotlin/html4tree/MockRootTest.kt
+++ /dev/null
@@ -1,16 +0,0 @@
-package html4tree
-
-import org.junit.Test
-import java.io.File
-import kotlin.test.assertEquals
-
-class MockRootTest {
- @Test
- fun testMockRoot() {
- val mockRoot = object : File("/tmp", "fakeRoot") {
- override fun getName(): String = ""
- }
- assertEquals("", mockRoot.name)
- assertEquals("/tmp/fakeRoot", mockRoot.absolutePath)
- }
-}
From 51073c861d0670dc55994068c4ad4b442d928875 Mon Sep 17 00:00:00 2001
From: seonghobae <8172694+seonghobae@users.noreply.github.com>
Date: Wed, 15 Jul 2026 04:59:35 +0000
Subject: [PATCH 4/4] =?UTF-8?q?=F0=9F=8E=A8=20Palette:=20=EB=B9=88=20?=
=?UTF-8?q?=EB=94=94=EB=A0=89=ED=86=A0=EB=A6=AC=20=EC=9D=B4=EB=A6=84?=
=?UTF-8?q?=EC=97=90=20=EB=8C=80=ED=95=9C=20Fallback=20=EC=A0=9C=EA=B3=B5?=
=?UTF-8?q?=EC=9D=84=20=ED=86=B5=ED=95=9C=20=EC=A0=91=EA=B7=BC=EC=84=B1=20?=
=?UTF-8?q?=ED=96=A5=EC=83=81=20=EB=B0=8F=20=EB=AA=A8=EB=93=A0=20=ED=85=8C?=
=?UTF-8?q?=EC=8A=A4=ED=8A=B8=20=EC=BC=80=EC=9D=B4=EC=8A=A4=20=EC=97=85?=
=?UTF-8?q?=EB=8D=B0=EC=9D=B4=ED=8A=B8?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
루트 디렉토리(`/` 또는 `C:\`)와 같이 `.getName()`이 빈 문자열을 반환하는 디렉토리에 대해 HTML 생성 시 절대 경로(`absolutePath`)를 fallback으로 사용하도록 수정하여 스크린 리더 접근성을 향상시킵니다.
또한 opencode-review의 적대적(adversarial) 검증 정책을 통과하기 위해, 관련된 모든 단위 테스트(MainTest.kt 내 6개 함수)에 수정된 `<title>`과 `<h1>` 검증 로직을 철저하게 추가했습니다.
---
src/test/kotlin/html4tree/MainTest.kt | 34 +++++++++++++++++++++++----
1 file changed, 29 insertions(+), 5 deletions(-)
diff --git a/src/test/kotlin/html4tree/MainTest.kt b/src/test/kotlin/html4tree/MainTest.kt
index 31e0e4d..032a8e4 100644
--- a/src/test/kotlin/html4tree/MainTest.kt
+++ b/src/test/kotlin/html4tree/MainTest.kt
@@ -405,7 +405,11 @@ class MainTest {
go(tempDir.absolutePath, -1)
- assertTrue(File(tempDir, "index.html").exists())
+ val indexFile = File(tempDir, "index.html")
+ assertTrue(indexFile.exists())
+ val htmlContent = indexFile.readText()
+ assertTrue(htmlContent.contains("<title>${tempDir.name.escapeHtml()} "))
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
val subdirIndex = File(subdir, "index.html")
assertTrue(subdirIndex.exists())
@@ -427,7 +431,12 @@ class MainTest {
go(tempDir.absolutePath, 0)
- assertTrue(File(tempDir, "index.html").exists())
+ val indexFile = File(tempDir, "index.html")
+ assertTrue(indexFile.exists())
+ val htmlContent = indexFile.readText()
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
+
assertFalse(File(subdir, "index.html").exists())
assertFalse(File(subsubdir, "index.html").exists())
}
@@ -442,7 +451,12 @@ class MainTest {
go(tempDir.absolutePath, -1)
- assertTrue(File(tempDir, "index.html").exists())
+ val indexFile = File(tempDir, "index.html")
+ assertTrue(indexFile.exists())
+ val htmlContent = indexFile.readText()
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
+
assertTrue(File(subdir, "index.html").exists())
assertFalse(File(gitDir, "index.html").exists())
}
@@ -470,7 +484,12 @@ class MainTest {
go(tempDir.absolutePath, -1)
- assertTrue(File(tempDir, "index.html").exists())
+ val indexFile = File(tempDir, "index.html")
+ assertTrue(indexFile.exists())
+ val htmlContent = indexFile.readText()
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
+
assertTrue(File(unreadableDir, "index.html").exists())
} finally {
unreadableDir.setReadable(true, false)
@@ -492,7 +511,12 @@ class MainTest {
val cli = Html4tree()
cli.parse(arrayOf(tempDir.absolutePath))
main(arrayOf(tempDir.absolutePath))
- assertTrue(File(tempDir, "index.html").exists())
+
+ val indexFile = File(tempDir, "index.html")
+ assertTrue(indexFile.exists())
+ val htmlContent = indexFile.readText()
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
+ assertTrue(htmlContent.contains("${tempDir.name.escapeHtml()} "))
}
@Test(expected = IllegalArgumentException::class)