Skip to content

fix: sheets scope typo, IM flag docs, dry-run membership warning, proxy warning env var #976

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mengqiuzhen wants to merge 3 commits into
base: main
Choose a base branch
from
+21 −12
Open

fix: sheets scope typo, IM flag docs, dry-run membership warning, proxy warning env var #976

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
1122127
fix(sheets): correct scope name from "read" to "readonly" matching Fe…
mengqiuzhen May 20, 2026
90a2a7a
fix(im): document relative-path restriction on media flags and add dr…
mengqiuzhen May 20, 2026
421a333
Revert "fix(sheets): correct scope name from "read" to "readonly" mat…
mengqiuzhen May 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion internal/util/proxy.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,9 @@ import (
const (
// EnvNoProxy disables automatic proxy support when set to any non-empty value.
EnvNoProxy = "LARK_CLI_NO_PROXY"
// EnvNoProxyWarning suppresses the one-time proxy warning when set to any non-empty value.
// The proxy itself is still used; only the stderr warning is silenced.
EnvNoProxyWarning = "LARK_CLI_NO_PROXY_WARNING"
)

// proxyEnvKeys lists environment variables that Go's ProxyFromEnvironment reads.
Expand Down Expand Up @@ -60,7 +63,7 @@ func redactProxyURL(raw string) string {
// are redacted. Safe to call multiple times; only the first call prints.
func WarnIfProxied(w io.Writer) {
proxyWarningOnce.Do(func() {
if os.Getenv(EnvNoProxy) != "" {
if os.Getenv(EnvNoProxy) != "" || os.Getenv(EnvNoProxyWarning) != "" {
return
}
key, val := DetectProxyEnv()
Expand Down
10 changes: 5 additions & 5 deletions shortcuts/im/im_messages_reply.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,11 @@ var ImMessagesReply = common.Shortcut{
{Name: "content", Desc: "(one of --content/--text/--markdown/--image/--file/--video/--audio required) message content JSON"},
{Name: "text", Desc: "plain text message (auto-wrapped as JSON)"},
{Name: "markdown", Desc: "markdown text (auto-wrapped as post format with style optimization; image URLs auto-resolved)"},
{Name: "image", Desc: "image_key, local file path"},
{Name: "file", Desc: "file_key, local file path"},
{Name: "video", Desc: "video file_key, local file path; must be used together with --video-cover"},
{Name: "video-cover", Desc: "video cover image_key, local file path; required when using --video"},
{Name: "audio", Desc: "audio file_key, local file path"},
{Name: "image", Desc: "image_key or relative path to file"},
{Name: "file", Desc: "file_key or relative path to file"},
{Name: "video", Desc: "video file_key or relative path to file; must be used together with --video-cover"},
{Name: "video-cover", Desc: "video cover image_key or relative path to file; required when using --video"},
{Name: "audio", Desc: "audio file_key or relative path to file"},
{Name: "reply-in-thread", Type: "bool", Desc: "reply in thread (message appears in thread stream instead of main chat)"},
{Name: "idempotency-key", Desc: "idempotency key (prevents duplicate sends)"},
},
Expand Down
18 changes: 12 additions & 6 deletions shortcuts/im/im_messages_send.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,11 +33,11 @@ var ImMessagesSend = common.Shortcut{
{Name: "text", Desc: "plain text message (auto-wrapped as JSON)"},
{Name: "markdown", Desc: "markdown text (auto-wrapped as post format with style optimization; image URLs auto-resolved)"},
{Name: "idempotency-key", Desc: "idempotency key (prevents duplicate sends)"},
{Name: "image", Desc: "image_key, local file path"},
{Name: "file", Desc: "file_key, local file path"},
{Name: "video", Desc: "video file_key, local file path; must be used together with --video-cover"},
{Name: "video-cover", Desc: "video cover image_key, local file path; required when using --video"},
{Name: "audio", Desc: "audio file_key, local file path"},
{Name: "image", Desc: "image_key or relative path to file"},
{Name: "file", Desc: "file_key or relative path to file"},
{Name: "video", Desc: "video file_key or relative path to file; must be used together with --video-cover"},
{Name: "video-cover", Desc: "video cover image_key or relative path to file; required when using --video"},
{Name: "audio", Desc: "audio file_key or relative path to file"},
},
DryRun: func(ctx context.Context, runtime *common.RuntimeContext) *common.DryRunAPI {
chatFlag := runtime.Str("chat-id")
Expand Down Expand Up @@ -81,10 +81,16 @@ var ImMessagesSend = common.Shortcut{
if desc != "" {
d.Desc(desc)
}
return d.
d = d.
POST("/open-apis/im/v1/messages").
Params(map[string]interface{}{"receive_id_type": receiveIdType}).
Body(body)

if chatFlag != "" {
d.Desc("dry-run validates request shape only — it does not verify that the selected identity is a member of the target chat; a real send can still fail with \"Bot/User can NOT be out of the chat\"")
}

return d
},
Validate: func(ctx context.Context, runtime *common.RuntimeContext) error {
chatFlag := runtime.Str("chat-id")
Expand Down
Loading