refactor(deploy): report --json problems as an errors map and warnings list

Replace the flat `checks` array with `errors` (a map of blocking problem → fix)
and `warnings` (messages), dropping informational pass/skip lines. A consumer
acts on the fixes directly instead of filtering checks by status.

Author: gu-stav

packages/@sanity/cli/src/actions/deploy/__tests__/deploymentPlan.test.ts

@@ -51,12 +51,13 @@ const studioPlan = (checks: DeployCheck[], files: DeploymentFile[] = []): Deploy
 })
 
 describe('deploymentPlanToJson', () => {
-  test('projects the plan into a machine-readable shape with a derived verdict and total', () => {
+  test('maps failing checks to fixes and warnings to messages, dropping pass/skip', () => {
     const json = deploymentPlanToJson(
       studioPlan(
         [
           {message: 'Project: p1', status: 'pass'},
-          {message: 'No studio hostname configured', status: 'fail'},
+          {message: 'No studio hostname configured', solution: 'Set `studioHost`', status: 'fail'},
+          {message: 'The autoUpdates config has moved', status: 'warn'},
         ],
         [{path: 'dist/index.html', size: 1_048_576}],
       ),
@@ -65,16 +66,20 @@ describe('deploymentPlanToJson', () => {
     expect(json).toEqual({
       applicationType: 'studio',
       applicationVersion: '3.99.0',
-      checks: [
-        {message: 'Project: p1', status: 'pass'},
-        {message: 'No studio hostname configured', status: 'fail'},
-      ],
       deployable: false,
+      errors: {'No studio hostname configured': 'Set `studioHost`'},
       files: [{path: 'dist/index.html', size: 1_048_576}],
       totalBytes: 1_048_576,
+      warnings: ['The autoUpdates config has moved'],
     })
   })
 
+  test('an error without a solution maps to null', () => {
+    const json = deploymentPlanToJson(studioPlan([{message: 'boom', status: 'fail'}]))
+    expect(json.errors).toEqual({boom: null})
+    expect(json.deployable).toBe(false)
+  })
+
   test('deployable is true when no check failed', () => {
     expect(deploymentPlanToJson(studioPlan([{message: 'ok', status: 'pass'}])).deployable).toBe(
       true,

packages/@sanity/cli/src/actions/deploy/deploymentPlan.ts

@@ -66,24 +66,34 @@ function totalBytes(files: DeploymentFile[]): number {
 }
 
 /**
- * The machine-readable form of the report `renderDeploymentPlan` prints. Both
- * derive from the plan, so the two can't drift.
+ * A problem-focused, machine-readable projection of the plan: blocking problems
+ * mapped to their fix, warnings as messages. Derived from the same checks the
+ * human report renders (its pass/skip lines are informational and omitted here).
  */
 export function deploymentPlanToJson(plan: DeploymentPlan): {
   applicationType: DeploymentPlan['type']
   applicationVersion: string | null
-  checks: DeployCheck[]
   deployable: boolean
+  errors: Record<string, string | null>
   files: DeploymentFile[]
   totalBytes: number
+  warnings: string[]
 } {
+  const errors: Record<string, string | null> = {}
+  const warnings: string[] = []
+  for (const check of plan.checks) {
+    if (check.status === 'fail') errors[check.message] = check.solution ?? null
+    else if (check.status === 'warn') warnings.push(check.message)
+  }
+
   return {
     applicationType: plan.type,
     applicationVersion: plan.version,
-    checks: plan.checks,
     deployable: isDeployable(plan),
+    errors,
     files: plan.files,
     totalBytes: totalBytes(plan.files),
+    warnings,
   }
 }
 

packages/@sanity/cli/test/integration/commands/deploy.studio.test.ts

@@ -316,10 +316,9 @@ describe('#deploy studio', () => {
     expect(plan.applicationType).toBe('studio')
     expect(typeof plan.applicationVersion).toBe('string')
     expect(plan.deployable).toBe(true)
+    expect(plan.errors).toEqual({})
+    expect(Array.isArray(plan.warnings)).toBe(true)
     expect(plan.files).toContainEqual(expect.objectContaining({path: 'dist/index.html'}))
-    expect(
-      plan.checks.some((check: {message: string}) => check.message.includes('existing studio')),
-    ).toBe(true)
   })
 
   test('should output the deploy result as JSON with --json', async () => {