@fugood/bricks-ctor 2.25.0-beta.56 → 2.25.0-beta.58

package/compile/__tests__/index.test.js

@@ -249,6 +249,37 @@ describe('compile event handlers', () => {
   })
 })
 
+describe('compile expanded state', () => {
+  test('defaults canvas, bricks, and data collapsed for generated configs', async () => {
+    const config = await compile(makeApp([], [makeData(undefined)]))
+
+    expect(config.subspace_map[SUBSPACE_ID]._expanded).toEqual({
+      brick: false,
+      generator: true,
+      canvas: { [CANVAS_ID]: false },
+      property_bank: false,
+      property_bank_calc: true,
+    })
+  })
+
+  test('preserves explicit source unexpanded state when present', async () => {
+    const app = makeApp()
+    app.rootSubspace.unexpanded = {
+      generator: true,
+      dataCalculation: true,
+    }
+
+    const config = await compile(app)
+    const expanded = config.subspace_map[SUBSPACE_ID]._expanded
+
+    expect(expanded.brick).toBe(true)
+    expect(expanded.generator).toBe(false)
+    expect(expanded.canvas).toBeUndefined()
+    expect(expanded.property_bank).toBe(true)
+    expect(expanded.property_bank_calc).toBe(false)
+  })
+})
+
 describe('compile data remote update', () => {
   test.each([
     [undefined, { bank_type: 'none' }],

package/compile/index.ts

@@ -751,6 +751,17 @@ const compileAutomation = (automationMap: AutomationMap) =>
     }),
   )
 
+const buildDefaultExpandedState = (subspace: Subspace) => ({
+  brick: false,
+  generator: true,
+  canvas: (subspace.canvases || []).reduce((acc, canvas) => {
+    if (canvas?.id) acc[canvas.id] = false
+    return acc
+  }, {}),
+  property_bank: false,
+  property_bank_calc: true,
+})
+
 // 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
@@ -868,7 +879,7 @@ export const compile = async (app: Application) => {
               property_bank: !subspace.unexpanded.data,
               property_bank_calc: !subspace.unexpanded.dataCalculation,
             }
-          : undefined,
+          : buildDefaultExpandedState(subspace),
         layout: {
           width: subspace.layout?.width,
           height: subspace.layout?.height,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fugood/bricks-ctor",
-  "version": "2.25.0-beta.56",
+  "version": "2.25.0-beta.58",
   "main": "index.ts",
   "scripts": {
     "typecheck": "tsc --noEmit",
@@ -11,7 +11,7 @@
     "@babel/parser": "7.28.5",
     "@babel/traverse": "7.28.5",
     "@babel/types": "7.28.5",
-    "@fugood/bricks-cli": "^2.25.0-beta.53",
+    "@fugood/bricks-cli": "^2.25.0-beta.58",
     "@huggingface/gguf": "^0.3.2",
     "@iarna/toml": "^3.0.0",
     "@modelcontextprotocol/sdk": "^1.15.0",
@@ -29,5 +29,5 @@
   "peerDependencies": {
     "oxfmt": "^0.36.0"
   },
-  "gitHead": "5d3de40404adc334a9a64046c5514e1cd86132db"
+  "gitHead": "355da5754f1136c10e073460af4727dae94f7e30"
 }

package/tools/mcp-tools/_editing-helpers.ts

@@ -56,3 +56,43 @@ export const removeObjectProperty = (object: t.ObjectExpression, key: string) =>
   })
   if (index >= 0) object.properties.splice(index, 1)
 }
