Skip to content

LLP 0269: grep search on the client - implementation plan for LLP 0264 (LLP-only) - #864

Closed
bgmcmullen wants to merge 1 commit into
masterfrom
llp/0269-grep-search-client-plan
Closed

LLP 0269: grep search on the client - implementation plan for LLP 0264 (LLP-only)#864
bgmcmullen wants to merge 1 commit into
masterfrom
llp/0269-grep-search-client-plan

Conversation

@bgmcmullen

Copy link
Copy Markdown
Contributor

LLP-only PR, no code changes. Status: Accepted - this is the plan doc neutral derives implementable tasks from; the merged LLP 0264 decision alone produced none.

Phases

  • T1 hoist the server's shared search modules (allowlist, matcher, snippet, hit shapes) into src/core/search/, exported for the server's later import swap
  • T2hypgrep hard dep + overrides pinning its hyparquet to 1.28.2 (LLP 0222 floor - correctness, not hygiene)
  • T3 the two-tier grep service: brute-scan mutable files, hypgrep sidecars on compacted files, LLP 0105 visibility wrap, spool flush first; ships before any index exists (unindexed = slower, never wrong)
  • T4query grep verb (tool: grep_search, schema-compatible with the server so --remote works day one) + the VerbRegistry replace affordance the server's collision fix needs
  • T5 sidecar build at maintenance: worker thread, retry quarantine, sidecar-existence idempotency (server LLP 0136/0158/0128 patterns)
  • T6 SKILL.md updates + a query_grep_two_tier smoke

Sequencing (resolves LLP 0264 #open)

Client ships T1-T4 without waiting on the server. The server-side pair (import swap + grep_search register-and-replace) must merge in hypaware-server before that repo bumps its hypaware dependency past T4 - a bump without it silently swaps the archive-backed tool for the kernel's local one.

🤖 Generated with Claude Code

…0264
LLP-only, Accepted: the plan doc neutral derives tasks from, which the
0264 decision alone did not provide.
Six phases: hoist the server's shared search modules into core, hypgrep
dependency with the LLP 0222 hyparquet-floor override, the two-tier grep
service (correct-before-fast: ships unindexed), the query grep verb plus
the VerbRegistry replace affordance the server needs, sidecar builds at
maintenance on the server's worker/quarantine/existence patterns, docs
and a two-tier smoke. Resolves LLP 0264 #open by sequencing the server
pair against the server's next hypaware dep bump, not the client release.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@bgmcmullen

Copy link
Copy Markdown
ContributorAuthor

Closing as a duplicate: this and #865 were authored in parallel as the plan doc for LLP 0264, and #865 (LLP 0265, llp/0265-grep-search-implementation.plan.md) merged first. LLP 0265 covers everything here - the shared-module hoist, the hypgrep dep with the LLP 0222 overrides, the two-tier service with the LLP 0105 wrap, the wire-compatible query grep verb plus the registry removal affordance hypaware-server#364 guards on, the maintenance-time sidecar build on the server's worker/quarantine patterns, and the same #open-resolving sequencing (server #364 first, client release, server dependency bump last) - plus a verified-against-the-tree section and the LLP 0215-style wave/dependency structure neutral derives from. LLP 0269's number is freed by this close; derive tasks from LLP 0265.

Sign up for freeto 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

@bgmcmullen