Skip to content

ci(release): read description from Cargo.toml instead of hardcoding it - #199

Merged
getappz merged 1 commit into
masterfrom
worktree-release-1.3.0
Jul 15, 2026
Merged

getappz merged 1 commit into
masterfrom
worktree-release-1.3.0

Conversation

@getappz

@getappz getappz commented Jul 15, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • CodeRabbit finding from chore: bump version to 1.3.0 #198 (Major/Quick win): release.yml hardcoded the pre-rewrite crate description in both the Homebrew formula and Scoop manifest generation steps -- v1.3.0 would have shipped stale package-manager metadata despite Cargo.toml's description having just been updated.
  • Extracts the description from Cargo.toml at workflow runtime instead, so the two can't drift apart again.
  • update-homebrew had no checkout step at all (it only clones the separate homebrew-agentflare repo) -- added one so Cargo.toml is actually readable there.

Test plan

  • Verified the extraction command (grep -m1 '^description = ' Cargo.toml | cut -d'"' -f2) against the real Cargo.toml locally -- produces the correct string
  • CI status checks pass
  • After merge and the v1.3.0 tag lands: confirm the generated Homebrew formula / Scoop manifest carry the new description

Summary by CodeRabbit

  • Chores
    • Improved release packaging so Homebrew and Scoop manifests automatically use the project’s current description.
    • Reduced the risk of outdated or inconsistent package descriptions in generated manifests.

…coding it

CodeRabbit flagged this on #198: release.yml hardcoded the pre-rewrite
description in both the Homebrew formula and Scoop manifest generation
steps, so v1.3.0 would have shipped stale package-manager metadata even
though Cargo.toml's description was just updated. Extract it at runtime
instead so the two can't drift apart again. update-homebrew had no
checkout step at all (it only clones the separate homebrew-agentflare
repo) -- added one so Cargo.toml is actually available to read.
@getappz
getappz enabled auto-merge July 15, 2026 15:06
@coderabbitai

coderabbitai Bot commented Jul 15, 2026 •

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 6c847a81-8f65-4643-8635-dc67b963d17c

📥 Commits

Reviewing files that changed from the base of the PR and between 00a213b and 2886acb.

📒 Files selected for processing (1)
  • .github/workflows/release.yml

📝 Walkthrough

Walkthrough

The release workflow now checks out the repository for Homebrew generation and sources both Homebrew and Scoop manifest descriptions from Cargo.toml instead of hardcoded strings.

Changes

Release manifest generation

Layer / File(s) Summary
Synchronize package descriptions
.github/workflows/release.yml
The Homebrew job checks out the repository, and both Homebrew and Scoop generation scripts extract the package description from Cargo.toml.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: dependabot[bot]

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch worktree-release-1.3.0

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

@getappz
getappz merged commit f5a4127 into master Jul 15, 2026
12 of 13 checks passed
@getappz
getappz deleted the worktree-release-1.3.0 branch July 15, 2026 15:07
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