TEST

.gitea/workflows/discovery-deploy.yml

@@ -0,0 +1,60 @@
+name: Build, Push and Deploy Discovery
+
+on:
+  push:
+    branches: [main]
+    paths:
+      - "discovery/**"
+      - ".gitea/workflows/discovery-deploy.yml"
+
+env:
+  AWS_REGION: us-east-1
+  ECR_REPOSITORY: discovery
+  ECS_CLUSTER_ARN: arn:aws:ecs:us-east-1:887083795404:cluster/app-cluster-384acc4
+  DISCOVERY_ECS_SERVICE_ARN: arn:aws:ecs:us-east-1:887083795404:service/app-cluster-384acc4/discovery-service-52aa4f6
+
+jobs:
+  build-push-deploy:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Configure AWS credentials
+        uses: aws-actions/configure-aws-credentials@v4
+        with:
+          aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
+          aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
+          aws-region: ${{ env.AWS_REGION }}
+
+      - name: Log in to Amazon ECR
+        id: login-ecr
+        uses: aws-actions/amazon-ecr-login@v2
+
+      - name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v3
+
+      - name: Build and push image
+        uses: docker/build-push-action@v6
+        with:
+          context: ./discovery
+          file: ./discovery/Dockerfile
+          push: true
+          tags: |
+            ${{ steps.login-ecr.outputs.registry }}/${{ env.ECR_REPOSITORY }}:latest
+            ${{ steps.login-ecr.outputs.registry }}/${{ env.ECR_REPOSITORY }}:${{ gitea.sha }}
+
+      - name: Install AWS CLI
+        run: |
+          curl "https://awscli.amazonaws.com/awscli-exe-linux-x86_64.zip" -o "awscliv2.zip"
+          unzip -q awscliv2.zip
+          sudo ./aws/install
+          rm -rf awscliv2.zip aws
+
+      - name: Deploy to ECS
+        run: |
+          aws ecs update-service \
+            --cluster "${{ env.ECS_CLUSTER_ARN }}" \
+            --service "${{ env.DISCOVERY_ECS_SERVICE_ARN }}" \
+            --force-new-deployment \
+            --region "${{ env.AWS_REGION }}"

CLAUDE.md

@@ -7,15 +7,28 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 ```
 .
 ├── cms/            Go service: video ingestion/CMS (implemented)
-├── discovery/      Go service: empty placeholder, no code yet
+├── discovery/      Go service: catalogue discovery/read side (skeleton)
 ├── infrastructure/ Pulumi (Go) program provisioning all AWS resources
 ├── tests/          Go module: Gherkin/gobdd scenarios run against a live cms
 └── .gitea/workflows/  Gitea Actions CI/CD pipelines
 ```
 
-`discovery` is provisioned in infra (its own ECR repo, ECS service, ALB,
+`discovery` is the read side. It owns no ingestion: it learns what exists by
-Postgres database/role) but has no application code yet — don't assume it's
+draining `discovery-catalogue-events`, the queue subscribed to the catalogue
-deployable.
+topic `cms` announces ready videos on, and keeps its own copy in its own
+database. Neither service reads the other's tables and nothing polls `cms`.
+
+It serves `GET /health`, `GET /api/videos/{id}`, `GET /api/videos` (the
+catalogue search — see below) and the Swagger UI. Migration `0001_init` still
+creates nothing (it predates the domain and exists only because
+`internal/db/migrate.go` embeds `migrations/*.sql`, which will not compile
+against an empty directory); `0002_create_videos_table` is where the schema
+actually starts, and `0003`–`0005` add what the search needs. It now has an `internal/services` (SQS and Redis) and an
+`internal/consumers`, and a task role scoped to that one queue — receive,
+delete, get-attributes, and nothing else. It cannot publish back onto the
+topic: it is a subscriber, not a participant. The Redis client needs nothing
+from that role: ElastiCache is reached over the Redis protocol on the private
+network, so the security group is what grants access, not IAM.
 
 ## Commands
 
@@ -29,6 +42,136 @@ go build ./...
 go vet ./...
 ```
 
+### discovery (Go 1.25, module `thamanyah/discovery`)
+
+Same commands as `cms`, and the same two entry paths (`runServer`,
+`runMigrate`) in `discovery/main.go` — only the port and the dependencies
+differ. Note the module path carries no `/v2`: unlike `cms`, this module has
+never had a v1.
+
+```bash
+cd discovery
+go run .                    # serves on :8080 (requires DB_* plus AWS_REGION/CATALOGUE_EVENTS_QUEUE_URL/REDIS_ADDR — no S3/MediaConvert)
+go run . migrate            # applies pending DB migrations, then exits (no HTTP server)
+go build ./...
+go vet ./...
+```
+
+Its OpenAPI spec is generated into `discovery/docs` by the same `swag init`
+invocation as `cms`, run from `discovery/`. In `docker-compose.yml` it is a
+service of its own on `127.0.0.1:8080`, wired to the `discovery` database and
+role LocalStack provisions, plus the AWS_* vars its consumer needs and
+`REDIS_ADDR` pointing at the `redis` container — and, like
+`cms`, started in server mode only, so a freshly created local database needs
+`docker exec discovery ./discovery migrate` once. Until that runs, the consumer
+logs `relation "videos" does not exist` per announcement and leaves them on the
+queue; they are picked up once the table exists.
+
+`discovery`'s data model is one table, `videos`: the catalogue's copy of an
+announced video, keyed by **the id `cms` issued** (no `DEFAULT` — the id
+arrives in the announcement and is reused verbatim, so both services name the
+same video the same way). `categories` is a denormalized `TEXT[]` of names:
+`cms` owns the vocabulary and announces names, so there is nothing to join to.
+`repositories.SaveVideo` is an **upsert** — the topic delivers at-least-once,
+so the same announcement can arrive twice, and a plain `INSERT` would fail on
+the second, wedging every message queued behind it. A later announcement wins.
+
+Three more columns/indexes exist purely for the search (`0003`–`0005`):
+`search_vector` is a **generated** `tsvector` column (`to_tsvector('arabic',
+title)`, `STORED`) with a GIN index; `categories` has a GIN index for the `&&`
+overlap; and `videos_recent_idx` is a btree on `(created_at DESC, id DESC)`.
+The text search configuration is `'arabic'`, not `'english'` or `'simple'`: it
+stems Arabic while leaving Latin-script tokens as written, which suits a mixed
+catalogue. The cost is that English is **not** stemmed, so "documentaries" does
+not find "documentary". It is named explicitly in the column *and* in every
+query — a query built under a different configuration stems its terms
+differently and silently matches nothing.
+
+### Catalogue search (`GET /api/videos`)
+
+Served on **GET**, not POST: a search is safe and idempotent, which POST is not,
+and its parameters fit a query string. It shares the path with
+`GET /api/videos/{id}`; `ServeMux` keeps them apart, the more specific pattern
+winning. Two things to know before touching it:
+
+- This endpoint was on **QUERY** until the method was changed to GET. The
+  parameters are now a query string, not a JSON body, so there is no
+  `api.SearchRequest` any more — `handlers.parseSearchQuery` reads
+  `r.URL.Query()` into an unexported `searchQuery`. Anything still sending a
+  JSON body gets the whole catalogue, since every parameter is optional and it
+  supplied none of them.
+- It **does** appear in the OpenAPI spec now, as `@Router /api/videos [get]`
+  with one `@Param … query` per parameter (`categories` carries
+  `collectionFormat(multi)`). That is the reason the change is worth anything
+  beyond method purity: QUERY had no `PathItem` slot in Swagger 2.0 or
+  OpenAPI 3.x, so `swag` rejected the annotation with `invalid method: QUERY`
+  and the endpoint had to be described in prose instead.
+
+`categories` is **repeated**, not comma-separated
+(`?categories=news&categories=documentary`), so a category name containing a
+comma stays one name; an empty value is dropped rather than treated as a
+category named `""`. `limit` is the only parameter that can be malformed — a
+non-numeric one is a 400 `Malformed Search`; zero or negative reads as "no
+preference" and takes the default, as omitting it does.
+
+Response `{videos, nextCursor}`.
+`title` is matched lexically, ANDing the words with the **last one as a prefix**
+(`desert & fal:*`) so a part-typed word still matches; `categories` narrows to
+videos filed under **any** of the names (`&&`, not `@>`); `limit` defaults to 20
+and is **capped at 100** rather than refused. Paging is **keyset**, not offset:
+the opaque cursor carries the last row's `(rank, created_at, id)`, so pages stay
+stable while the catalogue is being written to. The query asks for `limit+1`
+rows and hands back a cursor only when the extra row appears, which is how the
+last page reports itself as last.
+
+`tsQueryFor` builds the tsquery by splitting input on everything that is not a
+letter or digit. That drops every character tsquery gives a meaning to, which is
+what makes it safe to interpolate — `websearch_to_tsquery` parses user syntax
+but cannot express a prefix match.
+
+A measured caveat: for a **ranked** search, deep paging is not a seek. `ts_rank`
+is computed per row, so the keyset comparison lands as a filter rather than an
+index condition and every matching row is still scanned. Keyset paging still
+buys stability and avoids OFFSET's growing discard cost, but only the unranked
+browse path (no title term) is a true index seek — that is what
+`videos_recent_idx` is for, and it takes a 200k-row browse from ~15 ms to
+~0.03 ms.
+
+#### Search cache (Redis / ElastiCache)
+
+Because of that caveat, the search reads through a Redis cache before it
+touches Postgres — `handlers.cachedSearch`/`cacheSearch` around the repository
+call, storing the `api.SearchResults` a page serialises to. `REDIS_ADDR` points
+at the ElastiCache node `infrastructure/main.go` provisions (a `redis`
+container under compose). Locally it takes a repeated ranked search from
+~6.8 ms to ~0.5 ms.
+
+- **The key is `discovery:search:v1:` + a canonical JSON encoding of
+  `{t: title, c: categories, l: limit, p: cursor}`.** The cursor *is* the page
+  identity — keyset paging has no page number to key on. `limit` is in the key
+  because the same title/categories/cursor at a different limit is a different
+  set of rows. JSON rather than pasted-together separators so a category name
+  containing `,` or `|` cannot be mistaken for a boundary. The title is
+  lowercased (`tsQueryFor` folds case anyway) but the **categories are not**:
+  `categories && $2` compares them verbatim, so `News` and `news` really are
+  different searches. Categories are sorted and deduped — `&&` means "any of
+  these", so order never changed the answer — and never nil, so `"categories":
+  []` and an omitted member share an entry.
+- **TTL is 60 s and nothing invalidates on write.** The consumer writes rows
+  continuously and would have to know which cached pages a new title belongs
+  on — for a ranked search, every page it outranks. Expiry is cheaper and
+  bounds staleness to roughly the announcement's own delivery lag.
+- **Only 200s are cached**, so a malformed cursor still reaches the repository
+  and is still a 400.
+- **Every cache failure is a miss.** A Get/Set error is logged and the search
+  is answered from Postgres; the service also *starts* with an unreachable
+  cache, warning rather than panicking as the DB and SQS assertions do — a slow
+  read side beats no read side. Each call is bounded by
+  `services.cacheOperationTimeout` (100 ms, retries off), because an unbounded
+  cache miss on a dead node costs the dial timeout *plus* the query it was
+  avoiding. A total cache outage adds ~200 ms per search (a failed read and a
+  failed write), not seconds.
+
 `cms`, `discovery` and `infrastructure` hold no test files of their own. The
 only tests in the repo are the black-box BDD scenarios in `tests/` — see below.
 
@@ -98,7 +241,7 @@ on the same seams:
 | `internal/models` | the **domain types** the rest of the code passes around: `Video`, `Category`, `VideoStatus`. No JSON tags, no SQL. |
 | `internal/handlers` | HTTP: decode `api.*`, validate, call repositories/services, encode `api.*`. |
 | `internal/db` | the `*sql.DB` connection lifecycle and migrations; `internal/db/repositories` holds all SQL. |
-| `internal/services` | **AWS only** — S3 and MediaConvert. Nothing database-related lives here. |
+| `internal/services` | **AWS only** — S3, MediaConvert and SQS. Nothing database-related lives here. |
 
 `api` and `models` are deliberately separate types even where the fields look
 alike: `api.CompleteResponse` is the published schema, `models.Video` is the
@@ -226,9 +369,27 @@ it ever matters.
    `repositories.VideoRepo.CreateVideo` persists the `videos` row (status
    `"processing"`) and its `video_categories` links.
 4. Finished output lands in `encoded-bucket`, served via CloudFront.
+5. MediaConvert reports the job's state changes to an SNS topic that fans out
+   to `cms-mediaconvert-events`, which `internal/consumers` reads for the
+   lifetime of the process: it records the outcome (`ready`/`failed`) and the
+   playback URL, rewriting the `s3://` playlist path onto `PLAYBACK_BASE_URL`.
+   A job that came out **ready** is then announced on the `catalogue-events`
+   SNS topic via `services.CatalogueClient` — `{videoId, title, playbackUrl,
+   categories}`, categories by **name** so a subscriber needs nothing from
+   cms to interpret them. Failed jobs are recorded but never announced. A
+   topic, not a queue, so the announcement fans out: `discovery-catalogue-events`
+   subscribes today (raw delivery, so there is no SNS envelope to unwrap), and
+   a second reader can subscribe its own queue without cms changing. Note a
+   topic only delivers to subscriptions that exist when it publishes, so a new
+   subscriber's queue has to be in place before the announcement, not after.
+   Delivery is at-least-once: a failed publish leaves the *job* event on the
+   consumer's queue, and the outcome update is idempotent, so a redelivery
+   re-announces and a subscriber can see a video twice.
 
 Config wiring: cms reads `S3_BUCKET`, `MEDIACONVERT_INPUT_BUCKET`,
 `MEDIACONVERT_OUTPUT_BUCKET`, `MEDIACONVERT_ROLE_ARN`, `AWS_REGION`,
+`MEDIACONVERT_EVENTS_QUEUE_URL`, `CATALOGUE_EVENTS_TOPIC_ARN`,
+`PLAYBACK_BASE_URL`,
 `DB_HOST`, `DB_PORT`, `DB_NAME`, `DB_USER`, `DB_PASSWORD` from env vars
 injected by the ECS task definition (`extraEnv` and the unconditional DB_*
 vars in `deployFargateService`, `infrastructure/main.go`) — `main.go` panics
@@ -240,13 +401,25 @@ on boot if any required var is empty.
   backups — intentionally minimal). Each app (`cms`, `discovery`) gets its
   own login role and same-named database via the `postgresql` provider
   (`newServiceDatabase`), so services never share DB credentials.
+- **ElastiCache**: one `cache.t4g.micro` Redis node (`search-cache`, engine
+  7.1, single-AZ, no replica, no snapshots) that `discovery` answers repeated
+  catalogue searches from. Its contents are derivable from Postgres by
+  definition, so there is nothing to back up. Its security group admits 6379
+  from `ecs-service-sg` **only** — narrower than the database's, which also
+  admits the deployer's IP for the `postgresql` provider; there is nothing to
+  administer here from a laptop. The endpoint is the single node's address
+  (`CacheNodes[0]`, not `ConfigurationEndpoint` — that is a Memcached thing),
+  exported as `searchCacheAddress` and injected as `REDIS_ADDR`. Skipped under
+  LocalStack, where docker-compose runs a plain `redis:7-alpine` container
+  instead: ElastiCache picks its own endpoint, and compose needs a literal
+  `REDIS_ADDR` before anything is provisioned.
 - **ECS Fargate**: one cluster (`app-cluster`), one ALB *per service* (each
   gets its own DNS name rather than sharing a load balancer on different
   ports). `deployFargateService(...)` is the shared helper building a
   service's ECR repo, CloudWatch log group, task definition, ECS service,
   and ALB. `taskRole` is optional (nil = no AWS identity beyond the shared
   execution role); `extraEnv` appends container env vars beyond the DB_* set;
-  `runMigrations` (true for `cms`, false for `discovery`) adds a second,
+  `runMigrations` (true for both services) adds a second,
   non-essential `<name>-migrate` container to the task — same image,
   `command: ["migrate"]` — with the main container's `dependsOn` set to
   `condition: "COMPLETE"` on it. This is ECS's container-dependency
@@ -259,23 +432,39 @@ on boot if any required var is empty.
 - **S3 + CloudFront**: `encoded-bucket` holds finished transcoded output,
   served publicly via CloudFront using Origin Access Control (OAC) — the
   bucket itself blocks all public access; only CloudFront's OAC principal
-  can read it.
+  can read it. The distribution carries a **response headers policy** adding
+  permissive CORS headers (`*`, `GET`/`HEAD`/`OPTIONS`), and `OPTIONS` is in
+  the behaviour's allowed/cached methods so CloudFront answers preflights
+  itself: HLS is fetched by JavaScript, so a playlist or segment served
+  without `Access-Control-Allow-Origin` is discarded by the browser. The
+  bucket also carries its own equivalent CORS rule, which is what the
+  LocalStack stack relies on — there is no CloudFront there and
+  `PLAYBACK_BASE_URL` points the player straight at S3.
 - **S3 raw uploads**: `raw-uploads-bucket` is a separate, private bucket for
   pre-transcode uploads — deliberately kept apart from `encoded-bucket` so
   raw source video is never reachable through the public CDN. CORS is
   scoped to `PUT` only, from the `cms` ALB's own origin — correct back when
   cms served the upload page itself, but stale now that it serves no UI (see
   Known gaps).
-- **IAM roles** — four distinct roles/users, each scoped narrowly, don't
+- **IAM roles** — five distinct roles/users, each scoped narrowly, don't
   conflate them:
   - `ecs-task-execution-role` — shared by both services' ECS *agent* (image
     pull, log write, Secrets Manager read for DB password). Not usable by
     application code inside the container.
   - `cms-task-role` — the `cms` container's own AWS identity: S3
     `ListBucket`/`PutObject`/`GetObject` on `raw-uploads-bucket` only,
-    `mediaconvert:CreateJob`, and `iam:PassRole` scoped to the MediaConvert
+    `mediaconvert:CreateJob`, `iam:PassRole` scoped to the MediaConvert
-    service role (`iam:PassedToService` condition). `discovery` has no task
+    service role (`iam:PassedToService` condition), receive/delete on
-    role — it doesn't touch S3 or MediaConvert.
+    `cms-mediaconvert-events`, and `sns:Publish` on the `catalogue-events`
+    topic — the only thing it writes to. It has no access to
+    `discovery-catalogue-events`, the queue subscribed to that topic: cms
+    publishes, it does not reach into a subscriber.
+  - `discovery-task-role` — the `discovery` container's own AWS identity, and
+    its only one: receive/delete/get-attributes on
+    `discovery-catalogue-events`. No S3, no MediaConvert, and no `sns:Publish`
+    — it consumes the catalogue, it does not add to it. The search cache is
+    absent from it on purpose: ElastiCache is reached over the Redis protocol
+    inside the VPC, so `search-cache-sg` is the grant, not IAM.
   - `mediaconvert-service-role` — trusted by `mediaconvert.amazonaws.com`,
     not by ECS; the role MediaConvert itself assumes (passed as
     `CreateJobInput.Role`) to read `raw-uploads-bucket` and write
@@ -295,6 +484,11 @@ on boot if any required var is empty.
   a third-party action), then `aws ecs update-service --force-new-deployment`.
   Cluster/service ARNs come from `pulumi stack output` and are set as repo
   *variables* (not secrets — ARNs aren't sensitive).
+- **`discovery-deploy.yml`**: the same pipeline for `discovery`, on pushes
+  touching `discovery/**`. One difference: the ECS service ARN is read from
+  the `DISCOVERY_ECS_SERVICE_ARN` repo variable rather than pinned in the
+  workflow, so it has to be set (from `pulumi stack output
+  discoveryServiceArn`) before the first deploy can succeed.
 - **`infrastructure-deploy.yml`**: push to `main` touching
   `infrastructure/**`. Runs `pulumi up` using a separate, broader AWS
   credential (`PULUMI_AWS_ACCESS_KEY_ID`/`SECRET`) than `gitea-ci-user`,

README.md

@@ -1 +1 @@
-Hello
+Hello There

cms/go.mod

@@ -3,7 +3,7 @@ module thamanyah/cms/v2
 go 1.25.12
 
 require (
-	github.com/aws/aws-sdk-go-v2 v1.44.0
+	github.com/aws/aws-sdk-go-v2 v1.45.1
 	github.com/aws/aws-sdk-go-v2/config v1.32.36
 	github.com/aws/aws-sdk-go-v2/service/mediaconvert v1.97.2
 	github.com/aws/aws-sdk-go-v2/service/s3 v1.107.1
@@ -20,14 +20,15 @@ require (
 	github.com/aws/aws-sdk-go-v2/aws/protocol/eventstream v1.7.17 // indirect
 	github.com/aws/aws-sdk-go-v2/credentials v1.19.35 // indirect
 	github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.18.36 // indirect
-	github.com/aws/aws-sdk-go-v2/internal/configsources v1.4.40 // indirect
+	github.com/aws/aws-sdk-go-v2/internal/configsources v1.5.1 // indirect
-	github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.7.40 // indirect
+	github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.8.1 // indirect
 	github.com/aws/aws-sdk-go-v2/internal/v4a v1.4.37 // indirect
 	github.com/aws/aws-sdk-go-v2/service/internal/accept-encoding v1.13.16 // indirect
 	github.com/aws/aws-sdk-go-v2/service/internal/checksum v1.9.29 // indirect
 	github.com/aws/aws-sdk-go-v2/service/internal/presigned-url v1.13.36 // indirect
 	github.com/aws/aws-sdk-go-v2/service/internal/s3shared v1.19.37 // indirect
 	github.com/aws/aws-sdk-go-v2/service/signin v1.5.5 // indirect
+	github.com/aws/aws-sdk-go-v2/service/sns v1.44.1 // indirect
 	github.com/aws/aws-sdk-go-v2/service/sso v1.33.5 // indirect
 	github.com/aws/aws-sdk-go-v2/service/ssooidc v1.38.5 // indirect
 	github.com/aws/aws-sdk-go-v2/service/sts v1.45.5 // indirect

cms/go.sum

@@ -6,6 +6,8 @@ github.com/Microsoft/go-winio v0.6.2 h1:F2VQgta7ecxGYO8k3ZZz3RS8fVIXVxONVUPlNERo
 github.com/Microsoft/go-winio v0.6.2/go.mod h1:yd8OoFMLzJbo9gZq8j5qaps8bJ9aShtEA8Ipt1oGCvU=
 github.com/aws/aws-sdk-go-v2 v1.44.0 h1:4IbaHhtzy+4h37z4JQyO9a2QsiCml3CNYHtq5hIHigo=
 github.com/aws/aws-sdk-go-v2 v1.44.0/go.mod h1:bttEH6JqnUL8LepvDVfdrds/fZ5bCIxzpe3abyUrhDU=
+github.com/aws/aws-sdk-go-v2 v1.45.1 h1:iIoG3NaLhV6UZpPXyPXlDj2I9oS8tV/nMcMnITCC6Ks=
+github.com/aws/aws-sdk-go-v2 v1.45.1/go.mod h1:bttEH6JqnUL8LepvDVfdrds/fZ5bCIxzpe3abyUrhDU=
 github.com/aws/aws-sdk-go-v2/aws/protocol/eventstream v1.7.17 h1:mn+Vxb9zgz/FE/yDTcFim3DZ1qpcrxR+qBQkBrl6bzA=
 github.com/aws/aws-sdk-go-v2/aws/protocol/eventstream v1.7.17/go.mod h1:eDfmEFxu+BSVsUGLbzJhWjpOurv1mqczClS97yI8wdk=
 github.com/aws/aws-sdk-go-v2/config v1.32.36 h1:mX6ietU7UlB4w/2IUaexJdsyUDvhTd+jYPjVePiyi6s=
@@ -16,8 +18,12 @@ github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.18.36 h1:gucL1KH/PAYbpTpBg09CiV
 github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.18.36/go.mod h1:usTB+PHhNMhrx2dxUeHcM7OrT5pySvmjYI++IsefPN0=
 github.com/aws/aws-sdk-go-v2/internal/configsources v1.4.40 h1:UIXlbijuB2XK1Kr57fo8iIxCuaSHJzwZ1uo+2tbEYIk=
 github.com/aws/aws-sdk-go-v2/internal/configsources v1.4.40/go.mod h1:wcEsL6jscjZjVUinb0Q5qD/GXOG1yT3GNfmT9HuDwzU=
+github.com/aws/aws-sdk-go-v2/internal/configsources v1.5.1 h1:pc138gM1CW+XPc60rEwUlwwuwWFQK16CI1T7v1F9Oec=
+github.com/aws/aws-sdk-go-v2/internal/configsources v1.5.1/go.mod h1:1+koxpPIbfBdfzP6vojm5/zTpTQ/micYwlxIiNB3TxI=
 github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.7.40 h1:xLQVRDs2NddDmK9BEyh5KSlJ1Gpy5/GIJXrV6WcVGAE=
 github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.7.40/go.mod h1:XRXnpFVFGLaEVK+olDdFIM1vNa04ETW452oFGEPUxAo=
+github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.8.1 h1:K0JsbZQj+1h208Ro1zHeA4l7bMp0NvRffHQ91q8Ol1s=
+github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.8.1/go.mod h1:W3/vL6EtCIatICGy9ab29QhMuae+cOKPWcMxv02CO+Q=
 github.com/aws/aws-sdk-go-v2/internal/v4a v1.4.37 h1:oyd3ke4V9AhKcRR7rRgxk1VyI+DjK2CBQtbxh3OkdaA=
 github.com/aws/aws-sdk-go-v2/internal/v4a v1.4.37/go.mod h1:aA9D7SqfG9IC1b7FLD7Iyc8Q4JN0a8gHhNjN4zPlIaI=
 github.com/aws/aws-sdk-go-v2/service/internal/accept-encoding v1.13.16 h1:iE4NGbvqUZnHDqddQAauZzCILYtFjOHwRM5MOOKLB5A=
@@ -34,6 +40,8 @@ github.com/aws/aws-sdk-go-v2/service/s3 v1.107.1 h1:VUTtUJMuRNMkb/7NIKmd8NQaeQLP
 github.com/aws/aws-sdk-go-v2/service/s3 v1.107.1/go.mod h1:WvUaO0lP5GNMs1R6cs6qvB3mqo16GLta8yfOuf55Rpc=
 github.com/aws/aws-sdk-go-v2/service/signin v1.5.5 h1:0VTFBfOgPJrUSpGMgzoi8qLcXF5dbmiBuxpo14eBWUw=
 github.com/aws/aws-sdk-go-v2/service/signin v1.5.5/go.mod h1:sNZYlBxoohYMBYl47BO/bFtAM6I8HSsPa1qwwPPRGoQ=
+github.com/aws/aws-sdk-go-v2/service/sns v1.44.1 h1:dMIcbUQ8fPJPbX9jZV19JtL2lCAgEh1LUNlpe6sdgqE=
+github.com/aws/aws-sdk-go-v2/service/sns v1.44.1/go.mod h1:ucBUMGW8avqGmbyQoXyoC6Cgt+WsNBrhL9DA4Bb+jN4=
 github.com/aws/aws-sdk-go-v2/service/sqs v1.47.0 h1:vNsYthHgT4sUo0KVqpkZlz+8ZDqy/MdlqdvZdP6IoAc=
 github.com/aws/aws-sdk-go-v2/service/sqs v1.47.0/go.mod h1:FSB4mnod1TCBhs3vp2tWVVGHbqxluzA0Fo6LBOXZByw=
 github.com/aws/aws-sdk-go-v2/service/sso v1.33.5 h1:jDQARFp1mJ2PEnllQf01nfFXGfWMJ59e0/HCHUTTZCk=

cms/internal/consumers/mediaconvert.go

@@ -187,7 +187,7 @@ func (c MediaConvertEvents) handleJobEvent(ctx context.Context, body string) boo
 		}
 	}
 
-	err := repositories.VideoRepo.UpdateVideoOutcomeByJobID(ctx, event.Detail.JobID, status, playbackURL)
+	video, err := repositories.VideoRepo.UpdateVideoOutcomeByJobID(ctx, event.Detail.JobID, status, playbackURL)
 	if errors.Is(err, repositories.ErrVideoNotFound) {
 		// Not a fault: the topic carries every job in the account, including
 		// ones this service never submitted.
@@ -202,5 +202,29 @@ func (c MediaConvertEvents) handleJobEvent(ctx context.Context, body string) boo
 	}
 
 	log.Printf("Recorded A Job Outcome: job=%q status=%q playback=%q", event.Detail.JobID, status, playbackURL)
+
+	// Only a video that came out ready is worth showing, so only that one is
+	// announced. A failed job has already had its status recorded above, and
+	// the catalogue has no use for it.
+	if status != models.VideoStatusReady {
+		return true
+	}
+
+	if err := services.CatalogueClient.AnnounceVideo(ctx, services.CatalogueVideo{
+		VideoID:     video.ID,
+		Title:       video.Title,
+		PlaybackURL: video.PlaybackURL,
+		Categories:  video.CategoryNames,
+	}); err != nil {
+		// Worth another attempt: the outcome is already recorded, and the
+		// update is idempotent, so a redelivery re-runs it and tries the
+		// announcement again. The cost is that a consumer can see the same
+		// video twice, which is what at-least-once delivery means anyway.
+		log.Printf("Something Went Wrong Announcing A Ready Video: video=%q job=%q: %s",
+			video.ID, event.Detail.JobID, err)
+		return false
+	}
+
+	log.Printf("Announced A Ready Video: video=%q playback=%q", video.ID, video.PlaybackURL)
 	return true
 }

