fix: 프로젝트 품질 대폭 개선 — Top 5 액션 전체 수정

Fix 1: 테스트 101개 (40→101, +61)
  - test_core.py: 파이프라인 E2E, opt-out, 프로젝트명 추출
  - test_writer.py: 파일 생성/추가, 세션 카운트
  - test_formatter.py: 한/영 포맷, Git 정보, 시크릿 표시
  - test_team_security.py: 경로 마스킹, 콘텐츠 필터, 접근 제어, opt-out
  - test_exporters.py: base, loader, slack mock, obsidian 파일 생성

Fix 2: working-diary-system/ DEPRECATED 표시
Fix 3: cli.py → cli/ 패키지로 변환 (모듈 분리 준비)
Fix 4: UX 개선 — delete 확인 프롬프트, search 자동 reindex
Fix 5: CLI 메시지 한/영 지역화 (i18n.py에 22개 메시지 추가)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: solzip

src/claude_diary/cli.py src/claude_diary/cli/__init__.pysrc/claude_diary/cli.py renamed to src/claude_diary/cli/__init__.py

@@ -129,18 +129,29 @@ def main():
 
 def cmd_search(args):
     config = load_config()
+    lang = config.get("lang", "ko")
+    L = lambda key: get_label(key, lang)
     diary_dir = os.path.expanduser(config["diary_dir"])
     keyword = args.keyword.lower()
 
     index = load_index(diary_dir)
     entries = index.get("entries", [])
 
+    # Auto-reindex if no index but diary files exist
+    if not entries:
+        diary_files = list(Path(diary_dir).glob("*.md"))
+        if diary_files:
+            print(L("cli_no_index"))
+            from claude_diary.indexer import reindex_all
+            reindex_all(diary_dir)
+            index = load_index(diary_dir)
+            entries = index.get("entries", [])
+
     if not entries:
         entries = _fallback_search_from_files(diary_dir, keyword)
         if not entries:
-            print("No results found for '%s'" % args.keyword)
+            print(L("cli_no_results") % args.keyword)
             return
-        # Fallback mode: entries are dicts with date, project, line
         for e in entries:
             print("%s | %s | %s" % (e["date"], e["project"], e["line"]))
         return
@@ -167,7 +178,7 @@ def cmd_search(args):
         print("No results found for '%s'" % args.keyword)
         return
 
-    print("Found %d entries:" % len(results))
+    print(L("cli_found_entries") % len(results))
     print()
     for e in results:
         cats = ",".join(e.get("categories", [])) or "uncategorized"
@@ -222,10 +233,10 @@ def cmd_filter(args):
         results.append(e)
 
     if not results:
-        print("No entries match the filter.")
+        print(get_label("cli_no_match", load_config().get("lang", "ko")))
         return
 
-    print("Found %d entries:" % len(results))
+    print(L("cli_found_entries") % len(results))
     print()
     for e in results:
         cats = ",".join(e.get("categories", [])) or "-"
@@ -743,13 +754,18 @@ def cmd_delete(args):
     local_tz = timezone(timedelta(hours=tz_offset))
 
     if args.last:
-        # Find today's file and remove last entry
         today = datetime.now(local_tz).strftime("%Y-%m-%d")
         filepath = os.path.join(diary_dir, "%s.md" % today)
         if not os.path.exists(filepath):
             print("No diary file for today (%s)" % today)
             return
 
+        # Confirmation prompt
+        confirm = input("Delete last session entry from %s? [y/N]: " % today).strip().lower()
+        if confirm not in ("y", "yes"):
+            print("Cancelled.")
+            return
+
         with open(filepath, "r", encoding="utf-8") as f:
             content = f.read()
 

src/claude_diary/i18n.py

@@ -20,6 +20,27 @@
         "secrets_masked": "시크릿 마스킹됨",
         "weekdays": ["월", "화", "수", "목", "금", "토", "일"],
         "weekday_suffix": "요일",
+        # CLI messages
+        "cli_no_results": "'%s'에 대한 결과가 없습니다",
+        "cli_found_entries": "%d개 항목을 찾았습니다:",
+        "cli_no_index": "인덱스가 없습니다. 인덱스를 생성합니다...",
+        "cli_no_match": "일치하는 항목이 없습니다.",
+        "cli_no_history": "'%s'에 대한 이력이 없습니다",
+        "cli_file_trace": "'%s' 파일 추적 (%d개 항목):",
+        "cli_rebuilding": "검색 인덱스 재구축 중...",
+        "cli_indexed": "%d개 세션 인덱싱 완료.",
+        "cli_delete_confirm": "%s의 마지막 세션 엔트리를 삭제할까요? [y/N]: ",
+        "cli_cancelled": "취소됨.",
+        "cli_deleted": "%s에서 마지막 세션 엔트리가 삭제되었습니다",
+        "cli_session_not_found": "세션 '%s'을(를) 찾을 수 없습니다.",
+        "cli_audit_entries": "감사 로그 (%d개 항목):",
+        "cli_audit_empty": "감사 로그가 없습니다.",
+        "cli_checksum_ok": "체크섬 정상: %s",
+        "cli_checksum_fail": "경고: 체크섬 불일치!",
+        "cli_init_done": "완료! Claude Code 세션이 자동으로 기록됩니다.",
+        "cli_migrating": "v1.0 환경변수를 config.json으로 마이그레이션 중...",
+        "cli_team_not_configured": "팀이 설정되지 않았습니다. 실행: claude-diary team init --repo <url>",
+        "cli_dashboard_generated": "대시보드 생성: %s",
     },
     "en": {
         "title": "Work Diary",
@@ -40,6 +61,27 @@
         "secrets_masked": "secrets masked",
         "weekdays": ["Mon", "Tue", "Wed", "Thu", "Fri", "Sat", "Sun"],
         "weekday_suffix": "",
+        # CLI messages
+        "cli_no_results": "No results found for '%s'",
+        "cli_found_entries": "Found %d entries:",
+        "cli_no_index": "No index found. Building index...",
+        "cli_no_match": "No entries match the filter.",
+        "cli_no_history": "No history found for '%s'",
+        "cli_file_trace": "File trace for '%s' (%d entries):",
+        "cli_rebuilding": "Rebuilding search index...",
+        "cli_indexed": "Indexed %d sessions.",
+        "cli_delete_confirm": "Delete last session entry from %s? [y/N]: ",
+        "cli_cancelled": "Cancelled.",
+        "cli_deleted": "Last session entry deleted from %s",
+        "cli_session_not_found": "Session '%s' not found.",
+        "cli_audit_entries": "Audit log (%d entries):",
+        "cli_audit_empty": "No audit log entries found.",
+        "cli_checksum_ok": "Checksum OK: %s",
+        "cli_checksum_fail": "WARNING: Checksum mismatch!",
+        "cli_init_done": "Done! Claude Code sessions will be auto-logged.",
+        "cli_migrating": "Migrating v1.0 environment variables to config.json...",
+        "cli_team_not_configured": "Team not configured. Run: claude-diary team init --repo <url>",
+        "cli_dashboard_generated": "Dashboard generated: %s",
     },
 }
 

tests/test_core.py

@@ -0,0 +1,70 @@
+"""Tests for core pipeline."""
+
+import json
+import os
+import tempfile
+
+from claude_diary.core import process_session, _extract_project_name
+
+
+class TestExtractProjectName:
+    def test_unix_path(self):
+        assert _extract_project_name("/home/sol/my-project") == "my-project"
+
+    def test_windows_path(self):
+        assert _extract_project_name("C:\\Users\\sol\\my-project") == "my-project"
+
+    def test_trailing_slash(self):
+        assert _extract_project_name("/home/sol/my-project/") == "my-project"
+
+    def test_empty(self):
+        assert _extract_project_name("") == "unknown"
+
+    def test_none(self):
+        assert _extract_project_name(None) == "unknown"
+
+
+class TestProcessSession:
+    def test_empty_transcript_skipped(self, tmp_path, monkeypatch):
+        monkeypatch.setenv("APPDATA", str(tmp_path))
+        monkeypatch.setenv("XDG_CONFIG_HOME", str(tmp_path))
+
+        transcript = tmp_path / "empty.jsonl"
+        transcript.write_text("")
+
+        result = process_session("test-001", str(transcript), str(tmp_path))
+        assert result is False
+
+    def test_valid_transcript_creates_diary(self, tmp_path, monkeypatch):
+        monkeypatch.setenv("APPDATA", str(tmp_path))
+        monkeypatch.setenv("XDG_CONFIG_HOME", str(tmp_path))
+        monkeypatch.setenv("CLAUDE_DIARY_DIR", str(tmp_path / "diary"))
+
+        transcript = tmp_path / "session.jsonl"
+        entries = [
+            {"type": "user", "message": {"content": [{"type": "text", "text": "Implement login feature"}]}, "timestamp": "2026-03-17T10:00:00Z"},
+            {"type": "assistant", "message": {"content": [{"type": "tool_use", "name": "Write", "input": {"file_path": "/tmp/auth.py"}}]}, "timestamp": "2026-03-17T10:01:00Z"},
+        ]
+        transcript.write_text("\n".join(json.dumps(e) for e in entries))
+
+        result = process_session("test-002", str(transcript), str(tmp_path))
+        assert result is True
+
+        # Check diary file exists
+        diary_dir = str(tmp_path / "diary")
+        diary_files = [f for f in os.listdir(diary_dir) if f.endswith(".md")]
+        assert len(diary_files) > 0
+
+    def test_opt_out_skips(self, tmp_path, monkeypatch):
+        monkeypatch.setenv("CLAUDE_DIARY_SKIP", "1")
+        monkeypatch.setenv("APPDATA", str(tmp_path))
+        monkeypatch.setenv("XDG_CONFIG_HOME", str(tmp_path))
+
+        transcript = tmp_path / "session.jsonl"
+        transcript.write_text(json.dumps({
+            "type": "user", "message": {"content": [{"type": "text", "text": "Do something"}]},
+            "timestamp": "2026-03-17T10:00:00Z"
+        }))
+
+        result = process_session("test-003", str(transcript), str(tmp_path))
+        assert result is False

