Skip to content

CONTRIBUTING.md

Latest commit

 

History

History
86 lines (61 loc) · 2.72 KB

CONTRIBUTING.md

File metadata and controls

86 lines (61 loc) · 2.72 KB
Raw

Contributing

Which stack do I work on?

The repository contains two parallel source trees for each layer — Rust contracts, Node backend, and React frontend. From directory names alone it is not obvious which tree receives a given change.

The short rule:

  • Prefer COMEBACKHERE-contracts/ for contract changes
  • Prefer comebackhere-backend/ for backend changes
  • Prefer comebackhere-frontend/ for frontend changes

These are the trees built and tested by CI (make test, .github/workflows/ci-contracts.yml, ci.yml). They are the canonical source of truth and the only path that gets the full required-status-check matrix on every PR.

The sibling-less top-level contracts/, backend/, and frontend/ directories are also valid PR targets — they contain older in-tree copies of the same sources and are still referenced from documentation (for example, docs/error-codes.md cites contracts/invoice/src/lib.rs as the source of InvoiceError). They do not, however, have a dedicated CI workflow of their own, so changes there rely on whichever ci-*.yml job happens to match the files. When in doubt, target the canonical COMEBACKHERE-* tree.

See ARCHITECTURE.md for the full reasoning, the exact directory layout, and how the mirrors are kept in step.

Local hooks

Install pre-commit and enable the repository hooks:

pip install pre-commit
pre-commit install

Hooks run on each commit and enforce:

  • ABI snapshot hygiene (abis/*.json must change together with COMEBACKHERE-contracts/contracts/*/src/)
  • Markdown linting
  • Trailing whitespace detection
  • End-of-file fixing (ensure files end with a newline)
  • JSON validation

Run all hooks manually:

pre-commit run --all-files

Branch Protection

The main branch is protected. Direct pushes are not allowed; all changes must go through a pull request.

Required status checks

All of the following checks must pass before a PR can be merged:

  • contract-build — Soroban contract compilation
  • contract-tests — contract unit and integration tests
  • abi-snapshot-hygiene — ABI metadata in abis/ is consistent with contract source
  • markdown-lint — documentation linting
  • frontend-build — frontend build succeeds

Required reviews

  • At least 1 approving review is required for all PRs.
  • At least 2 approving reviews are required for PRs that touch mainnet-related paths (docs/MAINNET_DEPLOYMENT.md, deployment scripts, or governance configuration).

ABI snapshots

After changing contract interfaces (in COMEBACKHERE-contracts/), regenerate and verify ABI metadata:

make update-abi-snapshots
# or
just snapshot

make check-abi-snapshots
# or
just check-snapshot