Skip to content

Commit 81fb8a5

Browse files
authored
Merge pull request #15 from ayim/ayim/overwatch-codebase-rules
Add scoped Overwatch codebase policies
2 parents 36a9cbd + 5a89f98 commit 81fb8a5

6 files changed

Lines changed: 65 additions & 0 deletions

File tree

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
version: overwatch-rules-v1
2+
rules:
3+
- id: checkout-money-movement
4+
description: Checkout changes can affect money movement and order integrity.
5+
owner: payments
6+
review: required
7+
autoApprove: never
8+
requiredTests:
9+
- payment idempotency
10+
- duplicate-submit protection
11+
- failed payment recovery
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
version: overwatch-rules-v1
2+
rules:
3+
- id: catalog-product-experience
4+
description: Product-page changes are eligible for risk-based approval with catalog coverage.
5+
owner: catalog
6+
review: risk_based
7+
autoApprove: eligible
8+
requiredTests:
9+
- unavailable variant behavior
10+
- price and currency rendering
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
version: overwatch-rules-v1
2+
rules:
3+
- id: sign-in-critical-path
4+
description: The sign-in route is a universal critical path and cannot be auto-approved.
5+
owner: identity
6+
review: required
7+
autoApprove: never
8+
requiredTests:
9+
- successful and rejected sign-in flows
10+
- redirect safety
11+
- no credentials or PII in analytics
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
version: overwatch-rules-v1
2+
rules:
3+
- id: analytics-data-contract
4+
description: Analytics changes must preserve event contracts and avoid customer PII.
5+
owner: data-platform
6+
review: risk_based
7+
autoApprove: eligible
8+
requiredTests:
9+
- event name and property contract
10+
- synthetic traffic labeling
11+
- no customer PII
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
version: overwatch-rules-v1
2+
rules:
3+
- id: authentication-boundary
4+
description: Authentication and session changes always require a human reviewer.
5+
owner: identity
6+
review: required
7+
autoApprove: never
8+
requiredTests:
9+
- denial-by-default behavior
10+
- invalid and expired session handling
11+
- login failure without credential leakage
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
version: overwatch-rules-v1
2+
rules:
3+
- id: operator-order-mutations
4+
description: Admin order mutations require an operations reviewer.
5+
owner: commerce-operations
6+
review: required
7+
autoApprove: never
8+
requiredTests:
9+
- state transition authorization
10+
- cancellation and refund safeguards
11+
- audit trail preservation

0 commit comments

Comments
 (0)