chore(stables): remove Dune-backed large-transfer monitor - #353
Merged
Merged
Conversation
The hourly stables-dune-large-transfers task has been failing on every run: Dune rejects the query with "This performance tier is not available with your subscription". The script already defaults to the free cluster, so restoring it would need a paid plan rather than a config change. Removes the monitor and every reference to it: - protocols/stables/dune_large_transfers.py and its tests - the hourly task in automation/jobs.yaml - the task path and "Large Transfers" monitor in monitoring.yaml - the DUNE_* vars in .env.example - the dune-client dependency (pyproject.toml + regenerated uv.lock) This drops >=$5M transfer coverage for cUSD and iUSD; nothing else monitors those. Porting it to the Envio-backed pattern used by usdai/large_mints.py is the follow-up if we want the signal back. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.
Summary
Removes the
stables-dune-large-transfersmonitor and every reference to Dune in the repo. It was the only script using Dune.Why
The hourly task fails on every run:
The script already defaulted to the free cluster (
DEFAULT_PERFORMANCE_TIER = "free"), so this is a plan/billing limit rather than something a config change fixes. Rather than keep a permanently-failing task, remove it.Note the failures were never blocking other monitors —
automation/runner.pycontinues on per-task failure and the script caught its own exception — so this is noise cleanup, not an outage fix.Changes
protocols/stables/dune_large_transfers.pytests/test_dune_large_transfers.pyautomation/jobs.yamlhourlyprofilemonitoring.yaml.env.exampleDUNE_*varspyproject.toml/uv.lockdune-client==1.7.10, lock regeneratedprotocols/stables/has no README, so nothing to update there. The lone remaining "Dune" string is a docstring inutils/address_resolver.pylisting hypothetical future label sources — not a dependency.Coverage impact
This drops >=$5M transfer alerts for cUSD (0xcccc62962d17b8914c62d74ffb843d73b2a3cccc, routed to
cap) and iUSD (0x48f9e38f3070ad8945dfeae3fa70987722e3d89c, routed toinfinifi). Nothing else in the repo covers large transfers for those tokens. Porting to the Envio-backed pattern inprotocols/usdai/large_mints.pyis the follow-up if we want the signal back — happy to close this instead if we'd rather upgrade the Dune plan.Verification
uv run --extra dev pytest -q->855 passed, 4 skipped, 5 subtests passed. Thetests/test_monitoring_config.pyconsistency checks pass, sojobs.yamlandmonitoring.yamlstay in sync.uv run --extra dev ruff check .-> clean.uv lock --check-> in sync, so the deploy'suv sync --frozenwon't break.python -m automation run hourly --dry-run-> all 25 remaining hourly tasks still listed, Dune gone; other four profiles untouched.main(Duplicate module named "main"betweenprotocols/aaveandprotocols/3jane), unrelated to this change.Deploy note
Per the header comment in
automation/jobs.yaml, dependency changes still need a manualuv sync --frozen --extra ai+systemctl restart monitoringon the box.DUNE_API_KEYshould also be removed from the deployment's real env/secrets.🤖 Generated with Claude Code