fix: address 0.8.8 release readiness issues

Author: stefan-ernst

frontend/src/lib/features/collections/CollectionBoard.svelte

@@ -20,7 +20,7 @@
   import { buildIterationPickerConfig } from '../iterations/iterationPickerUtils.js';
   import ItemTypeIcon from '../../components/ItemTypeIcon.svelte';
   import { draggable, dropTargetForElements } from '@atlaskit/pragmatic-drag-and-drop/element/adapter';
-  import { autoScrollWindowForElements } from '@atlaskit/pragmatic-drag-and-drop-auto-scroll/element';
+  import { autoScrollForElements } from '@atlaskit/pragmatic-drag-and-drop-auto-scroll/element';
   import { attachClosestEdge, extractClosestEdge } from '@atlaskit/pragmatic-drag-and-drop-hitbox/closest-edge';
   import ItemDetail from '../items/ItemDetail.svelte';
   import PersonalTaskDetail from '../personal/PersonalTaskDetail.svelte';
@@ -87,7 +87,6 @@
   let loading = $state(true);
   let currentCollectionName = $derived(collectionStore.collectionName);
   let setupTimeout;
-  let autoScrollCleanup;
   let setupElements = new Map(); // Track which elements have drag/drop set up and their cleanup functions
   let pendingDrops = new Set(); // Track pending drop operations to prevent duplicates
   let showItemModal = $state(false);
@@ -145,7 +144,7 @@
   // Edge-based drag state
   let dragState = $state(new Map()); // Track drag state for each item: { isDragging: boolean, closestEdge: 'top'|'bottom'|null }
   let boardAnnouncement = $state('');
-  let boardViewElement = $state(null);
+  let boardScrollElement = $state(null);
 
   // Centralized gradient styling
   const styles = useGradientStyles();
@@ -178,7 +177,14 @@
 
   onDestroy(() => {
     collectionStore.clearBoardSearch();
-    autoScrollCleanup?.();
+  });
+
+  $effect(() => {
+    if (!boardScrollElement) return;
+    return autoScrollForElements({
+      element: boardScrollElement,
+      getAllowedAxis: () => 'horizontal',
+    });
   });
 
   // Listen for newly created items
@@ -412,12 +418,6 @@
   }
 
   onMount(async () => {
-    autoScrollCleanup = autoScrollWindowForElements({
-      canScroll: ({ source }) =>
-        source.data.type === 'work-item' && Boolean(boardViewElement?.contains(source.element)),
-      getAllowedAxis: () => 'horizontal',
-    });
-
     await Promise.all([
       workspaceId ? loadWorkspaceGradient(workspaceId) : Promise.resolve(),
       workspaceId
@@ -1542,12 +1542,17 @@
     <div class="animate-pulse">{t('common.loading')}</div>
   </div>
 {:else if workspace || !workspaceId}
-  <StaticViewBackground
-    backgroundStyle={styles.backgroundStyle}
-    contextVars={styles.contextVars}
-    contentClass="p-6 min-w-fit"
-    testid="collection-board-background"
+  <div
+    bind:this={boardScrollElement}
+    class="w-full min-w-0 max-w-full overflow-x-auto"
+    data-testid="board-scroll-container"
   >
+    <StaticViewBackground
+      backgroundStyle={styles.backgroundStyle}
+      contextVars={styles.contextVars}
+      contentClass="p-6 min-w-fit"
+      testid="collection-board-background"
+    >
     <!-- Content Container -->
       <!-- Header with view tabs -->
       <div class="mb-8">
@@ -1691,7 +1696,6 @@
       {:else}
         <!-- Board Columns / Swimlanes -->
         <div
-          bind:this={boardViewElement}
           class={selectedGroupByItemType ? 'space-y-4' : ''}
           data-testid="board-view"
         >
@@ -1892,7 +1896,8 @@
           </p>
         </div>
       {/if}
-  </StaticViewBackground>
+    </StaticViewBackground>
+  </div>
 {:else}
   <div class="p-6">
     <div class="text-center" style="color: var(--ds-text-subtle);">

frontend/src/lib/features/collections/CollectionMap.svelte

@@ -12,8 +12,8 @@
   import ItemTypeIcon from '../../components/ItemTypeIcon.svelte';
   import EmptyState from '../../components/EmptyState.svelte';
   import Textarea from '../../components/Textarea.svelte';
-  import { monitorForElements } from '@atlaskit/pragmatic-drag-and-drop/element/adapter';
-  import { draggable, dropTargetForElements } from '@atlaskit/pragmatic-drag-and-drop/element/adapter';
+  import { draggable, dropTargetForElements, monitorForElements } from '@atlaskit/pragmatic-drag-and-drop/element/adapter';
+  import { autoScrollForElements } from '@atlaskit/pragmatic-drag-and-drop-auto-scroll/element';
   import Tooltip from '../../components/Tooltip.svelte';
   import ViewHeader from '../../layout/ViewHeader.svelte';
   import StaticViewBackground from '../../layout/StaticViewBackground.svelte';
@@ -53,6 +53,7 @@
   // Item detail modal state
   let selectedItemId = $state(null);
   let showItemModal = $state(false);
+  let mapScrollElement = $state(null);
 
 
   // Centralized gradient styling
@@ -65,6 +66,14 @@
 
   useEventListener(() => window, 'popstate', handlePopState);
 
+  $effect(() => {
+    if (!mapScrollElement) return;
+    return autoScrollForElements({
+      element: mapScrollElement,
+      getAllowedAxis: () => 'horizontal',
+    });
+  });
+
   onMount(async () => {
     if (workspaceId) {
       await loadWorkspaceGradient(workspaceId);
@@ -568,6 +577,7 @@
     backgroundStyle={styles.backgroundStyle}
     contextVars={styles.contextVars}
     contentClass=""
+    rootStyle="width: 100%; min-width: 0; max-width: 100%;"
     testid="map-view"
   >
     <!-- Header -->
@@ -642,7 +652,11 @@
     </div>
 
     <!-- Story Map Container -->
-    <div class="p-6 overflow-x-auto">
+    <div
+      bind:this={mapScrollElement}
+      class="p-6 overflow-x-auto"
+      data-testid="map-scroll-container"
+    >
       <div class="min-w-max">
         <!-- Backbone (Horizontal) -->
         <div

frontend/src/lib/portal/PortalVerifyLink.svelte

@@ -10,6 +10,7 @@
   let errorMessage = $state('');
 
   let verificationRun = 0;
+  let lastVerificationKey = null;
 
   async function verify(currentSlug, currentToken, run) {
     if (!currentToken) {
@@ -45,6 +46,11 @@
   $effect(() => {
     const currentSlug = slug;
     const currentToken = token;
+    const verificationKey = currentSlug && currentToken
+      ? `${currentSlug}:${currentToken}`
+      : null;
+    if (verificationKey && verificationKey === lastVerificationKey) return;
+    lastVerificationKey = verificationKey;
     const run = ++verificationRun;
     status = 'verifying';
     errorMessage = '';

frontend/src/lib/stores/portalAuth.svelte.js

@@ -259,8 +259,9 @@ function createPortalAuthStore() {
       try {
         const response = await api.portalAuth.verifyMagicLink(slug, token);
         if (response.success) {
-          customer.set(response.customer);
-          isAuthenticated.set(true);
+          // The caller dismisses the one-time-token UI before checkAuth updates
+          // reactive auth state. Updating it here can remount the verifier and
+          // redeem the same token a second time.
           return { success: true, customer: response.customer };
         } else {
           error.set(response.message || 'Invalid or expired link');

internal/actionevents/actionevents.go

@@ -288,7 +288,7 @@ func RunTargets(ctx context.Context, store *TargetStore, eventKey string, callba
 	}
 
 	var targetErrors []error
-	permanent := false
+	hasTransientFailure := false
 	var executed int64
 	for _, target := range targets {
 		if target.State == "completed" || target.State == "skipped" {
@@ -298,26 +298,35 @@ func RunTargets(ctx context.Context, store *TargetStore, eventKey string, callba
 		if completionErr == nil && completed {
 			if err := store.markCompleted(ctx, eventKey, target.ActionID); err != nil {
 				targetErrors = append(targetErrors, err)
+				hasTransientFailure = true
 			}
 			continue
 		}
 		if completionErr != nil && !errors.Is(completionErr, sql.ErrNoRows) {
 			targetErrors = append(targetErrors, fmt.Errorf("load durable action execution: %w", completionErr))
+			hasTransientFailure = true
 			continue
 		}
 		if err := store.markRunning(ctx, eventKey, target.ActionID); err != nil {
 			targetErrors = append(targetErrors, err)
+			hasTransientFailure = true
 			continue
 		}
 		isPermanent, err := callbacks.Execute(target.ActionID)
 		if err != nil {
-			permanent = permanent || isPermanent
-			_ = store.markFailed(ctx, eventKey, target.ActionID, err)
+			if !isPermanent {
+				hasTransientFailure = true
+			}
+			if markErr := store.markFailed(ctx, eventKey, target.ActionID, err); markErr != nil {
+				targetErrors = append(targetErrors, markErr)
+				hasTransientFailure = true
+			}
 			targetErrors = append(targetErrors, err)
 			continue
 		}
 		if err := store.markCompleted(ctx, eventKey, target.ActionID); err != nil {
 			targetErrors = append(targetErrors, err)
+			hasTransientFailure = true
 			continue
 		}
 		executed++
@@ -326,8 +335,8 @@ func RunTargets(ctx context.Context, store *TargetStore, eventKey string, callba
 		return executed, nil
 	}
 	err = errors.Join(targetErrors...)
-	if permanent {
-		return executed, events.Permanent(err)
+	if hasTransientFailure {
+		return executed, err
 	}
-	return executed, err
+	return executed, events.Permanent(err)
 }

internal/database/migrations.go

@@ -499,6 +499,14 @@ var Catalog = []Migration{
 		Postgres:    `ALTER TABLE sso_providers ALTER COLUMN attribute_mapping SET DEFAULT '{"email":"email","name":"name","given_name":"given_name","family_name":"family_name","username":"preferred_username","email_verified":"email_verified"}'`,
 		ApplySQLite: applySQLiteSSOAttributeMappingDefault,
 	},
+	{
+		Version:       "20260901_daily_briefing_workspace_provenance",
+		Name:          "Record daily briefing workspace provenance",
+		CheckSQLite:   sqliteColumnCheck("daily_briefings", "source_workspace_ids"),
+		CheckPostgres: pgColumnCheck("daily_briefings", "source_workspace_ids"),
+		SQLite:        "ALTER TABLE daily_briefings ADD COLUMN source_workspace_ids TEXT",
+		Postgres:      "ALTER TABLE daily_briefings ADD COLUMN source_workspace_ids TEXT",
+	},
 }
 
 func applySQLiteSSOAttributeMappingDefault(db Database) (retErr error) {

internal/database/schema/daily_briefings.sql

@@ -3,6 +3,7 @@ CREATE TABLE IF NOT EXISTS daily_briefings (
     user_id INTEGER NOT NULL,
     date TEXT NOT NULL,
     content TEXT NOT NULL,
+    source_workspace_ids TEXT,
     generation_duration_ms INTEGER,
     error TEXT,
     lock_until DATETIME,

internal/database/schema/daily_briefings_postgres.sql

@@ -3,6 +3,7 @@ CREATE TABLE IF NOT EXISTS daily_briefings (
     user_id INTEGER NOT NULL,
     date TEXT NOT NULL,
     content TEXT NOT NULL,
+    source_workspace_ids TEXT,
     generation_duration_ms INTEGER,
     error TEXT,
     lock_until TIMESTAMPTZ,

internal/events/engine.go

@@ -97,6 +97,7 @@ type Engine struct {
 
 	mu            sync.Mutex
 	handlers      map[string]Handler
+	nextHandler   int
 	started       bool
 	used          bool
 	cancel        context.CancelFunc
@@ -349,7 +350,7 @@ func (e *Engine) handle(ctx context.Context, handler Handler, delivery Delivery)
 		return nil
 	}
 
-	retry := !isPermanent(err) && delivery.AttemptCount < e.config.MaxAttempts
+	retry := !IsPermanent(err) && delivery.AttemptCount < e.config.MaxAttempts
 	nextAttempt := e.now()
 	if retry {
 		nextAttempt = nextAttempt.Add(e.retryDelay(delivery.AttemptCount))
@@ -381,7 +382,15 @@ func (e *Engine) handlerKeys() []string {
 		keys = append(keys, key)
 	}
 	slices.Sort(keys)
-	return keys
+	if len(keys) == 0 {
+		return keys
+	}
+	start := e.nextHandler % len(keys)
+	e.nextHandler = (start + 1) % len(keys)
+	if start == 0 {
+		return keys
+	}
+	return append(keys[start:], keys[:start]...)
 }
 
 func (e *Engine) handler(key string) Handler {

internal/events/event.go

@@ -207,7 +207,8 @@ func Permanent(err error) error {
 	return permanentFailure{err: err}
 }
 
-func isPermanent(err error) bool {
+// IsPermanent reports whether an error disables automatic delivery retries.
+func IsPermanent(err error) bool {
 	var target permanentFailure
 	return errors.As(err, &target)
 }