fix: reject victorialogs raw rows query missing start/end args

monit-query rows dispatched --ds-type victorialogs with
--args victorialogs.type=raw straight to the backend without
checking for victorialogs.start/victorialogs.end. The backend
silently returns null (HTTP 200, exit 0) instead of an error,
which caused an agent session to retry the same broken call
~10+ times before finding the doc that explains the raw-mode
contract. Validate this known-bad combination client-side
before dispatch, mirroring the existing required-flags check
in the same RunE.

Author: ysyneu

internal/cli/monit_query.go

@@ -102,6 +102,10 @@ func newMonitQueryRowsCmd() *cobra.Command {
 			if err != nil {
 				return fmt.Errorf("invalid --args: %w", err)
 			}
+			if dsType == "victorialogs" && argsMap["victorialogs.type"] == "raw" &&
+				(argsMap["victorialogs.start"] == "" || argsMap["victorialogs.end"] == "") {
+				return fmt.Errorf("victorialogs raw mode requires --args victorialogs.start=<unix> --args victorialogs.end=<unix>; see skills/flashduty/reference/monit-query.md")
+			}
 
 			return runCommand(cmd, args, func(ctx *RunContext) error {
 				input := &flashduty.QueryRowsRequest{

internal/cli/monit_query_test.go

@@ -259,3 +259,25 @@ func TestMonitQueryRowsInvalidArgs(t *testing.T) {
 		t.Errorf("rows should not have been called: %d request(s)", stub.requests)
 	}
 }
+
+func TestMonitQueryRowsVictorialogsRawRequiresTimeRange(t *testing.T) {
+	saveAndResetGlobals(t)
+	stub := newGFStub(t)
+
+	_, err := execCommand(
+		"monit-query", "rows",
+		"--ds-type", "victorialogs",
+		"--ds-name", "vl-prod",
+		"--expr", "* | limit 2",
+		"--args", "victorialogs.type=raw",
+	)
+	if err == nil {
+		t.Fatal("expected error for victorialogs raw mode missing start/end, got nil")
+	}
+	if !strings.Contains(err.Error(), "victorialogs.start") || !strings.Contains(err.Error(), "victorialogs.end") {
+		t.Errorf("expected error to mention victorialogs.start/end, got %q", err.Error())
+	}
+	if stub.requests != 0 {
+		t.Errorf("rows should not have been called: %d request(s)", stub.requests)
+	}
+}