Skip to content

fix(logging): suppress health endpoint request logging#1959

Open
kilo-code-bot[bot] wants to merge 1 commit intomainfrom
fix/suppress-health-request-logging
Open

fix(logging): suppress health endpoint request logging#1959
kilo-code-bot[bot] wants to merge 1 commit intomainfrom
fix/suppress-health-request-logging

Conversation

@kilo-code-bot
Copy link
Copy Markdown
Contributor

@kilo-code-bot kilo-code-bot bot commented Apr 3, 2026

Summary

  • Suppress /health endpoint request logging in both the KiloClaw worker and the gastown container control server — these high-frequency health poll requests flood logs with noise, making it harder to spot meaningful requests.
  • Uses a QUIET_PATHS Set for O(1) lookups and easy extensibility, following the same pattern introduced in fix(logging): exclude noisy health/telemetry paths from request logs #1948.

Verification

  • pnpm typecheck passes in kiloclaw/ (tsgo --noEmit)
  • pnpm lint passes in kiloclaw/ (oxlint, 0 warnings, 0 errors)
  • pnpm typecheck passes in cloudflare-gastown/container/ (tsc --noEmit)
  • pnpm lint passes in cloudflare-gastown/container/ (oxlint, 0 warnings, 0 errors)
  • oxfmt --check passes on both changed files

Visual Changes

N/A

Reviewer Notes

  • The QUIET_PATHS pattern is intentionally a named constant (not inline) so future noisy paths can be added in one place.
  • The KiloClaw worker checks url.pathname (already computed) against the set; the control-server uses c.req.path (Hono's parsed path). Both are consistent with how the rest of each middleware already accesses the path.
  • The KiloClaw worker's /health route is a public route handled before auth — it never reaches the catch-all proxy, so only the logRequest middleware needs the suppression.

Built for Mark by Kilo for Slack

/health requests fire very frequently (health polls) and flood worker
logs with noise. Add a QUIET_PATHS set to skip request logging for
these high-frequency paths in both the KiloClaw worker and the gastown
container control server.
@kilo-code-bot
Copy link
Copy Markdown
Contributor Author

kilo-code-bot bot commented Apr 3, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • cloudflare-gastown/container/src/control-server.ts
  • kiloclaw/src/index.ts

Reviewed by gpt-5.4-20260305 · 500,978 tokens

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