Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .agent/personas/p6-release-manager.md
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ gh pr create --base main --head <type>/issue-<id>-<slug> \
### 3. Zero Auto-Merge Rule

> [!CAUTION]
> **P6 NEVER merges a PR without explicit human confirmation.** Present the PR link and stop. Merge only after explicit confirmation (e.g. *"Tu peux merger"*).
> **P6 NEVER merges a PR without explicit human confirmation.** Present the PR link and stop. Merge only after explicit confirmation (e.g. *"Tu peux merger"* / *"Approve merge"*).

### 4. Post-Merge Cleanup & Dual-Sync
```bash
Expand Down
2 changes: 1 addition & 1 deletion .agent/rules/agent-lifecycle.md
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@ gh pr create --base <base_branch> --head <type>/issue-<id>-<slug> \
### Step 3.3 Β· Zero Auto-Merge Rule

> [!CAUTION]
> The agent NEVER merges a PR autonomously. Present the PR link and stop. Merge only after the user says *"Tu peux merger"* or equivalent.
> The agent NEVER merges a PR autonomously. Present the PR link and stop. Merge only after the user says *"Tu peux merger"* / *"Approve merge"* or equivalent.

### Step 3.4 Β· Post-Merge Branch Cleanup & Dynamic Sync

Expand Down
2 changes: 1 addition & 1 deletion .agent/rules/git-workflow.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@

- **Zero Auto-Merge**:
> [!CAUTION]
> The agent NEVER merges a PR without explicit user approval. Present the PR link and stop. Merge only after the user says *"Tu peux merger"* or equivalent.
> The agent NEVER merges a PR without explicit user approval. Present the PR link and stop. Merge only after the user says *"Tu peux merger"* / *"Approve merge"* or equivalent.

- **One Branch per Task / Issue**: every modification begins with `<type>/issue-<id>-<slug>` cut from up-to-date `main`.

Expand Down
2 changes: 1 addition & 1 deletion .agent/skills/open-pr/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ GitHubMCP:create_pull_request {
### Step 6: Zero Auto-Merge Guardrail (STOP & WAIT)

> [!CAUTION]
> **Zero Auto-Merge Rule**: Persona 6 MUST NEVER merge the PR automatically. Present the PR link in chat and stop. Wait for explicit user approval (e.g. *"Tu peux merger"*).
> **Zero Auto-Merge Rule**: Persona 6 MUST NEVER merge the PR automatically. Present the PR link in chat and stop. Wait for explicit user approval (e.g. *"Tu peux merger"* / *"Approve merge"*).

### Step 7: Post-Merge Cleanup & Dual-Sync
Once user confirms merge:
Expand Down
2 changes: 1 addition & 1 deletion .agent/templates/pr-walkthrough.md
Original file line number Diff line number Diff line change
Expand Up @@ -87,5 +87,5 @@ Zero "N/A" lines or empty placeholders are permitted.

> [!CAUTION]
> **Zero Auto-Merge β€” Explicit approval required.**
> This PR will NOT be merged until the author explicitly confirms: *"Tu peux merger"* or equivalent.
> This PR will NOT be merged until the author explicitly confirms: *"Tu peux merger"* / *"Approve merge"* or equivalent.
> P6 presents this PR link and stops. Merge is a deliberate human action.
2 changes: 1 addition & 1 deletion .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -70,5 +70,5 @@ Zero "N/A" lines or empty placeholders are permitted.

> [!CAUTION]
> **Zero Auto-Merge β€” Explicit approval required.**
> This PR will NOT be merged until the author explicitly confirms: *"Tu peux merger"* or equivalent.
> This PR will NOT be merged until the author explicitly confirms: *"Tu peux merger"* / *"Approve merge"* or equivalent.
> P6 presents this PR link and stops. Merge is a deliberate human action.
4 changes: 2 additions & 2 deletions AGENTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,15 +44,15 @@ flowchart LR
3. **Phase 3 β€” Release (Persona 6)**:
- Activates **Persona 6 (Release Manager)**: pushes branch and opens PR via `GitHubMCP:create_pull_request`.
- PR targets `epic/**` with `skip-release` for intermediate child tasks; targets `main` for standalone or final consolidated Epic PRs.
- **Gate 3.5 β€” Zero Auto-Merge Lock**: Halts with PR link. Merges exclusively after explicit user confirmation (*"Tu peux merger"*).
- **Gate 3.5 β€” Zero Auto-Merge Lock**: Halts with PR link. Merges exclusively after explicit user confirmation (*"Tu peux merger"* / *"Approve merge"*).
- Executes squash merge, runs `.agent/hooks/post-merge-dual-sync.sh` (dynamically synchronizing base branch), and prunes branches.

### 1.2 Conditional P2 Design Gate

Pillar 1 (Design Spec) enforcement is contextual and adaptive:
- **UI & Composable Changes**: **Persona 2 (Design Lead)** must define Material 3 tokens (from `DESIGN.md`), 4-state UI matrix, and Roborazzi expectations.
- **Non-UI Changes**: For backend, Room, CI/CD, scripts, or doc chores, Pillar 1 is explicitly marked: `N/A β€” No visual/UI changes`.
- **Explicit User Override**: If prompt requests to skip design (*"skip design"*, *"sans design"*), P2 is immediately bypassed.
- **Explicit User Override**: If prompt requests to skip design (*"skip design"* / *"sans design"*), P2 is immediately bypassed.

### 1.3 The 6 Engineering Personas β€” Delegation Matrix

Expand Down
2 changes: 1 addition & 1 deletion ARCHITECTURE.md
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ stateDiagram-v2
}

state Gate_3_5 <<choice>>
note right of Gate_3_5 : ZERO AUTO-MERGE β€” Explicit author confirmation\n('Tu peux merger')
note right of Gate_3_5 : ZERO AUTO-MERGE β€” Explicit author confirmation\n('Tu peux merger' / 'Approve merge')

Gate_3_5 --> SquashMerge : Confirmed
SquashMerge --> PostMergeSync : post-merge-dual-sync.sh
Expand Down
12 changes: 6 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ flowchart TD

subgraph Release["Phase 3 Β· Release & Observability"]
Airbag -->|100% green checks| P6["P6: Release Manager\nGit Push & Open PR Walkthrough"]
P6 --> Gate35{"πŸ”’ GATE 3.5 : Auto-Merge Lock\nHuman Authorization Required\n('Tu peux merger')"}
P6 --> Gate35{"πŸ”’ GATE 3.5 : Auto-Merge Lock\nHuman Authorization Required\n('Tu peux merger' / 'Approve merge')"}
Gate35 -->|Written confirmation| Merge["Squash Merge & Dual-Sync\nKanban Closure & Branch Pruning"]
end

Expand Down Expand Up @@ -92,15 +92,15 @@ flowchart TD
- Activated after all Quality Airbag assertions pass 100% green.
- Persona 6 pushes the branch and opens a Pull Request with a structured Walkthrough via `GitHubMCP:create_pull_request`.
- PR targets `epic/**` with `skip-release` for intermediate child tasks; targets `main` for standalone or consolidated Epic releases.
- **Gate 3.5 β€” Zero Auto-Merge Lock**: Strict halt with PR link. Merges exclusively after explicit user confirmation (*"Tu peux merger"*).
- **Gate 3.5 β€” Zero Auto-Merge Lock**: Strict halt with PR link. Merges exclusively after explicit user confirmation (*"Tu peux merger"* / *"Approve merge"*).
- Executes squash merge, runs `.agent/hooks/post-merge-dual-sync.sh` (dynamically synchronizing base branch), and prunes branches.

### Conditional Persona 2 Design Gate

Pillar 1 (Design Spec) enforcement is contextual and adaptive:
* **UI & Composable Changes**: **Persona 2 (Design Lead)** must define Material 3 tokens (from [`DESIGN.md`](DESIGN.md)), 4-state UI matrix (`Loading`, `Empty`, `Error`, `Content`), and Roborazzi expectations.
* **Non-UI Changes**: For backend, Room, CI/CD, scripts, or doc chores, Pillar 1 is explicitly marked: `N/A β€” No visual/UI changes`.
* **Explicit User Override**: If prompt requests to skip design (*"skip design"*, *"sans design"*), P2 is immediately bypassed.
* **Explicit User Override**: If prompt requests to skip design (*"skip design"* / *"sans design"*), P2 is immediately bypassed.

---

Expand All @@ -126,7 +126,7 @@ The kernel divides operational responsibilities into 6 distinct personas. To gua
3. **Rule A (Append-Only Immutability)**: Issue title and body are permanently read-only once created. Revisions are appended exclusively via comments.
4. **WIP = 1 (Single Active Pair)**: Exactly 1 issue `In Progress` and at most 1 PR `In Review` at any time (Macro Epic in progress, Micro child WIP = 1).
5. **Dual-Write Pattern**: Canonical remote GitHub issue synchronized with local `implementation_plan.md` mirror for IDE harmony.
6. **Zero Auto-Merge Lock (Gate 3.5)**: The agent never merges autonomously without human confirmation (*"Tu peux merger"*).
6. **Zero Auto-Merge Lock (Gate 3.5)**: The agent never merges autonomously without human confirmation (*"Tu peux merger"* / *"Approve merge"*).
7. **Runtime Bypass Guardrails**:
* **`branch-guard.mjs`**: PreToolUse hook intercepting file write tools (`write_to_file`, `replace_file_content`), blocking edits on protected branches (`main`, `epic/**`).
* **`plan-guard.mjs`**: PreToolUse hook intercepting shell executions, blocking `--no-verify`, inline hooks overrides (`-c core.hooksPath`), and direct git pushes to protected branches.
Expand Down Expand Up @@ -239,7 +239,7 @@ All repository-specific parameters are centralized in `kernel.config.json`, form
### 1. Configure the Kernel
Clone the repository and adapt `kernel.config.json` with your project coordinates:
```bash
git clone https://github.com/<owner>/agentic-android-delivery-kernel.git my-app
git clone https://github.com/nicolasvd/agentic-android-delivery-kernel.git my-app
cd my-app
# Customize kernel.config.json
```
Expand Down Expand Up @@ -269,7 +269,7 @@ When initiating a feature or bug fix:
2. **Gate 1.4 Halt**: The agent halts and awaits your explicit approval.
3. **JIT Sealing**: Once approved, the issue is sealed automatically on GitHub with Projects v2 fields.
4. **Delivery & Testing**: Persona 5 implements code on `<type>/issue-<id>-<slug>` and verifies tests green.
5. **Gate 3.5 Release**: Persona 6 opens the PR and awaits your explicit command (*"Tu peux merger"*).
5. **Gate 3.5 Release**: Persona 6 opens the PR and awaits your explicit command (*"Tu peux merger"* / *"Approve merge"*).

### πŸ“‹ Live Kanban Delivery Board (GitHub Projects v2)

Expand Down
Loading