Skip to content

Unify shell helpers under a health dispatcher#35

Merged
Jammy2211 merged 2 commits into
mainfrom
feature/health-shell-api
Jul 7, 2026
Merged

Unify shell helpers under a health dispatcher#35
Jammy2211 merged 2 commits into
mainfrom
feature/health-shell-api

Conversation

@Jammy2211

Copy link
Copy Markdown
Collaborator

Summary

Adopt the health vocabulary on the shell surface, mirroring the Claude
/health door. Part of the "full health API" pass (paired PyAutoHeart +
PyAutoBrain PRs fold /pyauto-status-full into /health full).

Change

Rename the three sourced helper scripts + their functions, and add a dispatcher:

New Was Runs via
scripts/health_sync.sh_health_sync pyauto_status.sh / pyauto-status health / health sync
scripts/health_release.sh_health_release pyauto_status_full.sh / pyauto-status-full health release
scripts/health_audit.sh_health_audit pyauto_audit.sh / pyauto-audit health audit
scripts/health.shhealth() dispatcher (new)

The release-run sub-helpers become health-report / health-json /
health-triage (flat-named — niche views of the last run).

~/.bashrc callers (PyAuto()/PyAutoGPU()/PyAutoNoJAX()) move from
pyauto-status to health — a local change, not in this repo. Doc refs in
README.md + skills/OWNERSHIP.md repointed to the /health status /
/health full legs.

No back-compat aliases (clean rename).

Validation

Sourcing all four scripts in a fresh shell defines health, _health_sync,
_health_release, _health_audit, health-report, health-json,
health-triage; health help and unknown-subcommand handling work.

🤖 Generated with Claude Code

Adopt the health vocabulary on the shell surface, mirroring the Claude /health
door. Rename the three sourced helper scripts + their functions and add a
dispatcher:

- health_sync.sh    : _health_sync    (was pyauto_status.sh / pyauto-status)
- health_release.sh : _health_release (was pyauto_status_full.sh / pyauto-status-full)
                      + health-report / health-json / health-triage helpers
- health_audit.sh   : _health_audit   (was pyauto_audit.sh / pyauto-audit)
- health.sh         : the `health` dispatcher — bare/`sync`, `release`, `audit`

Callers in ~/.bashrc (PyAuto()/PyAutoGPU()/PyAutoNoJAX()) move from
`pyauto-status` to `health` (local change, not in this repo). Also repoint the
/pyauto-status(-full) references in README.md + skills/OWNERSHIP.md to the
`/health status` and `/health full` legs.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 7, 2026 20:10

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aligns the local shell “health” helpers with the /health vocabulary by introducing a health dispatcher function and renaming the underlying sourced helper functions/scripts, plus updating a couple of documentation references.

Changes:

  • Added scripts/health.sh implementing a health subcommand dispatcher for sync/release/audit.
  • Renamed the sourced helper functions to _health_sync, _health_release, _health_audit and updated related output strings.
  • Repointed documentation references in README.md and skills/OWNERSHIP.md to the /health status and /health full legs.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
skills/OWNERSHIP.md Updates ownership table entries to reflect /health legs and retirement notes.
scripts/health.sh New health() dispatcher front door for the shell helpers.
scripts/health_sync.sh Renames the cross-repo sync dashboard function to _health_sync and updates messages/docs.
scripts/health_release.sh Renames the release-prep dashboard function to _health_release and renames viewer helpers in text.
scripts/health_audit.sh Renames the structural audit function to _health_audit and updates messages/docs.
README.md Updates narrative about the single /health door and its legs.
Comments suppressed due to low confidence (2)

scripts/health_sync.sh:49

  • This PR renames the sync dashboard function to _health_sync, but scripts/status.sh --repos still sources scripts/pyauto_status.sh and runs pyauto-status. With the old file/function gone, --repos will fail at runtime. Either update status.sh to source the new script and call health/_health_sync, or add a small compatibility wrapper file/function so existing internal tooling keeps working.
    scripts/health_release.sh:196
  • The viewer functions were renamed to health-*, but they still rely on the helper _pyauto_run_file, which emits pyauto:-prefixed errors and has a comment referencing pyauto-{report,json,triage}. This creates confusing mixed branding in user-facing error output (e.g. health-report failing with a pyauto: message). Consider renaming the helper + its error prefixes to health as part of this rename.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread scripts/health.sh Outdated
Comment on lines +18 to +32
health() {
local sub="${1:-sync}"
case "$sub" in
sync)
shift 2>/dev/null || true
_health_sync "$@"
;;
release)
shift
_health_release "$@"
;;
audit)
shift
_health_audit "$@"
;;
Comment thread skills/OWNERSHIP.md Outdated
Comment on lines +34 to +35
| `pyauto-status/` | `PyAutoHeart/skills/` | **PyAutoHeart** (active-work dashboard) | **retired as command → `/health status` leg** |
| `pyauto-status-full/` | `PyAutoHeart/skills/` | **PyAutoHeart** (release-run dashboard) | **retired as command → `/health full` leg** |
…opilot)

- Restructure the dispatcher to shift the subcommand token once up front (only
  when given), replacing the confusing `shift 2>/dev/null` in the sync branch.
  Verified passthrough: bare health / health sync [args] / release <dir> / audit.
- OWNERSHIP table: drop trailing-slash names for consistency; note the leg
  reference file in the status column instead.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@Jammy2211 Jammy2211 merged commit 79b8afb into main Jul 7, 2026
@Jammy2211 Jammy2211 deleted the feature/health-shell-api branch July 7, 2026 20:17
Jammy2211 pushed a commit that referenced this pull request Jul 8, 2026
…d, seed autonomy_log.md; autonomy-contract shipped (PR #35)
Jammy2211 pushed a commit that referenced this pull request Jul 8, 2026
Jammy2211 pushed a commit that referenced this pull request Jul 8, 2026
Jammy2211 pushed a commit that referenced this pull request Jul 11, 2026
…pled ePSF back-end (deep design; Tier-1b reclassification)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018pUia6xzapBww82yopSZZG
Jammy2211 pushed a commit that referenced this pull request Jul 11, 2026
Jammy2211 pushed a commit that referenced this pull request Jul 11, 2026
Jammy2211 pushed a commit that referenced this pull request Jul 11, 2026
…ted, half-pixel trap fixed, #35)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018pUia6xzapBww82yopSZZG
Jammy2211 pushed a commit that referenced this pull request Jul 11, 2026
Jammy2211 pushed a commit that referenced this pull request Jul 11, 2026
… not stellar); definitive test needs stellar-field reduction (#35)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018pUia6xzapBww82yopSZZG
Jammy2211 pushed a commit that referenced this pull request Jul 11, 2026
Jammy2211 pushed a commit that referenced this pull request Jul 11, 2026
…closed, #37 follow-up open

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018pUia6xzapBww82yopSZZG
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants