Skip to content

Add validator snoopfear|PON (ID 206)#815

Merged
johnmarcou merged 3 commits intomonad-developers:mainfrom
snoopfear:main
Feb 6, 2026
Merged

Add validator snoopfear|PON (ID 206)#815
johnmarcou merged 3 commits intomonad-developers:mainfrom
snoopfear:main

Conversation

@snoopfear
Copy link
Contributor

Testnet validator info for snoopfear|PON

Added JSON file containing details for snoopfear|PON.
@greptile-apps
Copy link

greptile-apps bot commented Feb 6, 2026

Greptile Overview

Greptile Summary

Added validator info for snoopfear|PON (ID 206) with complete metadata including SECP and BLS keys, website, description, logo, and X profile.

Critical Issue:

  • File is placed in root directory instead of testnet/ directory, which will cause CI validation to skip it and prevent the file from being processed in the deployment workflow

Confidence Score: 0/5

  • This PR cannot be merged as-is due to incorrect file placement
  • Score reflects critical structural issue: the validator JSON file is in the root directory instead of testnet/ directory, which means it will not be validated by CI (check.yaml only processes files in testnet/ or mainnet/ paths) and will not be included in the deployment workflow (generate-validators.yml only processes files in those directories). This must be fixed before merge.
  • The file 0203a26b820dafdb794f1fc7117ba8e897830b184dcb08e45a44262f018deabaf3.json must be moved to the testnet/ directory

Important Files Changed

Filename Overview
0203a26b820dafdb794f1fc7117ba8e897830b184dcb08e45a44262f018deabaf3.json File placed in root instead of testnet/ directory; will not be validated by CI and won't be processed

Sequence Diagram

sequenceDiagram
    participant Author
    participant PR as Pull Request
    participant CI as GitHub CI
    participant Validator as validate.py
    participant Chain as Monad Testnet RPC

    Author->>PR: Add validator JSON file
    Note over Author,PR: File: 0203a26b820dafdb794f1fc7117ba8e897830b184dcb08e45a44262f018deabaf3.json
    
    PR->>CI: Trigger check.yaml workflow
    CI->>CI: Filter changed files (testnet/*.json OR mainnet/*.json)
    Note over CI: ❌ File in root directory is skipped
    CI-->>PR: No files to validate (PASS - incorrectly)
    
    Note over CI,Chain: Expected flow if file were in testnet/:
    CI->>Validator: Run validate.py on testnet/[filename].json
    Validator->>Validator: Check JSON schema
    Validator->>Validator: Validate name and logo URL
    Validator->>Chain: Fetch validator keys for ID 206
    Validator->>Validator: Compare on-chain vs local SECP/BLS keys
    Validator->>Validator: Verify filename matches SECP key
    Validator-->>CI: Validation result
    CI-->>PR: Report validation status
Loading

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment on lines 1 to 10
{
"id": 206,
"name": "snoopfear|PON",
"secp": "0203a26b820dafdb794f1fc7117ba8e897830b184dcb08e45a44262f018deabaf3",
"bls": "a5e8b15da155a204d87b4caf71d1694f002b471166e47813507d024b85dcd71635796c7ab746ae201b1f588b3b4f65c1",
"website": "https://t.me/snoopfear",
"description": "Reliability, performance, and 24/7 monitoring. Enterprise-grade infrastructure with a fair fee",
"logo": "https://raw.githubusercontent.com/snoopfear/logo/main/my_logo.png",
"x": "https://x.com/snoopfear"
}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

file is in the wrong location - must be in testnet/ directory

The CI workflow at .github/workflows/check.yaml:6-7 only validates files in testnet/ or mainnet/ directories. This file needs to be moved to testnet/0203a26b820dafdb794f1fc7117ba8e897830b184dcb08e45a44262f018deabaf3.json

@johnmarcou johnmarcou merged commit ae6ae93 into monad-developers:main Feb 6, 2026
4 checks passed
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.

2 participants