harvest: fresh-machine kit deltas + direct-path import defect (F2, F6) - #1
Merged
Conversation
Arrived at the same conclusion as finding-001 and G4 from the orchestrator side, independently: the false "proxied is the only TLS-capable mode" premise had also been written into the orchestrator's Phase-1 attach plan, and building the proxied path there is what disproved it. Main already carries the corrected shape, so this keeps only what is additive. F2 gains two kit items from a real second-machine attempt. doctor should assert the effective port rather than the configured one: bd path-hashes a port when BEADS_DOLT_SERVER_PORT is unset, and its auto-start then creates an empty server there which rejects the configured user, so a pointer problem surfaces as an authentication error and invites the wrong diagnosis. And a host-specific connection pointer must not be a shared artifact, since committing one is what carried the wrong port to the second machine in the first place. F6 gains (i): bd import miscounts on the direct path, reporting existing rows as created and disagreeing with --dry-run on the same input. Unlike (h) this is on callbook's path, because direct mode is primary and import is the seeding tool, so a seed silently reporting every row as new is a bad signal exactly when an operator is checking whether the seed took. F6(h) corrected: our import-leg PR #5181 was withdrawn as needing rework, not in flight. Refs: aae-orc finding-088
arcavenai
force-pushed
the
harvest/server-mode-attach
branch
from
July 31, 2026 04:31
85d077d to
e146e68
Compare
The orchestrator-side harvest split: finding-088 there is the direct-mode TLS disproof, and the fresh-machine trap that produced these two kit items is finding-089. Reference corrected so the pointer resolves to the artifact that actually carries the evidence.
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.
Why
Two sessions converged on the same correction from opposite ends.
main'sfinding-001 + G4 disproved the "proxied-server is the only TLS-capable
client" premise by wire-testing TLS against a deployed instance. Working
from the orchestrator side, I hit the same false premise written into the
Phase-1 attach plan, and building the proxied path is what disproved it
there: 95 proxied-server mode-refusals in
cmd/bdagainst zero fordirect mode, and
import/export/rememberall working in direct modewith ids, timestamps, labels and dependency edges intact.
That independent arrival is worth stating and then dropping:
mainalready has the better-evidenced version (dummy-password discrimination
separating TLS failure from auth failure is a sharper method than mine,
and it correctly isolates the gap to
bd init). This PR keeps only whatmaindoes not already say.What is additive
F2, two kit items from a real second-machine attempt (the failure
that started the whole investigation):
doctorshould assert the effective port, not the configured one.bd path-hashes a port when
BEADS_DOLT_SERVER_PORTis unset, and itsauto-start then creates an empty server there, which rejects the
configured user. So a pointer problem presents as an authentication
error, and the obvious diagnosis (credentials, keychain) is the wrong
one.
doctoralready knows both values, so it is the cheap place tocatch this.
Committing one is precisely how the wrong port reached the second
machine. Worth stating in the kit rather than leaving to convention.
F6 (i), a defect on the path
mainjust made primary:bd importmiscounts on the direct path.
importIssuesCoresetsCreated: len(importedIDs)and never setsUnchanged, so existing rowsare reported as created and a converged re-import disagrees with
--dry-runon the same input. Unlike (h), this is on callbook's path:direct mode is now primary and
importis the seeding tool, so a seedthat reports every row as new is a bad signal at exactly the moment an
operator is checking whether the seed took. Reproduced independently on
a second machine; queued for upstream filing. It pairs with aae-orc
finding-081 (
import --dry-runnever consults the db): the real run andthe dry run compute their counts independently, which is the underlying
weakness.
F6 (h) corrected: our import-leg PR #5181 is not "in flight." I
withdrew it as not fully formed and needing rework, after
mainwaswritten.
Docs only, one file. Em-dash, org-token, and shellcheck gates pass
locally; the
helmsteps ofjust checkneedhelm, which is notinstalled here, and no chart files are touched.
Fuller evidence, including the four other results from the session, is in
aae-orc
_kos/findings/finding-088-bd-server-mode-over-proxied-server.md.