chore(github): tighten bug report intake#4954
Conversation
Signed-off-by: San Dang <san1201.bkhn@gmail.com>
📝 WalkthroughWalkthroughGitHub issue templates are updated to encourage agent-first diagnostics. The bug report template now requires reporters to run an agent and submit diagnostic output before filing issues. Contact links redirect questions to Discussions and troubleshooting help to documentation. ChangesAgent-First Issue Handling
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 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)
Comment |
E2E Advisor RecommendationRequired E2E: None Full advisor summaryE2E Recommendation AdvisorBase: Required E2E
Optional E2E
New E2E recommendations
|
E2E Scenario Advisor RecommendationRequired scenario E2E: None Full scenario advisor summaryE2E Scenario AdvisorBase: Required scenario E2E
Optional scenario E2E
Relevant changed files
|
PR Review AdvisorFindings: 0 needs attention, 3 worth checking, 0 nice ideas Review findings🛠️ Needs attention
🔎 Worth checking
🌱 Nice ideas
Consider writing more tests for
Since last review detailsCurrent findings:
This is an automated advisory review. A human maintainer must make the final merge decision. |
|
@sandl99 for Bug Reports, can you also make sure the template:
|
Summary
This PR tightens NemoClaw bug report intake so maintainers get enough context to triage issues. Bug reports now require debug output and logs; agent diagnostics are encouraged but optional.
Bug reports now require:
DescriptionReproduction StepsEnvironmentDebug OutputLogsFeature request and documentation issue templates are unchanged.
Changes
Agent Diagnosticfield to bug reports.Debug OutputandLogsrequired in bug reports.Type of Change
Verification
npx prek run --all-filespassesnpm testpassesnpm run docsbuilds without warnings (doc changes only)Passing verification:
ruby -e 'require "yaml"; ARGV.each { |f| YAML.load_file(f); puts "ok #{f}" }' .github/ISSUE_TEMPLATE/bug_report.yml .github/ISSUE_TEMPLATE/feature_request.yml .github/ISSUE_TEMPLATE/doc_issue.yml .github/ISSUE_TEMPLATE/config.ymlnpx prek run --files .github/ISSUE_TEMPLATE/bug_report.yml .github/ISSUE_TEMPLATE/feature_request.yml .github/ISSUE_TEMPLATE/doc_issue.yml .github/ISSUE_TEMPLATE/config.ymlFull
npx prek run --all-fileswas attempted but still fails in unrelated existing CLI tests. No runtime code or tests are changed by this PR.Signed-off-by: San Dang san1201.bkhn@gmail.com