Skip to content

chore: improve ci#5

Merged
yordis merged 1 commit into
mainfrom
improve-ci
Sep 18, 2025
Merged

chore: improve ci#5
yordis merged 1 commit into
mainfrom
improve-ci

Conversation

@yordis

@yordis yordis commented Sep 17, 2025

Copy link
Copy Markdown
Member

No description provided.

@coderabbitai

coderabbitai Bot commented Sep 17, 2025

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

This PR updates release automation configuration, renames CI/CD workflow display names, adds a release workflow to build and publish native and WASM Buf plugins on GitHub Releases, ignores the WASM artifact in .gitignore, and introduces Taskfile tasks to build native and WASM plugin binaries.

Changes

Cohort / File(s) Summary of Changes
Release automation config
.github/.release-please-config.json
Enabled component in tag; set draft/prerelease false; added PR header; set tag separator to "@"; enabled separate PRs; defined package mapping for "." with component "protoc-gen".
CI workflow rename
.github/workflows/ci.yml
Renamed workflow display name from "CI" to "ci"; no functional changes.
CD workflows
.github/workflows/release-please.yml, .github/workflows/release.yml
Renamed Release Please workflow to "cd / release please". Added new "cd" workflow to build and publish native and WASM Buf plugins on release events using Buf CLI and 1Password-loaded secrets.
Build scripts & ignore
Taskfile.yml, .gitignore
Removed clean task; added build-plugin and build-plugin-wasm tasks. Ignored protoc-gen-connect-go-servicestruct.wasm.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Dev as GitHub Release
  participant WF as GHA Workflow "cd"
  participant OP as 1Password App
  participant Go as Go Toolchain
  participant Buf as Buf CLI
  participant BSR as Buf Registry (buf.build)

  Dev->>WF: release(created: released/prereleased)
  WF->>OP: Load OP_SERVICE_ACCOUNT_TOKEN, BUF_TOKEN
  OP-->>WF: Secrets
  WF->>Buf: buf registry login (BUF_TOKEN)
  par Native build
    WF->>Go: go build ./cmd/protoc-gen-connect-go-servicestruct
    Go-->>WF: protoc-gen-connect-go-servicestruct
    WF->>Buf: push native binary with labels (tag, latest)
    Buf-->>BSR: Publish native
  and WASM build
    WF->>Go: GOOS=wasip1 GOARCH=wasm go build ./cmd/...
    Go-->>WF: protoc-gen-connect-go-servicestruct.wasm
    WF->>Buf: push wasm binary with labels (tag-wasm, latest-wasm)
    Buf-->>BSR: Publish wasm
  end
  BSR-->>WF: Publish results
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20–30 minutes

Possibly related PRs

Poem

I twitch my nose at tags that sing,
With @-separators, releases spring.
I hop from Go to WASM land,
Buf-bound bundles, neatly planned.
On push I thump—two artifacts fly,
A carrot-shaped cloud in the registry sky. 🥕🚀

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch improve-ci

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ab4d219 and 375a454.

📒 Files selected for processing (6)
  • .github/.release-please-config.json (2 hunks)
  • .github/workflows/ci.yml (1 hunks)
  • .github/workflows/release-please.yml (1 hunks)
  • .github/workflows/release.yml (1 hunks)
  • .gitignore (1 hunks)
  • Taskfile.yml (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@yordis
yordis force-pushed the improve-ci branch 4 times, most recently from 470ba23 to 162b826 Compare September 18, 2025 03:33
@yordis yordis changed the title improve ci chore: improve ci Sep 18, 2025
@yordis
yordis marked this pull request as ready for review September 18, 2025 03:34
Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
@yordis
yordis merged commit 19f520c into main Sep 18, 2025
2 checks passed
@yordis
yordis deleted the improve-ci branch September 18, 2025 03:34
@coderabbitai coderabbitai Bot mentioned this pull request Apr 28, 2026
5 tasks
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