Platform Evolution
Assessment run-2026-07-27-1 · 213 of 213 repos assessed
Repos / phenix-quote

phenix-quote

Multi-solution monorepo for the Phenix Quote product: legacy WCF quote service, the production ASP.NET Core 2.2 Quote API, an ASP.NET MVC quote UI, a .NET 8 quote-components BFF, a non-production .NET 8 rewrite, and the Quote database scripts.

35 · redhighkernelOn-premZDD: a long roadapisecrets in source

Health score

35

0 to 100 across seven weighted factors; nothing is hidden. Green is 75 and above, amber 50 to 74, red below 50. How scoring works.


Runtime currency / 25
0
.NET Framework, out of mainstream support
Legacy transport / 15
2
Hosts WCF or SOAP endpoints with no successor on modern .NET
Test posture / 15
15
Good test coverage
Security hygiene / 15
0
Credentials or keys committed to source
Dangerous coupling / 15
3
shares a core database (sqlserver:EventLog); depends on pinning component(s): phenix-feature-service, phenix-passenger-configuration-service-contract, mail-service, mailtemplate-service
Activity / 10
10
Actively maintained
Deployment hygiene / 5
5
CI in place
Health over time
2026-07-24 (run-2026-07-24-1): 352026-07-27 (run-2026-07-27-1): 35
No change since 2026-07-24 (35 to 35).

Facts

Languagescsharp, sql, typescript
FrameworksASP.NET Core 2.2, ASP.NET MVC 5, ASP.NET Core 8, WCF, MediatR, MassTransit, SignalR, EF 6, EF Core 8, Carter
Runtimes.NET Framework 4.6.2, .NET Framework 4.7.2, .NET Framework 4.8.1, .NET 8
Tests / CIgood / CI in place
Last commit2026-07-24
OwnerOwner unknown (mapping to be curated)
DomainSelling and quoting (system generated; curate in repo-owners.json to override)
Product areaPhenix
HostedOn-prem Deployed to our own machines
  • Release "\commercial\quote\phenix-quote-api": QA: deployment-group phase (agents on our own machines)
  • Release "\commercial\quote\phenix-quote-api": Staging: deployment-group phase (agents on our own machines)
  • Release "\commercial\quote\phenix-quote-api": Production: deployment-group phase (agents on our own machines)
  • Release "\commercial\quote\phenix-quote-api": Training: deployment-group phase (agents on our own machines)
  • Release "\commercial\quote\phenix-quote-api": DR: deployment-group phase (agents on our own machines)
Releases6 release pipeline(s): \commercial\quote\phenix-quote-api; \commercial\quote\phenix-quote-db; \commercial\quote\phenix-quote-service; \commercial\quote\phenix-quote-ui; \commercial\quote\phenix-quote-ui2; \flights-stream\quote-components-api
Integrationrest, wcf, masstransit, sqs, redis, signalr, jwt
SourceOpen in Azure DevOps

Dependencies in context

Laying out the graph…
greenamberreddata store (square, dashed link)Node size follows fan-in. Click any node to open it.

Entanglement

kernel Bound into the core by 3 mechanism(s).

Detaches when its pinning dependencies are cut or multi-targeted (phenix-feature-service, phenix-passenger-configuration-service-contract, mail-service, mailtemplate-service); and when its WCF surface gains a modern interface or its WCF calls move to one; and when the shared database group it belongs to gains an owned API or completes schema separation (sqlserver:EventLog).

Migration playbooks

Generated from this repo's assessment data: what is feasible, what blocks it, and the steps in order. Ratings: green means actionable now, amber means sequence other work first, red means blocked, grey means not applicable.

ZDD adoption

A long road

Technically eligible one day, but the blockers are structural: hosted WCF or SOAP contracts that cannot run two versions at once, or heavy legacy coupling. Sequence the modernisation work first. Hosts WCF or SOAP endpoints: strict serialization contracts cannot safely run two versions at once (the shared-enum trap). End-of-life runtime compounds the work. Shares sqlserver:EventLog: schema changes move in convoy.

