@autobest-ui/agent 1.0.10 → 1.0.12

package/README.md

@@ -40,21 +40,35 @@ npx --yes \
   autobest-azurepr-mcp
 ```
 
+启动 Playwright Trace 人工录制 MCP：
+
+```bash
+npx --yes --package=@autobest-ui/agent@latest playwright install chromium
+
+BACKEND_BASE_URL="http://127.0.0.1:3001" \
+MAX_RECORD_DURATION="1800000" \
+npx --yes \
+  --package=@autobest-ui/agent@latest \
+  autobest-trace-mcp-recorder
+```
+
 启动成功后，MCP 会通过 stdio 持续等待客户端请求，终端没有输出属于正常状态，可按 `Ctrl+C` 停止。供 Codex 长期使用时，不要只执行上述临时命令，应将完整配置加入 `~/.codex/config.toml` 后重启 Codex。
 
-同一个 npm 包暴露三个独立命令：
+同一个 npm 包暴露多个独立命令：
 
-| 命令                   | 用途                           | 必需配置           |
-| ---------------------- | ------------------------------ | ------------------ |
-| `autobest-rag-mcp`     | 连接 PRD Knowledge RAG API     | `RAG_API_BASE_URL` |
-| `autobest-azurepr-mcp` | 读取 Azure DevOps Pull Request | `AZURE_DEVOPS_PAT` |
-| `figma-mcp-bridge`     | 连接本机 Figma Plugin          | 无                 |
+| 命令                            | 用途                           | 必需配置           |
+| ------------------------------- | ------------------------------ | ------------------ |
+| `autobest-rag-mcp`              | 连接 PRD Knowledge RAG API     | `RAG_API_BASE_URL` |
+| `autobest-azurepr-mcp`          | 读取 Azure DevOps Pull Request | `AZURE_DEVOPS_PAT` |
+| `autobest-trace-mcp-recorder`   | 录制并上传 Playwright Trace    | `BACKEND_BASE_URL` |
+| `figma-mcp-bridge`              | 连接本机 Figma Plugin          | 无                 |
 
 Codex 配置示例分别位于：
 
 - [RAG MCP 配置](mcp/rag-mcp-bridge/config.toml.example)
 - [Azure PR MCP 配置](mcp/azurepr-mcp-bridge/config.toml.example)
 - [Figma MCP 配置](mcp/figma-mcp-bridge/config.toml.example)
+- [Trace Recorder MCP 配置](mcp/trace-mcp-recorder/config.toml.example)
 
 生产环境可将 `@latest` 替换为明确版本，以固定 MCP 行为。
 

package/bin/sync-assets.test.mjs

@@ -23,7 +23,9 @@ test("installs common skills into the user-level skills directory", async () =>
       "code-pr-submit",
       "figma-ui-capture",
       "make-spec",
+      "repo-setup",
       "review-from-docs",
+      "trace-recorder",
       "ui-prd-scope",
     ]);
     await access(
@@ -33,6 +35,8 @@ test("installs common skills into the user-level skills directory", async () =>
     await access(
       path.join(result.targetRoot, "make-spec", "scripts", "validate-spec-traceability.mjs")
     );
+    await access(path.join(result.targetRoot, "repo-setup", "SKILL.md"));
+    await access(path.join(result.targetRoot, "trace-recorder", "SKILL.md"));
     await access(
       path.join(result.targetRoot, "review-from-docs", "references", "review-result-schema.md")
     );

package/mcp/code-mcp-pr/README.md

@@ -9,7 +9,7 @@ export AZURE_DEVOPS_PAT="your-personal-access-token"
 export DINGDING_ALERT_TOKEN="your-dingtalk-robot-access-token"
 ```
 
-`AZURE_DEVOPS_PAT` 是所有工具访问 Azure DevOps 的必需配置。`DINGDING_ALERT_TOKEN` 仅在 `code-mcp-pr` 同时收到钉钉 markdown 标题和正文时使用。两个 token 都不会作为 MCP 工具参数接收。
+`AZURE_DEVOPS_PAT` 是访问 Azure DevOps 的工具所需配置。`DINGDING_ALERT_TOKEN` 在 `code-mcp-pr` 创建新 PR 后发送通知，或 `send_pr_dingtalk` 为已有 PR 补发通知时使用。两个 token 都不会作为 MCP 工具参数接收。
 
 ## 启动
 
@@ -30,7 +30,8 @@ Codex 配置示例见 [config.toml.example](config.toml.example)。
 
 ## 工具
 
-- `code-mcp-pr`：`businessTargetBranch` 作为 ADO `sourceRefName`，`businessSourceBranch` 作为 `targetRefName`。仅在 PR 创建成功且两个 markdown 字段都有值时发送钉钉通知。
+- `code-mcp-pr`：`businessTargetBranch` 作为 ADO `sourceRefName`，`businessSourceBranch` 作为 `targetRefName`。目标为 `main/master` 时要求显式传入 `targetBranchConfirmed=true`。创建前检查相同方向的 active PR；已存在时返回 `alreadyExists=true` 和 `prWebUrl`，不重复发送钉钉。仅在新 PR 创建成功且两个 markdown 字段都有值时发送钉钉通知。
+- `send_pr_dingtalk`：在用户确认后，为一个已存在的 PR 链接补发钉钉 markdown 通知；不会创建或修改 PR。
 - `get_repo_default_branch`：读取仓库默认分支。
 - `guess_parent_branch`：以默认分支和 ADO merge-base 信息进行启发式推测。
 - `get_branch_diff`：读取 source 相对 target 的文件列表和受限 diff 片段。

package/mcp/code-mcp-pr/azure-devops.js

@@ -28,6 +28,7 @@ const DIFF_FILE_CONCURRENCY = 5;
  * @property {string} prTitle
  * @property {string=} prDescription
  * @property {string[]=} reviewers
+ * @property {boolean=} targetBranchConfirmed
  * @property {string=} dingtalkMarkdownTitle
  * @property {string=} dingtalkMarkdownContent
  */
@@ -35,6 +36,9 @@ const DIFF_FILE_CONCURRENCY = 5;
 /**
  * @typedef {object} CreatePullRequestResult
  * @property {boolean} success
+ * @property {boolean} created
+ * @property {boolean} alreadyExists
+ * @property {boolean} confirmationRequired
  * @property {string|null} prId
  * @property {string|null} prWebUrl
  * @property {JsonObject} adoApiPayload
@@ -43,6 +47,13 @@ const DIFF_FILE_CONCURRENCY = 5;
  * @property {string|null} errorMsg
  */
 
