feat: update claim entity custom UI (GEO-2976)

apps/web/app/space/(entity)/[id]/[entityId]/claim-record-page.tsx

@@ -0,0 +1,30 @@
+import { IdUtils } from '@geoprotocol/geo-sdk/lite';
+
+import { notFound } from 'next/navigation';
+
+import { CLAIM_TYPE_ID } from '~/core/claims/ontology';
+import { ID } from '~/core/id';
+import { isHiddenEntity } from '~/core/moderation/hidden';
+
+import { cachedFetchEntityPage } from './cached-fetch-entity';
+import DefaultEntityPage from './default-entity-page';
+
+export type ClaimRecordPageProps = {
+  params: Promise<{ id: string; entityId: string }>;
+  searchParams: Promise<{ [key: string]: string | string[] | undefined }>;
+};
+
+/** The shared server guard and page shell behind the claim's three record routes. */
+export async function ClaimRecordPage(props: ClaimRecordPageProps) {
+  const params = await props.params;
+  const searchParams = await props.searchParams;
+
+  if (!IdUtils.isValid(params.id) || !IdUtils.isValid(params.entityId)) notFound();
+
+  const result = await cachedFetchEntityPage(params.entityId, params.id);
+  if (isHiddenEntity(result?.entity) || !result?.entity?.types.some(type => ID.equals(type.id, CLAIM_TYPE_ID))) {
+    notFound();
+  }
+
+  return <DefaultEntityPage params={params} searchParams={searchParams} />;
+}

apps/web/app/space/(entity)/[id]/[entityId]/claims/page.tsx

@@ -0,0 +1,5 @@
+import { ClaimRecordPage, type ClaimRecordPageProps } from '../claim-record-page';
+
+export default function ClaimClaimsPage(props: ClaimRecordPageProps) {
+  return <ClaimRecordPage {...props} />;
+}

apps/web/app/space/(entity)/[id]/[entityId]/debates/page.tsx

@@ -0,0 +1,5 @@
+import { ClaimRecordPage, type ClaimRecordPageProps } from '../claim-record-page';
+
+export default function ClaimDebatesPage(props: ClaimRecordPageProps) {
+  return <ClaimRecordPage {...props} />;
+}

apps/web/app/space/(entity)/[id]/[entityId]/layout.tsx

@@ -15,6 +15,7 @@ import { EntityStoreProvider } from '~/core/state/entity-page-store/entity-store
 import { TabEntity } from '~/core/types';
 import { Entity, Relation } from '~/core/types';
 import { Entities } from '~/core/utils/entity';
+import { entityBrowseViewFromTypes } from '~/core/utils/entity-browse-view';
 import { sortRelations } from '~/core/utils/utils';
 
 import { Spacer } from '~/design-system/spacer';
@@ -77,9 +78,9 @@ export default async function ProfileLayout(props: Props) {
 
   const { children } = props;
   const result = await cachedFetchEntityPage(entityId, spaceId);
-  const typeIds = result?.entity?.types.map(t => t.id) ?? [];
+  const entityTypes = result?.entity?.types ?? [];
 
-  if (!typeIds.includes(SystemIds.PERSON_TYPE)) {
+  if (entityBrowseViewFromTypes(entityTypes) !== 'person') {
     return <>{children}</>;
   }
 

apps/web/app/space/(entity)/[id]/[entityId]/page.tsx

@@ -1,4 +1,4 @@
-import { IdUtils, SystemIds } from '@geoprotocol/geo-sdk/lite';
+import { IdUtils } from '@geoprotocol/geo-sdk/lite';
 
 import { notFound } from 'next/navigation';
 
@@ -8,6 +8,7 @@ import { getRecordingUrls } from '~/core/community-calls/recordings';
 import { DebateEntityView } from '~/core/debates/browse/debate-entity-view';
 import { isDebateEntity } from '~/core/debates/is-debate-entity';
 import { isHiddenEntity } from '~/core/moderation/hidden';
+import { entityBrowseViewFromTypes } from '~/core/utils/entity-browse-view';
 import { entityHasOnlyPostType } from '~/core/utils/entity/entities';
 
 import { BountyDetailHeader } from '~/partials/bounties/bounty-detail-header';
@@ -43,7 +44,7 @@ export default async function EntityTemplateStrategy(props: Props) {
     notFound();
   }
 
-  if (result?.entity?.types.map(t => t.id).includes(SystemIds.PERSON_TYPE)) {
+  if (result?.entity && entityBrowseViewFromTypes(result.entity.types) === 'person') {
     return <ProfileEntityServerContainer params={params} searchParams={searchParams} />;
   }
 

apps/web/app/space/(entity)/[id]/[entityId]/profile-layout.test.tsx

@@ -1,10 +1,18 @@
+import { SystemIds } from '@geoprotocol/geo-sdk/lite';
 import '@testing-library/jest-dom/vitest';
 import { cleanup, render } from '@testing-library/react';
 
 import * as React from 'react';
 
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 
+import { CLAIM_TYPE_ID } from '~/core/claims/ontology';
+
+// Statically, below the mocks vitest hoists above it. Importing this graph inside a test body ran
+// it against the 5s test timeout instead, which the route's module tree exceeds under a parallel
+// run — it is the layout, so it pulls in most of the entity page.
+import ProfileLayout from './layout';
+
 // The layout 404s on anything `IdUtils.isValid` rejects, so these are real ids rather than labels.
 /** The route params. The fetched person's id is deliberately a different one. */
 const ROUTE_ENTITY_ID = 'a1b2c3d4e5f6427a8b9c0d1e2f3a4b5c';
@@ -17,14 +25,16 @@ const mocks = vi.hoisted(() => ({
   heading: null as Record<string, unknown> | null,
   storeProvider: null as Record<string, unknown> | null,
   metadataHeader: null as Record<string, unknown> | null,
+  entityTypes: [] as { id: string }[],
 }));
 
-vi.mock('./cached-fetch-entity', async () => {
-  const { SystemIds } = await import('@geoprotocol/geo-sdk/lite');
+vi.mock('./cached-fetch-entity', () => {
   const person = {
     id: 'c3d4e5f6a7b8429c0d1e2f3a4b5c6d7e',
     name: 'Ada Lovelace',
-    types: [{ id: SystemIds.PERSON_TYPE }],
+    get types() {
+      return mocks.entityTypes;
+    },
     relations: [],
     values: [],
     spaces: [],
@@ -88,12 +98,8 @@ vi.mock('~/partials/entity-page/personal-profile-suggested-task-sync', () => ({
   PersonalProfileSuggestedTaskSync: () => null,
 }));
 
-// Statically, below the mocks vitest hoists above it. Importing this graph inside a test body ran
-// it against the 5s test timeout instead, which the route's module tree exceeds under a parallel
-// run — it is the layout, so it pulls in most of the entity page.
-import ProfileLayout from './layout';
-
 beforeEach(() => {
+  mocks.entityTypes = [{ id: SystemIds.PERSON_TYPE }];
   mocks.actions = null;
   mocks.heading = null;
   mocks.storeProvider = null;
@@ -140,4 +146,14 @@ describe('profile ProfileLayout', () => {
     // above is what covers it — the space it scopes by is the one assertable here.
     expect(mocks.metadataHeader).toMatchObject({ spaceId: SPACE_ID });
   });
+
+  it('does not wrap a mixed Claim and Person entity in the profile shell', async () => {
+    mocks.entityTypes = [{ id: SystemIds.PERSON_TYPE }, { id: CLAIM_TYPE_ID }];
+
+    const view = await renderProfileLayout();
+
+    expect(view.getByTestId('page')).toBeInTheDocument();
+    expect(mocks.storeProvider).toBeNull();
+    expect(mocks.actions).toBeNull();
+  });
 });

apps/web/app/space/(entity)/[id]/[entityId]/sources/page.tsx

@@ -0,0 +1,5 @@
+import { ClaimRecordPage, type ClaimRecordPageProps } from '../claim-record-page';
+
+export default function ClaimSourcesPage(props: ClaimRecordPageProps) {
+  return <ClaimRecordPage {...props} />;
+}