Blockers
Dependencies and gotchas
  • 11 system(s) depend on this repo; during a rolling deploy they will talk to old and new versions at once, so responses must stay compatible
  • Shares sqlserver:EventLog with 13 system(s): destructive schema change is prohibited in a single release; expand and contract only
  • Serialized state or messages in flight (Redis or queues): new code must deserialize objects written by the previous version
  • Feature flags come from phenix-feature-service (WCF-only): flag reads themselves are a legacy dependency to watch
  • Session is externalised to Redis: keep it that way, and check session object compatibility between versions
Steps
  1. Adopt the four-stage discipline from the ZDD engineering standard: pre-deployment compatibility, deployment, feature activation behind a flag, post-release cleanup.
  2. Make every change dual-version safe: backwards-compatible APIs (new major version for breaking wire changes), expand and contract for any schema change, and consumers tolerant of unknown fields.
  3. Audit serialized data: anything in Redis, queues or shared caches must deserialize cleanly across adjacent versions.
  4. Wrap behaviour changes in feature flags so code deploys dark and release becomes a business decision, not a deployment event.
  5. Route deployments through the Workflow Engine's automated drain, deploy, health-check, return cycle (the path proven by the first automated production run in mid-2026).
  6. Verify with area-scoped regression rather than the whole-platform cycle, and watch error rates after rollout with automated rollback on alert.

SQL Server exit

Blocked by a shared core database

sqlserver:EventLog is shared, so this system cannot change engine on its own: every sharer moves in convoy. The stage 4 work (named owner, declarative schema, owned API) has to land first; after that, the steps below apply to whatever slice this system ends up owning.

Blockers
Steps
  1. Inventory the schema and query surface of sqlserver:Quote, sqlserver:EventLog: tables, stored procedures, triggers, cross-database joins and anything else engine-specific.
  2. Confirm the target: PostgreSQL (relational model; keep transactions and joins, swap the engine).
  3. Replatform to modern .NET first (see the recommended actions): current PostgreSQL and MongoDB drivers and EF Core providers target modern .NET, and doing one risky change at a time is cheaper than doing two.
  4. Data access is not through an ORM, so port the SQL by hand: translate T-SQL constructs, replace stored procedures with application code or target-native equivalents, and add integration tests around the queries that matter first.
  5. Stand up the target alongside SQL Server, backfill with an ETL pass (pgloader or an export/import job), then dual-write or replicate until parity is proven.
  6. Move reads to the target, watch correctness and latency, then cut writes over and keep SQL Server read-only until confidence is earned.
  7. Decommission the SQL Server database and remove the connection strings and firewall paths that pointed at it.

Containerisation

Feasible after replatforming

Appropriate eventually, but not from here: the legacy runtime and the hosted WCF surface both have to go first, and that work is already in this repo's recommended actions.

Blockers
Steps
  1. Replatform to modern .NET first (see the recommended actions), replacing the hosted WCF/SOAP surface with REST, gRPC or messaging so consumers can follow. Containerising then becomes routine.
  2. Add a Dockerfile and make the app read all configuration from environment variables and the secrets manager, never from machine-local files.
  3. Log to stdout and expose a health check endpoint so ECS can supervise it.
  4. Adopt the shared ci.yml pipeline template with the container parameters (awsRegion, containerAfterBuild, ecsTerraformRepo), which is exactly how the estate's existing AWS services deploy.
  5. Define the service in the Terraform repo, deploy to a non-production environment, then shift traffic and disarm the deployment-group release pipeline.

Recommended actions

Rotate the credentials committed to phenix-quote

Risk flags indicate credentials or keys in source (plaintext-credentials). Rotate now and enable secret scanning in CI; this is independent of all other work.

Impact: Closes a live security exposure at near-zero cost

proposed

Replatform phenix-quote to modern .NET

