| Files | Last commit | Last update |
|---|---|---|
test(discussions): assert repo comment order param + 404→exit3 test + docs -R wording (review P1/P2) 评审 P1:①TestListRepoDiscussionCommentsBuildsPathAndParams 未断言 order=time_desc query → 补 gotQuery 断言;②4 个 comment/reply 无 404→exit3 单测 → 补 org comments list 404→ExitNotFound。P2:project comments docs -R 写「必填」不准(ResolveRepo 可省略)→ 改「可省略自动推断」与 list/view 一致。 | 2 months ago | |
feat(cli): 统一 list 命令分页 flag(--limit + --per-page 全覆盖) issue #501:list 类命令分页 flag 命名分裂——issue/label/milestone/ repo list/repo log 仅 --limit;discussions(org+project) 仅 --per-page; pr list/actions 两者并存。agent 无法预知某命令用哪个 flag(§6.1)。 方案 A:所有 list 命令统一提供 --per-page(API page size)+ --limit (客户端结果上限,默认 30)。语义对齐 pr list 既有双 flag 模式 (resolvePerPage + trim + LimitSet/PerPageSet)。 改动: - issue/label/milestone/repo list/repo log(5 命令)补 --per-page + resolvePerPage/trim + PerPage<0 校验 - discussions list(org) + project list(2 命令)补 --limit + trim + Limit<=0 校验 - pr list / actions(已合规)不改 - docs/COMMANDS.md 更新 7 命令分页 flag 说明(修正 label list 旧误述) - 单测:issue list 补 resolvePerPage/trim/validation 表驱动测试; discussions org+project 修 fixture(Limit:30) + 补 limit 校验用例 §5.9.B:已查 gc-api-doc 确认 issues/labels/milestones 端点均支持 per_page(无功能缺口,仅 flag 命名分裂)。 Closes #501 | 2 months ago | |
feat(cli): 统一 list 命令分页 flag(--limit + --per-page 全覆盖) issue #501:list 类命令分页 flag 命名分裂——issue/label/milestone/ repo list/repo log 仅 --limit;discussions(org+project) 仅 --per-page; pr list/actions 两者并存。agent 无法预知某命令用哪个 flag(§6.1)。 方案 A:所有 list 命令统一提供 --per-page(API page size)+ --limit (客户端结果上限,默认 30)。语义对齐 pr list 既有双 flag 模式 (resolvePerPage + trim + LimitSet/PerPageSet)。 改动: - issue/label/milestone/repo list/repo log(5 命令)补 --per-page + resolvePerPage/trim + PerPage<0 校验 - discussions list(org) + project list(2 命令)补 --limit + trim + Limit<=0 校验 - pr list / actions(已合规)不改 - docs/COMMANDS.md 更新 7 命令分页 flag 说明(修正 label list 旧误述) - 单测:issue list 补 resolvePerPage/trim/validation 表驱动测试; discussions org+project 修 fixture(Limit:30) + 补 limit 校验用例 §5.9.B:已查 gc-api-doc 确认 issues/labels/milestones 端点均支持 per_page(无功能缺口,仅 flag 命名分裂)。 Closes #501 | 2 months ago | |
refactor(discussions): DRY print helpers into render package org/project list+view 的 printDiscussions/printDiscussion 是逐字节复制 (~33+24 行 ×2),未来格式变更需改 4 处。抽取 PrintDiscussions/ PrintDiscussion 到已有的 pkg/cmd/discussions/render 包,4 个调用点统一 引用,删除本地副本。 行为 bit-identical:函数体原样搬迁,仅包名/调用方变化。 Closes #500 | 2 months ago | |
refactor(discussions): DRY print helpers into render package org/project list+view 的 printDiscussions/printDiscussion 是逐字节复制 (~33+24 行 ×2),未来格式变更需改 4 处。抽取 PrintDiscussions/ PrintDiscussion 到已有的 pkg/cmd/discussions/render 包,4 个调用点统一 引用,删除本地副本。 行为 bit-identical:函数体原样搬迁,仅包名/调用方变化。 Closes #500 | 2 months ago | |
feat(discussions): discussion comments + replies (org+project) (#465,#466,#469,#470) 新增 gc discussions comments 子命令组(org+project 双 scope,评论+回复)。 端点按 §9/§5.9.B 查 gc-api-doc 确认 4 个 GET 端点,评论/回复 schema 一致。 - api/queries_discussions.go: DiscussionComment 结构 + ListOrg/RepoDiscussionComments + ListOrg/RepoDiscussionCommentReplies + appendDiscussionCommentParams(page/per_page/order) - pkg/cmd/discussions/comments/: org comments 组(list/replies,--org) - pkg/cmd/discussions/project/comments/: project comments 组(list/replies,-R) - pkg/cmd/discussions/render/: 共享 PrintComments(独立包打破 import cycle) - 注册 comments 到 discussions + project - 404→ExitNotFound(3)(与 view 一致,agent-friendly) - docs/COMMANDS.md: 4 个 comments 子命令节 - 单测: api 4 端点路径/params/parse + 4 命令 run 级(text/json/validation) Closes #465 Closes #466 Closes #469 Closes #470 | 2 months ago |