feat(drive-integration): async import runs rearchitecture [INTEG-4526] by david-shibley-contentful · Pull Request #11092 · contentful/apps

david-shibley-contentful · 2026-07-16T21:10:58Z

INTEG-4526

Summary

Rearchitects the import flow from synchronous (20-min blocking spinner) to fully async: wizard fires off the agent run, saves to localStorage, redirects immediately to a new Runs page
New Runs page becomes the app home screen — lists all imports with live status badges (Running / Needs Review / Completed / Failed / Expired), polls every 10s while any run is in-flight, supports multiple concurrent runs
ReviewPage is now reachable directly from the Runs page via a Review CTA (no longer locked inside the wizard flow); entry creation stays synchronous

Key files

src/types/runs.ts — RunRecord, DisplayStatus, RunWithStatus, AppView types
src/hooks/useRunStorage.ts — localStorage-backed run history (max 50, scoped by space+env)
src/hooks/useRunsPolling.ts — parallel Promise.all polling, 10s interval while running
src/services/workflowService.ts — resumeAndPollWorkflow extracted from the hook
src/locations/Page/components/runs/ — RunsPage + RunRow components
src/locations/Page/Page.tsx — AppView discriminated union state machine replaces binary toggle
src/locations/Page/components/mainpage/ModalOrchestrator.tsx — fire-and-forget wizard exit

Test plan

Generated with Claude Code

Rearchitects the drive-integration import flow from synchronous (20-min blocking spinner) to fully async. Import wizard fires off the agent run, saves a RunRecord to localStorage, then immediately redirects to a new Runs page. The Runs page polls agentRun status every 10s for in-flight imports and supports multiple concurrent runs. ReviewPage is now reachable directly from the Runs page rather than only from the wizard flow. Key changes: - New RunsPage (home screen) + RunRow components with live status badges - useRunStorage hook (localStorage-backed RunRecord[] with 50-record cap) - useRunsPolling hook (parallel Promise.all, 10s interval while running) - workflowService.ts extracted from useWorkflowAgent (resumeAndPollWorkflow) - Page.tsx rewritten around AppView discriminated union state machine - ModalOrchestrator: fire-and-forget startWorkflow → addRun → onRunStarted - 104 tests passing across all new and modified files Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…table after import Three independent useRunStorage instances (Page, RunsPage, ModalOrchestrator) meant addRun() only updated local React state — RunsPage wouldn't see the new run without a page refresh. Lift the hook to Page as the single owner and pass runs/addRun/ removeRun/storageError down as props. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…y ESLint Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…ixtures from branch Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…E_ENABLE_MOCK_REVIEW_PAYLOAD branch Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

JuliRossi · 2026-07-20T13:53:10Z

+  const handleCancelReview = async (runId?: string) => {
+    if (runId) {
+      try {
+        await resumeAndPollWorkflow(sdk, runId, { cancelled: true });


⚔️ This is missing the entryBlockGraph. I think we want it for logging purposes

JuliRossi · 2026-07-20T14:05:15Z

+        onRunStarted={handleRunStarted}
+        onResetToMain={() => setAppView({ view: 'runs' })}


💭 These props do the same thing, setAppView({ view: 'runs' })

JuliRossi · 2026-07-20T14:08:06Z

+  const handleExitReview = () => {
    modalOrchestratorRef.current?.resetFlow();
-    handleReturnToMainPage();
+    setAppView({ view: 'runs' });


💭 Same logic as handleRunStarted(), would replace with that

JuliRossi · 2026-07-20T14:35:26Z

+                {run.createdEntryIds.map((entryId) => (
+                  <TextLink
+                    key={entryId}
+                    href={`https://app.contentful.com/spaces/${spaceId}/entries/${entryId}`}


❓ Should this use : sdk.hostnames.webapp instead of https://app.contentful.com ?

JuliRossi · 2026-07-20T14:39:03Z

+    default:
+      return 'expired';
+  }
+}


🤔 This could be a map, instead of s switch. Similar to what we do for FAILURE_REASON_MESSAGES

JuliRossi · 2026-07-20T15:17:58Z

+        const runId = await startWorkflow(contentTypeIds, documentSelection);
+
+        if (storageError) {
+          // Storage unavailable — surface error before proceeding
+          showWorkflowError(
+            new WorkflowRunError(
+              'Unable to track this import: browser storage is unavailable or full.',
+              WorkflowFailureReason.GENERIC
+            )
+          );
+          return;
+        }


❓ Do we want to start the workflow even if we've got no storage? How is the user going to be able to see the run?

JuliRossi · 2026-07-20T15:19:04Z

+  onDismiss: (runId: string) => void;
+}
+
+function formatDate(iso: string): string {


⛏️ This manual format handling could be avoided if use a library like Dayjs

JuliRossi · 2026-07-20T15:29:05Z

-    payload: MappingReviewSuspendPayload;
-    runId?: string;
-  } | null>(null);
+  const [appView, setAppView] = useState<AppView>({ view: 'runs' });


💭 Could we move runId outside AppView type? It makes AppView type a bit inconsistent. This could also make simpler with UseEffect in line 41

JuliRossi · 2026-07-20T15:38:41Z

+    if (appView.view !== 'review') {
+      setPendingReviewPayload(null);
+      return;


Suggested change

if (appView.view !== 'review') {

setPendingReviewPayload(null);

return;

if (appView.view !== 'review') {

setPendingReviewPayload(null);

setIsLoadingReviewPayload(false);

return;

JuliRossi · 2026-07-20T15:41:27Z

+  errorMessage?: string;
+}
+
+export type AppView = { view: 'runs' } | { view: 'import' } | { view: 'review'; runId: string };


💭 runs, import and review could be an enum

JuliRossi · 2026-07-20T16:00:07Z

❓ Should we allow the user to a way clear the runs they have on the page? So they can free space on the localstorage as well

Also if a run stuck in 'running' has no action button. The user can't dismiss it, can't cancel it, can't do anything. It just sits there with a spinner.

JuliRossi

Nice work! They way I review PRs:
⛏️ Nitpicks, non blockers
💭 / 🤔 Highly suggested, but non blockers
❓ Question, may be blocking or not depending on the answer
⚔️ Blockers

After testing it locally: There seems to be a bug with the document title that keep it as Untitled document instead of the actual title. We are also showing the entry id instead of the name in the RunPage

JuliRossi · 2026-07-20T16:15:09Z

💭 I think we no longer need the loading step for the ModalOrchestrator

david-shibley-contentful requested review from a team as code owners July 16, 2026 21:10

david-shibley-contentful force-pushed the feat/drive-integration-async-runs-INTEG-4526 branch from 3ca6d78 to 4bc5d43 Compare July 16, 2026 21:52

david-shibley-contentful and others added 3 commits July 16, 2026 16:00

chore(drive-integration): fix prettier formatting violations

2d23ce1

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

david-shibley-contentful force-pushed the feat/drive-integration-async-runs-INTEG-4526 branch from 2734bc2 to 2d23ce1 Compare July 16, 2026 22:00

david-shibley-contentful and others added 3 commits July 16, 2026 16:04

fix(drive-integration): remove unused imports and variables flagged b…

cd4c650

…y ESLint Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

chore(drive-integration): remove speckit planning artifacts and dev f…

66a8b7d

…ixtures from branch Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix(drive-integration): remove dangling FixtureHarness import and VIT…

db5e939

…E_ENABLE_MOCK_REVIEW_PAYLOAD branch Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

JuliRossi reviewed Jul 20, 2026

View reviewed changes

		onRunStarted={handleRunStarted}
		onResetToMain={() => setAppView({ view: 'runs' })}

Uh oh!

Conversation

david-shibley-contentful commented Jul 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Key files

Test plan

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

JuliRossi Jul 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

JuliRossi Jul 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

JuliRossi left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

david-shibley-contentful commented Jul 16, 2026 •

edited

Loading

JuliRossi Jul 20, 2026 •

edited

Loading

JuliRossi Jul 20, 2026 •

edited

Loading

JuliRossi left a comment •

edited

Loading