style(purchase): US-spell misspell linter findings in touched files

Fix British spellings flagged by golangci-lint misspell in the two files
touched by the #95 TODO-update commit: execution.go and
service_details_codec.go. Changes are comment-only.

behaviour -> behavior, recognise/normalise/honour -> recognize/normalize/honor

Author: cristim

internal/purchase/execution.go

@@ -473,7 +473,7 @@ func (m *Manager) resolveGCPProvider(ctx context.Context, account config.CloudAc
 
 // getMaxAccountParallelism is a thin alias over the shared
 // execution.ConcurrencyFromEnv so the purchase manager and the scheduler
-// both honour the same CUDLY_MAX_ACCOUNT_PARALLELISM override.
+// both honor the same CUDLY_MAX_ACCOUNT_PARALLELISM override.
 func getMaxAccountParallelism() int {
 	return execution.ConcurrencyFromEnv()
 }
@@ -648,7 +648,7 @@ func recordHistoryAuditGap(exec *config.PurchaseExecution, commitmentID string,
 // token and the provider dedupes the purchase. It falls back to ExecutionID
 // only for legacy rows persisted before migration 000066 (IdempotencyKey == "");
 // for a single un-retried execution that fallback is identical to the pre-fix
-// behaviour, and such legacy rows never gain a retry successor that could
+// behavior, and such legacy rows never gain a retry successor that could
 // diverge (the retry handler seeds the successor's key from the predecessor's
 // ExecutionID in that case, preserving the match).
 func idempotencyLineageKey(exec *config.PurchaseExecution) string {
@@ -812,7 +812,7 @@ func (m *Manager) buildPurchaseConfirmationData(exec *config.PurchaseExecution,
 }
 
 // logRecCtxErr emits a diagnostic log line when a per-recommendation context has
-// been cancelled or timed out. It distinguishes DeadlineExceeded (the 30s per-rec
+// been canceled or timed out. It distinguishes DeadlineExceeded (the 30s per-rec
 // budget fired) from Canceled (a parent context stopped the execution) so that
 // CloudWatch filters can tell the two apart without parsing error strings.
 // It is a no-op when recCtxErr is nil.
@@ -958,8 +958,8 @@ func (m *Manager) executeSinglePurchase(ctx context.Context, rec config.Recommen
 // mapServiceType maps a service string to common.ServiceType. Both the
 // canonical hyphenated slugs (compute, relational-db, cache, search,
 // data-warehouse) and the legacy AWS-only slugs (ec2, rds, elasticache,
-// opensearch, redshift, memorydb) are recognised; everything else passes
-// through verbatim. Savings Plans slugs are normalised by mapSavingsPlansSlug.
+// opensearch, redshift, memorydb) are recognized; everything else passes
+// through verbatim. Savings Plans slugs are normalized by mapSavingsPlansSlug.
 func (m *Manager) mapServiceType(service string) common.ServiceType {
 	if svc, ok := mapSavingsPlansSlug(service); ok {
 		return svc
@@ -1009,7 +1009,7 @@ func mapServiceSlug(service string) (common.ServiceType, bool) {
 	return svc, ok
 }
 
-// mapSavingsPlansSlug normalises both the canonical hyphenated SP slugs and
+// mapSavingsPlansSlug normalizes both the canonical hyphenated SP slugs and
 // the dash-free spellings the frontend has historically sent into the
 // matching common.ServiceType. The map covers the legacy umbrella plus the
 // four per-plan-type slugs in both spellings — pulled out of mapServiceType

pkg/common/service_details_codec.go

@@ -60,7 +60,7 @@ func MarshalServiceDetails(details ServiceDetails) (json.RawMessage, error) {
 // "savingsplans", "savings-plans-compute" and the dash-free spellings —
 // see internal/purchase/execution.go: mapServiceType / mapSavingsPlansSlug).
 //
-// Behaviour:
+// Behavior:
 //   - raw is empty AND service maps to a known *Details type → returns a
 //     zero-valued typed pointer (the legacy / pre-#453 fallback). The
 //     downstream service client's buildOfferingFilters tolerates zero-
@@ -79,7 +79,7 @@ func DecodeServiceDetailsFor(service string, raw json.RawMessage) (ServiceDetail
 	if !ok {
 		// Service has no *Details type — nothing to decode. Empty raw
 		// payload is fine; a non-empty payload on a service we don't
-		// recognise is suspicious but tolerated (writers and readers
+		// recognize is suspicious but tolerated (writers and readers
 		// might be on different versions during a rolling deploy).
 		return nil, nil
 	}
@@ -124,7 +124,7 @@ func newDetailsForService(service string) (ServiceDetails, bool) {
 	// values of ServiceSavingsPlans / ServiceSavingsPlansCompute etc.;
 	// "savings-plans" (with a dash) is the legacy umbrella alias that
 	// internal/purchase/execution.go: mapSavingsPlansSlug still
-	// recognises so purchase_executions JSONB rows persisted before
+	// recognizes so purchase_executions JSONB rows persisted before
 	// the rename in PR #94 (merged 2026-04-30) still resolve.
 	// TODO(#95): drop "savings-plans" here once the ~6-month retention
 	// window has passed (earliest 2026-10-30). See execution.go
@@ -139,7 +139,7 @@ func newDetailsForService(service string) (ServiceDetails, bool) {
 
 	// Services that don't currently type-assert Details in
 	// findOfferingID (OpenSearch / Redshift / MemoryDB read rec.ResourceType
-	// directly). We still recognise OpenSearch and Redshift here so a
+	// directly). We still recognize OpenSearch and Redshift here so a
 	// forthcoming refactor that starts asserting can flip the table
 	// value without changing call sites. MemoryDB has no dedicated
 	// *Details type yet, so it's intentionally absent — callers see