Skip to content

Add taxation-bucket tagging on report_revenue (ordinary, capital, return-of-capital) #856

Description

@Jagadeeshftw

Summary

Add taxation-bucket tagging on report_revenue (ordinary, capital, return-of-capital)

Why this matters

Revenue-share distributions have different tax treatments (ordinary income, capital gains, return of capital). Add a TaxBucket field on report_revenue so downstream indexers and tax engines can categorize each disbursement without out-of-band annotation.

Scope

Implement the requested behavior across Revora-Contracts/src/lib.rs. The solution must preserve existing compatibility, authorization, and production safeguards.

Priority

High

Acceptance criteria

  • The requested behavior is implemented across Revora-Contracts/src/lib.rs with a clear, reviewable contract.
  • Security, authorization, validation, and data-integrity requirements in the repository context are enforced and covered by tests.
  • Failure, retry, timeout, concurrency, and boundary behavior is explicit and produces safe, diagnosable outcomes where applicable.
  • Regression coverage includes empty, invalid, duplicate, and boundary inputs relevant to this flow.
  • Existing API, storage, and deployment compatibility is preserved unless a migration is explicitly documented.

Validation

Include focused tests for:

  • happy paths;
  • invalid input and authorization boundaries;
  • concurrency, retries, or failure recovery;
  • backward compatibility;
  • relevant integration or contract behavior.

The implementation must pass the repository’s existing build, lint, test, and formatting checks (use cargo test --all where applicable).

Non-goals

  • Typo-only, formatting-only, or documentation-only changes.
  • Unrelated refactors or dependency upgrades.
  • Weakening security, authorization, CI, or production safeguards.
  • Changing public behavior outside this issue’s scope.

Contributor application

Before implementation, comment with:

  1. relevant experience;
  2. a concise implementation approach;
  3. expected files or modules affected;
  4. an estimate for opening the first PR.

Wait for maintainer assignment before coding.

PR requirements

Use a feature branch and include Closes #.

The PR must:

  • address every acceptance criterion;
  • link each criterion to code and tests;
  • explain security and failure-mode handling;
  • include meaningful regression coverage;
  • document compatibility or migration considerations;
  • pass the repository checks.

Reward-readiness

This is a substantive quality issue. Merge status does not guarantee reward eligibility; final evaluation is determined separately.

Implementation context

Description

Revenue-share distributions have different tax treatments (ordinary income, capital gains, return of capital). Add a TaxBucket field on report_revenue so downstream indexers and tax engines can categorize each disbursement without out-of-band annotation.

Requirements and context

  • Must be secure, tested, and documented
  • Should be efficient and easy to review
  • Relevant code: Revora-Contracts/src/lib.rs
  • Bucket must be encoded into the indexed event topic for filterable subscription

Suggested execution

  • Fork the repo and create a branch
  • git checkout -b feat/tax-bucket-tagging
  • Implement changes
    • Add enum TaxBucket { Ordinary, Capital, ReturnOfCapital, Custom(Symbol) }
    • Persist bucket in PeriodRevenue and surface it on claim events
    • Document mapping to common jurisdictions in README
  • Validate security and correctness assumptions

Test and commit

  • Run tests
    • cargo test --all
  • Cover edge cases
    • Override revenue must preserve original bucket unless explicitly overridden
  • Include test output and security notes

Example commit message

feat: tag report_revenue with TaxBucket for downstream tax engines

Guidelines

  • Minimum 95 percent test coverage
  • Clear documentation
  • Timeframe: 96 hours

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Stellar WaveStellar Wave issue batch

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions