fix: omit SDP command from skill reference

Author: ysyneu

internal/skilldoc/generate.go

@@ -44,9 +44,13 @@ func FenceEnd(group string) string   { return fmt.Sprintf(fenceEndFmt, group) }
 func groupCommands(d Dump, group string) []Command {
 	var cmds []Command
 	for _, c := range d.Commands {
-		if c.Group == group {
-			cmds = append(cmds, c)
+		if c.Group != group {
+			continue
+		}
+		if c.Path == "incident sdp-request-list" {
+			continue
 		}
+		cmds = append(cmds, c)
 	}
 	sort.Slice(cmds, func(i, j int) bool { return cmds[i].Path < cmds[j].Path })
 	return cmds

internal/skilldoc/generate_test.go

@@ -5,6 +5,29 @@ import (
 	"testing"
 )
 
+func TestGenerateFence_ExcludesLowFrequencyCommand(t *testing.T) {
+
+	d := Dump{Commands: []Command{
+
+		{Path: "incident list", Group: "incident", Short: "List incidents"},
+
+		{Path: "incident sdp-request-list", Group: "incident", Short: "Get ServiceDeskPlus linked incidents"},
+	}}
+
+	got := GenerateFence(d, "incident")
+	if !strings.Contains(got, "### list") {
+
+		t.Fatalf("ordinary incident command missing from generated fence:\n%s", got)
+
+	}
+	if strings.Contains(got, "sdp-request-list") {
+
+		t.Fatalf("low-frequency command must not appear in generated fence:\n%s", got)
+
+	}
+
+}
+
 // generatorDump mirrors the real cligen Long shape: a Request fields block with
 // a required enum flag, a non-required flag, and a nested --data body field.
 func generatorDump() Dump {

skills/flashduty/reference/incident.md

@@ -328,20 +328,6 @@ Add incident responder
 - `<person-ids>` (positional, required) intSlice — Member IDs to add as responders.
 - body-only (`--data`): notify (object)
 
-### sdp-request-list
-Get ServiceDeskPlus linked incidents
-- `--asc` bool — When 'true', sort by internal record ID ascending; otherwise descending.
-- `--channel-ids` intSlice — Channel IDs to filter by.
-- `--end-time` string — Window end, Unix seconds. Must be greater than or equal to 'start_time'. Optional when 'incident_id' is provided. (min 0) Accepts a duration (7d, 24h), '+7d' for the future, 'now', a date, or Unix seconds.
-- `--incident-id` string — Flashduty incident ID. When set, the time window can be omitted. (≤64 chars)
-- `--integration-id` int64 — ServiceDeskPlus integration ID. (min 0)
-- `--limit` int64 — Page size. Defaults to 20; maximum 100. (0-100)
-- `--page` int64 — Page number starting at 1. Ignored when 'search_after_ctx' is set. (min 0)
-- `--request-id` string — ServiceDeskPlus request ID. (≤64 chars)
-- `--search-after-ctx` string — Cursor returned by the previous page.
-- `--start-time` string — Window start, Unix seconds. Optional when 'incident_id' is provided. (min 0) Accepts a duration (7d, 24h), '+7d' for the future, 'now', a date, or Unix seconds.
-- `--status` string — Synchronization status filter. · enum: success | failed
-
 ### similar <id>
 Find similar incidents
 - `--limit` int