Talos#2928
Merged
Merged
Conversation
…m/OriginProtocol/origin-dollar into sparrowDom/automated-actions-lite-chris2
- Add TypeScript, ESLint TS, and Prettier TS support to contracts/ - Create action() registration pattern for hardhat tasks with Winston logging - Convert 13 Defender actions to self-registering TS hardhat tasks - Add Loki transport for centralized logging - Add healthcheck task for validation - Fix ssv-scanner dependency (bloxapp → ssvlabs) - Fix @lodestar/types ESM compatibility via ts-node skipProject
…om/automated-actions-lite
Without this, the runner image ships /app/actions-catalog.json missing,
runner.ts falls back to actionsCatalog = {}, and Talos admin's
fetchRunnerEditableFlags() gets 404 on /actions/<task>/params for every
task, surfacing as "No editable flags are registered for this task" in
the action edit dialog (regardless of params actually defined on the
hardhat task).
Mirrors the dump step already in talos/docker/origin-dollar.Dockerfile
(used by docker-compose dev). The talos-owned Dockerfile is not the
one CI builds with — this workflow uses ./contracts/dockerfile-actions
— so the dump was silently absent in production. Self-contained
CommonJS script copied verbatim from talos/docker/dump-actions-catalog.cjs.
The old-org (412463071885) Talos stack is decommissioned. Drop the dual-push to both accounts and the old-account creds/login steps; build and push the runner image to the new account (114563866192) ECR only.
claimBribes pinned gasLimit: 20000000 on removeAllNFTIds, fetchNFTIds and claimBribes. The Base RPC endpoint rejects a 20M tx gas limit ("gas limit too high"; Base block limit is 400M, so it is the endpoint's per-tx cap), so the action aborted on its first call (removeAllNFTIds) and never claimed bribes.
Remove the explicit gasLimit so the nonce queue's estimate + 20% buffer sets it. On-chain estimates as operator 0x7392: removeAllNFTIds 28546, fetchNFTIds 29597 — ~700x under the rejected 20M and well within the cap.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
sparrowDom
requested review from
clement-ux,
naddison36 and
shahthepro
as code owners
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #2928 +/- ##
=======================================
Coverage 44.63% 44.63%
=======================================
Files 110 110
Lines 4920 4920
Branches 1362 1362
=======================================
Hits 2196 2196
Misses 2721 2721
Partials 3 3 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
shahthepro
approved these changes
Jun 17, 2026
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
4 participants
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.
Migrate actions to Talos
Code Change Checklist
To be completed before internal review begins:
Internal review: