Support targeted workflow delivery service bindings - #3595
Conversation
Allow delivery packages to map each external connection slot to an explicit workflow YAML location so multi-service merchant workflows can stay package-driven instead of runtime-special-cased. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e4545102e2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## feature/integrate #3595 +/- ##
=====================================================
+ Coverage 83.44% 83.45% +0.01%
=====================================================
Files 1709 1709
Lines 197727 197730 +3
Branches 23735 23735
=====================================================
+ Hits 164989 165013 +24
+ Misses 21696 21677 -19
+ Partials 11042 11040 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 7 files with indirect coverage changes 🚀 New features to boost your workflow:
|
Fail closed when package connection slots target the same YAML location so each external service binding remains unambiguous through catalog, actor, and projection paths. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Fail closed when delivery connection YAML pointers overlap configuration fields, and clear omitted optional connection slots so package defaults cannot leak into installed workflows. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 participant
Summary
YamlPointerto workflow delivery connection slots so package authors can bind each external service slot to an explicit YAML location.user_service_idnode globally.Test plan
dotnet test test/Aevatar.Studio.Tests/Aevatar.Studio.Tests.csproj --no-restore --nologo -clp:ErrorsOnlybash tools/ci/test_stability_guards.shbash tools/ci/architecture_guards.sh🤖 Generated with Claude Code