@mingfuyun/star-allinone 0.2.0 → 0.3.1

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

@@ -0,0 +1,44 @@
+# List page template
+
+The CLI writes this tree. Edit the generated files; do not recreate a parallel structure.
+
+```text
+config/route.ts
+src/pages/<Feature>/
+├── index.tsx
+├── index.less
+├── const.tsx
+└── components/
+    ├── <Feature>Drawer.tsx
+    └── <Feature>ConfirmModal.tsx
+```
+
+## index.tsx
+
+- Wrap with `PageContainer` when `src/components/PageContainer` exists.
+- `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()`.
+
+## const.tsx
+
+- Export `columns`, `getActionColumn`, `getToolBarRender`.
+- Put permission strings on actions and toolbar items when the project uses them.
+- Render empty cells as `'-'`.
+
+## Drawer
+
+- `CustomDrawer` + `HocModal`.
+- `ProForm` with `submitter={false}` and `layout="horizontal"`.
+- `validateFields()` in `try`. Create vs update from `record`.
+- Reset fields when `visible` becomes true.
+
+## Confirm modal
+
+- `BaseModal` + `HocModal`.
+- `confirmLoading` tied to the async `onConfirm`.
+- Do not use `Modal.confirm`.
+
+## 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.

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

@@ -75,7 +75,7 @@ Follow [overlays.md](overlays.md). Use `CustomDrawer`, `BaseModal`, and `HocModa
 - API-backed pages include types, service methods, aggregate exports, and page consumption in the same change.
 - Do not embed production fake data or invented response fields.
 - If the backend contract is unavailable, label proposed methods, paths, and fields as proposed.
-- Pair missing services with `.agents/skills/shared-api-to-service/SKILL.md`. Pair missing deterministic data with `.agents/skills/shared-api-mock/SKILL.md`.
+- Pair missing services with `.agents/skills/shared-api-to-service/SKILL.md`. Pair missing deterministic data with `.agents/skills/shared-api-to-service/references/mock.md`.
 
 ## Styling and fidelity
 

package/skills/shared/i-have-adhd/{SKILL.md → i-have-adhd.md}

@@ -1,13 +1,3 @@
----
-name: {{sharedSkillName}}
-description: 'Shape output for a reader with ADHD: lead with the next action, number multi-step work, restate state across turns, suppress tangents, give specific time estimates, make wins visible. Invoke with /i-have-adhd; stays on until "stop adhd mode".'
-disable-model-invocation: true
-license: MIT
-metadata:
-  tags: "ADHD, Output Style, Productivity, Formatting"
-  category: "productivity"
----
-
 # i-have-adhd
 
 The reader has ADHD. Output is not just brief. It is shaped so an ADHD brain can act on it.

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

@@ -4,6 +4,7 @@ Apply these rules to every non-trivial frontend change. Shared review, commit, m
 
 ## Architecture
 
+- Always-on agent instructions live in the repo-root `AGENTS.md`. Do not add `.cursor/rules`.
 - 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`.
@@ -38,6 +39,8 @@ pnpm exec prettier --check <changed-files>
 pnpm exec tsc --noEmit
 ```
 
+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.
 
 Distinguish failures introduced by the current change from pre-existing or environment failures.

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

@@ -1,10 +1,12 @@
 ---
 name: {{sharedSkillName}}
-description: Publish the current frontend Git branch to the project's Yunxiao test pipeline only after explicit confirmation such as 确认推送流水线 or 发布测试环境. Never run the pipeline automatically after git push.
+description: User-facing. Publish the current Git branch to the Yunxiao test pipeline after explicit confirmation such as 确认推送流水线. Never run automatically after git push.
 ---
 
 # Publish Test Environment
 
+User-facing entry for test-environment publish only.
+
 This skill is fully shared. The only project-specific value is `publishPipelineId` in `.star-allinone.json`.
 
 Do not run the pipeline script after a git push. Git push, `确认提交并推送`, and `确认` are not pipeline authorization.
@@ -58,4 +60,4 @@ The dashboard comment is `BUILD_REMARK` and contains branch, commit subject, and
 
 ## User-facing output
 
-Read and apply `.agents/skills/shared-i-have-adhd/SKILL.md` to every response this skill sends to the user.
+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/review-and-submit/SKILL.md

@@ -1,10 +1,12 @@
 ---
 name: {{sharedSkillName}}
-description: Run the frontend pre-submit workflow: impact analysis, verification, then confirmation-gated commit review. Use when the user asks for the full submit flow or says 使用完整流程提交代码. Never stage, commit, or push before explicit confirmation.
+description: User-facing. Pre-submit workflow: impact analysis, verify, then confirmation-gated commit. Use when the user wants to review, submit, or commit.
 ---
 
 # Review and Submit
 
+User-facing entry for review, submit, and commit. Read the files under `references/` in order. Do not tell the user to invoke other review skills.
+
 This is the unified pre-submit entry and is fully shared. Do not mutate Git until the user confirms the exact reviewed scope.
 
 If `.agents/project/review-rules.md` exists, apply extra domain checks only; do not weaken shared hard rules.
@@ -15,7 +17,7 @@ Execute in this order.
 
 ### 1. Impact analysis
 
-Follow `.agents/skills/shared-impact-analysis/SKILL.md`. Output:
+Follow [references/impact-analysis.md](references/impact-analysis.md). Output:
 
 - Change summary
 - Direct callers and affected pages
@@ -25,7 +27,7 @@ Follow `.agents/skills/shared-impact-analysis/SKILL.md`. Output:
 
 ### 2. Verification
 
-Follow `.agents/skills/shared-verify/SKILL.md`. Output:
+Follow [references/verify.md](references/verify.md). Output:
 
 - P0/P1/P2 blockers
 - Completeness of route, permission, API, types, mock, and page states
@@ -33,11 +35,11 @@ Follow `.agents/skills/shared-verify/SKILL.md`. Output:
 - Prettier, TypeScript, lint, and build results
 - Uncovered risks
 
-If the current change introduced P0/P1 issues or validation failures, stop. Do not enter commit review.
+Also apply [references/ai-review.md](references/ai-review.md). If the current change introduced P0/P1 issues or validation failures, stop. Do not enter commit review.
 
 ### 3. Commit review
 
-Follow `.agents/skills/shared-commit-current-changes/SKILL.md`. Output:
+Follow [references/commit.md](references/commit.md). Output:
 
 - Findings sorted P0–P2
 - Optional improvements
@@ -67,16 +69,6 @@ If the user only says `确认` but the scope is unclear, do not commit; ask for
 
 If the user says `提交代码` before a review proposal exists, run the three steps above first. Do not skip review.
 
-## Related skills
-
-| Skill | Purpose |
-| --- | --- |
-| `$shared-impact-analysis` | Read-only impact analysis |
-| `$shared-verify` | Convention and command verification |
-| `$shared-commit-current-changes` | Review plus confirmation-gated commit |
-| `$shared-ai-review` | Standalone review without commit |
-| `$shared-publish-test` | Publish to the test pipeline only after a separate `确认推送流水线`. Never run it automatically after git push. |
-
 ## User-facing output
 
-Read and apply `.agents/skills/shared-i-have-adhd/SKILL.md` to every response this skill sends to the user.
+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/{ai-review/SKILL.md → review-and-submit/references/ai-review.md}

@@ -1,16 +1,13 @@
----
-name: {{sharedSkillName}}
-description: Review frontend changes for correctness, API and permission contracts, React lifecycle, performance, maintainability, security, accessibility, and test gaps. Use for a standalone review without commit. Do not modify files.
----
-
 # AI Code Review
 
-This skill is fully shared. Review the current diff plus relevant callers and types. Before reviewing, read:
+Checklist used by `shared-review-and-submit`. Do not treat this file as a generate or submit entry.
+
+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`
 
-Do not require a project-local review-rules file. If `.agents/project/review-rules.md` exists, apply extra domain checks only; do not weaken shared hard rules.
+If `.agents/project/review-rules.md` exists, apply extra domain checks only; do not weaken shared hard rules.
 
 ## Required checks
 
@@ -27,7 +24,3 @@ Do not require a project-local review-rules file. If `.agents/project/review-rul
 Lead with findings ordered `[P0]`–`[P3]`, with file and line. Separate blocking issues from recommendations. Do not report style preferences as bugs.
 
 If none, write `No issues found.` Always include validation performed and gaps. Do not modify files unless the user explicitly asks for fixes.
-
-## User-facing output
-
-Read and apply `.agents/skills/shared-i-have-adhd/SKILL.md` to every response this skill sends to the user.

package/skills/shared/{commit-current-changes/SKILL.md → review-and-submit/references/commit.md}

@@ -1,11 +1,6 @@
----
-name: {{sharedSkillName}}
-description: Review the active frontend project's changes and prepare a confirmation-gated Conventional Commit and optional push. Use when the user asks to review current changes, submit code, or commit and push. Never commit before explicit confirmation.
----
-
 # Commit Current Changes
 
-This skill is fully shared. Review user-authorized working-tree changes, report findings and a commit proposal, and mutate Git only after explicit confirmation.
+Step 3 of `shared-review-and-submit`. Review user-authorized working-tree changes, report findings and a commit proposal, and mutate Git only after explicit confirmation.
 
 ## Write-operation gate
 
@@ -29,8 +24,9 @@ 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-project-conventions/references/architecture.md`
+4. [ai-review.md](ai-review.md)
 
-This skill is fully shared. If `.agents/project/review-rules.md` exists, apply extra domain checks only; do not weaken shared hard rules.
+If `.agents/project/review-rules.md` exists, apply extra domain checks only; do not weaken shared hard rules.
 
 Classify only concrete security, data-loss, runtime, contract, validation, performance, accessibility, or maintainability risks as P0–P2.
 
