Drop run-summary JSON visibility columns

Drop run-summary JSON visibility columns

Author: durable-workflow-ops

docs/search-attributes-architecture.md

@@ -326,8 +326,6 @@ is no separate v2-alpha cutover surface to preserve.
 Required cleanup before v2.0 stable:
 
 - runtime stops dual-writing the JSON column from `WorkflowExecutor`
-- runtime stops dual-reading the JSON column as a fallback in
-  `WorkflowRunSummary` and other projections
 - the `workflow_runs.search_attributes` column is dropped from the
   `create_workflow_runs_table` migration
 - this document and `docs/workflow-memos-architecture.md` describe the

docs/workflow-memos-architecture.md

@@ -255,8 +255,6 @@ is no separate v2-alpha cutover surface to preserve.
 Required cleanup before v2.0 stable:
 
 - runtime stops dual-writing the JSON column from `WorkflowExecutor`
-- runtime stops dual-reading the JSON column as a fallback in
-  `WorkflowRunSummary::getMemos()` and other projections
 - the `workflow_runs.memo` column is dropped from the
   `create_workflow_runs_table` migration
 - this document and `docs/search-attributes-architecture.md` describe

src/V2/Support/RunSummaryProjector.php

@@ -318,7 +318,6 @@ public static function project(WorkflowRun $run): WorkflowRunSummary
                 'namespace' => $run->namespace ?? $run->instance?->namespace,
                 'business_key' => $run->business_key ?? $run->instance?->business_key,
                 'visibility_labels' => $run->visibility_labels ?? $run->instance?->visibility_labels,
-                'search_attributes' => $run->search_attributes,
                 'compatibility' => $run->compatibility,
                 'declared_entry_mode' => $commandContract['entry_mode'],
                 'declared_contract_source' => $commandContract['source'],

src/migrations/2026_04_05_000106_create_workflow_run_summaries_table.php

@@ -52,10 +52,6 @@ public function up(): void
                 ->index('workflow_run_summaries_business_key_index');
             $table->json('visibility_labels')
                 ->nullable();
-            $table->json('memo')
-                ->nullable();
-            $table->json('search_attributes')
-                ->nullable();
 
             // Execution status
             $table->string('status')

tests/Unit/Providers/WorkflowServiceProviderTest.php

@@ -190,7 +190,8 @@ public function testProviderLoadsPackageMigrationsForFreshApps(): void
         $this->assertTrue(Schema::hasTable('workflow_instances'));
         $this->assertTrue(Schema::hasTable('workflow_runs'));
         $this->assertTrue(Schema::hasTable('workflow_run_summaries'));
-        $this->assertTrue(Schema::hasColumn('workflow_run_summaries', 'memo'));
+        $this->assertFalse(Schema::hasColumn('workflow_run_summaries', 'memo'));
+        $this->assertFalse(Schema::hasColumn('workflow_run_summaries', 'search_attributes'));
         $this->assertTrue(Schema::hasTable('workflow_commands'));
     }
 
@@ -220,7 +221,8 @@ public function testPublishedMigrationsCanRunAsInstallSource(): void
             $this->assertTrue(Schema::hasTable('workflow_instances'));
             $this->assertTrue(Schema::hasTable('workflow_runs'));
             $this->assertTrue(Schema::hasTable('workflow_run_summaries'));
-            $this->assertTrue(Schema::hasColumn('workflow_run_summaries', 'memo'));
+            $this->assertFalse(Schema::hasColumn('workflow_run_summaries', 'memo'));
+            $this->assertFalse(Schema::hasColumn('workflow_run_summaries', 'search_attributes'));
             $this->assertTrue(Schema::hasTable('workflow_schedules'));
             $this->assertTrue(Schema::hasTable('workflow_schedule_history_events'));
         } finally {

tests/Unit/V2/VisibilityFiltersTest.php

@@ -728,7 +728,6 @@ private function createSearchAttributeSummary(string $runId, string $instanceId,
             'engine_source' => 'v2',
             'class' => 'BillingWorkflow',
             'workflow_type' => 'billing.invoice-sync',
-            'search_attributes' => $searchAttributes,
             'status' => 'running',
             'status_bucket' => 'running',
         ]);

tests/Unit/V2/VisibilityMetadataContractDocumentationTest.php

@@ -97,7 +97,7 @@ public function testSearchAttributesDocNamesRequiredCleanupSurfaces(): void
     {
         $contents = $this->documentContents(self::SEARCH_ATTRIBUTES_DOCUMENT);
 
-        foreach (['WorkflowExecutor', 'WorkflowRunSummary', 'create_workflow_runs_table'] as $surface) {
+        foreach (['WorkflowExecutor', 'create_workflow_runs_table'] as $surface) {
             $this->assertStringContainsString(
                 $surface,
                 $contents,
@@ -113,11 +113,7 @@ public function testMemosDocNamesRequiredCleanupSurfaces(): void
     {
         $contents = $this->documentContents(self::MEMOS_DOCUMENT);
 
-        foreach ([
-            'WorkflowExecutor',
-            'WorkflowRunSummary::getMemos()',
-            'create_workflow_runs_table',
-        ] as $surface) {
+        foreach (['WorkflowExecutor', 'create_workflow_runs_table'] as $surface) {
             $this->assertStringContainsString(
                 $surface,
                 $contents,

tests/Unit/V2/WorkflowRunVisibilityMetadataAccessorsTest.php

@@ -75,12 +75,6 @@ public function testSummaryPropertiesPreferTypedVisibilityRows(): void
             'projection_schema_version' => 1,
             'class' => $run->workflow_class,
             'workflow_type' => $run->workflow_type,
-            'memo' => [
-                'legacy' => 'json',
-            ],
-            'search_attributes' => [
-                'legacy' => 'json',
-            ],
             'status' => $run->status->value,
             'status_bucket' => $statusBucket->value,
         ]);

tests/Unit/migrations/MigrationsTest.php

@@ -193,6 +193,8 @@ public function testDownMethodsDropTables(): void
         $this->assertTrue(Schema::hasColumn('workflow_run_summaries', 'task_problem'));
         $this->assertTrue(Schema::hasColumn('workflow_run_summaries', 'declared_entry_mode'));
         $this->assertTrue(Schema::hasColumn('workflow_run_summaries', 'declared_contract_source'));
+        $this->assertFalse(Schema::hasColumn('workflow_run_summaries', 'memo'));
+        $this->assertFalse(Schema::hasColumn('workflow_run_summaries', 'search_attributes'));
         $this->assertTrue(Schema::hasColumn('workflow_links', 'sequence'));
         $this->assertTrue(Schema::hasColumn('workflow_tasks', 'last_dispatch_attempt_at'));
         $this->assertTrue(Schema::hasColumn('workflow_tasks', 'last_dispatch_error'));