+/**
+ * @typedef {object} SendPullRequestDingtalkInput
+ * @property {string} prWebUrl
+ * @property {string} dingtalkMarkdownTitle
+ * @property {string} dingtalkMarkdownContent
+ */
+
 /**
  * @typedef {object} DefaultBranchResult
  * @property {boolean} success
@@ -203,6 +214,37 @@ function pullRequestWebUrl(input, responseBody, prId) {
   return `https://dev.azure.com/${encodeURIComponent(input.orgName)}/${encodeURIComponent(input.projectName)}/_git/${encodeURIComponent(repositoryName)}/pullrequest/${encodeURIComponent(prId)}`;
 }
 
+function activePullRequestsUrl(input, sourceRefName, targetRefName) {
+  const query = new URLSearchParams({
+    'searchCriteria.status': 'active',
+    'searchCriteria.sourceRefName': sourceRefName,
+    'searchCriteria.targetRefName': targetRefName,
+    '$top': '1'
+  });
+  return `${repositoryRoot(input)}/pullrequests?${query.toString()}`;
+}
+
+async function findActivePullRequest(input, sourceRefName, targetRefName, pat) {
+  const result = await adoRequest(activePullRequestsUrl(input, sourceRefName, targetRefName), pat);
+  if (!result.ok) return { pullRequest: null, errorMsg: result.errorMsg };
+  const body = asObject(result.data);
+  const pullRequests = Array.isArray(body.value) ? body.value : [];
+  return { pullRequest: asObject(pullRequests[0]), errorMsg: null };
+}
+
+function existingPullRequestResult(baseResult, input, pullRequest) {
+  const rawPrId = pullRequest.pullRequestId;
+  if (rawPrId === undefined || rawPrId === null) return null;
+  const prId = String(rawPrId);
+  return {
+    ...baseResult,
+    success: true,
+    alreadyExists: true,
+    prId,
+    prWebUrl: pullRequestWebUrl(input, pullRequest, prId)
+  };
+}
+
 async function sendDingtalkMarkdown(title, content) {
   const token = process.env.DINGDING_ALERT_TOKEN?.trim();
   if (!token) {
@@ -239,6 +281,9 @@ export async function createPullRequest(input) {
   };
   const baseResult = {
     success: false,
+    created: false,
+    alreadyExists: false,
+    confirmationRequired: false,
     prId: null,
     prWebUrl: null,
     adoApiPayload,
@@ -251,14 +296,35 @@ export async function createPullRequest(input) {
     return { ...baseResult, errorMsg: 'businessTargetBranch 不能等于 businessSourceBranch' };
   }
 
+  const targetBranch = branchVersion(targetRefName).toLowerCase();
+  if ((targetBranch === 'main' || targetBranch === 'master') && input.targetBranchConfirmed !== true) {
+    return {
+      ...baseResult,
+      confirmationRequired: true,
+      errorMsg: `目标分支 ${targetBranch} 是主分支，需要用户明确确认后才能创建PR`
+    };
+  }
+
   const pat = requirePat();
   if (!pat) return { ...baseResult, errorMsg: missingPatError() };
 
+  const existingLookup = await findActivePullRequest(input, sourceRefName, targetRefName, pat);
+  if (existingLookup.errorMsg) return { ...baseResult, errorMsg: existingLookup.errorMsg };
+  const existingResult = existingPullRequestResult(baseResult, input, existingLookup.pullRequest);
+  if (existingResult) return existingResult;
+
   const createResult = await adoRequest(`${repositoryRoot(input)}/pullrequests`, pat, {
     method: 'POST',
     body: JSON.stringify(adoApiPayload)
   });
-  if (!createResult.ok) return { ...baseResult, errorMsg: createResult.errorMsg };
+  if (!createResult.ok) {
+    if (createResult.status === 409) {
+      const raceLookup = await findActivePullRequest(input, sourceRefName, targetRefName, pat);
+      const racedExistingResult = existingPullRequestResult(baseResult, input, raceLookup.pullRequest);
+      if (racedExistingResult) return racedExistingResult;
+    }
+    return { ...baseResult, errorMsg: createResult.errorMsg };
+  }
 
   const response = asObject(createResult.data);
   const rawPrId = response.pullRequestId;
@@ -268,7 +334,7 @@ export async function createPullRequest(input) {
 
   const prId = String(rawPrId);
   const prWebUrl = pullRequestWebUrl(input, response, prId);
-  const successResult = { ...baseResult, success: true, prId, prWebUrl };
+  const successResult = { ...baseResult, success: true, created: true, prId, prWebUrl };
   const title = input.dingtalkMarkdownTitle;
   const content = input.dingtalkMarkdownContent;
   if (!title?.trim() || !content?.trim()) return successResult;
@@ -282,6 +348,19 @@ export async function createPullRequest(input) {
   };
 }
 
+/** @param {SendPullRequestDingtalkInput} input */
+export async function sendPullRequestDingtalk(input) {
+  const renderedContent = input.dingtalkMarkdownContent.replaceAll('{{PR_WEB_URL}}', input.prWebUrl);
+  const dingtalk = await sendDingtalkMarkdown(input.dingtalkMarkdownTitle, renderedContent);
+  return {
+    success: dingtalk.success,
+    prWebUrl: input.prWebUrl,
+    dingtalkSendSuccess: dingtalk.success,
+    dingtalkResponse: dingtalk.response,
+    errorMsg: dingtalk.success ? null : boundedJson(dingtalk.response) || '钉钉通知发送失败'
+  };
+}
+
 /** @param {RepositoryInput} input @returns {Promise<DefaultBranchResult>} */
 export async function getRepoDefaultBranch(input) {
   const pat = requirePat();

package/mcp/code-mcp-pr/index.js

@@ -8,7 +8,8 @@ import {
   getBranchDiff,
   getRepoDefaultBranch,
   guessParentBranch,
-  readFileAtRef
+  readFileAtRef,
+  sendPullRequestDingtalk
 } from './azure-devops.js';
 
 const server = new McpServer(
@@ -56,6 +57,10 @@ registerJsonTool(
       prTitle: z.string().min(1).describe('PR标题'),
       prDescription: z.string().optional().describe('PR描述，可选，条目化变更详情'),
       reviewers: z.array(z.string().min(1)).optional().describe('评审人ID列表，可选'),
+      targetBranchConfirmed: z
+        .boolean()
+        .optional()
+        .describe('目标分支为main或master时，必须在用户明确二次确认后传true'),
       dingtalkMarkdownTitle: z.string().optional().describe('钉钉通知markdown标题，可选；不传则不发送钉钉'),
       dingtalkMarkdownContent: z
         .string()
@@ -67,6 +72,9 @@ registerJsonTool(
   createPullRequest,
   errorMsg => ({
     success: false,
+    created: false,
+    alreadyExists: false,
+    confirmationRequired: false,
     prId: null,
     prWebUrl: null,
     adoApiPayload: {},
@@ -76,6 +84,31 @@ registerJsonTool(
   })
 );
 
+registerJsonTool(
+  'send_pr_dingtalk',
+  {
+    title: '发送已有 PR 钉钉通知',
+    description: '为已确认的 PR 链接发送钉钉 markdown 通知；不会创建或修改 PR。',
+    inputSchema: {
+      prWebUrl: z.string().url().describe('已存在 PR 的可访问 Web 链接'),
+      dingtalkMarkdownTitle: z.string().min(1).describe('钉钉通知 markdown 标题'),
+      dingtalkMarkdownContent: z
+        .string()
+        .min(1)
+        .describe('钉钉通知 markdown 正文；内部会替换{{PR_WEB_URL}}占位符为 PR 链接')
+    },
+    annotations: { readOnlyHint: false, destructiveHint: false, idempotentHint: false, openWorldHint: true }
+  },
+  sendPullRequestDingtalk,
+  errorMsg => ({
+    success: false,
+    prWebUrl: null,
+    dingtalkSendSuccess: false,
+    dingtalkResponse: null,
+    errorMsg
+  })
+);
+
 registerJsonTool(
   'get_repo_default_branch',
   {

package/mcp/code-mcp-pr/index.test.js

@@ -12,7 +12,8 @@ import {
   getRepoDefaultBranch,
   guessParentBranch,
   normalizeBranchRef,
-  readFileAtRef
+  readFileAtRef,
+  sendPullRequestDingtalk
 } from './azure-devops.js';
 
 const directory = path.dirname(fileURLToPath(import.meta.url));
@@ -57,6 +58,7 @@ test('creates the reverse PR with the required branch direction and sends render
   globalThis.fetch = async (url, options) => {
     calls.push({ url: String(url), options });
     if (String(url).startsWith('https://dev.azure.com/')) {
+      if (!options?.method || options.method === 'GET') return jsonResponse({ value: [] });
       return jsonResponse({
         pullRequestId: 42,
         repository: { name: 'web' },
@@ -75,12 +77,15 @@ test('creates the reverse PR with the required branch direction and sends render
         businessTargetBranch: 'release/with-change',
         businessSourceBranch: 'main',
         prTitle: 'Reverse PR',
+        targetBranchConfirmed: true,
         reviewers: ['reviewer-id'],
         dingtalkMarkdownTitle: 'PR created',
         dingtalkMarkdownContent: '[Open PR]({{PR_WEB_URL}})'
       });
 
       assert.equal(result.success, true);
+      assert.equal(result.created, true);
+      assert.equal(result.alreadyExists, false);
       assert.equal(result.dingtalkSendSuccess, true);
       assert.deepEqual(result.adoApiPayload, {
         sourceRefName: 'refs/heads/release/with-change',
@@ -88,14 +93,14 @@ test('creates the reverse PR with the required branch direction and sends render
         title: 'Reverse PR',
         reviewers: [{ id: 'reviewer-id' }]
       });
-      assert.equal(calls.length, 2);
-      assert.match(calls[0].url, new RegExp(`api-version=${API_VERSION.replaceAll('.', '\\.')}`));
-      assert.deepEqual(JSON.parse(calls[0].options.body), result.adoApiPayload);
+      assert.equal(calls.length, 3);
+      assert.match(calls[1].url, new RegExp(`api-version=${API_VERSION.replaceAll('.', '\\.')}`));
+      assert.deepEqual(JSON.parse(calls[1].options.body), result.adoApiPayload);
       assert.equal(
-        JSON.parse(calls[1].options.body).markdown.text,
+        JSON.parse(calls[2].options.body).markdown.text,
         '[Open PR](https://dev.azure.com/org/project/_git/web/pullrequest/42)'
       );
-      assert.doesNotMatch(calls[0].options.headers.Authorization, /secret-pat/);
+      assert.doesNotMatch(calls[1].options.headers.Authorization, /secret-pat/);
     });
   } finally {
     globalThis.fetch = originalFetch;
@@ -105,8 +110,9 @@ test('creates the reverse PR with the required branch direction and sends render
 test('does not call DingTalk after an Azure DevOps create failure', async () => {
   let callCount = 0;
   const originalFetch = globalThis.fetch;
-  globalThis.fetch = async () => {
+  globalThis.fetch = async (_url, options) => {
     callCount += 1;
+    if (!options?.method || options.method === 'GET') return jsonResponse({ value: [] });
     return jsonResponse({ message: 'denied' }, 401);
   };
 
@@ -119,12 +125,13 @@ test('does not call DingTalk after an Azure DevOps create failure', async () =>
         businessTargetBranch: 'feature',
         businessSourceBranch: 'main',
         prTitle: 'PR',
+        targetBranchConfirmed: true,
         dingtalkMarkdownTitle: 'title',
         dingtalkMarkdownContent: 'content'
       });
       assert.equal(result.success, false);
       assert.equal(result.dingtalkSendSuccess, null);
-      assert.equal(callCount, 1);
+      assert.equal(callCount, 2);
     });
   } finally {
     globalThis.fetch = originalFetch;
@@ -133,7 +140,10 @@ test('does not call DingTalk after an Azure DevOps create failure', async () =>
 
 test('keeps PR success when DingTalk token is missing', async () => {
   const originalFetch = globalThis.fetch;
-  globalThis.fetch = async () => jsonResponse({ pullRequestId: 7 });
+  globalThis.fetch = async (_url, options) =>
+    !options?.method || options.method === 'GET'
+      ? jsonResponse({ value: [] })
+      : jsonResponse({ pullRequestId: 7 });
   try {
     await withEnvironment({ AZURE_DEVOPS_PAT: 'pat', DINGDING_ALERT_TOKEN: undefined }, async () => {
       const result = await createPullRequest({
@@ -143,6 +153,7 @@ test('keeps PR success when DingTalk token is missing', async () => {
         businessTargetBranch: 'feature',
         businessSourceBranch: 'main',
         prTitle: 'PR',
+        targetBranchConfirmed: true,
         dingtalkMarkdownTitle: 'title',
         dingtalkMarkdownContent: 'content'
       });
@@ -155,6 +166,138 @@ test('keeps PR success when DingTalk token is missing', async () => {
   }
 });
 
+test('requires explicit confirmation before targeting main or master', async () => {
+  const originalFetch = globalThis.fetch;
+  globalThis.fetch = async () => assert.fail('fetch must not be called before confirmation');
+  try {
+    for (const targetBranch of ['main', 'refs/heads/master', 'MAIN']) {
+      const result = await createPullRequest({
+        orgName: 'org',
+        projectName: 'project',
+        repoId: 'repo',
+        businessTargetBranch: 'feature',
+        businessSourceBranch: targetBranch,
+        prTitle: 'PR'
+      });
+      assert.equal(result.success, false);
+      assert.equal(result.confirmationRequired, true);
+      assert.match(result.errorMsg, /明确确认/);
+    }
+  } finally {
+    globalThis.fetch = originalFetch;
+  }
+});
+
+test('returns the existing active PR link without sending DingTalk again', async () => {
+  const calls = [];
+  const originalFetch = globalThis.fetch;
+  globalThis.fetch = async (url, options) => {
+    calls.push({ url: String(url), options });
+    return jsonResponse({
+      value: [
+        {
+          pullRequestId: 88,
+          repository: { name: 'web' },
+          _links: { web: { href: 'https://dev.azure.com/org/project/_git/web/pullrequest/88' } }
+        }
+      ]
+    });
+  };
+
+  try {
+    await withEnvironment({ AZURE_DEVOPS_PAT: 'pat', DINGDING_ALERT_TOKEN: 'ding-token' }, async () => {
+      const result = await createPullRequest({
+        orgName: 'org',
+        projectName: 'project',
+        repoId: 'repo',
+        businessTargetBranch: 'feature',
+        businessSourceBranch: 'develop',
+        prTitle: 'PR',
+        dingtalkMarkdownTitle: 'title',
+        dingtalkMarkdownContent: '[PR]({{PR_WEB_URL}})'
+      });
+
+      assert.equal(result.success, true);
+      assert.equal(result.created, false);
+      assert.equal(result.alreadyExists, true);
+      assert.equal(result.prId, '88');
+      assert.equal(result.prWebUrl, 'https://dev.azure.com/org/project/_git/web/pullrequest/88');
+      assert.equal(result.dingtalkSendSuccess, null);
+      assert.equal(calls.length, 1);
+      const query = new URL(calls[0].url).searchParams;
+      assert.equal(query.get('searchCriteria.status'), 'active');
+      assert.equal(query.get('searchCriteria.sourceRefName'), 'refs/heads/feature');
+      assert.equal(query.get('searchCriteria.targetRefName'), 'refs/heads/develop');
+      assert.equal(calls[0].options?.method, undefined);
+    });
+  } finally {
+    globalThis.fetch = originalFetch;
+  }
+});
+
+test('returns the existing PR link when creation races with another submitter', async () => {
+  let adoCall = 0;
+  const originalFetch = globalThis.fetch;
+  globalThis.fetch = async () => {
+    adoCall += 1;
+    if (adoCall === 1) return jsonResponse({ value: [] });
+    if (adoCall === 2) return jsonResponse({ message: 'active pull request already exists' }, 409);
+    return jsonResponse({
+      value: [
+        {
+          pullRequestId: 99,
+          repository: { name: 'web' },
+          _links: { web: { href: 'https://dev.azure.com/org/project/_git/web/pullrequest/99' } }
+        }
+      ]
+    });
+  };
+
+  try {
+    await withEnvironment({ AZURE_DEVOPS_PAT: 'pat' }, async () => {
+      const result = await createPullRequest({
+        orgName: 'org',
+        projectName: 'project',
+        repoId: 'repo',
+        businessTargetBranch: 'feature',
+        businessSourceBranch: 'develop',
+        prTitle: 'PR'
+      });
+      assert.equal(result.success, true);
+      assert.equal(result.created, false);
+      assert.equal(result.alreadyExists, true);
+      assert.equal(result.prWebUrl, 'https://dev.azure.com/org/project/_git/web/pullrequest/99');
+      assert.equal(adoCall, 3);
+    });
+  } finally {
+    globalThis.fetch = originalFetch;
+  }
+});
+
+test('sends a confirmed DingTalk notification for an existing PR link', async () => {
+  const originalFetch = globalThis.fetch;
+  let markdownText;
+  globalThis.fetch = async (_url, options) => {
+    markdownText = JSON.parse(options.body).markdown.text;
+    return jsonResponse({ errcode: 0, errmsg: 'ok' });
+  };
+
+  try {
+    await withEnvironment({ DINGDING_ALERT_TOKEN: 'ding-token' }, async () => {
+      const result = await sendPullRequestDingtalk({
+        prWebUrl: 'https://dev.azure.com/org/project/_git/web/pullrequest/88',
+        dingtalkMarkdownTitle: 'Existing PR',
+        dingtalkMarkdownContent: '[Open]({{PR_WEB_URL}})'
+      });
+      assert.equal(result.success, true);
+      assert.equal(result.dingtalkSendSuccess, true);
+      assert.equal(markdownText, '[Open](https://dev.azure.com/org/project/_git/web/pullrequest/88)');
+    });
+  } finally {
+    globalThis.fetch = originalFetch;
+  }
+});
+
 test('rejects identical normalized PR branches without making an HTTP request', async () => {
   const originalFetch = globalThis.fetch;
   globalThis.fetch = async () => assert.fail('fetch must not be called');
@@ -266,7 +409,7 @@ test('returns a structured missing-PAT error', async () => {
   });
 });
 
-test('exposes all five tools over stdio', async () => {
+test('exposes all six tools over stdio', async () => {
   const client = new Client({ name: 'code-mcp-pr-test', version: '1.0.0' });
   const transport = new StdioClientTransport({
     command: process.execPath,
@@ -281,7 +424,14 @@ test('exposes all five tools over stdio', async () => {
     const { tools } = await client.listTools();
     assert.deepEqual(
       tools.map(tool => tool.name),
-      ['code-mcp-pr', 'get_repo_default_branch', 'guess_parent_branch', 'get_branch_diff', 'read_file_at_ref']
+      [
+        'code-mcp-pr',
+        'send_pr_dingtalk',
+        'get_repo_default_branch',
+        'guess_parent_branch',
+        'get_branch_diff',
+        'read_file_at_ref'
+      ]
     );
   } finally {
     await client.close();

package/mcp/trace-mcp-recorder/.env.example

@@ -0,0 +1,6 @@
+# 必填：私有后端服务基础地址，不要以 / 结尾。
+BACKEND_BASE_URL=http://127.0.0.1:3001
+
+# 可选：最大录制时长（毫秒），默认 1800000（30 分钟）。
+MAX_RECORD_DURATION=1800000
+

package/mcp/trace-mcp-recorder/README.md

@@ -0,0 +1,58 @@
+# trace-mcp-recorder
+
+`trace-mcp-recorder` 是基于 `@modelcontextprotocol/sdk` 和 `StdioServerTransport` 的 Node.js MCP Server。它打开非无头 Chromium 供测试人员手动操作，停止后上传 `trace.zip` 和浏览器元数据，并返回私有后端的 Trace Viewer 链接。同一进程同一时间只允许一个录制任务。
+
+## 安装与启动
+
+要求 Node.js 20 或更高版本。在本目录安装依赖及 Chromium：
+
+```bash
+npm install
+npx playwright install chromium
+```
+
+复制 `.env.example` 中的变量到 MCP 客户端环境。服务不会自动读取 `.env` 文件，也不会在代码中提供后端默认地址。直接启动：
+
+```bash
+BACKEND_BASE_URL="http://127.0.0.1:3001" \
+MAX_RECORD_DURATION="1800000" \
+npm start
+```
+
+服务使用 stdio 通信，启动后持续等待请求且不会向标准输出写日志。
+
+## Codex 接入
+
+将 [config.toml.example](config.toml.example) 的内容加入 `~/.codex/config.toml` 并按实际部署修改 `BACKEND_BASE_URL`，然后重启 Codex。发布包暴露的命令是 `autobest-trace-mcp-recorder`。
+
+配套 Skill 位于 `../../skills/common/trace-recorder/`，由 `autobest-agent-sync common` 安装，并作为 `skill://trace-recorder/SKILL.md` MCP 资源提供。Skill 的 `agents/openai.yaml` 声明了对 `trace-mcp-recorder` 的工具依赖。
+
+## 环境变量
+
+| 变量 | 默认值 | 说明 |
+| --- | --- | --- |
+| `BACKEND_BASE_URL` | 无，必填 | 私有后端基础地址，必须为 HTTP(S) URL |
+| `MAX_RECORD_DURATION` | `1800000` | 最大录制时长，单位毫秒；超时后自动停止并上传 |
+
+回放链接按 `${BACKEND_BASE_URL}/trace-viewer/?trace=${fileUrl}` 生成。服务地址、域名和端口均来自 `BACKEND_BASE_URL`。
+
+## MCP 工具
+
+- `start_trace_recording`：参数为必填 `url` 和可选 `version`、`title`、`operator`。`url` 独立确定被测页面；`title` 是录制标题或功能描述，也可以直接填写 `bug3452` 这类 Bug 编号。系统不额外传递独立的 Bug 字段。MCP 认证上下文中的 `operator`、`name` 或 `preferred_username` 优先于参数。
+- `stop_trace_recording`：首次调用不传参数，只停止 trace 并询问是否上传；用户确认后传 `confirm=upload`，取消时传 `confirm=cancel`。上传成功后返回回放链接及版本、标题、操作人、浏览器版本。
+
+录制超时后服务自动停止并自动上传（无需人工确认），随后第一次人工调用 `stop_trace_recording` 会取回自动停止的结果；再调用则返回没有活动会话。
+
+## 后端接口契约
+
+服务期望后端提供以下接口：
+
+- `POST /api/trace-sessions`：接收 `multipart/form-data`，必填字段为 `trace`、`target_url`，可选字段为 `version`、`title`、`operator`、`browser_name`、`browser_version`、`user_agent`、`os_name`、`viewport`；后端生成 UUID，创建并上传成功后返回 `{ "uuid": "...", "fileUrl": "...", "item": { "status": "completed" } }`。
+
+`metadata` 包含浏览器版本、User-Agent、viewport、MCP 所在操作系统信息、目标 URL 和录制业务字段。上传或状态更新失败时，本地 trace 仍会被清理，会话会尽力回写为 `failed`。
+
+## 验证
+
+```bash
+npm run verify
+```

package/mcp/trace-mcp-recorder/config.toml.example

@@ -0,0 +1,11 @@
+[mcp_servers.trace-mcp-recorder]
+command = "npx"
+args = ["--yes", "--package=@autobest-ui/agent@latest", "autobest-trace-mcp-recorder"]
+startup_timeout_sec = 30
+tool_timeout_sec = 180
+enabled = true
+
+[mcp_servers.trace-mcp-recorder.env]
+BACKEND_BASE_URL = "http://127.0.0.1:3001"
+MAX_RECORD_DURATION = "1800000"
+