Runs on .NET Framework 4.6.2, .NET Framework 4.7.2, .NET Framework 4.8.1, .NET 8, out of mainstream support. Blocked until stage 2 cuts: phenix-feature-service, phenix-passenger-configuration-service-contract, mail-service, mailtemplate-service. Shares sqlserver:EventLog, so batch it with its database group (stage 4).

Impact: Unblocks 11 dependent system(s)

proposed

Assessment notes

phenix-quote

Purpose

The Phenix Quote product in one repo, spanning three generations of backend: the legacy WCF quote service (net462, still in production), Quote API v1 (net472 + ASP.NET Core 2.2 — the current production HTTP API, MediatR/AutoMapper/FluentValidation with a MassTransit SalesOrderClosedConsumer), and the greenfield Quote Core API v2 (.NET 8, Carter + EF Core 8, explicitly non-production). Alongside sit the Quote UI (net481 ASP.NET MVC + SignalR + Razor), a .NET 8 Quote Components API (BFF over Cart and Phenix User for booking-step/itinerary/passenger/ribbon UIs), NuGet contract/client packages and the QuoteDatabase Create/Change SQL scripts.

Tech stack

C# across net462/net472/net481 and .NET 8 (one package even multi-targets net10.0). Shared DAL (TC.Phenix.Quote.DataAccess) used by both the WCF service and API v1, with EF 6 (QuoteSystemEntities) plus ConnectionBroker-brokered SQL/Redis/HTTP/WCF connections; TC.Cache multi-level caching with Redis storage, TC.RemoteSession Redis session state, PhenixAuth JWT middleware (401s deliberately swallowed for UI flows), Newtonsoft with TypeNameHandling.All on the v1 wire format. Frontend assets via npm @tc/phenixstyles and gulp. Excellent in-repo docs: CLAUDE.md/AGENTS.md describe architecture, conventions and forbidden patterns.

Deployment

release_count 6, the widest pipeline surface in the Phenix batch: nine build-quote-*.yml YAML pipelines (API, API client, components API, contact-centre, DB, packages, service contract, WCF service, UI) with path filters, and release pipelines under \commercial\quote\ (api, db, service, ui, ui2) plus \flights-stream\quote-components-api. Only API v1 is deployed to production; the v2 Core API has no pipeline.

Dependencies

Internal (from DAL clients and NuGet references): connection broker and exceptions libraries, Phenix booking services (TC.Phenix.Services.Client → backoffice), cart, user, feature, itineraries, travel-protection, place-repository, retail-configuration, passenger-configuration, destination-management, contact-centre WCF, user-account, balance-due-date, business-travel credit check, booking-download, virtual credit card and mail/mail-template services. External: AWS SQS (MassTransit transport Phenix_QuoteMasstransit), AWS Cognito auth client, New Relic.

Data stores

SQL Server Quote database via EF 6 QuoteSystemEntities (brokered) and the QuoteDatabase script project; the DAL metric logger also writes to the shared EventLog database. Redis for the remote cache, component cacher (RedisDirect) and UI session state. App.production.config contains a full production connection string with plaintext SQL credentials for the Phenix user.

Integration points

REST (API v1 and Components API), WCF/SOAP (legacy quote service), MassTransit over SQS (sales-order-closed consumer), SignalR in the UI, Redis, JWT auth.

Code health

Good test coverage breadth: twelve-plus test projects across UI, service, API (MSTest + NUnit/SpecFlow), DAL and components (.NET 8, xUnit). Actively developed (last commit 2026-07-24) and unusually well documented via CLAUDE.md, though the README proper is boilerplate. Load-bearing quirks are documented (401-swallowing auth handler, $type serialisation).

Modernisation risks

High: the production API runs ASP.NET Core 2.2 on .NET Framework 4.7.2 (both long EOL) and the WCF service on net462 is still live; the v2 .NET 8 rewrite exists but is stalled ("out of scope until a dedicated project picks it up"), so the repo carries three backend generations concurrently. Plaintext production DB credentials are committed in config. The shared DAL couples WCF and API generations, making incremental migration hard.