@@ -55,7 +51,7 @@ Lead with findings. Then present optional improvements, validation results, chan
 3. Commit without `--no-verify` unless the user explicitly authorizes it. Do not amend unless requested.
 4. Push to upstream, or `git push -u origin HEAD` when none exists. Never force-push.
 5. Report hash, subject, files, validation, and remote branch.
-6. Stop. Do not run `$shared-publish-test` or any Yunxiao pipeline script after git push.
+6. Stop. Do not run the test pipeline after git push.
 
 A confirmed commit or push does not authorize the test pipeline. If the user later says `确认推送流水线` or `发布测试环境`, follow `.agents/skills/shared-publish-test/SKILL.md`.
 
@@ -64,7 +60,3 @@ A confirmed commit or push does not authorize the test pipeline. If the user lat
 - Never expose credentials from diffs; stop if secrets are present.
 - Never use destructive Git commands to clean the working tree.
 - If push is rejected, report the reason and do not rewrite history without separate authorization.
-
-## User-facing output
-
-Read and apply `.agents/skills/shared-i-have-adhd/SKILL.md` to every response this skill sends to the user.

package/skills/shared/{impact-analysis/SKILL.md → review-and-submit/references/impact-analysis.md}

@@ -1,11 +1,6 @@
----
-name: {{sharedSkillName}}
-description: Analyze frontend Git changes and trace affected routes, pages, services, permissions, shared components, mocks, and regression checks. Read-only. Use for PR or pre-submit impact analysis.
----
-
 # Impact Analysis
 
-This skill does not modify files.
+Step 1 of `shared-review-and-submit`. Do not modify files.
 
 ## Workflow
 
@@ -28,7 +23,3 @@ Read `.agents/skills/shared-project-conventions/references/review-checklist.md`
 - Risks that cannot be confirmed statically, with why
 
 Use repository facts only. Do not invent backend behavior.
-
-## User-facing output
-
-Read and apply `.agents/skills/shared-i-have-adhd/SKILL.md` to every response this skill sends to the user.

package/skills/shared/{verify/SKILL.md → review-and-submit/references/verify.md}

@@ -1,11 +1,8 @@
----
-name: {{sharedSkillName}}
-description: Verify generated or manually changed frontend code against shared conventions, API contracts, permissions, async behavior, and validation commands. Use after API, mock, page, or refactor work. Do not generate business code.
----
-
 # Frontend Verification
 
-This skill is fully shared. Read the current diff, related callers, and:
+Step 2 of `shared-review-and-submit`.
+
+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`
@@ -28,7 +25,7 @@ Choose by risk. Do not auto-fix to make a command pass.
 git diff --check
 pnpm exec prettier --check <changed-files>
 pnpm exec eslint <changed-files>
-pnpm exec tsc --noEmit
+pnpm typecheck
 pnpm lint
 ```
 
@@ -42,7 +39,3 @@ Run the project's build script when routes, entry, or shared runtime change.
 4. Regression list: affected pages, suggested manual paths, uncovered risks.
 
 If there are no blockers, write `未发现阻断问题`. Do not report only that commands passed.
-
-## User-facing output
-
-Read and apply `.agents/skills/shared-i-have-adhd/SKILL.md` to every response this skill sends to the user.

package/skills/shared/project-conventions/SKILL.md

@@ -1,34 +0,0 @@
----
-name: {{sharedSkillName}}
-description: Apply the active frontend project's architecture, API, business, permission, performance, testing, and naming conventions to generated or reviewed code.
----
-
-# Project Conventions
-
-Use this skill for every non-trivial change. Preserve existing patterns and make the smallest scoped change.
-
-Read in this order:
-
-1. `.star-allinone.json`
-2. [references/architecture.md](references/architecture.md)
-3. [references/review-checklist.md](references/review-checklist.md)
-
-Shared rules are complete for review, commit, mock, and service generation. If `.agents/project/conventions.md` exists, use it only for extra paths or domain invariants. It must not weaken the shared hard rules below.
-
-## Hard rules
-
-- Pages consume `import Service from '@/services'`. No direct axios or request-helper calls from pages.
-- DTOs live in `src/services/<Domain>/typings.d.ts`. Pages do not redefine them.
-- Reuse `MFProTable`, `ProForm`, `CustomDrawer`, `BaseModal`, `HocModal`, and existing permission wrappers.
-- Backend field names stay in service types. Convert only at an adapter boundary.
-- Async surfaces handle loading, empty, error, retry, unauthorized, and duplicate-submit.
-- No unexplained `any`, `as any`, `@ts-ignore`, or `Record<string, any>` when the contract is known.
-- No new UI/state library without approval.
-
-## Completion
-
-Report changed files, API operations used, states covered, validation commands and results, and remaining gaps.
-
-## User-facing output
-
-Read and apply `.agents/skills/shared-i-have-adhd/SKILL.md` to every response this skill sends to the user.