From 0a13b4605eabd6e4b96238904b4111b6b67e0616 Mon Sep 17 00:00:00 2001
From: Fabio Manganiello <fabio@manganiello.tech>
Date: Thu, 12 Oct 2023 02:49:51 +0200
Subject: [PATCH] [Execute panel] Procedures merged into actions.

Plus, a last big refactor/redesign for the panel's components.
---
 .../components/panels/Execute/ActionArgs.vue  | 133 ++++++++
 .../components/panels/Execute/ActionDoc.vue   |  40 +++
 .../src/components/panels/Execute/Index.vue   | 283 ++++--------------
 .../components/panels/Execute/Response.vue    |   3 +
 .../src/components/panels/Execute/common.scss |   3 +-
 .../backend/http/webapp/src/utils/Text.vue    |   4 +-
 6 files changed, 246 insertions(+), 220 deletions(-)
 create mode 100644 platypush/backend/http/webapp/src/components/panels/Execute/ActionArgs.vue
 create mode 100644 platypush/backend/http/webapp/src/components/panels/Execute/ActionDoc.vue

diff --git a/platypush/backend/http/webapp/src/components/panels/Execute/ActionArgs.vue b/platypush/backend/http/webapp/src/components/panels/Execute/ActionArgs.vue
new file mode 100644
index 000000000..d6c2f8bc6
--- /dev/null
+++ b/platypush/backend/http/webapp/src/components/panels/Execute/ActionArgs.vue
@@ -0,0 +1,133 @@
+<template>
+  <div class="args-body">
+    <div class="args-list"
+         v-if="Object.keys(action.args).length || action.supportsExtraArgs">
+      <!-- Supported action arguments -->
+      <div class="arg" :key="name" v-for="name in Object.keys(action.args)">
+        <label>
+          <input type="text"
+                 class="action-arg-value"
+                 :class="{required: action.args[name].required}"
+                 :disabled="running"
+                 :placeholder="name"
+                 :value="action.args[name].value"
+                 @input="onArgEdit(name, $event)"
+                 @focus="onSelect(name)">
+          <span class="required-flag" v-if="action.args[name].required">*</span>
+        </label>
+
+        <Argdoc :name="selectedArg"
+                :args="action.args[selectedArg]"
+                :doc="selectedArgdoc"
+                :loading="loading"
+                is-mobile
+                v-if="selectedArgdoc && selectedArg && name === selectedArg" />
+      </div>
+
+      <!-- Extra action arguments -->
+      <div class="extra-args" v-if="Object.keys(action.extraArgs).length">
+        <div class="arg extra-arg" :key="i" v-for="(arg, i) in action.extraArgs">
+          <label class="col-5">
+            <input type="text"
+                   class="action-extra-arg-name"
+                   placeholder="Name"
+                   :disabled="running"
+                   :value="arg.name"
+                   @input="onExtraArgNameEdit(i, $event.target.value)">
+          </label>
+          <label class="col-6">
+            <input type="text"
+                   class="action-extra-arg-value"
+                   placeholder="Value"
+                   :disabled="running"
+                   :value="arg.value"
+                   @input="onExtraArgValueEdit(i, $event.target.value)">
+          </label>
+          <label class="col-1 buttons">
+            <button type="button" class="action-extra-arg-del" title="Remove argument" @click="$emit('remove', i)">
+              <i class="fas fa-trash" />
+            </button>
+          </label>
+        </div>
+      </div>
+
+      <div class="add-arg" v-if="action.supportsExtraArgs">
+        <button type="button" title="Add an argument" @click="onArgAdd">
+          <i class="fas fa-plus" />
+        </button>
+      </div>
+    </div>
+
+    <Argdoc :name="selectedArg"
+            :args="action.args[selectedArg]"
+            :doc="selectedArgdoc"
+            :loading="loading"
+            v-if="selectedArgdoc && selectedArg" />
+  </div>
+</template>
+
+<script>
+import Argdoc from "./Argdoc"
+
+export default {
+  name: 'ActionArgs',
+  components: { Argdoc },
+  emits: [
+    'add',
+    'arg-edit',
+    'extra-arg-name-edit',
+    'extra-arg-value-edit',
+    'remove',
+    'select',
+  ],
+  props: {
+    action: Object,
+    loading: Boolean,
+    running: Boolean,
+    selectedArg: String,
+    selectedArgdoc: String,
+  },
+
+  methods: {
+    onArgAdd() {
+      this.$emit('add')
+      this.$nextTick(() => {
+        const args = this.$el.querySelectorAll('.action-extra-arg-name')
+        if (!args.length)
+          return
+
+        args[args.length - 1].focus()
+      })
+    },
+
+    onArgEdit(name, event) {
+      this.$emit('arg-edit', {
+        name: name,
+        value: event.target.value,
+      })
+    },
+
+    onExtraArgNameEdit(i, value) {
+      this.$emit('extra-arg-name-edit', {
+        index: i,
+        value: value,
+      })
+    },
+
+    onExtraArgValueEdit(i, value) {
+      this.$emit('extra-arg-value-edit', {
+        index: i,
+        value: value,
+      })
+    },
+
+    onSelect(arg) {
+      this.$emit('select', arg)
+    },
+  },
+}
+</script>
+
+<style lang="scss" scoped>
+@import "common";
+</style>
diff --git a/platypush/backend/http/webapp/src/components/panels/Execute/ActionDoc.vue b/platypush/backend/http/webapp/src/components/panels/Execute/ActionDoc.vue
new file mode 100644
index 000000000..502863d91
--- /dev/null
+++ b/platypush/backend/http/webapp/src/components/panels/Execute/ActionDoc.vue
@@ -0,0 +1,40 @@
+<template>
+  <section class="doc-container" v-if="doc?.length">
+    <h2>
+      <div class="title">
+        <i class="fas fa-book" /> &nbsp;
+        <a :href="action?.doc_url">Action documentation</a>
+      </div>
+      <div class="buttons" v-if="action?.name">
+        <button type="button" title="cURL command" v-if="curlSnippet?.length" @click="$emit('curl-modal')">
+          <i class="fas fa-terminal" />
+        </button>
+      </div>
+    </h2>
+
+    <div class="doc html">
+      <Loading v-if="loading" />
+      <span v-html="doc" v-else />
+    </div>
+  </section>
+</template>
+
+<script>
+import Loading from "@/components/Loading"
+
+export default {
+  name: 'ActionDoc',
+  components: { Loading },
+  emits: ['curl-modal'],
+  props: {
+    action: Object,
+    doc: String,
+    curlSnippet: String,
+    loading: Boolean,
+  }
+}
+</script>
+
+<style lang="scss" scoped>
+@import "common";
+</style>
diff --git a/platypush/backend/http/webapp/src/components/panels/Execute/Index.vue b/platypush/backend/http/webapp/src/components/panels/Execute/Index.vue
index c7adfd8de..feeed7a9c 100644
--- a/platypush/backend/http/webapp/src/components/panels/Execute/Index.vue
+++ b/platypush/backend/http/webapp/src/components/panels/Execute/Index.vue
@@ -49,25 +49,12 @@
           </header>
 
           <!-- Action documentation container -->
-          <section class="doc-container" v-if="selectedDoc">
-            <h2>
-              <div class="title">
-                <i class="fas fa-book" /> &nbsp;
-                <a :href="this.action?.doc_url">Action documentation</a>
-              </div>
-              <div class="buttons" v-if="action?.name">
-                <button type="button" title="cURL command" v-if="curlSnippet?.length"
-                        @click="$refs.curlModal.show()">
-                  <i class="fas fa-terminal" />
-                </button>
-              </div>
-            </h2>
-
-            <div class="doc html">
-              <Loading v-if="docLoading" />
-              <span v-html="selectedDoc" v-else />
-            </div>
-          </section>
+          <ActionDoc
+            :action="action"
+            :curl-snippet="curlSnippet"
+            :loading="docLoading"
+            :doc="selectedDoc"
+            @curl-modal="$refs.curlModal.show()" />
 
           <!-- Action arguments container -->
           <section class="args"
@@ -77,66 +64,20 @@
               Arguments
             </h2>
 
-            <div class="args-body">
-              <div class="args-list"
-                   v-if="Object.keys(action.args).length || action.supportsExtraArgs">
-                <!-- Supported action arguments -->
-                <div class="arg" :key="name" v-for="name in Object.keys(action.args)">
-                  <label>
-                    <input
-                        type="text"
-                        class="action-arg-value"
-                        :class="{required: action.args[name].required}"
-                        :disabled="running"
-                        :placeholder="name"
-                        v-model="action.args[name].value"
-                        @focus="selectArgdoc(name)">
-                    <span class="required-flag" v-if="action.args[name].required">*</span>
-                  </label>
-
-                  <Argdoc :name="selectedArg"
-                          :args="action.args[selectedArg]"
-                          :doc="selectedArgdoc"
-                          :loading="docLoading"
-                          is-mobile
-                          v-if="selectedArgdoc && selectedArg && name === selectedArg" />
-                </div>
-
-                <!-- Extra action arguments -->
-                <div class="extra-args" v-if="Object.keys(action.extraArgs).length">
-                  <div class="arg extra-arg" :key="i" v-for="i in Object.keys(action.extraArgs)">
-                    <label class="col-5">
-                      <input type="text" class="action-extra-arg-name" :disabled="running"
-                             placeholder="Name" v-model="action.extraArgs[i].name">
-                    </label>
-                    <label class="col-6">
-                      <input type="text" class="action-extra-arg-value" :disabled="running"
-                             placeholder="Value" v-model="action.extraArgs[i].value">
-                    </label>
-                    <label class="col-1 buttons">
-                      <button type="button" class="action-extra-arg-del" title="Remove argument"
-                              @click="removeArg(i)">
-                        <i class="fas fa-trash" />
-                      </button>
-                    </label>
-                  </div>
-                </div>
-
-                <div class="add-arg" v-if="action.supportsExtraArgs">
-                  <button type="button" title="Add an argument" @click="addArg">
-                    <i class="fas fa-plus" />
-                  </button>
-                </div>
-              </div>
-
-              <Argdoc :name="selectedArg"
-                      :args="action.args[selectedArg]"
-                      :doc="selectedArgdoc"
-                      :loading="docLoading"
-                      v-if="selectedArgdoc && selectedArg" />
-            </div>
+            <ActionArgs :action="action"
+                        :loading="loading"
+                        :running="running"
+                        :selected-arg="selectedArg"
+                        :selected-argdoc="selectedArgdoc"
+                        @add="addArg"
+                        @select="selectArgdoc"
+                        @remove="removeArg"
+                        @arg-edit="action.args[$event.name].value = $event.value"
+                        @extra-arg-name-edit="action.extraArgs[$event.index].name = $event.value"
+                        @extra-arg-value-edit="action.extraArgs[$event.index].value = $event.value" />
           </section>
 
+          <!-- Structured response container -->
           <Response :response="response" :error="error" />
         </div>
 
@@ -151,45 +92,17 @@
             </button>
           </div>
 
+          <!-- Raw response container -->
           <Response :response="response" :error="error" />
         </div>
       </form>
     </main>
-
-    <!-- Procedures section (to be removed) -->
-    <div class="section procedures-container" v-if="Object.keys(procedures).length">
-      <h1>Execute Procedure</h1>
-      <div class="procedure" :class="selectedProcedure.name === name ? 'selected' : ''"
-           v-for="name in Object.keys(procedures).sort()" :key="name" @click="updateProcedure(name, $event)">
-        <form ref="procedureForm" autocomplete="off" @submit.prevent="executeProcedure">
-          <div class="head">
-            <div class="name col-no-margin-11" v-text="name" />
-            <div class="btn-container col-no-margin-1">
-              <button type="submit" class="run-btn btn-default" :disabled="running" title="Run"
-                      @click.stop="$emit('submit')" v-if="selectedProcedure.name === name">
-                <i class="fas fa-play" />
-              </button>
-            </div>
-          </div>
-
-          <div class="args-list" v-if="selectedProcedure.name === name">
-            <div class="arg"
-                 v-for="argname in Object.keys(selectedProcedure.args)"
-                 :key="argname">
-              <label>
-                <input type="text" class="action-arg-value" @click="$event.stopPropagation()" :disabled="running"
-                       :placeholder="argname" v-model="selectedProcedure.args[argname]">
-              </label>
-            </div>
-          </div>
-        </form>
-      </div>
-    </div>
   </div>
 </template>
 
 <script>
