Skip to content

fix(release): harden cloud validation and metadata#179

Merged
avivsinai merged 3 commits intomasterfrom
avivsinai/fix/release-hardening-0-26-1
Apr 22, 2026
Merged

fix(release): harden cloud validation and metadata#179
avivsinai merged 3 commits intomasterfrom
avivsinai/fix/release-hardening-0-26-1

Conversation

@avivsinai
Copy link
Copy Markdown
Owner

Summary

  • escape BBQL string literals and reject invalid pipeline UUIDs locally
  • fail clearly on diffstat truncation and preserve real git remote errors
  • stop embedding Cloud OAuth consumer credentials in public build artifacts
  • sync release docs/metadata and teach the release scripts to maintain changelog compare links

Verification

  • go test ./...
  • make check-skills
  • make build
  • bash -n scripts/release.sh scripts/check-release-version.sh
  • ./scripts/check-release-version.sh v0.26.0
  • goreleaser release --snapshot --clean --skip=publish

Escape BBQL string literals, reject invalid pipeline UUIDs locally, surface diffstat truncation, and preserve real git remote failures. Remove Cloud OAuth client credential embedding from public build artifacts and tighten the release metadata checks so changelog compare links stay current.

Co-Authored-By: Codex <noreply@openai.com>
@avivsinai avivsinai enabled auto-merge (squash) April 22, 2026 16:26
avivsinai and others added 2 commits April 22, 2026 19:26
Regenerate the auth rule markdown after updating the Cloud login help text so the checked-in skill output matches the command source used by CI.

Co-Authored-By: Codex <noreply@openai.com>
Staticcheck rejects capitalized error strings. Keep the new Cloud OAuth guidance, but phrase it as a conventional Go error so CI stays green.

Co-Authored-By: Codex <noreply@openai.com>
@sentry
Copy link
Copy Markdown

sentry Bot commented Apr 22, 2026

Codecov Report

❌ Patch coverage is 81.81818% with 14 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/bbcloud/client.go 76.92% 6 Missing and 3 partials ⚠️
internal/remote/remote.go 57.14% 2 Missing and 1 partial ⚠️
pkg/bbcloud/pullrequests.go 0.00% 1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@avivsinai avivsinai merged commit 821185b into master Apr 22, 2026
10 checks passed
@avivsinai avivsinai deleted the avivsinai/fix/release-hardening-0-26-1 branch April 22, 2026 17:19
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