chore(deps): refresh eligible parser and artifact action pins - #199
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review in progressClawSweeper is reviewing this revision. This supersedes any previous blocked status. |
|
Codex review: needs real behavior proof before merge. Reviewed September 13, 2026, 11:47 AM ET / 15:47 UTC. ClawSweeper reviewWhat this changesUpdates an indirect C-parser dependency to v4.29.6, pins the automation guide’s artifact-upload example to v7.0.1, and records the refresh in the changelog. Merge readiness⛔ Blocked before merge - 2 items remain The refresh remains distinct from main and v0.9.6. No actionable patch defect was found, but the promised real CLI validation has not yet been recorded. Priority: P3 Review scores
Verification
How this fits togetherGitcrawl stores searchable GitHub archives in SQLite; this refresh changes dependency selection in its Go module graph. The separate documentation example uploads generated triage reports as GitHub Actions artifacts. flowchart LR
A[Go module pins] --> B[SQLite dependency graph]
B --> C[Gitcrawl archive CLI]
C --> D[JSON triage reports]
D --> E[Documented artifact upload]
E --> F[GitHub Actions artifact]
Before merge
Agent review detailsSecurityNone. Review metricsNone. Technical reviewBest possible solution: Keep the refresh narrowly scoped and substantiate the unchanged archive behavior with the promised built-CLI validation. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR refreshes dependency pins rather than reporting broken existing behavior. Is this the best way to solve the issue? Yes: the small module override and immutable action pin fit the stated maintenance goal without changing SQLite/libc versions or application code. AGENTS.md: not found in the target repository. Codex review notes: model internal, reasoning medium; reviewed against 223f23dd3b75. 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 remaining eligible dependency pins after the previous sweep: modernc.org/cc/v4 moves from v4.29.2 to v4.29.6, and the automation guide now uses the verified SHA for actions/upload-artifact v7.0.1 instead of v4. The parser is a transitive test dependency of SQLite's libc; application behavior is unchanged. Regenerated go.sum and added one Unreleased entry.
The three-day cooldown holds CrawlKit v0.16.2, golden's September 13 pseudo-version, and cc/v4 v4.29.7. SQLite still requires libc v1.75.6, which also prevents ccgo/v4 v4.35.2 (requires libc v1.75.7). Charm's v2 stack requires a coordinated view/input/color migration and live terminal coverage, so it remains outside this dependency refresh. Go 1.27.1, Node 26.8.2, GoReleaser 2.18.1, sqlc 1.31.1, govulncheck 1.8.0, deadcode 0.50.0, Docker images, and executable workflow action pins are already current. The shared release-workflow v1 alias resolves to v1.8.2.
Validation on head
d6aee98315ab227333c0fb951dbe1dcac6b3215b:GOWORK=off GOFLAGS=-mod=readonly go mod downloadandgo mod verifypassed; module files remain tidy.vcs.modified=false.fixture/deps --numbers 101 --include-comments(1 issue, 1 comment); keyword searchSQLitereturned issue fix(store): order hydration observations durably #101, “SQLite dependency proof”; thread reads, status JSON, version, and TUI help all succeeded. Only synthetic data and a dummy fixture credential were used.No tests were removed or weakened. No release, tag, application version bump, or manual deployment. No open Dependabot PR is superseded.
Supplemental local validation: smoke, release-script tests, and all documentation tests/build passed. The full local coverage run hit Go’s 10-minute package limit in CLI/store without assertion failures; the two cases interrupted by the package timer passed isolated rechecks (1.36s and 8.98s). Full-suite and coverage-gate evidence is the green Linux/macOS CI above.