feat(ui): main_recursive.sh — walkthrough for --recursive across add/get/status#205
Open
CGMossa wants to merge 1 commit into
Open
feat(ui): main_recursive.sh — walkthrough for --recursive across add/get/status#205CGMossa wants to merge 1 commit into
CGMossa wants to merge 1 commit into
Conversation
…get/status Adds ui/main_recursive.sh (550 lines) exercising the matrix: - dvs add: explicitly rejects --recursive (glob is the recursion encoding; flag-based recursion was deliberately excluded for add). - dvs get: positional file, single-dir, single-dir + --recursive, '.' + --recursive, mixed file/dir inputs. - dvs status: same matrix, plus the no-path case where --recursive is a documented no-op (whole-repo iteration is already recursive). Each case logs both the command and the resulting state of the working tree so the CLI/R parity can be eyeballed in the published HTML. ui/CLAUDE.md: adds main_recursive.sh to the script roster + widens the comment column to accommodate the longer name. Carved out of #154. Stacked on the rpkg PR. ui-trace cleanup hunks that #154 used to carry are intentionally NOT here — they land separately via #198 (chore/ui-trace-cleanup).
CGMossa
force-pushed
the
feat/dvs-rpkg-get-recursive
branch
from
d034deb to
5456863
Compare
CGMossa
force-pushed
the
feat/ui-main-recursive
branch
from
3e9a174 to
50b9093
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
AI-written details
Summary
Carved out of #154 as the ui-walkthrough layer of the 3-way split. Stacked on #204 (rpkg binding), which depends on #203 (core+CLI).
ui/main_recursive.sh(+550 lines): exercises the recursive matrix:dvs add— explicitly rejects--recursive(recursion is encoded via glob; flag-based recursion was deliberately excluded foradd).dvs get— positional file; single-dir; single-dir +--recursive;.+--recursive; mixed file/dir inputs.dvs status— same matrix, plus the no-path case where--recursiveis a documented no-op (whole-repo iteration is already recursive).ui/CLAUDE.md: registersmain_recursive.shin the script roster; widens the comment column to accommodate the longer name.What's NOT here
The ui-trace cleanup hunks (
helpers.shregex extension + deferredset -xacrossmain.sh/main_parallel.sh/main_progress.sh/main_status.sh) that #154 used to carry are not in this PR. They're identical to the changes already in #198 (chore/ui-trace-cleanup). After #198 lands and this rebases on main, those hunks resolve to nothing.Test plan
bash ui/main_recursive.shruns to completion without unexpected nonzero exitjust ui-publish ui-recursiveproduces a readable HTMLui/CLAUDE.mdrenders without trailing-whitespace anomaliesSequencing
Drafted by Claude (claude-opus-4-7). Reviewed by the author.