Follow-up to #741 (pull kata task context into reviews).
The kata-integration work wires kata task context (kata_context.mode = current|open) into:
- daemon post-commit reviews (worker prompt builder)
- foreground
roborev review
- the daemon-free batch reviewer (
internal/review/batch.go)
The CI poller review path was intentionally left out of that change. The CI builders (internal/daemon/ci_poller.go:128 and :2690) construct prebuilt prompts via BuildWithAdditionalContextAndDiffFile with a diff-file placeholder and additional-context behavior, so wiring kata there warrants its own change and tests.
What's needed
- Attach
kata.NewCLIClient(repoPath) to the CI poller prompt builders so configured kata_context applies to CI PR reviews.
- Verify behavior when kata is unavailable on the CI host or the repo is unbound:
kata.ResolveContext is inert in those cases, so the default (mode off) and unbound repos are unaffected.
- Decide whether CI reviews of external/forked PRs should resolve task context at all (a self-hosted CI using kata vs. reviewing forks without kata bindings) and gate accordingly if needed.
Context: kata-integration branch implementing #741 / #749.
Follow-up to #741 (pull kata task context into reviews).
The kata-integration work wires kata task context (
kata_context.mode = current|open) into:roborev reviewinternal/review/batch.go)The CI poller review path was intentionally left out of that change. The CI builders (
internal/daemon/ci_poller.go:128and:2690) construct prebuilt prompts viaBuildWithAdditionalContextAndDiffFilewith a diff-file placeholder and additional-context behavior, so wiring kata there warrants its own change and tests.What's needed
kata.NewCLIClient(repoPath)to the CI poller prompt builders so configuredkata_contextapplies to CI PR reviews.kata.ResolveContextis inert in those cases, so the default (modeoff) and unbound repos are unaffected.Context: kata-integration branch implementing #741 / #749.