fix(client): support menu data override, fix #260

packages/client/app/layout/menu-item.vue

@@ -8,7 +8,7 @@
 
 <script lang="ts" setup>
 
-import { LegacyMenuItem, MaybeGetter, useContext } from '@koishijs/client'
+import { MaybeGetter, useContext } from '@koishijs/client'
 import { computed } from 'vue'
 
 const props = defineProps<{
@@ -30,15 +30,17 @@ const disabled = computed(() => {
   return toValue(props.item.disabled)
 })
 
+const scope = computed(() => ctx.$action.createScope({
+  [props.menuKey]: props.menuData,
+}))
+
 function toValue<T>(getter: MaybeGetter<T>): T {
   if (typeof getter !== 'function') return getter
-  return (getter as any)(ctx.$action.createScope())
+  return (getter as any)(scope.value)
 }
 
 function trigger() {
-  return props.item.action(ctx.$action.createScope({
-    [props.menuKey]: props.menuData,
-  }))
+  return props.item.action(scope.value)
 }
 
 </script>

packages/client/client/plugins/action.ts

@@ -140,17 +140,22 @@ export default class ActionService extends Service {
     })
   }
 
-  createScope(scope = this.ctx.internal.scope, prefix = '') {
-    return new Proxy({}, {
-      get: (target, key) => {
-        if (typeof key === 'symbol') return target[key]
-        key = prefix + key
-        if (key in scope) return toValue(scope[key])
-        const _prefix = key + '.'
-        if (Object.keys(scope).some(k => k.startsWith(_prefix))) {
-          return this.createScope(scope, key + '.')
-        }
-      },
-    })
+  createScope(override = {}) {
+    const scope = { ...this.ctx.internal.scope, ...override }
+    return createScope(scope)
   }
 }
+
+function createScope(scope: Store<ActionContext>, prefix = '') {
+  return new Proxy({}, {
+    get: (target, key) => {
+      if (typeof key === 'symbol') return target[key]
+      key = prefix + key
+      if (key in scope) return toValue(scope[key])
+      const _prefix = key + '.'
+      if (Object.keys(scope).some(k => k.startsWith(_prefix))) {
+        return createScope(scope, key + '.')
+      }
+    },
+  })
+}