Skip to content

Conversation

@M0nkeyFl0wer
Copy link

Summary

Fixes schema validation errors for 40 legacy projects that were causing test failures in multiple PRs.

Issues Fixed

1. Missing Required project_status Fields

The schema now requires three boolean fields in project_status:

  • live_status - indicates if project is actively deployed
  • testnet - testnet availability
  • mainnet - mainnet availability

Error before fix:

{base}.project_status must have required property 'live_status'

2. Ecosystem Capitalization Mismatch

Schema requires lowercase ecosystem values but projects had capitalized values.

Error before fix:

'0' property must be equal to one of the allowed values
path: {base}.ecosystem.0
allowedValues: ["ethereum", "bitcoin", "solana", ...]

Changes Made

✅ Added missing project_status fields to 40 projects
✅ Normalized ecosystem values to lowercase (e.g., "Ethereum" → "ethereum")
✅ Included migration script (migrate-project-status.mjs) for documentation

Migration Logic

// For projects with version: "Mainnet"
project_status: {
  live_status: true,
  version: "Mainnet",
  testnet: false,
  mainnet: true
}

// Ecosystem normalization
"Ethereum"  "ethereum"
"Bitcoin"  "bitcoin"
"Monero"  "monero"
Unknown  "other"

Testing

All 40 projects successfully migrated:
fileverse, mysterium-network, elusiv, zkvote, starkex, hopr, deeper-network, firo, oasis-network, zcash, privatepool, tornado-cash, iden3, circom, zksync, darkfi, sentinel, snarkjs, findora, cake-wallet, typhoon-network, iron-fish, concordium, zk-money, suterusu, oxen, orchid, rotki, mobilecoin, sienna-network, monero, zano, zeal, xx-network, mask-network, fluidkey, webb-protocol, wasabi-wallet, semaphore, incognito

Resolves

…let-team

Update Umbra wallet project with complete information
@M0nkeyFl0wer M0nkeyFl0wer force-pushed the fix/schema-migration-project-status-ecosystem branch from a6e64eb to 75059ef Compare November 11, 2025 05:03
@M0nkeyFl0wer
Copy link
Author

Closing this PR - found additional issue with usecases capitalization. Opening new PR with complete fix.

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