@temporalio/core-bridge 1.7.2 → 1.7.3

package/lib/index.d.ts

@@ -1,7 +1,7 @@
 import { SpanContext } from '@opentelemetry/api';
 import type { TLSConfig } from '@temporalio/common/lib/internal-non-workflow';
 export { TLSConfig };
-declare type Shadow<Base, New> = Base extends object ? New extends object ? {
+type Shadow<Base, New> = Base extends object ? New extends object ? {
     [K in keyof Base | keyof New]: K extends keyof Base ? K extends keyof New ? Shadow<Base[K], New[K]> : Base[K] : K extends keyof New ? New[K] : never;
 } : New : New;
 export interface RetryOptions {
@@ -74,7 +74,7 @@ export interface ForwardLogger {
  *
  * @experimental
  */
-export declare type Logger = ConsoleLogger | ForwardLogger;
+export type Logger = ConsoleLogger | ForwardLogger;
 /**
  * OpenTelemetry Collector options for exporting metrics or traces
  *
@@ -100,13 +100,13 @@ export interface OtelCollectorExporter {
     };
 }
 /** @experimental */
-export declare type CompiledOtelTraceExporter = Shadow<OtelCollectorExporter, {
+export type CompiledOtelTraceExporter = Shadow<OtelCollectorExporter, {
     otel: {
         metricsExportInterval?: never;
     };
 }>;
 /** @experimental */
-export declare type CompiledOtelMetricsExporter = Shadow<OtelCollectorExporter, {
+export type CompiledOtelMetricsExporter = Shadow<OtelCollectorExporter, {
     otel: {
         metricsExportInterval: number;
     };
@@ -136,7 +136,7 @@ export interface PrometheusMetricsExporter {
  *
  * @experimental
  */
-export declare type MetricsExporter = {
+export type MetricsExporter = {
     temporality?: 'cumulative' | 'delta';
 } & (PrometheusMetricsExporter | OtelCollectorExporter);
 /**
@@ -144,7 +144,7 @@ export declare type MetricsExporter = {
  *
  * @experimental
  */
-export declare type TraceExporter = OtelCollectorExporter;
+export type TraceExporter = OtelCollectorExporter;
 /** @experimental */
 export interface TelemetryOptions {
     /**
@@ -216,7 +216,7 @@ export interface TelemetryOptions {
     metrics?: MetricsExporter;
 }
 /** @experimental */
-export declare type CompiledTelemetryOptions = {
+export type CompiledTelemetryOptions = {
     noTemporalPrefixForMetrics?: boolean;
     logging: {
         filter: string;
@@ -290,7 +290,7 @@ export interface WorkerOptions {
     maxActivitiesPerSecond?: number;
 }
 /** Log level - must match rust log level names */
-export declare type LogLevel = 'TRACE' | 'DEBUG' | 'INFO' | 'WARN' | 'ERROR';
+export type LogLevel = 'TRACE' | 'DEBUG' | 'INFO' | 'WARN' | 'ERROR';
 export interface LogEntry {
     /** Log message */
     message: string;
@@ -306,7 +306,7 @@ export interface LogEntry {
 /**
  * Which version of the executable to run.
  */
-export declare type EphemeralServerExecutable = {
+export type EphemeralServerExecutable = {
     type: 'cached-download';
     /**
      * Download destination directory or the system's temp directory if none set.
@@ -390,7 +390,7 @@ export interface DevServerConfig {
  *
  * Both the time-skipping test server and Temporal CLI dev server are supported.
  */
-export declare type EphemeralServerConfig = TimeSkippingServerConfig | DevServerConfig;
+export type EphemeralServerConfig = TimeSkippingServerConfig | DevServerConfig;
 export interface Worker {
     type: 'Worker';
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@temporalio/core-bridge",
-  "version": "1.7.2",
+  "version": "1.7.3",
   "description": "Temporal.io SDK Core<>Node bridge",
   "main": "index.js",
   "types": "lib/index.d.ts",
@@ -22,8 +22,8 @@
   "author": "Temporal Technologies Inc. <sdk@temporal.io>",
   "license": "MIT",
   "dependencies": {
-    "@opentelemetry/api": "^1.3.0",
-    "@temporalio/common": "1.7.2",
+    "@opentelemetry/api": "^1.4.1",
+    "@temporalio/common": "1.7.3",
     "arg": "^5.0.2",
     "cargo-cp-artifact": "^0.1.6",
     "which": "^2.0.2"
@@ -53,5 +53,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "779561124eecdec8396e658c0a1305d343dfaff7"
+  "gitHead": "c78237bad40342b97a0aaec97ef01763b39a04f8"
 }

package/sdk-core/.buildkite/pipeline.yml

@@ -17,7 +17,7 @@ steps:
     agents:
       queue: "default"
       docker: "*"
-    command: "cargo lint"
+    command: "cargo lint && cargo test-lint"
     timeout_in_minutes: 15
     plugins:
       - docker-compose#v3.0.0:

package/sdk-core/.cargo/config.toml

@@ -4,3 +4,5 @@ wf-input-replay = ["run", "--package", "temporal-sdk-core", "--features", "save_
                    "--example", "wf_input_replay", "--"]
 lint = ["clippy", "--workspace", "--examples", "--all-features",
         "--test", "integ_tests", "--test", "heavy_tests", "--", "--D", "warnings"]
+test-lint = ["clippy", "--all", "--all-features", "--examples", "--workspace",
+             "--tests", "--", "--D", "warnings"]

package/sdk-core/CODEOWNERS

@@ -1,3 +1,3 @@
 # Primary owners
 
-* @Sushisource @bergundy @cretz @Spikhalskiy
+* @temporalio/sdk

package/sdk-core/core/src/core_tests/workflow_tasks.rs

@@ -896,10 +896,7 @@ async fn workflow_failures_only_reported_once() {
 #[tokio::test]
 async fn max_wft_respected() {
     let total_wfs = 100;
-    let wf_ids: Vec<_> = (0..total_wfs)
-        .into_iter()
-        .map(|i| format!("fake-wf-{i}"))
-        .collect();
+    let wf_ids: Vec<_> = (0..total_wfs).map(|i| format!("fake-wf-{i}")).collect();
     let hists = wf_ids.iter().map(|wf_id| {
         let hist = canned_histories::single_timer("1");
         FakeWfResponses {
@@ -1027,7 +1024,7 @@ async fn activity_not_canceled_when_also_completed_repro(hist_batches: &'static
 #[tokio::test]
 async fn lots_of_workflows() {
     let total_wfs = 500;
-    let hists = (0..total_wfs).into_iter().map(|i| {
+    let hists = (0..total_wfs).map(|i| {
         let wf_id = format!("fake-wf-{i}");
         let hist = canned_histories::single_timer("1");
         FakeWfResponses {
@@ -1705,9 +1702,7 @@ async fn pagination_works_with_tasks_from_completion() {
     t.add_by_type(EventType::WorkflowExecutionStarted);
     t.add_full_wf_task();
     t.add_we_signaled("sig", vec![]);
-    t.add_full_wf_task();
-    t.add_workflow_execution_completed();
-    let get_exec_resp: GetWorkflowExecutionHistoryResponse = t.get_history_info(2).unwrap().into();
+    t.add_workflow_task_scheduled_and_started();
 
     let mut mock = mock_workflow_client();
     let mut needs_pag_resp = hist_to_poll_resp(&t, wfid.to_owned(), ResponseType::OneTask(2)).resp;
@@ -1722,9 +1717,13 @@ async fn pagination_works_with_tasks_from_completion() {
     mock.expect_complete_workflow_task()
         .times(1)
         .returning(|_| Ok(Default::default()));
+
+    let get_exec_resp: GetWorkflowExecutionHistoryResponse =
+        t.get_full_history_info().unwrap().into();
     mock.expect_get_workflow_execution_history()
         .returning(move |_, _, _| Ok(get_exec_resp.clone()))
         .times(1);
+
     let mut mock = single_hist_mock_sg(wfid, t, [1], mock, true);
     mock.worker_cfg(|wc| wc.max_cached_workflows = 2);
     let core = mock_worker(mock);
@@ -2162,10 +2161,6 @@ async fn fetching_to_continue_replay_works() {
     t.add_full_wf_task(); // end 14
     let mut fetch_resp: GetWorkflowExecutionHistoryResponse =
         t.get_full_history_info().unwrap().into();
-    // Should only contain events after 7
-    if let Some(ref mut h) = fetch_resp.history {
-        h.events.retain(|e| e.event_id >= 8);
-    }
     // And indicate that even *more* needs to be fetched after this, so we see a request for the
     // next page happen.
     fetch_resp.next_page_token = vec![2];
@@ -2173,12 +2168,8 @@ async fn fetching_to_continue_replay_works() {
     let timer_started_event_id = t.add_by_type(EventType::TimerStarted);
     t.add_timer_fired(timer_started_event_id, "1".to_string());
     t.add_full_wf_task();
-    let mut final_fetch_resp: GetWorkflowExecutionHistoryResponse =
+    let final_fetch_resp: GetWorkflowExecutionHistoryResponse =
         t.get_full_history_info().unwrap().into();
-    // Should have only the final event
-    if let Some(ref mut h) = final_fetch_resp.history {
-        h.events.retain(|e| e.event_id >= 15);
-    }
 
     let tasks = vec![
         ResponseType::ToTaskNum(1),
@@ -2273,15 +2264,25 @@ async fn ensure_fetching_fail_during_complete_sends_task_failure() {
     t.add_full_wf_task(); // started 3
     t.add_we_signaled("sig1", vec![]);
     t.add_full_wf_task(); // started 7
-    t.add_we_signaled("sig2", vec![]);
+
+    // Need a command event after here so the paginator will know it has two complete WFTs and
+    // processing can begin before needing to fetch again
+    t.add_by_type(EventType::TimerStarted);
     t.add_full_wf_task(); // started 11
     t.add_workflow_execution_completed();
 
-    let mut first_poll = hist_to_poll_resp(&t, wfid, ResponseType::ToTaskNum(1)).resp;
-    first_poll.next_page_token = vec![1];
-    first_poll.previous_started_event_id = 3;
+    let mut first_poll = hist_to_poll_resp(&t, wfid, ResponseType::OneTask(4)).resp;
+    // History is partial so fetch will happen. We have to lie here and make up a previous started
+    // which really makes no sense, otherwise the paginator eagerly fetches and will fail before we
+    // ever start anything -- which is good -- but this test wants to make sure a fetching failure
+    // during a completion is handled correctly. That may no longer actually be a thing that can
+    // happen.
+    first_poll.previous_started_event_id = 0;
+    first_poll.started_event_id = 11;
 
-    let mut next_page: GetWorkflowExecutionHistoryResponse = t.get_history_info(2).unwrap().into();
+    let mut next_page: GetWorkflowExecutionHistoryResponse =
+        t.get_full_history_info().unwrap().into();
+    next_page.history.as_mut().unwrap().events.truncate(9);
     next_page.next_page_token = vec![2];
 
     let mut mock = mock_workflow_client();
@@ -2291,9 +2292,6 @@ async fn ensure_fetching_fail_during_complete_sends_task_failure() {
             Ok(next_page.clone())
         })
         .times(1);
-    let mut really_empty_fetch_resp: GetWorkflowExecutionHistoryResponse =
-        t.get_history_info(1).unwrap().into();
-    really_empty_fetch_resp.history = Some(Default::default());
     mock.expect_get_workflow_execution_history()
         .returning(move |_, _, _| {
             error!("Called fetch second time!");
@@ -2314,24 +2312,13 @@ async fn ensure_fetching_fail_during_complete_sends_task_failure() {
         .await
         .unwrap();
 
-    let wf_task = core.poll_workflow_activation().await.unwrap();
-    assert_matches!(
-        wf_task.jobs.as_slice(),
-        [WorkflowActivationJob {
-            variant: Some(workflow_activation_job::Variant::SignalWorkflow(_)),
-        },]
-    );
-    core.complete_workflow_activation(WorkflowActivationCompletion::empty(wf_task.run_id))
-        .await
-        .unwrap();
-
     // Expect to see eviction b/c of history fetching error here.
     let wf_task = core.poll_workflow_activation().await.unwrap();
     assert_matches!(
         wf_task.jobs.as_slice(),
         [WorkflowActivationJob {
-            variant: Some(workflow_activation_job::Variant::RemoveFromCache(_)),
-        },]
+            variant: Some(workflow_activation_job::Variant::RemoveFromCache(c)),
+        }] if c.message.contains("Fetching history")
     );
 
     core.shutdown().await;
@@ -2401,7 +2388,6 @@ async fn core_internal_flags() {
                 .copied()
                 .collect::<HashSet<_>>(),
             CoreInternalFlags::all_except_too_high()
-                .into_iter()
                 .map(|f| f as u32)
                 .collect()
         );

package/sdk-core/core/src/worker/workflow/history_update.rs

@@ -48,6 +48,7 @@ pub struct HistoryUpdate {
     /// True if this update contains the final WFT in history, and no more attempts to extract
     /// additional updates should be made.
     has_last_wft: bool,
+    wft_count: usize,
 }
 impl Debug for HistoryUpdate {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
@@ -176,11 +177,14 @@ impl HistoryPaginator {
             run_id: req.original_wft.work.execution.run_id.clone(),
             previous_wft_started_id: req.original_wft.work.update.previous_wft_started_id,
             wft_started_event_id: req.original_wft.work.update.wft_started_id,
-            id_of_last_event_in_last_extracted_update: None,
+            id_of_last_event_in_last_extracted_update: req
+                .original_wft
+                .paginator
+                .id_of_last_event_in_last_extracted_update,
             client,
             event_queue: Default::default(),
             next_page_token: NextPageToken::FetchFromStart,
-            final_events: vec![],
+            final_events: req.original_wft.work.update.events,
         };
         let first_update = paginator.extract_next_update().await?;
         req.original_wft.work.update = first_update;
@@ -244,7 +248,7 @@ impl HistoryPaginator {
     /// we have two, or until we are at the end of history.
     pub(crate) async fn extract_next_update(&mut self) -> Result<HistoryUpdate, tonic::Status> {
         loop {
-            let fetch_happened = !self.get_next_page().await?;
+            let no_next_page = !self.get_next_page().await?;
             let current_events = mem::take(&mut self.event_queue);
             let seen_enough_events = current_events
                 .back()
@@ -267,10 +271,7 @@ impl HistoryPaginator {
                 .id_of_last_event_in_last_extracted_update
                 .unwrap_or_default()
                 >= self.wft_started_event_id;
-            if current_events.is_empty()
-                && !fetch_happened
-                && already_sent_update_with_enough_events
-            {
+            if current_events.is_empty() && no_next_page && already_sent_update_with_enough_events {
                 // We must return an empty update which also says is contains the final WFT so we
                 // know we're done with replay.
                 return Ok(HistoryUpdate::from_events(
@@ -282,12 +283,15 @@ impl HistoryPaginator {
                 .0);
             }
 
-            if current_events.is_empty() || (fetch_happened && !seen_enough_events) {
+            if current_events.is_empty() || (no_next_page && !seen_enough_events) {
                 // If next page fetching happened, and we still ended up with no or insufficient
                 // events, something is wrong. We're expecting there to be more events to be able to
                 // extract this update, but server isn't giving us any. We have no choice except to
                 // give up and evict.
                 error!(
+                    current_events=?current_events,
+                    no_next_page,
+                    seen_enough_events,
                     "We expected to be able to fetch more events but server says there are none"
                 );
                 return Err(EMPTY_FETCH_ERR.clone());
@@ -302,6 +306,16 @@ impl HistoryPaginator {
                 self.wft_started_event_id,
                 no_more,
             );
+
+            // If there are potentially more events and we haven't extracted two WFTs yet, keep
+            // trying.
+            if !matches!(self.next_page_token, NextPageToken::Done) && update.wft_count < 2 {
+                // Unwrap the update and stuff it all back in the queue
+                self.event_queue.extend(update.events);
+                self.event_queue.extend(extra);
+                continue;
+            }
+
             let extra_eid_same = extra
                 .first()
                 .map(|e| e.event_id == first_event_id)
@@ -315,17 +329,19 @@ impl HistoryPaginator {
             }
             self.id_of_last_event_in_last_extracted_update =
                 update.events.last().map(|e| e.event_id);
+            #[cfg(debug_assertions)]
+            update.assert_contiguous();
             return Ok(update);
         }
     }
 
-    /// Fetches the next page and adds it to the internal queue. Returns true if a fetch was
-    /// performed, false if there is no next page.
+    /// Fetches the next page and adds it to the internal queue.
+    /// Returns true if we still have a next page token after fetching.
     async fn get_next_page(&mut self) -> Result<bool, tonic::Status> {
         let history = loop {
             let npt = match mem::replace(&mut self.next_page_token, NextPageToken::Done) {
                 // If the last page token we got was empty, we're done.
-                NextPageToken::Done => return Ok(false),
+                NextPageToken::Done => break None,
                 NextPageToken::FetchFromStart => vec![],
                 NextPageToken::Next(v) => v,
             };
@@ -352,8 +368,18 @@ impl HistoryPaginator {
             break fetch_res.history;
         };
 
-        self.event_queue
-            .extend(history.map(|h| h.events).unwrap_or_default());
+        let queue_back_id = self
+            .event_queue
+            .back()
+            .map(|e| e.event_id)
+            .unwrap_or_default();
+        self.event_queue.extend(
+            history
+                .map(|h| h.events)
+                .unwrap_or_default()
+                .into_iter()
+                .skip_while(|e| e.event_id <= queue_back_id),
+        );
         if matches!(&self.next_page_token, NextPageToken::Done) {
             // If finished, we need to extend the queue with the final events, skipping any
             // which are already present.
@@ -366,7 +392,7 @@ impl HistoryPaginator {
                 );
             }
         };
-        Ok(true)
+        Ok(!matches!(&self.next_page_token, NextPageToken::Done))
     }
 }
 
@@ -428,6 +454,7 @@ impl HistoryUpdate {
             previous_wft_started_id: -1,
             wft_started_id: -1,
             has_last_wft: false,
+            wft_count: 0,
         }
     }
     pub fn is_real(&self) -> bool {
@@ -437,6 +464,20 @@ impl HistoryUpdate {
         self.events.get(0).map(|e| e.event_id)
     }
 
+    #[cfg(debug_assertions)]
+    fn assert_contiguous(&self) -> bool {
+        use crate::abstractions::dbg_panic;
+
+        for win in self.events.as_slice().windows(2) {
+            if let &[e1, e2] = &win {
+                if e2.event_id != e1.event_id + 1 {
+                    dbg_panic!("HistoryUpdate isn't contiguous! {:?} -> {:?}", e1, e2);
+                }
+            }
+        }
+        true
+    }
+
     /// Create an instance of an update directly from events. If the passed in event iterator has a
     /// partial WFT sequence at the end, all events after the last complete WFT sequence (ending
     /// with WFT started) are returned back to the caller, since the history update only works in
@@ -451,8 +492,11 @@ impl HistoryUpdate {
         <I as IntoIterator>::IntoIter: Send + 'static,
     {
         let mut all_events: Vec<_> = events.into_iter().collect();
-        let mut last_end =
-            find_end_index_of_next_wft_seq(all_events.as_slice(), previous_wft_started_id);
+        let mut last_end = find_end_index_of_next_wft_seq(
+            all_events.as_slice(),
+            previous_wft_started_id,
+            has_last_wft,
+        );
         if matches!(last_end, NextWFTSeqEndIndex::Incomplete(_)) {
             return if has_last_wft {
                 (
@@ -461,6 +505,7 @@ impl HistoryUpdate {
                         previous_wft_started_id,
                         wft_started_id,
                         has_last_wft,
+                        wft_count: 1,
                     },
                     vec![],
                 )
@@ -471,23 +516,32 @@ impl HistoryUpdate {
                         previous_wft_started_id,
                         wft_started_id,
                         has_last_wft,
+                        wft_count: 0,
                     },
                     all_events,
                 )
             };
         }
+        let mut wft_count = 0;
         while let NextWFTSeqEndIndex::Complete(next_end_ix) = last_end {
+            wft_count += 1;
             let next_end_eid = all_events[next_end_ix].event_id;
             // To save skipping all events at the front of this slice, only pass the relevant
             // portion, but that means the returned index must be adjusted, hence the addition.
-            let next_end = find_end_index_of_next_wft_seq(&all_events[next_end_ix..], next_end_eid)
-                .add(next_end_ix);
+            let next_end = find_end_index_of_next_wft_seq(
+                &all_events[next_end_ix..],
+                next_end_eid,
+                has_last_wft,
+            )
+            .add(next_end_ix);
             if matches!(next_end, NextWFTSeqEndIndex::Incomplete(_)) {
                 break;
             }
             last_end = next_end;
         }
-        let remaining_events = if all_events.is_empty() {
+        // If we have the last WFT, there's no point in there being "remaining" events, because
+        // they must be considered part of the last sequence
+        let remaining_events = if all_events.is_empty() || has_last_wft {
             vec![]
         } else {
             all_events.split_off(last_end.index() + 1)
@@ -499,6 +553,7 @@ impl HistoryUpdate {
                 previous_wft_started_id,
                 wft_started_id,
                 has_last_wft,
+                wft_count,
             },
             remaining_events,
         )
@@ -521,6 +576,7 @@ impl HistoryUpdate {
             previous_wft_started_id,
             wft_started_id,
             has_last_wft: true,
+            wft_count: 0,
         }
     }
 
@@ -536,7 +592,8 @@ impl HistoryUpdate {
         if let Some(ix_first_relevant) = self.starting_index_after_skipping(from_wft_started_id) {
             self.events.drain(0..ix_first_relevant);
         }
-        let next_wft_ix = find_end_index_of_next_wft_seq(&self.events, from_wft_started_id);
+        let next_wft_ix =
+            find_end_index_of_next_wft_seq(&self.events, from_wft_started_id, self.has_last_wft);
         match next_wft_ix {
             NextWFTSeqEndIndex::Incomplete(siz) => {
                 if self.has_last_wft {
@@ -577,14 +634,17 @@ impl HistoryUpdate {
         if relevant_events.is_empty() {
             return relevant_events;
         }
-        let ix_end = find_end_index_of_next_wft_seq(relevant_events, from_wft_started_id).index();
+        let ix_end =
+            find_end_index_of_next_wft_seq(relevant_events, from_wft_started_id, self.has_last_wft)
+                .index();
         &relevant_events[0..=ix_end]
     }
 
     /// Returns true if this update has the next needed WFT sequence, false if events will need to
     /// be fetched in order to create a complete update with the entire next WFT sequence.
     pub fn can_take_next_wft_sequence(&self, from_wft_started_id: i64) -> bool {
-        let next_wft_ix = find_end_index_of_next_wft_seq(&self.events, from_wft_started_id);
+        let next_wft_ix =
+            find_end_index_of_next_wft_seq(&self.events, from_wft_started_id, self.has_last_wft);
         if let NextWFTSeqEndIndex::Incomplete(_) = next_wft_ix {
             if !self.has_last_wft {
                 return false;
@@ -644,12 +704,13 @@ impl NextWFTSeqEndIndex {
 fn find_end_index_of_next_wft_seq(
     events: &[HistoryEvent],
     from_event_id: i64,
+    has_last_wft: bool,
 ) -> NextWFTSeqEndIndex {
     if events.is_empty() {
         return NextWFTSeqEndIndex::Incomplete(0);
     }
     let mut last_index = 0;
-    let mut saw_any_non_wft_event = false;
+    let mut saw_any_command_event = false;
     for (ix, e) in events.iter().enumerate() {
         last_index = ix;
 
@@ -659,15 +720,8 @@ fn find_end_index_of_next_wft_seq(
             continue;
         }
 
-        if !matches!(
-            e.event_type(),
-            EventType::WorkflowTaskFailed
-                | EventType::WorkflowTaskTimedOut
-                | EventType::WorkflowTaskScheduled
-                | EventType::WorkflowTaskStarted
-                | EventType::WorkflowTaskCompleted
-        ) {
-            saw_any_non_wft_event = true;
+        if e.is_command_event() || e.event_type() == EventType::WorkflowExecutionStarted {
+            saw_any_command_event = true;
         }
         if e.is_final_wf_execution_event() {
             return NextWFTSeqEndIndex::Complete(last_index);
@@ -696,12 +750,20 @@ fn find_end_index_of_next_wft_seq(
                         if next_next_event.event_type() == EventType::WorkflowTaskScheduled {
                             continue;
                         } else {
-                            saw_any_non_wft_event = true;
+                            return NextWFTSeqEndIndex::Complete(ix);
                         }
+                    } else if !has_last_wft && !saw_any_command_event {
+                        // Don't have enough events to look ahead of the WorkflowTaskCompleted. Need
+                        // to fetch more.
+                        continue;
                     }
                 }
+            } else if !has_last_wft && !saw_any_command_event {
+                // Don't have enough events to look ahead of the WorkflowTaskStarted. Need to fetch
+                // more.
+                continue;
             }
-            if saw_any_non_wft_event {
+            if saw_any_command_event {
                 return NextWFTSeqEndIndex::Complete(ix);
             }
         }
@@ -715,11 +777,21 @@ pub mod tests {
     use super::*;
     use crate::{
         replay::{HistoryInfo, TestHistoryBuilder},
-        test_help::canned_histories,
+        test_help::{canned_histories, hist_to_poll_resp, mock_sdk_cfg, MockPollCfg, ResponseType},
         worker::client::mocks::mock_workflow_client,
     };
+    use futures::StreamExt;
     use futures_util::TryStreamExt;
-    use temporal_sdk_core_protos::temporal::api::workflowservice::v1::GetWorkflowExecutionHistoryResponse;
+    use std::sync::atomic::{AtomicUsize, Ordering};
+    use temporal_client::WorkflowOptions;
+    use temporal_sdk::WfContext;
+    use temporal_sdk_core_protos::{
+        temporal::api::{
+            common::v1::WorkflowExecution, enums::v1::WorkflowTaskFailedCause,
+            workflowservice::v1::GetWorkflowExecutionHistoryResponse,
+        },
+        DEFAULT_WORKFLOW_TYPE,
+    };
 
     impl From<HistoryInfo> for HistoryUpdate {
         fn from(v: HistoryInfo) -> Self {
@@ -1004,14 +1076,13 @@ pub mod tests {
     }
 
     // Like the above, but if the history happens to be cut off at a wft boundary, (even though
-    // there may have been many heartbeats after we have no way of knowing about), it's going to
-    // count events 7-20 as a WFT since there is started, completed, timer command, ..heartbeats..
+    // there may have been many heartbeats after we have no way of knowing about)
     #[tokio::test]
     async fn needs_fetch_after_complete_seq_with_heartbeats() {
         let t = three_wfts_then_heartbeats();
         let mut ends_in_middle_of_seq = t.as_history_update().events;
         ends_in_middle_of_seq.truncate(20);
-        let (mut update, remaining) = HistoryUpdate::from_events(
+        let (mut update, _) = HistoryUpdate::from_events(
             ends_in_middle_of_seq,
             0,
             t.get_full_history_info()
@@ -1019,7 +1090,6 @@ pub mod tests {
                 .workflow_task_started_event_id(),
             false,
         );
-        assert!(remaining.is_empty());
         let seq = update.take_next_wft_sequence(0).unwrap_events();
         assert_eq!(seq.last().unwrap().event_id, 3);
         let seq = update.take_next_wft_sequence(3).unwrap_events();
@@ -1248,8 +1318,554 @@ pub mod tests {
         assert_eq!(seq.last().unwrap().event_id, 3);
         let seq = update.take_next_wft_sequence(3).unwrap_events();
         assert_eq!(seq.last().unwrap().event_id, 8);
-        assert_matches!(update.take_next_wft_sequence(8), NextWFT::NeedFetch);
-        let mut update = paginator.extract_next_update().await.unwrap();
         assert_matches!(update.take_next_wft_sequence(8), NextWFT::ReplayOver);
     }
+
+    #[tokio::test]
+    async fn weird_pagination_doesnt_drop_wft_events() {
+        let wf_id = "fakeid";
+        // 1: EVENT_TYPE_WORKFLOW_EXECUTION_STARTED
+        // 2: EVENT_TYPE_WORKFLOW_TASK_SCHEDULED
+        // 3: EVENT_TYPE_WORKFLOW_TASK_STARTED
+        // 4: EVENT_TYPE_WORKFLOW_TASK_COMPLETED
+        // empty page
+        // 5: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 6: EVENT_TYPE_WORKFLOW_TASK_SCHEDULED
+        // 7: EVENT_TYPE_WORKFLOW_TASK_STARTED
+        // 8: EVENT_TYPE_WORKFLOW_TASK_FAILED
+        // empty page
+        // 9: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 10: EVENT_TYPE_WORKFLOW_TASK_SCHEDULED
+        // 11: EVENT_TYPE_WORKFLOW_TASK_STARTED
+        // empty page
+        let mut t = TestHistoryBuilder::default();
+        t.add_by_type(EventType::WorkflowExecutionStarted);
+        t.add_full_wf_task();
+
+        t.add_we_signaled("hi", vec![]);
+        t.add_workflow_task_scheduled_and_started();
+        t.add_workflow_task_failed_with_failure(
+            WorkflowTaskFailedCause::UnhandledCommand,
+            Default::default(),
+        );
+
+        t.add_we_signaled("hi", vec![]);
+        t.add_workflow_task_scheduled_and_started();
+
+        let workflow_task = t.get_full_history_info().unwrap();
+        let mut wft_resp = workflow_task.as_poll_wft_response();
+        wft_resp.workflow_execution = Some(WorkflowExecution {
+            workflow_id: wf_id.to_string(),
+            run_id: t.get_orig_run_id().to_string(),
+        });
+        // Just 9/10/11 in WFT
+        wft_resp.history.as_mut().unwrap().events.drain(0..8);
+
+        let mut resp_1: GetWorkflowExecutionHistoryResponse =
+            t.get_full_history_info().unwrap().into();
+        resp_1.next_page_token = vec![1];
+        resp_1.history.as_mut().unwrap().events.truncate(4);
+
+        let mut mock_client = mock_workflow_client();
+        mock_client
+            .expect_get_workflow_execution_history()
+            .returning(move |_, _, _| Ok(resp_1.clone()))
+            .times(1);
+        mock_client
+            .expect_get_workflow_execution_history()
+            .returning(move |_, _, _| {
+                Ok(GetWorkflowExecutionHistoryResponse {
+                    history: Some(History { events: vec![] }),
+                    raw_history: vec![],
+                    next_page_token: vec![2],
+                    archived: false,
+                })
+            })
+            .times(1);
+        let mut resp_2: GetWorkflowExecutionHistoryResponse =
+            t.get_full_history_info().unwrap().into();
+        resp_2.next_page_token = vec![3];
+        resp_2.history.as_mut().unwrap().events.drain(0..4);
+        resp_2.history.as_mut().unwrap().events.truncate(4);
+        mock_client
+            .expect_get_workflow_execution_history()
+            .returning(move |_, _, _| Ok(resp_2.clone()))
+            .times(1);
+        mock_client
+            .expect_get_workflow_execution_history()
+            .returning(move |_, _, _| {
+                Ok(GetWorkflowExecutionHistoryResponse {
+                    history: Some(History { events: vec![] }),
+                    raw_history: vec![],
+                    next_page_token: vec![],
+                    archived: false,
+                })
+            })
+            .times(1);
+
+        let wf_type = DEFAULT_WORKFLOW_TYPE;
+        let mh =
+            MockPollCfg::from_resp_batches(wf_id, t, [ResponseType::Raw(wft_resp)], mock_client);
+        let mut worker = mock_sdk_cfg(mh, |cfg| {
+            cfg.max_cached_workflows = 2;
+            cfg.ignore_evicts_on_shutdown = false;
+        });
+
+        let sig_ctr = Arc::new(AtomicUsize::new(0));
+        let sig_ctr_clone = sig_ctr.clone();
+        worker.register_wf(wf_type.to_owned(), move |ctx: WfContext| {
+            let sig_ctr_clone = sig_ctr_clone.clone();
+            async move {
+                let mut sigchan = ctx.make_signal_channel("hi");
+                while sigchan.next().await.is_some() {
+                    if sig_ctr_clone.fetch_add(1, Ordering::AcqRel) == 1 {
+                        break;
+                    }
+                }
+                Ok(().into())
+            }
+        });
+
+        worker
+            .submit_wf(
+                wf_id.to_owned(),
+                wf_type.to_owned(),
+                vec![],
+                WorkflowOptions::default(),
+            )
+            .await
+            .unwrap();
+        worker.run_until_done().await.unwrap();
+        assert_eq!(sig_ctr.load(Ordering::Acquire), 2);
+    }
+
+    #[tokio::test]
+    async fn extreme_pagination_doesnt_drop_wft_events_paginator() {
+        // 1: EVENT_TYPE_WORKFLOW_EXECUTION_STARTED
+        // 2: EVENT_TYPE_WORKFLOW_TASK_SCHEDULED
+        // 3: EVENT_TYPE_WORKFLOW_TASK_STARTED // <- previous_started_event_id
+        // 4: EVENT_TYPE_WORKFLOW_TASK_COMPLETED
+
+        // 5: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 6: EVENT_TYPE_WORKFLOW_TASK_SCHEDULED
+        // 7: EVENT_TYPE_WORKFLOW_TASK_STARTED
+        // 8: EVENT_TYPE_WORKFLOW_TASK_FAILED
+
+        // 9: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 10: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 11: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 12: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 13: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 14: EVENT_TYPE_WORKFLOW_TASK_SCHEDULED
+        // 15: EVENT_TYPE_WORKFLOW_TASK_STARTED // <- started_event_id
+
+        let mut t = TestHistoryBuilder::default();
+        t.add_by_type(EventType::WorkflowExecutionStarted);
+        t.add_full_wf_task();
+
+        t.add_we_signaled("hi", vec![]);
+        t.add_workflow_task_scheduled_and_started();
+        t.add_workflow_task_failed_with_failure(
+            WorkflowTaskFailedCause::UnhandledCommand,
+            Default::default(),
+        );
+
+        t.add_we_signaled("hi", vec![]);
+        t.add_we_signaled("hi", vec![]);
+        t.add_we_signaled("hi", vec![]);
+        t.add_we_signaled("hi", vec![]);
+        t.add_we_signaled("hi", vec![]);
+        t.add_workflow_task_scheduled_and_started();
+
+        let mut mock_client = mock_workflow_client();
+
+        let events: Vec<HistoryEvent> = t.get_full_history_info().unwrap().into_events();
+        let first_event = events[0].clone();
+        for (i, event) in events.into_iter().enumerate() {
+            // Add an empty page
+            mock_client
+                .expect_get_workflow_execution_history()
+                .returning(move |_, _, _| {
+                    Ok(GetWorkflowExecutionHistoryResponse {
+                        history: Some(History { events: vec![] }),
+                        raw_history: vec![],
+                        next_page_token: vec![(i * 10) as u8],
+                        archived: false,
+                    })
+                })
+                .times(1);
+
+            // Add a page with only event i
+            mock_client
+                .expect_get_workflow_execution_history()
+                .returning(move |_, _, _| {
+                    Ok(GetWorkflowExecutionHistoryResponse {
+                        history: Some(History {
+                            events: vec![event.clone()],
+                        }),
+                        raw_history: vec![],
+                        next_page_token: vec![(i * 10 + 1) as u8],
+                        archived: false,
+                    })
+                })
+                .times(1);
+        }
+
+        // Add an extra empty page at the end, with no NPT
+        mock_client
+            .expect_get_workflow_execution_history()
+            .returning(move |_, _, _| {
+                Ok(GetWorkflowExecutionHistoryResponse {
+                    history: Some(History { events: vec![] }),
+                    raw_history: vec![],
+                    next_page_token: vec![],
+                    archived: false,
+                })
+            })
+            .times(1);
+
+        let mut paginator = HistoryPaginator::new(
+            History {
+                events: vec![first_event],
+            },
+            3,
+            15,
+            "wfid".to_string(),
+            "runid".to_string(),
+            vec![1],
+            Arc::new(mock_client),
+        );
+
+        let mut update = paginator.extract_next_update().await.unwrap();
+        let seq = update.take_next_wft_sequence(0).unwrap_events();
+        assert_eq!(seq.first().unwrap().event_id, 1);
+        assert_eq!(seq.last().unwrap().event_id, 3);
+
+        let seq = update.take_next_wft_sequence(3).unwrap_events();
+        assert_eq!(seq.first().unwrap().event_id, 4);
+        assert_eq!(seq.last().unwrap().event_id, 15);
+    }
+
+    #[tokio::test]
+    async fn extreme_pagination_doesnt_drop_wft_events_worker() {
+        let wf_id = "fakeid";
+
+        // In this test, we add empty pages between each event
+
+        // 1: EVENT_TYPE_WORKFLOW_EXECUTION_STARTED
+        // 2: EVENT_TYPE_WORKFLOW_TASK_SCHEDULED
+        // 3: EVENT_TYPE_WORKFLOW_TASK_STARTED // <- previous_started_event_id
+        // 4: EVENT_TYPE_WORKFLOW_TASK_COMPLETED
+
+        // 5: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 6: EVENT_TYPE_WORKFLOW_TASK_SCHEDULED
+        // 7: EVENT_TYPE_WORKFLOW_TASK_STARTED
+        // 8: EVENT_TYPE_WORKFLOW_TASK_FAILED
+
+        // 9: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 10: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 11: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 12: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 13: EVENT_TYPE_WORKFLOW_EXECUTION_SIGNALED
+        // 14: EVENT_TYPE_WORKFLOW_TASK_SCHEDULED
+        // 15: EVENT_TYPE_WORKFLOW_TASK_STARTED // <- started_event_id
+
+        let mut t = TestHistoryBuilder::default();
+        t.add_by_type(EventType::WorkflowExecutionStarted);
+        t.add_full_wf_task();
+
+        t.add_we_signaled("hi", vec![]);
+        t.add_workflow_task_scheduled_and_started();
+        t.add_workflow_task_failed_with_failure(
+            WorkflowTaskFailedCause::UnhandledCommand,
+            Default::default(),
+        );
+
+        t.add_we_signaled("hi", vec![]);
+        t.add_we_signaled("hi", vec![]);
+        t.add_we_signaled("hi", vec![]);
+        t.add_we_signaled("hi", vec![]);
+        t.add_we_signaled("hi", vec![]);
+        t.add_workflow_task_scheduled_and_started();
+
+        /////
+
+        let events: Vec<HistoryEvent> = t.get_full_history_info().unwrap().into_events();
+        let first_event = events[0].clone();
+
+        let mut mock_client = mock_workflow_client();
+
+        for (i, event) in events.into_iter().enumerate() {
+            // Add an empty page
+            mock_client
+                .expect_get_workflow_execution_history()
+                .returning(move |_, _, _| {
+                    Ok(GetWorkflowExecutionHistoryResponse {
+                        history: Some(History { events: vec![] }),
+                        raw_history: vec![],
+                        next_page_token: vec![(i * 10 + 1) as u8],
+                        archived: false,
+                    })
+                })
+                .times(1);
+
+            // Add a page with just event i
+            mock_client
+                .expect_get_workflow_execution_history()
+                .returning(move |_, _, _| {
+                    Ok(GetWorkflowExecutionHistoryResponse {
+                        history: Some(History {
+                            events: vec![event.clone()],
+                        }),
+                        raw_history: vec![],
+                        next_page_token: vec![(i * 10) as u8],
+                        archived: false,
+                    })
+                })
+                .times(1);
+        }
+
+        // Add an extra empty page at the end, with no NPT
+        mock_client
+            .expect_get_workflow_execution_history()
+            .returning(move |_, _, _| {
+                Ok(GetWorkflowExecutionHistoryResponse {
+                    history: Some(History { events: vec![] }),
+                    raw_history: vec![],
+                    next_page_token: vec![],
+                    archived: false,
+                })
+            })
+            .times(1);
+
+        let workflow_task = t.get_full_history_info().unwrap();
+        let mut wft_resp = workflow_task.as_poll_wft_response();
+        wft_resp.workflow_execution = Some(WorkflowExecution {
+            workflow_id: wf_id.to_string(),
+            run_id: t.get_orig_run_id().to_string(),
+        });
+        wft_resp.history = Some(History {
+            events: vec![first_event],
+        });
+        wft_resp.next_page_token = vec![1];
+        wft_resp.previous_started_event_id = 3;
+        wft_resp.started_event_id = 15;
+
+        let wf_type = DEFAULT_WORKFLOW_TYPE;
+        let mh =
+            MockPollCfg::from_resp_batches(wf_id, t, [ResponseType::Raw(wft_resp)], mock_client);
+        let mut worker = mock_sdk_cfg(mh, |cfg| {
+            cfg.max_cached_workflows = 2;
+            cfg.ignore_evicts_on_shutdown = false;
+        });
+
+        let sig_ctr = Arc::new(AtomicUsize::new(0));
+        let sig_ctr_clone = sig_ctr.clone();
+        worker.register_wf(wf_type.to_owned(), move |ctx: WfContext| {
+            let sig_ctr_clone = sig_ctr_clone.clone();
+            async move {
+                let mut sigchan = ctx.make_signal_channel("hi");
+                while sigchan.next().await.is_some() {
+                    if sig_ctr_clone.fetch_add(1, Ordering::AcqRel) == 5 {
+                        break;
+                    }
+                }
+                Ok(().into())
+            }
+        });
+
+        worker
+            .submit_wf(
+                wf_id.to_owned(),
+                wf_type.to_owned(),
+                vec![],
+                WorkflowOptions::default(),
+            )
+            .await
+            .unwrap();
+        worker.run_until_done().await.unwrap();
+        assert_eq!(sig_ctr.load(Ordering::Acquire), 6);
+    }
+
+    #[tokio::test]
+    async fn finding_end_index_with_started_as_last_event() {
+        let wf_id = "fakeid";
+        let mut t = TestHistoryBuilder::default();
+        t.add_by_type(EventType::WorkflowExecutionStarted);
+        t.add_full_wf_task();
+
+        t.add_we_signaled("hi", vec![]);
+        t.add_workflow_task_scheduled_and_started();
+        // We need to see more after this - it's not sufficient to end on a started event when
+        // we know there might be more
+
+        let workflow_task = t.get_history_info(1).unwrap();
+        let prev_started_wft_id = workflow_task.previous_started_event_id();
+        let wft_started_id = workflow_task.workflow_task_started_event_id();
+        let mut wft_resp = workflow_task.as_poll_wft_response();
+        wft_resp.workflow_execution = Some(WorkflowExecution {
+            workflow_id: wf_id.to_string(),
+            run_id: t.get_orig_run_id().to_string(),
+        });
+        wft_resp.next_page_token = vec![1];
+
+        let mut resp_1: GetWorkflowExecutionHistoryResponse =
+            t.get_full_history_info().unwrap().into();
+        resp_1.next_page_token = vec![2];
+
+        let mut mock_client = mock_workflow_client();
+        mock_client
+            .expect_get_workflow_execution_history()
+            .returning(move |_, _, _| Ok(resp_1.clone()))
+            .times(1);
+        // Since there aren't sufficient events, we should try to see another fetch, and that'll
+        // say there aren't any
+        mock_client
+            .expect_get_workflow_execution_history()
+            .returning(move |_, _, _| Ok(Default::default()))
+            .times(1);
+
+        let mut paginator = HistoryPaginator::new(
+            workflow_task.into(),
+            prev_started_wft_id,
+            wft_started_id,
+            "wfid".to_string(),
+            "runid".to_string(),
+            NextPageToken::FetchFromStart,
+            Arc::new(mock_client),
+        );
+        let mut update = paginator.extract_next_update().await.unwrap();
+        let seq = update.take_next_wft_sequence(0).unwrap_events();
+        assert_eq!(seq.last().unwrap().event_id, 3);
+        let seq = update.take_next_wft_sequence(3).unwrap_events();
+        // We're done since the last fetch revealed nothing
+        assert_eq!(seq.last().unwrap().event_id, 7);
+    }
+
+    #[tokio::test]
+    async fn just_signal_is_complete_wft() {
+        let mut t = TestHistoryBuilder::default();
+        t.add_by_type(EventType::WorkflowExecutionStarted);
+        t.add_full_wf_task();
+        t.add_we_signaled("whatever", vec![]);
+        t.add_full_wf_task();
+        t.add_we_signaled("whatever", vec![]);
+        t.add_full_wf_task();
+        t.add_workflow_execution_completed();
+
+        let workflow_task = t.get_full_history_info().unwrap();
+        let prev_started_wft_id = workflow_task.previous_started_event_id();
+        let wft_started_id = workflow_task.workflow_task_started_event_id();
+        let mock_client = mock_workflow_client();
+        let mut paginator = HistoryPaginator::new(
+            workflow_task.into(),
+            prev_started_wft_id,
+            wft_started_id,
+            "wfid".to_string(),
+            "runid".to_string(),
+            NextPageToken::Done,
+            Arc::new(mock_client),
+        );
+        let mut update = paginator.extract_next_update().await.unwrap();
+        let seq = next_check_peek(&mut update, 0);
+        assert_eq!(seq.len(), 3);
+        let seq = next_check_peek(&mut update, 3);
+        assert_eq!(seq.len(), 4);
+        let seq = next_check_peek(&mut update, 7);
+        assert_eq!(seq.len(), 4);
+        let seq = next_check_peek(&mut update, 11);
+        assert_eq!(seq.len(), 2);
+    }
+
+    #[tokio::test]
+    async fn heartbeats_then_signal() {
+        let mut t = TestHistoryBuilder::default();
+        t.add_by_type(EventType::WorkflowExecutionStarted);
+        t.add_full_wf_task();
+        t.add_full_wf_task();
+        let mut need_fetch_resp =
+            hist_to_poll_resp(&t, "wfid".to_owned(), ResponseType::AllHistory).resp;
+        need_fetch_resp.next_page_token = vec![1];
+        t.add_full_wf_task();
+        t.add_we_signaled("whatever", vec![]);
+        t.add_workflow_task_scheduled_and_started();
+
+        let full_resp: GetWorkflowExecutionHistoryResponse =
+            t.get_full_history_info().unwrap().into();
+
+        let mut mock_client = mock_workflow_client();
+        mock_client
+            .expect_get_workflow_execution_history()
+            .returning(move |_, _, _| Ok(full_resp.clone()))
+            .times(1);
+
+        let mut paginator = HistoryPaginator::new(
+            need_fetch_resp.history.unwrap(),
+            // Pretend we have already processed first WFT
+            3,
+            6,
+            "wfid".to_string(),
+            "runid".to_string(),
+            NextPageToken::Next(vec![1]),
+            Arc::new(mock_client),
+        );
+        let mut update = paginator.extract_next_update().await.unwrap();
+        // Starting past first wft
+        let seq = next_check_peek(&mut update, 3);
+        assert_eq!(seq.len(), 6);
+        let seq = next_check_peek(&mut update, 9);
+        assert_eq!(seq.len(), 4);
+    }
+
+    #[tokio::test]
+    async fn cache_miss_with_only_one_wft_available_orders_properly() {
+        let mut t = TestHistoryBuilder::default();
+        t.add_by_type(EventType::WorkflowExecutionStarted);
+        t.add_full_wf_task();
+        t.add_by_type(EventType::TimerStarted);
+        t.add_full_wf_task();
+        t.add_by_type(EventType::TimerStarted);
+        t.add_workflow_task_scheduled_and_started();
+
+        let incremental_task =
+            hist_to_poll_resp(&t, "wfid".to_owned(), ResponseType::OneTask(3)).resp;
+
+        let mut mock_client = mock_workflow_client();
+        let mut one_task_resp: GetWorkflowExecutionHistoryResponse =
+            t.get_history_info(1).unwrap().into();
+        one_task_resp.next_page_token = vec![1];
+        mock_client
+            .expect_get_workflow_execution_history()
+            .returning(move |_, _, _| Ok(one_task_resp.clone()))
+            .times(1);
+        let mut up_to_sched_start: GetWorkflowExecutionHistoryResponse =
+            t.get_full_history_info().unwrap().into();
+        up_to_sched_start
+            .history
+            .as_mut()
+            .unwrap()
+            .events
+            .truncate(9);
+        mock_client
+            .expect_get_workflow_execution_history()
+            .returning(move |_, _, _| Ok(up_to_sched_start.clone()))
+            .times(1);
+
+        let mut paginator = HistoryPaginator::new(
+            incremental_task.history.unwrap(),
+            6,
+            9,
+            "wfid".to_string(),
+            "runid".to_string(),
+            NextPageToken::FetchFromStart,
+            Arc::new(mock_client),
+        );
+        let mut update = paginator.extract_next_update().await.unwrap();
+        let seq = next_check_peek(&mut update, 0);
+        assert_eq!(seq.last().unwrap().event_id, 3);
+        let seq = next_check_peek(&mut update, 3);
+        assert_eq!(seq.last().unwrap().event_id, 7);
+        let seq = next_check_peek(&mut update, 7);
+        assert_eq!(seq.last().unwrap().event_id, 11);
+    }
 }

package/sdk-core/core/src/worker/workflow/machines/child_workflow_state_machine.rs

@@ -531,7 +531,7 @@ impl TryFrom<HistEventData> for ChildWorkflowMachineEvents {
                 Self::ChildWorkflowExecutionCancelled
             }
             _ => {
-                return Err(WFMachinesError::Fatal(format!(
+                return Err(WFMachinesError::Nondeterminism(format!(
                     "Child workflow machine does not handle this event: {e:?}"
                 )))
             }

package/sdk-core/core/src/worker/workflow/managed_run/managed_wf_test.rs

@@ -95,7 +95,7 @@ impl ManagedWFFunc {
                 run_id: "runid".to_string(),
                 history: hist,
                 metrics: MetricsContext::no_op(),
-                capabilities: &DEFAULT_TEST_CAPABILITIES,
+                capabilities: DEFAULT_TEST_CAPABILITIES,
             },
             Box::new(driver).into(),
         );

package/sdk-core/core/src/worker/workflow/managed_run.rs

@@ -1031,9 +1031,6 @@ impl ManagedRun {
         new_local_acts: Vec<LocalActRequest>,
     ) -> Result<(), WFMachinesError> {
         let immediate_resolutions = self.local_activity_request_sink.sink_reqs(new_local_acts);
-        if !immediate_resolutions.is_empty() {
-            warn!("Immediate res: {:?}", &immediate_resolutions);
-        }
         for resolution in immediate_resolutions {
             self.wfm
                 .notify_of_local_result(LocalResolution::LocalActivity(resolution))?;

package/sdk-core/sdk/src/workflow_context/options.rs

@@ -272,7 +272,7 @@ impl Signal {
 }
 
 /// Data contained within a signal
-#[derive(Default)]
+#[derive(Default, Debug)]
 pub struct SignalData {
     /// The arguments the signal will receive
     pub input: Vec<Payload>,

package/sdk-core/sdk/src/workflow_context.rs

@@ -11,7 +11,7 @@ use crate::{
     SignalExternalWfResult, TimerResult, UnblockEvent, Unblockable,
 };
 use crossbeam::channel::{Receiver, Sender};
-use futures::{task::Context, FutureExt, Stream};
+use futures::{task::Context, FutureExt, Stream, StreamExt};
 use parking_lot::RwLock;
 use std::{
     collections::HashMap,
@@ -398,6 +398,14 @@ impl DrainableSignalStream {
         }
         signals
     }
+
+    pub fn drain_ready(&mut self) -> Vec<SignalData> {
+        let mut signals = vec![];
+        while let Some(s) = self.0.next().now_or_never().flatten() {
+            signals.push(s);
+        }
+        signals
+    }
 }
 
 impl Stream for DrainableSignalStream {