feat(wallet_screening): paginate Etherscan txlist and surface warnings (#214)

[Hendobox]

Summary

• Paginate Etherscan account.txlist (1k per page, cap 10k txs)
• Add metadata.warnings when history is truncated or unavailable
• Update agent instructions.md and bundle tests (mocked only)

Fixes #214 · Part of #115

Test plan

• pytest skills/finance/wallet_screening/test_skill.py tests/skills/finance/test_wallet_screening.py -q (18 passed)
• flake8 + black on changed files

[Hendobox]

@rosspeili #214 ready - Etherscan txlist pagination + metadata.warnings, instructions + tests. Scoped like #128/#140. Fixes #214 · part of #115.

[rosspeili]

Hey @Hendobox, thanks for picking up another clean #115 slice.

A small tip before merge, would be nice to have a line in docs/skills/wallet_screening.md about the 10k cap and warning codes would help operators who don’t read instructions.md first (fine as a follow-up if you prefer to keep this PR tight).

I’ll update the pinned #115 comment to mark #214 / #215 as in review under Ph 2.2b (slice). Happy to merge once CI is green. Nice work continuing the RFC pattern from #128 / #140. <3

[Hendobox]

Thanks @rosspeili — added the catalog note in docs/skills/wallet_screening.md (Limitations): paginated 10k cap + metadata.warnings codes (etherscan_txlist_truncated / unavailable / partial). Pushed on this PR so ops don’t have to dig into instructions.md. CI was already green; this should be a quick re-check. 🙏

[rosspeili]

@Hendobox Thanks for the quick follow-up,

This looks tight, I only see a conflict on changelog since we had 2 releases cut today, but it's a quick rebase fix. Also in terms of data schema, maybe add optional metadata.warnings to the JSON example in docs/skills/wallet_screening.md (eg. "warnings": ["etherscan_txlist_truncated"]) so the catalog matches the new output shape, also not sure if we need to do it now or when all items finish, bumping manifest.yaml version (eg 1.0.0 → 1.0.1) for the user-visible output change, or you want to make it 2.0.0 when the rfc is done? 100% up to you.

Once CI is green on the rebased branch, we’re good to merge. Thanks for continuing to this RFC and web3 area of Skillware bro <3

[rosspeili]

@Hendobox how you've been? This might need another rebase, and address the above after 0.4.5. Happy to rereview and merge your next commit when done, then will sort the trackers on the rfc.

[Hendobox]

Hey @rosspeili — thanks for the nudge. Rebased onto latest main (post-0.4.5), resolved the CHANGELOG conflict, and addressed the review notes:

• Catalog Data Schema JSON now includes optional metadata.warnings (e.g. etherscan_txlist_truncated)
• Limitations section already documents the 10k cap + warning codes
• Bumped manifest.yaml 1.0.0 → 1.0.1 for this user-visible output change; leaving 2.0.0 for when [RFC]: Harden wallet_screening data pipeline, parsing, coverage, and report schema v2 #115 schema v2 lands

Local wallet_screening tests: 18 passed. Happy for another look once CI is green. 🙏

[rosspeili]

Thanks @Hendobox, merged this one. Clean slice as always <3 Updated the #115 Tracker also. What would be the next step from the RFC picks?