baodhgbaodhg/FLOW/skills/flow-review/SKILL.md

flow-review

Skill2 starsChanged 7 days agoMITView on GitHub

Dùng khi cần một cặp mắt không dính vào lúc viết để soát thay đổi — sau một đợt task, trong bước đóng việc, trước khi mở PR, hoặc khi nghi có lặp mã. Người dùng hay nói "soát giúp", "xem lại code", "đọc diff này", "có lặp code không", "review", "đủ chuẩn để merge chưa".

2 other projects use a version of this.

Adopted by

Most recently taken up by

fifersheep/flowtheir own version, first seen Oct 20, 2026
emiliodominguez/flow-skillstheir own version, first seen Oct 15, 2026

Other versions2 projects use a version of this.
Word for word · 0
None found.
Edited · 0
None found.
Wrote their own version · 2
emiliodominguez/flow-skills★ 1same name
fifersheep/flow★ 0same name
How matches are found. Word for word: the exact same text. Edited: shares passages with this file (“99% same” means 99% of its five-word phrases also appear here; 10% or more counts). Their own version: similar meaning, or, weakest, the same name. We can’t tell which project had it first. Projects are counted among the 200 most-starred candidates, and mentions are links to the repository we found, up to 50 per site, so both counts are a floor.
In this repository13 other files it gives its agents.
- flow-contextSkill
- flow-executeSkill
- flow-exploreSkill
- flow-initSkill
- flow-learnSkill
- flow-planSkill
- flow-postSkill
- flow-runbookSkill
- flow-secretsSkill
- flowSkill
- flow-syncSkill
- flow-testSkill
- flow-worktreeSkill

The file

We split this skill into the rules it gives an agent

1 rule. About 0 are common, 1 is distinctive, and 0 are very specific to this project.

≈0
Common · estimate
Many files like this have a rule of this kind.
≈1
Distinctive · estimate
This author's own choices. Worth a look.
0
Very specific
Tied to their project's names, files or tools. Probably skip.

Common
Distinctive
Very specific

Distinctive:
Test chứng minh đúng hành vi plan nói
line 28

≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.

Keep what fits and drop the rest. Your version, ready for your agents.

Files

SKILL.md

flow-review — soát độc lập

Người viết thấy điều mình định viết, không thấy điều mình đã viết. Vì vậy review do một subagent mới làm, không phải chính phiên đang viết.

Gói giao cho subagent

Chỉ gồm:

Diff: git diff <nhánh gốc>...HEAD, hoặc diff của đợt task
Mục tiêu: mục tiêu plan và danh sách task, hoặc một câu nếu không có plan
Luật khớp: flow rules find --changed
Kết quả công cụ: flow check --base <nhánh gốc>, flow dup check --base <nhánh gốc>, verify

Không đưa kết luận hay lập luận của người viết, vì chúng neo người soát vào cùng một cách nhìn. Không nói "phần này đã ổn".

Hai kết luận riêng

Một thay đổi có thể đúng plan mà mã kém, hoặc mã đẹp mà lệch plan, nên không gộp.

Đúng plan

Mọi task đã làm đủ, không làm thừa ngoài phạm vi
Test chứng minh đúng hành vi plan nói
Thứ tự triển khai, hợp đồng, rollback còn khớp với plan

Chất lượng

DRY: có hàm mới trùng ý đồ với hàm có sẵn. Dùng flow dup find chứ không chỉ đọc diff, vì bản gốc thường nằm ngoài diff.
Ranh giới: logic nằm đúng tầng, không xuyên module
Lỗi im lặng: catch rỗng, mặc định che lỗi, kết quả bỏ qua
Đồng thời: tranh chấp ghi, thiếu transaction, thao tác không idempotent
Tên: nói đúng việc, theo quy ước repo
Bảo mật cơ bản: kiểm quyền, nối chuỗi vào SQL, in bí mật, log thông tin cá nhân

Định dạng kết quả

Đúng plan: đạt | không đạt
Chất lượng: đạt | không đạt

- [chặn] `loan/renew.ts:42` — thiếu transaction khi gia hạn `Loan` — hai yêu cầu đồng thời cùng qua — bọc trong transaction, thêm test đồng thời
- [nên sửa] `loan/format.ts:10` — trùng ý đồ với `formatDueDate` — dùng lại bản có sẵn
- [gợi ý] `loan/renew.ts:7` — tên `data` chưa nói gì

Mỗi mục có tệp:dòng, vấn đề, hệ quả, cách sửa.

Mức	Nghĩa	Xử lý
`chặn`	Sai hành vi, mất dữ liệu, vi phạm luật	Phải sửa trước khi đi tiếp
`nên sửa`	Làm việc được nhưng để lại nợ	Sửa, hoặc ghi lý do không sửa
`gợi ý`	Tuỳ ý	Người viết tự quyết

Vòng lặp

Tối đa hai vòng. Sửa xong gửi diff mới cho subagent mới. Sau hai vòng vẫn còn mục chặn thì hỏi người dùng trên chat, vì lúc đó thường là bất đồng về thiết kế, không phải lỗi mã.

Người viết không tự bác mục chặn. Không đồng ý thì nêu lý do cho người dùng.

Khi nào

Sau mỗi task rủi ro hoặc mỗi đợt task trong skill flow-execute: soát phần vừa làm.
Ở skill flow-post: review cuối soát cả nhánh, vì lỗi hay nằm ở chỗ các task ghép với nhau.

Download now (ZIP)

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

It worked
Partly
It didn't

What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.

Check your setup, free