Repository navigation
build: consume Grafeo fork main without a downstream patch - #4721
Closed
ScriptedAlchemy wants to merge 2 commits into
Closed
ScriptedAlchemy wants to merge 2 commits into
ScriptedAlchemy wants to merge 2 commits into
Conversation
|
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This was referenced Nov 3, 2026
Apply the reviewed fixture repair from fbca068 (#4731) after this branch's private Linux run failed three admission tests at the redundant generation-publication wait. The original fixture blob matches exactly. Retain the stronger fresh serving-seat wait and every admission assertion. No timeout increases, production changes, or validation skips. The independent cold-daemon Git topology failure is not addressed by this repair. The shared repair passed native Linux on #4731; the updated Grafeo branch still needs its own full hosted Bazel validation. No local Rust execution is claimed.
This was referenced Nov 3, 2026
Owner
Author
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.
Scope
Consume the allocation-accounting APIs maintained directly on
ScriptedAlchemy/grafeomain, rather than applying them only through a downstream Bazel patch.All five sibling Grafeo crates use
branch = "main"; Cargo.lock pins5116be3c7b9392006ef4d229634963e1306a7b6b, merged in ScriptedAlchemy/grafeo#14. The local patch and its Bazel annotation are removed. No unrelated dependency revisions change.Shared CI fixture repair
Commit
9651a9ff7f03d69bdfd2d1a0b5dfb7c55f8bf070applies the exact reviewed fix from #4731 after this branch's private Linux run failed three ignored-dependency admission tests at the redundant generation-publication wait. The original file matched #4731's parent byte-for-byte; the replacement blob is9135cf4f9077156831e3f0bb0141f40b99503b65.The fixture waits directly for its existing stronger fresh serving-generation state. Every admission assertion and the serving-readiness deadline remain unchanged. This is a non-force follow-up commit, not a merge into master.
Validation
The original implementation reported a successful bounded LCM compiler check, Grafeo allocation tests (3 with LPG/compact-store and 4 with temporal), focused formatting/Clippy, and patch-registry validation. Those results predate this follow-up and are not full current-branch runtime validation.
The original private Linux run 41606692236 passed 179/181 Bazel targets. This fixture repair addresses its code-index admission race; the separate cold-daemon Git-sync
git_topology_declared_authority_unavailablefailure remains unresolved.The shared fixture fix passed native Linux on #4731 (run 41665062193). This does not establish that the updated Grafeo integration passes. Fresh exact-head CI is queued at https://github.com/ScriptedAlchemy/tracedecay/actions/runs/41706760515. No local Rust compilation or tests ran during this follow-up; complete applicable hosted Bazel validation is required before merge.