Tổng thể làm khá tốt, scope gọn và test rất kỹ, đặc biệt phần permission trước credential resolution, error handling, pagination và không leak token.
Trước khi merge cần bổ sung/chốt thêm mấy điểm:
Phần identity/credential hiện vẫn đang dùng GITEA_TOKEN chung và repo map theo project. Với pilot read-only thì dùng service account được, nhưng cần tách Target Resolver và Credential Resolver ngay từ kiến trúc để sau này chuyển sang on-behalf-of mà không phải sửa lại tool. Target cũng nên tính tới org_unit/customer/project, không chỉ project.
Thêm 1 happy-path test đi xuyên qua ProjectProviderResolver + build_provider thật. Hiện happy path đang inject GiteaIssueProvider trực tiếp nên chưa chứng minh wiring production hoạt động end-to-end.
Bổ sung evidence/test Audit: call thành công/deny phải có correlation/audit record, đồng thời không ghi credential. Nếu shared runtime đã có test này thì link lại test/evidence trong PR là được.
Acceptance Criteria hiện chỉ nhận heading tiếng Anh. Task Core Team sau này chủ yếu dùng tiếng Việt nên support thêm Tiêu chí hoàn thành/Tiêu chí chấp nhận hoặc có cách parse chuẩn hơn, tránh fallback lấy nhầm checklist của section khác.
Ngoài ra check thêm output từ Gitea phải đi qua Untrusted Content Fence của Cowork trước khi đưa vào Agent; nếu đã có ở shared runtime thì ghi rõ evidence/link test trong PR.
Sau các điểm trên PR này đủ tốt để approve cho FSG pilot. 👍
Tổng thể làm khá tốt, scope gọn và test rất kỹ, đặc biệt phần permission trước credential resolution, error handling, pagination và không leak token.
Trước khi merge cần bổ sung/chốt thêm mấy điểm:
1. Phần identity/credential hiện vẫn đang dùng `GITEA_TOKEN` chung và repo map theo `project`. Với pilot read-only thì dùng service account được, nhưng cần tách Target Resolver và Credential Resolver ngay từ kiến trúc để sau này chuyển sang on-behalf-of mà không phải sửa lại tool. Target cũng nên tính tới `org_unit/customer/project`, không chỉ `project`.
2. Thêm 1 happy-path test đi xuyên qua `ProjectProviderResolver` + `build_provider` thật. Hiện happy path đang inject `GiteaIssueProvider` trực tiếp nên chưa chứng minh wiring production hoạt động end-to-end.
3. Bổ sung evidence/test Audit: call thành công/deny phải có correlation/audit record, đồng thời không ghi credential. Nếu shared runtime đã có test này thì link lại test/evidence trong PR là được.
4. Acceptance Criteria hiện chỉ nhận heading tiếng Anh. Task Core Team sau này chủ yếu dùng tiếng Việt nên support thêm `Tiêu chí hoàn thành/Tiêu chí chấp nhận` hoặc có cách parse chuẩn hơn, tránh fallback lấy nhầm checklist của section khác.
Ngoài ra check thêm output từ Gitea phải đi qua Untrusted Content Fence của Cowork trước khi đưa vào Agent; nếu đã có ở shared runtime thì ghi rõ evidence/link test trong PR.
Sau các điểm trên PR này đủ tốt để approve cho FSG pilot. 👍
gitea-admin
changed title from create feature mcp-issue-context to create feature mcp-issue-context & project knowledge search2026-09-04 01:48:34 +00:00
gitea-admin
approved these changes 2026-09-05 00:12:28 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Summary
What changed and why?
Change Type
Related Work
Cowork Task:
Core Repo: http://34.143.229.138/gitea-admin/fsg-ai-core-assets
Core AI Issue:
Core Task:
Related PR:
Scope
What is intentionally included?
What is intentionally NOT included?
Validation
Commands / evidence:
Security Impact
Permission / credential / network / customer data impact:
Compatibility
Reviewer Notes
Anything Cowork reviewers should pay attention to.
Tổng thể làm khá tốt, scope gọn và test rất kỹ, đặc biệt phần permission trước credential resolution, error handling, pagination và không leak token.
Trước khi merge cần bổ sung/chốt thêm mấy điểm:
Phần identity/credential hiện vẫn đang dùng
GITEA_TOKENchung và repo map theoproject. Với pilot read-only thì dùng service account được, nhưng cần tách Target Resolver và Credential Resolver ngay từ kiến trúc để sau này chuyển sang on-behalf-of mà không phải sửa lại tool. Target cũng nên tính tớiorg_unit/customer/project, không chỉproject.Thêm 1 happy-path test đi xuyên qua
ProjectProviderResolver+build_providerthật. Hiện happy path đang injectGiteaIssueProvidertrực tiếp nên chưa chứng minh wiring production hoạt động end-to-end.Bổ sung evidence/test Audit: call thành công/deny phải có correlation/audit record, đồng thời không ghi credential. Nếu shared runtime đã có test này thì link lại test/evidence trong PR là được.
Acceptance Criteria hiện chỉ nhận heading tiếng Anh. Task Core Team sau này chủ yếu dùng tiếng Việt nên support thêm
Tiêu chí hoàn thành/Tiêu chí chấp nhậnhoặc có cách parse chuẩn hơn, tránh fallback lấy nhầm checklist của section khác.Ngoài ra check thêm output từ Gitea phải đi qua Untrusted Content Fence của Cowork trước khi đưa vào Agent; nếu đã có ở shared runtime thì ghi rõ evidence/link test trong PR.
Sau các điểm trên PR này đủ tốt để approve cho FSG pilot. 👍
create feature mcp-issue-contextto create feature mcp-issue-context & project knowledge searche0ab653a61to3bd58b4ecb3bd58b4ecbtoe5fa21ecfdNew commits pushed, approval review dismissed automatically according to repository settings