Skip to content

Commit fb950ae

Browse files
committed
docs: add clear pre-merge todos for next session
- Add prioritized todo list to CLAUDE.md for immediate visibility - Update Next Steps with pre-merge and post-merge task separation - Include unit testing for JSON parsing as high priority - Add error boundaries and performance metrics to task list - Clear guidance for completing v0.4 before merge to main
1 parent fb09911 commit fb950ae

2 files changed

Lines changed: 44 additions & 3 deletions

File tree

CLAUDE.md

Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,40 @@
22

33
This file provides crucial guidance to Claude Code (claude.ai/code) when working with code in this repository.
44

5+
## 🎯 NEXT SESSION TODOS (Pre-merge to main)
6+
7+
**HIGH PRIORITY - Complete these before merging to main:**
8+
9+
1. **Test JSON parsing fix**
10+
- Run query on https://ask-intercom-production.up.railway.app/
11+
- Verify structured output instead of sparse "X customers, Y%" results
12+
- Check logs if issues: `/api/logs?lines=50`
13+
14+
2. **Add unit tests for structured JSON parsing**
15+
- Test `_cleanup_json_response` method in `src/ai_client.py`
16+
- Test various malformed JSON scenarios
17+
- Test successful parsing of valid responses
18+
19+
3. **Add React error boundaries**
20+
- Create ErrorBoundary component
21+
- Wrap main App component
22+
- Show user-friendly error messages
23+
24+
4. **Add basic performance metrics**
25+
- Track query response times
26+
- Log to structured logs
27+
- Add to `/api/debug` endpoint
28+
29+
5. **Integration test the full flow**
30+
- Test with real API keys locally
31+
- Verify Docker deployment works
32+
- Confirm Railway deployment matches local
33+
34+
6. **Final verification**
35+
- All tests passing
36+
- Documentation accurate
37+
- Ready for v0.4 merge to main
38+
539
## 🚀 Quick Start for New Claude Sessions
640

741
**⚠️ MANDATORY FIRST STEP - DO NOT SKIP:**

docs/05-Next-Steps.md

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -112,9 +112,16 @@
112112

113113
### Immediate Next Session Planning
114114
1.**Railway deployment**: Live at https://ask-intercom-production.up.railway.app/
115-
2. **Template marketplace**: Package and submit to Railway templates
116-
3. **User testing**: Share hosted URL with target personas for feedback
117-
4. **Performance monitoring**: Use Railway metrics and remote debugging tools
115+
2. **Pre-merge tasks** (HIGH PRIORITY):
116+
- Test JSON parsing fix with real query on live app
117+
- Add unit tests for structured JSON parsing
118+
- Add error boundaries to React frontend
119+
- Add basic performance metrics collection
120+
- Verify all improvements work end-to-end
121+
3. **Post-merge tasks** (MEDIUM PRIORITY):
122+
- Create Railway template for marketplace
123+
- User testing and feedback collection
124+
- Performance monitoring dashboard
118125

119126
### 🔧 Remote Debugging (Production)
120127
**Critical for monitoring live deployment:**

0 commit comments

Comments
 (0)