cms/internal/db/repositories/videos.go

@@ -21,25 +21,52 @@ var VideoRepo VideoRepository
 // is cleared along with the status. It returns ErrVideoNotFound when no video
 // carries that job id, which the consumer treats as a message to drop rather
 // than a fault.
-func (svc VideoRepository) UpdateVideoOutcomeByJobID(ctx context.Context, jobID string, status models.VideoStatus, playbackURL string) error {
+//
-	result, err := svc.SQLDB.ExecContext(ctx, `
+// It returns the video it updated, carrying the members an announcement is
+// built from — the row's id and title, and the names of the categories it is
+// filed under. The consumer would otherwise have to read back what it just
+// wrote, and reading it here keeps that SQL on this side of the seam.
+func (svc VideoRepository) UpdateVideoOutcomeByJobID(ctx context.Context, jobID string, status models.VideoStatus, playbackURL string) (models.Video, error) {
+	updated := models.Video{
+		Status:            status,
+		PlaybackURL:       playbackURL,
+		MediaConvertJobID: jobID,
+	}
+
+	err := svc.SQLDB.QueryRowContext(ctx, `
 		UPDATE videos
 		SET status = $1, playback_url = $2, updated_at = now()
 		WHERE mediaconvert_job_id = $3
-	`, string(status), playbackURL, jobID)
+		RETURNING id, title
+	`, string(status), playbackURL, jobID).Scan(&updated.ID, &updated.Title)
+	if errors.Is(err, sql.ErrNoRows) {
+		return models.Video{}, ErrVideoNotFound
+	}
 	if err != nil {
-		return err
+		return models.Video{}, err
 	}
 
-	affected, err := result.RowsAffected()
+	rows, err := svc.SQLDB.QueryContext(ctx, `
+		SELECT c.name
+		FROM video_categories vc
+		JOIN categories c ON c.id = vc.category_id
+		WHERE vc.video_id = $1
+		ORDER BY c.name
+	`, updated.ID)
 	if err != nil {
-		return err
+		return models.Video{}, err
 	}
-	if affected == 0 {
+	defer rows.Close()
-		return ErrVideoNotFound
+
+	for rows.Next() {
+		var name string
+		if err := rows.Scan(&name); err != nil {
+			return models.Video{}, err
+		}
+		updated.CategoryNames = append(updated.CategoryNames, name)
 	}
 
-	return nil
+	return updated, rows.Err()
 }
 
 // GetVideoByID reads one video and the ids of the categories it is filed

cms/internal/models/video.go

@@ -3,10 +3,14 @@ package models
 import "time"
 
 type Video struct {
-	ID                string
+	ID          string
-	Title             string
+	Title       string
-	Description       string
+	Description string
-	CategoryIDs       []int16
+	CategoryIDs []int16
+	// CategoryNames is the same filing read by name rather than id. It is
+	// filled only where a caller needs categories to be self-describing —
+	// the catalogue announcement — and is empty elsewhere.
+	CategoryNames     []string
 	Tags              string
 	FileName          string
 	StorageKey        string

cms/internal/services/catalogue.go

@@ -0,0 +1,75 @@
+package services
+
+import (
+	"context"
+	"encoding/json"
+	"fmt"
+
+	"github.com/aws/aws-sdk-go-v2/aws"
+	"github.com/aws/aws-sdk-go-v2/service/sns"
+)
+
+var CatalogueClient Catalogue
+
+// CatalogueVideo is what cms announces when a video becomes ready. It is a
+// published contract — every subscriber of the catalogue topic decodes it —
+// so the JSON names here are as much an API as the ones in internal/api, and
+// changing one is a change every subscriber sees.
+//
+// Categories are named rather than numbered on purpose: the ids are cms's
+// own, and a consumer holding a name needs nothing from cms to make sense
+// of it.
+type CatalogueVideo struct {
+	VideoID     string   `json:"videoId"`
+	Title       string   `json:"title"`
+	PlaybackURL string   `json:"playbackUrl"`
+	Categories  []string `json:"categories"`
+}
+
+type Catalogue interface {
+	AnnounceVideo(ctx context.Context, video CatalogueVideo) error
+}
+
+type CatalogueConcrete struct {
+	SNSClient *sns.Client
+	TopicARN  string
+}
+
+// AnnounceVideo publishes one video to the catalogue topic, which fans it out
+// to whatever has subscribed. Delivery is at-least-once: the caller
+// republishes when a later step fails, so a subscriber has to tolerate seeing
+// the same video twice.
+//
+// A topic delivers only to the subscriptions that exist at the moment it
+// publishes, so a subscriber that wants the backlog needs its queue in place
+// before the announcement, not after.
+func (svc CatalogueConcrete) AnnounceVideo(ctx context.Context, video CatalogueVideo) error {
+	// A nil slice marshals to null, which would make "has no categories" and
+	// "categories unknown" indistinguishable to a reader. An empty list says
+	// the first, which is what an unfiled video means.
+	if video.Categories == nil {
+		video.Categories = []string{}
+	}
+
+	body, err := json.Marshal(video)
+	if err != nil {
+		return err
+	}
+
+	_, err = svc.SNSClient.Publish(ctx, &sns.PublishInput{
+		TopicArn: aws.String(svc.TopicARN),
+		Message:  aws.String(string(body)),
+	})
+	return err
+}
+
+// AssertSuccessfulConnection verifies the topic exists and is readable with
+// the configured credentials. It panics on failure: a service that cannot
+// announce would transcode videos the catalogue never hears about.
+func (svc CatalogueConcrete) AssertSuccessfulConnection(ctx context.Context) {
+	if _, err := svc.SNSClient.GetTopicAttributes(ctx, &sns.GetTopicAttributesInput{
+		TopicArn: aws.String(svc.TopicARN),
+	}); err != nil {
+		panic(fmt.Errorf("sns: cannot connect to catalogue topic %q: %w", svc.TopicARN, err))
+	}
+}

cms/main.go

@@ -15,6 +15,7 @@ import (
 	"github.com/aws/aws-sdk-go-v2/config"
 	"github.com/aws/aws-sdk-go-v2/service/mediaconvert"
 	"github.com/aws/aws-sdk-go-v2/service/s3"
+	"github.com/aws/aws-sdk-go-v2/service/sns"
 	"github.com/aws/aws-sdk-go-v2/service/sqs"
 	_ "github.com/lib/pq"
 	httpSwagger "github.com/swaggo/http-swagger/v2"
@@ -73,6 +74,13 @@ func runServer() {
 		panic(fmt.Errorf("missing required env var: MEDIACONVERT_EVENTS_QUEUE_URL"))
 	}
 
+	// The topic ready videos are announced on, which fans out to whatever has
+	// subscribed — the read side's queue, today.
+	catalogueEventsTopicARN := os.Getenv("CATALOGUE_EVENTS_TOPIC_ARN")
+	if catalogueEventsTopicARN == "" {
+		panic(fmt.Errorf("missing required env var: CATALOGUE_EVENTS_TOPIC_ARN"))
+	}
+
 	// The public host the encoded output is served from — the CloudFront
 	// distribution in front of the output bucket. MediaConvert reports where
 	// it wrote as an s3:// path into a bucket that blocks public access, so
@@ -117,6 +125,14 @@ func runServer() {
 
 	services.SQSClient = concreteSQSClient
 
+	concreteCatalogueClient := &services.CatalogueConcrete{
+		SNSClient: sns.NewFromConfig(awsConfig),
+		TopicARN:  catalogueEventsTopicARN,
+	}
+	concreteCatalogueClient.AssertSuccessfulConnection(context.Background())
+
+	services.CatalogueClient = concreteCatalogueClient
+
 	concreteDBClient := db.CreateDBConnection(requireDBConnectionString())
 	defer db.CloseConnection(concreteDBClient)
 	db.AssertSuccessfulConnection(context.Background(), concreteDBClient)

discovery/.gitignore

@@ -0,0 +1,33 @@
+# Binaries
+*.exe
+*.exe~
+*.dll
+*.so
+*.dylib
+
+# Test binary, built with `go test -c`
+*.test
+
+# Output of the go coverage tool
+*.out
+coverage.txt
+
+# Dependency directories
+vendor/
+
+# Go workspace file
+go.work
+go.work.sum
+
+# Env files
+.env
+.env.local
+
+# Build output
+/bin/
+/dist/
+
+# IDE
+.idea/
+.vscode/
+*.swp

discovery/Dockerfile

@@ -0,0 +1,19 @@
+# syntax=docker/dockerfile:1
+
+FROM golang:1.25-alpine AS builder
+WORKDIR /src
+
+COPY go.mod go.sum ./
+RUN go mod download
+
+COPY . .
+RUN CGO_ENABLED=0 GOOS=linux go build -o /out/discovery .
+
+FROM alpine:3.20
+RUN apk add --no-cache ca-certificates
+
+WORKDIR /app
+COPY --from=builder /out/discovery ./discovery
+
+EXPOSE 8080
+ENTRYPOINT ["./discovery"]

discovery/docs/docs.go

@@ -0,0 +1,238 @@
+// Package docs Code generated by swaggo/swag. DO NOT EDIT
+package docs
+
+import "github.com/swaggo/swag"
+
+const docTemplate = `{
+    "schemes": {{ marshal .Schemes }},
+    "swagger": "2.0",
+    "info": {
+        "description": "{{escape .Description}}",
+        "title": "{{.Title}}",
+        "contact": {},
+        "version": "{{.Version}}"
+    },
+    "host": "{{.Host}}",
+    "basePath": "{{.BasePath}}",
+    "paths": {
+        "/api/videos": {
+            "get": {
+                "description": "Finds catalogued videos by the words of their title and the categories they are filed under, most relevant first. Every parameter is optional — a request with none of them browses the whole catalogue, newest first. Paging is by cursor rather than by offset, so a page stays the same page while videos are being announced into the catalogue around it.",
+                "produces": [
+                    "application/json"
+                ],
+                "tags": [
+                    "videos"
+                ],
+                "summary": "Search the catalogue",
+                "parameters": [
+                    {
+                        "type": "string",
+                        "example": "desert fal",
+                        "description": "Matched lexically against video titles: the words are all required, and the last is treated as a prefix so a part-typed word still finds something.",
+                        "name": "title",
+                        "in": "query"
+                    },
+                    {
+                        "type": "array",
+                        "items": {
+                            "type": "string"
+                        },
+                        "collectionFormat": "multi",
+                        "example": "documentary",
+                        "description": "Narrows the search to videos filed under any one of these names — not all of them. Repeat the parameter per category. Omitted means every category.",
+                        "name": "categories",
+                        "in": "query"
+                    },
+                    {
+                        "type": "integer",
+                        "default": 20,
+                        "description": "How many videos to return. Absent, zero or negative takes the default of 20; anything above the maximum of 100 is capped to it rather than refused, and the cursor still reaches the rest.",
+                        "name": "limit",
+                        "in": "query"
+                    },
+                    {
+                        "type": "string",
+                        "description": "Asks for the page after the one a previous search ended at. Send back the nextCursor from that search unchanged; it is opaque, and the only thing to do with it is return it.",
+                        "name": "cursor",
+                        "in": "query"
+                    }
+                ],
+                "responses": {
+                    "200": {
+                        "description": "OK",
+                        "schema": {
+                            "$ref": "#/definitions/api.SearchResults"
+                        }
+                    },
+                    "400": {
+                        "description": "Bad Request",
+                        "schema": {
+                            "$ref": "#/definitions/api.ProblemDetails"
+                        }
+                    },
+                    "500": {
+                        "description": "Internal Server Error",
+                        "schema": {
+                            "$ref": "#/definitions/api.ProblemDetails"
+                        }
+                    }
+                }
+            }
+        },
+        "/api/videos/{id}": {
+            "get": {
+                "description": "Returns the catalogue's copy of a video: what a reader needs to show and play it. A video appears here only after the CMS has announced it as ready, so a video that is still transcoding — or one the CMS never made ready — is a 404.",
+                "produces": [
+                    "application/json"
+                ],
+                "tags": [
+                    "videos"
+                ],
+                "summary": "Get a catalogued video",
+                "parameters": [
+                    {
+                        "type": "string",
+                        "description": "Video id, as issued by the CMS",
+                        "name": "id",
+                        "in": "path",
+                        "required": true
+                    }
+                ],
+                "responses": {
+                    "200": {
+                        "description": "OK",
+                        "schema": {
+                            "$ref": "#/definitions/api.Video"
+                        }
+                    },
+                    "404": {
+                        "description": "Not Found",
+                        "schema": {
+                            "$ref": "#/definitions/api.ProblemDetails"
+                        }
+                    },
+                    "500": {
+                        "description": "Internal Server Error",
+                        "schema": {
+                            "$ref": "#/definitions/api.ProblemDetails"
+                        }
+                    }
+                }
+            }
+        },
+        "/health": {
+            "get": {
+                "description": "Reports that the service is up and serving. Used as the ALB target group health check; it does not verify the database connection.",
+                "produces": [
+                    "application/json"
+                ],
+                "tags": [
+                    "system"
+                ],
+                "summary": "Health check",
+                "responses": {
+                    "200": {
+                        "description": "OK",
+                        "schema": {
+                            "$ref": "#/definitions/api.HealthResponse"
+                        }
+                    }
+                }
+            }
+        }
+    },
+    "definitions": {
+        "api.HealthResponse": {
+            "type": "object",
+            "properties": {
+                "status": {
+                    "type": "string",
+                    "example": "ok"
+                }
+            }
+        },
+        "api.ProblemDetails": {
+            "type": "object",
+            "properties": {
+                "detail": {
+                    "type": "string",
+                    "example": "The catalogue could not be read from the database. This is a server-side fault and the request was not processed; retrying in a few moments may succeed."
+                },
+                "status": {
+                    "type": "integer",
+                    "example": 500
+                },
+                "title": {
+                    "type": "string",
+                    "example": "Something Went Wrong Loading The Catalogue"
+                },
+                "type": {
+                    "type": "string",
+                    "example": "about:blank"
+                }
+            }
+        },
+        "api.SearchResults": {
+            "type": "object",
+            "properties": {
+                "nextCursor": {
+                    "description": "NextCursor reaches the page after this one. Empty on the last page,\nwhich is how a reader knows there is no more to ask for.",
+                    "type": "string"
+                },
+                "videos": {
+                    "description": "Videos are the matches, most relevant first. Never null: a search that\nmatches nothing is an empty list.",
+                    "type": "array",
+                    "items": {
+                        "$ref": "#/definitions/api.Video"
+                    }
+                }
+            }
+        },
+        "api.Video": {
+            "type": "object",
+            "properties": {
+                "categories": {
+                    "type": "array",
+                    "items": {
+                        "type": "string"
+                    },
+                    "example": [
+                        "documentary",
+                        "news"
+                    ]
+                },
+                "id": {
+                    "type": "string",
+                    "example": "0f8fad5b-d9cb-469f-a165-70867728950e"
+                },
+                "playbackUrl": {
+                    "type": "string",
+                    "example": "https://d111111abcdef8.cloudfront.net/videos/abc123/index.m3u8"
+                },
+                "title": {
+                    "type": "string",
+                    "example": "The Evening Bulletin"
+                }
+            }
+        }
+    }
+}`
+
+// SwaggerInfo holds exported Swagger Info so clients can modify it
+var SwaggerInfo = &swag.Spec{
+	Version:          "1.0",
+	Host:             "",
+	BasePath:         "/",
+	Schemes:          []string{},
+	Title:            "Thamanyah Discovery API",
+	Description:      "JSON API for browsing the Thamanyah catalogue. Read-side counterpart to the CMS, which is what ingests videos.\n\nThe catalogue search is `GET /api/videos`, listed below: title is matched lexically with the last word as a prefix, categories narrow to videos filed under any one of them, limit defaults to 20 and is capped at 100, and cursor is the opaque `nextCursor` of a previous search.",
+	InfoInstanceName: "swagger",
+	SwaggerTemplate:  docTemplate,
+	LeftDelim:        "{{",
+	RightDelim:       "}}",
+}
+
+func init() {
+	swag.Register(SwaggerInfo.InstanceName(), SwaggerInfo)
+}

discovery/docs/swagger.json

@@ -0,0 +1,213 @@
+{
+    "swagger": "2.0",
+    "info": {
+        "description": "JSON API for browsing the Thamanyah catalogue. Read-side counterpart to the CMS, which is what ingests videos.\n\nThe catalogue search is `GET /api/videos`, listed below: title is matched lexically with the last word as a prefix, categories narrow to videos filed under any one of them, limit defaults to 20 and is capped at 100, and cursor is the opaque `nextCursor` of a previous search.",
+        "title": "Thamanyah Discovery API",
+        "contact": {},
+        "version": "1.0"
+    },
+    "basePath": "/",
+    "paths": {
+        "/api/videos": {
+            "get": {
+                "description": "Finds catalogued videos by the words of their title and the categories they are filed under, most relevant first. Every parameter is optional — a request with none of them browses the whole catalogue, newest first. Paging is by cursor rather than by offset, so a page stays the same page while videos are being announced into the catalogue around it.",
+                "produces": [
+                    "application/json"
+                ],
+                "tags": [
+                    "videos"
+                ],
+                "summary": "Search the catalogue",
+                "parameters": [
+                    {
+                        "type": "string",
+                        "example": "desert fal",
+                        "description": "Matched lexically against video titles: the words are all required, and the last is treated as a prefix so a part-typed word still finds something.",
+                        "name": "title",
+                        "in": "query"
+                    },
+                    {
+                        "type": "array",
+                        "items": {
+                            "type": "string"
+                        },
+                        "collectionFormat": "multi",
+                        "example": "documentary",
+                        "description": "Narrows the search to videos filed under any one of these names — not all of them. Repeat the parameter per category. Omitted means every category.",
+                        "name": "categories",
+                        "in": "query"
+                    },
+                    {
+                        "type": "integer",
+                        "default": 20,
+                        "description": "How many videos to return. Absent, zero or negative takes the default of 20; anything above the maximum of 100 is capped to it rather than refused, and the cursor still reaches the rest.",
+                        "name": "limit",
+                        "in": "query"
+                    },
+                    {
+                        "type": "string",
+                        "description": "Asks for the page after the one a previous search ended at. Send back the nextCursor from that search unchanged; it is opaque, and the only thing to do with it is return it.",
+                        "name": "cursor",
+                        "in": "query"
+                    }
+                ],
+                "responses": {
+                    "200": {
+                        "description": "OK",
+                        "schema": {
+                            "$ref": "#/definitions/api.SearchResults"
+                        }
+                    },
+                    "400": {
+                        "description": "Bad Request",
+                        "schema": {
+                            "$ref": "#/definitions/api.ProblemDetails"
+                        }
+                    },
+                    "500": {
+                        "description": "Internal Server Error",
+                        "schema": {
+                            "$ref": "#/definitions/api.ProblemDetails"
+                        }
+                    }
+                }
+            }
+        },
+        "/api/videos/{id}": {
+            "get": {
+                "description": "Returns the catalogue's copy of a video: what a reader needs to show and play it. A video appears here only after the CMS has announced it as ready, so a video that is still transcoding — or one the CMS never made ready — is a 404.",
+                "produces": [
+                    "application/json"
+                ],
+                "tags": [
+                    "videos"
+                ],
+                "summary": "Get a catalogued video",
+                "parameters": [
+                    {
+                        "type": "string",
+                        "description": "Video id, as issued by the CMS",
+                        "name": "id",
+                        "in": "path",
+                        "required": true
+                    }
+                ],
+                "responses": {
+                    "200": {
+                        "description": "OK",
+                        "schema": {
+                            "$ref": "#/definitions/api.Video"
+                        }
+                    },
+                    "404": {
+                        "description": "Not Found",
+                        "schema": {
+                            "$ref": "#/definitions/api.ProblemDetails"
+                        }
+                    },
+                    "500": {
+                        "description": "Internal Server Error",
+                        "schema": {
+                            "$ref": "#/definitions/api.ProblemDetails"
+                        }
+                    }
+                }
+            }
+        },
+        "/health": {
+            "get": {
+                "description": "Reports that the service is up and serving. Used as the ALB target group health check; it does not verify the database connection.",
+                "produces": [
+                    "application/json"
+                ],
+                "tags": [
+                    "system"
+                ],
+                "summary": "Health check",
+                "responses": {
+                    "200": {
+                        "description": "OK",
+                        "schema": {
+                            "$ref": "#/definitions/api.HealthResponse"
+                        }
+                    }
+                }
+            }
+        }
+    },
+    "definitions": {
+        "api.HealthResponse": {
+            "type": "object",
+            "properties": {
+                "status": {
+                    "type": "string",
+                    "example": "ok"
+                }
+            }
+        },
+        "api.ProblemDetails": {
+            "type": "object",
+            "properties": {
+                "detail": {
+                    "type": "string",
+                    "example": "The catalogue could not be read from the database. This is a server-side fault and the request was not processed; retrying in a few moments may succeed."
+                },
+                "status": {
+                    "type": "integer",
+                    "example": 500
+                },
+                "title": {
+                    "type": "string",
+                    "example": "Something Went Wrong Loading The Catalogue"
+                },
+                "type": {
+                    "type": "string",
+                    "example": "about:blank"
+                }
+            }
+        },
+        "api.SearchResults": {
+            "type": "object",
+            "properties": {
+                "nextCursor": {
+                    "description": "NextCursor reaches the page after this one. Empty on the last page,\nwhich is how a reader knows there is no more to ask for.",
+                    "type": "string"
+                },
+                "videos": {
+                    "description": "Videos are the matches, most relevant first. Never null: a search that\nmatches nothing is an empty list.",
+                    "type": "array",
+                    "items": {
+                        "$ref": "#/definitions/api.Video"
+                    }
+                }
+            }
+        },
+        "api.Video": {
+            "type": "object",
+            "properties": {
+                "categories": {
+                    "type": "array",
+                    "items": {
+                        "type": "string"
+                    },
+                    "example": [
+                        "documentary",
+                        "news"
+                    ]
+                },
+                "id": {
+                    "type": "string",
+                    "example": "0f8fad5b-d9cb-469f-a165-70867728950e"
+                },
+                "playbackUrl": {
+                    "type": "string",
+                    "example": "https://d111111abcdef8.cloudfront.net/videos/abc123/index.m3u8"
+                },
+                "title": {
+                    "type": "string",
+                    "example": "The Evening Bulletin"
+                }
+            }
+        }
+    }
+}

discovery/docs/swagger.yaml

@@ -0,0 +1,167 @@
+basePath: /
+definitions:
+  api.HealthResponse:
+    properties:
+      status:
+        example: ok
+        type: string
+    type: object
+  api.ProblemDetails:
+    properties:
+      detail:
+        example: The catalogue could not be read from the database. This is a server-side
+          fault and the request was not processed; retrying in a few moments may succeed.
+        type: string
+      status:
+        example: 500
+        type: integer
+      title:
+        example: Something Went Wrong Loading The Catalogue
+        type: string
+      type:
+        example: about:blank
+        type: string
+    type: object
+  api.SearchResults:
+    properties:
+      nextCursor:
+        description: |-
+          NextCursor reaches the page after this one. Empty on the last page,
+          which is how a reader knows there is no more to ask for.
+        type: string
+      videos:
+        description: |-
+          Videos are the matches, most relevant first. Never null: a search that
+          matches nothing is an empty list.
+        items:
+          $ref: '#/definitions/api.Video'
+        type: array
+    type: object
+  api.Video:
+    properties:
+      categories:
+        example:
+        - documentary
+        - news
+        items:
+          type: string
+        type: array
+      id:
+        example: 0f8fad5b-d9cb-469f-a165-70867728950e
+        type: string
+      playbackUrl:
+        example: https://d111111abcdef8.cloudfront.net/videos/abc123/index.m3u8
+        type: string
+      title:
+        example: The Evening Bulletin
+        type: string
+    type: object
+info:
+  contact: {}
+  description: |-
+    JSON API for browsing the Thamanyah catalogue. Read-side counterpart to the CMS, which is what ingests videos.
+
+    The catalogue search is `GET /api/videos`, listed below: title is matched lexically with the last word as a prefix, categories narrow to videos filed under any one of them, limit defaults to 20 and is capped at 100, and cursor is the opaque `nextCursor` of a previous search.
+  title: Thamanyah Discovery API
+  version: "1.0"
+paths:
+  /api/videos:
+    get:
+      description: Finds catalogued videos by the words of their title and the categories
+        they are filed under, most relevant first. Every parameter is optional — a
+        request with none of them browses the whole catalogue, newest first. Paging
+        is by cursor rather than by offset, so a page stays the same page while videos
+        are being announced into the catalogue around it.
+      parameters:
+      - description: 'Matched lexically against video titles: the words are all required,
+          and the last is treated as a prefix so a part-typed word still finds something.'
+        example: desert fal
+        in: query
+        name: title
+        type: string
+      - collectionFormat: multi
+        description: Narrows the search to videos filed under any one of these names
+          — not all of them. Repeat the parameter per category. Omitted means every
+          category.
+        example: documentary
+        in: query
+        items:
+          type: string
+        name: categories
+        type: array
+      - default: 20
+        description: How many videos to return. Absent, zero or negative takes the
+          default of 20; anything above the maximum of 100 is capped to it rather
+          than refused, and the cursor still reaches the rest.
+        in: query
+        name: limit
+        type: integer
+      - description: Asks for the page after the one a previous search ended at. Send
+          back the nextCursor from that search unchanged; it is opaque, and the only
+          thing to do with it is return it.
+        in: query
+        name: cursor
+        type: string
+      produces:
+      - application/json
+      responses:
+        "200":
+          description: OK
+          schema:
+            $ref: '#/definitions/api.SearchResults'
+        "400":
+          description: Bad Request
+          schema:
+            $ref: '#/definitions/api.ProblemDetails'
+        "500":
+          description: Internal Server Error
+          schema:
+            $ref: '#/definitions/api.ProblemDetails'
+      summary: Search the catalogue
+      tags:
+      - videos
+  /api/videos/{id}:
+    get:
+      description: 'Returns the catalogue''s copy of a video: what a reader needs
+        to show and play it. A video appears here only after the CMS has announced
+        it as ready, so a video that is still transcoding — or one the CMS never made
+        ready — is a 404.'
+      parameters:
+      - description: Video id, as issued by the CMS
+        in: path
+        name: id
+        required: true
+        type: string
+      produces:
+      - application/json
+      responses:
+        "200":
+          description: OK
+          schema:
+            $ref: '#/definitions/api.Video'
+        "404":
+          description: Not Found
+          schema:
+            $ref: '#/definitions/api.ProblemDetails'
+        "500":
+          description: Internal Server Error
+          schema:
+            $ref: '#/definitions/api.ProblemDetails'
+      summary: Get a catalogued video
+      tags:
+      - videos
+  /health:
+    get:
+      description: Reports that the service is up and serving. Used as the ALB target
+        group health check; it does not verify the database connection.
+      produces:
+      - application/json
+      responses:
+        "200":
+          description: OK
+          schema:
+            $ref: '#/definitions/api.HealthResponse'
+      summary: Health check
+      tags:
+      - system
+swagger: "2.0"

discovery/go.mod

@@ -0,0 +1,44 @@
+module thamanyah/discovery
+
+go 1.25.12
+
+require (
+	github.com/aws/aws-sdk-go-v2/config v1.33.1
+	github.com/aws/aws-sdk-go-v2/service/sqs v1.48.1
+	github.com/golang-migrate/migrate/v4 v4.19.1
+	github.com/lib/pq v1.12.3
+	github.com/redis/go-redis/v9 v9.22.0
+	github.com/swaggo/http-swagger/v2 v2.0.2
+	github.com/swaggo/swag v1.16.6
+)
+
+require (
+	github.com/KyleBanks/depth v1.2.1 // indirect
+	github.com/aws/aws-sdk-go-v2 v1.45.1 // indirect
+	github.com/aws/aws-sdk-go-v2/credentials v1.20.1 // indirect
+	github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.19.1 // indirect
+	github.com/aws/aws-sdk-go-v2/internal/configsources v1.5.1 // indirect
+	github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.8.1 // indirect
+	github.com/aws/aws-sdk-go-v2/internal/v4a v1.5.1 // indirect
+	github.com/aws/aws-sdk-go-v2/service/internal/accept-encoding v1.13.19 // indirect
+	github.com/aws/aws-sdk-go-v2/service/internal/presigned-url v1.14.1 // indirect
+	github.com/aws/aws-sdk-go-v2/service/signin v1.7.1 // indirect
+	github.com/aws/aws-sdk-go-v2/service/sso v1.35.1 // indirect
+	github.com/aws/aws-sdk-go-v2/service/ssooidc v1.40.1 // indirect
+	github.com/aws/aws-sdk-go-v2/service/sts v1.47.1 // indirect
+	github.com/aws/smithy-go v1.28.1 // indirect
+	github.com/cespare/xxhash/v2 v2.3.0 // indirect
+	github.com/go-openapi/jsonpointer v0.19.5 // indirect
+	github.com/go-openapi/jsonreference v0.20.0 // indirect
+	github.com/go-openapi/spec v0.20.6 // indirect
+	github.com/go-openapi/swag v0.19.15 // indirect
+	github.com/josharian/intern v1.0.0 // indirect
+	github.com/mailru/easyjson v0.7.6 // indirect
+	github.com/swaggo/files/v2 v2.0.0 // indirect
+	go.uber.org/atomic v1.11.0 // indirect
+	golang.org/x/mod v0.29.0 // indirect
+	golang.org/x/sync v0.18.0 // indirect
+	golang.org/x/sys v0.38.0 // indirect
+	golang.org/x/tools v0.38.0 // indirect
+	gopkg.in/yaml.v2 v2.4.0 // indirect
+)

discovery/go.sum

@@ -0,0 +1,161 @@
+github.com/Azure/go-ansiterm v0.0.0-20230124172434-306776ec8161 h1:L/gRVlceqvL25UVaW/CKtUDjefjrs0SPonmDGUVOYP0=
+github.com/Azure/go-ansiterm v0.0.0-20230124172434-306776ec8161/go.mod h1:xomTg63KZ2rFqZQzSB4Vz2SUXa1BpHTVz9L5PTmPC4E=
+github.com/KyleBanks/depth v1.2.1 h1:5h8fQADFrWtarTdtDudMmGsC7GPbOAu6RVB3ffsVFHc=
+github.com/KyleBanks/depth v1.2.1/go.mod h1:jzSb9d0L43HxTQfT+oSA1EEp2q+ne2uh6XgeJcm8brE=
+github.com/Microsoft/go-winio v0.6.2 h1:F2VQgta7ecxGYO8k3ZZz3RS8fVIXVxONVUPlNERoyfY=
+github.com/Microsoft/go-winio v0.6.2/go.mod h1:yd8OoFMLzJbo9gZq8j5qaps8bJ9aShtEA8Ipt1oGCvU=
+github.com/aws/aws-sdk-go-v2 v1.45.1 h1:iIoG3NaLhV6UZpPXyPXlDj2I9oS8tV/nMcMnITCC6Ks=
+github.com/aws/aws-sdk-go-v2 v1.45.1/go.mod h1:bttEH6JqnUL8LepvDVfdrds/fZ5bCIxzpe3abyUrhDU=
+github.com/aws/aws-sdk-go-v2/config v1.33.1 h1:bq9jze1hQ5YTCLoVxNnbp0T7rglrlOE7N9YsHqjGkEw=
+github.com/aws/aws-sdk-go-v2/config v1.33.1/go.mod h1:2A3HQwG4zaL5Tm80rc6RZj8LmWWv4WYT5v8raSz/L7A=
+github.com/aws/aws-sdk-go-v2/credentials v1.20.1 h1:Z8GRNEx0u9sDkZOq4PUnN8mjGwbUQGRzMSXpvt3d8xQ=
+github.com/aws/aws-sdk-go-v2/credentials v1.20.1/go.mod h1:uBIK00kFo95dnemqfFMTWx0X8YRqsh6ecIoCjjOkZqM=
+github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.19.1 h1:YIEBqcqRnpi4Pfv0YHImtgi6czGCwKHANC7SwmUAVD0=
+github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.19.1/go.mod h1:imEf0oufgAo8KAkCHhrOdqGEC0YWx1PPBQH82shSxGw=
+github.com/aws/aws-sdk-go-v2/internal/configsources v1.5.1 h1:pc138gM1CW+XPc60rEwUlwwuwWFQK16CI1T7v1F9Oec=
+github.com/aws/aws-sdk-go-v2/internal/configsources v1.5.1/go.mod h1:1+koxpPIbfBdfzP6vojm5/zTpTQ/micYwlxIiNB3TxI=
+github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.8.1 h1:K0JsbZQj+1h208Ro1zHeA4l7bMp0NvRffHQ91q8Ol1s=
+github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.8.1/go.mod h1:W3/vL6EtCIatICGy9ab29QhMuae+cOKPWcMxv02CO+Q=
+github.com/aws/aws-sdk-go-v2/internal/v4a v1.5.1 h1:yhw5KD1phVyP9vijxOUzDfEtJx+bt+L63k+VfuiYFAA=
+github.com/aws/aws-sdk-go-v2/internal/v4a v1.5.1/go.mod h1:ZW2e0d7DYlRxlS9hEiMXE47gTdX5KRN4byUiNbUpG+Q=
+github.com/aws/aws-sdk-go-v2/service/internal/accept-encoding v1.13.19 h1:bAdDl/HkGCcGPoe25ToSHEw23VIxt6CT5fLcg111BKg=
+github.com/aws/aws-sdk-go-v2/service/internal/accept-encoding v1.13.19/go.mod h1:KaUzbLxv4CeSxh6ZCl9B4m7CuFenS8kUEaDs+f/DQr4=
+github.com/aws/aws-sdk-go-v2/service/internal/presigned-url v1.14.1 h1:RmmWQPREQdk9U+PfqeHW3MqZaBaNK7TpV9W3RY+b+7g=
+github.com/aws/aws-sdk-go-v2/service/internal/presigned-url v1.14.1/go.mod h1:0A3W4F+68ZnNk5XcNL/e9HFMwnP8RlEicFfy6eOEDyw=
+github.com/aws/aws-sdk-go-v2/service/signin v1.7.1 h1:mdMtSVKdQ3+mzBh+l0ogrFYZVQUCg6pJZOirA2ARsYE=
+github.com/aws/aws-sdk-go-v2/service/signin v1.7.1/go.mod h1:9IqUlsJDbUPcg6cgx3WEzXdjrbWzLDQrak0aaSqlTcI=
+github.com/aws/aws-sdk-go-v2/service/sqs v1.48.1 h1:jXP3BdVenFa8RfLVH+D2gswrWZHJcgtygKCf22APFqo=
+github.com/aws/aws-sdk-go-v2/service/sqs v1.48.1/go.mod h1:d4DToDhLnEofHKvFu4yCF0Be65pZW267COfKOztsZOQ=
+github.com/aws/aws-sdk-go-v2/service/sso v1.35.1 h1:B6WFn91tobD6gG4724ONHaqrpKsoETGnv98LHe/yIGM=
+github.com/aws/aws-sdk-go-v2/service/sso v1.35.1/go.mod h1:tWuiVBUtPBr8/rgRiYS8Uf85sHcAN+G7XS3D3CEoUh8=
+github.com/aws/aws-sdk-go-v2/service/ssooidc v1.40.1 h1:6yeYCWFvgbI2TI3K6jr9LtBNhXgJ7g4xqD+DEiaDDmM=
+github.com/aws/aws-sdk-go-v2/service/ssooidc v1.40.1/go.mod h1:naFe83jSMuYkH+QjQPX8n1MLhBkeCFM5Lsnh5m5wz3c=
+github.com/aws/aws-sdk-go-v2/service/sts v1.47.1 h1:Sv2xPnRHlThSUtVujYuUBPI/Il8si6UPHXL8DMiB/F0=
+github.com/aws/aws-sdk-go-v2/service/sts v1.47.1/go.mod h1:mKo/CzaCz8qytGW70NG4vIIGAx1HXTlb5lHNkC5k3lk=
+github.com/aws/smithy-go v1.28.1 h1:R/nXH00c8qcfCzQVELtRw+eLQWtzv+VAIEFJ1/xxXlQ=
+github.com/aws/smithy-go v1.28.1/go.mod h1:YE2RhdIuDbA5E5bTdciG9KrW3+TiEONeUWCqxX9i1Fc=
+github.com/bsm/ginkgo/v2 v2.12.0 h1:Ny8MWAHyOepLGlLKYmXG4IEkioBysk6GpaRTLC8zwWs=
+github.com/bsm/ginkgo/v2 v2.12.0/go.mod h1:SwYbGRRDovPVboqFv0tPTcG1sN61LM1Z4ARdbAV9g4c=
+github.com/bsm/gomega v1.27.10 h1:yeMWxP2pV2fG3FgAODIY8EiRE3dy0aeFYt4l7wh6yKA=
+github.com/bsm/gomega v1.27.10/go.mod h1:JyEr/xRbxbtgWNi8tIEVPUYZ5Dzef52k01W3YH0H+O0=
+github.com/cespare/xxhash/v2 v2.3.0 h1:UL815xU9SqsFlibzuggzjXhog7bL6oX9BbNZnL2UFvs=
+github.com/cespare/xxhash/v2 v2.3.0/go.mod h1:VGX0DQ3Q6kWi7AoAeZDth3/j3BFtOZR5XLFGgcrjCOs=
+github.com/containerd/errdefs v1.0.0 h1:tg5yIfIlQIrxYtu9ajqY42W3lpS19XqdxRQeEwYG8PI=
+github.com/containerd/errdefs v1.0.0/go.mod h1:+YBYIdtsnF4Iw6nWZhJcqGSg/dwvV7tyJ/kCkyJ2k+M=
+github.com/containerd/errdefs/pkg v0.3.0 h1:9IKJ06FvyNlexW690DXuQNx2KA2cUJXx151Xdx3ZPPE=
+github.com/containerd/errdefs/pkg v0.3.0/go.mod h1:NJw6s9HwNuRhnjJhM7pylWwMyAkmCQvQ4GpJHEqRLVk=
+github.com/creack/pty v1.1.9/go.mod h1:oKZEueFk5CKHvIhNR5MUki03XCEU+Q6VDXinZuGJ33E=
+github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
+github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
+github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc h1:U9qPSI2PIWSS1VwoXQT9A3Wy9MM3WgvqSxFWenqJduM=
+github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
+github.com/dhui/dktest v0.4.6 h1:+DPKyScKSEp3VLtbMDHcUq6V5Lm5zfZZVb0Sk7Ahom4=
+github.com/dhui/dktest v0.4.6/go.mod h1:JHTSYDtKkvFNFHJKqCzVzqXecyv+tKt8EzceOmQOgbU=
+github.com/distribution/reference v0.6.0 h1:0IXCQ5g4/QMHHkarYzh5l+u8T3t73zM5QvfrDyIgxBk=
+github.com/distribution/reference v0.6.0/go.mod h1:BbU0aIcezP1/5jX/8MP0YiH4SdvB5Y4f/wlDRiLyi3E=
+github.com/docker/docker v28.3.3+incompatible h1:Dypm25kh4rmk49v1eiVbsAtpAsYURjYkaKubwuBdxEI=
+github.com/docker/docker v28.3.3+incompatible/go.mod h1:eEKB0N0r5NX/I1kEveEz05bcu8tLC/8azJZsviup8Sk=
+github.com/docker/go-connections v0.5.0 h1:USnMq7hx7gwdVZq1L49hLXaFtUdTADjXGp+uj1Br63c=
+github.com/docker/go-connections v0.5.0/go.mod h1:ov60Kzw0kKElRwhNs9UlUHAE/F9Fe6GLaXnqyDdmEXc=
+github.com/docker/go-units v0.5.0 h1:69rxXcBk27SvSaaxTtLh/8llcHD8vYHT7WSdRZ/jvr4=
+github.com/docker/go-units v0.5.0/go.mod h1:fgPhTUdO+D/Jk86RDLlptpiXQzgHJF7gydDDbaIK4Dk=
+github.com/felixge/httpsnoop v1.0.4 h1:NFTV2Zj1bL4mc9sqWACXbQFVBBg2W3GPvqp8/ESS2Wg=
+github.com/felixge/httpsnoop v1.0.4/go.mod h1:m8KPJKqk1gH5J9DgRY2ASl2lWCfGKXixSwevea8zH2U=
+github.com/go-logr/logr v1.4.3 h1:CjnDlHq8ikf6E492q6eKboGOC0T8CDaOvkHCIg8idEI=
+github.com/go-logr/logr v1.4.3/go.mod h1:9T104GzyrTigFIr8wt5mBrctHMim0Nb2HLGrmQ40KvY=
+github.com/go-logr/stdr v1.2.2 h1:hSWxHoqTgW2S2qGc0LTAI563KZ5YKYRhT3MFKZMbjag=
+github.com/go-logr/stdr v1.2.2/go.mod h1:mMo/vtBO5dYbehREoey6XUKy/eSumjCCveDpRre4VKE=
+github.com/go-openapi/jsonpointer v0.19.3/go.mod h1:Pl9vOtqEWErmShwVjC8pYs9cog34VGT37dQOVbmoatg=
+github.com/go-openapi/jsonpointer v0.19.5 h1:gZr+CIYByUqjcgeLXnQu2gHYQC9o73G2XUeOFYEICuY=
+github.com/go-openapi/jsonpointer v0.19.5/go.mod h1:Pl9vOtqEWErmShwVjC8pYs9cog34VGT37dQOVbmoatg=
+github.com/go-openapi/jsonreference v0.20.0 h1:MYlu0sBgChmCfJxxUKZ8g1cPWFOB37YSZqewK7OKeyA=
+github.com/go-openapi/jsonreference v0.20.0/go.mod h1:Ag74Ico3lPc+zR+qjn4XBUmXymS4zJbYVCZmcgkasdo=
+github.com/go-openapi/spec v0.20.6 h1:ich1RQ3WDbfoeTqTAb+5EIxNmpKVJZWBNah9RAT0jIQ=
+github.com/go-openapi/spec v0.20.6/go.mod h1:2OpW+JddWPrpXSCIX8eOx7lZ5iyuWj3RYR6VaaBKcWA=
+github.com/go-openapi/swag v0.19.5/go.mod h1:POnQmlKehdgb5mhVOsnJFsivZCEZ/vjK9gh66Z9tfKk=
+github.com/go-openapi/swag v0.19.15 h1:D2NRCBzS9/pEY3gP9Nl8aDqGUcPFrwG2p+CNFrLyrCM=
+github.com/go-openapi/swag v0.19.15/go.mod h1:QYRuS/SOXUCsnplDa677K7+DxSOj6IPNl/eQntq43wQ=
+github.com/gogo/protobuf v1.3.2 h1:Ov1cvc58UF3b5XjBnZv7+opcTcQFZebYjWzi34vdm4Q=
+github.com/gogo/protobuf v1.3.2/go.mod h1:P1XiOD3dCwIKUDQYPy72D8LYyHL2YPYrpS2s69NZV8Q=
+github.com/golang-migrate/migrate/v4 v4.19.1 h1:OCyb44lFuQfYXYLx1SCxPZQGU7mcaZ7gH9yH4jSFbBA=
+github.com/golang-migrate/migrate/v4 v4.19.1/go.mod h1:CTcgfjxhaUtsLipnLoQRWCrjYXycRz/g5+RWDuYgPrE=
+github.com/google/go-cmp v0.6.0 h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI=
+github.com/google/go-cmp v0.6.0/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY=
+github.com/josharian/intern v1.0.0 h1:vlS4z54oSdjm0bgjRigI+G1HpF+tI+9rE5LLzOg8HmY=
+github.com/josharian/intern v1.0.0/go.mod h1:5DoeVV0s6jJacbCEi61lwdGj/aVlrQvzHFFd8Hwg//Y=
+github.com/klauspost/cpuid/v2 v2.2.10 h1:tBs3QSyvjDyFTq3uoc/9xFpCuOsJQFNPiAhYdw2skhE=
+github.com/klauspost/cpuid/v2 v2.2.10/go.mod h1:hqwkgyIinND0mEev00jJYCxPNVRVXFQeu1XKlok6oO0=
+github.com/kr/pretty v0.1.0/go.mod h1:dAy3ld7l9f0ibDNOQOHHMYYIIbhfbHSm3C4ZsoJORNo=
+github.com/kr/pty v1.1.1/go.mod h1:pFQYn66WHrOpPYNljwOMqo10TkYh1fy3cYio2l3bCsQ=
+github.com/kr/text v0.1.0/go.mod h1:4Jbv+DJW3UT/LiOwJeYQe1efqtUx/iVham/4vfdArNI=
+github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY=
+github.com/kr/text v0.2.0/go.mod h1:eLer722TekiGuMkidMxC/pM04lWEeraHUUmBw8l2grE=
+github.com/lib/pq v1.12.3 h1:tTWxr2YLKwIvK90ZXEw8GP7UFHtcbTtty8zsI+YjrfQ=
+github.com/lib/pq v1.12.3/go.mod h1:/p+8NSbOcwzAEI7wiMXFlgydTwcgTr3OSKMsD2BitpA=
+github.com/mailru/easyjson v0.0.0-20190614124828-94de47d64c63/go.mod h1:C1wdFJiN94OJF2b5HbByQZoLdCWB1Yqtg26g4irojpc=
+github.com/mailru/easyjson v0.0.0-20190626092158-b2ccc519800e/go.mod h1:C1wdFJiN94OJF2b5HbByQZoLdCWB1Yqtg26g4irojpc=
+github.com/mailru/easyjson v0.7.6 h1:8yTIVnZgCoiM1TgqoeTl+LfU5Jg6/xL3QhGQnimLYnA=
+github.com/mailru/easyjson v0.7.6/go.mod h1:xzfreul335JAWq5oZzymOObrkdz5UnU4kGfJJLY9Nlc=
+github.com/moby/docker-image-spec v1.3.1 h1:jMKff3w6PgbfSa69GfNg+zN/XLhfXJGnEx3Nl2EsFP0=
+github.com/moby/docker-image-spec v1.3.1/go.mod h1:eKmb5VW8vQEh/BAr2yvVNvuiJuY6UIocYsFu/DxxRpo=
+github.com/moby/term v0.5.0 h1:xt8Q1nalod/v7BqbG21f8mQPqH+xAaC9C3N3wfWbVP0=
+github.com/moby/term v0.5.0/go.mod h1:8FzsFHVUBGZdbDsJw/ot+X+d5HLUbvklYLJ9uGfcI3Y=
+github.com/morikuni/aec v1.0.0 h1:nP9CBfwrvYnBRgY6qfDQkygYDmYwOilePFkwzv4dU8A=
+github.com/morikuni/aec v1.0.0/go.mod h1:BbKIizmSmc5MMPqRYbxO4ZU0S0+P200+tUnFx7PXmsc=
+github.com/niemeyer/pretty v0.0.0-20200227124842-a10e7caefd8e h1:fD57ERR4JtEqsWbfPhv4DMiApHyliiK5xCTNVSPiaAs=
+github.com/niemeyer/pretty v0.0.0-20200227124842-a10e7caefd8e/go.mod h1:zD1mROLANZcx1PVRCS0qkT7pwLkGfwJo4zjcN/Tysno=
+github.com/opencontainers/go-digest v1.0.0 h1:apOUWs51W5PlhuyGyz9FCeeBIOUDA/6nW8Oi/yOhh5U=
+github.com/opencontainers/go-digest v1.0.0/go.mod h1:0JzlMkj0TRzQZfJkVvzbP0HBR3IKzErnv2BNG4W4MAM=
+github.com/opencontainers/image-spec v1.1.0 h1:8SG7/vwALn54lVB/0yZ/MMwhFrPYtpEHQb2IpWsCzug=
+github.com/opencontainers/image-spec v1.1.0/go.mod h1:W4s4sFTMaBeK1BQLXbG4AdM2szdn85PY75RI83NrTrM=
+github.com/pkg/errors v0.9.1 h1:FEBLx1zS214owpjy7qsBeixbURkuhQAwrK5UwLGTwt4=
+github.com/pkg/errors v0.9.1/go.mod h1:bwawxfHBFNV+L2hUp1rHADufV3IMtnDRdf1r5NINEl0=
+github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
+github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 h1:Jamvg5psRIccs7FGNTlIRMkT8wgtp5eCXdBlqhYGL6U=
+github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
+github.com/redis/go-redis/v9 v9.22.0 h1:laDvpYXTJtZLloinw1fA5Kqd6HAEH2XKxOkG/PDq2F0=
+github.com/redis/go-redis/v9 v9.22.0/go.mod h1:y2g0Wj8rQvuK0ELM+oxSudcLtC09JScs98I/X9gRWY4=
+github.com/stretchr/objx v0.1.0/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME=
+github.com/stretchr/testify v1.3.0/go.mod h1:M5WIy9Dh21IEIfnGCwXGc5bZfKNJtfHm1UVUgZn+9EI=
+github.com/stretchr/testify v1.6.1/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/h/Wwjteg=
+github.com/stretchr/testify v1.10.0 h1:Xv5erBjTwe/5IxqUQTdXv5kgmIvbHo3QQyRwhJsOfJA=
+github.com/stretchr/testify v1.10.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
+github.com/swaggo/files/v2 v2.0.0 h1:hmAt8Dkynw7Ssz46F6pn8ok6YmGZqHSVLZ+HQM7i0kw=
+github.com/swaggo/files/v2 v2.0.0/go.mod h1:24kk2Y9NYEJ5lHuCra6iVwkMjIekMCaFq/0JQj66kyM=
+github.com/swaggo/http-swagger/v2 v2.0.2 h1:FKCdLsl+sFCx60KFsyM0rDarwiUSZ8DqbfSyIKC9OBg=
+github.com/swaggo/http-swagger/v2 v2.0.2/go.mod h1:r7/GBkAWIfK6E/OLnE8fXnviHiDeAHmgIyooa4xm3AQ=
+github.com/swaggo/swag v1.16.6 h1:qBNcx53ZaX+M5dxVyTrgQ0PJ/ACK+NzhwcbieTt+9yI=
+github.com/swaggo/swag v1.16.6/go.mod h1:ngP2etMK5a0P3QBizic5MEwpRmluJZPHjXcMoj4Xesg=
+github.com/zeebo/xxh3 v1.1.0 h1:s7DLGDK45Dyfg7++yxI0khrfwq9661w9EN78eP/UZVs=
+github.com/zeebo/xxh3 v1.1.0/go.mod h1:IisAie1LELR4xhVinxWS5+zf1lA4p0MW4T+w+W07F5s=
+go.opentelemetry.io/auto/sdk v1.1.0 h1:cH53jehLUN6UFLY71z+NDOiNJqDdPRaXzTel0sJySYA=
+go.opentelemetry.io/auto/sdk v1.1.0/go.mod h1:3wSPjt5PWp2RhlCcmmOial7AvC4DQqZb7a7wCow3W8A=
+go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.61.0 h1:F7Jx+6hwnZ41NSFTO5q4LYDtJRXBf2PD0rNBkeB/lus=
+go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.61.0/go.mod h1:UHB22Z8QsdRDrnAtX4PntOl36ajSxcdUMt1sF7Y6E7Q=
+go.opentelemetry.io/otel v1.37.0 h1:9zhNfelUvx0KBfu/gb+ZgeAfAgtWrfHJZcAqFC228wQ=
+go.opentelemetry.io/otel v1.37.0/go.mod h1:ehE/umFRLnuLa/vSccNq9oS1ErUlkkK71gMcN34UG8I=
+go.opentelemetry.io/otel/metric v1.37.0 h1:mvwbQS5m0tbmqML4NqK+e3aDiO02vsf/WgbsdpcPoZE=
+go.opentelemetry.io/otel/metric v1.37.0/go.mod h1:04wGrZurHYKOc+RKeye86GwKiTb9FKm1WHtO+4EVr2E=
+go.opentelemetry.io/otel/trace v1.37.0 h1:HLdcFNbRQBE2imdSEgm/kwqmQj1Or1l/7bW6mxVK7z4=
+go.opentelemetry.io/otel/trace v1.37.0/go.mod h1:TlgrlQ+PtQO5XFerSPUYG0JSgGyryXewPGyayAWSBS0=
+go.uber.org/atomic v1.11.0 h1:ZvwS0R+56ePWxUNi+Atn9dWONBPp/AUETXlHW0DxSjE=
+go.uber.org/atomic v1.11.0/go.mod h1:LUxbIzbOniOlMKjJjyPfpl4v+PKK2cNJn91OQbhoJI0=
+golang.org/x/mod v0.29.0 h1:HV8lRxZC4l2cr3Zq1LvtOsi/ThTgWnUk/y64QSs8GwA=
+golang.org/x/mod v0.29.0/go.mod h1:NyhrlYXJ2H4eJiRy/WDBO6HMqZQ6q9nk4JzS3NuCK+w=
+golang.org/x/sync v0.18.0 h1:kr88TuHDroi+UVf+0hZnirlk8o8T+4MrK6mr60WkH/I=
+golang.org/x/sync v0.18.0/go.mod h1:9KTHXmSnoGruLpwFjVSX0lNNA75CykiMECbovNTZqGI=
+golang.org/x/sys v0.38.0 h1:3yZWxaJjBmCWXqhN1qh02AkOnCQ1poK6oF+a7xWL6Gc=
+golang.org/x/sys v0.38.0/go.mod h1:OgkHotnGiDImocRcuBABYBEXf8A9a87e/uXjp9XT3ks=
+golang.org/x/tools v0.38.0 h1:Hx2Xv8hISq8Lm16jvBZ2VQf+RLmbd7wVUsALibYI/IQ=
+golang.org/x/tools v0.38.0/go.mod h1:yEsQ/d/YK8cjh0L6rZlY8tgtlKiBNTL14pGDJPJpYQs=
+gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
+gopkg.in/check.v1 v1.0.0-20180628173108-788fd7840127/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
+gopkg.in/check.v1 v1.0.0-20200227125254-8fa46927fb4f h1:BLraFXnmrev5lT+xlilqcH8XK9/i0At2xKjWk4p6zsU=
+gopkg.in/check.v1 v1.0.0-20200227125254-8fa46927fb4f/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
+gopkg.in/yaml.v2 v2.2.2/go.mod h1:hI93XBmqTisBFMUTm0b8Fm+jr3Dg1NNxqwp+5A1VGuI=
+gopkg.in/yaml.v2 v2.4.0 h1:D8xgwECY7CYvx+Y2n4sBz93Jn9JRvxdiyyo8CTfuKaY=
+gopkg.in/yaml.v2 v2.4.0/go.mod h1:RDklbk79AGWmwhnvt/jBztapEOGDOx6ZbXqjP6csGnQ=
+gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=
+gopkg.in/yaml.v3 v3.0.0-20200615113413-eeeca48fe776/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=
+gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA=
+gopkg.in/yaml.v3 v3.0.1/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=

discovery/internal/api/api.go

@@ -0,0 +1,23 @@
+// Package api holds the request and response bodies of the HTTP API — the
+// wire contract, kept apart from the handlers that serve it. The swaggo
+// annotations on the handlers reference these types by name
+// (api.HealthResponse and so on), so renaming one changes the generated spec.
+package api
+
+// HealthResponse is the body of GET /health.
+type HealthResponse struct {
+	Status string `json:"status" example:"ok"`
+}
+
+// ProblemDetails is an error body in the RFC 9457 "Problem Details for HTTP
+// APIs" format. Type stays "about:blank" — the value RFC 9457 defines for
+// problems with no dedicated documentation URI. Title is a short summary that
+// stays identical for every occurrence of the same problem, so clients can
+// branch on it; Detail explains this particular occurrence and is the only
+// member that varies with request data.
+type ProblemDetails struct {
+	Type   string `json:"type" example:"about:blank"`
+	Title  string `json:"title" example:"Something Went Wrong Loading The Catalogue"`
+	Status int    `json:"status" example:"500"`
+	Detail string `json:"detail,omitempty" example:"The catalogue could not be read from the database. This is a server-side fault and the request was not processed; retrying in a few moments may succeed."`
+}

discovery/internal/api/videos.go

@@ -0,0 +1,28 @@
+package api
+
+// Video is the catalogue's copy of a video, as GET /api/videos/{id} serves it.
+// It is deliberately not cms's video record of the same name: the catalogue
+// publishes what a reader needs to show and play a video, and none of the
+// ingestion detail — no storage key, no transcoding job id, no status, since
+// a video that is not ready never reaches the catalogue at all.
+type Video struct {
+	ID          string   `json:"id" example:"0f8fad5b-d9cb-469f-a165-70867728950e"`
+	Title       string   `json:"title" example:"The Evening Bulletin"`
+	PlaybackURL string   `json:"playbackUrl" example:"https://d111111abcdef8.cloudfront.net/videos/abc123/index.m3u8"`
+	Categories  []string `json:"categories" example:"documentary,news"`
+}
+
+// The search itself takes no body type: GET /api/videos reads its parameters
+// — title, categories, limit and cursor — from the query string, so there is
+// no request document to publish a schema for. They are documented as
+// parameters on handlers.SearchVideos, and appear in the generated spec there.
+
+// SearchResults is one page of search results.
+type SearchResults struct {
+	// Videos are the matches, most relevant first. Never null: a search that
+	// matches nothing is an empty list.
+	Videos []Video `json:"videos"`
+	// NextCursor reaches the page after this one. Empty on the last page,
+	// which is how a reader knows there is no more to ask for.
+	NextCursor string `json:"nextCursor"`
+}

discovery/internal/consumers/catalogue.go

@@ -0,0 +1,110 @@
+// Package consumers holds the queue-driven half of the service: work that
+// arrives on a queue rather than as an HTTP request. It is to SQS what
+// internal/handlers is to HTTP — it decodes a message, calls repositories, and
+// decides what the message's fate is — so the same seams apply, and no SQL
+// lives here.
+package consumers
+
+import (
+	"context"
+	"encoding/json"
+	"log"
+	"strings"
+	"thamanyah/discovery/internal/db/repositories"
+	"thamanyah/discovery/internal/models"
+	"thamanyah/discovery/internal/services"
+	"time"
+)
+
+// receiveBackoff is how long the loop waits after a failed receive, so a queue
+// that is unreachable produces a slow trickle of log lines rather than a spin.
+const receiveBackoff = 5 * time.Second
+
+// announcement is what cms publishes to the catalogue topic when a video
+// becomes ready. It is cms's published contract, mirrored here rather than
+// imported: the two services share no code, and this struct is exactly the
+// coupling between them — so a field renamed on one side has to be renamed
+// here too, deliberately.
+//
+// The subscription delivers raw, so this is the whole message body; there is
+// no SNS envelope to unwrap.
+type announcement struct {
+	VideoID     string   `json:"videoId"`
+	Title       string   `json:"title"`
+	PlaybackURL string   `json:"playbackUrl"`
+	Categories  []string `json:"categories"`
+}
+
+// CatalogueEvents consumes the announcements cms publishes.
+type CatalogueEvents struct{}
+
+// Run consumes announcements until ctx is cancelled. It is meant to be run in
+// its own goroutine for the lifetime of the process.
+func (c CatalogueEvents) Run(ctx context.Context) {
+	log.Println("catalogue announcements consumer started")
+
+	for {
+		if ctx.Err() != nil {
+			log.Println("catalogue announcements consumer stopped")
+			return
+		}
+
+		messages, err := services.SQSClient.ReceiveMessages(ctx)
+		if err != nil {
+			if ctx.Err() != nil {
+				log.Println("catalogue announcements consumer stopped")
+				return
+			}
+			log.Printf("Something Went Wrong Receiving Announcements: %s", err)
+			time.Sleep(receiveBackoff)
+			continue
+		}
+
+		for _, message := range messages {
+			if !c.handleAnnouncement(ctx, message.Body) {
+				// Left on the queue on purpose: it becomes visible again when
+				// the visibility timeout expires, and reaches the dead-letter
+				// queue if it keeps failing.
+				continue
+			}
+
+			if err := services.SQSClient.DeleteMessage(ctx, message.ReceiptHandle); err != nil {
+				log.Printf("Something Went Wrong Acknowledging An Announcement: %s", err)
+			}
+		}
+	}
+}
+
+// handleAnnouncement processes one message and reports whether it is finished
+// with — that is, whether it should be deleted from the queue. It returns
+// false only when a retry could plausibly succeed. Anything a retry cannot fix
+// (a body that will never parse, an announcement naming no video) is finished
+// with, however little it accomplished: leaving it on the queue would only
+// stall the announcements behind it.
+func (c CatalogueEvents) handleAnnouncement(ctx context.Context, body string) bool {
+	var announced announcement
+	if err := json.Unmarshal([]byte(body), &announced); err != nil {
+		log.Printf("Discarding An Announcement That Is Not JSON: %s", err)
+		return true
+	}
+
+	if strings.TrimSpace(announced.VideoID) == "" {
+		log.Printf("Discarding An Announcement That Names No Video")
+		return true
+	}
+
+	if err := repositories.VideoRepo.SaveVideo(ctx, models.Video{
+		ID:          announced.VideoID,
+		Title:       announced.Title,
+		PlaybackURL: announced.PlaybackURL,
+		Categories:  announced.Categories,
+	}); err != nil {
+		// Worth another attempt: the save is an upsert, so a redelivery
+		// re-runs it harmlessly.
+		log.Printf("Something Went Wrong Saving An Announced Video: video=%q: %s", announced.VideoID, err)
+		return false
+	}
+
+	log.Printf("Catalogued An Announced Video: video=%q title=%q", announced.VideoID, announced.Title)
+	return true
+}

discovery/internal/db/client.go

@@ -0,0 +1,30 @@
+package db
+
+import (
+	"context"
+	"database/sql"
+	"fmt"
+	"log"
+)
+
+func CreateDBConnection(connectionString string) *sql.DB {
+	sqlDB, err := sql.Open("postgres", connectionString)
+	if err != nil {
+		panic(fmt.Errorf("db: cannot open connection: %w", err))
+	}
+
+	return sqlDB
+}
+
+func CloseConnection(SQLDB *sql.DB) {
+	err := SQLDB.Close()
+	if err != nil {
+		log.Fatalf("Error Closing DB Connection: %s", err)
+	}
+}
+
+func AssertSuccessfulConnection(ctx context.Context, SQLDB *sql.DB) {
+	if err := SQLDB.PingContext(ctx); err != nil {
+		panic(fmt.Errorf("db: cannot connect: %w", err))
+	}
+}

discovery/internal/db/migrate.go

@@ -0,0 +1,38 @@
+package db
+
+import (
+	"database/sql"
+	"embed"
+	"errors"
+	"fmt"
+
+	"github.com/golang-migrate/migrate/v4"
+	"github.com/golang-migrate/migrate/v4/database/postgres"
+	"github.com/golang-migrate/migrate/v4/source/iofs"
+)
+
+//go:embed migrations/*.sql
+var migrationsFS embed.FS
+
+func Migrate(sqlDB *sql.DB) error {
+	driver, err := postgres.WithInstance(sqlDB, &postgres.Config{})
+	if err != nil {
+		return fmt.Errorf("db: creating migration driver: %w", err)
+	}
+
+	source, err := iofs.New(migrationsFS, "migrations")
+	if err != nil {
+		return fmt.Errorf("db: loading embedded migrations: %w", err)
+	}
+
+	m, err := migrate.NewWithInstance("iofs", source, "postgres", driver)
+	if err != nil {
+		return fmt.Errorf("db: initializing migrator: %w", err)
+	}
+
+	if err := m.Up(); err != nil && !errors.Is(err, migrate.ErrNoChange) {
+		return fmt.Errorf("db: applying migrations: %w", err)
+	}
+
+	return nil
+}

discovery/internal/db/migrations/0001_init.down.sql

@@ -0,0 +1,2 @@
+-- Nothing to undo: 0001 creates no objects.
+SELECT 1;

discovery/internal/db/migrations/0001_init.up.sql

@@ -0,0 +1,6 @@
+-- Discovery's schema starts here. The service owns its own database — the
+-- `discovery` role and same-named database provisioned by newServiceDatabase
+-- in infrastructure/main.go — and shares no tables with cms, so nothing is
+-- created yet. The file exists because internal/db/migrate.go embeds
+-- migrations/*.sql: with no migration at all the package would not compile.
+SELECT 1;

discovery/internal/db/migrations/0002_create_videos_table.down.sql

@@ -0,0 +1 @@
+DROP TABLE videos;

discovery/internal/db/migrations/0002_create_videos_table.up.sql

@@ -0,0 +1,19 @@
+-- The catalogue's own copy of a video, built from the announcements cms
+-- publishes on the catalogue topic. It is a read model, not a second source of
+-- truth: every column here arrives in an announcement, and nothing in this
+-- service ever writes a video of its own.
+--
+-- id is the id cms issued, carried in the announcement and reused verbatim, so
+-- the two services can talk about the same video without a mapping table. It
+-- is therefore NOT generated here: no DEFAULT, unlike cms's videos.id.
+CREATE TABLE videos (
+    id UUID PRIMARY KEY,
+    title TEXT NOT NULL,
+    playback_url TEXT NOT NULL DEFAULT '',
+    -- Denormalized on purpose. cms owns the category vocabulary and sends
+    -- names, so there is nothing here to join to and no id to keep in step;
+    -- an array is what the read side actually serves.
+    categories TEXT[] NOT NULL DEFAULT '{}',
+    created_at TIMESTAMPTZ NOT NULL DEFAULT now(),
+    updated_at TIMESTAMPTZ NOT NULL DEFAULT now()
+);

discovery/internal/db/migrations/0003_add_video_search.down.sql

@@ -0,0 +1,2 @@
+DROP INDEX IF EXISTS videos_search_vector_idx;
+ALTER TABLE videos DROP COLUMN IF EXISTS search_vector;

discovery/internal/db/migrations/0003_add_video_search.up.sql

@@ -0,0 +1,25 @@
+-- Lexical search over video titles.
+--
+-- The vector is a generated column rather than something the consumer writes:
+-- the catalogue's only writer is SaveVideo, and a title that arrives in an
+-- announcement should be searchable because it is stored, not because a second
+-- statement remembered to index it. STORED because a GIN index needs the value
+-- on disk to index it at all.
+--
+-- 'arabic' is the text search configuration, not 'english' or 'simple': it
+-- stems Arabic (الوثائقي -> وثايق) while leaving Latin-script tokens as written,
+-- which is the right trade for a mixed catalogue. The cost is that English
+-- words are not stemmed, so "documentaries" does not find "documentary".
+--
+-- The configuration is named explicitly rather than left to default_text_search_config,
+-- which is a per-session GUC — an expression that reads it is only STABLE, and
+-- a generated column requires IMMUTABLE. Naming it is also what keeps the
+-- column and every query in step: a query built with a different
+-- configuration would stem its terms differently and silently match nothing.
+ALTER TABLE videos
+    ADD COLUMN search_vector tsvector
+    GENERATED ALWAYS AS (to_tsvector('arabic', title)) STORED;
+
+-- GIN, not GiST: the catalogue is read far more than it is written, and GIN
+-- answers @@ faster at the cost of a slower update — the right way round here.
+CREATE INDEX videos_search_vector_idx ON videos USING GIN (search_vector);

discovery/internal/db/migrations/0004_add_video_categories_index.down.sql

@@ -0,0 +1 @@
+DROP INDEX IF EXISTS videos_categories_idx;

discovery/internal/db/migrations/0004_add_video_categories_index.up.sql

@@ -0,0 +1,7 @@
+-- Narrowing a search by category.
+--
+-- categories is a TEXT[] of names, and the search overlaps it against the
+-- names a reader asked for (&&). GIN is the index type that operator can use;
+-- without it the overlap is a filter applied after the rows are read, which is
+-- the whole table once the title term is broad.
+CREATE INDEX videos_categories_idx ON videos USING GIN (categories);

discovery/internal/db/migrations/0005_add_video_recency_index.down.sql

@@ -0,0 +1 @@
+DROP INDEX IF EXISTS videos_recent_idx;

discovery/internal/db/migrations/0005_add_video_recency_index.up.sql

@@ -0,0 +1,17 @@
+-- Browsing the catalogue, and paging through it.
+--
+-- Every search orders by (rank, created_at, id). When there is no title term
+-- the rank is a constant, and Postgres folds a constant sort key away — so the
+-- order that remains is exactly this index, and a browse becomes an index-only
+-- scan that stops as soon as the page is full instead of sorting the table.
+--
+-- Measured at 200k rows: a first page of a browse goes from a 15 ms parallel
+-- sequential scan with a top-N sort to 0.03 ms. It matters more for the cursor
+-- than for the first page — the keyset comparison (created_at, id) < (…)
+-- becomes an index condition rather than a filter, so a deep page is a seek
+-- to the right place in the index rather than a walk through everything
+-- before it.
+--
+-- DESC on both columns to match the ORDER BY: a btree can be read backwards,
+-- but only as a whole, so a mixed-direction ordering could not use it.
+CREATE INDEX videos_recent_idx ON videos (created_at DESC, id DESC);

discovery/internal/db/repositories/errors.go

@@ -0,0 +1,14 @@
+package repositories
+
+import "errors"
+
+// Sentinel errors the repositories return so handlers can tell "there is no
+// such row" from "the query failed", without importing database/sql to do it.
+
+// ErrVideoNotFound reports that the catalogue holds no video with that id.
+var ErrVideoNotFound = errors.New("video not found")
+
+// ErrInvalidCursor reports that a paging cursor could not be read. It is the
+// caller's mistake, not a fault in the catalogue, so it must not reach the
+// handler as an indistinguishable query failure.
+var ErrInvalidCursor = errors.New("invalid cursor")

discovery/internal/db/repositories/repositories.go

@@ -0,0 +1,7 @@
+// Package repositories holds every SQL statement the service runs. A
+// repository is a struct carrying the *sql.DB opened by db.CreateDBConnection
+// and is assigned to a package-level var in main, the way cms wires VideoRepo
+// and CatagoriesRepo, so handlers call through something substitutable.
+//
+// Empty until discovery has tables to read.
+package repositories

discovery/internal/db/repositories/videos.go

@@ -0,0 +1,252 @@
+package repositories
+
+import (
+	"context"
+	"database/sql"
+	"encoding/base64"
+	"encoding/json"
+	"errors"
+	"strings"
+	"thamanyah/discovery/internal/models"
+	"time"
+	"unicode"
+
+	"github.com/lib/pq"
+)
+
+type VideoRepository struct {
+	SQLDB *sql.DB
+}
+
+var VideoRepo VideoRepository
+
+// SaveVideo records an announced video, replacing what the catalogue already
+// held for that id.
+//
+// The upsert is what makes the consumer safe to retry: the catalogue topic
+// delivers at-least-once, so the same announcement can arrive twice, and a
+// plain INSERT would fail the second time on the primary key. Re-announcing a
+// video is also how it is corrected — a later announcement wins, which is why
+// every column is overwritten rather than merged.
+func (svc VideoRepository) SaveVideo(ctx context.Context, v models.Video) error {
+	_, err := svc.SQLDB.ExecContext(ctx, `
+		INSERT INTO videos (id, title, playback_url, categories)
+		VALUES ($1, $2, $3, $4)
+		ON CONFLICT (id) DO UPDATE
+		SET title = EXCLUDED.title,
+		    playback_url = EXCLUDED.playback_url,
+		    categories = EXCLUDED.categories,
+		    updated_at = now()
+	`, v.ID, v.Title, v.PlaybackURL, pq.Array(v.Categories))
+	return err
+}
+
+// GetVideoByID reads the catalogue's copy of one video. It returns
+// ErrVideoNotFound when the catalogue has not heard about it — which, for a
+// video cms has only just made ready, is a matter of timing rather than a
+// mistake.
+func (svc VideoRepository) GetVideoByID(ctx context.Context, id string) (models.Video, error) {
+	var v models.Video
+
+	err := svc.SQLDB.QueryRowContext(ctx, `
+		SELECT id, title, playback_url, categories, created_at, updated_at
+		FROM videos
+		WHERE id = $1
+	`, id).Scan(&v.ID, &v.Title, &v.PlaybackURL, pq.Array(&v.Categories), &v.CreatedAt, &v.UpdatedAt)
+	if errors.Is(err, sql.ErrNoRows) {
+		return models.Video{}, ErrVideoNotFound
+	}
+	if err != nil {
+		return models.Video{}, err
+	}
+
+	return v, nil
+}
+
+// searchConfig is the text search configuration the search_vector column is
+// generated with. Every query has to name the same one: a tsquery built under
+// a different configuration stems its terms differently and matches nothing.
+const searchConfig = "arabic"
+
+// tsQueryFor turns what a reader typed into a tsquery.
+//
+// The words are ANDed, and the last one is given a :* prefix match — someone
+// typing into a search box is usually part-way through their last word, so
+// "desert fal" should still find "Desert Falcons". Earlier words are matched
+// whole, because they have been finished.
+//
+// The input is split on everything that is not a letter or a digit, which
+// drops every character tsquery gives a meaning to (&, |, !, :, *, parens) and
+// leaves nothing that could change the shape of the query. That is what makes
+// it safe to interpolate the result into to_tsquery: the alternative,
+// websearch_to_tsquery, parses user syntax but cannot express a prefix match.
+//
+// It returns "" when there is nothing to search for, which callers read as
+// "no title filter" rather than "match nothing".
+func tsQueryFor(term string) string {
+	words := strings.FieldsFunc(term, func(r rune) bool {
+		return !unicode.IsLetter(r) && !unicode.IsDigit(r)
+	})
+	if len(words) == 0 {
+		return ""
+	}
+
+	for i, word := range words {
+		words[i] = strings.ToLower(word)
+	}
+	return strings.Join(words, " & ") + ":*"
+}
+
+// searchCursor is where a page of results stopped: the sort key of its last
+// row. Paging resumes by asking for the rows that sort after it.
+//
+// It carries the rank as well as the row's identity because the results are
+// ordered by relevance first, and "the row after this one" is only a
+// well-defined place if every term of the ordering is pinned.
+type searchCursor struct {
+	Rank      float64   `json:"r"`
+	CreatedAt time.Time `json:"t"`
+	ID        string    `json:"i"`
+}
+
+// encode renders a cursor as an opaque string. Opaque on purpose: it is a
+// place in a result set, not a number a client should do arithmetic on, and
+// making it look like one invites callers to guess a page.
+func (c searchCursor) encode() (string, error) {
+	raw, err := json.Marshal(c)
+	if err != nil {
+		return "", err
+	}
+	return base64.RawURLEncoding.EncodeToString(raw), nil
+}
+
+func decodeCursor(encoded string) (searchCursor, error) {
+	raw, err := base64.RawURLEncoding.DecodeString(encoded)
+	if err != nil {
+		return searchCursor{}, ErrInvalidCursor
+	}
+
+	var c searchCursor
+	if err := json.Unmarshal(raw, &c); err != nil {
+		return searchCursor{}, ErrInvalidCursor
+	}
+	if strings.TrimSpace(c.ID) == "" {
+		return searchCursor{}, ErrInvalidCursor
+	}
+
+	return c, nil
+}
+
+// SearchVideos finds catalogued videos whose title matches the given term and
+// which are filed under any of the given categories, most relevant first.
+//
+// It returns the page and the cursor that reaches the page after it; that
+// cursor is empty on the last page, which is how a caller knows to stop.
+//
+// The title predicate is skipped entirely when the term yields no words, and
+// the category predicate when no categories are named, so a search with
+// neither browses the whole catalogue newest-first.
+//
+// Paging is by key rather than by offset. OFFSET makes the database walk and
+// discard every row it skips, so deep pages get steadily more expensive, and
+// it addresses a page by its position — which moves whenever a video is
+// announced, repeating or skipping rows for a reader mid-browse. Comparing
+// against the last row's sort key has neither problem: the work is the same at
+// any depth, and the page after a given row is that same page however much has
+// been added since.
+func (svc VideoRepository) SearchVideos(ctx context.Context, term string, categories []string, limit int, cursor string) ([]models.Video, string, error) {
+	query := tsQueryFor(term)
+
+	// Never nil: a nil slice reaches Postgres as NULL, and cardinality(NULL)
+	// is NULL rather than 0, so the "no category filter" branch would not fire
+	// and every search would come back empty.
+	if categories == nil {
+		categories = []string{}
+	}
+
+	// A zero cursor is passed as NULL, which the query reads as "start at the
+	// beginning" rather than as a row to seek past.
+	var after *searchCursor
+	if strings.TrimSpace(cursor) != "" {
+		decoded, err := decodeCursor(cursor)
+		if err != nil {
+			return nil, "", err
+		}
+		after = &decoded
+	}
+
+	var (
+		afterRank *float64
+		afterTime *time.Time
+		afterID   *string
+	)
+	if after != nil {
+		afterRank, afterTime, afterID = &after.Rank, &after.CreatedAt, &after.ID
+	}
+
+	// One row more than asked for: if it comes back, there is another page and
+	// the extra row is discarded. That is what lets the last page say so,
+	// rather than handing out a cursor to an empty page.
+	//
+	// The rank is computed in the subquery and referenced by name in both the
+	// cursor comparison and the ORDER BY, so the two cannot drift apart — if
+	// they did, "the row after this one" would mean something different from
+	// the order the rows are actually in, and pages would overlap.
+	rows, err := svc.SQLDB.QueryContext(ctx, `
+		SELECT id, title, playback_url, categories, created_at, rank
+		FROM (
+			SELECT id, title, playback_url, categories, created_at,
+			       CASE WHEN $1 = '' THEN 0::float8
+			            ELSE ts_rank(search_vector, to_tsquery('`+searchConfig+`', $1))::float8
+			       END AS rank
+			FROM videos
+			WHERE ($1 = '' OR search_vector @@ to_tsquery('`+searchConfig+`', $1))
+			  AND (cardinality($2::text[]) = 0 OR categories && $2)
+		) ranked
+		WHERE $4::float8 IS NULL
+		   OR (rank, created_at, id) < ($4::float8, $5::timestamptz, $6::uuid)
+		ORDER BY rank DESC, created_at DESC, id DESC
+		LIMIT $3
+	`, query, pq.Array(categories), limit+1, afterRank, afterTime, afterID)
+	if err != nil {
+		return nil, "", err
+	}
+	defer rows.Close()
+
+	// Never nil: a search that matches nothing is an empty list on the wire,
+	// not null.
+	videos := []models.Video{}
+	ranks := []float64{}
+	for rows.Next() {
+		var (
+			v    models.Video
+			rank float64
+		)
+		if err := rows.Scan(&v.ID, &v.Title, &v.PlaybackURL, pq.Array(&v.Categories), &v.CreatedAt, &rank); err != nil {
+			return nil, "", err
+		}
+		videos = append(videos, v)
+		ranks = append(ranks, rank)
+	}
+	if err := rows.Err(); err != nil {
+		return nil, "", err
+	}
+
+	if len(videos) <= limit {
+		return videos, "", nil
+	}
+
+	videos, ranks = videos[:limit], ranks[:limit]
+	last := videos[len(videos)-1]
+
+	next, err := searchCursor{
+		Rank:      ranks[len(ranks)-1],
+		CreatedAt: last.CreatedAt,
+		ID:        last.ID,
+	}.encode()
+	if err != nil {
+		return nil, "", err
+	}
+
+	return videos, next, nil
+}

discovery/internal/handlers/handlers.go

@@ -0,0 +1,41 @@
+package handlers
+
+import (
+	"encoding/json"
+	"log"
+	"net/http"
+	"thamanyah/discovery/internal/api"
+)
+
+// Health is the liveness probe the ALB target group polls.
+//
+//	@Summary		Health check
+//	@Description	Reports that the service is up and serving. Used as the ALB target group health check; it does not verify the database connection.
+//	@Tags			system
+//	@Produce		json
+//	@Success		200	{object}	api.HealthResponse
+//	@Router			/health [get]
+func Health(w http.ResponseWriter, r *http.Request) {
+	writeJSON(w, http.StatusOK, api.HealthResponse{Status: "ok"})
+}
+
+func writeJSON(w http.ResponseWriter, status int, body any) {
+	writeJSONContent(w, "application/json", status, body)
+}
+
+func writeProblem(w http.ResponseWriter, status int, title, detail string) {
+	writeJSONContent(w, "application/problem+json", status, api.ProblemDetails{
+		Type:   "about:blank",
+		Title:  title,
+		Status: status,
+		Detail: detail,
+	})
+}
+
+func writeJSONContent(w http.ResponseWriter, contentType string, status int, body any) {
+	w.Header().Set("Content-Type", contentType)
+	w.WriteHeader(status)
+	if err := json.NewEncoder(w).Encode(body); err != nil {
+		log.Printf("Something Went Wrong Writing The Response: %s", err)
+	}
+}

discovery/internal/handlers/videos.go

@@ -0,0 +1,306 @@
+package handlers
+
+import (
+	"context"
+	"encoding/json"
+	"errors"
+	"fmt"
+	"log"
+	"net/http"
+	"net/url"
+	"slices"
+	"strconv"
+	"strings"
+	"thamanyah/discovery/internal/api"
+	"thamanyah/discovery/internal/db/repositories"
+	"thamanyah/discovery/internal/services"
+	"time"
+)
+
+// GetVideo serves the catalogue's copy of one video.
+//
+//	@Summary		Get a catalogued video
+//	@Description	Returns the catalogue's copy of a video: what a reader needs to show and play it. A video appears here only after the CMS has announced it as ready, so a video that is still transcoding — or one the CMS never made ready — is a 404.
+//	@Tags			videos
+//	@Produce		json
+//	@Param			id	path		string	true	"Video id, as issued by the CMS"
+//	@Success		200	{object}	api.Video
+//	@Failure		404	{object}	api.ProblemDetails
+//	@Failure		500	{object}	api.ProblemDetails
+//	@Router			/api/videos/{id} [get]
+func GetVideo(w http.ResponseWriter, r *http.Request) {
+	id := r.PathValue("id")
+
+	video, err := repositories.VideoRepo.GetVideoByID(r.Context(), id)
+	if errors.Is(err, repositories.ErrVideoNotFound) {
+		writeProblem(w, http.StatusNotFound, "Video Not In The Catalogue",
+			"No video with that id is in the catalogue. A video appears here once the CMS announces it as ready, so one that is still being transcoded is not here yet.")
+		return
+	}
+	if err != nil {
+		log.Printf("Something Went Wrong Loading A Catalogued Video: id=%q: %s", id, err)
+		writeProblem(w, http.StatusInternalServerError, "Something Went Wrong Loading The Catalogue",
+			"The video could not be read from the database. This is a server-side fault and the request was not processed; retrying in a few moments may succeed.")
+		return
+	}
+
+	// Translated field by field rather than returned as-is: models.Video is the
+	// row, api.Video is the published schema, and neither should drag the other
+	// along when it changes.
+	writeJSON(w, http.StatusOK, api.Video{
+		ID:          video.ID,
+		Title:       video.Title,
+		PlaybackURL: video.PlaybackURL,
+		// Never null on the wire: a video with no categories has an empty
+		// list, which is a different thing from "unknown".
+		Categories: append([]string{}, video.Categories...),
+	})
+}
+
+const (
+	// searchPageSize is how many videos a search returns when the caller does
+	// not say. Small enough to keep the common request cheap.
+	searchPageSize = 20
+
+	// maxSearchPageSize is the largest page the catalogue will build, however
+	// big a page the caller asks for. The endpoint is read constantly, and a
+	// page size is a request for work: without a ceiling, one client asking
+	// for the whole catalogue in a single response makes every other reader
+	// wait behind it.
+	//
+	// Asking for more is not an error — the page is simply capped, and the
+	// cursor still reaches the rest — because a client that wants everything
+	// is doing something legitimate, just not in one request.
+	maxSearchPageSize = 100
+
+	// searchCacheTTL is how long a page of results stays servable from the
+	// cache before it has to be built again.
+	//
+	// Nothing invalidates an entry when a video is announced: the consumer
+	// writes rows continuously and would have to know which of the cached
+	// pages a new title belongs on, which for a ranked search is every page it
+	// outranks. Expiry is the cheaper answer, and it bounds the staleness — a
+	// video is findable within a minute of being catalogued, which is the same
+	// order as the announcement's own delivery lag.
+	searchCacheTTL = time.Minute
+)
+
+// searchCacheKey is what one page of search results is stored under: the title
+// searched for, the categories it was narrowed to, and the page within that
+// result set.
+//
+// The page is named by the cursor, because with keyset paging that is what a
+// page *is* — there is no page number to key on, and the cursor identifies the
+// same rows whenever it is presented. The empty cursor is the first page.
+//
+// The limit is in the key too, even though a reader would not call it part of
+// "which page". It has to be: the same title, categories and cursor with a
+// different limit is a different set of rows, and sharing an entry between
+// them would hand a client asking for 50 a page of 20.
+//
+// The parts are JSON-encoded rather than pasted together with separators so
+// that no category name can be mistaken for the boundary between two of them —
+// {"c":["a,b"]} and {"c":["a","b"]} stay distinguishable, which "a,b" and
+// "a,b" would not. The key stays readable in redis-cli either way.
+func searchCacheKey(title string, categories []string, limit int, cursor string) string {
+	// The title is lowercased because the search itself is: tsQueryFor folds
+	// case before it builds the tsquery, so two spellings that differ only in
+	// case are the same search and should be the same entry. The categories
+	// are not — they are compared to the stored array verbatim, so "News" and
+	// "news" really do ask different questions.
+	//
+	// Sorting the categories is safe for the same reason the SQL uses &&:
+	// they mean "any of these", so their order never changed the answer. Two
+	// readers naming the same categories in a different order now share one
+	// entry instead of building the same page twice.
+	//
+	// Never nil, so that a client sending no categories at all and one whose
+	// categories parsed away to none — the same search, since both mean "every
+	// category" — land on one entry rather than on `null` and `[]`.
+	sorted := append([]string{}, categories...)
+	slices.Sort(sorted)
+	sorted = slices.Compact(sorted)
+
+	parts := struct {
+		Title      string   `json:"t"`
+		Categories []string `json:"c"`
+		Limit      int      `json:"l"`
+		Cursor     string   `json:"p"`
+	}{
+		Title:      strings.ToLower(strings.TrimSpace(title)),
+		Categories: sorted,
+		Limit:      limit,
+		Cursor:     cursor,
+	}
+
+	// Cannot fail: every field is a string, a string slice or an int.
+	encoded, _ := json.Marshal(parts)
+	return "discovery:search:v1:" + string(encoded)
+}
+
+// cachedSearch returns the page held under key, or false when there is none to
+// serve.
+//
+// Every failure reads as "no cached page": a cache that is unreachable, slow
+// or holding something unreadable must cost a reader nothing more than the
+// database query they would have paid for anyway. Only faults worth acting on
+// are logged — a miss is not one.
+func cachedSearch(ctx context.Context, key string) (api.SearchResults, bool) {
+	encoded, err := services.CacheClient.Get(ctx, key)
+	if errors.Is(err, services.ErrCacheMiss) {
+		return api.SearchResults{}, false
+	}
+	if err != nil {
+		log.Printf("Could Not Read The Search Cache, Falling Back To The Database: key=%q: %s", key, err)
+		return api.SearchResults{}, false
+	}
+
+	var results api.SearchResults
+	if err := json.Unmarshal(encoded, &results); err != nil {
+		// Only reachable if something else wrote this key, or the shape
+		// changed without the v1 in the prefix changing with it.
+		log.Printf("Could Not Read A Cached Search Page, Falling Back To The Database: key=%q: %s", key, err)
+		return api.SearchResults{}, false
+	}
+
+	return results, true
+}
+
+func cacheSearch(ctx context.Context, key string, results api.SearchResults) {
+	encoded, err := json.Marshal(results)
+	if err != nil {
+		log.Printf("Could Not Encode A Search Page For The Cache: key=%q: %s", key, err)
+		return
+	}
+
+	if err := services.CacheClient.Set(ctx, key, encoded, searchCacheTTL); err != nil {
+		log.Printf("Could Not Write The Search Cache: key=%q: %s", key, err)
+	}
+}
+
+// searchQuery is a search as the query string spells it out. Every parameter
+// is optional: a request carrying none of them browses the whole catalogue,
+// newest first.
+type searchQuery struct {
+	title      string
+	categories []string
+	limit      int
+	cursor     string
+}
+
+// parseSearchQuery reads the search out of the URL, applying the default and
+// the cap to the page size so that the rest of the handler works with a limit
+// it can use as given.
+//
+// The only parameter that can be malformed is the limit: every other one is a
+// string the search takes as written, so there is nothing to spell wrong.
+func parseSearchQuery(values url.Values) (searchQuery, error) {
+	parsed := searchQuery{
+		title: values.Get("title"),
+		// Repeated — ?categories=news&categories=documentary — rather than one
+		// comma-separated value, for the reason searchCacheKey encodes them as
+		// JSON rather than joining them: a category name is a name, and one
+		// containing a comma must not read as two.
+		//
+		// Empty values are dropped so that a client sending ?categories= means
+		// "every category", as omitting it does, rather than narrowing the
+		// search to a category named "".
+		categories: nonEmpty(values["categories"]),
+		cursor:     values.Get("cursor"),
+		limit:      searchPageSize,
+	}
+
+	if raw := values.Get("limit"); raw != "" {
+		limit, err := strconv.Atoi(raw)
+		if err != nil {
+			return searchQuery{}, fmt.Errorf("limit %q is not a whole number", raw)
+		}
+		// Zero and negative are read as "no preference", the same as omitting
+		// the parameter. A page of nothing is not what anyone meant by it, and
+		// the request is still answerable, so answering it beats refusing it.
+		if limit > 0 {
+			parsed.limit = limit
+		}
+	}
+
+	if parsed.limit > maxSearchPageSize {
+		parsed.limit = maxSearchPageSize
+	}
+
+	return parsed, nil
+}
+
+func nonEmpty(values []string) []string {
+	kept := make([]string, 0, len(values))
+	for _, v := range values {
+		if v != "" {
+			kept = append(kept, v)
+		}
+	}
+	return kept
+}
+
+// SearchVideos serves the catalogue's lexical search.
+//
+// It is served on GET: a search is safe and idempotent, which POST is not, and
+// its parameters are small enough to say in a query string — which keeps the
+// endpoint reachable from anything that speaks HTTP, describable in the
+// OpenAPI spec, and cacheable by the intermediaries between the reader and
+// this service.
+//
+//	@Summary		Search the catalogue
+//	@Description	Finds catalogued videos by the words of their title and the categories they are filed under, most relevant first. Every parameter is optional — a request with none of them browses the whole catalogue, newest first. Paging is by cursor rather than by offset, so a page stays the same page while videos are being announced into the catalogue around it.
+//	@Tags			videos
+//	@Produce		json
+//	@Param			title		query		string		false	"Matched lexically against video titles: the words are all required, and the last is treated as a prefix so a part-typed word still finds something."	example(desert fal)
+//	@Param			categories	query		[]string	false	"Narrows the search to videos filed under any one of these names — not all of them. Repeat the parameter per category. Omitted means every category."	collectionFormat(multi)	example(documentary)
+//	@Param			limit		query		int			false	"How many videos to return. Absent, zero or negative takes the default of 20; anything above the maximum of 100 is capped to it rather than refused, and the cursor still reaches the rest."	default(20)
+//	@Param			cursor		query		string		false	"Asks for the page after the one a previous search ended at. Send back the nextCursor from that search unchanged; it is opaque, and the only thing to do with it is return it."
+//	@Success		200			{object}	api.SearchResults
+//	@Failure		400			{object}	api.ProblemDetails
+//	@Failure		500			{object}	api.ProblemDetails
+//	@Router			/api/videos [get]
+func SearchVideos(w http.ResponseWriter, r *http.Request) {
+	request, err := parseSearchQuery(r.URL.Query())
+	if err != nil {
+		writeProblem(w, http.StatusBadRequest, "Malformed Search",
+			"The search parameters could not be read: "+err.Error()+". Every parameter is optional; a request with none of them browses the whole catalogue.")
+		return
+	}
+
+	cacheKey := searchCacheKey(request.title, request.categories, request.limit, request.cursor)
+	if cached, hit := cachedSearch(r.Context(), cacheKey); hit {
+		writeJSON(w, http.StatusOK, cached)
+		return
+	}
+
+	found, next, err := repositories.VideoRepo.SearchVideos(r.Context(), request.title, request.categories, request.limit, request.cursor)
+	if errors.Is(err, repositories.ErrInvalidCursor) {
+		writeProblem(w, http.StatusBadRequest, "Malformed Search Cursor",
+			"The cursor could not be read. Send back the nextCursor from a previous search unchanged, or omit it to start from the first page.")
+		return
+	}
+	if err != nil {
+		log.Printf("Something Went Wrong Searching The Catalogue: title=%q: %s", request.title, err)
+		writeProblem(w, http.StatusInternalServerError, "Something Went Wrong Searching The Catalogue",
+			"The catalogue could not be searched. This is a server-side fault and the request was not processed; retrying in a few moments may succeed.")
+		return
+	}
+
+	// Translated field by field, for the same reason GetVideo does it: the row
+	// and the published schema are different things.
+	videos := make([]api.Video, 0, len(found))
+	for _, v := range found {
+		videos = append(videos, api.Video{
+			ID:          v.ID,
+			Title:       v.Title,
+			PlaybackURL: v.PlaybackURL,
+			Categories:  append([]string{}, v.Categories...),
+		})
+	}
+
+	results := api.SearchResults{Videos: videos, NextCursor: next}
+	cacheSearch(r.Context(), cacheKey, results)
+
+	writeJSON(w, http.StatusOK, results)
+}

discovery/internal/models/models.go

@@ -0,0 +1,8 @@
+// Package models holds the domain types the rest of the service passes
+// around, the way cms holds Video and Category: plain Go structs with no JSON
+// tags and no SQL, so that renaming a field here never moves the published
+// API (that lives in internal/api) and never rewrites a query (that lives in
+// internal/db/repositories).
+//
+// Empty until discovery has a domain to model.
+package models

discovery/internal/models/video.go

@@ -0,0 +1,18 @@
+package models
+
+import "time"
+
+// Video is the catalogue's copy of a video cms has announced as ready. Every
+// member is filled from the announcement — discovery originates none of it.
+type Video struct {
+	// ID is the id cms issued, reused verbatim so both services name the same
+	// video the same way.
+	ID          string
+	Title       string
+	PlaybackURL string
+	// Categories are names, not ids: cms owns the vocabulary and announces it
+	// by name, so nothing here has to track its numbering.
+	Categories []string
+	CreatedAt  time.Time
+	UpdatedAt  time.Time
+}

discovery/internal/services/cache.go

@@ -0,0 +1,58 @@
+package services
+
+import (
+	"context"
+	"errors"
+	"fmt"
+	"log"
+	"time"
+
+	"github.com/redis/go-redis/v9"
+)
+
+var CacheClient Cache
+
+var ErrCacheMiss = errors.New("cache: miss")
+
+type Cache interface {
+	Get(ctx context.Context, key string) ([]byte, error)
+	Set(ctx context.Context, key string, value []byte, ttl time.Duration) error
+}
+
+type RedisConcrete struct {
+	Redis *redis.Client
+}
+
+const cacheOperationTimeout = 100 * time.Millisecond
+
+func (svc RedisConcrete) Get(ctx context.Context, key string) ([]byte, error) {
+	ctx, cancel := context.WithTimeout(ctx, cacheOperationTimeout)
+	defer cancel()
+
+	value, err := svc.Redis.Get(ctx, key).Bytes()
+	if errors.Is(err, redis.Nil) {
+		return nil, ErrCacheMiss
+	}
+	if err != nil {
+		return nil, err
+	}
+	return value, nil
+}
+
+func (svc RedisConcrete) Set(ctx context.Context, key string, value []byte, ttl time.Duration) error {
+	ctx, cancel := context.WithTimeout(ctx, cacheOperationTimeout)
+	defer cancel()
+
+	return svc.Redis.Set(ctx, key, value, ttl).Err()
+}
+
+func (svc RedisConcrete) Close() error {
+	return svc.Redis.Close()
+}
+
+func (svc RedisConcrete) AssertSuccessfulConnection(ctx context.Context) {
+	if err := svc.Redis.Ping(ctx).Err(); err != nil {
+		log.Printf("WARNING: %s", fmt.Errorf("redis: cannot reach cache at %q, searches will be served from the database: %w",
+			svc.Redis.Options().Addr, err))
+	}
+}

discovery/internal/services/sqs.go

@@ -0,0 +1,83 @@
+// Package services holds this service's AWS clients. Discovery reaches exactly
+// one AWS API — the SQS queue subscribed to the catalogue topic — so unlike
+// cms there is no S3 or MediaConvert here, and nothing database-related.
+package services
+
+import (
+	"context"
+	"fmt"
+
+	"github.com/aws/aws-sdk-go-v2/service/sqs"
+)
+
+var SQSClient SQS
+
+// QueueMessage is one message taken off the queue. ReceiptHandle is what
+// identifies it for deletion — it belongs to this delivery, not to the
+// message, so it cannot be held across receives.
+type QueueMessage struct {
+	Body          string
+	ReceiptHandle string
+}
+
+type SQS interface {
+	ReceiveMessages(ctx context.Context) ([]QueueMessage, error)
+	DeleteMessage(ctx context.Context, receiptHandle string) error
+}
+
+type SQSConcrete struct {
+	SQSClient *sqs.Client
+	QueueURL  string
+}
+
+// receiveWaitSeconds turns every receive into a long poll: the call parks on
+// the server until a message arrives or this elapses, so an event is picked up
+// within milliseconds of being published while an idle consumer costs one
+// request every twenty seconds rather than spinning. 20 is the AWS maximum.
+const receiveWaitSeconds = 20
+
+func (svc SQSConcrete) ReceiveMessages(ctx context.Context) ([]QueueMessage, error) {
+	output, err := svc.SQSClient.ReceiveMessage(ctx, &sqs.ReceiveMessageInput{
+		QueueUrl:            &svc.QueueURL,
+		MaxNumberOfMessages: 10,
+		WaitTimeSeconds:     receiveWaitSeconds,
+	})
+	if err != nil {
+		return nil, err
+	}
+
+	messages := make([]QueueMessage, 0, len(output.Messages))
+	for _, message := range output.Messages {
+		if message.Body == nil || message.ReceiptHandle == nil {
+			continue
+		}
+		messages = append(messages, QueueMessage{
+			Body:          *message.Body,
+			ReceiptHandle: *message.ReceiptHandle,
+		})
+	}
+
+	return messages, nil
+}
+
+// DeleteMessage acknowledges a message. Until this is called the message is
+// merely invisible, and it returns to the queue when its visibility timeout
+// expires — which is how a handler that failed gets another attempt.
+func (svc SQSConcrete) DeleteMessage(ctx context.Context, receiptHandle string) error {
+	_, err := svc.SQSClient.DeleteMessage(ctx, &sqs.DeleteMessageInput{
+		QueueUrl:      &svc.QueueURL,
+		ReceiptHandle: &receiptHandle,
+	})
+	return err
+}
+
+// AssertSuccessfulConnection verifies the queue is reachable and readable with
+// the configured credentials. It panics on failure, since a service whose
+// consumer cannot start would serve a catalogue that silently stops growing.
+func (svc SQSConcrete) AssertSuccessfulConnection(ctx context.Context) {
+	if _, err := svc.SQSClient.GetQueueAttributes(ctx, &sqs.GetQueueAttributesInput{
+		QueueUrl: &svc.QueueURL,
+	}); err != nil {
+		panic(fmt.Errorf("sqs: cannot connect to queue %q: %w", svc.QueueURL, err))
+	}
+}

discovery/main.go

@@ -0,0 +1,174 @@
+package main
+
+import (
+	"context"
+	"fmt"
+	"log"
+	"net/http"
+	"os"
+	"thamanyah/discovery/internal/consumers"
+	"thamanyah/discovery/internal/db"
+	"thamanyah/discovery/internal/db/repositories"
+	"thamanyah/discovery/internal/handlers"
+	"thamanyah/discovery/internal/services"
+	"time"
+
+	awsconfig "github.com/aws/aws-sdk-go-v2/config"
+	"github.com/aws/aws-sdk-go-v2/service/sqs"
+	_ "github.com/lib/pq"
+	"github.com/redis/go-redis/v9"
+	httpSwagger "github.com/swaggo/http-swagger/v2"
+
+	_ "thamanyah/discovery/docs"
+)
+
+//	@title			Thamanyah Discovery API
+//	@version		1.0
+//	@description	JSON API for browsing the Thamanyah catalogue. Read-side counterpart to the CMS, which is what ingests videos.
+//	@description
+//	@description	The catalogue search is `GET /api/videos`, listed below: title is matched lexically with the last word as a prefix, categories narrow to videos filed under any one of them, limit defaults to 20 and is capped at 100, and cursor is the opaque `nextCursor` of a previous search.
+//	@BasePath		/
+
+func main() {
+	if len(os.Args) > 1 && os.Args[1] == "migrate" {
+		runMigrate()
+		return
+	}
+	runServer()
+}
+
+func runMigrate() {
+	concreteDBClient := db.CreateDBConnection(requireDBConnectionString())
+	defer db.CloseConnection(concreteDBClient)
+	if err := db.Migrate(concreteDBClient); err != nil {
+		panic(err)
+	}
+	log.Println("migrations applied successfully")
+}
+
+func runServer() {
+	// The queue subscribed to cms's catalogue topic. This is the only AWS
+	// service discovery talks to.
+	catalogueQueueURL := os.Getenv("CATALOGUE_EVENTS_QUEUE_URL")
+	if catalogueQueueURL == "" {
+		panic(fmt.Errorf("missing required env var: CATALOGUE_EVENTS_QUEUE_URL"))
+	}
+
+	awsConfig, err := awsconfig.LoadDefaultConfig(context.Background())
+	if err != nil {
+		panic(err)
+	}
+
+	concreteSQSClient := &services.SQSConcrete{
+		SQSClient: sqs.NewFromConfig(awsConfig),
+		QueueURL:  catalogueQueueURL,
+	}
+	concreteSQSClient.AssertSuccessfulConnection(context.Background())
+
+	services.SQSClient = concreteSQSClient
+
+	// The ElastiCache Redis node the catalogue search is answered from. Not an
+	// AWS API call — the address resolves inside the VPC and the security
+	// group is what grants access — so it needs nothing from the task role and
+	// nothing from awsConfig.
+	//
+	// The address is required because a missing one is a deployment mistake,
+	// not a choice; whether the node actually answers is a different question,
+	// and one the service deliberately survives getting "no" to.
+	redisAddress := os.Getenv("REDIS_ADDR")
+	if redisAddress == "" {
+		panic(fmt.Errorf("missing required env var: REDIS_ADDR"))
+	}
+
+	concreteCacheClient := &services.RedisConcrete{
+		Redis: redis.NewClient(&redis.Options{
+			Addr: redisAddress,
+			// Per-call deadlines on top of the budget RedisConcrete already
+			// applies, so a connection that hangs rather than refusing cannot
+			// tie up a goroutine past the request that opened it. Retries are
+			// off for the same reason the budget is small: a second attempt at
+			// a cache costs more than the query it is saving.
+			DialTimeout:  100 * time.Millisecond,
+			ReadTimeout:  100 * time.Millisecond,
+			WriteTimeout: 100 * time.Millisecond,
+			MaxRetries:   -1,
+		}),
+	}
+	defer func() { _ = concreteCacheClient.Close() }()
+	concreteCacheClient.AssertSuccessfulConnection(context.Background())
+
+	services.CacheClient = concreteCacheClient
+
+	concreteDBClient := db.CreateDBConnection(requireDBConnectionString())
+	defer db.CloseConnection(concreteDBClient)
+	db.AssertSuccessfulConnection(context.Background(), concreteDBClient)
+
+	repositories.VideoRepo = repositories.VideoRepository{
+		SQLDB: concreteDBClient,
+	}
+
+	// Consumes announcements for the lifetime of the process. Cancelled when
+	// runServer returns, which today only happens if the server itself fails.
+	consumerCtx, stopConsumer := context.WithCancel(context.Background())
+	defer stopConsumer()
+	go consumers.CatalogueEvents{}.Run(consumerCtx)
+
+	mux := http.NewServeMux()
+
+	mux.HandleFunc("GET /health", handlers.Health)
+	mux.HandleFunc("GET /api/videos/{id}", handlers.GetVideo)
+	// The catalogue search. GET rather than POST because a search is safe and
+	// idempotent, and its parameters — title, categories, limit, cursor — fit
+	// a query string. ServeMux keeps this apart from the {id} pattern above:
+	// the more specific one wins.
+	mux.HandleFunc("GET /api/videos", handlers.SearchVideos)
+
+	// Swagger UI and the generated spec. The UI assets are embedded in the
+	// binary by swaggo/files, so this needs no static directory on disk.
+	mux.Handle("GET /swagger/", httpSwagger.WrapHandler)
+
+	addr := ":8080"
+	log.Printf("listening on %s", addr)
+	if err := http.ListenAndServe(addr, mux); err != nil {
+		log.Fatal(err)
+	}
+}
+
+func requireDBConnectionString() string {
+	dbHost := os.Getenv("DB_HOST")
+	if dbHost == "" {
+		panic(fmt.Errorf("missing required env var: DB_HOST"))
+	}
+
+	dbPort := os.Getenv("DB_PORT")
+	if dbPort == "" {
+		panic(fmt.Errorf("missing required env var: DB_PORT"))
+	}
+
+	dbUser := os.Getenv("DB_USER")
+	if dbUser == "" {
+		panic(fmt.Errorf("missing required env var: DB_USER"))
+	}
+
+	dbPassword := os.Getenv("DB_PASSWORD")
+	if dbPassword == "" {
+		panic(fmt.Errorf("missing required env var: DB_PASSWORD"))
+	}
+
+	dbName := os.Getenv("DB_NAME")
+	if dbName == "" {
+		panic(fmt.Errorf("missing required env var: DB_NAME"))
+	}
+
+	// Defaults to require: RDS terminates TLS, and only a local Postgres that
+	// serves none (LocalStack's RDS emulation) has any business overriding it.
+	dbSSLMode := os.Getenv("DB_SSLMODE")
+	if dbSSLMode == "" {
+		dbSSLMode = "require"
+	}
+
+	return fmt.Sprintf(
+		"host=%s port=%s user=%s password=%s dbname=%s sslmode=%s",
+		dbHost, dbPort, dbUser, dbPassword, dbName, dbSSLMode,
+	)
+}

docker-compose.yml

@@ -55,7 +55,7 @@ services:
     environment:
       - PULUMI_CONFIG_PASSPHRASE=${PULUMI_CONFIG_PASSPHRASE:-local}
       - PULUMI_SKIP_UPDATE_CHECK=true
-      - DB_PASSWORD=${DB_PASSWORD:?}
+      - DB_PASSWORD=example_password
     volumes:
       - "./infrastructure:/infra"
       - "pulumi-state:/state" # stack state, kept out of the repo
@@ -100,6 +100,9 @@ services:
       # of the process. Pinned by the localstack branch in infrastructure/main.go;
       # LocalStack always uses account 000000000000.
       - MEDIACONVERT_EVENTS_QUEUE_URL=${MEDIACONVERT_EVENTS_QUEUE_URL:-http://localhost.localstack.cloud:4566/000000000000/cms-mediaconvert-events}
+      # The topic ready videos are announced on, which fans out to the read
+      # side's queue. Pinned by the localstack branch in infrastructure/main.go.
+      - CATALOGUE_EVENTS_TOPIC_ARN=${CATALOGUE_EVENTS_TOPIC_ARN:-arn:aws:sns:us-east-1:000000000000:catalogue-events}
       # Where finished HLS output is served from. In AWS this is the CloudFront
       # distribution; there is none under LocalStack, so the encoded bucket is
       # addressed directly — path-style, for the same reason S3 is elsewhere.
@@ -110,7 +113,7 @@ services:
       - DB_PORT=${DB_PORT:-4510}
       - DB_NAME=${DB_NAME:-cms}
       - DB_USER=${DB_USER:-cms}
-      - DB_PASSWORD=${DB_PASSWORD:?}
+      - DB_PASSWORD=example_password
       - DB_SSLMODE=${DB_SSLMODE:-disable}
     depends_on:
       infra:
@@ -123,6 +126,64 @@ services:
         - action: rebuild
           path: ./cms
 
+  # Stands in for the ElastiCache node infrastructure/main.go provisions on the
+  # AWS stacks. Not provisioned through LocalStack like the buckets and the
+  # database are: ElastiCache hands out an endpoint on a port it chooses, and
+  # the literal REDIS_ADDR below has to be known before anything is created.
+  # Nothing here is worth persisting — every entry is derivable from Postgres,
+  # which is the whole point of a cache — so there is no volume.
+  redis:
+    container_name: "${REDIS_DOCKER_NAME:-redis}"
+    image: redis:7-alpine
+    ports:
+      - "127.0.0.1:6379:6379"
+    healthcheck:
+      test: ["CMD", "redis-cli", "ping"]
+      interval: 5s
+      timeout: 3s
+      retries: 10
+
+  discovery:
+    container_name: "${DISCOVERY_DOCKER_NAME:-discovery}"
+    build:
+      context: ./discovery
+      dockerfile: Dockerfile
+    ports:
+      - "127.0.0.1:8080:8080" # JSON API + Swagger UI at /swagger/
+    environment:
+      # AWS: discovery reaches exactly one AWS API — the queue subscribed to
+      # cms's catalogue topic, which it drains for the lifetime of the process.
+      # That is why it now has a task role in infrastructure/main.go, scoped to
+      # this queue and nothing else.
+      - AWS_REGION=${AWS_REGION:-us-east-1}
+      - AWS_ACCESS_KEY_ID=${AWS_ACCESS_KEY_ID:-test}
+      - AWS_SECRET_ACCESS_KEY=${AWS_SECRET_ACCESS_KEY:-test}
+      - AWS_ENDPOINT_URL=${AWS_ENDPOINT_URL:-http://localhost.localstack.cloud:4566}
+      - CATALOGUE_EVENTS_QUEUE_URL=${CATALOGUE_EVENTS_QUEUE_URL:-http://localhost.localstack.cloud:4566/000000000000/discovery-catalogue-events}
+      # The catalogue search's cache — the redis service below, standing in for
+      # ElastiCache. main.go panics without an address but only warns if the
+      # node does not answer: a cold cache means slower searches, not none.
+      - REDIS_ADDR=${REDIS_ADDR:-redis:6379}
+      # Postgres: its own role and database, provisioned alongside the cms ones
+      # by newServiceDatabase, so the two services share no credentials.
+      - DB_HOST=${DISCOVERY_DB_HOST:-localhost.localstack.cloud}
+      - DB_PORT=${DISCOVERY_DB_PORT:-4510}
+      - DB_NAME=${DISCOVERY_DB_NAME:-discovery}
+      - DB_USER=${DISCOVERY_DB_USER:-discovery}
+      - DB_PASSWORD=example_password
+      - DB_SSLMODE=${DISCOVERY_DB_SSLMODE:-disable}
+    depends_on:
+      infra:
+        condition: service_completed_successfully
+      redis:
+        condition: service_healthy
+    develop:
+      watch:
+        # Same as cms: a compiled binary, so every change means a new image.
+        # Needs `docker compose up --watch` (or `docker compose watch`).
+        - action: rebuild
+          path: ./discovery
+
 volumes:
   pulumi-state:
   pulumi-home:

infrastructure/main.go

@@ -13,6 +13,7 @@ import (
 	"github.com/pulumi/pulumi-aws/sdk/v7/go/aws/ec2"
 	"github.com/pulumi/pulumi-aws/sdk/v7/go/aws/ecr"
 	"github.com/pulumi/pulumi-aws/sdk/v7/go/aws/ecs"
+	"github.com/pulumi/pulumi-aws/sdk/v7/go/aws/elasticache"
 	"github.com/pulumi/pulumi-aws/sdk/v7/go/aws/iam"
 	"github.com/pulumi/pulumi-aws/sdk/v7/go/aws/lb"
 	"github.com/pulumi/pulumi-aws/sdk/v7/go/aws/rds"
@@ -384,6 +385,7 @@ func main() {
 			cmsRepo, discoveryRepo                 *ecr.Repository
 			cmsService, discoveryService           *ecs.Service
 			cmsAlb, discoveryAlb                   *lb.LoadBalancer
+			searchCacheAddress                     pulumi.StringOutput
 		)
 
 		// Bucket names are left to Pulumi's auto-naming in AWS, but pinned under
@@ -426,6 +428,42 @@ func main() {
 				return err
 			}
 
+			// HLS is played by JavaScript (hls.js fetches the .m3u8 playlist
+			// and every .ts/.m4s segment with XHR), so the CDN has to answer
+			// with CORS headers or the browser drops the response — the
+			// "CORS Missing Allow Origin" failure. The S3 origin sends none of
+			// its own, so CloudFront adds them here, at the edge, for cached
+			// and uncached responses alike.
+			corsHeaders, err := cloudfront.NewResponseHeadersPolicy(ctx, "encoded-bucket-cors-headers", &cloudfront.ResponseHeadersPolicyArgs{
+				Comment: pulumi.String("CORS headers so browsers can fetch HLS playlists and segments"),
+				CorsConfig: &cloudfront.ResponseHeadersPolicyCorsConfigArgs{
+					AccessControlAllowCredentials: pulumi.Bool(false),
+					AccessControlAllowHeaders: &cloudfront.ResponseHeadersPolicyCorsConfigAccessControlAllowHeadersArgs{
+						Items: pulumi.ToStringArray([]string{"*"}),
+					},
+					AccessControlAllowMethods: &cloudfront.ResponseHeadersPolicyCorsConfigAccessControlAllowMethodsArgs{
+						Items: pulumi.ToStringArray([]string{"GET", "HEAD", "OPTIONS"}),
+					},
+					// Playback is public and unauthenticated, so any origin may
+					// read it. A wildcard also keeps Origin out of the cache
+					// key: with a fixed list CloudFront would have to vary the
+					// cached response on the request's Origin header.
+					AccessControlAllowOrigins: &cloudfront.ResponseHeadersPolicyCorsConfigAccessControlAllowOriginsArgs{
+						Items: pulumi.ToStringArray([]string{"*"}),
+					},
+					// Content-Range/Content-Length are what a player reads back
+					// when it seeks with a Range request.
+					AccessControlExposeHeaders: &cloudfront.ResponseHeadersPolicyCorsConfigAccessControlExposeHeadersArgs{
+						Items: pulumi.ToStringArray([]string{"Content-Length", "Content-Range", "Date", "ETag"}),
+					},
+					AccessControlMaxAgeSec: pulumi.Int(3000),
+					OriginOverride:         pulumi.Bool(true),
+				},
+			})
+			if err != nil {
+				return err
+			}
+
 			distribution, err = cloudfront.NewDistribution(ctx, "encoded-bucket-cdn", &cloudfront.DistributionArgs{
 				Enabled: pulumi.Bool(true),
 				Comment: pulumi.String("Edge caching for encoded-bucket assets"),
@@ -442,10 +480,14 @@ func main() {
 				DefaultCacheBehavior: &cloudfront.DistributionDefaultCacheBehaviorArgs{
 					TargetOriginId:       pulumi.String(originId),
 					ViewerProtocolPolicy: pulumi.String("redirect-to-https"),
-					AllowedMethods:       pulumi.ToStringArray([]string{"GET", "HEAD"}),
+					// OPTIONS is listed so CloudFront answers CORS preflights
-					CachedMethods:        pulumi.ToStringArray([]string{"GET", "HEAD"}),
+					// itself from the response headers policy below; without it
-					Compress:             pulumi.Bool(true),
+					// a preflight is rejected with 403 before the policy runs.
-					CachePolicyId:        pulumi.String(cachingOptimizedPolicyId),
+					AllowedMethods:          pulumi.ToStringArray([]string{"GET", "HEAD", "OPTIONS"}),
+					CachedMethods:           pulumi.ToStringArray([]string{"GET", "HEAD", "OPTIONS"}),
+					Compress:                pulumi.Bool(true),
+					CachePolicyId:           pulumi.String(cachingOptimizedPolicyId),
+					ResponseHeadersPolicyId: corsHeaders.ID(),
 				},
 				Restrictions: &cloudfront.DistributionRestrictionsArgs{
 					GeoRestriction: &cloudfront.DistributionRestrictionsGeoRestrictionArgs{
@@ -731,6 +773,67 @@ func main() {
 			}
 		}
 
+		if !localstack {
+			cacheSubnetGroup, err := elasticache.NewSubnetGroup(ctx, "search-cache-subnet-group", &elasticache.SubnetGroupArgs{
+				SubnetIds: pulumi.ToStringArray(subnetIDs),
+			})
+			if err != nil {
+				return err
+			}
+
+			cacheSecurityGroup, err := ec2.NewSecurityGroup(ctx, "search-cache-sg", &ec2.SecurityGroupArgs{
+				Description: pulumi.String("Allow the ECS tasks to reach the search cache"),
+				VpcId:       pulumi.String(vpcID),
+				Ingress: ec2.SecurityGroupIngressArray{
+					&ec2.SecurityGroupIngressArgs{
+						Protocol:       pulumi.String("tcp"),
+						FromPort:       pulumi.Int(6379),
+						ToPort:         pulumi.Int(6379),
+						SecurityGroups: pulumi.StringArray{serviceSecurityGroup.ID()},
+						Description:    pulumi.String("Redis, from the service tasks only"),
+					},
+				},
+				Egress: ec2.SecurityGroupEgressArray{
+					&ec2.SecurityGroupEgressArgs{
+						Protocol:   pulumi.String("-1"),
+						FromPort:   pulumi.Int(0),
+						ToPort:     pulumi.Int(0),
+						CidrBlocks: pulumi.ToStringArray([]string{"0.0.0.0/0"}),
+					},
+				},
+			})
+			if err != nil {
+				return err
+			}
+
+			searchCache, err := elasticache.NewCluster(ctx, "search-cache", &elasticache.ClusterArgs{
+				Engine:             pulumi.String("redis"),
+				EngineVersion:      pulumi.String("7.1"),
+				NodeType:           pulumi.String("cache.t4g.micro"),
+				NumCacheNodes:      pulumi.Int(1),
+				ParameterGroupName: pulumi.String("default.redis7"),
+				Port:               pulumi.Int(6379),
+				SubnetGroupName:    cacheSubnetGroup.Name,
+				SecurityGroupIds:   pulumi.StringArray{cacheSecurityGroup.ID()},
+				ApplyImmediately:   pulumi.Bool(true),
+			})
+			if err != nil {
+				return err
+			}
+
+			// A single-node Redis cluster has no configuration endpoint — that
+			// is a Memcached thing — so the address is the one cache node's.
+			searchCacheAddress = pulumi.All(searchCache.CacheNodes, searchCache.Port).ApplyT(
+				func(args []any) (string, error) {
+					nodes := args[0].([]elasticache.ClusterCacheNode)
+					if len(nodes) == 0 || nodes[0].Address == nil {
+						return "", fmt.Errorf("search cache reported no node address")
+					}
+					return fmt.Sprintf("%s:%d", *nodes[0].Address, args[1].(int)), nil
+				},
+			).(pulumi.StringOutput)
+		}
+
 		// Private bucket for raw video uploads (pre-transcode). Kept separate
 		// from encoded-bucket, which is fronted by CloudFront/OAC for public
 		// delivery of finished renditions — raw source video must not be
@@ -854,6 +957,158 @@ func main() {
 			return err
 		}
 
+		// The catalogue topic: where cms announces a video once its transcode
+		// has finished. A topic rather than a queue so the announcement fans
+		// out — the read side subscribes the queue below, and a second reader
+		// (search indexing, notifications) can subscribe its own without cms
+		// knowing it exists or this wiring changing.
+		catalogueTopicArgs := &sns.TopicArgs{}
+		catalogueQueueArgs := &sqs.QueueArgs{}
+		catalogueDeadLetterArgs := &sqs.QueueArgs{}
+		if localstack {
+			// docker-compose.yml and the tests/ suite refer to these literally.
+			catalogueTopicArgs.Name = pulumi.String("catalogue-events")
+			catalogueQueueArgs.Name = pulumi.String("discovery-catalogue-events")
+			catalogueDeadLetterArgs.Name = pulumi.String("discovery-catalogue-events-dlq")
+		}
+
+		catalogueTopic, err := sns.NewTopic(ctx, "catalogue-events", catalogueTopicArgs)
+		if err != nil {
+			return err
+		}
+
+		catalogueDeadLetter, err := sqs.NewQueue(ctx, "discovery-catalogue-events-dlq", catalogueDeadLetterArgs)
+		if err != nil {
+			return err
+		}
+
+		catalogueQueueArgs.VisibilityTimeoutSeconds = pulumi.Int(60)
+		catalogueQueueArgs.RedrivePolicy = catalogueDeadLetter.Arn.ApplyT(func(arn string) (string, error) {
+			b, err := json.Marshal(map[string]any{
+				"deadLetterTargetArn": arn,
+				"maxReceiveCount":     5,
+			})
+			return string(b), err
+		}).(pulumi.StringOutput)
+
+		// The read side's own queue on the catalogue topic. It exists before
+		// discovery has code to drain it so the announcements pile up rather
+		// than being dropped on the floor — a topic delivers only to the
+		// subscriptions that exist when it publishes.
+		catalogueQueue, err := sqs.NewQueue(ctx, "discovery-catalogue-events", catalogueQueueArgs)
+		if err != nil {
+			return err
+		}
+
+		// SNS is not an IAM principal the queue trusts by default; without this
+		// the subscription is created and silently delivers nothing.
+		catalogueQueuePolicy := pulumi.All(catalogueQueue.Arn, catalogueTopic.Arn).ApplyT(
+			func(args []any) (string, error) {
+				queueArn := args[0].(string)
+				topicArn := args[1].(string)
+
+				doc := map[string]any{
+					"Version": "2012-10-17",
+					"Statement": []map[string]any{
+						{
+							"Sid":       "AllowCatalogueTopicToSend",
+							"Effect":    "Allow",
+							"Principal": map[string]string{"Service": "sns.amazonaws.com"},
+							"Action":    "sqs:SendMessage",
+							"Resource":  queueArn,
+							"Condition": map[string]any{
+								"ArnEquals": map[string]string{"aws:SourceArn": topicArn},
+							},
+						},
+					},
+				}
+				b, err := json.Marshal(doc)
+				return string(b), err
+			},
+		).(pulumi.StringOutput)
+
+		_, err = sqs.NewQueuePolicy(ctx, "discovery-catalogue-events-queue-policy", &sqs.QueuePolicyArgs{
+			QueueUrl: catalogueQueue.ID(),
+			Policy:   catalogueQueuePolicy,
+		})
+		if err != nil {
+			return err
+		}
+
+		_, err = sns.NewTopicSubscription(ctx, "discovery-catalogue-events-subscription", &sns.TopicSubscriptionArgs{
+			Topic:    catalogueTopic.Arn,
+			Protocol: pulumi.String("sqs"),
+			Endpoint: catalogueQueue.Arn,
+			// The queue receives the announcement itself rather than an SNS
+			// envelope carrying it as a JSON string, so a consumer parses one
+			// document instead of unwrapping two.
+			RawMessageDelivery: pulumi.Bool(true),
+		})
+		if err != nil {
+			return err
+		}
+
+		// A second subscriber, for the tests/ suite only. The BDD scenarios
+		// assert on the announcement itself — the field names cms publishes,
+		// which no assertion through discovery's API can pin — and a consumer
+		// destroys what it reads. Sharing discovery's queue would mean the two
+		// racing for every message and each seeing about half, so the suite
+		// gets its own subscription. This is what the topic is for.
+		//
+		// LocalStack only: in AWS this queue would fill up unread, since
+		// nothing runs the suite there.
+		if localstack {
+			testsCatalogueQueue, err := sqs.NewQueue(ctx, "tests-catalogue-events", &sqs.QueueArgs{
+				Name: pulumi.String("tests-catalogue-events"),
+			})
+			if err != nil {
+				return err
+			}
+
+			testsCatalogueQueuePolicy := pulumi.All(testsCatalogueQueue.Arn, catalogueTopic.Arn).ApplyT(
+				func(args []any) (string, error) {
+					queueArn := args[0].(string)
+					topicArn := args[1].(string)
+
+					doc := map[string]any{
+						"Version": "2012-10-17",
+						"Statement": []map[string]any{
+							{
+								"Sid":       "AllowCatalogueTopicToSend",
+								"Effect":    "Allow",
+								"Principal": map[string]string{"Service": "sns.amazonaws.com"},
+								"Action":    "sqs:SendMessage",
+								"Resource":  queueArn,
+								"Condition": map[string]any{
+									"ArnEquals": map[string]string{"aws:SourceArn": topicArn},
+								},
+							},
+						},
+					}
+					b, err := json.Marshal(doc)
+					return string(b), err
+				},
+			).(pulumi.StringOutput)
+
+			_, err = sqs.NewQueuePolicy(ctx, "tests-catalogue-events-queue-policy", &sqs.QueuePolicyArgs{
+				QueueUrl: testsCatalogueQueue.ID(),
+				Policy:   testsCatalogueQueuePolicy,
+			})
+			if err != nil {
+				return err
+			}
+
+			_, err = sns.NewTopicSubscription(ctx, "tests-catalogue-events-subscription", &sns.TopicSubscriptionArgs{
+				Topic:              catalogueTopic.Arn,
+				Protocol:           pulumi.String("sqs"),
+				Endpoint:           testsCatalogueQueue.Arn,
+				RawMessageDelivery: pulumi.Bool(true),
+			})
+			if err != nil {
+				return err
+			}
+		}
+
 		// SNS is not an IAM principal the queue trusts by default; without this
 		// the subscription is created and silently delivers nothing.
 		jobEventsQueuePolicy := pulumi.All(jobEventsQueue.Arn, jobEventsTopic.Arn).ApplyT(
@@ -969,11 +1224,12 @@ func main() {
 			return err
 		}
 
-		cmsTaskPolicy := pulumi.All(rawUploadsBucket.Arn, mediaConvertRole.Arn, jobEventsQueue.Arn).ApplyT(
+		cmsTaskPolicy := pulumi.All(rawUploadsBucket.Arn, mediaConvertRole.Arn, jobEventsQueue.Arn, catalogueTopic.Arn).ApplyT(
 			func(args []any) (string, error) {
 				rawUploadsArn := args[0].(string)
 				mediaConvertRoleArn := args[1].(string)
 				jobEventsQueueArn := args[2].(string)
+				catalogueTopicArn := args[3].(string)
 
 				doc := map[string]any{
 					"Version": "2012-10-17",
@@ -1009,6 +1265,15 @@ func main() {
 							},
 							"Resource": jobEventsQueueArn,
 						},
+						{
+							"Sid":    "AnnounceReadyVideos",
+							"Effect": "Allow",
+							"Action": []string{
+								"sns:Publish",
+								"sns:GetTopicAttributes",
+							},
+							"Resource": catalogueTopicArn,
+						},
 						{
 							"Sid":      "PassMediaConvertRole",
 							"Effect":   "Allow",
@@ -1035,6 +1300,45 @@ func main() {
 			return err
 		}
 
+		// discovery's own ECS task role. It used to have none — it touched no
+		// AWS API at all — and it still gets nothing but the one queue it
+		// drains: no S3, no MediaConvert, and no ability to publish back onto
+		// the catalogue topic. It is a subscriber, not a participant.
+		discoveryTaskRole, err := iam.NewRole(ctx, "discovery-task-role", &iam.RoleArgs{
+			AssumeRolePolicy: pulumi.String(execRoleAssumePolicy),
+		})
+		if err != nil {
+			return err
+		}
+
+		discoveryTaskPolicy := catalogueQueue.Arn.ApplyT(func(catalogueQueueArn string) (string, error) {
+			doc := map[string]any{
+				"Version": "2012-10-17",
+				"Statement": []map[string]any{
+					{
+						"Sid":    "ConsumeCatalogueAnnouncements",
+						"Effect": "Allow",
+						"Action": []string{
+							"sqs:ReceiveMessage",
+							"sqs:DeleteMessage",
+							"sqs:GetQueueAttributes",
+						},
+						"Resource": catalogueQueueArn,
+					},
+				},
+			}
+			b, err := json.Marshal(doc)
+			return string(b), err
+		}).(pulumi.StringOutput)
+
+		_, err = iam.NewRolePolicy(ctx, "discovery-task-role-policy", &iam.RolePolicyArgs{
+			Role:   discoveryTaskRole.ID(),
+			Policy: discoveryTaskPolicy,
+		})
+		if err != nil {
+			return err
+		}
+
 		// The two ECS services and the CI user that deploys them. Under
 		// LocalStack the equivalent of all this is the cms service in
 		// docker-compose.yml, which reads the same env vars from .env.
@@ -1046,6 +1350,7 @@ func main() {
 				{Name: "MEDIACONVERT_OUTPUT_BUCKET", Value: bucket.ID().ToStringOutput()},
 				{Name: "MEDIACONVERT_ROLE_ARN", Value: mediaConvertRole.Arn},
 				{Name: "MEDIACONVERT_EVENTS_QUEUE_URL", Value: jobEventsQueue.Url},
+				{Name: "CATALOGUE_EVENTS_TOPIC_ARN", Value: catalogueTopic.Arn},
 				// The CDN in front of the encoded bucket, which is the only way
 				// that bucket is readable — it blocks public access and grants
 				// only CloudFront's OAC principal. cms rewrites the s3:// paths
@@ -1072,6 +1377,27 @@ func main() {
 			corsOrigins = pulumi.StringArray{pulumi.Sprintf("http://%s", cmsAlb.DnsName)}
 		}
 
+		// The encoded bucket needs its own CORS rule for the LocalStack stack,
+		// where there is no CloudFront in front of it and PLAYBACK_BASE_URL
+		// points the player straight at S3. Behind CloudFront the response
+		// headers policy already covers playback, but the rule is harmless
+		// there and keeps a direct-to-bucket player working either way.
+		_, err = s3.NewBucketCorsConfigurationV2(ctx, "encoded-bucket-cors", &s3.BucketCorsConfigurationV2Args{
+			Bucket: bucket.ID(),
+			CorsRules: s3.BucketCorsConfigurationV2CorsRuleArray{
+				&s3.BucketCorsConfigurationV2CorsRuleArgs{
+					AllowedMethods: pulumi.ToStringArray([]string{"GET", "HEAD"}),
+					AllowedOrigins: pulumi.ToStringArray([]string{"*"}),
+					AllowedHeaders: pulumi.ToStringArray([]string{"*"}),
+					ExposeHeaders:  pulumi.ToStringArray([]string{"Content-Length", "Content-Range", "Date", "ETag"}),
+					MaxAgeSeconds:  pulumi.Int(3000),
+				},
+			},
+		})
+		if err != nil {
+			return err
+		}
+
 		_, err = s3.NewBucketCorsConfigurationV2(ctx, "raw-uploads-bucket-cors", &s3.BucketCorsConfigurationV2Args{
 			Bucket: rawUploadsBucket.ID(),
 			CorsRules: s3.BucketCorsConfigurationV2CorsRuleArray{
@@ -1089,9 +1415,22 @@ func main() {
 		}
 
 		if !localstack {
+			// runMigrations is true for the same reason it is for cms:
+			// discovery embeds its own golang-migrate migrations and applies
+			// them through `./discovery migrate`, so ECS runs that container to
+			// completion before the service accepts traffic.
+			discoveryExtraEnv := []envVar{
+				{Name: "AWS_REGION", Value: pulumi.String(awsRegion).ToStringOutput()},
+				{Name: "CATALOGUE_EVENTS_QUEUE_URL", Value: catalogueQueue.Url},
+				// The search cache. Reached over the Redis protocol on the
+				// private network, so unlike the queue above it needs no
+				// matching grant on discovery-task-role.
+				{Name: "REDIS_ADDR", Value: searchCacheAddress},
+			}
+
 			discoveryRepo, discoveryService, discoveryAlb, err = deployFargateService(ctx, "discovery", 8080,
-				cluster, execRole, nil, nil, vpcID, subnetIDs, albSecurityGroup, serviceSecurityGroup,
+				cluster, execRole, discoveryTaskRole, discoveryExtraEnv, vpcID, subnetIDs, albSecurityGroup, serviceSecurityGroup,
-				db.Address, db.Port, discoveryPassword, false)
+				db.Address, db.Port, discoveryPassword, true)
 			if err != nil {
 				return err
 			}
@@ -1180,6 +1519,7 @@ func main() {
 			ctx.Export("ecsClusterArn", cluster.Arn)
 			ctx.Export("cmsServiceArn", cmsService.Arn)
 			ctx.Export("discoveryServiceArn", discoveryService.Arn)
+			ctx.Export("searchCacheAddress", searchCacheAddress)
 		}
 		return nil
 	})

tests/README.md

@@ -12,7 +12,7 @@ The upload to storage is a plain `PUT` to the presigned URL the API hands out,
 with no AWS SDK involved, because that is exactly what a real client does.
 
 ```
-features/video_upload.feature   the scenarios, in Gherkin
+features/*.feature              the scenarios, in Gherkin
 suite_test.go                   the gobdd suite: step registration + per-scenario world
 steps_test.go                   what each step does
 client_test.go                  HTTP plumbing and the wire types
@@ -68,6 +68,39 @@ Both steps of the upload flow, end to end:
   the category lookup itself failing produces
 - answering 409 for a key that was already registered, rather than queueing a
   second transcoding job for the same file
+- announcing a video on the catalogue topic once its job comes back COMPLETE,
+  carrying its id, title, playback URL and the **names** of its categories —
+  and announcing nothing for a job that ended in an error
+
+- the read side building its catalogue from those announcements: the video
+  turns up in discovery under the id cms issued, a video still transcoding is
+  not there at all, and a video announced a second time is updated rather than
+  colliding with its own row
+
+- searching that catalogue over `GET /api/videos`: finding a video by a word
+  in its title and by a part-typed one, narrowing to a category and being left
+  out of another, several categories meaning "any of", paging by cursor so the
+  pages tile the results exactly once, refusing a cursor no search issued, and
+  capping a page size that asks for the whole catalogue
+
+Search scenarios give every video they catalogue a nonce in its title, and
+assert relative to the video they created ("returns that video") rather than on
+absolute counts. Nothing cleans up between runs, so a fixed title accumulates a
+copy per run and a count would stop meaning anything. The paging scenario shares
+one nonce across its videos so that searching for it matches that run's cohort
+and nothing else.
+
+The publication scenarios read `tests-catalogue-events`, the suite's **own**
+queue on the catalogue topic — not discovery's. A consumer destroys what it
+reads, so sharing discovery's queue would have the suite and the running
+service race for every announcement and each see about half. Delivery is raw,
+so a message body is the announcement with no SNS envelope.
+
+The ingestion scenarios need `discovery` up as well as `cms`; they read it over
+HTTP at `DISCOVERY_BASE_URL` (default `http://localhost:8080`).
+Like the job-state publisher they stand in for AWS, so they are the second
+place the suite reaches for the AWS SDK; everything a real client does still
+goes over plain HTTP.
 
 ## Adding a scenario
 

tests/client_test.go

@@ -6,7 +6,9 @@ import (
 	"fmt"
 	"io"
 	"net/http"
+	"net/url"
 	"os"
+	"strconv"
 	"strings"
 	"time"
 )
@@ -23,6 +25,19 @@ func baseURL() string {
 	return defaultBaseURL
 }
 
+// defaultDiscoveryBaseURL is where docker-compose publishes discovery on the
+// host.
+const defaultDiscoveryBaseURL = "http://localhost:8080"
+
+// discoveryBaseURL is the Discovery service the scenarios read the catalogue
+// from. Override it the same way as CMS_BASE_URL.
+func discoveryBaseURL() string {
+	if v := strings.TrimSpace(os.Getenv("DISCOVERY_BASE_URL")); v != "" {
+		return strings.TrimRight(v, "/")
+	}
+	return defaultDiscoveryBaseURL
+}
+
 // response is one HTTP exchange, kept whole so a failing assertion can print
 // the body it actually got rather than just a status code.
 type response struct {
@@ -107,6 +122,95 @@ func newClient() *client {
 	}
 }
 
+// newDiscoveryClient is newClient pointed at the read side. Same plumbing —
+// only the host differs, since both services speak the same JSON dialect.
+func newDiscoveryClient() *client {
+	return &client{
+		base: discoveryBaseURL(),
+		http: &http.Client{Timeout: 30 * time.Second},
+	}
+}
+
+// catalogueVideoBody is GET /api/videos/{id} on discovery: the catalogue's own
+// copy of a video, which is a different published shape from the CMS record of
+// the same name.
+type catalogueVideoBody struct {
+	ID          string   `json:"id"`
+	Title       string   `json:"title"`
+	PlaybackURL string   `json:"playbackUrl"`
+	Categories  []string `json:"categories"`
+}
+
+// searchRequest is a search of GET /api/videos on discovery: what a reader is
+// looking for, plus where in the results to carry on from.
+//
+// Every field is optional. A search with none of them set is the whole
+// catalogue, newest first.
+type searchRequest struct {
+	Title      string
+	Categories []string
+	Limit      int
+	Cursor     string
+}
+
+// query renders the search as the query string discovery reads it from.
+// Categories are repeated rather than joined, since that is how the endpoint
+// takes several of them, and an unset field is left out entirely so that the
+// service applies its own default.
+func (r searchRequest) query() url.Values {
+	values := url.Values{}
+	if r.Title != "" {
+		values.Set("title", r.Title)
+	}
+	for _, category := range r.Categories {
+		values.Add("categories", category)
+	}
+	if r.Limit != 0 {
+		values.Set("limit", strconv.Itoa(r.Limit))
+	}
+	if r.Cursor != "" {
+		values.Set("cursor", r.Cursor)
+	}
+	return values
+}
+
+// searchResultsBody is what that query answers with: the page of videos, and
+// the cursor that reaches the page after it. NextCursor is empty on the last
+// page, which is how a client knows to stop.
+type searchResultsBody struct {
+	Videos     []catalogueVideoBody `json:"videos"`
+	NextCursor string               `json:"nextCursor"`
+}
+
+// ids lists the video ids in the page, for asserting on which videos came back
+// without caring about the rest of their fields.
+func (b searchResultsBody) ids() []string {
+	found := make([]string, 0, len(b.Videos))
+	for _, v := range b.Videos {
+		found = append(found, v.ID)
+	}
+	return found
+}
+
+func (b searchResultsBody) holds(id string) bool {
+	for _, v := range b.Videos {
+		if v.ID == id {
+			return true
+		}
+	}
+	return false
+}
+
+// search sends the catalogue search: a plain GET with the search spelled out
+// in the query string.
+func (c *client) search(path string, request searchRequest) (response, error) {
+	query := request.query().Encode()
+	if query != "" {
+		path += "?" + query
+	}
+	return c.get(path)
+}
+
 // postJSON marshals payload and posts it to a path on the CMS.
 func (c *client) postJSON(path string, payload any) (response, error) {
 	encoded, err := json.Marshal(payload)

tests/events_test.go

@@ -13,6 +13,7 @@ import (
 	awsconfig "github.com/aws/aws-sdk-go-v2/config"
 	"github.com/aws/aws-sdk-go-v2/credentials"
 	"github.com/aws/aws-sdk-go-v2/service/sns"
+	"github.com/aws/aws-sdk-go-v2/service/sqs"
 )
 
 // These scenarios stand in for EventBridge, not for a client: they publish the
@@ -30,6 +31,15 @@ const (
 	defaultRegion          = "us-east-1"
 	defaultOutputBucket    = "encoded-bucket"
 	defaultPlaybackBaseURL = "http://localhost.localstack.cloud:4566/encoded-bucket"
+	// The suite's own queue on the catalogue topic cms announces ready videos
+	// to. The suite reads a queue rather than the topic because that is what a
+	// subscriber sees — SNS has nothing to poll.
+	//
+	// Deliberately not discovery's queue: a consumer destroys what it reads, so
+	// sharing one would have the suite and the running service race for every
+	// announcement and each see about half. Pinned by the localstack branch in
+	// infrastructure/main.go; LocalStack always uses account 000000000000.
+	defaultCatalogueQueueURL = "http://localhost.localstack.cloud:4566/000000000000/tests-catalogue-events"
 )
 
 // playbackBaseURL is the public host the encoded output is served from — the
@@ -97,21 +107,27 @@ type eventPublisher struct {
 	topicARN string
 }
 
-func newEventPublisher(ctx context.Context) (*eventPublisher, error) {
+// awsConfig is the SDK configuration both ends of the event seam use: the
-	endpoint := envOr("AWS_ENDPOINT_URL", defaultEndpointURL)
+// publisher that stands in for EventBridge, and the reader that stands in for
-	region := envOr("AWS_REGION", defaultRegion)
+// whoever consumes the catalogue queue.
-
+func awsConfig(ctx context.Context) (aws.Config, error) {
-	cfg, err := awsconfig.LoadDefaultConfig(ctx,
+	return awsconfig.LoadDefaultConfig(ctx,
-		awsconfig.WithRegion(region),
+		awsconfig.WithRegion(envOr("AWS_REGION", defaultRegion)),
 		// LocalStack accepts any credentials; these keep the SDK from hunting
 		// for a profile that a developer's machine may not have.
 		awsconfig.WithCredentialsProvider(credentials.NewStaticCredentialsProvider(
 			envOr("AWS_ACCESS_KEY_ID", "test"), envOr("AWS_SECRET_ACCESS_KEY", "test"), "")),
 	)
+}
+
+func newEventPublisher(ctx context.Context) (*eventPublisher, error) {
+	cfg, err := awsConfig(ctx)
 	if err != nil {
 		return nil, err
 	}
 
+	endpoint := envOr("AWS_ENDPOINT_URL", defaultEndpointURL)
+
 	return &eventPublisher{
 		sns:      sns.NewFromConfig(cfg, func(o *sns.Options) { o.BaseEndpoint = aws.String(endpoint) }),
 		topicARN: envOr("MEDIACONVERT_EVENTS_TOPIC_ARN", defaultTopicARN),
@@ -160,3 +176,84 @@ func (p *eventPublisher) publishJobState(ctx context.Context, jobID, state, play
 	}
 	return nil
 }
+
+// --- the catalogue queue ----------------------------------------------------
+
+// catalogueAnnouncement is what cms puts on the catalogue queue when a video
+// becomes ready. This is the published contract, so the suite spells it out
+// rather than importing it: a change here is a change other services see.
+type catalogueAnnouncement struct {
+	VideoID     string   `json:"videoId"`
+	Title       string   `json:"title"`
+	PlaybackURL string   `json:"playbackUrl"`
+	Categories  []string `json:"categories"`
+}
+
+// catalogueReader stands in for the read side, draining its own queue on the
+// catalogue topic. It consumes what it reads, exactly as a real subscriber
+// would. The subscription delivers raw, so a message body is the announcement
+// itself with no SNS envelope to unwrap.
+type catalogueReader struct {
+	sqs      *sqs.Client
+	queueURL string
+}
+
+func newCatalogueReader(ctx context.Context) (*catalogueReader, error) {
+	cfg, err := awsConfig(ctx)
+	if err != nil {
+		return nil, err
+	}
+
+	endpoint := envOr("AWS_ENDPOINT_URL", defaultEndpointURL)
+
+	return &catalogueReader{
+		sqs:      sqs.NewFromConfig(cfg, func(o *sqs.Options) { o.BaseEndpoint = aws.String(endpoint) }),
+		queueURL: envOr("CATALOGUE_QUEUE_URL", defaultCatalogueQueueURL),
+	}, nil
+}
+
+// await waits for the announcement naming videoID and returns it. Messages for
+// other videos are consumed and discarded on the way: scenarios run one at a
+// time, so anything else on the queue is a leftover, and leaving it would only
+// make the next scenario wade through it. It returns nil if nothing names that
+// video before the deadline.
+func (r *catalogueReader) await(ctx context.Context, videoID string, within time.Duration) (*catalogueAnnouncement, error) {
+	deadline := time.Now().Add(within)
+
+	for time.Now().Before(deadline) {
+		output, err := r.sqs.ReceiveMessage(ctx, &sqs.ReceiveMessageInput{
+			QueueUrl:            aws.String(r.queueURL),
+			MaxNumberOfMessages: 10,
+			WaitTimeSeconds:     2,
+		})
+		if err != nil {
+			return nil, fmt.Errorf("receiving from %s: %w", r.queueURL, err)
+		}
+
+		var found *catalogueAnnouncement
+		for _, message := range output.Messages {
+			if message.Body == nil || message.ReceiptHandle == nil {
+				continue
+			}
+
+			var announcement catalogueAnnouncement
+			if err := json.Unmarshal([]byte(*message.Body), &announcement); err == nil &&
+				announcement.VideoID == videoID {
+				found = &announcement
+			}
+
+			if _, err := r.sqs.DeleteMessage(ctx, &sqs.DeleteMessageInput{
+				QueueUrl:      aws.String(r.queueURL),
+				ReceiptHandle: message.ReceiptHandle,
+			}); err != nil {
+				return nil, fmt.Errorf("acknowledging a message on %s: %w", r.queueURL, err)
+			}
+		}
+
+		if found != nil {
+			return found, nil
+		}
+	}
+
+	return nil, nil
+}

tests/features/catalogue_ingestion.feature

@@ -0,0 +1,40 @@
+Feature: Building the catalogue from announcements
+  Discovery is the read side. It owns no ingestion of its own: it learns what
+  exists by subscribing to the catalogue topic cms announces ready videos on,
+  and keeps its own copy in its own database. Nothing polls cms and neither
+  service reads the other's tables.
+
+  A video therefore reaches the catalogue only once its transcode has finished
+  — which is the point, since a catalogue entry nobody can play is no use.
+
+  Background:
+    Given the CMS API is available
+    And the Discovery API is available
+
+  Scenario: A video the CMS announces appears in the catalogue
+    Given I have registered a video that is being transcoded
+    When MediaConvert reports that the job reached "COMPLETE"
+    Then the catalogue eventually holds that video
+    And the catalogue's copy carries its title, playback URL and categories
+
+  # The catalogue is built from announcements, and cms announces only what came
+  # out ready. A video part-way through its transcode is therefore genuinely
+  # absent rather than merely late, and saying so is the honest answer.
+  Scenario: A video that is still being transcoded is not in the catalogue
+    Given I have registered a video that is being transcoded
+    Then the catalogue does not hold that video
+    And the request is rejected with status 404
+    And the problem title is "Video Not In The Catalogue"
+
+  # The catalogue topic delivers at-least-once, and a job's outcome can be
+  # reported more than once, so the same video can be announced again. The
+  # later announcement is the truth: it lands on top of what is already there
+  # rather than colliding with it. Announcing it again with no playlist is what
+  # makes that visible — a second announcement carrying identical content is
+  # indistinguishable from never having arrived.
+  Scenario: A video announced again is updated rather than colliding
+    Given I have registered a video that is being transcoded
+    And MediaConvert reports that the job reached "COMPLETE"
+    And the catalogue eventually holds that video
+    When MediaConvert reports that the job reached "COMPLETE" without a playlist
+    Then the catalogue eventually holds that video with no playback URL

tests/features/catalogue_publication.feature

@@ -0,0 +1,43 @@
+Feature: Announcing ready videos to the catalogue
+  A video is only worth showing once its transcode has finished. When
+  MediaConvert reports a job complete, cms announces that video on a queue of
+  its own — the seam the read side reads to build the public catalogue, so
+  nothing downstream has to poll cms or reach into its database.
+
+  The announcement is the published contract: the video's id, its title, the
+  URL a player can open, and the categories it is filed under, named rather
+  than numbered so a reader needs nothing from cms to make sense of it.
+
+  Background:
+    Given the CMS API is available
+
+  Scenario: A finished job announces the video to the catalogue
+    Given I have registered a video that is being transcoded
+    When MediaConvert reports that the job reached "COMPLETE"
+    Then the video eventually has status "ready"
+    And the catalogue is told about the video
+    And the announcement carries the video's title, playback URL and categories
+
+  # The catalogue is a list of things worth showing. A job that ended in an
+  # error produced nothing to play, so it is recorded against the video and
+  # goes no further.
+  Scenario Outline: A job that did not finish is not announced
+    Given I have registered a video that is being transcoded
+    When MediaConvert reports that the job reached "<state>"
+    Then the video eventually has status "failed"
+    And the catalogue is told nothing about the video
+
+    Examples:
+      | state    |
+      | ERROR    |
+      | CANCELED |
+
+  # A video can be filed under several categories, so the announcement carries
+  # all of them. Names, not the ids the register call took: what cms numbers
+  # them is its own business.
+  Scenario: The announcement lists every category the video is filed under
+    Given I have registered a video being transcoded under categories "documentary, news, podcast"
+    When MediaConvert reports that the job reached "COMPLETE"
+    Then the video eventually has status "ready"
+    And the catalogue is told about the video
+    And the announcement files it under "documentary, news, podcast"

tests/features/catalogue_search.feature

@@ -0,0 +1,78 @@
+Feature: Searching the catalogue
+  Discovery holds a copy of every video cms has announced as ready. Reaching
+  one by id is no use to a reader who does not have an id — finding something
+  to watch means searching for it by what it is called and what it is filed
+  under.
+
+  The search is lexical: it matches the words of a title, not its meaning. The
+  last word a reader types is treated as a prefix, because someone typing into
+  a search box is usually part-way through a word.
+
+  Background:
+    Given the CMS API is available
+    And the Discovery API is available
+
+  Scenario: Finding a catalogued video by a word in its title
+    Given the catalogue holds a video titled "Desert Falcons"
+    When I search the catalogue for that video's title
+    Then the search returns that video
+
+  # The last word is matched as a prefix, so the results keep up with someone
+  # still typing. Earlier words are matched whole — they have been finished.
+  Scenario: Finding a video from a part-typed word
+    Given the catalogue holds a video titled "Desert Falcons"
+    When I search the catalogue for "Desert Fal"
+    Then the search returns that video
+
+  # Categories narrow a search rather than widening it: a video filed elsewhere
+  # is not an answer to a reader who asked for one category in particular.
+  Scenario: A search narrowed to another category leaves the video out
+    Given the catalogue holds a video titled "Desert Falcons" filed under "documentary"
+    When I search the catalogue for that video's title in category "news"
+    Then the search does not return that video
+
+  Scenario: A search narrowed to the video's own category still finds it
+    Given the catalogue holds a video titled "Desert Falcons" filed under "documentary"
+    When I search the catalogue for that video's title in category "documentary"
+    Then the search returns that video
+
+  # Several categories mean "any of these", not "all of these": naming more of
+  # them offers the reader more, rather than demanding the video be filed under
+  # every one at once.
+  Scenario: Naming several categories matches a video filed under any of them
+    Given the catalogue holds a video titled "Desert Falcons" filed under "documentary"
+    When I search the catalogue for that video's title in categories "news, documentary"
+    Then the search returns that video
+
+  # Paging is by cursor rather than by offset: the catalogue is added to while
+  # people are reading it, and an offset silently repeats or skips a video when
+  # something is announced between one page and the next. A cursor names where
+  # the last page stopped, so the page after it is the same page whenever it is
+  # asked for.
+  Scenario: Paging through the results a page at a time
+    Given the catalogue holds 3 videos titled "Mountain Wolves"
+    When I search the catalogue for that title 2 at a time
+    Then the search returns 2 videos
+    And the search offers a cursor to the next page
+    When I follow the cursor
+    Then the search returns 1 video
+    And the search offers no further cursor
+    And the pages together hold every one of those videos exactly once
+
+  # A cursor is opaque, so a client that makes one up has made a mistake rather
+  # than found a fault. Saying 400 is what tells the two apart.
+  Scenario: Refusing a cursor no search issued
+    Given the catalogue holds a video titled "Desert Falcons"
+    When I search the catalogue for that video's title from the cursor "not-a-cursor"
+    Then the request is rejected with status 400
+    And the problem title is "Malformed Search Cursor"
+
+  # A page size is a request for work, and an uncapped one is a request for all
+  # of it — the catalogue is read constantly, and one client asking for every
+  # row at once should not be able to make everyone else wait. The cap applies
+  # silently rather than as a rejection: the reader still gets a page, and the
+  # cursor still reaches the rest.
+  Scenario: Capping a page size that asks for the whole catalogue
+    When I search the whole catalogue 5000 at a time
+    Then the search returns at most 100 videos
+    And the search offers a cursor to the next page

tests/go.mod

@@ -2,20 +2,23 @@ module thamanyah/tests
 
 go 1.25.12
 
-require github.com/go-bdd/gobdd v1.1.4
+require (
+	github.com/aws/aws-sdk-go-v2 v1.45.1
+	github.com/aws/aws-sdk-go-v2/config v1.32.40
+	github.com/aws/aws-sdk-go-v2/credentials v1.19.39
+	github.com/aws/aws-sdk-go-v2/service/sns v1.43.0
+	github.com/aws/aws-sdk-go-v2/service/sqs v1.48.1
+	github.com/go-bdd/gobdd v1.1.4
+)
 
 require (
-	github.com/aws/aws-sdk-go-v2 v1.44.0 // indirect
-	github.com/aws/aws-sdk-go-v2/config v1.32.40 // indirect
-	github.com/aws/aws-sdk-go-v2/credentials v1.19.39 // indirect
 	github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.18.40 // indirect
-	github.com/aws/aws-sdk-go-v2/internal/configsources v1.4.40 // indirect
+	github.com/aws/aws-sdk-go-v2/internal/configsources v1.5.1 // indirect
-	github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.7.40 // indirect
+	github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.8.1 // indirect
 	github.com/aws/aws-sdk-go-v2/internal/v4a v1.4.41 // indirect
 	github.com/aws/aws-sdk-go-v2/service/internal/accept-encoding v1.13.19 // indirect
 	github.com/aws/aws-sdk-go-v2/service/internal/presigned-url v1.13.40 // indirect
 	github.com/aws/aws-sdk-go-v2/service/signin v1.6.0 // indirect
-	github.com/aws/aws-sdk-go-v2/service/sns v1.43.0 // indirect
 	github.com/aws/aws-sdk-go-v2/service/sso v1.34.0 // indirect
 	github.com/aws/aws-sdk-go-v2/service/ssooidc v1.39.0 // indirect
 	github.com/aws/aws-sdk-go-v2/service/sts v1.46.0 // indirect

tests/go.sum

@@ -1,15 +1,15 @@
-github.com/aws/aws-sdk-go-v2 v1.44.0 h1:4IbaHhtzy+4h37z4JQyO9a2QsiCml3CNYHtq5hIHigo=
+github.com/aws/aws-sdk-go-v2 v1.45.1 h1:iIoG3NaLhV6UZpPXyPXlDj2I9oS8tV/nMcMnITCC6Ks=
-github.com/aws/aws-sdk-go-v2 v1.44.0/go.mod h1:bttEH6JqnUL8LepvDVfdrds/fZ5bCIxzpe3abyUrhDU=
+github.com/aws/aws-sdk-go-v2 v1.45.1/go.mod h1:bttEH6JqnUL8LepvDVfdrds/fZ5bCIxzpe3abyUrhDU=
 github.com/aws/aws-sdk-go-v2/config v1.32.40 h1:lAVC9gMmKusmqDRe32dPtgKl/BWvJmMJoWELKHCAObw=
 github.com/aws/aws-sdk-go-v2/config v1.32.40/go.mod h1:8xOJLbe/hOj1g4PVsfJYV7O2byq+UGET1onDdUgbwqc=
 github.com/aws/aws-sdk-go-v2/credentials v1.19.39 h1:XOg8LC3Kgnsa3WiPQjc7Bi8k5IBN92cPYfIV9XMFss0=
 github.com/aws/aws-sdk-go-v2/credentials v1.19.39/go.mod h1:GonTDBQ+mTpCVNwaHjj0PagspfrYYMEqOx7FehoEP/I=
 github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.18.40 h1:r5aGipEVgI9aT/tAGjdrPbDQvIAKdTrS3rUPQtG4Rmo=
 github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.18.40/go.mod h1:vOD3CnPxAdkL6MWZeROkZsTlskklMFfgVFkHzx/oZpY=
-github.com/aws/aws-sdk-go-v2/internal/configsources v1.4.40 h1:UIXlbijuB2XK1Kr57fo8iIxCuaSHJzwZ1uo+2tbEYIk=
+github.com/aws/aws-sdk-go-v2/internal/configsources v1.5.1 h1:pc138gM1CW+XPc60rEwUlwwuwWFQK16CI1T7v1F9Oec=
-github.com/aws/aws-sdk-go-v2/internal/configsources v1.4.40/go.mod h1:wcEsL6jscjZjVUinb0Q5qD/GXOG1yT3GNfmT9HuDwzU=
+github.com/aws/aws-sdk-go-v2/internal/configsources v1.5.1/go.mod h1:1+koxpPIbfBdfzP6vojm5/zTpTQ/micYwlxIiNB3TxI=
-github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.7.40 h1:xLQVRDs2NddDmK9BEyh5KSlJ1Gpy5/GIJXrV6WcVGAE=
+github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.8.1 h1:K0JsbZQj+1h208Ro1zHeA4l7bMp0NvRffHQ91q8Ol1s=
-github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.7.40/go.mod h1:XRXnpFVFGLaEVK+olDdFIM1vNa04ETW452oFGEPUxAo=
+github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.8.1/go.mod h1:W3/vL6EtCIatICGy9ab29QhMuae+cOKPWcMxv02CO+Q=
 github.com/aws/aws-sdk-go-v2/internal/v4a v1.4.41 h1:nv/ILuCY0yXACzMQwvtt/HbqDDjemZiI0AeDbxGQlnU=
 github.com/aws/aws-sdk-go-v2/internal/v4a v1.4.41/go.mod h1:dzvOSpxaPqQ3j0xS6Lc1vyVuWW0RBj7s/QqYpzu3Q/0=
 github.com/aws/aws-sdk-go-v2/service/internal/accept-encoding v1.13.19 h1:bAdDl/HkGCcGPoe25ToSHEw23VIxt6CT5fLcg111BKg=
@@ -20,6 +20,8 @@ github.com/aws/aws-sdk-go-v2/service/signin v1.6.0 h1:agcr0j8YeFEzdXNo17Rg9MbbjL
 github.com/aws/aws-sdk-go-v2/service/signin v1.6.0/go.mod h1:qU5PxgQ4JiUOOMotzfO3+5oUda5W+8JDVKyLQqlrJik=
 github.com/aws/aws-sdk-go-v2/service/sns v1.43.0 h1:VPYjwn0BoX34hb44OT8T+Ikgn4NzsN7fHetaHaevsDc=
 github.com/aws/aws-sdk-go-v2/service/sns v1.43.0/go.mod h1:I1vnLPvvi9KBqxddu8nJ4vktoPJvaIG05UmjBD9sqm8=
+github.com/aws/aws-sdk-go-v2/service/sqs v1.48.1 h1:jXP3BdVenFa8RfLVH+D2gswrWZHJcgtygKCf22APFqo=
+github.com/aws/aws-sdk-go-v2/service/sqs v1.48.1/go.mod h1:d4DToDhLnEofHKvFu4yCF0Be65pZW267COfKOztsZOQ=
 github.com/aws/aws-sdk-go-v2/service/sso v1.34.0 h1:FxaN8/sn61DTXNI6Gt678tFJUY8iUsCchm6Y/F/RjaA=
 github.com/aws/aws-sdk-go-v2/service/sso v1.34.0/go.mod h1:vu4OY6s8LJtT8BtYG2LD6BGSZMptkYn3o5hvCPB22jc=
 github.com/aws/aws-sdk-go-v2/service/ssooidc v1.39.0 h1:crWKPeGYTBTuBxQ3p73kjfJvt4brUIsr+Fuypko8FxY=

tests/steps_test.go

@@ -3,8 +3,10 @@ package tests
 import (
 	"bytes"
 	"context"
+	"fmt"
 	"net/url"
 	"slices"
+	"strconv"
 	"strings"
 	"time"
 
@@ -422,6 +424,25 @@ func registerVideoBeingTranscoded(t gobdd.StepTest, ctx gobdd.Context) {
 	}
 }
 
+// registerVideoBeingTranscodedUnder is registerVideoBeingTranscoded for a
+// scenario that cares which categories the video is filed under.
+func registerVideoBeingTranscodedUnder(t gobdd.StepTest, ctx gobdd.Context, categories string) {
+	w := worldOf(t, ctx)
+
+	requestUploadSlot(t, ctx, "transcoding.mp4", "video/mp4")
+	uploadTheFile(t, ctx)
+	registerUploadedVideo(t, ctx, "A Video In Several Categories", categories)
+
+	if w.last.status != 201 {
+		t.Fatalf("could not register a video to transcode: %s", w.last.summary())
+		return
+	}
+	if strings.TrimSpace(w.video.MediaConvertJobID) == "" {
+		t.Fatalf("the registered video carries no transcoding job id, so no event could name it")
+		return
+	}
+}
+
 func mediaConvertReportsState(t gobdd.StepTest, ctx gobdd.Context, state string) {
 	w := worldOf(t, ctx)
 
@@ -581,3 +602,618 @@ func askForThatVideoWrittenAs(t gobdd.StepTest, ctx gobdd.Context, form string)
 		}
 	}
 }
+
+// --- the catalogue queue ----------------------------------------------------
+
+// announcementTimeout bounds how long a scenario waits for the announcement.
+// It is the status timeout's sibling: the publish happens in the same consumer
+// pass as the status write, so a video that is ready and still unannounced
+// after this is not slow, it is unannounced.
+const announcementTimeout = 20 * time.Second
+
+func theCatalogueIsToldAboutTheVideo(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if strings.TrimSpace(w.video.ID) == "" {
+		t.Fatalf("no video has been registered, so nothing could be announced")
+		return
+	}
+
+	background := context.Background()
+
+	reader, err := newCatalogueReader(background)
+	if err != nil {
+		t.Fatalf("could not reach the catalogue queue: %s", err)
+		return
+	}
+
+	announcement, err := reader.await(background, w.video.ID, announcementTimeout)
+	if err != nil {
+		t.Fatalf("could not read the catalogue queue: %s", err)
+		return
+	}
+	if announcement == nil {
+		t.Errorf("the catalogue was never told about video %q within %s", w.video.ID, announcementTimeout)
+		return
+	}
+
+	w.announcement = announcement
+}
+
+// announcementHoldWindow is how long "told nothing" watches for before it is
+// satisfied. The announcement rides in the same consumer pass as the status
+// write, and the status is already settled by the time this step runs, so a
+// message that is not here by now is not coming.
+const announcementHoldWindow = 5 * time.Second
+
+func theCatalogueIsToldNothingAboutTheVideo(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if strings.TrimSpace(w.video.ID) == "" {
+		t.Fatalf("no video has been registered, so there is nothing to look for")
+		return
+	}
+
+	background := context.Background()
+
+	reader, err := newCatalogueReader(background)
+	if err != nil {
+		t.Fatalf("could not reach the catalogue queue: %s", err)
+		return
+	}
+
+	announcement, err := reader.await(background, w.video.ID, announcementHoldWindow)
+	if err != nil {
+		t.Fatalf("could not read the catalogue queue: %s", err)
+		return
+	}
+	if announcement != nil {
+		t.Errorf("the catalogue was told about video %q, which has nothing to play", w.video.ID)
+	}
+}
+
+func theAnnouncementCarriesTheVideosDetails(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if w.announcement == nil {
+		t.Fatalf("no announcement has been received, so there is nothing to inspect")
+		return
+	}
+
+	if w.announcement.Title != w.sent.Title {
+		t.Errorf("the announcement calls the video %q, but it was registered as %q",
+			w.announcement.Title, w.sent.Title)
+	}
+
+	if want := playbackURLFor(w.slot.Key); w.announcement.PlaybackURL != want {
+		t.Errorf("the announcement points a player at %q, but the output landed at %q",
+			w.announcement.PlaybackURL, want)
+	}
+
+	// Named, not numbered: a reader of the queue has no access to the category
+	// ids cms issues, so the names are what make the message self-contained.
+	want := make([]string, 0, len(w.sent.CategoryIDs))
+	for _, id := range w.sent.CategoryIDs {
+		want = append(want, w.categoryName(t, id))
+	}
+
+	got := slices.Clone(w.announcement.Categories)
+	slices.Sort(got)
+	slices.Sort(want)
+
+	if !slices.Equal(got, want) {
+		t.Errorf("the announcement files the video under %v, but it was registered under %v",
+			w.announcement.Categories, want)
+	}
+}
+
+func theAnnouncementFilesItUnder(t gobdd.StepTest, ctx gobdd.Context, names string) {
+	w := worldOf(t, ctx)
+
+	if w.announcement == nil {
+		t.Fatalf("no announcement has been received, so there is nothing to inspect")
+		return
+	}
+
+	want := []string{}
+	for _, name := range strings.Split(names, ",") {
+		if name = strings.TrimSpace(name); name != "" {
+			want = append(want, name)
+		}
+	}
+
+	got := slices.Clone(w.announcement.Categories)
+	slices.Sort(got)
+	slices.Sort(want)
+
+	if !slices.Equal(got, want) {
+		t.Errorf("the announcement files the video under %v, but it was registered under %v",
+			w.announcement.Categories, want)
+	}
+}
+
+// --- the catalogue, as discovery serves it ----------------------------------
+
+// catalogueSettleTimeout bounds how long a scenario waits for the read side to
+// catch up. It is longer than the CMS's own status timeout because two hops
+// have to happen — cms consuming the job event and announcing, then discovery
+// consuming that announcement — each with its own long poll.
+const catalogueSettleTimeout = 30 * time.Second
+
+func theDiscoveryAPIIsAvailable(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	w.discovery = newDiscoveryClient()
+
+	result, err := w.discovery.get("/health")
+	if err != nil {
+		t.Fatalf("no Discovery at %s (%s) — start one with `docker compose up` from the repo root, "+
+			"or set DISCOVERY_BASE_URL to point at a running instance", w.discovery.base, err)
+		return
+	}
+	if result.status != 200 {
+		t.Fatalf("Discovery at %s is not healthy: %s", w.discovery.base, result.summary())
+	}
+}
+
+func theCatalogueEventuallyHoldsThatVideo(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if strings.TrimSpace(w.video.ID) == "" {
+		t.Fatalf("no video has been registered, so the catalogue could hold nothing")
+		return
+	}
+
+	deadline := time.Now().Add(catalogueSettleTimeout)
+	last := response{}
+	for time.Now().Before(deadline) {
+		result, err := w.discovery.get("/api/videos/" + w.video.ID)
+		if err != nil {
+			t.Fatalf("could not ask the catalogue for the video: %s", err)
+			return
+		}
+		last = result
+
+		if result.status == 200 {
+			var body catalogueVideoBody
+			if err := result.json(&body); err != nil {
+				t.Fatalf("could not decode the catalogue's copy: %s", err)
+				return
+			}
+			w.catalogued = &body
+			return
+		}
+
+		time.Sleep(statusPollInterval)
+	}
+
+	t.Errorf("the catalogue never held video %q within %s; asking for it still gives %s",
+		w.video.ID, catalogueSettleTimeout, last.summary())
+}
+
+func theCataloguesCopyCarriesTheDetails(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if w.catalogued == nil {
+		t.Fatalf("the catalogue holds no copy of the video, so there is nothing to inspect")
+		return
+	}
+
+	if w.catalogued.ID != w.video.ID {
+		t.Errorf("the catalogue filed the video as %q, but the CMS issued id %q",
+			w.catalogued.ID, w.video.ID)
+	}
+
+	if w.catalogued.Title != w.sent.Title {
+		t.Errorf("the catalogue calls the video %q, but it was registered as %q",
+			w.catalogued.Title, w.sent.Title)
+	}
+
+	if want := playbackURLFor(w.slot.Key); w.catalogued.PlaybackURL != want {
+		t.Errorf("the catalogue points a player at %q, but the output landed at %q",
+			w.catalogued.PlaybackURL, want)
+	}
+
+	want := make([]string, 0, len(w.sent.CategoryIDs))
+	for _, id := range w.sent.CategoryIDs {
+		want = append(want, w.categoryName(t, id))
+	}
+
+	got := slices.Clone(w.catalogued.Categories)
+	slices.Sort(got)
+	slices.Sort(want)
+
+	if !slices.Equal(got, want) {
+		t.Errorf("the catalogue files the video under %v, but it was registered under %v",
+			w.catalogued.Categories, want)
+	}
+}
+
+// catalogueAbsenceWindow is how long "does not hold" watches before it is
+// satisfied. Held rather than sampled once: the read side is asynchronous, so
+// a video that is absent the instant after registration proves nothing — it
+// might simply not have been announced yet.
+const catalogueAbsenceWindow = 5 * time.Second
+
+func theCatalogueDoesNotHoldThatVideo(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if strings.TrimSpace(w.video.ID) == "" {
+		t.Fatalf("no video has been registered, so there is nothing to look for")
+		return
+	}
+
+	deadline := time.Now().Add(catalogueAbsenceWindow)
+	for {
+		result, err := w.discovery.get("/api/videos/" + w.video.ID)
+		if err != nil {
+			t.Fatalf("could not ask the catalogue for the video: %s", err)
+			return
+		}
+
+		// Kept as the scenario's last exchange so the assertions that follow
+		// can read the status and the problem body the usual way.
+		w.last = result
+
+		if result.status != 404 {
+			t.Errorf("the catalogue holds video %q, which the CMS never announced: %s",
+				w.video.ID, result.summary())
+			return
+		}
+
+		if time.Now().After(deadline) {
+			return
+		}
+		time.Sleep(statusPollInterval)
+	}
+}
+
+func theCatalogueEventuallyHoldsThatVideoWithNoPlaybackURL(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if strings.TrimSpace(w.video.ID) == "" {
+		t.Fatalf("no video has been registered, so the catalogue could hold nothing")
+		return
+	}
+
+	deadline := time.Now().Add(catalogueSettleTimeout)
+	last := ""
+	for time.Now().Before(deadline) {
+		result, err := w.discovery.get("/api/videos/" + w.video.ID)
+		if err != nil {
+			t.Fatalf("could not ask the catalogue for the video: %s", err)
+			return
+		}
+
+		if result.status == 200 {
+			var body catalogueVideoBody
+			if err := result.json(&body); err != nil {
+				t.Fatalf("could not decode the catalogue's copy: %s", err)
+				return
+			}
+			if body.PlaybackURL == "" {
+				w.catalogued = &body
+				return
+			}
+			last = body.PlaybackURL
+		}
+
+		time.Sleep(statusPollInterval)
+	}
+
+	t.Errorf("the catalogue still points a player at %q for video %q after %s; the later "+
+		"announcement carried no playlist and should have replaced it", last, w.video.ID, catalogueSettleTimeout)
+}
+
+// --- Catalogue search ------------------------------------------------------
+
+// searchLimit is the page size the search scenarios ask for unless they are
+// about paging itself.
+const searchLimit = 20
+
+// uniqueTitle appends a nonce to the title a scenario names.
+//
+// Scenarios write real rows and nothing cleans up after them, so a fixed title
+// accumulates a copy per run and "the search returns exactly this video" stops
+// meaning anything. The nonce keeps every run's video findable on its own
+// while the feature file still reads in plain words.
+func uniqueTitle(title string) string {
+	return fmt.Sprintf("%s %s", title, strings.ToUpper(strconv.FormatInt(time.Now().UnixNano(), 36)))
+}
+
+// theCatalogueHoldsAVideoTitled puts one video into the catalogue under a
+// title the scenario chose: register it, report its transcode finished, and
+// wait for the announcement to land on the read side.
+func theCatalogueHoldsAVideoTitled(t gobdd.StepTest, ctx gobdd.Context, title string) {
+	catalogueVideoTitledUnder(t, ctx, title, "other")
+}
+
+// catalogueVideoTitledUnder is the whole write-to-read round trip for one
+// video, which is what "the catalogue holds …" costs: cms issues the id, the
+// job completes, cms announces it, discovery ingests it.
+func catalogueVideoTitledUnder(t gobdd.StepTest, ctx gobdd.Context, title, categories string) {
+	w := worldOf(t, ctx)
+
+	full := uniqueTitle(title)
+
+	requestUploadSlot(t, ctx, "searchable.mp4", "video/mp4")
+	uploadTheFile(t, ctx)
+	registerUploadedVideo(t, ctx, full, categories)
+
+	if w.last.status != 201 {
+		t.Fatalf("could not register a video to search for: %s", w.last.summary())
+		return
+	}
+
+	w.searchTitle = full
+
+	mediaConvertReportsState(t, ctx, "COMPLETE")
+	theCatalogueEventuallyHoldsThatVideo(t, ctx)
+}
+
+// searchTheCatalogueForThatTitle searches for the exact title the scenario's
+// video was catalogued under, nonce included — the scenario says "that video's
+// title" precisely so it does not have to know about the nonce.
+func searchTheCatalogueForThatTitle(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if strings.TrimSpace(w.searchTitle) == "" {
+		t.Fatalf("no video has been catalogued, so there is no title to search for")
+		return
+	}
+
+	searchTheCatalogue(t, w, searchRequest{Title: w.searchTitle, Limit: searchLimit})
+}
+
+// searchTheCatalogue performs the query and decodes the page, leaving both the
+// raw exchange and the decoded results on the world so a "Then" step can
+// assert on either.
+func searchTheCatalogue(t gobdd.StepTest, w *world, request searchRequest) {
+	if w.discovery == nil {
+		w.discovery = newDiscoveryClient()
+	}
+
+	result, err := w.discovery.search("/api/videos", request)
+	if err != nil {
+		t.Fatalf("could not search the catalogue: %s", err)
+		return
+	}
+
+	w.last = result
+	w.results = searchResultsBody{}
+
+	// A rejected search has no page to decode; the scenario asserting the
+	// rejection reads w.last instead.
+	if result.status == 200 {
+		if err := result.json(&w.results); err != nil {
+			t.Fatalf("could not decode the search results: %s", err)
+		}
+	}
+}
+
+func theSearchReturnsThatVideo(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if w.last.status != 200 {
+		t.Fatalf("the search did not succeed: %s", w.last.summary())
+		return
+	}
+	if !w.results.holds(w.video.ID) {
+		t.Errorf("the search for %q did not return video %q; it returned %v",
+			w.searchTitle, w.video.ID, w.results.ids())
+	}
+}
+
+// searchTheCatalogueFor searches for a term the scenario spells out, rather
+// than for the title of the video it catalogued.
+func searchTheCatalogueFor(t gobdd.StepTest, ctx gobdd.Context, term string) {
+	searchTheCatalogue(t, worldOf(t, ctx), searchRequest{Title: term, Limit: searchLimit})
+}
+
+// theCatalogueHoldsAVideoTitledUnder is theCatalogueHoldsAVideoTitled for a
+// scenario that cares which categories the video is filed under.
+func theCatalogueHoldsAVideoTitledUnder(t gobdd.StepTest, ctx gobdd.Context, title, categories string) {
+	catalogueVideoTitledUnder(t, ctx, title, categories)
+}
+
+func searchTheCatalogueForThatTitleInCategory(t gobdd.StepTest, ctx gobdd.Context, category string) {
+	w := worldOf(t, ctx)
+
+	if strings.TrimSpace(w.searchTitle) == "" {
+		t.Fatalf("no video has been catalogued, so there is no title to search for")
+		return
+	}
+
+	searchTheCatalogue(t, w, searchRequest{
+		Title:      w.searchTitle,
+		Categories: []string{category},
+		Limit:      searchLimit,
+	})
+}
+
+func theSearchDoesNotReturnThatVideo(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if w.last.status != 200 {
+		t.Fatalf("the search did not succeed: %s", w.last.summary())
+		return
+	}
+	if w.results.holds(w.video.ID) {
+		t.Errorf("the search returned video %q, which it should have left out; it returned %v",
+			w.video.ID, w.results.ids())
+	}
+}
+
+// searchTheCatalogueForThatTitleInCategories is the several-categories form,
+// for pinning that they mean "any of" rather than "all of".
+func searchTheCatalogueForThatTitleInCategories(t gobdd.StepTest, ctx gobdd.Context, categories string) {
+	w := worldOf(t, ctx)
+
+	if strings.TrimSpace(w.searchTitle) == "" {
+		t.Fatalf("no video has been catalogued, so there is no title to search for")
+		return
+	}
+
+	names := []string{}
+	for _, name := range strings.Split(categories, ",") {
+		if name = strings.TrimSpace(name); name != "" {
+			names = append(names, name)
+		}
+	}
+
+	searchTheCatalogue(t, w, searchRequest{
+		Title:      w.searchTitle,
+		Categories: names,
+		Limit:      searchLimit,
+	})
+}
+
+// theCatalogueHoldsVideosTitled catalogues several videos under one shared
+// title, which is what a paging scenario needs: enough matches to fill more
+// than one page.
+//
+// The nonce is generated once and shared, rather than per video, so that
+// searching for it matches this scenario's videos and no others. Runs leave
+// their videos behind, so a term that also matched a previous run's would make
+// "the search returns 2 videos" mean nothing.
+func theCatalogueHoldsVideosTitled(t gobdd.StepTest, ctx gobdd.Context, count int, title string) {
+	w := worldOf(t, ctx)
+
+	w.searchTitle = uniqueTitle(title)
+	w.cohort = nil
+	w.pageSeen = nil
+
+	for i := 0; i < count; i++ {
+		requestUploadSlot(t, ctx, "searchable.mp4", "video/mp4")
+		uploadTheFile(t, ctx)
+		registerUploadedVideo(t, ctx, w.searchTitle, "other")
+
+		if w.last.status != 201 {
+			t.Fatalf("could not register video %d of %d to page through: %s", i+1, count, w.last.summary())
+			return
+		}
+
+		mediaConvertReportsState(t, ctx, "COMPLETE")
+		theCatalogueEventuallyHoldsThatVideo(t, ctx)
+
+		w.cohort = append(w.cohort, w.video.ID)
+	}
+}
+
+func searchTheCatalogueForThatTitleAPageAtATime(t gobdd.StepTest, ctx gobdd.Context, size int) {
+	w := worldOf(t, ctx)
+
+	w.pageSeen = nil
+	searchTheCatalogue(t, w, searchRequest{Title: w.searchTitle, Limit: size})
+	recordPage(w)
+}
+
+// followTheCursor asks for the page after the one just returned, with the same
+// term and page size — a cursor says where to carry on from, not what to look
+// for.
+func followTheCursor(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if w.results.NextCursor == "" {
+		t.Fatalf("the last page offered no cursor, so there is no next page to follow")
+		return
+	}
+
+	searchTheCatalogue(t, w, searchRequest{
+		Title:  w.searchTitle,
+		Limit:  len(w.results.Videos),
+		Cursor: w.results.NextCursor,
+	})
+	recordPage(w)
+}
+
+// recordPage remembers what a page held, so a later step can check the pages
+// tile the results.
+func recordPage(w *world) {
+	w.pageSeen = append(w.pageSeen, w.results.ids()...)
+}
+
+func theSearchReturnsNVideos(t gobdd.StepTest, ctx gobdd.Context, count int) {
+	w := worldOf(t, ctx)
+
+	if w.last.status != 200 {
+		t.Fatalf("the search did not succeed: %s", w.last.summary())
+		return
+	}
+	if len(w.results.Videos) != count {
+		t.Errorf("expected %d videos in the page, got %d: %v", count, len(w.results.Videos), w.results.ids())
+	}
+}
+
+func theSearchOffersACursor(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if strings.TrimSpace(w.results.NextCursor) == "" {
+		t.Errorf("the page offered no cursor, so there is no way to ask for the next one")
+	}
+}
+
+func theSearchOffersNoFurtherCursor(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	if w.results.NextCursor != "" {
+		t.Errorf("the last page still offered cursor %q, so a reader cannot tell they have reached the end",
+			w.results.NextCursor)
+	}
+}
+
+// thePagesTileTheCohort checks the pages between them held each of the
+// scenario's videos exactly once — the property a cursor exists to give, and
+// the one an offset loses as soon as the catalogue is written to.
+func thePagesTileTheCohort(t gobdd.StepTest, ctx gobdd.Context) {
+	w := worldOf(t, ctx)
+
+	seen := map[string]int{}
+	for _, id := range w.pageSeen {
+		seen[id]++
+	}
+
+	for _, id := range w.cohort {
+		switch seen[id] {
+		case 1:
+		case 0:
+			t.Errorf("video %q was catalogued but appeared in none of the pages; the pages held %v",
+				id, w.pageSeen)
+		default:
+			t.Errorf("video %q appeared in %d pages; a video should be on exactly one", id, seen[id])
+		}
+	}
+
+	if len(w.pageSeen) != len(w.cohort) {
+		t.Errorf("the pages held %d videos between them, but %d were catalogued: %v",
+			len(w.pageSeen), len(w.cohort), w.pageSeen)
+	}
+}
+
+func searchTheCatalogueFromCursor(t gobdd.StepTest, ctx gobdd.Context, cursor string) {
+	w := worldOf(t, ctx)
+	searchTheCatalogue(t, w, searchRequest{
+		Title:  w.searchTitle,
+		Limit:  searchLimit,
+		Cursor: cursor,
+	})
+}
+
+// searchTheWholeCatalogue searches with no term and no categories, which is
+// how a reader browses rather than searches.
+func searchTheWholeCatalogue(t gobdd.StepTest, ctx gobdd.Context, size int) {
+	searchTheCatalogue(t, worldOf(t, ctx), searchRequest{Limit: size})
+}
+
+func theSearchReturnsAtMostNVideos(t gobdd.StepTest, ctx gobdd.Context, most int) {
+	w := worldOf(t, ctx)
+
+	if w.last.status != 200 {
+		t.Fatalf("the search did not succeed: %s", w.last.summary())
+		return
+	}
+	if len(w.results.Videos) > most {
+		t.Errorf("the search returned %d videos, more than the %d it should cap at",
+			len(w.results.Videos), most)
+	}
+}

tests/suite_test.go

@@ -16,6 +16,10 @@ import (
 type world struct {
 	client *client
 
+	// discovery is the read side, set by the Background step that checks it is
+	// up. Nil in scenarios that never mention the catalogue.
+	discovery *client
+
 	// last is the most recent exchange with the CMS. Every "Then the request
 	// …" step reads it.
 	last response
@@ -36,6 +40,32 @@ type world struct {
 	// category instead of hard-coding the id the seed migration happened to
 	// give it.
 	categoriesByName map[string]int16
+
+	// announcement is the message the catalogue topic carried for this
+	// scenario's video, once a step has waited for it.
+	announcement *catalogueAnnouncement
+
+	// catalogued is the read side's own copy of the video, once a step has
+	// waited for it to arrive.
+	catalogued *catalogueVideoBody
+
+	// searchTitle is the exact title the scenario's video was catalogued
+	// under. Scenarios name a plain title like "Desert Falcons"; the step
+	// registering it appends a nonce, because nothing cleans up between runs
+	// and a title reused across runs would make a result count meaningless.
+	searchTitle string
+
+	// results is the last page the catalogue search answered with.
+	results searchResultsBody
+
+	// pageSeen accumulates the ids from every page a paging scenario has
+	// walked, so it can assert the pages tile the results rather than
+	// repeating or dropping one.
+	pageSeen []string
+
+	// cohort is the ids of the videos a scenario catalogued together under one
+	// shared title, in the order they were created.
+	cohort []string
 }
 
 type worldKey struct{}
@@ -91,6 +121,19 @@ func (w *world) categoryID(t gobdd.StepTest, name string) int16 {
 	return id
 }
 
+// categoryName resolves a category id back to its name, for asserting on a
+// message that names categories rather than numbering them.
+func (w *world) categoryName(t gobdd.StepTest, id int16) string {
+	w.categoryID(t, "other") // ensures the list is loaded
+	for name, known := range w.categoriesByName {
+		if known == id {
+			return name
+		}
+	}
+	t.Fatalf("the CMS knows no category with id %d; it offers %v", id, w.categoriesByName)
+	return ""
+}
+
 // categoryIDs turns a comma-separated list of category names into the ids the
 // register endpoint expects. An empty list stays empty — that is a scenario in
 // its own right.
@@ -130,6 +173,7 @@ func TestVideoUpload(t *testing.T) {
 
 	// Given — the stack is up, and steps that arrange state a later When acts on.
 	suite.AddStep(`^the CMS API is available$`, theAPIIsAvailable)
+	suite.AddStep(`^the Discovery API is available$`, theDiscoveryAPIIsAvailable)
 	suite.AddStep(`^I have requested an upload slot for "(.*)" of type "(.*)"$`, requestUploadSlot)
 	suite.AddStep(`^I have uploaded the file to the upload URL$`, uploadTheFile)
 	suite.AddStep(`^I have registered the uploaded video titled "(.*)" under categories "(.*)"$`, registerUploadedVideo)
@@ -142,6 +186,7 @@ func TestVideoUpload(t *testing.T) {
 	suite.AddStep(`^I ask the CMS for that video$`, askForThatVideo)
 	suite.AddStep(`^I ask the CMS for the video with id "(.*)"$`, askForVideoWithID)
 	suite.AddStep(`^I have registered a video that is being transcoded$`, registerVideoBeingTranscoded)
+	suite.AddStep(`^I have registered a video being transcoded under categories "(.*)"$`, registerVideoBeingTranscodedUnder)
 	suite.AddStep(`^I have registered a video$`, registerVideoBeingTranscoded)
 	suite.AddStep(`^I ask the CMS for that video with its id written (.*)$`, askForThatVideoWrittenAs)
 	suite.AddStep(`^MediaConvert reports that the job reached "(.*)"$`, mediaConvertReportsState)
@@ -151,6 +196,32 @@ func TestVideoUpload(t *testing.T) {
 	suite.AddStep(`^the video keeps status "(.*)"$`, theVideoKeepsStatus)
 	suite.AddStep(`^the video has a playback URL for its HLS playlist$`, theVideoHasAPlaybackURL)
 	suite.AddStep(`^the video has no playback URL$`, theVideoHasNoPlaybackURL)
+	suite.AddStep(`^the catalogue is told about the video$`, theCatalogueIsToldAboutTheVideo)
+	suite.AddStep(`^the catalogue is told nothing about the video$`, theCatalogueIsToldNothingAboutTheVideo)
+	suite.AddStep(`^the announcement carries the video's title, playback URL and categories$`, theAnnouncementCarriesTheVideosDetails)
+	suite.AddStep(`^the announcement files it under "(.*)"$`, theAnnouncementFilesItUnder)
+	suite.AddStep(`^the catalogue eventually holds that video$`, theCatalogueEventuallyHoldsThatVideo)
+	suite.AddStep(`^the catalogue does not hold that video$`, theCatalogueDoesNotHoldThatVideo)
+	suite.AddStep(`^the catalogue eventually holds that video with no playback URL$`, theCatalogueEventuallyHoldsThatVideoWithNoPlaybackURL)
+	suite.AddStep(`^the catalogue's copy carries its title, playback URL and categories$`, theCataloguesCopyCarriesTheDetails)
+	suite.AddStep(`^the catalogue holds a video titled "([^"]*)"$`, theCatalogueHoldsAVideoTitled)
+	suite.AddStep(`^I search the catalogue for that video's title$`, searchTheCatalogueForThatTitle)
+	suite.AddStep(`^I search the catalogue for "([^"]*)"$`, searchTheCatalogueFor)
+	suite.AddStep(`^the catalogue holds a video titled "([^"]*)" filed under "([^"]*)"$`, theCatalogueHoldsAVideoTitledUnder)
+	suite.AddStep(`^I search the catalogue for that video's title in category "([^"]*)"$`, searchTheCatalogueForThatTitleInCategory)
+	suite.AddStep(`^I search the catalogue for that video's title in categories "([^"]*)"$`, searchTheCatalogueForThatTitleInCategories)
+	suite.AddStep(`^the catalogue holds (\d+) videos titled "([^"]*)"$`, theCatalogueHoldsVideosTitled)
+	suite.AddStep(`^I search the catalogue for that title (\d+) at a time$`, searchTheCatalogueForThatTitleAPageAtATime)
+	suite.AddStep(`^I search the whole catalogue (\d+) at a time$`, searchTheWholeCatalogue)
+	suite.AddStep(`^I follow the cursor$`, followTheCursor)
+	suite.AddStep(`^I search the catalogue for that video's title from the cursor "([^"]*)"$`, searchTheCatalogueFromCursor)
+	suite.AddStep(`^the search returns (\d+) videos?$`, theSearchReturnsNVideos)
+	suite.AddStep(`^the search returns at most (\d+) videos$`, theSearchReturnsAtMostNVideos)
+	suite.AddStep(`^the search offers a cursor to the next page$`, theSearchOffersACursor)
+	suite.AddStep(`^the search offers no further cursor$`, theSearchOffersNoFurtherCursor)
+	suite.AddStep(`^the pages together hold every one of those videos exactly once$`, thePagesTileTheCohort)
+	suite.AddStep(`^the search does not return that video$`, theSearchDoesNotReturnThatVideo)
+	suite.AddStep(`^the search returns that video$`, theSearchReturnsThatVideo)
 	suite.AddStep(`^I register the uploaded video titled "(.*)" under categories "(.*)"$`, registerUploadedVideo)
 	suite.AddStep(`^I register the uploaded video titled "(.*)" under category id (\d+)$`, registerUnderCategoryID)
 	suite.AddStep(`^I register a video titled "(.*)" with the key "(.*)" under categories "(.*)"$`, registerWithKey)

upload-video-demo.sh

@@ -0,0 +1,357 @@
+#!/usr/bin/env bash
+#
+# upload-video.sh — drive the cms upload flow end to end from a local file.
+#
+#   1. GET  /api/categories        pick what the video is filed under
+#   2. POST /api/videos/presign    ask for an upload slot
+#   3. PUT  <presigned url>        send the file straight to S3, not through cms
+#   4. POST /api/videos            register it, which queues the transcode
+#   5. GET  /api/videos/{id}       poll until playbackUrl appears
+#
+# The playback URL is empty until the MediaConvert job reports success and
+# cms's consumer records it, so step 5 is a wait, not a formality. A job that
+# comes back "failed" never gets a URL — the script stops rather than polls on.
+#
+# Usage: scripts/upload-video.sh [options]
+# Run with --help for the full list. With no options it asks for everything.
+
+set -euo pipefail
+
+readonly DEFAULT_BASE_URL="http://cms-alb-d478b02-648162889.us-east-1.elb.amazonaws.com"
+readonly DEFAULT_POLL_INTERVAL=5
+readonly DEFAULT_POLL_TIMEOUT=900
+
+base_url="${CMS_BASE_URL:-$DEFAULT_BASE_URL}"
+file_path=""
+title=""
+description=""
+tags=""
+categories_arg=""
+poll_interval="$DEFAULT_POLL_INTERVAL"
+poll_timeout="$DEFAULT_POLL_TIMEOUT"
+skip_poll=false
+
+usage() {
+  cat <<'USAGE'
+Upload a video to the Thamanyah cms and wait for its playback URL.
+
+Usage: upload-video.sh [options]
+
+Options:
+  -f, --file PATH          Video file to upload (.mp4 or .mov). Prompted for if omitted.
+  -t, --title TITLE        Video title. Prompted for if omitted.
+  -d, --description TEXT   Video description. Optional.
+      --tags TAGS          Free-text comma-separated tags. Optional.
+  -c, --categories LIST    Comma-separated category names or ids (e.g. "news,podcast"
+                           or "2,4"). Prompted for if omitted.
+  -u, --url URL            cms base URL. Default: $CMS_BASE_URL or http://localhost:8081
+  -i, --interval SECONDS   Seconds between status polls. Default: 5
+      --timeout SECONDS    Give up waiting after this long. Default: 900
+      --no-poll            Register the video and exit without waiting.
+  -h, --help               Show this help.
+
+Requires: curl, jq.
+
+Exit codes: 0 ready, 1 usage/upload error, 2 transcode failed, 3 poll timed out.
+USAGE
+}
+
+die() {
+  printf '\nerror: %s\n' "$*" >&2
+  exit 1
+}
+
+info() { printf '%s\n' "$*" >&2; }
+
+# The poll writes over one line on a terminal and one line per check when
+# redirected — an escape-littered log file helps nobody.
+progress() {
+  if [[ -t 2 ]]; then
+    printf '\r\033[K  %s' "$*" >&2
+  else
+    printf '  %s\n' "$*" >&2
+  fi
+}
+
+clear_progress() { [[ -t 2 ]] && printf '\r\033[K' >&2 || true; }
+
+require_tools() {
+  local missing=()
+  for tool in curl jq; do
+    command -v "$tool" >/dev/null 2>&1 || missing+=("$tool")
+  done
+  [[ ${#missing[@]} -eq 0 ]] || die "missing required command(s): ${missing[*]}"
+}
+
+# The API reports failures as RFC 9457 problem details; surface the human parts
+# of that rather than dumping the raw body.
+report_api_error() {
+  local context="$1" status="$2" body="$3" problem_title problem_detail
+
+  problem_title=$(jq -r '.title // empty' <<<"$body" 2>/dev/null || true)
+  problem_detail=$(jq -r '.detail // empty' <<<"$body" 2>/dev/null || true)
+
+  if [[ -n "$problem_title" ]]; then
+    printf '\nerror: %s (HTTP %s)\n  %s\n' "$context" "$status" "$problem_title" >&2
+    [[ -n "$problem_detail" ]] && printf '  %s\n' "$problem_detail" >&2
+  else
+    printf '\nerror: %s (HTTP %s)\n  %s\n' "$context" "$status" "${body:-<empty response>}" >&2
+  fi
+  exit 1
+}
+
+# Splits curl's "body + trailing status line" into the two globals the callers
+# read, so a request needs one subshell rather than two round trips.
+http_status=""
+http_body=""
+request() {
+  local response
+  response=$(curl --silent --show-error --location --write-out $'\n%{http_code}' "$@") ||
+    die "request to cms failed — is it running at $base_url ?"
+  http_status="${response##*$'\n'}"
+  http_body="${response%$'\n'*}"
+}
+
+content_type_for() {
+  case "${1,,}" in
+  *.mp4) printf 'video/mp4' ;;
+  *.mov) printf 'video/quicktime' ;;
+  *) die "unsupported file type: $1 — cms accepts only .mp4 and .mov" ;;
+  esac
+}
+
+prompt_required() {
+  local varname="$1" prompt="$2" value=""
+  [[ -t 0 ]] || die "$varname not given and stdin is not a terminal — pass it as an option"
+  while [[ -z "$value" ]]; do
+    read -r -e -p "$prompt" value
+    value="${value#"${value%%[![:space:]]*}"}"
+  done
+  printf '%s' "$value"
+}
+
+while [[ $# -gt 0 ]]; do
+  case "$1" in
+  -f | --file)
+    file_path="${2:-}"
+    shift 2
+    ;;
+  -t | --title)
+    title="${2:-}"
+    shift 2
+    ;;
+  -d | --description)
+    description="${2:-}"
+    shift 2
+    ;;
+  --tags)
+    tags="${2:-}"
+    shift 2
+    ;;
+  -c | --categories)
+    categories_arg="${2:-}"
+    shift 2
+    ;;
+  -u | --url)
+    base_url="${2:-}"
+    shift 2
+    ;;
+  -i | --interval)
+    poll_interval="${2:-}"
+    shift 2
+    ;;
+  --timeout)
+    poll_timeout="${2:-}"
+    shift 2
+    ;;
+  --no-poll)
+    skip_poll=true
+    shift
+    ;;
+  -h | --help)
+    usage
+    exit 0
+    ;;
+  *)
+    usage >&2
+    die "unknown option: $1"
+    ;;
+  esac
+done
+
+require_tools
+
+base_url="${base_url%/}"
+[[ "$poll_interval" =~ ^[0-9]+$ && "$poll_interval" -gt 0 ]] || die "--interval must be a positive integer"
+[[ "$poll_timeout" =~ ^[0-9]+$ && "$poll_timeout" -gt 0 ]] || die "--timeout must be a positive integer"
+
+# ---------------------------------------------------------------- the file ---
+
+if [[ -z "$file_path" ]]; then
+  # -e gives readline's filename completion, which is the whole point of
+  # asking here rather than making the flag mandatory.
+  file_path=$(prompt_required "--file" "Video file to upload: ")
+fi
+file_path="${file_path/#\~/$HOME}"
+
+[[ -f "$file_path" ]] || die "no such file: $file_path"
+[[ -r "$file_path" ]] || die "file is not readable: $file_path"
+[[ -s "$file_path" ]] || die "file is empty: $file_path"
+
+file_name=$(basename -- "$file_path")
+content_type=$(content_type_for "$file_name")
+
+info "Checking cms at $base_url ..."
+request "$base_url/health"
+[[ "$http_status" == "200" ]] || report_api_error "cms is not healthy" "$http_status" "$http_body"
+
+# ---------------------------------------------------------- the categories ---
+
+request "$base_url/api/categories"
+[[ "$http_status" == "200" ]] || report_api_error "could not list categories" "$http_status" "$http_body"
+categories_json="$http_body"
+
+if [[ -z "$categories_arg" ]]; then
+  info ""
+  info "Available categories:"
+  jq -r '.categories[] | "  \(.id)) \(.name)"' <<<"$categories_json" >&2
+  info ""
+  categories_arg=$(prompt_required "--categories" "Categories (comma-separated ids or names): ")
+fi
+
+# Accept either spelling — an id straight from the list, or the name it goes by
+# — and resolve both to the ids POST /api/videos wants. Unmatched entries come
+# back in their own member rather than as a jq error, so they can be named.
+resolved=$(jq -c --arg raw "$categories_arg" '
+	[$raw | split(",") | .[] | gsub("^\\s+|\\s+$"; "") | select(length > 0)] as $wanted
+	| {
+		ids: [ $wanted[] as $w
+			| $categories.categories[]
+			| select((.id | tostring) == $w or (.name | ascii_downcase) == ($w | ascii_downcase))
+			| .id ] | unique,
+		unknown: [ $wanted[] as $w
+			| select([ $categories.categories[]
+				| select((.id | tostring) == $w or (.name | ascii_downcase) == ($w | ascii_downcase)) ] | length == 0)
+			| $w ]
+	  }
+' --argjson categories "$categories_json" -n)
+
+unknown_categories=$(jq -r '.unknown | join(", ")' <<<"$resolved")
+[[ -z "$unknown_categories" ]] || die "no category is called \"$unknown_categories\" — pick from the list with GET $base_url/api/categories"
+
+category_ids_json=$(jq -c '.ids' <<<"$resolved")
+[[ "$category_ids_json" != "[]" ]] || die "at least one category is required"
+
+category_names=$(jq -r --argjson ids "$category_ids_json" \
+  '[.categories[] | select(.id as $i | $ids | index($i)) | .name] | join(", ")' <<<"$categories_json")
+
+# -------------------------------------------------------------- the metadata ---
+
+[[ -n "$title" ]] || title=$(prompt_required "--title" "Title: ")
+
+info ""
+info "Uploading   : $file_path ($content_type)"
+info "Title       : $title"
+info "Categories  : $category_names"
+info ""
+
+# ------------------------------------------------------------- 1. presign ---
+
+info "Requesting an upload URL ..."
+presign_body=$(jq -n --arg fileName "$file_name" --arg contentType "$content_type" \
+  '{fileName: $fileName, contentType: $contentType}')
+
+request -X POST "$base_url/api/videos/presign" \
+  -H 'Content-Type: application/json' \
+  --data-binary "$presign_body"
+[[ "$http_status" == "200" ]] || report_api_error "could not get an upload URL" "$http_status" "$http_body"
+
+upload_url=$(jq -r '.uploadUrl' <<<"$http_body")
+storage_key=$(jq -r '.key' <<<"$http_body")
+[[ -n "$upload_url" && "$upload_url" != "null" ]] || die "cms returned no upload URL"
+
+# ----------------------------------------------------- 2. PUT the file to S3 ---
+
+# The content type is signed into the URL, so this header has to match the one
+# sent to /presign exactly or S3 rejects the PUT as a signature mismatch.
+info "Uploading $(du -h -- "$file_path" | cut -f1) to storage ..."
+upload_status=$(curl --silent --show-error --progress-bar \
+  --request PUT \
+  --header "Content-Type: $content_type" \
+  --upload-file "$file_path" \
+  --write-out '%{http_code}' \
+  --output /dev/null \
+  "$upload_url") || die "upload to storage failed"
+
+[[ "$upload_status" =~ ^2 ]] || die "storage rejected the upload (HTTP $upload_status)"
+info "Upload complete: $storage_key"
+
+# ------------------------------------------- 3. register + queue transcoding ---
+
+info "Registering the video ..."
+complete_body=$(jq -n \
+  --arg title "$title" \
+  --arg description "$description" \
+  --arg tags "$tags" \
+  --arg fileName "$file_name" \
+  --arg key "$storage_key" \
+  --argjson categoryIds "$category_ids_json" \
+  '{title: $title, description: $description, categoryIds: $categoryIds, tags: $tags, fileName: $fileName, key: $key}')
+
+request -X POST "$base_url/api/videos" \
+  -H 'Content-Type: application/json' \
+  --data-binary "$complete_body"
+[[ "$http_status" == "201" ]] || report_api_error "could not register the video" "$http_status" "$http_body"
+
+video_id=$(jq -r '.id' <<<"$http_body")
+info "Registered as $video_id (status: $(jq -r '.status' <<<"$http_body"))"
+
+if [[ "$skip_poll" == true ]]; then
+  info ""
+  info "Not waiting for transcoding (--no-poll). Follow it with:"
+  info "  curl -s $base_url/api/videos/$video_id | jq"
+  printf '%s\n' "$video_id"
+  exit 0
+fi
+
+# ------------------------------------------ 4. poll until the URL is there ---
+
+info ""
+info "Waiting for transcoding to finish (timeout ${poll_timeout}s, checking every ${poll_interval}s) ..."
+
+started_at=$SECONDS
+while :; do
+  request "$base_url/api/videos/$video_id"
+  [[ "$http_status" == "200" ]] || report_api_error "could not read the video" "$http_status" "$http_body"
+
+  status=$(jq -r '.status' <<<"$http_body")
+  playback_url=$(jq -r '.playbackUrl // empty' <<<"$http_body")
+  elapsed=$((SECONDS - started_at))
+
+  # The URL is what the caller is actually waiting for, and it is written
+  # alongside the status — so wait for the URL, not merely for "ready".
+  if [[ -n "$playback_url" ]]; then
+    clear_progress
+    info "Ready after ${elapsed}s."
+    info ""
+    info "Playback URL:"
+    printf '%s\n' "$playback_url"
+    exit 0
+  fi
+
+  if [[ "$status" == "failed" ]]; then
+    clear_progress
+    printf '\nerror: transcoding failed for video %s after %ss — no playback URL will be produced.\n' "$video_id" "$elapsed" >&2
+    exit 2
+  fi
+
+  if ((elapsed >= poll_timeout)); then
+    clear_progress
+    printf '\nerror: timed out after %ss with status %q and no playback URL.\n  Check again with: curl -s %s/api/videos/%s | jq\n' \
+      "$elapsed" "$status" "$base_url" "$video_id" >&2
+    exit 3
+  fi
+
+  progress "status: $status — ${elapsed}s elapsed"
+  sleep "$poll_interval"
+done