Fix 6.3 blog: correct AiService hrInsight() to use ApiResult<T> wrapper and map(r => r.value)

Author: workcontrolgit

blogs/series-6-ai-app-features/6.3-angular-ai-chat-widget.md

@@ -89,8 +89,15 @@ The AI endpoints don't fit the CRUD pattern in `BaseApiService<T>`. Create a foc
 import { Injectable, inject } from '@angular/core';
 import { HttpClient } from '@angular/common/http';
 import { Observable } from 'rxjs';
+import { map } from 'rxjs/operators';
 import { environment } from '../../../environments/environment';
 
+interface ApiResult<T> {
+  isSuccess: boolean;
+  value: T;
+  errors: string[];
+}
+
 export interface AiChatResponse {
   reply: string;
 }
@@ -133,7 +140,9 @@ export class AiService {
   }
 
   hrInsight(question: string): Observable<HrInsightResponse> {
-    return this.http.post<HrInsightResponse>(`${this.apiUrl}/ai/hr-insight`, { question });
+    return this.http.post<ApiResult<HrInsightResponse>>(`${this.apiUrl}/ai/hr-insight`, {
+      question,
+    }).pipe(map(r => r.value));
   }
 
   nlEmployeeSearch(query: string): Observable<NlEmployeeFilter> {
@@ -152,6 +161,7 @@ export class AiService {
 
 * **Does not extend `BaseApiService<T>`** — The AI endpoints are not CRUD endpoints. A focused service with four methods is self-documenting.
 * **Four methods, four endpoints** — `chat`, `hrInsight`, `nlEmployeeSearch`, `semanticPositionSearch`. Each article in this series introduces one method; the full service is shown here for reference.
+* **`hrInsight` unwraps `ApiResult<T>`** — The `POST /ai/hr-insight` endpoint returns the standard `Result<HrInsightDto>` envelope (`{ isSuccess, value, errors }`) used by all MediatR-dispatched endpoints. The private `ApiResult<T>` interface models this shape, and `.pipe(map(r => r.value))` unwraps it so components receive a plain `HrInsightResponse`. The `chat` and `nlEmployeeSearch` endpoints return their payloads directly (no envelope), so no unwrapping is needed for those.
 * **`inject()` not constructor injection** — Consistent with every other service in this codebase.
 
 **Add to the barrel export** in `src/app/services/api/index.ts`: