Files
SnapOtter/CONTRIBUTING.md
T
SnapOtter fa50266ea4 fix: address review issues in contribution infrastructure
- CODEOWNERS: use @snapotter-hq username instead of non-existent team
- PR template: use absolute URLs (relative links break in PR body)
- CLA: strengthen entity definition with successor/assignee clause
- CLA workflow: store signatures on dedicated branch, add owner to allowlist
- CONTRIBUTING: clarify approval means the "approved" label, reword CCLA
- Bug template: allow "running from source" for developer contributors
2026-05-16 16:08:16 +08:00

5.2 KiB

Contributing to SnapOtter

Thanks for your interest in contributing. This guide covers how to participate, what we accept, and how to set up your development environment.

Ways to Contribute

Issues (no setup required)

  • Bug reports - Something broken? Open a bug report with reproduction steps.
  • Feature requests - Have an idea? Open a feature request describing the problem it solves.

Code (requires CLA)

We accept pull requests for:

Type Process
Bug fixes Open a PR directly (link the issue if one exists)
New translations Open a PR directly (see Translation Guide)
Documentation improvements Open a PR directly
Test coverage improvements Open a PR directly
New tools or features Open an issue first, wait for a maintainer to add the approved label before writing code
Refactors or architecture changes Open an issue first, wait for a maintainer to add the approved label before writing code

What We Will Not Accept

  • Changes to CI/CD workflows, release config, or linter/compiler config
  • PRs without a signed Contributor License Agreement
  • PRs over 400 lines of change (break large work into smaller PRs)
  • Features that were not discussed and approved in an issue first
  • Changes to packages/ai/ without prior discussion

Contributor License Agreement

Before we can merge your first PR, you must sign our Individual CLA. This is a one-time requirement.

Why: SnapOtter is dual-licensed (AGPLv3 + commercial). The CLA grants us the right to distribute your contributions under both licenses. You retain full copyright ownership of your work.

How: When you open your first PR, the CLA Assistant bot will comment with a link. Click it, review the agreement, and sign with your GitHub account. Takes 30 seconds.

If you are contributing on behalf of your employer and your employer retains IP rights over your work, contact contact@snapotter.com to arrange a Corporate CLA before submitting.

Development Setup

Prerequisites

  • Node.js 22+
  • pnpm 9+
  • Python 3.11+ (only for AI tools)
  • Docker (optional, for full integration testing)

Getting Started

# Fork and clone
git clone https://github.com/<your-username>/snapotter.git
cd snapotter

# Install dependencies
pnpm install

# Start dev servers (web on :1349, API on :13490)
pnpm dev

Running Checks

Before submitting a PR, ensure all checks pass locally:

pnpm lint          # Biome lint + format check
pnpm typecheck     # TypeScript across monorepo
pnpm test          # Vitest unit + integration tests

To run a single test file:

pnpm vitest run tests/unit/my-test.test.ts
pnpm vitest run tests/integration/my-test.test.ts

Code Style

  • Biome handles formatting and linting (double quotes, semicolons, 2-space indent)
  • Pre-commit hook runs biome check --write on staged files automatically
  • If the linter complains, fix the code (do not modify Biome config)
  • ES modules everywhere (import/export, .js extensions on relative imports)
  • Conventional commits: feat:, fix:, refactor:, docs:, test:, chore:

Architecture Quick Reference

A tool lives in three places sharing a toolId string:

  1. Shared metadata - packages/shared/src/constants.ts (TOOLS array)
  2. API route - apps/api/src/routes/tools/<toolId>.ts (uses createToolRoute factory)
  3. Frontend settings - apps/web/src/components/tools/<toolId>-settings.tsx

For full architecture details, see the Developer Guide.

Pull Request Process

  1. Fork the repo and create a branch from main (feat/my-feature or fix/issue-123)
  2. Make your changes in focused, reviewable commits using conventional commits
  3. Add or update tests for your changes
  4. Run pnpm lint && pnpm typecheck && pnpm test locally
  5. Open a PR against main and fill out the template
  6. Sign the CLA if prompted
  7. Wait for CI to pass and a maintainer to review

Review Expectations

  • We aim to respond to PRs within 7 days
  • Small, focused PRs get reviewed faster
  • If you have not heard back in 7 days, leave a comment pinging the thread
  • We may request changes, suggest a different approach, or close the PR if it does not align with project direction

After Your PR is Merged

Your contribution will be included in the next release and credited in the changelog.

Security Vulnerabilities

Do not open a public PR or issue for security vulnerabilities. Report them privately through GitHub Security Advisories or email contact@snapotter.com. See SECURITY.md for full details.

Questions?