Skip to content

Remove relocation stub from active reactor + repo-wide URL cleanup#149

Merged
bernardladenthin merged 1 commit into
mainfrom
claude/llm-wiki-architecture-plan-0nvriu
Jul 17, 2026
Merged

Remove relocation stub from active reactor + repo-wide URL cleanup#149
bernardladenthin merged 1 commit into
mainfrom
claude/llm-wiki-architecture-plan-0nvriu

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

Summary

  • Removed the relocation stub (net.ladenthin:llamacpp-ai-index-maven-plugin) from the active reactor. It was published once at 1.0.4 as part of the 1.1.1 release, and the redirect was verified working end-to-end — a Maven resolution of the old coordinates from a completely clean local repository correctly follows the relocation through to srcmorph-maven-plugin:1.1.1 and its full dependency graph. The published artifact is permanent on Maven Central regardless of this repo's module list and will never need another release, so there's no reason to keep carrying it (and the versions:set -Dexcludes=... footgun it required) in active development.
  • Updated CLAUDE.md, README.md, TODO.md, CHANGELOG.md, docs/RELEASE.md, srcmorph-maven-plugin/README.md to reflect the 3-module reactor and the stub's now-historical status.
  • .github/workflows/publish.yml comment updated (no functional change needed — the stub never produced a jar, so no per-module CI step ever referenced it beyond the aggregate reactor build).
  • Found and fixed a real staleness bug along the way: srcmorph's own spotbugs-exclude.xml comments still referred to the sibling plugin module by its pre-rename name (llamacpp-ai-index-maven-plugin module) instead of srcmorph-maven-plugin module — missed during the original Step 9 rename.
  • Repo-wide GitHub link cleanup: renamed bernardladenthin/llamacpp-ai-index-maven-plugin URLs to the canonical bernardladenthin/srcmorph in README.md, CONTRIBUTING.md, SECURITY.md, SUPPORT.md, .github/ISSUE_TEMPLATE/config.yml, and REUSE.toml's package metadata. Left third-party service-specific identifiers untouched (SonarCloud project key, Snyk/FOSSA/Libraries.io/Scorecard/REUSE.software badges) since those may be registered under the old name independently of the GitHub repo and can't be verified from here. Updated the README's Maven Central + Snapshot badges to point at srcmorph-maven-plugin (the real, current plugin artifact) instead of the now-deprecated stub coordinates.

Test plan

  • mvn clean verify from repo root — BUILD SUCCESS
  • Reactor build order confirmed as 4 projects (parent + 3 real modules), no stub
  • Full repo-wide grep for lingering stub/old-URL references, classified and addressed (historical mentions in CHANGELOG.md/module READMEs left as accurate history; historical benchmark output files in docs/ai-index-benchmark/ deliberately untouched, consistent with existing precedent)
  • CI green on this branch (pending)

Related issues / PRs

Follow-up to #144#148 — closes out the relocation-stub lifecycle after confirming the redirect works.

Checklist

  • I have read CONTRIBUTING.md and CODE_OF_CONDUCT.md
  • My commits follow Conventional Commits
  • No security-sensitive changes

Generated by Claude Code

…eanup

The relocation stub (net.ladenthin:llamacpp-ai-index-maven-plugin) was
published once at 1.0.4 as part of the 1.1.1 release, and the redirect
was verified working end-to-end: a Maven resolution of the old
coordinates from a completely clean local repository correctly follows
the relocation through to srcmorph-maven-plugin:1.1.1 and its full
dependency graph. The published artifact is permanent on Maven Central
regardless of this repo's module list, and it will never need another
release, so there's no reason to keep carrying it (and the
`versions:set -Dexcludes=...` footgun it required) in active
development.

- Removed llamacpp-ai-index-maven-plugin/ and its <module> entry from
  the root pom.
- Updated CLAUDE.md, README.md, TODO.md, CHANGELOG.md, docs/RELEASE.md,
  srcmorph-maven-plugin/README.md to reflect the 3-module reactor and
  the stub's now-historical status; removed the now-moot versions:set
  exclusion caveat from TODO.md and CLAUDE.md.
- .github/workflows/publish.yml comment updated (no functional change
  needed - the stub never produced a jar, so no per-module CI step
  ever referenced it beyond the aggregate reactor build).
- Fixed a real staleness bug found along the way: srcmorph's own
  spotbugs-exclude.xml comments still called the sibling plugin module
  by its pre-rename name ("llamacpp-ai-index-maven-plugin module")
  instead of "srcmorph-maven-plugin module" - missed during the
  original Step 9 rename.
- Repo-wide GitHub link cleanup: renamed bernardladenthin/llamacpp-ai-index-maven-plugin
  URLs to the canonical bernardladenthin/srcmorph in README.md,
  CONTRIBUTING.md, SECURITY.md, SUPPORT.md, .github/ISSUE_TEMPLATE/config.yml,
  and REUSE.toml's package metadata. Left third-party service-specific
  identifiers untouched (SonarCloud project key, Snyk/FOSSA/Libraries.io/
  Scorecard/REUSE.software badges) since those may be registered under
  the old name independently of the GitHub repo and I can't verify
  whether they were updated. Updated the README's Maven Central +
  Snapshot badges to point at srcmorph-maven-plugin (the real, current
  plugin artifact) instead of the now-deprecated stub coordinates.

Verified: mvn clean verify - BUILD SUCCESS. Reactor build order confirmed
as 4 projects (parent + 3 real modules), no stub.
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 29483214795

Coverage remained the same at 90.779%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 2567
Covered Lines: 2404
Line Coverage: 93.65%
Relevant Branches: 925
Covered Branches: 766
Branch Coverage: 82.81%
Branches in Coverage %: Yes
Coverage Strength: 3.9 hits per line

💛 - Coveralls

@bernardladenthin
bernardladenthin merged commit bd7d291 into main Jul 17, 2026
20 of 25 checks passed
@bernardladenthin
bernardladenthin deleted the claude/llm-wiki-architecture-plan-0nvriu branch July 17, 2026 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants