cure(md): 8 autonomous tidies, skim+companion-gated — for review #40

Merged

calliope merged 10 commits from erato/cure-md into erato/focus 2026-06-23 15:32:06 +00:00 AGit

Conversation 1 Commits 10 Files changed 1 +2 -1

erato commented 2026-06-22 22:08:27 +00:00

Collaborator

Copy link

No description provided.

erato added 8 commits 2026-06-22 22:08:27 +00:00

refactor(md): improve readability by storing id length in variable f48bda069b

refactor(md-utils): refactor condition logic for section grabbing 594d19526d

tidy(lib): make link variable local in catalog_resolve 64fee25659

tidy(lib): Declare variable 'n' as local 9b5daf3667

tidy(lib): declare matches as local in _md_section_search 74c81996f5

tidy(md): declare variable n as local in roadmap_read function 731b01a5be

tidy(lib): make matches variable local in roadmap_read function fbc386c0e7

tidy(lib): add local variable declaration in md.sh 847de51339

calliope commented 2026-06-23 08:32:31 +00:00

Collaborator

Copy link

daktyl review (head 5ac2867)

gate	status	duration	note
bash-parse	✅ PASS	4ms	(no changed .sh failed parse)
lint-bash-traps	✅ PASS	7ms	(no bash traps in the diff)
shellcheck	✅ PASS	35ms	(no findings at -S warning)
lint-language	✅ PASS	19ms	(no vocabulary violations in diff)
daisy-validate	✅ PASS	124173ms	(full suite green)

Verdict: 5/5 gates green. ✅ Awaiting judgment.

## daktyl review (head `5ac2867`) | gate | status | duration | note | |---|---|---|---| | bash-parse | ✅ PASS | 4ms | (no changed .sh failed parse) | | lint-bash-traps | ✅ PASS | 7ms | (no bash traps in the diff) | | shellcheck | ✅ PASS | 35ms | (no findings at -S warning) | | lint-language | ✅ PASS | 19ms | (no vocabulary violations in diff) | | daisy-validate | ✅ PASS | 124173ms | (full suite green) | **Verdict:** 5/5 gates green. ✅ Awaiting judgment.

calliope added the

gates-red

label 2026-06-23 08:32:34 +00:00

erato added 1 commit 2026-06-23 11:37:57 +00:00

fix(md): SC2155 — declare then assign ... aa2e649177

Split the coupling cure's introduced 'local x=$(cmd)' into 'local x' / 'x=$(cmd)' so the command's return is not masked — the two-line form the reviewer's shellcheck expects (5 line(s)).

erato added 1 commit 2026-06-23 14:09:00 +00:00

fix(md): revert awk-corrupting cures + redundant locals ... 5ac286787d

Legibility cures had injected bash (local, if/then/fi) into md_section's and menu_index's awk programs — corrupting md_section, which broke clio-anchor via menu_read. Reverted those hunks to the original awk; removed redundant 'local matches/n/link' (already declared). Kept the valid length_four awk intermediate. clio-anchor + md green; md.sh shellcheck-clean.

calliope added

awaiting-judgment

gates-green

and removed

gates-red

labels 2026-06-23 15:29:01 +00:00

calliope merged commit 75d644d896 into erato/focus 2026-06-23 15:32:06 +00:00

calliope referenced this pull request from a commit 2026-06-23 15:32:08 +00:00

Merge pull request 'cure(md): 8 autonomous tidies, skim+companion-gated — for review' (#40) from erato/cure-md into erato/focus

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

awaiting-judgment

  

gates-green

  

gates-red

No labels

awaiting-judgment

gates-green

gates-red

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

fqqdk/daisy!40

No description provided.