Skip to content

Add /ask natural-language search page (frontend) - #204

Closed
skearnes wants to merge 1 commit into
mainfrom
nl-query-frontend
Closed

Add /ask natural-language search page (frontend)#204
skearnes wants to merge 1 commit into
mainfrom
nl-query-frontend

Conversation

@skearnes

Copy link
Copy Markdown
Member

Follow-up to #203 (the natural-language query backend). Adds the web UI for /api/nl_query.

  • A dedicated /ask page (MainNLSearch), separate from /search, with an "Interpreted as:" panel (roles, modes, resolved SMILES, resolver), example chips, and shareable ?q= URLs; reuses SearchResults.
  • useNLQuery hook — calls /api/nl_query and deserializes the result protos.
  • NL types in app/src/types/search.ts; /ask route in App.tsx; "Ask" nav link in HeaderNav.

Depends on #203 (the /api/nl_query endpoint). Draft until that merges; will rebase on main afterward.

🤖 Generated with Claude Code

The web UI for the natural-language query endpoint: a dedicated /ask page
separate from /search, with an "Interpreted as:" panel showing roles, modes,
resolved SMILES, and the resolver; example chips; and shareable ?q= URLs.
Reuses SearchResults for rendering.

- app/src/views/nl-search/MainNLSearch.{tsx,scss}
- app/src/hooks/useNLQuery.ts (calls /api/nl_query, deserializes result protos)
- types in app/src/types/search.ts; /ask route in App.tsx; "Ask" nav link

Depends on the backend endpoint from the natural-language query PR.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@skearnes

Copy link
Copy Markdown
Member Author

Folding the frontend back into #203 so the natural-language feature can be reviewed and iterated on as a single full-stack PR. The /ask page, useNLQuery hook, types, route, and nav link are now part of #203. Closing this split-out branch.

🤖 Generated with Claude Code

@skearnes skearnes closed this Jun 27, 2026
@skearnes
skearnes deleted the nl-query-frontend branch June 27, 2026 22:06
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