Changeset a6e33d1

Timestamp:

07/02/26 23:35:59 (3 days ago)

Author:

kikisrbinoska <srbinoskakristina07@…>

Branches:

main

Parents:

3ae4bab

Message:

AI suggestions fixed

Files:

• ChapterX.API/Controllers/AISuggestionsController.cs (modified) (2 diffs)
• ChapterX.Application/AISuggestion/Queries/GetByChapterHandler.cs (added)
• ChapterX.Application/AISuggestion/Queries/GetByChapterRequest.cs (added)
• ChapterX.Application/AISuggestion/Queries/GetByChapterResponse.cs (added)
• ChapterX.Infrastructure/Repositories/AISuggestionRepository.cs (modified) (1 diff)
• chapterx-frontend/src/components/writer/AISuggestionPanel.tsx (modified) (2 diffs)
• chapterx-frontend/src/store/storyStore.ts (modified) (6 diffs)

ChapterX.API/Controllers/AISuggestionsController.cs

@@ -5 +5 @@
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.Extensions.Logging;
+using System.Linq;
 
 namespace ChapterX.API.Controllers
@@ -37 +38 @@
             var response = await _mediator.Send(new GetRequest(id));
             return Ok(response);
+        }
+
+        [HttpGet("chapter/{chapterId:int}")]
+        [AllowAnonymous]
+        public async Task<ActionResult> GetByChapter(int chapterId)
+        {
+            _logger.LogInformation("Fetching AI suggestions for ChapterId: {ChapterId}", chapterId);
+            var response = await _mediator.Send(new GetByChapterRequest(chapterId));
+            var result = response.AISuggestions.Select(s => new
+            {
+                id = s.Id,
+                originalText = s.OriginalText,
+                suggestedText = s.SuggestedText,
+                accepted = s.Accepted,
+                createdAt = s.CreatedAt,
+                appliedAt = s.AppliedAt,
+                storyId = s.StoryId,
+                suggestionTypes = s.SuggestionTypes.Select(t => t.SuggestionTypeValue),
+            });
+            return Ok(result);
         }
 

ChapterX.Infrastructure/Repositories/AISuggestionRepository.cs

@@ -21 +21 @@
         {
             return await _dbSet
-                .Where(a => a.StoryId == chapterId)
+                .Include(a => a.SuggestionTypes)
+                .Where(a => a.NeedApprovals.Any(na => na.ChapterId == chapterId))
                 .ToListAsync(cancellationToken);
         }

chapterx-frontend/src/components/writer/AISuggestionPanel.tsx

@@ -100 +100 @@
 }
 
-export const AISuggestionPanel: React.FC<AISuggestionPanelProps> = ({ chapterId, storyId }) => {
+export const AISuggestionPanel: React.FC<AISuggestionPanelProps> = ({ chapterId }) => {
   const { aiSuggestions, acceptSuggestion, rejectSuggestion, fetchSuggestions } = useStoryStore()
   const { addToast } = useUIStore()
@@ -108 +108 @@
   useEffect(() => {
     setLoading(true)
-    fetchSuggestions().finally(() => setLoading(false))
+    fetchSuggestions(chapterId).finally(() => setLoading(false))
   }, [chapterId])
 
-  // Filter by storyId if available (backend), else fall back to chapterId (mock)
-  const chapterSuggestions = aiSuggestions.filter(s =>
-    storyId ? (s as any).story_id === storyId : s.chapter_id === chapterId
-  )
+  // Backend already scopes results to this chapter via NEED_APPROVAL
+  const chapterSuggestions = aiSuggestions
   const pending = chapterSuggestions.filter(s => s.accepted === null)
   const accepted = chapterSuggestions.filter(s => s.accepted === true)

chapterx-frontend/src/store/storyStore.ts

@@ -7 +7 @@
   Collaboration,
   AISuggestion,
+  SuggestionType,
   Genre,
   ReadingList,
@@ -105 +106 @@
 
   // AI Suggestion actions
-  fetchSuggestions: () => Promise<void>
+  fetchSuggestions: (chapterId: number) => Promise<void>
   acceptSuggestion: (id: number) => Promise<void>
   rejectSuggestion: (id: number) => Promise<void>
@@ -434 +435 @@
   },
 
-  fetchSuggestions: async () => {
-    try {
-      const res = await axios.get(`${API}/aisuggestions`)
-      const data = res.data.aiSuggestions ?? res.data
+  fetchSuggestions: async (chapterId) => {
+    try {
+      const res = await axios.get(`${API}/aisuggestions/chapter/${chapterId}`)
+      const data: any[] = res.data ?? []
       const mapped: AISuggestion[] = data.map((s: any) => ({
         suggestion_id: s.id,
-        chapter_id: s.storyId,
+        chapter_id: chapterId,
         story_id: s.storyId,
         original_text: s.originalText,
         suggested_text: s.suggestedText,
-        suggestion_type: 'style' as const,
+        suggestion_type: (s.suggestionTypes?.[0] ?? 'style') as SuggestionType,
         accepted: s.accepted === true ? true : s.accepted === false ? false : null,
         applied_at: s.appliedAt ?? undefined,
@@ -471 +472 @@
         suggestedText: s.suggested_text,
         accepted: true,
-      })
+      }, { headers: getAuthHeaders() })
     } catch {
       // keep optimistic update even if backend fails
@@ -491 +492 @@
         suggestedText: s.suggested_text,
         accepted: false,
-      })
+      }, { headers: getAuthHeaders() })
     } catch {
       // keep optimistic update even if backend fails
@@ -502 +503 @@
     set(state => ({ aiSuggestions: [...state.aiSuggestions, { ...suggestion, suggestion_id: tempId }] }))
     try {
-      const res = await axios.post('${API}/aisuggestions', {
+      const res = await axios.post(`${API}/aisuggestions`, {
         originalText: suggestion.original_text,
         suggestedText: suggestion.suggested_text,
         storyId: suggestion.chapter_id,
-      })
+      }, { headers: getAuthHeaders() })
       const newId = res.data.id ?? tempId
       set(state => ({