Skip to content

chore(deps): bump kai-engine to v0.6.70 - #108

Merged
jschatz1 merged 1 commit into
mainfrom
chore/bump-engine-screenshot-locates
Sep 11, 2026
Merged

chore(deps): bump kai-engine to v0.6.70#108
jschatz1 merged 1 commit into
mainfrom
chore/bump-engine-screenshot-locates

Conversation

@jschatz1

Copy link
Copy Markdown
Member

Summary

  • Pins kai-engine v0.6.70, which carries kaicontext/kai-engine#97: a screenshot locates, it does not name (doctrine + caption guard + .css in the ui-verify gate + workspace-edit reporting).
  • Opened by hand: the adopt-engine workflow's bot push to main is refused by the pull_request ruleset (its last three runs failed at the push step).

Tests

  • CGO_ENABLED=1 go build ./... and go test -timeout 5m -skip TestRunCompletion ./... green locally, the same commands the adoption workflow runs.

🤖 Generated with Claude Code

Brings in the harness change "a screenshot locates, it does not name"
(kaicontext/kai-engine#97): doctrine for screenshot-driven tasks, the
caption guard, .css as a renderer file for the ui-verify gate, and the
outer agent reporting workspace edits as workspace edits.

Done by hand because adopt-engine's bot push is refused by the
pull_request rule on main (last three runs).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

jschatz1 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@coderabbitai

coderabbitai Bot commented Sep 11, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 25204c53-add5-44a8-8535-dc186e5dc4cb


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@kaicontext kaicontext Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kai review

Kai Summary

Read the whole thing — it does what it says, and nothing jumped out at me. ✅

A clean dependency pin that moves kai-engine to v0.6.70 with no local source edits; correctness rests on the engine's contract, which is outside this repo and unverifiable here.

Important files changed
File Change
go.mod modified · +1 −1
go.sum modified · +2 −2
What I opened — 2 files, 6 turns, 1m7s
  • go.mod
  • go.sum
Full read-through

Scope: read kaicontext/kai-cli at the working tree (commit 20d4b9d), including go.mod and the full go.sum. The diff is a dependency pin only — go.mod/go.sum — moving github.com/kaicontext/kai-engine from a pre-release pseudo-version (v0.6.69-0.20260910160213-68f0588df199) to the tagged v0.6.70. I could not read github.com/kaicontext/kai-engine itself; everything the bump actually delivers (screenshot doctrine, caption guard, .css renderer-file handling for the ui-verify gate, workspace-edit reporting) lives in that module and rests on its v0.6.70 contract, which is outside this repo.

The change re-points the one require entry (go.mod:64, now v0.6.70) and updates the two corresponding go.sum lines (112–113) — the module zip hash (h1:kN2AVoVDbp20IrCsYHRABWvtQYKKBUNkSTPygx65ahg=) and the go.mod hash (h1:ZHSy+4n9n6o3zSTTrRwRhwY0A8KFg7KjBxj1aKeoneY=). The go.mod hash is unchanged in value from the previous version, which is expected: kai-engine's own go.mod content did not move between the pseudo-version and the tag, so its h1: is identical; only the module zip hash changed. There is no replace directive anywhere in go.mod that would shadow the pinned version, and the single require block matches the single go.sum entry pair. The go.sum is otherwise internally consistent — no orphaned or duplicate kai-engine lines remain (the old pseudo-version lines were replaced, not left alongside), and every module with a require in go.mod has a matching h1:/go.mod h1: pair. No Go source in this repo is touched; the api/* shim packages and cmd/kai/* consumers import github.com/kaicontext/kai-engine/<pkg> by version-agnostic path, so there are no call-site edits to miss and no signature breakage visible from this side. The build/test claim (go build ./..., go test -skip TestRunCompletion ./... green) is the load-bearing check, and nothing in the diff contradicts it; I could not re-run it.

I found no defects. The unfixable caveat is that I cannot verify v0.6.70's contract — whether any exported symbol the ~38 importing files reference changed shape — but that is a stated limitation of a dependency bump into a module whose source this checkout does not hold, not a finding against this change. There is no evidence here of a missed call-site update, a conflicting replace, or a malformed go.sum.

One process note, not a defect: the author states the bump was done by hand because the adopt-engine workflow's bot push to main is refused by the pull_request ruleset (last three runs failed). That explains this change but means the next automated adoption will hit the same wall until the ruleset or workflow is fixed, so manual bumps will keep being needed.

Ready to merge as-is — it's a clean, internally consistent version-string bump with no local code edits to review and no conflicting replace directive.

+3 −3 · 2 files · reaches 0 · the full analysis
💬 Reply to any of my comments and I'll answer, or say @kaicontext anywhere on this PR — a question, or "take another look at the retry logic".

@jschatz1
jschatz1 merged commit 0a5a971 into main Sep 11, 2026
8 checks passed
@jschatz1
jschatz1 deleted the chore/bump-engine-screenshot-locates branch September 11, 2026 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant