shirokuma-docs 0.1.0-alpha.1

package/plugin/shirokuma-skills-en/skills/reviewing-on-issue/roles/testing.md

@@ -0,0 +1,180 @@
+# Test Review Role
+
+## Responsibilities
+
+Test code quality review covering:
+- TDD compliance verification
+- Test coverage analysis
+- Test patterns and anti-patterns
+- Mock usage review
+- E2E test quality
+
+## Required Knowledge
+
+Load these files for context:
+
+**Skill files (explicit load):**
+- `../criteria/testing.md` - Testing criteria
+- `../patterns/server-actions.md` - Mock patterns
+- `../patterns/e2e-testing.md` - E2E patterns
+
+**Rules (auto-loaded from `.claude/rules/`):**
+- `tech-stack.md` - Version information
+- `known-issues.md` - Testing issues
+
+## Review Checklist
+
+### Test Documentation (shirokuma-docs lint-tests)
+
+Run test documentation lint before reviewing:
+
+```bash
+# Run test documentation lint
+shirokuma-docs lint-tests -p . -c shirokuma-docs.config.yaml -f summary
+
+# For detailed report
+shirokuma-docs lint-tests -p . -c shirokuma-docs.config.yaml -f terminal
+```
+
+Check for:
+- [ ] @testdoc comments present (Japanese test descriptions)
+- [ ] @purpose tags explaining test intent
+- [ ] @expected tags with expected outcomes
+- [ ] No duplicate @testdoc descriptions
+- [ ] Coverage threshold met (configurable)
+
+JSDoc format for test documentation:
+```typescript
+/**
+ * @testdoc 日本語でのテスト説明
+ * @purpose テストの目的
+ * @precondition 前提条件
+ * @expected 期待する結果
+ */
+it("english test name", () => { ... });
+```
+
+### Implementation-Test Coverage (shirokuma-docs lint-coverage)
+
+Run coverage check:
+```bash
+# Check source-test file mapping
+shirokuma-docs lint-coverage -p . -c shirokuma-docs.config.yaml
+
+# Summary only
+shirokuma-docs lint-coverage -p . -c shirokuma-docs.config.yaml -f summary
+
+# Strict mode (fail on missing tests)
+shirokuma-docs lint-coverage -p . -c shirokuma-docs.config.yaml -s
+```
+
+Check for:
+- [ ] All source files have corresponding test files
+- [ ] @skip-test annotations have valid reasons
+- [ ] No orphan test files (tests without implementations)
+
+@skip-test annotation for files that don't need tests:
+```typescript
+/**
+ * @skip-test 自動生成コードのためテスト不要
+ */
+export const generatedSchema = { ... }
+```
+
+Valid skip reasons:
+- 自動生成 (auto-generated code)
+- shadcn/ui (library components)
+- E2Eでカバー (covered by E2E tests)
+- 外部ライブラリ (external library wrapper)
+- 単純なre-export (simple re-export)
+
+### TDD Compliance
+- [ ] Tests committed before implementation
+- [ ] Red-Green-Refactor cycle followed
+- [ ] Tests describe behavior, not implementation
+- [ ] Git history shows test-first approach
+
+### Test Coverage
+- [ ] Server Actions: 80%+
+- [ ] Components: 70%+
+- [ ] Utilities: 90%+
+- [ ] Critical paths covered
+
+### Unit Test Quality
+- [ ] Success cases tested
+- [ ] Error cases tested
+- [ ] Edge cases covered
+- [ ] Proper beforeEach/afterEach
+- [ ] Mock cleanup between tests
+
+### E2E Test Quality
+- [ ] Semantic locators used
+- [ ] i18n variants handled
+- [ ] Loading states tested
+- [ ] Error states tested
+- [ ] No arbitrary timeouts
+
+### Mock Patterns
+- [ ] Queue-based mocks for DB
+- [ ] Auth properly mocked
+- [ ] External deps only mocked
+- [ ] No over-mocking
+
+## Test Anti-patterns to Detect
+
+Check for the following test violations during review:
+
+### TDD Violation Anti-patterns
+- [ ] Tests written after implementation (not test-first)
+- [ ] Tests commented out or skipped
+- [ ] Tests depending on implementation details (should test behavior)
+- [ ] Broken tests left unfixed
+
+### Flaky Test Anti-patterns
+- [ ] Using arbitrary timeouts (`waitForTimeout`)
+- [ ] Time-dependent tests (`new Date()` comparisons)
+- [ ] Order-dependent tests
+- [ ] External service dependencies not mocked
+
+### Mock Anti-patterns
+- [ ] Over-mocking (not testing real code)
+- [ ] Mocks not reset between tests
+- [ ] Missing `jest.clearAllMocks()` in `beforeEach`
+- [ ] Mocks diverging from actual API
+
+### E2E Anti-patterns
+- [ ] Forcing clicks with `force: true`
+- [ ] Direct URL navigation skipping user journey
+- [ ] Using `test.skip` when seed data should exist
+- [ ] Overusing CSS selectors instead of semantic locators
+- [ ] Using `textbox` role for `type="password"` inputs
+
+### Coverage Anti-patterns
+- [ ] Tests written only for coverage metrics
+- [ ] Missing error case tests
+- [ ] Missing boundary value tests
+- [ ] Insufficient auth/authorization tests
+
+## Verify TDD with Git
+
+```bash
+# Check if tests were committed before implementation
+git log --oneline --name-only -- "*.test.ts" "*.spec.ts"
+```
+
+## Report Format
+
+Use template from `templates/report.md`:
+
+1. **TDD Assessment**: Compliance level
+2. **Coverage Report**: Current vs required
+3. **Test Quality Issues**: Anti-patterns found
+4. **Missing Tests**: Gaps identified
+5. **Recommendations**: Improvements
+
+## Trigger Keywords
+
+- "test review"
+- "テストレビュー"
+- "review tests"
+- "test quality"

package/plugin/shirokuma-skills-en/skills/reviewing-on-issue/templates/report.md

@@ -0,0 +1,121 @@
+# Review Report Template
+
+## Header
+
+```markdown
+# [Review Type] Review Report
+
+**Date**: YYYY-MM-DD HH:MM
+**Reviewer**: Claude (reviewer agent)
+**Target**: [file/directory path]
+**Role**: [code|security|testing|nextjs|plan]
+```
+
+## Sections
+
+### 1. Summary
+
+Brief overview (2-3 sentences):
+- What was reviewed
+- Overall assessment (Good/Needs Work/Critical)
+- Key finding
+
+### 2. Critical Issues
+
+**Must fix before merge** (See [Severity Definitions](#severity-definitions))
+
+| Issue | Location | Severity | Description |
+|-------|----------|----------|-------------|
+| Issue name | file:line | Critical/High | What's wrong and why |
+
+### 3. Improvements
+
+**Recommended changes** (See [Severity Definitions](#severity-definitions))
+
+| Issue | Location | Severity | Description |
+|-------|----------|----------|-------------|
+| Issue name | file:line | Medium/Low | What to improve |
+
+### 4. Best Practices
+
+**Positive findings**
+
+- What's done well
+- Patterns followed correctly
+- Good examples to replicate
+
+### 5. Recommendations
+
+**Prioritized action items**
+
+1. **Immediate**: Critical fixes
+2. **Before merge**: Important improvements
+3. **Future**: Technical debt items
+
+---
+
+## Example Report
+
+```markdown
+# Security Review Report
+
+**Date**: 2025-11-28 14:30
+**Reviewer**: Claude (reviewer agent)
+**Target**: apps/admin/lib/actions/
+**Role**: security
+
+## Summary
+
+Reviewed 5 Server Action files. Overall security posture is **Good** with one critical issue: missing authorization check in `deleteComment`. All other actions follow secure patterns.
+
+## Critical Issues
+
+| Issue | Location | Severity | Description |
+|-------|----------|----------|-------------|
+| Missing auth check | comments.ts:45 | Critical | `deleteComment` has no session verification |
+| ILIKE injection | posts.ts:112 | High | Search query not escaped |
+
+## Improvements
+
+| Issue | Location | Severity | Description |
+|-------|----------|----------|-------------|
+| Rate limiting | auth.ts:23 | Medium | Consider stricter limits for password reset |
+| Error logging | categories.ts:78 | Low | Add structured logging context |
+
+## Best Practices
+
+- All actions use Zod validation
+- Proper ownership checks in post mutations
+- Structured error responses
+- revalidatePath used consistently
+
+## Recommendations
+
+1. **Immediate**: Add auth check to `deleteComment`
+2. **Before merge**: Escape ILIKE wildcards in search
+3. **Future**: Implement rate limiting for sensitive actions
+```
+
+## Severity Definitions
+
+| Level | Impact | Action |
+|-------|--------|--------|
+| **Critical** | Security vulnerability, data loss risk | Block merge |
+| **High** | Significant bug, security gap | Fix before merge |
+| **Medium** | Performance issue, maintainability | Fix recommended |
+| **Low** | Style issue, minor improvement | Track for later |
+
+## Output Location
+
+**Create Discussion in Reports category:**
+
+```bash
+shirokuma-docs discussions create \
+  --category Reports \
+  --title "[Review] security: apps/admin/lib/actions/" \
+  --body "$(cat report.md)"
+```
+
+Report the Discussion URL to the user.
+
+> See `rules/output-destinations.md` for output destination policy.

package/plugin/shirokuma-skills-en/skills/showing-github/SKILL.md

@@ -0,0 +1,276 @@
+---
+name: showing-github
+description: Display GitHub project data (dashboard, items, issues, handovers, specs). Use when "show dashboard", "show items", "show handovers", "show issues", "show specs".
+allowed-tools: Bash, Read, Glob
+---
+
+# Showing GitHub
+
+Display GitHub project data. Consolidates dashboard, items, issues, handovers, and specs into one skill.
+
+> **Reference**: See `reference/github-operations.md` for CLI commands, Status Workflow, and error handling.
+
+## /show-dashboard
+
+Full project dashboard aggregating GitHub data.
+
+```
+/show-dashboard              # Full dashboard
+/show-dashboard --quick      # Quick summary only
+```
+
+### Workflow
+
+1. Get repository info: `gh repo view --json nameWithOwner -q '.nameWithOwner'`
+2. Run in parallel:
+   - `shirokuma-docs issues list` (project items by status)
+   - `gh issue list --state open --json number | jq length` (open issues count)
+   - `gh pr list --state open --json number | jq length` (open PRs count)
+   - `gh api repos/{owner}/{repo}/commits?per_page=5` (recent commits)
+   - `shirokuma-docs discussions list --category Handovers --limit 3` (recent handovers)
+
+### Display Format
+
+```markdown
+# Project Dashboard
+
+**Repository:** {owner}/{repo}
+**Generated:** {timestamp}
+
+## Project Items
+| Status | Count | Bar |
+|--------|-------|-----|
+| In Progress | 1 | ██ |
+| Backlog | 2 | ████ |
+
+**Total:** {total} | **Completion:** {done/total * 100}%
+
+## Activity
+| Metric | Count |
+|--------|-------|
+| Open Issues | {count} |
+| Open PRs | {count} |
+| Commits (7d) | {count} |
+
+## Recent Handovers
+| Date | Title |
+|------|-------|
+| {date} | {title} |
+```
+
+### Quick Mode (--quick)
+
+```markdown
+## Quick Status
+**Items:** 6 Done / 1 In Progress / 2 Backlog
+**Issues:** 3 open | **PRs:** 1 open
+**Last commit:** {message} ({time ago})
+```
+
+---
+
+## /show-items [filter]
+
+GitHub Project items with Status filter.
+
+```
+/show-items              # Active items (excludes Done/Released)
+/show-items all          # All items including Done
+/show-items ready        # Items with "Ready" status
+/show-items in-progress  # Items with "In Progress" status
+```
+
+### Workflow
+
+```bash
+# Default (open issues)
+shirokuma-docs issues list
+
+# With filter
+shirokuma-docs issues list --all
+shirokuma-docs issues list --status Ready
+shirokuma-docs issues list --status "In Progress" --status Ready
+```
+
+### Display Format (Grouped View)
+
+```markdown
+## Project Items
+
+**In Progress (1):**
+- #9 Task title (XL, Medium)
+
+**Backlog (2):**
+- #10 Feature A (M, High)
+
+**Icebox (1):**
+- #8 Future enhancement (L, Low)
+
+---
+Total: 4 active items
+```
+
+### Filtered View
+
+```markdown
+## Ready Items (2)
+| # | Title | Priority | Type | Size |
+|---|-------|----------|------|------|
+| #10 | Feature A | High | Feature | M |
+```
+
+---
+
+## /show-issues [--label X] [--assignee X]
+
+GitHub Issues list with filtering.
+
+```
+/show-issues                 # All open issues
+/show-issues --all           # Include closed
+/show-issues --label bug     # Filter by label
+/show-issues --assignee @me  # My issues
+```
+
+### Workflow
+
+```bash
+gh issue list --state open \
+  --json number,title,state,labels,assignees,createdAt,updatedAt \
+  --limit 20
+```
+
+### Display Format
+
+```markdown
+## Issues
+
+**Filter:** {description} | **Total:** {count}
+
+| # | Title | Labels | Assignee | Updated |
+|---|-------|--------|----------|---------|
+| #123 | Fix login bug | `bug` | @user | 2d ago |
+```
+
+---
+
+## /show-handovers [count]
+
+Past session handover information.
+
+```
+/show-handovers       # Last 5 handovers
+/show-handovers 10    # Last 10
+/show-handovers all   # All handovers
+```
+
+### Data Sources (try in order)
+
+1. GitHub Discussions (Handovers category)
+2. Local files (`.claude/sessions/*.md`)
+
+### Workflow
+
+```bash
+# From Discussions
+shirokuma-docs discussions list --category Handovers --limit {count}
+
+# Get specific handover
+shirokuma-docs discussions get {number}
+
+# From local files (fallback)
+ls -t .claude/sessions/*-handover.md 2>/dev/null | head -{count}
+```
+
+### Display Format (List View)
+
+```markdown
+## Recent Handovers
+
+**2025-01-25** - Blog CMS article management
+   Summary: Implemented post CRUD with draft/publish workflow
+   Next: Add category filtering
+   Issues: #10, #12
+
+**2025-01-24** - Session management skill
+   Summary: Created session skills
+   Next: Test with Discussions
+```
+
+### Parsing Handover Content
+
+| Section | Pattern |
+|---------|---------|
+| Summary | Text after `## Summary` |
+| Issues | Lines matching `#\d+` after `## Related` |
+| Next Steps | Checkbox items after `## Next Steps` |
+
+---
+
+## /show-specs [--recent] ["keyword"]
+
+Spec Discussions from Ideas category.
+
+```
+/show-specs              # All specs
+/show-specs --recent     # Last 5
+/show-specs "keyword"    # Search
+```
+
+### Workflow
+
+```bash
+gh api graphql -f query='{
+  repository(owner: "{owner}", name: "{repo}") {
+    discussions(first: 20, categoryId: "{ideas_id}", orderBy: {field: CREATED_AT, direction: DESC}) {
+      nodes { number title createdAt author { login } comments { totalCount } url }
+    }
+  }
+}'
+```
+
+### Display Format
+
+```markdown
+## Specifications
+
+| # | Title | Author | Comments | Created |
+|---|-------|--------|----------|---------|
+| #10 | [Spec] Auth Flow | @user | 5 | 1w ago |
+```
+
+### Status Indicators
+
+```
+Draft | Review | Approved | Rejected | Implementing
+```
+
+---
+
+## Error Handling
+
+| Error | Action |
+|-------|--------|
+| No project found | Show only Issues/PRs/Commits |
+| No items match filter | "No items with status '{filter}'. Try `/show-items`." |
+| No Discussions/category | Check local files or skip gracefully |
+| No handovers found | "No handover history. Start with `/ending-session`." |
+| No specs found | "No specs found. Create one with `/create-spec`." |
+| gh not authenticated | Prompt: `gh auth login` |
+
+## Reference Documents
+
+### Skill Documents
+
+| Document | Content | When to Read |
+|----------|---------|--------------|
+| [reference/github-operations.md](reference/github-operations.md) | GitHub CLI commands and status workflow | All subcommands |
+
+## Notes
+
+- All data fetched on-demand (not cached)
+- Items sorted by Priority within each status
+- Combine Discussion and local handovers if both exist
+- Specs stored in "Ideas" category by convention
+- When the request is ambiguous, use AskUserQuestion to confirm which subcommand (dashboard/items/issues/handovers/specs)
+- This is a display-only task; TodoWrite is not needed