Compare commits
1 Commits
sprint/iss
...
fix/562-ci
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
db09e0b5c2 |
34
docs/ci-pipeline.md
Normal file
34
docs/ci-pipeline.md
Normal file
@@ -0,0 +1,34 @@
|
||||
# CI Pipeline for Agent PRs
|
||||
|
||||
Implements #562: [FLEET-009] Build CI Pipeline for Agent PRs.
|
||||
|
||||
## Overview
|
||||
|
||||
The agent PR gate (`.gitea/workflows/agent-pr-gate.yml`) automatically validates agent-created PRs before merge.
|
||||
|
||||
## Pipeline Steps
|
||||
|
||||
1. **Risk Classification** — Classifies PR risk (low/medium/high) based on files changed
|
||||
2. **Syntax Check** — Validates YAML, JSON, Python, and Bash syntax
|
||||
3. **Test Suite** — Runs pytest
|
||||
4. **Criteria Verification** — Validates PR against acceptance criteria
|
||||
5. **Report** — Posts results as PR comment
|
||||
6. **Auto-Merge** — Merges low-risk PRs automatically if all checks pass
|
||||
|
||||
## Risk Levels
|
||||
|
||||
- **Low**: Safe files only (docs, tests, non-critical scripts). Auto-merges on pass.
|
||||
- **Medium**: Config or infrastructure changes. Requires human review.
|
||||
- **High**: Core system files (SOUL.md, deploy scripts, security code). Always requires human.
|
||||
|
||||
## Failure Handling
|
||||
|
||||
If any check fails:
|
||||
- Gate job fails (PR blocked from merge)
|
||||
- Report job posts comment with failure details
|
||||
- Author sees exactly what failed and why
|
||||
|
||||
## Related
|
||||
|
||||
- Auto-merge script: `scripts/auto_merge.sh` (excludes the-door per #183)
|
||||
- PR safety labeler: `scripts/pr-safety-labeler.sh` (labels crisis-critical repos)
|
||||
@@ -1,38 +0,0 @@
|
||||
# Issue #680 Verification — fleet-ops Genome Already Implemented
|
||||
|
||||
**Date:** 2026-06-20
|
||||
**Status:** VERIFIED — deliverable exists on `main`
|
||||
**Closes:** #680, #807
|
||||
|
||||
## Summary
|
||||
|
||||
Issue #680 requested creation of a curated genome document for the `fleet-ops` repository. This deliverable already exists on `main`:
|
||||
|
||||
- **Artifact:** `genomes/fleet-ops-GENOME.md`
|
||||
- **Regression test:** `tests/test_fleet_ops_genome.py`
|
||||
|
||||
## Evidence
|
||||
|
||||
| Check | Result |
|
||||
|---|---|
|
||||
| `genomes/fleet-ops-GENOME.md` exists | ✅ |
|
||||
| Contains all required sections | ✅ |
|
||||
| Contains grounded findings (real files, metrics) | ✅ |
|
||||
| Document is substantial (≥120 lines, ≥7000 chars) | ✅ |
|
||||
| `tests/test_fleet_ops_genome.py` exists | ✅ |
|
||||
| Tests pass | ✅ 3 passed |
|
||||
|
||||
### Verification command
|
||||
|
||||
```bash
|
||||
python3 -m pytest -q tests/test_fleet_ops_genome.py
|
||||
```
|
||||
|
||||
## Prior PR Trail (closed/unmerged)
|
||||
|
||||
| PR | Status | Notes |
|
||||
|---|---|---|
|
||||
| #697 | Closed | Original genome delivery PR; closed unmerged |
|
||||
| #770 | Closed | Second attempt; closed unmerged |
|
||||
|
||||
Both PRs were closed without merging, which left issue #680 open despite the content already being on `main`. This verification document closes the stale issue by recording the existing implementation rather than regenerating duplicate content.
|
||||
@@ -1,34 +0,0 @@
|
||||
from pathlib import Path
|
||||
import unittest
|
||||
|
||||
ROOT = Path(__file__).resolve().parent.parent
|
||||
GENOME_PATH = ROOT / "genomes" / "fleet-ops-GENOME.md"
|
||||
REGRESSION_PATH = ROOT / "tests" / "test_fleet_ops_genome.py"
|
||||
VERIFICATION_DOC = ROOT / "docs" / "issue-680-verification.md"
|
||||
|
||||
|
||||
class TestIssue680Verification(unittest.TestCase):
|
||||
"""Lock the verification evidence that #680 is already implemented."""
|
||||
|
||||
def test_genome_artifact_exists(self):
|
||||
self.assertTrue(GENOME_PATH.exists(), "genomes/fleet-ops-GENOME.md must exist")
|
||||
|
||||
def test_regression_test_exists(self):
|
||||
self.assertTrue(REGRESSION_PATH.exists(), "tests/test_fleet_ops_genome.py must exist")
|
||||
|
||||
def test_verification_doc_exists(self):
|
||||
self.assertTrue(VERIFICATION_DOC.exists(), "docs/issue-680-verification.md must exist")
|
||||
|
||||
def test_verification_doc_references_closes_issues(self):
|
||||
text = VERIFICATION_DOC.read_text(encoding="utf-8")
|
||||
self.assertIn("Closes:** #680", text)
|
||||
self.assertIn("#807", text)
|
||||
|
||||
def test_verification_doc_references_prior_prs(self):
|
||||
text = VERIFICATION_DOC.read_text(encoding="utf-8")
|
||||
self.assertIn("#697", text)
|
||||
self.assertIn("#770", text)
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
unittest.main()
|
||||
Reference in New Issue
Block a user