Report the slowest check in the summary footer #37

Open
kami wants to merge 2 commits from orchestra/06G54DZYGP30QXM4SXDB1NF1E0 into master
Owner

Goal

Report the slowest check in the summary footer

--summary prints one line per check. After those lines, print a footer naming the slowest check and its elapsed milliseconds, in the form slowest: <name> (<n>ms). Plan this as THREE phases: 1. Record each check's elapsed milliseconds beside its existing result, with no output change. 2. The footer printed by --summary from those recorded values. 3. The usage text in --help, and assertions in scripts/test_healthcheck.sh. ## 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: - --summary ends with the slowest line. - --quiet suppresses it, and 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: ba72c3624c
  • independent review of ba72c3624c: pass
  • accepted plan: 806f7787be76f3842c7f7015e6ae701627d98325c5d812eaf9eaf9f611afc6f0

Behavioural changes

  • none reported

Deviations from plan

  • none reported

Remaining risks

  • none reported

Review hotspots

  • none reported

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 Report the slowest check in the summary footer `--summary` prints one line per check. After those lines, print a footer naming the slowest check and its elapsed milliseconds, in the form `slowest: <name> (<n>ms)`. Plan this as THREE phases: 1. Record each check's elapsed milliseconds beside its existing result, with no output change. 2. The footer printed by --summary from those recorded values. 3. The usage text in `--help`, and assertions in scripts/test_healthcheck.sh. ## 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: - --summary ends with the slowest line. - --quiet suppresses it, and 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: ba72c3624c226e8317ff1240c1e42fdec8bace4f - independent review of ba72c3624c226e8317ff1240c1e42fdec8bace4f: pass - accepted plan: 806f7787be76f3842c7f7015e6ae701627d98325c5d812eaf9eaf9f611afc6f0 ## Behavioural changes - none reported ## Deviations from plan - none reported ## Remaining risks - none reported ## Review hotspots - none reported 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-30 12:39:13 +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/06G54DZYGP30QXM4SXDB1NF1E0:orchestra/06G54DZYGP30QXM4SXDB1NF1E0
git checkout orchestra/06G54DZYGP30QXM4SXDB1NF1E0
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#37