build(deps): refresh Go toolchain and storage dependencies - #106
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Codex review: needs real behavior proof before merge. Reviewed September 4, 2026, 5:05 AM ET / 09:05 UTC. ClawSweeper reviewWhat this changesUpdates SQLite, cryptography, and terminal-width dependencies, prefers Go 1.27.1 while retaining the Go 1.27.0 minimum, and documents the SQLite/libc pairing. Merge readiness⛔ Blocked before merge - 2 items remain This refresh remains useful: main and v0.14.8 retain the older dependency versions. No actionable patch defect was found, but the reported consumer runs need inspectable real-behavior evidence before merge. Priority: P2 Review scores
Verification
How this fits togetherCrawlkit supplies shared storage, snapshots, encrypted backups, and terminal browsing to downstream crawler applications. Its module dependencies provide the database engine, cryptography, and text-width handling used by those helpers. flowchart TD
A[Downstream crawler apps] --> B[Crawlkit shared helpers]
C[Go module versions] --> B
B --> D[SQLite storage and snapshots]
B --> E[Encrypted backups]
B --> F[Terminal text layout]
D --> G[Local archives]
E --> G
Before merge
Agent review detailsSecurityNone. Review metricsNone. Technical reviewBest possible solution: Retain the focused dependency refresh and existing Go minimum, with auditable consumer results supporting storage, backup, and terminal compatibility. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR refreshes dependencies rather than reporting a reproducible existing-behavior defect. Is this the best way to solve the issue? Yes: version and checksum updates with matching documentation are a narrow maintenance approach; no competing implementation or API expansion is introduced. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against cf646f20a726. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rank-up movesOptional improvements that raise the rating; they are not merge blockers.
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
|
Refresh the storage and terminal dependencies while keeping the supported Go minimum at 1.27.0. The preferred development toolchain is now Go 1.27.1; SQLite moves to v1.58.0, x/crypto to v0.56.0, and go-runewidth to v0.0.29.
SQLite v1.58.0 explicitly requires libc v1.75.6. This patch retains that exact runtime pairing and documents why libc must not be upgraded independently. The module's
goline remains 1.27.0, while itstoolchainline selects 1.27.1 for development and CI.Validation:
make checkpassed with Go 1.27.1: tidy, formatting, vet, deadcode, govulncheck, full unit/race suites, and all 25 release guard tests.GOTOOLCHAIN=go1.27.0 GOWORK=off go test -count=1 ./...passed all packages, preserving the documented floor.The dependency refresh is prepared for maintainer review. No release, tag, or deployment is included.