René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"cz bump crashes with version_provider=cargo on workspace virtual manifest (missing [package] / TypeError on version.workspace)","articleBody":"### Description\n\nCommitizen crashes when using `version_provider = \"cargo\"` on a Cargo **workspace virtual manifest** (root `Cargo.toml` contains `[workspace]` and `[workspace.package]` but **no** `[package]`). Our workspace members inherit version via `version.workspace = true`. Running `cz bump` fails inside `commitizen/providers/cargo_provider.py` while trying to update `Cargo.lock`.\n\nThis prevents using Commitizen’s cargo provider for monorepo-style Rust workspaces where the version is centralized in `[workspace.package].version`.\n\n\n\n\n### Steps to reproduce\n\n1. Create a Cargo workspace that uses a **virtual manifest** at the root (no `[package]`), with a centralized version:\n\n   ```toml\n   # ./Cargo.toml (workspace root)\n   [workspace]\n   members = [\"crates/a\"]\n\n   [workspace.package]\n   version = \"0.1.0\"\n   edition = \"2024\"\n   ```\n\n2. Create a member crate that inherits the version via `version.workspace = true`:\n\n   ```toml\n   # ./crates/a/Cargo.toml\n   [package]\n   name = \"a\"\n   version.workspace = true\n   edition.workspace = true\n   ```\n\n3. Configure Commitizen to use the cargo provider:\n\n   ```toml\n   # ./cz.toml\n   [tool.commitizen]\n   name = \"cz_conventional_commits\"\n   version_scheme = \"semver\"\n   version_provider = \"cargo\"\n   tag_format = \"v$version\"\n   update_changelog_on_bump = true\n   ```\n\n4. Run:\n\n   ```bash\n   cz bump --yes\n   ```\n\n5. Observe the crash inside `commitizen/providers/cargo_provider.py` (during manifest parsing / Cargo.lock update).\n\n\n### Current behavior\n\n`cz bump` crashes with exceptions from `cargo_provider.py`:\n\n* `tomlkit.exceptions.NonExistentKey: 'Key \"package\" does not exist.'` (root manifest has no `[package]`)\n* then `TypeError: string indices must be integers, not 'str'` (when accessing `package_content[\"version\"][\"workspace\"]` for members)\n\nThe command terminates without completing the bump/tag operation.\n\n### Desired behavior\n\n`cz bump` should support Cargo workspaces with a **virtual manifest** (root `Cargo.toml` has `[workspace]` / `[workspace.package]` but no `[package]`) and members using `version.workspace = true`.\n\nConcretely:\n\n* Read and bump the canonical version from `[workspace.package].version`.\n* Do not assume `[package]` exists in the workspace root manifest.\n* When scanning/updating member manifests, correctly handle `version.workspace = true` (a nested table / boolean) without type errors.\n\n### Screenshots\n\n_No response_\n\n### Environment\n\n(computation) juncheng@localhost:~/projects/computation$ cz version --report\nCommitizen Version: 4.10.1\nPython Version: 3.11.12 (main, Apr  9 2025, 04:04:00) [Clang 20.1.0 ]\nOperating System: Linux","author":{"url":"https://github.com/jcfangc","@type":"Person","name":"jcfangc"},"datePublished":"2025-12-23T03:55:05.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/1732/commitizen/issues/1732"}