fix: sync versions to 0.7.2 and migrate to .cz.yaml config by OzBenSimhonTraceloop · Pull Request #70 · traceloop/hub

OzBenSimhonTraceloop · 2025-10-16T14:16:03Z

Changes

Added helm/Chart.yaml:version to commitizen version_files (was missing, causing version drift)
Migrated .cz.toml → .cz.yaml to align with other repos
Synced all versions to 0.7.2 (Cargo.toml, helm/Chart.yaml, .cz.yaml)
Changed version_provider: cargo → commitizen (required for updating multiple files)

Problem This Fixes

There was a version mismatch between Docker images and Helm charts:

Docker Hub traceloop/hub: 0.7.2 ✅
Docker Hub traceloop/helm-hub: 0.5.0 ❌ (outdated)

This happened because the commitizen configuration was only updating Cargo.toml during version bumps, leaving helm/Chart.yaml and .cz.toml stuck at version 0.5.0 and 0.7.0.

Important

Synchronize version to 0.7.2 across files and migrate commitizen config to YAML.

Version Synchronization:
- Synced version to 0.7.2 in Cargo.toml, helm/Chart.yaml, and .cz.yaml.
- Added helm/Chart.yaml:version to version_files in .cz.yaml to prevent version drift.
Configuration Migration:
- Migrated from .cz.toml to .cz.yaml for commitizen configuration.
- Changed version_provider from cargo to commitizen in .cz.yaml to support multiple file updates.

^{This description was created by}^{for c93188d. You can customize this summary. It will automatically update as commits are pushed.}

Summary by CodeRabbit

Chores
- Version bumped to 0.7.2.
- Replaced prior versioning configuration with a new Commitizen-based conventional commits setup.
- Enabled coordinated versioning across package and chart artifacts and automatic changelog updates on version bumps.

CLAassistant · 2025-10-16T14:16:13Z

All committers have signed the CLA.

coderabbitai · 2025-10-16T14:16:32Z

Walkthrough

Removed the old Commitizen TOML config, added a new .cz.yaml Commitizen config, and bumped package/chart versions to 0.7.2 in Cargo.toml and helm/Chart.yaml. Version files tracked now include Cargo.toml and helm/Chart.yaml.

Changes

Cohort / File(s)	Summary
Commitizen config changed `\.cz\.toml`, `\.cz\.yaml`	Removed `.cz.toml` (deleted `[tool.commitizen]` block). Added `.cz.yaml` with `cz_conventional_commits` adapter, `tag_format: v$version`, `version: 0.7.2`, `version_files` mapping including `Cargo.toml:^version` and `helm/Chart.yaml:version`, `version_provider: commitizen`, `version_scheme: semver`, and `update_changelog_on_bump: true` (`changelog_file: CHANGELOG.md`).
Version bump files `Cargo.toml`, `helm/Chart.yaml`	`Cargo.toml`: package version bumped `0.7.1` → `0.7.2`. `helm/Chart.yaml`: chart `version` bumped `0.5.0` → `0.7.2`. No other functional code changes.

Sequence Diagram(s)

sequenceDiagram
    participant Dev as Developer
    participant CI as CI / Release job
    participant CZ as Commitizen (.cz.yaml)
    participant Files as Version files
    rect rgb(230, 248, 230)
      Note right of CZ: New config tracks\nCargo.toml & helm/Chart.yaml
    end
    Dev->>CI: Trigger release (commit bump)
    CI->>CZ: Run commitizen bump (provider: commitizen)
    CZ->>Files: Update `Cargo.toml` (version)
    CZ->>Files: Update `helm/Chart.yaml` (version)
    Files-->>CI: Files updated
    CI->>Dev: Release artifacts / changelog updated

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

fix: fix bump package #67 — Related Commitizen versioning/config changes (similar modifications to .cz.toml / version_files handling).

Suggested reviewers

galkleinman

Poem

🐰 Hopping bump to 0.7.2,
Two files tuned, one config new,
Toml hopped out, YAML hopped in,
Changelog waits for the next spin,
🥕 release carrots for the crew.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title clearly summarizes the primary changes by indicating both the version synchronization to 0.7.2 and the migration to a .cz.yaml configuration, which directly matches the PR objectives and follows a conventional commit style without superfluous detail.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch obs/fix-tags-and-helm-miss-alignment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 5b31020 and c93188d.

📒 Files selected for processing (1)

