Sort the check names in --list-checks #30

2026-08-29T18:11:01+02:00

kami commented

2026-08-29 18:11:01 +02:00

Goal

Sort the check names in --list-checks

--list-checks prints the ten check names in the order they run. Add a --sorted modifier that prints them in alphabetical order instead, leaving run order untouched. Plan this as THREE phases: 1. A helper that returns the check names in alphabetical order, with no caller yet. 2. --sorted accepted alongside --list-checks, using that helper. Alone, --sorted changes nothing. 3. The usage text in --help, and assertions in scripts/test_healthcheck.sh for both orders. ## Work one phase at a time Implement phase N, request its verification, and only then begin phase N+1. ## Verification policy for this project Only two commands may appear on a run: line, exactly: - ["bash", "-n", ""] - ["bash", "scripts/orchestra_e2e_healthcheck.sh"] Anything else is refused when you seal the plan. Give every phase at least one automated check from that list and at least one manual check. ## Research first Establish at least three findings with distinct confidence: at least one fact, one inference, one assumption. Acceptance: - --list-checks --sorted prints the ten names alphabetically. - --list-checks alone keeps run order. - Every other flag is unchanged.

Acceptance

not stated in the task contract

Verification

bash -n scripts/*.sh && bash scripts/orchestra_e2e_healthcheck.sh exited 0
commit: 49635d38e6
independent review of 49635d38e6: pass
minor findings, not fixed: 2
accepted plan: d8cca86d5469bd6e83faae6171ed66cfdd5de6cf480ab5892b77381bd36e76f1

Behavioural changes

none reported

Deviations from plan

none reported

Remaining risks

none reported

Review hotspots

none reported

Reviewer findings

minor: scripts/orchestra_e2e_healthcheck.sh:119 Flag precedence for the undocumented combination --list-checks --help is inverted: it printed the ten check names before, it prints the usage text now.
minor: scripts/orchestra_e2e_healthcheck.sh:139 The deferred list print uses printf directly, bypassing emit, which the file documents as the single output path.

The sections above are derived from Orchestra state. The reported
changes, deviations, risks, and hotspots are the implementing agent's
own account and are not verified.

## Goal Sort the check names in --list-checks `--list-checks` prints the ten check names in the order they run. Add a `--sorted` modifier that prints them in alphabetical order instead, leaving run order untouched. Plan this as THREE phases: 1. A helper that returns the check names in alphabetical order, with no caller yet. 2. `--sorted` accepted alongside `--list-checks`, using that helper. Alone, `--sorted` changes nothing. 3. The usage text in `--help`, and assertions in scripts/test_healthcheck.sh for both orders. ## Work one phase at a time Implement phase N, request its verification, and only then begin phase N+1. ## Verification policy for this project Only two commands may appear on a run: line, exactly: - ["bash", "-n", "<path>"] - ["bash", "scripts/orchestra_e2e_healthcheck.sh"] Anything else is refused when you seal the plan. Give every phase at least one automated check from that list and at least one manual check. ## Research first Establish at least three findings with distinct confidence: at least one fact, one inference, one assumption. Acceptance: - `--list-checks --sorted` prints the ten names alphabetically. - `--list-checks` alone keeps run order. - Every other flag is unchanged. ## Acceptance - not stated in the task contract ## Verification - `bash -n scripts/*.sh && bash scripts/orchestra_e2e_healthcheck.sh` exited 0 - commit: 49635d38e69ffb862ef3557588f4363d443b1f03 - independent review of 49635d38e69ffb862ef3557588f4363d443b1f03: pass - minor findings, not fixed: 2 - accepted plan: d8cca86d5469bd6e83faae6171ed66cfdd5de6cf480ab5892b77381bd36e76f1 ## Behavioural changes - none reported ## Deviations from plan - none reported ## Remaining risks - none reported ## Review hotspots - none reported ## Reviewer findings - minor: `scripts/orchestra_e2e_healthcheck.sh:119` Flag precedence for the undocumented combination `--list-checks --help` is inverted: it printed the ten check names before, it prints the usage text now. - minor: `scripts/orchestra_e2e_healthcheck.sh:139` The deferred list print uses printf directly, bypassing emit, which the file documents as the single output path. The sections above are derived from Orchestra state. The reported changes, deviations, risks, and hotspots are the implementing agent's own account and are not verified.

kami added 2 commits 2026-08-29 18:11:01 +02:00

orchestra: TASK.md 0312dd9871

orchestra: pre-release WIP snapshot (orchestra-06g4wdj06g7fk00qf73z9ddbt4-920248b2) 49635d38e6

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin orchestra/06G4WDJ06G7FK00QF73Z9DDBT4:orchestra/06G4WDJ06G7FK00QF73Z9DDBT4

git checkout orchestra/06G4WDJ06G7FK00QF73Z9DDBT4

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: kami/test-e2e#30