@mingfuyun/star-allinone 0.4.2 → 0.4.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mingfuyun/star-allinone",
-  "version": "0.4.2",
+  "version": "0.4.4",
   "description": "星系运维前端能力包",
   "main": "dist/index.js",
   "bin": {

package/scripts/cli.js

@@ -19,7 +19,7 @@ function usage() {
   star-allinone skills sync [--target <dir>] [--dry-run]
   star-allinone doctor [--target <dir>]
   star-allinone gen service --operation "GET /api/v1/devices/" [--spec <file>] [--domain Devices] [--mock-file] [--dry-run]
-  star-allinone gen page --feature Widgets --route /widgets --title 组件管理 --list-operation "GET /api/v1/widgets/" [--create-operation ...] [--update-operation ...] [--delete-operation ...] [--dry-run]
+  star-allinone gen page --feature Widgets --route /widgets --title 组件管理 --list-operation "GET /api/v1/widgets/" [--create-operation ...] [--update-operation ...] [--delete-operation ...] [--no-tracking] [--dry-run]
 
 Run from a business project containing .star-allinone.json.
 
@@ -30,7 +30,8 @@ If <project>/AGENTS.md is missing, it copies examples/AGENTS.md once.
 doctor checks config, OpenAPI, MFRequest, MFProTable, skill sync, and bad imports.
 
 gen service writes one MFRequest method and types from OpenAPI/Swagger.
-gen page scaffolds a list + drawer + confirm page from OpenAPI schema and can generate services first.`);
+gen page scaffolds a list + drawer + confirm page from OpenAPI schema and can generate services first.
+When src/utils/tracking/index.ts exists, generated pages include getTrackData wiring unless --no-tracking is passed.`);
   process.exitCode = 1;
 }
 
@@ -65,6 +66,7 @@ function parseArgs(args) {
     if (arg === '--dry-run') options.dryRun = true;
     else if (arg === '--mock-file') options.mockFile = true;
     else if (arg === '--force') options.force = true;
+    else if (arg === '--no-tracking') options.noTracking = true;
     else if (arg.startsWith('--')) {
       const key = arg.slice(2).replace(/-([a-z])/g, (_, ch) => ch.toUpperCase());
       options[key] = args[++index];

package/scripts/gen/fixtures/run.js

@@ -49,6 +49,11 @@ function makeProject() {
   fs.mkdirSync(path.join(root, 'src/components/CommonTable'), { recursive: true });
   fs.writeFileSync(path.join(root, 'src/components/CommonTable/MFProTable.tsx'), 'export default function MFProTable() { return null; }\n');
   fs.mkdirSync(path.join(root, 'src/pages'), { recursive: true });
+  fs.mkdirSync(path.join(root, 'src/utils/tracking'), { recursive: true });
+  fs.writeFileSync(
+    path.join(root, 'src/utils/tracking/index.ts'),
+    "export const getTrackData = (...args: unknown[]) => JSON.stringify(args);\n",
+  );
   fs.writeFileSync(path.join(root, 'package.json'), JSON.stringify({
     dependencies: { '@mingfuyun/star-allinone': '^0.3.2' },
     scripts: { typecheck: 'tsc --noEmit' },
@@ -120,11 +125,19 @@ function main() {
   assert(columns.includes("dataIndex: 'kind'"), columns);
   assert(columns.includes("valueType: 'select'"), columns);
   assert(columns.includes("valueType: 'digit'"), columns);
+  assert(columns.includes("getTrackData('create', \"新增\""), columns);
+  assert(columns.includes("getTrackData('edit', \"编辑\""), columns);
+  assert(columns.includes("getTrackData('delete', \"删除\""), columns);
   const drawer = fs.readFileSync(path.join(root, 'src/pages/Widgets/components/WidgetsDrawer.tsx'), 'utf8');
   assert(drawer.includes('ProFormSelect'), drawer);
   assert(drawer.includes("name=\"status\""), drawer);
   assert(drawer.includes("name=\"name\""), drawer);
   assert(drawer.includes('required: true'), drawer);
+  assert(drawer.includes('cancelTrack={getTrackData('), drawer);
+  assert(drawer.includes('okTrack={getTrackData('), drawer);
+  const confirm = fs.readFileSync(path.join(root, 'src/pages/Widgets/components/WidgetsConfirmModal.tsx'), 'utf8');
+  assert(confirm.includes("'data-track': getTrackData('cancel', \"取消\""), confirm);
+  assert(confirm.includes("'data-track': getTrackData('confirm', \"确定\""), confirm);
   const route = fs.readFileSync(path.join(root, 'config/route.ts'), 'utf8');
   assert(route.includes("path: '/widgets'"), route);
 

package/scripts/gen/page.js

@@ -19,6 +19,20 @@ function detectLayout(root) {
   return { hasPageContainer, hasProTableLess, hasContainerConfig };
 }
 
+function hasTracking(root, config) {
+  const modulePath = (config.trackingModule || '@/utils/tracking').replace(/^@\//, 'src/');
+  return fs.existsSync(path.join(root, `${modulePath}.ts`))
+    || fs.existsSync(path.join(root, `${modulePath}/index.ts`));
+}
+
+function trackingImport(config) {
+  return config.trackingModule || '@/utils/tracking';
+}
+
+function emitTrackData(action, label, context) {
+  return `getTrackData('${action}', ${JSON.stringify(label)}, ${JSON.stringify(context)})`;
+}
+
 function featureNames(options) {
   const feature = options.feature.replace(/\\/g, '/').replace(/^\/+|\/+$/g, '');
   if (!feature) throw new Error('Missing --feature. Example: --feature Widgets');
@@ -43,7 +57,7 @@ function searchFieldNames(fields) {
     .map((field) => field.name);
 }
 
-function emitColumns(names, itemType, fields, permissions, queryNames) {
+function emitColumns(names, itemType, fields, permissions, queryNames, tracking, trackingModule) {
   const columnFields = fields.length ? fields : [{ name: 'id', title: 'ID', type: 'string' }];
   const searchable = new Set(
     queryNames?.length
@@ -74,7 +88,19 @@ function emitColumns(names, itemType, fields, permissions, queryNames) {
   const permCreate = permissions.create ? `      permission: '${permissions.create}',\n` : '';
   const permEdit = permissions.edit ? `      permission: '${permissions.edit}',\n` : '';
   const permDelete = permissions.delete ? `      permission: '${permissions.delete}',\n` : '';
-  return `import type {
+  const trackImport = tracking
+    ? `import { getTrackData } from '${trackingModule}';\n`
+    : '';
+  const trackEdit = tracking
+    ? `    track: ${emitTrackData('edit', '编辑', { key: 'action', name: '操作' })},\n`
+    : '';
+  const trackDelete = tracking
+    ? `    track: ${emitTrackData('delete', '删除', { key: 'action', name: '操作' })},\n`
+    : '';
+  const trackCreate = tracking
+    ? `    track: ${emitTrackData('create', '新增', { key: 'toolbar', name: '工具栏' })},\n`
+    : '';
+  return `${trackImport}import type {
   ITableProps,
   TableSearchType,
 } from '@/components/CommonTable/type';
@@ -99,12 +125,12 @@ export const getActionColumn = ({
 >['actions'] => [
   {
     key: 'edit',
-${permEdit}    getText: () => '编辑',
+${permEdit}${trackEdit}    getText: () => '编辑',
     onClick: handleEdit,
   },
   {
     key: 'delete',
-${permDelete}    getText: () => '删除',
+${permDelete}${trackDelete}    getText: () => '删除',
     onClick: handleDelete,
   },
 ];
@@ -116,7 +142,7 @@ export const getToolBarRender = ({
     key: 'create',
     text: '新增',
     type: 'primary',
-${permCreate}    onClick: handleCreate,
+${permCreate}${trackCreate}    onClick: handleCreate,
   },
 ];
 `;
@@ -324,7 +350,7 @@ function emitFormItem(field) {
         />`;
 }
 
-function emitDrawer(names, types, fns, formFields) {
+function emitDrawer(names, types, fns, formFields, tracking, trackingModule) {
   const itemType = types.itemType || 'Record<string, unknown>';
   const fields = formFields.length
     ? formFields
@@ -336,11 +362,18 @@ function emitDrawer(names, types, fns, formFields) {
   const updateCall = fns.update
     ? `        await Service.${fns.update}(record.id, values);`
     : '        // TODO: call update service';
+  const trackImport = tracking
+    ? `import { getTrackData } from '${trackingModule}';\n`
+    : '';
+  const drawerTracks = tracking
+    ? `\n      cancelTrack={${emitTrackData('cancel', '取消', { key: 'editor', name: names.title })}}\n      okTrack={${emitTrackData('confirm', '确定', { key: 'editor', name: names.title })}}`
+    : '';
   return `import Service from '@/services';
 import { ${components.join(', ')} } from '@ant-design/pro-components';
 import { CustomDrawer, HocModal } from '@mingfuyun/star-allinone';
 import { Form, message } from 'antd';
 import { useEffect, useState } from 'react';
+${trackImport}
 
 type ${names.drawer}Args = {
   record?: ${itemType};
@@ -397,7 +430,7 @@ ${createCall}
       loading={loading}
       open={visible}
       title={editing ? '编辑${names.title}' : '新增${names.title}'}
-      width={480}
+      width={480}${drawerTracks}
       onClose={handleCancel}
       onConfirm={handleConfirm}
     >
@@ -417,10 +450,17 @@ export default HocModal(${names.drawer});
 `;
 }
 
-function emitConfirm(names) {
+function emitConfirm(names, tracking, trackingModule) {
+  const trackImport = tracking
+    ? `import { getTrackData } from '${trackingModule}';\n`
+    : '';
+  const modalTracks = tracking
+    ? `\n      cancelButtonProps={{\n        'data-track': ${emitTrackData('cancel', '取消', { key: 'confirm', name: '确认' })},\n      }}\n      okButtonProps={{\n        'data-track': ${emitTrackData('confirm', '确定', { key: 'confirm', name: '确认' })},\n      }}`
+    : '';
   return `import { BaseModal, HocModal } from '@mingfuyun/star-allinone';
 import { Typography } from 'antd';
 import { useState } from 'react';
+${trackImport}
 
 export type ${names.confirm}Args = {
   title: string;
@@ -466,7 +506,7 @@ const ${names.confirm}: React.FC<Props> = ({
       maskClosable={false}
       title={title}
       visible={visible}
-      width={480}
+      width={480}${modalTracks}
       onCancel={handleCancel}
       onOk={handleOk}
     >
@@ -613,10 +653,16 @@ function generatePage(cwd, options) {
   };
 
   const queryNames = querySearchNames(operationByKey.list?.queryParams);
+  const tracking = !options.noTracking && hasTracking(project.root, project.config);
+  const trackingModule = trackingImport(project.config);
   writeFile(files[1], emitLess(layout), project.dryRun);
-  writeFile(files[2], emitColumns(names, types.itemType, fields, permissions, queryNames), project.dryRun);
-  writeFile(files[3], emitDrawer(names, types, fns, formFields), project.dryRun);
-  writeFile(files[4], emitConfirm(names), project.dryRun);
+  writeFile(
+    files[2],
+    emitColumns(names, types.itemType, fields, permissions, queryNames, tracking, trackingModule),
+    project.dryRun,
+  );
+  writeFile(files[3], emitDrawer(names, types, fns, formFields, tracking, trackingModule), project.dryRun);
+  writeFile(files[4], emitConfirm(names, tracking, trackingModule), project.dryRun);
   if (layout.hasContainerConfig) {
     writeFile(files[0], emitContainerIndex(), project.dryRun);
     writeFile(files[5], emitIndex(names, layout, types, fns, pagination, 'list'), project.dryRun);
@@ -631,4 +677,4 @@ function generatePage(cwd, options) {
   return names;
 }
 
-module.exports = { generatePage, featureNames, patchContainerConfig };
+module.exports = { generatePage, featureNames, patchContainerConfig, hasTracking };

package/skills/README.md

@@ -134,11 +134,11 @@ git commit -m "chore(skill): 更新公共 skill"
 | 入口 | 规则位置 | 覆盖内容 |
 | --- | --- | --- |
 | `shared-api-to-service` | SKILL + `references/service-conventions.md`、`api-artifact-generation.md`、`mock.md` | 指定 operation、`MFRequest`、namespace 类型、snake_case、mock |
-| `shared-design-to-page` | SKILL + `page-conventions.md`、`overlays.md`、`list-page-template.md` | 页面分层、标准列表 CLI、弹层合同 |
-| `shared-review-and-submit` | SKILL + `impact-analysis.md`、`verify.md`、`ai-review.md`、`commit.md` | 影响分析 → 验证 → 确认后提交 |
+| `shared-design-to-page` | SKILL + `page-conventions.md`、`overlays.md`、`list-page-template.md`、`tracking.md` | 页面分层、标准列表 CLI、弹层合同、按钮埋点 |
+| `shared-review-and-submit` | SKILL + `impact-analysis.md`、`verify.md`、`ai-review.md`、`commit.md` | 影响分析 → 验证（含埋点完整性）→ 确认后提交 |
 | `shared-publish-test` | SKILL + `scripts/publish-current-branch.sh` | 云效测发；只读 `publishPipelineId` |
 | `shared-tracking` | SKILL | 埋点接入：能力包版本、配置文件、脚本、初始化、失败上报、权限按钮、操作列、抽屉弹层、表单按钮 |
-| （非入口）`shared-project-conventions` | `architecture.md`、`mfprotable.md`、`component-contract.md`、`review-checklist.md`、`quality-checklist.md` | 架构、导入合同、质量清单 |
+| （非入口）`shared-project-conventions` | `architecture.md`、`mfprotable.md`、`component-contract.md`、`review-checklist.md`（含埋点审查）、`quality-checklist.md` | 架构、导入合同、质量清单 |
 | （非入口）`shared-i-have-adhd` | `i-have-adhd.md` | 用户输出格式 |
 
 以下内容**不**写进公共 skill 源码：

package/skills/shared/design-to-page/SKILL.md

@@ -15,6 +15,7 @@ Before implementation, read:
 2. [references/page-conventions.md](references/page-conventions.md)
 3. [references/overlays.md](references/overlays.md) for any modal, drawer, or confirmation
 4. `.agents/skills/shared-project-conventions/references/architecture.md`
+5. [references/tracking.md](references/tracking.md) when `src/utils/tracking/index.ts` exists (or the project tracking path in conventions)
 
 If `.agents/project/conventions.md` exists, use it only for extra paths.
 
@@ -51,6 +52,7 @@ Add `--permission-create`, `--permission-edit`, `--permission-delete` when codes
 3. The CLI fills columns and drawer fields from the OpenAPI schema (required, enum → `ProFormSelect`, dates/numbers). Review labels and permission codes; do not recreate the file tree. See [references/list-page-template.md](references/list-page-template.md).
 4. If `src/components/Page/Container/config.tsx` exists, `gen page` writes a Container `index.tsx`, puts the table in `components/List.tsx`, and registers a Tab with `loader: () => import(...)`. Do not invent a second page shell.
 5. Do not overwrite existing page files. Tell the user the CLI refused, then edit in place.
+6. **Tracking:** When the project has tracking configured, `gen page` auto-wires default button tracking. Agent must scan and wire **all** in-scope buttons per [references/tracking.md](references/tracking.md) and `$shared-tracking` (permission, drawer/modal footers, action column, ProForm, copy)—do not leave manual steps for the user. If tracking is not set up yet, tell the user to run `$shared-tracking` first; do not invent a second tracking helper.
 
 ## Design path
 
@@ -63,6 +65,7 @@ For non-standard layouts:
 5. API-backed pages include types, `MFRequest` methods, aggregate exports, and page consumption in the same change. Use `.agents/skills/shared-api-to-service/SKILL.md` for missing services. Use `.agents/skills/shared-api-to-service/references/mock.md` for extra mock scenarios. Do not embed production fake data.
 6. Compare desktop and narrow layouts against the design. Fix structure and spacing before decoration.
 7. Run focused verification. Do not claim visual fidelity without inspecting the rendered page.
+8. When tracking is enabled, Agent applies [references/tracking.md](references/tracking.md) for every toolbar, action, copy, drawer, confirm, form submit, and standalone permission button on the page.
 
 ## Hard rules
 
@@ -76,6 +79,7 @@ For non-standard layouts:
 - Do not use `Modal.confirm` or feature-level Ant Design `Modal` / `Drawer` shells.
 - Do not add a UI library, state library, or CSS-in-JS system without approval.
 - Do not refactor unrelated code.
+- When `src/utils/tracking/index.ts` exists, Agent wires `getTrackData` on every button covered by [references/tracking.md](references/tracking.md) (including copy). Do not call `track()` in handlers. Do not ask the user to add tracks manually.
 
 ## Verification
 
@@ -86,6 +90,8 @@ pnpm typecheck
 
 Run `pnpm lint` for broad or shared changes. Run the project's documented build command when routes, entry, or multiple pages change.
 
+When tracking is configured, scan changed page files: toolbar `track`, action `track`, copy `track`, drawer `cancelTrack` / `okTrack` or custom-footer `data-track`, confirm modal buttons, ProForm submitters, and standalone permission buttons. Agent fixes missing wiring before calling the page done; do not defer to the user.
+
 ## User-facing output
 
 Read and apply `.agents/skills/shared-i-have-adhd/i-have-adhd.md` to every response this skill sends to the user.

package/skills/shared/design-to-page/references/list-page-template.md

@@ -27,6 +27,7 @@ When `src/components/Page/Container/config.tsx` exists, `index.tsx` is a Contain
 - Export `columns`, `getActionColumn`, `getToolBarRender`.
 - Columns come from the list item schema: types, enums, and the first string fields as search.
 - Put permission strings on actions and toolbar items when the project uses them.
+- When tracking is configured, add `track: getTrackData(...)` beside each toolbar and action item. See [tracking.md](tracking.md).
 - Render empty cells as `'-'`.
 
 ## Drawer
@@ -36,12 +37,14 @@ When `src/components/Page/Container/config.tsx` exists, `index.tsx` is a Contain
 - Fields come from the create/update body schema: required rules, `enum` → `ProFormSelect`, numbers → `ProFormDigit`, dates → date pickers.
 - `validateFields()` in `try`. Create vs update from `record`.
 - Reset fields when `visible` becomes true.
+- When tracking is configured, set `cancelTrack` and `okTrack` on `CustomDrawer` (default footer). Do not add duplicate submit tracks on `ProForm`.
 
 ## Confirm modal
 
 - `BaseModal` + `HocModal`.
 - `confirmLoading` tied to the async `onConfirm`.
 - Do not use `Modal.confirm`.
+- When tracking is configured, set `cancelButtonProps` and `okButtonProps` with `'data-track'` from `getTrackData`. See [tracking.md](tracking.md).
 
 ## Container tabs
 

package/skills/shared/design-to-page/references/overlays.md

@@ -41,6 +41,8 @@ Do not duplicate open/visible, selected-record, and close state in the page when
 
 Use `CustomDrawer` confirmation controls and disable the ProForm submitter. Reset or initialize fields whenever the overlay becomes visible.
 
+When the project has tracking, wire `cancelTrack` / `okTrack` on the default drawer footer, or `data-track` on each button in a custom footer. For `BaseModal`, use `cancelButtonProps` / `okButtonProps`. Details: [tracking.md](tracking.md).
+
 ```tsx
 const handleConfirm = async () => {
   try {

package/skills/shared/design-to-page/references/page-conventions.md

@@ -62,6 +62,10 @@ Follow [overlays.md](overlays.md). Use `CustomDrawer`, `BaseModal`, and `HocModa
 - Do not treat hidden buttons as authorization. Backend enforcement must remain.
 - Add explicit routes in `config/route.ts` when the page needs navigation. Preserve `/403`, `/404`, and nested menu behavior.
 
+## Tracking
+
+When `src/utils/tracking/index.ts` exists, every new or changed page wires button tracking in the same PR. Follow [tracking.md](tracking.md) and `.agents/skills/shared-tracking/SKILL.md` for SDK setup. Use `getTrackData`; do not call `track()` in handlers.
+
 ## Async behavior
 
 - Every `await` lives in `try-catch`, including handlers, loaders, and submissions.

package/skills/shared/design-to-page/references/tracking.md

@@ -0,0 +1,63 @@
+# Page tracking
+
+Apply when the business project already has `src/utils/tracking/index.ts` (or the path documented in project conventions). Full SDK setup and **Agent 全站批量补齐规则** live in `.agents/skills/shared-tracking/SKILL.md`. This file covers **page / 生成场景**。
+
+**Agent 规则：** 生成或修改页面时，由 Agent 写入全部按钮埋点，**不要**留给用户手工补。与「接入埋点」同一套五类扫描标准（含复制按钮）。
+
+Import:
+
+```tsx
+import { getTrackData } from '@/utils/tracking';
+```
+
+Click reporting uses `data-track` JSON from `getTrackData`. Do not call `track()` in business handlers.
+
+## Required on every new or changed page
+
+| Surface | Where | Pattern |
+| --- | --- | --- |
+| Toolbar / batch / search extras | `const.tsx` or page `toolBarRender` | `track: getTrackData(..., { key: 'toolbar', name: '工具栏' })` |
+| Permission buttons | `withAuth(Button)` / `AccessButton` | `data-track={getTrackData(...)}` |
+| Row actions | `const.tsx` → `actionColumn` / `TablePermissionActions` | `track: getTrackData(..., { key: 'action', name: '操作' })` |
+| Copy actions | action column / toolbar / custom footer | `getTrackData('copy', '复制', …)` or `复制IP` |
+| Drawer default footer | `<Feature>Drawer.tsx` | `cancelTrack` / `okTrack` on `CustomDrawer` |
+| Drawer / Modal custom footer | each `Button` | `data-track={getTrackData(...)}` |
+| Confirm dialog | `BaseModal` | custom footer + `data-track` when button props unavailable |
+| Inline form submit | `ProForm` / `Form` | `submitButtonProps` / `resetButtonProps` |
+
+`MFProTable` query and reset are handled inside the table (`filter_query` / `查询`, `filter_reset` / `重置`). Do not duplicate in page code.
+
+## Drawer with `submitter={false}`
+
+Submit lives on `CustomDrawer`. Do not add a second submit track on `ProForm`.
+
+```tsx
+<CustomDrawer
+  cancelTrack={getTrackData('cancel', '取消', { key: 'editor', name: '<页面中文名>' })}
+  okTrack={getTrackData('confirm', '确定', { key: 'editor', name: '<页面中文名>' })}
+  onClose={handleCancel}
+  onConfirm={handleConfirm}
+/>
+```
+
+## Custom footer
+
+Every business `Button` in custom `footer` needs `data-track`. Do not replace the default footer only to add tracking.
+
+## Do not track
+
+- Column settings only.
+- Table overflow 「更多」 ellipsis icon (menu items inside still need `track`).
+- Buttons hidden by permission (no render → no track).
+
+**Copy buttons must be tracked** — see shared-tracking §5.
+
+## After `gen page`
+
+When `src/utils/tracking/index.ts` exists, `gen page` wires toolbar, action edit/delete, drawer, and modal by default. Agent still reviews and adds tracking for extra buttons (copy, batch, custom footer, ProForm). Pass `--no-tracking` only when user explicitly asks.
+
+For pages **not** from `gen page`, Agent scans and wires the full set per shared-tracking — not a user manual task.
+
+## Verification
+
+Static scan changed files: every toolbar / action / copy / drawer / modal / form button has `track`, `cancelTrack`, `okTrack`, or `data-track`.

package/skills/shared/project-conventions/references/review-checklist.md

@@ -26,6 +26,19 @@ This checklist is complete for shared review and commit. If `.agents/project/rev
 - Check loading, empty, error, disabled, long content, and narrow-screen layout.
 - Table empty values render `'-'` unless the design requires another value.
 
+## Tracking
+
+Apply when the project has `src/utils/tracking/index.ts` (or the path in project conventions). Read `.agents/skills/shared-tracking/SKILL.md` and `.agents/skills/shared-design-to-page/references/tracking.md`.
+
+- Toolbar and batch actions in `getToolBarRender` need `track: getTrackData(...)`.
+- Action column entries need `track` beside `permission` when present.
+- Standalone permission buttons need `data-track={getTrackData(...)}`.
+- `CustomDrawer` with the default footer needs `cancelTrack` and `okTrack`; custom footers need `data-track` on each business button.
+- `BaseModal` confirm dialogs need `cancelButtonProps` / `okButtonProps` with `'data-track'`.
+- `ProForm` with a visible default submitter needs submit/reset button props; when `submitter={false}`, tracking belongs on the overlay footer only.
+- Require tracking on copy actions; do not require tracking on column settings, table overflow ellipsis icon, or buttons hidden by permission.
+- Missing in-scope button tracking on a changed page is **P1**.
+
 ## Shared components and regression
 
 - Changes to `MFProTable`, permission components, the request wrapper, or shared hooks must list direct callers.

package/skills/shared/review-and-submit/SKILL.md

@@ -30,8 +30,9 @@ Follow [references/impact-analysis.md](references/impact-analysis.md). Output:
 Follow [references/verify.md](references/verify.md). Output:
 
 - P0/P1/P2 blockers
-- Completeness of route, permission, API, types, mock, and page states
+- Completeness of route, permission, API, types, mock, page states, and button tracking when the project has tracking configured
 - Async, loading, overlay, and list-reload behavior
+- Missing `track` / `data-track` / `cancelTrack` / `okTrack` on in-scope buttons per `shared-tracking` and `shared-design-to-page/references/tracking.md` (P1 when tracking is enabled)
 - Prettier, TypeScript, lint, and build results
 - Uncovered risks
 

package/skills/shared/review-and-submit/references/ai-review.md

@@ -6,6 +6,7 @@ Review the current diff plus relevant callers and types. Before reviewing, read:
 
 1. `.agents/skills/shared-project-conventions/references/review-checklist.md`
 2. `.agents/skills/shared-project-conventions/references/quality-checklist.md`
+3. `.agents/skills/shared-tracking/SKILL.md` and `.agents/skills/shared-design-to-page/references/tracking.md` when the project has tracking configured
 
 If `.agents/project/review-rules.md` exists, apply extra domain checks only; do not weaken shared hard rules.
 
@@ -14,14 +15,15 @@ If `.agents/project/review-rules.md` exists, apply extra domain checks only; do
 1. React Hooks: stale closures, incomplete dependencies, request races, updates after unmount, unnecessary rerenders.
 2. API contracts: method/path/params, response unwrapping, pagination, nullability, enum drift, error handling, DTO mismatch.
 3. Permissions: route access, button visibility, mutation authorization, client-only enforcement.
-4. Forms/tables: validation, reset, duplicate submit, refresh after mutation, column-state, long text, empty/error.
-5. Performance (hard checks):
+4. **Tracking** (when `src/utils/tracking/index.ts` or project equivalent exists): scan changed pages for missing `track` / `cancelTrack` / `okTrack` / `data-track` on in-scope buttons—toolbar, action column, permission buttons, drawer and confirm footers (default and custom), inline form submitters, and **copy** actions. Flag missing wiring as **P1**. Do not require tracking on column settings, table overflow ellipsis icon, or permission-hidden buttons. Do not flag handlers that call `track()` when `data-track` is the project standard.
+5. Forms/tables: validation, reset, duplicate submit, refresh after mutation, column-state, long text, empty/error.
+6. Performance (hard checks):
    - New or changed files must not statically `import 'xlsx'`, `@uiw/react-codemirror`, `@codemirror/*`, `echarts`, `@ant-design/charts`, or `react-json-view` at module top level. Those must be `import()`.
    - Do not add heavy modules to `src/app.ts`, `src/global.tsx`, or layout bootstrap.
    - Flag generated list pages that pull Excel/editor/chart code into the route entry instead of a feature-level dynamic import.
    - Also check duplicate libraries, large lists without virtualization, unstable columns/handlers, and avoidable requests.
-6. Security/privacy: tokens, user data, URLs, unsafe HTML, sensitive logs, environment config.
-7. Tests and accessibility: critical paths, failure states, keyboard labels, disabled/loading semantics.
+7. Security/privacy: tokens, user data, URLs, unsafe HTML, sensitive logs, environment config.
+8. Tests and accessibility: critical paths, failure states, keyboard labels, disabled/loading semantics.
 
 ## Output
 

package/skills/shared/review-and-submit/references/verify.md

@@ -6,6 +6,7 @@ Read the current diff, related callers, and:
 
 1. `.agents/skills/shared-project-conventions/references/review-checklist.md`
 2. `.agents/skills/shared-project-conventions/references/architecture.md`
+3. `.agents/skills/shared-tracking/SKILL.md` when the project has `src/utils/tracking/index.ts` (or the tracking path in project conventions)
 
 Do not auto-fix. If `.agents/project/review-rules.md` exists, apply extra domain checks only.
 
@@ -18,6 +19,7 @@ Do not auto-fix. If `.agents/project/review-rules.md` exists, apply extra domain
 5. Empty table values, duplicate-submit guards, permission wrappers, and narrow-screen overflow.
 6. Heavy deps (`xlsx`, editors, charts) are `import()`; nothing new lands in `src/app.ts`.
 7. No `MFTablePro` / table-from-package imports; overlays come from `@mingfuyun/star-allinone`.
+8. **Tracking** (when tracking is configured): every in-scope button has `track`, `cancelTrack`, `okTrack`, or `data-track` per `.agents/skills/shared-tracking/SKILL.md` and `.agents/skills/shared-design-to-page/references/tracking.md`. Check toolbar items, action column entries, standalone permission buttons, drawer default or custom footers, confirm modals, inline `ProForm` submitters, and **copy** buttons. Missing wiring is **P1** when the button is user-visible and in scope. Excluded controls (column settings, table overflow ellipsis icon) must not be flagged. Permission-hidden buttons may omit tracking.
 
 ## Commands
 
@@ -36,7 +38,7 @@ Run the project's build script when routes, entry, or shared runtime change.
 ## Output
 
 1. Blocking issues P0–P2, with file, line, impact, and fix direction.
-2. Completeness: route, permission, API, types, mock, page states.
+2. Completeness: route, permission, API, types, mock, page states, and tracking on in-scope buttons when tracking is configured.
 3. Command results, distinguishing new, existing, and environment failures.
 4. Regression list: affected pages, suggested manual paths, uncovered risks.
 

package/skills/shared/tracking/SKILL.md

@@ -9,6 +9,56 @@ description: User-facing. Connect a galaxy admin app to the shared tracking SDK.
 
 先确认埋点平台已登记本项目的 `appId`。测试用 `test-<project>`，生产用 `<project>`。不一致时只改配置，不改包。
 
+## Agent 必须做的事（不要留给用户手工补）
+
+用户说「接入埋点」「补齐埋点」「全部补齐」时，**Agent 负责扫描业务仓库并写入全部按钮埋点**。不要把 wiring 清单丢给用户，不要说「请你在各页 const.tsx 里手动加 track」。
+
+完成标准：
+
+1. 基础设施（SDK、`tracking/index.ts`、`app.tsx`、请求封装）已就绪。
+2. **全站**下列五类可点击按钮均已带 `track` / `cancelTrack` / `okTrack` / `data-track`（无权限未渲染的除外）。
+3. `pnpm typecheck` 通过。
+4. 向用户汇报已覆盖的文件范围，而不是留 TODO 让用户自己改。
+
+### 必须扫描并补齐的五类按钮
+
+| # | 范围 | 扫描位置 | 写法 |
+| --- | --- | --- | --- |
+| 1 | **权限相关按钮** | `withAuth` / `AccessButton`；`TableSearchType` 带 `permission`；`TablePermissionActions` | `data-track` 或 `track:` |
+| 2 | **CustomDrawer / HocModal** | 所有 `CustomDrawer`、`BaseModal`（含 `HocModal` 包裹） | 默认底部：`cancelTrack`/`okTrack`；**自定义 footer 每个 Button** 带 `data-track` |
+| 3 | **MFProTable 操作列** | `actionColumn`、`listActionColumn`、`TableActions`、`TablePermissionActions` | `ActionItem.track` |
+| 4 | **ProForm / Form** | `submitter` 默认栏、`submitter.render`、antd `Form` 提交/重置 | `submitButtonProps` / `resetButtonProps` 的 `data-track` |
+| 5 | **复制相关按钮** | 文案含「复制」「一键复制」、`key` 含 `copy`；含复制 IP | `getTrackData('copy', '…', …)` |
+
+另需覆盖（同属 Agent 批量任务）：
+
+- 工具栏 / 批量底部 / 搜索附加按钮（`toolBarRender`、`multiFooter`、`searchButton`）
+- `MFProTable` 查询/重置（表格组件内已处理则不要重复）
+- 抽屉 `submitter={false}` 时，提交在 Drawer 底部，不在 Form 重复
+
+### 扫描命令（Agent 自行执行）
+
+```bash
+rg "CustomDrawer|BaseModal|HocModal" src/pages src/components -g "*.tsx"
+rg "toolBarRender|actionColumn|listActionColumn|TablePermissionActions" src/pages -g "*.{tsx,ts}"
+rg "withAuth|AccessButton" src/pages -g "*.tsx"
+rg "ProForm|submitter" src/pages -g "*.tsx"
+rg "复制|copy|Copy" src/pages src/components -g "*.tsx"
+```
+
+对每个命中文件：**打开、判断是否在五类范围内、补全、保存**。历史页与 `gen page` 新页同一标准。
+
+### 不要做的「半套接入」
+
+- ❌ 只建 `tracking/index.ts` 和 SDK，不扫页面按钮
+- ❌ 只改一两个示例页，告诉用户「其余页按同样方式手动加」
+- ❌ 在组件层做 track 自动推断（不改 `withAuth` / `TableActions` 自动生成 track）
+- ❌ 把复制按钮排除在补齐范围外
+
+仍不上报：**列设置**、表格 **「更多」省略号**（仅展开菜单的 icon；菜单内各项仍要 track）。
+
+---
+
 ## 1. 更新能力包
 
 依赖至少 `0.4.0`。这一版才有 `createTracking`、`TRACKING_REPORT_URLS`，以及 `CustomDrawer` 的 `cancelTrack` / `okTrack`。
@@ -99,17 +149,15 @@ Umi：
 
 在项目自己的请求封装里，业务 `code` 非 0 或 HTTP 失败时调用 `reportApiTracking`。成功不报。字段：去掉 query 的 `url`、大写 `method`、`duration`、`success: false`、`status`、`errorMessage`。保留原来的 `message.error`。页面不要直接调 axios。
 
-### 按钮
+### 按钮写法（Agent 批量写入业务代码）
 
-`getTrackData(action, name, { key, name })` 生成 `data-track` JSON。`id` 用下划线，展示名用 `-` 连接。点击由 SDK 监听 `data-track`，业务代码不要再调 `track`。
+`getTrackData(action, name, { key, name })` 生成 `data-track` JSON。`id` 用下划线，展示名用 `-` 连接。点击由 SDK 监听 `data-track`，业务代码不要再调 `track()`。
 
-权限按钮、操作列、抽屉/弹层底部、表单提交都要上报。复制、列设置、表格「更多」省略号不上报。搜索栏里除查询和重置以外的按钮，除非用户点名，否则不加。
-
-#### 权限按钮
+#### 1. 权限相关按钮
 
 `withAuth` 会把除 `permission`、`fallback` 以外的 props 传给原按钮。无权限时组件不渲染，不会上报。
 
-工具栏、批量底部、搜索附加按钮已经走 `TableSearchType.track`，`MFProTable` 把它写成 `data-track`。在 `const.tsx` 里和 `permission` 一起写：
+工具栏、批量底部、搜索附加按钮走 `TableSearchType.track`，`MFProTable` 把它写成 `data-track`：
 
 ```tsx
 {
@@ -122,7 +170,7 @@ Umi：
 }
 ```
 
-页面上单独的 `withAuth(Button)` 直接写 `data-track`，不要为了埋点再包一层：
+页面上单独的 `withAuth(Button)`：
 
 ```tsx
 <AccessButton
@@ -135,11 +183,13 @@ Umi：
 </AccessButton>
 ```
 
-#### CustomDrawer、HocModal
+`TablePermissionActions` 的 `actionList` 每项加 `track`。
+
+#### 2. CustomDrawer、HocModal
 
 `HocModal` 只控制显隐，不渲染按钮。埋点写在它包住的 `CustomDrawer` 或 `BaseModal` 上。
 
-`CustomDrawer` 默认底部用 `cancelTrack` 和 `okTrack`。不要为了埋点整段替换 `footer`。
+**默认底部** — 用 `cancelTrack` / `okTrack`，不要整段替换 `footer`：
 
 ```tsx
 <CustomDrawer
@@ -150,49 +200,33 @@ Umi：
 />
 ```
 
-自定义 `footer` 会换掉默认底部。每个业务按钮自己带 `data-track`，包括预览、上一步、取消、确定。
+**自定义 footer** — 每个业务 Button 带 `data-track`（取消、确定、关闭、预览、复制 IP 等）。
 
-`BaseModal` 用 antd 的按钮 props。自定义 `footer` 时同样把 `data-track` 写在自己的按钮上。
+`BaseModal` 若无 `cancelButtonProps` 类型，用**自定义 footer** + `data-track`。
 
-```tsx
-<BaseModal
-  cancelButtonProps={{
-    'data-track': getTrackData('cancel', '取消', { key: 'confirm', name: '确认' }),
-  }}
-  okButtonProps={{
-    'data-track': getTrackData('confirm', '确定', { key: 'confirm', name: '确认' }),
-  }}
-  onCancel={handleCancel}
-  onOk={handleOk}
-/>
-```
+#### 3. MFProTable 操作列
 
-#### MFProTable 操作列
-
-`actionColumn` 的配置在页面 `const.tsx`。给 `ActionItem` 增加 `track?: string`，和 `permission` 放在一起。模块固定 `{ key: 'action', name: '操作' }`，`action` 用该项的 `key`，`name` 用按钮文案。
+`actionColumn` 在页面 `const.tsx`。给 `ActionItem` 增加 `track`：
 
 ```tsx
 {
-  key: 'allocate',
-  permission: FULL_MENU.example.allocate,
-  track: getTrackData('allocate', '分配', { key: 'action', name: '操作' }),
-  getText: () => '分配',
-  onClick: handleAllocate,
+  key: 'edit',
+  permission: FULL_MENU.example.edit,
+  track: getTrackData('edit', '编辑', { key: 'action', name: '操作' }),
+  getText: () => '编辑',
+  onClick: handleEdit,
 }
 ```
 
-业务仓的 `TableActions` 要把 `track` 写到可点击节点上：
-
-- 行内 `<a>` 加 `data-track={action.track}`。
-- 超出 `maxCount` 收进下拉时，菜单项 `label` 用 `<span data-track={action.track}>`。不要给「更多」省略号按钮加埋点。
+业务仓 `TableActions` 已支持 `data-track={action.track}`。不要给「更多」省略号 icon 加埋点；下拉菜单内各项仍要 `track`。
 
-`MFProTable` 查询按钮用 `filter_query` / `查询`，重置按钮用 `filter_reset` / `重置`。ProTable 重置按钮的 key 是 `rest`。
+查询/重置：`MFProTable` 内为 `filter_query`/`查询`、`filter_reset`/`重置`（ProTable 重置 key 为 `rest`）。
 
-#### ProForm、Form
+#### 4. ProForm、Form
 
-抽屉里的 `ProForm` 若 `submitter={false}`，提交按钮在抽屉或弹层底部，按上一节上报，不要在表单里再加一套。
+抽屉内 `ProForm` 若 `submitter={false}`，提交在 `CustomDrawer` 底部，按上一节写，**不要在 Form 里重复**。
 
-`ProForm` 使用默认提交栏时，写在 `submitter` 上：
+默认提交栏：
 
 ```tsx
 <ProForm
@@ -207,9 +241,24 @@ Umi：
 />
 ```
 
-`submitter.render` 换掉默认按钮时，返回的每个提交、重置按钮自己带 `data-track`。
+`submitter.render` 或 antd `Form` 自定义按钮：每个提交/重置 Button 单独 `data-track`。
 
-antd `Form` 的提交、重置按钮同样写 `data-track`。只上报会触发提交或清空表单的按钮，不上报字段里的辅助按钮。
+#### 5. 复制相关按钮
+
+**必须上报**。含操作列「复制」、群看板「一键复制」、复制 IP 等。
+
+```tsx
+{
+  key: 'action-copy',
+  track: getTrackData('copy', '复制', { key: 'action', name: '操作' }),
+  getText: () => '复制',
+  onClick: handleCopy,
+}
+```
+
+```tsx
+track: getTrackData('copy', '复制IP', { key: 'toolbar', name: '群看板' })
+```
 
 ## 不要做的事
 
@@ -218,6 +267,8 @@ antd `Form` 的提交、重置按钮同样写 `data-track`。只上报会触发
 - 不要在包外重写队列。上限 50 条、失败补报已经在 `createTracking` 里。
 - 不要上报成功请求。
 - 不要为了埋点改用户接口或请求错误提示。
+- **不要**在组件层做 track 自动推断。
+- **不要**只完成基础设施就把按钮埋点留给用户。
 
 ## 验证
 
@@ -226,6 +277,6 @@ antd `Form` 的提交、重置按钮同样写 `data-track`。只上报会触发
 1. 页面请求了 OSS 上的脚本，并且出现 `window.TrackingSDK`。
 2. 进入业务页、初始状态完成后存在 `window.__tracker`。`/` 上可以还没有。
 3. 一次失败请求发出事件名 `api_request`。
-4. 查询、重置、权限按钮、操作列、抽屉或弹层底部、表单提交按钮带 `data-track`。无权限而不渲染的按钮可以没有。
+4. 五类按钮（含复制）带 `data-track` / `cancelTrack` / `okTrack`。
 
 `appId` 与平台不一致时，只改 `src/utils/tracking/index.ts`。