@mingfuyun/star-allinone 0.3.1 → 0.3.3

package/scripts/gen/fields.js

@@ -0,0 +1,118 @@
+const { deref } = require('./spec');
+
+const SKIP_COLUMNS = new Set([
+  'created_at',
+  'updated_at',
+  'createTime',
+  'updateTime',
+  'deleted_at',
+]);
+
+const SKIP_FORM = new Set([
+  'id',
+  'pk',
+  'created_at',
+  'updated_at',
+  'createTime',
+  'updateTime',
+  'deleted_at',
+]);
+
+const SKIP_SEARCH = new Set([
+  'page',
+  'page_size',
+  'pageSize',
+  'current',
+  'limit',
+  'offset',
+  'ordering',
+  'sort',
+  'sort_by',
+]);
+
+function schemaType(schema) {
+  if (!schema) return undefined;
+  if (schema.enum?.length) return 'string';
+  const types = Array.isArray(schema.type)
+    ? schema.type.filter((item) => item !== 'null')
+    : [schema.type];
+  return types[0];
+}
+
+function fieldFromSchema(spec, name, schema, required) {
+  const resolved = deref(spec, schema) || {};
+  return {
+    name,
+    title: resolved.title || name,
+    required: Boolean(required),
+    type: schemaType(resolved),
+    format: resolved.format,
+    enum: Array.isArray(resolved.enum) ? resolved.enum : undefined,
+    readOnly: Boolean(resolved.readOnly),
+  };
+}
+
+function schemaFields(spec, schema, options = {}) {
+  if (!spec || !schema) return [];
+  const skip = options.skip || SKIP_COLUMNS;
+  const resolved = deref(spec, schema);
+  const properties = resolved?.properties || {};
+  const required = new Set(resolved?.required || []);
+  const fields = [];
+  for (const [name, prop] of Object.entries(properties)) {
+    if (skip.has(name)) continue;
+    const field = fieldFromSchema(spec, name, prop, required.has(name));
+    if (options.forForm && field.readOnly) continue;
+    fields.push(field);
+  }
+  if (typeof options.limit === 'number') return fields.slice(0, options.limit);
+  return fields;
+}
+
+function formComponent(field) {
+  if (field.enum?.length) return 'ProFormSelect';
+  if (field.type === 'boolean') return 'ProFormSwitch';
+  if (field.type === 'integer' || field.type === 'number') return 'ProFormDigit';
+  if (field.format === 'date-time') return 'ProFormDateTimePicker';
+  if (field.format === 'date') return 'ProFormDatePicker';
+  if (/desc|note|remark|comment|content|message/i.test(field.name)) return 'ProFormTextArea';
+  return 'ProFormText';
+}
+
+function columnValueType(field) {
+  if (field.enum?.length) return 'select';
+  if (field.type === 'boolean') return 'select';
+  if (field.type === 'integer' || field.type === 'number') return 'digit';
+  if (field.format === 'date-time') return 'dateTime';
+  if (field.format === 'date') return 'date';
+  return undefined;
+}
+
+function fallbackFields(itemTypeSource = '') {
+  const names = [...itemTypeSource.matchAll(/^\s+(\w+)\??:/gm)]
+    .map((match) => match[1])
+    .filter((name) => !SKIP_COLUMNS.has(name));
+  return names.slice(0, 6).map((name) => ({
+    name,
+    title: name,
+    required: false,
+    type: 'string',
+  }));
+}
+
+function querySearchNames(queryParams = []) {
+  return queryParams
+    .map((param) => param?.name)
+    .filter((name) => name && !SKIP_SEARCH.has(name));
+}
+
+module.exports = {
+  SKIP_COLUMNS,
+  SKIP_FORM,
+  SKIP_SEARCH,
+  schemaFields,
+  formComponent,
+  columnValueType,
+  fallbackFields,
+  querySearchNames,
+};

package/scripts/gen/fixtures/run.js

@@ -3,6 +3,7 @@ const os = require('os');
 const path = require('path');
 const { generateService } = require('../service');
 const { generatePage } = require('../page');
+const { runDoctor } = require('../doctor');
 
 function assert(condition, message) {
   if (!condition) throw new Error(message);
@@ -15,6 +16,7 @@ function makeProject() {
     JSON.stringify(
       {
         project: 'fixture',
+        publicSkillPrefix: 'shared-',
         serviceDir: 'src/services',
         openapi: 'openapi.json',
       },
@@ -45,10 +47,41 @@ 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.writeFileSync(path.join(root, 'package.json'), JSON.stringify({
+    dependencies: { '@mingfuyun/star-allinone': '^0.3.2' },
+    scripts: { typecheck: 'tsc --noEmit' },
+  }, null, 2));
   return root;
 }
 
+function writeContainerConfig(root) {
+  const dir = path.join(root, 'src/components/Page/Container');
+  fs.mkdirSync(dir, { recursive: true });
+  fs.writeFileSync(
+    path.join(dir, 'config.tsx'),
+    `export default {
+  '/existing': [
+    { key: 'tab-existing', label: 'Existing', children: <div /> },
+  ],
+};
+`,
+  );
+}
+
+function generateWidgetsPage(root, feature = 'Widgets', route = '/widgets') {
+  generatePage(root, {
+    feature,
+    route,
+    title: '组件管理',
+    listOperation: 'GET /api/v1/widgets/',
+    createOperation: 'POST /api/v1/widgets/',
+    updateOperation: 'PATCH /api/v1/widgets/{id}/',
+    deleteOperation: 'DELETE /api/v1/widgets/{id}/',
+  });
+}
+
 function main() {
   const root = makeProject();
   const first = generateService(root, {
@@ -78,21 +111,53 @@ function main() {
   assert(patched.includes('widgetsPartialUpdate'), patched);
   assert(patched.includes('MFRequest.patch<WIDGETS_API.Widget>'), patched);
 
-  generatePage(root, {
-    feature: 'Widgets',
-    route: '/widgets',
-    title: '组件管理',
-    listOperation: 'GET /api/v1/widgets/',
-    createOperation: 'POST /api/v1/widgets/',
-    updateOperation: 'PATCH /api/v1/widgets/{id}/',
-    deleteOperation: 'DELETE /api/v1/widgets/{id}/',
-  });
+  generateWidgetsPage(root);
   const page = fs.readFileSync(path.join(root, 'src/pages/Widgets/index.tsx'), 'utf8');
   assert(page.includes('Service.widgetsList'), page);
-  assert(page.includes("rowKey=\"id\""), page);
+  assert(page.includes('rowKey="id"'), page);
+  const columns = fs.readFileSync(path.join(root, 'src/pages/Widgets/const.tsx'), 'utf8');
+  assert(columns.includes("dataIndex: 'name'"), columns);
+  assert(columns.includes("dataIndex: 'kind'"), columns);
+  assert(columns.includes("valueType: 'select'"), columns);
+  assert(columns.includes("valueType: 'digit'"), 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);
   const route = fs.readFileSync(path.join(root, 'config/route.ts'), 'utf8');
   assert(route.includes("path: '/widgets'"), route);
+
+  const tabRoot = makeProject();
+  writeContainerConfig(tabRoot);
+  generateService(tabRoot, { operation: 'GET /api/v1/widgets/' });
+  generateWidgetsPage(tabRoot);
+  const shell = fs.readFileSync(path.join(tabRoot, 'src/pages/Widgets/index.tsx'), 'utf8');
+  assert(shell.includes("import Container from '@/components/Page/Container'"), shell);
+  const list = fs.readFileSync(path.join(tabRoot, 'src/pages/Widgets/components/List.tsx'), 'utf8');
+  assert(list.includes('Service.widgetsList'), list);
+  const tabConfig = fs.readFileSync(path.join(tabRoot, 'src/components/Page/Container/config.tsx'), 'utf8');
+  assert(tabConfig.includes("'/widgets'"), tabConfig);
+  assert(tabConfig.includes("@/pages/Widgets/components/List"), tabConfig);
+
+  const healthy = runDoctor(root);
+  assert(healthy.some((item) => item.message.includes('MFProTable')), JSON.stringify(healthy));
+  assert(!healthy.some((item) => item.level === 'error' && item.message.includes('MFRequest')), JSON.stringify(healthy));
+
+  fs.mkdirSync(path.join(root, 'src/pages/Bad'), { recursive: true });
+  fs.writeFileSync(
+    path.join(root, 'src/pages/Bad/index.tsx'),
+    "import { MFTablePro } from '@mingfuyun/star-allinone';\nimport CustomDrawer from '@/components/CustomDrawer';\n",
+  );
+  fs.mkdirSync(path.join(root, 'src/components/CustomDrawer'), { recursive: true });
+  fs.writeFileSync(path.join(root, 'src/components/CustomDrawer/index.tsx'), 'export default function CustomDrawer() { return null; }\n');
+  const sick = runDoctor(root);
+  assert(sick.some((item) => item.level === 'error' && /MFTablePro/.test(item.message)), JSON.stringify(sick));
+  assert(sick.some((item) => item.level === 'error' && /Local overlay/.test(item.message)), JSON.stringify(sick));
+
   fs.rmSync(root, { recursive: true, force: true });
+  fs.rmSync(tabRoot, { recursive: true, force: true });
+  process.exitCode = 0;
   console.log('codegen fixtures passed');
 }
 

package/scripts/gen/fixtures/widgets.openapi.json

@@ -165,6 +165,7 @@
         "properties": {
           "id": { "type": "string" },
           "name": { "type": "string" },
+          "kind": { "type": "string", "enum": ["gadget", "tool"] },
           "status": { "type": "string", "nullable": true },
           "count": { "type": "integer" }
         }
@@ -174,7 +175,7 @@
         "required": ["name"],
         "properties": {
           "name": { "type": "string" },
-          "status": { "type": "string" }
+          "status": { "type": "string", "enum": ["active", "inactive"] }
         }
       },
       "WidgetList": {

package/scripts/gen/page.js

@@ -1,8 +1,16 @@
 const fs = require('fs');
 const path = require('path');
-const { toPascal, parseOperationSelector } = require('./names');
+const { toPascal } = require('./names');
 const { generateService } = require('./service');
 const { loadProject, writeFile } = require('./project');
+const {
+  schemaFields,
+  formComponent,
+  columnValueType,
+  fallbackFields,
+  SKIP_FORM,
+  querySearchNames,
+} = require('./fields');
 
 function detectLayout(root) {
   const hasPageContainer = fs.existsSync(path.join(root, 'src/components/PageContainer/index.tsx'));
@@ -28,30 +36,39 @@ function featureNames(options) {
   };
 }
 
-function columnFields(itemTypeSource = '') {
-  const fields = [...itemTypeSource.matchAll(/^\s+(\w+)\??:/gm)].map((match) => match[1]);
-  const preferred = fields.filter((name) => !['created_at', 'updated_at', 'createTime', 'updateTime'].includes(name));
-  return (preferred.length ? preferred : fields).slice(0, 6);
+function searchFieldNames(fields) {
+  return fields
+    .filter((field) => field.name !== 'id' && (field.type === 'string' || !field.type))
+    .slice(0, 2)
+    .map((field) => field.name);
 }
 
-function emitColumns(names, itemType, fields, permissions) {
-  const columns = fields.map((field, index) => {
-    const search = index < 2;
+function emitColumns(names, itemType, fields, permissions, queryNames) {
+  const columnFields = fields.length ? fields : [{ name: 'id', title: 'ID', type: 'string' }];
+  const searchable = new Set(
+    queryNames?.length
+      ? columnFields.filter((field) => queryNames.includes(field.name)).map((field) => field.name)
+      : searchFieldNames(columnFields),
+  );
+  const columns = columnFields.map((field) => {
+    const search = searchable.has(field.name);
     return `  {
-    title: '${field}',
-    dataIndex: '${field}',
-    key: '${field}',
+    title: ${JSON.stringify(field.title)},
+    dataIndex: '${field.name}',
+    key: '${field.name}',
     hideInSearch: ${search ? 'false' : 'true'},
-    ellipsis: true,
-    render: (_, record) => record.${field} ?? '-',
+    ellipsis: true,${valueTypeBlock(field)}${enumBlock(field)}
+    render: (_, record) => record.${field.name} ?? '-',
   }`;
   });
-  if (!columns.length) {
+  for (const name of queryNames || []) {
+    if (columnFields.some((field) => field.name === name)) continue;
     columns.push(`  {
-    title: 'ID',
-    dataIndex: 'id',
-    key: 'id',
-    hideInSearch: true,
+    title: ${JSON.stringify(name)},
+    dataIndex: '${name}',
+    key: '${name}',
+    hideInTable: true,
+    hideInSearch: false,
   }`);
   }
   const permCreate = permissions.create ? `      permission: '${permissions.create}',\n` : '';
@@ -105,14 +122,47 @@ ${permCreate}    onClick: handleCreate,
 `;
 }
 
-function emitIndex(names, layout, types, fns, pagination) {
-  const pageImport = layout.hasPageContainer
+function valueTypeBlock(field) {
+  const valueType = columnValueType(field);
+  return valueType ? `\n    valueType: '${valueType}',` : '';
+}
+
+function enumBlock(field) {
+  if (field.enum?.length) {
+    const entries = field.enum
+      .map((value) => `      ${JSON.stringify(String(value))}: { text: ${JSON.stringify(String(value))} }`)
+      .join(',\n');
+    return `\n    valueEnum: {\n${entries},\n    },`;
+  }
+  if (field.type === 'boolean') {
+    return `\n    valueEnum: { true: { text: '是' }, false: { text: '否' } },`;
+  }
+  return '';
+}
+
+function emitContainerIndex() {
+  return `import Container from '@/components/Page/Container';
+
+export default () => {
+  return <Container />;
+};
+`;
+}
+
+function emitIndex(names, layout, types, fns, pagination, variant = 'page') {
+  const isList = variant === 'list';
+  const usePageContainer = layout.hasPageContainer && !layout.hasContainerConfig && !isList;
+  const pageImport = usePageContainer
     ? `import PageContainer from '@/components/PageContainer';\n`
     : '';
-  const open = layout.hasPageContainer
+  const overlayDir = isList ? './' : './components/';
+  const constImport = isList ? '../const' : './const';
+  const stylesImport = isList ? '../index.less' : './index.less';
+  const exportName = isList ? 'List' : names.leaf;
+  const open = usePageContainer
     ? `    <PageContainer header={{ title: undefined }}>`
     : '    <>';
-  const close = layout.hasPageContainer ? `    </PageContainer>` : '    </>';
+  const close = usePageContainer ? `    </PageContainer>` : '    </>';
   const listAccess = pagination === 'results-count'
     ? 'list: data?.results || [],\n          total: data?.count || 0,'
     : 'list: data?.list || [],\n          total: data?.total || 0,';
@@ -130,14 +180,14 @@ import { message } from 'antd';
 import { useCallback, useRef } from 'react';
 import ${names.confirm}, {
   type ${names.confirm}Ref,
-} from './components/${names.confirm}';
+} from '${overlayDir}${names.confirm}';
 import ${names.drawer}, {
   type ${names.drawer}Ref,
-} from './components/${names.drawer}';
-import { columns, getActionColumn, getToolBarRender } from './const';
-import styles from './index.less';
+} from '${overlayDir}${names.drawer}';
+import { columns, getActionColumn, getToolBarRender } from '${constImport}';
+import styles from '${stylesImport}';
 
-export default function ${names.leaf}() {
+export default function ${exportName}() {
   const tableRef = useRef<MFTableRef>(null);
   const drawerRef = useRef<${names.drawer}Ref>(null);
   const confirmRef = useRef<${names.confirm}Ref>(null);
@@ -224,8 +274,62 @@ function emitLess(layout) {
 `;
 }
 
-function emitDrawer(names, types, fns) {
+function emitFormItem(field) {
+  const component = formComponent(field);
+  const label = JSON.stringify(field.title);
+  const requiredVerb = component === 'ProFormSelect' ? '请选择' : '请输入';
+  const required = field.required
+    ? `\n          rules={[{ required: true, message: ${JSON.stringify(`${requiredVerb}${field.title}`)} }]}`
+    : '';
+  if (component === 'ProFormSelect') {
+    const options = (field.enum || [])
+      .map((value) => `{ label: ${JSON.stringify(String(value))}, value: ${JSON.stringify(value)} }`)
+      .join(', ');
+    return `        <ProFormSelect
+          name="${field.name}"
+          label={${label}}
+          options={[${options}]}${required}
+        />`;
+  }
+  if (component === 'ProFormSwitch') {
+    return `        <ProFormSwitch
+          name="${field.name}"
+          label={${label}}
+        />`;
+  }
+  if (component === 'ProFormDigit') {
+    return `        <ProFormDigit
+          name="${field.name}"
+          label={${label}}
+          placeholder={${JSON.stringify(`请输入${field.title}`)}}${required}
+        />`;
+  }
+  if (component === 'ProFormDateTimePicker' || component === 'ProFormDatePicker') {
+    return `        <${component}
+          name="${field.name}"
+          label={${label}}${required}
+        />`;
+  }
+  if (component === 'ProFormTextArea') {
+    return `        <ProFormTextArea
+          name="${field.name}"
+          label={${label}}
+          placeholder={${JSON.stringify(`请输入${field.title}`)}}${required}
+        />`;
+  }
+  return `        <ProFormText
+          name="${field.name}"
+          label={${label}}
+          placeholder={${JSON.stringify(`请输入${field.title}`)}}${required}
+        />`;
+}
+
+function emitDrawer(names, types, fns, formFields) {
   const itemType = types.itemType || 'Record<string, unknown>';
+  const fields = formFields.length
+    ? formFields
+    : [{ name: 'name', title: '名称', required: true, type: 'string' }];
+  const components = ['ProForm', ...new Set(fields.map(formComponent))];
   const createCall = fns.create
     ? `        await Service.${fns.create}(values);`
     : '        // TODO: call create service';
@@ -233,7 +337,7 @@ function emitDrawer(names, types, fns) {
     ? `        await Service.${fns.update}(record.id, values);`
     : '        // TODO: call update service';
   return `import Service from '@/services';
-import { ProForm, ProFormText } from '@ant-design/pro-components';
+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';
@@ -303,12 +407,7 @@ ${createCall}
         layout="horizontal"
         submitter={false}
       >
-        <ProFormText
-          name="name"
-          label="名称"
-          placeholder="请输入名称"
-          rules={[{ required: true, message: '请输入名称' }]}
-        />
+${fields.map(emitFormItem).join('\n')}
       </ProForm>
     </CustomDrawer>
   );
@@ -407,6 +506,49 @@ function patchRoute(root, names, dryRun) {
   writeFile(file, next, dryRun);
 }
 
+function insertImport(source, line) {
+  if (source.includes(line.trim())) return source;
+  const matches = [...source.matchAll(/^import .*;\n/gm)];
+  if (!matches.length) return `${line}${source}`;
+  const last = matches[matches.length - 1];
+  const index = last.index + last[0].length;
+  return `${source.slice(0, index)}${line}${source.slice(index)}`;
+}
+
+function patchContainerConfig(root, names, dryRun) {
+  const file = path.join(root, 'src/components/Page/Container/config.tsx');
+  if (!fs.existsSync(file)) return false;
+  let current = fs.readFileSync(file, 'utf8');
+  const importPath = `@/pages/${names.featurePath}/components/List`;
+  const hasRoute = current.includes(`'${names.route}'`) || current.includes(`"${names.route}"`);
+  if (current.includes(importPath) && hasRoute) {
+    console.log(`tab container already has ${names.route}`);
+    return false;
+  }
+  current = insertImport(current, `import ${names.leaf} from '${importPath}';\n`);
+  if (!hasRoute) {
+    const entry = `  '${names.route}': [
+    {
+      key: 'tab-${names.leaf}',
+      label: ${JSON.stringify(names.title)},
+      children: <${names.leaf} />,
+    },
+  ],
+`;
+    if (/export default \{\r?\n/.test(current)) {
+      current = current.replace(/export default \{\r?\n/, (match) => `${match}${entry}`);
+    } else if (current.includes('export default {')) {
+      current = current.replace('export default {', `export default {\n${entry}`);
+    } else {
+      console.log('Could not patch src/components/Page/Container/config.tsx; register the tab manually.');
+      return false;
+    }
+  }
+  writeFile(file, current, dryRun);
+  console.log(`${dryRun ? '[dry-run] ' : ''}registered tab ${names.route} in src/components/Page/Container/config.tsx`);
+  return true;
+}
+
 function generatePage(cwd, options) {
   const operations = [
     ['list', options.listOperation],
@@ -429,6 +571,9 @@ function generatePage(cwd, options) {
     path.join(pageDir, 'components', `${names.drawer}.tsx`),
     path.join(pageDir, 'components', `${names.confirm}.tsx`),
   ];
+  if (layout.hasContainerConfig) {
+    files.push(path.join(pageDir, 'components', 'List.tsx'));
+  }
   const exists = files.filter((file) => fs.existsSync(file));
   if (exists.length) {
     throw new Error(`Refusing to overwrite existing page files:\n${exists.map((file) => path.relative(project.root, file)).join('\n')}`);
@@ -438,6 +583,7 @@ function generatePage(cwd, options) {
   let types = { itemType: `${toPascal(names.leaf)}_API.${names.leaf}Item`, queryType: undefined };
   let pagination = 'list-total';
   let itemSource = '';
+  const operationByKey = {};
 
   for (const [key, selector] of operations) {
     const result = generateService(cwd, {
@@ -446,6 +592,7 @@ function generatePage(cwd, options) {
       mockFile: false,
     });
     fns[key] = result.fn;
+    operationByKey[key] = result.operation;
     if (key === 'list') {
       types = {
         itemType: result.itemType || types.itemType,
@@ -459,25 +606,37 @@ function generatePage(cwd, options) {
     }
   }
 
-  const fields = columnFields(itemSource);
+  const spec = project.spec;
+  let fields = schemaFields(spec, operationByKey.list?.itemSchema, { limit: 8 });
+  if (!fields.length) fields = fallbackFields(itemSource);
+  const formFields = schemaFields(
+    spec,
+    operationByKey.create?.bodySchema || operationByKey.update?.bodySchema || operationByKey.list?.itemSchema,
+    { skip: SKIP_FORM, forForm: true },
+  );
   const permissions = {
     create: options.permissionCreate,
     edit: options.permissionEdit,
     delete: options.permissionDelete,
   };
 
-  writeFile(files[0], emitIndex(names, layout, types, fns, pagination), project.dryRun);
+  const queryNames = querySearchNames(operationByKey.list?.queryParams);
   writeFile(files[1], emitLess(layout), project.dryRun);
-  writeFile(files[2], emitColumns(names, types.itemType, fields, permissions), project.dryRun);
-  writeFile(files[3], emitDrawer(names, types, fns), 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);
-  patchRoute(project.root, names, project.dryRun);
-
   if (layout.hasContainerConfig) {
-    console.log(`starscope-style tab container detected. Register ${names.featurePath} in src/components/Page/Container/config.tsx if this page should appear as a tab.`);
+    writeFile(files[0], emitContainerIndex(), project.dryRun);
+    writeFile(files[5], emitIndex(names, layout, types, fns, pagination, 'list'), project.dryRun);
+    patchRoute(project.root, names, project.dryRun);
+    patchContainerConfig(project.root, names, project.dryRun);
+  } else {
+    writeFile(files[0], emitIndex(names, layout, types, fns, pagination), project.dryRun);
+    patchRoute(project.root, names, project.dryRun);
   }
+
   console.log(`${project.dryRun ? '[dry-run] ' : ''}generated page ${names.route} -> src/pages/${names.featurePath}`);
   return names;
 }
 
-module.exports = { generatePage, featureNames };
+module.exports = { generatePage, featureNames, patchContainerConfig };

package/skills/README.md

@@ -46,11 +46,13 @@ Cursor / Codex 的常驻说明只放仓库根 `AGENTS.md`（模板见 `examples/
         ├── design-to-page/          # 页面；标准列表内部跑 gen page
         ├── review-and-submit/       # 审查并提交；步骤在 references/
         ├── publish-test/            # 测发
-        ├── project-conventions/     # architecture / review-checklist / quality-checklist
+        ├── project-conventions/     # architecture / component-contract / mfprotable / review-checklist / quality-checklist
         └── i-have-adhd/             # 用户输出格式
 ```
 
-公共入口写死两边已对齐的硬规则，例如：页面走 `MFRequest`、DTO 放 namespace typings、`MFProTable` + `ProForm`、弹层用 `BaseModal` / `CustomDrawer` / `HocModal`、`await` 必须 try/catch、禁止无说明的 `any`。
+公共入口写死两边已对齐的硬规则，例如：页面走 `MFRequest`、DTO 放 namespace typings、表格用业务仓 `@/components/CommonTable/MFProTable`（不要从包里找 `MFTablePro`）、`ProForm`、弹层从 `@mingfuyun/star-allinone` 导入 `BaseModal` / `CustomDrawer` / `HocModal`、`await` 必须 try/catch、禁止无说明的 `any`、React 18。
+
+接入或升级后跑 `pnpm exec star-allinone doctor`。
 
 业务项目的 `.agents/project/` 只写差异：路径、权限体系、OpenAPI 位置、领域特例。项目文件可以补充，但不能削弱公共硬规则。
 
@@ -70,6 +72,7 @@ npm 包中的 skills/shared
 
 ```bash
 pnpm exec star-allinone skills sync
+pnpm exec star-allinone doctor
 ```
 
 同步会整体替换 `shared-*` 目录，并删除包中已不存在的过期 `shared-*`，但不会改动 `custom-*`。
@@ -132,7 +135,7 @@ git commit -m "chore(skill): 更新公共 skill"
 | `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-publish-test` | SKILL + `scripts/publish-current-branch.sh` | 云效测发；只读 `publishPipelineId` |
-| （非入口）`shared-project-conventions` | `architecture.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 源码：
@@ -141,6 +144,16 @@ git commit -m "chore(skill): 更新公共 skill"
 - OpenAPI 快照：业务仓库数据文件
 - 真正的业务特例：权限常量名、IP 校验、设备/节点领域规则
 
+## 云效门禁
+
+业务仓通常没有 pipeline YAML。在云效构建控制台显式增加：
+
+```bash
+pnpm typecheck && pnpm lint
+```
+
+再跑现有 `pnpm build`。`pnpm test` 若实际是 `max dev`（测网开发服务），改名为 `start:test` / `dev:test`，避免 Agent 当成单测。
+
 ## 最终推荐流程
 
 ```text