Remove SLA section from SDLC skill and diagram
SLA timelines are meaningless to agents operating in heartbeats. Removed the 48-hour PR review SLA from SKILL.md and the SLA gantt chart from sdlc-diagram.md. Co-Authored-By: Paperclip <noreply@paperclip.ing>
This commit is contained in:
committed by
Countess von Containerheim [agent]
parent
da86aa7754
commit
40caf8cfee
@@ -67,16 +67,6 @@ Requires **3 approving GitHub reviews** before the CEO merges:
|
||||
|
||||
**Review order is mandatory: CI → UAT → QA → CTO → CEO merge.** Each stage gates the next. No agent merges their own PRs.
|
||||
|
||||
## 48-Hour PR Review SLA
|
||||
|
||||
Every open PR must receive its first review within 48 hours. Each reviewer's SLA starts when the previous stage approves.
|
||||
|
||||
- **24h:** CEO tags reviewer and surfaces PR in daily status
|
||||
- **48h:** SLA violation; CEO escalates to reviewer's manager
|
||||
- **72h+:** Critical-path PRs block the next release
|
||||
|
||||
Reviewers who cannot meet SLA must hand off within the window. No exceptions without board approval.
|
||||
|
||||
## Pipeline
|
||||
|
||||
### Pipeline A: Plugin/Feature Changes
|
||||
|
||||
@@ -108,21 +108,6 @@ flowchart TD
|
||||
class A_ENG,B_ENG,A_CEO,B_CEO agent
|
||||
```
|
||||
|
||||
## PR Review SLA
|
||||
|
||||
```mermaid
|
||||
gantt
|
||||
title PR Review SLA Timeline (per stage)
|
||||
dateFormat X
|
||||
axisFormat %d h
|
||||
|
||||
section SLA Windows
|
||||
Normal review window :done, 0, 24
|
||||
CEO escalation at 24h :active, 24, 48
|
||||
SLA violation at 48h :crit, 48, 72
|
||||
Release blocker at 72h :crit, 72, 96
|
||||
```
|
||||
|
||||
## Handoff Protocol
|
||||
|
||||
```mermaid
|
||||
|
||||
Reference in New Issue
Block a user