feat(platform-web): hide global agreements from the workspace agreements list (#37)

Global service agreements are managed on the admin surface and remain
reachable when attaching to a service or adding a workspace default, so
they no longer belong in a workspace's own /app/:slug/service-agreements
list. Filter them out in workspace scope (admin surface unchanged), drop
the now-constant Scope column, and reword the subtitle.

Author: sidmclaughlin

apps/platform-web/src/components/console/service-agreements/agreements-list.tsx

@@ -21,6 +21,10 @@ export function AgreementsList({ scope }: { scope: AgreementScope }) {
   // Workspace scope may still be resolving (empty id) → gate the query; global (null) is always ready.
   const ready = scope.kind === 'admin' || scope.workspaceId !== '';
   const { data: items = [] } = useQuery({ ...agreementsQueryOptions(workspaceId), enabled: ready });
+  // Workspace scope lists only the workspace's OWN agreements — globals are managed on the admin
+  // surface and reachable when attaching to a service / adding a default, not here (feature 150). The
+  // admin surface (global-only) is unfiltered.
+  const visible = scope.kind === 'admin' ? items : items.filter((item) => !item.isGlobal);
 
   const goNew = () => {
     if (scope.kind === 'workspace') {
@@ -36,7 +40,7 @@ export function AgreementsList({ scope }: { scope: AgreementScope }) {
         <span className="text-sm text-muted-foreground">
           {scope.kind === 'admin'
             ? 'Global consent documents shared across all workspaces.'
-            : 'Consent documents in this workspace (plus global ones).'}
+            : 'Consent documents in this workspace.'}
         </span>
         <Button size="sm" type="button" disabled={!ready} onClick={goNew}>
           <Plus className="size-4" aria-hidden />
@@ -49,18 +53,17 @@ export function AgreementsList({ scope }: { scope: AgreementScope }) {
             <TableRow>
               <TableHead>Title</TableHead>
               <TableHead>Status</TableHead>
-              <TableHead>Scope</TableHead>
             </TableRow>
           </TableHeader>
           <TableBody>
-            {items.length === 0 ? (
+            {visible.length === 0 ? (
               <TableRow>
-                <TableCell colSpan={3} className="py-10 text-center text-muted-foreground">
+                <TableCell colSpan={2} className="py-10 text-center text-muted-foreground">
                   No service agreements yet — create one with the New button.
                 </TableCell>
               </TableRow>
             ) : (
-              items.map((item: ServiceAgreementSummary) => (
+              visible.map((item: ServiceAgreementSummary) => (
                 <TableRow key={item.id}>
                   <TableCell>
                     {scope.kind === 'workspace' ? (
@@ -84,13 +87,6 @@ export function AgreementsList({ scope }: { scope: AgreementScope }) {
                   <TableCell>
                     <Badge color={STATUS_COLOR[item.status]}>{item.status}</Badge>
                   </TableCell>
-                  <TableCell>
-                    {item.isGlobal ? (
-                      <Badge color="grey">Global</Badge>
-                    ) : (
-                      <span className="text-muted-foreground">Workspace</span>
-                    )}
-                  </TableCell>
                 </TableRow>
               ))
             )}

apps/platform-web/test/service-agreements-console.test.tsx

@@ -63,17 +63,14 @@ function withAgreements(base: ReturnType<typeof mockAuth>) {
 }
 
 describe('service agreements console', () => {
-  it('lists a workspace’s agreements (plus global) with scope badges', async () => {
+  it('lists a workspace’s OWN agreements only — globals are hidden here (feature 150)', async () => {
     withAgreements(mockAuth(authedUser, { workspaces: [riverton] }));
     renderApp('/app/riverton/service-agreements');
     expect(
       await screen.findByRole('link', { name: 'Terms of service' }, { timeout: 8000 }),
     ).toBeInTheDocument();
-    // The global agreement is visible too, badged "Global".
-    expect(
-      await screen.findByRole('link', { name: 'Privacy policy' }, { timeout: 8000 }),
-    ).toBeInTheDocument();
-    expect(await screen.findByText('Global', undefined, { timeout: 8000 })).toBeInTheDocument();
+    // The global agreement is filtered out of the workspace list (still attachable / defaultable).
+    expect(screen.queryByRole('link', { name: 'Privacy policy' })).not.toBeInTheDocument();
   });
 
   it('opens the New agreement modal (title + description) at /new', async () => {