feat: add module support to JustfileFixture#5
feat: add module support to JustfileFixture#5bluefing wants to merge 1 commit intoDataBooth:mainfrom
Conversation
Changed Files
|
944eeeb to
0e816cb
Compare
Flattens the nested module tree from just --dump into a flat dict keyed by recipe namepaths (e.g. module::submodule::recipe). All existing methods (is_private, dependencies, parameters, etc.) work transparently with full module paths. Adds all_modules property for module discovery. Backwards compatible — justfiles without modules work unchanged.
0e816cb to
46a61b3
Compare
|
Great direction and this adds high-value capability for modular justfile layouts. Assessment What looks good Points to consider before merge
|
Just does not allow duplicates, if I duplicate a recipe the in the same module I get this error
Good question. I think you are right, if the dump is malformed we should fail fast.
Needs some thought - not sure what the right answer is here.
Well spotted - I asked casey to implement a function to return the current module, he settled on
What do you think? |
Adds module-aware recipe resolution to
JustfileFixture. Recipes from nested modules are flattened by namepath (e.g.infra::deploy::up), so all existing methods work transparently with full module paths.Closes #1
Changes
_flattened— cached single-pass walk of the dump, returning both recipes and modules_recipes— now returns the flattened dict keyed by namepath (was root-only)_flatten— static recursive walk of the module treeall_modules— property returning discovered module pathsTests
9 tests covering: namepath flattening, privacy across modules, dependencies, parameters, unknown namepath errors, module discovery, private recipe filtering, and standalone
_flattenusage.