Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
564d852
📝 docs: add git commit guidelines with gitmoji and confidence index
baptou12 Nov 6, 2025
2180dc8
📝 docs: add task management structure in .claude/tasks
baptou12 Nov 6, 2025
53a6669
🐛 fix: remove debug console.log and duplicate scroll effect
baptou12 Nov 6, 2025
143baef
✨ feat: add comprehensive error handling to webhook
baptou12 Nov 6, 2025
5ad40bd
♿️ a11y: add aria-labels to chat input elements
baptou12 Nov 6, 2025
18dcfa3
✅ test: skip failing test pending investigation
baptou12 Nov 6, 2025
cc8e834
📝 docs: mark critical fixes as completed
baptou12 Nov 6, 2025
a8f9982
✅ test: fix failing "shows AI response after loading" test
baptou12 Nov 6, 2025
1ce65c0
📝 docs: mark Task 0 as completed in testing tasks
baptou12 Nov 6, 2025
cb349a2
♻️ refactor: replace fetch stub with MSW in tests
baptou12 Nov 6, 2025
0a30cc9
📝 docs: add AskUserQuestion usage guidelines
baptou12 Nov 6, 2025
fbdccbf
✅ test: add comprehensive tests for useChatWebhook hook
baptou12 Nov 6, 2025
59c4659
📝 docs: add comprehensive testing guidelines
baptou12 Nov 6, 2025
6accf94
♻️ refactor: align useChatWebhook tests with guidelines
baptou12 Nov 6, 2025
90b46e6
📝 docs: add GIVEN-WHEN-THEN structure guideline for tests
baptou12 Nov 6, 2025
864defc
✅ test: add comprehensive tests for useTypingText hook
baptou12 Nov 6, 2025
5a5002e
✅ test: fix all useChatWebhook test timeouts and failures
baptou12 Nov 6, 2025
792589d
♻️ refactor: extract magic numbers to constants
baptou12 Nov 7, 2025
9d906c7
🔧 config: add bash permissions and ignore tsbuildinfo
baptou12 Nov 7, 2025
0992dff
🐛 fix: ensure unique message IDs with counter
baptou12 Nov 9, 2025
eec5f07
📦 build: add @testing-library/user-event dependency
baptou12 Nov 9, 2025
17364cf
✅ test: improve existing test quality and add edge cases
baptou12 Nov 9, 2025
db494a7
✅ test: add comprehensive tests for AssistantMessage component
baptou12 Nov 9, 2025
4ac649a
♻️ refactor: fix hydration warning and remove unused params
baptou12 Nov 9, 2025
321bc88
🎨 style: fix Biome formatting issues
baptou12 Nov 9, 2025
572ec64
🐛 fix: resolve TypeScript type error in SyntaxHighlighter
baptou12 Nov 9, 2025
0aae754
♻️ refactor: restructure TESTING.md for clearer conceptual hierarchy
baptou12 Nov 9, 2025
074c0a2
🔧 config: replace local settings with generalized permission patterns
baptou12 Nov 9, 2025
04c3504
🔧 config: add git commit to ask permissions
baptou12 Nov 9, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
471 changes: 471 additions & 0 deletions .claude/guidelines/TESTING.md

Large diffs are not rendered by default.

23 changes: 23 additions & 0 deletions .claude/settings.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
{
"permissions": {
"allow": [
"Bash(pnpm :*)",
"Bash(git :*)",
"Bash(gh :*)",
"Bash(mkdir:*)"
],
"deny": [
"Bash(git push --force:*)",
"Bash(git reset --hard:*)",
"Bash(git clean:*)",
"Bash(git branch -D:*)",
"Bash(gh repo delete:*)",
"Bash(gh pr merge:*)",
"Bash(gh release delete:*)",
"Bash(pnpm publish:*)"
],
"ask": [
"Bash(git commit)"
]
}
}
215 changes: 215 additions & 0 deletions .claude/tasks/01-documentation.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,215 @@
---
title: Documentation Tasks
category: documentation
priority: low
total_tasks: 7
estimated_time: 45min
status: pending
created: 2025-11-06
---

# 📚 Documentation Tasks

Tasks to align documentation with actual implementation.

---

## Task 1: Fix /src/api Architecture Documentation

**Status:** 🔲 Pending
**Priority:** 🟢 Low
**Time:** ~5 min
**File:** `CLAUDE.md`

### Problem
- Documentation mentions `/src/api` directory for "API integration and data fetching"
- This directory doesn't exist
- API calls are actually handled in `/src/hooks/useChatWebhook.ts`

### Action
Remove or update the `/src/api` reference in CLAUDE.md line 40.

