Forgot to use /bug, so here's an overview of how things went haywire but then okay
Resolved 💬 1 comment Opened Feb 25, 2025 by cartermp Closed Mar 25, 2025
Full session log here: https://gist.github.com/cartermp/eeb1d00bca606f804c877d7fa9bacb36
I asked Claude Code to implement trace critical path calculation in the refinery codebase. It:
- read the codebase and wrote up a quick file around things like code style
- used that to proceed with implementation
- it picked an implementation I'd probably do myself
I then asked it to add tests, which it did:
- Added comprehensive tests
- When running the tests, some things failed to build, it got into a loop (missing things like
fmtimports) but fixed itself - Accidentally deleted most of
collect.go
I then said yo why'd you delete that bro and it said haha whoops
- Correctly re-added the code it deleted
I re-affirmed that memory pressure matters for refinery, and asked it to really go over the implementation
- It added benchmark tests
- Went into a loop of running tests, adjusting implementation, etc.
And then it was done! There are likely some subtleties it missed, but Refinery is a pretty complex codebase so I wouldn't be peeved about that if it were to have missed some things. Full code change is here: https://github.com/honeycombio/refinery/pull/1493
This issue has 1 comment on GitHub. Read the full discussion on GitHub ↗