From cac60a25bb6bf65c6b874997b08ec213655fd7bd Mon Sep 17 00:00:00 2001
From: Yeuoly <admin@srmxy.cn>
Date: Thu, 30 Oct 2025 15:27:02 +0800
Subject: [PATCH 01/17] cleanup: migrations

---
 ...-4558cfabe44e_add_workflow_trigger_logs.py |  67 -----
 ...5871f634954d_add_workflow_webhook_table.py |  47 ----
 ...203-9ee7d347f4c1_add_app_triggers_table.py |  47 ----
 ...c19938f630b6_add_workflow_schedule_plan.py |  47 ----
 ..._10_27_1205-132392a2635f_plugin_trigger.py | 102 --------
 ..._1752-5ed4b21dbb8d_trigger_log_metadata.py |  32 ---
 ..._30_1518-669ffd70119c_introduce_trigger.py | 235 ++++++++++++++++++
 api/models/trigger.py                         |   2 +-
 8 files changed, 236 insertions(+), 343 deletions(-)
 delete mode 100644 api/migrations/versions/2025_10_27_1201-4558cfabe44e_add_workflow_trigger_logs.py
 delete mode 100644 api/migrations/versions/2025_10_27_1202-5871f634954d_add_workflow_webhook_table.py
 delete mode 100644 api/migrations/versions/2025_10_27_1203-9ee7d347f4c1_add_app_triggers_table.py
 delete mode 100644 api/migrations/versions/2025_10_27_1204-c19938f630b6_add_workflow_schedule_plan.py
 delete mode 100644 api/migrations/versions/2025_10_27_1205-132392a2635f_plugin_trigger.py
 delete mode 100644 api/migrations/versions/2025_10_27_1752-5ed4b21dbb8d_trigger_log_metadata.py
 create mode 100644 api/migrations/versions/2025_10_30_1518-669ffd70119c_introduce_trigger.py

