chore(repo): clarify autonomous research network and remove unused code - #230
Merged
Conversation
Ground the overview in the whitepaper's reusable research thesis and separate proposed synthesis from the currently implemented mechanisms. Document existing Proof judge, persistence, and emission limitations. Remove unused chain stubs, helpers, a superseded prompt, the obsolete Proof average, and unused dependencies. Share metadata storage projection and cover the retained behavior with regression tests. Refresh contributor, miner, architecture, and deployment guidance without changing scoring rules, signatures, migration history, or deployed names. Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
Contributor
Author
|
@greptileai review |
Greptile SummarySummary
Merge safetySafe to merge. No issues requiring changes were identified. Confidence Score: 5/5Safe to merge: no actionable defects were identified in the reviewed changes. No final findings remain after exercising gateway request forwarding, live metadata snapshot projection, and Proof scoring and emission behavior before and after the changes. Files Needing Attention: None.
What T-Rex did
Reviews (1): Last reviewed commit: "chore(repo): clarify research network an..." | Re-trigger Greptile |
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.
Summary
ChainClient,FakeChain, productionchain-live, active v3 prompts, and Proof's consumedprism-*harvest stack.See
docs/CLEANUP.mdfor deletion/retention evidence anddocs/WHITEPAPER.mdfor the proposal-versus-implementation comparison. The PDF itself is unchanged.Greptile
@greptileai reviewThree local simplification reviews covered the entire 50-file change: reuse, quality/compatibility provenance, and efficiency. No introduced actionable code issues were found. Their routing-persistence documentation clarification was applied.
Test plan
cargo fmt --all -- --checkcargo clippy --workspace --all-targets -- -D warningscargo test --workspace: 1,119 passed, 0 failed, 11 ignoredcargo-deny checkusing an isolated installation: advisories, bans, licenses, and sources pass; non-blocking dependency warnings remaincargo run -p xtask -- loc-capcargo run -p xtask -- consensus-lintcargo run -p xtask -- spec-checkcargo run -p xtask -- design-checkcargo run -p xtask -- external-docs-checkcargo clippy -p validator-bin --features dcap --all-targets -- -D warningsbash deploy/scripts/assert-compose-matrix.shNot run: paid Lium/GPU experiments, deployment, live on-chain submission, or the 11 ignored Rust tests. Local HTTP/challenge and signed-leaf integration tests are included in the workspace suite; these are not presented as proof of live scientific reproduction or payments.
Risk
No change to scoring formulas, emission shares, trust roots, signature domains, applied migrations, deployment settings, or pinned image digests. Compose changes are comments only. Removed Rust surfaces had no workspace consumers; the chain crate is unpublished. Existing scientific/economic readiness gaps are documented rather than hidden or altered during cleanup.
Naming
BASE_*,/opt/base,/run/base, existing registry paths, cryptographic domain tags, compatibility routes, frozen specifications, historical miner links, and operational evidence remain intact.