[codex] Add semantic search Roam API types#61
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@codex review |
|
Picked this up from the earlier |
Summary
Adds TypeScript coverage for the new Roam semantic search API surface:
SemanticSearchArgs,SemanticSearchHit, andSemanticSearchwindow.roamAlphaAPI.data.semanticSearchEnabled()window.roamAlphaAPI.data.async.semanticSearch(...)No mock environment defaults are included.
Impact
Extension authors can now call the new async semantic search API with typed arguments and hit records. Callers can use
semanticSearchEnabled()before invoking the async API, matching Roam's availability guard contract.Validation
npm run buildpassesnpm run lintpasses with one existing warning insrc/types/index.tsnpm testwas previously blocked during test loading by existing missing modules:../src/backend/getRoamJSUserandreact-dom/client