ponderants/app

Fork 0

Commit Graph

Author	SHA1	Message	Date
Albert	a520814771	feat: Implement node deletion with shared modal and fix SurrealDB RecordId handling Implements complete node deletion functionality for both galaxy view and debug panel: Core Changes: - Created shared DeleteNodeModal component used by both ThoughtGalaxy and UserMenu - Modal provides consistent UX with proper confirmation messaging - Deletion follows write-through cache pattern: ATproto first, then SurrealDB SurrealDB RecordId Fixes: - Fixed SELECT query to use type::thing($table, $recordId) for UUID-based RecordIds - Fixed DELETE query to use type::thing() instead of db.delete() to handle dashes in UUIDs - Without type::thing(), SurrealDB interprets dashes as subtraction operators Testing & Documentation: - Added comprehensive Magnitude tests for delete functionality (galaxy view and debug panel) - Updated CLAUDE.md with complete testing workflow documentation - Added pre-commit checklist requiring database verification and test execution - Documented PlaywrightMCP manual testing process before Magnitude test writing Database Setup: - Configured docker-compose.yml to use environment variables for credentials - Updated namespace/database to match .env configuration (ponderants/main) File Changes: - app/api/nodes/[id]/route.ts: Fixed RecordId query patterns (SELECT and DELETE) - components/DeleteNodeModal.tsx: New shared modal component - components/ThoughtGalaxy.tsx: Uses shared DeleteNodeModal - components/UserMenu.tsx: Replaced browser confirm() with shared DeleteNodeModal - tests/magnitude/03-delete-node.mag.ts: Added debug panel delete test - AGENTS.md: Added testing workflow and pre-commit checklist documentation - docker-compose.yml: Environment variable configuration 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-11-10 13:25:01 +00:00
Albert	63c955c848	feat: Add delete functionality for user-authored nodes - Add DELETE /api/nodes/[id] endpoint for deleting nodes - Verify user authentication and ownership before deletion - Delete from ATproto (source of truth) first, then SurrealDB cache - Add delete button in ThoughtGalaxy component for user's own nodes - Add confirmation modal before deletion - Fix Modal z-index to appear above node detail panel (zIndex: 1001) - Fix RecordId encoding issue (strip angle brackets ⟨⟩ from IDs) - Remove deleted node and associated links from local state - Add comprehensive Magnitude tests for delete functionality 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-11-10 01:49:48 +00:00

Author

SHA1

Message

Date

Albert

a520814771

feat: Implement node deletion with shared modal and fix SurrealDB RecordId handling

Implements complete node deletion functionality for both galaxy view and debug panel:

**Core Changes:**
- Created shared DeleteNodeModal component used by both ThoughtGalaxy and UserMenu
- Modal provides consistent UX with proper confirmation messaging
- Deletion follows write-through cache pattern: ATproto first, then SurrealDB

**SurrealDB RecordId Fixes:**
- Fixed SELECT query to use type::thing($table, $recordId) for UUID-based RecordIds
- Fixed DELETE query to use type::thing() instead of db.delete() to handle dashes in UUIDs
- Without type::thing(), SurrealDB interprets dashes as subtraction operators

**Testing & Documentation:**
- Added comprehensive Magnitude tests for delete functionality (galaxy view and debug panel)
- Updated CLAUDE.md with complete testing workflow documentation
- Added pre-commit checklist requiring database verification and test execution
- Documented PlaywrightMCP manual testing process before Magnitude test writing

**Database Setup:**
- Configured docker-compose.yml to use environment variables for credentials
- Updated namespace/database to match .env configuration (ponderants/main)

**File Changes:**
- app/api/nodes/[id]/route.ts: Fixed RecordId query patterns (SELECT and DELETE)
- components/DeleteNodeModal.tsx: New shared modal component
- components/ThoughtGalaxy.tsx: Uses shared DeleteNodeModal
- components/UserMenu.tsx: Replaced browser confirm() with shared DeleteNodeModal
- tests/magnitude/03-delete-node.mag.ts: Added debug panel delete test
- AGENTS.md: Added testing workflow and pre-commit checklist documentation
- docker-compose.yml: Environment variable configuration

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

2025-11-10 13:25:01 +00:00

Albert

63c955c848

feat: Add delete functionality for user-authored nodes

- Add DELETE /api/nodes/[id] endpoint for deleting nodes
- Verify user authentication and ownership before deletion
- Delete from ATproto (source of truth) first, then SurrealDB cache
- Add delete button in ThoughtGalaxy component for user's own nodes
- Add confirmation modal before deletion
- Fix Modal z-index to appear above node detail panel (zIndex: 1001)
- Fix RecordId encoding issue (strip angle brackets ⟨⟩ from IDs)
- Remove deleted node and associated links from local state
- Add comprehensive Magnitude tests for delete functionality

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

2025-11-10 01:49:48 +00:00

2 Commits