feat: add multi-platform sync testing (amd64-windows/linux + arm64-linux)#2921
Open
bruno-dasilva wants to merge 12 commits into
Open
feat: add multi-platform sync testing (amd64-windows/linux + arm64-linux)#2921bruno-dasilva wants to merge 12 commits into
bruno-dasilva wants to merge 12 commits into
Conversation
Collaborator
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Apr 11, 2026
bruno-dasilva
force-pushed
the
bruno/coding-docs-3
branch
from
fbc2fb8 to
ada0cab
Compare
bruno-dasilva
force-pushed
the
bruno/fightertest-synctest
branch
from
406efe5 to
1c5634e
Compare
This was referenced Apr 11, 2026
bruno-dasilva
force-pushed
the
bruno/coding-docs-3
branch
from
ada0cab to
7a4ce40
Compare
bruno-dasilva
force-pushed
the
bruno/fightertest-synctest
branch
from
1c5634e to
347360c
Compare
bruno-dasilva
changed the base branch from
bruno/coding-docs-3
to
graphite-base/2921
bruno-dasilva
force-pushed
the
bruno/fightertest-synctest
branch
from
347360c to
479f08e
Compare
bruno-dasilva
changed the base branch from
graphite-base/2921
to
bruno/fightertest-checksum-helper-engine-changes
bruno-dasilva
force-pushed
the
bruno/fightertest-synctest
branch
from
479f08e to
627e5b9
Compare
This was referenced Apr 12, 2026
Closed
bruno-dasilva
force-pushed
the
bruno/fightertest-synctest
branch
from
0da9634 to
a590fdc
Compare
bruno-dasilva
force-pushed
the
bruno/fightertest-checksum-helper-engine-changes
branch
from
d043767 to
fdb3ebe
Compare
bruno-dasilva
force-pushed
the
bruno/fightertest-synctest
branch
from
a590fdc to
b9dd08d
Compare
bruno-dasilva
force-pushed
the
bruno/fightertest-checksum-helper-engine-changes
branch
from
fdb3ebe to
21cb29b
Compare
bruno-dasilva
force-pushed
the
bruno/fightertest-synctest
branch
from
b9dd08d to
2ee2068
Compare
bruno-dasilva
force-pushed
the
bruno/fightertest-checksum-helper-engine-changes
branch
from
21cb29b to
20d298b
Compare
bruno-dasilva
force-pushed
the
bruno/fightertest-synctest
branch
from
2ee2068 to
ead137e
Compare
bruno-dasilva
force-pushed
the
bruno/fightertest-checksum-helper-engine-changes
branch
from
20d298b to
64ea15d
Compare
bruno-dasilva
changed the title
bruno-dasilva
added
the
area: devops/gitops
bruno-dasilva
force-pushed
the
bruno/fightertest-synctest
branch
from
ead137e to
bc51925
Compare
bruno-dasilva
force-pushed
the
bruno/fightertest-checksum-helper-engine-changes
branch
from
64ea15d to
a4d28b8
Compare
# Purpose We're working towards multi-platform sync testing, to detect when there's sync divergences. For now this change just focuses on linux and doesn't even have a functional sync-hash extraction mechanism, but this will at least get the scaffolding of the workflow in so it can be iterated on.
…hashes The collect-hashes job now uploads a combined synchashes manifest per tested commit, fetches historical manifests via the GitHub API, renders a markdown summary table, and fails on branch builds when hashes diverge from the merge-base commit. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…nux) Expand the fightertest workflow to run on both amd64-windows, amd64-linux, and arm64-linux, producing 12 parallel jobs (4 tests x 3 platforms). Split collect-hashes into two jobs: cross-platform-check (merge-blocking, asserts all platforms produce identical hashes) and hash-history (informational, compares against prior commits with warnings only). Closes #2906. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: bruno-dasilva <8520801+bruno-dasilva@users.noreply.github.com>
bruno-dasilva
force-pushed
the
bruno/fightertest-synctest
branch
from
bc51925 to
9271922
Compare
Collaborator
Author
|
@p2004a could you take a look when you have a sec? Should be good to go (I'll coordinate merge of the stack if you approve). |
sprunk
reviewed
Jul 1, 2026
| handicap=0; | ||
| allyteam=1; | ||
| teamleader=0; | ||
| rgbcolor=0.99609375 0.546875 0; |
Collaborator
There was a problem hiding this comment.
do we need to specify random crap like rgb color? would be nice if the file was minimal
Collaborator
Author
There was a problem hiding this comment.
fair enough good idea, this was copied straight out of what chobby generated hence the artifacts.
bruno-dasilva
commented
Jul 1, 2026
| contents: read | ||
|
|
||
| env: | ||
| MAP: "Jade Empress 1.3" |
Collaborator
Author
There was a problem hiding this comment.
Suggested change
| MAP: "Jade Empress 1.3" | |
| MAP: "Jade Empress 1.41" |
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
2 participants
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.
Purpose
This should mostly implement multi-platform sync testing, to detect when there's sync divergences. At least a good v1.0.
Currently runs just amd64-linux, arm64-linux, and amd64-windows - but this can be expanded easily in the matrix later.
Addresses #2910 #2906
Example run: https://github.com/bruno-dasilva/RecoilEngine/actions/runs/26504320539
AI Disclosure: initial passes by claude code, significant cleanup and changes by me :^)