You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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:
relevant experience;
a concise implementation approach;
expected files or modules affected;
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
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
TaxBucketfield onreport_revenueso 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
Validation
Include focused tests for:
The implementation must pass the repository’s existing build, lint, test, and formatting checks (use cargo test --all where applicable).
Non-goals
Contributor application
Before implementation, comment with:
Wait for maintainer assignment before coding.
PR requirements
Use a feature branch and include Closes #.
The PR must:
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
TaxBucketfield onreport_revenueso downstream indexers and tax engines can categorize each disbursement without out-of-band annotation.Requirements and context
Revora-Contracts/src/lib.rsSuggested execution
git checkout -b feat/tax-bucket-taggingenum TaxBucket { Ordinary, Capital, ReturnOfCapital, Custom(Symbol) }PeriodRevenueand surface it onclaimeventsTest and commit
cargo test --allExample commit message
feat: tag report_revenue with TaxBucket for downstream tax enginesGuidelines