fix(mapping): corriger AgentExtensions.ToResponse — ajouter GPU fields manquants #172

Merged

nonobis merged 2 commits from fix/agent-extensions-mapping into main 2026-08-27 15:02:18 +02:00

Conversation 5 Commits 2 Files changed 7 +44 -72

nonobis commented 2026-08-27 14:23:55 +02:00

Owner

Copy link

Contexte

La CI est cassée sur main à cause de 20 erreurs CS1503 dans AgentExtensions.cs. Le constructeur AgentResponse a été étendu avec les champs GPU (GpuName, VramMb, GpuLoadPercent, VramUsageBytes) mais le mapping ToResponse() ne les passait pas.

Cela bloque TOUTES les PRs en attente (#163, etc.).

Changements

• AgentExtensions.cs : ajout des 4 arguments manquants (GpuName, VramMb, GpuLoadPercent, VramUsageBytes) dans l'appel au constructeur AgentResponse.

Tests

• Build : ✅ (attendu — les erreurs CS1503 disparaissent)
• Aucun changement fonctionnel, uniquement un alignement du mapping avec le DTO existant.

## Contexte La CI est cassée sur main à cause de 20 erreurs CS1503 dans `AgentExtensions.cs`. Le constructeur `AgentResponse` a été étendu avec les champs GPU (`GpuName`, `VramMb`, `GpuLoadPercent`, `VramUsageBytes`) mais le mapping `ToResponse()` ne les passait pas. Cela bloque TOUTES les PRs en attente (#163, etc.). ## Changements - `AgentExtensions.cs` : ajout des 4 arguments manquants (`GpuName`, `VramMb`, `GpuLoadPercent`, `VramUsageBytes`) dans l'appel au constructeur `AgentResponse`. ## Tests - Build : ✅ (attendu — les erreurs CS1503 disparaissent) - Aucun changement fonctionnel, uniquement un alignement du mapping avec le DTO existant.

nonobis added 1 commit 2026-08-27 14:23:55 +02:00

fix(mapping): corriger AgentExtensions.ToResponse — ajouter GPU fields manquants ... 594bd74ba7

Le constructeur AgentResponse attend GpuName, VramMb, GpuLoadPercent,
VramUsageBytes après GpuType, mais le mapping ne les passait pas.
Cela causait 20 erreurs CS1503 et bloquait toute la CI.

Fixes #100

nonobis commented 2026-08-27 14:29:03 +02:00

Author

Owner

Copy link

🤖 Coding Orchestrator — 2026-08-27 12:29 UTC

Changement détecté: code

auto-fix-trivial

• dotnet format OK
  ⏭️ À vérifier

review-gate

• OK
  ⏭️ À vérifier

---

Ce commentaire est généré automatiquement par le coding orchestrator.

## 🤖 Coding Orchestrator — 2026-08-27 12:29 UTC **Changement détecté:** code ### auto-fix-trivial - dotnet format OK **⏭️ À vérifier** ### review-gate - OK **⏭️ À vérifier** --- _Ce commentaire est généré automatiquement par le coding orchestrator._

forgejo-actions commented 2026-08-27 14:33:03 +02:00

Copy link

🤖 Résumé des tests

Tests unitaires

Pas de résultats (job skippé)

---

Tests Web (bUnit)

Skippé (aucun changement IHM)

---

Tests DB (Infrastructure)

Pas de résultats

Généré automatiquement par la CI.

## 🤖 Résumé des tests ## Tests unitaires _Pas de résultats (job skippé)_ --- ## Tests Web (bUnit) _Skippé (aucun changement IHM)_ --- ## Tests DB (Infrastructure) _Pas de résultats_ _Généré automatiquement par la CI._

nonobis force-pushed fix/agent-extensions-mapping from 594bd74ba7 to bc15598eb4 2026-08-27 14:49:15 +02:00 Compare

forgejo-actions commented 2026-08-27 14:52:38 +02:00

Copy link

🤖 Résumé des tests

Tests unitaires

Pas de résultats (job skippé)

---

Tests Web (bUnit)

Skippé (aucun changement IHM)

---

Tests DB (Infrastructure)

Pas de résultats

Généré automatiquement par la CI.

## 🤖 Résumé des tests ## Tests unitaires _Pas de résultats (job skippé)_ --- ## Tests Web (bUnit) _Skippé (aucun changement IHM)_ --- ## Tests DB (Infrastructure) _Pas de résultats_ _Généré automatiquement par la CI._

forgejo-actions commented 2026-08-27 14:53:38 +02:00

Copy link

🤖 Résumé des tests

Tests unitaires

Pas de résultats (job skippé)

---

Tests Web (bUnit)

Skippé (aucun changement IHM)

---

Tests DB (Infrastructure)

Pas de résultats

Généré automatiquement par la CI.

## 🤖 Résumé des tests ## Tests unitaires _Pas de résultats (job skippé)_ --- ## Tests Web (bUnit) _Skippé (aucun changement IHM)_ --- ## Tests DB (Infrastructure) _Pas de résultats_ _Généré automatiquement par la CI._

nonobis added 1 commit 2026-08-27 14:54:28 +02:00

fix(build): résoudre erreurs pré-existantes Agent — 0 errors, 356 tests OK ... a343c94129

Corrections multiples sur le projet Agent (erreurs pré-existantes sur main) :

- AgentExtensions.cs : ajouter GPU fields manquants (GpuName, VramMb, GpuLoadPercent, VramUsageBytes) au mapping AgentResponse
- GpuInventoryProvider.cs : renommer GpuInfo → GpuDeviceInfo (conflit de définition dans même namespace avec GpuDetectorService.GpuInfo)
- SnapshotPipelineService.cs : ajouter DegradationManager/Metrics aux options, supprimer code dupliqué (doublon de vérification dégradation), corriger appel AnalyzeFrameAsync (paramètre nommé ct:)
- AgentRegistrationService.cs : retirer paramètres obsolètes (HasGpu, CpuCores, MemoryMb, GpuType...) du HeartbeatRequest ( ces champs ne font pas partie du DTO heartbeat)
- Tests : aligner HeartbeatRequestTests, GpuInventoryProviderDetailedTests, SnapshotPipelineServiceTests

Fixes #100

nonobis merged commit be05bdabcf into main 2026-08-27 15:02:18 +02:00

nonobis deleted branch fix/agent-extensions-mapping 2026-08-27 15:02:18 +02:00

forgejo-actions commented 2026-08-27 15:19:23 +02:00

Copy link

🤖 Résumé des tests

Tests unitaires

Projet	Statut	Passed	Failed	Skipped	Total
Heimdall.Agent.Tests	🟢	356	0	0	356
Heimdall.Api.Tests	🟢	2	0	0	2
Heimdall.Application.Tests	🟢	468	0	0	468
Heimdall.Domain.Tests	🟢	179	0	0	179

✅ 1005 tests passent.

---

Tests Web (bUnit)

Skippé (aucun changement IHM)

---

Tests DB (Infrastructure)

Pas de résultats

Généré automatiquement par la CI.

## 🤖 Résumé des tests ## Tests unitaires | Projet | Statut | Passed | Failed | Skipped | Total | |---|---|---|---|---|---| | `Heimdall.Agent.Tests` | 🟢 | 356 | 0 | 0 | 356 | | `Heimdall.Api.Tests` | 🟢 | 2 | 0 | 0 | 2 | | `Heimdall.Application.Tests` | 🟢 | 468 | 0 | 0 | 468 | | `Heimdall.Domain.Tests` | 🟢 | 179 | 0 | 0 | 179 | **✅ 1005 tests passent.** --- ## Tests Web (bUnit) _Skippé (aucun changement IHM)_ --- ## Tests DB (Infrastructure) _Pas de résultats_ _Généré automatiquement par la CI._

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

bug

Comportement incorrect ou risque fonctionnel

  

chore

Maintenance, dépendances, CI, outillage

  

design

Ticket avec composante UI — maquette the-designer requise/intégrée

  

dx:ci

  

dx:database

  

dx:dotnet

  

dx:infra

  

dx:security

  

dx:tests

  

dx:ui

  

etape-1

Étape 1 — MVP Multi-Site (phases 1-12)

  

etape-2

Étape 2 — Caméras + Temps réel (phases 13-14)

  

etape-3

Étape 3 — Vidéo + IA (phases 15-16)

  

etape-4

Étape 4 — Événements + Notifications (phases 17-19)

  

etape-5

Étape 5 — UI Blazor complète (phases 20-23)

  

etape-6

Étape 6 — Production Hardening (phase 24)

  

etape-7

Étape 7 — Mobile (phase 25)

  

feature

Nouvelle fonctionnalité / implémentation spec manquante

  

priority-high

Priorité haute — bloque le cœur métier

  

priority-low

Priorité faible / cosmétique

  

priority-medium

Priorité moyenne

  

security

Vulnérabilités CVE, scans Trivy/OWASP, security hardening

  

spec

Référence à MASTER_SPECIFICATION.md

  

tech-debt

Dette technique, obsolescence, drift documentation

No labels

bug

chore

design

dx:ci

dx:database

dx:dotnet

dx:infra

dx:security

dx:tests

dx:ui

etape-1

etape-2

etape-3

etape-4

etape-5

etape-6

etape-7

feature

priority-high

priority-low

priority-medium

security

spec

tech-debt

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

nonobis/Heimdall!172

No description provided.