Skip to content

Agentic UI: Add live pages and posts search to the preview omnibox - #4373

Open
bcotrim wants to merge 27 commits into
trunkfrom
stu-2012-add-preview-site-search
Open

Agentic UI: Add live pages and posts search to the preview omnibox#4373
bcotrim wants to merge 27 commits into
trunkfrom
stu-2012-add-preview-site-search

Conversation

@bcotrim

@bcotrim bcotrim commented Jul 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Related issues

How AI was used in this PR

Claude Code extracted this slice from the exploration branch and ran lint/typecheck/tests.

Proposed Changes

  • Typing words in the omnibox now shows live results from the site's pages and posts, with Searching… / No matches / Search unavailable states; clicking a result navigates the preview.
  • The zero state now shows real front-end permalinks (latest post + a page) instead of guessed paths.
  • Backed by a new authenticated site REST proxy in the main process: path-only GETs, rejects requests escaping /wp-json/ (the SSRF guard has a dedicated test).

Testing Instructions

  • Full app restart (main-process changes).
  • Open a running site's preview, type words in the address bar → live results appear; click one → preview navigates there.
  • Only published content appears (drafts don't).
  • Verify in light + dark.

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

🤖 Generated with Claude Code
bcotrim added 2 commits July 28, 2026 16:32
@bcotrim
Merge remote-tracking branch 'origin/stu-2012-add-site-preview-toolba…
164dab9 
…r-tabs' into stu-2012-add-preview-omnibox

# Conflicts:
#	apps/ui/src/components/site-preview/address-bar.tsx
@bcotrim
Leave the site-search Enter fallback to the search PR
a93c4f0
@bcotrim bcotrim mentioned this pull request Jul 28, 2026
Merged
1 task
bcotrim added 12 commits July 28, 2026 17:33
@bcotrim
Replace the omnibox popup with a destinations menu until search lands
b182c96
@bcotrim
Remove WP Admin and Database rows from the destinations menu
1fabe51
@bcotrim
Merge remote-tracking branch 'origin/stu-2012-add-preview-omnibox' into
3712b2a 
stu-2012-add-preview-site-search

# Conflicts:
#	apps/ui/src/components/site-preview/address-bar.test.tsx
#	apps/ui/src/components/site-preview/address-bar.tsx
#	apps/ui/src/components/site-preview/index.tsx
@bcotrim
Merge remote-tracking branch 'origin/trunk' into stu-2012-add-preview…
511a3cc 
…-omnibox

# Conflicts:
#	apps/ui/src/components/site-preview/address-bar.module.css
#	apps/ui/src/components/site-preview/address-bar.test.tsx
#	apps/ui/src/components/site-preview/address-bar.tsx
#	apps/ui/src/components/site-preview/index.tsx
@bcotrim
Mark the current destination in the address bar menu
d304c93
@bcotrim
Highlight the current destination and match rewritten WP Admin URLs
8b12abd
@bcotrim
Match renamed site-editor route slugs when marking the current destin…
1fa3c42 
…ation
@bcotrim
Merge remote-tracking branch 'origin/stu-2012-add-preview-omnibox' into
6478d45 
stu-2012-add-preview-site-search

# Conflicts:
#	apps/ui/src/components/site-preview/address-bar.test.tsx
#	apps/ui/src/components/site-preview/address-bar.tsx
#	apps/ui/src/components/site-preview/index.tsx
#	apps/ui/src/data/core/types.ts
@bcotrim
Keep the omnibox results list mounted so typing doesn't lose focus
4dca7f2
@bcotrim
Return a 502 from the site REST proxy when the site is unreachable
6dfa8b3
@bcotrim
Dial site REST requests via localhost so IPv6-bound site servers connect
fc560e1
@bcotrim
Clarify the match-score dummy base and drop the localhost comment
d890d56
Base automatically changed from stu-2012-add-preview-omnibox to trunk July 31, 2026 09:48
bcotrim added 4 commits July 31, 2026 10:53
@bcotrim
Merge remote-tracking branch 'origin/trunk' into stu-2012-add-preview…
102d508 
…-site-search

# Conflicts:
#	apps/ui/src/components/site-preview/address-bar.module.css
#	apps/ui/src/components/site-preview/address-bar.test.tsx
#	apps/ui/src/components/site-preview/address-bar.tsx
@bcotrim
Dedupe omnibox results that appear as both permalink rows and search …
ed5a557 
…matches
@bcotrim
Skip the site REST nonce request when auto-login returns no login cookie
b9c9761
@bcotrim
Warn once per site when REST auth prep keeps failing
246fa4a
@bcotrim
bcotrim requested a review from sejas July 31, 2026 10:45
@bcotrim
bcotrim requested a review from nightnei July 31, 2026 10:45
@bcotrim
bcotrim marked this pull request as ready for review July 31, 2026 10:45
@wpmobilebot

wpmobilebot commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing c7ae680 vs trunk

app-size

Metric trunk c7ae680 Diff Change
App Size (Mac) 1367.52 MB 1367.60 MB +0.08 MB ⚪ 0.0%

site-editor

Metric trunk c7ae680 Diff Change
load 1040 ms 1067 ms +27 ms ⚪ 0.0%

site-startup

Metric trunk c7ae680 Diff Change
siteCreation 6532 ms 6499 ms 33 ms ⚪ 0.0%
siteStartup 2364 ms 2378 ms +14 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)
@nightnei

nightnei commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Path search feels broken:

  1. It works only when it matches fully
    Screenshot 2026-07-31 at 12 27 14
  2. But, when I fill in the full path - I see the default state - all links that are available by default:
    Screenshot 2026-07-31 at 12 27 46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@bcotrim @wpmobilebot @nightnei