Method/function props of exported objects are not server functions

This is a follow-up fix for #72969. Method or function properties of
exported objects in `"use cache"` or `"use server"` files must not be
compiled into server functions.

crates/next-custom-transforms/src/transforms/server_actions.rs

@@ -1190,13 +1190,15 @@ impl<C: Comments> VisitMut for ServerActions<C> {
 
     fn visit_mut_prop_or_spread(&mut self, n: &mut PropOrSpread) {
         let old_arrow_or_fn_expr_ident = self.arrow_or_fn_expr_ident.clone();
+        let old_in_exported_expr = self.in_exported_expr;
 
         match n {
             PropOrSpread::Prop(box Prop::KeyValue(KeyValueProp {
                 key: PropName::Ident(ident_name),
                 value: box Expr::Arrow(_) | box Expr::Fn(_),
                 ..
             })) => {
+                self.in_exported_expr = false;
                 self.arrow_or_fn_expr_ident = Some(ident_name.clone().into());
             }
             PropOrSpread::Prop(box Prop::Method(MethodProp { key, .. })) => {
@@ -1205,7 +1207,9 @@ impl<C: Comments> VisitMut for ServerActions<C> {
                     self.arrow_or_fn_expr_ident = Some(ident_name.clone().into());
                 }
                 self.rewrite_expr_to_proxy_expr = None;
+                self.in_exported_expr = false;
                 n.visit_mut_children_with(self);
+                self.in_exported_expr = old_in_exported_expr;
                 if let Some(expr) = &self.rewrite_expr_to_proxy_expr {
                     *n = PropOrSpread::Prop(Box::new(Prop::KeyValue(KeyValueProp {
                         key,
@@ -1230,6 +1234,7 @@ impl<C: Comments> VisitMut for ServerActions<C> {
 
         n.visit_mut_children_with(self);
         self.arrow_or_fn_expr_ident = old_arrow_or_fn_expr_ident;
+        self.in_exported_expr = old_in_exported_expr;
     }
 
     fn visit_mut_callee(&mut self, n: &mut Callee) {

crates/next-custom-transforms/tests/fixture/server-actions/server/56/input.js

@@ -0,0 +1,13 @@
+'use cache'
+
+// No method nor function property should be considered a cache function.
+export const obj = {
+  foo() {
+    return 1
+  },
+  async bar() {
+    return 2
+  },
+  baz: () => 2,
+  qux: async () => 3,
+}

crates/next-custom-transforms/tests/fixture/server-actions/server/56/output.js

@@ -0,0 +1,14 @@
+/* __next_internal_action_entry_do_not_use__ {} */ import { registerServerReference } from "private-next-rsc-server-reference";
+import { encryptActionBoundArgs, decryptActionBoundArgs } from "private-next-rsc-action-encryption";
+import { cache as $$cache__ } from "private-next-rsc-cache-wrapper";
+// No method nor function property should be considered a cache function.
+export const obj = {
+    foo () {
+        return 1;
+    },
+    async bar () {
+        return 2;
+    },
+    baz: ()=>2,
+    qux: async ()=>3
+};