Skip to content

fix(graph-db): report and verify drifting corruption verdicts - #4578

Merged
ScriptedAlchemy merged 2 commits into
masterfrom
fix/corruption-event-assertions
Oct 27, 2026
Merged

ScriptedAlchemy merged 2 commits into
masterfrom
fix/corruption-event-assertions

Conversation

@ScriptedAlchemy

@ScriptedAlchemy ScriptedAlchemy commented Oct 27, 2026 •

Copy link
Copy Markdown
Owner

Why

#4574 was merged independently while its writable repair branch was being verified. Master now contains the drifting-corruption ERROR event and the generated runtime-core test dependency correction. The existing corruption-refusal regression still needs to verify the event severity and diagnostic fields.

Changes

Validation

  • Root and independent source review verified the event capture configuration and assertions against the actual tracing output format.
  • The rebased corruption test and generated Bazel dependency are byte-identical to the previously reviewed repair.
  • The maintained BUILD generator check passed: all 60 generated BUILD files are current. The dependency correction is now inherited from master.
  • Pinned Rust formatting, whitespace checks, and maintained commit hooks passed; the two formatting blobs were independently reproduced using the pinned formatter.
  • Hosted CI must execute the native graph-store regression and finish the required gates before merge. The full native dependency build was not repeated locally after exceeding this environment's memory.

Original failing job: https://github.com/ScriptedAlchemy/tracedecay/actions/runs/40405385367/job/121862232508


Devin Review

@changeset-bot

changeset-bot Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 47b1b8e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@ScriptedAlchemy
ScriptedAlchemy force-pushed the fix/corruption-event-assertions branch from 767ba89 to 3088330 Compare October 27, 2026 02:49
@ScriptedAlchemy
ScriptedAlchemy enabled auto-merge (squash) October 27, 2026 02:50
Apply the pinned repository formatter to the Pi and hook replay files
introduced by the concurrent #4567 merge. The corruption assertions and
all other source changes remain unchanged.
@ScriptedAlchemy
ScriptedAlchemy merged commit c696f28 into master Oct 27, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant