build(deps): dependency-submission action v6, and record that main does not build - #207
Merged
Merged
Conversation
Closes cross-repo drift rather than chasing a new release. Dependabot opened this bump in streambuffer only (#156, merged 2026-09-09); the other three sibling repos stayed on v5 until their own weekly Dependabot run, so the four pipelines that are meant to run the same toolchain briefly did not. v6.0.0 moves the action runtime from Node 20 to Node 24 and is already proven in this workspace: streambuffer's Report job -- the job that actually runs the action, with no continue-on-error on the step -- went green on it. Workflow-file change only; no source, no dependency, no test touched. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AnNYn8W1xuVxVJtyL34GyH
Found during a cross-repo dependency sweep, not by CI going red — CI has been red since 2026-09-01 and nobody could see it. srcmorph/pom.xml pins <llama.version>5.2.0</llama.version> (5b4abeb, "pin net.ladenthin:llama to the 5.2.0 release, not the snapshot"). That release does not exist: Central's newest net.ladenthin:llama is 5.1.0 and llama-5.2.0.pom returns HTTP 404. This reactor declares no repository besides Central, so the dependency cannot resolve anywhere except a machine where an earlier `mvn install` of java-llama.cpp left 5.2.0 in ~/.m2 -- which is exactly the state a local `mvn verify` runs in, and why commit messages from that window report a green reactor build in good faith. Why it stayed invisible for eight days: every PR run since then fails at the first Maven step (run 33962068246, job 101297417993 -- "Could not find artifact net.ladenthin:llama:jar:5.2.0 in central", BUILD FAILURE after 3.35 s), but the merge commits' main runs were all cancelled by the start gate, so the repo never showed a red main and the PR failures were attributed to whatever else was in flight at the time. Recorded rather than fixed because the fix is a real decision, not a revert: 229903c raised the pin from 5.1.0 to 5.2.0-SNAPSHOT precisely because the newer binding can express flashAttn, so dropping back to 5.1.0 may not compile. java-llama.cpp's main is at 5.2.0-SNAPSHOT, so publishing that release is the likelier fix and is the owner's call. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AnNYn8W1xuVxVJtyL34GyH
bernardladenthin
had a problem deploying
to
maven-central
September 9, 2026 12:07 — with
GitHub Actions
Failure
bernardladenthin
had a problem deploying
to
maven-central
September 9, 2026 12:07 — with
GitHub Actions
Failure
Review Summary✅ Excellent documentation of a critical blockerBoth commits are well-executed: 1. Action version bump (
2. Build blocker documentation (
Technical assessment
Ready to merge. The critical fix step (updating java-llama.cpp or reverting the pin) is separate, but documenting the problem this thoroughly unblocks whoever takes that decision. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two commits from a cross-repo dependency sweep on 2026-09-09. The second one matters far more than the first.
mainhas not been buildable since 2026-09-01srcmorph/pom.xmlpins<llama.version>5.2.0</llama.version>(commit5b4abeb, "pin net.ladenthin:llama to the 5.2.0 release, not the snapshot") — but that release was never published. Central's newestnet.ladenthin:llamais5.1.0, andllama-5.2.0.pomreturns HTTP 404. This reactor declares no repository besides Central, so nothing can resolve it.Proven two independent ways:
and in CI — run
33962068246, job101297417993:Why it stayed invisible for eight days. Every PR run since 2026-09-01 dies at that first Maven step, but the merge commits'
mainruns were allcancelledby the start gate — so the repo never showed a redmain, and the PR failures were attributed to whatever else was in flight. It also passes on any machine where an earliermvn installof java-llama.cpp left 5.2.0 in~/.m2, which is exactly the state a localmvn verifyruns in; that is why "verified locally: reactor mvn clean verify green" appears, in good faith, in commit messages from that window.Recorded, not fixed — the fix is a decision, not a revert.
229903chad raised the pin from5.1.0to5.2.0-SNAPSHOTprecisely because the newer binding "can express" flashAttn, so dropping back to5.1.0may not compile. java-llama.cpp'smainsits at5.2.0-SNAPSHOT, so publishing that release is the likelier fix. That call is the owner's, so this PR adds aTODO.mdentry with the reproduction rather than guessing.advanced-security/maven-dependency-submission-actionv5 → v6Closing cross-repo drift: Dependabot opened this in streambuffer only (bernardladenthin/streambuffer#156, merged), leaving the other three on v5 until their own staggered weekly run.
v6.0.0moves the runtime from Node 20 to Node 24, and streambuffer'sReportjob — where the action actually runs, nocontinue-on-error— went green on it first.Expect this PR's CI to fail on the dependency resolution above; that failure is the subject of the second commit, not a regression from either change here.
🤖 Generated with Claude Code
https://claude.ai/code/session_01AnNYn8W1xuVxVJtyL34GyH
Generated by Claude Code