Skip to content

Vault: restore clean lib.rs build (merge artifact removal) #117

@greatest0fallt1me

Description

@greatest0fallt1me

Description

The vault crate must compile cleanly on stable Rust with soroban-sdk 22; remove merge markers, duplicate impl blocks, and inconsistent function signatures.

Requirements and context

  • This is blocking for CI, coverage, and audits.
  • Reconcile the intended public API with README.md.

Suggested execution

  • Fork the repo and create a branch: git checkout -b fix/vault-lib-merge
  • Implement changes in the Callora-Contracts Soroban workspace (contracts/*/src/).
  • Run cargo fmt, cargo clippy --all-targets --all-features -- -D warnings, and cargo test from the workspace root.
  • For WASM release builds: cargo build --target wasm32-unknown-unknown --release -p callora-vault (adjust -p if the crate name differs).

Deliverables

  • Repair contracts/vault/src/lib.rs and fix test.rs accordingly.
  • Run ./scripts/check-wasm-size.sh after release build.

Test and commit

  • Run tests and ./scripts/coverage.sh (or cargo tarpaulin per tarpaulin.toml).
  • Cover edge cases; include summarized test output and brief security notes in the PR description.

Example commit message

fix(vault): restore compiling vault implementation

Guidelines

  • Minimum 95% line coverage for touched crates (workspace policy).
  • Clear documentation (Rust /// on public items where applicable; repo markdown as needed).
  • Timeframe: 96 hours from assignment unless agreed otherwise.

Metadata

Metadata

Assignees

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions