@flamingo-stack/openframe-frontend-core 0.0.565-1861.5100.1 → 0.0.565

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flamingo-stack/openframe-frontend-core",
-  "version": "0.0.565-1861.5100.1",
+  "version": "0.0.565",
   "description": "Shared design system and components for all Flamingo platforms",
   "type": "module",
   "main": "./dist/index.js",

package/src/components/.comment-card.md

@@ -1,18 +1,18 @@
 <!-- source-hash: 2dc22952dfcc6e283cfc3b6d4d636ade -->
-Renders a styled comment card that adapts its header to the surface: it leads with the vendor a comment is about on a user profile, or with the commenter everywhere else. Delete is a caller-supplied rule.
+Renders a styled comment card that adapts its layout based on context — showing vendor information when viewing a user profile, or user information when viewing a vendor page — with optional delete functionality based on auth state.
 
 ## Key Components
 
 ### `CommentCard` (default export)
 
-The comment tile: one card for every comment surface, with the header lead, the body length and deletability all supplied as props.
+A context-aware card component for displaying user-submitted vendor comments.
 
 **Props (`CommentCardProps`):**
 
 | Prop | Type | Default | Description |
 |------|------|---------|-------------|
 | `comment` | `object` | — | Comment data including content, author, vendor, and metadata |
-| `leadWith` | `'vendor' \| 'commenter'` | `'commenter'` | Which header the tile leads with: the vendor the comment is about (profile page) or the person who wrote it |
+| `context` | `'profile' \| 'vendor'` | `'profile'` | Controls layout and delete permission logic |
 | `onViewProduct` | `(slug: string) => void` | — | Callback when vendor button is clicked |
 | `onDeleteComment` | `(id: string) => void` | — | Callback when delete is triggered |
 | `showVendorInfo` | `boolean` | `true` | Toggles vendor info display |
@@ -20,13 +20,14 @@ The comment tile: one card for every comment surface, with the header lead, the
 
 **Internal logic:**
 
-- `canUserDeleteComment()` — Deletability is the CALLER's rule: an explicit `comment.canDelete` wins, and the default is "your own comment" (`currentUser.id === comment.user.id`). No per-surface branch.
+- `canUserDeleteComment()` — Grants delete access to authenticated users; in `profile` context, respects the `canDelete` prop; in `vendor` context, compares `currentUser.id` against `comment.user.id`
 - `formatActivityTime()` — Delegates to `formatRelativeTime` utility for human-readable timestamps
 
 ## Usage Example
 
 ```typescript
 <CommentCard
+  context="vendor"
   comment={{
     id: "abc123",
     content: "Solid RMM tool for small MSPs.",

package/src/components/chat/__tests__/ticket-event.test.ts

@@ -134,6 +134,21 @@ describe('chat stream reducer — ticket event', () => {
     ])
     expect(events(trailingSegments(reducer))).toHaveLength(3)
   })
+
+  it('stamps a live card with its arrival time, not the bubble timestamp', () => {
+    // The chunk carries no event time, so arrival IS the event time for a
+    // genuinely live receipt. Without the stamp the card renders the trailing
+    // bubble's timestamp — the turn's FIRST row, minutes stale by the time a
+    // resolve/reopen lands.
+    const reducer = createChatStreamReducer({ transport: 'nats' })
+    const before = Date.now()
+    feed(reducer, [resolvedChunk(1)])
+    const after = Date.now()
+    const [card] = events(trailingSegments(reducer))
+    expect(card.occurredAt).toBeInstanceOf(Date)
+    expect(card.occurredAt!.getTime()).toBeGreaterThanOrEqual(before)
+    expect(card.occurredAt!.getTime()).toBeLessThanOrEqual(after)
+  })
 })
 
 describe('accumulator — hydrate/catch-up overlap', () => {
@@ -262,4 +277,75 @@ describe('processHistoricalMessages — ticket event', () => {
     ])
     expect(messages.flatMap((m) => events(m.content))).toHaveLength(0)
   })
+
+  it('stamps each card with its OWN row time, not the bubble timestamp', () => {
+    // The reported bug: resolve/reopen rows grouped into one assistant bubble
+    // all rendered the bubble's timestamp — the FIRST row of the turn (the
+    // ticket-creation-era greeting), so every lifecycle card read the same
+    // stale time.
+    const rowAt = (id: string, createdAt: string, data: Record<string, unknown>): HistoricalMessage => ({
+      id,
+      createdAt,
+      owner: { type: 'ASSISTANT' },
+      messageData: [data as never],
+    })
+    const { messages } = processHistoricalMessages([
+      rowAt('m1', '2026-08-20T13:54:00Z', { type: 'TEXT', text: 'Hi! How can I help?' }),
+      rowAt('m2', '2026-08-20T14:10:00Z', { type: 'TICKET_EVENT', kind: 'RESOLVED', actorType: 'AI' }),
+      rowAt('m3', '2026-08-20T14:33:00Z', {
+        type: 'TICKET_EVENT',
+        kind: 'REOPENED',
+        actorType: 'CLIENT',
+        targetStatusKind: 'AI_ASSISTANCE',
+      }),
+    ])
+    expect(messages).toHaveLength(1)
+    // The bubble keeps the first row's time…
+    expect(messages[0].timestamp).toEqual(new Date('2026-08-20T13:54:00Z'))
+    // …but each card carries the time of its own event.
+    const cards = events(messages[0].content)
+    expect(cards.map((c) => c.occurredAt)).toEqual([
+      new Date('2026-08-20T14:10:00Z'),
+      new Date('2026-08-20T14:33:00Z'),
+    ])
+  })
+})
+
+describe('accumulator — occurredAt across the hydrate/catch-up overlap', () => {
+  it('keeps the hydrated row time when the seq-stamped redelivery carries none', () => {
+    const accumulator = new MessageSegmentAccumulator()
+    accumulator.addTicketEvent(
+      { kind: 'RESOLVED', actorId: 'fae', actorName: 'Fae', actorType: 'AI' },
+      undefined,
+      new Date('2026-08-20T14:10:00Z'),
+    )
+    // Same event redelivered live (JetStream catch-up): seq known, time not.
+    const segments = accumulator.addTicketEvent(
+      { kind: 'RESOLVED', actorId: 'fae', actorName: 'Fae', actorType: 'AI' },
+      412,
+    )
+    const rendered = events(segments)
+    expect(rendered).toHaveLength(1)
+    expect(rendered[0].streamSeq).toBe(412)
+    expect(rendered[0].occurredAt).toEqual(new Date('2026-08-20T14:10:00Z'))
+  })
+
+  it('keeps the hydrated row time over a LATE redelivery arrival stamp', () => {
+    // Reconnect catch-up redelivers an hour-old event stamped with its (late)
+    // arrival time — the hydrated twin's real row time must win.
+    const accumulator = new MessageSegmentAccumulator()
+    accumulator.addTicketEvent(
+      { kind: 'RESOLVED', actorId: 'fae', actorName: 'Fae', actorType: 'AI' },
+      undefined,
+      new Date('2026-08-20T14:10:00Z'),
+    )
+    const segments = accumulator.addTicketEvent(
+      { kind: 'RESOLVED', actorId: 'fae', actorName: 'Fae', actorType: 'AI' },
+      412,
+      new Date('2026-08-20T15:10:00Z'),
+    )
+    const rendered = events(segments)
+    expect(rendered).toHaveLength(1)
+    expect(rendered[0].occurredAt).toEqual(new Date('2026-08-20T14:10:00Z'))
+  })
 })

package/src/components/chat/chat-message-enhanced.tsx

@@ -652,7 +652,9 @@ const ChatMessageEnhanced = forwardRef<HTMLDivElement, ChatMessageEnhancedProps>
                 } else if (segment.type === 'ticket_escalated') {
                   return <TicketEscalatedMessage key={index} data={segment.data} timestamp={timestamp} />
                 } else if (segment.type === 'ticket_event') {
-                  return <TicketEventMessage key={index} data={segment.data} timestamp={timestamp} />
+                  // The card's own event time; the bubble timestamp is the
+                  // turn's FIRST row and lags every later lifecycle event.
+                  return <TicketEventMessage key={index} data={segment.data} timestamp={segment.occurredAt ?? timestamp} />
                 } else if (segment.type === 'error') {
                   return (
                     <ErrorMessageDisplay

package/src/components/chat/entity-cards/employee-entry-card.tsx

@@ -38,12 +38,6 @@ const STATUS_BADGE_CLASS: Record<string, string> = {
   published: 'bg-ods-success-secondary text-ods-success',
   draft: 'bg-ods-warning-secondary text-ods-warning',
   archived: 'bg-ods-border text-ods-text-secondary',
-  // Design-doc lifecycle (product-hub) — same ODS pairs, no new palette.
-  in_review: 'bg-ods-warning-secondary text-ods-warning',
-  approved: 'bg-ods-success-secondary text-ods-success',
-  building: 'bg-ods-success-secondary text-ods-success',
-  shipped: 'bg-ods-success-secondary text-ods-success',
-  abandoned: 'bg-ods-border text-ods-text-secondary',
 }
 
 /** Shared badge treatment so an entity's own badges (discipline, level) sit

package/src/components/chat/stream/chat-stream-reducer.ts

@@ -1378,7 +1378,13 @@ export function createChatStreamReducer(
           reason: event.reason,
           targetStatusKind: event.targetStatusKind,
         }
-        const segments = accumulator.addTicketEvent(data, seq)
+        // The chunk carries no event time, so stamp arrival — for a genuinely
+        // live event that IS the event time. Without it the card falls back to
+        // the enclosing bubble's timestamp: the turn's FIRST row, minutes
+        // stale by the time a resolve/reopen lands. A catch-up redelivery of
+        // an already-hydrated event does NOT regress to arrival time —
+        // `addTicketEvent` keeps the first-known `occurredAt` on upsert.
+        const segments = accumulator.addTicketEvent(data, seq, new Date())
         applyAccumulated(before, segments)
         // The card alone is not enough for a host that also tracks the
         // ticket's state: a RESOLVED/REOPENED arriving live must move the

package/src/components/chat/types/message.types.ts

@@ -359,6 +359,14 @@ export type TicketEventSegment = {
   type: 'ticket_event'
   data: TicketEventData
   streamSeq?: number
+  /** When THIS event happened (history: the persisted row's `createdAt`;
+   *  live: chunk arrival). Kept beside `streamSeq`, not in `data`: the
+   *  upsert's payload-equality fallback must keep matching a redelivered
+   *  chunk to its hydrated twin, and their times come from different clocks.
+   *  Without it the card inherits the surrounding assistant bubble's
+   *  timestamp — the FIRST row of the whole turn, so every lifecycle card
+   *  reads the same stale time. */
+  occurredAt?: Date
 }
 
 export type ErrorSegment = {

package/src/components/chat/utils/message-segment-accumulator.ts

@@ -550,11 +550,12 @@ export class MessageSegmentAccumulator {
    * by the same actor is payload-identical to the first resolve, and matching
    * the old card meant the final one never rendered.
    */
-  addTicketEvent(data: TicketEventData, streamSeq?: number): MessageSegment[] {
+  addTicketEvent(data: TicketEventData, streamSeq?: number, occurredAt?: Date): MessageSegment[] {
     const segment: TicketEventSegment = {
       type: 'ticket_event',
       data,
       ...(streamSeq !== undefined ? { streamSeq } : {}),
+      ...(occurredAt !== undefined ? { occurredAt } : {}),
     }
     let existingIndex =
       streamSeq !== undefined
@@ -581,7 +582,13 @@ export class MessageSegmentAccumulator {
       }
     }
     if (existingIndex !== -1) {
-      this.segments[existingIndex] = segment
+      // The FIRST-known time wins: the hydrated twin carries the row's real
+      // `createdAt`, while a catch-up redelivery only knows its (late) arrival
+      // time — replacing wholesale would regress the card to arrival or, with
+      // neither side stamped, to the bubble timestamp.
+      const existing = this.segments[existingIndex] as TicketEventSegment
+      const occurredAt = existing.occurredAt ?? segment.occurredAt
+      this.segments[existingIndex] = occurredAt !== undefined ? { ...segment, occurredAt } : segment
       return this.getSegments()
     }
     this.segments.push(segment)
@@ -734,7 +741,7 @@ export class MessageSegmentAccumulator {
           this.addTicketEscalated(segment.data)
           break
         case 'ticket_event':
-          this.addTicketEvent(segment.data, segment.streamSeq)
+          this.addTicketEvent(segment.data, segment.streamSeq, segment.occurredAt)
           break
         case 'error':
           this.addError(segment.title, segment.details)

package/src/components/chat/utils/process-historical-messages.ts

@@ -330,6 +330,7 @@ function applyHistoryEvent(
   options: MessageProcessingOptions,
   escalatedApprovals?: EscalatedApprovals,
   offerResolutions?: OfferResolutions,
+  rowCreatedAt?: Date,
 ): void {
   // batchApprovalsEnabled is owned by the consumer (oss-tenant chat client /
   // openframe-frontend tickets). Defaults to ON so consumers that haven't
@@ -362,14 +363,21 @@ function applyHistoryEvent(
     // same event matches this segment by payload equality (see
     // `addTicketEvent`).
     case 'ticket-event':
-      accumulator.addTicketEvent({
-        kind: event.kind,
-        actorId: event.actorId,
-        actorName: event.actorName,
-        actorType: event.actorType,
-        reason: event.reason,
-        targetStatusKind: event.targetStatusKind,
-      })
+      // `rowCreatedAt` is the persisted row's own time — without it the card
+      // renders the enclosing assistant bubble's timestamp, i.e. the FIRST
+      // row of the turn, and every lifecycle card reads the same stale time.
+      accumulator.addTicketEvent(
+        {
+          kind: event.kind,
+          actorId: event.actorId,
+          actorName: event.actorName,
+          actorType: event.actorType,
+          reason: event.reason,
+          targetStatusKind: event.targetStatusKind,
+        },
+        undefined,
+        rowCreatedAt,
+      )
       break
 
     case 'escalation-offer-resolved':
@@ -738,6 +746,7 @@ export function processHistoricalMessages(
           { displayApprovalTypes, batchApprovalsEnabled, escalationOfferStates },
           escalatedApprovals,
           offerResolutions,
+          new Date(msg.createdAt),
         )
       })
 

package/src/components/comment-card.tsx

@@ -37,23 +37,7 @@ interface CommentCardProps {
   onDeleteComment?: (commentId: string) => void
   showVendorInfo?: boolean
   compact?: boolean
-  /**
-   * Which HEADER this tile leads with — the only thing that ever varied by
-   * surface. Two states, because there are two headers: the profile page shows
-   * a person's comments, so it leads with the VENDOR each one is about;
-   * everywhere else the vendor is the page and the commenter is the news.
-   * Every BEHAVIOUR is a prop, so a new surface never needs a new member here.
-   */
-  leadWith?: 'vendor' | 'commenter'
-  /**
-   * Read the body in FULL (thread reading) instead of the fixed-height,
-   * three-line list tile. Defaults to the list tile.
-   */
-  fullBody?: boolean
-  /** Rendered under the body — badges + per-surface actions (design-doc threads). */
-  footer?: React.ReactNode
-  /** Rendered after the card — nested replies + reply composer (design-doc threads). */
-  children?: React.ReactNode
+  context: 'profile' | 'vendor'
 }
 
 export function CommentCard({ 
@@ -62,10 +46,7 @@ export function CommentCard({
   onDeleteComment, 
   showVendorInfo = true,
   compact = false,
-  leadWith = 'commenter',
-  fullBody = false,
-  footer,
-  children
+  context = 'profile'
 }: CommentCardProps) {
   const { user: currentUser } = useAuth()
   
@@ -75,28 +56,32 @@ export function CommentCard({
   }
 
   // Check if current user can delete this comment
-  /**
-   * Deletability is the CALLER's rule, never a per-surface branch in here: an
-   * explicit `comment.canDelete` always wins (design docs compute own-or-DRI
-   * with their gate module and hand it in), and the default is the only rule
-   * this component can honestly know by itself — your own comment.
-   */
   const canUserDeleteComment = () => {
+    // If not authenticated, can't delete
     if (!currentUser) return false
-    if (typeof comment.canDelete === 'boolean') return comment.canDelete
-    return !!comment.user && currentUser.id === comment.user.id
+    
+    // For profile context, use the existing canDelete prop (since it's the user's own comments)
+    if (context === 'profile') {
+      return comment.canDelete !== false
+    }
+    
+    // For vendor context, check if current user is the comment creator
+    if (context === 'vendor' && comment.user) {
+      return currentUser.id === comment.user.id
+    }
+    
+    return false
   }
 
   const showDeleteButton = canUserDeleteComment()
 
   return (
-    <>
     <div className="bg-ods-card border border-ods-border rounded-lg p-4 hover:border-ods-accent transition-all group overflow-hidden w-full max-w-full box-border" style={{ maxWidth: '100%', wordBreak: 'break-word' }}>
       {/* Comment Header */}
       <div className="flex flex-col gap-3 mb-3 min-[420px]:flex-row min-[420px]:items-center min-[420px]:justify-between min-[420px]:gap-2 w-full">
         {/* Row 1: Info + Timestamp */}
         <div className="flex items-center gap-2 min-w-0 flex-1">
-          {leadWith === 'vendor' && comment.vendor ? (
+          {context === 'profile' && comment.vendor ? (
             <>
               {/* Vendor Button - Icon + Name */}
               <VendorDisplayButton
@@ -109,7 +94,7 @@ export function CommentCard({
                 {formatActivityTime(comment.createdAt)}
               </span>
             </>
-          ) : comment.user ? (
+          ) : context === 'vendor' && comment.user ? (
             <UserSummary
               name={comment.user.name}
               email=""
@@ -167,12 +152,6 @@ export function CommentCard({
             </h4>
           </div>
         )}
-        {fullBody ? (
-          // Thread bodies are read in full — no fixed height, no clamp.
-          <p className="text-h4 text-ods-text-primary whitespace-pre-wrap" style={{ wordBreak: 'break-word', overflowWrap: 'break-word', maxWidth: '100%' }}>
-            {comment.content}
-          </p>
-        ) : (
         <div className={compact ? "h-[60px] flex items-center" : "h-[72px] flex items-center"}>
           <p className="text-h4 text-ods-text-primary line-clamp-3" style={{ 
             wordBreak: 'break-word', 
@@ -187,11 +166,7 @@ export function CommentCard({
             {comment.content}
           </p>
         </div>
-        )}
-        {footer ? <div className="pt-2 flex flex-wrap items-center gap-2">{footer}</div> : null}
       </div>
     </div>
-    {children}
-    </>
   )
 }

package/src/components/embeds/index.ts

@@ -11,8 +11,6 @@ export type { GoogleSheetsViewerProps } from './google-sheets-viewer'
 
 export { FigmaEmbed } from './figma-embed'
 export type { FigmaEmbedProps } from './figma-embed'
-export { ClaudeEmbed } from './claude-embed'
-export type { ClaudeEmbedProps, ClaudeEmbedKind } from './claude-embed'
 
 export { OGLinkPreview, OGLinkErrorBoundary } from './og-link-preview'
 export type {

package/src/components/features/array-entry-manager.tsx

@@ -159,8 +159,7 @@ export function ArrayEntryManager<T extends { [key: string]: any }>({
             </div>
           )}
 
-          {/* min-w-0: a wide renderLabel (e.g. a DeliveryRow) must truncate, not push the row past its host. */}
-          <div className="flex-1 min-w-0 space-y-2">
+          <div className="flex-1 space-y-2">
             {renderLabel && renderLabel(item, index)}
             <Input
               placeholder={placeholder}

package/src/components/features/clickup-tasks-manager.tsx

@@ -14,19 +14,13 @@ interface ClickUpTasksManagerProps {
   onChange: (tasks: ClickUpTaskLink[]) => void;
   title?: ReactNode; // Support string or ReactNode for badge integration
   className?: string;
-  /**
-   * Per-row label renderer (passthrough to ArrayEntryManager) — lets a host
-   * hydrate the bare task id with the mirror's title/status row.
-   */
-  renderLabel?: (item: ClickUpTaskLink, index: number) => ReactNode;
 }
 
 export function ClickUpTasksManager({
   tasks,
   onChange,
   title = 'ClickUp Roadmap Tasks', // Default title
-  className = '',
-  renderLabel
+  className = ''
 }: ClickUpTasksManagerProps) {
   return (
     <ArrayEntryManager
@@ -39,7 +33,6 @@ export function ClickUpTasksManager({
       addButtonText="Add Task"
       icon={<Lightbulb className="w-5 h-5 text-ods-text-secondary" />}
       className={className}
-      renderLabel={renderLabel}
     />
   );
 }

package/src/components/index.ts

@@ -16,7 +16,6 @@ export * from './hover-card'
 export * from './categories-cart'
 export * from './category-card'
 export * from './comment-card'
-export * from './comment-composer'
 export * from './content-loading-container'
 export * from './dynamic-skeleton'
 export * from './empty-state'

package/src/components/shared/delivery/delivery-row.tsx

@@ -68,17 +68,12 @@ export interface DeliveryRowProps {
    *  paired with `scroll-mt-24` on the outer element so the row lands
    *  BELOW the sticky chrome after the scroll. */
   id?: string
-  /** `'_blank'` renders a plain external anchor (new tab, `rel` hardened) —
-   *  for rows whose href leaves the app (e.g. the raw ClickUp task URL on the
-   *  design-doc surfaces). Internal deep-links omit it and keep the Link shim. */
-  target?: '_blank'
   className?: string
 }
 
 export function DeliveryRow({
   item,
   href,
-  target,
   caption,
   id,
   className,
@@ -105,10 +100,7 @@ export function DeliveryRow({
           </h3>
         </div>
         <div className="min-h-[20px] flex items-center">
-          {/* Relative time is wall-clock-derived: on an SSR host the server and
-              client render different strings a moment apart — text-only, so
-              suppress instead of forcing a client-only render. */}
-          <p className="text-h5 text-ods-text-secondary uppercase tracking-[-0.28px] truncate" suppressHydrationWarning>
+          <p className="text-h5 text-ods-text-secondary uppercase tracking-[-0.28px] truncate">
             {subtitle}
           </p>
         </div>
@@ -163,14 +155,6 @@ export function DeliveryRow({
     className,
   )
 
-  if (href && target === '_blank') {
-    return (
-      <a href={href} id={id} className={baseClass} target="_blank" rel="noopener noreferrer">
-        {inner}
-      </a>
-    )
-  }
-
   if (href) {
     // `Link` is the env-aware embed-shim — delegates to `next/link` on
     // a Next.js host (soft RSC nav, back-button restores the previous

package/src/components/shared/roadmap/roadmap-grid.tsx

@@ -100,15 +100,6 @@ export interface RoadmapGridProps {
   /** Show the desktop left margin (~120px) that aligns the grid with the page
    *  hero. Default `true`. Related-content rails pass `false`. */
   showLeftMargin?: boolean;
-  /**
-   * Columns at `md` and up. Default `2` (the roadmap page).
-   *
-   * A card in this grid needs real width — its title, status badge, target
-   * version and vote row sit on one line each. Two columns inside a NARROW
-   * host (a modal rail, a sidebar) halve an already-small container and crush
-   * every one of them, so such hosts pass `1`.
-   */
-  columns?: 1 | 2;
   /** URL builder for the per-task refresh call after a successful vote. */
   buildRefreshUrl?: (taskId: string) => string;
   /** Voting hook options (vote endpoint + storage key). */
@@ -131,26 +122,18 @@ export interface RoadmapGridProps {
 function RoadmapGridSingle({
   items,
   showLeftMargin,
-  columns = 2,
   getVote,
   onVote,
   votingTasks,
 }: {
   items: RoadmapItem[];
   showLeftMargin: boolean;
-  columns?: 1 | 2;
   getVote: (taskId: string) => 'up' | 'down' | null;
   onVote: (taskId: string, voteType: 'up' | 'down') => void;
   votingTasks: Set<string>;
 }) {
   return (
-    // LITERAL class strings: an interpolated `md:grid-cols-${n}` is invisible
-    // to Tailwind's scanner and is never generated.
-    <div
-      className={`grid grid-cols-1 gap-6 ${columns === 2 ? 'md:grid-cols-2' : ''} ${
-        showLeftMargin ? 'md:ml-[120px]' : ''
-      }`}
-    >
+    <div className={`grid grid-cols-1 md:grid-cols-2 gap-6 ${showLeftMargin ? 'md:ml-[120px]' : ''}`}>
       {items.map((item) => (
         // DOM id + sticky-header scroll offset live ON RoadmapCard's own
         // outer element (no wrapper div). Anchor mirrors
@@ -174,7 +157,6 @@ export function RoadmapGrid({
   items,
   onItemUpdate,
   showLeftMargin = true,
-  columns = 2,
   buildRefreshUrl = DEFAULT_BUILD_REFRESH_URL,
   votingOptions,
   groupByQuarter = false,
@@ -273,7 +255,6 @@ export function RoadmapGrid({
       <RoadmapGridSingle
         items={items}
         showLeftMargin={showLeftMargin}
-        columns={columns}
         getVote={getVote}
         onVote={handleVote}
         votingTasks={votingTasks}
@@ -324,7 +305,6 @@ export function RoadmapGrid({
               <RoadmapGridSingle
                 items={itemsByQuarter[quarter]}
                 showLeftMargin={showLeftMargin}
-                columns={columns}
                 getVote={getVote}
                 onVote={handleVote}
                 votingTasks={votingTasks}

package/src/components/ui/card.tsx

@@ -11,10 +11,7 @@ const Card = React.forwardRef<
   <div
     ref={ref}
     className={cn(
-      // `border` with NO color token resolves to `currentColor` — on a dark card
-      // with light text that paints a WHITE outline. Every ODS surface borders
-      // on `--color-border`; say so.
-      "rounded-lg border border-ods-border bg-card text-card-foreground shadow-sm",
+      "rounded-lg border bg-card text-card-foreground shadow-sm",
       className
     )}
     {...props}

package/src/components/ui/index.ts

@@ -122,7 +122,6 @@ export * from './content-loader'
 
 // Table components
 export {
-  COMPACT_ROW_MIN_HEIGHT_PX,
   Table, TableCardSkeleton, TableCell,
   TableDescriptionCell, TableEmptyState, TableHeader,
   TableRow, TableTimestampCell

package/src/components/ui/markdown/rich/__tests__/shortcodes.test.ts

@@ -56,47 +56,3 @@ describe('processShortcodes script stripping', () => {
     expect(output).toContain('<script>');
   });
 });
-
-/**
- * Design-doc embeds (R11): the spec's `{{figma:FILE_KEY[:NODE_ID]}}` grammar is
- * rewritten onto the canonical `{{figma:URL}}` rule, and Claude artifacts get
- * the SAME treatment — a shortcode → a block the renderer maps to `ClaudeEmbed`.
- * Both are markdown blocks, so they render identically in a spec body, a
- * comment, or a links rail, and no surface hand-assembles its own card.
- */
-describe('processShortcodes design-doc embeds', () => {
-  it('rewrites {{figma:KEY:NODE}} to the canonical design URL and renders the figma embed', () => {
-    const output = processShortcodes('{{figma:aB3xK9:12-405}}');
-    expect(output).toContain('class="figma-embed"');
-    expect(output).toContain('data-figma-url="https://www.figma.com/design/aB3xK9?node-id=12-405"');
-  });
-
-  it('accepts a colon-separated node id and a bare key', () => {
-    expect(processShortcodes('{{figma:aB3xK9:12:405}}')).toContain('node-id=12-405');
-    expect(processShortcodes('{{figma:aB3xK9}}')).toContain('data-figma-url="https://www.figma.com/design/aB3xK9"');
-  });
-
-  it('leaves the existing {{figma:URL}} grammar untouched', () => {
-    const output = processShortcodes('{{figma:https://www.figma.com/proto/aB3xK9/Flows?node-id=1-2}}');
-    expect(output).toContain('data-figma-url="https://www.figma.com/proto/aB3xK9/Flows?node-id=1-2"');
-  });
-
-  it('renders a Claude artifact / Claude Design link as an embed block, like figma', () => {
-    const artifact = processShortcodes('{{claude-artifact:https://claude.ai/public/artifacts/abc-123}}');
-    expect(artifact).toContain('class="claude-embed"');
-    expect(artifact).toContain('data-url="https://claude.ai/public/artifacts/abc-123"');
-    expect(artifact).toContain('data-kind="artifact"');
-
-    const design = processShortcodes('{{claude-design:https://claude.ai/design/xyz}}');
-    expect(design).toContain('data-kind="design"');
-  });
-
-  it('escapes the url attribute so a crafted link cannot break out of the block', () => {
-    const output = processShortcodes('{{claude-artifact:https://claude.ai/a"><script>alert(1)</script>}}');
-    expect(output).not.toContain('<script>');
-    expect(output).toContain('&quot;');
-  });
-
-
-
-});