Repository navigation
test: closed features end to end (do not merge) - #10
Draft
Svilen-Stefanov wants to merge 3 commits into
Draft
Svilen-Stefanov wants to merge 3 commits into
Svilen-Stefanov wants to merge 3 commits into
Conversation
CodeBoarding reviewStatus: 6 changed components See the full change in CodeBoarding. graph LR
n_MAUI_Client_Application["MAUI Client Application"]
n_Ordering_Service_and_Processor["Ordering Service and Processor"]
n_Identity_Service["Identity Service"]
n_Web_and_Hybrid_Client_Applications["Web and Hybrid Client Applications"]
n_Catalog_and_Event_Log_Service["Catalog and Event Log Service"]
n_Webhooks_Service_and_Client["Webhooks Service and Client"]
n_Event_Bus_Infrastructure["Event Bus Infrastructure"]
n_Basket_Service["Basket Service"]
n_Aspire_AppHost_and_Service_Defaults["Aspire AppHost and Service Defaults"]
n_Payment_Processor["Payment Processor"]
n_Payment_Processor -- "publishes payment integration events to" --> n_Event_Bus_Infrastructure
n_Ordering_Service_and_Processor -- "persists integration event outbox logs to" --> n_Catalog_and_Event_Log_Service
n_Ordering_Service_and_Processor -- "uses database migration helpers in" --> n_Webhooks_Service_and_Client
n_Ordering_Service_and_Processor -- "publishes and subscribes to order events via" --> n_Event_Bus_Infrastructure
n_Ordering_Service_and_Processor -- "configures authentication and defaults from" --> n_Aspire_AppHost_and_Service_Defaults
n_Web_and_Hybrid_Client_Applications -- "subscribes to order status events via" --> n_Event_Bus_Infrastructure
n_Web_and_Hybrid_Client_Applications -- "configures HTTP and authentication defaults usi…" --> n_Aspire_AppHost_and_Service_Defaults
n_Catalog_and_Event_Log_Service -- "uses database migration helpers in" --> n_Webhooks_Service_and_Client
n_Catalog_and_Event_Log_Service -- "publishes catalog events and subscribes via" --> n_Event_Bus_Infrastructure
n_Catalog_and_Event_Log_Service -- "calls" --> n_Basket_Service
n_Webhooks_Service_and_Client -- "subscribes to domain events via" --> n_Event_Bus_Infrastructure
n_Webhooks_Service_and_Client -- "configures HTTP client and auth defaults using" --> n_Aspire_AppHost_and_Service_Defaults
n_Event_Bus_Infrastructure -- "dispatches integration events to" --> n_Payment_Processor
n_Event_Bus_Infrastructure -- "dispatches integration events to" --> n_Ordering_Service_and_Processor
n_Event_Bus_Infrastructure -- "dispatches order status events to" --> n_Web_and_Hybrid_Client_Applications
n_Event_Bus_Infrastructure -- "dispatches stock validation events to" --> n_Catalog_and_Event_Log_Service
n_Event_Bus_Infrastructure -- "dispatches integration events to" --> n_Webhooks_Service_and_Client
n_Event_Bus_Infrastructure -- "dispatches order events to" --> n_Basket_Service
n_Basket_Service -- "queries product existence via HTTP" --> n_Catalog_and_Event_Log_Service
n_Basket_Service -- "subscribes to integration events via" --> n_Event_Bus_Infrastructure
n_Basket_Service -- "configures authentication and defaults from" --> n_Aspire_AppHost_and_Service_Defaults
classDef added fill:#1f883d,stroke:#0b5d23,color:#ffffff;
classDef modified fill:#bf8700,stroke:#7d4e00,color:#ffffff;
classDef deleted fill:#cf222e,stroke:#82071e,color:#ffffff,stroke-dasharray:5 3;
class n_Catalog_and_Event_Log_Service,n_Basket_Service modified;
linkStyle 9,18 stroke:#0b5d23,stroke-width:2px;
|
Introduce a direct Basket-to-Catalog HTTP connection with Aspire service discovery. Validate complete batches before saving and preserve baskets on catalog failures. Cover missing products, outages, cancellation, authentication and input limits. Amp-Thread-ID: https://ampcode.com/threads/T-01a0d851-f6a1-75de-9f3b-578690b1f00f Co-authored-by: Amp <amp@ampcode.com>
Catalog calls BasketPreview.Create in the Basket assembly without HTTP or the integration event bus. Keep previews read-only and reuse Basket validation. Preserve Catalog host settings when publishing the cross-service reference. Amp-Thread-ID: https://ampcode.com/threads/T-01a0d851-f6a1-75de-9f3b-578690b1f00f Co-authored-by: Amp <amp@ampcode.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Svilen-Stefanov
force-pushed
the
test/closed-features
branch
from
November 1, 2026 00:19
1a49a5c to
ecd8474
Compare
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.
End-to-end test of the closed features runner: the Action on
feat/closed-featuresagainst the dev closed functions. Carries the demo change from #8 so the architecture warnings have something real to read. Do not merge.🤖 Generated with Claude Code