core: Restart store on revert errors to clear poisoned writer (#6422)

The runner FSM refactor mapped Action::Restart from handle_revert to
RestartReason::DataSourceExpired, skipping store.restart(). Fix by using
RestartReason::StoreError for revert errors and remove the unused
DataSourceExpired variant.

Author: incrypto32

core/src/subgraph/runner/mod.rs

@@ -619,7 +619,7 @@ where
         match action {
             Action::Continue => Ok(RunnerState::AwaitingBlock { block_stream }),
             Action::Restart => Ok(RunnerState::Restarting {
-                reason: RestartReason::DataSourceExpired,
+                reason: RestartReason::StoreError,
             }),
             Action::Stop => Ok(RunnerState::Stopped {
                 reason: StopReason::Canceled,

core/src/subgraph/runner/state.rs

@@ -74,8 +74,6 @@ pub enum RunnerState<C: Blockchain> {
 pub enum RestartReason {
     /// New dynamic data source was created that requires filter updates.
     DynamicDataSourceCreated,
-    /// A data source reached its end block.
-    DataSourceExpired,
     /// Store error occurred and store needs to be restarted.
     StoreError,
     /// Possible reorg detected, need to restart to detect it.