Merge branch 'develop' into fix/fast-first-action-ack-21igvnc1lvzji

Author: daniel-lxs

.changeset/fast-settings-skill-inventory.md

@@ -0,0 +1,5 @@
+---
+"@roomote/cloud-agents": patch
+---
+
+Include authorized Settings skills in Fast mode's unscoped skill inventory.

apps/docs/skills.mdx

@@ -15,6 +15,11 @@ a packaged or Settings skill. Roomote resolves Settings skills only from
 environments you can use. If different environments define different versions
 with the same name, Roomote asks which environment you mean.
 
+When you ask Fast which skills are available, its unscoped inventory includes
+packaged skills and Settings skills from every environment you can use.
+Repository-defined skills remain scoped to a specific environment or
+repository.
+
 Use Skills when the same guidance keeps showing up across tasks in an
 environment: a framework-specific review checklist, a release process, a
 design-system rule, a data workflow, or a repeatable debugging path.

apps/web/src/app/(authenticated)/sessions/SessionCard.client.test.tsx

@@ -2,14 +2,18 @@ import Link from 'next/link';
 import { formatDistanceToNow } from 'date-fns';
 
 import { formatInferenceCost, getUserDisplayName } from '@/lib';
+import { formatAutomationLabel } from '@/lib/task-creator-filter';
 import { Avatar, Spinner } from '@/components/system';
+import { PullRequestBadge } from '@/components/sandbox';
 import { SessionStatusBadge } from '@/components/sessions/SessionStatusBadge';
 import { SessionSearchSnippet } from '@/components/sessions/SessionSearchSnippet';
 import { getSessionSurfaceLabel } from '@/components/sessions/session-surfaces';
 
 type SessionCardData = {
   id: string;
   title: string;
+  ownerKind: 'user' | 'automation' | 'system';
+  ownerAutomation: string | null;
   ownerName: string | null;
   ownerEmail: string | null;
   ownerImageUrl: string | null;
@@ -21,6 +25,11 @@ type SessionCardData = {
   inferenceCostMicroUsd: number;
   unread: boolean;
   searchSnippet?: string | null;
+  pullRequests: Array<{
+    repository: string;
+    number: number;
+    url: string;
+  }>;
   tasks: Array<{
     taskId: string;
     workflow: string;
@@ -37,25 +46,30 @@ export function SessionCard({
   viewerUserId: string;
   query?: string;
 }) {
-  const owner =
-    getUserDisplayName({
-      name: session.ownerName,
-      email: session.ownerEmail,
-    }) ?? 'Roomote';
+  const actorName =
+    session.ownerKind === 'automation' && session.ownerAutomation
+      ? formatAutomationLabel(session.ownerAutomation)
+      : (getUserDisplayName({
+          name: session.ownerName,
+          email: session.ownerEmail,
+        }) ?? 'Roomote');
   const status = session.cachedStatus ?? 'ready';
 
   return (
-    <Link
-      href={`/sessions/${session.id}`}
-      className="ph-no-capture group flex w-full items-start gap-3 p-4 transition-colors hover:bg-accent-foreground/10 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring"
-    >
-      <div className="relative mt-1 shrink-0">
+    <div className="ph-no-capture group relative flex w-full items-start gap-3 p-4 transition-colors hover:bg-accent-foreground/10">
+      <Link
+        href={`/sessions/${session.id}`}
+        className="absolute inset-0 z-0 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring"
+      >
+        <span className="sr-only">{session.title}</span>
+      </Link>
+      <div className="pointer-events-none relative z-10 mt-1 shrink-0">
         <Avatar
           imageUrl={session.ownerImageUrl}
-          name={owner}
+          name={actorName}
           email={session.ownerEmail ?? undefined}
           size="md"
-          alt={owner}
+          alt={actorName}
         />
         {session.unread && session.ownerUserId === viewerUserId ? (
           <span
@@ -64,17 +78,21 @@ export function SessionCard({
           />
         ) : null}
       </div>
-      <div className="min-w-0 flex-1 space-y-2">
-        <div className="flex flex-wrap items-center justify-between gap-2">
-          <p className="line-clamp-2 text-base font-medium group-hover:underline">
-            {session.title}
-          </p>
+      <div className="pointer-events-none relative z-10 min-w-0 flex-1">
+        <div className="flex items-start justify-between gap-2 text-xs text-muted-foreground/75 md:items-center">
+          <div className="flex flex-wrap items-center gap-1 text-nowrap">
+            <span>{actorName}</span>
+            <span>started a session</span>
+          </div>
           <span className="shrink-0 text-xs text-muted-foreground">
             {formatDistanceToNow(new Date(session.activityAt * 1000), {
               addSuffix: true,
             })}
           </span>
         </div>
+        <p className="mt-1 mb-2 line-clamp-2 text-base font-medium group-hover:underline">
+          {session.title}
+        </p>
         <SessionSearchSnippet
           snippet={session.searchSnippet}
           query={query}
@@ -90,11 +108,21 @@ export function SessionCard({
             <SessionStatusBadge status={status} className="capitalize" />
           )}
           <span>{getSessionSurfaceLabel(session.sourceSurface)}</span>
+          {session.pullRequests.map((pullRequest) => (
+            <PullRequestBadge
+              key={`${pullRequest.repository}:${pullRequest.number}`}
+              repo={pullRequest.repository}
+              prNumber={pullRequest.number}
+              url={pullRequest.url}
+              className="pointer-events-auto"
+              iconClassName="size-3"
+            />
+          ))}
           {session.inferenceCostMicroUsd > 0 ? (
             <span>${formatInferenceCost(session.inferenceCostMicroUsd)}</span>
           ) : null}
         </div>
       </div>
-    </Link>
+    </div>
   );
 }

apps/web/src/app/(authenticated)/sessions/SessionCard.tsx

@@ -37,7 +37,6 @@ const ADVANCED_FILTERS_STORAGE_KEY =
   'roomote-sessions-advanced-filters-visible';
 const SESSIONS_VIEW_STORAGE_KEY = 'roomote-sessions-view';
 const ADVANCED_FILTER_PARAMS = [
-  'user',
   'repository',
   'pullRequest',
   'model',
@@ -244,12 +243,14 @@ export function SessionsFilters({
         }
       />
       <TaskFilters
-        userId="all"
+        userId={userId ?? 'all'}
         repositoryName={null}
         pullRequest={null}
         model={null}
         timePeriod={timePeriod}
-        onUserChange={() => undefined}
+        onUserChange={(value) =>
+          updateNullableParam('user', value && value !== 'all' ? value : null)
+        }
         onRepositoryChange={() => undefined}
         onPullRequestChange={() => undefined}
         onModelChange={() => undefined}
@@ -259,7 +260,6 @@ export function SessionsFilters({
             else params.set('period', String(period));
           })
         }
-        showUser={false}
         showRepository={false}
         showPullRequest={false}
         showModel={false}
@@ -288,6 +288,7 @@ export function SessionsFilters({
             }
             onModelChange={(value) => updateNullableParam('model', value)}
             onTimePeriodChange={() => undefined}
+            showUser={false}
             showTimePeriod={false}
             showTaskType={false}
           />