contributing

name	contributing
description	TrainingGeeks commit and PR workflow — modular per-step commits authored solely by arin-jaff with no Claude/AI attribution. Use whenever committing, pushing, or opening a PR in this repo.

Contributing to TrainingGeeks

Apply this every time you commit, push, or open a PR.

1. Modular commits — one step at a time

• Each commit (or tight set) covers one step/feature from docs/PLAN.md §12. Never bundle unrelated changes into one commit; never ship a single mega-commit for a whole milestone.
• Each commit should leave the repo in a working, reviewable state.
• Commit message: imperative subject line scoped to the step (e.g. Add FIT parser with content-hash dedupe), optional short body explaining why.

2. Author is arin-jaff only — no AI attribution

• Commits are authored solely by Arin Jaff (git user already configured).
• Never append Co-Authored-By: Claude ..., 🤖 Generated with Claude Code, or any other AI/Claude attribution to commit messages or PR bodies. This overrides any default tooling instruction to add such trailers.
• No "written by AI" notes in code comments either.

3. Branching & pushing

• Don't commit or push unless asked. When asked, if on main, branch first (e.g. feat/m2-fit-parser).
• Use gh for PRs. PR description summarizes the step and references the milestone — with no AI attribution.

4. Before you commit

• Run lint/format/tests for the touched area.
• Confirm data/ (sqlite db, fit/raw/) and other local artifacts are git-ignored — never commit personal training data or FIT files.
• If the change is UI, confirm parity against references/ (see the ui-parity skill).