From dda3e7adb048677a65ff19ecd5604afdc810302e Mon Sep 17 00:00:00 2001 From: zhengzhijie Date: Fri, 17 Jul 2026 15:54:51 +0800 Subject: [PATCH] fix(sheets): mark undo shortcut high risk --- shortcuts/sheets/data/flag-defs.json | 2 +- shortcuts/sheets/flag_defs_gen.go | 2 +- shortcuts/sheets/lark_sheet_undo.go | 2 +- shortcuts/sheets/lark_sheet_undo_test.go | 3 +++ 4 files changed, 6 insertions(+), 3 deletions(-) diff --git a/shortcuts/sheets/data/flag-defs.json b/shortcuts/sheets/data/flag-defs.json index 0b744bb41..b4cc0841c 100644 --- a/shortcuts/sheets/data/flag-defs.json +++ b/shortcuts/sheets/data/flag-defs.json @@ -4976,7 +4976,7 @@ ] }, "+undo": { - "risk": "write", + "risk": "high-risk-write", "flags": [ { "name": "url", diff --git a/shortcuts/sheets/flag_defs_gen.go b/shortcuts/sheets/flag_defs_gen.go index a4faaaa3c..56455830a 100644 --- a/shortcuts/sheets/flag_defs_gen.go +++ b/shortcuts/sheets/flag_defs_gen.go @@ -998,7 +998,7 @@ var flagDefs = map[string]commandDef{ }, }, "+undo": { - Risk: "write", + Risk: "high-risk-write", Flags: []flagDef{ {Name: "url", Kind: "public", Type: "string", Required: "xor", Desc: "Spreadsheet locator"}, {Name: "spreadsheet-token", Kind: "public", Type: "string", Required: "xor", Desc: "Spreadsheet locator"}, diff --git a/shortcuts/sheets/lark_sheet_undo.go b/shortcuts/sheets/lark_sheet_undo.go index a768a1a9d..7c06eff3c 100644 --- a/shortcuts/sheets/lark_sheet_undo.go +++ b/shortcuts/sheets/lark_sheet_undo.go @@ -13,7 +13,7 @@ var Undo = common.Shortcut{ Service: "sheets", Command: "+undo", Description: "Undo the current user's latest spreadsheet write.", - Risk: "write", + Risk: "high-risk-write", Scopes: []string{"sheets:spreadsheet:write_only"}, AuthTypes: []string{"user"}, HasFormat: true, diff --git a/shortcuts/sheets/lark_sheet_undo_test.go b/shortcuts/sheets/lark_sheet_undo_test.go index d910a9ee9..236bc7460 100644 --- a/shortcuts/sheets/lark_sheet_undo_test.go +++ b/shortcuts/sheets/lark_sheet_undo_test.go @@ -10,6 +10,9 @@ import ( func TestUndo_DryRun(t *testing.T) { t.Parallel() + if Undo.Risk != "high-risk-write" { + t.Fatalf("Undo.Risk = %q, want high-risk-write", Undo.Risk) + } args := []string{"--url", testURL, "--as", "user"} callURL := dryRunFirstCallURL(t, Undo, args)