From c6568f57fedfbb60fefb53dfc671e7b6d0b1a813 Mon Sep 17 00:00:00 2001
From: w940853815 <940853815@qq.com>
Date: Wed, 9 Apr 2025 18:44:22 +0800
Subject: [PATCH] perf: Optimized operation logic

---
 src/views/ops/Adhoc/QuickJob.vue              | 28 +++++++++++++++++--
 .../components/ConfirmRunAssetsDialog.vue     | 14 ++++++++--
 .../ops/Adhoc/components/QuickJobTerm.vue     | 17 +++++++++++
 3 files changed, 54 insertions(+), 5 deletions(-)

diff --git a/src/views/ops/Adhoc/QuickJob.vue b/src/views/ops/Adhoc/QuickJob.vue
index 21bff99e5..5fc376db0 100644
--- a/src/views/ops/Adhoc/QuickJob.vue
+++ b/src/views/ops/Adhoc/QuickJob.vue
@@ -34,8 +34,10 @@
             <QuickJobTerm
               ref="xterm"
               :show-tool-bar="true"
+              :select-assets="selectAssets"
               :xterm-config="xtermConfig"
               :execution-info="executionInfo"
+              @view-assets="viewConfirmRunAssets"
             />
           </div>
           <div style="display: flex;margin-top:10px;justify-content: space-between" />
@@ -47,6 +49,7 @@
 
 <script>
 import $ from '@/utils/jquery-vendor.js'
+import _isequal from 'lodash.isequal'
 import AssetTreeTable from '@/components/Apps/AssetTreeTable'
 import QuickJobTerm from '@/views/ops/Adhoc/components/QuickJobTerm.vue'
 import CodeEditor from '@/components/Form/FormFields/CodeEditor'
@@ -325,7 +328,9 @@ export default {
         error: [],
         runnable: [],
         skipped: []
-      }
+      },
+      selectAssets: [],
+      lastRequestPayload: null
     }
   },
   computed: {
@@ -455,6 +460,12 @@ export default {
       })
       return { hosts, nodes }
     },
+    shouldReRequest(payload) {
+      if (!this.lastRequestPayload) return true
+      const current = _.omit(payload, ['args'])
+      const last = _.omit(this.lastRequestPayload, ['args'])
+      return !_isequal(current, last)
+    },
     execute() {
       // const size = 'rows=' + this.xterm.rows + '&cols=' + this.xterm.cols
       const { hosts, nodes } = this.getSelectedNodesAndHosts()
@@ -471,13 +482,22 @@ export default {
         this.$message.error(this.$tc('RequiredRunas'))
         return
       }
-      this.$axios.post('/api/v1/ops/inventory/classified-hosts/', {
+      const payload = {
         assets: hosts,
         nodes: nodes,
         module: this.module,
         args: this.command,
         runas: this.runas,
         runas_policy: this.runasPolicy
+      }
+      if (!this.shouldReRequest(payload)) {
+        this.onConfirmRunAsset(this.selectAssets, nodes)
+        return
+      }
+
+      this.lastRequestPayload = { ...payload }
+      this.$axios.post('/api/v1/ops/classified-hosts/', {
+        ...payload
       }).then(data => {
         this.classifiedAssets = data
         if (this.classifiedAssets.error.length === 0) {
@@ -514,6 +534,10 @@ export default {
         this.writeExecutionOutput()
         this.setBtn()
       })
+      this.selectAssets = assets
+    },
+    viewConfirmRunAssets() {
+      this.showConfirmRunAssetsDialog = true
     },
     stop() {
       stopJob({ task_id: this.currentTaskId }).then(() => {
diff --git a/src/views/ops/Adhoc/components/ConfirmRunAssetsDialog.vue b/src/views/ops/Adhoc/components/ConfirmRunAssetsDialog.vue
index 8675e7ad4..b0ceb7db3 100644
--- a/src/views/ops/Adhoc/components/ConfirmRunAssetsDialog.vue
+++ b/src/views/ops/Adhoc/components/ConfirmRunAssetsDialog.vue
@@ -1,7 +1,7 @@
 <template>
   <Dialog
     :title="$t('ConfirmRunningAssets')"
-    :visible.sync="visible"
+    :visible.sync="iVisible"
     :show-buttons="true"
     :show-confirm="true"
     :show-cancel="true"
@@ -48,7 +48,7 @@
       </div>
     </div>
     <div>
-      <div class="selected-count">{{ $t('AssetsSelected', {count: selectedAssets.length}) }}</div>
+      <div class="selected-count">{{ selectedAssets.length }}{{ $t('AssetsSelected') }}</div>
     </div>
   </Dialog>
 </template>
@@ -80,6 +80,14 @@ export default {
     }
   },
   computed: {
+    iVisible: {
+      set(val) {
+        this.$emit('update:visible', val)
+      },
+      get() {
+        return this.visible
+      }
+    },
     runnableAssets() {
       return this.assets.runnable
     },
@@ -89,7 +97,7 @@ export default {
   },
   watch: {
     visible(val) {
-      if (val === true) {
+      if (val === true && this.selectedAssets.length === 0) {
         this.selectedAssets = this.runnableAssets.map((item) => item.id)
       }
     }
diff --git a/src/views/ops/Adhoc/components/QuickJobTerm.vue b/src/views/ops/Adhoc/components/QuickJobTerm.vue
index 18625547a..2078d6fd6 100644
--- a/src/views/ops/Adhoc/components/QuickJobTerm.vue
+++ b/src/views/ops/Adhoc/components/QuickJobTerm.vue
@@ -6,6 +6,16 @@
         v-if="executionInfo.status"
         class="header-status"
       >
+        <span class="status-item">
+          <el-link
+            @click="viewConfirmRunAssets"
+          >
+            <span>{{ selectAssets.length }}</span>
+          </el-link>
+          <span style="display: inline-block;" @click="viewConfirmRunAssets">
+            {{ $t('AssetsSelected') }}
+          </span>
+        </span>
         <span class="status-item">
           <span>{{ $tc('Status') }}: </span>
           <span
@@ -67,6 +77,10 @@ export default {
       type: Object,
       // eslint-disable-next-line vue/require-valid-default-prop
       default: {}
+    },
+    selectAssets: {
+      type: Array,
+      default: () => []
     }
   },
   data() {
@@ -153,6 +167,9 @@ export default {
         this.showScrollButton = true
         this.xterm.scrollToBottom()
       })
+    },
+    viewConfirmRunAssets() {
+      this.$emit('view-assets')
     }
   }
 }