Skip to content

Add bug report and feature request issue templates - #988

Merged
TheGreatAxios merged 3 commits into
mainfrom
cl-7890-add-bug-report-and-feature-request-issue-templates
Sep 13, 2026
Merged

Add bug report and feature request issue templates#988
TheGreatAxios merged 3 commits into
mainfrom
cl-7890-add-bug-report-and-feature-request-issue-templates

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Closes CL-7890.

Summary

.github/ had PULL_REQUEST_TEMPLATE.md and workflows/ and nothing else, so a new issue opened as an empty box. What arrived was whatever the reporter thought to include — usually not the version, platform, provider, or model a triager needs before starting.

What is here

bug-report.yml — an issue form, so the fields are required rather than suggested: what happened, what was expected, steps to reproduce, corbits --version, OS/arch, install method, provider and model, relevant output, and whether it reproduces on a clean config.

Provider and model are required but answerable with "not inference" — the field only costs something when it matters, and inference bugs are unactionable without it. The clean-config dropdown is the single most useful thing a reporter can tell us, so it is required too.

feature-request.md — markdown, deliberately. A form here is friction on someone doing us a favour. Asks for the problem before the solution, the current workaround, and what they already ruled out.

config.yml — routes security reports to private disclosure, plus links to docs/ and CONTRIBUTING.md.

Two judgement calls

Blank issues stay enabled. I disabled them first, then checked: has_discussions is false on this repo. Disabling blank issues with no Discussions leaves anyone with a question pushed into the wrong template. There is a comment in config.yml saying to turn it off if Discussions is ever enabled.

No Discussions link. My first draft linked to /discussions, which would have been a dead 404 for the same reason.

Constraint honored

Templates link to CONTRIBUTING.md rather than restating any of it. Nothing here describes commit or PR conventions, which are changing under #985 and would have gone stale immediately.

The bug report also warns against pasting credentials or ~/.corbits/settings.json contents — worth having on a tool whose config file holds provider API keys.

Testing

Both YAML files parse. Rendering in the GitHub new-issue picker cannot be verified until the branch is on the default branch — that Outcome item stays unticked.

@linear-code

linear-code Bot commented Sep 13, 2026

Copy link
Copy Markdown

CL-7890

.github had a pull-request template and nothing for issues, so a new
issue opened as an empty box and arrived without the version, platform,
provider, or model needed to start on it.

The bug report is an issue form, so version, platform, install method,
reproduction, and whether it reproduces on a clean config are required
rather than suggested. Provider and model are required too, answerable
with "not inference" — the field only costs something when it matters.
The feature request stays markdown: a form there is friction on someone
doing us a favour.

Blank issues stay enabled. Discussions is not turned on for this
repository, so disabling them would leave anyone with a question pushed
into the wrong template.

Templates link to CONTRIBUTING.md rather than restating it, and route
security reports to private disclosure.

CL-7890
@TheGreatAxios
TheGreatAxios force-pushed the cl-7890-add-bug-report-and-feature-request-issue-templates branch from 076a864 to 07cc82a Compare September 13, 2026 18:02
@TheGreatAxios TheGreatAxios changed the title feat(github): add bug report and feature request issue templates Add bug report and feature request issue templates Sep 13, 2026
Provider and model was marked required while its own description told
non-inference reporters to type "not inference" to get past it. GitHub
issue forms have no conditional-required, so that combination just forces
boilerplate. It is optional now, with the description doing the asking.

Install method comes off the required list for the same reason: it is
rarely what unblocks a triager, and gating submission on it buys noise.

"What happened" and the reproduction box were collecting the same
paragraph twice; the first now asks only for the observed behavior.

Four fields still gate submission: what happened, what was expected,
reproduction, and version — plus platform and the clean-config answer.

CL-7890
@TheGreatAxios

Copy link
Copy Markdown
Collaborator Author

Audit — Approve

Fit: this PR fills a real gap (.github/ had no issue templates, so new issues arrived as an empty box). The bug-report form requires what triage needs (repro steps, version, platform, clean-config signal) and the feature-request stays markdown to keep friction low on someone doing us a favour. Security routing to private disclosure plus the credentials warning is the right call for a tool whose config holds provider API keys. bug label on the form matches an existing repo label. Blank-issues-left-on is justified while Discussions is disabled (has_discussions: false).

One fix applied (pushed to this branch as 64527c7c): feature-request.md frontmatter set labels: ["feature"], but the repo has no feature label (has bug, enhancement, …) so the auto-label silently no-ops. Changed to labels: ["enhancement"]; frontmatter re-parsed to confirm.

Stale-body-text note for the author: the PR body says provider/model are "required but answerable" — the form as merged makes provider-model optional (no required: true, description says leave blank for non-inference bugs), which is the better call. The body text should be corrected to match.

@TheGreatAxios
TheGreatAxios merged commit 3f22fc9 into main Sep 13, 2026
9 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.

1 participant