Repository navigation
feat: record transaction transition audit history and dispatch committed hooks - #793
Conversation
052fae2 to
22597f6
Compare
|
Warning Possible malicious code: please review before merging. An automated security scan flagged files in this pull request that appear to contain malicious code. Code like this can run automatically when a project is opened in an editor or its dependencies are installed, so it could compromise the machine of anyone who checks out this branch. Automated scans can be wrong, so this may be a false positive. Maintainers: please review the flagged files carefully before merging, on GitHub in your browser rather than by checking out the branch. The malicious part is often hidden far off to the right of an ordinary-looking line, so scroll sideways through each file. @cindybby18: this pull request adds or changes these files, possibly without your knowledge. Malware like this spreads by adding itself to repositories on machines it has infected, so your machine might be compromised. Please undo these changes in this pull request, and don't open or run other projects on that machine until you've checked it. Flagged files at commit Only merge once you're confident these files are safe. ℹ️ Posted automatically by Drips Wave. Questions? Contact support@drips.network. |
Overview
This PR adds a durable audit record for each transaction status transition and dispatches the matching plugin hook only after the transition is committed. Hook dispatch follows a recoverable policy so that a failed update emits no successful hook and replay does not duplicate events.
Related Issue
Closes #AK-058
Changes
🧾 Transition Audit and Hook Dispatch
[MODIFY]
src/runtime/database/sql-database-adapter.ts[MODIFY]
src/types/plugin.ts[MODIFY]
src/core/factory.tsVerification Results
Reproducible focused check (expected to pass):
Expected result: all transition-audit assertions pass, including one audit entry per committed transition, no hook on failed update, and no duplicate event on replay.
Applicable checks to run before merge:
Closes #717