.cz.yaml (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

.cz.yaml

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

GitHub Check: build
GitHub Check: Analyze (rust)

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

ellipsis-dev

Important

Looks good to me! 👍

Reviewed everything up to 6fff2d4 in 43 seconds. Click for details.

Reviewed 38 lines of code in 3 files
Skipped 0 files when reviewing.
Skipped posting 3 draft comments. View those below.
Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. .cz.toml:8

Draft comment:
Good update: now both Cargo.toml and helm/Chart.yaml are listed for version bumps. Ensure documentation reflects this change.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

2. Cargo.toml:3

Draft comment:
Bump package version to 0.7.2 for consistency with Helm chart and tag version.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

3. helm/Chart.yaml:5

Draft comment:
Updated Helm chart version to 0.7.2. If the chart uses appVersion elsewhere, consider updating it too.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_RYijB0kboV76vCO5

^{You can customize}^{by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

ellipsis-dev

Important

Looks good to me! 👍

Reviewed 5b31020 in 42 seconds. Click for details.

Reviewed 19 lines of code in 1 files
Skipped 0 files when reviewing.
Skipped posting 2 draft comments. View those below.
Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. .cz.yaml:4

Draft comment:
Consider quoting 'v$version' to prevent any YAML parser misinterpretation of the dollar sign.
Reason this comment was not posted:
Confidence changes required: 33% <= threshold 50% None

2. .cz.yaml:7

Draft comment:
Verify that using a regex (^version) for Cargo.toml and a plain key ('version') for helm/Chart.yaml is intentional and consistent.
Reason this comment was not posted:
Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_hcV4hra4PcXufG11

^{You can customize}^{by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

coderabbitai

Actionable comments posted: 1

🧹 Nitpick comments (1)

.cz.yaml (1)
3-12: Confirm pre-1.0 bump semantics (major_version_zero).

If you rely on 0.y.z semantics (no “major” bumps before 1.0), explicitly set major_version_zero; the TOML previously had this key. Omitting it may change bump behavior.
 commitizen:
   name: cz_conventional_commits
   tag_format: v$version
   version: 0.7.2
+  major_version_zero: true
   version_files:

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 6fff2d4 and 5b31020.

📒 Files selected for processing (2)

.cz.toml (0 hunks)
.cz.yaml (1 hunks)

💤 Files with no reviewable changes (1)

.cz.toml

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

GitHub Check: build
GitHub Check: Analyze (rust)

🔇 Additional comments (1)

.cz.yaml (1)

5-12: Versions are synchronized across files. All version declarations in .cz.yaml, Cargo.toml, and helm/Chart.yaml match at 0.7.2.

coderabbitai · 2025-10-16T14:37:55Z

.cz.yaml

+  version_files:
+  - Cargo.toml:^version
+  - helm/Chart.yaml:version


🛠️ Refactor suggestion | 🟠 Major

Anchor regexes to avoid accidental matches in Chart.yaml (apiVersion/appVersion).

Pattern helm/Chart.yaml:version can match apiVersion and appVersion. Anchor and include the separator for precision; optionally keep appVersion in sync too.

version_files: - - Cargo.toml:^version - - helm/Chart.yaml:version + - Cargo.toml:^version\s*= + - helm/Chart.yaml:^version\s*: + # Optional: keep chart appVersion aligned with the released app/image tag + # - helm/Chart.yaml:^appVersion\s*:

🤖 Prompt for AI Agents

In .cz.yaml around lines 6 to 8, the pattern "helm/Chart.yaml:version" is too loose and may match apiVersion or appVersion; update it to anchor the key and include the YAML key separator (e.g. use a regex like '^version:' for the version key in Chart.yaml) and optionally add a separate anchored entry for appVersion (e.g. '^appVersion:') so only the intended fields are matched.

ellipsis-dev

Important

Looks good to me! 👍

Reviewed c93188d in 34 seconds. Click for details.

Reviewed 12 lines of code in 1 files
Skipped 0 files when reviewing.
Skipped posting 1 draft comments. View those below.
Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. .cz.yaml:11

Draft comment:
The addition of 'major_version_zero: true' is appropriate for managing 0.x versions; consider adding a brief inline comment explaining its purpose for clarity.
Reason this comment was not posted:
Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_LoISxnrfL34Jm9U1

^{You can customize}^{by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

wip

6fff2d4

ellipsis-dev bot reviewed Oct 16, 2025

View reviewed changes

OzBenSimhonTraceloop changed the title ~~fix: sync versions to 0.7.2 and configure commitizen to update helm chart~~ fix: sync versions to 0.7.2 and migrate to .cz.yaml config Oct 16, 2025

OzBenSimhonTraceloop added 2 commits October 16, 2025 17:32

wip

81d4c1a

wip

5b31020

ellipsis-dev bot reviewed Oct 16, 2025

View reviewed changes

coderabbitai bot reviewed Oct 16, 2025

View reviewed changes

wip

c93188d

ellipsis-dev bot reviewed Oct 16, 2025

View reviewed changes

galkleinman approved these changes Oct 19, 2025

View reviewed changes

OzBenSimhonTraceloop merged commit 3cc62a4 into main Oct 19, 2025
6 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: sync versions to 0.7.2 and migrate to .cz.yaml config#70

fix: sync versions to 0.7.2 and migrate to .cz.yaml config#70
OzBenSimhonTraceloop merged 4 commits intomainfrom
obs/fix-tags-and-helm-miss-alignment

OzBenSimhonTraceloop commented Oct 16, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

CLAassistant commented Oct 16, 2025 •

edited

Loading

Uh oh!

coderabbitai bot commented Oct 16, 2025 •

edited

Loading

Uh oh!

ellipsis-dev bot left a comment

Uh oh!

ellipsis-dev bot left a comment

Uh oh!

coderabbitai bot left a comment

Uh oh!

coderabbitai bot Oct 16, 2025

Uh oh!

ellipsis-dev bot left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

OzBenSimhonTraceloop commented Oct 16, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

Problem This Fixes

Summary by CodeRabbit

Uh oh!

CLAassistant commented Oct 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai bot commented Oct 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related PRs

Suggested reviewers

Poem

Pre-merge checks and finishing touches

Uh oh!

ellipsis-dev bot left a comment

Choose a reason for hiding this comment

Uh oh!

ellipsis-dev bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Oct 16, 2025

Choose a reason for hiding this comment

Uh oh!

ellipsis-dev bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

OzBenSimhonTraceloop commented Oct 16, 2025 •

edited by coderabbitai bot

Loading

CLAassistant commented Oct 16, 2025 •

edited

Loading

coderabbitai bot commented Oct 16, 2025 •

edited

Loading