mirror of
https://github.com/larksuite/cli.git
synced 2026-07-05 15:47:54 +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.
80 lines
2.4 KiB
Go
80 lines
2.4 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/shortcuts/common"
|
||
)
|
||
|
||
// AppsCreate creates a new Miaoda app.
|
||
var AppsCreate = common.Shortcut{
|
||
Service: appsService,
|
||
Command: "+create",
|
||
Description: "Create a new Miaoda app",
|
||
Risk: "write",
|
||
Scopes: []string{"spark:app:write"},
|
||
AuthTypes: []string{"user"},
|
||
HasFormat: true,
|
||
Flags: []common.Flag{
|
||
{Name: "name", Desc: "app display name", Required: true},
|
||
{Name: "app-type", Desc: "app type (currently only: HTML)", Required: true},
|
||
{Name: "description", Desc: "app description"},
|
||
{Name: "icon-url", Desc: "app icon URL (server uses default if omitted)"},
|
||
},
|
||
Validate: func(ctx context.Context, rctx *common.RuntimeContext) error {
|
||
if strings.TrimSpace(rctx.Str("name")) == "" {
|
||
return output.ErrValidation("--name is required")
|
||
}
|
||
appType := strings.TrimSpace(rctx.Str("app-type"))
|
||
if appType == "" {
|
||
return output.ErrValidation("--app-type is required")
|
||
}
|
||
if !validAppTypes[appType] {
|
||
return output.ErrValidation(fmt.Sprintf("--app-type %q is not supported (allowed: HTML)", appType))
|
||
}
|
||
return nil
|
||
},
|
||
DryRun: func(ctx context.Context, rctx *common.RuntimeContext) *common.DryRunAPI {
|
||
return common.NewDryRunAPI().
|
||
POST(apiBasePath + "/apps").
|
||
Desc("Create a Miaoda app").
|
||
Body(buildAppsCreateBody(rctx))
|
||
},
|
||
Execute: func(ctx context.Context, rctx *common.RuntimeContext) error {
|
||
data, err := rctx.CallAPI("POST", apiBasePath+"/apps", nil, buildAppsCreateBody(rctx))
|
||
if err != nil {
|
||
return err
|
||
}
|
||
rctx.OutFormat(data, nil, func(w io.Writer) {
|
||
fmt.Fprintf(w, "created: %s\n", common.GetString(data, "app", "app_id"))
|
||
})
|
||
return nil
|
||
},
|
||
}
|
||
|
||
// 应用类型枚举。当前只有 HTML，未来会扩展（SPA、NATIVE、...）。
|
||
var validAppTypes = map[string]bool{
|
||
"HTML": true,
|
||
}
|
||
|
||
func buildAppsCreateBody(rctx *common.RuntimeContext) map[string]interface{} {
|
||
body := map[string]interface{}{
|
||
"name": strings.TrimSpace(rctx.Str("name")),
|
||
"app_type": strings.TrimSpace(rctx.Str("app-type")),
|
||
}
|
||
if desc := strings.TrimSpace(rctx.Str("description")); desc != "" {
|
||
body["description"] = desc
|
||
}
|
||
if icon := strings.TrimSpace(rctx.Str("icon-url")); icon != "" {
|
||
body["icon_url"] = icon
|
||
}
|
||
return body
|
||
}
|