Repository navigation
Organize repository root and preserve historical experiment assets - #614
Conversation
Reviewer's GuideThis draft PR cleans the repository root by categorizing 88 relocated assets and updating all active consumers, CI, documentation, and tests to the new layout, while using a hash/blob manifest and preservation checks to retain historical bytes, provenance, fixtures, and evidence boundaries. Sequence diagram for relocated dependency profiles in CIsequenceDiagram
participant CI as GitHub Actions CI
participant Workflow as Workflow definition
participant Profile as requirements/ profiles
participant Tests as Targeted pytest checks
CI->>Workflow: Trigger on repository path changes
Workflow->>Profile: Install selected profile
Profile-->>Workflow: Pinned environment dependencies
Workflow->>Tests: Run interop and memory validation
Tests-->>CI: Report pass or environment skip
Flow diagram for path migration and preservation validationflowchart TD
Old[Root-level files and assets]
Classify[Categorize files by role]
Move[Relocate 88 files]
Repair[Repair references in live consumers]
Record[Record paths blobs modes and SHA-256]
Verify[Run layout byte and path-integrity checks]
Preserve[Historical bytes fixtures and receipts preserved]
Old --> Classify
Classify --> Move
Move --> Repair
Move --> Record
Repair --> Verify
Record --> Verify
Verify --> Preserve
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Sorry @EmergentMonk, you've used your own review budget of 250,000 diff characters for the last 7 days. You can request another review in 20 hours and 55 minutes by commenting |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 98dcca0cdc
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - [Commands and capability metadata](docs/CAPABILITIES.md) · [IRC operator guide](docs/IRC_BOT_GUIDE.md) | ||
| - [Qutrit construction benchmark and limits](docs/QUTRIT_CONSTRUCTION_BENCHMARK.md) | ||
| - [Roadmap](ROADMAP.md) · [Detailed historical overview](docs/archive/README_BEFORE_V177_1_CLEANUP.md) | ||
| - [Roadmap](docs/roadmap/ROADMAP.md) · [Detailed historical overview](docs/archive/README_BEFORE_V177_1_CLEANUP.md) |
There was a problem hiding this comment.
Rebase links in the historical overview
After relocating the root documentation, the historical overview linked here still points to ../../ROADMAP.md, ../../INSTALL.md, ../../DOCUMENTATION.md, and ../../USAGE.md. Those files no longer exist at the repository root, so readers following its roadmap, setup, documentation, and TUI links receive 404s; rebase them to the new docs/roadmap, docs/guides, and docs/README.md locations.
AGENTS.md reference: AGENTS.md:L96-L99
Useful? React with 👍 / 👎.
|
@codex Please Review this exact SHA 8be76f8 against the existing contract. Report only actionable correctness defects, with a minimal reproduction, expected versus actual behavior, and affected lines. State whether each reproduction was executed or statically inferred. Don’t repeat fixed findings without a new failing case. Keep architectural suggestions separate and non-blocking. |
|
Codex Review: Didn't find any major issues. Keep it up! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
Validation
98dcca0cdcb3a3c1cbed8fb92e737c42c1edfc5a: all 20 GitHub checks passed. Current P2 link-fix head8be76f8e8ba5ffeb3d0592eb8cb34a2135246b3c: seven layout tests passed; independent review verified all 22 local overview links resolve and prose is unchanged. All 20 exact-head CI checks passed on8be76f8e8ba5ffeb3d0592eb8cb34a2135246b3c, including full repository, container and acoustic regressions. GitHub currently reports this PR ready for review; this correction did not change its review state or merge it.98dcca0cdcb3a3c1cbed8fb92e737c42c1edfc5a; stdlib integrity verification passed there. It was not rerun for the subsequent documentation-and-link-test-only head8be76f8e8ba5ffeb3d0592eb8cb34a2135246b3c. GPU commands correctly fail for absent GPU dependencies, so this is not new GPU validation.Boundaries
No scientific algorithm, decoder, dependency pin or historical receipt changes. ZIPs remained opaque: no extraction or execution. Historical commands and newly archived source documents remain byte-preserved; the migration index supplies current paths. The user-requested P2 correction rebases nine navigation links in the pre-existing historical overview without altering its prose; a bounded migration-link regression guards them. No merge or release requested.
Summary by Sourcery
Organize the repository into focused package, documentation, dependency, script, and archive areas while preserving historical assets and their provenance.
New Features:
Bug Fixes:
Enhancements:
Build:
CI:
Documentation:
Tests:
Chores: