RomanNum3ral
/
paperclip
mirror of https://github.com/paperclipai/paperclip.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix(runner): stop after delivered terminal results

This commit is contained in:
Dotta 2026-09-02 22:59:09 -05:00
parent c8598416ef
commit 534ce69adb
1 changed files with 34 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

41
packages/paperclip-runner/runner/crates/runner-core/src/durable/runner.rs
View File

@ -291,9 +291,21 @@ pub fn run_durable_runner<E: CommandExecutor>(
break;
}
}
if !disconnected && send_outbox(&mut transport, &state, &mut sent_source_seq).is_err() {
state.record_diagnostic("outbox delivery failed; unacknowledged suffix will replay");
disconnected = true;
if !disconnected {
if let Err(error) = send_outbox(&mut transport, &state, &mut sent_source_seq) {
state.record_diagnostic(
"outbox delivery failed; unacknowledged suffix remains durable",
);
if lifecycle_after_reply.durable_state().is_some() {
// The terminal result was delivered above. Never reconnect
// this process and overwrite its durable terminal state as
// ready merely to retry a later outbox frame.
store.save(&state)?;
let _ = executor.shutdown();
return Err(error);
}
disconnected = true;
}
}
if let Some(durable_lifecycle) = lifecycle_after_reply
.durable_state()
@ -382,16 +394,31 @@ pub fn run_durable_runner<E: CommandExecutor>(
durable_lifecycle,
)?;
}
let delivery = transport
.send_json(&command_result_envelope(&state, &result))
.and_then(|()| send_outbox(&mut transport, &state, &mut sent_source_seq));
if let Err(error) = delivery {
if let Err(error) =
transport.send_json(&command_result_envelope(&state, &result))
{
disconnected_since.get_or_insert_with(Instant::now);
state.record_diagnostic(error.to_string());
state.reconnect_count = state.reconnect_count.saturating_add(1);
store.save(&state)?;
break;
}
if let Err(error) = send_outbox(&mut transport, &state, &mut sent_source_seq) {
state.record_diagnostic(
"outbox delivery failed; unacknowledged suffix remains durable",
);
store.save(&state)?;
if lifecycle.durable_state().is_some() {
// The controller has accepted this terminal result.
// Stop even though a later outbox frame failed so a
// reconnect cannot restore the runner to ready.
let _ = executor.shutdown();
return Err(error);
}
disconnected_since.get_or_insert_with(Instant::now);
state.reconnect_count = state.reconnect_count.saturating_add(1);
break;
}
if let Some(durable_lifecycle) = lifecycle.durable_state() {
debug_assert_eq!(state.lifecycle, durable_lifecycle);
executor.shutdown()?;