Skip to content

Add Search API CLI commands - #266

Open
chruffins wants to merge 5 commits into
mainfrom
hypeship/search-api-cli
Open

chruffins wants to merge 5 commits into
mainfrom
hypeship/search-api-cli

Conversation

@chruffins

@chruffins chruffins commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add kernel search [query], search get <id>, and search providers [--slug] using the existing authenticated SDK transport and project scope.
  • Support provider discovery, automatic or pinned routing, result limits, complete JSON requests from inline input/files/stdin, and preserved JSON responses.
  • Disable automatic create retries to avoid duplicate billable searches.
  • Add command help, README examples, and focused HTTP-level tests.

Scope and assumptions

  • The merged Search API contract is authoritative; deferred content retrieval remains unavailable and is not exposed.
  • Inline content and advanced strategies/options are passed through complete JSON requests with server-side validation.
  • The pinned SDK does not expose a Search service, so this uses its generic Execute helper without dependency changes.
  • JSON-only output preserves warnings, attempts, native fields, and large numbers. Search API access must be enabled for the organization.

Validation

  • go test ./cmd -run TestSearch -count=1
  • make test (go vet and full test suite)
  • make build
  • Search and provider command help smoke checks
  • git diff main...HEAD --check
  • No production API calls performed.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 00dc5a1. Configure here.

Comment thread cmd/search.go Outdated
@chruffins
chruffins marked this pull request as ready for review September 22, 2026 21:10
@chruffins
chruffins requested a review from rgarcia September 22, 2026 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant