fix: remove default-members in workspace

[nv-anants]

Overview:

Remove default-members from the Cargo workspace to eliminate the drift between default-members and members.

The default-members list was originally introduced to exclude packages (mainly dynamo-run) that are slow to build and are not shipped as flagship features.

Clippy fixes

lib/bindings/python/codegen/src/prometheus_parser.rs
unnecessary_unwrap: Replaced if is_macro_generated && macro_prefix.is_some() { let prefix = macro_prefix.as_ref().unwrap(); ... } with if is_macro_generated { if let Some(prefix) = macro_prefix.as_ref() { ... } }.

lib/kv-router/src/concurrent_radix_tree.rs
unnecessary_cast: Removed redundant as u64 in two places (lines 1142 and 1162) where the value was already u64.

lib/kv-router/src/indexer.rs
redundant_closure: Replaced all .map(|i| LocalBlockHash(i)) with .map(LocalBlockHash) (27 occurrences).
unnecessary_get_then_check: Replaced scores.scores.get(&WorkerWithDpRank::new(0, 0)).is_none() with !scores.scores.contains_key(&WorkerWithDpRank::new(0, 0)).

closes: OPS-3275

Summary by CodeRabbit

Release Notes

• Chores

  • Updated workspace configuration to include all members by default.

• Refactor

  • Simplified conditional logic and code structure for improved maintainability.

• Tests

  • Enhanced test implementations with improved type inference and assertion clarity.

[coderabbitai]

Walkthrough

This pull request removes the explicit default-members block from the Cargo workspace configuration, allowing the workspace to include all members by default. Simultaneously, it refactors conditional logic, simplifies type inference in test code across multiple crates, and adjusts assertion condition patterns in tests.

Changes

Cohort / File(s)	Summary
Workspace Configuration Cargo.toml	Removed explicit default-members block; workspace now defaults to including all members rather than selectively excluding crates.
Prometheus Parser Refactor lib/bindings/python/codegen/src/prometheus_parser.rs	Refactored conditional logic for applying macro-generated prefixes using nested if-let pattern; functional behavior unchanged.
KV Router Test Updates lib/kv-router/src/concurrent_radix_tree.rs, lib/kv-router/src/indexer.rs	Simplified type inference in test constructors by removing explicit numeric casts; refactored assertion conditions from is_none() checks to contains_key() pattern.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Members now dance where they're meant to be seen,
Conditionals nested in patterns pristine,
Type inference flows through the tests with such grace,
A workspace renewed in its rightful place! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: removing default-members from the workspace configuration, which is the primary modification in Cargo.toml.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Description check	✅ Passed	The pull request description includes all required sections: Overview clearly states the main change (removing default-members), Details provides context about the original purpose and rationale for the change, and includes specific Clippy fixes with file locations. Related Issues section closes OPS-3275. The description is well-structured and complete.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[grahamking]

I was hoping someone would do this :-)