Skip to content

fix(build): tidy obsolete module checksums - #1700

Merged
dnlrsls merged 1 commit into
Gentleman-Programming:mainfrom
dnlrsls:fix/release-3.2.1-tidy
Oct 7, 2026
Merged

dnlrsls merged 1 commit into
Gentleman-Programming:mainfrom
dnlrsls:fix/release-3.2.1-tidy

Conversation

@dnlrsls

@dnlrsls dnlrsls commented Oct 7, 2026

Copy link
Copy Markdown
Member

🔗 Linked Issue

Closes #1699

🏷️ PR Type

  • type:bug — Bug fix
  • type:feature — New feature
  • type:question — Question requiring tracked work
  • type:docs — Documentation only
  • type:refactor — Code refactoring
  • type:chore — Maintenance
  • type:breaking-change — Breaking change

📝 Summary

  • Remove fourteen obsolete checksum rows with the release's Go 1.25.10 toolchain.
  • Restore go mod tidy -diff without changing go.mod, resolved dependency versions, runtime code, or the release guard.
  • Prepare the approved v3.2.1 recovery; preserve the already-published v3.2.0 tag and Cloud images.

📂 Changes

File Change
go.sum Remove obsolete checksum and /go.mod pairs for seven older golang.org/x module versions

🧪 Test Plan

  • Regression oracle: Go 1.25.10 go mod tidy -diff — RED exit1 before tidy, GREEN exit0/no diff afterward.
  • go list -m all before/after — identical module graph; git diff -- go.mod empty.
  • go mod verify — all modules verified.
  • git diff --check — pass; candidate is one file, 0 additions/14 deletions.
  • CGO_ENABLED=0 go build -o .release-tools/engram.exe -ldflags='-X main.version=3.2.1' ./cmd/engram with Go 1.25.10 — pass.
  • Independent verification — actual failed release logs match the removed rows; independent Go1.25.10 tidy check passes.
  • Native review review-bf1e5cf27864f6e6 approved and acknowledged for exact tree 04375400d24f8eecb6175dc1b8d0661a65849c5f.
  • Runtime unit tests: no new behavior/test introduced; broad unit/E2E/plugin/lint/platform suites run in GitHub CI, not duplicated locally for checksum cleanup.

🤖 Automated Checks

Required issue-reference, approved-issue, type-label, Unit Tests, E2E Tests, and Plugin Tests: pending until GitHub runs them. Applicable lint, Windows, and transient-artifact checks are also pending.

✅ Contributor Checklist

  • Linked approved issue fix(build): tidy module checksums to unblock core release #1699.
  • Exactly one type:* label.
  • Recorded actual regression and applicable local checks; candidate CI remains pending.
  • Documentation/help: N/A — no user-facing behavior, command, option, or message changed.
  • Conventional commit; no Co-Authored-By trailers.
  • Only go.sum enters this PR; cache, binary, local ODD tracking, and unrelated changes excluded.

💬 Notes for Reviewers

Release failure: https://github.com/Gentleman-Programming/engram/actions/runs/37561305090
Cloud v3.2.0 publication succeeded separately; this PR does not re-tag, bypass guards, publish npm, or merge automatically. A new v3.2.1 tag is planned only after explicit merge approval and successful merged-candidate checks.

Native acknowledgement succeeded; its optional candidate-view cleanup was deferred with candidate-view-git-failure. No out-of-band cleanup or new review was attempted.

Remove unused checksums with Go 1.25.10 so the release tidiness guard passes without changing resolved dependency versions.

Closes Gentleman-Programming#1699
@dnlrsls dnlrsls added the type:bug Bug fix label Oct 7, 2026
@coderabbitai

coderabbitai Bot commented Oct 7, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum, !go.sum

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 98c1dd8a-4548-4877-9b8c-c4ae45f2a5d1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@dnlrsls
dnlrsls added this pull request to the merge queue Oct 7, 2026
Merged via the queue into Gentleman-Programming:main with commit 2d4a150 Oct 7, 2026
19 of 20 checks passed
@barbatdev

Copy link
Copy Markdown
Contributor

Fleet review (qwen)

Closes #1699. Fix: tidy obsolete module checksums. go.sum contained 14 obsolete golang.org/x checksums blocking v3.2.0 release workflow. Removed obsolete entries only. go mod tidy -diff passes. 0 new behavior/test.

Suggestion (1)

Verdict: clean

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(build): tidy module checksums to unblock core release

2 participants