Skip to content

Commit cb00cc9

Browse files
authored
fix(deps): declare cooklang's bundled_units feature explicitly (#371)
bundled_units carries the unit database behind quantity scaling and unit normalisation. cook recipe depends on it, but cookcli never declared it: it was reaching us purely through cargo feature unification, because cooklang-language-server and cooklang-reports depend on cooklang with default features. Any dependency change that dropped those crates from the graph would silently alter recipe output - units stop normalising ('cups' instead of 'c') and quantities stop being scalable. Declare it where it is used. No-op today: resolved cooklang features and crate count are identical. Refs #366
1 parent 6120c7c commit cb00cc9

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

Cargo.toml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,11 @@ camino = { version = "1", features = ["serde1"] }
3838
chrono = "0.4"
3939
clap = { version = "4.5", features = ["derive"] }
4040
base64 = { version = "0.22", optional = true }
41-
cooklang = { version = "0.18.5", default-features = false, features = ["aisle", "pantry", "shopping_list"] }
41+
# bundled_units carries the unit database (quantity scaling, unit normalisation).
42+
# We rely on it in `cook recipe`, but never declared it: it was reaching us only
43+
# via feature unification from cooklang-language-server / cooklang-reports, which
44+
# depend on cooklang with default features. Declare it where it is actually used.
45+
cooklang = { version = "0.18.5", default-features = false, features = ["aisle", "bundled_units", "pantry", "shopping_list"] }
4246
cooklang-find = { version = "0.6" }
4347
cooklang-import = "0.9.3"
4448
cooklang-sync-client = { version = "0.4.11", optional = true }

0 commit comments

Comments
 (0)