fix: boucle infinie Maximum update depth dans useAutoTagging + toolbar
- use-auto-tagging: onQuotaExceeded via ref stable → n'invalide plus useCallback analyzeContent à chaque render parent - note-editor-context: filteredSuggestions et existingLabelsLower stabilisés avec useMemo (était recalculé sans memo → nouvelle ref à chaque render → état useMemo state se réexécutait → boucle) - deepseek.ts: generateTags via generateText (pas generateObject) pour éviter response_format:json_schema non supporté Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This commit is contained in:
@@ -211,11 +211,15 @@ export function NoteEditorProvider({ note, readOnly = false, fullPage = false, o
|
|||||||
const [comparisonNotes, setComparisonNotes] = useState<Array<Partial<Note>>>([])
|
const [comparisonNotes, setComparisonNotes] = useState<Array<Partial<Note>>>([])
|
||||||
const [fusionNotes, setFusionNotes] = useState<Array<Partial<Note>>>([])
|
const [fusionNotes, setFusionNotes] = useState<Array<Partial<Note>>>([])
|
||||||
|
|
||||||
const existingLabelsLower = (note.labels || []).map((l) => l.toLowerCase())
|
const existingLabelsLower = useMemo(
|
||||||
const filteredSuggestions = suggestions.filter(s => {
|
() => (note.labels || []).map((l) => l.toLowerCase()),
|
||||||
|
// eslint-disable-next-line react-hooks/exhaustive-deps
|
||||||
|
[JSON.stringify(note.labels)]
|
||||||
|
)
|
||||||
|
const filteredSuggestions = useMemo(() => suggestions.filter(s => {
|
||||||
if (!s || !s.tag) return false
|
if (!s || !s.tag) return false
|
||||||
return !dismissedTags.includes(s.tag) && !existingLabelsLower.includes(s.tag.toLowerCase())
|
return !dismissedTags.includes(s.tag) && !existingLabelsLower.includes(s.tag.toLowerCase())
|
||||||
})
|
}), [suggestions, dismissedTags, existingLabelsLower])
|
||||||
|
|
||||||
const colorClasses = NOTE_COLORS[color as NoteColor] || NOTE_COLORS.default
|
const colorClasses = NOTE_COLORS[color as NoteColor] || NOTE_COLORS.default
|
||||||
|
|
||||||
|
|||||||
@@ -25,6 +25,9 @@ export function useAutoTagging({ content, notebookId, enabled = true, onQuotaExc
|
|||||||
const previousNotebookId = useRef<string | null | undefined>(notebookId);
|
const previousNotebookId = useRef<string | null | undefined>(notebookId);
|
||||||
// AbortController for cancelling in-flight requests
|
// AbortController for cancelling in-flight requests
|
||||||
const abortRef = useRef<AbortController | null>(null);
|
const abortRef = useRef<AbortController | null>(null);
|
||||||
|
// Stable ref for onQuotaExceeded — avoids re-creating analyzeContent on every render
|
||||||
|
const onQuotaExceededRef = useRef(onQuotaExceeded);
|
||||||
|
onQuotaExceededRef.current = onQuotaExceeded;
|
||||||
|
|
||||||
const analyzeContent = useCallback(async (contentToAnalyze: string, currentNotebookId?: string | null, currentLanguage?: string) => {
|
const analyzeContent = useCallback(async (contentToAnalyze: string, currentNotebookId?: string | null, currentLanguage?: string) => {
|
||||||
if (!contentToAnalyze || contentToAnalyze.length < 10) {
|
if (!contentToAnalyze || contentToAnalyze.length < 10) {
|
||||||
@@ -63,8 +66,8 @@ export function useAutoTagging({ content, notebookId, enabled = true, onQuotaExc
|
|||||||
if (controller.signal.aborted) return;
|
if (controller.signal.aborted) return;
|
||||||
|
|
||||||
if (!response.ok) {
|
if (!response.ok) {
|
||||||
if (response.status === 402 && onQuotaExceeded) {
|
if (response.status === 402) {
|
||||||
onQuotaExceeded();
|
onQuotaExceededRef.current?.();
|
||||||
}
|
}
|
||||||
throw new Error('Error during analysis');
|
throw new Error('Error during analysis');
|
||||||
}
|
}
|
||||||
@@ -79,7 +82,7 @@ export function useAutoTagging({ content, notebookId, enabled = true, onQuotaExc
|
|||||||
setIsAnalyzing(false);
|
setIsAnalyzing(false);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}, [hasAiConsent, requestAiConsent, onQuotaExceeded]);
|
}, [hasAiConsent, requestAiConsent]); // onQuotaExceeded via ref — stable
|
||||||
|
|
||||||
// Trigger on content change
|
// Trigger on content change
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
|
|||||||
@@ -30,21 +30,23 @@ export class DeepSeekProvider implements AIProvider {
|
|||||||
this.embeddingModel = deepseek.embedding(embeddingModelName);
|
this.embeddingModel = deepseek.embedding(embeddingModelName);
|
||||||
}
|
}
|
||||||
|
|
||||||
async generateTags(content: string): Promise<TagSuggestion[]> {
|
async generateTags(content: string, language?: string): Promise<TagSuggestion[]> {
|
||||||
try {
|
try {
|
||||||
const { object } = await generateObject({
|
// DeepSeek doesn't support response_format: json_schema — use generateText + manual parse
|
||||||
|
const { text } = await aiGenerateText({
|
||||||
model: this.model,
|
model: this.model,
|
||||||
schema: z.object({
|
prompt: `Analyze the following note and suggest 1 to 5 relevant tags as a JSON array.
|
||||||
tags: z.array(z.object({
|
Return ONLY a JSON array like: [{"tag":"example","confidence":0.9}]
|
||||||
tag: z.string().describe('Short tag name in lowercase'),
|
Note content: "${content.substring(0, 1500)}"`,
|
||||||
confidence: z.number().min(0).max(1).describe('Confidence level between 0 and 1')
|
|
||||||
}))
|
|
||||||
}),
|
|
||||||
prompt: `Analyze the following note and suggest 1 to 5 relevant tags.
|
|
||||||
Note content: "${content}"`,
|
|
||||||
});
|
});
|
||||||
|
|
||||||
return object.tags;
|
const clean = text.replace(/^```json\n?/, '').replace(/\n?```$/, '').trim();
|
||||||
|
const parsed = JSON.parse(clean);
|
||||||
|
const arr = Array.isArray(parsed) ? parsed : (parsed.tags || []);
|
||||||
|
return arr.map((t: any) => ({
|
||||||
|
tag: t.tag || t.label || t.name || '',
|
||||||
|
confidence: t.confidence || t.score || 0.7,
|
||||||
|
}));
|
||||||
} catch (e) {
|
} catch (e) {
|
||||||
console.error('Error generating tags (DeepSeek):', e);
|
console.error('Error generating tags (DeepSeek):', e);
|
||||||
return [];
|
return [];
|
||||||
|
|||||||
Reference in New Issue
Block a user