Skip to content

fix: harden concurrent protocol lifecycle handling - #67

Closed
Zhoujie628 wants to merge 3 commits into
project-openan:mainfrom
Zhoujie628:main
Closed

Zhoujie628 wants to merge 3 commits into
project-openan:mainfrom
Zhoujie628:main

Conversation

@Zhoujie628

Copy link
Copy Markdown
Contributor

Closes #66

Summary

  • isolate workflow negotiation and event callbacks per invocation
  • treat raw SSE comments as transport activity without creating business events
  • make task/notification timeouts cancellable and clarify transport ownership
  • reject ambiguous client and credential configuration
  • add configurable orchestration-center connect/read timeouts
  • align direct dependencies and Chinese/English integration documentation

Verification

  • mvn -B -ntp -pl workflow-engine -Dtest=LoadPsopSearchTest,DefaultWorkflowEngineClientNegotiationTest,ObservedHttpClientTest,NotificationSubscriptionTest,ExtensionSenderTest,ExecutePsopTest,WorkflowExecutorTest test (79 passed)
  • direct and Order simulator focused regressions passed on the dev overlay
  • mvn -B -ntp -DskipTests verify passed, including sources, Javadoc, and dependency convergence

Branch policy

The PR contains transport-neutral engine and direct-transport changes only. The Order SDK response-chunk activity adapter remains isolated on the dev branch.

Signed-off-by: Zhoujie628 <zhoujie628@users.noreply.github.com>
Signed-off-by: Zhoujie628 <zhoujie628@users.noreply.github.com>
@Zhoujie628
Zhoujie628 requested a review from a team as a code owner September 13, 2026 09:26
fix: harden concurrent protocol lifecycle handling
@Zhoujie628 Zhoujie628 closed this Sep 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: harden concurrent protocol lifecycle handling

1 participant