@simplysm/sd-claude 13.0.71 → 13.0.74

package/claude/skills/sd-document/extract_xlsx.py

@@ -1,5 +1,5 @@
 #!/usr/bin/env python3
-"""XLSX 파일에서 데이터와 이미지를 셀 위치와 함께 추출한다."""
+"""Extract data and images from XLSX files with cell positions."""
 
 import sys
 import io
@@ -17,7 +17,7 @@ def ensure_packages():
         try:
             __import__(import_name)
         except ImportError:
-            print(f"패키지 설치 중: {pip_name}...", file=sys.stderr)
+            print(f"Installing package: {pip_name}...", file=sys.stderr)
             subprocess.check_call([sys.executable, "-m", "pip", "install", pip_name],
                                   stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
 
@@ -37,10 +37,10 @@ def extract(file_path):
         ws = wb[sheet_name]
         print(f"## Sheet: {sheet_name}\n")
 
-        # 데이터 추출
+        # Data extraction
         rows = list(ws.iter_rows(values_only=False))
         if not rows:
-            print("(빈 시트)\n")
+            print("(empty sheet)\n")
             continue
 
         for row in rows:
@@ -53,7 +53,7 @@ def extract(file_path):
                     cells.append(str(val).strip())
             print(f"[{row[0].coordinate.split('1')[0]}{row[0].row}] " + " | ".join(cells))
 
-        # 이미지 추출
+        # Image extraction
         if ws._images:
             for img in ws._images:
                 img_idx += 1
@@ -70,9 +70,9 @@ def extract(file_path):
         print()
 
     if img_idx > 0:
-        print(f"---\n이미지 {img_idx}개 저장: {out_dir}")
+        print(f"---\n{img_idx} image(s) saved: {out_dir}")
     else:
-        print("---\n이미지 없음")
+        print("---\nNo images")
 
 
 if __name__ == "__main__":

package/claude/skills/sd-email-analyze/email-analyzer.py

@@ -12,13 +12,13 @@ import base64
 from email.policy import default as default_policy
 from pathlib import Path
 
-# stdout UTF-8 강제 (Windows 호환)
+# Force stdout UTF-8 (Windows compatibility)
 sys.stdout = io.TextIOWrapper(sys.stdout.buffer, encoding="utf-8", errors="replace")
 sys.stderr = io.TextIOWrapper(sys.stderr.buffer, encoding="utf-8", errors="replace")
 
 
 def ensure_packages():
-    """필요한 패키지 자동 설치."""
+    """Auto-install required packages."""
     packages = {"extract-msg": "extract_msg"}
     missing = []
     for pip_name, import_name in packages.items():
@@ -27,7 +27,7 @@ def ensure_packages():
         except ImportError:
             missing.append(pip_name)
     if missing:
-        print(f"패키지 설치 중: {', '.join(missing)}...", file=sys.stderr)
+        print(f"Installing packages: {', '.join(missing)}...", file=sys.stderr)
         subprocess.check_call(
             [sys.executable, "-m", "pip", "install", "-q", *missing],
             stdout=subprocess.DEVNULL,
@@ -326,45 +326,45 @@ def build_report(filepath):
         saved_attachments = save_files(attachments, out_dir)
 
     out = []
-    out.append("# 이메일 분석서\n")
-    out.append(f"**원본 파일**: `{os.path.basename(filepath)}`\n")
-
-    # ── 메일 정보
-    out.append("## 메일 정보\n")
-    out.append("| 항목 | 내용 |")
-    out.append("|------|------|")
-    out.append(f"| **제목** | {headers['subject']} |")
-    out.append(f"| **보낸 사람** | {headers['from']} |")
-    out.append(f"| **받는 사람** | {headers['to']} |")
+    out.append("# Email Analysis Report\n")
+    out.append(f"**Source file**: `{os.path.basename(filepath)}`\n")
+
+    # ── Email info
+    out.append("## Email Info\n")
+    out.append("| Field | Value |")
+    out.append("|-------|-------|")
+    out.append(f"| **Subject** | {headers['subject']} |")
+    out.append(f"| **From** | {headers['from']} |")
+    out.append(f"| **To** | {headers['to']} |")
     if headers["cc"]:
-        out.append(f"| **참조** | {headers['cc']} |")
-    out.append(f"| **날짜** | {headers['date']} |")
-    out.append(f"| **첨부파일** | {len(saved_attachments)}개 |")
+        out.append(f"| **CC** | {headers['cc']} |")
+    out.append(f"| **Date** | {headers['date']} |")
+    out.append(f"| **Attachments** | {len(saved_attachments)} |")
     if all_inline:
-        out.append(f"| **본문 이미지** | {len(all_inline)}개 |")
+        out.append(f"| **Inline images** | {len(all_inline)} |")
     out.append("")
 
-    # ── 본문
-    out.append("## 본문 내용\n")
+    # ── Body
+    out.append("## Body\n")
     body = body_plain
     if not body and body_html:
         body = strip_html(body_html)
-    out.append(body.strip() if body else "_(본문 없음)_")
+    out.append(body.strip() if body else "_(No body)_")
     out.append("")
 
-    # ── 본문 삽입 이미지
+    # ── Inline images
     if all_inline:
-        out.append("## 본문 삽입 이미지\n")
-        out.append("| # | 파일명 | 크기 | 저장 경로 |")
+        out.append("## Inline Images\n")
+        out.append("| # | Filename | Size | Saved path |")
         out.append("|---|--------|------|-----------|")
         for i, img in enumerate(all_inline, 1):
             out.append(f"| {i} | {img['filename']} | {fmt_size(img['size'])} | `{img['saved_path']}` |")
         out.append("")
 
-    # ── 첨부파일
+    # ── Attachments
     if saved_attachments:
-        out.append("## 첨부파일\n")
-        out.append("| # | 파일명 | 크기 | 저장 경로 |")
+        out.append("## Attachments\n")
+        out.append("| # | Filename | Size | Saved path |")
         out.append("|---|--------|------|-----------|")
         for i, a in enumerate(saved_attachments, 1):
             out.append(f"| {i} | {a['filename']} | {fmt_size(a['size'])} | `{a['saved_path']}` |")
@@ -382,12 +382,12 @@ if __name__ == "__main__":
 
     path = sys.argv[1]
     if not os.path.isfile(path):
-        print(f"파일을 찾을 수 없습니다: {path}", file=sys.stderr)
+        print(f"File not found: {path}", file=sys.stderr)
         sys.exit(1)
 
     ext = Path(path).suffix.lower()
     if ext not in (".eml", ".msg"):
-        print(f"지원하지 않는 형식: {ext} (.eml 또는 .msg만 지원)", file=sys.stderr)
+        print(f"Unsupported format: {ext} (only .eml and .msg are supported)", file=sys.stderr)
         sys.exit(1)
 
     print(build_report(path))

package/claude/skills/sd-plan/SKILL.md

@@ -49,6 +49,15 @@ Assume they are a skilled developer, but know almost nothing about our toolset o
 ---
 ```
 
+## Package Manager Detection
+
+When writing run commands in the plan, detect the package manager:
+- If `pnpm-lock.yaml` exists in project root → use `pnpm`
+- If `yarn.lock` exists in project root → use `yarn`
+- Otherwise → use `npm`
+
+`$PM` in the task template below refers to the detected package manager.
+
 ## Task Structure
 
 ```markdown
@@ -70,7 +79,7 @@ test("specific behavior", () => {
 
 **Step 2: Run test to verify it fails**
 
-Run: `pnpm vitest exact/path/to/tests/file.spec.ts --run`
+Run: `$PM run vitest exact/path/to/tests/file.spec.ts --run`
 Expected: FAIL with "functionUnderTest is not defined"
 
 **Step 3: Write minimal implementation**
@@ -83,7 +92,7 @@ function functionUnderTest(input: InputType): OutputType {
 
 **Step 4: Run test to verify it passes**
 
-Run: `pnpm vitest exact/path/to/tests/file.spec.ts --run`
+Run: `$PM run vitest exact/path/to/tests/file.spec.ts --run`
 Expected: PASS
 
 **Step 5: Commit**

package/claude/skills/sd-plan-dev/SKILL.md

@@ -221,11 +221,13 @@ Done!
 
 ## Batch Integration Check
 
-Between batches, run targeted verification on affected packages before starting the next batch:
+Between batches, run targeted verification on affected packages before starting the next batch.
+
+Detect the package manager first (`pnpm-lock.yaml` → pnpm, `yarn.lock` → yarn, otherwise → npm):
 
 ```bash
-npm run typecheck [affected packages]
-npm run lint [affected packages]
+$PM run typecheck [affected packages]
+$PM run lint [affected packages]
 ```
 
 This catches cross-task integration issues early — especially when the next batch depends on the current batch's output. Do NOT skip this even if individual task reviews passed.

package/claude/skills/sd-plan-dev/final-review-prompt.md

@@ -37,9 +37,9 @@ Individual tasks already passed spec and quality reviews. Focus on cross-task in
 
 ### Verification
 
-Run and report results:
-- `npm run typecheck [affected packages]`
-- `npm run lint [affected packages]`
+Detect the package manager (`pnpm-lock.yaml` → pnpm, `yarn.lock` → yarn, otherwise → npm), then run and report results:
+- `$PM run typecheck [affected packages]`
+- `$PM run lint [affected packages]`
 
 ### Report
 

package/claude/skills/sd-readme/SKILL.md

@@ -7,73 +7,59 @@ model: sonnet
 
 # sd-readme
 
-Sync package README.md source index with current exports from `index.ts`.
+Sync package README.md with current source code by comparing exports against documentation.
 
 ## Purpose
 
-`@simplysm/*` packages ship `.ts` source and test files. Claude Code reads source files directly from `node_modules/` for API details (props, types, signatures, usage patterns).
+README.md is the **sole API documentation source for Claude Code**. When Claude Code works in a consumer app using `@simplysm/*` packages, it reads README.md from `node_modules/` to understand the library API. Claude Code does NOT read JSDoc from source files.
 
-**README.md is a source file index** — it tells Claude Code **which files to read**, not the API itself.
+**Therefore: every exported symbol must be documented in README.**
 
 ## Modes
 
 - **Single package** (`$ARGUMENTS` = package name or path): Update one package's README
 - **Batch** (`$ARGUMENTS` empty): Discover and update all packages in parallel
 
-## What Goes Where
+## README Writing Rules
 
-| In README | NOT in README (read from source) |
-|-----------|----------------------------------|
-| Package description (one-line) | Prop tables |
-| Installation + peer dependencies | Code examples |
-| Setup/configuration (Provider tree, Tailwind preset) | Type/interface definitions |
-| **Source index table** | API signatures |
-| License | Behavioral descriptions |
+- Written in **English**
+- All code examples must include **import paths**: `import { ... } from "@simplysm/..."`
+- **Every export** from `index.ts` must be documented — including those with `@internal` JSDoc
+- No changelog, version history, or "recently updated" sections
+- Section organization follows `index.ts` `#region` structure
+- Heading levels: `##` for major sections, `###` for sub-sections
 
-## README Structure
+### Standard Structure
 
 ```markdown
 # @simplysm/{package-name}
 
-{One-line description from package.json}
+{One-line description}
 
 ## Installation
 
-pnpm add @simplysm/{package-name}
-
-**Peer Dependencies:** (if any)
+## Main Modules
 
-## Configuration
+### {Category matching index.ts #region}
 
-{Only for setup patterns — Provider wrapping, Tailwind preset, etc.}
-{Include brief code snippets for wiring, NOT per-component API}
+- Description + code examples per export
 
-## Source Index
+## Types
 
-### {Category matching index.ts #region}
+## Dependencies (only when peer deps exist)
+```
 
-| Source | Exports | Description | Test |
-|--------|---------|-------------|------|
-| `src/path/to/File.tsx` | `ComponentA`, `ComponentB` | Brief one-line description | `File.test.tsx` |
-| `src/path/to/types.ts` | `TypeA`, `TypeB` | Type definitions for X | - |
+### docs/ Subfolder Rules
 
-## License
-```
+When README exceeds ~500 lines, split detailed documentation into `docs/`:
 
-**Source index table rules:**
-- One row per `export *` statement in `index.ts`
-- Exports column: list all exported symbols from that file
-- Description column: brief one-line summary of what the file provides (enough for Claude Code to decide whether to read the source)
-- Test column: test file name if exists, `-` if not
-- Source paths relative to package root
+- README.md becomes an **overview/index** with links: `[functionName](docs/category.md#anchor)`
+- docs/ files contain detailed descriptions, full code examples, parameter tables
+- File organization follows index.ts `#region` (e.g., `docs/types.md`, `docs/utils.md`)
 
-## General Rules
+When README is under ~500 lines, keep everything inline.
 
-- Written in **English**
-- Sections follow `index.ts` `#region` structure
-- **Every `export *`** from `index.ts` must appear in source index
-- No changelog, version history, or "recently updated" sections
-- Configuration section: only setup/wiring patterns, NOT per-component API
+**If docs/ already exists, maintain and update it. Do not remove an existing docs/ structure.**
 
 ## Single Package Mode
 
@@ -84,66 +70,62 @@ If not starting with `packages/`, prepend it.
 
 ### Step 2: Build Export Map (Source of Truth)
 
-1. Read `<pkg-path>/src/index.ts` — get all `export *` statements and their `#region` grouping
-2. For each `export * from "./path/to/file"`:
-   - Resolve actual file path (`.ts`, `.tsx`)
-   - Read file to extract exported symbol names
-   - Write a brief one-line description of what the file provides
-   - Check if test file exists (e.g., `File.test.tsx`, `File.spec.ts`)
+1. Read `<pkg-path>/src/index.ts` — get all exports and their `#region` grouping
+2. For each exported module, read the source file to extract:
+   - Function signatures (params, return type, overloads)
+   - Class public API (constructor, methods, properties)
+   - Type/interface definitions
+   - Default values, options objects
 
-### Step 3: Build Index Map
+### Step 3: Build Documentation Map
 
-Read `<pkg-path>/README.md` (if exists).
-Extract existing source index table entries — map each source file to its row.
+Read `<pkg-path>/README.md` (if exists). If docs/ exists, read those files too.
+Map each documented item to its current documentation content.
 
 ### Step 4: Diff and Report
 
-Compare export map (Step 2) against index map (Step 3):
+Compare export map (Step 2) against documentation map (Step 3):
 
-| Status | Meaning |
-|--------|---------|
-| **ADDED** | `export *` in index.ts but source file not in README index |
-| **REMOVED** | In README index but no longer in index.ts |
-| **CHANGED** | Same file, but exported symbols changed |
-| **OK** | Index entry matches source |
+| Status      | Meaning                              |
+| ----------- | ------------------------------------ |
+| **ADDED**   | Exported in source but not in README |
+| **REMOVED** | In README but no longer exported     |
+| **CHANGED** | Both exist but API signature differs |
+| **OK**      | Documentation matches source         |
 
 **Report to user before editing:**
 
 ```
-ADDED (2):
-  - src/providers/i18n/I18nContext.tsx (useI18n, useI18nOptional, I18nProvider)
-  - src/providers/i18n/I18nContext.types.ts (I18nContextValue, I18nConfigureOptions, FlatDict)
+ADDED (3):
+  - strToCamelCase (from utils/str.ts)
+  - objGetChainValueByDepth (from utils/obj.ts)
+  - ZipArchiveProgress type (from zip/sd-zip.ts)
 
 REMOVED (1):
-  - src/components/SelectList.tsx (no longer exported)
+  - oldFunction (no longer exported)
 
-CHANGED (1):
-  - src/features/shared-data/SharedDataSelectList.tsx: exports changed
+CHANGED (2):
+  - objMerge: added `deep` parameter
+  - Set.toggle: added `addOrDel` optional parameter
 
-OK: 42 entries unchanged
+OK: 45 items unchanged
 ```
 
 **Wait for user confirmation before proceeding to edit.**
 
 ### Step 5: Apply Updates
 
-- **ADDED**: Add row to source index table in the section matching the item's `#region` in index.ts.
-- **REMOVED**: Delete row from source index table.
-- **CHANGED**: Update exports column.
+- **ADDED**: Write documentation matching existing style. Place in the section matching the item's `#region` in index.ts. Include description + code example with import path.
+- **REMOVED**: Delete the documentation entry.
+- **CHANGED**: Update existing entry to match current API. Preserve code examples if still valid.
 - **OK**: Do not touch.
 
-### Step 6: Cleanup Check
+**If README uses docs/ links**: update the corresponding docs/ file, not just README.
 
-If `docs/` subfolder exists from old full-documentation approach, report it:
-
-```
-NOTE: docs/ subfolder exists with old-style full documentation.
-  Source index approach makes docs/ redundant.
-  Files: docs/form-controls.md, docs/providers.md, ...
-  Remove docs/ folder? (requires user confirmation)
-```
+### Step 6: Size Check
 
-**Do not auto-delete docs/ — always ask user first.**
+After updates, if README exceeds ~500 lines and no docs/ exists:
+suggest splitting to the user (do not auto-split without confirmation).
 
 ## Batch Mode
 
@@ -155,33 +137,31 @@ When `$ARGUMENTS` is empty:
 **Per-package subagent prompt template:**
 
 ```
-Update README.md source index for package {pkg-name} at {pkg-path}.
+Update README.md for package {pkg-name} at {pkg-path}.
 
-PURPOSE: README.md is a source file index. @simplysm/* packages ship .ts source + tests.
-Claude Code reads source files directly. README tells Claude WHICH files to read.
+PURPOSE: README.md is the sole API documentation source for Claude Code.
+Every exported symbol must be documented. Claude Code does NOT read JSDoc.
 
 STEPS:
-1. Read {pkg-path}/src/index.ts — get all `export *` statements and #region grouping.
-2. For each export, resolve source file path and find test file.
-3. Read each source file to extract exported symbol names.
-4. Read {pkg-path}/README.md (if exists).
-5. Compare exports vs README source index:
-   - ADDED: add row to source index table.
-   - REMOVED: delete row.
-   - CHANGED: update exports/description column.
+1. Read {pkg-path}/src/index.ts — get all exports and #region grouping.
+2. For each export, read the source file for signatures, classes, types.
+3. Read {pkg-path}/README.md (and docs/ if exists).
+4. Compare exports vs documentation:
+   - ADDED (in source, not in README): add documentation with description + code example.
+     Include import path: import { X } from "@simplysm/{pkg-name}"
+   - REMOVED (in README, not in source): delete documentation.
+   - CHANGED (both exist, API differs): update to match current API.
    - OK: don't touch.
-6. Section organization follows index.ts #region structure.
-7. Write in English. No changelog sections.
-8. If README doesn't exist, create with structure:
+5. Section organization follows index.ts #region structure.
+6. Write in English. No changelog sections.
+7. If README doesn't exist, create with standard structure:
    # @simplysm/{pkg-name}
    {description from package.json}
    ## Installation
-   ## Source Index (tables per #region)
-   ## License
-9. Do NOT write prop tables, code examples, or type definitions.
-   Source index table only: Source | Exports | Description | Test
-   Description: brief one-line summary of what the file provides.
-10. Report: list of ADDED/REMOVED/CHANGED items.
+   ## Main Modules (sections per #region)
+   ## Types
+8. If docs/ subfolder exists, update those files too.
+9. Report: list of ADDED/REMOVED/CHANGED items.
 ```
 
 **Project root subagent prompt:**
@@ -198,14 +178,14 @@ Report what was changed.
 
 ## Common Mistakes
 
-| Mistake | Fix |
-|---------|-----|
-| Writing prop tables or code examples | README is a source index — Claude reads source files for API details |
-| Missing or vague Description | Each row needs a brief description so Claude can decide whether to read the file |
-| Skipping exports from index.ts | Every `export *` must appear in the source index |
-| Putting per-component API in Configuration | Configuration = setup patterns only (Provider tree, Tailwind preset) |
-| Arbitrary section reorganization | Follow index.ts `#region` structure |
-| Writing in Korean | README must be in English |
-| Adding changelog sections | Never add version history |
-| Editing before reporting diff | Always report and wait for confirmation |
-| Auto-deleting docs/ folder | Report docs/ existence, ask user before removing |
+| Mistake                               | Fix                                                                 |
+| ------------------------------------- | ------------------------------------------------------------------- |
+| Skipping exports with @internal JSDoc | Document ALL exports — Claude Code doesn't read JSDoc               |
+| Rewriting OK sections                 | Only touch ADDED/REMOVED/CHANGED items                              |
+| Ignoring existing docs/ structure     | If docs/ exists, update those files too                             |
+| Arbitrary section reorganization      | Follow index.ts `#region` structure                                 |
+| Missing import paths in examples      | Always: `import { X } from "@simplysm/..."`                         |
+| Writing in Korean                     | README must be in English                                           |
+| Adding changelog sections             | Never add version history                                           |
+| Editing before reporting diff         | Always report ADDED/REMOVED/CHANGED and wait for confirmation       |
+| Destroying docs/ link format          | If README uses `[name](docs/file.md#anchor)`, preserve that pattern |