build(deps): refresh dependencies - #112
Conversation
|
Codex review: needs maintainer review before merge. Reviewed August 1, 2026, 8:06 PM ET / August 2, 2026, 00:06 UTC. ClawSweeper reviewWhat this changesThe branch updates CrawlKit, SQLite-related Go modules, Merge readinessKeep this PR open for normal maintainer review: it is a focused, passing dependency and workflow maintenance patch from a collaborator, and the earlier changelog objection is not supported by the checked-in contribution guidance. The contributor’s reply correctly distinguishes the release workflow opening the next Priority: P3 Review scores
Verification
How this fits togetherSlacrawl is a Go CLI for local Slack archives. Its module manifest selects crawler, SQLite, and network libraries, while GitHub Actions run release snapshots and stale-item automation around the CLI. flowchart LR
A[Go module manifest] --> B[Dependency resolver]
B --> C[Slacrawl CLI]
C --> D[Local Slack archive]
D --> E[SQLite queries]
F[CI workflow pins] --> G[Release snapshots]
F --> H[Stale-item automation]
Before merge
Agent review detailsSecurityNone. Review metrics
Technical reviewBest possible solution: Merge the focused refresh through normal collaborator review, retaining the two Unreleased maintenance bullets and the stated live CLI validation as release context. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR is routine dependency and workflow maintenance rather than a report of broken user behavior. The contributor nevertheless supplied a real built-CLI validation path in the PR body. Is this the best way to solve the issue? Yes. Updating the selected module versions and existing Action pins is the narrow maintainable approach, and the changelog entries match the repository’s current Unreleased maintenance convention. AGENTS.md: not found in the target repository. Codex review notes: model internal, reasoning high; reviewed against 53dd8d09b584. LabelsLabel justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (1 earlier review cycle)
|
|
Keeping the two The substantive dependency/action diff, full validation, and live built-CLI proof are unchanged. The bot finding does not identify a code, build, or release-process defect. @clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
|
Landed via squash as Pre-merge verification at exact head
The normal squash merge was refused only because branch protection requires a review the author cannot self-approve, so the approved admin override was used. The remote PR branch has been deleted. |
Summary
golang.org/x/netfrom 0.55.0 to 0.57.0actions/stalefrom v10 to v11 and the pinned GoReleaser action from 7.2.2 to 7.2.3Verification
actionlintmake check(module verification/tidy, formatting, vet, govulncheck, deadcode, all Go tests, CLI smoke, and four-platform GoReleaser snapshot)govulncheck: no vulnerabilities foundslacrawl sqlThe first final
make checkattempt hit a TLS timeout fetching the vulnerability database; the unchanged retry completed successfully.