feat(targethasher): collapse bzlmod external targets to repo-level hash - #312

Open

yushan8 wants to merge 23 commits into

mainfrom

bzlmod-collapse

Commits on Sep 10, 2026

feat(targethasher): collapse bzlmod external targets to repo-level hash

yushan8
and
claude
committed
496cc84
Fix collapseBzlmodExternalTargets to respect excluded_files regex

yushan8
and
claude
committed
986eea4
Consolidate bzlmod/WORKSPACE external handling into else block and add tests

yushan8
and
claude
committed
e46077c
Use Bazel marker file hashes for bzlmod external target collapsing

yushan8
and
claude
committed
f419ccf
Return error when marker files can't be read instead of silent fallback

yushan8
and
claude
committed
fedc0e2
Address Uber Go style guide issues

yushan8
and
claude
committed
b765037
Pass OutputBase via NativeGraphRunnerParams instead of Bazel interface

yushan8
and
claude
committed
4b26933
Require OutputBase for bzlmod repos instead of silently skipping

yushan8
and
claude
committed
c761c91
Simplify marker reading: hash the whole file instead of parsing hex

yushan8
and
claude
committed
6cf661d
Hash only the first line of marker files

yushan8
and
claude
committed
81c3dba

Commits on Sep 11, 2026

Move OutputBase to Bazel interface, resolve after query

yushan8
and
claude
committed
4528d39
Extract shouldCollapse helper, update copyright year

yushan8
and
claude
committed
d1c7440
Fix documentation to match current implementation

yushan8
and
claude
committed
5ac76f3
Guard against nil targets, empty output_base, and invalid inputs

yushan8
and
claude
committed
439fc7b
Move markers.go to core/targethasher, revert Bazel interface change

yushan8
and
claude
committed
908d969
Move bazelOutputBase to core/bazel as exported OutputBase

yushan8
and
claude
committed
b5d6608
Remove trailing blank lines to fix lint

yushan8
and
claude
committed
6ac21c2
Unexport detectBazelExecutable, now only used within core/bazel

yushan8
and
claude
committed
c81f9f5
Fix detectBazelExecutable comment to say bazelisk

yushan8
and
claude
committed
3b8841e
Add test for bzlmod collapse path in graphrunner

yushan8
and
claude
committed
393c771
Hash full marker file (excluding ENV lines) for patch correctness

yushan8
and
claude
committed
7791cee
Address PR review comments

yushan8
and
claude
committed
0d1cf05
Error when bzlmod repo has targets but no marker file

yushan8
and
claude
committed
9241589