fix(doctor): recognize embedded bd Dolt JSON output#1
Open
onliner10 wants to merge 1 commit into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This fixes
doctor.bd-dolt/autocodex harness preflight --strictcompatibility with newer Beads (bd) embedded Dolt JSON output.Issue
With
bd version 1.0.3, a healthy embedded Dolt workspace can report:{ "backend": "dolt", "data_dir": "/path/to/.beads/embeddeddolt", "database": "weather_agent", "embedded": true, "schema_version": 1 }Autocodex currently expects
bd dolt show --jsonto expose an explicitmodefield, for example:{ "backend": "dolt", "mode": "embedded" }When
modeis absent, the doctor cannot determine Dolt mode and strict harness preflight reports:Root Cause
assessBDDoltShowJSONonly treats embedded Dolt as healthy whenmode == "embedded".Current Beads embedded-mode JSON may encode the same state as
embedded: truewithout amodefield. The text/status/context commands still identify embedded mode, but the JSON parser path used by Autocodex does not infer mode from that boolean.Fix
embeddedboolean frombd dolt show --json.modeandembedded: trueas embedded mode.mode=embeddedin the summarized doctor details so preflight output remains clear.embedded: true/ nomodeJSON shape.After this change, the same healthy workspace is reported as:
Verification
Ran:
Also verified the patched binary against the original failing workspace with:
Contributing Checklist
Followed
CONTRIBUTING.md:onliner10:fix-bd-embedded-json-modefix(doctor): recognize embedded bd Dolt JSON outputgofmtapplied to touched Go filesgo test ./...passes