diff --git a/api/migrations/versions/2025_10_27_1201-4558cfabe44e_add_workflow_trigger_logs.py b/api/migrations/versions/2025_10_27_1201-4558cfabe44e_add_workflow_trigger_logs.py
deleted file mode 100644
index 1fe46972c1..0000000000
--- a/api/migrations/versions/2025_10_27_1201-4558cfabe44e_add_workflow_trigger_logs.py
+++ /dev/null
@@ -1,67 +0,0 @@
-"""Add workflow trigger logs table
-
-Revision ID: 4558cfabe44e
-Revises: ae662b25d9bc
-Create Date: 2025-10-27 12:01:00.000000
-
-"""
-from alembic import op
-import models as models
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = '4558cfabe44e'
-down_revision = '03f8dcbc611e'
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    op.create_table('workflow_trigger_logs',
-    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
-    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
-    sa.Column('app_id', models.types.StringUUID(), nullable=False),
-    sa.Column('workflow_id', models.types.StringUUID(), nullable=False),
-    sa.Column('workflow_run_id', models.types.StringUUID(), nullable=True),
-    sa.Column('root_node_id', sa.String(length=255), nullable=True),
-    sa.Column('trigger_type', sa.String(length=50), nullable=False),
-    sa.Column('trigger_data', sa.Text(), nullable=False),
-    sa.Column('inputs', sa.Text(), nullable=False),
-    sa.Column('outputs', sa.Text(), nullable=True),
-    sa.Column('status', sa.String(length=50), nullable=False),
-    sa.Column('error', sa.Text(), nullable=True),
-    sa.Column('queue_name', sa.String(length=100), nullable=False),
-    sa.Column('celery_task_id', sa.String(length=255), nullable=True),
-    sa.Column('retry_count', sa.Integer(), nullable=False),
-    sa.Column('elapsed_time', sa.Float(), nullable=True),
-    sa.Column('total_tokens', sa.Integer(), nullable=True),
-    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.Column('created_by_role', sa.String(length=255), nullable=False),
-    sa.Column('created_by', sa.String(length=255), nullable=False),
-    sa.Column('triggered_at', sa.DateTime(), nullable=True),
-    sa.Column('finished_at', sa.DateTime(), nullable=True),
-    sa.PrimaryKeyConstraint('id', name='workflow_trigger_log_pkey')
-    )
-    with op.batch_alter_table('workflow_trigger_logs', schema=None) as batch_op:
-        batch_op.create_index('workflow_trigger_log_created_at_idx', ['created_at'], unique=False)
-        batch_op.create_index('workflow_trigger_log_status_idx', ['status'], unique=False)
-        batch_op.create_index('workflow_trigger_log_tenant_app_idx', ['tenant_id', 'app_id'], unique=False)
-        batch_op.create_index('workflow_trigger_log_workflow_id_idx', ['workflow_id'], unique=False)
-        batch_op.create_index('workflow_trigger_log_workflow_run_idx', ['workflow_run_id'], unique=False)
-
-    # ### end Alembic commands ###
-
-
-def downgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    with op.batch_alter_table('workflow_trigger_logs', schema=None) as batch_op:
-        batch_op.drop_index('workflow_trigger_log_workflow_run_idx')
-        batch_op.drop_index('workflow_trigger_log_workflow_id_idx')
-        batch_op.drop_index('workflow_trigger_log_tenant_app_idx')
-        batch_op.drop_index('workflow_trigger_log_status_idx')
-        batch_op.drop_index('workflow_trigger_log_created_at_idx')
-
-    op.drop_table('workflow_trigger_logs')
-    # ### end Alembic commands ###
diff --git a/api/migrations/versions/2025_10_27_1202-5871f634954d_add_workflow_webhook_table.py b/api/migrations/versions/2025_10_27_1202-5871f634954d_add_workflow_webhook_table.py
deleted file mode 100644
index 43466a0697..0000000000
--- a/api/migrations/versions/2025_10_27_1202-5871f634954d_add_workflow_webhook_table.py
+++ /dev/null
@@ -1,47 +0,0 @@
-"""Add workflow webhook table
-
-Revision ID: 5871f634954d
-Revises: 4558cfabe44e
-Create Date: 2025-10-27 12:02:00.000000
-
-"""
-from alembic import op
-import models as models
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = '5871f634954d'
-down_revision = '4558cfabe44e'
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    op.create_table('workflow_webhook_triggers',
-    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
-    sa.Column('app_id', models.types.StringUUID(), nullable=False),
-    sa.Column('node_id', sa.String(length=64), nullable=False),
-    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
-    sa.Column('webhook_id', sa.String(length=24), nullable=False),
-    sa.Column('created_by', models.types.StringUUID(), nullable=False),
-    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.Column('updated_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.PrimaryKeyConstraint('id', name='workflow_webhook_trigger_pkey'),
-    sa.UniqueConstraint('app_id', 'node_id', name='uniq_node'),
-    sa.UniqueConstraint('webhook_id', name='uniq_webhook_id')
-    )
-    with op.batch_alter_table('workflow_webhook_triggers', schema=None) as batch_op:
-        batch_op.create_index('workflow_webhook_trigger_tenant_idx', ['tenant_id'], unique=False)
-
-    # ### end Alembic commands ###
-
-
-def downgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    with op.batch_alter_table('workflow_webhook_triggers', schema=None) as batch_op:
-        batch_op.drop_index('workflow_webhook_trigger_tenant_idx')
-
-    op.drop_table('workflow_webhook_triggers')
-    # ### end Alembic commands ###
diff --git a/api/migrations/versions/2025_10_27_1203-9ee7d347f4c1_add_app_triggers_table.py b/api/migrations/versions/2025_10_27_1203-9ee7d347f4c1_add_app_triggers_table.py
deleted file mode 100644
index fe4cd24ad6..0000000000
--- a/api/migrations/versions/2025_10_27_1203-9ee7d347f4c1_add_app_triggers_table.py
+++ /dev/null
@@ -1,47 +0,0 @@
-"""Add app triggers table
-
-Revision ID: 9ee7d347f4c1
-Revises: 5871f634954d
-Create Date: 2025-10-27 12:03:00.000000
-
-"""
-from alembic import op
-import models as models
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = '9ee7d347f4c1'
-down_revision = '5871f634954d'
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    op.create_table('app_triggers',
-    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
-    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
-    sa.Column('app_id', models.types.StringUUID(), nullable=False),
-    sa.Column('node_id', sa.String(length=64), nullable=False),
-    sa.Column('trigger_type', sa.String(length=50), nullable=False),
-    sa.Column('title', sa.String(length=255), nullable=False),
-    sa.Column('provider_name', sa.String(length=255), server_default='', nullable=True),
-    sa.Column('status', sa.String(length=50), nullable=False),
-    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.Column('updated_at', sa.DateTime(), nullable=False),
-    sa.PrimaryKeyConstraint('id', name='app_trigger_pkey')
-    )
-    with op.batch_alter_table('app_triggers', schema=None) as batch_op:
-        batch_op.create_index('app_trigger_tenant_app_idx', ['tenant_id', 'app_id'], unique=False)
-
-    # ### end Alembic commands ###
-
-
-def downgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    with op.batch_alter_table('app_triggers', schema=None) as batch_op:
-        batch_op.drop_index('app_trigger_tenant_app_idx')
-
-    op.drop_table('app_triggers')
-    # ### end Alembic commands ###
diff --git a/api/migrations/versions/2025_10_27_1204-c19938f630b6_add_workflow_schedule_plan.py b/api/migrations/versions/2025_10_27_1204-c19938f630b6_add_workflow_schedule_plan.py
deleted file mode 100644
index 85e7e0c735..0000000000
--- a/api/migrations/versions/2025_10_27_1204-c19938f630b6_add_workflow_schedule_plan.py
+++ /dev/null
@@ -1,47 +0,0 @@
-"""Add workflow schedule plan table
-
-Revision ID: c19938f630b6
-Revises: 9ee7d347f4c1
-Create Date: 2025-10-27 12:04:00.000000
-
-"""
-from alembic import op
-import models as models
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = 'c19938f630b6'
-down_revision = '9ee7d347f4c1'
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    op.create_table('workflow_schedule_plans',
-    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
-    sa.Column('app_id', models.types.StringUUID(), nullable=False),
-    sa.Column('node_id', sa.String(length=64), nullable=False),
-    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
-    sa.Column('cron_expression', sa.String(length=255), nullable=False),
-    sa.Column('timezone', sa.String(length=64), nullable=False),
-    sa.Column('next_run_at', sa.DateTime(), nullable=True),
-    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.Column('updated_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.PrimaryKeyConstraint('id', name='workflow_schedule_plan_pkey'),
-    sa.UniqueConstraint('app_id', 'node_id', name='uniq_app_node')
-    )
-    with op.batch_alter_table('workflow_schedule_plans', schema=None) as batch_op:
-        batch_op.create_index('workflow_schedule_plan_next_idx', ['next_run_at'], unique=False)
-
-    # ### end Alembic commands ###
-
-
-def downgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    with op.batch_alter_table('workflow_schedule_plans', schema=None) as batch_op:
-        batch_op.drop_index('workflow_schedule_plan_next_idx')
-
-    op.drop_table('workflow_schedule_plans')
-    # ### end Alembic commands ###
diff --git a/api/migrations/versions/2025_10_27_1205-132392a2635f_plugin_trigger.py b/api/migrations/versions/2025_10_27_1205-132392a2635f_plugin_trigger.py
deleted file mode 100644
index 426be1b071..0000000000
--- a/api/migrations/versions/2025_10_27_1205-132392a2635f_plugin_trigger.py
+++ /dev/null
@@ -1,102 +0,0 @@
-"""plugin_trigger
-
-Revision ID: 132392a2635f
-Revises: c19938f630b6
-Create Date: 2025-10-27 12:05:00.000000
-
-"""
-from alembic import op
-import models as models
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = '132392a2635f'
-down_revision = 'c19938f630b6'
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    op.create_table('trigger_oauth_system_clients',
-    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
-    sa.Column('plugin_id', sa.String(length=512), nullable=False),
-    sa.Column('provider', sa.String(length=255), nullable=False),
-    sa.Column('encrypted_oauth_params', sa.Text(), nullable=False),
-    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.Column('updated_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.PrimaryKeyConstraint('id', name='trigger_oauth_system_client_pkey'),
-    sa.UniqueConstraint('plugin_id', 'provider', name='trigger_oauth_system_client_plugin_id_provider_idx')
-    )
-    op.create_table('trigger_oauth_tenant_clients',
-    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
-    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
-    sa.Column('plugin_id', sa.String(length=512), nullable=False),
-    sa.Column('provider', sa.String(length=255), nullable=False),
-    sa.Column('enabled', sa.Boolean(), server_default=sa.text('true'), nullable=False),
-    sa.Column('encrypted_oauth_params', sa.Text(), nullable=False),
-    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.Column('updated_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.PrimaryKeyConstraint('id', name='trigger_oauth_tenant_client_pkey'),
-    sa.UniqueConstraint('tenant_id', 'plugin_id', 'provider', name='unique_trigger_oauth_tenant_client')
-    )
-    op.create_table('trigger_subscriptions',
-    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
-    sa.Column('name', sa.String(length=255), nullable=False, comment='Subscription instance name'),
-    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
-    sa.Column('user_id', models.types.StringUUID(), nullable=False),
-    sa.Column('provider_id', sa.String(length=255), nullable=False, comment='Provider identifier (e.g., plugin_id/provider_name)'),
-    sa.Column('endpoint_id', sa.String(length=255), nullable=False, comment='Subscription endpoint'),
-    sa.Column('parameters', sa.JSON(), nullable=False, comment='Subscription parameters JSON'),
-    sa.Column('properties', sa.JSON(), nullable=False, comment='Subscription properties JSON'),
-    sa.Column('credentials', sa.JSON(), nullable=False, comment='Subscription credentials JSON'),
-    sa.Column('credential_type', sa.String(length=50), nullable=False, comment='oauth or api_key'),
-    sa.Column('credential_expires_at', sa.Integer(), nullable=False, comment='OAuth token expiration timestamp, -1 for never'),
-    sa.Column('expires_at', sa.Integer(), nullable=False, comment='Subscription instance expiration timestamp, -1 for never'),
-    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.Column('updated_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.PrimaryKeyConstraint('id', name='trigger_provider_pkey'),
-    sa.UniqueConstraint('tenant_id', 'provider_id', 'name', name='unique_trigger_provider')
-    )
-    with op.batch_alter_table('trigger_subscriptions', schema=None) as batch_op:
-        batch_op.create_index('idx_trigger_providers_endpoint', ['endpoint_id'], unique=True)
-        batch_op.create_index('idx_trigger_providers_tenant_endpoint', ['tenant_id', 'endpoint_id'], unique=False)
-        batch_op.create_index('idx_trigger_providers_tenant_provider', ['tenant_id', 'provider_id'], unique=False)
-
-    # Create workflow_plugin_triggers table with final schema (merged from all 4 migrations)
-    op.create_table('workflow_plugin_triggers',
-    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
-    sa.Column('app_id', models.types.StringUUID(), nullable=False),
-    sa.Column('node_id', sa.String(length=64), nullable=False),
-    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
-    sa.Column('provider_id', sa.String(length=512), nullable=False),
-    sa.Column('subscription_id', sa.String(length=255), nullable=False),
-    sa.Column('event_name', sa.String(length=255), nullable=False),
-    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.Column('updated_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
-    sa.PrimaryKeyConstraint('id', name='workflow_plugin_trigger_pkey'),
-    sa.UniqueConstraint('app_id', 'node_id', name='uniq_app_node_subscription')
-    )
-    with op.batch_alter_table('workflow_plugin_triggers', schema=None) as batch_op:
-        batch_op.create_index('workflow_plugin_trigger_tenant_subscription_idx', ['tenant_id', 'subscription_id', 'event_name'], unique=False)
-
-    # ### end Alembic commands ###
-
-
-def downgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    with op.batch_alter_table('workflow_plugin_triggers', schema=None) as batch_op:
-        batch_op.drop_index('workflow_plugin_trigger_tenant_subscription_idx')
-
-    op.drop_table('workflow_plugin_triggers')
-    with op.batch_alter_table('trigger_subscriptions', schema=None) as batch_op:
-        batch_op.drop_index('idx_trigger_providers_tenant_provider')
-        batch_op.drop_index('idx_trigger_providers_tenant_endpoint')
-        batch_op.drop_index('idx_trigger_providers_endpoint')
-
-    op.drop_table('trigger_subscriptions')
-    op.drop_table('trigger_oauth_tenant_clients')
-    op.drop_table('trigger_oauth_system_clients')
-
-    # ### end Alembic commands ###
diff --git a/api/migrations/versions/2025_10_27_1752-5ed4b21dbb8d_trigger_log_metadata.py b/api/migrations/versions/2025_10_27_1752-5ed4b21dbb8d_trigger_log_metadata.py
deleted file mode 100644
index 089246d2fa..0000000000
--- a/api/migrations/versions/2025_10_27_1752-5ed4b21dbb8d_trigger_log_metadata.py
+++ /dev/null
@@ -1,32 +0,0 @@
-"""trigger_log_metadata
-
-Revision ID: 5ed4b21dbb8d
-Revises: 132392a2635f
-Create Date: 2025-10-27 17:52:35.658975
-
-"""
-from alembic import op
-import models as models
-import sqlalchemy as sa
-
-
-# revision identifiers, used by Alembic.
-revision = '5ed4b21dbb8d'
-down_revision = '132392a2635f'
-branch_labels = None
-depends_on = None
-
-
-def upgrade():
-    with op.batch_alter_table('workflow_trigger_logs', schema=None) as batch_op:
-        batch_op.add_column(sa.Column('trigger_metadata', sa.Text(), nullable=True))
-
-    # ### end Alembic commands ###
-
-
-def downgrade():
-    # ### commands auto generated by Alembic - please adjust! ###
-    with op.batch_alter_table('workflow_trigger_logs', schema=None) as batch_op:
-        batch_op.drop_column('trigger_metadata')
-
-    # ### end Alembic commands ###
diff --git a/api/migrations/versions/2025_10_30_1518-669ffd70119c_introduce_trigger.py b/api/migrations/versions/2025_10_30_1518-669ffd70119c_introduce_trigger.py
new file mode 100644
index 0000000000..c03d64b234
--- /dev/null
+++ b/api/migrations/versions/2025_10_30_1518-669ffd70119c_introduce_trigger.py
@@ -0,0 +1,235 @@
+"""introduce_trigger
+
+Revision ID: 669ffd70119c
+Revises: 03f8dcbc611e
+Create Date: 2025-10-30 15:18:49.549156
+
+"""
+from alembic import op
+import models as models
+import sqlalchemy as sa
+
+from models.enums import AppTriggerStatus, AppTriggerType
+
+
+# revision identifiers, used by Alembic.
+revision = '669ffd70119c'
+down_revision = '03f8dcbc611e'
+branch_labels = None
+depends_on = None
+
+
+def upgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table('app_triggers',
+    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
+    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
+    sa.Column('app_id', models.types.StringUUID(), nullable=False),
+    sa.Column('node_id', sa.String(length=64), nullable=False),
+    sa.Column('trigger_type', models.types.EnumText(AppTriggerType, length=50), nullable=False),
+    sa.Column('title', sa.String(length=255), nullable=False),
+    sa.Column('provider_name', sa.String(length=255), server_default='', nullable=True),
+    sa.Column('status', models.types.EnumText(AppTriggerStatus, length=50), nullable=False),
+    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.Column('updated_at', sa.DateTime(), nullable=False),
+    sa.PrimaryKeyConstraint('id', name='app_trigger_pkey')
+    )
+    with op.batch_alter_table('app_triggers', schema=None) as batch_op:
+        batch_op.create_index('app_trigger_tenant_app_idx', ['tenant_id', 'app_id'], unique=False)
+
+    op.create_table('trigger_oauth_system_clients',
+    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuid_generate_v4()'), nullable=False),
+    sa.Column('plugin_id', sa.String(length=512), nullable=False),
+    sa.Column('provider', sa.String(length=255), nullable=False),
+    sa.Column('encrypted_oauth_params', sa.Text(), nullable=False),
+    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.Column('updated_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.PrimaryKeyConstraint('id', name='trigger_oauth_system_client_pkey'),
+    sa.UniqueConstraint('plugin_id', 'provider', name='trigger_oauth_system_client_plugin_id_provider_idx')
+    )
+    op.create_table('trigger_oauth_tenant_clients',
+    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuid_generate_v4()'), nullable=False),
+    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
+    sa.Column('plugin_id', sa.String(length=512), nullable=False),
+    sa.Column('provider', sa.String(length=255), nullable=False),
+    sa.Column('enabled', sa.Boolean(), server_default=sa.text('true'), nullable=False),
+    sa.Column('encrypted_oauth_params', sa.Text(), nullable=False),
+    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.Column('updated_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.PrimaryKeyConstraint('id', name='trigger_oauth_tenant_client_pkey'),
+    sa.UniqueConstraint('tenant_id', 'plugin_id', 'provider', name='unique_trigger_oauth_tenant_client')
+    )
+    op.create_table('trigger_subscriptions',
+    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuid_generate_v4()'), nullable=False),
+    sa.Column('name', sa.String(length=255), nullable=False, comment='Subscription instance name'),
+    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
+    sa.Column('user_id', models.types.StringUUID(), nullable=False),
+    sa.Column('provider_id', sa.String(length=255), nullable=False, comment='Provider identifier (e.g., plugin_id/provider_name)'),
+    sa.Column('endpoint_id', sa.String(length=255), nullable=False, comment='Subscription endpoint'),
+    sa.Column('parameters', sa.JSON(), nullable=False, comment='Subscription parameters JSON'),
+    sa.Column('properties', sa.JSON(), nullable=False, comment='Subscription properties JSON'),
+    sa.Column('credentials', sa.JSON(), nullable=False, comment='Subscription credentials JSON'),
+    sa.Column('credential_type', sa.String(length=50), nullable=False, comment='oauth or api_key'),
+    sa.Column('credential_expires_at', sa.Integer(), nullable=False, comment='OAuth token expiration timestamp, -1 for never'),
+    sa.Column('expires_at', sa.Integer(), nullable=False, comment='Subscription instance expiration timestamp, -1 for never'),
+    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.Column('updated_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.PrimaryKeyConstraint('id', name='trigger_provider_pkey'),
+    sa.UniqueConstraint('tenant_id', 'provider_id', 'name', name='unique_trigger_provider')
+    )
+    with op.batch_alter_table('trigger_subscriptions', schema=None) as batch_op:
+        batch_op.create_index('idx_trigger_providers_endpoint', ['endpoint_id'], unique=True)
+        batch_op.create_index('idx_trigger_providers_tenant_endpoint', ['tenant_id', 'endpoint_id'], unique=False)
+        batch_op.create_index('idx_trigger_providers_tenant_provider', ['tenant_id', 'provider_id'], unique=False)
+
+    op.create_table('workflow_plugin_triggers',
+    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuid_generate_v4()'), nullable=False),
+    sa.Column('app_id', models.types.StringUUID(), nullable=False),
+    sa.Column('node_id', sa.String(length=64), nullable=False),
+    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
+    sa.Column('provider_id', sa.String(length=512), nullable=False),
+    sa.Column('event_name', sa.String(length=255), nullable=False),
+    sa.Column('subscription_id', sa.String(length=255), nullable=False),
+    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.Column('updated_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.PrimaryKeyConstraint('id', name='workflow_plugin_trigger_pkey'),
+    sa.UniqueConstraint('app_id', 'node_id', name='uniq_app_node_subscription')
+    )
+    with op.batch_alter_table('workflow_plugin_triggers', schema=None) as batch_op:
+        batch_op.create_index('workflow_plugin_trigger_tenant_subscription_idx', ['tenant_id', 'subscription_id', 'event_name'], unique=False)
+
+    op.create_table('workflow_schedule_plans',
+    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
+    sa.Column('app_id', models.types.StringUUID(), nullable=False),
+    sa.Column('node_id', sa.String(length=64), nullable=False),
+    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
+    sa.Column('cron_expression', sa.String(length=255), nullable=False),
+    sa.Column('timezone', sa.String(length=64), nullable=False),
+    sa.Column('next_run_at', sa.DateTime(), nullable=True),
+    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.Column('updated_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.PrimaryKeyConstraint('id', name='workflow_schedule_plan_pkey'),
+    sa.UniqueConstraint('app_id', 'node_id', name='uniq_app_node')
+    )
+    with op.batch_alter_table('workflow_schedule_plans', schema=None) as batch_op:
+        batch_op.create_index('workflow_schedule_plan_next_idx', ['next_run_at'], unique=False)
+
+    op.create_table('workflow_trigger_logs',
+    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
+    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
+    sa.Column('app_id', models.types.StringUUID(), nullable=False),
+    sa.Column('workflow_id', models.types.StringUUID(), nullable=False),
+    sa.Column('workflow_run_id', models.types.StringUUID(), nullable=True),
+    sa.Column('root_node_id', sa.String(length=255), nullable=True),
+    sa.Column('trigger_metadata', sa.Text(), nullable=False),
+    sa.Column('trigger_type', models.types.EnumText(AppTriggerType, length=50), nullable=False),
+    sa.Column('trigger_data', sa.Text(), nullable=False),
+    sa.Column('inputs', sa.Text(), nullable=False),
+    sa.Column('outputs', sa.Text(), nullable=True),
+    sa.Column('status', models.types.EnumText(AppTriggerStatus, length=50), nullable=False),
+    sa.Column('error', sa.Text(), nullable=True),
+    sa.Column('queue_name', sa.String(length=100), nullable=False),
+    sa.Column('celery_task_id', sa.String(length=255), nullable=True),
+    sa.Column('retry_count', sa.Integer(), nullable=False),
+    sa.Column('elapsed_time', sa.Float(), nullable=True),
+    sa.Column('total_tokens', sa.Integer(), nullable=True),
+    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.Column('created_by_role', sa.String(length=255), nullable=False),
+    sa.Column('created_by', sa.String(length=255), nullable=False),
+    sa.Column('triggered_at', sa.DateTime(), nullable=True),
+    sa.Column('finished_at', sa.DateTime(), nullable=True),
+    sa.PrimaryKeyConstraint('id', name='workflow_trigger_log_pkey')
+    )
+    with op.batch_alter_table('workflow_trigger_logs', schema=None) as batch_op:
+        batch_op.create_index('workflow_trigger_log_created_at_idx', ['created_at'], unique=False)
+        batch_op.create_index('workflow_trigger_log_status_idx', ['status'], unique=False)
+        batch_op.create_index('workflow_trigger_log_tenant_app_idx', ['tenant_id', 'app_id'], unique=False)
+        batch_op.create_index('workflow_trigger_log_workflow_id_idx', ['workflow_id'], unique=False)
+        batch_op.create_index('workflow_trigger_log_workflow_run_idx', ['workflow_run_id'], unique=False)
+
+    op.create_table('workflow_webhook_triggers',
+    sa.Column('id', models.types.StringUUID(), server_default=sa.text('uuidv7()'), nullable=False),
+    sa.Column('app_id', models.types.StringUUID(), nullable=False),
+    sa.Column('node_id', sa.String(length=64), nullable=False),
+    sa.Column('tenant_id', models.types.StringUUID(), nullable=False),
+    sa.Column('webhook_id', sa.String(length=24), nullable=False),
+    sa.Column('created_by', models.types.StringUUID(), nullable=False),
+    sa.Column('created_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.Column('updated_at', sa.DateTime(), server_default=sa.text('CURRENT_TIMESTAMP'), nullable=False),
+    sa.PrimaryKeyConstraint('id', name='workflow_webhook_trigger_pkey'),
+    sa.UniqueConstraint('app_id', 'node_id', name='uniq_node'),
+    sa.UniqueConstraint('webhook_id', name='uniq_webhook_id')
+    )
+    with op.batch_alter_table('workflow_webhook_triggers', schema=None) as batch_op:
+        batch_op.create_index('workflow_webhook_trigger_tenant_idx', ['tenant_id'], unique=False)
+
+    with op.batch_alter_table('celery_taskmeta', schema=None) as batch_op:
+        batch_op.alter_column('task_id',
+               existing_type=sa.VARCHAR(length=155),
+               nullable=False)
+        batch_op.alter_column('status',
+               existing_type=sa.VARCHAR(length=50),
+               nullable=False)
+
+    with op.batch_alter_table('celery_tasksetmeta', schema=None) as batch_op:
+        batch_op.alter_column('taskset_id',
+               existing_type=sa.VARCHAR(length=155),
+               nullable=False)
+
+    with op.batch_alter_table('providers', schema=None) as batch_op:
+        batch_op.drop_column('credential_status')
+
+    # ### end Alembic commands ###
+
+
+def downgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    with op.batch_alter_table('providers', schema=None) as batch_op:
+        batch_op.add_column(sa.Column('credential_status', sa.VARCHAR(length=20), server_default=sa.text("'active'::character varying"), autoincrement=False, nullable=True))
+
+    with op.batch_alter_table('celery_tasksetmeta', schema=None) as batch_op:
+        batch_op.alter_column('taskset_id',
+               existing_type=sa.VARCHAR(length=155),
+               nullable=True)
+
+    with op.batch_alter_table('celery_taskmeta', schema=None) as batch_op:
+        batch_op.alter_column('status',
+               existing_type=sa.VARCHAR(length=50),
+               nullable=True)
+        batch_op.alter_column('task_id',
+               existing_type=sa.VARCHAR(length=155),
+               nullable=True)
+
+    with op.batch_alter_table('workflow_webhook_triggers', schema=None) as batch_op:
+        batch_op.drop_index('workflow_webhook_trigger_tenant_idx')
+
+    op.drop_table('workflow_webhook_triggers')
+    with op.batch_alter_table('workflow_trigger_logs', schema=None) as batch_op:
+        batch_op.drop_index('workflow_trigger_log_workflow_run_idx')
+        batch_op.drop_index('workflow_trigger_log_workflow_id_idx')
+        batch_op.drop_index('workflow_trigger_log_tenant_app_idx')
+        batch_op.drop_index('workflow_trigger_log_status_idx')
+        batch_op.drop_index('workflow_trigger_log_created_at_idx')
+
+    op.drop_table('workflow_trigger_logs')
+    with op.batch_alter_table('workflow_schedule_plans', schema=None) as batch_op:
+        batch_op.drop_index('workflow_schedule_plan_next_idx')
+
+    op.drop_table('workflow_schedule_plans')
+    with op.batch_alter_table('workflow_plugin_triggers', schema=None) as batch_op:
+        batch_op.drop_index('workflow_plugin_trigger_tenant_subscription_idx')
+
+    op.drop_table('workflow_plugin_triggers')
+    with op.batch_alter_table('trigger_subscriptions', schema=None) as batch_op:
+        batch_op.drop_index('idx_trigger_providers_tenant_provider')
+        batch_op.drop_index('idx_trigger_providers_tenant_endpoint')
+        batch_op.drop_index('idx_trigger_providers_endpoint')
+
+    op.drop_table('trigger_subscriptions')
+    op.drop_table('trigger_oauth_tenant_clients')
+    op.drop_table('trigger_oauth_system_clients')
+    with op.batch_alter_table('app_triggers', schema=None) as batch_op:
+        batch_op.drop_index('app_trigger_tenant_app_idx')
+
+    op.drop_table('app_triggers')
+    # ### end Alembic commands ###
diff --git a/api/models/trigger.py b/api/models/trigger.py
index 5237a512e4..22bdcbca33 100644
--- a/api/models/trigger.py
+++ b/api/models/trigger.py
@@ -196,7 +196,7 @@ class WorkflowTriggerLog(Base):
     workflow_id: Mapped[str] = mapped_column(StringUUID, nullable=False)
     workflow_run_id: Mapped[Optional[str]] = mapped_column(StringUUID, nullable=True)
     root_node_id: Mapped[Optional[str]] = mapped_column(String(255), nullable=True)
-    trigger_metadata: Mapped[Optional[str]] = mapped_column(sa.Text, nullable=True)
+    trigger_metadata: Mapped[str] = mapped_column(sa.Text, nullable=False)
     trigger_type: Mapped[str] = mapped_column(EnumText(AppTriggerType, length=50), nullable=False)
     trigger_data: Mapped[str] = mapped_column(sa.Text, nullable=False)  # Full TriggerData as JSON
     inputs: Mapped[str] = mapped_column(sa.Text, nullable=False)  # Just inputs for easy viewing

From 1d03e0e9fc6507275b87da42540909c61d78b76a Mon Sep 17 00:00:00 2001
From: yessenia <yessenia.contact@gmail.com>
Date: Wed, 29 Oct 2025 18:51:27 +0800
Subject: [PATCH 02/17] fix(trigger): hide input params when no subscription

---
 .../components/plugins/readme-panel/index.tsx |  50 ++++----
 .../_base/components/workflow-panel/index.tsx | 114 ++++++++++--------
 .../workflow-panel/trigger-subscription.tsx   |  33 +----
 .../components/trigger-form/index.tsx         |   6 +-
 .../components/trigger-form/item.tsx          |   6 +-
 .../workflow/nodes/trigger-plugin/panel.tsx   |   7 +-
 .../nodes/trigger-plugin/use-config.ts        |  20 ++-
 7 files changed, 111 insertions(+), 125 deletions(-)

diff --git a/web/app/components/plugins/readme-panel/index.tsx b/web/app/components/plugins/readme-panel/index.tsx
index b77d59fb0b..70d1e0db2c 100644
--- a/web/app/components/plugins/readme-panel/index.tsx
+++ b/web/app/components/plugins/readme-panel/index.tsx
@@ -2,7 +2,6 @@
 import ActionButton from '@/app/components/base/action-button'
 import Loading from '@/app/components/base/loading'
 import { Markdown } from '@/app/components/base/markdown'
-import Modal from '@/app/components/base/modal'
 import { useLanguage } from '@/app/components/header/account-setting/model-provider-page/hooks'
 import { usePluginReadme } from '@/service/use-plugins'
 import cn from '@/utils/classnames'
@@ -85,29 +84,36 @@ const ReadmePanel: FC = () => {
     </div>
   )
 
-  return showType === ReadmeShowType.drawer ? createPortal(
-    <div className='pointer-events-none fixed inset-0 z-[9997] flex justify-start'>
-      <div
-        className={cn(
-          'pointer-events-auto mb-2 ml-2 mr-2 mt-16 w-[600px] max-w-[600px] justify-start rounded-2xl border-[0.5px] border-components-panel-border bg-components-panel-bg p-0 shadow-xl',
-        )}
-      >
-        {children}
+  const portalContent = showType === ReadmeShowType.drawer
+    ? (
+      <div className='pointer-events-none fixed inset-0 z-[9997] flex justify-start'>
+        <div
+          className={cn(
+            'pointer-events-auto mb-2 ml-2 mr-2 mt-16 w-[600px] max-w-[600px] justify-start rounded-2xl border-[0.5px] border-components-panel-border bg-components-panel-bg p-0 shadow-xl',
+          )}
+        >
+          {children}
+        </div>
       </div>
-    </div>,
+    )
+    : (
+      <div className='pointer-events-none fixed inset-0 z-[9997] flex items-center justify-center p-2'>
+        <div
+          className={cn(
+            'pointer-events-auto relative h-[calc(100vh-16px)] w-full max-w-[800px] rounded-2xl bg-components-panel-bg p-0 shadow-xl',
+          )}
+          onClick={(event) => {
+            event.stopPropagation()
+          }}
+        >
+          {children}
+        </div>
+      </div>
+    )
+
+  return createPortal(
+    portalContent,
     document.body,
-  ) : (
-    <Modal
-      isShow={!!detail}
-      onClose={onClose}
-      overlayOpacity={true}
-      className='h-[calc(100vh-16px)] max-w-[800px] p-0'
-      wrapperClassName='!z-[102]'
-      containerClassName='p-2'
-      clickOutsideNotClose={true}
-    >
-      {children}
-    </Modal>
   )
 }
 
diff --git a/web/app/components/workflow/nodes/_base/components/workflow-panel/index.tsx b/web/app/components/workflow/nodes/_base/components/workflow-panel/index.tsx
index 32b9cb2671..93589795a6 100644
--- a/web/app/components/workflow/nodes/_base/components/workflow-panel/index.tsx
+++ b/web/app/components/workflow/nodes/_base/components/workflow-panel/index.tsx
@@ -1,5 +1,17 @@
 import { useStore as useAppStore } from '@/app/components/app/store'
+import { Stop } from '@/app/components/base/icons/src/vender/line/mediaAndDevices'
 import Tooltip from '@/app/components/base/tooltip'
+import { useLanguage } from '@/app/components/header/account-setting/model-provider-page/hooks'
+import {
+  AuthCategory,
+  AuthorizedInDataSourceNode,
+  AuthorizedInNode,
+  PluginAuth,
+  PluginAuthInDataSourceNode,
+} from '@/app/components/plugins/plugin-auth'
+import { usePluginStore } from '@/app/components/plugins/plugin-detail-panel/store'
+import type { SimpleSubscription } from '@/app/components/plugins/plugin-detail-panel/subscription-list'
+import { ReadmeEntrance } from '@/app/components/plugins/readme-panel/entrance'
 import BlockIcon from '@/app/components/workflow/block-icon'
 import {
   WorkflowHistoryEvent,
@@ -11,7 +23,14 @@ import {
   useToolIcon,
   useWorkflowHistory,
 } from '@/app/components/workflow/hooks'
+import { useHooksStore } from '@/app/components/workflow/hooks-store'
+import useInspectVarsCrud from '@/app/components/workflow/hooks/use-inspect-vars-crud'
 import Split from '@/app/components/workflow/nodes/_base/components/split'
+import DataSourceBeforeRunForm from '@/app/components/workflow/nodes/data-source/before-run-form'
+import type { CustomRunFormProps } from '@/app/components/workflow/nodes/data-source/types'
+import { DataSourceClassification } from '@/app/components/workflow/nodes/data-source/types'
+import { useLogs } from '@/app/components/workflow/run/hooks'
+import SpecialResultPanel from '@/app/components/workflow/run/special-result-panel'
 import { useStore } from '@/app/components/workflow/store'
 import { BlockEnum, type Node, NodeRunningStatus } from '@/app/components/workflow/types'
 import {
@@ -20,16 +39,18 @@ import {
   hasRetryNode,
   isSupportCustomRunForm,
 } from '@/app/components/workflow/utils'
+import { useModalContext } from '@/context/modal-context'
+import { useAllBuiltInTools } from '@/service/use-tools'
 import { useAllTriggerPlugins } from '@/service/use-triggers'
+import { FlowType } from '@/types/common'
+import { canFindTool } from '@/utils'
 import cn from '@/utils/classnames'
 import {
   RiCloseLine,
   RiPlayLargeLine,
 } from '@remixicon/react'
-import type {
-  FC,
-  ReactNode,
-} from 'react'
+import { debounce } from 'lodash-es'
+import type { FC, ReactNode } from 'react'
 import React, {
   cloneElement,
   memo,
@@ -42,44 +63,18 @@ import React, {
 import { useTranslation } from 'react-i18next'
 import { useShallow } from 'zustand/react/shallow'
 import { useResizePanel } from '../../hooks/use-resize-panel'
+import BeforeRunForm from '../before-run-form'
+import PanelWrap from '../before-run-form/panel-wrap'
 import ErrorHandleOnPanel from '../error-handle/error-handle-on-panel'
 import HelpLink from '../help-link'
 import NextStep from '../next-step'
 import PanelOperator from '../panel-operator'
 import RetryOnPanel from '../retry/retry-on-panel'
-import {
-  DescriptionInput,
-  TitleInput,
-} from '../title-description-input'
-import Tab, { TabType } from './tab'
-// import AuthMethodSelector from '@/app/components/workflow/nodes/trigger-plugin/components/auth-method-selector'
-import { Stop } from '@/app/components/base/icons/src/vender/line/mediaAndDevices'
-import {
-  AuthCategory,
-  AuthorizedInDataSourceNode,
-  AuthorizedInNode,
-  PluginAuth,
-  PluginAuthInDataSourceNode,
-} from '@/app/components/plugins/plugin-auth'
-import type { SimpleSubscription } from '@/app/components/plugins/plugin-detail-panel/subscription-list'
-import { useHooksStore } from '@/app/components/workflow/hooks-store'
-import useInspectVarsCrud from '@/app/components/workflow/hooks/use-inspect-vars-crud'
-import DataSourceBeforeRunForm from '@/app/components/workflow/nodes/data-source/before-run-form'
-import type { CustomRunFormProps } from '@/app/components/workflow/nodes/data-source/types'
-import { DataSourceClassification } from '@/app/components/workflow/nodes/data-source/types'
-import { useLogs } from '@/app/components/workflow/run/hooks'
-import SpecialResultPanel from '@/app/components/workflow/run/special-result-panel'
-import { useModalContext } from '@/context/modal-context'
-import { FlowType } from '@/types/common'
-import { canFindTool } from '@/utils'
-import { debounce } from 'lodash-es'
-import BeforeRunForm from '../before-run-form'
-import PanelWrap from '../before-run-form/panel-wrap'
+import { DescriptionInput, TitleInput } from '../title-description-input'
 import LastRun from './last-run'
 import useLastRun from './last-run/use-last-run'
+import Tab, { TabType } from './tab'
 import { TriggerSubscription } from './trigger-subscription'
-import { ReadmeEntrance } from '@/app/components/plugins/readme-panel/entrance'
-import { useAllBuiltInTools } from '@/service/use-tools'
 
 const getCustomRunForm = (params: CustomRunFormProps): React.JSX.Element => {
   const nodeType = params.payload.type
@@ -103,6 +98,7 @@ const BasePanel: FC<BasePanelProps> = ({
   children,
 }) => {
   const { t } = useTranslation()
+  const language = useLanguage()
   const { showMessageLogModal } = useAppStore(useShallow(state => ({
     showMessageLogModal: state.showMessageLogModal,
   })))
@@ -224,6 +220,7 @@ const BasePanel: FC<BasePanelProps> = ({
   useEffect(() => {
     hasClickRunning.current = false
   }, [id])
+
   const {
     nodesMap,
   } = useNodesMetaData()
@@ -278,12 +275,7 @@ const BasePanel: FC<BasePanelProps> = ({
   }, [pendingSingleRun, id, handleSingleRun, handleStop, setPendingSingleRun])
 
   const logParams = useLogs()
-  const passedLogParams = (() => {
-    if ([BlockEnum.Tool, BlockEnum.Agent, BlockEnum.Iteration, BlockEnum.Loop].includes(data.type))
-      return logParams
-
-    return {}
-  })()
+  const passedLogParams = useMemo(() => [BlockEnum.Tool, BlockEnum.Agent, BlockEnum.Iteration, BlockEnum.Loop].includes(data.type) ? logParams : {}, [data.type, logParams])
 
   const storeBuildInTools = useStore(s => s.buildInTools)
   const { data: buildInTools } = useAllBuiltInTools()
@@ -295,16 +287,32 @@ const BasePanel: FC<BasePanelProps> = ({
     return data.type === BlockEnum.Tool && currToolCollection?.allow_delete
   }, [data.type, currToolCollection?.allow_delete])
 
-  const { data: triggerProviders = [] } = useAllTriggerPlugins()
-  const currentTriggerProvider = useMemo(() => {
-    if (!data.provider_id || !data.provider_name)
+  // only fetch trigger plugins when the node is a trigger plugin
+  const { data: triggerPlugins = [] } = useAllTriggerPlugins(data.type === BlockEnum.TriggerPlugin)
+  const currentTriggerPlugin = useMemo(() => {
+    if (data.type !== BlockEnum.TriggerPlugin || !data.plugin_id || !triggerPlugins?.length)
       return undefined
-    return triggerProviders.find(p => p.name === data.provider_id) // todo: confirm
-  }, [data.type, data.provider_id, data.provider_name, triggerProviders])
+    return triggerPlugins?.find(p => p.plugin_id === data.plugin_id)
+  }, [data.type, data.plugin_id, triggerPlugins])
+  const { setDetail } = usePluginStore()
 
-  const showTriggerConfig = useMemo(() => {
-    return data.type === BlockEnum.TriggerPlugin && currentTriggerProvider
-  }, [data.type, currentTriggerProvider])
+  useEffect(() => {
+    if (currentTriggerPlugin) {
+      setDetail({
+        name: currentTriggerPlugin.label[language],
+        plugin_id: currentTriggerPlugin.plugin_id || '',
+        provider: currentTriggerPlugin.name,
+        declaration: {
+          tool: undefined,
+          // @ts-expect-error just remain the necessary fields
+          trigger: {
+            subscription_schema: currentTriggerPlugin.subscription_schema || [],
+            subscription_constructor: currentTriggerPlugin.subscription_constructor,
+          },
+        },
+      })
+    }
+  }, [currentTriggerPlugin, setDetail])
 
   const dataSourceList = useStore(s => s.dataSourceList)
 
@@ -352,14 +360,14 @@ const BasePanel: FC<BasePanelProps> = ({
         pluginDetail = currentDataSource
         break
       case BlockEnum.TriggerPlugin:
-        pluginDetail = currentTriggerProvider
+        pluginDetail = currentTriggerPlugin
         break
 
       default:
         break
     }
     return !pluginDetail ? null : <ReadmeEntrance pluginDetail={pluginDetail as any} className='mt-auto' />
-  }, [data.type, currToolCollection, currentDataSource, currentTriggerProvider])
+  }, [data.type, currToolCollection, currentDataSource, currentTriggerPlugin])
 
   if (logParams.showSpecialResultPanel) {
     return (
@@ -558,9 +566,9 @@ const BasePanel: FC<BasePanelProps> = ({
             )
           }
           {
-            showTriggerConfig && (
+            currentTriggerPlugin && (
               <TriggerSubscription
-                data={data}
+                subscriptionIdSelected={data.subscription_id}
                 onSubscriptionChange={handleSubscriptionChange}
               >
                 <Tab
@@ -571,7 +579,7 @@ const BasePanel: FC<BasePanelProps> = ({
             )
           }
           {
-            !needsToolAuth && !currentDataSource && !showTriggerConfig && (
+            !needsToolAuth && !currentDataSource && !currentTriggerPlugin && (
               <div className='flex items-center justify-between pl-4 pr-3'>
                 <Tab
                   value={tabType}
diff --git a/web/app/components/workflow/nodes/_base/components/workflow-panel/trigger-subscription.tsx b/web/app/components/workflow/nodes/_base/components/workflow-panel/trigger-subscription.tsx
index 4ae499580f..811516df3d 100644
--- a/web/app/components/workflow/nodes/_base/components/workflow-panel/trigger-subscription.tsx
+++ b/web/app/components/workflow/nodes/_base/components/workflow-panel/trigger-subscription.tsx
@@ -1,52 +1,25 @@
-import { useLanguage } from '@/app/components/header/account-setting/model-provider-page/hooks'
 import type { SimpleSubscription } from '@/app/components/plugins/plugin-detail-panel/subscription-list'
 import { CreateButtonType, CreateSubscriptionButton } from '@/app/components/plugins/plugin-detail-panel/subscription-list/create'
 import { SubscriptionSelectorEntry } from '@/app/components/plugins/plugin-detail-panel/subscription-list/selector-entry'
-import { usePluginStore } from '@/app/components/plugins/plugin-detail-panel/store'
 import { useSubscriptionList } from '@/app/components/plugins/plugin-detail-panel/subscription-list/use-subscription-list'
-import useConfig from '@/app/components/workflow/nodes/trigger-plugin/use-config'
-import type { Node } from '@/app/components/workflow/types'
 import cn from '@/utils/classnames'
 import type { FC } from 'react'
-import { useEffect } from 'react'
 
 type TriggerSubscriptionProps = {
-  data: Node['data']
+  subscriptionIdSelected?: string
   onSubscriptionChange: (v: SimpleSubscription, callback?: () => void) => void
   children: React.ReactNode
 }
 
-export const TriggerSubscription: FC<TriggerSubscriptionProps> = ({ data, onSubscriptionChange, children }) => {
-  // @ts-expect-error TODO: fix this
-  const { currentProvider } = useConfig(data.id as string, data)
-  const { setDetail } = usePluginStore()
-  const language = useLanguage()
+export const TriggerSubscription: FC<TriggerSubscriptionProps> = ({ subscriptionIdSelected, onSubscriptionChange, children }) => {
   const { subscriptions } = useSubscriptionList()
   const subscriptionCount = subscriptions?.length || 0
 
-  useEffect(() => {
-    if (currentProvider) {
-      setDetail({
-        name: currentProvider.label[language],
-        plugin_id: currentProvider.plugin_id || '',
-        provider: currentProvider.name,
-        declaration: {
-          tool: undefined,
-          // @ts-expect-error just remain the necessary fields
-          trigger: {
-            subscription_schema: currentProvider.subscription_schema || [],
-            subscription_constructor: currentProvider.subscription_constructor,
-          },
-        },
-      })
-    }
-  }, [currentProvider, setDetail])
-
   return <div className={cn('px-4', subscriptionCount > 0 && 'flex items-center justify-between pr-3')}>
     {!subscriptionCount && <CreateSubscriptionButton buttonType={CreateButtonType.FULL_BUTTON} />}
     {children}
     {subscriptionCount > 0 && <SubscriptionSelectorEntry
-      selectedId={data.subscription_id}
+      selectedId={subscriptionIdSelected}
       onSelect={onSubscriptionChange}
     />}
   </div>
diff --git a/web/app/components/workflow/nodes/trigger-plugin/components/trigger-form/index.tsx b/web/app/components/workflow/nodes/trigger-plugin/components/trigger-form/index.tsx
index 17c71b6b95..93bf788c34 100644
--- a/web/app/components/workflow/nodes/trigger-plugin/components/trigger-form/index.tsx
+++ b/web/app/components/workflow/nodes/trigger-plugin/components/trigger-form/index.tsx
@@ -14,7 +14,7 @@ type Props = {
   onChange: (value: PluginTriggerVarInputs) => void
   onOpen?: (index: number) => void
   inPanel?: boolean
-  currentTrigger?: Event
+  currentEvent?: Event
   currentProvider?: TriggerWithProvider
   extraParams?: Record<string, any>
   disableVariableInsertion?: boolean
@@ -27,7 +27,7 @@ const TriggerForm: FC<Props> = ({
   value,
   onChange,
   inPanel,
-  currentTrigger,
+  currentEvent,
   currentProvider,
   extraParams,
   disableVariableInsertion = false,
@@ -44,7 +44,7 @@ const TriggerForm: FC<Props> = ({
             value={value}
             onChange={onChange}
             inPanel={inPanel}
-            currentTrigger={currentTrigger}
+            currentEvent={currentEvent}
             currentProvider={currentProvider}
             extraParams={extraParams}
             disableVariableInsertion={disableVariableInsertion}
diff --git a/web/app/components/workflow/nodes/trigger-plugin/components/trigger-form/item.tsx b/web/app/components/workflow/nodes/trigger-plugin/components/trigger-form/item.tsx
index 22331aa578..678c12f02a 100644
--- a/web/app/components/workflow/nodes/trigger-plugin/components/trigger-form/item.tsx
+++ b/web/app/components/workflow/nodes/trigger-plugin/components/trigger-form/item.tsx
@@ -22,7 +22,7 @@ type Props = {
   value: PluginTriggerVarInputs
   onChange: (value: PluginTriggerVarInputs) => void
   inPanel?: boolean
-  currentTrigger?: Event
+  currentEvent?: Event
   currentProvider?: TriggerWithProvider
   extraParams?: Record<string, any>
   disableVariableInsertion?: boolean
@@ -35,7 +35,7 @@ const TriggerFormItem: FC<Props> = ({
   value,
   onChange,
   inPanel,
-  currentTrigger,
+  currentEvent,
   currentProvider,
   extraParams,
   disableVariableInsertion = false,
@@ -91,7 +91,7 @@ const TriggerFormItem: FC<Props> = ({
         value={value}
         onChange={onChange}
         inPanel={inPanel}
-        currentTool={currentTrigger}
+        currentTool={currentEvent}
         currentProvider={currentProvider}
         providerType='trigger'
         extraParams={extraParams}
diff --git a/web/app/components/workflow/nodes/trigger-plugin/panel.tsx b/web/app/components/workflow/nodes/trigger-plugin/panel.tsx
index f7dc8374e7..9b4d8058b1 100644
--- a/web/app/components/workflow/nodes/trigger-plugin/panel.tsx
+++ b/web/app/components/workflow/nodes/trigger-plugin/panel.tsx
@@ -22,7 +22,8 @@ const Panel: FC<NodePanelProps<PluginTriggerNodeType>> = ({
     outputSchema,
     hasObjectOutput,
     currentProvider,
-    currentTrigger,
+    currentEvent,
+    subscriptionSelected,
   } = useConfig(id, data)
   const disableVariableInsertion = data.type === BlockEnum.TriggerPlugin
 
@@ -36,7 +37,7 @@ const Panel: FC<NodePanelProps<PluginTriggerNodeType>> = ({
   return (
     <div className='mt-2'>
       {/* Dynamic Parameters Form - Only show when authenticated */}
-      {triggerParameterSchema.length > 0 && (
+      {triggerParameterSchema.length > 0 && subscriptionSelected && (
         <>
           <div className='px-4 pb-4'>
             <TriggerForm
@@ -46,7 +47,7 @@ const Panel: FC<NodePanelProps<PluginTriggerNodeType>> = ({
               value={triggerParameterValue}
               onChange={setTriggerParameterValue}
               currentProvider={currentProvider}
-              currentTrigger={currentTrigger}
+              currentEvent={currentEvent}
               disableVariableInsertion={disableVariableInsertion}
             />
           </div>
diff --git a/web/app/components/workflow/nodes/trigger-plugin/use-config.ts b/web/app/components/workflow/nodes/trigger-plugin/use-config.ts
index 7d75aa5e00..6ea711cd31 100644
--- a/web/app/components/workflow/nodes/trigger-plugin/use-config.ts
+++ b/web/app/components/workflow/nodes/trigger-plugin/use-config.ts
@@ -86,6 +86,7 @@ const useConfig = (id: string, payload: PluginTriggerNodeType) => {
     event_name: event_name,
     config = {},
     event_parameters: rawEventParameters = {},
+    subscription_id,
   } = inputs
 
   const event_parameters = useMemo(
@@ -97,16 +98,6 @@ const useConfig = (id: string, payload: PluginTriggerNodeType) => {
     [config],
   )
 
-  // Construct provider for authentication check
-  const authProvider = useMemo(() => {
-    return provider_name || ''
-  }, [provider_id, provider_name])
-
-  const { data: subscriptions = [] } = useTriggerSubscriptions(
-    authProvider,
-    !!authProvider,
-  )
-
   const currentProvider = useMemo<TriggerWithProvider | undefined>(() => {
     return triggerPlugins.find(
       provider =>
@@ -116,6 +107,12 @@ const useConfig = (id: string, payload: PluginTriggerNodeType) => {
     )
   }, [triggerPlugins, provider_name, provider_id])
 
+  const { data: subscriptions = [] } = useTriggerSubscriptions(provider_id || '')
+
+  const subscriptionSelected = useMemo(() => {
+    return subscriptions?.find(s => s.id === subscription_id)
+  }, [subscriptions, subscription_id])
+
   const currentEvent = useMemo<Event | undefined>(() => {
     return currentProvider?.events.find(
       event => event.name === event_name,
@@ -221,7 +218,7 @@ const useConfig = (id: string, payload: PluginTriggerNodeType) => {
     readOnly,
     inputs,
     currentProvider,
-    currentTrigger: currentEvent,
+    currentEvent,
     triggerParameterSchema,
     triggerParameterValue,
     setTriggerParameterValue,
@@ -229,6 +226,7 @@ const useConfig = (id: string, payload: PluginTriggerNodeType) => {
     outputSchema,
     hasObjectOutput,
     subscriptions,
+    subscriptionSelected,
   }
 }
 

From 6e0765fbaf6313a7f87ccf12c16c55c30bf31767 Mon Sep 17 00:00:00 2001
From: lyzno1 <yuanyouhuilyz@gmail.com>
Date: Thu, 30 Oct 2025 14:15:30 +0800
Subject: [PATCH 03/17] feat: add install check for tools, triggers and
 datasources

---
 .../workflow/block-selector/data-sources.tsx  |   1 +
 .../block-selector/tool/action-item.tsx       |   1 +
 .../workflow/block-selector/tool/tool.tsx     |   2 +
 .../workflow/block-selector/types.ts          |   2 +
 .../hooks/use-node-plugin-installation.ts     | 208 ++++++++++++++++++
 .../workflow/nodes/data-source/node.tsx       |  32 ++-
 .../workflow/nodes/data-source/types.ts       |   1 +
 .../components/workflow/nodes/tool/node.tsx   |  78 ++++---
 .../components/workflow/nodes/tool/types.ts   |   1 +
 .../workflow/nodes/trigger-plugin/node.tsx    |  22 +-
 .../workflow/nodes/trigger-plugin/types.ts    |   1 +
 web/app/components/workflow/types.ts          |   1 +
 12 files changed, 318 insertions(+), 32 deletions(-)
 create mode 100644 web/app/components/workflow/hooks/use-node-plugin-installation.ts

diff --git a/web/app/components/workflow/block-selector/data-sources.tsx b/web/app/components/workflow/block-selector/data-sources.tsx
index 3961f63dbe..b98a52dcff 100644
--- a/web/app/components/workflow/block-selector/data-sources.tsx
+++ b/web/app/components/workflow/block-selector/data-sources.tsx
@@ -63,6 +63,7 @@ const DataSources = ({
       datasource_name: toolDefaultValue?.tool_name,
       datasource_label: toolDefaultValue?.tool_label,
       title: toolDefaultValue?.title,
+      plugin_unique_identifier: toolDefaultValue?.plugin_unique_identifier,
     }
     // Update defaultValue with fileExtensions if this is the local file data source
     if (toolDefaultValue?.provider_id === 'langgenius/file' && toolDefaultValue?.provider_name === 'file') {
diff --git a/web/app/components/workflow/block-selector/tool/action-item.tsx b/web/app/components/workflow/block-selector/tool/action-item.tsx
index e2c28602f8..01c319327a 100644
--- a/web/app/components/workflow/block-selector/tool/action-item.tsx
+++ b/web/app/components/workflow/block-selector/tool/action-item.tsx
@@ -72,6 +72,7 @@ const ToolItem: FC<Props> = ({
             provider_type: provider.type,
             provider_name: provider.name,
             plugin_id: provider.plugin_id,
+            plugin_unique_identifier: provider.plugin_unique_identifier,
             provider_icon: normalizeProviderIcon(provider.icon),
             tool_name: payload.name,
             tool_label: payload.label[language],
diff --git a/web/app/components/workflow/block-selector/tool/tool.tsx b/web/app/components/workflow/block-selector/tool/tool.tsx
index 5ac043e933..38be8d19d6 100644
--- a/web/app/components/workflow/block-selector/tool/tool.tsx
+++ b/web/app/components/workflow/block-selector/tool/tool.tsx
@@ -94,6 +94,7 @@ const Tool: FC<Props> = ({
                 provider_type: payload.type,
                 provider_name: payload.name,
                 plugin_id: payload.plugin_id,
+                plugin_unique_identifier: payload.plugin_unique_identifier,
                 provider_icon: normalizeProviderIcon(payload.icon),
                 tool_name: tool.name,
                 tool_label: tool.label[language],
@@ -175,6 +176,7 @@ const Tool: FC<Props> = ({
               provider_type: payload.type,
               provider_name: payload.name,
               plugin_id: payload.plugin_id,
+              plugin_unique_identifier: payload.plugin_unique_identifier,
               provider_icon: normalizeProviderIcon(payload.icon),
               tool_name: tool.name,
               tool_label: tool.label[language],
diff --git a/web/app/components/workflow/block-selector/types.ts b/web/app/components/workflow/block-selector/types.ts
index 512621a552..e995974b87 100644
--- a/web/app/components/workflow/block-selector/types.ts
+++ b/web/app/components/workflow/block-selector/types.ts
@@ -59,6 +59,7 @@ export type ToolDefaultValue = PluginCommonDefaultValue & {
   meta?: PluginMeta
   plugin_id?: string
   provider_icon?: Collection['icon']
+  plugin_unique_identifier?: string
 }
 
 export type DataSourceDefaultValue = Omit<PluginCommonDefaultValue, 'provider_id'> & {
@@ -69,6 +70,7 @@ export type DataSourceDefaultValue = Omit<PluginCommonDefaultValue, 'provider_id
   datasource_label: string
   title: string
   fileExtensions?: string[]
+  plugin_unique_identifier?: string
 }
 
 export type PluginDefaultValue = ToolDefaultValue | DataSourceDefaultValue | TriggerDefaultValue
diff --git a/web/app/components/workflow/hooks/use-node-plugin-installation.ts b/web/app/components/workflow/hooks/use-node-plugin-installation.ts
new file mode 100644
index 0000000000..d82ed0ac61
--- /dev/null
+++ b/web/app/components/workflow/hooks/use-node-plugin-installation.ts
@@ -0,0 +1,208 @@
+import { useCallback, useMemo } from 'react'
+import { BlockEnum, type CommonNodeType } from '../types'
+import type { ToolNodeType } from '../nodes/tool/types'
+import type { PluginTriggerNodeType } from '../nodes/trigger-plugin/types'
+import type { DataSourceNodeType } from '../nodes/data-source/types'
+import { CollectionType } from '@/app/components/tools/types'
+import {
+  useAllBuiltInTools,
+  useAllCustomTools,
+  useAllMCPTools,
+  useAllWorkflowTools,
+  useInvalidToolsByType,
+} from '@/service/use-tools'
+import {
+  useAllTriggerPlugins,
+  useInvalidateAllTriggerPlugins,
+} from '@/service/use-triggers'
+import { useInvalidDataSourceList } from '@/service/use-pipeline'
+import { useStore } from '../store'
+import { canFindTool } from '@/utils'
+
+type InstallationState = {
+  isChecking: boolean
+  isMissing: boolean
+  uniqueIdentifier?: string
+  canInstall: boolean
+  onInstallSuccess: () => void
+  shouldDim: boolean
+}
+
+const useToolInstallation = (data: ToolNodeType): InstallationState => {
+  const builtInQuery = useAllBuiltInTools()
+  const customQuery = useAllCustomTools()
+  const workflowQuery = useAllWorkflowTools()
+  const mcpQuery = useAllMCPTools()
+  const invalidateTools = useInvalidToolsByType(data.provider_type)
+
+  const collectionInfo = useMemo(() => {
+    switch (data.provider_type) {
+      case CollectionType.builtIn:
+        return {
+          list: builtInQuery.data,
+          isLoading: builtInQuery.isLoading,
+        }
+      case CollectionType.custom:
+        return {
+          list: customQuery.data,
+          isLoading: customQuery.isLoading,
+        }
+      case CollectionType.workflow:
+        return {
+          list: workflowQuery.data,
+          isLoading: workflowQuery.isLoading,
+        }
+      case CollectionType.mcp:
+        return {
+          list: mcpQuery.data,
+          isLoading: mcpQuery.isLoading,
+        }
+      default:
+        return undefined
+    }
+  }, [
+    builtInQuery.data,
+    builtInQuery.isLoading,
+    customQuery.data,
+    customQuery.isLoading,
+    data.provider_type,
+    mcpQuery.data,
+    mcpQuery.isLoading,
+    workflowQuery.data,
+    workflowQuery.isLoading,
+  ])
+
+  const collection = collectionInfo?.list
+  const isLoading = collectionInfo?.isLoading ?? false
+  const isResolved = !!collectionInfo && !isLoading
+
+  const matchedCollection = useMemo(() => {
+    if (!collection || !collection.length)
+      return undefined
+
+    return collection.find((toolWithProvider) => {
+      if (data.plugin_id && toolWithProvider.plugin_id === data.plugin_id)
+        return true
+      if (canFindTool(toolWithProvider.id, data.provider_id))
+        return true
+      if (toolWithProvider.name === data.provider_name)
+        return true
+      return false
+    })
+  }, [collection, data.plugin_id, data.provider_id, data.provider_name])
+
+  const uniqueIdentifier = data.plugin_unique_identifier || data.plugin_id || data.provider_id
+  const canInstall = Boolean(data.plugin_unique_identifier)
+
+  const onInstallSuccess = useCallback(() => {
+    if (invalidateTools)
+      invalidateTools()
+  }, [invalidateTools])
+
+  return {
+    isChecking: !!collectionInfo && !isResolved,
+    isMissing: isResolved && !matchedCollection,
+    uniqueIdentifier,
+    canInstall,
+    onInstallSuccess,
+  }
+}
+
+const useTriggerInstallation = (data: PluginTriggerNodeType): InstallationState => {
+  const triggerPluginsQuery = useAllTriggerPlugins()
+  const invalidateTriggers = useInvalidateAllTriggerPlugins()
+
+  const triggerProviders = triggerPluginsQuery.data
+  const isLoading = triggerPluginsQuery.isLoading
+
+  const matchedProvider = useMemo(() => {
+    if (!triggerProviders || !triggerProviders.length)
+      return undefined
+
+    return triggerProviders.find(provider =>
+      provider.name === data.provider_name
+      || provider.id === data.provider_id
+      || (data.plugin_id && provider.plugin_id === data.plugin_id),
+    )
+  }, [
+    data.plugin_id,
+    data.provider_id,
+    data.provider_name,
+    triggerProviders,
+  ])
+
+  const uniqueIdentifier = data.plugin_unique_identifier || data.plugin_id || data.provider_id
+  const canInstall = Boolean(data.plugin_unique_identifier)
+
+  const onInstallSuccess = useCallback(() => {
+    invalidateTriggers()
+  }, [invalidateTriggers])
+
+  return {
+    isChecking: isLoading,
+    isMissing: !isLoading && !!triggerProviders && !matchedProvider,
+    uniqueIdentifier,
+    canInstall,
+    onInstallSuccess,
+  }
+}
+
+const useDataSourceInstallation = (data: DataSourceNodeType): InstallationState => {
+  const dataSourceList = useStore(s => s.dataSourceList)
+  const invalidateDataSourceList = useInvalidDataSourceList()
+
+  const matchedPlugin = useMemo(() => {
+    if (!dataSourceList || !dataSourceList.length)
+      return undefined
+
+    return dataSourceList.find((item) => {
+      if (data.plugin_unique_identifier && item.plugin_unique_identifier === data.plugin_unique_identifier)
+        return true
+      if (data.plugin_id && item.plugin_id === data.plugin_id)
+        return true
+      if (data.provider_name && item.provider === data.provider_name)
+        return true
+      return false
+    })
+  }, [data.plugin_id, data.plugin_unique_identifier, data.provider_name, dataSourceList])
+
+  const uniqueIdentifier = data.plugin_unique_identifier || data.plugin_id
+  const canInstall = Boolean(data.plugin_unique_identifier)
+
+  const onInstallSuccess = useCallback(() => {
+    invalidateDataSourceList()
+  }, [invalidateDataSourceList])
+
+  const hasLoadedList = dataSourceList !== undefined
+
+  return {
+    isChecking: !hasLoadedList,
+    isMissing: hasLoadedList && !matchedPlugin,
+    uniqueIdentifier,
+    canInstall,
+    onInstallSuccess,
+  }
+}
+
+export const useNodePluginInstallation = (data: CommonNodeType): InstallationState => {
+  const toolInstallation = useToolInstallation(data as ToolNodeType)
+  const triggerInstallation = useTriggerInstallation(data as PluginTriggerNodeType)
+  const dataSourceInstallation = useDataSourceInstallation(data as DataSourceNodeType)
+
+  switch (data.type as BlockEnum) {
+    case BlockEnum.Tool:
+      return toolInstallation
+    case BlockEnum.TriggerPlugin:
+      return triggerInstallation
+    case BlockEnum.DataSource:
+      return dataSourceInstallation
+    default:
+      return {
+        isChecking: false,
+        isMissing: false,
+        uniqueIdentifier: undefined,
+        canInstall: false,
+        onInstallSuccess: () => undefined,
+      }
+  }
+}
diff --git a/web/app/components/workflow/nodes/data-source/node.tsx b/web/app/components/workflow/nodes/data-source/node.tsx
index f97098e52f..6e6c565dc2 100644
--- a/web/app/components/workflow/nodes/data-source/node.tsx
+++ b/web/app/components/workflow/nodes/data-source/node.tsx
@@ -1,10 +1,36 @@
 import type { FC } from 'react'
 import { memo } from 'react'
-import type { DataSourceNodeType } from './types'
 import type { NodeProps } from '@/app/components/workflow/types'
-const Node: FC<NodeProps<DataSourceNodeType>> = () => {
+import { InstallPluginButton } from '@/app/components/workflow/nodes/_base/components/install-plugin-button'
+import { useNodePluginInstallation } from '@/app/components/workflow/hooks/use-node-plugin-installation'
+import type { DataSourceNodeType } from './types'
+
+const Node: FC<NodeProps<DataSourceNodeType>> = ({
+  data,
+}) => {
+  const {
+    isChecking,
+    isMissing,
+    uniqueIdentifier,
+    canInstall,
+    onInstallSuccess,
+  } = useNodePluginInstallation(data)
+
+  const showInstallButton = !isChecking && isMissing && canInstall && uniqueIdentifier
+
+  if (!showInstallButton)
+    return null
+
   return (
-    <div>
+    <div className='relative mb-1 px-3 py-1'>
+      <div className='absolute right-3 top-[-32px] z-20'>
+        <InstallPluginButton
+          size='small'
+          className='!font-medium !text-text-accent'
+          uniqueIdentifier={uniqueIdentifier!}
+          onSuccess={onInstallSuccess}
+        />
+      </div>
     </div>
   )
 }
diff --git a/web/app/components/workflow/nodes/data-source/types.ts b/web/app/components/workflow/nodes/data-source/types.ts
index da887244b8..cd22b305d1 100644
--- a/web/app/components/workflow/nodes/data-source/types.ts
+++ b/web/app/components/workflow/nodes/data-source/types.ts
@@ -30,6 +30,7 @@ export type DataSourceNodeType = CommonNodeType & {
   datasource_label: string
   datasource_parameters: ToolVarInputs
   datasource_configurations: Record<string, any>
+  plugin_unique_identifier?: string
 }
 
 export type CustomRunFormProps = {
diff --git a/web/app/components/workflow/nodes/tool/node.tsx b/web/app/components/workflow/nodes/tool/node.tsx
index 8cc3ec580d..466cbb577f 100644
--- a/web/app/components/workflow/nodes/tool/node.tsx
+++ b/web/app/components/workflow/nodes/tool/node.tsx
@@ -1,46 +1,68 @@
 import type { FC } from 'react'
 import React from 'react'
-import type { ToolNodeType } from './types'
 import type { NodeProps } from '@/app/components/workflow/types'
 import { FormTypeEnum } from '@/app/components/header/account-setting/model-provider-page/declarations'
+import { InstallPluginButton } from '@/app/components/workflow/nodes/_base/components/install-plugin-button'
+import { useNodePluginInstallation } from '@/app/components/workflow/hooks/use-node-plugin-installation'
+import type { ToolNodeType } from './types'
 
 const Node: FC<NodeProps<ToolNodeType>> = ({
   data,
 }) => {
   const { tool_configurations, paramSchemas } = data
   const toolConfigs = Object.keys(tool_configurations || {})
+  const {
+    isChecking,
+    isMissing,
+    uniqueIdentifier,
+    canInstall,
+    onInstallSuccess,
+  } = useNodePluginInstallation(data)
+  const showInstallButton = !isChecking && isMissing && canInstall && uniqueIdentifier
 
-  if (!toolConfigs.length)
+  const hasConfigs = toolConfigs.length > 0
+
+  if (!showInstallButton && !hasConfigs)
     return null
 
   return (
-    <div className='mb-1 px-3 py-1'>
-      <div className='space-y-0.5'>
-        {toolConfigs.map((key, index) => (
-          <div key={index} className='flex h-6 items-center justify-between space-x-1 rounded-md  bg-workflow-block-parma-bg px-1 text-xs font-normal text-text-secondary'>
-            <div title={key} className='max-w-[100px] shrink-0 truncate text-xs font-medium uppercase text-text-tertiary'>
-              {key}
+    <div className='relative mb-1 px-3 py-1'>
+      {showInstallButton && (
+        <div className='absolute right-3 top-[-32px] z-20'>
+          <InstallPluginButton
+            size='small'
+            className='!font-medium !text-text-accent'
+            uniqueIdentifier={uniqueIdentifier!}
+            onSuccess={onInstallSuccess}
+          />
+        </div>
+      )}
+      {hasConfigs && (
+        <div className='space-y-0.5'>
+          {toolConfigs.map((key, index) => (
+            <div key={index} className='flex h-6 items-center justify-between space-x-1 rounded-md  bg-workflow-block-parma-bg px-1 text-xs font-normal text-text-secondary'>
+              <div title={key} className='max-w-[100px] shrink-0 truncate text-xs font-medium uppercase text-text-tertiary'>
+                {key}
+              </div>
+              {typeof tool_configurations[key].value === 'string' && (
+                <div title={tool_configurations[key].value} className='w-0 shrink-0 grow truncate text-right text-xs font-normal text-text-secondary'>
+                  {paramSchemas?.find(i => i.name === key)?.type === FormTypeEnum.secretInput ? '********' : tool_configurations[key].value}
+                </div>
+              )}
+              {typeof tool_configurations[key].value === 'number' && (
+                <div title={Number.isNaN(tool_configurations[key].value) ? '' : tool_configurations[key].value} className='w-0 shrink-0 grow truncate text-right text-xs font-normal text-text-secondary'>
+                  {Number.isNaN(tool_configurations[key].value) ? '' : tool_configurations[key].value}
+                </div>
+              )}
+              {typeof tool_configurations[key] !== 'string' && tool_configurations[key]?.type === FormTypeEnum.modelSelector && (
+                <div title={tool_configurations[key].model} className='w-0 shrink-0 grow truncate text-right text-xs font-normal text-text-secondary'>
+                  {tool_configurations[key].model}
+                </div>
+              )}
             </div>
-            {typeof tool_configurations[key].value === 'string' && (
-              <div title={tool_configurations[key].value} className='w-0 shrink-0 grow truncate text-right text-xs font-normal text-text-secondary'>
-                {paramSchemas?.find(i => i.name === key)?.type === FormTypeEnum.secretInput ? '********' : tool_configurations[key].value}
-              </div>
-            )}
-            {typeof tool_configurations[key].value === 'number' && (
-              <div title={Number.isNaN(tool_configurations[key].value) ? '' : tool_configurations[key].value} className='w-0 shrink-0 grow truncate text-right text-xs font-normal text-text-secondary'>
-                {Number.isNaN(tool_configurations[key].value) ? '' : tool_configurations[key].value}
-              </div>
-            )}
-            {typeof tool_configurations[key] !== 'string' && tool_configurations[key]?.type === FormTypeEnum.modelSelector && (
-              <div title={tool_configurations[key].model} className='w-0 shrink-0 grow truncate text-right text-xs font-normal text-text-secondary'>
-                {tool_configurations[key].model}
-              </div>
-            )}
-          </div>
-
-        ))}
-
-      </div>
+          ))}
+        </div>
+      )}
     </div>
   )
 }
diff --git a/web/app/components/workflow/nodes/tool/types.ts b/web/app/components/workflow/nodes/tool/types.ts
index 12a4283cf6..6e6ef858dc 100644
--- a/web/app/components/workflow/nodes/tool/types.ts
+++ b/web/app/components/workflow/nodes/tool/types.ts
@@ -22,4 +22,5 @@ export type ToolNodeType = CommonNodeType & {
   params?: Record<string, any>
   plugin_id?: string
   provider_icon?: Collection['icon']
+  plugin_unique_identifier?: string
 }
diff --git a/web/app/components/workflow/nodes/trigger-plugin/node.tsx b/web/app/components/workflow/nodes/trigger-plugin/node.tsx
index 9be517e97d..bfd807f30e 100644
--- a/web/app/components/workflow/nodes/trigger-plugin/node.tsx
+++ b/web/app/components/workflow/nodes/trigger-plugin/node.tsx
@@ -3,6 +3,8 @@ import type { NodeProps } from '@/app/components/workflow/types'
 import type { FC } from 'react'
 import React, { useMemo } from 'react'
 import { useTranslation } from 'react-i18next'
+import { InstallPluginButton } from '@/app/components/workflow/nodes/_base/components/install-plugin-button'
+import { useNodePluginInstallation } from '@/app/components/workflow/hooks/use-node-plugin-installation'
 import type { PluginTriggerNodeType } from './types'
 import useConfig from './use-config'
 
@@ -42,6 +44,14 @@ const Node: FC<NodeProps<PluginTriggerNodeType>> = ({
   const { subscriptions } = useConfig(id, data)
   const { config = {}, subscription_id } = data
   const configKeys = Object.keys(config)
+  const {
+    isChecking,
+    isMissing,
+    uniqueIdentifier,
+    canInstall,
+    onInstallSuccess,
+  } = useNodePluginInstallation(data)
+  const showInstallButton = !isChecking && isMissing && canInstall && uniqueIdentifier
 
   const { t } = useTranslation()
 
@@ -50,7 +60,17 @@ const Node: FC<NodeProps<PluginTriggerNodeType>> = ({
   }, [subscription_id, subscriptions])
 
   return (
-    <div className="mb-1 px-3 py-1">
+    <div className="relative mb-1 px-3 py-1">
+      {showInstallButton && (
+        <div className="absolute right-3 top-[-32px] z-20">
+          <InstallPluginButton
+            size="small"
+            className="!font-medium !text-text-accent"
+            uniqueIdentifier={uniqueIdentifier!}
+            onSuccess={onInstallSuccess}
+          />
+        </div>
+      )}
       <div className="space-y-0.5">
         {!isValidSubscription && <NodeStatus status={NodeStatusEnum.warning} message={t('pluginTrigger.node.status.warning')} />}
         {isValidSubscription && configKeys.map((key, index) => (
diff --git a/web/app/components/workflow/nodes/trigger-plugin/types.ts b/web/app/components/workflow/nodes/trigger-plugin/types.ts
index 43268e9096..6dba97d795 100644
--- a/web/app/components/workflow/nodes/trigger-plugin/types.ts
+++ b/web/app/components/workflow/nodes/trigger-plugin/types.ts
@@ -16,6 +16,7 @@ export type PluginTriggerNodeType = CommonNodeType & {
   event_node_version?: string
   plugin_id?: string
   config?: Record<string, any>
+  plugin_unique_identifier?: string
 }
 
 // Use base types directly
diff --git a/web/app/components/workflow/types.ts b/web/app/components/workflow/types.ts
index a4e7002960..d126daa350 100644
--- a/web/app/components/workflow/types.ts
+++ b/web/app/components/workflow/types.ts
@@ -451,6 +451,7 @@ export type MoreInfo = {
 export type ToolWithProvider = Collection & {
   tools: Tool[]
   meta: PluginMeta
+  plugin_unique_identifier?: string
 }
 
 export type RAGRecommendedPlugins = {

From ff0f645e543cf60efccb7f4ecfd79654f8f61734 Mon Sep 17 00:00:00 2001
From: lyzno1 <yuanyouhuilyz@gmail.com>
Date: Thu, 30 Oct 2025 14:42:52 +0800
Subject: [PATCH 04/17] Fix plugin install detection for tool nodes

---
 .../components/install-plugin-button.tsx      | 24 +++++++++++++++----
 .../workflow/nodes/data-source/node.tsx       |  4 ++++
 .../components/workflow/nodes/tool/node.tsx   |  5 ++++
 .../workflow/nodes/trigger-plugin/node.tsx    |  5 ++++
 4 files changed, 34 insertions(+), 4 deletions(-)

diff --git a/web/app/components/workflow/nodes/_base/components/install-plugin-button.tsx b/web/app/components/workflow/nodes/_base/components/install-plugin-button.tsx
index 23119f0213..c387cb8630 100644
--- a/web/app/components/workflow/nodes/_base/components/install-plugin-button.tsx
+++ b/web/app/components/workflow/nodes/_base/components/install-plugin-button.tsx
@@ -7,15 +7,25 @@ import { useCheckInstalled, useInstallPackageFromMarketPlace } from '@/service/u
 
 type InstallPluginButtonProps = Omit<ComponentProps<typeof Button>, 'children' | 'loading'> & {
   uniqueIdentifier: string
+  extraIdentifiers?: string[]
   onSuccess?: () => void
 }
 
 export const InstallPluginButton = (props: InstallPluginButtonProps) => {
-  const { className, uniqueIdentifier, onSuccess, ...rest } = props
+  const {
+    className,
+    uniqueIdentifier,
+    extraIdentifiers = [],
+    onSuccess,
+    ...rest
+  } = props
   const { t } = useTranslation()
+  const identifiers = Array.from(new Set(
+    [uniqueIdentifier, ...extraIdentifiers].filter((item): item is string => Boolean(item)),
+  ))
   const manifest = useCheckInstalled({
-    pluginIds: [uniqueIdentifier],
-    enabled: !!uniqueIdentifier,
+    pluginIds: identifiers,
+    enabled: identifiers.length > 0,
   })
   const install = useInstallPackageFromMarketPlace()
   const isLoading = manifest.isLoading || install.isPending
@@ -31,7 +41,13 @@ export const InstallPluginButton = (props: InstallPluginButtonProps) => {
     })
   }
   if (!manifest.data) return null
-  if (manifest.data.plugins.some(plugin => plugin.id === uniqueIdentifier)) return null
+  const identifierSet = new Set(identifiers)
+  const isInstalled = manifest.data.plugins.some(plugin => (
+    identifierSet.has(plugin.id)
+    || (plugin.plugin_unique_identifier && identifierSet.has(plugin.plugin_unique_identifier))
+    || (plugin.plugin_id && identifierSet.has(plugin.plugin_id))
+  ))
+  if (isInstalled) return null
   return <Button
     variant={'secondary'}
     disabled={isLoading}
diff --git a/web/app/components/workflow/nodes/data-source/node.tsx b/web/app/components/workflow/nodes/data-source/node.tsx
index 6e6c565dc2..ada649270e 100644
--- a/web/app/components/workflow/nodes/data-source/node.tsx
+++ b/web/app/components/workflow/nodes/data-source/node.tsx
@@ -26,6 +26,10 @@ const Node: FC<NodeProps<DataSourceNodeType>> = ({
       <div className='absolute right-3 top-[-32px] z-20'>
         <InstallPluginButton
           size='small'
+          extraIdentifiers={[
+            data.plugin_id,
+            data.provider_name,
+          ].filter(Boolean) as string[]}
           className='!font-medium !text-text-accent'
           uniqueIdentifier={uniqueIdentifier!}
           onSuccess={onInstallSuccess}
diff --git a/web/app/components/workflow/nodes/tool/node.tsx b/web/app/components/workflow/nodes/tool/node.tsx
index 466cbb577f..0ebee2966c 100644
--- a/web/app/components/workflow/nodes/tool/node.tsx
+++ b/web/app/components/workflow/nodes/tool/node.tsx
@@ -32,6 +32,11 @@ const Node: FC<NodeProps<ToolNodeType>> = ({
           <InstallPluginButton
             size='small'
             className='!font-medium !text-text-accent'
+            extraIdentifiers={[
+              data.plugin_id,
+              data.provider_id,
+              data.provider_name,
+            ].filter(Boolean) as string[]}
             uniqueIdentifier={uniqueIdentifier!}
             onSuccess={onInstallSuccess}
           />
diff --git a/web/app/components/workflow/nodes/trigger-plugin/node.tsx b/web/app/components/workflow/nodes/trigger-plugin/node.tsx
index bfd807f30e..53694c8759 100644
--- a/web/app/components/workflow/nodes/trigger-plugin/node.tsx
+++ b/web/app/components/workflow/nodes/trigger-plugin/node.tsx
@@ -65,6 +65,11 @@ const Node: FC<NodeProps<PluginTriggerNodeType>> = ({
         <div className="absolute right-3 top-[-32px] z-20">
           <InstallPluginButton
             size="small"
+            extraIdentifiers={[
+              data.plugin_id,
+              data.provider_id,
+              data.provider_name,
+            ].filter(Boolean) as string[]}
             className="!font-medium !text-text-accent"
             uniqueIdentifier={uniqueIdentifier!}
             onSuccess={onInstallSuccess}

From 8a48db6d0dc6e92b06a538c6b7b27e50c372692e Mon Sep 17 00:00:00 2001
From: lyzno1 <yuanyouhuilyz@gmail.com>
Date: Thu, 30 Oct 2025 15:04:14 +0800
Subject: [PATCH 05/17] Improve workflow tool install flow

---
 .../workflow/block-selector/tool-picker.tsx   | 16 +++++-
 .../components/install-plugin-button.tsx      | 55 +++++++++++++++++--
 2 files changed, 64 insertions(+), 7 deletions(-)

diff --git a/web/app/components/workflow/block-selector/tool-picker.tsx b/web/app/components/workflow/block-selector/tool-picker.tsx
index c438e45042..660cdad71e 100644
--- a/web/app/components/workflow/block-selector/tool-picker.tsx
+++ b/web/app/components/workflow/block-selector/tool-picker.tsx
@@ -23,7 +23,16 @@ import {
 } from '@/service/tools'
 import type { CustomCollectionBackend } from '@/app/components/tools/types'
 import Toast from '@/app/components/base/toast'
-import { useAllBuiltInTools, useAllCustomTools, useAllMCPTools, useAllWorkflowTools, useInvalidateAllBuiltInTools, useInvalidateAllCustomTools } from '@/service/use-tools'
+import {
+  useAllBuiltInTools,
+  useAllCustomTools,
+  useAllMCPTools,
+  useAllWorkflowTools,
+  useInvalidateAllBuiltInTools,
+  useInvalidateAllCustomTools,
+  useInvalidateAllMCPTools,
+  useInvalidateAllWorkflowTools,
+} from '@/service/use-tools'
 import { useFeaturedToolsRecommendations } from '@/service/use-plugins'
 import { useGlobalPublicStore } from '@/context/global-public-context'
 import cn from '@/utils/classnames'
@@ -70,6 +79,8 @@ const ToolPicker: FC<Props> = ({
   const { data: workflowTools } = useAllWorkflowTools()
   const { data: mcpTools } = useAllMCPTools()
   const invalidateBuiltInTools = useInvalidateAllBuiltInTools()
+  const invalidateWorkflowTools = useInvalidateAllWorkflowTools()
+  const invalidateMcpTools = useInvalidateAllMCPTools()
 
   const {
     plugins: featuredPlugins = [],
@@ -193,6 +204,9 @@ const ToolPicker: FC<Props> = ({
             showFeatured={scope === 'all' && enable_marketplace}
             onFeaturedInstallSuccess={async () => {
               invalidateBuiltInTools()
+              invalidateCustomTools()
+              invalidateWorkflowTools()
+              invalidateMcpTools()
             }}
           />
         </div>
diff --git a/web/app/components/workflow/nodes/_base/components/install-plugin-button.tsx b/web/app/components/workflow/nodes/_base/components/install-plugin-button.tsx
index c387cb8630..b0d878d53d 100644
--- a/web/app/components/workflow/nodes/_base/components/install-plugin-button.tsx
+++ b/web/app/components/workflow/nodes/_base/components/install-plugin-button.tsx
@@ -1,8 +1,11 @@
 import Button from '@/app/components/base/button'
 import { RiInstallLine, RiLoader2Line } from '@remixicon/react'
 import type { ComponentProps, MouseEventHandler } from 'react'
+import { useState } from 'react'
 import classNames from '@/utils/classnames'
 import { useTranslation } from 'react-i18next'
+import checkTaskStatus from '@/app/components/plugins/install-plugin/base/check-task-status'
+import { TaskStatus } from '@/app/components/plugins/types'
 import { useCheckInstalled, useInstallPackageFromMarketPlace } from '@/service/use-plugins'
 
 type InstallPluginButtonProps = Omit<ComponentProps<typeof Button>, 'children' | 'loading'> & {
@@ -28,15 +31,55 @@ export const InstallPluginButton = (props: InstallPluginButtonProps) => {
     enabled: identifiers.length > 0,
   })
   const install = useInstallPackageFromMarketPlace()
-  const isLoading = manifest.isLoading || install.isPending
-  // await for refetch to get the new installed plugin, when manifest refetch, this component will unmount
-  || install.isSuccess
+  const [isTracking, setIsTracking] = useState(false)
+  const isLoading = manifest.isLoading || install.isPending || isTracking
   const handleInstall: MouseEventHandler = (e) => {
     e.stopPropagation()
+    if (isLoading)
+      return
+    setIsTracking(true)
     install.mutate(uniqueIdentifier, {
-      onSuccess: async () => {
-        await manifest.refetch()
-        onSuccess?.()
+      onSuccess: async (response) => {
+        const finish = async () => {
+          await manifest.refetch()
+          onSuccess?.()
+          setIsTracking(false)
+          install.reset()
+        }
+
+        if (!response) {
+          await finish()
+          return
+        }
+
+        if (response.all_installed) {
+          await finish()
+          return
+        }
+
+        const { check } = checkTaskStatus()
+        try {
+          const { status } = await check({
+            taskId: response.task_id,
+            pluginUniqueIdentifier: uniqueIdentifier,
+          })
+
+          if (status === TaskStatus.failed) {
+            setIsTracking(false)
+            install.reset()
+            return
+          }
+
+          await finish()
+        }
+        catch {
+          setIsTracking(false)
+          install.reset()
+        }
+      },
+      onError: () => {
+        setIsTracking(false)
+        install.reset()
       },
     })
   }

From 4d37d618514a9e7f25e73733ddcfb31cbc99dcda Mon Sep 17 00:00:00 2001
From: lyzno1 <yuanyouhuilyz@gmail.com>
Date: Thu, 30 Oct 2025 15:33:32 +0800
Subject: [PATCH 06/17] feat: dim node

---
 .../hooks/use-node-plugin-installation.ts       | 10 ++++++++++
 .../workflow/nodes/data-source/node.tsx         | 17 ++++++++++++++++-
 web/app/components/workflow/nodes/tool/node.tsx | 17 ++++++++++++++++-
 .../workflow/nodes/trigger-plugin/node.tsx      | 16 +++++++++++++++-
 4 files changed, 57 insertions(+), 3 deletions(-)

diff --git a/web/app/components/workflow/hooks/use-node-plugin-installation.ts b/web/app/components/workflow/hooks/use-node-plugin-installation.ts
index d82ed0ac61..96e3919e67 100644
--- a/web/app/components/workflow/hooks/use-node-plugin-installation.ts
+++ b/web/app/components/workflow/hooks/use-node-plugin-installation.ts
@@ -99,12 +99,15 @@ const useToolInstallation = (data: ToolNodeType): InstallationState => {
       invalidateTools()
   }, [invalidateTools])
 
+  const shouldDim = (!!collectionInfo && !isResolved) || (isResolved && !matchedCollection)
+
   return {
     isChecking: !!collectionInfo && !isResolved,
     isMissing: isResolved && !matchedCollection,
     uniqueIdentifier,
     canInstall,
     onInstallSuccess,
+    shouldDim,
   }
 }
 
@@ -138,12 +141,15 @@ const useTriggerInstallation = (data: PluginTriggerNodeType): InstallationState
     invalidateTriggers()
   }, [invalidateTriggers])
 
+  const shouldDim = isLoading || (!isLoading && !!triggerProviders && !matchedProvider)
+
   return {
     isChecking: isLoading,
     isMissing: !isLoading && !!triggerProviders && !matchedProvider,
     uniqueIdentifier,
     canInstall,
     onInstallSuccess,
+    shouldDim,
   }
 }
 
@@ -175,12 +181,15 @@ const useDataSourceInstallation = (data: DataSourceNodeType): InstallationState
 
   const hasLoadedList = dataSourceList !== undefined
 
+  const shouldDim = !hasLoadedList || (hasLoadedList && !matchedPlugin)
+
   return {
     isChecking: !hasLoadedList,
     isMissing: hasLoadedList && !matchedPlugin,
     uniqueIdentifier,
     canInstall,
     onInstallSuccess,
+    shouldDim,
   }
 }
 
@@ -203,6 +212,7 @@ export const useNodePluginInstallation = (data: CommonNodeType): InstallationSta
         uniqueIdentifier: undefined,
         canInstall: false,
         onInstallSuccess: () => undefined,
+        shouldDim: false,
       }
   }
 }
diff --git a/web/app/components/workflow/nodes/data-source/node.tsx b/web/app/components/workflow/nodes/data-source/node.tsx
index ada649270e..354758074b 100644
--- a/web/app/components/workflow/nodes/data-source/node.tsx
+++ b/web/app/components/workflow/nodes/data-source/node.tsx
@@ -1,11 +1,13 @@
 import type { FC } from 'react'
-import { memo } from 'react'
+import { memo, useEffect } from 'react'
 import type { NodeProps } from '@/app/components/workflow/types'
 import { InstallPluginButton } from '@/app/components/workflow/nodes/_base/components/install-plugin-button'
 import { useNodePluginInstallation } from '@/app/components/workflow/hooks/use-node-plugin-installation'
+import { useNodeDataUpdate } from '@/app/components/workflow/hooks/use-node-data-update'
 import type { DataSourceNodeType } from './types'
 
 const Node: FC<NodeProps<DataSourceNodeType>> = ({
+  id,
   data,
 }) => {
   const {
@@ -14,7 +16,20 @@ const Node: FC<NodeProps<DataSourceNodeType>> = ({
     uniqueIdentifier,
     canInstall,
     onInstallSuccess,
+    shouldDim,
   } = useNodePluginInstallation(data)
+  const { handleNodeDataUpdate } = useNodeDataUpdate()
+
+  useEffect(() => {
+    if (data._dimmed === shouldDim)
+      return
+    handleNodeDataUpdate({
+      id,
+      data: {
+        _dimmed: shouldDim,
+      },
+    })
+  }, [data._dimmed, handleNodeDataUpdate, id, shouldDim])
 
   const showInstallButton = !isChecking && isMissing && canInstall && uniqueIdentifier
 
diff --git a/web/app/components/workflow/nodes/tool/node.tsx b/web/app/components/workflow/nodes/tool/node.tsx
index 0ebee2966c..584d6ff3de 100644
--- a/web/app/components/workflow/nodes/tool/node.tsx
+++ b/web/app/components/workflow/nodes/tool/node.tsx
@@ -1,12 +1,14 @@
 import type { FC } from 'react'
-import React from 'react'
+import React, { useEffect } from 'react'
 import type { NodeProps } from '@/app/components/workflow/types'
 import { FormTypeEnum } from '@/app/components/header/account-setting/model-provider-page/declarations'
 import { InstallPluginButton } from '@/app/components/workflow/nodes/_base/components/install-plugin-button'
 import { useNodePluginInstallation } from '@/app/components/workflow/hooks/use-node-plugin-installation'
+import { useNodeDataUpdate } from '@/app/components/workflow/hooks/use-node-data-update'
 import type { ToolNodeType } from './types'
 
 const Node: FC<NodeProps<ToolNodeType>> = ({
+  id,
   data,
 }) => {
   const { tool_configurations, paramSchemas } = data
@@ -17,8 +19,21 @@ const Node: FC<NodeProps<ToolNodeType>> = ({
     uniqueIdentifier,
     canInstall,
     onInstallSuccess,
+    shouldDim,
   } = useNodePluginInstallation(data)
   const showInstallButton = !isChecking && isMissing && canInstall && uniqueIdentifier
+  const { handleNodeDataUpdate } = useNodeDataUpdate()
+
+  useEffect(() => {
+    if (data._dimmed === shouldDim)
+      return
+    handleNodeDataUpdate({
+      id,
+      data: {
+        _dimmed: shouldDim,
+      },
+    })
+  }, [data._dimmed, handleNodeDataUpdate, id, shouldDim])
 
   const hasConfigs = toolConfigs.length > 0
 
diff --git a/web/app/components/workflow/nodes/trigger-plugin/node.tsx b/web/app/components/workflow/nodes/trigger-plugin/node.tsx
index 53694c8759..0aa7f656e2 100644
--- a/web/app/components/workflow/nodes/trigger-plugin/node.tsx
+++ b/web/app/components/workflow/nodes/trigger-plugin/node.tsx
@@ -1,10 +1,11 @@
 import NodeStatus, { NodeStatusEnum } from '@/app/components/base/node-status'
 import type { NodeProps } from '@/app/components/workflow/types'
 import type { FC } from 'react'
-import React, { useMemo } from 'react'
+import React, { useEffect, useMemo } from 'react'
 import { useTranslation } from 'react-i18next'
 import { InstallPluginButton } from '@/app/components/workflow/nodes/_base/components/install-plugin-button'
 import { useNodePluginInstallation } from '@/app/components/workflow/hooks/use-node-plugin-installation'
+import { useNodeDataUpdate } from '@/app/components/workflow/hooks/use-node-data-update'
 import type { PluginTriggerNodeType } from './types'
 import useConfig from './use-config'
 
@@ -50,9 +51,22 @@ const Node: FC<NodeProps<PluginTriggerNodeType>> = ({
     uniqueIdentifier,
     canInstall,
     onInstallSuccess,
+    shouldDim,
   } = useNodePluginInstallation(data)
+  const { handleNodeDataUpdate } = useNodeDataUpdate()
   const showInstallButton = !isChecking && isMissing && canInstall && uniqueIdentifier
 
+  useEffect(() => {
+    if (data._dimmed === shouldDim)
+      return
+    handleNodeDataUpdate({
+      id,
+      data: {
+        _dimmed: shouldDim,
+      },
+    })
+  }, [data._dimmed, handleNodeDataUpdate, id, shouldDim])
+
   const { t } = useTranslation()
 
   const isValidSubscription = useMemo(() => {

From b3866288e09ffadc885ccf78186a7b75556e8d63 Mon Sep 17 00:00:00 2001
From: Yeuoly <admin@srmxy.cn>
Date: Thu, 30 Oct 2025 15:54:25 +0800
Subject: [PATCH 07/17] fix: docker compose

---
 docker/.env.example        | 5 +++++
 docker/docker-compose.yaml | 1 +
 2 files changed, 6 insertions(+)

diff --git a/docker/.env.example b/docker/.env.example
index fb815ea160..74a436b082 100644
--- a/docker/.env.example
+++ b/docker/.env.example
@@ -24,6 +24,11 @@ CONSOLE_WEB_URL=
 # Example: https://api.dify.ai
 SERVICE_API_URL=
 
+# Trigger external URL
+# used to display trigger endpoint API Base URL to the front-end.
+# Example: https://api.dify.ai
+TRIGGER_URL=
+
 # WebApp API backend Url,
 # used to declare the back-end URL for the front-end API.
 # If empty, it is the same domain.
diff --git a/docker/docker-compose.yaml b/docker/docker-compose.yaml
index 81f2a5e625..6d1f2e8352 100644
--- a/docker/docker-compose.yaml
+++ b/docker/docker-compose.yaml
@@ -8,6 +8,7 @@ x-shared-env: &shared-api-worker-env
   CONSOLE_API_URL: ${CONSOLE_API_URL:-}
   CONSOLE_WEB_URL: ${CONSOLE_WEB_URL:-}
   SERVICE_API_URL: ${SERVICE_API_URL:-}
+  TRIGGER_URL: ${TRIGGER_URL:-}
   APP_API_URL: ${APP_API_URL:-}
   APP_WEB_URL: ${APP_WEB_URL:-}
   FILES_URL: ${FILES_URL:-}

From defea962f6a9912fe21572c6001b36fb89361302 Mon Sep 17 00:00:00 2001
From: Yeuoly <45712896+Yeuoly@users.noreply.github.com>
Date: Thu, 30 Oct 2025 15:58:41 +0800
Subject: [PATCH 08/17] Update
 api/controllers/console/workspace/trigger_providers.py

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
---
 api/controllers/console/workspace/trigger_providers.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/api/controllers/console/workspace/trigger_providers.py b/api/controllers/console/workspace/trigger_providers.py
index 3f398fbbc3..309dbde1b6 100644
--- a/api/controllers/console/workspace/trigger_providers.py
+++ b/api/controllers/console/workspace/trigger_providers.py
@@ -427,7 +427,7 @@ class TriggerOAuthCallbackApi(Resource):
         expires_at = credentials_response.expires_at
 
         if not credentials:
-            raise Exception("Failed to get OAuth credentials")
+            raise ValueError("Failed to get OAuth credentials from the provider.")
 
         # Update subscription builder
         TriggerSubscriptionBuilderService.update_trigger_subscription_builder(

From 49ebbd05b5022d799a9ef93343cb82a890583148 Mon Sep 17 00:00:00 2001
From: Yeuoly <45712896+Yeuoly@users.noreply.github.com>
Date: Thu, 30 Oct 2025 15:58:58 +0800
Subject: [PATCH 09/17] Update api/controllers/console/app/workflow.py

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
---
 api/controllers/console/app/workflow.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/api/controllers/console/app/workflow.py b/api/controllers/console/app/workflow.py
index 62d50303cd..1b1babb235 100644
--- a/api/controllers/console/app/workflow.py
+++ b/api/controllers/console/app/workflow.py
@@ -1144,7 +1144,7 @@ class DraftWorkflowTriggerRunAllApi(Resource):
         except InvokeRateLimitError as ex:
             raise InvokeRateLimitHttpError(ex.description)
         except Exception:
-            logger.exception("Error running draft workflow trigger webhook run")
+            logger.exception("Error running draft workflow trigger run-all")
             return jsonable_encoder(
                 {
                     "status": "error",

From 447127cee4f7aff575679dc05555a17bf6459b2d Mon Sep 17 00:00:00 2001
From: Yeuoly <45712896+Yeuoly@users.noreply.github.com>
Date: Thu, 30 Oct 2025 15:59:37 +0800
Subject: [PATCH 10/17] Update api/controllers/console/app/workflow.py

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
---
 api/controllers/console/app/workflow.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/api/controllers/console/app/workflow.py b/api/controllers/console/app/workflow.py
index 1b1babb235..3b2c769960 100644
--- a/api/controllers/console/app/workflow.py
+++ b/api/controllers/console/app/workflow.py
@@ -1072,7 +1072,7 @@ class DraftWorkflowTriggerNodeApi(Resource):
             return jsonable_encoder(node_execution)
         except Exception as e:
             logger.exception("Error running draft workflow trigger node")
-            return jsonable_encoder({"status": "error", "error": str(e)}), 500
+            return jsonable_encoder({"status": "error", "error": "An unexpected error occurred while running the node."}), 500
 
 
 @console_ns.route("/apps/<uuid:app_id>/workflows/draft/trigger/run-all")

From 08e61d76d63f3db191a88df4f6a656027f6d9f45 Mon Sep 17 00:00:00 2001
From: Yeuoly <45712896+Yeuoly@users.noreply.github.com>
Date: Thu, 30 Oct 2025 16:00:24 +0800
Subject: [PATCH 11/17] Potential fix for code scanning alert no. 244:
 Information exposure through an exception

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
---
 api/services/trigger/trigger_subscription_builder_service.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/api/services/trigger/trigger_subscription_builder_service.py b/api/services/trigger/trigger_subscription_builder_service.py
index 3b61d01648..d7324aec3e 100644
--- a/api/services/trigger/trigger_subscription_builder_service.py
+++ b/api/services/trigger/trigger_subscription_builder_service.py
@@ -473,7 +473,8 @@ class TriggerSubscriptionBuilderService:
             )
             return response
         except Exception as e:
-            error_response = Response(status=500, response=str(e))
+            logger.exception("Error during validation endpoint dispatch for endpoint_id=%s", endpoint_id)
+            error_response = Response(status=500, response="An internal error has occurred.")
             cls.append_log(endpoint_id=endpoint_id, request=request, response=error_response)
             return error_response
 

From 2dbfd9ea5affdbbf45883e9f6121623369e6b259 Mon Sep 17 00:00:00 2001
From: Yeuoly <45712896+Yeuoly@users.noreply.github.com>
Date: Thu, 30 Oct 2025 16:00:44 +0800
Subject: [PATCH 12/17] Potential fix for code scanning alert no. 241: Use of a
 broken or weak cryptographic hashing algorithm on sensitive data

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
---
 api/core/trigger/debug/event_bus.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/api/core/trigger/debug/event_bus.py b/api/core/trigger/debug/event_bus.py
index e5f2ac2c00..9d10e1a0e0 100644
--- a/api/core/trigger/debug/event_bus.py
+++ b/api/core/trigger/debug/event_bus.py
@@ -107,7 +107,7 @@ class TriggerDebugEventBus:
         Returns:
             Event object if available, None otherwise
         """
-        address_id: str = hashlib.sha1(f"{user_id}|{app_id}|{node_id}".encode()).hexdigest()
+        address_id: str = hashlib.sha256(f"{user_id}|{app_id}|{node_id}".encode()).hexdigest()
         address: str = f"trigger_debug_inbox:{tenant_id}:{address_id}"
 
         try:

From 8bc5035624017e1f611de2ce0e35c8d2ae6d18b1 Mon Sep 17 00:00:00 2001
From: Yeuoly <45712896+Yeuoly@users.noreply.github.com>
Date: Thu, 30 Oct 2025 16:00:57 +0800
Subject: [PATCH 13/17] Potential fix for code scanning alert no. 211:
 Information exposure through an exception

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
---
 api/controllers/trigger/webhook.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/api/controllers/trigger/webhook.py b/api/controllers/trigger/webhook.py
index 0eb42e8f0d..cec5c3d8ae 100644
--- a/api/controllers/trigger/webhook.py
+++ b/api/controllers/trigger/webhook.py
@@ -102,4 +102,4 @@ def handle_webhook_debug(webhook_id: str):
         raise
     except Exception as e:
         logger.exception("Webhook debug processing failed for %s", webhook_id)
-        return jsonify({"error": "Internal server error", "message": str(e)}), 500
+        return jsonify({"error": "Internal server error", "message": "An internal error has occurred."}), 500

From 6260a1a28c5bee861fb8d9d40a85287aa8ec3d85 Mon Sep 17 00:00:00 2001
From: Yeuoly <admin@srmxy.cn>
Date: Thu, 30 Oct 2025 16:09:39 +0800
Subject: [PATCH 14/17] fix: cycle imports

---
 api/controllers/console/app/workflow.py       |  4 +-
 .../console/workspace/trigger_providers.py    |  6 +-
 api/services/trigger/schedule_service.py      | 10 ++-
 api/services/trigger/trigger_service.py       | 66 +----------------
 .../trigger_subscription_operator_service.py  | 70 +++++++++++++++++++
 api/tasks/trigger_processing_tasks.py         |  6 +-
 6 files changed, 88 insertions(+), 74 deletions(-)
 create mode 100644 api/services/trigger/trigger_subscription_operator_service.py

diff --git a/api/controllers/console/app/workflow.py b/api/controllers/console/app/workflow.py
index 3b2c769960..a6e2d64b19 100644
--- a/api/controllers/console/app/workflow.py
+++ b/api/controllers/console/app/workflow.py
@@ -1072,7 +1072,9 @@ class DraftWorkflowTriggerNodeApi(Resource):
             return jsonable_encoder(node_execution)
         except Exception as e:
             logger.exception("Error running draft workflow trigger node")
-            return jsonable_encoder({"status": "error", "error": "An unexpected error occurred while running the node."}), 500
+            return jsonable_encoder(
+                {"status": "error", "error": "An unexpected error occurred while running the node."}
+            ), 500
 
 
 @console_ns.route("/apps/<uuid:app_id>/workflows/draft/trigger/run-all")
diff --git a/api/controllers/console/workspace/trigger_providers.py b/api/controllers/console/workspace/trigger_providers.py
index 309dbde1b6..bbbbe12fb0 100644
--- a/api/controllers/console/workspace/trigger_providers.py
+++ b/api/controllers/console/workspace/trigger_providers.py
@@ -20,8 +20,8 @@ from models.account import Account
 from models.provider_ids import TriggerProviderID
 from services.plugin.oauth_service import OAuthProxyService
 from services.trigger.trigger_provider_service import TriggerProviderService
-from services.trigger.trigger_service import TriggerService
 from services.trigger.trigger_subscription_builder_service import TriggerSubscriptionBuilderService
+from services.trigger.trigger_subscription_operator_service import TriggerSubscriptionOperatorService
 
 logger = logging.getLogger(__name__)
 
@@ -265,7 +265,7 @@ class TriggerSubscriptionDeleteApi(Resource):
     @setup_required
     @login_required
     @account_initialization_required
-    def post(self, subscription_id):
+    def post(self, subscription_id: str):
         """Delete a subscription instance"""
         user = current_user
         assert isinstance(user, Account)
@@ -282,7 +282,7 @@ class TriggerSubscriptionDeleteApi(Resource):
                     subscription_id=subscription_id,
                 )
                 # Delete plugin triggers
-                TriggerService.delete_plugin_trigger_by_subscription(
+                TriggerSubscriptionOperatorService.delete_plugin_trigger_by_subscription(
                     session=session,
                     tenant_id=user.current_tenant_id,
                     subscription_id=subscription_id,
diff --git a/api/services/trigger/schedule_service.py b/api/services/trigger/schedule_service.py
index b0f39bf929..97f2def0a1 100644
--- a/api/services/trigger/schedule_service.py
+++ b/api/services/trigger/schedule_service.py
@@ -14,6 +14,7 @@ from libs.schedule_utils import calculate_next_run_at, convert_12h_to_24h
 from models.account import Account, TenantAccountJoin
 from models.trigger import WorkflowSchedulePlan
 from models.workflow import Workflow
+from services.errors.account import AccountNotFoundError
 
 logger = logging.getLogger(__name__)
 
@@ -124,7 +125,7 @@ class ScheduleService:
         session.flush()
 
     @staticmethod
-    def get_tenant_owner(session: Session, tenant_id: str) -> Optional[Account]:
+    def get_tenant_owner(session: Session, tenant_id: str) -> Account:
         """
         Returns an account to execute scheduled workflows on behalf of the tenant.
         Prioritizes owner over admin to ensure proper authorization hierarchy.
@@ -144,7 +145,12 @@ class ScheduleService:
             ).scalar_one_or_none()
 
         if result:
-            return session.get(Account, result.account_id)
+            account = session.get(Account, result.account_id)
+            if not account:
+                raise AccountNotFoundError(f"Account not found: {result.account_id}")
+            return account
+        else:
+            raise AccountNotFoundError(f"Account not found for tenant: {tenant_id}")
 
     @staticmethod
     def update_next_run_at(
diff --git a/api/services/trigger/trigger_service.py b/api/services/trigger/trigger_service.py
index 8a04a3c642..0255e42546 100644
--- a/api/services/trigger/trigger_service.py
+++ b/api/services/trigger/trigger_service.py
@@ -6,7 +6,7 @@ from typing import Any
 
 from flask import Request, Response
 from pydantic import BaseModel
-from sqlalchemy import and_, select
+from sqlalchemy import select
 from sqlalchemy.orm import Session
 
 from core.plugin.entities.plugin_daemon import CredentialType
@@ -22,7 +22,7 @@ from extensions.ext_database import db
 from extensions.ext_redis import redis_client
 from models.model import App
 from models.provider_ids import TriggerProviderID
-from models.trigger import AppTrigger, AppTriggerStatus, TriggerSubscription, WorkflowPluginTrigger
+from models.trigger import TriggerSubscription, WorkflowPluginTrigger
 from models.workflow import Workflow
 from services.trigger.trigger_provider_service import TriggerProviderService
 from services.trigger.trigger_request_service import TriggerHttpRequestCachingService
@@ -148,68 +148,6 @@ class TriggerService:
             )
         return dispatch_response.response
 
-    @classmethod
-    def get_subscriber_triggers(
-        cls, tenant_id: str, subscription_id: str, event_name: str
-    ) -> list[WorkflowPluginTrigger]:
-        """
-        Get WorkflowPluginTriggers for a subscription and trigger.
-
-        Args:
-            tenant_id: Tenant ID
-            subscription_id: Subscription ID
-            event_name: Event name
-        """
-        with Session(db.engine, expire_on_commit=False) as session:
-            subscribers = session.scalars(
-                select(WorkflowPluginTrigger)
-                .join(
-                    AppTrigger,
-                    and_(
-                        AppTrigger.tenant_id == WorkflowPluginTrigger.tenant_id,
-                        AppTrigger.app_id == WorkflowPluginTrigger.app_id,
-                        AppTrigger.node_id == WorkflowPluginTrigger.node_id,
-                    ),
-                )
-                .where(
-                    WorkflowPluginTrigger.tenant_id == tenant_id,
-                    WorkflowPluginTrigger.subscription_id == subscription_id,
-                    WorkflowPluginTrigger.event_name == event_name,
-                    AppTrigger.status == AppTriggerStatus.ENABLED,
-                )
-            ).all()
-            return list(subscribers)
-
-    @classmethod
-    def delete_plugin_trigger_by_subscription(
-        cls,
-        session: Session,
-        tenant_id: str,
-        subscription_id: str,
-    ) -> None:
-        """Delete a plugin trigger by tenant_id and subscription_id within an existing session
-
-        Args:
-            session: Database session
-            tenant_id: The tenant ID
-            subscription_id: The subscription ID
-
-        Raises:
-            NotFound: If plugin trigger not found
-        """
-        # Find plugin trigger using indexed columns
-        plugin_trigger = session.scalar(
-            select(WorkflowPluginTrigger).where(
-                WorkflowPluginTrigger.tenant_id == tenant_id,
-                WorkflowPluginTrigger.subscription_id == subscription_id,
-            )
-        )
-
-        if not plugin_trigger:
-            return
-
-        session.delete(plugin_trigger)
-
     @classmethod
     def sync_plugin_trigger_relationships(cls, app: App, workflow: Workflow):
         """
diff --git a/api/services/trigger/trigger_subscription_operator_service.py b/api/services/trigger/trigger_subscription_operator_service.py
new file mode 100644
index 0000000000..5d7785549e
--- /dev/null
+++ b/api/services/trigger/trigger_subscription_operator_service.py
@@ -0,0 +1,70 @@
+from sqlalchemy import and_, select
+from sqlalchemy.orm import Session
+
+from extensions.ext_database import db
+from models.enums import AppTriggerStatus
+from models.trigger import AppTrigger, WorkflowPluginTrigger
+
+
+class TriggerSubscriptionOperatorService:
+    @classmethod
+    def get_subscriber_triggers(
+        cls, tenant_id: str, subscription_id: str, event_name: str
+    ) -> list[WorkflowPluginTrigger]:
+        """
+        Get WorkflowPluginTriggers for a subscription and trigger.
+
+        Args:
+            tenant_id: Tenant ID
+            subscription_id: Subscription ID
+            event_name: Event name
+        """
+        with Session(db.engine, expire_on_commit=False) as session:
+            subscribers = session.scalars(
+                select(WorkflowPluginTrigger)
+                .join(
+                    AppTrigger,
+                    and_(
+                        AppTrigger.tenant_id == WorkflowPluginTrigger.tenant_id,
+                        AppTrigger.app_id == WorkflowPluginTrigger.app_id,
+                        AppTrigger.node_id == WorkflowPluginTrigger.node_id,
+                    ),
+                )
+                .where(
+                    WorkflowPluginTrigger.tenant_id == tenant_id,
+                    WorkflowPluginTrigger.subscription_id == subscription_id,
+                    WorkflowPluginTrigger.event_name == event_name,
+                    AppTrigger.status == AppTriggerStatus.ENABLED,
+                )
+            ).all()
+            return list(subscribers)
+
+    @classmethod
+    def delete_plugin_trigger_by_subscription(
+        cls,
+        session: Session,
+        tenant_id: str,
+        subscription_id: str,
+    ) -> None:
+        """Delete a plugin trigger by tenant_id and subscription_id within an existing session
+
+        Args:
+            session: Database session
+            tenant_id: The tenant ID
+            subscription_id: The subscription ID
+
+        Raises:
+            NotFound: If plugin trigger not found
+        """
+        # Find plugin trigger using indexed columns
+        plugin_trigger = session.scalar(
+            select(WorkflowPluginTrigger).where(
+                WorkflowPluginTrigger.tenant_id == tenant_id,
+                WorkflowPluginTrigger.subscription_id == subscription_id,
+            )
+        )
+
+        if not plugin_trigger:
+            return
+
+        session.delete(plugin_trigger)
diff --git a/api/tasks/trigger_processing_tasks.py b/api/tasks/trigger_processing_tasks.py
index fca7e47c95..75c36ef677 100644
--- a/api/tasks/trigger_processing_tasks.py
+++ b/api/tasks/trigger_processing_tasks.py
@@ -32,6 +32,7 @@ from services.async_workflow_service import AsyncWorkflowService
 from services.end_user_service import EndUserService
 from services.trigger.trigger_provider_service import TriggerProviderService
 from services.trigger.trigger_request_service import TriggerHttpRequestCachingService
+from services.trigger.trigger_subscription_operator_service import TriggerSubscriptionOperatorService
 from services.workflow.entities import PluginTriggerData, PluginTriggerDispatchData, PluginTriggerMetadata
 
 logger = logging.getLogger(__name__)
@@ -121,10 +122,7 @@ def dispatch_triggered_workflow(
     request = TriggerHttpRequestCachingService.get_request(request_id)
     payload = TriggerHttpRequestCachingService.get_payload(request_id)
 
-    from services.trigger.trigger_service import TriggerService
-    # FIXME: we should avoid import modules inside methods
-
-    subscribers: list[WorkflowPluginTrigger] = TriggerService.get_subscriber_triggers(
+    subscribers: list[WorkflowPluginTrigger] = TriggerSubscriptionOperatorService.get_subscriber_triggers(
         tenant_id=subscription.tenant_id, subscription_id=subscription.id, event_name=event_name
     )
     if not subscribers:

From dd089b1b21cc14f618aa7d9395aea7b54955bbba Mon Sep 17 00:00:00 2001
From: Yeuoly <admin@srmxy.cn>
Date: Thu, 30 Oct 2025 16:10:54 +0800
Subject: [PATCH 15/17] fix: coding style

---
 api/agent_skills/coding_style.md | 1 -
 1 file changed, 1 deletion(-)

diff --git a/api/agent_skills/coding_style.md b/api/agent_skills/coding_style.md
index 26be5990aa..c2f161fa22 100644
--- a/api/agent_skills/coding_style.md
+++ b/api/agent_skills/coding_style.md
@@ -19,7 +19,6 @@
 - Prefer simple functions over classes for lightweight helpers.
 - Keep files below 800 lines; split when necessary.
 - Keep code readable—no clever hacks.
-- Never use type annotations.
 - Never use `print`; log with `logger = logging.getLogger(__name__)`.
 
 ## Guiding Principles

From 501698d844c47b56788e74c72f0f4e29518a7d7b Mon Sep 17 00:00:00 2001
From: Yeuoly <45712896+Yeuoly@users.noreply.github.com>
Date: Thu, 30 Oct 2025 16:11:32 +0800
Subject: [PATCH 16/17] Potential fix for code scanning alert no. 243:
 Information exposure through an exception

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
---
 api/controllers/trigger/trigger.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/api/controllers/trigger/trigger.py b/api/controllers/trigger/trigger.py
index 4640f0b198..169b5302aa 100644
--- a/api/controllers/trigger/trigger.py
+++ b/api/controllers/trigger/trigger.py
@@ -41,4 +41,4 @@ def trigger_endpoint(endpoint_id: str):
         return jsonify({"error": "Endpoint processing failed", "message": str(e)}), 500
     except Exception as e:
         logger.exception("Webhook processing failed for {endpoint_id}")
-        return jsonify({"error": "Internal server error", "message": str(e)}), 500
+        return jsonify({"error": "Internal server error"}), 500

From e9078eedbd16caeb90c24d048b53893aa47bea0a Mon Sep 17 00:00:00 2001
From: Yeuoly <admin@srmxy.cn>
Date: Thu, 30 Oct 2025 16:14:13 +0800
Subject: [PATCH 17/17] fix: Variable e is not accessed (reportUnusedVariable)

---
 api/services/trigger/trigger_subscription_builder_service.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/api/services/trigger/trigger_subscription_builder_service.py b/api/services/trigger/trigger_subscription_builder_service.py
index d7324aec3e..571393c782 100644
--- a/api/services/trigger/trigger_subscription_builder_service.py
+++ b/api/services/trigger/trigger_subscription_builder_service.py
@@ -472,7 +472,7 @@ class TriggerSubscriptionBuilderService:
                 response=response,
             )
             return response
-        except Exception as e:
+        except Exception:
             logger.exception("Error during validation endpoint dispatch for endpoint_id=%s", endpoint_id)
             error_response = Response(status=500, response="An internal error has occurred.")
             cls.append_log(endpoint_id=endpoint_id, request=request, response=error_response)