Run a single named check with --only #28

Open
kami wants to merge 4 commits from orchestra/06G4W63545T3RV8SSKGKN11B3G into master
Owner

Goal

Run a single named check with --only

Give scripts/orchestra_e2e_healthcheck.sh a --only <name> flag that runs a single named check instead of all ten, then prints the usual success line. Behaviour: - --only repo_root_readable runs that check alone and exits 0 when it passes. - --only nonsense prints unknown check: nonsense on stderr and exits 2. - --only composes with --summary, --json and --quiet, each keeping its current shape over the single result. - Without --only, every existing flag behaves exactly as it does today. Plan this as FIVE phases: 1. A helper that resolves a name to a check function and reports whether the name is known. 2. Flag parsing for --only <name>, with the unknown-name refusal and exit 2. 3. Running the single resolved check and recording its result through the existing results state. 4. Composition with --summary and --json. 5. The usage text in --help, and assertions in scripts/test_healthcheck.sh for each behaviour above. ## Work one phase at a time Implement phase N, request its verification, and only then begin phase N+1. Do not write the whole change first and verify the phases afterwards. ## Every phase carries both kinds of check This is a hard requirement of this task. Each phase must have at least one automated check from the policy below. Each phase must also have at least one manual check, confirmed by a human reading the printed output. Orchestra runs the automated ones. A human owns the manual ones, and the phase is not finished until that human signs it off. ## 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. ## Research first Establish at least three findings with distinct confidence: at least one fact, one inference, one assumption. ## Handoff behaviour Behave normally. If Orchestra asks you to write a handoff report, write it and stop. Acceptance: - --only <name> runs exactly one check. - An unknown name exits 2 with a message on stderr. - --only composes with --summary, --json and --quiet. - Every existing flag is unchanged when --only is absent.

Acceptance

  • not stated in the task contract

Human decisions

  • correction (plan_phase_verification:53f7918b77b3712499fb442b1773a2274a84cb54c2019f66be82204ce1f50e69:phase-1): orchestra verify phase-1 Read the printed output for this phase and confirmed it by hand.

Verification

  • bash -n scripts/*.sh && bash scripts/orchestra_e2e_healthcheck.sh exited 0
  • commit: 6ce0033d9e
  • independent review of 6ce0033d9e: pass
  • minor findings, not fixed: 1
  • accepted plan: 53f7918b77b3712499fb442b1773a2274a84cb54c2019f66be82204ce1f50e69

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:214 Two stray comment lines unrelated to --only were left at the end of the script: '# Rig: an operator edit made after phase-1 was signed off.' (line 214) and '# Rig: a second tree move, with no new sign-off.' (line 216).

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 Run a single named check with --only Give scripts/orchestra_e2e_healthcheck.sh a `--only <name>` flag that runs a single named check instead of all ten, then prints the usual success line. Behaviour: - `--only repo_root_readable` runs that check alone and exits 0 when it passes. - `--only nonsense` prints `unknown check: nonsense` on stderr and exits 2. - `--only` composes with `--summary`, `--json` and `--quiet`, each keeping its current shape over the single result. - Without `--only`, every existing flag behaves exactly as it does today. Plan this as FIVE phases: 1. A helper that resolves a name to a check function and reports whether the name is known. 2. Flag parsing for `--only <name>`, with the unknown-name refusal and exit 2. 3. Running the single resolved check and recording its result through the existing results state. 4. Composition with `--summary` and `--json`. 5. The usage text in `--help`, and assertions in scripts/test_healthcheck.sh for each behaviour above. ## Work one phase at a time Implement phase N, request its verification, and only then begin phase N+1. Do not write the whole change first and verify the phases afterwards. ## Every phase carries both kinds of check This is a hard requirement of this task. Each phase must have at least one automated check from the policy below. Each phase must also have at least one manual check, confirmed by a human reading the printed output. Orchestra runs the automated ones. A human owns the manual ones, and the phase is not finished until that human signs it off. ## 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. ## Research first Establish at least three findings with distinct confidence: at least one fact, one inference, one assumption. ## Handoff behaviour Behave normally. If Orchestra asks you to write a handoff report, write it and stop. Acceptance: - `--only <name>` runs exactly one check. - An unknown name exits 2 with a message on stderr. - `--only` composes with `--summary`, `--json` and `--quiet`. - Every existing flag is unchanged when `--only` is absent. ## Acceptance - not stated in the task contract ## Human decisions - correction (plan_phase_verification:53f7918b77b3712499fb442b1773a2274a84cb54c2019f66be82204ce1f50e69:phase-1): orchestra verify phase-1 Read the printed output for this phase and confirmed it by hand. ## Verification - `bash -n scripts/*.sh && bash scripts/orchestra_e2e_healthcheck.sh` exited 0 - commit: 6ce0033d9e9e9dcc060ef5ba6434a37e3d9a583a - independent review of 6ce0033d9e9e9dcc060ef5ba6434a37e3d9a583a: pass - minor findings, not fixed: 1 - accepted plan: 53f7918b77b3712499fb442b1773a2274a84cb54c2019f66be82204ce1f50e69 ## 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:214` Two stray comment lines unrelated to --only were left at the end of the script: '# Rig: an operator edit made after phase-1 was signed off.' (line 214) and '# Rig: a second tree move, with no new sign-off.' (line 216). 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 4 commits 2026-08-29 17:50:48 +02:00
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/06G4W63545T3RV8SSKGKN11B3G:orchestra/06G4W63545T3RV8SSKGKN11B3G
git checkout orchestra/06G4W63545T3RV8SSKGKN11B3G
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: kami/test-e2e#28