@qijenchen/design-system 0.1.0-beta.8 → 0.1.0-beta.81

package/ds-canonical/hooks/tests/test_check_overlay_handcraft.sh

@@ -0,0 +1,126 @@
+#!/bin/bash
+# Tests for check_overlay_handcraft.sh
+# Focus: Check 6(2026-05-01)— overlay body 重新引入 stripped-padding boolean variant 攔阻
+set -u
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+HOOK="$SCRIPT_DIR/../lib/_overlay_handcraft.sh"
+[ -x "$HOOK" ] || { echo "FATAL: hook not executable"; exit 1; }
+PASS=0; FAIL=0
+
+setup() { TMP=$(mktemp -d); mkdir -p "$TMP/.claude/hooks"; echo 'log_hook_fire(){ :; }' > "$TMP/.claude/hooks/_log-fire.sh"; }
+teardown() { rm -rf "$TMP"; }
+
+run() {
+  local fp="$1"
+  STDOUT=$(echo "{\"tool_input\":{\"file_path\":\"$fp\"}}" | bash "$HOOK" 2>&1)
+  EXIT=$?
+}
+
+# Test 1: out-of-scope file(non-overlay path)→ silent
+echo "Test 1: out-of-scope path → silent for Check 6"
+setup
+mkdir -p "$TMP/packages/design-system/src/components/Button"
+cat > "$TMP/packages/design-system/src/components/Button/button.tsx" <<'EOF'
+interface ButtonProps { flush?: boolean }
+EOF
+run "$TMP/packages/design-system/src/components/Button/button.tsx"
+echo "$STDOUT" | grep -q "stripped-padding boolean variant" && { echo "  FAIL: false positive on Button"; FAIL=$((FAIL+1)); } || { echo "  PASS"; PASS=$((PASS+1)); }
+teardown
+
+# Test 2: DialogBody re-introducing flush?: boolean → flagged
+echo "Test 2: DialogBody flush?: boolean → flagged"
+setup
+mkdir -p "$TMP/packages/design-system/src/components/Dialog"
+cat > "$TMP/packages/design-system/src/components/Dialog/dialog.tsx" <<'EOF'
+interface DialogBodyProps {
+  flush?: boolean
+}
+EOF
+run "$TMP/packages/design-system/src/components/Dialog/dialog.tsx"
+echo "$STDOUT" | grep -q "stripped-padding boolean variant" && { echo "  PASS"; PASS=$((PASS+1)); } || { echo "  FAIL: $STDOUT"; FAIL=$((FAIL+1)); }
+teardown
+
+# Test 3: SheetBody flush = false destructure default → flagged
+echo "Test 3: SheetBody flush = false destructure → flagged"
+setup
+mkdir -p "$TMP/packages/design-system/src/components/Sheet"
+cat > "$TMP/packages/design-system/src/components/Sheet/sheet.tsx" <<'EOF'
+const SheetBody = ({ flush = false }) => null
+interface X { flush?: boolean }
+EOF
+run "$TMP/packages/design-system/src/components/Sheet/sheet.tsx"
+echo "$STDOUT" | grep -q "stripped-padding boolean variant" && { echo "  PASS"; PASS=$((PASS+1)); } || { echo "  FAIL: $STDOUT"; FAIL=$((FAIL+1)); }
+teardown
+
+# Test 4: PopoverBody naked?: boolean(equivalent rename)→ flagged
+echo "Test 4: PopoverBody naked?: boolean → flagged"
+setup
+mkdir -p "$TMP/packages/design-system/src/components/Popover"
+cat > "$TMP/packages/design-system/src/components/Popover/popover.tsx" <<'EOF'
+interface PopoverBodyProps { naked?: boolean }
+EOF
+run "$TMP/packages/design-system/src/components/Popover/popover.tsx"
+echo "$STDOUT" | grep -q "stripped-padding boolean variant" && { echo "  PASS"; PASS=$((PASS+1)); } || { echo "  FAIL: $STDOUT"; FAIL=$((FAIL+1)); }
+teardown
+
+# Test 5: bare?: boolean(equivalent rename)→ flagged
+echo "Test 5: DialogBody bare?: boolean → flagged"
+setup
+mkdir -p "$TMP/packages/design-system/src/components/Dialog"
+cat > "$TMP/packages/design-system/src/components/Dialog/dialog.tsx" <<'EOF'
+interface Y { bare?: boolean }
+EOF
+run "$TMP/packages/design-system/src/components/Dialog/dialog.tsx"
+echo "$STDOUT" | grep -q "stripped-padding boolean variant" && { echo "  PASS"; PASS=$((PASS+1)); } || { echo "  FAIL: $STDOUT"; FAIL=$((FAIL+1)); }
+teardown
+
+# Test 6: noPadding?: boolean(equivalent rename)→ flagged
+echo "Test 6: SheetBody noPadding?: boolean → flagged"
+setup
+mkdir -p "$TMP/packages/design-system/src/components/Sheet"
+cat > "$TMP/packages/design-system/src/components/Sheet/sheet.tsx" <<'EOF'
+interface Z { noPadding?: boolean }
+EOF
+run "$TMP/packages/design-system/src/components/Sheet/sheet.tsx"
+echo "$STDOUT" | grep -q "stripped-padding boolean variant" && { echo "  PASS"; PASS=$((PASS+1)); } || { echo "  FAIL: $STDOUT"; FAIL=$((FAIL+1)); }
+teardown
+
+# Test 7: stories.tsx 內出現 flush?: boolean → silent(stories scope skip)
+echo "Test 7: stories.tsx flush?: boolean → skipped(out-of-scope)"
+setup
+mkdir -p "$TMP/packages/design-system/src/components/Dialog"
+cat > "$TMP/packages/design-system/src/components/Dialog/dialog.stories.tsx" <<'EOF'
+interface ExampleProps { flush?: boolean }
+EOF
+run "$TMP/packages/design-system/src/components/Dialog/dialog.stories.tsx"
+echo "$STDOUT" | grep -q "stripped-padding boolean variant" && { echo "  FAIL: false positive on stories"; FAIL=$((FAIL+1)); } || { echo "  PASS"; PASS=$((PASS+1)); }
+teardown
+
+# Test 8: allowlist comment escape hatch → silent
+echo "Test 8: allowlist escape hatch → silent"
+setup
+mkdir -p "$TMP/packages/design-system/src/components/Dialog"
+cat > "$TMP/packages/design-system/src/components/Dialog/dialog.tsx" <<'EOF'
+// overlay-body-stripped-variant-allow: 已對照 Polaris/Material/X 三家,multi-row 已驗證 hold
+interface DialogBodyProps { flush?: boolean }
+EOF
+run "$TMP/packages/design-system/src/components/Dialog/dialog.tsx"
+echo "$STDOUT" | grep -q "stripped-padding boolean variant" && { echo "  FAIL: allowlist not honored"; FAIL=$((FAIL+1)); } || { echo "  PASS"; PASS=$((PASS+1)); }
+teardown
+
+# Test 9: clean DialogBody(post-2026-05-01 canonical)→ silent
+echo "Test 9: clean DialogBody(no flush variant)→ silent"
+setup
+mkdir -p "$TMP/packages/design-system/src/components/Dialog"
+cat > "$TMP/packages/design-system/src/components/Dialog/dialog.tsx" <<'EOF'
+const DialogBody = React.forwardRef<HTMLDivElement, React.ComponentPropsWithoutRef<typeof ScrollArea>>(
+  ({ className, children, ...props }, ref) => null
+)
+EOF
+run "$TMP/packages/design-system/src/components/Dialog/dialog.tsx"
+echo "$STDOUT" | grep -q "stripped-padding boolean variant" && { echo "  FAIL: false positive on clean canonical"; FAIL=$((FAIL+1)); } || { echo "  PASS"; PASS=$((PASS+1)); }
+teardown
+
+echo ""
+echo "Summary: $PASS passed / $FAIL failed"
+[ "$FAIL" -eq 0 ] || exit 1

package/ds-canonical/hooks/tests/test_check_overlay_open_focus_escape_probe.sh

@@ -0,0 +1,191 @@
+#!/bin/bash
+# Tests for check_overlay_open_focus_escape_probe.sh(P0 BLOCKER,codex M31 finding 2026-05-27)
+#
+# Hook 規則(PreToolUse,Edit|Write|MultiEdit,只在 *.stories.tsx):
+#   - Reads content from tool_input.new_string // tool_input.content。
+#   - 只在 file_path 匹配 \.stories\.tsx$ 才作用,否則 exit 0 silent。
+#   - File-level escapes(任一命中 → silent):
+#       `@overlay-open-skip:` OR `@story-trait-allow:.*missing-opensnapshot`
+#   - 偵測 overlay primitive trigger usage:
+#       <(DS\.)?(Tooltip|Popover|Dialog|Sheet|DropdownMenu|HoverCard)Trigger\b
+#   - 若用了 overlay trigger 但無 open-state mechanism →
+#       BLOCK exit 2 + stderr「OVERLAY-OPEN-PROBE BLOCKER」。
+#     open-state mechanism = defaultOpen | open={true|isOpen|isVisible}
+#                            | play: async | play(...click
+#
+# M34 broad-vs-narrow symmetry:
+#   - near-miss(over-broad guard):<DialogTriggerWrapper> 不該 fire(\b 阻擋)
+#   - real violation(over-narrow guard):<DialogTrigger> 無 open → 必 fire
+
+set -u
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+HOOK="$SCRIPT_DIR/../check_overlay_open_focus_escape_probe.sh"
+
+if [ ! -x "$HOOK" ]; then
+  echo "FATAL: hook not executable: $HOOK"
+  exit 1
+fi
+
+PASS=0
+FAIL=0
+FAILED_TESTS=""
+
+TMP_DIR=$(mktemp -d)
+trap 'rm -rf "$TMP_DIR"' EXIT
+
+# Override CLAUDE_PROJECT_DIR so any _log-fire.sh state lands in TMP_DIR
+# (避免污染 repo .claude/logs/)
+export CLAUDE_PROJECT_DIR="$TMP_DIR"
+mkdir -p "$TMP_DIR/.claude/logs"
+
+# run-hook helper: pipes JSON into the hook via stdin.
+#   $1 tool_name, $2 file_path, $3 content (mapped to new_string)
+run_hook() {
+  local tool="$1"; local file_path="$2"; local content="$3"
+  local payload
+  payload=$(jq -n \
+    --arg t "$tool" --arg fp "$file_path" --arg c "$content" \
+    '{hook_event_name:"PreToolUse", tool_name:$t, tool_input:{file_path:$fp, new_string:$c}}')
+  STDOUT=$(mktemp); STDERR=$(mktemp)
+  set +e
+  printf '%s' "$payload" | bash "$HOOK" >"$STDOUT" 2>"$STDERR"
+  EXIT=$?
+  set -e
+  STDERR_TEXT=$(cat "$STDERR" 2>/dev/null)
+  rm -f "$STDOUT" "$STDERR"
+}
+
+# Variant that puts content under tool_input.content (Write convention) to
+# exercise the `.tool_input.new_string // .tool_input.content` fallback.
+run_hook_content_field() {
+  local tool="$1"; local file_path="$2"; local content="$3"
+  local payload
+  payload=$(jq -n \
+    --arg t "$tool" --arg fp "$file_path" --arg c "$content" \
+    '{hook_event_name:"PreToolUse", tool_name:$t, tool_input:{file_path:$fp, content:$c}}')
+  STDOUT=$(mktemp); STDERR=$(mktemp)
+  set +e
+  printf '%s' "$payload" | bash "$HOOK" >"$STDOUT" 2>"$STDERR"
+  EXIT=$?
+  set -e
+  STDERR_TEXT=$(cat "$STDERR" 2>/dev/null)
+  rm -f "$STDOUT" "$STDERR"
+}
+
+expect_pass_silent() {
+  local name="$1"
+  if [ "$EXIT" = "0" ] && [ -z "$STDERR_TEXT" ]; then
+    echo "  PASS  $name"; PASS=$((PASS+1))
+  else
+    echo "  FAIL  $name (expected silent exit=0, got exit=$EXIT, stderr=$([ -n "$STDERR_TEXT" ] && echo non-empty || echo empty))"
+    echo "  --- stderr ---"; echo "$STDERR_TEXT" | sed 's/^/    /'; echo "  --- end ---"
+    FAIL=$((FAIL+1)); FAILED_TESTS="${FAILED_TESTS}\n  - $name"
+  fi
+}
+
+expect_block() {
+  local name="$1"; local needle="$2"
+  if [ "$EXIT" = "2" ] && echo "$STDERR_TEXT" | grep -qF "$needle"; then
+    echo "  PASS  $name"; PASS=$((PASS+1))
+  else
+    echo "  FAIL  $name (expected BLOCK exit=2 + '$needle', got exit=$EXIT)"
+    echo "  --- stderr ---"; echo "$STDERR_TEXT" | sed 's/^/    /'; echo "  --- end ---"
+    FAIL=$((FAIL+1)); FAILED_TESTS="${FAILED_TESTS}\n  - $name"
+  fi
+}
+
+STORY="apps/template/src/components/Demo.stories.tsx"
+BLOCK_NEEDLE="OVERLAY-OPEN-PROBE BLOCKER"
+
+echo "=== check_overlay_open_focus_escape_probe tests ==="
+
+# ---------------------------------------------------------------------------
+# POSITIVE cases — SHOULD trigger BLOCKER (exit 2)
+# ---------------------------------------------------------------------------
+
+# P1. Dialog trigger, no open mechanism → BLOCK (over-narrow guard: real violation)
+run_hook "Edit" "$STORY" \
+  'export const Basic = () => (<Dialog><DialogTrigger>Open</DialogTrigger><DialogContent>Hi</DialogContent></Dialog>)'
+expect_block "P1. DialogTrigger no open → BLOCK" "$BLOCK_NEEDLE"
+
+# P2. Namespaced DS.PopoverTrigger, no open mechanism → BLOCK
+run_hook "Write" "$STORY" \
+  'export const Pop = () => (<DS.Popover><DS.PopoverTrigger>x</DS.PopoverTrigger><DS.PopoverContent>c</DS.PopoverContent></DS.Popover>)'
+expect_block "P2. DS.PopoverTrigger no open → BLOCK" "$BLOCK_NEEDLE"
+
+# P3. Tooltip trigger with open={false} (NOT a real open state) → BLOCK
+#     guards that regex anchors on true/isOpen/isVisible, not any open= prop
+run_hook "Edit" "$STORY" \
+  'export const T = () => (<Tooltip open={false}><TooltipTrigger>?</TooltipTrigger><TooltipContent>tip</TooltipContent></Tooltip>)'
+expect_block "P3. TooltipTrigger open={false} (not real open) → BLOCK" "$BLOCK_NEEDLE"
+
+# P4. content field (Write convention) instead of new_string → still BLOCK (field fallback)
+run_hook_content_field "Write" "$STORY" \
+  'export const S = () => (<Sheet><SheetTrigger>menu</SheetTrigger><SheetContent>side</SheetContent></Sheet>)'
+expect_block "P4. SheetTrigger via tool_input.content fallback → BLOCK" "$BLOCK_NEEDLE"
+
+# ---------------------------------------------------------------------------
+# NEGATIVE cases — should NOT trigger (silent exit 0)
+# ---------------------------------------------------------------------------
+
+# N1. DialogTrigger WITH defaultOpen → silent
+run_hook "Edit" "$STORY" \
+  'export const Open = () => (<Dialog defaultOpen><DialogTrigger>x</DialogTrigger><DialogContent>c</DialogContent></Dialog>)'
+expect_pass_silent "N1. DialogTrigger + defaultOpen → silent"
+
+# N2. PopoverTrigger WITH open={true} controlled → silent
+run_hook "Edit" "$STORY" \
+  'export const Ctl = () => (<Popover open={true}><PopoverTrigger>x</PopoverTrigger><PopoverContent>c</PopoverContent></Popover>)'
+expect_pass_silent "N2. PopoverTrigger + open={true} → silent"
+
+# N3. DropdownMenuTrigger WITH play() click interaction → silent
+run_hook "Edit" "$STORY" \
+  'export const Play = {render: () => (<DropdownMenu><DropdownMenuTrigger>m</DropdownMenuTrigger></DropdownMenu>), play: async ({canvasElement}) => { await userEvent.click(trigger); }}'
+expect_pass_silent "N3. DropdownMenuTrigger + play: async → silent"
+
+# N4. HoverCardTrigger no open BUT file-level @overlay-open-skip escape → silent
+run_hook "Edit" "$STORY" \
+  '// @overlay-open-skip: behavior-only test, no visual snapshot needed
+export const HC = () => (<HoverCard><HoverCardTrigger>@u</HoverCardTrigger><HoverCardContent>card</HoverCardContent></HoverCard>)'
+expect_pass_silent "N4. HoverCardTrigger + @overlay-open-skip escape → silent"
+
+# N5. HoverCardTrigger no open BUT @story-trait-allow missing-opensnapshot escape → silent
+run_hook "Edit" "$STORY" \
+  '// @story-trait-allow: missing-opensnapshot (HoverCard codex exception)
+export const HC2 = () => (<HoverCard><HoverCardTrigger>@u</HoverCardTrigger></HoverCard>)'
+expect_pass_silent "N5. HoverCardTrigger + @story-trait-allow missing-opensnapshot → silent"
+
+# N6. No overlay primitive at all (plain Button story) → silent
+run_hook "Edit" "$STORY" \
+  'export const Btn = () => (<Button variant="primary">Save</Button>)'
+expect_pass_silent "N6. plain Button story (no overlay) → silent"
+
+# N7. Non-stories file (component .tsx) using a trigger → silent (scope excluded)
+run_hook "Edit" "apps/template/src/components/Demo.tsx" \
+  'export const D = () => (<Dialog><DialogTrigger>x</DialogTrigger></Dialog>)'
+expect_pass_silent "N7. non-stories .tsx file → silent (scope excluded)"
+
+# N8. Non-Edit/Write/MultiEdit tool (Read) → silent
+run_hook "Read" "$STORY" \
+  'export const D = () => (<Dialog><DialogTrigger>x</DialogTrigger></Dialog>)'
+expect_pass_silent "N8. tool=Read → silent (event excluded)"
+
+# N9. OVER-BROAD GUARD (near-miss): <DialogTriggerWrapper> is a DIFFERENT identifier.
+#     \b after 'Trigger' must NOT match because 'W' is a word char → no false fire.
+run_hook "Edit" "$STORY" \
+  'export const W = () => (<DialogTriggerWrapper>custom non-DS wrapper, not a real overlay trigger</DialogTriggerWrapper>)'
+expect_pass_silent "N9. near-miss <DialogTriggerWrapper> → silent (over-broad guard)"
+
+# N10. Empty content payload → silent (hook guards [ -z CONTENT ])
+run_hook "Edit" "$STORY" ''
+expect_pass_silent "N10. empty content → silent"
+
+echo ""
+echo "=== Summary ==="
+echo "Passed: $PASS / $((PASS + FAIL))"
+if [ "$FAIL" -gt 0 ]; then
+  printf "Failed:%b\n" "$FAILED_TESTS"
+  exit 1
+fi
+exit 0

package/ds-canonical/hooks/tests/test_check_pattern_invariants.sh

@@ -0,0 +1,148 @@
+#!/bin/bash
+# Tests for check_pattern_invariants.sh — merged 4 sub-rules(C.1-C.4)
+
+set -u
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+HOOK="$SCRIPT_DIR/../check_pattern_invariants.sh"
+[ -x "$HOOK" ] || { echo "FATAL: hook not executable"; exit 1; }
+
+PASS=0; FAIL=0; FAILED_TESTS=""
+
+run_hook() {
+  local file_path="$1" content="$2"
+  STDOUT=$(mktemp); STDERR=$(mktemp)
+  set +e
+  jq -n --arg fp "$file_path" --arg c "$content" \
+    '{tool_name:"Write", tool_input:{file_path:$fp, content:$c}}' \
+    | bash "$HOOK" >"$STDOUT" 2>"$STDERR"
+  EXIT=$?
+  set -e
+  STDERR_TEXT=$(cat "$STDERR"); rm -f "$STDOUT" "$STDERR"
+}
+
+expect_exit() {
+  local name="$1" expect="$2" needle="${3:-}"
+  if [ "$EXIT" = "$expect" ] && { [ -z "$needle" ] || echo "$STDERR_TEXT" | grep -qF "$needle"; }; then
+    echo "  PASS  $name"; PASS=$((PASS+1))
+  else
+    echo "  FAIL  $name (expected exit $expect${needle:+ + needle '$needle'}, got exit $EXIT)"
+    echo "  --- stderr ---"; echo "$STDERR_TEXT" | sed 's/^/    /'; echo "  --- end ---"
+    FAIL=$((FAIL+1)); FAILED_TESTS="${FAILED_TESTS}\n  - $name"
+  fi
+}
+
+echo "=== C.1 overlay panel scroll chain ==="
+
+# 1. SurfaceBody + wrapper without flex flex-col h-full → WARN stderr (exit 0)
+run_hook "/r/src/app/foo.tsx" '
+function Panel() {
+  return (
+    <div className="w-[640px]">
+      <SurfaceHeader>x</SurfaceHeader>
+      <SurfaceBody>y</SurfaceBody>
+    </div>
+  )
+}
+'
+expect_exit "C.1.1 SurfaceBody w/o flex chain → WARN stderr" 0 "overlay scroll chain"
+
+# 2. SurfaceBody + correct flex chain → silent
+run_hook "/r/src/app/foo.tsx" '
+function Panel() {
+  return (
+    <div className="w-[640px] flex flex-col h-full min-h-0">
+      <SurfaceBody>y</SurfaceBody>
+    </div>
+  )
+}
+'
+expect_exit "C.1.2 SurfaceBody w/ flex chain → silent" 0 ""
+
+# 3. allowlist → silent
+run_hook "/r/src/app/foo.tsx" '
+// @scroll-chain-allow: fixed widget no scroll
+function Panel() { return <div className="w-[640px]"><SurfaceBody/></div> }
+'
+expect_exit "C.1.3 allowlist → silent" 0 ""
+
+echo ""
+echo "=== C.2 inline-action canonical gap ==="
+
+# 4. components/ ItemInlineAction + gap-3 → WARN stderr
+run_hook "/r/packages/design-system/src/components/Foo/foo.tsx" '
+<div className="flex gap-3"><ItemInlineAction /></div>
+'
+expect_exit "C.2.1 gap-3 inline-action → WARN" 0 "inline-action gap"
+
+# 5. components/ ItemInlineAction + gap-2 → silent
+run_hook "/r/packages/design-system/src/components/Foo/foo.tsx" '
+<div className="flex gap-2"><ItemInlineAction /></div>
+'
+expect_exit "C.2.2 gap-2 inline-action → silent" 0 ""
+
+# 6. SSOT host item-anatomy → skip
+run_hook "/r/packages/design-system/src/patterns/element-anatomy/item-anatomy.tsx" '
+<div className="flex gap-3"><ItemInlineAction /></div>
+'
+expect_exit "C.2.3 SSOT host skip → silent" 0 ""
+
+echo ""
+echo "=== C.3 primitive wrapper padding (BLOCK) ==="
+
+# 7. <div p-2><DateGrid /></div> → BLOCK
+run_hook "/r/src/app/foo.tsx" '
+function Foo() {
+  return <div className="p-2"><DateGrid /></div>
+}
+'
+expect_exit "C.3.1 p-2 wrapping DateGrid → BLOCK" 2 "primitive wrapper padding"
+
+# 8. <DateGrid /> direct → silent
+run_hook "/r/src/app/foo.tsx" '
+function Foo() { return <DateGrid /> }
+'
+expect_exit "C.3.2 DateGrid direct → silent" 0 ""
+
+# 9. file-level allowlist (first 5 lines) → silent
+run_hook "/r/src/app/foo.tsx" '// @primitive-padding-allow: legacy migration
+
+function Foo() {
+  return <div className="p-2"><DateGrid /></div>
+}
+'
+expect_exit "C.3.3 file allowlist → silent" 0 ""
+
+echo ""
+echo "=== C.4 row slot handcraft (BLOCK) ==="
+
+# 10. components/ self-coded h-[1lh] shrink-0 flex items-center → BLOCK
+run_hook "/r/packages/design-system/src/components/Bad/bad.tsx" '
+function F() { return <span className="h-[1lh] shrink-0 flex items-center"><Icon/></span> }
+'
+expect_exit "C.4.1 row slot handcraft → BLOCK" 2 "row slot handcraft"
+
+# 11. SSOT host item-anatomy → skip
+run_hook "/r/packages/design-system/src/patterns/element-anatomy/item-anatomy.tsx" '
+const x = "h-[1lh] shrink-0 flex items-center"
+'
+expect_exit "C.4.2 item-anatomy SSOT skip → silent" 0 ""
+
+# 12. Using ItemPrefix → silent
+run_hook "/r/packages/design-system/src/components/Good/good.tsx" '
+import { ItemPrefix } from "@/design-system/patterns/element-anatomy/item-anatomy"
+function F() { return <ItemPrefix><Icon/></ItemPrefix> }
+'
+expect_exit "C.4.3 ItemPrefix consume → silent" 0 ""
+
+# 13. allowlist → silent
+run_hook "/r/packages/design-system/src/components/Edge/edge.tsx" '
+// @row-slot-handcraft-allow: forced width
+function F() { return <span className="h-[1lh] shrink-0 flex items-center"><Icon/></span> }
+'
+expect_exit "C.4.4 row slot allowlist → silent" 0 ""
+
+echo ""
+echo "═══ Results: $PASS PASS, $FAIL FAIL ═══"
+[ "$FAIL" -gt 0 ] && { printf "Failed:%b\n" "$FAILED_TESTS"; exit 1; }
+exit 0

package/ds-canonical/hooks/tests/test_check_peoplepicker_ssot_drift.sh

@@ -0,0 +1,108 @@
+#!/bin/bash
+# Tests for check_peoplepicker_ssot_drift.sh(2026-05-15)
+#
+# Hook 規則:PreToolUse Edit/Write on PeoplePicker/people-picker.tsx /
+# PeoplePicker/person-display.tsx / Combobox/combobox.tsx — 若 NEW_CONTENT
+# 含 SSOT-bearing keyword(tagRenderer/overflowShape/...)但無 cite
+# `people-picker.spec.md §A-§F` → 經由 stdout 輸出 hookSpecificOutput JSON warn。
+# 非目標檔案 / 無 SSOT keyword / 含 spec cite → silent。
+
+set -u
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+HOOK="$SCRIPT_DIR/../check_peoplepicker_ssot_drift.sh"
+
+if [ ! -x "$HOOK" ]; then
+  echo "FATAL: hook not executable: $HOOK"
+  exit 1
+fi
+
+PASS=0
+FAIL=0
+FAILED_TESTS=""
+
+run_hook() {
+  local file_path="$1"
+  local new_string="$2"
+  local payload
+  payload=$(jq -n --arg fp "$file_path" --arg ns "$new_string" \
+    '{tool_name: "Edit", tool_input: {file_path: $fp, new_string: $ns}}')
+  STDOUT=$(mktemp); STDERR=$(mktemp)
+  set +e
+  printf '%s' "$payload" | bash "$HOOK" >"$STDOUT" 2>"$STDERR"
+  EXIT=$?
+  set -e
+  STDOUT_TEXT=$(cat "$STDOUT")
+  STDERR_TEXT=$(cat "$STDERR")
+  rm -f "$STDOUT" "$STDERR"
+}
+
+expect_pass_silent() {
+  local name="$1"
+  if [ "$EXIT" = "0" ] && [ -z "$STDOUT_TEXT" ] && [ -z "$STDERR_TEXT" ]; then
+    echo "  PASS  $name"; PASS=$((PASS+1))
+  else
+    echo "  FAIL  $name (expected silent, exit=$EXIT, stdout-empty=$([ -z "$STDOUT_TEXT" ] && echo yes), stderr-empty=$([ -z "$STDERR_TEXT" ] && echo yes))"
+    echo "  --- stdout ---"; echo "$STDOUT_TEXT" | sed 's/^/    /'; echo "  --- end ---"
+    echo "  --- stderr ---"; echo "$STDERR_TEXT" | sed 's/^/    /'; echo "  --- end ---"
+    FAIL=$((FAIL+1)); FAILED_TESTS="${FAILED_TESTS}\n  - $name"
+  fi
+}
+
+expect_warn() {
+  local name="$1"; local needle="$2"
+  if [ "$EXIT" = "0" ] && echo "$STDOUT_TEXT" | grep -qF "$needle"; then
+    echo "  PASS  $name"; PASS=$((PASS+1))
+  else
+    echo "  FAIL  $name (expected warn '$needle', got exit $EXIT)"
+    echo "  --- stdout ---"; echo "$STDOUT_TEXT" | sed 's/^/    /'; echo "  --- end ---"
+    FAIL=$((FAIL+1)); FAILED_TESTS="${FAILED_TESTS}\n  - $name"
+  fi
+}
+
+echo "=== check_peoplepicker_ssot_drift tests ==="
+
+# 1. people-picker.tsx 動 tagRenderer 無 spec cite → warn
+run_hook "/repo/packages/design-system/src/components/PeoplePicker/people-picker.tsx" '
+const customTagRenderer = (item) => <PersonAvatarTag {...item} />;
+'
+expect_warn "1. tagRenderer change without spec cite → warn" "PeoplePicker SSOT-bearing edit"
+
+# 2. people-picker.tsx 動 overflowShape 無 cite → warn
+run_hook "/repo/packages/design-system/src/components/PeoplePicker/people-picker.tsx" '
+overflowShape="round"
+'
+expect_warn "2. overflowShape change → warn" "people-picker.spec.md"
+
+# 3. 含 spec cite §A → silent
+run_hook "/repo/packages/design-system/src/components/PeoplePicker/people-picker.tsx" '
+// per people-picker.spec.md §A 元件本質繼承表
+const tagRenderer = (item) => <PersonAvatarTag {...item} />;
+'
+expect_pass_silent "3. tagRenderer with §A cite → silent"
+
+# 4. 非 PeoplePicker 檔案 → silent skip
+run_hook "/repo/packages/design-system/src/components/Button/button.tsx" '
+const tagRenderer = (item) => <Tag {...item} />;
+'
+expect_pass_silent "4. non-target file → silent skip"
+
+# 5. combobox.tsx 動 selectedItemRenderer 無 cite → warn
+run_hook "/repo/packages/design-system/src/components/Combobox/combobox.tsx" '
+selectedItemRenderer={renderPerson}
+'
+expect_warn "5. combobox selectedItemRenderer change → warn" "PeoplePicker SSOT-bearing"
+
+# 6. person-display.tsx edit 不含 SSOT keyword → silent
+run_hook "/repo/packages/design-system/src/components/PeoplePicker/person-display.tsx" '
+const helperVar = "noop";
+'
+expect_pass_silent "6. person-display.tsx edit without SSOT keyword → silent"
+
+echo ""
+echo "=== Summary ==="
+echo "Passed: $PASS / $((PASS + FAIL))"
+if [ "$FAIL" -gt 0 ]; then
+  echo "Failed:$FAILED_TESTS"
+  exit 1
+fi

