openclaw - 💡(How to fix) Fix CLI search --limit accepts partially numeric values [3 pull requests]

Official PRs (…)
ON THIS PAGE

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…

Fix Action

Fixed

Code Example

$ pnpm openclaw skills search calendar --limit 1abc --json
{
  "results": [
    {
      "score": 4.39479889756764,
      "slug": "calendar",
      "displayName": "Calendar",
      "summary": "Calendar management and scheduling. Create events, manage meetings, and sync across calendar providers.",
      "version": null,
      "updatedAt": 1778491574637,
      "ownerHandle": "ndcccccc",
      "owner": {
        "handle": "ndcccccc",
        "displayName": "NDCCCCCC",
        "image": "https://avatars.githubusercontent.com/u/63100268?v=4"
      }
    }
  ]
}

$ pnpm openclaw plugins search calendar --limit 1abc --json
{
  "results": []
}
RAW_BUFFERClick to expand / collapse

Problem

openclaw plugins search and openclaw skills search parse --limit with Number.parseInt, so partially numeric values are accepted instead of rejected. That can turn typos such as 1abc into 1, or let non-positive values fall through to a default/remote behavior instead of telling the user the option is invalid.

Current behavior on upstream main

Checked on origin/main at 67c12e036802224e76547092938e3b9759846956.

$ pnpm openclaw skills search calendar --limit 1abc --json
{
  "results": [
    {
      "score": 4.39479889756764,
      "slug": "calendar",
      "displayName": "Calendar",
      "summary": "Calendar management and scheduling. Create events, manage meetings, and sync across calendar providers.",
      "version": null,
      "updatedAt": 1778491574637,
      "ownerHandle": "ndcccccc",
      "owner": {
        "handle": "ndcccccc",
        "displayName": "NDCCCCCC",
        "image": "https://avatars.githubusercontent.com/u/63100268?v=4"
      }
    }
  ]
}

$ pnpm openclaw plugins search calendar --limit 1abc --json
{
  "results": []
}

Both commands accept the invalid option and continue.

Expected behavior

--limit should require a whole positive integer and fail fast for values such as 1abc, 5.5, 0, or -1, before calling ClawHub.

Duplicate/collision checks

I searched current open/closed issues and open PRs for:

  • "plugins search" "--limit"
  • "skills search" "--limit"
  • "plugins search" "1abc"
  • "skills search" "1abc"
  • open PRs touching plugins-cli.ts or skills-cli.ts

No exact issue or open PR was found for this CLI search limit parser behavior.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

FAQ

Expected behavior

--limit should require a whole positive integer and fail fast for values such as 1abc, 5.5, 0, or -1, before calling ClawHub.

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING