Files
cowork-local/ui/mcp_servers_dialog.py
d2101e1f87
CI / test (pull_request) Canceled after 0s
fix(i18n): dịch nốt chữ do Qt tự vẽ, và thêm dòng phiên bản ở góc phải
Người dùng báo: chọn tiếng Nhật mà nhóm Sandbox Security, nút Save/Cancel và
nhiều chỗ khác vẫn tiếng Anh. Bộ test i18n cũ vẫn xanh vì nó chỉ bắt lỗi "có
dịch nhưng không ai áp lại" — hai lỗ thật nằm chỗ khác:

* Chuỗi HARDCODE không đi qua ``tr()`` bao giờ (``QPushButton("Unlock")``), nên
  phép đo "đổi ngôn ngữ rồi tìm chỗ không mang mốc" thấy nó đứng yên ở cả hai
  lần chụp và coi là bình thường.
* Nhãn nút do CHÍNH Qt vẽ. ``QDialogButtonBox``, ``QMessageBox.question`` và
  ``QInputDialog.get*`` lấy chữ từ bảng dịch riêng của Qt; ứng dụng không cài
  ``QTranslator`` nào và bản PySide6 đang dùng cũng không đóng gói file
  ``qtbase_*.qm`` nào để cài — nên chúng luôn rơi về tiếng Anh.

``ui/dialog_buttons.py`` gán nhãn của dự án đè lên nhãn Qt: ``dialog_buttons``
(10 hộp thoại), ``confirm`` (13 hộp Có/Không), ``ask_text``/``ask_multiline``/
``ask_item`` (16 hộp nhập liệu). Cùng với 17 chuỗi hardcode và 5 câu lỗi mà
``core/tasks.py`` trả thẳng ra hộp thoại — nay trả KHOÁ i18n, nơi hiển thị mới
gọi ``tr()`` — là 61 chỗ.

Ba chỗ nữa cùng lớp lỗi, phát hiện khi rà lại:

* ``_add_section`` nhận chuỗi ĐÃ dịch nên bốn tiêu đề mục của Step config đứng
  nguyên ở ngôn ngữ lúc dựng panel. Nay nhận khoá + ``bind_dynamic`` để không
  mất trạng thái gập/mở khi đổi ngôn ngữ.
* Thẻ tool ở Giám sát ▸ Công cụ hiện thẳng ``spec.description`` — chuỗi gửi cho
  MÔ HÌNH trong schema function-calling, phải giữ tiếng Anh. Thêm bộ mô tả hiển
  thị riêng cho 9 tool.
* Tên nhóm catalog ở tab Connector ("Other (any generic MCP server)").

Kèm theo, phần giao diện người dùng yêu cầu:

* ``__version__`` 2.26.0 -> 0.0.1, một nguồn cho tiêu đề cửa sổ, tab Giới thiệu
  và dòng mới ở góc phải thanh trạng thái (thay dòng ghi công tác giả).
* Tắt size grip: nó vẽ một vệt ngay bên phải dòng phiên bản. Cửa sổ vẫn kéo
  giãn được từ các cạnh.
* ``_NAV_SETTINGS_GAP`` 10 -> 4: hàng Cài đặt bớt xa nhóm Dashboard/Giám sát.

Hai test SẼ TREO nếu không sửa kèm: chúng patch ``QInputDialog.getText/getItem``
để tự trả lời, mà code nay gọi ``ask_text``/``ask_item`` — patch không còn chặn
được và hộp thoại thật sẽ mở ra chờ người bấm.

Ba cổng mới trong ``tests/ui/test_i18n_khong_hardcode_chu.py`` canh ở mức cấu
trúc (không ai được dựng lại kiểu cũ); đã kiểm chúng CẮN trên bản trước khi sửa:
10 + 13 + 17 vi phạm.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
2026-09-10 01:43:22 +09:00

72 lines
2.8 KiB
Python

"""Add/edit one external MCP (Model Context Protocol) server entry — Settings'
"🔌 MCP Servers" section (see settings_dialog.py for the list/CRUD)."""
from __future__ import annotations
import shlex
from typing import Optional
from PySide6.QtWidgets import (
QDialog, QDialogButtonBox, QLabel, QLineEdit, QVBoxLayout,
)
from ..i18n import tr
from .dialog_buttons import dialog_buttons
class McpServerEditDialog(QDialog):
"""Form thêm/sửa một máy chủ MCP: tên, lệnh chạy, tham số và công tắc bật.
Đã được ``ui/connectors_panel.py`` thay thế; giữ lại làm bản đối chiếu.
"""
def __init__(self, parent=None, server: Optional[dict] = None):
"""Dựng form. ``server`` để None thì đây là form thêm mới."""
super().__init__(parent)
server = server or {}
self.setWindowTitle(tr("mcp.edit_title") if server else tr("mcp.add_title"))
self.setMinimumWidth(480)
self._enabled = bool(server.get("enabled", True))
lay = QVBoxLayout(self)
lay.addWidget(QLabel(tr("mcp.name_label")))
self.name = QLineEdit(server.get("name", ""))
self.name.setPlaceholderText(tr("mcp.name_placeholder"))
lay.addWidget(self.name)
lay.addWidget(QLabel(tr("mcp.command_label")))
self.command = QLineEdit(server.get("command", ""))
self.command.setPlaceholderText(tr("mcp.command_placeholder"))
lay.addWidget(self.command)
lay.addWidget(QLabel(tr("mcp.args_label")))
self.args = QLineEdit(" ".join(server.get("args", []) or []))
self.args.setPlaceholderText(tr("mcp.args_placeholder"))
lay.addWidget(self.args)
buttons = dialog_buttons(QDialogButtonBox.Save | QDialogButtonBox.Cancel)
buttons.accepted.connect(self._on_accept)
buttons.rejected.connect(self.reject)
lay.addWidget(buttons)
def _on_accept(self) -> None:
"""Chỉ đóng khi đã có tên và lệnh chạy — thiếu một trong hai thì máy chủ không
khởi động được. Con trỏ nhảy về ô tên thay vì hiện hộp lỗi.
"""
if not self.name.text().strip() or not self.command.text().strip():
self.name.setFocus()
return
self.accept()
def result_server(self) -> dict:
"""Nội dung form dưới dạng dict.
Tham số được tách bằng ``shlex`` chứ không phải ``split()``: đường dẫn có
khoảng trắng phải đặt trong dấu nháy và vẫn tính là MỘT tham số.
"""
args_text = self.args.text().strip()
return {
"name": self.name.text().strip(),
"command": self.command.text().strip(),
"args": shlex.split(args_text) if args_text else [],
"enabled": self._enabled,
}