+ {/* The verb shortcut row carried over from the retired `/?mode=tools` hub.
+ Sourced from `accessibleTools`, not `filteredTools`: the shortcuts are a
+ fixed way in, so a query or category filter must not empty the row. Hidden
+ once a query is running, where the ranked results are the answer and a
+ static row above them is just noise. */}
+
+ {query.trim() ? null : (
+
+ )}
;
/**
@@ -17,6 +36,19 @@ export function legacyHomeRedirectUrl(requestUrl: LegacyHomeRequestUrl, method:
if ((method !== "GET" && method !== "HEAD") || requestUrl.pathname !== "/") return null;
const mode = requestUrl.searchParams.get("mode");
+
+ const aliasOnlyPath = mode ? aliasOnlyModePaths[mode as keyof typeof aliasOnlyModePaths] : undefined;
+ if (aliasOnlyPath) {
+ const aliasDestination = new URL(requestUrl.toString());
+ aliasDestination.pathname = aliasOnlyPath;
+ aliasDestination.hash = "";
+ // `mode` is consumed by the destination pathname; everything else (a query, a
+ // submitted `run=1`, navigation context) rides along, because `/tools` reads the
+ // same query string the alias carried.
+ aliasDestination.searchParams.delete("mode");
+ return aliasDestination;
+ }
+
const destinationPath = mode ? legacyModePaths[mode as keyof typeof legacyModePaths] : undefined;
if (!destinationPath) return null;
diff --git a/src/lib/tools-catalog.ts b/src/lib/tools-catalog.ts
index bd7b5e90d9..1bac15ffad 100644
--- a/src/lib/tools-catalog.ts
+++ b/src/lib/tools-catalog.ts
@@ -59,6 +59,17 @@ export type ToolCatalogRecord = {
output: string;
};
+/**
+ * Tools the composer's smart search already answers directly, so surfacing them again
+ * as local results or shortcuts would send a clinician the long way round to something
+ * they have just been handed.
+ *
+ * One definition, three consumers. It was copied into the launcher and the tools
+ * directory separately, and a third copy was nearly added when the quick-action row
+ * moved out of the launcher.
+ */
+export const localSmartExcludedToolIds = new Set(["clinical-kb-search", "documents", "favourites"]);
+
export const toolCatalogRecords: ToolCatalogRecord[] = [
{
id: "clinical-kb-search",
diff --git a/tests/audit-navigation-auth-regressions.test.ts b/tests/audit-navigation-auth-regressions.test.ts
index 0d1026b78d..36e705fc83 100644
--- a/tests/audit-navigation-auth-regressions.test.ts
+++ b/tests/audit-navigation-auth-regressions.test.ts
@@ -120,6 +120,25 @@ describe("audit navigation and auth regressions", () => {
expect(source("src/proxy.ts")).toContain("legacyHomeRedirectUrl(request.nextUrl, request.method)");
});
+ // Tools is the one alias that forwards whether or not the URL is submitted. It has no
+ // shared home to fall back to (`shouldShowSharedHome` excludes `tools`), so `/?mode=tools`
+ // used to render a second, hub-shaped launcher and Tools had two surfaces depending on how
+ // the clinician arrived. The hub's verb shortcut row moved to `/tools`, so the alias has
+ // nothing of its own left to show.
+ it("forwards the tools alias to the canonical directory whether or not it is submitted", () => {
+ expect(legacyHomeRedirectUrl(new URL("https://clinical-kb.test/?mode=tools"), "GET")?.toString()).toBe(
+ "https://clinical-kb.test/tools",
+ );
+ expect(
+ legacyHomeRedirectUrl(
+ new URL("https://clinical-kb.test/?mode=tools&q=medications&focus=1&run=1#detail"),
+ "GET",
+ )?.toString(),
+ ).toBe("https://clinical-kb.test/tools?q=medications&focus=1&run=1");
+ // A non-navigation method still falls through, same as every other alias.
+ expect(legacyHomeRedirectUrl(new URL("https://clinical-kb.test/?mode=tools"), "POST")).toBeNull();
+ });
+
// This redirect used to rebuild the destination from scratch (`destination.search = ""`,
// then only q/focus/run re-added), so `queryMode` and the scope filters were already gone
// one hop before `consolidatedModeHomeTarget` — whose own doc promises "every other query
diff --git a/tests/privacy-readiness-contract.test.ts b/tests/privacy-readiness-contract.test.ts
index cdb7900116..bc5c0fdeef 100644
--- a/tests/privacy-readiness-contract.test.ts
+++ b/tests/privacy-readiness-contract.test.ts
@@ -1,3 +1,4 @@
+import { execFileSync } from "node:child_process";
import { readFileSync } from "node:fs";
import { describe, expect, it } from "vitest";
import { validatePrivacyReadiness } from "../scripts/check-privacy-readiness.mjs";
@@ -11,9 +12,50 @@ const retentionParityMigration = readFileSync(
"utf8",
);
+/*
+ * Web-container sessions start on a shallow clone, so `reviewedCommit` is often
+ * absent from local history. Without this the validator reports
+ * `reviewedCommit does not exist: `, which reads as a governance breach
+ * rather than a missing object, and every cloud session sees a spurious
+ * regression on an untouched file. Mirrors the guards already carried by
+ * tests/clinical-hazard-controls.test.ts and tests/rag-plan-package-parity.test.ts
+ * (`#1M0J6D`); this suite was the third with the same failure and the only one
+ * left without the guard.
+ */
+function isShallowClone(): boolean {
+ try {
+ return (
+ execFileSync("git", ["rev-parse", "--is-shallow-repository"], {
+ encoding: "utf8",
+ stdio: ["ignore", "pipe", "ignore"],
+ }).trim() === "true"
+ );
+ } catch {
+ return false;
+ }
+}
+
+function isCommitAvailable(commit: string): boolean {
+ try {
+ execFileSync("git", ["cat-file", "-e", `${commit}^{commit}`], {
+ stdio: ["ignore", "ignore", "ignore"],
+ });
+ return true;
+ } catch {
+ return false;
+ }
+}
+
describe("privacy readiness contract", () => {
it("accepts the honest structural register", () => {
- expect(validatePrivacyReadiness(manifest)).toEqual([]);
+ let checkGit = true;
+ if (!isCommitAvailable(manifest.reviewedCommit) && isShallowClone()) {
+ console.warn(
+ `PRIVACY_READINESS_SHALLOW_CLONE: reviewedCommit ${manifest.reviewedCommit} is unavailable in shallow clone; skipping commit ancestry check.`,
+ );
+ checkGit = false;
+ }
+ expect(validatePrivacyReadiness(manifest, { checkGit })).toEqual([]);
});
it("keeps Railway processor evidence linked to the privacy impact assessment", () => {
diff --git a/tests/ui-smoke.spec.ts b/tests/ui-smoke.spec.ts
index b44074d60c..211349d9d0 100644
--- a/tests/ui-smoke.spec.ts
+++ b/tests/ui-smoke.spec.ts
@@ -1525,6 +1525,28 @@ test.describe("PsychSift UI smoke coverage", () => {
expect(csp).toContain("https://*.supabase.co");
});
+ /*
+ * The Documents half of the same defect. `/documents` redirects to the shared
+ * home, and `documents/page.tsx` records the idle Documents view as deliberately
+ * retired — but clearing the composer on a submitted search left `run=1` in the
+ * URL, so the shared home stayed suppressed and DocumentSearchResultsPanel fell
+ * back to that retired "Start here" home (`document-search-empty-state`).
+ */
+ test("clearing a documents search returns the shared home, never the retired Start here view", async ({ page }) => {
+ await page.setViewportSize({ width: 1280, height: 900 });
+ await mockDemoApi(page);
+ await gotoApp(page, "/documents/search?q=lithium+monitoring&run=1");
+
+ await page
+ .getByRole("button", { name: /clear search question|clear search/i })
+ .first()
+ .click();
+
+ await expect(page).toHaveURL(/\/\?mode=documents&focus=1$/);
+ await expect(page.getByTestId("shared-home-empty-state")).toBeVisible();
+ await expect(page.getByTestId("document-search-empty-state")).toHaveCount(0);
+ });
+
test("static agent guidance is available and documents mode avoids the app error boundary", async ({ page }) => {
const llms = await page.request.get("/llms.txt");
expect(llms.status()).toBe(200);
@@ -5106,28 +5128,27 @@ test.describe("PsychSift UI smoke coverage", () => {
expect(requestCounts.quality).toBe(0);
});
- test("tools mode searches the existing applications registry inside the dashboard", async ({ page }) => {
+ // The legacy `/?mode=tools` URL used to render a second, hub-shaped launcher. It now
+ // redirects to `/tools`, so the query it carries has to survive the hop and land on
+ // the one tools directory with the same registry search behind it.
+ test("the legacy tools URL carries its query to the tools directory", async ({ page }) => {
await page.setViewportSize({ width: 1280, height: 900 });
await mockPrivateUnauthenticatedApi(page);
await gotoApp(page, "/?mode=tools&q=medications&focus=1&run=1");
+ await page.waitForURL(/\/tools\?/);
await expect(page.getByRole("button", { name: "Mode Tools" })).toBeVisible();
- await expect(page.locator('input[placeholder="Search tools..."]:visible').first()).toHaveValue("medications");
- await expect(page.getByTestId("tools-hub")).toBeVisible();
- const queryRibbon = page.getByTestId("tools-hub").getByTestId("search-query-ribbon");
+ const results = page.getByTestId("tools-search-results-page");
+ await expect(results).toBeVisible();
+ const queryRibbon = results.getByTestId("search-query-ribbon");
await expect(queryRibbon.getByRole("heading", { name: "medications" })).toBeVisible();
await expect(queryRibbon.getByRole("group", { name: "Filter tools by category" })).toBeVisible();
- await expect(page.getByTestId("tools-hub").getByTestId("application-row-medication-prescribing")).toContainText(
- "Medication Prescribing",
- );
- await expect(page.getByTestId("tools-hub").getByText("Selected tool")).toHaveCount(0);
- const detailsButton = page
- .getByTestId("tools-hub")
- .getByRole("button", { name: "View details for Medication Prescribing" });
- await expect(detailsButton).toHaveAttribute("aria-haspopup", "dialog");
- await detailsButton.click();
+ await expect(results.getByRole("heading", { level: 2, name: "Medication Prescribing" }).first()).toBeVisible();
+ // The verb shortcut row is for an unqueried catalogue, so a running query hides it.
+ await expect(page.getByTestId("tools-shortcuts")).toHaveCount(0);
+ await results.getByRole("button", { name: "View details for Medication Prescribing" }).click();
await expect(
- page.getByRole("dialog", { name: "Medication Prescribing" }).locator('a[href="/medications"]').first(),
+ results.getByRole("complementary", { name: "Medication Prescribing" }).locator('a[href="/medications"]').first(),
).toBeVisible();
await expectNoPageHorizontalOverflow(page);
});
diff --git a/tests/ui-tools-show-all.spec.ts b/tests/ui-tools-show-all.spec.ts
index dd78924cfa..0a75363300 100644
--- a/tests/ui-tools-show-all.spec.ts
+++ b/tests/ui-tools-show-all.spec.ts
@@ -6,26 +6,23 @@ async function gotoLauncher(page: Page, path = "/tools") {
await expect(page.locator("#main-content").first()).toBeVisible({ timeout: 15_000 });
}
-test("Show all opens the unfiltered tools search mode", async ({ page }) => {
+/**
+ * Tools used to have two surfaces: a hub at `/?mode=tools` whose "Show all" chip
+ * linked across to the directory at `/tools`. Which one a clinician landed on
+ * depended only on how they arrived. `/?mode=tools` now redirects, so the chip has
+ * nothing left to link to and the assertion worth keeping is that the alias lands on
+ * the one directory with its verb shortcuts intact.
+ */
+test("the legacy tools alias lands on the single tools directory", async ({ page }) => {
await page.setViewportSize({ width: 390, height: 844 });
await gotoLauncher(page, "/?mode=tools");
- const showAll = page.getByRole("link", { name: "Show all tools" });
- await expect(showAll).toBeVisible();
- await expect(showAll).toHaveText("Show all");
- await expect(showAll).toHaveAttribute("href", "/tools");
-
- const showAllBox = await showAll.boundingBox();
- expect(showAllBox?.height).toBeGreaterThanOrEqual(48);
-
- await showAll.focus();
- await expect(showAll).toBeFocused();
-
- await Promise.all([page.waitForURL(/\/tools$/), page.keyboard.press("Enter")]);
+ await page.waitForURL(/\/tools$/);
const results = page.getByTestId("tools-search-results-page");
await expect(results).toBeVisible();
await expect(results.getByRole("heading", { level: 1, name: "All tools" })).toBeVisible();
+ await expect(page.getByTestId("tools-shortcuts")).toBeVisible();
await expect(results.getByRole("link", { name: "Open PsychSift Search" })).toHaveAttribute("href", "/?mode=answer");
await expect(results.getByRole("button", { name: "View details for PsychSift Search" })).toBeVisible();
await expect(page.getByTestId("global-search-input")).toHaveCount(0);
diff --git a/tests/ui-tools.spec.ts b/tests/ui-tools.spec.ts
index 22ceaa8543..d6df76f2cf 100644
--- a/tests/ui-tools.spec.ts
+++ b/tests/ui-tools.spec.ts
@@ -475,42 +475,47 @@ test.describe("PsychSift tools directory and legacy launcher", () => {
{ name: "mobile", width: 390, height: 820 },
{ name: "desktop", width: 1280, height: 900 },
] as const) {
- test(`tools launcher is usable at ${viewport.name}`, async ({ page }) => {
+ /**
+ * This case used to load `/?mode=tools`, the hub that was one of two Tools
+ * surfaces. That alias now redirects to the directory, so the same contract is
+ * asserted here on the single surface: the verb shortcut row ported over from
+ * the hub, the category filter, the detail sheet with a real launch link, and no
+ * shared search chrome on a route that owns its own filtering.
+ */
+ test(`the tools directory is usable at ${viewport.name}`, async ({ page }) => {
await page.setViewportSize({ width: viewport.width, height: viewport.height });
- await gotoLauncher(page, "/?mode=tools");
+ await gotoLauncher(page, "/tools");
- await expect(page.getByRole("heading", { level: 1, name: "Tools" })).toBeVisible();
- await expect(page.getByRole("region", { name: "Quick tool shortcuts" })).toBeVisible();
- await expect(page.getByRole("heading", { name: "All tools" })).toBeVisible();
- await expect(page.locator("#launcher-results-panel")).toHaveAttribute("role", "group");
- await expect(page.locator("#launcher-results-panel")).toHaveAttribute("aria-label", "All tools");
+ const results = visibleByTestId(page, "tools-search-results-page");
+ await expect(results.getByRole("heading", { level: 1, name: "All tools" })).toBeVisible();
+ await expect(page.getByRole("region", { name: "Quick tool shortcuts" }).first()).toBeVisible();
+ await expect(results.getByRole("region", { name: "Tool results" })).toBeVisible();
if (viewport.name === "mobile") {
- const categoryTrigger = page.getByTestId("tool-filter-trigger-phone");
+ const categoryTrigger = results.getByTestId("tools-search-filter-trigger-phone");
await expect(categoryTrigger).toBeVisible();
- await expect(categoryTrigger).toHaveAccessibleName(/No filters active/);
await categoryTrigger.click();
- await page.getByRole("radiogroup", { name: "Category" }).getByRole("radio", { name: "Assess" }).click();
- await expect(page.locator("#launcher-results-panel")).toHaveAttribute("aria-label", "Assess tools");
- await expect(categoryTrigger).toHaveAccessibleName(/1 filter active/);
+ const filterSheet = page.locator('[data-testid="tools-search-filter-sheet"]:visible');
+ await filterSheet.getByRole("radio", { name: /Assess/ }).click();
+ await filterSheet.getByTestId("tools-search-filter-sheet-done").click();
+ await expect(results.getByRole("heading", { level: 2, name: "Medication Prescribing" })).toHaveCount(0);
await categoryTrigger.click();
- await page.getByRole("radiogroup", { name: "Category" }).getByRole("radio", { name: "All tools" }).click();
- await page.getByTestId("application-row-medication-prescribing").click();
- const selectedSheet = page.getByRole("dialog", { name: "Medication Prescribing" });
- await expect(selectedSheet).toBeVisible();
- await expect(selectedSheet.getByRole("heading", { name: "Medication Prescribing" })).toBeVisible();
- const mobileLaunchLink = selectedSheet.locator('a[href="/medications"]').first();
+ await filterSheet.getByRole("radio", { name: /All tools/ }).click();
+ await filterSheet.getByTestId("tools-search-filter-sheet-done").click();
+
+ await results.getByRole("button", { name: "View details for Medication Prescribing" }).click();
+ const detailSheet = page.locator('[data-testid="tools-search-detail-sheet"]:visible');
+ await expect(detailSheet.getByRole("heading", { name: "Medication Prescribing" })).toBeVisible();
+ const mobileLaunchLink = detailSheet.locator('a[href="/medications"]').first();
await expect(mobileLaunchLink).toBeVisible();
- await expect(mobileLaunchLink).toHaveAttribute("href", "/medications");
await expect(mobileLaunchLink).not.toHaveAttribute("target", "_blank");
- await page.getByRole("button", { name: "Close Medication Prescribing" }).click();
- await expect(selectedSheet).toBeHidden();
+ await detailSheet.getByRole("button", { name: "Close Medication Prescribing" }).click();
+ await expect(detailSheet).toHaveCount(0);
} else {
- await expect(page.getByRole("button", { name: "View details for PsychSift Search" })).toBeVisible();
+ await expect(results.getByRole("button", { name: "View details for PsychSift Search" })).toBeVisible();
}
- await expect(page.getByLabel("Mode Tools")).toBeVisible();
+ await expect(page.getByRole("button", { name: "Mode Tools" })).toBeVisible();
await expect(visibleGlobalSearchInput(page)).toHaveCount(0);
await expect(page.locator("form.answer-footer-search-dock")).toHaveCount(0);
- await expect(page.getByTestId("tools-local-search-input")).toBeVisible();
await expectNoPageHorizontalOverflow(page);
});
}
@@ -531,7 +536,10 @@ test.describe("PsychSift tools directory and legacy launcher", () => {
);
await expect(visibleGlobalSearchInput(page)).toHaveCount(0);
await expect(page.locator("form.answer-footer-search-dock")).toHaveCount(0);
- await expect(page.getByTestId("tools-local-search-input")).toHaveCount(0);
+ // The route owns its filtering, so it has its own in-flow box and no shared
+ // composer. This box came here from the retired `/?mode=tools` hub, which was
+ // the only place a tools search could be typed until that alias redirected.
+ await expect(page.getByTestId("tools-local-search-input")).toBeVisible();
const categories = results.getByRole("radiogroup", { name: "Tool category" });
await categories.getByRole("radio", { name: /Treat/ }).click();
@@ -668,10 +676,14 @@ test.describe("PsychSift tools directory and legacy launcher", () => {
await expectNoPageHorizontalOverflow(page);
});
- test("launcher links point to the expected in-app modes", async ({ page }) => {
+ // Moved off `/?mode=tools` when that alias started redirecting here. The hrefs are
+ // the point of the case and are unchanged: both surfaces read them from the same
+ // tools catalogue, so the directory proves the same contract the hub used to.
+ test("tool links point to the expected in-app modes", async ({ page }) => {
await page.setViewportSize({ width: 1280, height: 900 });
- await gotoLauncher(page, "/?mode=tools");
+ await gotoLauncher(page, "/tools");
+ const results = visibleByTestId(page, "tools-search-results-page");
for (const [title, href] of [
["Medication Prescribing", "/medications"],
["Documents", "/documents"],
@@ -680,12 +692,10 @@ test.describe("PsychSift tools directory and legacy launcher", () => {
["Saved workflows", "/favourites"],
["PsychSift Search", "/?mode=answer"],
] as const) {
- const detailsButton = page.getByRole("button", { name: `View details for ${title}` });
- await expect(detailsButton).toHaveAttribute("aria-haspopup", "dialog");
- await detailsButton.click();
- const dialog = page.getByRole("dialog", { name: title });
- await expect(dialog.locator(`a[href="${href}"]`).first()).toBeVisible();
- await page.getByRole("button", { name: `Close ${title}` }).click();
+ await expect(results.getByRole("link", { name: `Open ${title}` })).toHaveAttribute("href", href);
+ await results.getByRole("button", { name: `View details for ${title}` }).click();
+ const detail = results.getByRole("complementary", { name: title });
+ await expect(detail.locator(`a[href="${href}"]`).first()).toBeVisible();
}
// External companion-app launchers were removed; no localhost links should remain.
await expect(page.locator('a[href^="http://localhost"], a[href^="http://127.0.0.1"]')).toHaveCount(0);
@@ -1317,11 +1327,12 @@ test.describe("PsychSift tools directory and legacy launcher", () => {
}
});
- test("tablet legacy Tools alias uses its local filter without shared search chrome", async ({ page }) => {
+ test("tablet legacy Tools alias redirects to the directory without shared search chrome", async ({ page }) => {
await page.setViewportSize({ width: 768, height: 1024 });
await gotoLauncher(page, "/?mode=tools");
- await expect(page.getByRole("heading", { level: 1, name: "Tools" })).toBeVisible();
+ await page.waitForURL(/\/tools$/);
+ await expect(page.getByRole("heading", { level: 1, name: "All tools" })).toBeVisible();
await expect(visibleGlobalSearchInput(page)).toHaveCount(0);
await expect(page.locator("form.answer-footer-search-dock")).toHaveCount(0);
await expect(page.getByTestId("tools-local-search-input")).toBeVisible();
@@ -3293,6 +3304,28 @@ test.describe("Responsive layout guards", () => {
// click/query assertions have nothing left to click. The still-live half of
// the second test — the submitted-search results view — survives below,
// reached directly instead of via a pill click.
+ /*
+ * Clearing the composer used to empty the React query while leaving `q` and
+ * `run=1` in the URL. `showSharedHome` reads `run=1` off the URL, so it stayed
+ * suppressed while MedicationPrescribingWorkspace, now seeing no query, fell back
+ * to `medication-home` — the Dose/Safety/Monitoring tiles `/medications` was
+ * consolidated away from, and which this file's own note above calls "retired and
+ * no longer reachable from any route". It was reachable, by this exact click.
+ */
+ test("clearing a prescribing search returns the shared home, never the retired medication home", async ({ page }) => {
+ await mockAnswerDashboardApi(page);
+ await gotoLauncher(page, "/?mode=prescribing&q=acamprosate%20renal%20dose&run=1");
+
+ await page
+ .getByRole("button", { name: /clear search question|clear search/i })
+ .first()
+ .click();
+
+ await expect(page).toHaveURL(/\/\?mode=prescribing&focus=1$/);
+ await expect(page.getByTestId("shared-home-empty-state")).toBeVisible();
+ await expect(page.getByTestId("medication-home")).toHaveCount(0);
+ });
+
test("prescribing submitted search keeps results above the phone bottom dock", async ({ page }) => {
await page.setViewportSize({ width: 320, height: 760 });
await mockAnswerDashboardApi(page);
diff --git a/tests/ui-universal-search.spec.ts b/tests/ui-universal-search.spec.ts
index f783c42524..d563df3129 100644
--- a/tests/ui-universal-search.spec.ts
+++ b/tests/ui-universal-search.spec.ts
@@ -578,6 +578,34 @@ test.describe("universal search smart affordances", () => {
await expect(page.getByTestId("universal-also-matches")).toHaveCount(0);
});
+ test("clearing a pending Answer request stays on the shared home after the response settles", async ({ page }) => {
+ await page.route(/\/api\/answer(?:\/stream)?(?:\?.*)?$/, async (route) => {
+ await new Promise((resolve) => setTimeout(resolve, 1_000));
+ try {
+ await route.fulfill({ json: syntheticAnswer });
+ } catch {
+ // Aborting the request is the expected clear-path outcome.
+ }
+ });
+
+ const input = await openComposer(page, "/?mode=answer&focus=1");
+ await input.fill("acamprosat");
+ await page.getByRole("button", { name: "Generate source-backed answer" }).click();
+ await expect(page.getByTestId("answer-progress")).toBeVisible();
+
+ await page
+ .getByRole("button", { name: /clear search question|clear search/i })
+ .first()
+ .click();
+
+ await expect(page).toHaveURL(/\/\?mode=answer&focus=1$/);
+ await expect(page.getByTestId("shared-home-empty-state")).toBeVisible();
+ await page.waitForTimeout(1_250);
+ await expect(page).toHaveURL(/\/\?mode=answer&focus=1$/);
+ await expect(page.getByTestId("shared-home-empty-state")).toBeVisible();
+ await expect(page.locator('[data-dashboard-stage="answer-surface"]')).toHaveCount(0);
+ });
+
test("keeps a saved exact match first in Favourites", async ({ page }) => {
await mockSmartSearch(page);
const input = await openComposer(page, "/favourites?focus=1");