package/ds-canonical/hooks/tests/test_check_person_data_richness.sh

@@ -0,0 +1,58 @@
+#!/bin/bash
+# Tests for check_person_data_richness.sh
+set -u
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+HOOK="$SCRIPT_DIR/../lib/_person_data_richness.sh"
+[ -x "$HOOK" ] || { echo "FATAL: hook not executable"; exit 1; }
+PASS=0; FAIL=0
+
+setup() { TMP=$(mktemp -d); mkdir -p "$TMP/.claude/hooks"; echo 'log_hook_fire(){ :; }' > "$TMP/.claude/hooks/_log-fire.sh"; }
+teardown() { rm -rf "$TMP"; }
+
+run() {
+  STDOUT=$(echo "{\"tool_input\":{\"file_path\":\"$1\"}}" | bash "$HOOK" 2>&1)
+  EXIT=$?
+}
+
+# Test 1: non-tsx file → silent
+echo "Test 1: non-tsx → silent"
+setup
+echo "data" > "$TMP/foo.txt"
+run "$TMP/foo.txt"
+[ "$EXIT" = "0" ] && [ -z "$STDOUT" ] && { echo "  PASS"; PASS=$((PASS+1)); } || { echo "  FAIL"; FAIL=$((FAIL+1)); }
+teardown
+
+# Test 2: sparse PersonData → flag
+echo "Test 2: sparse {name,avatarUrl} → flagged"
+setup
+cat > "$TMP/test.tsx" <<EOF
+const p = { name: 'Alice', avatarUrl: 'http://x' }
+EOF
+run "$TMP/test.tsx"
+echo "$STDOUT" | grep -q "Sparse PersonData" && { echo "  PASS"; PASS=$((PASS+1)); } || { echo "  FAIL: $STDOUT"; FAIL=$((FAIL+1)); }
+teardown
+
+# Test 3: rich PersonData → silent
+echo "Test 3: rich PersonData → silent"
+setup
+cat > "$TMP/test.tsx" <<EOF
+const p: PersonData = { name: 'Alice', avatarUrl: 'http://x', description: 'Eng', status: 'online', statusMessage: 'OK', fields: [] }
+EOF
+run "$TMP/test.tsx"
+[ "$EXIT" = "0" ] && [ -z "$STDOUT" ] && { echo "  PASS"; PASS=$((PASS+1)); } || { echo "  FAIL: $STDOUT"; FAIL=$((FAIL+1)); }
+teardown
+
+# Test 4: NameCard / Avatar self files skipped(real DS path pattern)
+echo "Test 4: src/.../NameCard/name-card.stories.tsx → skip"
+setup
+mkdir -p "$TMP/packages/design-system/src/components/NameCard"
+cat > "$TMP/packages/design-system/src/components/NameCard/name-card.stories.tsx" <<EOF
+const p = { name: 'A', avatarUrl: 'x' }
+EOF
+run "$TMP/packages/design-system/src/components/NameCard/name-card.stories.tsx"
+[ "$EXIT" = "0" ] && [ -z "$STDOUT" ] && { echo "  PASS"; PASS=$((PASS+1)); } || { echo "  FAIL: $STDOUT"; FAIL=$((FAIL+1)); }
+teardown
+
+echo ""
+echo "── Summary: $PASS / $((PASS+FAIL)) passed ──"
+[ "$FAIL" -eq 0 ] && echo "✅ All passed" || exit 1