Skip to content

Conversation

MoonBoi9001
Copy link
Member

@MoonBoi9001 MoonBoi9001 commented Sep 25, 2025

Summary

Renames the project from Service Quality Oracle (easy to confuse with Quality of Service Oracle) to Rewards Eligibility Oracle and reduces initial eligibility requirements for launch, to make for a smoother transition period.

Motivation

  • Align naming with the oracle's actual purpose of determining rewards eligibility
  • Reduce barrier to entry for initial launch by requiring only 1 subgraph instead of 10

Key Changes

Naming Updates

  • Updated all references across:
    • Documentation (README, CHANGELOG, CLAUDE.md, ELIGIBILITY_CRITERIA.md)
    • Source code (blockchain_client.py, service_quality_oracle.py)
    • Tests (test_blockchain_client.py, test_configuration.py, test_service_quality_oracle.py)
    • Configuration (config.toml.example, k8s/configmap.yaml)
    • Contract ABI (contracts/contract.abi.json)

Eligibility Requirements

  • Reduced MIN_SUBGRAPHS from 10 to 1 for initial launch
  • Updated justification to focus on "syncing and serving subgraph data" rather than requiring multiple subgraphs
  • Applied consistently across configuration, tests, and documentation

Additional Updates

  • Fixed markdown linting in CLAUDE.md and STYLE_GUIDELINES.md

Impact

  • Contract Interface: Updated ABI to match RewardsEligibilityOracle contract
  • Configuration: All deployments will need to update config values
  • Testing: All tests updated to reflect new requirements
  • Documentation: Clear explanation of new eligibility criteria

Testing

All unit tests have been updated and pass with the new requirements.

@MoonBoi9001 MoonBoi9001 changed the title refactor(REO): rename to RewardsEligibilityOracle refactor(REO): rename to RewardsEligibilityOracle, reduce initial requirements Sep 25, 2025
Updates the BigQuery SQL query snapshot to reflect the reduced MIN_SUBGRAPHS requirement from 10 to 1 for launch.
Copy link

@RembrandtK RembrandtK left a comment

Choose a reason for hiding this comment

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

Name change looks good to me.

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