Skip to content

Conversation

@jamesefhawkins
Copy link

@jamesefhawkins jamesefhawkins commented Dec 16, 2025

  • Remove AI slop from this code
  • Get the plan to persist across separate sessions (not sure how this will work yet - this could be a second PR though)
  1. I did wonder if we should make this mode a configuration - i'd love feedback if this style of UX is useful for others. I like it personally....
  2. should it say "live plan" not "plan" to be clearer

if i had more time, i'd want a bit more focus in the prompts on it creating a real todo list and ticking things off - it's somewhat haphazard atm

@jamesefhawkins jamesefhawkins requested a review from a team as a code owner December 16, 2025 11:41
@charlesvien charlesvien marked this pull request as draft December 16, 2025 19:18
@charlesvien charlesvien changed the title DRAFT - live plan mode feat: Live plan mode Dec 16, 2025
- Fix duplicate taskId extraction in claude.ts
- Fix tool name references (mcp__acp__ReadPlan/WritePlan)
- Move require('fs') to top-level import
- Add type safety for _meta field with NewSessionMeta type
- Extract shared path utilities to packages/agent/src/paths.ts
- Add proper layout migration (v8→v9) that preserves user layouts
- Inject system reminder when user mentions "plan" to ensure agent reads plan before implementing
- Hide injected system reminders from UI
- Remove unused RichTextEditor from PlanEditor
- Remove debug UI (taskId, repoPath, branchName display)
- Remove junk files (.posthog/, sounds/) and add to .gitignore

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@jamesefhawkins jamesefhawkins marked this pull request as ready for review December 19, 2025 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants