mirror of
https://github.com/larksuite/cli.git
synced 2026-07-03 14:02:43 +08:00
Introduce a typed error contract framework for lark-cli so in-process
Go callers can branch via errors.As(&errs.XxxError{}) and shell scripts,
AI agents, and protocol adapters can branch on stable JSON type/subtype
fields instead of regex-parsing free-form messages.
Adds:
- Canonical taxonomy under errs/ (9 categories + typed Error structs
embedding a shared Problem, RFC 7807-aligned)
- Centralized Lark code metadata + identity-aware BuildAPIError dispatch
- Typed JSON envelope writer alongside the legacy envelope writer
- MCP / OAuth (RFC 6750 Bearer) projection adapters
- Five CI lint guards preventing ad-hoc taxonomy drift
Backward compatibility: legacy *output.ExitError producers (ErrAPI,
ErrWithHint, Errorf, ErrBare) and business shortcuts that use them
continue to render the legacy envelope unchanged. SecurityPolicyError
wire format and exit code are preserved via a carve-out; taxonomy
migration is deferred to PR 2. Domain-specific business migration is
staged across PR 3+.
Framework-direct paths now return typed *errs.*Error: ErrAuth /
ErrValidation / ErrNetwork emit category literals on the wire
(authentication / validation / network), *core.ConfigError is promoted
at the cmd/root boundary with exit code aligned from 2 to 3, and Lark
API permission denials classified by BuildAPIError exit 3.
At the SDK boundary, WrapDoAPIError preserves any already-classified
error (legacy *output.ExitError or typed *errs.*) so output.ErrAuth
from missing credentials surfaces with the auth category and exit 3
intact instead of being downgraded to a network error. Policy responses
classified by BuildAPIError (codes 21000 / 21001) extract challenge_url
and the canonical hint from the response body, matching what the
auth transport already surfaces at the HTTP layer; non-https
challenge URLs are dropped.
First PR in the feat/error-contract-* series.
215 lines
6.8 KiB
Go
215 lines
6.8 KiB
Go
// Copyright (c) 2026 Lark Technologies Pte. Ltd.
|
|
// SPDX-License-Identifier: MIT
|
|
|
|
package cmd
|
|
|
|
import (
|
|
"errors"
|
|
"fmt"
|
|
"strings"
|
|
|
|
"github.com/spf13/cobra"
|
|
|
|
"github.com/larksuite/cli/errs"
|
|
internalauth "github.com/larksuite/cli/internal/auth"
|
|
"github.com/larksuite/cli/internal/cmdutil"
|
|
"github.com/larksuite/cli/internal/core"
|
|
"github.com/larksuite/cli/internal/output"
|
|
"github.com/larksuite/cli/internal/registry"
|
|
"github.com/larksuite/cli/shortcuts"
|
|
shortcutcommon "github.com/larksuite/cli/shortcuts/common"
|
|
)
|
|
|
|
// applyNeedAuthorizationHint augments a typed *errs.AuthenticationError with a
|
|
// "current command requires scope(s): X, Y" hint when the underlying error is
|
|
// a need_user_authorization signal AND the current command declares scopes
|
|
// locally (via shortcut registration or service-method metadata).
|
|
//
|
|
// Stage-1: this typed path is dormant — no production code returns a typed
|
|
// *errs.AuthenticationError. Kept so per-domain stage-2 migrations can plug
|
|
// in without re-architecting. The active stage-1 path is
|
|
// enrichMissingScopeError below, which operates on legacy *output.ExitError.
|
|
func applyNeedAuthorizationHint(f *cmdutil.Factory, err error) {
|
|
if err == nil || f == nil {
|
|
return
|
|
}
|
|
if !internalauth.IsNeedUserAuthorizationError(err) {
|
|
return
|
|
}
|
|
var authErr *errs.AuthenticationError
|
|
if !errors.As(err, &authErr) {
|
|
return
|
|
}
|
|
scopes := resolveDeclaredScopesForCurrentCommand(f)
|
|
if len(scopes) == 0 {
|
|
return
|
|
}
|
|
scopeHint := fmt.Sprintf("current command requires scope(s): %s", strings.Join(scopes, ", "))
|
|
if authErr.Hint == "" {
|
|
authErr.Hint = scopeHint
|
|
return
|
|
}
|
|
authErr.Hint += "\n" + scopeHint
|
|
}
|
|
|
|
// enrichMissingScopeError appends a "current command requires scope(s): X"
|
|
// hint to a legacy *output.ExitError when the underlying error carries the
|
|
// need_user_authorization marker AND the current command declares scopes
|
|
// locally. Matches pre-PR behaviour byte-for-byte; lives on the legacy
|
|
// envelope path until per-domain stage-2 typed migration.
|
|
//
|
|
// Deprecated: stage-1 enrichment for the legacy *output.ExitError surface.
|
|
// Stage-2 typed migration will lift this into AuthenticationError.Hint on
|
|
// the typed envelope via applyNeedAuthorizationHint and remove this helper.
|
|
func enrichMissingScopeError(f *cmdutil.Factory, exitErr *output.ExitError) {
|
|
if exitErr == nil || exitErr.Detail == nil {
|
|
return
|
|
}
|
|
if !internalauth.IsNeedUserAuthorizationError(exitErr) {
|
|
return
|
|
}
|
|
scopes := resolveDeclaredScopesForCurrentCommand(f)
|
|
if len(scopes) == 0 {
|
|
return
|
|
}
|
|
scopeHint := fmt.Sprintf("current command requires scope(s): %s", strings.Join(scopes, ", "))
|
|
if exitErr.Detail.Hint == "" {
|
|
exitErr.Detail.Hint = scopeHint
|
|
return
|
|
}
|
|
exitErr.Detail.Hint += "\n" + scopeHint
|
|
}
|
|
|
|
// resolveDeclaredScopesForCurrentCommand returns the scopes declared by the
|
|
// current command for the resolved identity, checking shortcuts first and then
|
|
// service methods from local registry metadata.
|
|
func resolveDeclaredScopesForCurrentCommand(f *cmdutil.Factory) []string {
|
|
if f == nil || f.CurrentCommand == nil {
|
|
return nil
|
|
}
|
|
|
|
identity := string(f.ResolvedIdentity)
|
|
if identity == "" {
|
|
identity = string(core.AsUser)
|
|
}
|
|
if identity != string(core.AsUser) && identity != string(core.AsBot) {
|
|
return nil
|
|
}
|
|
|
|
if scopes := resolveDeclaredShortcutScopes(f.CurrentCommand, identity); len(scopes) > 0 {
|
|
return scopes
|
|
}
|
|
return resolveDeclaredServiceMethodScopes(f.CurrentCommand, identity)
|
|
}
|
|
|
|
// resolveDeclaredShortcutScopes returns the scopes declared by a mounted
|
|
// shortcut command for the given identity.
|
|
func resolveDeclaredShortcutScopes(cmd *cobra.Command, identity string) []string {
|
|
if cmd == nil || cmd.Parent() == nil || !strings.HasPrefix(cmd.Name(), "+") {
|
|
return nil
|
|
}
|
|
|
|
service := cmd.Parent().Name()
|
|
for _, sc := range shortcuts.AllShortcuts() {
|
|
if sc.Service != service || sc.Command != cmd.Name() || !shortcutSupportsIdentity(sc, identity) {
|
|
continue
|
|
}
|
|
scopes := sc.DeclaredScopesForIdentity(identity)
|
|
if len(scopes) == 0 {
|
|
return nil
|
|
}
|
|
return append([]string(nil), scopes...)
|
|
}
|
|
return nil
|
|
}
|
|
|
|
// resolveDeclaredServiceMethodScopes returns the scopes declared by a
|
|
// service/resource/method command from the embedded from_meta registry.
|
|
func resolveDeclaredServiceMethodScopes(cmd *cobra.Command, identity string) []string {
|
|
// Service-method scope lookup only applies to commands mounted as
|
|
// root -> service -> resource -> method. Non-resource/method commands
|
|
// intentionally return no scopes here so auth-hint enrichment does not
|
|
// change runtime semantics for other command shapes.
|
|
if cmd == nil || cmd.Parent() == nil || cmd.Parent().Parent() == nil || cmd.Parent().Parent().Parent() == nil {
|
|
return nil
|
|
}
|
|
if strings.HasPrefix(cmd.Name(), "+") {
|
|
return nil
|
|
}
|
|
|
|
service := cmd.Parent().Parent().Name()
|
|
resource := cmd.Parent().Name()
|
|
method := cmd.Name()
|
|
|
|
spec := registry.LoadFromMeta(service)
|
|
if spec == nil {
|
|
return nil
|
|
}
|
|
resources, _ := spec["resources"].(map[string]interface{})
|
|
resMap, _ := resources[resource].(map[string]interface{})
|
|
if resMap == nil {
|
|
return nil
|
|
}
|
|
methods, _ := resMap["methods"].(map[string]interface{})
|
|
methodMap, _ := methods[method].(map[string]interface{})
|
|
if methodMap == nil {
|
|
return nil
|
|
}
|
|
return declaredScopesForMethod(methodMap, identity)
|
|
}
|
|
|
|
// declaredScopesForMethod returns all requiredScopes when present; otherwise it
|
|
// resolves the single recommended scope from the method's scopes list.
|
|
func declaredScopesForMethod(method map[string]interface{}, identity string) []string {
|
|
if requiredRaw, ok := method["requiredScopes"].([]interface{}); ok && len(requiredRaw) > 0 {
|
|
return interfaceStrings(requiredRaw)
|
|
}
|
|
|
|
rawScopes, _ := method["scopes"].([]interface{})
|
|
if len(rawScopes) == 0 {
|
|
return nil
|
|
}
|
|
recommended := registry.SelectRecommendedScope(rawScopes, identity)
|
|
if recommended == "" {
|
|
for _, raw := range rawScopes {
|
|
if scope, ok := raw.(string); ok && scope != "" {
|
|
recommended = scope
|
|
break
|
|
}
|
|
}
|
|
}
|
|
if recommended == "" {
|
|
return nil
|
|
}
|
|
return []string{recommended}
|
|
}
|
|
|
|
// interfaceStrings converts a []interface{} containing strings into a compact
|
|
// []string, skipping empty or non-string values.
|
|
func interfaceStrings(values []interface{}) []string {
|
|
scopes := make([]string, 0, len(values))
|
|
for _, value := range values {
|
|
scope, ok := value.(string)
|
|
if !ok || scope == "" {
|
|
continue
|
|
}
|
|
scopes = append(scopes, scope)
|
|
}
|
|
return scopes
|
|
}
|
|
|
|
// shortcutSupportsIdentity reports whether a shortcut supports the requested
|
|
// identity, applying the default user-only behavior when AuthTypes is empty.
|
|
func shortcutSupportsIdentity(sc shortcutcommon.Shortcut, identity string) bool {
|
|
authTypes := sc.AuthTypes
|
|
if len(authTypes) == 0 {
|
|
authTypes = []string{string(core.AsUser)}
|
|
}
|
|
for _, authType := range authTypes {
|
|
if authType == identity {
|
|
return true
|
|
}
|
|
}
|
|
return false
|
|
}
|