**Option A:** Remove the line
**Option B:** Update to:
```markdown
- `/src/hooks`: Custom React hooks including API integration (useChatWebhook)
```

### Files to modify
- `CLAUDE.md` (line 40)

---

## Task 2: Fix biome.json JSX Runtime Configuration

**Status:** 🔲 Pending
**Priority:** 🟢 Low
**Time:** ~2 min
**File:** `biome.json`

### Problem
- `biome.json` line 16 sets `"jsxRuntime": "reactClassic"`
- Project uses React 19 with automatic JSX transformation
- Causes inconsistency with tsconfig `"jsx": "react-jsx"`

### Action
Update `biome.json` line 16:

```json
"jsxRuntime": "transparent"
```

Or remove the setting entirely to use default.

### Files to modify
- `biome.json` (line 16)

---

## Task 3: Update pnpm Version

**Status:** 🔲 Pending
**Priority:** 🟢 Low
**Time:** ~1 min
**File:** `CLAUDE.md`

### Problem
- Documentation states `pnpm v10.6.3`
- Actual version is `pnpm v10.14.0`

### Action
Update CLAUDE.md line 15:

```markdown
- **Package Manager**: pnpm v10.14.0
```

### Files to modify
- `CLAUDE.md` (line 15)

---

## Task 4: Document Missing Folders

**Status:** 🔲 Pending
**Priority:** 🟢 Low
**Time:** ~5 min
**File:** `CLAUDE.md`

### Problem
Actual project structure has folders not documented:
- `/src/domain` - Contains entities (Message, Roles enum)
- `/src/lib` - Utility functions (utils.ts with `cn()` helper)
- `/src/contexts` - Empty folder

### Action
Add to CLAUDE.md "Architecture Notes" section:

```markdown
- Project structure:
- `/src/components`: React components
- `/src/hooks`: Custom React hooks
- `/src/domain`: Domain entities and types
- `/src/lib`: Utility functions and helpers
- `/src/contexts`: React contexts (currently empty)
```

### Files to modify
- `CLAUDE.md` (lines 39-41)

---

## Task 5: Update Naming Convention Documentation

**Status:** 🔲 Pending
**Priority:** 🟢 Low
**Time:** ~3 min
**File:** `CLAUDE.md`

### Problem
- Documentation says: "Types/Interfaces: PascalCase with prefix (IUser, TConfig)"
- Actual codebase uses: `ChatContainerProps`, `WebhookResponse` (no I/T prefix)
- Uses "Props" suffix pattern instead

### Action
Update CLAUDE.md line 21:

```markdown
- **Naming Conventions**:
- Components: PascalCase
- Functions/Variables: camelCase
- Types/Interfaces: PascalCase (Props suffix for component props)
- Example: `ChatContainerProps`, `WebhookResponse`
```

### Files to modify
- `CLAUDE.md` (line 21)

---

## Task 6: Document Key Dependencies

**Status:** 🔲 Pending
**Priority:** 🟢 Low
**Time:** ~10 min
**File:** `CLAUDE.md`

### Problem
Important dependencies are used but not documented:
- `react-markdown` + `remark-gfm` - Markdown rendering
- `react-syntax-highlighter` - Code block syntax highlighting
- `lucide-react` - Icon library
- `class-variance-authority` - Component variants (used by shadcn)

### Action
Add new section in CLAUDE.md after "UI Components":

```markdown
- **Additional Libraries**:
- `react-markdown` + `remark-gfm`: Markdown rendering with GitHub flavored syntax
- `react-syntax-highlighter`: Code block highlighting in chat messages
- `lucide-react`: Icon library for UI elements
- `class-variance-authority`: Component variant utilities
```

### Files to modify
- `CLAUDE.md` (add after line 30)

---

## Task 7: Decide on Empty /src/contexts Folder

**Status:** 🔲 Pending
**Priority:** 🟢 Low
**Time:** ~2 min
**File:** File system

### Problem
- `/src/contexts` folder exists but is completely empty
- Not documented in CLAUDE.md
- Unclear if it's intentional or leftover

### Action
**Option A:** Remove the folder if not needed:
```bash
rm -rf src/contexts
```

**Option B:** Keep and document intent in CLAUDE.md

**Option C:** Add a README.md inside explaining it's reserved for future use

### Files to modify
- Either delete `src/contexts/` or document it

---

## Checklist

- [ ] Task 1: Fix /src/api documentation
- [ ] Task 2: Fix biome.json jsxRuntime
- [ ] Task 3: Update pnpm version
- [ ] Task 4: Document missing folders
- [ ] Task 5: Update naming conventions
- [ ] Task 6: Document key dependencies
- [ ] Task 7: Handle empty contexts folder
Loading
Loading