chore(deps): update CrawlKit and build with Go 1.27.1 - #170
Conversation
Bumps [github.com/openclaw/crawlkit](https://github.com/openclaw/crawlkit) from 0.14.7 to 0.14.8. - [Release notes](https://github.com/openclaw/crawlkit/releases) - [Changelog](https://github.com/openclaw/crawlkit/blob/main/CHANGELOG.md) - [Commits](openclaw/crawlkit@v0.14.7...v0.14.8) --- updated-dependencies: - dependency-name: github.com/openclaw/crawlkit dependency-version: 0.14.8 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
|
🦞👀 Pull request received. I will update this pull request when review starts. |
|
Codex review: blocked before merge. Reviewed September 4, 2026, 4:29 AM ET / 08:29 UTC. ClawSweeper reviewWhat this changesUpdates CrawlKit, SQLite, supporting dependencies and build tools, aligns source and Docker builds on Go 1.27.1, and documents the new macOS 13 minimum. Merge readiness⛔ Blocked before merge - 4 items remain The dependency update remains absent from main and v0.9.4. The revised branch addresses the previous Docker/documentation mismatch, and captured managed CodeQL checks pass; no concrete patch defect remains. Priority: P2 Review scores
Verification
How this fits togetherGitcrawl mirrors GitHub issues and pull requests into a local SQLite archive for search and maintainer workflows. CrawlKit supplies shared storage helpers, while the Go toolchain builds the distributed CLI. flowchart LR
A[GitHub responses] --> B[Gitcrawl sync]
B --> C[CrawlKit storage helpers]
C --> D[SQLite archive]
D --> E[Search and terminal interface]
F[Go toolchain and dependencies] --> B
Decision needed
Why: The compatibility change is explicit and the previous CodeQL concern has positive check evidence, but successful builds cannot decide the supported-platform policy. Before merge
Agent review detailsSecurityNone. Review metrics
Root-cause clusterRelationship: Members:
Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything. Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Ship a maintainer-approved Go 1.27.1 baseline with explicit macOS 13 requirements and preserved access to existing archives. Do we have a high-confidence way to reproduce the issue? Not applicable to the remaining maintenance proposal. Current source resolves the previously reported build-version mismatch; no failing current-main scenario was executed. Is this the best way to solve the issue? Yes, aligning the module, Docker builder and installation requirements is a focused solution; the supported-platform cutoff needs explicit acceptance. AGENTS.md: not found in the target repository. Codex review notes: model internal, reasoning high; reviewed against 3c9e6d58548f. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
History |
Merge the compatible refresh into the existing CrawlKit update. Align the module, Docker builder and source documentation on Go 1.27.1, retain SQLite's exact libc requirement, and document the new macOS 13 minimum. Apply only the formatting required by the new Go toolchain. Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Merge main after #169 and preserve both the cursor-fix thanks and the integrated dependency notes. Keep the Go 1.27.1 candidate separate from main until managed CodeQL and the full build/runtime checks confirm compatibility.
|
ClawSweeper status: review started. I am starting a fresh review of this pull request: chore(deps): update CrawlKit and build with Go 1.27.1 This is item 1/1 in the current shard. Shard 0/1. This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking. Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted. |
CrawlKit v0.14.8 requires Go 1.27. The original dependency bump raised go.mod while Docker and the source-build documentation still used Go 1.26.7; Docker could not download the module, and Go 1.27's formatter rejected one existing indentation block.
This candidate aligns go.mod, the Docker builder, and source-build documentation on Go 1.27.1. The source-build minimum rises from Go 1.26.7 to Go 1.27.1, and newly built macOS binaries require macOS 13 Ventura or newer. These requirements follow CrawlKit's Go 1.27 dependency and the Go toolchain's supported platforms. The application-code change is limited to the formatting required by Go 1.27.
The compatible updates from #171 are incorporated through normal merge commits: SQLite 1.58.0, go-runewidth 0.0.29, current pprof/golden snapshots, TruffleHog 3.97.4, and Dockerfile frontend 1.27. SQLite retains its exact required libc version, 1.75.6. The original Dependabot commit and #171's maintenance commits remain in the ancestry; no history was rewritten. Main's landed #169 cursor fix and @SebTardif's changelog credit are preserved.
#171 remains open until this integrated PR lands; it should not be merged separately.
Validation completed locally:
3c9e6d58548fa2655189b268a04e25f0ff586c0e: no actionable findings.make checkpasses with Go 1.27.1: tidy, formatting, vet, vulnerability scan, deadcode, full suite (85.6% coverage), CLI smoke, release-script tests, and all six snapshot targets. No vulnerabilities found.The named Go 1.27 hold is resolved by managed CodeQL on this candidate: CodeQL CLI 2.26.4 configured Go 1.27.1 with
GOTOOLCHAIN=local, then completed Go autobuild and analysis successfully. No CodeQL configuration or gate was bypassed.The macOS minimum follows the Go 1.27 platform requirement; the arm64 snapshot also reports
LC_BUILD_VERSION minos 13.0.Final prepared head:
bec2e085087e9e43c3db9eaf64e07643495a0de9, based on main3c9e6d58548fa2655189b268a04e25f0ff586c0e. All exact-head checks pass: Linux/macOS CI, Docker build/runtime smoke, managed CodeQL, and secret scanning. This PR remains open for maintainer review; #171 stays open until this candidate actually lands.