booking-tags-api
CRUD API for managing booking tag definitions and applying tags to bookings in the Phenix Accounts system.
Health score
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.
Facts
| Languages | csharp |
| Frameworks | ASP.NET Core 8, Dapper 2.1, Mediator 3.0, FluentValidation 12, Serilog |
| Runtimes | .NET 8 |
| Tests / CI | some / CI in place |
| Last commit | 2026-03-25 |
| Owner | Owner unknown (mapping to be curated) |
| Domain | Bookings and fulfilment (system generated; curate in repo-owners.json to override) |
| Product area | Phenix |
| Hosted | On-prem Deployed to our own machines
|
| Releases | 2 release pipeline(s): booking-tags-api; booking-tags-api Database |
| Integration | rest |
| Source | Open in Azure DevOps |
Dependencies in context
Entanglement
loose No binding mechanisms apply: this system needs no untangling work.
Depends on / used by
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
Ready with guardrailsClose: the deployment path exists, but specific checks are needed first, typically raising tests, auditing serialized data, or respecting a shared schema with expand and contract. Test coverage is thin; ZDD shifts the safety net onto automation.
- Adopt the four-stage discipline from the ZDD engineering standard: pre-deployment compatibility, deployment, feature activation behind a flag, post-release cleanup.
- Raise automated test coverage first: ZDD trades the end-of-sprint manual gate for continuous automated confidence, so the tests are the entry ticket.
- 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.
- Audit serialized data: anything in Redis, queues or shared caches must deserialize cleanly across adjacent versions.
- Wrap behaviour changes in feature flags so code deploys dark and release becomes a business decision, not a deployment event.
- 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).
- 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 databasesqlserver:Accounts is shared, so this system cannot change engine on its own: every sharer moves in convoy. Either agree the move with all sharers as one programme, or first split this system's data out behind an owned API, then follow the steps below.
- Inventory the schema and query surface of sqlserver:Accounts: tables, stored procedures, triggers, cross-database joins and anything else engine-specific.
- Confirm the target: PostgreSQL (relational model; keep transactions and joins, swap the engine).
- 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.
- 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.
- Move reads to the target, watch correctness and latency, then cut writes over and keep SQL Server read-only until confidence is earned.
- Decommission the SQL Server database and remove the connection strings and firewall paths that pointed at it.
Containerisation
Ready to containeriseModern runtime, no hosted legacy transport: this service can follow the estate's proven container path (Dockerfile, shared pipeline template, ECS via Terraform).
- Add a Dockerfile and make the app read all configuration from environment variables and the secrets manager, never from machine-local files.
- Log to stdout and expose a health check endpoint so ECS can supervise it.
- 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.
- 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
No specific actions for this repo. It either needs only normal maintenance, or its work is covered by an estate-level action. See the full programme.
Assessment notes
booking-tags-api
Purpose
A small CRUD API for booking tags: managing per-company BookingTag definitions and applying/removing PhenixBookingTag assignments on bookings within the Phenix Accounts system.
Tech stack
- C# on .NET 8, single ASP.NET Core web project (
TC.BookingTags.Api,.slnxsolution). - Dapper over
Microsoft.Data.SqlClientcalling stored procedures; Mediator (source-generated) handlers; FluentValidation; API versioning; Swashbuckle; Serilog; PhenixAuth authentication scheme. - Connection strings resolved centrally via
TC.ConnectionBroker.Sql(CONNECTIONBROKERENVIRONMENT).
Deployment
release_count 2. Build pipelines booking-tags-api and booking-tags-api db; release pipelines booking-tags-api and booking-tags-api Database. The repo's azure-pipelines.yml uses the shared azure-pipelines-templates (dotnet SDK 8.x); the database schema changes are released via the separate Database pipeline (no SQL project is present in this repo).
Dependencies
- Internal: relies on the shared Phenix Accounts SQL Server database and its
Travel.BookingTag_*/Travel.PhenixBookingTag_*stored procedures; uses TC ConnectionBroker and PhenixAuth NuGet libraries. No confirmed source dependency on another estate repo. - External: none beyond standard telemetry/logging libraries.
Data stores
- SQL Server
Accountsdatabase (shared Phenix database), accessed via stored procedures only.
Integration points
- Versioned REST API with Swagger; authenticated with the PhenixAuth scheme. No message bus usage.
Code health
Clean, modern codebase with a single test project (TC.BookingTags.Api.Tests), CI via shared pipeline templates, health checks, and a Backstage catalog-info.yaml. Last commit 2026-03-25.
Modernisation risks
Low: current .NET 8 and tidy structure. The main risk is coupling to the shared Accounts database and stored procedures owned outside this repo, which constrains schema evolution.