Address code review feedback: improve types, comments, and naming

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

apps/console/objectstack.shared.ts

@@ -102,8 +102,9 @@ export const sharedConfig = {
 const allConfigs = [crmConfig, todoConfig, kitchenSinkConfig];
 
 // defineStack() validates the config but strips non-standard properties like
-// listViews and actions from objects. Re-compose after validation so the
-// runtime protocol serves objects with their view and action definitions.
+// listViews and actions from objects. A second composeStacks pass restores
+// these runtime properties onto the validated objects. This double-pass is
+// necessary because defineStack's Zod schema doesn't preserve custom fields.
 const validated = defineStack(sharedConfig as Parameters<typeof defineStack>[0]);
 export default {
   ...validated,

objectstack.config.ts

@@ -58,7 +58,10 @@ const mergedApp = defineStack({
   pages: composed.pages,
 } as any);
 
-// Re-compose after defineStack validation to restore listViews and actions
+// defineStack() validates the config but strips non-standard properties like
+// listViews and actions from objects. A second composeStacks pass restores
+// these runtime properties onto the validated objects. This double-pass is
+// necessary because defineStack's Zod schema doesn't preserve custom fields.
 const mergedAppWithViews = {
   ...mergedApp,
   objects: composeStacks([

packages/core/src/utils/compose-stacks.ts

@@ -11,13 +11,19 @@
  * - Handles duplicate object names via `objectConflict` option
  * - Merges stack-level views (listViews) into corresponding objects
  * - Merges stack-level actions into objects via explicit `objectName` field
+ *
+ * Note: Uses Record<string, any> because inputs can be raw defineStack output,
+ * plugin getConfig() results, or partial stacks with only views/actions.
  */
 
 export interface ComposeStacksOptions {
   /** How to resolve two objects with the same `name`. Default: 'override' (last wins). */
   objectConflict?: 'override' | 'error';
 }
 
+/** A partial or full stack definition used as input to composeStacks. */
+export type StackInput = Record<string, any>;
+
 /**
  * Compose multiple stack definitions into a single merged definition.
  *
@@ -26,7 +32,7 @@ export interface ComposeStacksOptions {
  * @returns       A merged stack definition (unvalidated — call defineStack if needed)
  */
 export function composeStacks(
-  stacks: Record<string, any>[],
+  stacks: StackInput[],
   options: ComposeStacksOptions = {},
 ): Record<string, any> {
   const { objectConflict = 'override' } = options;
@@ -57,7 +63,7 @@ export function composeStacks(
   for (const obj of allObjects) {
     const name = obj?.name;
     if (!name) {
-      objectMap.set(`__anon_${objectMap.size}`, obj);
+      objectMap.set(`__unnamed_${objectMap.size}`, obj);
       continue;
     }
     if (objectMap.has(name)) {