feat(web): overhaul shared-skills admin table and sharing UX

Replace the workspace shared-skills card list with a searchable, filterable table (Synced/Manual filter, Added by, Auto toggle), backed by source and creator fields on the management item. Sharing a synced skill now goes through a confirmation dialog warning about owner/member repo visibility, shown only when permission syncing is on. Add cross-navigation between the account and admin skill views, and drop the redundant Make personal menu item in favor of the Shared toggle.

Author: jsourcebot

packages/web/src/app/(app)/settings/skills/page.tsx

@@ -5,19 +5,29 @@ import { ServiceErrorException } from "@/lib/serviceError";
 import { isServiceError } from "@/lib/utils";
 import { authenticatedPage } from "@/middleware/authenticatedPage";
 import { OrgRole } from "@sourcebot/db";
+import { env } from "@sourcebot/shared";
 import { SkillsEntitlementMessage } from "./skillsEntitlementMessage";
 
-export default authenticatedPage(async ({ user, role }) => {
+interface PageProps extends Record<string, unknown> {
+    // `skill` deep-links a specific skill as the initially-selected one (used by
+    // the workspace admin table's skill-name links).
+    searchParams: Promise<{ skill?: string }>;
+}
+
+export default authenticatedPage<PageProps>(async ({ user, role }, { searchParams }) => {
     // Skills are part of Ask Sourcebot. Gate the EE skill UI behind the `ask`
     // entitlement so it never renders on a non-entitled deployment; show the FSL
     // upsell panel instead.
     if (!(await hasEntitlement('ask'))) {
         return <SkillsEntitlementMessage />;
     }
 
-    const [personalSkills, sharedSkills] = await Promise.all([
-        listPersonalAgentSkills(),
-        listSharedAgentSkillCatalog(),
+    const [{ skill: initialSelectedId }, [personalSkills, sharedSkills]] = await Promise.all([
+        searchParams,
+        Promise.all([
+            listPersonalAgentSkills(),
+            listSharedAgentSkillCatalog(),
+        ]),
     ]);
     if (isServiceError(personalSkills)) {
         throw new ServiceErrorException(personalSkills);
@@ -32,6 +42,8 @@ export default authenticatedPage(async ({ user, role }) => {
             initialSharedSkills={sharedSkills}
             currentUserEmail={user.email ?? ""}
             isOwner={role === OrgRole.OWNER}
+            initialSelectedId={initialSelectedId}
+            permissionSyncEnabled={env.PERMISSION_SYNC_ENABLED === 'true'}
         />
     );
 });

packages/web/src/app/(app)/settings/workspaceAskAgent/page.tsx

@@ -13,6 +13,8 @@ interface PageProps extends Record<string, unknown> {
         status?: string;
         server?: string;
         message?: string;
+        // Pre-fills the shared-skills search box (deep link from the account page).
+        skillSearch?: string;
     }>;
 }
 
@@ -34,7 +36,7 @@ export default authenticatedPage<PageProps>(async ({ org, prisma }, { searchPara
         }
     }
 
-    const { status, server, message } = await searchParams;
+    const { status, server, message, skillSearch } = await searchParams;
     const orgSkills = hasAskEntitlement ? await listSharedAgentSkillManagement() : [];
     if (isServiceError(orgSkills)) {
         throw new ServiceErrorException(orgSkills);
@@ -47,6 +49,7 @@ export default authenticatedPage<PageProps>(async ({ org, prisma }, { searchPara
             callbackMessage={message}
             oauthRedirectUrl={getMcpOAuthCallbackUrl()}
             initialOrgSkills={orgSkills}
+            initialSkillSearch={skillSearch}
         />
     );
 }, { minRole: OrgRole.OWNER, redirectTo: '/settings' });

packages/web/src/app/(app)/settings/workspaceAskAgent/workspaceAskAgentPage.test.tsx

@@ -2,6 +2,7 @@ import { act, cleanup, fireEvent, render, screen, waitFor, within } from '@testi
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
 import { afterEach, describe, expect, test, vi } from 'vitest';
 import { TooltipProvider } from '@/components/ui/tooltip';
+import type { ReactNode } from 'react';
 import type { SharedAgentSkillManagementItem } from '@/ee/features/chat/skills/types';
 
 vi.mock('next/navigation', () => ({
@@ -10,6 +11,15 @@ vi.mock('next/navigation', () => ({
     }),
 }));
 
+// The shared-skills manager links skill names via next/link; render a plain anchor.
+vi.mock('next/link', async () => {
+    const { createElement } = await import('react');
+    return {
+        default: ({ href, children }: { href: string; children: ReactNode }) =>
+            createElement('a', { href }, children),
+    };
+});
+
 vi.mock('@/app/api/(client)/client', () => ({
     getMcpConfiguration: vi.fn(),
     getMcpServersWithStatus: vi.fn(),
@@ -81,6 +91,8 @@ describe('WorkspaceAskAgentPage', () => {
             description: 'Review risky changes.',
             enabled: true,
             autoEnrolled: false,
+            source: null,
+            createdByEmail: 'creator@sourcebot.dev',
             createdAt: '2026-06-18T00:00:00.000Z',
             updatedAt: '2026-06-18T00:00:00.000Z',
         };
@@ -93,7 +105,7 @@ describe('WorkspaceAskAgentPage', () => {
         await screen.findByText('Ask Sourcebot connectors are unavailable');
 
         expect(screen.queryByText('Review')).toBeNull();
-        expect(screen.queryByText('Manage shared skills available to everyone in your workspace.')).toBeNull();
+        expect(screen.queryByText('Available to everyone in your workspace.')).toBeNull();
     });
 
     test('disables shared skill delete while a flag update is pending', async () => {
@@ -105,6 +117,8 @@ describe('WorkspaceAskAgentPage', () => {
             description: 'Review risky changes.',
             enabled: true,
             autoEnrolled: false,
+            source: null,
+            createdByEmail: 'creator@sourcebot.dev',
             createdAt: '2026-06-18T00:00:00.000Z',
             updatedAt: '2026-06-18T00:00:00.000Z',
         };
@@ -133,6 +147,8 @@ describe('WorkspaceAskAgentPage', () => {
             description: 'Review risky changes.',
             enabled: true,
             autoEnrolled: false,
+            source: null,
+            createdByEmail: 'creator@sourcebot.dev',
             createdAt: '2026-06-18T00:00:00.000Z',
             updatedAt: '2026-06-18T00:00:00.000Z',
         };
@@ -144,6 +160,8 @@ describe('WorkspaceAskAgentPage', () => {
             description: 'Review risky changes.',
             enabled: true,
             autoEnrolled: false,
+            source: null,
+            createdByEmail: 'creator@sourcebot.dev',
             createdAt: '2026-06-18T00:00:00.000Z',
             updatedAt: '2026-06-18T00:00:00.000Z',
         };
@@ -185,6 +203,8 @@ describe('WorkspaceAskAgentPage', () => {
             description: 'Review risky changes.',
             enabled: true,
             autoEnrolled: false,
+            source: null,
+            createdByEmail: 'creator@sourcebot.dev',
             createdAt: '2026-06-18T00:00:00.000Z',
             updatedAt: '2026-06-18T00:00:00.000Z',
         };
@@ -196,6 +216,8 @@ describe('WorkspaceAskAgentPage', () => {
             description: 'Review risky changes.',
             enabled: true,
             autoEnrolled: false,
+            source: null,
+            createdByEmail: 'creator@sourcebot.dev',
             createdAt: '2026-06-18T00:00:00.000Z',
             updatedAt: '2026-06-18T00:00:00.000Z',
         };