Skip to content

ci: publish auth-provider, not auth-auth-provider - #7

Merged
y1o1 merged 1 commit into
developfrom
ci/image-name-auth-provider
Jul 26, 2026
Merged

ci: publish auth-provider, not auth-auth-provider#7
y1o1 merged 1 commit into
developfrom
ci/image-name-auth-provider

Conversation

@y1o1

@y1o1 y1o1 commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

The name said nothing twice

The image name was assembled as auth-${{ matrix.generator }}, and one generator is already called auth-provider. So the registry carried:

ghcr.io/provin-line/auth-auth-provider

The doubling was an artifact of building the string, not a distinction anyone intended.

The fix

image is now stated per matrix entry instead of derived:

generator image
policy-verifier auth-policy-verifier
auth-provider auth-provider

That keeps the prefix where it does work — whose policy verifier is a real question about a generic-sounding component — and drops it where the component already says auth. Asymmetric to look at, correct in both cases.

Why now

This breaks anyone pinning the old name, which is precisely the argument for doing it today: the packages became public minutes ago and no release has been announced, so the set of people who could be pinning it is empty. Every day after this, it gets more expensive and eventually not worth doing at all.

What follows this PR

GHCR has no rename and no alias, so the operation is publish-the-right-name, then delete-the-wrong-one:

  1. this PR lands
  2. workflow_dispatch on publish-images republishes both images (no new version tag needed — the workflow already supports manual dispatch)
  3. the new auth-provider package is created private (this repo still is), so its visibility needs flipping by hand
  4. provin.oss's quickstart compose and the publication runbook move to the new name
  5. auth-auth-provider is deleted rather than left as a stale duplicate for someone to find later

All of it has to complete before the oss flip, because that is when the quickstart becomes the public front door and needs images that exist.

CHANGELOG.md's existing entry naming the old image is left alone — it records what was true when it was written. A new Unreleased / Changed entry documents the break.

🤖 Generated with Claude Code

The image name was built as auth-${{ matrix.generator }}, and one generator is
already called auth-provider — so the registry carried
ghcr.io/provin-line/auth-auth-provider, a name that says nothing twice. The
doubling was an artifact of assembling the string, not a distinction anyone
intended.

The name is now stated per matrix entry rather than derived. That keeps the
prefix where it does work — auth-policy-verifier, because "whose policy
verifier?" is a real question about a generic-sounding component — and drops it
where the component already says auth.

Breaking for anyone pinning the old name, which is exactly why it happens now:
the packages went public minutes ago and no release has been announced, so the
set of people who could be pinning it is empty. GHCR has no rename and no
alias, so the operation is publish-the-right-name and delete-the-wrong-one;
after this lands, a workflow_dispatch republishes and auth-auth-provider gets
deleted rather than left as a stale duplicate for someone to find later.

CHANGELOG.md's existing entry for the old name is left alone — it records what
was true when it was written.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@y1o1
y1o1 merged commit 81b466a into develop Jul 26, 2026
3 checks passed
@y1o1
y1o1 deleted the ci/image-name-auth-provider branch July 26, 2026 14:49
y1o1 added a commit to provin-line/oss that referenced this pull request Jul 26, 2026
Renamed upstream (provin-line/auth#7): the old auth-auth-provider was an
`auth-` prefix concatenated onto a generator already called auth-provider, so
the registry carried a name that said nothing twice.

Both references move — the compose file that pulls it, and the source-build
fallback Dockerfile's comment describing what compose uses. Nothing else in
this repository named it.

Done before this repository goes public, while the set of people who could be
pinning the old name is still empty.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.

1 participant