depot: replace --target local with --out DIR, matching crucible nwsync #66

New Issue

2026-07-28T07:35:32Z

archvillainette commented

2026-07-28 07:35:32 +00:00

crucible depot selects its destination with --target bunny|cdn|local, where local means "read/write a depot tree on disk". While settling the crucible nwsync surface (#56) the same choice came up and landed the other way: --out DIR present means write there, absent means upload to the zone.

--out DIR reads better. It names the directory in the same breath as choosing local, instead of --target local plus a second flag (--source / --dest, depending on the verb) that says where. It also removes the case where --target local and a missing directory flag are individually valid but jointly meaningless.

Adopt the same convention in depot so the two groups do not disagree:

--out DIR replaces --target local wherever local means "a tree on disk".
--target bunny|cdn keeps its meaning for remote backends.
Existing wrapper callers in wrappers/consumers.txt must keep working — check whether --target local needs to stay as a hidden alias.

Cosmetic-but-real; not on the route to a synced client, so it is not part of the NWSync map (#54).

`crucible depot` selects its destination with `--target bunny|cdn|local`, where `local` means "read/write a depot tree on disk". While settling the `crucible nwsync` surface (#56) the same choice came up and landed the other way: `--out DIR` present means write there, absent means upload to the zone. `--out DIR` reads better. It names the directory in the same breath as choosing local, instead of `--target local` plus a second flag (`--source` / `--dest`, depending on the verb) that says where. It also removes the case where `--target local` and a missing directory flag are individually valid but jointly meaningless. Adopt the same convention in `depot` so the two groups do not disagree: - `--out DIR` replaces `--target local` wherever local means "a tree on disk". - `--target bunny|cdn` keeps its meaning for remote backends. - Existing wrapper callers in `wrappers/consumers.txt` must keep working — check whether `--target local` needs to stay as a hidden alias. Cosmetic-but-real; not on the route to a synced client, so it is not part of the NWSync map (#54).

archvillainette referenced this issue

2026-07-28 07:36:10 +00:00

Map: NWSync publishing — emit at artifact birth, assemble at module release #54

archvillainette referenced this issue from a commit

2026-07-28 08:53:33 +00:00

feat(depot): select a depot tree on disk with --out DIR

archvillainette referenced a pull request that will close this issue

2026-07-28 08:53:43 +00:00

depot: replace --target local with --out DIR #67

archvillainette closed this issue

2026-07-28 11:07:59 +00:00

archvillainette referenced this issue from a commit