bd dolt pull/push have no timeout — a hung network stalls merge/wrap-up nf-knn ← Beads

open priority 2 task unassigned

Twinned from https://github.com/cfsh/ninefold/issues/1109 by tools/beads/import_github.py's reconcile pass.

bd dolt pull/push (design 011 T5, #1107) route through Bd.raw() (tools/nflib/bd.py), which does not accept a timeout kwarg — unlike Git.raw()/Git.fetch() (tools/nflib/git.py), whose docstrings already establish the house rule for the one verb that reaches the network. `dolt pull/push` are exactly that verb, called synchronously and unconditionally from merge_pr.py's merge() and wrap_up.py's remove(). A hung network there doesn't hit the "never fatal" note: path — it never returns, so the merge/wrap-up just stops responding with no message (DEV_PRINCIPLES D2: "fail noisily and as soon as possible").

Not urgent — bd is still prototype-status, and this mirrors an existing gap in Git.push()/Bd.export() too. Fix: thread a timeout through Bd.raw()/dolt_pull/dolt_push and pass one from beads.sync.sync_bd_dolt.

Raised by ninefold-reviewer's review of #1107 (design 011 T5).

Dependencies

None.

Comments

No comments.

Add a comment