Skip to content

fix: prefer local patched imsg binary#10

Merged
kacy merged 1 commit intomainfrom
fix/local-imsg-binary-override
Apr 5, 2026
Merged

fix: prefer local patched imsg binary#10
kacy merged 1 commit intomainfrom
fix/local-imsg-binary-override

Conversation

@kacy
Copy link
Copy Markdown
Owner

@kacy kacy commented Apr 5, 2026

what changed

this teaches the bridge how to resolve imsg from a few override points instead of assuming the binary only comes from PATH.

it now prefers:

  • the explicit -imsg-bin flag
  • IMSGBRIDGE_IMSG_BIN
  • a sibling checkout at ../imsg/bin/imsg when that exists
  • plain imsg on PATH as the fallback

make run also passes the resolved local binary path so local bridge work will pick up a patched checkout automatically.

why

history performance work currently lives in a local imsg fork, not upstream. without an override, the daemon keeps using the installed imsg binary and misses that faster history path.

impact

local development can point the bridge at a patched imsg build without editing code each time.

production behavior stays the same unless an override is provided or the sibling checkout layout is present.

validation

  • go test ./...

@kacy kacy marked this pull request as ready for review April 5, 2026 16:54
@kacy kacy merged commit 449b46e into main Apr 5, 2026
1 check failed
@kacy kacy deleted the fix/local-imsg-binary-override branch April 5, 2026 16:54
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.

1 participant