Skip to content

fix: correct indentation in worker/hono-app.ts#1447

Merged
jaypatrick merged 1 commit intomainfrom
claude/fix-ci-on-main
Mar 28, 2026
Merged

fix: correct indentation in worker/hono-app.ts#1447
jaypatrick merged 1 commit intomainfrom
claude/fix-ci-on-main

Conversation

@Claude
Copy link
Copy Markdown
Contributor

@Claude Claude AI commented Mar 28, 2026

Description

CI was failing on main due to a format check error in worker/hono-app.ts. Line 585 had 2 spaces of indentation instead of the required 4 spaces.

Changes

  • Fixed indentation at worker/hono-app.ts:585 from 2 to 4 spaces

Testing

  • CI passes

Zero Trust Architecture Checklist

Required for every PR touching worker/ or frontend/.
Check each item that applies. If an item doesn't apply, check it and note "N/A".

Worker / Backend

  • Every handler verifies auth before executing business logic — N/A (formatting-only change)
  • CORS origin allowlist enforced (not *) on write/authenticated endpoints — N/A (formatting-only change)
  • All secrets accessed via Worker Secret bindings (not [vars]) — N/A (formatting-only change)
  • All external inputs Zod-validated before use — N/A (formatting-only change)
  • All D1 queries use parameterized .prepare().bind() (no string interpolation) — N/A (formatting-only change)
  • Security events emitted to Analytics Engine on auth failures — N/A (formatting-only change)

Frontend / Angular

  • Protected routes have functional CanActivateFn auth guards — N/A (backend-only change)
  • Auth tokens managed via Clerk SDK (not localStorage) — N/A (backend-only change)
  • HTTP interceptor attaches ****** (no manual token passing) — N/A (backend-only change)
  • API responses validated with Zod schemas before consumption — N/A (backend-only change)

If this PR does not touch worker/ or frontend/, the ZTA checklist is not required.

Fixes CI format check failure on main branch.
The line had 2 spaces instead of required 4 spaces of indentation.

Agent-Logs-Url: https://github.com/jaypatrick/adblock-compiler/sessions/e061edbe-3a89-4a79-bc3b-de49911b5235

Co-authored-by: jaypatrick <1800595+jaypatrick@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

@jaypatrick jaypatrick marked this pull request as ready for review March 28, 2026 08:23
@jaypatrick jaypatrick merged commit 4c9215b into main Mar 28, 2026
31 checks passed
@jaypatrick jaypatrick deleted the claude/fix-ci-on-main branch March 28, 2026 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working deployment Deployment tasks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants