feat: full-OpenAPI coverage + help-reliability fixes from the agent eval

Bumps go-flashduty to the merged docs#67 sync (AuditRecordIDRequest split,
progress enum, export_fields enum, post-mortem desc) + the restored toon tag,
and lands the CLI-side eval fixes so an SRE agent can call correctly from
`--help` alone:

- cligen (internal/cmd/cligen): list help states the {request_id,error,data}
  envelope is unwrapped — pipe `jq '.items[]'` for generated lists, not
  `.data.items[]`.
- incident.go: `incident list` --since/--until window must be < 31 days, --limit
  max 100. audit.go: `audit search` window < 90 days, --limit max 99.
- command.go: WriteRaw streams *export CSV bodies to the writer verbatim so
  `> file.csv` captures bytes instead of the "OK: POST ..." acknowledgment.
- Regenerated all 254 commands against the new SDK (toon keys snake_case;
  rule-audit-detail --id documented as the audit-record id, not the rule id).

go.mod pins go-flashduty to the merged main pseudo-version (no release).
Build + go vet green.

Author: ysyneu

go.mod

@@ -3,7 +3,7 @@ module github.com/flashcatcloud/flashduty-cli
 go 1.25.1
 
 require (
-	github.com/flashcatcloud/go-flashduty v0.5.2
+	github.com/flashcatcloud/go-flashduty v0.5.3-0.20260602031007-62b37649b2f0
 	github.com/mattn/go-runewidth v0.0.23
 	github.com/spf13/cobra v1.10.2
 	github.com/spf13/pflag v1.0.10

go.sum

@@ -1,8 +1,8 @@
 github.com/clipperhouse/uax29/v2 v2.2.0 h1:ChwIKnQN3kcZteTXMgb1wztSgaU+ZemkgWdohwgs8tY=
 github.com/clipperhouse/uax29/v2 v2.2.0/go.mod h1:EFJ2TJMRUaplDxHKj1qAEhCtQPW2tJSwu5BF98AuoVM=
 github.com/cpuguy83/go-md2man/v2 v2.0.6/go.mod h1:oOW0eioCTA6cOiMLiUPZOpcVxMig6NIQQ7OS05n1F4g=
-github.com/flashcatcloud/go-flashduty v0.5.2 h1:mYg/M0jqkil30WTLdICVtTJVGxEIGmae/3zBpRkwLRQ=
-github.com/flashcatcloud/go-flashduty v0.5.2/go.mod h1:aA0RtZEs0AYOwwdNKdtVeD8YMOdnmVY1zAlVD+9Ovx8=
+github.com/flashcatcloud/go-flashduty v0.5.3-0.20260602031007-62b37649b2f0 h1:mk9ryHQVssVA3qqyH4ryqeWa6sW0tYdww3JWalN3ZH0=
+github.com/flashcatcloud/go-flashduty v0.5.3-0.20260602031007-62b37649b2f0/go.mod h1:aA0RtZEs0AYOwwdNKdtVeD8YMOdnmVY1zAlVD+9Ovx8=
 github.com/inconshreveable/mousetrap v1.1.0 h1:wN+x4NVGpMsO7ErUn/mUI3vEoE6Jt13X2s0bqwp9tc8=
 github.com/inconshreveable/mousetrap v1.1.0/go.mod h1:vpF70FUmC8bwa3OWnCshd2FqLfsEA9PFc4w1p2J65bw=
 github.com/mattn/go-runewidth v0.0.23 h1:7ykA0T0jkPpzSvMS5i9uoNn2Xy3R383f9HDx3RybWcw=

internal/cli/audit.go

@@ -27,7 +27,7 @@ func newAuditSearchCmd() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   "search",
 		Short: "Search audit logs",
-		Long:  curatedLong("Search audit logs within a time window, optionally filtered by person and operation type.", "AuditLogs", "Search"),
+		Long:  curatedLong("Search audit logs within a time window, optionally filtered by person and operation type. The --since/--until window must be < 90 days; --limit max is 99.", "AuditLogs", "Search"),
 		RunE: func(cmd *cobra.Command, args []string) error {
 			return runCommand(cmd, args, func(ctx *RunContext) error {
 				startTime, err := timeutil.Parse(since)
@@ -107,7 +107,7 @@ func newAuditSearchCmd() *cobra.Command {
 	cmd.Flags().StringVar(&since, "since", "7d", "Start time")
 	cmd.Flags().StringVar(&until, "until", "now", "End time")
 	cmd.Flags().Int64Var(&person, "person", 0, "Filter by person ID")
-	cmd.Flags().StringVar(&operation, "operation", "", "Filter by operation type")
+	cmd.Flags().StringVar(&operation, "operation", "", "Filter by exact operation name(s) from 'fduty audit operation-list' (e.g. monitRule:write:update); comma-separate to match several in one call. Prefixes do NOT match (\"monitRule\" returns nothing).")
 	cmd.Flags().IntVar(&limit, "limit", 20, "Max results (max 99)")
 	cmd.Flags().IntVar(&page, "page", 1, "Page number")
 

internal/cli/command.go

@@ -73,6 +73,15 @@ func (ctx *RunContext) WriteResult(message string) {
 	writeResult(ctx.Writer, message)
 }
 
+// WriteRaw writes a non-JSON response body (e.g. a CSV/file download surfaced
+// on Response.Raw by the *export endpoints) straight to the output writer, so
+// shell redirection (`> file.csv`) captures the bytes verbatim instead of the
+// canned "OK: POST ..." acknowledgment.
+func (ctx *RunContext) WriteRaw(body []byte) error {
+	_, err := ctx.Writer.Write(body)
+	return err
+}
+
 // WriteResultJSON outputs structured data in JSON or TOON mode, or a
 // human-readable message in table mode. JSON stays indented (byte-compatible
 // with the legacy --json path); TOON routes through the SDK marshaller.

internal/cli/incident.go

@@ -80,7 +80,7 @@ func newIncidentListCmd() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   "list",
 		Short: "List incidents",
-		Long:  curatedLong("List incidents matching the given filters.", "Incidents", "List"),
+		Long:  curatedLong("List incidents matching the given filters. The --since/--until window must be < 31 days; --limit max is 100.", "Incidents", "List"),
 		RunE: func(cmd *cobra.Command, args []string) error {
 			return runCommand(cmd, args, func(ctx *RunContext) error {
 				startTime, err := timeutil.Parse(since)
@@ -119,7 +119,7 @@ func newIncidentListCmd() *cobra.Command {
 	cmd.Flags().StringVar(&severity, "severity", "", "Filter: Critical,Warning,Info")
 	cmd.Flags().Int64Var(&channelID, "channel", 0, "Filter by channel ID")
 	cmd.Flags().StringVar(&query, "query", "", "Free-text search across title/labels/content (also resolves a 24-char incident ID or 6-char incident num to a direct lookup)")
-	cmd.Flags().StringVar(&since, "since", "24h", "Start time (duration, date, datetime, or unix timestamp)")
+	cmd.Flags().StringVar(&since, "since", "24h", "Start time (duration, date, datetime, or unix timestamp; --since→--until window must be < 31 days)")
 	cmd.Flags().StringVar(&until, "until", "now", "End time")
 	cmd.Flags().IntVar(&limit, "limit", 20, "Max results (max 100)")
 	cmd.Flags().IntVar(&page, "page", 1, "Page number")
@@ -135,25 +135,36 @@ func newIncidentGetCmd() *cobra.Command {
 		Args:  requireArgs("incident_id"),
 		RunE: func(cmd *cobra.Command, args []string) error {
 			return runCommand(cmd, args, func(ctx *RunContext) error {
-				result, _, err := ctx.Client.Incidents.List(cmdContext(ctx.Cmd), &flashduty.ListIncidentsRequest{
-					IncidentIDs: ctx.Args,
-				})
-				if err != nil {
-					return err
+				// Fetch each incident by ID via /incident/info: it works for
+				// any id with no time window. /incident/list cannot serve a
+				// plain get-by-id — it mandates a start/end window and caps it
+				// at 31 days, so it 400s ("StartTime is a required field") when
+				// queried by id alone.
+				items := make([]flashduty.IncidentInfo, 0, len(ctx.Args))
+				for _, id := range ctx.Args {
+					info, _, err := ctx.Client.Incidents.Info(cmdContext(ctx.Cmd), &flashduty.IncidentInfoRequest{
+						IncidentID: id,
+					})
+					if err != nil {
+						return fmt.Errorf("get incident %s: %w", id, err)
+					}
+					if info != nil {
+						items = append(items, *info)
+					}
 				}
 
 				if ctx.Structured() {
-					return ctx.Printer.Print(result.Items, nil)
+					return ctx.Printer.Print(items, nil)
 				}
 
 				// Single incident: vertical detail view
-				if len(ctx.Args) == 1 && len(result.Items) == 1 {
-					printIncidentDetail(ctx.Writer, result.Items[0])
+				if len(items) == 1 {
+					printIncidentDetail(ctx.Writer, items[0])
 					return nil
 				}
 
 				// Multiple: table
-				return ctx.Printer.Print(result.Items, incidentColumns())
+				return ctx.Printer.Print(items, incidentColumns())
 			})
 		},
 	}