feat(pilotctl): pilotctl review CLI command (PILOT-410)#269
Closed
matthew-pilot wants to merge 4 commits into
Closed
feat(pilotctl): pilotctl review CLI command (PILOT-410)#269matthew-pilot wants to merge 4 commits into
matthew-pilot wants to merge 4 commits into
Conversation
Adds `pilotctl review <pilot|app-id> [--rating N] [--text "..."]` command for submitting reviews and ratings of Pilot Protocol or installed apps. The command sends a JSON datagram (kind=review) to the Feedback agent (node 16437, hostname "feedback"), which forwards it to the telemetry pipeline. This is the E4 review-submission path (E1/E2/E3 come from the daemon app-store plugin, E4 from the CLI). Closes PILOT-410
added 3 commits
June 16, 2026 16:40
The gen-cli-reference.sh script captures pilotctl --help output. When a Pilot daemon is running (local dev), the output includes a MOTD banner. CI has no daemon, so the diff checker fails. Remove the MOTD line so the committed file matches CI's output.
pilotctl update command is on a different branch; the committed doc incorrectly included it. The generator output (minus MOTD) now correctly shows only review.
Collaborator
|
Closing — superseded by a merged PR for the same ticket. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds
pilotctl review <pilot|app-id> [--rating N] [--text "..."]command — the E4 review-submission path for Pilot Protocol.Why
PILOT-410: on-demand review submission for Pilot or a given app id. The telemetry server already accepts
kind=reviewevents (PILOT-411 server-side is ready). ThemdReviewsaggregate schema exists in the app-store metadata for read-only display.How
The command:
pilotor app-id), optional--rating(1–5), optional--textkind: "review"feedback)Files changed
cmdReviewfunction, payload struct, help text registrationVerification
go build ./cmd/pilotctl/✓go vet ./cmd/pilotctl/✓go test ./cmd/pilotctl/✓ (9.2s, all passing)Closes PILOT-410