Daily Edition Sources +6

Kieran Klukas Made Crush Share Its Config Safely

Three public Crush commits by Kieran Klukas turned shared config, OAuth refresh, and malformed tool-call state into tested failure boundaries.

Diagram Punk poster showing two Crush sessions pointing at one locked config file, with Kieran Klukas and commit evidence cards for copy-on-write config, token refresh singleflight, tool JSON guards, race tests, and a caveat stamp about tests not every provider.
Diagram Punkshared state gets locks, snapshots, and tests.
repo charmbracelet/crush evidence
6 source signals 1 repo 3 linked commits
Evidence: 3 linked commits / June 26, 2026 / Daily Edition
Open Edition Evidence below

The June 26 companion story starts with commit b10f890f, where Kieran Klukas changed Crush's config store so readers see copy-on-write snapshots instead of racing writers during startup and settings changes.

That contribution mattered because the failure mode was ordinary operator life: two sessions, reloads, model settings, Docker MCP config, compact mode, and credential refresh all touch shared state while an agent is still running.

The public trail

The copy-on-write patch added clone isolation and race tests, then replaced narrower reload locking with a single write lock for every operation that produces a new in-memory config. The commit message says the race detector had flagged concurrent startup and settings changes.

A second Klukas commit, de679203, tackled a more human failure: two Crush instances sharing one rotating OAuth login could invalidate each other's refresh token and both end up with authentication failures.

Why this contribution matters

The repair did not hide the problem behind a retry. It added in-process singleflight, a per-provider cross-process refresh lock, disk adoption of a peer's newer token, and tests that simulate two stores racing over the same credential.

Klukas also landed 21a457d5, which validates provider tool-call JSON before storing it so malformed arguments become an explicit tool error instead of a stuck conversation on every resend.

The conversation to open

The constructive question for Crush is whether these shared-state repairs should become a visible release note category for operators running multiple sessions or providers with rotating credentials.

The fairness limit is important: the commits prove source-level repairs and tests, not that every provider, filesystem, or multi-process edge case is closed.

Evidence Trail

Receipts below the story

The article above is the public narrative. This section keeps the source trail, limits, and reporting notes on the same page.

Edition
DateJune 26, 2026
LaneDaily Edition
Confidence87%
Sources6
Reposcharmbracelet/crush

Reporter Notes

This companion is a contributor-centered reliability repair trail. It uses public commits by Kieran Klukas and public source files only; it does not infer private motivation or identity beyond public authorship.

Primary Evidence

Evidence Limits

  • The evidence proves public source repairs and tests, not every provider, filesystem, or multi-process edge case.
  • The article credits public commit authorship and avoids claims about private motivation.
  • The article does not claim these fixes are already present in a public release note unless the project publishes one.
Letters & Corrections

Send a note to the desk

Corrections, missing context, or a follow-up lead.