Add what a public repository is asked for: security policy, templates, CI matrix - #16
Merged
Conversation
…, CI matrix Nothing here changes the SDK. It is the paperwork a repository needs once strangers can open issues against it. SECURITY.md routes vulnerability reports to GitHub's private advisory flow instead of a public issue, and names what counts as one HERE: a secret reaching a log or an error string (a webhook path is full access to a portal), a way past the application-token check or the DOMAIN validation, client_secret escaping the oauth package. It also names what does not, so ACCESS_DENIED from a portal does not arrive as a security report. The issue templates carry the two warnings that would otherwise be learned the hard way: do not paste a webhook URL into a public issue, and per-method wrappers are not something this SDK adds — every method is callable by name already. The feature template asks for the workaround code, because that is the evidence a gap is real; the whole of v0.2 was found that way. CI now runs twice: the minimum Go from go.mod, which is what someone on an older toolchain actually builds with, and stable, where a new vet check lands first. The minimum is read from go.mod rather than repeated as a literal, so the two cannot drift. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Nothing here changes the SDK. It is the paperwork a repository needs once
strangers can open issues against it.
SECURITY.mdroutes vulnerability reports to GitHub's private advisory flowinstead of a public issue, and names what counts as one here: a secret
reaching a log or an error string (a webhook path is full access to a portal), a
way past the
application_tokencheck or theDOMAINvalidation,client_secretescaping theoauthpackage. It also names what does not, soACCESS_DENIEDfrom a portal does not arrive as a security report.Issue templates carry the two warnings that would otherwise be learned the
hard way: do not paste a webhook URL into a public issue, and per-method
wrappers are not something this SDK adds — every method is callable by name
already. The feature template asks for the workaround code, because that is the
evidence a gap is real; the whole of v0.2 was found exactly that way.
A PR template with the four CI checks and a reminder that tests here explain
why they exist and assert behaviour rather than shape.
CI now runs twice: the minimum Go from
go.mod, which is what someone on anolder toolchain actually builds with, and
stable, where a new vet check or achanged stdlib default lands first. The minimum is read from
go.modratherthan repeated as a literal, so the two cannot drift.
Merges cleanly with #15 in either order; CI checked on the combined result.
🤖 Generated with Claude Code