Add ExecutionContext module and missing determinism test suite#64
Closed
Copilot wants to merge 2 commits into
Closed
Add ExecutionContext module and missing determinism test suite#64Copilot wants to merge 2 commits into
Copilot wants to merge 2 commits into
Conversation
Co-authored-by: aidoruao <174227749+aidoruao@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add ExecutionContext class for managing state
Add ExecutionContext module and missing determinism test suite
Mar 8, 2026
aidoruao
added a commit
that referenced
this pull request
Apr 4, 2026
…ring issues (#64, #65, #66, #67, #69, #44, #42) - Analyzed shader integration with Iris/Angelica - Identified OpenGL state management issues (GL_BLEND, GL_ALPHA_TEST) - Found compatibility gaps with Mineshot camera and Chisel blocks - Documented shader pack specific issues (Photon, Solas) - Generated JSON gap analysis + copy-paste GitHub comments for each issue - Created master report with cross-batch patterns - Pipeline: DH-STANDALONE-001
aidoruao
added a commit
that referenced
this pull request
Apr 8, 2026
Adds 5 vendored repositories under investigations/darkshadow44/: 1. DistantHorizonsStandalone (GPLv3, 84,441 LOC, 669 files) - Primary target for issue #51 and 24 other investigations - Includes existing DH_SOURCE_INDEX.json and WIKI.md 2. Angelica (LGPLv3, 100,467 LOC, 1,292 files) - Referenced in 6 issue analyses (#56, #42, #47, #64, #65, #72) - Critical for understanding SplashProgress replacement 3. ArchaicFix (LGPLv3 with caveats, 9,149 LOC, 190 files) - Canonical 1.7.10 performance mod - Shows what DH should NOT re-fix 4. Spool (LGPLv3, 20,195 LOC, 158 files) - Multithreading mod for 1.7.10 Forge - Context for issue #51 'different architecture' comment 5. SeasonalHorizons (LGPLv2.1, 1,336 LOC, 51 files) - DarkShadow44's original code (not a fork) - Shows his architectural patterns and patch acceptance style Each repository includes: - VENDOR_MANIFEST.json with commit hash and Merkle root - LICENSE file (verbatim from original) - ATTRIBUTION.md with non-affiliation statement - sha256_manifest.txt for byte-level verification - SOURCE_INDEX.json and WIKI.md (for analysis notes) - src/ containing full source tree (immutable) Also adds global_merkle.py for computing cross-repo Merkle tree roots: - Computes individual repo roots from src/ contents - Computes global root across all repos - Supports --update-manifests and --verify modes Global Merkle Root: 6b809444934f0457bb13e0ffc2185e7e0637bcefe82cb7ffa00dc4c60927ec7b This eliminates epistemic dependency on external clones and makes all analysis claims falsifiable by providing byte-verifiable source artifacts.
aidoruao
added a commit
that referenced
this pull request
Apr 8, 2026
…merkle.py; reconcile directories ## Changes ### Task 1: Fix global_merkle.py SOURCE_EXTS - Added .java, .kts, .gradle, .toml, .properties, .xml extensions - Recomputed global Merkle root: 162d7969cc8ac4c5cabf9b807490ce5d87cf9f7ab6d7c071246bd61256a99214 - Now includes all 6,116 source files (up from ~2,400) ### Task 2: Angelica SOURCE_INDEX.json + WIKI.md - 1,185 Java files, 100,467 LOC indexed - Identified MixinSplashProgress.java as SplashProgress replacement (critical for DH #56) - Catalogued 237 mixins with DH overlap analysis - Documented GLStateManager.java (1,200 LOC) GL context protection - Cross-referenced DH issues #56, #42, #47, #64, #65, #72 ### Task 3: ArchaicFix SOURCE_INDEX.json + WIKI.md - 177 Java files, 9,149 LOC indexed - Documented 111 mixins and 45+ config options - Identified fixes DH should NOT duplicate (Phosphor lighting, tick sync, etc.) - Analyzed 4 high-risk overlap areas with DH ### Task 4: Spool SOURCE_INDEX.json + WIKI.md - 145 Java files, 20,195 LOC indexed - Documented 3 threading modes (distance-based, dimension, sequential) - Analyzed 50ms tick budget impact - Compared ConcurrentLinkedQueue usage with DH ### Task 5: SeasonalHorizons SOURCE_INDEX.json + WIKI.md - 25 Java files, 1,336 LOC indexed - Documented DarkShadow44's architectural patterns - Mixin naming convention: seasonalHorizons$ prefix - Patch acceptance criteria based on his coding style ### Task 6: Reconcile old investigation directory - Moved tools/, batch1-4/, issue_*_corrected/ to darkshadow44/DistantHorizonsStandalone/ - Removed duplicate DH_SOURCE_INDEX.json and WIKI.md - Updated references in domain.py, README.md, DEVIN_ONBOARDING.md - Deleted empty distanthorizons_standalone/ directory ### All repos now have: - VENDOR_MANIFEST.json with Merkle roots - SOURCE_INDEX.json with census and critical classes - WIKI.md with architecture and issue cross-references - ATTRIBUTION.md with verification instructions - LICENSE file verbatim from original - sha256_manifest.txt for byte-level verification
Contributor
|
Closing as stale (Stage D housekeeping). This Copilot draft has been open without progress and is superseded by:
If the underlying work here is still wanted, re-open a fresh PR rebased on current |
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.
The
pr28-determinism.ymlCI workflow callspython tests/test_determinism.pywhich did not exist, causing every PR run to fail. The problem statement specifies a full deterministic execution pipeline built around anExecutionContextclass.New files
execution_context.py— Core implementation:ExecutionContext— state container (seed, manifest, commands, repo_state, audit_log)canonicalize()— normalizes CRLF→LF, strips trailing whitespace, sorts JSON keys; returns UTF-8 bytesmerkle_root()— binary Merkle tree over canonicalized file contents (odd-node duplication)execute_command()—MODE_1_EXECUTEmutatesrepo_state;MODE_2_SIMULATEis a no-op; both append an audit recordintegrity_loop()— up toMAX_REPAIR_ATTEMPTS=3canonicalization passes beforeenter_mode_0_halt()test_determinism()— runs the command sequence 5× and asserts all Merkle roots agreetests/test_determinism.py— 13 standalone tests (D_001–D_011), runnable viapython tests/test_determinism.py(matching the pattern of the existingtest_falsification.py):Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.