Skip to content

fix: Goのinstallで本体チェックサムと間接依存を揃える - #15

Merged
owayo merged 1 commit into
mainfrom
fix/issue-14-go-install-checksums
Oct 6, 2026
Merged

owayo merged 1 commit into
mainfrom
fix/issue-14-go-install-checksums

Conversation

@owayo

@owayo owayo commented Oct 6, 2026

Copy link
Copy Markdown
Owner

概要

Goプロジェクトで depup --install の後に go build が missing go.sum entry で失敗する問題を修正します。Goのinstallを go mod tidy に変更し、ソースに必要な go.mod と go.sum を揃えます。

Go 1.17以降、引数なしの go mod download はモジュール本体のハッシュを保存しません。download all でハッシュだけを追加しても、新版が必要とする間接依存のrequireが不足し得るため、tidyで両方を同期します。tidyによる未使用依存の削除、ディレクティブの変更、vendorの更新手順は英語・日本語の文書に記載しました。

検証

  • ローカルのfile proxyと隔離したGoキャッシュを使う回帰テストを追加(外部通信・新しい依存の追加なし)。Goがない環境ではこのテストをスキップ。
  • 修正前は新版の本体ハッシュの検証で失敗、修正後は成功。
  • 旧downloadの成功だけではreadonly buildが失敗することを確認。
  • install後の直接・間接依存の本体ハッシュとindirect requireを検証し、go build -mod=readonly ./... と go test -mod=readonly ./... が成功。
  • make ci: fmt-check / clippy / 全テスト。
  • ZIP fixtureの再生成で同一のバイト列になることを確認。
  • astro-sightの影響・API・未使用シンボル検査とCodexの差分レビュー。

Closes #14

引数なしのgo mod downloadはGo 1.17以降でモジュール本体のハッシュを
go.sumへ記録せず、更新直後のreadonly buildが失敗していた。
go mod tidyでgo.modとgo.sumをソースに合わせて同期する。

ローカルのGo proxyを使った回帰テストで旧処理の失敗、新しい間接依存と
本体ハッシュの追加、readonly build/testの成功を検証する。
英語と日本語の文書に依存整理とvendorの更新手順を記載する。
@owayo owayo self-assigned this Oct 6, 2026
@coderabbitai

coderabbitai Bot commented Oct 6, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5cbfe071-5d46-4677-9861-e14ba1040b8b
  • 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.

@owayo
owayo merged commit 7daf485 into main Oct 6, 2026
8 checks passed
@owayo
owayo deleted the fix/issue-14-go-install-checksums branch October 6, 2026 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant