openclaw/clawhub
1
0
Fork 0
mirror of https://github.com/openclaw/clawhub.git synced 2026-07-21 19:25:25 -04:00
Code Issues 183 Packages Projects Releases 1 Wiki Activity

[PR #36] [CLOSED] fix: make /search mode-aware #223

New Issue
Closed
opened 2026-02-15 17:16:22 -05:00 by yindo · 0 comments
yindo commented 2026-02-15 17:16:22 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/openclaw/clawhub/pull/36
Author: @zats
Created: 1/25/2026
Status: ❌ Closed

Base: main ← Head: fix/search-route-mode-aware

📝 Commits (1)

  • 34583a6 fix: make /search mode-aware

📊 Changes

2 files changed (+35 additions, -10 deletions)

View changed files

📝 src/components/Header.tsx (+2 -2)
📝 src/routes/search.tsx (+33 -8)

📄 Description

Summary

  • make /search mode-aware with a default redirect
  • point header Search link to /search

Rationale

  • Standardizing on /search gives us stable, shareable URLs across deployments. It lets us map search to the right mode today while keeping a single bookmarkable entry point that we can rewire later without breaking links.

Heads-up

  • Quick check: do our SSR deploy envs always set the intended mode (e.g., via VITE_SITE_MODE)? If not, /search could resolve to the wrong mode when VITE_SOULHUB_SITE_URL is set. src/routes/search.tsx:9-31

Testing

  • bun run lint (pass)
  • bun run test (fail: src/lib/site.test.ts; src/tests/search-route.test.ts; src/tests/upload.route.test.tsx)
  • bun run test:e2e (fail: 3 tests in e2e/clawdhub.e2e.test.ts)
  • bun run coverage (fail: same as bun run test)
  • bun run test:pw (fail: preview server couldn’t start; build required)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/openclaw/clawhub/pull/36 **Author:** [@zats](https://github.com/zats) **Created:** 1/25/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/search-route-mode-aware` --- ### 📝 Commits (1) - [`34583a6`](https://github.com/openclaw/clawhub/commit/34583a640c984add8a898f4b54daabb2f630d1ee) fix: make /search mode-aware ### 📊 Changes **2 files changed** (+35 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `src/components/Header.tsx` (+2 -2) 📝 `src/routes/search.tsx` (+33 -8) </details> ### 📄 Description ## Summary - make /search mode-aware with a default redirect - point header Search link to /search ## Rationale - Standardizing on `/search` gives us stable, shareable URLs across deployments. It lets us map search to the right mode today while keeping a single bookmarkable entry point that we can rewire later without breaking links. ## Heads-up - Quick check: do our SSR deploy envs always set the intended mode (e.g., via VITE_SITE_MODE)? If not, `/search` could resolve to the wrong mode when VITE_SOULHUB_SITE_URL is set. src/routes/search.tsx:9-31 ## Testing - bun run lint (pass) - bun run test (fail: src/lib/site.test.ts; src/__tests__/search-route.test.ts; src/__tests__/upload.route.test.tsx) - bun run test:e2e (fail: 3 tests in e2e/clawdhub.e2e.test.ts) - bun run coverage (fail: same as bun run test) - bun run test:pw (fail: preview server couldn’t start; build required) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 17:16:22 -05:00
yindo closed this issue 2026-02-15 17:16:22 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: openclaw/clawhub#223
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?