@mingfuyun/star-allinone 0.4.2 → 0.4.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mingfuyun/star-allinone",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "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. Review and extend per [references/tracking.md](references/tracking.md). Add tracking for any extra buttons the CLI did not generate. 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, apply [references/tracking.md](references/tracking.md) for every toolbar, action, drawer, confirm, 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, new or changed pages must wire `getTrackData` on every button covered by [references/tracking.md](references/tracking.md). Do not call `track()` in handlers.
 
 ## 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`, drawer `cancelTrack` / `okTrack` or custom-footer `data-track`, confirm modal button props, and standalone permission buttons. Report any missing wiring before calling the page done.
+
 ## 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,83 @@
+# Page tracking
+
+Apply when the business project already has `src/utils/tracking/index.ts` (or the path documented in project conventions). Full SDK setup lives in `.agents/skills/shared-tracking/SKILL.md`. This file covers **page generation only**.
+
+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 page
+
+| Surface | Where | Pattern |
+| --- | --- | --- |
+| Toolbar create / batch actions | `const.tsx` → `getToolBarRender` | `track: getTrackData('<action>', '<文案>', { key: 'toolbar', name: '工具栏' })` |
+| Row actions | `const.tsx` → `getActionColumn` | `track: getTrackData('<action>', '<文案>', { key: 'action', name: '操作' })` |
+| Standalone permission buttons | page or `const.tsx` | `data-track={getTrackData(...)}` on the button |
+| Drawer footer | `<Feature>Drawer.tsx` | `cancelTrack` / `okTrack` on `CustomDrawer` when using the default footer |
+| Confirm dialog | `<Feature>ConfirmModal.tsx` | `cancelButtonProps` / `okButtonProps` with `'data-track'` on `BaseModal` |
+| Inline form submit | `ProForm` with default submitter | `submitButtonProps` / `resetButtonProps` |
+
+`MFProTable` query and reset are handled inside the table (`filter_query` / `查询`, `filter_reset` / `重置`). Do not duplicate them in page code unless the project documents otherwise.
+
+## 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}
+/>
+```
+
+Use the page menu title or feature name for `name`. Keep `key: 'editor'` for create/edit drawers.
+
+## Confirm modal
+
+```tsx
+<BaseModal
+  cancelButtonProps={{
+    'data-track': getTrackData('cancel', '取消', { key: 'confirm', name: '确认' }),
+  }}
+  okButtonProps={{
+    'data-track': getTrackData('confirm', '确定', { key: 'confirm', name: '确认' }),
+  }}
+  onCancel={handleCancel}
+  onOk={handleOk}
+/>
+```
+
+## Custom footer
+
+If `CustomDrawer` or `BaseModal` uses a custom `footer`, every business button in that footer needs its own `data-track`. Do not replace the default footer only to add tracking.
+
+## Do not track
+
+- Copy, column settings, table overflow 「更多」 ellipsis.
+- Search-bar buttons other than query and reset, unless the user explicitly asked.
+- Buttons hidden by permission (no render → no track).
+
+## After `gen page`
+
+When `src/utils/tracking/index.ts` exists, `gen page` already wires:
+
+- `track` on toolbar create and action edit/delete in `const.tsx`
+- `cancelTrack` / `okTrack` on `CustomDrawer`
+- `cancelButtonProps` / `okButtonProps` on `BaseModal`
+
+Review and adjust labels or module path (`trackingModule` in `.star-allinone.json` when non-default). Add tracking for any extra buttons the CLI did not generate. Pass `--no-tracking` to skip auto wiring.
+
+For pages built without `gen page`, follow the table above manually.
+
+## Verification
+
+Static scan changed page files for:
+
+- Every rendered toolbar / action / drawer / confirm button listed above has `track`, `cancelTrack`, `okTrack`, or `data-track`.
+- No `data-track` on excluded controls.

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.
+- Do not require tracking on copy, column settings, table overflow ellipsis, 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, inline form submitters. Flag missing wiring as **P1**. Do not require tracking on copy, column settings, table overflow ellipsis, 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, and inline `ProForm` submitters. Missing wiring is **P1** when the button is user-visible and in scope. Excluded controls (copy, column settings, table overflow ellipsis, non-default search buttons) 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.