fix: reject positional arguments in shortcuts

[MaxHuang22]

Summary

• Shortcuts 之前会静默忽略位置参数（如 lark-cli docs +search "hello"），导致空搜索结果
• 在所有声明式 shortcut 的 cobra 命令上添加 Args 校验器，传入位置参数时打印 usage 并报错：positional arguments are not supported (got "hello"); pass values via flags
• 错误使用普通 error 而非 ExitError，避免输出 JSON 信封，保持 CLI 原生体验

Test plan

• go test ./shortcuts/common/... -run TestRejectPositionalArgs 通过
• go test ./shortcuts/common/... 全量通过
• go build ./... 编译通过
• 手动验证 lark-cli docs +search "hello" 输出 usage + 错误提示
• 手动验证 lark-cli docs +search --query "hello" 正常工作

🤖 Generated with Claude Code

Summary by CodeRabbit

• Bug Fixes
  • Shortcut commands now reject positional arguments; values must be supplied via flags. Errors clearly explain positional args are unsupported and show the offending values.
• Tests
  • Added unit tests to validate rejection behavior for no args, single arg, and multiple args.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3f488169-1e87-49c5-83ce-878cda951ef3

📥 Commits

Reviewing files that changed from the base of the PR and between b4c020c and 1b6d962.

📒 Files selected for processing (2)

• shortcuts/common/runner.go
• shortcuts/common/runner_args_test.go

🚧 Files skipped from review as they are similar to previous changes (2)

• shortcuts/common/runner.go
• shortcuts/common/runner_args_test.go

---

📝 Walkthrough

Walkthrough

Added a positional-argument validator and wired it into Cobra command construction so subcommands reject positional args; includes unit tests covering single, multiple, and empty/nil argument cases.

Changes

Cohort / File(s)	Summary
Positional Args Validation shortcuts/common/runner.go	Added rejectPositionalArgs() returning a cobra.PositionalArgs validator that returns nil for no args or a formatted fmt.Errorf listing provided positional values and instructing to use flags. Integrated via Args: rejectPositionalArgs().
Validation Tests shortcuts/common/runner_args_test.go	Added three parallel tests: TestRejectPositionalArgs_WithArgs, TestRejectPositionalArgs_MultipleArgs, and TestRejectPositionalArgs_NoArgs verifying rejection messages for one and multiple args and acceptance for nil/empty slices.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested reviewers

• fangshuyu-768

Poem

I nibble code in morning light,
Positional berries—nope, not right.
Flags are tidy, neat, and sage,
Hop this change into the cage. 🐇✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 16.67% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'fix: reject positional arguments in shortcuts' accurately and concisely describes the main change—adding validation to reject positional arguments in shortcut commands.
Description check	✅ Passed	The description covers all required sections (Summary, Changes via bullet points, Test Plan with checkboxes, and Related Issues). It provides clear motivation, implementation details, and comprehensive testing evidence.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/reject-positional-args

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[greptile-apps]

Greptile Summary

This PR fixes silent swallowing of positional arguments in declarative shortcut commands by wiring a rejectPositionalArgs() validator into mountDeclarative. Previously, passing a bare value like lark-cli docs +search "hello" would silently produce empty results; now cobra rejects it with a clear usage message.

Key changes:

• shortcuts/common/runner.go: rejectPositionalArgs() is a plain func() cobra.PositionalArgs (no unused parameters) that formats the full args slice with %q, covering single and multiple positional arguments in the error message. Using a plain error (not ExitError) keeps cobra's default usage printing and avoids wrapping the message in a JSON envelope.
• shortcuts/common/runner_args_test.go: Three table-style tests cover the single-arg, multi-arg, and no-arg paths; all previous review concerns (unused *Shortcut parameter, first-arg-only message, missing multi-arg test) have been resolved.

Confidence Score: 5/5

Safe to merge — the change is minimal, well-tested, and all prior review concerns have been addressed.

No P0 or P1 findings remain. All three issues flagged in the previous review round (unused parameter, first-arg-only error message, missing multi-arg test) are fully resolved in this revision. The implementation is correct, the test coverage is solid, and the change has a very small blast radius (only declarative shortcut commands).

No files require special attention.

Important Files Changed

Filename	Overview
shortcuts/common/runner.go	Adds rejectPositionalArgs() validator to mountDeclarative and defines the helper; all previous review concerns (unused parameter, first-arg-only message, missing multi-arg test) are fully addressed.
shortcuts/common/runner_args_test.go	New test file covering single arg, multiple args, and no-args cases; TestRejectPositionalArgs_MultipleArgs was added to address the previous-round feedback.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["User runs: lark-cli docs +search hello"] --> B["cobra parses command"]
    B --> C{"Args validator\nrejectPositionalArgs()"}
    C -- "len(args) == 0" --> D["RunE: runShortcut()"]
    C -- "len(args) > 0" --> E["Return plain error\npositional arguments are not supported"]
    E --> F["cobra prints Usage + Error line\nno JSON envelope"]
    D --> G["API call succeeds"]

Loading

_{Reviews (2): Last reviewed commit: "fix: address PR review comments" | Re-trigger Greptile}

[github-actions]

🚀 PR Preview Install Guide

🧰 CLI update

npm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@1b6d962a43deb5741654ad4ef2c0aa1f6588a282

🧩 Skill update

npx skills add larksuite/cli#fix/reject-positional-args -y -g