**Changements majeurs** : 1. **Suppression ADR-010 (Commandes volant et likes)** : - Contenu consolidé dans Règle 05 (section 5.3) - Raison : ADR-010 était du métier déguisé en architecture - Section "Implémentation Technique" ajoutée à Règle 05 - Pattern correct (addition) vs incorrect (multiplication) 2. **Déplacement ADR-011 → Compliance** : - `docs/adr/011-conformite-stores.md` → `docs/compliance/stores-submission.md` - Raison : Nature opérationnelle/légale, pas architecture technique - Nouveau dossier `/docs/compliance/` créé 3. **Renumérotation ADR (010-022)** : - Combler les trous de numérotation (010 et 011) - ADR-012→010, ADR-013→011, ..., ADR-024→022 - 22 ADR numérotés en continu (001-022) - Historique Git préservé (git mv) 4. **Mise à jour références** : - Règle 03 : ADR-010 → Règle 05 (section 5.3) - Règle 09 : ADR-010 → Règle 05 (section 5.3) - INCONSISTENCIES-ANALYSIS.md : toutes références mises à jour - Incohérence #15 annulée (faux problème : modes séparés) **Résultat** : - ✅ Séparation claire ADR (technique) vs Règles métier (fonctionnel) - ✅ Documentation compliance séparée - ✅ Numérotation ADR continue sans trous - ✅ Single Source of Truth (pas de redondance) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
12 KiB
ADR-022 : Stratégie CI/CD avec Path Filters pour Monorepo
Statut : Accepté (non implémenté) Date : 2026-02-01
Contexte
RoadWave est organisé en monorepo contenant backend Go, mobile Flutter, documentation et features BDD (ADR-014). Sans optimisation, chaque commit déclencherait tous les builds (backend + mobile + docs), même si seul un composant a changé.
Problématique :
- ❌ Temps de CI/CD inutilement longs (rebuild complet ~15 min)
- ❌ Gaspillage de ressources GitHub Actions
- ❌ Ralentissement du feedback développeur
- ❌ Coûts CI/CD élevés pour changements isolés
Exemple : Modification d'un fichier markdown dans /docs déclenche les tests backend (5 min) + tests mobile (8 min) + build (2 min) alors que seule la validation docs (~30s) est nécessaire.
Décision
Workflows GitHub Actions séparés avec path filters pour rebuild sélectif basé sur les fichiers modifiés.
Alternatives considérées
| Option | Optimisation | Complexité | Maintenance | Coût CI |
|---|---|---|---|---|
| Workflows séparés + path filters | Excellente (~70%) | Faible | Simple | Minimal |
| Workflow monolithique | Nulle | Très faible | Simple | Élevé |
| Turborepo/Nx orchestration | Bonne (~50%) | Élevée | Complexe | Moyen |
| Multirepo (repos séparés) | Excellente | Élevée | Complexe | Variable |
Architecture
Structure Workflows
.github/workflows/
├── backend.yml # Backend Go (tests, lint, build)
├── mobile.yml # Mobile Flutter (tests, lint, build)
└── shared.yml # Docs + code partagé (validation, génération)
Configuration Path Filters
Workflow Backend (backend.yml)
name: Backend CI
on:
push:
branches: [main, develop]
paths:
- 'backend/**' # Code Go modifié
- 'features/api/**' # Tests API modifiés
- 'features/e2e/**' # Tests E2E (impliquent backend)
- '.github/workflows/backend.yml'
pull_request:
branches: [main, develop]
paths:
- 'backend/**'
- 'features/api/**'
- 'features/e2e/**'
jobs:
test-unit:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-go@v5
with:
go-version: '1.21'
- run: cd backend && go test ./...
test-integration:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-go@v5
with:
go-version: '1.21'
- run: cd backend && make test-integration
test-bdd:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-go@v5
with:
go-version: '1.21'
- run: |
go install github.com/cucumber/godog/cmd/godog@latest
godog run features/api/ features/e2e/
lint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: golangci/golangci-lint-action@v4
build:
runs-on: ubuntu-latest
needs: [test-unit, test-integration, test-bdd, lint]
steps:
- uses: actions/checkout@v4
- uses: actions/setup-go@v5
with:
go-version: '1.21'
- run: cd backend && make build
Déclenché par :
- Modifications dans
/backend(code Go, migrations, config) - Nouvelles features API dans
/features/api - Tests end-to-end dans
/features/e2e(backend impliqué)
Workflow Mobile (mobile.yml)
name: Mobile CI
on:
push:
branches: [main, develop]
paths:
- 'mobile/**' # Code Flutter modifié
- 'features/ui/**' # Tests UI modifiés
- 'features/e2e/**' # Tests E2E (impliquent mobile)
- '.github/workflows/mobile.yml'
pull_request:
branches: [main, develop]
paths:
- 'mobile/**'
- 'features/ui/**'
- 'features/e2e/**'
jobs:
test-unit:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: subosito/flutter-action@v2
with:
flutter-version: '3.16.0'
- run: cd mobile && flutter test
test-integration:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: subosito/flutter-action@v2
with:
flutter-version: '3.16.0'
- run: cd mobile && flutter test integration_test/
lint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: subosito/flutter-action@v2
with:
flutter-version: '3.16.0'
- run: cd mobile && flutter analyze
build-android:
runs-on: ubuntu-latest
needs: [test-unit, test-integration, lint]
steps:
- uses: actions/checkout@v4
- uses: subosito/flutter-action@v2
with:
flutter-version: '3.16.0'
- run: cd mobile && flutter build apk --release
build-ios:
runs-on: macos-latest
needs: [test-unit, test-integration, lint]
steps:
- uses: actions/checkout@v4
- uses: subosito/flutter-action@v2
with:
flutter-version: '3.16.0'
- run: cd mobile && flutter build ios --release --no-codesign
Déclenché par :
- Modifications dans
/mobile(code Flutter/Dart, assets, config) - Nouvelles features UI dans
/features/ui - Tests end-to-end dans
/features/e2e(mobile impliqué)
Workflow Shared (shared.yml)
name: Shared CI
on:
push:
branches: [main, develop]
paths:
- 'docs/**' # Documentation modifiée
- 'shared/**' # Code partagé modifié
- '.github/workflows/shared.yml'
pull_request:
branches: [main, develop]
paths:
- 'docs/**'
- 'shared/**'
jobs:
docs-validation:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-python@v5
with:
python-version: '3.11'
- run: |
pip install mkdocs mkdocs-material
mkdocs build --strict
docs-links:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: lycheeverse/lychee-action@v1
with:
args: 'docs/**/*.md'
shared-tests:
runs-on: ubuntu-latest
if: contains(github.event.head_commit.modified, 'shared/')
steps:
- uses: actions/checkout@v4
# Tests pour code partagé si nécessaire
Déclenché par :
- Modifications dans
/docs(ADR, règles métier, documentation technique) - Modifications dans
/shared(contrats API, types partagés)
Matrice de Déclenchement
| Modification | Backend | Mobile | Shared | Temps total |
|---|---|---|---|---|
backend/internal/auth/service.go |
✅ | ❌ | ❌ | ~5 min |
mobile/lib/screens/home.dart |
❌ | ✅ | ❌ | ~8 min |
features/api/authentication/*.feature |
✅ | ❌ | ❌ | ~5 min |
features/ui/audio-guides/*.feature |
❌ | ✅ | ❌ | ~8 min |
features/e2e/abonnements/*.feature |
✅ | ✅ | ❌ | ~13 min (parallèle) |
docs/adr/016-email.md |
❌ | ❌ | ✅ | ~30s |
| Commit mixte (backend + mobile + docs) | ✅ | ✅ | ✅ | ~13 min (parallèle) |
Économie de temps :
- Commit backend-only : ~5 min (vs 15 min sans path filters) = 67% plus rapide
- Commit docs-only : ~30s (vs 15 min) = 97% plus rapide
Cas Particulier : Features E2E
Les tests end-to-end dans /features/e2e/ déclenchent les deux workflows (backend ET mobile) car ils testent l'intégration complète :
Exemples de features E2E :
features/e2e/abonnements/: Formulaire mobile → API Zitadel → API RoadWave → Mangopay → Confirmation UIfeatures/e2e/error-handling/: Perte réseau → Fallback mode offline → Reprise auto après reconnexion
Justification : Les features E2E valident le contrat backend-mobile, donc les deux doivent être testés.
Justification
Avantages
✅ Rebuild sélectif : changement backend → seulement backend rebuild (~5 min au lieu de 15 min)
✅ Économie de temps CI : ~70% de réduction sur commits isolés (backend-only ou mobile-only)
✅ Économie de coûts : GitHub Actions facturé à la minute → réduction proportionnelle des coûts
✅ Parallélisation : workflows indépendants exécutés en parallèle par GitHub Actions
✅ Features E2E : déclenchent backend ET mobile → cohérence end-to-end garantie
✅ Feedback rapide : développeur backend voit uniquement résultats backend (moins de bruit)
✅ Scalabilité : ajout de nouveaux composants (admin panel, worker, etc.) = nouveau workflow isolé
Inconvénients
❌ Duplication config : certains éléments (checkout, cache) dupliqués entre workflows
❌ Maintenance : 3 workflows à maintenir au lieu de 1
❌ Complexité initiale : setup plus complexe que workflow monolithique
Mitigation :
- Utiliser des composite actions pour partager la config commune
- Documentation claire dans ce ADR
- Coût initial faible (~2h setup) vs gains à long terme importants
Implémentation
Prérequis
- ✅ Code backend implémenté avec tests unitaires + intégration
- ✅ Code mobile implémenté avec tests Flutter
- ✅ Step definitions BDD implémentées pour
api/,ui/,e2e/ - ✅ Catégorisation features BDD en
/features/{api,ui,e2e}(ADR-007)
Plan d'Implémentation
Phase 1 : Setup workflows de base (~1h)
- Créer
backend.ymlavec jobs test + lint + build - Créer
mobile.ymlavec jobs test + lint + build - Créer
shared.ymlavec validation docs
Phase 2 : Configuration path filters (~30 min)
- Ajouter
paths:à chaque workflow - Tester avec commits isolés (backend-only, mobile-only, docs-only)
Phase 3 : Optimisations (~30 min)
- Ajouter caching (Go modules, Flutter dependencies, node_modules)
- Créer composite actions pour config partagée
- Ajouter badges status dans README
Effort total estimé : ~2h
Validation
# Test 1 : Commit backend-only
git add backend/
git commit -m "test: backend change"
git push
# → Vérifier que SEULEMENT backend.yml s'exécute
# Test 2 : Commit mobile-only
git add mobile/
git commit -m "test: mobile change"
git push
# → Vérifier que SEULEMENT mobile.yml s'exécute
# Test 3 : Commit E2E
git add features/e2e/
git commit -m "test: e2e change"
git push
# → Vérifier que backend.yml ET mobile.yml s'exécutent
# Test 4 : Commit docs-only
git add docs/
git commit -m "docs: update ADR"
git push
# → Vérifier que SEULEMENT shared.yml s'exécute
Conséquences
Positives
- Productivité développeur : feedback 3x plus rapide sur commits isolés
- Coûts réduits : ~70% d'économie sur minutes GitHub Actions
- Meilleure expérience PR : reviews plus rapides, moins d'attente CI
- Scalabilité : facile d'ajouter nouveaux composants (worker, admin, etc.)
Négatives
- Maintenance : 3 workflows à maintenir vs 1
- Setup initial : ~2h vs ~30 min pour workflow simple
- Complexité : nécessite compréhension path filters GitHub Actions
Risques
⚠️ Faux négatifs : path filter mal configuré → test non exécuté → bug en production
Mitigation :
- Features E2E déclenchent toujours backend + mobile (safety net)
- Tests de validation dans le plan d'implémentation
- Review obligatoire des modifications de workflows
Références
- ADR-007 - Tests BDD et Catégorisation Features
- ADR-014 - Organisation Monorepo
- GitHub Actions - Path Filters
- Monorepo CI/CD Best Practices
Statut d'Implémentation
Actuel : Documenté mais non implémenté
Quand : Lors du Sprint d'implémentation backend/mobile (code production)
Pourquoi reporté : Le projet est actuellement en phase de documentation uniquement. Aucun code backend/mobile n'est implémenté, donc pas de tests à exécuter. L'implémentation CI/CD sera faite lors du développement.
Prochaines étapes :
- Implémenter code backend avec tests
- Implémenter code mobile avec tests
- Implémenter step definitions BDD
- Créer workflows CI/CD selon ce ADR
- Valider avec commits de test