@mingfuyun/star-allinone 0.3.2 → 0.3.3

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

@@ -48,8 +48,8 @@ pnpm exec star-allinone gen page \
 Add `--permission-create`, `--permission-edit`, `--permission-delete` when codes are known.
 
 2. If only services are missing, run `pnpm exec star-allinone gen service --operation "METHOD /path/"` first. Follow `.agents/skills/shared-api-to-service/SKILL.md`.
-3. Fill generated placeholders using [references/list-page-template.md](references/list-page-template.md): columns, form fields, rowKey, permission constants, and pagination adapter (`results/count` vs `list/total`).
-4. If `src/components/Page/Container/config.tsx` exists, register the new page as a tab after generating the standalone files. Do not invent a second page shell.
+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. Do not invent a second page shell.
 5. Do not overwrite existing page files. Tell the user the CLI refused, then edit in place.
 
 ## Design path
@@ -67,7 +67,7 @@ For non-standard layouts:
 ## Hard rules
 
 - React function components, TypeScript, Ant Design 5, Pro Components, Less.
-- `ProForm` for business forms. `MFProTable` for management lists.
+- `ProForm` for business forms. `MFProTable` from `@/components/CommonTable/MFProTable` (not from the npm package).
 - Overlays from `@mingfuyun/star-allinone`: `CustomDrawer`, `BaseModal`, `HocModal`.
 - Pages consume `import Service from '@/services'` only.
 - Keep columns and toolbar builders in `const.tsx`. Empty cells render as `-`.

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

@@ -13,9 +13,11 @@ src/pages/<Feature>/
     └── <Feature>ConfirmModal.tsx
 ```
 
-## index.tsx
+When `src/components/Page/Container/config.tsx` exists, `index.tsx` is a Container shell and the table lives in `components/List.tsx`. The CLI also registers a tab for the new route.
 
-- Wrap with `PageContainer` when `src/components/PageContainer` exists.
+## index.tsx / List.tsx
+
+- Wrap with `PageContainer` when `src/components/PageContainer` exists and the page is not a Container tab.
 - `fetchData` is a `useCallback` that calls `Service.<listFn>`, maps pagination, and returns `{ code, data: { list, total } }`.
 - Open the drawer with `drawerRef.current?.show({ record })`.
 - Destructive actions go through the confirm modal ref, then `tableRef.current?.reload()`.
@@ -23,6 +25,7 @@ src/pages/<Feature>/
 ## const.tsx
 
 - 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.
 - Render empty cells as `'-'`.
 
@@ -30,6 +33,7 @@ src/pages/<Feature>/
 
 - `CustomDrawer` + `HocModal`.
 - `ProForm` with `submitter={false}` and `layout="horizontal"`.
+- 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.
 
@@ -41,4 +45,10 @@ src/pages/<Feature>/
 
 ## Container tabs
 
-If the project uses `src/components/Page/Container/config.tsx`, the generated `src/pages/<Feature>/index.tsx` is still the source of truth. Add a tab entry that renders the feature component. Do not duplicate fetch logic inside Container.
+If the project uses `src/components/Page/Container/config.tsx`, the CLI:
+
+1. Writes `index.tsx` as `import Container from '@/components/Page/Container'`.
+2. Writes the table in `components/List.tsx`.
+3. Registers `'<route>': [{ key, label, children: <List /> }]` in `config.tsx`.
+
+Do not duplicate fetch logic inside Container. Do not wrap the List with a second `PageContainer`.

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

@@ -2,6 +2,12 @@
 
 Use this reference for every modal, confirmation dialog, and drawer. Import `@mingfuyun/star-allinone/dist/styles.css` once from the application entry, not from feature components.
 
+Overlays belong to the npm package. Do not add `src/components/CustomDrawer`, `src/components/BaseModal`, or `src/components/HocModal`.
+
+```tsx
+import { CustomDrawer, BaseModal, HocModal } from '@mingfuyun/star-allinone';
+```
+
 ## Required components
 
 - `CustomDrawer` for create, edit, detail, or configuration drawers.

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

@@ -31,7 +31,8 @@ Styles live in the current page directory. Reuse shared mixins such as `@/styles
 
 ## Tables
 
-- Prefer `MFProTable` for management lists. Use Ant Design `Table` only for a deliberately simple static shell.
+- Prefer `MFProTable` for management lists. Import it from `@/components/CommonTable/MFProTable`. See [mfprotable.md](../../project-conventions/references/mfprotable.md). Use Ant Design `Table` only for a deliberately simple static shell.
+- Do not import `MFTablePro` or any table from `@mingfuyun/star-allinone`.
 - Extract `fetchData` as an `async` method. Do not inline request logic in JSX props.
 - Wrap `await` in `try-catch` and return a table-safe empty shape on failure.
 - Adapt backend pagination to the table contract. Common backend shapes are `{ results, count }` or `{ list, total }`; the table adapter is typically `{ code, data: { list, total } }`. Follow the nearest page.

package/skills/shared/project-conventions/references/architecture.md

@@ -8,10 +8,11 @@ Apply these rules to every non-trivial frontend change. Shared review, commit, m
 - Use React function components and TypeScript.
 - Put transport code in `src/services`. Pages must not call axios or the request helper directly.
 - Keep API DTOs in the domain `typings.d.ts`. Use named types and adapters instead of `any`.
-- Reuse `MFProTable`, `BaseModal`, `CustomDrawer`, `HocModal`, permission wrappers, table cells, hooks, and shared styles before adding abstractions.
+- Reuse `MFProTable` from `@/components/CommonTable/MFProTable` (see [component-contract.md](./component-contract.md) and [mfprotable.md](./mfprotable.md)). It is not exported by `@mingfuyun/star-allinone`.
+- Overlays (`CustomDrawer`, `BaseModal`, `HocModal`) and `ErrorBoundary` / `Cron` come from `@mingfuyun/star-allinone`. Do not keep local copies under `src/components`.
 - Use `@/` imports for application modules and keep feature-local imports relative.
 - Do not add a UI/state library or duplicate an existing component without explicit approval.
-- Prefer feature-level code splitting for Excel, editors, charts, drag-and-drop, and other heavy modules.
+- Prefer feature-level code splitting for Excel, editors, charts, drag-and-drop, and other heavy modules. `xlsx`, `@uiw/react-codemirror` / `@codemirror` / monaco, and chart libraries must use `import()`. Do not static-import them from `src/app.ts` or generated list pages.
 
 ## Business behavior
 
@@ -43,4 +44,6 @@ Prefer `pnpm typecheck` when that script exists.
 
 Run `pnpm lint` for broad or shared changes. Run `pnpm run build` when routes, entry behavior, shared runtime, or multiple page modules change. If `.star-allinone.json` documents a TypeScript `baseUrl` or build script, use that command.
 
+Yunxiao (and any CI without in-repo pipeline YAML) must run `pnpm typecheck && pnpm lint` as an explicit build step. `pnpm test` / `pnpm start:test` in business apps starts the test-env dev server; it is not a unit-test suite.
+
 Distinguish failures introduced by the current change from pre-existing or environment failures.

package/skills/shared/project-conventions/references/component-contract.md

@@ -0,0 +1,38 @@
+# Component contract
+
+Use this file as the import source of truth. Do not invent package exports.
+
+## From `@mingfuyun/star-allinone`
+
+The npm package ships overlays, Cron, and ErrorBoundary. It does **not** export a table.
+
+```ts
+import {
+  BaseModal,
+  Cron,
+  CustomDrawer,
+  ErrorBoundary,
+  HocModal,
+} from '@mingfuyun/star-allinone';
+import '@mingfuyun/star-allinone/dist/styles.css';
+```
+
+Do not write any of the following:
+
+- `import { MFTablePro } from '@mingfuyun/star-allinone'`
+- `import { MFProTable } from '@mingfuyun/star-allinone'`
+- `@mingfuyun/star-allinone/components/BaseModal` (use the main export)
+
+## From the business app
+
+Management lists use the local wrapper. Props and usage are in [mfprotable.md](./mfprotable.md).
+
+```ts
+import MFProTable from '@/components/CommonTable/MFProTable';
+```
+
+Pages still consume APIs with `import Service from '@/services'`. Overlay shells come from the package above, not from `@/components/CustomDrawer` or `@/components/BaseModal`.
+
+## Runtime
+
+Peer React is **18.3.x**. Do not assume React 19 APIs. Ant Design 5 and Pro Components live in the business app.

package/skills/shared/project-conventions/references/mfprotable.md

@@ -0,0 +1,59 @@
+# MFProTable contract
+
+`MFProTable` lives in the **business app**, not in `@mingfuyun/star-allinone`. Do not invent `MFTablePro` and do not import a table from the npm package.
+
+```tsx
+import MFProTable from '@/components/CommonTable/MFProTable';
+import type { MFTableRef } from '@/components/CommonTable/type';
+```
+
+The capability package will not absorb this component until peer/externals/Umi versions are stable across apps.
+
+## Required usage
+
+```tsx
+<MFProTable<Item, Query>
+  ref={tableRef}
+  actionColumn={{
+    actions: getActionColumn({ handleEdit, handleDelete }),
+  }}
+  bordered
+  columns={columns()}
+  fetchData={fetchData}
+  queryParams={{}}
+  rowKey="id"
+  scroll={{ x: 'max-content' }}
+  topTitle={false}
+  toolBarRender={getToolBarRender({ handleCreate })}
+/>
+```
+
+- `fetchData` is an `async` callback. On failure return `{ code: 0, data: { list: [], total: 0 } }`.
+- Adapt backend `{ results, count }` or `{ list, total }` to `{ code, data: { list, total } }`.
+- Put columns, action builders, and toolbar items in `const.tsx`.
+- Empty display values render as `'-'`.
+- Open overlays through refs. Do not store drawer visibility on the page when `HocModal` owns it.
+
+## Common props
+
+These names match `src/components/CommonTable/type.ts` (`ITableProps` / `MFTableRef`):
+
+| Prop | Role |
+| --- | --- |
+| `columns` | `ProColumns<T>[]` from `const.tsx` |
+| `fetchData` | `(params) => Promise<{ code, data: { list, total } }>` |
+| `queryParams` | Extra query merged into requests |
+| `rowKey` | Stable id field, usually `id` |
+| `actionColumn.actions` | Row actions; `permission` when the project uses it |
+| `toolBarRender` | Toolbar buttons (`TableSearchType[]`) |
+| `scroll` | Wide tables: `{ x: 'max-content' }` |
+| `columnsStateKey` | Persist column state via the project hook |
+| `ref.reload` / `ref.reloadAndReset` | After mutations |
+
+Permission filtering is implemented **in the app** (`useAuthFilter`, `withAuth`, `TableActions`). The package overlays do not wrap table permissions.
+
+## Do not
+
+- `import { MFTablePro } from '@mingfuyun/star-allinone'`
+- `import { MFProTable } from '@mingfuyun/star-allinone'`
+- Copy `MFProTable` into a second path and import both.

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

@@ -17,3 +17,17 @@ Check unsafe HTML, URL construction, upload/download names, sensitive logs/stora
 ## Testing
 
 Request tests for concrete regression boundaries such as transformations, races, conditional validation, reusable hooks, complex transitions, or likely recurring bugs. Run focused Prettier, TypeScript, lint, `git diff --check`, and a production build in proportion to risk.
+
+## CI / Yunxiao
+
+These apps do not keep pipeline YAML in the repo. In the Yunxiao build step run:
+
+```bash
+pnpm typecheck && pnpm lint
+```
+
+`pnpm test` is not a unit-test command in the business apps (use `start:test` / `dev:test` for the test-env dev server). `@mingfuyun/star-allinone` `npm test` only covers codegen fixtures.
+
+## Performance
+
+Flag new top-level imports of `xlsx`, `@uiw/react-codemirror` / CodeMirror, chart libraries, and similar heavy modules. They must be `import()` inside the feature that needs them. Generated pages must not pull them into `src/app.ts`.

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

@@ -19,8 +19,8 @@ This checklist is complete for shared review and commit. If `.agents/project/rev
 
 ## Pages and forms
 
-- Management lists prefer `MFProTable`. Column and toolbar configuration belong outside the render function.
-- Business forms use `ProForm`. Drawers use `CustomDrawer`. Dialogs use `BaseModal` / `HocModal`.
+- Management lists prefer `MFProTable` from `@/components/CommonTable/MFProTable`. Column and toolbar configuration belong outside the render function.
+- Business forms use `ProForm`. Drawers use `CustomDrawer`. Dialogs use `BaseModal` / `HocModal` from `@mingfuyun/star-allinone`.
 - Successful mutations refresh the parent list. Closing an overlay clears form, loading, selected record, and other async state.
 - Every `await` has error handling. Loading restores in `finally`.
 - Check loading, empty, error, disabled, long content, and narrow-screen layout.

package/skills/shared/publish-test/SKILL.md

@@ -41,6 +41,16 @@ Read `.star-allinone.json` before running. `publishPipelineId` is required:
 
 Never ask the user to repeat the pipeline ID. Never copy another project's pipeline ID.
 
+## Yunxiao build gates
+
+The pipeline lives in the Yunxiao console (there is no in-repo YAML). The build step must run:
+
+```bash
+pnpm typecheck && pnpm lint
+```
+
+before or as part of `max build`. Do not treat `pnpm test` as unit tests; in business apps that script was the test-env dev server and is now `start:test` / `dev:test`. `ANALYZE=1` is a manual pre-release chunk check, not a pipeline substitute.
+
 ## Preconditions
 
 - Current HEAD is a named branch, not detached.

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

@@ -15,7 +15,11 @@ If `.agents/project/review-rules.md` exists, apply extra domain checks only; do
 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: eager `xlsx`/editor imports, duplicate libraries, large lists without virtualization, unstable columns/handlers, avoidable requests.
+5. 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.
 

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

@@ -16,6 +16,8 @@ Do not auto-fix. If `.agents/project/review-rules.md` exists, apply extra domain
 3. Every `await` has `try-catch`; loading restores in `finally`; overlays clear state on close; successful mutations reload lists.
 4. Scan new `any`, `as any`, `@ts-ignore`, direct Ant Design `Modal` / `Drawer` / `<Form>` containers, and production fake data.
 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`.
 
 ## Commands