From 71b35e298da35585a9bbd5383302de00a3e07650 Mon Sep 17 00:00:00 2001 From: Arvin A <51036481+DeveloperTheExplorer@users.noreply.github.com> Date: Thu, 28 May 2026 12:58:21 +0100 Subject: [PATCH] test(ai-builder): Skip flaky sequential-interrupt checkpoint test (no-changelog) (#31280) Co-authored-by: Claude Opus 4.7 (1M context) --- .../src/test/checkpoint-persistence.test.ts | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/packages/@n8n/ai-workflow-builder.ee/src/test/checkpoint-persistence.test.ts b/packages/@n8n/ai-workflow-builder.ee/src/test/checkpoint-persistence.test.ts index e193be63d9b..aabd05822f2 100644 --- a/packages/@n8n/ai-workflow-builder.ee/src/test/checkpoint-persistence.test.ts +++ b/packages/@n8n/ai-workflow-builder.ee/src/test/checkpoint-persistence.test.ts @@ -126,7 +126,12 @@ describe('LangGraph Checkpoint Message Persistence', () => { expect(contents).toContain('user-answers'); }); - it('should persist Command.update messages across two sequential interrupts and resumes', async () => { + // Skipped: flaky due to a race in @langchain/langgraph@1.0.2's + // Command.update + Command.resume + interrupt handling. On a failing run, the + // first Command.resume is dropped — step1 re-interrupts instead of returning, so + // the graph never advances to step2's final write. Tracked in AI-2531. + // eslint-disable-next-line n8n-local-rules/no-skipped-tests + it.skip('should persist Command.update messages across two sequential interrupts and resumes', async () => { // Two separate interrupt points in sequence (mimics questions interrupt → plan interrupt) const parent = new StateGraph(ParentState) .addNode('step1', (state) => {