Skip to content

feat: (workflow) - Exploratory bug fixes - #908

Open
WhitWaldo wants to merge 14 commits into
mainfrom
workflow-bug-fixes
Open

WhitWaldo wants to merge 14 commits into
mainfrom
workflow-bug-fixes

Conversation

@WhitWaldo

Copy link
Copy Markdown
Contributor

Description

A collection of bug fixes for workflows based on exploratory testing.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

… for each

Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
@WhitWaldo WhitWaldo added this to the v1.19 milestone Oct 19, 2026
@WhitWaldo WhitWaldo self-assigned this Oct 19, 2026
@WhitWaldo
WhitWaldo requested review from a team as code owners October 19, 2026 20:13
WhitWaldo and others added 9 commits October 19, 2026 15:20
Preserve binary-store payload handling alongside workflow reliability fixes and replay-version metadata. Cover offloaded status and output together with versioned replay over HTTP/2.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
Preserve workflow patch capability detection alongside OpenTelemetry context restoration, and retain both workflow documentation sections.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
Reconcile workflow patch support (#915) with the internal semantics marker:
the response version leads with the SDK marker on non-legacy executions and
appends the patches evaluated via ctx.isPatched().

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@codecov

codecov Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.01471% with 19 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.56%. Comparing base (839fbe4) to head (821efcb).

Files with missing lines Patch % Lines
src/workflow/engine/transport/TaskHubWorker.ts 85.71% 10 Missing and 6 partials ⚠️
...rc/workflow/engine/worker/OrchestrationExecutor.ts 95.65% 0 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #908      +/-   ##
==========================================
- Coverage   51.04%   48.56%   -2.48%     
==========================================
  Files         144      146       +2     
  Lines        4680     4859     +179     
  Branches      940      984      +44     
==========================================
- Hits         2389     2360      -29     
- Misses       2133     2376     +243     
+ Partials      158      123      -35     
Flag Coverage Δ
e2e 16.99% <11.76%> (-6.57%) ⬇️
unit 48.56% <93.01%> (+2.92%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Preserve activity concurrency limits alongside stream-scoped retries and graceful shutdown. Cover queued activity redelivery after stream failure.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
@WhitWaldo
WhitWaldo enabled auto-merge October 20, 2026 15:05
@WhitWaldo
WhitWaldo disabled auto-merge October 20, 2026 15:06

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant