perf(appbuilder): reduce number of ListRemoteTestEvent calls by remot…

…e invoke #6559 Problem: Remote Invoke call ListRemoteTestEvent every time the webview is instantiated. Solution: Introduce a condition for which this api calls can be made thereby reducing the spike in number failures we see in our dashboard.
aws · Feb 12, 2025 · 7d5a339 · 7d5a339
1 parent 8126a56
commit 7d5a339
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/packages/core/src/lambda/vue/remoteInvoke/remoteInvoke.vue b/packages/core/src/lambda/vue/remoteInvoke/remoteInvoke.vue
@@ -67,6 +67,7 @@
                                 value="savedEvents"
                                 v-model="payload"
                                 name="payload_request"
+                                @change="$emit('loadRemoteTestEvents')"
                             />
                             <label class="label-selector" for="savedEvents"> Remote saved events</label>
                         </div>

diff --git a/packages/core/src/lambda/vue/remoteInvoke/remoteInvokeFrontend.ts b/packages/core/src/lambda/vue/remoteInvoke/remoteInvokeFrontend.ts
@@ -132,6 +132,21 @@ export default defineComponent({
                 }
             )
         },
+
+        async loadRemoteTestEvents() {
+            const shouldLoadEvents =
+                this.payload === 'savedEvents' &&
+                this.initialData.FunctionArn &&
+                this.initialData.FunctionRegion &&
+                !this.initialData.TestEvents
+
+            if (shouldLoadEvents) {
+                this.initialData.TestEvents = await client.listRemoteTestEvents(
+                    this.initialData.FunctionArn,
+                    this.initialData.FunctionRegion
+                )
+            }
+        },
     },
     mixins: [saveData],
 })