[Pull Request] Dev=workflow sep#294
Merged
Merged
Conversation
…tandard+Dashboard 子模块指针
…etRequiredService 失败、ServiceLocator 与 TriggerManager 构造注入经集成测试验证通过(4工作流读取/触发器注册/RealPluginManager单例共享均正常)
Member
|
The origin title of this PR is: 'Dev=workflow sep'. GitHub Action is checking and format it ... |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.Coreproject into a new dedicatedKitX.Workflowlibrary. 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 singleAddKitXWorkflow()extension method provided by the new library. This change clarifies project boundaries and streamlines dependency management.Project structure and dependency changes:
KitX.Coreto the newKitX.Workflowproject, with namespaces updated accordingly. TheKitX.Coreproject now referencesKitX.Workflowas a project dependency. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13]Dependency injection and service registration:
CoreServiceCollectionExtensions.cshave been replaced with a single call toservices.AddKitXWorkflow(), delegating all workflow-related DI setup to the new library. This includes the registration ofRealPluginManager, block scripting, blueprint, and workflow storage services. [1] [2] [3]KitX.Workflownamespaces.Interface and contract alignment:
IDeviceHttpClientinterface is now referenced fromKitX.Core.ContractinDeviceHttpClient.cs, clarifying its contract location and usage. (F91f25b7L1)Other updates:
KitX Dashboardsubmodule reference has been updated to a new commit.These changes collectively modularize workflow functionality, improve maintainability, and simplify the dependency graph.