chore(repo): move v1/ → archive/v1/ + archive policy README#430
Merged
Conversation
The Rust port at v2/ has been the primary codebase since the rename in #427. The Python implementation at v1/ is no longer the active target; the only load-bearing path is the deterministic proof bundle at v1/data/proof/ (per ADR-011 / ADR-028 witness verification). Move the whole Python tree into archive/v1/ and document the policy in archive/README.md: no new features, bug fixes only when they affect a still-load-bearing path (currently just the proof), CI continues to verify the proof on every push and PR. Path references updated in 26 files via path-pattern sed (only matches v1/<known-child> patterns, never bare v1 or API URLs like /api/v1/). Two double-prefix typos (archive/archive/v1/) caught and hand-fixed in verify-pipeline.yml and ADR-011. Validated: - Python proof verify.py imports cleanly at archive/v1/data/proof/ (numpy/scipy still required; CI installs requirements-lock.txt from archive/v1/ now) - cargo test --workspace --no-default-features → 1,539 passed, 0 failed, 8 ignored (unaffected by Python tree relocation) - ESP32-S3 on COM7 untouched (no firmware paths changed) After-merge: contributors should re-run any local `python v1/...` commands as `python archive/v1/...` (CLAUDE.md and CHANGELOG already updated). Co-Authored-By: claude-flow <ruv@ruv.net>
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.
Why
The Rust port at
v2/has been the primary codebase since the rename in #427. The Python implementation atv1/is no longer the active target; the only load-bearing path is the deterministic proof bundle atv1/data/proof/(per ADR-011 / ADR-028 witness verification).Move the whole Python tree into
archive/v1/so the repo's top level reflects what's actually maintained. Addarchive/README.mdto make the policy explicit so future contributors know what they can and can't do in there.Scope
git mv v1 archive/v1— preserves history for all 180+ files.archive/README.md— new file, documents:archive/archive/v1/typos inverify-pipeline.yml:79andADR-011:329.What did NOT get rewritten
The sed pattern only matches
v1/<known-child>patterns. It deliberately does NOT touch:/api/v1/...URL paths (REST API versioning)0.6.1,v0.X.Yv1references that aren't pathsValidation
python archive/v1/data/proof/verify.pyimports cleanly at the new path. (CI installs numpy/scipy fromarchive/v1/requirements-lock.txt.)cargo test --workspace --no-default-features→ 1,539 passed, 0 failed, 8 ignored (unaffected by Python tree relocation).verify-pipeline.ymlpaths-on filter updated (archive/v1/src/core/**,archive/v1/src/hardware/**,archive/v1/data/proof/**).Dockerfile.pythonpaths updated.After-merge
Contributors with local checkouts should re-run any
python v1/...commands aspython archive/v1/.... CLAUDE.md and CHANGELOG already document the new paths.Test plan
python archive/v1/data/proof/verify.pyresolves at new pathcargo test --workspace --no-default-features→ 1,539 / 0archive/archive/typos remain🤖 Generated with claude-flow