chore: add check:format script and CI formatting step (#569)

Add formatting check to catch unformatted code in CI and locally:
- New script: check:format (prettier --check .)
- Added to the check script (runs format → lint → types → deps)
- New CI step in lint.yml: Format Check (runs before Lint)
- Fix 7 files with existing formatting issues

Co-Authored-By: Claude

Author: binoy14

.github/workflows/lint.yml

@@ -30,6 +30,9 @@ jobs:
         if: github.run_attempt > 1
         run: echo "TURBO_FORCE=true" >> $GITHUB_ENV
 
+      - name: Format Check
+        run: pnpm check:format
+
       - name: Lint
         run: pnpm check:lint
 

package.json

@@ -21,8 +21,9 @@
   "scripts": {
     "build": "turbo run build build:types",
     "build:cli": "turbo run build build:types --filter=./packages/**",
-    "check": "npm run check:lint && npm run check:types && npm run check:deps",
+    "check": "npm run check:format && npm run check:lint && npm run check:types && npm run check:deps",
     "check:deps": "knip",
+    "check:format": "prettier --check .",
     "check:lint": "turbo run lint -- --fix",
     "check:types": "turbo run check:types --filter=@sanity/cli --filter=@sanity/cli-core",
     "clean": "rimraf packages/@sanity/*/lib packages/*/lib packages/@sanity/*/dist packages/*/dist",

packages/@repo/coverage-delta/src/lib.ts

@@ -8,9 +8,7 @@ function isCoverageMetric(value: unknown): value is {pct: number} {
   )
 }
 
-function isFileCoverageData(
-  value: unknown,
-): value is {
+function isFileCoverageData(value: unknown): value is {
   branches: {pct: number}
   functions: {pct: number}
   lines: {pct: number}

packages/@sanity/cli/package.json

@@ -155,8 +155,8 @@
     "@types/lodash-es": "^4.17.12",
     "@types/minimist": "^1.2.5",
     "@types/node": "catalog:",
-    "@types/react": "^19.2.14",
     "@types/picomatch": "^4.0.2",
+    "@types/react": "^19.2.14",
     "@types/react-dom": "^19.2.3",
     "@types/react-is": "^19.2.0",
     "@types/semver": "^7.7.1",

packages/@sanity/cli/src/commands/documents/__tests__/validate.flags.test.ts

@@ -116,9 +116,7 @@ describe('#documents:validate flag passthrough', () => {
     ])
 
     if (error) throw error
-    expect(mocks.validateDocuments).toHaveBeenCalledWith(
-      expect.objectContaining({level: 'error'}),
-    )
+    expect(mocks.validateDocuments).toHaveBeenCalledWith(expect.objectContaining({level: 'error'}))
   })
 
   test('passes --max-custom-validation-concurrency to validateDocuments action', async () => {

packages/@sanity/cli/src/commands/documents/query.ts

@@ -64,12 +64,7 @@ export class QueryDocumentCommand extends SanityCommand<typeof QueryDocumentComm
   public async run(): Promise<void> {
     const {args, flags} = await this.parse(QueryDocumentCommand)
     const {query} = args
-    const {
-      anonymous,
-      'api-version': apiVersion,
-      dataset,
-      pretty,
-    } = flags
+    const {anonymous, 'api-version': apiVersion, dataset, pretty} = flags
 
     const cliConfig = await this.getCliConfig()
     const projectId = await this.getProjectId({

packages/@sanity/cli/src/commands/documents/validate.ts

@@ -39,7 +39,8 @@ export class ValidateDocumentsCommand extends SanityCommand<typeof ValidateDocum
 
   static flags = {
     ...getProjectIdFlag({
-      description: 'Override the project ID used. By default, this is derived from the given workspace',
+      description:
+        'Override the project ID used. By default, this is derived from the given workspace',
     }),
     ...getDatasetFlag({
       description:

packages/@sanity/cli/src/commands/graphql/__tests__/undeploy.test.ts

@@ -246,11 +246,9 @@ describe('graphql undeploy', () => {
   })
 
   test('errors when both --api and --project-id are specified', async () => {
-    const {error} = await testCommand(
-      Undeploy,
-      ['--api', 'ios', '--project-id', 'test-project'],
-      {mocks: defaultMocks},
-    )
+    const {error} = await testCommand(Undeploy, ['--api', 'ios', '--project-id', 'test-project'], {
+      mocks: defaultMocks,
+    })
 
     expect(error).toBeInstanceOf(Error)
     expect(error?.message).toContain('cannot also be provided when using --api')

packages/@sanity/cli/src/services/__tests__/grants.test.ts

@@ -38,9 +38,7 @@ describe('getUserGrants', () => {
       organizations: {},
       projects: {
         'project-a': {
-          'sanity.project.datasets': [
-            {grants: [{name: 'read', params: {}}]},
-          ],
+          'sanity.project.datasets': [{grants: [{name: 'read', params: {}}]}],
         },
       },
     }