chore: remove the HNSW badge; widen nightly's #2885 scope note - #141
Merged
Conversation
…ly's #2885 scope note The footer's SONA line showed a green "⚡ HNSW" badge purely on .swarm/hnsw.index existing. Upstream ruflo#2922 (3.38.7) established that the default memory search path is brute-force cosine even when that index exists — they renamed their own APIs over exactly this overclaim. The badge now reads "HNSW idx": the file proves an index was BUILT, never that search uses it. Revisit if upstream wires HNSW into the default path or persists an algorithm signal the footer can read. The badge gains the segment tests it never had (plan 45 → 47). nightly.yml's non-blocking guard cited ruflo#2885 as a neural-train crash; upstream evidence (2026-08-12) shows the same macOS teardown abort poisoning exit codes of store-touching commands (memory search → correct output, rc 134), so the comment now describes the widened scope — a future x verify memory step would need the same guard. The guard itself is unchanged and stays until #2885 closes. Fixes #139 Fixes #140
…feature The activation footer shows only features genuinely active in the project (its founding contract). Upstream ruflo#2922 established the default search path is brute-force cosine even when .swarm/hnsw.index exists, so the file proves nothing a user can act on. Removed rather than relabeled; reintroduce only if upstream wires HNSW into the default path or persists an algorithm signal the footer can read.
pacphi
changed the title
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.
Fixes #139, fixes #140. Two follow-ups from the ruflo 3.38.5–3.38.8 impact assessment.
Remove the HNSW badge (#140)
The activation footer's founding contract (10fc4e7) is to show only features genuinely active in the project. The
⚡ HNSWbadge keyed off.swarm/hnsw.indexexisting — but upstream ruflo#2922 (3.38.7) established the default memory search path is brute-force cosine even when that index exists (upstream renamedbridgeSearchHNSW→bridgeSearchBruteForceCosineover exactly this overclaim). The file therefore proves no active feature, so the badge is removed rather than relabeled. Reintroduction condition: upstream wires HNSW into the default search path, or persists an algorithm signal the footer can read.TDD: regression test asserts no HNSW claim renders even with the index file present (watched fail against the old badge); segment suite plan 45 → 46.
nightly.yml #2885 scope note (#139)
The non-blocking guard's comment described ruflo#2885 as a
neural traincrash ("open as of 2026-08-06"). Upstream evidence (2026-08-12) shows the same macOS arm64 libc++abi teardown abort poisoning exit codes of store-touching commands (memory search→ correct output, rc 134). Comment rewritten to the widened scope, with the note that a futurex verify memorystep would need the same guard. Guard behavior unchanged; comment-only.Full
pnpm run checkgreen (1469 node tests + 46/46 segment tests).🤖 Generated with Claude Code