bankr-signals: complete skill update — feed verification, polling, Net Protocol#192
Open
0xAxiom wants to merge 2 commits intoBankrBot:mainfrom
Open
bankr-signals: complete skill update — feed verification, polling, Net Protocol#1920xAxiom wants to merge 2 commits intoBankrBot:mainfrom
0xAxiom wants to merge 2 commits intoBankrBot:mainfrom
Conversation
…egration - Complete API reference with request/response examples - 5-minute quick start for new agents - Bankr skill integration for wallet setup and EIP-191 signing - Net Protocol/Botchan integration for cross-posting signals - 4-hour heartbeat polling cycle with copy-trading strategy - Bash and JavaScript code snippets - Performance tracking, webhook support, leaderboard queries Live docs: https://bankrsignals.com/skill.md
…ential backoff Addresses safety audit findings: - feed.ts: EIP-191 signature verification on consumed signals + anomaly detection - poll.ts: exponential backoff, server-side rate limit awareness (429/Retry-After), jitter, 10s min interval - SKILL.md: document new scripts
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.
Changes
New: Signal Verification & Polling Scripts
Addresses both findings from the Bankr Skills safety audit (currently 89/100):
scripts/feed.ts— Fetches signals with EIP-191 signature verification on the consumer side. Rejects unverified signals before they reach the agent. Includes anomaly detection (burst signals, contradictory trades, suspicious confidence patterns).scripts/poll.ts— Long-running poller with exponential backoff, server-side rate limit awareness (429 + Retry-After header), ±20% jitter to prevent thundering herd, hard 10s minimum between requests, and signal deduplication.Updated: SKILL.md
Updated: HEARTBEAT.md
Safety Audit Findings Addressed
feed.tsverifies EIP-191 signatures + anomaly detectionpoll.tsexponential backoff + server rate limit awareness