fix(sheets): migrate +table-put to typed error contract

The merge from main brought in #1449 (retire legacy error envelopes),
which removed output.ExitError / output.ErrDetail and forbids
constructing them. Port tablePutPartial off the legacy envelope:

- no sheets written -> typed errs.APIError (plain failure)
- some sheets written -> ok:false result via runtime.OutPartialFailure
  carrying written_sheets, returning the partial-failure exit signal

Also fix two drifts the same merge introduced:
- regenerate flag_defs_gen.go to match the committed flag-defs.json
- update the --max-chars flag test to assert visible (no longer hidden)

Author: xiongyuanwen-byted

shortcuts/sheets/flag_defs_gen.go

@@ -65,9 +65,9 @@ func TestFlagsFor_MapsAllFields(t *testing.T) {
 	if url == nil || url.Required {
 		t.Errorf("+sheet-create --url should not be cobra-required: %+v", url)
 	}
-	// hidden + int default
+	// visible + int default
 	cap := byName("+cells-get", "max-chars")
-	if cap == nil || !cap.Hidden || cap.Default != "500000" {
+	if cap == nil || cap.Hidden || cap.Default != "500000" {
 		t.Errorf("+cells-get --max-chars not mapped: %+v", cap)
 	}
 	// input sources

shortcuts/sheets/flag_defs_test.go

@@ -13,7 +13,6 @@ import (
 	"time"
 
 	"github.com/larksuite/cli/errs"
-	"github.com/larksuite/cli/internal/output"
 	"github.com/larksuite/cli/internal/util"
 	"github.com/larksuite/cli/shortcuts/common"
 )
@@ -818,7 +817,7 @@ func renameSheet(ctx context.Context, runtime *common.RuntimeContext, token, she
 func tablePutWrite(ctx context.Context, runtime *common.RuntimeContext, token string, payload *tablePayload, styles *workbookCreateSheetStyles) error {
 	written, err := writeTypedSheets(ctx, runtime, token, payload, "", styles)
 	if err != nil {
-		return tablePutPartial(token, nil, written, err.Error())
+		return tablePutPartial(runtime, token, nil, written, err.Error())
 	}
 	runtime.Out(map[string]interface{}{
 		"spreadsheet_token": token,
@@ -1005,39 +1004,28 @@ func listSheetIDsByName(ctx context.Context, runtime *common.RuntimeContext, tok
 	return byName, dimsByName, nil
 }
 
-// tablePutPartial builds a structured error for a multi-sheet write that failed
-// partway. When some sheets already landed it's a partial_success (their
-// summaries are surfaced so callers can retry the rest or delete the workbook);
-// when nothing landed — the first or only sheet failed — it's a plain failure,
-// so we don't misleadingly claim "some sheets were written".
-func tablePutPartial(token string, spreadsheet interface{}, written []interface{}, reason string) error {
-	detail := map[string]interface{}{
+// tablePutPartial reports a multi-sheet write that failed partway. When some
+// sheets already landed it is a partial_success: their summaries are the primary
+// machine-readable output, so we emit an ok:false result envelope on stdout (via
+// OutPartialFailure) carrying written_sheets, and return the partial-failure exit
+// signal — callers can retry the rest or delete the workbook. When nothing landed
+// — the first or only sheet failed — it is a plain failure, so we return a typed
+// errs.APIError rather than misleadingly claiming "some sheets were written".
+func tablePutPartial(runtime *common.RuntimeContext, token string, spreadsheet interface{}, written []interface{}, reason string) error {
+	if len(written) == 0 {
+		return errs.NewAPIError(errs.SubtypeServerError, "table-put failed on %s: %s", token, reason).
+			WithHint("no sheets were written; fix the cause and retry")
+	}
+	data := map[string]interface{}{
 		"spreadsheet_token": token,
 		"written_sheets":    written,
+		"reason":            reason,
+		"hint":              "some sheets were written; inspect written_sheets, then retry the remaining sheets or delete the spreadsheet",
 	}
 	if spreadsheet != nil {
-		detail["spreadsheet"] = spreadsheet
-	}
-	if len(written) == 0 {
-		return &output.ExitError{
-			Code: output.ExitAPI,
-			Detail: &output.ErrDetail{
-				Type:    "api_error",
-				Message: fmt.Sprintf("table-put failed on %s: %s", token, reason),
-				Hint:    "no sheets were written; fix the cause and retry",
-				Detail:  detail,
-			},
-		}
-	}
-	return &output.ExitError{
-		Code: output.ExitAPI,
-		Detail: &output.ErrDetail{
-			Type:    "partial_success",
-			Message: fmt.Sprintf("table-put partially applied to %s: %s", token, reason),
-			Hint:    "some sheets were written; inspect written_sheets, then retry the remaining sheets or delete the spreadsheet",
-			Detail:  detail,
-		},
+		data["spreadsheet"] = spreadsheet
 	}
+	return runtime.OutPartialFailure(data, nil)
 }
 
 // ─── dry-run ──────────────────────────────────────────────────────────

shortcuts/sheets/lark_sheet_table_io.go

@@ -5,11 +5,13 @@ package sheets
 
 import (
 	"encoding/json"
+	"errors"
 	"fmt"
 	"strings"
 	"testing"
 
 	"github.com/larksuite/cli/internal/httpmock"
+	"github.com/larksuite/cli/internal/output"
 )
 
 // ─── pure helpers: date serial, typed cell mapping ────────────────────
@@ -654,15 +656,22 @@ func TestTablePut_ExecutePartialFailure(t *testing.T) {
 			`{"sheets":[{"name":"汇总","columns":["a"],"data":[["x"]]},{"name":"明细","columns":["a"],"data":[["y"]]}]}`},
 		structure, writeOK, writeErr)
 	if err == nil {
-		t.Fatalf("expected partial-success error; got nil. out=%s", out)
-	}
-	if !strings.Contains(err.Error(), "partially applied") && !strings.Contains(out, "partially applied") {
-		t.Errorf("expected partial_success (not total failure); got err=%v out=%s", err, out)
-	}
-	// The failing sheet is named in the message; the written one lives in the
-	// structured written_sheets detail.
-	if !strings.Contains(err.Error(), "明细") {
-		t.Errorf("partial_success should name the failed sheet 明细; got err=%v", err)
+		t.Fatalf("expected partial-failure exit signal; got nil. out=%s", out)
+	}
+	// Per the typed-error contract, a partial success is a result, not an error
+	// envelope: the ok:false result (with written_sheets and the failing sheet's
+	// name) lands on stdout, and err is the bare partial-failure exit signal.
+	var pfErr *output.PartialFailureError
+	if !errors.As(err, &pfErr) {
+		t.Errorf("expected *output.PartialFailureError exit signal; got %T %v", err, err)
+	}
+	if !strings.Contains(out, "written_sheets") {
+		t.Errorf("expected written_sheets in the partial-success result; got out=%s", out)
+	}
+	// The failing sheet is named in the reason; the written one lives in
+	// written_sheets — both on stdout.
+	if !strings.Contains(out, "明细") {
+		t.Errorf("partial_success should name the failed sheet 明细; got out=%s", out)
 	}
 }