From 4cad7f1c46d9fe4d6e541ed58a3cea7fe6915f7a Mon Sep 17 00:00:00 2001 From: github-ops Date: Sat, 27 Jun 2026 08:33:16 +0000 Subject: [PATCH] Simplify approval payload settings --- apps/ui/html/wizard/step3-rest.html | 17 ++++------------- apps/ui/js/i18n.js | 10 ++-------- apps/ui/js/wizard-live.js | 4 +--- apps/ui/tests/e2e/wizard.spec.js | 4 ++-- 4 files changed, 9 insertions(+), 26 deletions(-) diff --git a/apps/ui/html/wizard/step3-rest.html b/apps/ui/html/wizard/step3-rest.html index 09561b6..0293d92 100644 --- a/apps/ui/html/wizard/step3-rest.html +++ b/apps/ui/html/wizard/step3-rest.html @@ -159,19 +159,10 @@ -
- -
- - -
-
+ diff --git a/apps/ui/js/i18n.js b/apps/ui/js/i18n.js index eb026dd..b811fcd 100644 --- a/apps/ui/js/i18n.js +++ b/apps/ui/js/i18n.js @@ -609,10 +609,7 @@ var TRANSLATIONS = { 'wizard.approval.elicitation_message_placeholder': 'Confirm operation execution.', 'wizard.approval.elicitation_message_hint': 'Short protocol message. The MCP client still controls the confirmation UI.', 'wizard.approval.ttl': 'How long to wait for confirmation', - 'wizard.approval.show_payload': 'Show request parameters', - 'wizard.approval.payload_mode': 'How to show parameters', - 'wizard.approval.payload.summary': 'Short summary', - 'wizard.approval.payload.masked_json': 'JSON with masked secrets', + 'wizard.approval.show_payload': 'Send call parameters to the confirmation flow', 'wizard.step5.security_level_title': 'Operation security', 'wizard.step5.community_security_note': '', 'wizard.step5.live_title': 'Check and publish', @@ -1474,10 +1471,7 @@ var TRANSLATIONS = { 'wizard.approval.elicitation_message_placeholder': 'Подтвердите выполнение операции.', 'wizard.approval.elicitation_message_hint': 'Короткое протокольное сообщение. Внешний вид окна подтверждения всё равно определяет MCP-клиент.', 'wizard.approval.ttl': 'Сколько ждать подтверждение', - 'wizard.approval.show_payload': 'Показывать параметры запроса', - 'wizard.approval.payload_mode': 'Как показывать параметры', - 'wizard.approval.payload.summary': 'Краткое описание', - 'wizard.approval.payload.masked_json': 'JSON с маскированием секретов', + 'wizard.approval.show_payload': 'Передавать параметры вызова в подтверждение', 'wizard.step5.security_level_title': 'Защита операции', 'wizard.step5.community_security_note': '', 'wizard.step5.live_title': 'Проверка и публикация', diff --git a/apps/ui/js/wizard-live.js b/apps/ui/js/wizard-live.js index 8f32282..8401475 100644 --- a/apps/ui/js/wizard-live.js +++ b/apps/ui/js/wizard-live.js @@ -47,7 +47,7 @@ function buildApprovalPolicy() { risk_level: 'normal', ttl_seconds: normalizeApprovalTtlSeconds(textValue('approval-ttl-seconds')), show_payload_preview: checkedValue('approval-show-payload-preview'), - payload_preview_mode: textValue('approval-payload-preview-mode') || 'summary', + payload_preview_mode: 'summary', elicitation_message: textValue('approval-mode') === 'elicitation' ? (textValue('approval-elicitation-message') || null) : null, @@ -197,7 +197,6 @@ function setApprovalPolicyEditor(policy) { if (showPayload) { showPayload.checked = !policy || policy.show_payload_preview !== false; } - setValue('approval-payload-preview-mode', policy && policy.payload_preview_mode ? policy.payload_preview_mode : 'summary'); updateApprovalPolicyUi(); } @@ -223,7 +222,6 @@ function bindApprovalPolicyControls() { 'approval-elicitation-message', 'approval-ttl-seconds', 'approval-show-payload-preview', - 'approval-payload-preview-mode', ].forEach(function(id) { var element = document.getElementById(id); if (!element || element.dataset.approvalBound === 'true') return; diff --git a/apps/ui/tests/e2e/wizard.spec.js b/apps/ui/tests/e2e/wizard.spec.js index ebf78b2..8ac561c 100644 --- a/apps/ui/tests/e2e/wizard.spec.js +++ b/apps/ui/tests/e2e/wizard.spec.js @@ -830,7 +830,7 @@ test('wizard edit mode preserves explicit request mapping targets on save', asyn await expect(page.locator('#approval-mode')).toHaveValue('custom'); await expect(page.locator('#approval-risk-level')).toHaveCount(0); await expect(page.locator('#approval-ttl-seconds')).toHaveValue('180'); - await expect(page.locator('#approval-payload-preview-mode')).toHaveValue('masked_json'); + await expect(page.locator('#approval-payload-preview-mode')).toHaveCount(0); await page.locator('.btn-save-draft').click(); await expect.poll(() => updatePayload).not.toBeNull(); @@ -862,7 +862,7 @@ test('wizard edit mode preserves explicit request mapping targets on save', asyn risk_level: 'normal', ttl_seconds: 180, show_payload_preview: true, - payload_preview_mode: 'masked_json', + payload_preview_mode: 'summary', elicitation_message: null, }, });