chore(repo): drop host and topology references from public surfaces - #771
Merged
Merged
Conversation
added 2 commits
September 16, 2026 10:45
The published repo described the specific machines it builds on: named CI and live-instance hosts plus a core count in the .kanon-ci.toml header, a fleet memory budget restated in CONTRIBUTING.md, an internal LAN hostname in the forge-fallback note, and that same hostname as a URL fixture in komide's validation tests. None of it is part of harmonia's public contract. Each removal keeps the invariant that made the prose worth writing: the --jobs 8 / --test-threads 8 pin still states that it bounds a single run's memory envelope, so a reader who cannot see the sizing will not delete the pin. That rationale now lives once, in the .kanon-ci.toml header, and CONTRIBUTING.md points at it rather than carrying a second copy that had already drifted to a different host.
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
1 participant
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.
Removes private fleet topology from surfaces that are public in this repository.
What this closes
.kanon-ci.tomlconcurrency rationale. The co-tenancy reason for the--jobs 8pin is preserved, because contention with other resident work is the claim the original actually made — not that an unpinned run exhausts the machine.CLAUDE.md,docs/README.md, anddocs/architecture/errors.md..lanhostname inCONTRIBUTING.mdand in one test fixture.At HEAD, a repo-wide search for the removed classes returns nothing: operator home paths, build-host names, CPU model numbers, internal
.lanhostnames, private host-operations repo references, and co-resident service names.What this provably cannot close
This is a forward-only commit. Every removed string remains reachable in this repository's history, and nothing here rewrites or removes it. Anyone with read access can recover all of it from prior commits.
It also does not touch commit author metadata, a separate class that content-level changes do not reach at all.
Both are deliberately left to the forge's history-cut capability rather than hand-rolled here, so one owner performs the operation once across the fleet.
Verification
Unlike the rest of this sweep, harmonia changes one line of executable code: the input string in
crates/komide/src/service/tests.rs::validate_url_accepts_lan_host. Config equivalence is not a sufficient proof for that, so the test was run directly:The substitution is semantically equivalent for what the test asserts — both values are
.lanhosts, and the test's purpose is that a LAN host is accepted rather than that one specific host is. The remaining changes in this PR are prose and comments; the parsed.kanon-ci.tomlis identical tomain's.