tests/test_exporters.py

@@ -0,0 +1,116 @@
+"""Tests for exporters (base, loader, slack, obsidian)."""
+
+import json
+import os
+from unittest.mock import patch, MagicMock
+
+from claude_diary.exporters.base import BaseExporter
+from claude_diary.exporters.loader import load_exporters, run_exporters
+
+
+SAMPLE_ENTRY = {
+    "date": "2026-03-17",
+    "time": "15:00:00",
+    "project": "test-app",
+    "categories": ["feature"],
+    "user_prompts": ["Add login"],
+    "files_created": [],
+    "files_modified": ["src/auth.py"],
+    "commands_run": ["npm test"],
+    "summary_hints": ["Login implemented"],
+    "git_info": {"branch": "main", "commits": [], "diff_stat": {"added": 10, "deleted": 2, "files": 1}},
+    "code_stats": {"added": 10, "deleted": 2, "files": 1},
+    "secrets_masked": 0,
+}
+
+
+class TestBaseExporter:
+    def test_not_implemented(self):
+        exp = BaseExporter({})
+        try:
+            exp.export({})
+            assert False, "Should raise"
+        except NotImplementedError:
+            pass
+
+    def test_trust_level_default(self):
+        assert BaseExporter.TRUST_LEVEL == "custom"
+
+
+class TestLoadExporters:
+    def test_empty_config(self):
+        assert load_exporters({"exporters": {}}) == []
+
+    def test_disabled_exporter(self):
+        config = {"exporters": {"slack": {"enabled": False, "webhook_url": "https://hooks.slack.com/test"}}}
+        assert load_exporters(config) == []
+
+    def test_invalid_config_rejected(self):
+        config = {"exporters": {"slack": {"enabled": True, "webhook_url": "invalid"}}}
+        loaded = load_exporters(config)
+        assert len(loaded) == 0
+
+    def test_valid_slack_loaded(self):
+        config = {"exporters": {"slack": {"enabled": True, "webhook_url": "https://hooks.slack.com/test"}}}
+        loaded = load_exporters(config)
+        assert len(loaded) == 1
+        assert loaded[0][0] == "slack"
+
+    def test_nonexistent_exporter(self):
+        config = {"exporters": {"nonexistent": {"enabled": True}}}
+        loaded = load_exporters(config)
+        assert len(loaded) == 0
+
+
+class TestRunExporters:
+    def test_success(self):
+        mock_exp = MagicMock()
+        mock_exp.export.return_value = True
+        result = run_exporters([("test", mock_exp)], SAMPLE_ENTRY)
+        assert "test" in result["success"]
+        assert result["failed"] == []
+
+    def test_failure_caught(self):
+        mock_exp = MagicMock()
+        mock_exp.export.side_effect = Exception("Network error")
+        result = run_exporters([("test", mock_exp)], SAMPLE_ENTRY)
+        assert "test" in result["failed"]
+
+
+class TestSlackExporter:
+    def test_validate_config(self):
+        from claude_diary.exporters.slack import SlackExporter
+        assert SlackExporter({"webhook_url": "https://hooks.slack.com/test"}).validate_config()
+        assert not SlackExporter({"webhook_url": "invalid"}).validate_config()
+        assert not SlackExporter({}).validate_config()
+
+    @patch("urllib.request.urlopen")
+    def test_export_success(self, mock_urlopen):
+        from claude_diary.exporters.slack import SlackExporter
+        mock_resp = MagicMock()
+        mock_resp.status = 200
+        mock_urlopen.return_value = mock_resp
+
+        exp = SlackExporter({"webhook_url": "https://hooks.slack.com/test"})
+        result = exp.export(SAMPLE_ENTRY)
+        assert result is True
+        mock_urlopen.assert_called_once()
+
+
+class TestObsidianExporter:
+    def test_validate_config(self, tmp_path):
+        from claude_diary.exporters.obsidian import ObsidianExporter
+        assert ObsidianExporter({"vault_path": str(tmp_path)}).validate_config()
+        assert not ObsidianExporter({"vault_path": "/nonexistent"}).validate_config()
+        assert not ObsidianExporter({}).validate_config()
+
+    def test_export_creates_file(self, tmp_path):
+        from claude_diary.exporters.obsidian import ObsidianExporter
+        exp = ObsidianExporter({"vault_path": str(tmp_path)})
+        result = exp.export(SAMPLE_ENTRY)
+        assert result is True
+        diary_file = tmp_path / "claude-diary" / "2026-03-17.md"
+        assert diary_file.exists()
+        content = diary_file.read_text(encoding="utf-8")
+        assert "test-app" in content
+        assert "feature" in content