format step (#114)

* format step

* apply format

Author: owens1127

.github/workflows/static-checks.yml

@@ -34,6 +34,19 @@ jobs:
             - name: Lint
               run: bun lint
 
+    format:
+        runs-on: ubuntu-latest
+        steps:
+            - uses: actions/checkout@v4
+            - uses: oven-sh/setup-bun@v1
+
+            - name: Bun Install
+              run: |
+                  bun install --frozen-lockfile
+
+            - name: Format
+              run: bun format --check
+
     verify-open-api-doc:
         runs-on: ubuntu-latest
         steps:

src/core/RaidHubRouter.ts

@@ -48,9 +48,7 @@ export class RaidHubRouter implements IRaidHubRoute {
     }
 
     getFullPath(child: IRaidHubRoute): string {
-        const path = this.routes.find(({ route }) =>
-            normalizeRoute(route).includes(child)
-        )?.path
+        const path = this.routes.find(({ route }) => normalizeRoute(route).includes(child))?.path
         if (!path) throw new Error("Child not found")
 
         return (this.parent ? this.parent.getFullPath(this) : "") + path

src/integrations/bungie/client.ts

@@ -18,7 +18,7 @@ const inFlightRequests = new Map<string, Promise<unknown>>()
 export const bungiePlatformHttp = (opts: { ttl: number }) => ({
     fetch: async <T>(config: BungieFetchConfig) => {
         const cacheKey = config.url.toString()
-        
+
         // Check cache first
         if (inMemoryCache.has(cacheKey)) {
             return inMemoryCache.get(cacheKey)!.data as T
@@ -82,9 +82,12 @@ export const bungiePlatformHttp = (opts: { ttl: number }) => ({
                         cause: body
                     })
                 } else {
-                    throw new Error(`Invalid response (${response.status}): ${response.statusText}`, {
-                        cause: body
-                    })
+                    throw new Error(
+                        `Invalid response (${response.status}): ${response.statusText}`,
+                        {
+                            cause: body
+                        }
+                    )
                 }
             }
         })()

src/integrations/clickhouse/client.ts

@@ -1,4 +1,4 @@
-import { Logger, LogFields } from "@/lib/utils/logging"
+import { LogFields, Logger } from "@/lib/utils/logging"
 import {
     ClickHouseLogLevel,
     ErrorLogParams,

src/middleware/duration-metrics.ts

@@ -3,31 +3,32 @@ import { RequestHandler } from "express"
 import { ParamsDictionary, Query } from "express-serve-static-core"
 import { RaidHubLocals } from "./types"
 
-
 /**
  * Factory function to create duration metrics middleware.
  * Tracks request duration in Prometheus with path and status code labels.
- * 
+ *
  * @param path The route path pattern (e.g., "/player/:membershipId")
  */
 export const durationMetrics = <
     P extends ParamsDictionary = ParamsDictionary,
     ResBody = unknown,
     ReqBody = unknown,
     ReqQuery extends Query = Query
->(path: string): RequestHandler<P, ResBody, ReqBody, ReqQuery, RaidHubLocals> => {
+>(
+    path: string
+): RequestHandler<P, ResBody, ReqBody, ReqQuery, RaidHubLocals> => {
     return (_, res, next) => {
         const start = Date.now()
         res.locals._startTime = start
-        
+
         res.once("finish", () => {
             const responseTimeInMs = Date.now() - start
             res.locals._duration = responseTimeInMs
 
             const code = res.statusCode.toString()
             httpRequestTimer.labels(path, code).observe(responseTimeInMs)
         })
-        
+
         next()
     }
 }

src/middleware/region-metrics.ts

@@ -1,9 +1,7 @@
-import {
-    httpRequestCountByRegion
-} from "@/integrations/prometheus/metrics"
+import { httpRequestCountByRegion } from "@/integrations/prometheus/metrics"
 import { RequestHandler } from "express"
-import { RaidHubLocals } from "./types"
 import { ParamsDictionary, Query } from "express-serve-static-core"
+import { RaidHubLocals } from "./types"
 
 /**
  * Helper to extract a string header value, handling both string and array cases
@@ -23,7 +21,7 @@ function extractHeader(header: string | string[] | undefined): string {
  * - Country (CF-IPCountry) - ISO 3166-1 alpha-2 country code (always available)
  * - Continent (CF-IPContinent) - Continent code (requires "Add visitor location headers" Managed Transform)
  * - ASN (CF-IPASNum) - Autonomous System Number (always available)
- * 
+ *
  * Adds _region, _continent, and _asn properties to the locals.
  */
 export const regionMetrics = <

src/middleware/request-logging.ts

@@ -9,7 +9,7 @@ const logger = new Logger("REQUEST_LOGGING")
  * Middleware to log request completion with duration and related metadata.
  * Should be used alongside duration metrics middleware, which is responsible
  * for populating timing information (e.g. `_duration`) on `res.locals`.
- * 
+ *
  * This middleware logs the request duration and Cloudflare region/ASN/continent.
  */
 export const requestLogging = <
@@ -20,7 +20,7 @@ export const requestLogging = <
 >(): RequestHandler<P, ResBody, ReqBody, ReqQuery, RaidHubLocals> => {
     return (req, res, next) => {
         res.once("finish", () => {
-            const locals = (res.locals as RaidHubLocals)
+            const locals = res.locals as RaidHubLocals
             logger.debug("REQUEST_COMPLETED", {
                 path: req.path,
                 url: req.ip,

src/middleware/types.ts

@@ -6,4 +6,4 @@ export interface RaidHubLocals {
     // populated by duration-metrics
     _startTime: number
     _duration: number
-}
+}

src/routes/admin/reporting/index.ts

@@ -1,8 +1,8 @@
 import { RaidHubRouter } from "@/core/RaidHubRouter"
 import { blacklistInstanceRoute } from "./blacklist"
 import { reportingStandingInstanceRoute } from "./instance-standing"
-import { getPlayerStanding } from "./player-standing"
 import { patchPlayer } from "./player"
+import { getPlayerStanding } from "./player-standing"
 
 export const reportingRouter = new RaidHubRouter({
     routes: [

src/routes/admin/reporting/player-standing.ts

@@ -4,10 +4,7 @@ import { ErrorCode } from "@/schema/errors/ErrorCode"
 import { zBigIntString } from "@/schema/input"
 import { zInt64 } from "@/schema/output"
 import { getPlayer } from "@/services/player"
-import {
-    getPlayerBlacklistedInstances,
-    getPlayerRecentFlags
-} from "@/services/reporting/standing"
+import { getPlayerBlacklistedInstances, getPlayerRecentFlags } from "@/services/reporting/standing"
 import { z } from "zod"
 
 export const getPlayerStanding = new RaidHubRoute({