### Overview

When running `entire enable`, it adds a search skill. Investigate if that's correct.

3w ago · 6.7s · 74 calls

### Investigation Results

* `entire enable` was installing a managed "search subagent" under:
  * `.claude/agents/entire-search.md`
  * `.codex/agents/entire-search.toml`
  * `.gemini/agents/entire-search.md`
* This was wired directly from `setupAgentHooks(...)`, so adding/enabling an agent also scaffolded the search agent.
* The text you pasted (`A trail ties together...`) is separate: it’s the trail context injection from `cmd/entire/cli/lifecycle.go`, not a search skill/subagent.

### Changes Made

* Removed search subagent scaffolding from enable/agent hook setup.
* Deleted the managed search subagent templates and tests.
* Updated integration tests to only assert hook installation.
* Removed Gemini E2E acknowledgment setup for the deleted search agent.
* Added a changelog entry.

### Verification

* `go test ./cmd/entire/cli ./e2e/testutil`
* `go test -tags=integration ./cmd/entire/cli/integration_test -run 'TestSetup(Claude|Codex|Gemini)Hooks_AddsAllRequiredHooks'`
* Manual temp-repo smoke test: `entire enable --agent codex` no longer creates `.codex/agents/entire-search.toml`.

### Summary of Changes

* Removed search skill installation from default `entire enable` / agent setup.
* Added opt-in flag:
  * `entire enable --agent claude-code --search-skill`
  * `entire agent add claude-code --search-skill`
* Kept managed search skill scaffolding available behind that flag.
* Removed checked-in generated `entire-search` files from `.claude`, `.codex`, and `.gemini`.
* Updated tests/docs/changelog.

### Further Verification

* `go test ./cmd/entire/cli ./e2e/testutil`
* `go test -tags=integration ./cmd/entire/cli/integration_test -run 'TestSetupCodexHooks_(AddsAllRequiredHooks|SearchSkillOptIn)' -count=1`

### Next Steps

Create a trail and commit push the changes to the new branch.

3w ago · 11.3s · 11 calls

Branch: `dip/search-skill-opt-in`
Trail: [Trail Link](/content/gh/entireio/cli/trails/662/index.html)
Commit: `31c66680d` — `Make Entire search skill opt-in`
Pushed to: `origin/dip/search-skill-opt-in`

### Verification Run

* `mise run lint`
* `go test ./cmd/entire/cli ./e2e/testutil`
* Integration opt-in tests for Codex search skill.

### PR Updates

Add a PR body explaining the changes positively [PR Link](https://github.com/entireio/cli/pull/1521/)

3w ago · 6.9s · 3 calls

Updated PR #1521 with a positive summary, rationale, and validation section.

### Removal of Unnecessary Changes

* Removed the `CHANGELOG.md` entry, committed, and pushed.
Commit: `85082f382` — `Remove changelog entry`

* Removed the `README.md` change, committed, and pushed.
Commit: `609eb860d` — `Remove README search skill docs`
Also ran `mise run lint` before pushing; it passed.