Skip to content

Add status page incident event commands - #57

Merged
kwent merged 4 commits into
masterfrom
agent/status-page-events
Aug 13, 2026
Merged

Add status page incident event commands#57
kwent merged 4 commits into
masterfrom
agent/status-page-events

Conversation

@kwent

@kwent kwent commented Aug 13, 2026

Copy link
Copy Markdown
Member

Summary

  • add a status-pages resource group with paginated discovery using the same --name, --slug, sorting, and output conventions as neighboring commands
  • add incident event list, create, update, and resolve operations
  • support dynamic server-defined event statuses instead of rejecting maintenance and team lifecycle statuses locally
  • expose create-time subscriber notifications and RFC3339 --started-at controls where the server implements them
  • return server validation details for rejected event updates
  • cover API payloads, incident ID normalization, lifecycle commands, and timestamp validation with tests

Interface

rootly status-pages list [--name=...] [--slug=...]
rootly status-pages events list <incident-id>
rootly status-pages events create <incident-id> --status-page=<id> --status=<status> --message=... [--notify-subscribers]
rootly status-pages events update <event-id> [--status=...] [--message=...] [--started-at=<rfc3339>]
rootly status-pages events resolve <event-id> --message=...

--notify-subscribers is intentionally create-only: the server notification callback does not run for event updates. Tweeting is not exposed because the API controller does not accept the documented field, and component selection remains out of scope because it is feature-gated and requires component discovery.

Closes #52

Testing

  • go test ./...
  • command help for the status-page group and event mutations
  • git diff --check

@kwent

kwent commented Aug 13, 2026

Copy link
Copy Markdown
Member Author

@greptileai

@greptile-apps

greptile-apps Bot commented Aug 13, 2026

Copy link
Copy Markdown

Greptile Summary

The PR adds status-page discovery and incident-event lifecycle commands to the CLI.

  • Adds paginated status-page and incident-event listing.
  • Adds create, update, and resolve event operations with RFC3339 timestamp handling.
  • Adds JSON:API request/response models, validation-error reporting, command registration, and tests.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
internal/api/client.go Adds status-page models and API methods for listing pages and creating, listing, updating, and resolving incident events.
internal/cmd/statuspages/events_mutations.go Implements event lifecycle commands, required flags, update-field detection, timestamp validation, and formatted output.
internal/cmd/statuspages/events_list.go Implements paginated event listing with incident ID normalization and structured output.
internal/cmd/statuspages/list.go Implements filtered and sorted status-page discovery with pagination reporting.
internal/api/client_status_pages_test.go Covers pagination validation, request payloads, response parsing, and server validation errors.
internal/cmd/statuspages/cmd_test.go Covers command output, incident ID normalization, lifecycle mutations, required update fields, and timestamp validation.

Reviews (5): Last reviewed commit: "validate status page pagination" | Re-trigger Greptile

@kwent

kwent commented Aug 13, 2026

Copy link
Copy Markdown
Member Author

@greptileai

1 similar comment
@kwent

kwent commented Aug 13, 2026

Copy link
Copy Markdown
Member Author

@greptileai

@kwent
kwent marked this pull request as ready for review August 13, 2026 04:07
@kwent
kwent force-pushed the agent/status-page-events branch from d8669b7 to c1461a1 Compare August 13, 2026 04:09
@kwent

kwent commented Aug 13, 2026

Copy link
Copy Markdown
Member Author

@greptileai

@kwent
kwent merged commit 8ceb2d5 into master Aug 13, 2026
7 checks passed
@kwent
kwent deleted the agent/status-page-events branch August 13, 2026 04:14
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.

Feature request: status page commands (incident resolve leaves the public page open)

1 participant