Skip to content

feat: add read-only remote capability doctor#1

Merged
J3m5 merged 4 commits into
mainfrom
feat/doctor
Jul 13, 2026
Merged

feat: add read-only remote capability doctor#1
J3m5 merged 4 commits into
mainfrom
feat/doctor

Conversation

@J3m5

@J3m5 J3m5 commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add a strictly read-only doctor command with a versioned JSON result and human/GitHub summaries
  • inventory the portable remote command profile, optional tools, normalized tar capabilities, and observable root/workdir permissions
  • require pinned known_hosts by default, with explicit non-persistent discovery limited to doctor and report
  • expose doctor through the composite Action without building an archive
  • document the public CLI, result contract, Action workflow, and security policy

Compatibility

  • GNU tar 1.34 is the initial implementation/version pair covered by SSH E2E; other versions return undetermined
  • doctor never creates the remote workdir or writes a probe file; actual write access remains an apply preflight concern
  • no consumer-specific capabilities or private project references are encoded in the public implementation
  • the Action path is validated statically and through the locally built PEX; it becomes directly consumable after a release publishes a PEX containing this change

Validation

  • mise run check — 91 tests plus Ruff, Pyrefly, Actionlint, Action Validator, whitespace, conflict, and private-key checks
  • mise run test:e2e — 15 SSH scenarios, including remote state equality before/after doctor
  • mise run test:e2e:pex — 15 SSH scenarios through the distributed PEX shape

@J3m5 J3m5 self-assigned this Jul 13, 2026
@J3m5
J3m5 merged commit 56e291a into main Jul 13, 2026
3 checks passed
@J3m5
J3m5 deleted the feat/doctor branch July 13, 2026 10:05
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.

1 participant