Conversation
Adds private vulnerability reporting guidance, a standard code of conduct with maintainer enforcement contact, minimal CODEOWNERS for canonical paths, and routes security reports away from public issues in SUPPORT.md. Fixes conorbronsdon#255 Co-authored-by: Sharad. <sharadvc@users.noreply.github.com>
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewTip of the day💡 Did you know, you can start a comment with 'qodo' or '@qodo' to chat about any finding |
PR Summary by QodoAdd repository governance and security reporting policies
AI Description
Diagram
High-Level Assessment
Files changed (4)
|
conorbronsdon
left a comment
There was a problem hiding this comment.
Codex review, with a GPT-5.6 Sol reviewer, at Conor's request.
Thanks, @sharadvc, for filling out the governance files and review ownership. CODEOWNERS is well scoped, and the SUPPORT linkage is sensible. The private reporting paths need to be concrete before #255 can close:
- The private-advisory route has not been confirmed enabled; the PR's own checkbox is still open.
- GitHub 'private communication options' and a private channel supplied later are not usable confidential contacts for security or conduct reports.
- The acknowledgment/update deadlines and disclosure commitments need a maintainer decision; no such decision is recorded on the issue.
Please keep this open until those contacts and commitments are settled. CODEOWNERS could be split into its own small PR and landed independently. SUPPORT should link SECURITY only once a working reporting path exists. The files merge cleanly and integrated tests pass; the blocker is an actionable reporting policy, not formatting.
|
Follow-up from Codex: the independently usable CODEOWNERS file is included in #287 with explicit Sharad co-credit. That lets the review-ownership part land now while this PR remains open for confirmed confidential contacts and response expectations. Thanks for providing that separable improvement; the security/conduct policies and SUPPORT linkage are not included in #287. |
Integrate PRs conorbronsdon#177, conorbronsdon#194, conorbronsdon#278, conorbronsdon#279, conorbronsdon#281, conorbronsdon#282, and conorbronsdon#286 with their original heads as parents. Carry the useful stock-reaction naming and evidence wording from conorbronsdon#178 while leaving its corpus preprocessing for revision. Include only the independently useful CODEOWNERS file from conorbronsdon#276. Complete body-timeout error context, full test-suite retention, bounded corpus downloads, committed-hash checks, title-case scope, push-safe CI concurrency, and the final self-scan snapshot. Co-authored-by: Francisco Quinteros <155692279+JavierQuinan@users.noreply.github.com> Co-authored-by: Sharad <sharadvc@users.noreply.github.com> Co-authored-by: anujsingh-cse <anujsingh0543@gmail.com> Co-authored-by: Emirhan Karaca <emirhan.karaca@std.yildiz.edu.tr> Co-authored-by: Codex <codex@openai.com>
|
Triage update: I checked the repository setting directly through GitHub's private-vulnerability-reporting endpoint; it currently reports that the feature is disabled. That confirms the PR's unchecked setup item is a real maintainer-side blocker, not something the contributor can resolve in this branch. I'm leaving the PR open and changes-requested. I have not changed the repository setting. Once a maintainer enables a working confidential route and decides the response/disclosure commitments, the policy text can be revised against those facts. |
|
Reviewed the requested comment and pushed verified repairs, preserving the original contribution:
Changes still required before merge: CODE_OF_CONDUCT.md lines 64-66 correctly records that no private maintainer contact has been designated. A maintainer must supply that contact and approve adoption/commitments, then remove the draft-only wording and obtain a review of the resulting commit. This PR remains open and must not merge yet. Validation: Independent, tool-isolated reviews of the final SHA completed through authenticated The PR description now includes the remaining adoption checklist and no longer claims to close #255. |
|
Deferring this proposal for now. We are not adopting the Code of Conduct or establishing a private conduct-reporting process at this time. The existing security-reporting policy remains in place. Thank you for preparing the draft; this can be reopened when we are ready to take on the reporting and enforcement commitments. |
Summary
Prepares Contributor Covenant 2.1 as a draft pending a maintainer-designated private conduct-reporting contact. The proposal explicitly distinguishes GitHub Support's abuse-reporting process from a private route to project maintainers.
Current main already contains SECURITY.md, SUPPORT.md's security-reporting link, and CODEOWNERS. The branch now incorporates main and retains those files exactly, resolving the stale conflicts and removing the earlier proposed security-response deadlines. The original contribution and commit history are preserved.
Private vulnerability reporting is enabled. The remaining blocker is the private conduct-reporting contact and policy-adoption decision. This PR must remain open and unmerged until those are settled.
Related to #255; this draft does not close it.
Validation
npm test: 20 suites passed, zero failed.npm run self-scan:checkpassed.git diff --checkpassed; no unresolved merge conflicts.Before merge