Skip to content
This repository was archived by the owner on Aug 20, 2026. It is now read-only.

Don't gate release/tag pipelines on the ambient CVE backlog - #19

Merged
satsura merged 1 commit into
mainfrom
fix/dont-gate-tag-pipelines
Aug 20, 2026
Merged

Don't gate release/tag pipelines on the ambient CVE backlog#19
satsura merged 1 commit into
mainfrom
fix/dont-gate-tag-pipelines

Conversation

@satsura

@satsura satsura commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Summary

Caught live on the first real release after #18: a GitLab tag pipeline's CI_PIPELINE_SOURCE is "push", same as a default-branch push, so scope detection fell through to default_branch and gated unscoped — startmatter/up's v1.1.0 release failed cerberus over 4 pre-existing critical dependency CVEs, unrelated to that release (deploy and create-release both succeeded fine).

Adds a tag scope (GitLab: CI_COMMIT_TAG set; GitHub: GITHUB_REF starts with refs/tags/) that still runs the scan for visibility but never gates — the tagged commit already earned its gate on the merge request.

Test plan

  • npm run typecheck, npm test (36/36), npm run build
  • CI on this PR

A GitLab tag pipeline's CI_PIPELINE_SOURCE is "push", same as a
default-branch push — so scope detection was falling through to
"default_branch" and gating unscoped on every tag. Caught live: v1.1.0
on startmatter/up failed cerberus over 4 pre-existing critical
dependency CVEs unrelated to that release, even though deploy and
create-release both succeeded fine.

The tagged commit already scanned and gated on its merge request.
Re-running an unscoped sweep on every release just makes shipping
hostage to the entire dependency tree's current CVE state rather than
what that release actually changed. New "tag" scope still runs the
scan for visibility, just never fails the pipeline.
@satsura
satsura merged commit f5a6073 into main Aug 20, 2026
3 checks passed
@satsura
satsura deleted the fix/dont-gate-tag-pipelines branch August 20, 2026 11:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant