feat(#82,#83): Protocol versioning, enrichment guardrails, and documentation fixes

docs/IMPLEMENTATION_ROADMAP.md

@@ -76,7 +76,7 @@ This document tracks the implementation progress of the Holiday Peak Hub platfor
 - ✅ Entra ID configuration documentation
 
 #### Testing & Quality
-- ✅ **508 tests passing** (↑343 from v1.0.0)
+- ✅ **635 tests passing** (↑470 from v1.0.0)
 - ✅ Connector test suites (Oracle: 48 tests, Salesforce: 48 tests)
 - ✅ Enterprise hardening tests (circuit breaker, bulkhead, rate limiter, telemetry)
 - ✅ PIM writeback tests (25 tests)

docs/architecture/README.md

@@ -20,12 +20,9 @@ This folder contains comprehensive architectural documentation for the Holiday P
 - [Backend Governance](../governance/backend-governance.md) — Python, FastAPI, agent development standards
 - [Infrastructure Governance](../governance/infrastructure-governance.md) — Bicep, AKS, Azure services standards
 
-### Governance and Compliance
+### Reference Architecture Patterns
 
-- [Governance Overview](../governance/README.md) — Standards and compliance guidelines
-- [Frontend Governance](../governance/frontend-governance.md) — Next.js, React, TypeScript standards
-- [Backend Governance](../governance/backend-governance.md) — Python, FastAPI, agent development standards
-- [Infrastructure Governance](../governance/infrastructure-governance.md) — Bicep, AKS, Azure services standards
+- [Reference Architectures](reference/) — Connector, adapter, and multi-tenant patterns (issue #84 — in progress)
 
 ## Quick Links
 

docs/architecture/adrs/adr-003-adapter-pattern.md

@@ -22,7 +22,7 @@ The accelerator must support pluggable integrations without modifying agent or a
 ### Structure
 ```python
 # lib/src/holiday_peak_lib/adapters/base.py
-class RetailAdapter(ABC):
+class BaseAdapter(ABC):
     @abstractmethod
     async def fetch_inventory(self, sku: str) -> InventoryStatus:
         pass
@@ -32,7 +32,7 @@ class RetailAdapter(ABC):
         pass
 
 # Retailer implements adapter
-class LevisInventoryAdapter(RetailAdapter):
+class LevisInventoryAdapter(BaseAdapter):
     async def fetch_inventory(self, sku: str) -> InventoryStatus:
         # Call Levis API
         ...

docs/architecture/adrs/adr-023-enterprise-resilience-patterns.md

@@ -29,7 +29,7 @@ Key questions addressed:
 Prevents repeated calls to failing services, allowing them time to recover.
 
 ```python
-from holiday_peak_lib.resilience import CircuitBreaker, CircuitState
+from holiday_peak_lib.utils.circuit_breaker import CircuitBreaker, CircuitState
 
 class OracleFusionConnector:
     def __init__(self, config: OracleFusionConfig):
@@ -66,7 +66,7 @@ HALF_OPEN → (failure) → OPEN
 Isolates resources to prevent a single slow dependency from consuming all capacity.
 
 ```python
-from holiday_peak_lib.resilience import Bulkhead
+from holiday_peak_lib.utils.bulkhead import Bulkhead
 
 class SAPConnector:
     def __init__(self, config: SAPConfig):
@@ -96,7 +96,7 @@ class SAPConnector:
 Enforces API rate limits to prevent throttling errors.
 
 ```python
-from holiday_peak_lib.resilience import RateLimiter
+from holiday_peak_lib.utils.rate_limiter import RateLimiter
 
 class SalesforceConnector:
     def __init__(self, config: SalesforceConfig):

docs/architecture/business-summary.md

@@ -352,7 +352,7 @@ Each app addresses a specific retail process:
 - Rapid iteration and feature deployment
 
 **Quality and Reliability**:
-- Production-ready code with 508 tests and 80%+ coverage
+- Production-ready code with 635 tests and 85%+ coverage
 - WCAG 2.1 AA accessibility compliance
 - SOC 2, GDPR, PCI DSS frameworks included
 - Proven architecture patterns

docs/project-status.md

@@ -15,7 +15,7 @@
 - **PIM Writeback Module**: Opt-in writeback with conflict detection and audit trail
 - **HITL Staff Review UI**: Review queue, evidence panel, bulk approval
 - **Frontend API Integration**: Enhanced checkout, order tracking, inventory pages
-- **Test Coverage**: 508 tests passing (up from 386)
+- **Test Coverage**: 635 tests passing (up from 386)
 
 ### Merged PRs (v1.1.0)
 | # | Title | Category |
@@ -47,7 +47,7 @@
 ### Build Status
 | Workflow | Run | Status | Notes |
 |---|---|---|---|
-| `ci.yml` | Latest | ✅ Passing | 508 tests passing, all lint checks green |
+| `ci.yml` | Latest | ✅ Passing | 635 tests passing, all lint checks green |
 | `build-push` | Latest | ✅ Passing | Docker images published to GHCR |
 | `deploy-azd` | Latest | ✅ Passing | Azure deployment successful |
 
@@ -73,6 +73,11 @@
 | [#103](https://github.com/Azure-Samples/holiday-peak-hub/issues/103) | Phase 3: HITL Staff Review UI pages | High | Truth Layer | ✅ Closed (PR #127) |
 | [#107](https://github.com/Azure-Samples/holiday-peak-hub/issues/107) | Phase 5: PIM writeback module | Low | Truth Layer | ✅ Closed (PR #116) |
 | [#110](https://github.com/Azure-Samples/holiday-peak-hub/issues/110) | Phase 5: Enterprise hardening | Low | Hardening | ✅ Closed (PR #119) |
+| [#79](https://github.com/Azure-Samples/holiday-peak-hub/issues/79) | Connector Registry Pattern | Medium | Architecture | ✅ Closed |
+| [#80](https://github.com/Azure-Samples/holiday-peak-hub/issues/80) | Event-Driven Connector Sync | Medium | Architecture | ✅ Closed |
+| [#81](https://github.com/Azure-Samples/holiday-peak-hub/issues/81) | Multi-Tenant Connector Config | Medium | Architecture | ✅ Closed |
+| [#82](https://github.com/Azure-Samples/holiday-peak-hub/issues/82) | Protocol Interface Evolution | Medium | Architecture | ✅ Closed |
+| [#83](https://github.com/Azure-Samples/holiday-peak-hub/issues/83) | Internal Data Enrichment Guardrails | Medium | Architecture | ✅ Closed |
 
 ---
 
@@ -92,7 +97,7 @@
 | [#128](https://github.com/Azure-Samples/holiday-peak-hub/pull/128) | Truth Export service | 4 | 🔄 Draft |
 | [#129](https://github.com/Azure-Samples/holiday-peak-hub/pull/129) | Truth-layer CRUD routes | 4 | 🔄 Draft |
 | [#117](https://github.com/Azure-Samples/holiday-peak-hub/pull/117) | Evidence extraction module | 5 | 🔄 Draft |
-| [#135](https://github.com/Azure-Samples/holiday-peak-hub/pull/135) | Data guardrails for AI agents | Architecture | 🔄 Draft |
+
 
 ---
 
@@ -146,11 +151,11 @@ Marked `priority: medium`. Can proceed in parallel with Truth Layer work.
 
 | # | Title | Status |
 |---|---|---|
-| [#79](https://github.com/Azure-Samples/holiday-peak-hub/issues/79) | Architecture: Connector Registry Pattern | Open |
-| [#80](https://github.com/Azure-Samples/holiday-peak-hub/issues/80) | Architecture: Event-Driven Sync Pattern | PR #132 (Ready) |
-| [#81](https://github.com/Azure-Samples/holiday-peak-hub/issues/81) | Architecture: Multi-Tenant Configuration | Open |
-| [#82](https://github.com/Azure-Samples/holiday-peak-hub/issues/82) | Architecture: Protocol Evolution | Open |
-| [#83](https://github.com/Azure-Samples/holiday-peak-hub/issues/83) | Architecture: Data Guardrails | PR #135 (Draft) |
+| [#79](https://github.com/Azure-Samples/holiday-peak-hub/issues/79) | Architecture: Connector Registry Pattern | ✅ Closed |
+| [#80](https://github.com/Azure-Samples/holiday-peak-hub/issues/80) | Architecture: Event-Driven Sync Pattern | ✅ Closed |
+| [#81](https://github.com/Azure-Samples/holiday-peak-hub/issues/81) | Architecture: Multi-Tenant Configuration | ✅ Closed |
+| [#82](https://github.com/Azure-Samples/holiday-peak-hub/issues/82) | Architecture: Protocol Evolution | ✅ Closed |
+| [#83](https://github.com/Azure-Samples/holiday-peak-hub/issues/83) | Architecture: Data Guardrails | ✅ Closed |
 | [#84](https://github.com/Azure-Samples/holiday-peak-hub/issues/84) | Architecture: Reference Architecture Patterns | Open |
 
 ---
@@ -270,7 +275,7 @@ These issues are superseded by the Truth Layer epic or are long-running backgrou
 ### Completed
 - **20 PRs merged** (19 feature PRs + 1 CI fix)
 - **16 issues closed** (12 Truth Layer, 3 Connectors, 1 Payment)
-- **508 tests** passing (122 new tests)
+- **635 tests** passing (249 new tests)
 - **4 Enterprise Connectors** production-ready
 - **Enterprise Hardening** complete
 
@@ -285,6 +290,6 @@ These issues are superseded by the Truth Layer epic or are long-running backgrou
 |---|---|---|---|
 | `Platform_Quality` | 1 (#31) | 6 (#28-#30, #32-#33, #112) | 🔴 Review First |
 | `Truth_Layer_Pipeline` | 12 | 8 (PRs #144-#151, #125-#129) | 🟠 High |
-| `Architecture_Patterns` | 0 | 6 (#79-#84) | 🟡 Medium |
+| `Architecture_Patterns` | 5 (#79-#83) | 1 (#84) | 🟡 Medium |
 | `Enterprise_Connectors` | 5 | ~35 remaining | 🟡 Low |
 | `Truth_Layer_Hardening` | 2 (#107, #110) | 2 (#108, #109) | 🟢 Low / Optional |

lib/src/holiday_peak_lib/agents/guardrails/__init__.py

@@ -1,11 +1,19 @@
 """Guardrails for AI agent enrichment pipelines."""
 
 from holiday_peak_lib.agents.guardrails.enrichment_guardrail import (
+    ContentAttributor,
     EnrichmentGuardrail,
+    GuardrailMiddleware,
+    SourceRef,
     SourceValidationResult,
+    SourceValidator,
 )
 
 __all__ = [
+    "ContentAttributor",
     "EnrichmentGuardrail",
+    "GuardrailMiddleware",
+    "SourceRef",
     "SourceValidationResult",
+    "SourceValidator",
 ]