@fugood/bricks-project 2.25.0-beta.5 → 2.25.0-beta.50

package/compile/action-name-map.ts

@@ -350,6 +350,63 @@ export const templateActionNameMap = {
       strokeWidth: 'BRICK_SKETCH_STROKE_WIDTH',
     },
   },
+  BRICK_SCENE_3D: {
+    BRICK_SCENE_3D_ADD_OBJECT: {
+      objectId: 'BRICK_SCENE_3D_OBJECT_ID',
+      objectType: 'BRICK_SCENE_3D_OBJECT_TYPE',
+      objectUrl: 'BRICK_SCENE_3D_OBJECT_URL',
+      objectMd5: 'BRICK_SCENE_3D_OBJECT_MD5',
+      objectPosition: 'BRICK_SCENE_3D_OBJECT_POSITION',
+      objectRotation: 'BRICK_SCENE_3D_OBJECT_ROTATION',
+      objectScale: 'BRICK_SCENE_3D_OBJECT_SCALE',
+      objectColor: 'BRICK_SCENE_3D_OBJECT_COLOR',
+    },
+    BRICK_SCENE_3D_REMOVE_OBJECT: {
+      objectId: 'BRICK_SCENE_3D_OBJECT_ID',
+    },
+    BRICK_SCENE_3D_UPDATE_OBJECT: {
+      objectId: 'BRICK_SCENE_3D_OBJECT_ID',
+      objectPosition: 'BRICK_SCENE_3D_OBJECT_POSITION',
+      objectRotation: 'BRICK_SCENE_3D_OBJECT_ROTATION',
+      objectScale: 'BRICK_SCENE_3D_OBJECT_SCALE',
+      objectVisible: 'BRICK_SCENE_3D_OBJECT_VISIBLE',
+      objectColor: 'BRICK_SCENE_3D_OBJECT_COLOR',
+      objectNodes: 'BRICK_SCENE_3D_OBJECT_NODES',
+    },
+    BRICK_SCENE_3D_SET_CAMERA: {
+      cameraPosition: 'BRICK_SCENE_3D_CAMERA_POSITION',
+      cameraTarget: 'BRICK_SCENE_3D_CAMERA_TARGET',
+      cameraFov: 'BRICK_SCENE_3D_CAMERA_FOV',
+      cameraAnimateMs: 'BRICK_SCENE_3D_CAMERA_ANIMATE_MS',
+    },
+    BRICK_SCENE_3D_LOOK_AT: {
+      objectId: 'BRICK_SCENE_3D_OBJECT_ID',
+    },
+    BRICK_SCENE_3D_PLAY_ANIMATION: {
+      objectId: 'BRICK_SCENE_3D_OBJECT_ID',
+      animationName: 'BRICK_SCENE_3D_ANIMATION_NAME',
+      animationLoop: 'BRICK_SCENE_3D_ANIMATION_LOOP',
+      animationSpeed: 'BRICK_SCENE_3D_ANIMATION_SPEED',
+    },
+    BRICK_SCENE_3D_STOP_ANIMATION: {
+      objectId: 'BRICK_SCENE_3D_OBJECT_ID',
+      animationName: 'BRICK_SCENE_3D_ANIMATION_NAME',
+    },
+    BRICK_SCENE_3D_SET_BACKGROUND: {
+      backgroundColor: 'BRICK_SCENE_3D_BACKGROUND_COLOR',
+      backgroundHdrUrl: 'BRICK_SCENE_3D_BACKGROUND_HDR_URL',
+      backgroundMd5: 'BRICK_SCENE_3D_BACKGROUND_MD5',
+    },
+    BRICK_SCENE_3D_SET_CONTROLS: {
+      controlsEnabled: 'BRICK_SCENE_3D_CONTROLS_ENABLED',
+      controlsAutoRotate: 'BRICK_SCENE_3D_CONTROLS_AUTO_ROTATE',
+      controlsAutoRotateSpeed: 'BRICK_SCENE_3D_CONTROLS_AUTO_ROTATE_SPEED',
+    },
+    BRICK_SCENE_3D_SCREENSHOT: {
+      screenshotFormat: 'BRICK_SCENE_3D_SCREENSHOT_FORMAT',
+      screenshotQuality: 'BRICK_SCENE_3D_SCREENSHOT_QUALITY',
+    },
+  },
 
   GENERATOR_FILE: {
     GENERATOR_FILE_READ_CONTENT: {
@@ -782,6 +839,11 @@ export const templateActionNameMap = {
       seed: 'GENERATOR_LLM_SEED',
       typicalP: 'GENERATOR_LLM_TYPICAL_P',
       ignoreEos: 'GENERATOR_LLM_IGNORE_EOS',
+      mtpSpeculativeDecoding: 'GENERATOR_LLM_MTP_SPECULATIVE_DECODING',
+      mtpDraftTokens: 'GENERATOR_LLM_MTP_DRAFT_TOKENS',
+      mtpDraftMinTokens: 'GENERATOR_LLM_MTP_DRAFT_MIN_TOKENS',
+      mtpDraftMinProbability: 'GENERATOR_LLM_MTP_DRAFT_MIN_PROBABILITY',
+      mtpDraftSplitProbability: 'GENERATOR_LLM_MTP_DRAFT_SPLIT_PROBABILITY',
       functionCallEnabled: 'GENERATOR_LLM_FUNCTION_CALL_ENABLED',
       functionCallSchema: 'GENERATOR_LLM_FUNCTION_CALL_SCHEMA',
     },
@@ -894,10 +956,12 @@ export const templateActionNameMap = {
     GENERATOR_APPLE_STT_TRANSCRIBE_FILE: {
       fileUrl: 'GENERATOR_APPLE_STT_FILE_URL',
       language: 'GENERATOR_APPLE_STT_LANGUAGE',
+      contextualStrings: 'GENERATOR_APPLE_STT_CONTEXTUAL_STRINGS',
     },
     GENERATOR_APPLE_STT_TRANSCRIBE_DATA: {
       data: 'GENERATOR_APPLE_STT_DATA',
       language: 'GENERATOR_APPLE_STT_LANGUAGE',
+      contextualStrings: 'GENERATOR_APPLE_STT_CONTEXTUAL_STRINGS',
     },
   },
   GENERATOR_APPLE_TTS: {

package/compile/config-diff.ts

@@ -0,0 +1,155 @@
+import { readFile } from 'node:fs/promises'
+import path from 'node:path'
+
+// The compiled config artifact written by `bun compile` (see the generated
+// project's `compile.ts`; bricks-project-generator/index.js:762).
+export const BUILD_CONFIG_RELATIVE = '.bricks/build/application-config.json'
+
+// `compile()` derives these top-level fields from `Date.now()`
+// (compile/index.ts `title: \`${app.name}(${timestamp})\`` and `update_timestamp`),
+// so they differ on every run regardless of source. Excluded from the comparison —
+// only the *top-level* keys are dropped, so nested subspace/brick `title`s still diff.
+const VOLATILE_TOP_LEVEL_FIELDS = ['title', 'update_timestamp']
+
+// A minimal, path-keyed config change (Option A): a generic patch, not coupled to
+// bricks-config-editor's action vocabulary. `path` is an array of object keys /
+// array indices, ready for the editor's `setYValueAtPath` (or a delete) so applying
+// the patch is a single Yjs `'local'` transaction == one undo/redo entry.
+export type ConfigPatchOp =
+  | { op: 'set'; path: Array<string | number>; value: unknown }
+  | { op: 'unset'; path: Array<string | number> }
+
+export type ConfigChange =
+  | { status: 'ok'; ops: ConfigPatchOp[]; opCount: number }
+  | { status: 'no_baseline' }
+  | { status: 'unavailable' }
+
+const isRecord = (value: unknown): value is Record<string, unknown> =>
+  typeof value === 'object' && value !== null && !Array.isArray(value)
+
+const isJsonDroppedValue = (value: unknown) =>
+  value === undefined || typeof value === 'function' || typeof value === 'symbol'
+
+const toJsonComparableScalar = (value: unknown) => {
+  if (typeof value === 'number' && !Number.isFinite(value)) return null
+  return value
+}
+
+const toJsonCompatibleValue = (value: unknown): unknown => {
+  if (isJsonDroppedValue(value)) return undefined
+  if (Array.isArray(value)) {
+    return value.map((item) => (isJsonDroppedValue(item) ? null : toJsonCompatibleValue(item)))
+  }
+  if (isRecord(value)) {
+    return Object.entries(value).reduce((acc, [key, item]) => {
+      if (!isJsonDroppedValue(item)) acc[key] = toJsonCompatibleValue(item)
+      return acc
+    }, {})
+  }
+  return toJsonComparableScalar(value)
+}
+
+const hasJsonObjectKey = (value: Record<string, unknown>, key: string) =>
+  Object.prototype.hasOwnProperty.call(value, key) && !isJsonDroppedValue(value[key])
+
+const getJsonArrayItem = (value: unknown[], index: number) => {
+  const item = value[index]
+  return isJsonDroppedValue(item) ? null : item
+}
+
+const deepEqual = (a: unknown, b: unknown): boolean => {
+  a = toJsonComparableScalar(a)
+  b = toJsonComparableScalar(b)
+  if (a === b) return true
+  if (Array.isArray(a) && Array.isArray(b)) {
+    if (a.length !== b.length) return false
+    for (let index = 0; index < a.length; index += 1) {
+      if (!deepEqual(getJsonArrayItem(a, index), getJsonArrayItem(b, index))) return false
+    }
+    return true
+  }
+  if (isRecord(a) && isRecord(b)) {
+    let comparableAKeys = 0
+    for (const key of Object.keys(a)) {
+      if (isJsonDroppedValue(a[key])) continue
+      comparableAKeys += 1
+      if (!hasJsonObjectKey(b, key) || !deepEqual(a[key], b[key])) return false
+    }
+
+    let comparableBKeys = 0
+    for (const key of Object.keys(b)) {
+      if (!isJsonDroppedValue(b[key])) comparableBKeys += 1
+    }
+    return comparableAKeys === comparableBKeys
+  }
+  return false
+}
+
+const normalizeConfig = (config: unknown) => {
+  if (!isRecord(config)) return config
+  const normalized = { ...config }
+  for (const field of VOLATILE_TOP_LEVEL_FIELDS) delete normalized[field]
+  return normalized
+}
+
+// Objects recurse key-wise; equal-length arrays recurse element-wise; everything else
+// (scalars, type changes, length-changed arrays) emits one whole-value `set` at that
+// path. The editor's `applyJsonDiffToYType` minimizes the actual Yjs ops downstream,
+// so a whole-array `set` on insert/remove still yields a minimal CRDT mutation.
+const diffInto = (
+  before: unknown,
+  after: unknown,
+  currentPath: Array<string | number>,
+  ops: ConfigPatchOp[],
+) => {
+  if (deepEqual(before, after)) return
+
+  if (isRecord(before) && isRecord(after)) {
+    const keys = new Set([...Object.keys(before), ...Object.keys(after)])
+    for (const key of keys) {
+      const beforeHasKey = hasJsonObjectKey(before, key)
+      const afterHasKey = hasJsonObjectKey(after, key)
+      if (!beforeHasKey && !afterHasKey) continue
+
+      const nextPath = [...currentPath, key]
+      if (!afterHasKey) ops.push({ op: 'unset', path: nextPath })
+      else if (!beforeHasKey) {
+        ops.push({ op: 'set', path: nextPath, value: toJsonCompatibleValue(after[key]) })
+      } else diffInto(before[key], after[key], nextPath, ops)
+    }
+    return
+  }
+
+  if (Array.isArray(before) && Array.isArray(after) && before.length === after.length) {
+    for (let index = 0; index < after.length; index += 1) {
+      diffInto(
+        getJsonArrayItem(before, index),
+        getJsonArrayItem(after, index),
+        [...currentPath, index],
+        ops,
+      )
+    }
+    return
+  }
+
+  ops.push({ op: 'set', path: currentPath, value: toJsonCompatibleValue(after) })
+}
+
+// Diff two compiled configs. `before == null` means there was no prior build to compare
+// (first compile); `after == null` means the fresh artifact could not be read.
+export const computeConfigChange = (before: unknown, after: unknown): ConfigChange => {
+  if (before == null) return { status: 'no_baseline' }
+  if (after == null) return { status: 'unavailable' }
+  const ops: ConfigPatchOp[] = []
+  diffInto(normalizeConfig(before), normalizeConfig(after), [], ops)
+  return { status: 'ok', ops, opCount: ops.length }
+}
+
+// Read the last-compiled config artifact. Returns null when it is absent or unreadable.
+export const readBuildConfig = async (projectDir: string): Promise<unknown> => {
+  try {
+    return JSON.parse(await readFile(path.join(projectDir, BUILD_CONFIG_RELATIVE), 'utf8'))
+  } catch {
+    return null
+  }
+}

package/compile/index.ts

@@ -6,10 +6,14 @@ import omit from 'lodash/omit'
 import { parse as parseAST } from 'acorn'
 import type { ExportNamedDeclaration, FunctionDeclaration } from 'acorn'
 import escodegen from 'escodegen'
-import { makeId } from '../utils/id'
+import { makeSeededId } from '../utils/id'
 import { generateCalulationMap } from './util'
 import { templateActionNameMap } from './action-name-map'
 import { templateEventPropsMap } from '../utils/event-props'
+import { sh } from '../tools/_shell'
+import { computeConfigChange, readBuildConfig } from './config-diff'
+import { appendEditRecord, editProvenance } from '../tools/_edits-log'
+import { isTruthyEnv } from '../tools/mcp-env'
 import type {
   Application,
   Data,
@@ -154,11 +158,13 @@ const basicAnimationEvents = ['show', 'standby', 'breatheStart']
 
 const compileAnimations = (
   templateKey: string,
-  animations: { [key: string]: Animation },
+  animations: { [key: string]: Animation | (() => Animation) },
   errorReference: string,
 ) =>
   Object.entries(animations).reduce((acc, [key, animation]) => {
-    const animationId = assertEntryId(animation?.id, 'ANIMATION', errorReference)
+    // Animation events accept either a direct Animation or a getter; unwrap.
+    const resolved = typeof animation === 'function' ? animation() : animation
+    const animationId = assertEntryId(resolved?.id, 'ANIMATION', errorReference)
     acc[convertEventKey(basicAnimationEvents.includes(key) ? 'BRICK' : templateKey, key)] =
       `ANIMATION#${animationId}`
     return acc
@@ -179,10 +185,17 @@ const compileEvents = (
 
       let handlerKey
       let handlerTemplateKey
-      if (handler === 'system' || typeof handler === 'string') {
-        if (handler.startsWith('SUBSPACE_')) handlerKey = handler
-        else handlerKey = handler.toUpperCase()
-        if (handlerKey === 'SYSTEM') handlerTemplateKey = 'SYSTEM'
+      if (typeof handler === 'string') {
+        // Only the literal 'system' handler is normalized to the SYSTEM template key.
+        // SubspaceID (SUBSPACE_*) and ItemBrickID handlers are kept verbatim: the runtime
+        // resolves them case-sensitively (see mapEventMapHandlersWithNewId), so uppercasing
+        // a mixed-case ItemBrickID would break handler-to-item event wiring.
+        if (handler === 'system') {
+          handlerKey = 'SYSTEM'
+          handlerTemplateKey = 'SYSTEM'
+        } else {
+          handlerKey = handler
+        }
       } else if (typeof handler === 'function') {
         let instance = handler()
         if (instance?.id) {
@@ -301,6 +314,162 @@ const animationTypeMap = {
   AnimationTimingConfig: 'timing',
   AnimationSpringConfig: 'spring',
   AnimationDecayConfig: 'decay',
+} as const
+
+type CompiledAnimationType = (typeof animationTypeMap)[keyof typeof animationTypeMap]
+type WarningMetadata = Record<string, unknown>
+
+const animationProperties = new Set([
+  'transform.translateX',
+  'transform.translateY',
+  'transform.scale',
+  'transform.scaleX',
+  'transform.scaleY',
+  'transform.rotate',
+  'transform.rotateX',
+  'transform.rotateY',
+  'opacity',
+])
+
+const animationComposeTypes = new Set(['parallel', 'sequence'])
+const springConfigFamilies = [
+  ['stiffness', 'damping', 'mass'],
+  ['tension', 'friction'],
+  ['bounciness', 'speed'],
+]
+const springConfigFamilyKeys = new Set(springConfigFamilies.flat())
+
+const isRecord = (value: unknown): value is Record<string, unknown> =>
+  Boolean(value) && typeof value === 'object' && !Array.isArray(value)
+
+const hasDefinedConfigValue = (config: Record<string, unknown>, key: string) =>
+  config[key] !== undefined
+
+const assertConfigValue = (
+  config: Record<string, unknown>,
+  key: string,
+  errorReference: string,
+) => {
+  if (!hasDefinedConfigValue(config, key)) {
+    throw new Error(`Invalid animation config ${errorReference}: missing "${key}"`)
+  }
+}
+
+const assertAnimationProperty = (property: unknown, errorReference: string) => {
+  if (typeof property !== 'string' || !animationProperties.has(property)) {
+    throw new Error(
+      `Invalid animation property${errorReference ? ` ${errorReference}` : ''}: ${String(
+        property,
+      )}`,
+    )
+  }
+  return property
+}
+
+const getAnimationType = (config: unknown, errorReference: string): CompiledAnimationType => {
+  if (!isRecord(config)) {
+    throw new Error(`Invalid animation config ${errorReference}: config must be an object`)
+  }
+
+  const animationType = animationTypeMap[config.__type as keyof typeof animationTypeMap]
+  if (!animationType) {
+    throw new Error(`Invalid animation config type ${errorReference}: ${String(config.__type)}`)
+  }
+  return animationType
+}
+
+const assertAnimationComposeType = (composeType: unknown, errorReference: string) => {
+  if (typeof composeType !== 'string' || !animationComposeTypes.has(composeType)) {
+    throw new Error(`Invalid animation compose type ${errorReference}: ${String(composeType)}`)
+  }
+  return composeType
+}
+
+const pickDefinedConfigValues = (config: Record<string, unknown>, keys: string[]) =>
+  keys.reduce((acc, key) => {
+    if (hasDefinedConfigValue(config, key)) acc[key] = config[key]
+    return acc
+  }, {})
+
+const getDefinedConfigKeys = (config: Record<string, unknown>, keys: string[]) =>
+  keys.filter((key) => hasDefinedConfigValue(config, key))
+
+const formatWarningMetadata = (metadata: WarningMetadata = {}) =>
+  Object.entries(metadata)
+    .filter(([, value]) => value !== undefined && value !== null && value !== '')
+    .map(([key, value]) => `${key}: ${String(value)}`)
+    .join(', ')
+
+const formatWarningReference = (metadata?: WarningMetadata) => {
+  const metadataText = formatWarningMetadata(metadata)
+  return metadataText ? ` [${metadataText}]` : ''
+}
+
+const normalizeSpringConfig = (
+  config: Record<string, unknown>,
+  errorReference: string,
+  warningMetadata?: WarningMetadata,
+): Record<string, unknown> => {
+  assertConfigValue(config, 'toValue', errorReference)
+
+  const usedFamilies = springConfigFamilies.filter((keys) =>
+    keys.some((key) => hasDefinedConfigValue(config, key)),
+  )
+  if (usedFamilies.length <= 1) return config
+
+  const configWithoutSpringFamily = Object.entries(config).reduce((acc, [key, value]) => {
+    if (!springConfigFamilyKeys.has(key)) acc[key] = value
+    return acc
+  }, {})
+
+  // Match runtime normalization: physical spring values are most explicit,
+  // otherwise preserve BRICKS' historical tension/friction controls.
+  const resolvedFamily =
+    usedFamilies.find((keys) => keys.includes('stiffness')) ||
+    usedFamilies.find((keys) => keys.includes('tension')) ||
+    usedFamilies[0]
+  const resolvedFamilyKeys = getDefinedConfigKeys(config, resolvedFamily)
+  const droppedFamilyKeys = usedFamilies
+    .filter((keys) => keys !== resolvedFamily)
+    .flatMap((keys) => getDefinedConfigKeys(config, keys))
+
+  console.warn(
+    `[Warning] Resolved animation spring config${formatWarningReference(
+      warningMetadata,
+    )}: using ${resolvedFamilyKeys.join('/')}, dropping ${droppedFamilyKeys.join('/')}`,
+  )
+
+  return {
+    ...configWithoutSpringFamily,
+    ...pickDefinedConfigValues(config, resolvedFamily),
+  }
+}
+
+const compileAnimationConfig = (
+  animationType: CompiledAnimationType,
+  config: unknown,
+  errorReference: string,
+  warningMetadata?: WarningMetadata,
+) => {
+  if (!isRecord(config)) {
+    throw new Error(`Invalid animation config ${errorReference}: config must be an object`)
+  }
+
+  const compiledConfig = compileProperty(omit(config, '__type'), errorReference)
+
+  if (!isRecord(compiledConfig)) {
+    throw new Error(`Invalid animation config ${errorReference}: config must compile to an object`)
+  }
+
+  if (animationType === 'timing') {
+    assertConfigValue(compiledConfig, 'toValue', errorReference)
+  } else if (animationType === 'spring') {
+    return normalizeSpringConfig(compiledConfig, errorReference, warningMetadata)
+  } else if (animationType === 'decay') {
+    assertConfigValue(compiledConfig, 'velocity', errorReference)
+  }
+
+  return compiledConfig
 }
 
 const compileFrame = (frame: Canvas['items'][number]['frame']) => ({
@@ -324,9 +493,12 @@ const preloadTypes = [
   'media-resource-audio',
   'media-resource-file',
   'lottie-file-uri',
+  'rive-file-uri',
   'ggml-model-asset',
   'gguf-model-asset',
   'binary-asset',
+  'mlx-model-asset',
+  'scene3d-objects',
 ]
 
 const compileKind = (kind: Data['kind']) => {
@@ -353,9 +525,10 @@ const compileKind = (kind: Data['kind']) => {
 }
 
 const compileRemoteUpdate = (remoteUpdate: Data['remoteUpdate']) => {
-  if (!remoteUpdate) return {}
-  if (remoteUpdate.type === 'auto') return { enable_remote_update: true }
+  if (!remoteUpdate) return { bank_type: 'none' }
+  if (remoteUpdate.type === 'auto') return { bank_type: 'create', enable_remote_update: true }
   return {
+    bank_type: remoteUpdate.type === 'device-specific' ? 'create-device-specific' : 'global',
     enable_remote_update: true,
     ...(remoteUpdate.type === 'device-specific' ? { use_remote_id_prefix: true } : {}),
     ...(remoteUpdate.type === 'global-data' ? { global_remote_update_prop: remoteUpdate.id } : {}),
@@ -485,6 +658,7 @@ const compileAutomationTest = (
 
   return {
     id: testId,
+    alias: test.alias,
     title: test.title,
     hide_short_ref: test.hideShortRef,
     timeout: test.timeout,
@@ -538,8 +712,38 @@ const compileAutomation = (automationMap: AutomationMap) =>
     }),
   )
 
+// Record the minimal compiled-config delta this compile produced to the shared audit
+// log (`.bricks/edits.jsonl`), so editing files directly and running `bun compile`
+// leaves the same trail as the MCP source-editing tools. Maintained only in the
+// editing-tools context (`BRICKS_CTOR_ENABLE_EDITING_TOOLS`); the source-editing tools
+// turn it off for their verify compiles (see _verify.ts) so a tool edit records one
+// richer entry instead of an extra generic compile entry. Also silent when there is no
+// prior build to diff against (fresh projects, package tests, tooling outside a project).
+const recordConfigChange = async (previousConfig: unknown, config: unknown) => {
+  if (previousConfig == null) return
+  if (!isTruthyEnv(process.env.BRICKS_CTOR_ENABLE_EDITING_TOOLS)) return
+  // The baseline was parsed from JSON; `computeConfigChange` applies the same
+  // JSON-omitted-field rules lazily so compile avoids cloning the full config.
+  const change = computeConfigChange(previousConfig, config)
+  if (change.status !== 'ok') return
+  await appendEditRecord(process.cwd(), {
+    ts: new Date().toISOString(),
+    tool: 'compile',
+    provenance: editProvenance(),
+    outcome: 'ok',
+    summary:
+      change.opCount === 0
+        ? 'compile: no config change'
+        : `compile: ${change.opCount} config op(s)`,
+    configChange: change,
+  }).catch(() => undefined)
+}
+
 export const compile = async (app: Application) => {
   await new Promise((resolve) => setImmediate(resolve, 0))
+  // Snapshot the prior build artifact before the caller's compile.ts overwrites it, so
+  // the config change introduced by this compile can be recorded on return.
+  const previousConfig = await readBuildConfig(process.cwd())
   const timestamp = Date.now()
   // Pre-index subspace ids so the canvas-item validation below stays O(1).
   const subspaceIdSet = new Set(app.subspaces.map((s) => s.id))
@@ -559,7 +763,14 @@ export const compile = async (app: Application) => {
       // validation (root_canvas_id is required before the conditional
       // schema fix is published).
       if (subspace.module?.link) {
-        const placeholderCanvasId = makeId('canvas')
+        // Seed the placeholder id from the (stable) subspace id. `makeId('canvas')` would take
+        // the count-fallback branch (a process-global counter that is never reset), so the
+        // placeholder id depended on how many prior count-fallback ids had been minted — making
+        // it differ between recompiles and breaking compile's byte-stable-output contract
+        // (phantom config-change ops). `makeSeededId` keeps no global state, so identical source
+        // recompiles to an identical id. (`makeId('canvas', alias)` would instead throw
+        // "Duplicate makeId alias" on the second compile in a long-lived process.)
+        const placeholderCanvasId = makeSeededId('canvas', `${subspaceId}:module-placeholder`)
         subspaceMap[subspaceId] = {
           title: subspace.title,
           description: subspace.description,
@@ -629,39 +840,63 @@ export const compile = async (app: Application) => {
             `(animation index: ${animationIndex}, subspace: ${subspaceId})`,
           )
 
-          if (animation.__typename === 'Animation') {
+          const animationTypename = animation.__typename
+          if (animationTypename === 'Animation') {
             const animationDef = animation as AnimationDef
+            const animationErrorReference = `(animation: ${animationId}, subspace ${subspaceId})`
+            const animationWarningMetadata = {
+              animationIndex,
+              animationTitle: animationDef.title,
+              animationAlias: animationDef.alias,
+              animationProperty: animationDef.property,
+              subspaceIndex,
+              subspaceTitle: subspace.title,
+            }
+            const animationType = getAnimationType(animationDef.config, animationErrorReference)
             map[animationId] = {
               alias: animationDef.alias,
               title: animationDef.title,
               description: animationDef.description,
               hide_short_ref: animationDef.hideShortRef,
               animationRunType: animationDef.runType,
-              property: animationDef.property,
-              type: animationTypeMap[animationDef.config.__type],
-              config: compileProperty(
-                omit(animationDef.config, '__type'),
-                `(animation: ${animationId}, subspace ${subspaceId})`,
+              property: assertAnimationProperty(animationDef.property, animationErrorReference),
+              type: animationType,
+              config: compileAnimationConfig(
+                animationType,
+                animationDef.config,
+                animationErrorReference,
+                animationWarningMetadata,
               ),
             }
-          } else if (animation.__typename === 'AnimationCompose') {
+          } else if (animationTypename === 'AnimationCompose') {
             const animationDef = animation as AnimationComposeDef
+            const animationErrorReference = `(animation: ${animationId}, subspace ${subspaceId})`
             map[animationId] = {
               alias: animationDef.alias,
               title: animationDef.title,
               description: animationDef.description,
               hide_short_ref: animationDef.hideShortRef,
               animationRunType: animationDef.runType,
-              compose_type: animationDef.composeType,
+              compose_type: assertAnimationComposeType(
+                animationDef.composeType,
+                animationErrorReference,
+              ),
               item_list: animationDef.items.map((item, index) => {
                 const innerAnimation = item()
-                if (!innerAnimation?.id)
-                  throw new Error(
-                    `Invalid animation index: ${index} (animation: ${innerAnimation.id}, subspace ${subspaceId})`,
-                  )
-                return { animation_id: innerAnimation.id }
+                const innerAnimationId = assertEntryId(
+                  innerAnimation?.id,
+                  'ANIMATION',
+                  `(animation item index: ${index}, animation: ${animationId}, subspace ${subspaceId})`,
+                )
+                return { animation_id: innerAnimationId }
               }),
             }
+          } else {
+            throw new Error(
+              `Invalid animation typename (animation: ${animationId}, subspace ${subspaceId}): ${String(
+                animationTypename,
+              )}`,
+            )
           }
           return map
         }, {}),
@@ -1009,7 +1244,7 @@ export const compile = async (app: Application) => {
             ...compileRemoteUpdate(data.remoteUpdate),
             routing: data.routing,
             schema: data.schema,
-            type: data.type,
+            type: data.type === 'boolean' ? 'bool' : data.type,
             ...compileKind(data.kind),
             value: compileProperty(data.value, `(data: ${dataId}, subspace ${subspaceId})`),
             event_map: compileEvents('PROPERTY_BANK', data.events || {}, {
@@ -1029,6 +1264,7 @@ export const compile = async (app: Application) => {
           )
 
           const calc: any = {
+            alias: dataCalc.alias,
             title: dataCalc.title,
             description: dataCalc.description,
             hide_short_ref: dataCalc.hideShortRef,
@@ -1227,12 +1463,7 @@ export const compile = async (app: Application) => {
                 : null,
             }
 
-            Object.assign(
-              calc,
-              generateCalulationMap(calc.script_config, {
-                snapshotMode: process.env.BRICKS_SNAPSHOT_MODE === '1',
-              }),
-            )
+            Object.assign(calc, generateCalulationMap(calc.script_config, dataCalcId))
           }
           map[dataCalcId] = calc
           return map
@@ -1269,10 +1500,20 @@ export const compile = async (app: Application) => {
     automation_map: compiledAutomationMap || app.metadata?.TEMP_automation_map || {},
     update_timestamp: timestamp,
   }
+  await recordConfigChange(previousConfig, config)
   return config
 }
 
 export const checkConfig = async (configPath: string) => {
-  const { sh } = await import('../tools/_shell')
-  await sh`bricks app check-config ${configPath}`
+  // --validate-automation surfaces broken automation_map / test_map refs early,
+  // which catches agent-authored automations that reference deleted bricks.
+  await sh`bricks app check-config --validate-automation ${configPath}`
+  // Doctor adds semantic lint checks after structural validation. Warnings are
+  // surfaced in the compile log, but only errors fail by default. Older published
+  // bricks-cli builds lack `app doctor` — skip rather than fail the compile.
+  const doctor = await sh`bricks app doctor --validate-automation ${configPath}`.nothrow()
+  if (doctor.exitCode !== 0) {
+    if (/unknown command/i.test(doctor.stderr?.toString() ?? '')) return
+    throw new Error(`bricks app doctor failed with exit ${doctor.exitCode}`)
+  }
 }