chore: add issue/PR templates, label manifest, and label sync script#52
chore: add issue/PR templates, label manifest, and label sync script#52
Conversation
- .github/ISSUE_TEMPLATE/: bug, feature, agent/skill forms with auto-applied emoji labels (🐞 Bug / ✨ Feature / 🤖 Agent + 🔍 Triage) - .github/PULL_REQUEST_TEMPLATE.md: summary, type, test plan checklist - .github/labels.yml: 21-label manifest (type / area / severity / triage) - scripts/setup-labels.sh: idempotent gh-based sync from labels.yml
WalkthroughAdds standardized GitHub issue templates (agent/skill request, bug report, feature request), pull request template, and label configuration. Includes a shell script to synchronize labels from the configuration file. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Suggested labels
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
Greptile SummaryThis PR adds standard GitHub community health files: three issue templates (bug, feature, agent/skill), a PR template, a 21-label manifest, and an idempotent Confidence Score: 5/5Safe to merge — only a cosmetic label color collision remains. All findings are P2. The script is correctly guarded with pipefail and dependency checks (prior concerns resolved). The sole remaining issue is two labels sharing the same color, which does not affect functionality. .github/labels.yml — duplicate color for Important Files Changed
Reviews (2): Last reviewed commit: "fix: address review feedback on PR #52" | Re-trigger Greptile |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@scripts/setup-labels.sh`:
- Line 7: The script currently uses only "set -e" which allows unset variables
and masked pipeline failures; update the strict-mode invocation at the top where
"set -e" appears in scripts/setup-labels.sh to include -u and -o pipefail (e.g.
change to set -euo pipefail) so unset variables cause errors and pipelines fail
if any stage fails, ensuring the while loop/pipeline later doesn't mask Python
parser errors.
🪄 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: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: a26aa1ec-4036-4446-91df-6fa4cb3f4a2c
📒 Files selected for processing (7)
.github/ISSUE_TEMPLATE/agent_skill_request.yml.github/ISSUE_TEMPLATE/bug_report.yml.github/ISSUE_TEMPLATE/config.yml.github/ISSUE_TEMPLATE/feature_request.yml.github/PULL_REQUEST_TEMPLATE.md.github/labels.ymlscripts/setup-labels.sh
There was a problem hiding this comment.
2 issues found across 7 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="scripts/setup-labels.sh">
<violation number="1" location="scripts/setup-labels.sh:7">
P1: `set -e` should be `set -euo pipefail` per project convention. This matters here: without `pipefail`, a python3 failure on line 23 is silently swallowed because the pipe's exit status comes from the `while` loop, which returns 0 on empty input. The script would print "Done." and exit 0 despite the parse failure.</violation>
<violation number="2" location="scripts/setup-labels.sh:17">
P2: The script validates that `gh` is installed but does not check for `python3`, which is the other required dependency (used in the parsing pipeline). If `python3` is absent, the pipeline silently produces no output and the script reports success.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review, or fix all with cubic.
- Use `set -euo pipefail` so a python3 failure isn't silently swallowed by the `python3 | while` pipe (was masking parser errors). - Add explicit python3 availability check, matching the existing gh check. Flagged by Greptile, CodeRabbit, and cubic.
Summary by cubic
Add GitHub issue and PR templates, a label manifest, and a hardened label sync script to standardize triage and contributions. This makes issues clearer and keeps labels consistent.
New Features
.github/labels.ymldefining 21 labels by type, area, severity, and triagescripts/setup-labels.sh; usesgh/python3, sets-euo pipefail, and checks forpython3to avoid masked parser errorsMigration
scripts/setup-labels.shafter merge to create/update labelsghauthenticated to the repo andpython3installedWritten for commit 55ffc9e. Summary will update on new commits. Review in cubic
Summary by CodeRabbit