chore: sync pull request template format

[peteski22]

Description

Syncs mcpd's PR template to the checkbox-based format used in sibling repos, while keeping mcpd-specific validation commands.

Summary by CodeRabbit

• Documentation
  • Updated the pull request template with a clearer structure and revised wording.
  • Replaced emoji-based PR type options with checkbox categories.
  • Refreshed the checklist to include linting and testing steps.
  • Simplified the AI usage section with clearer selection options and added fields for tool details and extra notes.

[coderabbitai]

Walkthrough

The PR restructures the GitHub pull request template, replacing emoji-based PR type bullets with checkboxes, updating checklist wording to include lint/test commands, and reworking the AI usage section into a checkbox selector with new fields for AI model/tool and additional details.

Changes

Pull request template update

Layer / File(s)	Summary
Template restructure .github/pull_request_template.md	PR type changed from emoji bullets to checkbox categories (including "Infrastructure / CI"); relevant issues heading reworded; checklist updated with make lint/make test guidance; AI Usage section replaced with checkbox selector and new "AI Model/Tool used" and additional-details fields, with updated reviewer-response note.

Possibly related PRs

• mozilla-ai/mcpd#267: Originally added the same .github/pull_request_template.md file that this PR restructures.

Suggested reviewers: dpoulopoulos

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description is too brief and omits most required template sections, including PR type, issues, checklist, and AI usage details.	Expand the description to cover all template sections, especially PR Type, Relevant issues, Checklist, and AI Usage Information.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly summarises the template sync and matches the main change.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/pr-template-sync

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/pull_request_template.md:
- Around line 18-25: The PR checklist in the pull request template repeats work
by listing both lint and test commands even though make test already covers
lint. Update the checklist item in the pull request template to reference make
test alone, or explicitly state that it includes lint, so contributors are not
instructed to run redundant checks.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 49bafbd9-b6f6-4481-a988-1c057fa7b511

📥 Commits

Reviewing files that changed from the base of the PR and between c832a36 and 27ba95b.

📒 Files selected for processing (1)

• .github/pull_request_template.md