@fugood/bricks-project 2.24.0-beta.6 → 2.24.0-beta.8

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "@fugood/bricks-project",
-  "version": "2.24.0-beta.6",
+  "version": "2.24.0-beta.8",
   "main": "index.ts",
   "scripts": {
     "build": "bun scripts/build.js"
   },
   "dependencies": {
-    "@fugood/bricks-cli": "^2.24.0-beta.6",
+    "@fugood/bricks-cli": "^2.24.0-beta.8",
     "@huggingface/gguf": "^0.3.2",
     "@iarna/toml": "^3.0.0",
     "@modelcontextprotocol/sdk": "^1.15.0",
@@ -19,5 +19,5 @@
     "lodash": "^4.17.4",
     "uuid": "^8.3.1"
   },
-  "gitHead": "893f3e56ec87e71bfc093434c9a8517fca6c0f99"
+  "gitHead": "c611e18b2c6dccf46a84efd88b959bf5c2cba3ad"
 }

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

@@ -4,15 +4,16 @@ E2E testing and scheduled execution for BRICKS applications. Simulates user beha
 
 ## Automation Types
 
-| Type | Description |
-|------|-------------|
-| `launch` | Run on application launch (restarts app when run manually) |
-| `anytime` | Execute anytime via manual trigger |
-| `cron` | Scheduled execution using crontab expressions |
+| Type      | Description                                                |
+| --------- | ---------------------------------------------------------- |
+| `launch`  | Run on application launch (restarts app when run manually) |
+| `anytime` | Execute anytime via manual trigger                         |
+| `cron`    | Scheduled execution using crontab expressions              |
 
 ## Simulation Actions
 
 Automations can simulate:
+
 - **Brick Press**: Tap/click on bricks
 - **Key Events**: Key up/down for keyboard input
 - **HTTP Request**: API calls
@@ -21,6 +22,7 @@ Automations can simulate:
 ## Assertions
 
 Automations can validate:
+
 - **Brick Exists**: Check if brick is rendered
 - **Event Triggered**: Verify event from Brick/Generator/Canvas
 - **Canvas Changed**: Confirm canvas navigation
@@ -100,25 +102,58 @@ const testLoginFlow: AutomationTest = {
 
 ## Test Methods
 
-| Method | Signature | Description |
-|--------|-----------|-------------|
-| `brick_press` | `[subspace, brick, options?]` | Simulate brick press |
-| `brick_exists` | `[subspace, brick, frame?]` | Check brick exists |
-| `wait_until_brick_exists` | `[subspace, brick, timeout?, frame?]` | Wait for brick |
-| `wait_until_event_trigger` | `[subspace, sender, eventKey, timeout?]` | Wait for event |
-| `wait_until_canvas_change` | `[subspace, canvas, timeout?]` | Wait for canvas |
-| `keydown` | `[keyCode, pressedKey?, flags?]` | Key down event |
-| `keyup` | `[keyCode, pressedKey?, flags?]` | Key up event |
-| `http_request` | `[url, options?]` | HTTP request |
-| `assert_property` | `[subspace, property, value]` | Assert data value |
-| `wait_until_property_change` | `[subspace, property, value, timeout?]` | Wait for value |
-| `execute_action` | `[subspace, handler, action, params?, options?]` | Execute action |
-| `match_screenshot` | `[name, threshold?, maxRetry?]` | Screenshot compare |
-| `delay` | `[subspace?, property?, defaultValue?]` | Delay execution |
+| Method                       | Signature                                        | Description          |
+| ---------------------------- | ------------------------------------------------ | -------------------- |
+| `brick_press`                | `[subspace, brick, options?]`                    | Simulate brick press |
+| `brick_exists`               | `[subspace, brick, frame?]`                      | Check brick exists   |
+| `wait_until_brick_exists`    | `[subspace, brick, timeout?, frame?]`            | Wait for brick       |
+| `wait_until_event_trigger`   | `[subspace, sender, eventKey, timeout?]`         | Wait for event       |
+| `wait_until_canvas_change`   | `[subspace, canvas, timeout?]`                   | Wait for canvas      |
+| `keydown`                    | `[keyCode, pressedKey?, flags?]`                 | Key down event       |
+| `keyup`                      | `[keyCode, pressedKey?, flags?]`                 | Key up event         |
+| `http_request`               | `[url, options?]`                                | HTTP request         |
+| `assert_property`            | `[subspace, property, value]`                    | Assert data value    |
+| `wait_until_property_change` | `[subspace, property, value, timeout?]`          | Wait for value       |
+| `execute_action`             | `[subspace, handler, action, params?, options?]` | Execute action       |
+| `match_screenshot`           | `[name, threshold?, maxRetry?]`                  | Screenshot compare   |
+| `delay`                      | `[subspace?, property?, defaultValue?]`          | Delay execution      |
+
+### 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.
+
+```typescript
+// CORRECT — use runtime event property key
+run: ['execute_action', () => subspace0, bricks.bInput.id, 'BRICK_TEXT_INPUT_SET_TEXT',
+  { BRICK_TEXT_INPUT_TEXT: 'hello' }]
+
+// WRONG — action config input name doesn't work in automation
+run: ['execute_action', () => subspace0, bricks.bInput.id, 'BRICK_TEXT_INPUT_SET_TEXT',
+  { text: 'hello' }]
+```
+
+Reference `event-props.ts` for the correct runtime keys (e.g., `BRICK_TEXT_INPUT_TEXT`, `GENERATOR_MQTT_PAYLOAD`).
+
+### Prefer UI Interactions Over Direct Generator Calls
+
+For realistic E2E testing, prefer simulating user actions (set text input + press button) over calling generator actions directly:
+
+```typescript
+// GOOD — simulates real user behavior
+{ run: ['execute_action', () => sub, bricks.bInput.id, 'BRICK_TEXT_INPUT_SET_TEXT',
+    { BRICK_TEXT_INPUT_TEXT: 'hello' }] },
+{ run: ['brick_press', () => sub, () => bricks.bSendBtn] },
+{ run: ['wait_until_property_change', () => sub, () => data.dPayload, 'hello', 10000] },
+
+// AVOID — bypasses UI, doesn't test the full flow
+{ run: ['execute_action', () => sub, generators.gClient.id, 'GENERATOR_MQTT_PUBLISH',
+    { topic: 'test', payload: 'hello', qos: '0' }] },
+```
 
 ## Recording Automations
 
 In BRICKS Editor Preview mode:
+
 1. Perform operations normally
 2. Open menu (right-bottom corner)
 3. Select "Record Events as Automation"
@@ -127,12 +162,15 @@ In BRICKS Editor Preview mode:
 ## Running Automations
 
 ### Manual Run
+
 `Menu` → `Automations` → Select automation → `Run`
 
 ### On Launch
+
 `Bind Device` → `Select Automation` (only `launch` or `cron` types)
 
 ### Scheduled (Cron)
+
 `Bind Device` → `Cron Automation` (allows multi-select)
 
 Use [crontab.guru](https://crontab.guru) to build cron expressions.
@@ -156,6 +194,7 @@ Visual regression testing with screenshot comparison:
 ```
 
 Screenshots can be stored:
+
 - Local file system
 - Media Flow workspace
 
@@ -165,11 +204,18 @@ First run captures baseline. Use "Run with Update" to update baseline.
 
 Automations work with Modules. Use Manual Run in Preview mode for module testing.
 
+## Important Notes
+
+- **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'`.
+- **handler in execute_action**: Pass the entity's `.id` string (e.g., `bricks.bInput.id`), not a getter function.
+
 ## Best Practices
 
 1. **Test culture**: Create automations for every significant flow
 2. **CI/CD integration**: Use `launch` automations for deployment validation
-3. **Incremental waits**: Use `EXPECT_*` steps with appropriate timeouts
+3. **Incremental waits**: Use `wait_until_property_change` with appropriate timeouts
 4. **Visual testing**: Add screenshot comparisons for critical UI states
 5. **Cron monitoring**: Schedule health checks for production displays
 6. **Isolation**: Each automation should be independent and idempotent
+7. **UI-first testing**: Simulate real user interactions (text input, button press) rather than calling generators directly

package/tools/preview-main.mjs

@@ -42,13 +42,17 @@ let config = JSON.parse(await readFile(`${cwd}/.bricks/build/application-config.
 let testId = values['test-id'] || null
 if (!testId && values['test-title-like']) {
   const titleLike = values['test-title-like'].toLowerCase()
-  const testMap = config.test_map || {}
-  const found = Object.entries(testMap).find(([, test]) =>
-    test.title?.toLowerCase().includes(titleLike),
-  )
-  if (found) {
-    ;[testId] = found
-  } else {
+  const automationMap = config.automation_map || {}
+  for (const group of Object.values(automationMap)) {
+    const found = Object.entries(group.map || {}).find(([, test]) =>
+      test.title?.toLowerCase().includes(titleLike),
+    )
+    if (found) {
+      ;[testId] = found
+      break
+    }
+  }
+  if (!testId) {
     throw new Error(`No automation found matching title: ${values['test-title-like']}`)
   }
 }

package/utils/calc.ts

@@ -33,11 +33,12 @@ export const generateDataCalculationMapEditorInfo = (
   nodes.forEach((node) => {
     // Count and track inputs
     if ('inputs' in node) {
-      const inputs = node.inputs.reduce((sum, input) => {
-        if (input === null) return sum
-        if (Array.isArray(input)) return sum + input.length
-        return sum + 1
-      }, 0)
+      let inputs = 0
+      for (const input of node.inputs) {
+        if (input === null) continue
+        if (Array.isArray(input)) inputs += input.length
+        else inputs += 1
+      }
       inputCounts.set(node, inputs)
 
       // Track connections
@@ -60,11 +61,12 @@ export const generateDataCalculationMapEditorInfo = (
 
     // Count outputs
     if ('outputs' in node) {
-      const outputs = node.outputs.reduce((sum, output) => {
-        if (output === null) return sum
-        if (Array.isArray(output)) return sum + output.length
-        return sum + 1
-      }, 0)
+      let outputs = 0
+      for (const output of node.outputs) {
+        if (output === null) continue
+        if (Array.isArray(output)) outputs += output.length
+        else outputs += 1
+      }
       outputCounts.set(node, outputs)
     } else {
       outputCounts.set(node, 0)