Skip to content

[Pull Request] Dev=workflow sep#294

Merged
StardustSeemsInk merged 2 commits into
dev=UI-separate-SIfrom
dev=Workflow-Sep
Jun 17, 2026
Merged

[Pull Request] Dev=workflow sep#294
StardustSeemsInk merged 2 commits into
dev=UI-separate-SIfrom
dev=Workflow-Sep

Conversation

@StardustSeemsInk

Copy link
Copy Markdown
Member

What does this PR do?

将Workflow模块从Core中分离。但仍然是与Core平级的Clint内部子系统而不是可独立出来到Standard或SDK中的独立库。

This pull request migrates all workflow-related code (including BlockScript and Blueprint services) from the KitX.Core project into a new dedicated KitX.Workflow library. As a result, all relevant classes, service registrations, and dependencies have been updated to use the new namespace and project structure. Additionally, the core service registration is simplified by leveraging a single AddKitXWorkflow() extension method provided by the new library. This change clarifies project boundaries and streamlines dependency management.

Project structure and dependency changes:

  • All workflow and block scripting files have been moved from KitX.Core to the new KitX.Workflow project, with namespaces updated accordingly. The KitX.Core project now references KitX.Workflow as a project dependency. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13]

Dependency injection and service registration:

  • The workflow service registrations in CoreServiceCollectionExtensions.cs have been replaced with a single call to services.AddKitXWorkflow(), delegating all workflow-related DI setup to the new library. This includes the registration of RealPluginManager, block scripting, blueprint, and workflow storage services. [1] [2] [3]
  • All using statements and type references in the core and test projects have been updated to use the new KitX.Workflow namespaces.

Interface and contract alignment:

  • The IDeviceHttpClient interface is now referenced from KitX.Core.Contract in DeviceHttpClient.cs, clarifying its contract location and usage. (F91f25b7L1)

Other updates:

  • The KitX Dashboard submodule reference has been updated to a new commit.

These changes collectively modularize workflow functionality, improve maintainability, and simplify the dependency graph.


…etRequiredService 失败、ServiceLocator 与 TriggerManager 构造注入经集成测试验证通过(4工作流读取/触发器注册/RealPluginManager单例共享均正常)
@Dynesshely

Copy link
Copy Markdown
Member

The origin title of this PR is: 'Dev=workflow sep'. GitHub Action is checking and format it ...

@Dynesshely Dynesshely changed the title Dev=workflow sep [Pull Request] Dev=workflow sep Jun 17, 2026
@StardustSeemsInk StardustSeemsInk merged commit 83fbbf1 into dev=UI-separate-SI Jun 17, 2026
2 checks passed
@github-actions github-actions Bot added the merged Pull Request had been merged label Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged Pull Request had been merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants