Repository navigation
CodeRabbit: repository review configuration #9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
3 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,43 @@ | ||
| # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json | ||
| language: en-US | ||
| tone_instructions: "Concise and technical. No praise or summaries of unchanged code. Lead with security and correctness; mark style points as nitpicks." | ||
| reviews: | ||
| profile: chill | ||
| poem: false | ||
| in_progress_fortune: false | ||
| auto_review: | ||
| enabled: true | ||
| drafts: false | ||
| ignore_usernames: ["dependabot[bot]"] | ||
| path_filters: | ||
| - "!go.sum" | ||
| - "!dist/**" | ||
| - "!**/*.png" | ||
| path_instructions: | ||
| - path: "internal/verify/**" | ||
| instructions: | | ||
| Signature and token verification; treat every change as security-sensitive. | ||
| - Secret-dependent comparisons must be constant time (hmac.Equal, crypto/subtle on fixed-size digests). Flag ==, bytes.Equal, early returns or length checks that leak secret or signature data through timing. | ||
| - The HMAC is computed over the raw body bytes exactly as received: no trimming, decoding, re-encoding, JSON parsing or normalisation before verification. | ||
| - Error strings are logged: they may name headers but must never contain secrets, signatures, computed MACs or body content. | ||
| - Every check fails closed: a missing, repeated or malformed header, an unknown encoding or an unresolved secret is a rejection, never a pass. | ||
| - New parsing paths need unit tests and fuzz coverage (FuzzHMAC). | ||
| - path: "internal/gateway/**" | ||
| instructions: | | ||
| - The body is read through http.MaxBytesReader before any verification, and only bytes that passed every check are forwarded, unchanged. | ||
| - Only allowlisted headers (pass_headers) and configured set_headers reach the upstream; sender credentials must never be forwarded. | ||
| - Nothing is forwarded for a not-ready source or a failed check. Retries go only to transport errors and 5xx. | ||
| - Check timeouts, bounded reads of upstream responses, goroutine and connection leaks, and graceful shutdown (drain_delay, shutdown_timeout). | ||
| - Logs and metrics never carry secrets, signatures or bodies; metric labels stay low-cardinality (source names, fixed outcomes). | ||
| - path: "internal/config/**" | ||
| instructions: | | ||
| Unknown keys stay errors. Secrets are references (env or file), never literals; validation errors name the reference, never the value. New keys need defaults, validation and an entry in docs/configuration.md. | ||
| - path: ".github/workflows/**" | ||
| instructions: | | ||
| Actions pinned to full commit SHAs with a version comment; top-level permissions read-only, write scopes per job only where needed; no pull_request_target that checks out or runs PR code; no untrusted input interpolated into run scripts. | ||
| - path: "examples/**" | ||
| instructions: | | ||
| Images pinned by version and digest; no real secrets; every documented command must work as written. | ||
| tools: | ||
| golangci-lint: | ||
| enabled: false | ||
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.