Skip to content

Fix invalid Docker image tag on PRs to main#46

Merged
mvalancy merged 1 commit into
developfrom
fix/docker-tag-invalid-on-pr
Jun 13, 2026
Merged

Fix invalid Docker image tag on PRs to main#46
mvalancy merged 1 commit into
developfrom
fix/docker-tag-invalid-on-pr

Conversation

@mvalancy

Copy link
Copy Markdown
Member

The build-and-push jobs failed on every PR targeting main (e.g. the develop→main promotion PR #45) with:

ERROR: failed to build: invalid tag "ghcr.io/graphdone/graphdone-web:-69c4029": invalid reference format

Cause: docker-publish.yml had type=sha,prefix={{branch}}-. On a pull_request event the metadata-action {{branch}} placeholder resolves to empty, so the tag became :-<sha> (leading dash → invalid reference).

Fix: use a static prefix=sha-, giving a valid sha-<shortsha> tag on branch pushes, version tags, and PRs alike. The other tags (type=ref,event=prpr-N, semver, latest on default branch) are unaffected.

This unblocks the prod image pipeline for PRs to main.

🤖 Generated with Claude Code

docker-publish.yml used `type=sha,prefix={{branch}}-`, but on a
pull_request event the metadata-action `{{branch}}` placeholder is empty,
so the computed tag became `ghcr.io/...:-<sha>` — an invalid reference
that failed every build-and-push job on PRs targeting main (blocking the
develop→main promotion / prod image pipeline). Use a static `sha-` prefix
so the tag is valid on branch pushes, tags, and PRs alike.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

🧪 Comprehensive Test Suite

  • Unit suites (Node 18.x & 20.x) — core, web, server, mcp-server: ✅ passed
  • Installer & deploy config: ✅ passed

Full-stack smoke gate runs in the CI workflow.

@mvalancy mvalancy merged commit 8d75ca4 into develop Jun 13, 2026
16 checks passed
@mvalancy mvalancy deleted the fix/docker-tag-invalid-on-pr branch June 13, 2026 20:32
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