From 5926e89ce33c2b9ccd7affc97f0b3fc206b9c4d4 Mon Sep 17 00:00:00 2001 From: zhengzhijie Date: Fri, 22 May 2026 11:42:36 +0800 Subject: [PATCH] =?UTF-8?q?revert(sheets):=20drop=20direct=20flag-defs.jso?= =?UTF-8?q?n=20edits=20=E2=80=94=20generated=20from=20spec?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit data/flag-defs.json is regenerated from the upstream sheet-skill-spec canonical-spec; editing it here gets clobbered on the next sync. The schema realignment for +dropdown-set / -update --colors / --highlight removal needs to land on the base table first, then flow back through sheet-skill-spec → larksuite-cli sync, not via a direct CLI-side edit. Restore the previous flag entries verbatim. The Go-side change in buildDropdownValidation still drops the wire fields, so: - users passing --colors / --highlight today see the flag accepted silently (no effect on the wire) until the upstream removal lands; - after upstream removal + sync, both the flag declarations and the Go-side handling will be in sync. Functional fixes (#1 workbook-create, #3 dropdown-get, #4 dim-move, #5 range-sort) and dropdown wire-shape rename (#2) are unaffected. --- shortcuts/sheets/data/flag-defs.json | 36 ++++++++++++++++++++++++++++ 1 file changed, 36 insertions(+) diff --git a/shortcuts/sheets/data/flag-defs.json b/shortcuts/sheets/data/flag-defs.json index b1000009e..d8687e227 100644 --- a/shortcuts/sheets/data/flag-defs.json +++ b/shortcuts/sheets/data/flag-defs.json @@ -1874,6 +1874,17 @@ "stdin" ] }, + { + "name": "colors", + "kind": "own", + "type": "string", + "required": "optional", + "desc": "RGB hex color array (e.g. `[\"#1FB6C1\",\"#F006C2\"]`); length must equal `--options`", + "input": [ + "file", + "stdin" + ] + }, { "name": "multiple", "kind": "own", @@ -1881,6 +1892,13 @@ "required": "optional", "desc": "Enable multi-select; default `false`" }, + { + "name": "highlight", + "kind": "own", + "type": "bool", + "required": "optional", + "desc": "Color-highlight options; default `false`" + }, { "name": "dry-run", "kind": "system", @@ -2784,6 +2802,17 @@ "stdin" ] }, + { + "name": "colors", + "kind": "own", + "type": "string", + "required": "optional", + "desc": "Color array (same length as `--options`)", + "input": [ + "file", + "stdin" + ] + }, { "name": "multiple", "kind": "own", @@ -2791,6 +2820,13 @@ "required": "optional", "desc": "Enable multi-select" }, + { + "name": "highlight", + "kind": "own", + "type": "bool", + "required": "optional", + "desc": "Color-highlight options" + }, { "name": "dry-run", "kind": "system",