@fugood/bricks-project 2.25.0-beta.6 → 2.25.0-beta.60

package/tools/mcp-tools/_verify.ts

@@ -0,0 +1,50 @@
+import { sh } from '../_shell'
+import { computeConfigChange, readBuildConfig, type ConfigChange } from '../../compile/config-diff'
+
+// Result of an editing tool's compile verification, carrying the config delta the
+// compile produced. Shared by entry-editing and data-calc-editing.
+export type VerificationResult = {
+  status: 'skipped' | 'compile:ok' | 'compile:failed'
+  errors: string[]
+  configChange?: ConfigChange
+}
+
+// Strip ANSI colors from the spawned compile so the captured output stays plain text.
+const noColorEnv = { FORCE_COLOR: '0', NO_COLOR: '1' }
+
+// Recompile the project and diff the result against the prior build artifact.
+export const compileAndDiff = async (projectDir: string): Promise<VerificationResult> => {
+  // Snapshot the prior compiled config before recompiling (the new artifact overwrites it).
+  const before = await readBuildConfig(projectDir)
+  try {
+    // Turn off the editing-tools audit for this spawned compile so compile() doesn't write
+    // a duplicate generic entry — the calling editing tool records its own richer entry.
+    await sh`bun compile`
+      .cwd(projectDir)
+      .env({ ...noColorEnv, BRICKS_CTOR_ENABLE_EDITING_TOOLS: '0' })
+      .text()
+  } catch (err: any) {
+    const stdout = err.stdout?.toString() ?? ''
+    const stderr = err.stderr?.toString() ?? ''
+    const output = [stdout, stderr, err.message].filter(Boolean).join('\n').trim()
+    return { status: 'compile:failed', errors: output ? [output] : ['bun compile failed'] }
+  }
+  const after = await readBuildConfig(projectDir)
+  return { status: 'compile:ok', errors: [], configChange: computeConfigChange(before, after) }
+}
+
+const shouldVerify = (inputVerify?: boolean) => {
+  if (inputVerify !== undefined) return inputVerify
+  const value = process.env.BRICKS_CTOR_MCP_EDIT_VERIFY ?? 'true'
+  return !['0', 'false', 'no', 'off'].includes(value.toLowerCase())
+}
+
+// Compile verification shared by the source-editing tools: per-call `verify` wins,
+// then the BRICKS_CTOR_MCP_EDIT_VERIFY env toggle, defaulting to on.
+export const verifyProject = async (
+  projectDir: string,
+  verify?: boolean,
+): Promise<VerificationResult> => {
+  if (!shouldVerify(verify)) return { status: 'skipped', errors: [] }
+  return compileAndDiff(projectDir)
+}

package/tools/mcp-tools/compile.ts

@@ -3,24 +3,33 @@ import { readFile } from 'node:fs/promises'
 import { z } from 'zod'
 import { sh } from '../_shell'
 
+// Disable ANSI color codes from spawned tools so MCP text output stays readable
+// (the host renders raw text; escape sequences leak into the result otherwise).
+const noColorEnv = { FORCE_COLOR: '0', NO_COLOR: '1' }
+
 export function register(server: McpServer, projectDir: string) {
   const { dirname } = import.meta
 
+  // `bun compile` records the resulting config delta to `.bricks/edits.jsonl` itself
+  // (see compile() in compile/index.ts), so the spawned output below already reflects it.
   server.tool('compile', {}, async () => {
-    let log = ''
+    let log = 'Type checking & Compiling...\n'
     try {
-      log += 'Type checking & Compiling...'
-      log += await sh`bun compile`.cwd(projectDir).text()
-    } catch (err) {
-      log += `${err.stdout.toString()}\n${err.stderr.toString()}`
+      log += await sh`bun compile`.cwd(projectDir).env(noColorEnv).text()
+    } catch (err: any) {
+      const stdout = err.stdout?.toString() ?? ''
+      const stderr = err.stderr?.toString() ?? ''
+      log += [stdout, stderr].filter(Boolean).join('\n')
     }
     return {
       content: [{ type: 'text', text: log }],
     }
   })
 
+  if (process.env.BRICKS_CTOR_MCP_DISABLE_PREVIEW === '1') return
+
   server.tool(
-    'preview',
+    'simulator',
     {
       delay: z
         .number()
@@ -61,11 +70,14 @@ export function register(server: McpServer, projectDir: string) {
         ]
         if (testId) args.push('--test-id', testId)
         if (testTitleLike) args.push('--test-title-like', testTitleLike)
-        log = await sh`bunx --bun electron ${toolsDir}/preview-main.mjs ${args}`
+        log = await sh`bunx --bun electron ${toolsDir}/simulator-main.mjs ${args}`
           .cwd(projectDir)
+          .env(noColorEnv)
           .text()
-      } catch (err) {
-        log = `${err.stdout.toString()}\n${err.stderr.toString()}`
+      } catch (err: any) {
+        const stdout = err.stdout?.toString() ?? ''
+        const stderr = err.stderr?.toString() ?? ''
+        log = [stdout, stderr].filter(Boolean).join('\n')
         error = true
       }
       let screenshotBase64: string | null = null