Skip to content

Clear research page #387

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jerry609 merged 5 commits into from
Mar 13, 2026
Merged

Clear research page #387

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
0187fac
chore(research): snapshot before removing post-search component
Linjie-top Mar 13, 2026
7e9802d
refactor(research): remove post-search track context panel
Linjie-top Mar 13, 2026
3b4db94
chore(research): remove unused ResearchTrackContextPanel and test
Linjie-top Mar 13, 2026
5a1b01e
test(research): assert Track Snapshot panel removed and post-search U…
Linjie-top Mar 13, 2026
808ac57
test(research): wait for post-search state before asserting removed p…
Linjie-top Mar 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Empty file added web/.npm-cache/_update-notifier-last-checked
View file
Open in desktop
Empty file.
14 changes: 14 additions & 0 deletions web/e2e/research.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,4 +25,18 @@ test.describe("Research Page E2E", () => {
content?.includes("Topic");
expect(hasResearchContent).toBeTruthy();
});

test("post-search view renders without Track Snapshot panel", async ({ page }) => {
// Navigate with a query to trigger post-search UI state
await page.goto("/research?query=rag");

// 1) Wait for a stable post-search marker (auto-retrying)
await expect(page.getByRole("link", { name: "Open Workflows" })).toBeVisible();

// 2) Assert the old panel label is absent (auto-retrying)
await expect(page.locator("text=Track Snapshot")).toHaveCount(0);

// 3) Sanity: a summary badge/label remains visible (one is enough)
await expect(page.locator("text=Results:").first()).toBeVisible();
});
});
10 changes: 1 addition & 9 deletions web/src/components/research/ResearchPageNew.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,6 @@ import { SavedTab } from "./SavedTab"
import { CreateTrackModal } from "./CreateTrackModal"
import { EditTrackModal } from "./EditTrackModal"
import { ManageTracksModal } from "./ManageTracksModal"
import { ResearchTrackContextPanel } from "./ResearchTrackContextPanel"
import type { Track } from "./TrackSelector"
import type { Paper } from "./PaperCard"
import type { ResearchTrackContextResponse } from "@/lib/types"
Expand Down Expand Up @@ -638,14 +637,7 @@ export default function ResearchPageNew() {
/>
</div>

{trackContext ? (
<div className="mb-6 animate-in fade-in slide-in-from-bottom-2 duration-500">
<ResearchTrackContextPanel
context={trackContext}
onOpenMemory={() => setMemoryOpen(true)}
/>
</div>
) : null}
{/* Track context panel removed per request */}
Comment thread
Linjie-top marked this conversation as resolved.
Comment thread
Linjie-top marked this conversation as resolved.

{/* Track Pills - only show before search */}
{!hasSearched && tracks.length > 0 && (
Expand Down
76 changes: 0 additions & 76 deletions web/src/components/research/ResearchTrackContextPanel.test.tsx
View file
Open in desktop

This file was deleted.

213 changes: 0 additions & 213 deletions web/src/components/research/ResearchTrackContextPanel.tsx
View file
Open in desktop

This file was deleted.

Loading