+
+export const insertImport = (ast: t.File, declaration: t.ImportDeclaration) => {
+  const body = ast.program.body
+  const lastImportIndex = body.findLastIndex((statement) => t.isImportDeclaration(statement))
+  body.splice(lastImportIndex + 1, 0, declaration)
+}
+
+export const ensureBricksCtorImport = (
+  ast: t.File,
+  importKind: 'type' | 'value',
+  names: Iterable<string>,
+) => {
+  const missing = new Set(Array.from(names).filter(Boolean))
+  if (missing.size === 0) return
+
+  for (const statement of ast.program.body) {
+    if (!t.isImportDeclaration(statement) || statement.source.value !== 'bricks-ctor') continue
+    const isTypeImport = statement.importKind === 'type'
+    if ((importKind === 'type') !== isTypeImport) continue
+
+    statement.specifiers.forEach((specifier) => {
+      if (!t.isImportSpecifier(specifier)) return
+      const imported = specifier.imported
+      if (t.isIdentifier(imported)) missing.delete(imported.name)
+      if (t.isStringLiteral(imported)) missing.delete(imported.value)
+    })
+
+    missing.forEach((name) => {
+      statement.specifiers.push(t.importSpecifier(t.identifier(name), t.identifier(name)))
+    })
+    return
+  }
+
+  const declaration = t.importDeclaration(
+    Array.from(missing).map((name) => t.importSpecifier(t.identifier(name), t.identifier(name))),
+    t.stringLiteral('bricks-ctor'),
+  )
+  if (importKind === 'type') declaration.importKind = 'type'
+  insertImport(ast, declaration)
+}

package/tools/mcp-tools/data-calc-editing.ts

@@ -10,8 +10,10 @@ import { z } from 'zod'
 
 import { verifyProject } from './_verify'
 import {
+  ensureBricksCtorImport,
   getObjectProperty,
   getStringProperty,
+  insertImport,
   isRecord,
   makeObjectKey,
   oxfmtOptions,
@@ -328,12 +330,6 @@ const relativeImportSource = (fromFile: string, toFile: string) => {
   return rel.replace(/\.(ts|tsx|js|jsx)$/, '')
 }
 
-const insertImport = (ast: t.File, declaration: t.ImportDeclaration) => {
-  const body = ast.program.body
-  const lastImportIndex = body.findLastIndex((statement) => t.isImportDeclaration(statement))
-  body.splice(lastImportIndex + 1, 0, declaration)
-}
-
 const ensureNamespaceImport = (parsed: ParsedFile, source: string, preferredLocal: string) => {
   for (const statement of parsed.ast.program.body) {
     if (!t.isImportDeclaration(statement) || statement.source.value !== source) continue
@@ -352,40 +348,6 @@ const ensureNamespaceImport = (parsed: ParsedFile, source: string, preferredLoca
   return preferredLocal
 }
 
-const ensureBricksCtorImport = (
-  ast: t.File,
-  importKind: 'type' | 'value',
-  names: Iterable<string>,
-) => {
-  const missing = new Set(Array.from(names).filter(Boolean))
-  if (missing.size === 0) return
-
-  for (const statement of ast.program.body) {
-    if (!t.isImportDeclaration(statement) || statement.source.value !== 'bricks-ctor') continue
-    const isTypeImport = statement.importKind === 'type'
-    if ((importKind === 'type') !== isTypeImport) continue
-
-    statement.specifiers.forEach((specifier) => {
-      if (!t.isImportSpecifier(specifier)) return
-      const imported = specifier.imported
-      if (t.isIdentifier(imported)) missing.delete(imported.name)
-      if (t.isStringLiteral(imported)) missing.delete(imported.value)
-    })
-
-    missing.forEach((name) => {
-      statement.specifiers.push(t.importSpecifier(t.identifier(name), t.identifier(name)))
-    })
-    return
-  }
-
-  const declaration = t.importDeclaration(
-    Array.from(missing).map((name) => t.importSpecifier(t.identifier(name), t.identifier(name))),
-    t.stringLiteral('bricks-ctor'),
-  )
-  if (importKind === 'type') declaration.importKind = 'type'
-  insertImport(ast, declaration)
-}
-
 const ensureReadFileImport = (ast: t.File) => {
   for (const statement of ast.program.body) {
     if (!t.isImportDeclaration(statement) || statement.source.value !== 'node:fs/promises') {

package/tools/mcp-tools/entry-editing.ts

@@ -11,9 +11,11 @@ import { z } from 'zod'
 
 import { verifyProject } from './_verify'
 import {
+  ensureBricksCtorImport,
   getObjectProperty,
   getPropertyKeyName,
   getStringProperty,
+  insertImport,
   isRecord,
   makeObjectKey,
   oxfmtOptions,
@@ -576,13 +578,6 @@ const getTopLevelNames = (ast: t.File) => {
   return names
 }
 
-const insertImport = (ast: t.File, declaration: t.ImportDeclaration) => {
-  const lastImportIndex = ast.program.body.findLastIndex((statement) =>
-    t.isImportDeclaration(statement),
-  )
-  ast.program.body.splice(lastImportIndex + 1, 0, declaration)
-}
-
 const ensureNamespaceImport = (
   parsed: ParsedFile,
   namespace: string,
@@ -611,40 +606,6 @@ const ensureNamespaceImport = (
   return localName
 }
 
-const ensureBricksCtorImport = (
-  ast: t.File,
-  importKind: 'type' | 'value',
-  names: Iterable<string>,
-) => {
-  const missing = new Set(Array.from(names).filter(Boolean))
-  if (missing.size === 0) return
-
-  for (const statement of ast.program.body) {
-    if (!t.isImportDeclaration(statement) || statement.source.value !== 'bricks-ctor') continue
-    const isTypeImport = statement.importKind === 'type'
-    if ((importKind === 'type') !== isTypeImport) continue
-
-    statement.specifiers.forEach((specifier) => {
-      if (!t.isImportSpecifier(specifier)) return
-      const imported = specifier.imported
-      if (t.isIdentifier(imported)) missing.delete(imported.name)
-      if (t.isStringLiteral(imported)) missing.delete(imported.value)
-    })
-
-    missing.forEach((name) => {
-      statement.specifiers.push(t.importSpecifier(t.identifier(name), t.identifier(name)))
-    })
-    return
-  }
-
-  const declaration = t.importDeclaration(
-    Array.from(missing).map((name) => t.importSpecifier(t.identifier(name), t.identifier(name))),
-    t.stringLiteral('bricks-ctor'),
-  )
-  if (importKind === 'type') declaration.importKind = 'type'
-  insertImport(ast, declaration)
-}
-
 const applyPendingImports = (ctx: EditContext) => {
   ensureBricksCtorImport(ctx.parsed.ast, 'type', ctx.typeImports)
   ensureBricksCtorImport(ctx.parsed.ast, 'value', ctx.valueImports)

package/types/bricks/Items.d.ts

@@ -269,6 +269,7 @@ Default property:
                   y?: number | DataLink
                   width?: number | DataLink
                   height?: number | DataLink
+                  type?: string | DataLink
                   standbyMode?: 'custom' | 'top' | 'bottom' | 'left' | 'right' | DataLink
                   standbyFrame?: DataLink | {}
                   standbyOpacity?: number | DataLink
@@ -319,6 +320,7 @@ Default property:
                 }
             show?: string | DataLink
             pressToOpenDetail?: boolean | DataLink
+            pressToBackList?: boolean | DataLink
           }
       >
     | DataLink
@@ -346,6 +348,7 @@ Default property:
                   y?: number | DataLink
                   width?: number | DataLink
                   height?: number | DataLink
+                  type?: string | DataLink
                   standbyMode?: 'custom' | 'top' | 'bottom' | 'left' | 'right' | DataLink
                   standbyFrame?: DataLink | {}
                   standbyOpacity?: number | DataLink
@@ -395,6 +398,7 @@ Default property:
                   renderOutOfViewport?: boolean | DataLink
                 }
             show?: string | DataLink
+            pressToOpenDetail?: boolean | DataLink
             pressToBackList?: boolean | DataLink
           }
       >