Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions .agents/skills/brainstorming/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ description: Use when receiving an ambiguous feature request, when scope is uncl
把模糊需求通过"一次一个问题"的方式,收敛成一份可用于 `writing-plans` 的设计规格。

<HARD-GATE>
在完成规格并拿到用户批准之前,**不得**调用 `writing-plans`、不得动任何源文件、不得派 subagent 执行实现
在完成规格并拿到用户批准之前,**不得**调用 `writing-plans`、不得动任何源文件、不得开始实施
逃生条款:用户明确说 "skip brainstorming"、"已经有规格"、"直接按 X 做" 时可跳过。
</HARD-GATE>

Expand Down Expand Up @@ -41,7 +41,7 @@ description: Use when receiving an ambiguous feature request, when scope is uncl
5. **写规格文档** — 保存到 `docs/specs/YYYY-MM-DD-<topic>.md`
6. **规格自查** — 有无占位符 `TODO`、有无自相矛盾、范围是否闭合
7. **请用户审阅规格文件** — 等明确 "OK/确认/继续"
8. **移交** — 规格确认后回到主流程(Read relevant docs → Planning Gate),由 Planning Gate 统一判定是否需要 plan
8. **移交** — 规格确认后回到主流程(Read relevant docs → 判定影响面),统一判定是否需要 plan

## Process Flow

Expand Down Expand Up @@ -73,7 +73,7 @@ Self-review (placeholders / contradictions / scope)
User reviews spec file ── approved? ─NO→ revise
│ YES
回到主流程 → Read relevant docs → Planning Gate(统一判定
回到主流程 → Read relevant docs → 影响面判定(统一一次
```

## Anti-Pattern:「这个需求太简单不需要规格」
Expand Down Expand Up @@ -105,7 +105,7 @@ User reviews spec file ── approved? ─NO→ revise
brainstorming 收敛规格并拿到用户确认后,**不自行判定影响面**,直接回到主流程:

```
规格确认 → Read relevant docs → Planning Gate(统一判定
规格确认 → Read relevant docs → 影响面判定(统一一次
```

影响面判定权归 Planning Gate 一处(`using-growingio-sdk-skills` meta-skill),brainstorming 不重复判定、不自行分叉。
影响面判定(改动文件 ≥3 或涉及公开 API 变更 → 先写 plan)在规格确认之后统一做一次,brainstorming 不重复判定、不自行分叉。
8 changes: 4 additions & 4 deletions .agents/skills/finishing-a-development-branch/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ description: Use after verification-before-completion passes and code review is

**强制:**
- `verification-before-completion` skill 通过后
- `sdk-code-review` skill 通过后
- 对照 `docs/sdk-review-checklist.md` 自查通过后
- 用户说"这个功能做完了" / "收个尾" / "合并吧"

**不触发:**
Expand Down Expand Up @@ -153,7 +153,7 @@ git push origin --delete <branch-name>
| "发版分支忘了打 tag 也没事" | 丢失版本追溯能力,补 tag 需要 cherry-pick 成本 |
| "改动都 commit 了,用 `git add -A` 一把梭" | 容易把 .env / 密钥 / 构建产物带进去 |
| "合并完分支保留吧以后说不定还用" | 仓库分支列表会膨胀;真要用再 checkout commit |
| "没触发 Planning Gate 就不用走收尾" | 只要有 commit 产生就应该走本 skill,小改动可以跳过 PR 但不能跳过用户确认 |
| "改动小、没写 plan 就不用走收尾" | 只要有 commit 产生就应该走本 skill,小改动可以跳过 PR 但不能跳过用户确认 |

## Red Flags — STOP if you catch yourself thinking these

Expand All @@ -164,7 +164,7 @@ git push origin --delete <branch-name>

## 关联 skill

- **上游触发:** `verification-before-completion` 通过且 `sdk-code-review` 通过
- **调度 subagent:** 无(本 skill 由控制器执行,用户决策哪条路径)
- **上游触发:** `verification-before-completion` 通过,且 `docs/sdk-review-checklist.md` 自查通过
- **决策方:** 用户决定走哪条终态路径
- **完成后交接:** 合并到 master / PR 待评审 / 分支保留 / 分支废弃 四种终态之一
- **替代路径:** 发版分支 → 调用 `jira-ticket`(发版单) + `ohpm-publish`(OHPM 发布) + `git-conventions`(tag 命名)
4 changes: 2 additions & 2 deletions .agents/skills/growingio-arkts-coding-style/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -391,6 +391,6 @@ import { LogUtil } from '../utils/LogUtil'
## 关联 skill

- **上游触发:** 写或审查任何 `.ets` / `.ts` 文件时
- **调度 subagent:** 无(本 skill 是规范手册,由编码者/审查者自己对照
- **完成后交接:** `sdk-code-review` 的 code-reviewer subagent 必须用本 skill 的约束清单做审查
- **使用方式:** 本 skill 是规范手册,由编码者/审查者自己对照
- **完成后交接:** 审查阶段按 `docs/sdk-review-checklist.md` Part 1 第 1 项,用本 skill 的约束清单逐条核对
- **替代路径:** 遇到 TypeScript 代码需迁移 → 交叉参考 `docs/typescript-to-arkts-migration-guide.md`
187 changes: 0 additions & 187 deletions .agents/skills/plan-document-review/SKILL.md

This file was deleted.

Loading