fix(pipeline): incorporate unresolved review comments into CI gate and auto-fix by khaira777 · Pull Request #890 · kunchenguid/no-mistakes

feat(ci): integrate unresolved review comments into issue detection and readiness gate (#870)

khaira777

committed

a736c1e

no-mistakes(review): Fix review dedup, findings, bot coverage; focused tests pass

khaira777

committed

8a7de8b

no-mistakes(review): Support GitHub Advanced Security review comments; focused tests pass

khaira777

committed

236198a

no-mistakes(review): Fix review pagination, timeout state, IPC bounds; tests pass

khaira777

committed

d705c6b

no-mistakes(review): Preserve review evidence, sanitize controls; focused tests pass

khaira777

committed

a910275

no-mistakes(review): Bound provider errors, preserved CI context; tests pass

khaira777

committed

0afe162

no-mistakes(review): Respect review policy, selection, redaction, and dedup

khaira777

committed

ca99482

no-mistakes(review): Split review budgets, preserve scope, and retain review findings

khaira777

committed

935f2f8

no-mistakes(review): Persist review state and enforce scoped repair budgets

khaira777

committed

cba746e

no-mistakes(review): Block review autofix during unresolved CI reruns

khaira777

committed

4f9aa8c

no-mistakes(review): Gate review autofix on readiness; mark findings actionable

khaira777

committed

e7977d9

no-mistakes(review): Preserve review targets through CI fix timeouts

khaira777

committed

b0a57d6

no-mistakes(review): {"summary":"bound timeout review targets and validate PR head"}

khaira777

committed

142f4fa

no-mistakes(document): update CI step documentation for review-comment auto-fix budgets

khaira777

committed

64c653d

no-mistakes(ci): Diagnosed and resolved the failing checks: 1. Fixed `test (macos-latest)` package timeout in `internal/pipeline/steps` caused by a non-parallel 31-second sleep in `TestIntentStep_S…

khaira777

committed

31c4539

no-mistakes(ci): Diagnosed and resolved the failing CI checks: 1. `test (macos-latest)` package timeout in `internal/pipeline/steps`: Tests in `ci_autofix_test.go` and `ci_checks_test.go` lacked pa…

khaira777

committed

72fa550

no-mistakes(ci): Fixed Greptile Review failure by requesting `pullRequest { headRefOid }` in the nested review thread comments GraphQL query (`reviewThreadCommentsQuery`), validating the PR head ac…

khaira777

committed

1716bec

fix(scm/github): fail closed when review comments response lacks PR head

khaira777

committed

f762362

no-mistakes(document): document CI review-comment gate and auto-fix budgets

khaira777

committed

06e4e50

no-mistakes(ci): Diagnosed and fixed test package timeout in `internal/pipeline/steps`. In `TestCIStep_ReviewAutoFixWaitsForKnownReadiness`, the poll loop previously executed with an unmocked clock…

khaira777

committed

261c629

Navigation Menu

fix(pipeline): incorporate unresolved review comments into CI gate and auto-fix - #890

fix(pipeline): incorporate unresolved review comments into CI gate and auto-fix#890
khaira777 wants to merge 20 commits into
kunchenguid:mainfrom
khaira777:fix/870-review-comments-gate

Commits on Aug 29, 2026

feat(ci): integrate unresolved review comments into issue detection and readiness gate (#870)

no-mistakes(review): Fix review dedup, findings, bot coverage; focused tests pass

no-mistakes(review): Support GitHub Advanced Security review comments; focused tests pass

no-mistakes(review): Fix review pagination, timeout state, IPC bounds; tests pass

no-mistakes(review): Preserve review evidence, sanitize controls; focused tests pass

no-mistakes(review): Bound provider errors, preserved CI context; tests pass

no-mistakes(review): Respect review policy, selection, redaction, and dedup

no-mistakes(review): Split review budgets, preserve scope, and retain review findings

no-mistakes(review): Persist review state and enforce scoped repair budgets

no-mistakes(review): Block review autofix during unresolved CI reruns

no-mistakes(review): Gate review autofix on readiness; mark findings actionable

no-mistakes(review): Preserve review targets through CI fix timeouts

no-mistakes(review): {"summary":"bound timeout review targets and validate PR head"}

no-mistakes(document): update CI step documentation for review-comment auto-fix budgets

no-mistakes(ci): Diagnosed and resolved the failing checks: 1. Fixed `test (macos-latest)` package timeout in `internal/pipeline/steps` caused by a non-parallel 31-second sleep in `TestIntentStep_S…

no-mistakes(ci): Diagnosed and resolved the failing CI checks: 1. `test (macos-latest)` package timeout in `internal/pipeline/steps`: Tests in `ci_autofix_test.go` and `ci_checks_test.go` lacked pa…

no-mistakes(ci): Fixed Greptile Review failure by requesting `pullRequest { headRefOid }` in the nested review thread comments GraphQL query (`reviewThreadCommentsQuery`), validating the PR head ac…

fix(scm/github): fail closed when review comments response lacks PR head

no-mistakes(document): document CI review-comment gate and auto-fix budgets

no-mistakes(ci): Diagnosed and fixed test package timeout in `internal/pipeline/steps`. In `TestCIStep_ReviewAutoFixWaitsForKnownReadiness`, the poll loop previously executed with an unmocked clock…