@fugood/bricks-project 2.25.0-beta.23 → 2.25.0-beta.25

package/compile/index.ts

@@ -1460,5 +1460,7 @@ export const compile = async (app: Application) => {
 
 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}`
 }

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "@fugood/bricks-project",
-  "version": "2.25.0-beta.23",
+  "version": "2.25.0-beta.25",
   "main": "index.ts",
   "scripts": {
     "typecheck": "tsc --noEmit",
     "build": "bun scripts/build.js"
   },
   "dependencies": {
-    "@fugood/bricks-cli": "^2.25.0-beta.21",
+    "@fugood/bricks-cli": "^2.25.0-beta.25",
     "@huggingface/gguf": "^0.3.2",
     "@iarna/toml": "^3.0.0",
     "@modelcontextprotocol/sdk": "^1.15.0",

package/package.json.bak

@@ -1,13 +1,13 @@
 {
   "name": "@fugood/bricks-ctor",
-  "version": "2.25.0-beta.23",
+  "version": "2.25.0-beta.25",
   "main": "index.ts",
   "scripts": {
     "typecheck": "tsc --noEmit",
     "build": "bun scripts/build.js"
   },
   "dependencies": {
-    "@fugood/bricks-cli": "^2.25.0-beta.21",
+    "@fugood/bricks-cli": "^2.25.0-beta.25",
     "@huggingface/gguf": "^0.3.2",
     "@iarna/toml": "^3.0.0",
     "@modelcontextprotocol/sdk": "^1.15.0",

package/skills/bricks-ctor/rules/automations.md

@@ -118,6 +118,16 @@ const testLoginFlow: AutomationTest = {
 | `match_screenshot`           | `[name, threshold?, maxRetry?]`                  | Screenshot compare   |
 | `delay`                      | `[subspace?, property?, defaultValue?]`          | Delay execution      |
 
+In project TypeScript source, pass entity getters for BRICKS entities:
+
+```typescript
+run: ['brick_press', () => mainSubspace, () => bricks.bSubmitButton]
+run: ['wait_until_canvas_change', () => mainSubspace, () => canvases.cDone, 5000]
+run: ['assert_property', () => mainSubspace, () => data.dStep, 'done']
+```
+
+The compiler resolves these getters to the current generated IDs.
+
 ### execute_action Params
 
 The `params` object in `execute_action` uses **runtime event property keys** from `event-props.ts`, NOT the action config `input` names from type definitions.
@@ -208,6 +218,7 @@ Automations work with Modules. Use Manual Run in Preview mode for module testing
 
 - **Automation map key**: Always use `'AUTOMATION_MAP_DEFAULT'` as the automation map ID (not `makeId()`). The preview test runner reads from `automationMap['AUTOMATION_MAP_DEFAULT']?.map`.
 - **Valid makeId types**: Use `'test'` for AutomationTest, `'test_case'` for TestCase, `'test_var'` for TestVariable. Do NOT use `'automation_test'` or `'automation_test_map'`.
+- **Entity references in run arrays**: Use getter references (`() => subspace`, `() => bricks.bButton`, `() => data.dValue`) in TypeScript source so compile resolves fresh IDs.
 - **handler in execute_action**: Pass the entity's `.id` string (e.g., `bricks.bInput.id`), not a getter function.
 
 ## Best Practices

package/utils/id.ts

@@ -1,38 +1,83 @@
 import { v4 as uuid } from 'uuid'
+import { createHash } from 'node:crypto'
+import { readFileSync } from 'node:fs'
+import { join } from 'node:path'
 
 let count = 0
 
-// Used for snapshot mode
-const countUUID = () => {
-  const current = count
-  count += 1
-  return `00000000-0000-0000-0000-${current.toString().padStart(12, '0')}`
+type IdType =
+  | 'animation'
+  | 'brick'
+  | 'canvas'
+  | 'generator'
+  | 'data'
+  | 'switch'
+  | 'property_bank_command'
+  | 'property_bank_calc'
+  | 'dynamic-brick'
+  | 'automation_map'
+  | 'test'
+  | 'test_case'
+  | 'test_var'
+  | 'subspace'
+
+type IdOptions = {
+  snapshotMode?: boolean
+}
+
+const APPLICATION_ID_FALLBACK = 'unknown-application'
+
+const usedStableAliases = new Set<string>()
+let applicationIdCache: { cwd: string; id: string } | null = null
+
+const readApplicationId = () => {
+  const cwd = process.cwd()
+  if (applicationIdCache?.cwd === cwd) return applicationIdCache.id
+
+  let id = APPLICATION_ID_FALLBACK
+  try {
+    const app = JSON.parse(readFileSync(join(cwd, 'application.json'), 'utf8'))
+    if (typeof app.id === 'string' && app.id) id = app.id
+  } catch {
+    // `makeId` is also used from tests and utilities outside project roots.
+  }
+
+  applicationIdCache = { cwd, id }
+  return id
+}
+
+const hashToRandomBytes = (parts: string[]) => {
+  const hash = createHash('sha256').update(JSON.stringify(parts)).digest()
+  return new Uint8Array(hash.subarray(0, 16))
 }
 
-// Make a random id if not want to use fixed id
-export const makeId = (
-  type:
-    | 'animation'
-    | 'brick'
-    | 'canvas'
-    | 'generator'
-    | 'data'
-    | 'switch'
-    | 'property_bank_command'
-    | 'property_bank_calc'
-    | 'dynamic-brick'
-    | 'automation_map'
-    | 'test'
-    | 'test_case'
-    | 'test_var'
-    | 'subspace',
-  opts?: {
-    snapshotMode?: boolean
-  },
-) => {
+const makeStableUuid = (type: string, alias?: string) => {
+  const applicationId = readApplicationId()
+  const seed = alias ?? String(count)
+  if (alias === undefined) count += 1
+
+  if (alias !== undefined) {
+    const aliasKey = JSON.stringify([applicationId, type, alias])
+    if (usedStableAliases.has(aliasKey)) {
+      throw new Error(`Duplicate makeId alias '${alias}' for type '${type}'`)
+    }
+    usedStableAliases.add(aliasKey)
+  }
+
+  return uuid({
+    random: hashToRandomBytes([applicationId, type, seed]),
+  })
+}
+
+// Make stable ids by default; explicit snapshotMode: false preserves the random escape hatch.
+export const makeId = (type: IdType, aliasOrOpts?: string | IdOptions, opts?: IdOptions) => {
   if (type === 'subspace') {
     throw new Error('Currently subspace is not supported for ID generation, please use a fixed ID')
   }
+
+  const alias = typeof aliasOrOpts === 'string' ? aliasOrOpts : undefined
+  const options = typeof aliasOrOpts === 'string' ? opts : (aliasOrOpts ?? opts)
+
   let prefix = ''
   switch (type) {
     case 'animation':
@@ -76,5 +121,11 @@ export const makeId = (
       break
     default:
   }
-  return `${prefix}${opts?.snapshotMode ? countUUID() : uuid()}`
+
+  const useCountFallback = aliasOrOpts === undefined && opts === undefined
+  const id =
+    alias !== undefined || options?.snapshotMode || useCountFallback
+      ? makeStableUuid(type, alias)
+      : uuid()
+  return `${prefix}${id}`
 }