Skip to content

Add Standardized GitHub Issue Templates (Closes #121)#124

Merged
SamXop123 merged 4 commits into
SamXop123:mainfrom
Nitya-003:main
Feb 19, 2026
Merged

Add Standardized GitHub Issue Templates (Closes #121)#124
SamXop123 merged 4 commits into
SamXop123:mainfrom
Nitya-003:main

Conversation

@Nitya-003

@Nitya-003 Nitya-003 commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

Description

This PR introduces structured GitHub Issue Templates to improve issue quality, streamline triaging, and enhance contributor onboarding for RigCrafter.

Previously, contributors were presented with a blank issue box, often resulting in incomplete reports and additional back-and-forth from maintainers. These templates ensure all necessary technical and contextual details are provided upfront.

Closes #121


Changes Made

Added .github/ISSUE_TEMPLATE/ directory containing:

config.yml

  • Keeps blank issues enabled
  • Provides guidance links to README and Discussions

bug_report.md

  • Structured fields for reproduction steps
  • Expected vs actual behavior
  • Environment details
  • Compatibility engine targeting
  • Screenshots/logs

feature_request.md

  • Problem statement
  • Proposed solution
  • Affected logic layer (UI / compatibility / suggestions / data)
  • Impact assessment

documentation_update.md

  • Structured improvements for README and technical documentation
  • Contributor clarity enhancement

Why This Matters

  • Improves issue clarity and completeness
  • Reduces maintainer triage time
  • Encourages professional open-source practices
  • Aligns repository with mature GitHub standards

Type of Change

  • Documentation
  • Repository Maintenance
  • Bug Fix
  • Feature Addition

Summary by CodeRabbit

Release Notes

  • Chores
    • Added standardized GitHub issue templates for bug reports, feature requests, and documentation updates.
    • Configured issue template settings to enable blank issues and provide quick access to documentation and discussion resources.

Added issue template configuration with contact links.
Added a bug report template for issue tracking.
Added a feature request template for GitHub issues.
This template allows users to suggest improvements to documentation, including README and guides.
@vercel

vercel Bot commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

@Nitya-003 is attempting to deploy a commit to the Dot_NotSam's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@coderabbitai

coderabbitai Bot commented Feb 19, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

This PR establishes standardized GitHub issue templates to improve issue reporting quality. Four files are added: a bug report template, feature request template, documentation update template, and template configuration with blank issue settings and contact links.

Changes

Cohort / File(s) Summary
GitHub Issue Templates
.github/ISSUE_TEMPLATE/bug_report.md, .github/ISSUE_TEMPLATE/feature_request.md, .github/ISSUE_TEMPLATE/documentation_update.md, .github/ISSUE_TEMPLATE/config.yml
Adds four template files: bug report template with steps to reproduce and environment details, feature request template with problem statement and impact sections, documentation update template with checklist for documentation areas, and config file enabling blank issues with contact links for documentation and general discussion.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 Hop, hop, templates take form,
Bug reports, features in sturdy form,
No more blank boxes causing despair,
Contributors now know what to share!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding standardized GitHub issue templates and references the closed issue #121.
Linked Issues check ✅ Passed All coding requirements from issue #121 are met: .github/ISSUE_TEMPLATE/ directory created with bug_report.md and feature_request.md templates containing specified fields.
Out of Scope Changes check ✅ Passed The PR includes an additional documentation_update.md template beyond the two specified in issue #121, though this aligns with the stated benefits and doesn't conflict with objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (2)
.github/ISSUE_TEMPLATE/bug_report.md (1)

40-45: Consider decoupling the checklist from internal source paths.

The backtick-quoted paths (lib/compatibility.ts, lib/suggestions.ts, lib/data.ts) are a helpful hint right now, but they will silently become misleading if any of those files are renamed or moved during a refactor. Use logical layer names only (matching the feature_request.md style) or keep paths as optional parenthetical hints that are easier to update.

♻️ Proposed change
 ## Affected Area
 - [ ] UI (Component Selector / Build Summary)
-- [ ] Compatibility Engine (`lib/compatibility.ts`)
-- [ ] Suggestion Engine (`lib/suggestions.ts`)
-- [ ] Data Layer (`lib/data.ts`)
+- [ ] Compatibility Engine
+- [ ] Suggestion Engine
+- [ ] Data Layer
 - [ ] Other:
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.github/ISSUE_TEMPLATE/bug_report.md around lines 40 - 45, The checklist in
the bug report template exposes brittle internal source paths
(`lib/compatibility.ts`, `lib/suggestions.ts`, `lib/data.ts`) that can become
misleading; update the checklist under "Affected Area" to use stable logical
layer names (e.g., "Compatibility Engine", "Suggestion Engine", "Data Layer")
only or move file paths into optional parenthetical hints (not the primary
label) so renames won’t break the guidance — modify the checkbox lines shown in
the diff accordingly to remove or demote the backtick-quoted paths.
.github/ISSUE_TEMPLATE/documentation_update.md (1)

29-30: Remove emoji from section heading for consistency across templates.

The 🎯 prefix on this heading is the only emoji used across all three templates' section headings. Keeping headings in plain text aligns with bug_report.md and feature_request.md.

♻️ Proposed change
-## 🎯 Why This Matters
+## Why This Matters
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.github/ISSUE_TEMPLATE/documentation_update.md around lines 29 - 30, Remove
the emoji from the "## 🎯 Why This Matters" heading so it matches the other
templates; replace that heading text with "## Why This Matters" (update the
heading string in the template where "## 🎯 Why This Matters" appears).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/ISSUE_TEMPLATE/bug_report.md:
- Around line 32-36: Add an "App Version" entry to the "## Environment" section
so reporters can include the app version or commit SHA; update the Environment
list to include a line like "App Version (e.g., v1.2.3 or commit SHA):" directly
under the existing items to ensure reproducible reports when fixes are deployed.

---

Nitpick comments:
In @.github/ISSUE_TEMPLATE/bug_report.md:
- Around line 40-45: The checklist in the bug report template exposes brittle
internal source paths (`lib/compatibility.ts`, `lib/suggestions.ts`,
`lib/data.ts`) that can become misleading; update the checklist under "Affected
Area" to use stable logical layer names (e.g., "Compatibility Engine",
"Suggestion Engine", "Data Layer") only or move file paths into optional
parenthetical hints (not the primary label) so renames won’t break the guidance
— modify the checkbox lines shown in the diff accordingly to remove or demote
the backtick-quoted paths.

In @.github/ISSUE_TEMPLATE/documentation_update.md:
- Around line 29-30: Remove the emoji from the "## 🎯 Why This Matters" heading
so it matches the other templates; replace that heading text with "## Why This
Matters" (update the heading string in the template where "## 🎯 Why This
Matters" appears).

Comment on lines +32 to +36
## Environment
- OS:
- Browser:
- Device:
- Screen Size (if UI related):

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Consider adding an App Version field to the Environment section.

Without a version or commit SHA, it can be difficult to reproduce or close a bug after a fix is deployed.

🛡️ Proposed addition
 ## Environment
 - OS:
 - Browser:
 - Device:
 - Screen Size (if UI related):
+- RigCrafter Version / Commit:
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.github/ISSUE_TEMPLATE/bug_report.md around lines 32 - 36, Add an "App
Version" entry to the "## Environment" section so reporters can include the app
version or commit SHA; update the Environment list to include a line like "App
Version (e.g., v1.2.3 or commit SHA):" directly under the existing items to
ensure reproducible reports when fixes are deployed.

@SamXop123 SamXop123 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! @Nitya-003

@SamXop123 SamXop123 merged commit 262aa23 into SamXop123:main Feb 19, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement GitHub Issue Templates for bug reports and feature requests

2 participants