build: fix prettier configuration, reformat files, and run prettier in CI (#227)

* build: fix prettier configuration

* style: reformat files with fixed prettier

* ci: run prettier in CI

* ci: rename format job

* style: reformat again after rebasing

Author: serhalp

.github/workflows/format.yaml

@@ -0,0 +1,23 @@
+name: Check formatting
+on:
+  push:
+    branches: [main]
+  pull_request:
+    types: [opened, synchronize, reopened]
+  merge_group:
+jobs:
+  format:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+      - name: Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: lts/*
+          cache: 'npm'
+      - name: Install dependencies
+        run: npm ci
+      - name: Check formatting
+        run: npm run format:ci

.github/workflows/test.yaml

@@ -47,4 +47,4 @@ jobs:
       - name: Build
         run: npm run build --workspaces=true
       - name: Tests
-        run: npm run test --workspaces=true
+        run: npm run test --workspaces=true

.prettierignore

@@ -1,2 +1,4 @@
 .circleci
+dist
 coverage
+CHANGELOG.md

packages/cache/src/cache-status/cache-status.ts

@@ -42,14 +42,17 @@ export const parseCacheStatusValue = (value: string) => {
   const parts = value.split(';').map((part) => part.trim())
   const [namePart, ...attributeParts] = parts
   const name = (namePart ?? '').replace(/"/g, '').toLowerCase()
-  const attributes = attributeParts.reduce((acc, part) => {
-    const [key, value = ''] = part.split('=')
+  const attributes = attributeParts.reduce(
+    (acc, part) => {
+      const [key, value = ''] = part.split('=')
 
-    return {
-      ...acc,
-      [key]: value,
-    }
-  }, {} as Record<string, string>)
+      return {
+        ...acc,
+        [key]: value,
+      }
+    },
+    {} as Record<string, string>,
+  )
 
   return {
     attributes,

packages/dev-utils/src/lib/base64.test.ts

@@ -4,6 +4,6 @@ import { shouldBase64Encode } from './base64.js'
 
 describe('`shouldBase64Encode` helper', () => {
   test('Returns `false` for `text/` content types', async () => {
-    expect(shouldBase64Encode("text/html")).toBeFalsy()
+    expect(shouldBase64Encode('text/html')).toBeFalsy()
   })
-})
+})

packages/dev/README.md

@@ -1,7 +1,6 @@
 # @netlify/dev
 
-> [!WARNING]
-> This module is under active development and does **not** yet support all Netlify platform features.
+> [!WARNING] This module is under active development and does **not** yet support all Netlify platform features.
 
 `@netlify/dev` is a local emulator for the Netlify production environment. While it can be used directly by advanced
 users, it is primarily designed as a foundational library for higher-level tools like the

packages/dev/src/main.ts

@@ -144,7 +144,7 @@ export class NetlifyDev {
           publishDir: this.#config?.config.build.publish ?? undefined,
           logger: this.#logger,
         })
-      : {handle: async (_request: Request, response: Response) => response}
+      : { handle: async (_request: Request, response: Response) => response }
 
     // Redirects
     const redirects = this.#features.redirects

packages/functions/README.md

@@ -68,7 +68,7 @@ To use Scheduled Functions, wrap your function handler with the `schedule` funct
   ```ts
   import { schedule } from '@netlify/functions'
 
-  export const handler = schedule("5 4 * * *", async () => {
+  export const handler = schedule('5 4 * * *', async () => {
     console.log("It's 04:05 AM!")
   })
   ```

packages/functions/test/helpers/main.mjs

@@ -16,4 +16,3 @@ export const invokeLambda = (handler, { method = 'GET', ...options } = {}) => {
     resolve(handler(event, {}, callback))
   })
 }
-

packages/functions/test/helpers/mock_fetch.mjs

@@ -1,4 +1,4 @@
-import assert from "node:assert"
+import assert from 'node:assert'
 
 export class MockFetch {
   constructor() {

packages/otel/src/bootstrap/main.ts

@@ -1,13 +1,13 @@
 import { GET_TRACER, SHUTDOWN_TRACERS } from '../constants.js'
 
 export const createTracerProvider = async (options: {
-  headers: Headers,
-  serviceName: string,
-  serviceVersion: string,
-  deploymentEnvironment: string,
-  siteUrl: string,
-  siteId: string,
-  siteName: string,
+  headers: Headers
+  serviceName: string
+  serviceVersion: string
+  deploymentEnvironment: string
+  siteUrl: string
+  siteId: string
+  siteName: string
 }) => {
   if (!options.headers.has('x-nf-enable-tracing')) {
     return
@@ -52,7 +52,7 @@ export const createTracerProvider = async (options: {
 
       const { default: pkg } = await import('../../package.json', { with: { type: 'json' } })
       return new SugaredTracer(trace.getTracer(pkg.name, pkg.version))
-    }
+    },
   })
 
   Object.defineProperty(globalThis, SHUTDOWN_TRACERS, {
@@ -61,7 +61,6 @@ export const createTracerProvider = async (options: {
     writable: false,
     value: async () => {
       return await nodeTracerProvider.shutdown()
-    }
+    },
   })
 }
-

packages/otel/src/main.ts

@@ -1,15 +1,15 @@
-import { type SugaredTracer } from '@opentelemetry/api/experimental';
-import { GET_TRACER, SHUTDOWN_TRACERS } from './constants.js';
+import { type SugaredTracer } from '@opentelemetry/api/experimental'
+import { GET_TRACER, SHUTDOWN_TRACERS } from './constants.js'
 
 type GlobalThisExtended = typeof globalThis & {
-    [GET_TRACER]?: (name?: string, version?: string) => SugaredTracer | undefined;
-    [SHUTDOWN_TRACERS]?: () => void;
+  [GET_TRACER]?: (name?: string, version?: string) => SugaredTracer | undefined
+  [SHUTDOWN_TRACERS]?: () => void
 }
 
 export const getTracer = async (name?: string, version?: string): Promise<SugaredTracer | undefined> => {
-    return (globalThis as GlobalThisExtended)[GET_TRACER]?.(name, version);
+  return (globalThis as GlobalThisExtended)[GET_TRACER]?.(name, version)
 }
 
 export const shutdownTracers = async (): Promise<void> => {
-    return (globalThis as GlobalThisExtended)[SHUTDOWN_TRACERS]?.();
-}
+  return (globalThis as GlobalThisExtended)[SHUTDOWN_TRACERS]?.()
+}

packages/runtime-utils/src/lib/base64.ts

@@ -1,6 +1,9 @@
-
 // eslint-disable-next-line n/prefer-global/buffer
-export const base64Decode =  globalThis.Buffer ? (input: string) => Buffer.from(input, 'base64').toString() : (input: string) => atob(input)
+export const base64Decode = globalThis.Buffer
+  ? (input: string) => Buffer.from(input, 'base64').toString()
+  : (input: string) => atob(input)
 
 // eslint-disable-next-line n/prefer-global/buffer
-export const base64Encode = globalThis.Buffer ? (input: string) => Buffer.from(input).toString('base64') : (input: string) => btoa(input)
+export const base64Encode = globalThis.Buffer
+  ? (input: string) => Buffer.from(input).toString('base64')
+  : (input: string) => btoa(input)

packages/static/src/lib/fs.ts

@@ -17,6 +17,6 @@ export const fileExists = async (path: string) => {
 
 export const getReadableStreamFromFile = (path: string) => {
   const stream = createReadStream(path)
-  
+
   return Readable.toWeb(stream) as ReadableStream
 }

packages/static/src/lib/paths.ts

@@ -1,12 +1,12 @@
-import path from "node:path"
+import path from 'node:path'
 
 // Used as an optimization to avoid dual lookups for missing assets
 const assetExtensionRegExp = /\.(html?|png|jpg|js|css|svg|gif|ico|woff|woff2)$/
 
-export const getFilePathsForURL = (pathname: string, baseDirectory = "") => {
+export const getFilePathsForURL = (pathname: string, baseDirectory = '') => {
   const urlVariations = getURLVariations(pathname)
-  const possiblePaths = urlVariations.map(urlVariation => {
-    const parts = urlVariation.split("/").filter(Boolean) as string[]
+  const possiblePaths = urlVariations.map((urlVariation) => {
+    const parts = urlVariation.split('/').filter(Boolean) as string[]
 
     return path.resolve.apply(null, [baseDirectory, ...parts])
   })
@@ -16,7 +16,7 @@ export const getFilePathsForURL = (pathname: string, baseDirectory = "") => {
 
 export const getURLVariations = (pathname: string) => {
   const paths: string[] = []
-  
+
   if (pathname[pathname.length - 1] === '/') {
     const end = pathname.length - 1
 
@@ -29,9 +29,9 @@ export const getURLVariations = (pathname: string) => {
     paths.push(`${pathname}.html`, `${pathname}.htm`, `${pathname}/index.html`, `${pathname}/index.htm`)
   }
 
-  if (!(paths.includes(pathname))) {
+  if (!paths.includes(pathname)) {
     return [pathname, ...paths]
   }
 
   return paths
-}
+}

packages/types/README.md

@@ -16,7 +16,8 @@ npm install @netlify/types
 
 ## Usage
 
-This package provides TypeScript type definitions for various Netlify platform primitives. Import the types you need in your TypeScript code:
+This package provides TypeScript type definitions for various Netlify platform primitives. Import the types you need in
+your TypeScript code:
 
 ```typescript
 import type { Context } from '@netlify/types'
@@ -32,4 +33,5 @@ The package includes TypeScript definitions for:
 
 ## Contributing
 
-Contributions are welcome! If you encounter any issues or have suggestions for improvements, please open an issue or submit a pull request.
+Contributions are welcome! If you encounter any issues or have suggestions for improvements, please open an issue or
+submit a pull request.

packages/vite-plugin/README.md

@@ -1,6 +1,7 @@
 # @netlify/vite-plugin
 
-> [!WARNING] This is an experimental Vite plugin for Netlify. It is under active development and does **not** yet support all Netlify platform features.
+> [!WARNING] This is an experimental Vite plugin for Netlify. It is under active development and does **not** yet
+> support all Netlify platform features.
 
 A Vite plugin that integrates with Netlify's platform features.
 
@@ -14,7 +15,8 @@ npm install @netlify/vite-plugin
 
 The plugin accepts the following options:
 
-- `middleware` (boolean, default: `true`): Attach a Vite middleware that intercepts requests and handles them in the same way as the Netlify production environment
+- `middleware` (boolean, default: `true`): Attach a Vite middleware that intercepts requests and handles them in the
+  same way as the Netlify production environment
 - `blobs`: Configure blob storage functionality
 - `functions`: Configure serverless functions
 - `headers`: Configure response headers
@@ -30,6 +32,6 @@ import { defineConfig } from 'vite'
 import netlify from '@netlify/vite-plugin'
 
 export default defineConfig({
-  plugins: [netlify()]
+  plugins: [netlify()],
 })
 ```

renovate.json

@@ -1,6 +1,4 @@
 {
   "$schema": "https://docs.renovatebot.com/renovate-schema.json",
-  "extends": [
-    "local>netlify/renovate-config"
-  ]
+  "extends": ["local>netlify/renovate-config"]
 }