Skip to content

add todo context to tool#1969

Closed
taigrr wants to merge 3 commits intocharmbracelet:mainfrom
taigrr:feat/todo-context
Closed

add todo context to tool#1969
taigrr wants to merge 3 commits intocharmbracelet:mainfrom
taigrr:feat/todo-context

Conversation

@taigrr
Copy link
Contributor

@taigrr taigrr commented Jan 23, 2026

  • I have read CONTRIBUTING.md.
  • I have created a discussion that was approved by a maintainer (for new features).

Summary

• Adds optional context field to todos for storing relevant files, functions, reasoning, or implementation notes
• Context persists with the todo, helping maintain task understanding after conversation summarization

Motivation

When conversations get summarized, the surrounding discussion that gave a todo meaning gets compressed or lost. A todo
like "Fix the validation bug" becomes useless without knowing which file, what validation, or why it matters.

The context field lets the agent capture this information directly on the task, so it survives summarization intact.

Example

{
    "content": "Fix token refresh validation",
    "status": "pending",
    "active_form": "Fixing token refresh validation",
    "context": "src/auth/refresh.go:validateToken() - expiry check missing when token refreshed via cookie auth"
}

@taigrr taigrr requested a review from a team as a code owner January 23, 2026 20:54
@taigrr taigrr requested review from andreynering and aymanbagabas and removed request for a team January 23, 2026 20:54
@kujtimiihoxha
Copy link
Contributor

@taigrr I am wondering if its better to update the summarize agent to ask it to give that info? instead of asking the model to add the context. The main issue is that currently to update the todo's we ask the agent to rewrite them, this can be costly and unnecessary token spend 🤔

@taigrr
Copy link
Contributor Author

taigrr commented Jan 26, 2026

The summarize agent is not working how I would hope. I can tell it in my initial prompt "don't forget the following information in any summaries" and it will still not include it sometimes. Very nondeterministic.

Is there a better way to do todos instead?

@taigrr taigrr force-pushed the feat/todo-context branch from 8439128 to e28e565 Compare January 31, 2026 06:01
@francescoalemanno
Copy link

francescoalemanno commented Feb 7, 2026

The summarize agent is not working how I would hope. I can tell it in my initial prompt "don't forget the following information in any summaries" and it will still not include it sometimes. Very nondeterministic.

Is there a better way to do todos instead?

There is a better way to perform summarisation, #2142
This by construction is almost loss-less, and it can be wired to provide context for todos too, Amp has the best implementation available of this mechanism atm.

@taigrr
Copy link
Contributor Author

taigrr commented Mar 5, 2026

seems to be superceded

@taigrr taigrr closed this Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants