refactor: simplify expressionEvaluator user fallback; document context-fresh closure

Agent-Logs-Url: https://github.com/objectstack-ai/objectui/sessions/9bbb4b9a-dc45-41ea-b03e-55d3172b7dde

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

Author: Copilot

packages/app-shell/src/console/AppContent.tsx

@@ -145,6 +145,13 @@ export function AppContent({ extraRoutes, extraRoutesNoApp }: AppContentProps =
     // The `objectName` param falls back to the action context's
     // `objectName` (set per view) so action buttons mounted inside an
     // ObjectView can omit it.
+    // NOTE on duplication below: each handler reads `runner.getContext()`
+    // INSIDE its closure (at action-invocation time) rather than once at
+    // registration. Hoisting the call outside the registrations would
+    // freeze the context to whatever it was when the effect last ran,
+    // breaking dynamic per-view `runner.updateContext({ objectName, ... })`
+    // calls (used by ObjectView / RecordDetailView). Keep the call where
+    // it is.
     runner.registerHandler('navigate_create', async (action: any) => {
       const ctx = runner.getContext?.() ?? {};
       const result = resolveNavigateCreateUrl({

packages/app-shell/src/views/RecordFormPage.tsx

@@ -139,11 +139,13 @@ export function RecordFormPage({ mode }: RecordFormPageProps) {
   const expressionEvaluator = useMemo(
     () =>
       new ExpressionEvaluator({
-        user: user ? expressionUser : {},
+        // expressionUser already handles the anonymous fallback, so we can
+        // pass it through unconditionally.
+        user: expressionUser,
         app: { name: appName },
         data: {},
       }),
-    [user, expressionUser, appName],
+    [expressionUser, appName],
   );
 
   // Resolve the field list using the same visibility-aware logic as the