Repository navigation
Add pre-commit hooks for rustfmt and eslint #89
Description
Summary
Formatting failures reach CI because nothing runs locally.
Context
StellarForge is a Soroban workspace (contracts/*) plus a TypeScript SDK (sdk/). Phase 1 is feature complete, so this stage removes friction for new contributors and makes the toolchain trustworthy before more contracts land.
This issue belongs to Stage 1: Repository, Tooling and Developer Experience.
Objective
Deliver the change described above so that: a commit with bad formatting is rejected locally.
Requirements
- Add a
.pre-commit-config.yaml - Run
cargo fmt --checkand eslint on staged files - Document install steps
Out of Scope
Changes to contract behavior or SDK public APIs unless the issue says so.
Acceptance Criteria
- A commit with bad formatting is rejected locally
- CI is green and a maintainer has reviewed the change
Definition of Done
- The tool runs from a clean checkout and its usage is documented
- The pull request links this issue and updates
CHANGELOG.mdwhen behavior changes
References
scripts/- Makefile
Activity
Clararosey67 commentedon Oct 17, 2026
@Clararosey67 has applied to work on this issue as part of the Stellar Wave Program's 10th wave.
ℹ️ Repo Maintainers: To accept this application, review their application or assign @Clararosey67 to this issue.
OrderStream commentedon Oct 21, 2026
@OrderStream has applied to work on this issue as part of the Stellar Wave Program's 10th wave.
ℹ️ Repo Maintainers: To accept this application, review their application or assign @OrderStream to this issue.
OrderStream commentedon Oct 21, 2026
Hi @0xLizTech! I have implemented the pre-commit hooks and submitted PR #637 with complete rustfmt and eslint hook support, Makefile install target, and documentation updates.
Please assign @OrderStream on Drips Wave so the points can be properly credited:
👉 https://www.drips.network/wave/maintainers/issues/b5474e31-a45f-43d3-986c-76d7a327df86
Thank you!
Adeolu01 commentedon Oct 22, 2026
@Adeolu01 has applied to work on this issue as part of the Stellar Wave Program's 10th wave.
ℹ️ Repo Maintainers: To accept this application, review their application or assign @Adeolu01 to this issue.
drips-wave commentedon Oct 22, 2026
Congratulations, @Adeolu01! 🎉 Your application was accepted by the repo's maintainers, and the issue is due on October 22, 2026.
🧑💻 @Adeolu01: Please resolve the issue such that the repo's maintainers have enough time to review your contribution before the due date. You'll earn Points for completing the issue on-time, which will make you eligible for a share of the Stellar Wave Program's reward pool.
Warning
When opening a PR, please link it to this issue to ensure it gets tracked accurately. Points are awarded when this issue is marked as completed by the maintainer.
🤠 Repo maintainers: Please keep an eye on the contributor's progress and review their work before the due date. You can manage this issue, including adjusting its complexity and points, here.
🌊 Happy Wave 🌊
drips-wave commentedon Oct 26, 2026
This issue has been completed by @Adeolu01 as part of the Stellar Wave Program's 10th Wave 🥳
😎 @Adeolu01: You earned 200 Points for completing this issue! Once the Wave's rewards are finalized, you'll be eligible for a percentage of the Wave's reward pool based on the percentage of total points you've earned. Learn more here. You can also Leave a review to share your experience working on this issue.
🧑💻 Repo maintainers: How'd the contributor do? Leave a review to share your experience working with them.
chore(ci): add stage 1 developer-experience tooling
feat(ci): add pre-commit hooks for rustfmt and eslint (#89)
Merge pull request #637 from OrderStream/feat/pre-commit-hooks-89
chore(ci): add commitlint, pre-commit, cargo-machete, gitattributes