Lớp vendored lên kit 2.18.1 — 9 → 15 tệp (giữ fork stale-scope) - #127
Closed
phanlemanh wants to merge 1 commit into
Closed
phanlemanh wants to merge 1 commit into
phanlemanh wants to merge 1 commit into
Conversation
Plugin scope project: acceptance-gate + feature-loop 2.16.0 → 2.18.1. Lớp CI theo INIT-CI-COPY-LIST của kit 2.18.1 (15 tệp). Sáu tệp vendor lần đầu (nhan-canh-gay, nguong-o-co-hoi, out-of-contract, product-map, trang-thai-ho-so, khong-can-nguoi); hai tệp lib/scripts đổi được chép nguyên bản; sáu tệp còn lại đã trùng khít. Không có `lib/out-of-contract.js` cũ để xoá. Tệp cổng là fork của kho (`feature_scope`) nên đi đường gộp BA CHIỀU — nền kit 11e98c47 (bản 2.17.0 mà lần nâng trước đã gộp vào; tám tệp còn lại trùng khít nền này), của-ta là fork, của-họ là gói 2.18.1. Gộp sạch, 0 xung đột; +78 dòng, các hunk trùng đúng từng dòng với diff nền→2.18.1 của kit; `feature_scope` còn nguyên bốn chỗ. Ba chỗ đi kèm, đều theo lệ sẵn có của kho cho tệp thước vendored: - `_acceptance/config.yaml`: sáu tệp mới vào `t1_skip_globs`. Thiếu dòng này chính PR nâng thước đỏ ở luật T1-escape (đo: exit 1, 1 VIOLATION). - `biome.json`: sáu tệp mới vào danh sách loại trừ. Thiếu thì `lint:check` đỏ vì định dạng/luật của mã kit. - `skills/acceptance/references/opportunity-template.md`: `product-map.mjs` 2.18.1 đọc khuôn này; vắng nó bản vendored thoát 2. Tệp NẰM NGOÀI danh sách 15 của kit. PRODUCT-MAP.md vẽ lại bằng product-map 2.18.1: đúng một dòng — `normalize-text-vi` mang nhãn «đã nghỉ, giữ sử liệu». Đo TRƯỚC↔SAU trên cây này: `pre-merge-check --base origin/main` exit 0 → exit 0, từng dòng giống hệt (chỉ khác số đếm «suppressed»); với `--recheck-all` cũng giống hệt. Không hồ sơ nào đổi phán quyết. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Owner
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tóm tắt
Nâng oneflow lên acceptance-gate-kit 2.18.1:
acceptance-gatevàfeature-looptừ 2.16.0 lên 2.18.1. Việc này nằm trong registry plugin trên máy, không có tệp nào trong kho.INIT-CI-COPY-LIST(GUIDE §5.3).Đã đo trên chính cây này: không hồ sơ nào đổi phán quyết. Kho này chưa có hồ sơ nào mà bản mới chấm khác đi (chi tiết bên dưới).
Cách nâng
nhan-canh-gay,nguong-o-co-hoi,out-of-contract(lib),product-map,trang-thai-ho-so,khong-can-nguoi(scripts)lib/workspace-record.cjs,scripts/recheck-evidence.cjsscripts/pre-merge-check.shfeature_scope), đi đường gộp ba chiều, chi tiết ngay dướilib/out-of-contract.js(đuôi cũ)Gộp ba chiều:
11e98c47, tức bản 2.17.0 mà Lớp vendored lên kit 2.17.0 — lối cho một hồ sơ nghỉ (giữ fork stale-scope) #126 đã gộp vào. Tám tệp không phải fork trùng khít bản này.11e98c47 → 2.18.1của kit.feature_scopecòn nguyên 4 chỗ.Ba chỗ đi kèm (theo lệ sẵn có cho tệp thước vendored)
_acceptance/config.yaml: thêm 6 tệp mới vàot1_skip_globs. Thiếu thì chính PR này đỏ ở luật T1-escape (đo được: exit 1,VIOLATION [PR]: non-T1 files changed … NO _acceptance/<slug>/ artifacts).biome.json: thêm 6 tệp mới vào danh sách loại trừ. Thiếu thìpnpm lint:checkđỏ vì định dạng và luật lint trên mã của kit.skills/acceptance/references/opportunity-template.md:product-map.mjs2.18.1 đọc khuôn này. Vắng nó, bản vendored thoát 2 (khuôn opportunity-template không đọc được … ENOENT). Tệp này không có trong danh sách 15 tệp của kit.PRODUCT-MAP.mdđược vẽ lại bằng product-map 2.18.1 và chỉ đổi một dòng:normalize-text-vinay mang nhãn «đã nghỉ, giữ sử liệu».TRƯỚC ↔ SAU
pre-merge-check --base origin/maincleanclean, từng dòng giống hệt (chỉ khác số đếm «suppressed»)pre-merge-check --base HEAD --recheck-allnormalize-text-vihang-rao-doc-nham-loi-thanh-khong-co-gi,pnpm-build-approvals,repin-khong-chay-lai-eval)skill-1-footage-kho-clip(sắp mở vòng)Kho không có hồ sơ nào ở
machine-cleared, không có tệpreview-findings.md, không có dòng quan sát prod. Vì vậy ba tính năng chính của 2.18.x (hoSoDaKhep, làn V đọc tệp phát hiện, «đã chấm bởi thực tế») chưa có vật gì để tác động ở đây.CI
Chạy ở máy:
lint:check,typecheck,test,build,sdk pytest,verify:plugins,pre-merge-checkđều xanh.Job Acceptance Gate sẽ đỏ, nhưng không phải do PR này.
main@389b6eeđã đỏ sẵn ở bước «Roadmap ledger freshness»:skill-system-v1đã ký nhưng chưa được phân loại trong sổ cái. Bản sửa nằm trên nhánh chưa mergedocs/dong-vong-skill-system-v1. Chạy ở máy trênorigin/maincho thấy các bước sau đó (đang bị bỏ qua trên CI) cũng đỏ sẵn, với đúng tập lỗi như trên nhánh này:check-product-map.mjs,check-plan-freeze-teeth,check-plan-docs(+teeth).Một điều anh cần biết
scripts/ci/check-product-map.mjsghi rõ quyết định không vendorproduct-map.mjs, để tránh có hai cây thước cho cùng một việc. Kit 2.18.1 thì đòi vendor tệp đó. PR này làm theo kit. Bản vendored hiện chưa bước CI nào gọi tới. Hai cây thước đã lệch nhau sẵn trênmain:product-map.mjs --checkcủa kit: khớp.check-product-map.mjscủa kho: 6 chỗ lệch.Nên giữ cây thước nào là quyết định của anh. PR này không đổi gì ở chỗ đó.
🤖 Generated with Claude Code