-import Argdoc from "./Argdoc"
+import ActionArgs from "./ActionArgs"
+import ActionDoc from "./ActionDoc"
 import Autocomplete from "@/components/elements/Autocomplete"
 import Loading from "@/components/Loading"
 import Modal from "@/components/Modal";
@@ -200,7 +113,16 @@ import Utils from "@/Utils"
 
 export default {
   name: "Execute",
-  components: {Argdoc, Autocomplete, Loading, Modal, Response, Tab, Tabs},
+  components: {
+    ActionArgs,
+    ActionDoc,
+    Autocomplete,
+    Loading,
+    Modal,
+    Response,
+    Tab,
+    Tabs,
+  },
   mixins: [Utils],
 
   data() {
@@ -212,11 +134,6 @@ export default {
       selectedDoc: undefined,
       selectedArg: undefined,
       selectedArgdoc: undefined,
-      selectedProcedure: {
-        name: undefined,
-        args: {},
-      },
-
       response: undefined,
       error: undefined,
       rawRequest: undefined,
@@ -273,7 +190,7 @@ export default {
         }, {}),
 
         ...this.action.extraArgs.reduce((args, arg) => {
-          let value = args[arg.value]
+          let value = arg.value
           try {
             value = JSON.parse(value)
           } catch (e) {
@@ -301,11 +218,16 @@ export default {
         args: this.requestArgs,
       }
 
+      const reqStr = JSON.stringify(request, null, 2)
+
       return (
-        'curl -XPOST -H "Content-Type: application/json" \\\n\t' +
+        'curl -XPOST -H "Content-Type: application/json" \\\n  ' +
         `-H "Cookie: session_token=${this.getCookies()['session_token']}"`+
-        " \\\n\t -d '" +
-        this.indent(JSON.stringify(request, null, 2), 2).trim() + "' \\\n\t" +
+        " \\\n  -d '\n  {\n    " +
+        this.indent(
+          reqStr.split('\n').slice(1, reqStr.length - 2).join('\n'), 2
+        ).trim() +
+        "' \\\n  " +
         `'${this.curlURL}'`
       )
     },
@@ -316,12 +238,36 @@ export default {
       this.loading = true
 
       try {
-        this.procedures = await this.request('inspect.get_procedures')
-        this.plugins = await this.request('inspect.get_all_plugins')
+        [this.procedures, this.plugins] = await Promise.all([
+          this.request('inspect.get_procedures'),
+          this.request('inspect.get_all_plugins'),
+        ])
       } finally {
         this.loading = false
       }
 
+      // Register procedures as actions
+      this.plugins.procedure = {
+        name: 'procedure',
+        actions: Object.entries(this.procedures || {}).reduce((actions, [name, procedure]) => {
+          actions[name] = {
+            name: name,
+            args: (procedure.args || []).reduce((args, arg) => {
+              args[arg] = {
+                name: arg,
+                required: false,
+              }
+
+              return args
+            }, {}),
+            supportsExtraArgs: true,
+          }
+
+          return actions
+        }, {}),
+      }
+
+      // Parse actions from the plugins map
       for (const plugin of Object.values(this.plugins)) {
         for (const action of Object.values(plugin.actions)) {
           action.name = plugin.name + '.' + action.name
@@ -380,34 +326,6 @@ export default {
       return await this.request('utils.rst_to_html', {text: docString})
     },
 
-    updateProcedure(name, event) {
-      if (event.target.getAttribute('type') === 'submit') {
-        return
-      }
-
-      if (this.selectedProcedure.name === name) {
-        this.selectedProcedure = {
-          name: undefined,
-          args: {},
-        }
-
-        return
-      }
-
-      if (!(name in this.procedures)) {
-        console.warn('Procedure not found: ' + name)
-        return
-      }
-
-      this.selectedProcedure = {
-        name: name,
-        args: (this.procedures[name].args || []).reduce((args, arg) => {
-          args[arg] = undefined
-          return args
-        }, {})
-      }
-    },
-
     addArg() {
       this.action.extraArgs.push({
         name: undefined,
@@ -494,33 +412,6 @@ export default {
       }
     },
 
-    executeProcedure(event) {
-      if (!this.selectedProcedure.name || this.running)
-        return
-
-      event.stopPropagation()
-      this.running = true
-      const args = {
-        ...Object.entries(this.selectedProcedure.args).reduce((args, arg) => {
-          if (arg[1] != null) {
-            let value = arg[1]
-            try {
-              value = JSON.parse(value)
-            } catch (e) {
-              console.debug('Not a valid JSON value')
-              console.debug(value)
-            }
-
-            args[arg[0]] = value
-          }
-          return args
-        }, {}),
-      }
-
-      this.request('procedure.' + this.selectedProcedure.name, args)
-          .then(this.onResponse).catch(this.onError).finally(this.onDone)
-    },
-
     onClick(event) {
       // Intercept any clicks from RST rendered links and open them in a new tab
       if (event.target.tagName.toLowerCase() === 'a') {
@@ -569,48 +460,6 @@ export default {
     margin: 0 .5em;
   }
 
-  .procedures-container {
-    .procedure {
-      background: $background-color;
-      border-bottom: $default-border-2;
-      padding: 1.5em .5em;
-      cursor: pointer;
-
-      &:hover {
-        background: $hover-bg;
-      }
-
-      &.selected {
-        background: $selected-bg;
-      }
-
-      form {
-        background: none;
-        display: flex;
-        margin-bottom: 0 !important;
-        flex-direction: column;
-        box-shadow: none;
-      }
-
-      .head {
-        display: flex;
-        align-items: center;
-      }
-
-      .btn-container {
-        text-align: right;
-      }
-
-      button {
-        background: $procedure-submit-btn-bg;
-      }
-    }
-
-    .action-arg-value {
-      margin: 0.25em 0;
-    }
-  }
-
   .run-btn {
     background: $background-color;
     border-radius: .25em;
diff --git a/platypush/backend/http/webapp/src/components/panels/Execute/Response.vue b/platypush/backend/http/webapp/src/components/panels/Execute/Response.vue
index efe7fb998..5309a6d48 100644
--- a/platypush/backend/http/webapp/src/components/panels/Execute/Response.vue
+++ b/platypush/backend/http/webapp/src/components/panels/Execute/Response.vue
@@ -21,8 +21,11 @@
 </template>
 
 <script>
+import Utils from "@/Utils"
+
 export default {
   name: 'Response',
+  mixins: [Utils],
   props: {
     response: String,
     error: String,
diff --git a/platypush/backend/http/webapp/src/components/panels/Execute/common.scss b/platypush/backend/http/webapp/src/components/panels/Execute/common.scss
index e075822f0..4b1dfcd38 100644
--- a/platypush/backend/http/webapp/src/components/panels/Execute/common.scss
+++ b/platypush/backend/http/webapp/src/components/panels/Execute/common.scss
@@ -19,7 +19,7 @@ h1 {
   padding: .75em .5em;
   box-shadow: $title-shadow;
   font-size: 1.1em;
-  margin-bottom: 0 !important;
+  margin: 0;
 
   @include from($desktop) {
     border-radius: 0.5em 0.5em 0 0;
@@ -246,6 +246,7 @@ textarea.curl-snippet {
   height: 100vh;
   max-width: 40em;
   max-height: 25em;
+  font-family: monospace;
   line-break: anywhere;
   overflow: auto;
   padding: 0.5em;
diff --git a/platypush/backend/http/webapp/src/utils/Text.vue b/platypush/backend/http/webapp/src/utils/Text.vue
index 425e448d6..6b1b2465d 100644
--- a/platypush/backend/http/webapp/src/utils/Text.vue
+++ b/platypush/backend/http/webapp/src/utils/Text.vue
@@ -13,8 +13,8 @@ export default {
       return text.split('_').map((t) => this.capitalize(t)).join(' ')
     },
 
-    indent(text, tabs = 1) {
-      return text.split('\n').map((t) => `${'\t'.repeat(tabs)}${t}`).join('\n')
+    indent(text, spaces = 2) {
+      return text.split('\n').map((t) => `${' '.repeat(spaces)}${t}`).join('\n')
     },
   },
 }