From b54ae272ebc2d31b32b049d44e2299a6be7f153c Mon Sep 17 00:00:00 2001
From: xiaojunnuo <xiaojunnuo@qq.com>
Date: Mon, 25 Nov 2024 11:35:16 +0800
Subject: [PATCH] =?UTF-8?q?perf:=20=E9=80=9A=E7=9F=A5=E6=B8=A0=E9=81=93?=
 =?UTF-8?q?=E6=94=AF=E6=8C=81=E6=B5=8B=E8=AF=95=E6=8C=89=E9=92=AE?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 packages/core/pipeline/src/access/api.ts      |  4 ++
 .../core/pipeline/src/access/decorator.ts     |  2 +-
 .../core/pipeline/src/notification/api.ts     | 15 ++++++
 .../pipeline/src/notification/decorator.ts    |  8 ++--
 .../components/plugins/common/api-test.vue    | 19 +++++---
 .../plugins/common/remote-select.vue          | 12 +++--
 .../src/components/plugins/lib/index.ts       | 10 ++--
 .../plugins/synology/device-id-getter.vue     |  1 -
 .../src/views/certd/access/common.tsx         |  3 ++
 .../src/views/certd/notification/common.tsx   | 25 +++++++++-
 .../pipeline/component/step-form/index.vue    |  3 ++
 .../controller/pipeline/handle-controller.ts  | 46 ++++++++++++++++++-
 .../plugin/deploy-to-cdn/index.ts             |  2 +-
 13 files changed, 126 insertions(+), 24 deletions(-)

diff --git a/packages/core/pipeline/src/access/api.ts b/packages/core/pipeline/src/access/api.ts
index 72937697..a350e8ef 100644
--- a/packages/core/pipeline/src/access/api.ts
+++ b/packages/core/pipeline/src/access/api.ts
@@ -41,6 +41,10 @@ export type AccessContext = {
 export abstract class BaseAccess implements IAccess {
   ctx!: AccessContext;
 
+  setCtx(ctx: AccessContext) {
+    this.ctx = ctx;
+  }
+
   async onRequest(req: AccessRequestHandleReq) {
     if (!req.action) {
       throw new Error("action is required");
diff --git a/packages/core/pipeline/src/access/decorator.ts b/packages/core/pipeline/src/access/decorator.ts
index 2abcd904..a0c19eee 100644
--- a/packages/core/pipeline/src/access/decorator.ts
+++ b/packages/core/pipeline/src/access/decorator.ts
@@ -56,6 +56,6 @@ export function newAccess(type: string, input: any, ctx?: AccessContext) {
       utils,
     };
   }
-  access.ctx = ctx;
+  access.setCtx(ctx);
   return access;
 }
diff --git a/packages/core/pipeline/src/notification/api.ts b/packages/core/pipeline/src/notification/api.ts
index 8488a690..338b8033 100644
--- a/packages/core/pipeline/src/notification/api.ts
+++ b/packages/core/pipeline/src/notification/api.ts
@@ -93,4 +93,19 @@ export abstract class BaseNotification implements INotification {
     }
     throw new Error(`action ${req.action} not found`);
   }
+
+  async onTestRequest() {
+    await this.send({
+      userId: 0,
+      title: "测试通知",
+      content: "测试通知",
+      pipeline: {
+        id: 1,
+        title: "测试流水线",
+      } as any,
+      pipelineId: 1,
+      historyId: 1,
+      url: "http://www.baidu.com",
+    });
+  }
 }
diff --git a/packages/core/pipeline/src/notification/decorator.ts b/packages/core/pipeline/src/notification/decorator.ts
index ac80b14c..66b17b4d 100644
--- a/packages/core/pipeline/src/notification/decorator.ts
+++ b/packages/core/pipeline/src/notification/decorator.ts
@@ -44,13 +44,13 @@ export function newNotification(type: string, input: any, ctx: NotificationConte
     throw new Error(`notification ${type} not found`);
   }
   // @ts-ignore
-  const access = new register.target();
+  const plugin = new register.target();
   for (const key in input) {
-    access[key] = input[key];
+    plugin[key] = input[key];
   }
   if (!ctx) {
     throw new Error("ctx is required");
   }
-  access.ctx = ctx;
-  return access;
+  plugin.setCtx(ctx);
+  return plugin;
 }
diff --git a/packages/ui/certd-client/src/components/plugins/common/api-test.vue b/packages/ui/certd-client/src/components/plugins/common/api-test.vue
index 7749e49f..23e3f60e 100644
--- a/packages/ui/certd-client/src/components/plugins/common/api-test.vue
+++ b/packages/ui/certd-client/src/components/plugins/common/api-test.vue
@@ -11,12 +11,15 @@
 </template>
 <script setup lang="ts">
 import { ComponentPropsType, doRequest } from "/@/components/plugins/lib";
-import { ref } from "vue";
+import { ref, inject } from "vue";
 
 defineOptions({
   name: "ApiTest"
 });
 
+const getScope: any = inject("get:scope");
+const getPluginType: any = inject("get:plugin:type");
+
 const props = defineProps<{} & ComponentPropsType>();
 
 const emit = defineEmits<{
@@ -31,16 +34,20 @@ const doTest = async () => {
     return;
   }
 
+  const { form } = getScope();
+  const pluginType = getPluginType();
+
   message.value = "";
   hasError.value = false;
   loading.value = true;
+  debugger;
   try {
     const res = await doRequest(
       {
-        type: props.type,
-        typeName: props.typeName,
+        type: pluginType,
+        typeName: form.type,
         action: props.action,
-        input: props.form
+        input: form
       },
       {
         onError(err: any) {
@@ -50,9 +57,7 @@ const doTest = async () => {
         showErrorNotify: false
       }
     );
-    if (res && res.length > 0) {
-      message.value = "测试请求成功";
-    }
+    message.value = "测试请求成功";
   } finally {
     loading.value = false;
   }
diff --git a/packages/ui/certd-client/src/components/plugins/common/remote-select.vue b/packages/ui/certd-client/src/components/plugins/common/remote-select.vue
index f4a9a0a8..63ed81f7 100644
--- a/packages/ui/certd-client/src/components/plugins/common/remote-select.vue
+++ b/packages/ui/certd-client/src/components/plugins/common/remote-select.vue
@@ -42,6 +42,9 @@ const emit = defineEmits<{
 const attrs = useAttrs();
 
 const getCurrentPluginDefine: any = inject("getCurrentPluginDefine");
+const getScope: any = inject("get:scope");
+const getPluginType: any = inject("get:plugin:type");
+
 const optionsRef = ref([]);
 const message = ref("");
 const hasError = ref(false);
@@ -75,13 +78,16 @@ const getOptions = async () => {
   hasError.value = false;
   loading.value = true;
   optionsRef.value = [];
+  const { form } = getScope();
+  const pluginType = getPluginType();
+
   try {
     const res = await doRequest(
       {
-        type: props.type,
-        typeName: props.typeName,
+        type: pluginType,
+        typeName: form.type,
         action: props.action,
-        input: props.form
+        input: form
       },
       {
         onError(err: any) {
diff --git a/packages/ui/certd-client/src/components/plugins/lib/index.ts b/packages/ui/certd-client/src/components/plugins/lib/index.ts
index 4ecebf32..eb6ac83d 100644
--- a/packages/ui/certd-client/src/components/plugins/lib/index.ts
+++ b/packages/ui/certd-client/src/components/plugins/lib/index.ts
@@ -1,9 +1,9 @@
 import { request } from "/@/api/service";
 export type ComponentPropsType = {
-  type: string;
-  typeName: string;
-  action?: string;
-  form: any;
+  type?: string;
+  typeName?: string;
+  action: string;
+  form?: any;
   value?: any;
 };
 export type RequestHandleReq<T = any> = {
@@ -15,7 +15,7 @@ export type RequestHandleReq<T = any> = {
 };
 
 export async function doRequest(req: RequestHandleReq, opts: any = {}) {
-  const url = req.type === "access" ? "/pi/handle/access" : "/pi/handle/plugin";
+  const url = `/pi/handle/${req.type}`;
   const { typeName, action, data, input } = req;
   const res = await request({
     url,
diff --git a/packages/ui/certd-client/src/components/plugins/synology/device-id-getter.vue b/packages/ui/certd-client/src/components/plugins/synology/device-id-getter.vue
index b19b2696..37b523b2 100644
--- a/packages/ui/certd-client/src/components/plugins/synology/device-id-getter.vue
+++ b/packages/ui/certd-client/src/components/plugins/synology/device-id-getter.vue
@@ -19,7 +19,6 @@ defineOptions({
 });
 
 const props = defineProps<ComponentPropsType>();
-
 const emit = defineEmits<{
   "update:value": any;
 }>();
diff --git a/packages/ui/certd-client/src/views/certd/access/common.tsx b/packages/ui/certd-client/src/views/certd/access/common.tsx
index 8b0cf045..5c6cbf82 100644
--- a/packages/ui/certd-client/src/views/certd/access/common.tsx
+++ b/packages/ui/certd-client/src/views/certd/access/common.tsx
@@ -6,6 +6,9 @@ import SecretPlainGetter from "/@/views/certd/access/access-selector/access/secr
 
 export function getCommonColumnDefine(crudExpose: any, typeRef: any, api: any) {
   provide("accessApi", api);
+  provide("get:plugin:type", () => {
+    return "access";
+  });
   const AccessTypeDictRef = dict({
     url: "/pi/access/accessTypeDict"
   });
diff --git a/packages/ui/certd-client/src/views/certd/notification/common.tsx b/packages/ui/certd-client/src/views/certd/notification/common.tsx
index b33710c3..c96f9f14 100644
--- a/packages/ui/certd-client/src/views/certd/notification/common.tsx
+++ b/packages/ui/certd-client/src/views/certd/notification/common.tsx
@@ -1,10 +1,13 @@
-import { ColumnCompositionProps, dict } from "@fast-crud/fast-crud";
+import { ColumnCompositionProps, compute, dict } from "@fast-crud/fast-crud";
 import { computed, provide, ref, toRef } from "vue";
 import { useReference } from "/@/use/use-refrence";
 import { forEach, get, merge, set } from "lodash-es";
 
 export function getCommonColumnDefine(crudExpose: any, typeRef: any, api: any) {
   provide("notificationApi", api);
+  provide("get:plugin:type", () => {
+    return "notification";
+  });
   const notificationTypeDictRef = dict({
     url: "/pi/notification/getTypeDict"
   });
@@ -125,6 +128,26 @@ export function getCommonColumnDefine(crudExpose: any, typeRef: any, api: any) {
         })
       }
     } as ColumnCompositionProps,
+    test: {
+      title: "测试",
+      form: {
+        component: {
+          name: "api-test",
+          type: "notification",
+          typeName: compute(({ form }) => {
+            return form.type;
+          }),
+          action: "TestRequest",
+          form: compute(({ form }) => {
+            return form;
+          })
+        },
+        order: 999
+      },
+      column: {
+        show: false
+      }
+    },
     setting: {
       column: { show: false },
       form: {
diff --git a/packages/ui/certd-client/src/views/certd/pipeline/pipeline/component/step-form/index.vue b/packages/ui/certd-client/src/views/certd/pipeline/pipeline/component/step-form/index.vue
index 0cf67338..284d7a71 100644
--- a/packages/ui/certd-client/src/views/certd/pipeline/pipeline/component/step-form/index.vue
+++ b/packages/ui/certd-client/src/views/certd/pipeline/pipeline/component/step-form/index.vue
@@ -222,6 +222,9 @@ export default {
       provide("getCurrentPluginDefine", () => {
         return currentPluginDefine;
       });
+      provide("get:plugin:type", () => {
+        return "plugin";
+      });
 
       function getContext() {
         return {
diff --git a/packages/ui/certd-server/src/controller/pipeline/handle-controller.ts b/packages/ui/certd-server/src/controller/pipeline/handle-controller.ts
index 2d778857..56a85784 100644
--- a/packages/ui/certd-server/src/controller/pipeline/handle-controller.ts
+++ b/packages/ui/certd-server/src/controller/pipeline/handle-controller.ts
@@ -1,11 +1,21 @@
 import { ALL, Body, Controller, Inject, Post, Provide } from '@midwayjs/core';
 import { Constants } from '@certd/lib-server';
-import { AccessRequestHandleReq, ITaskPlugin, newAccess, pluginRegistry, PluginRequestHandleReq, TaskInstanceContext } from '@certd/pipeline';
+import {
+  AccessRequestHandleReq,
+  ITaskPlugin,
+  newAccess,
+  newNotification,
+  NotificationRequestHandleReq,
+  pluginRegistry,
+  PluginRequestHandleReq,
+  TaskInstanceContext,
+} from '@certd/pipeline';
 import { BaseController } from '@certd/lib-server';
 import { AccessService } from '../../modules/pipeline/service/access-service.js';
 import { EmailService } from '../../modules/basic/service/email-service.js';
 import { AccessGetter } from '../../modules/pipeline/service/access-getter.js';
 import { http, HttpRequestConfig, logger, mergeUtils, utils } from '@certd/basic';
+import { NotificationService } from '../../modules/pipeline/service/notification-service.js';
 
 @Provide()
 @Controller('/api/pi/handle')
@@ -16,12 +26,18 @@ export class HandleController extends BaseController {
   @Inject()
   emailService: EmailService;
 
+  @Inject()
+  notificationService: NotificationService;
+
   @Post('/access', { summary: Constants.per.authOnly })
   async accessRequest(@Body(ALL) body: AccessRequestHandleReq) {
     let inputAccess = body.input.access;
     if (body.input.id > 0) {
       const oldEntity = await this.accessService.info(body.input.id);
       if (oldEntity) {
+        if (oldEntity.userId !== this.getUserId()) {
+          throw new Error('access not found');
+        }
         const param: any = {
           type: body.typeName,
           setting: JSON.stringify(body.input.access),
@@ -38,6 +54,34 @@ export class HandleController extends BaseController {
     return this.ok(res);
   }
 
+  @Post('/notification', { summary: Constants.per.authOnly })
+  async notificationRequest(@Body(ALL) body: NotificationRequestHandleReq) {
+    const input = body.input.body;
+    // if (body.input.id > 0) {
+    //   const oldEntity = await this.notificationService.info(body.input.id);
+    //   if (oldEntity) {
+    //     if (oldEntity.userId !== this.getUserId()) {
+    //       throw new Error('notification not found');
+    //     }
+    //     const param: any = {
+    //       type: body.typeName,
+    //       setting: JSON.stringify(body.input.access),
+    //     };
+    //   }
+    // }
+
+    const notification = newNotification(body.typeName, input, {
+      http,
+      logger,
+      utils,
+      emailService: this.emailService,
+    });
+
+    const res = await notification.onRequest(body);
+
+    return this.ok(res);
+  }
+
   @Post('/plugin', { summary: Constants.per.authOnly })
   async pluginRequest(@Body(ALL) body: PluginRequestHandleReq) {
     const userId = this.getUserId();
diff --git a/packages/ui/certd-server/src/plugins/plugin-tencent/plugin/deploy-to-cdn/index.ts b/packages/ui/certd-server/src/plugins/plugin-tencent/plugin/deploy-to-cdn/index.ts
index 3d6831f0..e4f0accc 100644
--- a/packages/ui/certd-server/src/plugins/plugin-tencent/plugin/deploy-to-cdn/index.ts
+++ b/packages/ui/certd-server/src/plugins/plugin-tencent/plugin/deploy-to-cdn/index.ts
@@ -7,7 +7,7 @@ import { CertInfo } from '@certd/plugin-cert';
   title: '部署到腾讯云CDN',
   icon: 'svg:icon-tencentcloud',
   group: pluginGroups.tencent.key,
-  desc: '可能不支持国际版',
+  desc: '已废弃，请使用v2版',
   default: {
     strategy: {
       runStrategy: RunStrategy.SkipWhenSucceed,