Skip to content

internal dev cache-expert agent skill#11720

Merged
sipsma merged 1 commit intodagger:mainfrom
sipsma:skills
Jan 22, 2026
Merged

internal dev cache-expert agent skill#11720
sipsma merged 1 commit intodagger:mainfrom
sipsma:skills

Conversation

@sipsma
Copy link
Contributor

@sipsma sipsma commented Jan 20, 2026

Had some promising but not quite fruitful trials w/ latest LLM coding models (codex 5.2-xhigh + opus 4.5) when it comes to debugging super gnarly engine issues around caching.

Seemed like one bottleneck was that it had to learn (and re-learn after context compaction) the whole codebase every time (which is often necessary for particularly tricky problems). So giving it detailed developer docs that focus it in on the parts that truly matter when debugging cache stuff may save a lot of file read/reasoning tokens.

Another issue was that it needed to be told all the time (or discover on its own) generally how to debug. Not just what commands to run but the techniques (i.e. println debugging of cache keys will get you very far).

Trying to package this all up in a skill. ATM the main utility of skill vs other packaging is just that LLMs have baked in knowledge on progressive disclosure of various skill docs, which is highly important here so we don't spend all our tokens just reading these docs.


This is still a WIP, it's basically become "the ultimate brain dump". I've been chipping away in the evenings, doing a mix of hand human writing and having LLMs interview me about a topic and then write everything it learned from me and from the code.

There's still a lot left to cover but probably reaching the point of utility. May give it a try on some caching related flakes and see how it does.

Been working with Claude/Codex to brain dump how caching works and how
to debug caching issues into a set of docs loadable through a skill.

This is still WIP but it's easier to keep iterating on it once merged as
things change. Can't hurt in the meantime either.

Signed-off-by: Erik Sipsma <erik@sipsma.dev>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doc is quite out of date and seemed to confuse the LLM more than help, so I just rm'd it here. It's largely covered in new docs added to the skill here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc @vito since you wrote it way back when

@sipsma sipsma marked this pull request as ready for review January 22, 2026 02:35
@sipsma sipsma requested a review from a team January 22, 2026 02:48
@sipsma sipsma merged commit 996e343 into dagger:main Jan 22, 2026
49 of 50 checks passed
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