@nextclaw/ui 0.24.2 → 0.24.3

package/src/features/projects/pages/project-home-page.tsx

@@ -1,13 +1,8 @@
 import { useState } from "react";
 import { useNavigate, useParams } from "react-router-dom";
-import { useMutation, useQueryClient } from "@tanstack/react-query";
-import type { ObservedRequest } from "@nextclaw/client-sdk";
-import { toast } from "sonner";
-import { useProjects, useRemoveProject } from "@/shared/hooks/use-projects";
+import { useProjects } from "@/shared/hooks/use-projects";
 import { useViewportLayout } from "@/app/hooks/use-viewport-layout";
 import { t } from "@/shared/lib/i18n";
-import { Button } from "@/shared/components/ui/button";
-import { useConfirmDialog } from "@/shared/hooks/use-confirm-dialog";
 import {
   Tabs,
   TabsContent,
@@ -17,7 +12,6 @@ import {
 import { ProjectAgreement } from "@/features/projects/components/project-agreement";
 import { ProjectArtifacts } from "@/features/projects/components/project-artifacts";
 import { ProjectOverview } from "@/features/projects/components/project-overview";
-import { ProjectRequests } from "@/features/projects/components/project-requests";
 import { ProjectSkills } from "@/features/projects/components/project-skills";
 import { ProjectWorkItemDrawer } from "@/features/projects/components/work/project-work-item-drawer";
 import { ProjectWorkItems } from "@/features/projects/components/work/project-work-items";
@@ -26,18 +20,13 @@ import {
   usePresenter,
 } from "@/features/chat";
 import {
-  projectObservationQueryKey,
-  useProjectObservation,
-} from "@/features/projects/hooks/use-project-observation";
+  useProjectAgreement,
+  useProjectSkills,
+} from "@/features/projects/hooks/use-project-materials";
 import {
-  getOpenProjectRequests,
   isProjectHomeTab,
   type ProjectHomeTab,
 } from "@/features/projects/presenters/project-home.presenter";
-import {
-  sendProjectRequestResponse,
-  type ProjectRequestDecision,
-} from "@/features/projects/utils/project-request-response.utils";
 import { joinProjectPath } from "@/features/projects/utils/project-artifact-view.utils";
 
 const PROJECT_TABS: ProjectHomeTab[] = [
@@ -55,73 +44,23 @@ export function ProjectsPage() {
   }>();
   const navigate = useNavigate();
   const presenter = usePresenter();
-  const queryClient = useQueryClient();
   const { isMobile } = useViewportLayout();
   const projects = useProjects();
-  const projectRemoval = useRemoveProject();
-  const { confirm, ConfirmDialog } = useConfirmDialog();
   const [selectedWorkItemId, setSelectedWorkItemId] = useState<string | null>(
     null,
   );
-  const [pendingRequestId, setPendingRequestId] = useState<string | null>(null);
   const registered = projects.data?.projects ?? [];
   const selectedProject =
     registered.find((project) => project.id === projectId) ?? null;
   const tab: ProjectHomeTab | null = isProjectHomeTab(tabParam)
     ? tabParam
     : null;
-  const needsObservation =
-    tab === "artifacts" || tab === "skills" || tab === "agreement";
-  const observation = useProjectObservation(
-    selectedProject?.id ?? null,
-    selectedProject?.rootPath ?? null,
-    needsObservation,
+  const agreement = useProjectAgreement(
+    tab === "agreement" ? (selectedProject?.id ?? null) : null,
+  );
+  const skills = useProjectSkills(
+    tab === "skills" ? (selectedProject?.id ?? null) : null,
   );
-  const response = useMutation({
-    mutationFn: sendProjectRequestResponse,
-    onMutate: (input) => setPendingRequestId(input.requestId),
-    onSuccess: async () => {
-      if (selectedProject)
-        await queryClient.invalidateQueries({
-          queryKey: projectObservationQueryKey(selectedProject.id),
-        });
-      toast.success(t("projectsResponseSent"));
-    },
-    onError: (error) =>
-      toast.error(
-        `${t("projectsResponseFailed")}: ${error instanceof Error ? error.message : String(error)}`,
-      ),
-    onSettled: () => setPendingRequestId(null),
-  });
-  const reply = (
-    request: ObservedRequest,
-    decision: ProjectRequestDecision,
-  ) => {
-    if (!request.reference.sessionId) return;
-    response.mutate({
-      requestId: request.id,
-      sessionId: request.reference.sessionId,
-      decision,
-      prompt: request.prompt,
-    });
-  };
-  const removeSelectedProject = async () => {
-    if (!selectedProject) return;
-    const confirmed = await confirm({
-      title: `${t("projectsRemoveConfirmTitle")} “${selectedProject.name}”`,
-      description: t("projectsRemoveConfirmDescription"),
-      confirmLabel: t("projectsRemoveConfirmAction"),
-      variant: "destructive",
-    });
-    if (!confirmed) return;
-    try {
-      await projectRemoval.mutateAsync(selectedProject.id);
-      navigate("/projects");
-    } catch {
-      // The mutation owns the visible error and the current project stays open.
-    }
-  };
-
   if (projects.isLoading)
     return (
       <main className="flex h-full items-center justify-center p-6 text-sm text-muted-foreground">
@@ -154,35 +93,16 @@ export function ProjectsPage() {
       viewMode: "preview",
       previewViewer: "rendered",
     });
-  const snapshot = observation.data;
   return (
     <>
       <main className="h-full min-w-0 flex-1 overflow-y-auto p-3 sm:p-4 md:p-6">
         <div className="mx-auto max-w-6xl space-y-4">
-          <header className="flex flex-col gap-3 rounded-2xl border border-border/60 bg-card p-3 sm:flex-row sm:items-start sm:justify-between sm:p-4">
-            <div className="min-w-0">
-              <h1 className="text-lg font-semibold">{selectedProject.name}</h1>
-              <p className="mt-1 break-all text-xs text-muted-foreground">
-                {selectedProject.rootPath}
-              </p>
-            </div>
-            <Button
-              type="button"
-              variant="destructive"
-              size="sm"
-              disabled={projectRemoval.isPending}
-              onClick={() => void removeSelectedProject()}
-            >
-              {t("projectsRemoveFromList")}
-            </Button>
+          <header className="rounded-2xl border border-border/60 bg-card p-3 sm:p-4">
+            <h1 className="text-lg font-semibold">{selectedProject.name}</h1>
+            <p className="mt-1 break-all text-xs text-muted-foreground">
+              {selectedProject.rootPath}
+            </p>
           </header>
-          {snapshot ? (
-            <ProjectRequests
-              requests={getOpenProjectRequests(snapshot)}
-              pendingRequestId={pendingRequestId}
-              onReply={reply}
-            />
-          ) : null}
           <Tabs
             value={tab}
             onValueChange={(value) =>
@@ -218,38 +138,43 @@ export function ProjectsPage() {
                 onOpenWorkItem={setSelectedWorkItemId}
               />
             </TabsContent>
-            {needsObservation && observation.isLoading ? (
-              <div className="mt-4 rounded-xl border border-border/60 p-5 text-sm text-muted-foreground">
-                {t("projectsLoading")}
-              </div>
-            ) : null}
-            {needsObservation && observation.isError ? (
-              <div className="mt-4 rounded-xl border border-destructive/40 p-5 text-sm text-destructive">
-                {t("projectsLoadFailed")}: {observation.error.message}
-              </div>
-            ) : null}
-            {snapshot ? (
-              <>
-                <TabsContent value="artifacts" className="mt-4">
-                  <ProjectArtifacts
-                    snapshot={snapshot}
-                    onOpenFile={openProjectFile}
-                  />
-                </TabsContent>
-                <TabsContent value="skills" className="mt-4">
-                  <ProjectSkills
-                    snapshot={snapshot}
-                    onOpen={(skill) => openProjectFile(skill.path, skill.name)}
-                  />
-                </TabsContent>
-                <TabsContent value="agreement" className="mt-4">
-                  <ProjectAgreement
-                    snapshot={snapshot}
-                    onOpenFile={openProjectFile}
-                  />
-                </TabsContent>
-              </>
-            ) : null}
+            <TabsContent value="artifacts" className="mt-4">
+              <ProjectArtifacts
+                projectId={selectedProject.id}
+                onOpenFile={(path, label) =>
+                  openProjectFile(
+                    joinProjectPath(selectedProject.rootPath, path),
+                    label,
+                  )
+                }
+              />
+            </TabsContent>
+            <TabsContent value="skills" className="mt-4">
+              <ProjectSkills
+                skills={skills.data ?? []}
+                isLoading={skills.isLoading}
+                isError={skills.isError}
+                onOpen={(skill) =>
+                  openProjectFile(
+                    joinProjectPath(selectedProject.rootPath, skill.path),
+                    skill.name,
+                  )
+                }
+              />
+            </TabsContent>
+            <TabsContent value="agreement" className="mt-4">
+              <ProjectAgreement
+                agreement={agreement.data}
+                isLoading={agreement.isLoading}
+                isError={agreement.isError}
+                onOpenFile={(path, label) =>
+                  openProjectFile(
+                    joinProjectPath(selectedProject.rootPath, path),
+                    label,
+                  )
+                }
+              />
+            </TabsContent>
           </Tabs>
         </div>
       </main>
@@ -267,7 +192,6 @@ export function ProjectsPage() {
         sessionKey={null}
         projectRoot={selectedProject.rootPath}
       />
-      <ConfirmDialog />
     </>
   );
 }

package/src/features/projects/pages/projects-page.test.tsx

@@ -5,32 +5,21 @@ import { beforeEach, describe, expect, it, vi } from "vitest";
 import { ProjectsPage } from "./project-home-page";
 
 const mocks = vi.hoisted(() => ({
-  confirm: vi.fn(),
+  agreementProps: vi.fn(),
+  artifactsProps: vi.fn(),
   drawerProps: vi.fn(),
-  removeProject: vi.fn(),
-  useProjectObservation: vi.fn(),
+  skillsProps: vi.fn(),
+  useProjectAgreement: vi.fn(),
+  useProjectSkills: vi.fn(),
   useProjects: vi.fn(),
 }));
 
 vi.mock("@/shared/hooks/use-projects", () => ({
   useProjects: mocks.useProjects,
-  useRemoveProject: () => ({
-    mutateAsync: mocks.removeProject,
-    isPending: false,
-  }),
 }));
-vi.mock("@/shared/hooks/use-confirm-dialog", () => ({
-  useConfirmDialog: () => ({
-    confirm: mocks.confirm,
-    ConfirmDialog: () => <div data-testid="confirm-dialog" />,
-  }),
-}));
-vi.mock("@/features/projects/hooks/use-project-observation", () => ({
-  projectObservationQueryKey: (projectId: string) => [
-    "project-observation",
-    projectId,
-  ],
-  useProjectObservation: mocks.useProjectObservation,
+vi.mock("@/features/projects/hooks/use-project-materials", () => ({
+  useProjectAgreement: mocks.useProjectAgreement,
+  useProjectSkills: mocks.useProjectSkills,
 }));
 vi.mock("@/features/chat", () => ({
   ChatConversationWorkspaceSection: () => <div data-testid="workspace" />,
@@ -64,18 +53,23 @@ vi.mock("@/features/projects/components/work/project-work-item-drawer", () => ({
   },
 }));
 vi.mock("@/features/projects/components/project-artifacts", () => ({
-  ProjectArtifacts: () => <div>Artifact content</div>,
+  ProjectArtifacts: (props: { projectId: string }) => {
+    mocks.artifactsProps(props);
+    return <div>Artifact content</div>;
+  },
 }));
 vi.mock("@/features/projects/components/project-skills", () => ({
-  ProjectSkills: () => <div>Skills content</div>,
+  ProjectSkills: (props: { skills: unknown[] }) => {
+    mocks.skillsProps(props);
+    return <div>Skills content</div>;
+  },
 }));
 vi.mock("@/features/projects/components/project-agreement", () => ({
-  ProjectAgreement: () => <div>Agreement content</div>,
-}));
-vi.mock("@/features/projects/components/project-requests", () => ({
-  ProjectRequests: () => null,
+  ProjectAgreement: (props: { agreement?: unknown }) => {
+    mocks.agreementProps(props);
+    return <div>Agreement content</div>;
+  },
 }));
-
 function renderPage(path: string) {
   return render(
     <QueryClientProvider client={new QueryClient()}>
@@ -92,10 +86,9 @@ function renderPage(path: string) {
 describe("ProjectsPage", () => {
   beforeEach(() => {
     mocks.drawerProps.mockReset();
-    mocks.confirm.mockReset();
-    mocks.confirm.mockResolvedValue(true);
-    mocks.removeProject.mockReset();
-    mocks.removeProject.mockResolvedValue(undefined);
+    mocks.agreementProps.mockReset();
+    mocks.artifactsProps.mockReset();
+    mocks.skillsProps.mockReset();
     mocks.useProjects.mockReturnValue({
       data: {
         projects: [
@@ -105,38 +98,35 @@ describe("ProjectsPage", () => {
       isLoading: false,
       isError: false,
     });
-    mocks.useProjectObservation.mockReturnValue({
+    mocks.useProjectAgreement.mockReturnValue({
+      data: undefined,
+      isLoading: false,
+      isError: false,
+    });
+    mocks.useProjectSkills.mockReturnValue({
       data: undefined,
       isLoading: false,
       isError: false,
     });
   });
 
-  it("keeps overview scan-free and opens its work item in the unified drawer", () => {
+  it("keeps overview material-free and opens its work item in the unified drawer", () => {
     renderPage("/projects/project-1/overview");
 
-    expect(mocks.useProjectObservation).toHaveBeenCalledWith(
-      "project-1",
-      "/tmp/research",
-      false,
-    );
+    expect(mocks.useProjectAgreement).toHaveBeenCalledWith(null);
+    expect(mocks.useProjectSkills).toHaveBeenCalledWith(null);
     expect(screen.getAllByRole("tab")).toHaveLength(5);
     fireEvent.click(screen.getByRole("button", { name: "Overview work" }));
     expect(screen.getByTestId("work-drawer").textContent).toBe("work-overview");
   });
 
-  it("enables legacy observation only for preserved artifact, skill, and agreement surfaces", () => {
-    mocks.useProjectObservation.mockReturnValue({
-      data: { requests: [] },
-      isLoading: false,
-      isError: false,
-    });
+  it("loads artifacts from project work without material observation", () => {
     renderPage("/projects/project-1/artifacts");
 
-    expect(mocks.useProjectObservation).toHaveBeenCalledWith(
-      "project-1",
-      "/tmp/research",
-      true,
+    expect(mocks.useProjectAgreement).toHaveBeenCalledWith(null);
+    expect(mocks.useProjectSkills).toHaveBeenCalledWith(null);
+    expect(mocks.artifactsProps).toHaveBeenCalledWith(
+      expect.objectContaining({ projectId: "project-1" }),
     );
     expect(screen.getByText("Artifact content")).toBeTruthy();
     expect(screen.getByRole("tab", { name: /Skills/ })).toBeTruthy();
@@ -147,47 +137,50 @@ describe("ProjectsPage", () => {
 
   it("asks the user to choose from the sidebar when no project route is selected", () => {
     renderPage("/projects");
-    expect(mocks.useProjectObservation).toHaveBeenCalledWith(null, null, false);
+    expect(mocks.useProjectAgreement).toHaveBeenCalledWith(null);
+    expect(mocks.useProjectSkills).toHaveBeenCalledWith(null);
     expect(screen.getByText(/Choose a project|选择项目/)).toBeTruthy();
   });
 
-  it("explains the impact before removing a project and returns to project selection", async () => {
-    renderPage("/projects/project-1/overview");
-
-    fireEvent.click(
-      screen.getByRole("button", {
-        name: /Remove from project list|从项目列表移除/,
+  it("loads each fixed project material only on its own tab", () => {
+    mocks.useProjectAgreement.mockReturnValue({
+      data: { path: "AGENTS.md", available: true },
+      isLoading: false,
+      isError: false,
+    });
+    const { unmount } = renderPage("/projects/project-1/agreement");
+    expect(mocks.useProjectAgreement).toHaveBeenCalledWith("project-1");
+    expect(mocks.useProjectSkills).toHaveBeenCalledWith(null);
+    expect(mocks.agreementProps).toHaveBeenCalledWith(
+      expect.objectContaining({
+        agreement: { path: "AGENTS.md", available: true },
       }),
     );
+    unmount();
 
-    await vi.waitFor(() =>
-      expect(mocks.confirm).toHaveBeenCalledWith(
-        expect.objectContaining({
-          description: expect.stringMatching(/local folder|本地目录/),
-          variant: "destructive",
-        }),
-      ),
-    );
-    await vi.waitFor(() =>
-      expect(mocks.removeProject).toHaveBeenCalledWith("project-1"),
-    );
-    await vi.waitFor(() =>
-      expect(screen.getByText(/Choose a project|选择项目/)).toBeTruthy(),
+    mocks.useProjectSkills.mockReturnValue({
+      data: [{ ref: "project:alpha", name: "alpha" }],
+      isLoading: false,
+      isError: false,
+    });
+    renderPage("/projects/project-1/skills");
+    expect(mocks.useProjectAgreement).toHaveBeenLastCalledWith(null);
+    expect(mocks.useProjectSkills).toHaveBeenLastCalledWith("project-1");
+    expect(mocks.skillsProps).toHaveBeenCalledWith(
+      expect.objectContaining({
+        skills: [{ ref: "project:alpha", name: "alpha" }],
+      }),
     );
   });
 
-  it("keeps the project when removal is cancelled", async () => {
-    mocks.confirm.mockResolvedValue(false);
+  it("does not expose project removal as a persistent detail-page action", () => {
     renderPage("/projects/project-1/overview");
 
-    fireEvent.click(
-      screen.getByRole("button", {
+    expect(
+      screen.queryByRole("button", {
         name: /Remove from project list|从项目列表移除/,
       }),
-    );
-
-    await vi.waitFor(() => expect(mocks.confirm).toHaveBeenCalledOnce());
-    expect(mocks.removeProject).not.toHaveBeenCalled();
+    ).toBeNull();
     expect(screen.getByText("Research")).toBeTruthy();
   });
 });

package/src/features/projects/presenters/project-home.presenter.test.ts

@@ -1,17 +1,5 @@
 import { describe, expect, it } from "vitest";
-import type { ObservedRequest, ObservedWorkItem, ObservedWorkflow } from "@nextclaw/client-sdk";
-import {
-  canReplyToProjectRequest,
-  createProjectBoardColumns,
-  getScheduledWorkItems,
-  isProjectHomeTab,
-} from "./project-home.presenter";
-
-const reference = {
-  kind: "ai-report" as const,
-  label: "AI report",
-  observedAt: "2026-08-30T00:00:00.000Z",
-};
+import { isProjectHomeTab } from "./project-home.presenter";
 
 describe("project home presenter", () => {
   it("accepts only the project home tabs represented in the route", () => {
@@ -20,55 +8,4 @@ describe("project home presenter", () => {
     expect(isProjectHomeTab("missing")).toBe(false);
     expect(isProjectHomeTab(undefined)).toBe(false);
   });
-
-  it("projects the same work items into board and scheduled views without inventing facts", () => {
-    const workflows: ObservedWorkflow[] = [{
-      id: "research",
-      label: "Research",
-      stages: [{ id: "draft", label: "Draft" }],
-      reference,
-    }];
-    const workItems: ObservedWorkItem[] = [
-      {
-        id: "with-schedule",
-        name: "Scheduled",
-        status: "active",
-        workflowId: "research",
-        stageId: "draft",
-        schedule: { start: "2026-08-30", end: "2026-09-02", milestone: false, dependsOn: [] },
-        updatedAt: reference.observedAt,
-        reference,
-      },
-      {
-        id: "without-schedule",
-        name: "Unscheduled",
-        status: "blocked",
-        updatedAt: reference.observedAt,
-        reference,
-      },
-    ];
-
-    expect(createProjectBoardColumns(workflows, workItems)[0]?.items.map((item) => item.id))
-      .toEqual(["with-schedule"]);
-    expect(getScheduledWorkItems(workItems).map((item) => item.id))
-      .toEqual(["with-schedule"]);
-  });
-
-  it("offers direct replies only for open confirm-reject requests with a source session", () => {
-    const request: ObservedRequest = {
-      id: "approval",
-      status: "open",
-      response: "confirm-reject",
-      prompt: "Ship?",
-      updatedAt: reference.observedAt,
-      reference: { ...reference, sessionId: "session-1" },
-    };
-    expect(canReplyToProjectRequest(request)).toBe(true);
-    expect(canReplyToProjectRequest({ ...request, reply: {
-      decision: "confirmed",
-      sentAt: reference.observedAt,
-      messageId: "response-1",
-    } })).toBe(false);
-    expect(canReplyToProjectRequest({ ...request, reference })).toBe(false);
-  });
 });

package/src/features/projects/presenters/project-home.presenter.ts

@@ -1,55 +1,18 @@
-import type {
-  ObservedRequest,
-  ObservedWorkItem,
-  ObservedWorkflow,
-  ProjectObservationSnapshot,
-} from "@nextclaw/client-sdk";
-
-export type ProjectHomeTab = "overview" | "work" | "artifacts" | "skills" | "agreement";
-export type ProjectWorkView = "list" | "board" | "gantt";
-
-export type ProjectBoardColumn = {
-  id: string;
-  label: string;
-  items: ObservedWorkItem[];
-};
-
-export function isProjectHomeTab(value: string | undefined): value is ProjectHomeTab {
-  return value === "overview" || value === "work" || value === "artifacts" || value === "skills" || value === "agreement";
-}
-
-export function createProjectBoardColumns(
-  workflows: readonly ObservedWorkflow[],
-  workItems: readonly ObservedWorkItem[],
-): ProjectBoardColumn[] {
-  return workflows.flatMap((workflow) => workflow.stages.map((stage) => ({
-    id: `${workflow.id}:${stage.id}`,
-    label: stage.label,
-    items: workItems.filter((item) => (
-      item.workflowId === workflow.id && item.stageId === stage.id
-    )),
-  })));
-}
-
-export function getUnstagedWorkItems(
-  workItems: readonly ObservedWorkItem[],
-): ObservedWorkItem[] {
-  return workItems.filter((item) => !item.workflowId || !item.stageId);
-}
-
-export function getScheduledWorkItems(
-  workItems: readonly ObservedWorkItem[],
-): ObservedWorkItem[] {
-  return workItems.filter((item) => item.schedule?.start || item.schedule?.end);
-}
-
-export function canReplyToProjectRequest(request: ObservedRequest): boolean {
-  return request.status === "open"
-    && request.response === "confirm-reject"
-    && Boolean(request.reference.sessionId)
-    && !request.reply;
-}
-
-export function getOpenProjectRequests(snapshot: ProjectObservationSnapshot): ObservedRequest[] {
-  return snapshot.requests.filter((request) => request.status === "open");
+export type ProjectHomeTab =
+  | "overview"
+  | "work"
+  | "artifacts"
+  | "skills"
+  | "agreement";
+
+export function isProjectHomeTab(
+  value: string | undefined,
+): value is ProjectHomeTab {
+  return (
+    value === "overview" ||
+    value === "work" ||
+    value === "artifacts" ||
+    value === "skills" ||
+    value === "agreement"
+  );
 }

package/src/features/projects/utils/project-artifact-view.utils.test.ts

@@ -2,55 +2,9 @@ import { describe, expect, it } from "vitest";
 import {
   formatProjectRelativeTime,
   joinProjectPath,
-  sortProjectArtifacts,
 } from "./project-artifact-view.utils";
 
-const artifacts = [
-  {
-    id: "older-update",
-    path: "docs/designs/older.md",
-    categoryId: "designs",
-    categoryLabel: "Designs",
-    exists: true,
-    fileCreatedAt: "2026-08-01T00:00:00.000Z",
-    fileUpdatedAt: "2026-08-28T00:00:00.000Z",
-    references: [],
-  },
-  {
-    id: "newer-update",
-    path: "docs/designs/newer.md",
-    categoryId: "designs",
-    categoryLabel: "Designs",
-    exists: true,
-    fileCreatedAt: "2026-08-29T00:00:00.000Z",
-    fileUpdatedAt: "2026-08-30T00:00:00.000Z",
-    references: [],
-  },
-  {
-    id: "untimed",
-    path: "docs/designs/alpha.md",
-    categoryId: "designs",
-    categoryLabel: "Designs",
-    exists: false,
-    references: [],
-  },
-];
-
 describe("project artifact view", () => {
-  it("prioritizes recent updates by default and supports alternate sort orders", () => {
-    expect(
-      sortProjectArtifacts(artifacts).map((artifact) => artifact.id),
-    ).toEqual(["newer-update", "older-update", "untimed"]);
-    expect(
-      sortProjectArtifacts(artifacts, "created-desc").map(
-        (artifact) => artifact.id,
-      ),
-    ).toEqual(["newer-update", "older-update", "untimed"]);
-    expect(
-      sortProjectArtifacts(artifacts, "name").map((artifact) => artifact.id),
-    ).toEqual(["untimed", "newer-update", "older-update"]);
-  });
-
   it("renders recent timestamps in people-friendly language", () => {
     const now = new Date("2026-08-30T12:00:00.000Z");
     expect(formatProjectRelativeTime("2026-08-30T11:57:00.000Z", now)).toBe(