@nextclaw/ui 0.26.5 → 0.27.1

package/src/features/right-panel-resources/components/__tests__/system-object-resource.test.tsx

@@ -1,8 +1,41 @@
 import type * as ReactQuery from "@tanstack/react-query";
-import { cleanup, render, screen } from "@testing-library/react";
+import type { ReactNode } from "react";
+import type * as ReactRouter from "react-router-dom";
+import { cleanup, fireEvent, render, screen } from "@testing-library/react";
 import { afterEach, describe, expect, it, vi } from "vitest";
 import { SYSTEM_OBJECT_RESOURCE_RENDERERS } from "@/features/right-panel-resources/components/system-object-resource";
+import { WORKSPACE_FILE_PANEL_RENDERERS } from "@/features/right-panel-resources/components/workspace-file-resource";
+import { createWorkspaceFilePanelTarget } from "@/features/chat";
 import type { DocBrowserCustomTabRenderParams } from "@/shared/components/doc-browser/doc-browser-renderer.types";
+import type { WorkspaceTextExcerpt } from "@/features/chat";
+import { NativeObjectResource } from "@/features/right-panel-resources/components/native-object-resource";
+
+vi.mock("@/features/agents", () => ({ AgentsPage: ({ resourceId }: { resourceId: string }) => <div>agent:{resourceId}</div> }));
+vi.mock("@/features/cron", () => ({ CronJobResource: ({ jobId }: { jobId: string }) => <div>cron-job:{jobId}</div> }));
+vi.mock("@/features/projects", () => ({ ProjectsPage: ({ resourceId, resourceWorkId }: { resourceId: string; resourceWorkId?: string }) => <div>project:{resourceId}{resourceWorkId ? `/${resourceWorkId}` : ""}</div> }));
+vi.mock("@/features/inbox", () => ({ InboxPage: ({ resourceId }: { resourceId: string }) => <div>inbox-delivery:{resourceId}</div> }));
+vi.mock("@/features/apps", () => ({ AppsPanel: ({ serviceResourceId }: { serviceResourceId: string }) => <div>service-app:{serviceResourceId}</div> }));
+vi.mock("@/features/marketplace", () => ({ McpMarketplacePage: ({ resourceId }: { resourceId: string }) => <div>mcp-server:{resourceId}</div> }));
+vi.mock("@/features/panel-apps", () => ({
+  PanelAppRuntimeSurface: ({ appId }: { appId: string }) => <div>panel-app:{appId}</div>,
+  PanelAppHostProvider: ({ children }: { children: ReactNode }) => children,
+}));
+
+const actions = vi.hoisted(() => ({ addExcerptToChat: vi.fn(), navigate: vi.fn() }));
+vi.mock("@/features/chat", async () => ({
+  ...await import("@/features/chat/features/workspace/utils/workspace-file-panel-route.utils"),
+  ...await import("@/features/chat/features/workspace/utils/chat-workspace-file-tab.utils"),
+  ...await import("@/features/chat/features/workspace/utils/workspace-text-excerpt.utils"),
+  ...await import("@/features/chat/features/workspace/components/chat-session-workspace-file-preview"),
+}));
+vi.mock("@/app/components/app-presenter-provider", () => ({
+  useAppPresenter: () => ({ pageResourceManager: actions }),
+}));
+vi.mock("react-router-dom", async (importOriginal) => ({
+  ...await importOriginal<typeof ReactRouter>(),
+  useLocation: () => ({ pathname: "/chat/current-session" }),
+  useNavigate: () => actions.navigate,
+}));
 
 vi.mock("@tanstack/react-query", async (importOriginal) => ({
   ...await importOriginal<typeof ReactQuery>(),
@@ -14,13 +47,44 @@ vi.mock("@tanstack/react-query", async (importOriginal) => ({
   }),
 }));
 vi.mock("@/features/chat/features/workspace/components/chat-session-workspace-file-preview", () => ({
-  ChatSessionWorkspaceFilePreview: ({ file }: { file: { key: string; rawText: string } }) => (
-    <div data-testid="preview" data-resource-key={file.key}>{file.rawText}</div>
+  ChatSessionWorkspaceFilePreview: ({ file, onTextExcerptAdd }: {
+    file: { key: string; rawText: string };
+    onTextExcerptAdd?: (excerpt: WorkspaceTextExcerpt) => void;
+  }) => (
+    <div data-testid="preview" data-resource-key={file.key}>{file.rawText}
+      <button onClick={() => onTextExcerptAdd?.({ path: "agent.md", label: "agent.md", excerpt: "Agent snapshot", startLine: 1, endLine: 1 })}>Add selection</button>
+    </div>
   ),
 }));
 afterEach(cleanup);
 
 describe("system object reading identity", () => {
+  it.each(["agent", "cron-job", "project", "inbox-delivery", "service-app", "mcp-server", "panel-app"])("opens %s with its native renderer and unchanged identity", async (objectType) => {
+    render(<NativeObjectResource objectType={objectType} objectId="raw:id" openTarget={vi.fn()} />);
+    expect(await screen.findByText(`${objectType}:raw:id`)).toBeTruthy();
+    expect(screen.queryByTestId("preview")).toBeNull();
+  });
+
+  it("keeps project and work item identities separate", async () => {
+    render(<NativeObjectResource objectType="project-work" objectId={JSON.stringify(["project", "work"])} openTarget={vi.fn()} />);
+    expect(await screen.findByText("project:project/work")).toBeTruthy();
+  });
+
+  it("retains a global file's source context when quoting into a different conversation", () => {
+    const target = createWorkspaceFilePanelTarget({
+      key: "source-file", path: "agent.md", label: "agent.md", parentSessionKey: "source-session", viewMode: "preview",
+    }, { workingDir: "/source/workspace", projectRoot: "/source/workspace" });
+    render(WORKSPACE_FILE_PANEL_RENDERERS["workspace-file"].renderContent!({
+      currentUrl: target.url,
+      tab: { ...target, id: target.url, currentUrl: target.url, history: [target.url], historyIndex: 0, navVersion: 0 },
+      open: vi.fn(), openTarget: vi.fn(), refreshIframe: vi.fn(),
+    }));
+    fireEvent.click(screen.getByRole("button", { name: "Add selection" }));
+    expect(actions.addExcerptToChat).toHaveBeenLastCalledWith({
+      path: target.resourceUri, label: "agent.md", excerpt: "Agent snapshot", startLine: 1, endLine: 1,
+    }, "/chat/current-session", actions.navigate);
+  });
+
   it("isolates same-named snapshots by resource URI and preserves identity on reopen", () => {
     const content = (uri: string) => SYSTEM_OBJECT_RESOURCE_RENDERERS["system-object"].renderContent!({
       currentUrl: uri,
@@ -29,14 +93,18 @@ describe("system object reading identity", () => {
       openTarget: vi.fn(),
       refreshIframe: vi.fn(),
     } satisfies DocBrowserCustomTabRenderParams);
-    const first = "nextclaw://objects/agent/first";
-    const second = "nextclaw://objects/agent/second";
+    const first = "nextclaw://objects/skill/first";
+    const second = "nextclaw://objects/skill/second";
     const view = render(content(first));
     expect(screen.getByTestId("preview").getAttribute("data-resource-key")).toBe(first);
     view.rerender(content(second));
     expect(screen.getByTestId("preview").getAttribute("data-resource-key")).toBe(second);
     view.rerender(content(first));
     expect(screen.getByTestId("preview").getAttribute("data-resource-key")).toBe(first);
-    expect(screen.getByTestId("preview").textContent).toBe("# Agent snapshot");
+    expect(screen.getByTestId("preview").textContent).toContain("# Agent snapshot");
+    fireEvent.click(screen.getByRole("button", { name: "Add selection" }));
+    expect(actions.addExcerptToChat).toHaveBeenCalledWith({
+      path: first, label: "Agent", excerpt: "Agent snapshot", startLine: null, endLine: null,
+    }, "/chat/current-session", actions.navigate);
   });
 });

package/src/features/right-panel-resources/components/native-object-resource.tsx

@@ -0,0 +1,134 @@
+import { lazy, Suspense, useState } from "react";
+import type { DocBrowserCustomTabRenderParams } from "@/shared/components/doc-browser/doc-browser-renderer.types";
+import { t } from "@/shared/lib/i18n";
+import {
+  createPanelAppRightPanelResourceTarget,
+  type RightPanelAppsTab,
+} from "@/features/right-panel-resources/utils/right-panel-resource-uri.utils";
+
+const AgentsPage = lazy(async () => ({
+  default: (await import("@/features/agents"))
+    .AgentsPage,
+}));
+const CronJobResource = lazy(async () => ({
+  default: (await import("@/features/cron"))
+    .CronJobResource,
+}));
+const ProjectsPage = lazy(async () => ({
+  default: (await import("@/features/projects"))
+    .ProjectsPage,
+}));
+const InboxPage = lazy(async () => ({
+  default: (await import("@/features/inbox")).InboxPage,
+}));
+const AppsPanel = lazy(async () => ({
+  default: (await import("@/features/apps")).AppsPanel,
+}));
+const McpMarketplacePage = lazy(async () => ({
+  default: (
+    await import("@/features/marketplace")
+  ).McpMarketplacePage,
+}));
+const PanelAppRuntimeSurface = lazy(async () => ({
+  default: (
+    await import("@/features/panel-apps")
+  ).PanelAppRuntimeSurface,
+}));
+const PanelAppHostProvider = lazy(async () => ({
+  default: (
+    await import("@/features/panel-apps")
+  ).PanelAppHostProvider,
+}));
+
+function ServiceAppResource({
+  objectId,
+  openTarget,
+}: {
+  objectId: string;
+  openTarget: DocBrowserCustomTabRenderParams["openTarget"];
+}) {
+  const [activeTab, setActiveTab] = useState<RightPanelAppsTab>("service-apps");
+  return (
+    <AppsPanel
+      serviceResourceId={objectId}
+      activeTab={activeTab}
+      onActiveTabChange={setActiveTab}
+      onOpenPanelApp={(entry) =>
+        openTarget(createPanelAppRightPanelResourceTarget(entry))
+      }
+    />
+  );
+}
+
+/** Object identity chooses domain content; the workbench owns placement. */
+export function NativeObjectResource({
+  objectType,
+  objectId,
+  openTarget,
+}: {
+  objectType: string;
+  objectId: string;
+  openTarget: DocBrowserCustomTabRenderParams["openTarget"];
+}) {
+  let content;
+  switch (objectType) {
+    case "agent":
+      content = <AgentsPage resourceId={objectId} />;
+      break;
+    case "cron-job":
+      content = <CronJobResource jobId={objectId} />;
+      break;
+    case "project":
+      content = <ProjectsPage resourceId={objectId} />;
+      break;
+    case "project-work": {
+      let ids: unknown;
+      try {
+        ids = JSON.parse(objectId);
+      } catch {
+        ids = null;
+      }
+      if (
+        !Array.isArray(ids) ||
+        ids.length !== 2 ||
+        ids.some((id) => typeof id !== "string" || !id)
+      )
+        break;
+      content = <ProjectsPage resourceId={ids[0]} resourceWorkId={ids[1]} />;
+      break;
+    }
+    case "inbox-delivery":
+      content = <InboxPage resourceId={objectId} />;
+      break;
+    case "mcp-server":
+      content = <McpMarketplacePage resourceId={objectId} />;
+      break;
+    case "service-app":
+      content = (
+        <ServiceAppResource objectId={objectId} openTarget={openTarget} />
+      );
+      break;
+    case "panel-app":
+      content = (
+        <PanelAppHostProvider>
+          <PanelAppRuntimeSurface appId={objectId} restorationScope="main" />
+        </PanelAppHostProvider>
+      );
+      break;
+  }
+  return (
+    <Suspense
+      fallback={
+        <p role="status" className="p-4">
+          {t("loading")}
+        </p>
+      }
+    >
+      {content ?? (
+        <p role="alert" className="p-4">
+          {t("resourceInvalid")}
+        </p>
+      )}
+    </Suspense>
+  );
+}

package/src/features/right-panel-resources/components/page-resource-icon.tsx

@@ -22,7 +22,6 @@ import {
   Github,
   Globe,
   LayoutDashboard,
-  Link2,
   MessageSquare,
   PanelsTopLeft,
   Plus,
@@ -44,7 +43,7 @@ const RESOURCE_ICONS = {
   app: AppWindow,
   docs: BookOpen,
   website: Globe,
-  generic: Link2,
+  generic: null,
   page: PanelsTopLeft,
   overview: LayoutDashboard,
   "child-sessions": GitBranch,
@@ -82,6 +81,7 @@ function resourceTypeIcon(
       ? (kind as keyof typeof RESOURCE_ICONS)
       : "page";
   }
+  if (parseSessionKeyFromPanelUrl(uri)) return "session";
   const object = parseSystemObjectReferenceUri(uri);
   if (object) return OBJECT_TYPE_ICONS[object.objectType] ?? "generic";
   if (
@@ -90,11 +90,11 @@ function resourceTypeIcon(
   )
     return "skill";
   if (uri.startsWith("nextclaw://page?")) return "page";
-  if (uri.startsWith("nextclaw://chat-session")) return "session";
   if (
     uri.startsWith("nextclaw://workspace-file") ||
     uri.startsWith("file:") ||
-    uri.startsWith("nextclaw://file/")
+    uri.startsWith("nextclaw://file/") ||
+    (!/^[a-z][a-z0-9+.-]*:/i.test(uri.replace(/:\d+(?::\d+)?$/, "")) && /\.[a-z0-9]{1,16}(?::\d+(?::\d+)?)?(?:[?#]|$)/i.test(uri))
   )
     return "file";
   if (
@@ -104,7 +104,7 @@ function resourceTypeIcon(
   )
     return "app";
   if (uri.startsWith("nextclaw://docs")) return "docs";
-  return /^https?:/.test(uri) ? "website" : "generic";
+  return "generic";
 }
 
 function appIcon(value?: string): DocBrowserDockIcon | undefined {
@@ -222,20 +222,28 @@ function ResourceIcon({
         {specific.value}
       </span>
     );
-  let pathname = uri;
+  let pathname = uri.replace(/:\d+(?::\d+)?$/, "");
   try {
-    pathname = new URL(uri).searchParams.get("path") ?? new URL(uri).pathname;
+    const url = new URL(pathname);
+    pathname = url.searchParams.get("path") ?? url.pathname;
   } catch {
     /* Local file link. */
   }
+  pathname = pathname.split(/[?#]/, 1)[0].replace(/:\d+(?::\d+)?$/, "");
+  try {
+    pathname = decodeURIComponent(pathname);
+  } catch {
+    /* Keep malformed percent escapes as literal file name characters. */
+  }
   if (
     resourceTypeIcon(uri, appId) === "file" &&
-    /\.[a-z0-9]{1,8}(?:[?#]|$)/i.test(pathname)
+    /\.[a-z0-9]{1,16}$/i.test(pathname)
   )
     return <FileTypeIcon fileName={pathname} size="compact" />;
   const Icon =
     (specific?.type === "builtin" && BUILTIN_ICONS[specific.name]) ||
     RESOURCE_ICONS[resourceTypeIcon(uri, appId)];
+  if (!Icon) return null;
   return (
     <Icon
       aria-hidden="true"

package/src/features/right-panel-resources/components/system-object-resource.tsx

@@ -9,11 +9,63 @@ import type {
   DocBrowserCustomTabRenderers,
 } from "@/shared/components/doc-browser/doc-browser-renderer.types";
 import { t } from "@/shared/lib/i18n";
+import { useAppPresenter } from "@/app/components/app-presenter-provider";
+import { useLocation, useNavigate } from "react-router-dom";
+import { parseSystemObjectReferenceUri } from "@nextclaw/shared";
+import { NativeObjectResource } from "./native-object-resource";
+import { ChatTextSelectionAction } from "@nextclaw/agent-chat-ui";
+import { WORKSPACE_TEXT_EXCERPT_MAX_CHARACTERS } from "@/features/chat";
 
 function SystemObjectResource({
   tab,
   openTarget,
 }: DocBrowserCustomTabRenderParams) {
+  const app = useAppPresenter();
+  const { pathname } = useLocation();
+  const navigate = useNavigate();
+  const uri = tab.resourceUri ?? tab.currentUrl;
+  const identity = parseSystemObjectReferenceUri(uri);
+  if (!identity)
+    return (
+      <p role="alert" className="p-4">
+        {t("resourceInvalid")}
+      </p>
+    );
+  if (identity.objectType !== "skill") {
+    return (
+      <ChatTextSelectionAction
+        actionLabel={t("chatWorkspaceAddToChat")}
+        className="h-full min-h-0"
+        maxCharacters={WORKSPACE_TEXT_EXCERPT_MAX_CHARACTERS}
+        selectionTooLongLabel={t("chatWorkspaceExcerptSelectionTooLong")}
+        onAddToChat={({ text }) =>
+          app.pageResourceManager.addExcerptToChat(
+            {
+              path: uri,
+              label: tab.title,
+              excerpt: text,
+              startLine: null,
+              endLine: null,
+            },
+            pathname,
+            navigate,
+          )
+        }
+      >
+        <NativeObjectResource key={uri} {...identity} openTarget={openTarget} />
+      </ChatTextSelectionAction>
+    );
+  }
+  return <SystemObjectDocument tab={tab} openTarget={openTarget} />;
+}
+
+function SystemObjectDocument({
+  tab,
+  openTarget,
+}: Pick<DocBrowserCustomTabRenderParams, "tab" | "openTarget">) {
+  const app = useAppPresenter();
+  const { pathname } = useLocation();
+  const navigate = useNavigate();
   const uri = tab.resourceUri ?? tab.currentUrl;
   const query = useQuery({
     queryKey: ["resource-object-snapshot", uri],
@@ -22,7 +74,7 @@ function SystemObjectResource({
         await nextclawClient.systemObjectReferences.resolve(uri);
       const response = await fetch(
         buildNcpAssetContentUrl(reference.assetUri),
-        { signal },
+        { signal: AbortSignal.any([signal, AbortSignal.timeout(30_000)]) },
       );
       if (!response.ok)
         throw new Error(`Unable to read resource (${response.status})`);
@@ -77,6 +129,19 @@ function SystemObjectResource({
       file={{ ...file, key: uri }}
       sessionWorkingDir={null}
       sessionProjectRoot={null}
+      onTextExcerptAdd={(excerpt) =>
+        app.pageResourceManager.addExcerptToChat(
+          {
+            ...excerpt,
+            path: uri,
+            label: reference.label,
+            startLine: null,
+            endLine: null,
+          },
+          pathname,
+          navigate,
+        )
+      }
       onFileOpen={(action) => {
         const next = createWorkspaceFileTab(action, null);
         if (next)

package/src/features/right-panel-resources/components/workspace-file-resource.tsx

@@ -11,11 +11,16 @@ import {
 } from "@/features/chat";
 import { createWorkspaceFileTab } from "@/features/chat";
 import { t } from "@/shared/lib/i18n";
+import { useAppPresenter } from "@/app/components/app-presenter-provider";
+import { useLocation, useNavigate } from "react-router-dom";
 
 function WorkspaceFilePanelContent({
   tab,
   openTarget,
 }: DocBrowserCustomTabRenderParams) {
+  const app = useAppPresenter();
+  const { pathname } = useLocation();
+  const navigate = useNavigate();
   const view = readWorkspaceFilePanelView(tab);
   if (!view)
     return (
@@ -32,6 +37,10 @@ function WorkspaceFilePanelContent({
         file={view.file}
         sessionWorkingDir={view.workingDir}
         sessionProjectRoot={view.projectRoot}
+        onTextExcerptAdd={(excerpt) => app.pageResourceManager.addExcerptToChat({
+          ...excerpt,
+          path: tab.resourceUri ?? tab.currentUrl,
+        }, pathname, navigate)}
         onFileOpen={(action) => {
           const file = createWorkspaceFileTab(
             action,

package/src/features/right-panel-resources/configs/right-panel-resource-routes.config.ts

@@ -33,7 +33,8 @@ function isUrlWithPrefix(url: string, prefix: string): boolean {
 function resolveNextclawDocsUrl(uri: ParsedResourceUri): string {
   const docsPath = uri.pathSegments.join('/');
   const rawPath = docsPath ? `/${docsPath}` : '/';
-  return normalizeUrlByKind(rawPath, 'docs');
+  const parsed = new URL(uri.raw);
+  return normalizeUrlByKind(`${rawPath}${parsed.search}${parsed.hash}`, 'docs');
 }
 
 function createDocsResourceUriFromSegments(pathSegments: string[]): string {
@@ -45,20 +46,9 @@ function createDocsResourceUriFromSegments(pathSegments: string[]): string {
 }
 
 function createDocsResourceUriFromUrl(url: string): string {
-  const normalized = normalizeDocUrl(url);
-  const pathname = normalized.startsWith('/')
-    ? normalized
-    : (() => {
-        try {
-          return new URL(url).pathname;
-        } catch {
-          return '';
-        }
-      })();
-  if (!pathname) {
-    return 'nextclaw://docs';
-  }
-  return createDocsResourceUriFromSegments(pathname.split('/'));
+  const parsed = new URL(url);
+  const pathname = parsed.pathname.replace(/\.html$/, '').replace(/\/$/, '');
+  return `${createDocsResourceUriFromSegments(pathname.split('/'))}${parsed.search}${parsed.hash}`;
 }
 
 function isPanelAppContentUri(uri: ParsedResourceUri): boolean {
@@ -97,7 +87,7 @@ function arePanelAppUrlsEquivalent(left: string, right: string): boolean {
 export const RIGHT_PANEL_RESOURCE_ROUTE_DEFINITIONS: RightPanelResourceRouteDefinition[] = [
   {
     id: 'system-object', kind: 'system-object', defaultUrl: () => 'nextclaw://objects',
-    match: (uri) => Boolean(parseSystemObjectReferenceUri(uri.raw)),
+    match: (uri) => !parseSessionKeyFromPanelUrl(uri.raw) && Boolean(parseSystemObjectReferenceUri(uri.raw)),
     resolve: (uri) => ({ kind: 'system-object', title: parseSystemObjectReferenceUri(uri.raw)!.objectId, url: uri.raw, resourceUri: uri.raw, historyPolicy: 'none' }),
     areEquivalent: (left, right) => left === right,
   },
@@ -120,10 +110,10 @@ export const RIGHT_PANEL_RESOURCE_ROUTE_DEFINITIONS: RightPanelResourceRouteDefi
     areEquivalent: (left, right) => left === right,
   },
   {
-    defaultUrl: () => 'nextclaw://chat-session/',
+    defaultUrl: () => 'nextclaw://sessions/',
     id: CHAT_SESSION_PANEL_KIND,
     kind: CHAT_SESSION_PANEL_KIND,
-    match: (uri) => uri.scheme === 'nextclaw' && uri.authority === 'chat-session',
+    match: (uri) => parseSessionKeyFromPanelUrl(uri.raw) !== null,
     resolve: (uri) => {
       const sessionKey = parseSessionKeyFromPanelUrl(uri.raw);
       const url = sessionKey ? buildSessionPanelUrl(sessionKey) : uri.raw;
@@ -136,7 +126,7 @@ export const RIGHT_PANEL_RESOURCE_ROUTE_DEFINITIONS: RightPanelResourceRouteDefi
         historyPolicy: 'none',
       };
     },
-    areEquivalent: (left, right) => left === right,
+    areEquivalent: (left, right) => parseSessionKeyFromPanelUrl(left) === parseSessionKeyFromPanelUrl(right),
   },
   {
     defaultUrl: () => RIGHT_PANEL_HOME_URL,
@@ -179,7 +169,7 @@ export const RIGHT_PANEL_RESOURCE_ROUTE_DEFINITIONS: RightPanelResourceRouteDefi
       return {
         historyPolicy: 'managed',
         kind: 'docs',
-        resourceUri: createDocsResourceUriFromSegments(uri.pathSegments),
+        resourceUri: `${createDocsResourceUriFromSegments(uri.pathSegments)}${new URL(uri.raw).search}${new URL(uri.raw).hash}`,
         title: t('docBrowserHelp'),
         url,
       };
@@ -198,7 +188,7 @@ export const RIGHT_PANEL_RESOURCE_ROUTE_DEFINITIONS: RightPanelResourceRouteDefi
         historyPolicy: 'managed',
         kind: RIGHT_PANEL_APPS_TAB_KIND,
         resourceUri: url,
-        title: url === RIGHT_PANEL_APPS_URL ? t('appsTitle') : t('serviceAppsTitle'),
+        title: t(new URL(url).searchParams.get('tab') === 'service-apps' ? 'serviceAppsTitle' : 'appsTitle'),
         url,
       };
     },

package/src/features/right-panel-resources/index.ts

@@ -1,4 +1,7 @@
 export {
+  createRightPanelAppsUrl,
+  getRightPanelAppsTabFromUrl,
+  type RightPanelAppsTab,
   createChatUiResourceReferenceFromTab,
   createPanelAppContentPath,
   createPanelAppResourceUri,

package/src/features/right-panel-resources/managers/__tests__/page-resource.manager.test.ts

@@ -17,7 +17,7 @@ import {
 function setup() {
   const app = {
     rightPanelResourceRouteResolver: new RightPanelResourceRouteResolver(),
-    docBrowserManager: { openTarget: vi.fn() },
+    docBrowserManager: { openTarget: vi.fn(), setActiveTab: vi.fn() },
     workbenchSurfaceManager: {
       restore: vi.fn(),
       toggleMaximize: vi.fn(),
@@ -28,6 +28,7 @@ function setup() {
       requestUiResourceReference: vi.fn(),
       requestFileReference: vi.fn(),
       requestSystemObjectReference: vi.fn(),
+      requestExcerptReference: vi.fn(),
     },
   };
   return {
@@ -38,6 +39,16 @@ function setup() {
 }
 
 describe("shared resource opening policy", () => {
+  it.each(["/chat/current-session", "/chat", "/agents"])("routes a selected resource excerpt from %s to its chat composer", (pathname) => {
+    const { manager, app, navigate } = setup();
+    const excerpt = { path: "nextclaw://objects/agent/philosopher", label: "灵魂哲学家", excerpt: "Runtime: native", startLine: null, endLine: null };
+    manager.addExcerptToChat(excerpt, pathname, navigate);
+    expect(app.chatComposerIntentManager.requestExcerptReference).toHaveBeenCalledWith({
+      ...excerpt, targetSessionKey: pathname === "/chat/current-session" ? "current-session" : null,
+    });
+    if (pathname === "/agents") expect(navigate).toHaveBeenCalledWith("/chat");
+    else expect(navigate).not.toHaveBeenCalled();
+  });
   beforeEach(() => {
     usePageNavigationStore.setState({ pinned: [] });
     useDocBrowserStore.setState({ snapshot: createDefaultDocBrowserState() });
@@ -74,9 +85,8 @@ describe("shared resource opening policy", () => {
       },
     });
     manager.open(page, "default", navigate);
-    expect(app.docBrowserManager.openTarget).toHaveBeenCalledWith(page.target, {
-      newTab: true,
-    });
+    expect(app.docBrowserManager.setActiveTab).toHaveBeenCalledWith(page.uri);
+    expect(app.docBrowserManager.openTarget).not.toHaveBeenCalled();
     expect(navigate).not.toHaveBeenCalled();
     manager.open(page, "main", navigate);
     expect(navigate).toHaveBeenCalledWith("/apps/panel/example-notes");