@fugood/bricks-ctor 2.25.0-beta.42 → 2.25.0-beta.45

package/tools/mcp-tools/__tests__/mcp-env.test.js

@@ -0,0 +1,19 @@
+import { isTruthyEnv, shouldRegisterEditingTools } from '../../mcp-env'
+
+describe('mcp env helpers', () => {
+  test('recognizes explicit truthy env values', () => {
+    for (const value of ['1', 'true', 'TRUE', 'yes', 'on', ' on ']) {
+      expect(isTruthyEnv(value)).toBe(true)
+    }
+  })
+
+  test('does not enable editing tools by default', () => {
+    expect(shouldRegisterEditingTools({})).toBe(false)
+    expect(shouldRegisterEditingTools({ BRICKS_CTOR_ENABLE_EDITING_TOOLS: '0' })).toBe(false)
+    expect(shouldRegisterEditingTools({ BRICKS_CTOR_ENABLE_EDITING_TOOLS: 'false' })).toBe(false)
+  })
+
+  test('enables editing tools when explicitly requested', () => {
+    expect(shouldRegisterEditingTools({ BRICKS_CTOR_ENABLE_EDITING_TOOLS: '1' })).toBe(true)
+  })
+})

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

@@ -10,6 +10,8 @@ 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 = 'Type checking & Compiling...\n'
     try {