cli: add code search via peregrine (ENT-993, ENT-994) by evisdren · Pull Request #1615 · entireio/cli

evisdren · 2026-07-02T19:58:18Z

https://entire.io/gh/entireio/cli/trails/739

Summary

Adds entire search --code <query> to search code content through peregrine via entire-api cells
New codesearch package with client for peregrine's /search/api/search endpoint
Gated behind ENTIRE_CODE_SEARCH=1 — returns a clear error otherwise, keeping it dark until launch
Auth via NewEntireAPICellClient (jurisdictional identity tokens), same pattern as entire api --to cell

Details

Output modes: grep-style repo:path:line: context (terminal) or structured JSON (--json / piped)
Flags: --code, --case-sensitive, --repo, --limit — incompatible flags (--page, --all-repos, --case-sensitive without --code) are rejected with clear errors
Response overflow detection: 8 MiB cap with explicit error instead of silent truncation
Timeout: 30s scoped to the search POST only, not auth resolution
V1 scope: Single-region (home cell). Multi-region fan-out is ENT-995 (future)

Linear tickets

ENT-993: CLI code search client package
ENT-994: Wire code search into search command UX
Parent: ENT-990: Cross-Region Code Search

Test plan

Unit tests for codesearch.Search() (success, API error, non-JSON error, response overflow)
Unit tests for env gate (ENTIRE_CODE_SEARCH on/off/falsy values)
Unit tests for flag validation (--code without gate, without query, --case-sensitive without --code, --all-repos with --code, --page with --code)
Unit tests for text and JSON output rendering (including empty results)
mise run check passes (fmt + lint + all tests)

🤖 Generated with Claude Code

Note

Medium Risk
Adds authenticated calls to a new cell API surface with jurisdictional routing; impact is limited by the ENTIRE_CODE_SEARCH=1 preview gate and bounded response handling.

Overview
Adds preview code content search to the hidden entire search command via --code, calling peregrine through entire-api at POST /search/api/search. A new codesearch package defines request/response types and Search(), including an 8 MiB response cap and structured api.HTTPError handling on failures.

The --code path is gated on ENTIRE_CODE_SEARCH=1; otherwise users get a clear “not yet available” error. It authenticates with NewEntireAPICellClient, applies a 30s timeout only around the search POST, and supports --repo / inline repo: (with repo:* / --all-repos treated as no repo filter), --limit, and --case-sensitive (rejected unless --code). Results render as grep-style repo:path:line: on a TTY or JSON when --json or stdout is piped.

Unit tests cover the client (success, API/non-JSON errors, oversized body), env gating, flag validation, and text/JSON output.

^{Reviewed by Cursor Bugbot for commit 55ceb6e. Configure here.}

Add `entire search --code <query>` to search code content through peregrine via entire-api cells. Gated behind ENTIRE_CODE_SEARCH=1 env var so it stays dark until launch. - New `codesearch` package: client for peregrine's /search/api/search endpoint with request/response types matching the peregrine API - Auth via NewEntireAPICellClient (jurisdictional identity tokens), same pattern as `entire api --to cell` - Output modes: grep-style text (terminal) or JSON (--json / piped) - Flags: --code, --case-sensitive, --repo, --limit - Rejects incompatible flags (--page, --all-repos, --case-sensitive without --code) with clear error messages - Response overflow detection (8 MiB cap with explicit error) - Timeout scoped to the search call only (30s), not auth resolution Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Copilot

Pull request overview

Adds an env-gated (ENTIRE_CODE_SEARCH=1) code-content search mode to the existing entire search command, backed by a new codesearch client that calls peregrine via the authenticated entire-api cell.

Changes:

Adds entire search --code <query> with --case-sensitive, --repo, and --limit, plus validation to reject incompatible checkpoint-search flags.
Introduces cmd/entire/cli/codesearch package to POST to /search/api/search with an 8 MiB response cap and typed request/response structs.
Adds unit tests covering the env gate, flag validation, output rendering, and codesearch.Search() behavior.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File	Description
cmd/entire/cli/search_cmd.go	Wires `--code` path into `entire search`, including env gate, flag validation, request mapping, and text/JSON rendering.
cmd/entire/cli/search_cmd_test.go	Adds tests for env gating, flag incompatibilities, and output helpers for the new code-search mode.
cmd/entire/cli/codesearch/codesearch.go	Implements peregrine code search client with response size limiting and error handling.
cmd/entire/cli/codesearch/codesearch_test.go	Adds unit tests for success/error/oversize response handling in the codesearch client.

+	if opts.repoFilter != "" {
+		if err := search.ValidateRepoFilters([]string{opts.repoFilter}); err != nil {
+			return fmt.Errorf("validating repo filter: %w", err)
+		}
+	}
+
+	req := codesearch.SearchRequest{
+		Query:         opts.query,
+		CaseSensitive: opts.caseSensitive,
+	}
+	if opts.limit > 0 {
+		req.MaxResults = opts.limit
+	}
+	if opts.repoFilter != "" {
+		req.Repos = []string{opts.repoFilter}
+	}


 	cmd.Flags().BoolVar(&jsonOutput, "json", false, "Output as JSON")
+	cmd.Flags().BoolVar(&codeFlag, "code", false, "Search code content via peregrine (requires ENTIRE_CODE_SEARCH=1)")
+	cmd.Flags().BoolVar(&caseSensitive, "case-sensitive", false, "Case-sensitive code search (only with --code)")
 	cmd.Flags().IntVar(&limitFlag, "limit", resultsPerPage, "Maximum number of results per page")


+	if resp.StatusCode != http.StatusOK {
+		var errResp struct {
+			Error string `json:"error"`
+		}
+		if json.Unmarshal(body, &errResp) == nil && errResp.Error != "" {
+			return nil, fmt.Errorf("code search error (%d): %s", resp.StatusCode, errResp.Error)
+		}
+		return nil, fmt.Errorf("code search returned %d: %s", resp.StatusCode, string(body))


- Parse inline repo: filters for --code path (was passing them as literal query text to peregrine) - Handle repo:* and --all-repos with --code as "no filter" instead of passing literal "*" to the API - Use api.ErrorResponse/HTTPError for error parsing instead of inline struct (supports both legacy and newer error envelopes) - Fix --limit help text to clarify behavior differs for --code - Drop --page/--all-repos rejection (silently ignored is less disruptive than breaking shared scripts) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

evisdren · 2026-07-02T20:33:39Z

bugbot run

cursor

Cursor Bugbot has reviewed your changes and found 2 potential issues.

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

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit 55ceb6e. Configure here.}

cursor · 2026-07-02T20:34:57Z

+					caseSensitive: caseSensitive,
+					jsonOutput:    jsonOutput,
+					insecureHTTP:  insecureHTTPAuth,
+				})


Page flag ignored for code

Medium Severity

The --code branch returns before checkpoint search runs and never checks --page, so entire search --code --page 2 … still returns the first result set with no error. Code search has no pagination, so the flag should be rejected like --case-sensitive without --code.

^{Reviewed by Cursor Bugbot for commit 55ceb6e. Configure here.}

cursor · 2026-07-02T20:34:57Z

+				codeRepo := repoFlag
+				if codeRepo == "" && len(parsed.Repos) > 0 {
+					codeRepo = parsed.Repos[0]
+				}


Multiple repo filters silently ignored

Medium Severity

For --code, inline repo: filters are parsed into parsed.Repos, but only parsed.Repos[0] is used and search.ValidateRepoFilters is never run on the full list. Queries like repo:a/b repo:c/d term or repo:a/b,c/d term search one repo without the error checkpoint search returns for multiple repo filters.

^{Reviewed by Cursor Bugbot for commit 55ceb6e. Configure here.}

evisdren · 2026-07-02T20:43:55Z

Superseded by #1616 (client package) and #1617 (command wiring) — split into two PRs per request.

evisdren requested a review from a team as a code owner July 2, 2026 19:58

Copilot AI review requested due to automatic review settings July 2, 2026 19:58

Copilot started reviewing on behalf of evisdren July 2, 2026 19:58 View session

cursor Bot reviewed Jul 2, 2026

View reviewed changes

Comment thread cmd/entire/cli/search_cmd.go Outdated

Comment thread cmd/entire/cli/search_cmd.go

Copilot AI reviewed Jul 2, 2026

View reviewed changes

cursor Bot reviewed Jul 2, 2026

View reviewed changes

evisdren closed this Jul 2, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

cli: add code search via peregrine (ENT-993, ENT-994)#1615

cli: add code search via peregrine (ENT-993, ENT-994)#1615
evisdren wants to merge 2 commits into
mainfrom
evis/ent-990-cross-region-code-search-cli-peregrine-us-first-then-eu

evisdren commented Jul 2, 2026 •

edited by cursor Bot

Loading

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

evisdren commented Jul 2, 2026

Uh oh!

cursor Bot left a comment

Uh oh!

cursor Bot Jul 2, 2026

Uh oh!

cursor Bot Jul 2, 2026

Uh oh!

evisdren commented Jul 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

evisdren commented Jul 2, 2026 • edited by cursor Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Details

Linear tickets

Test plan

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

evisdren commented Jul 2, 2026

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor Bot Jul 2, 2026

Choose a reason for hiding this comment

Page flag ignored for code

Uh oh!

cursor Bot Jul 2, 2026

Choose a reason for hiding this comment

Multiple repo filters silently ignored

Uh oh!

evisdren commented Jul 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

2 participants

evisdren commented Jul 2, 2026 •

edited by cursor Bot

Loading