mirror of
https://github.com/larksuite/cli.git
synced 2026-07-04 23:15:25 +08:00
0dda56914d
* fix(apps): read app object from data.app for +create and +update
The Miaoda OpenAPI returns the application object nested under
data.app for both POST /apps and PATCH /apps/{appId}. The CLI text
helper was reading common.GetString(data, "app_id"), which yields an
empty string against the wire format -- so `lark-cli apps +create
--format pretty` printed `created: ` with no ID.
Navigate the new nested path via GetString(data, "app", "app_id") for
both create and update. Update unit-test mocks to wrap the response
under `app`. Refresh the lark-apps skill references (example response
shape + jq paths) so agents reading them follow the right path.
Wire format is passed through to the user's JSON envelope untouched
-- no unwrapping in CLI. Consumers reading the response should use
.data.app.app_id.
The GET /apps list endpoint is unchanged: per the design doc its
items[] are flat objects, no wrapper.
* docs(apps): add required --app-type HTML to scenario 2 snippet
The "用户没有 app_id" snippet in lark-apps-html-publish.md was missing
the required --app-type flag, so copy-pasting it triggered Validate
("--app-type is required") and left $APP empty -- the following
+html-publish then failed with --app-id "". Bring the snippet in line
with every other apps +create example in the skill.
* docs(apps): simplify auth-recovery rule to error.type == missing_scope
Every apps shortcut declares Scopes, so the precheck path in
shortcuts/common/runner.go:825 is always the one that fires on scope
violations and the envelope's error.type is the stable discriminator.
Drop the keyword-sniffing of error.hint, the chain explanation, and the
bot caveat — they all reduce to one boolean: error.type == "missing_scope"
→ run `lark-cli auth login --domain apps`.
Also collapse the corresponding bullet in 快速决策 to point at this rule.
72 lines
2.2 KiB
Go
72 lines
2.2 KiB
Go
// Copyright (c) 2026 Lark Technologies Pte. Ltd.
|
|
// SPDX-License-Identifier: MIT
|
|
|
|
package apps
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
"io"
|
|
"strings"
|
|
|
|
"github.com/larksuite/cli/internal/output"
|
|
"github.com/larksuite/cli/internal/validate"
|
|
"github.com/larksuite/cli/shortcuts/common"
|
|
)
|
|
|
|
// AppsUpdate partially updates a Miaoda app's name / description.
|
|
var AppsUpdate = common.Shortcut{
|
|
Service: appsService,
|
|
Command: "+update",
|
|
Description: "Partially update a Miaoda app (only provided fields are sent)",
|
|
Risk: "write",
|
|
Scopes: []string{"spark:app:write"},
|
|
AuthTypes: []string{"user"},
|
|
HasFormat: true,
|
|
Flags: []common.Flag{
|
|
{Name: "app-id", Desc: "app ID", Required: true},
|
|
{Name: "name", Desc: "new app display name"},
|
|
{Name: "description", Desc: "new app description"},
|
|
},
|
|
Validate: func(ctx context.Context, rctx *common.RuntimeContext) error {
|
|
if strings.TrimSpace(rctx.Str("app-id")) == "" {
|
|
return output.ErrValidation("--app-id is required")
|
|
}
|
|
body := buildAppsUpdateBody(rctx)
|
|
if len(body) == 0 {
|
|
return output.ErrValidation("provide at least one of --name or --description")
|
|
}
|
|
return nil
|
|
},
|
|
DryRun: func(ctx context.Context, rctx *common.RuntimeContext) *common.DryRunAPI {
|
|
appID := strings.TrimSpace(rctx.Str("app-id"))
|
|
return common.NewDryRunAPI().
|
|
PATCH(fmt.Sprintf("%s/apps/%s", apiBasePath, validate.EncodePathSegment(appID))).
|
|
Desc("Update a Miaoda app").
|
|
Body(buildAppsUpdateBody(rctx))
|
|
},
|
|
Execute: func(ctx context.Context, rctx *common.RuntimeContext) error {
|
|
appID := strings.TrimSpace(rctx.Str("app-id"))
|
|
path := fmt.Sprintf("%s/apps/%s", apiBasePath, validate.EncodePathSegment(appID))
|
|
data, err := rctx.CallAPI("PATCH", path, nil, buildAppsUpdateBody(rctx))
|
|
if err != nil {
|
|
return err
|
|
}
|
|
rctx.OutFormat(data, nil, func(w io.Writer) {
|
|
fmt.Fprintf(w, "updated: %s\n", common.GetString(data, "app", "app_id"))
|
|
})
|
|
return nil
|
|
},
|
|
}
|
|
|
|
func buildAppsUpdateBody(rctx *common.RuntimeContext) map[string]interface{} {
|
|
body := map[string]interface{}{}
|
|
if v := strings.TrimSpace(rctx.Str("name")); v != "" {
|
|
body["name"] = v
|
|
}
|
|
if v := strings.TrimSpace(rctx.Str("description")); v != "" {
|
|
body["description"] = v
|
|
}
|
|
return body
|
|
}
|