Skip to content

Add experimental Harper-backed Fulltext runtime - #30

Merged
kylebernhardy merged 6 commits into
mainfrom
codex/harper-vertical-slice
Sep 10, 2026
Merged

Add experimental Harper-backed Fulltext runtime#30
kylebernhardy merged 6 commits into
mainfrom
codex/harper-vertical-slice

Conversation

@kylebernhardy

Copy link
Copy Markdown
Member

Summary

  • add an experimental @harperfast/fulltext/harper entry point that runs Tantivy through a synchronous Harper-owned key-value view
  • persist the derived-index cursor in the same Tantivy commit generation and reload the reader before publish() resolves
  • enforce owner-worker writer exclusivity, bounded transport admission, callback fencing, and safe worker teardown
  • preserve the native Tantivy entry point while keeping RocksDB and rocksdb-js out of this package
  • document ambiguous publish outcomes by making committedPayload unavailable until reopen

This is the narrow Fulltext-side runtime slice for Phase 0: prove the Harper-backed Tantivy storage and derived-index path #7. It intentionally does not add Harper schema declarations, Table.search, non-owner readers, generation retirement, or release enablement.

Verification

  • npm run check
  • cargo clippy --locked --all-targets --features node-api -- -D warnings
  • cargo clippy --locked --all-targets --features host-storage -- -D warnings
  • local plan review by three independent reviewers
  • committed implementation review by Claude and direct Gemini CLI review; relevant findings resolved

Advances #7

Comment generated by kAIle (GPT-5)

@kylebernhardy
kylebernhardy merged commit 7085d39 into main Sep 10, 2026
6 checks passed
@kylebernhardy
kylebernhardy deleted the codex/harper-vertical-slice branch September 10, 2026 22:51
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