pull_request_template.md

What

Why

Closes #

How

Checklist

Code

• Tests pass (npm test)
• Linter is clean (npm run check)
• No new any introduced
• No dead code or commented-out blocks
• All other style rules per CODING_STYLE.md

Scope

• This PR addresses one concern (not a mix of bug fixes, refactors, and features)
• Cross-module changes are documented in the description

Documentation

• Inline comments explain why, not what
• Relevant docs updated if behaviour changed

Tests

• New behaviour is covered by tests
• If a new endpoint was added: handler integration test included
• If a new service method was added: unit test with stubbed repo included

Git

• Commits follow convco / Conventional Commits
• Commits are signed (strongly encouraged — see CONTRIBUTING.md §6)
• Branch name follows <type>/<scope>/<description> convention