From 380d73a535570bfeee9ec6d78a4b4e79c63be52b Mon Sep 17 00:00:00 2001
From: master <>
Date: Sun, 29 Mar 2026 13:39:12 +0300
Subject: [PATCH] =?UTF-8?q?Map=20policy=20API=20response=20fields=20to=20U?=
 =?UTF-8?q?I=20model=20(packId=E2=86=92id,=20displayName=E2=86=92name)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The backend returns PolicyPackSummaryDto with fields: packId,
displayName, createdAt, versions. The UI model expects: id, name,
description, version, status, etc.

Added mapPackSummary() to translate API fields to UI model fields.
The sortPacks filter was removing all packs because p.id was undefined
(the API field is packId, not id).

Also maps createPack response and sends displayName in the POST body
to match the backend's expected field name.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 .../services/policy-api.service.ts            | 31 +++++++++++++++++--
 1 file changed, 29 insertions(+), 2 deletions(-)

diff --git a/src/Web/StellaOps.Web/src/app/features/policy-studio/services/policy-api.service.ts b/src/Web/StellaOps.Web/src/app/features/policy-studio/services/policy-api.service.ts
index 2357c06cf..04feca9cb 100644
--- a/src/Web/StellaOps.Web/src/app/features/policy-studio/services/policy-api.service.ts
+++ b/src/Web/StellaOps.Web/src/app/features/policy-studio/services/policy-api.service.ts
@@ -14,6 +14,7 @@
 import { Injectable, inject } from '@angular/core';
 import { HttpClient, HttpParams } from '@angular/common/http';
 import { Observable } from 'rxjs';
+import { map } from 'rxjs/operators';
 
 import type {
   PolicyPackSummary,
@@ -68,7 +69,24 @@ export class PolicyApiService {
     if (params?.limit) httpParams = httpParams.set('limit', params.limit.toString());
     if (params?.offset) httpParams = httpParams.set('offset', params.offset.toString());
 
-    return this.http.get<PolicyPackSummary[]>(`${API_BASE}/packs`, { params: httpParams });
+    return this.http.get<any[]>(`${API_BASE}/packs`, { params: httpParams }).pipe(
+      map((items) => (items ?? []).map((item) => this.mapPackSummary(item)))
+    );
+  }
+
+  private mapPackSummary(raw: any): PolicyPackSummary {
+    return {
+      id: raw.packId ?? raw.id ?? '',
+      name: raw.displayName ?? raw.name ?? raw.packId ?? 'Unnamed Pack',
+      description: raw.description ?? '',
+      version: raw.versions?.length > 0 ? `v${Math.max(...raw.versions)}` : 'latest',
+      status: raw.status ?? 'active',
+      createdAt: raw.createdAt ?? '',
+      modifiedAt: raw.modifiedAt ?? raw.createdAt ?? '',
+      createdBy: raw.createdBy ?? '',
+      modifiedBy: raw.modifiedBy ?? raw.createdBy ?? '',
+      tags: raw.tags ?? [],
+    };
   }
 
   /**
@@ -95,7 +113,16 @@ export class PolicyApiService {
     content: string;
     tags?: string[];
   }): Observable<PolicyPack> {
-    return this.http.post<PolicyPack>(`${API_BASE}/packs`, pack);
+    return this.http.post<any>(`${API_BASE}/packs`, {
+      displayName: pack.name,
+      ...pack,
+    }).pipe(
+      map((raw) => ({
+        ...raw,
+        id: raw?.packId ?? raw?.id ?? '',
+        name: raw?.displayName ?? raw?.name ?? pack.name,
+      }))
+    );
   }
 
   /**