Skip to content

v0.1.1 - #2

Merged
Ret2Hell merged 4 commits into
mainfrom
dev
Mar 17, 2026
Merged

Ret2Hell merged 4 commits into
mainfrom
dev

Conversation

@Ret2Hell

@Ret2Hell Ret2Hell commented Mar 17, 2026 •

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Chores

    • Added CI workflow to run tests and linters on pushes and PRs.
    • Added linting configuration and pinned lint tool version.
    • Updated runtime port to 3000 and adjusted health-check target.
  • Bug Fixes

    • Improved request logging safety and hardened gzip request handling to surface invalid payloads.

was using wrong host & port :)
@coderabbitai

coderabbitai Bot commented Mar 17, 2026 •

Copy link
Copy Markdown
ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: 362a5d42-3eeb-406f-9427-09f78fe6400e

📥 Commits

Reviewing files that changed from the base of the PR and between f203f2d and b98eb73.

📒 Files selected for processing (3)
  • .github/workflows/ci.yml
  • internal/server/middleware.go
  • mise.toml
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

Adds CI (GitHub Actions) and linting configuration, updates container port and healthcheck in the Dockerfile, introduces a golangci-lint tool entry, and makes small safer error-handling changes in server middleware.

Changes

Cohort / File(s) Summary
CI Workflow
​.github/workflows/ci.yml
Adds GitHub Actions workflow with test and lint jobs (checkout, setup Go with go.mod, caching; runs go vet/go test and golangci-lint).
Lint Configuration & Tooling
.golangci.yml, mise.toml
Adds GolangCI-Lint config enabling errcheck, govet, staticcheck and sets tool version golangci-lint = "2.11.3" in mise.toml.
Container Configuration
Dockerfile
Changes exposed port from 8080 to 3000 and updates healthcheck to 127.0.0.1:3000/healthz.
Server Middleware
internal/server/middleware.go
Safer extraction of request_id from context/locals; adjusts gzip decompression to explicitly close reader with error handling and return 400 on close errors.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A pipeline wakes, the checks align,
Lints hum softly, every rule in line,
Port three-thousand opens to the day,
Middlewares tidy in a careful way,
I hop and cheer — the build's on its way! 🥕✨

📝 Coding Plan
  • Generate coding plan for human review comments

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud

Copy link
Copy Markdown

@Ret2Hell
Ret2Hell merged commit 51ac7c9 into main Mar 17, 2026
4 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