Skip to content

feat(anomaly): anomalous tx detection interface#68

Open
0xAlcibiades wants to merge 1 commit into
phylaxsystems:masterfrom
0xAlcibiades:alcibiadeseth/ano-2-create-interface-for-anomaly-detection-inference
Open

feat(anomaly): anomalous tx detection interface#68
0xAlcibiades wants to merge 1 commit into
phylaxsystems:masterfrom
0xAlcibiades:alcibiadeseth/ano-2-create-interface-for-anomaly-detection-inference

Conversation

@0xAlcibiades
Copy link
Copy Markdown

Adds interface for detecting and reacting to anomalous transactions. Notably omits test coverage for the moment until cheat codes land in phorge.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 21, 2026

CLA assistant check
All committers have signed the CLA.

Adds interface for detecting and reacting to anomalous transactions.
Notably omits test coverage for the moment until cheat codes land
in `phorge`.
@0xAlcibiades 0xAlcibiades force-pushed the alcibiadeseth/ano-2-create-interface-for-anomaly-detection-inference branch from 1dba7e0 to b55897e Compare May 21, 2026 14:58
Comment thread src/PhEvm.sol
function inflowContext() external view returns (InflowContext memory ctx);

// ---------------------------------------------------------------
// V2: Anomaly detection
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.

Maybe we should have a new spec with anomaly detection? Wdyt cc: @odyslam @fredo

Comment thread src/Assertion.sol
/// @param target The address whose anomaly score this assertion observes.
/// @param fnSelector The assertion function to invoke when `target` is
/// scored.
function watchAnomaly(address target, bytes4 fnSelector) internal view {
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.

No way to specify what anomalies to specifically watch? I might have a wrong mental model of the system

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.

3 participants