Refactor parseSummary to always save summary regardless of missing fields
- Removed validation for required fields in parseSummary function. - Added a note emphasizing the importance of saving the summary even if some fields are missing.
This commit is contained in:
File diff suppressed because one or more lines are too long
+14
-11
@@ -130,18 +130,21 @@ export function parseSummary(text: string, sessionId?: number): ParsedSummary |
|
|||||||
const next_steps = extractField(summaryContent, 'next_steps');
|
const next_steps = extractField(summaryContent, 'next_steps');
|
||||||
const notes = extractField(summaryContent, 'notes'); // Optional
|
const notes = extractField(summaryContent, 'notes'); // Optional
|
||||||
|
|
||||||
|
// NOTE FROM THEDOTMACK: 100% of the time we must SAVE the summary, even if fields are missing. 10/24/2025
|
||||||
|
// NEVER DO THIS NONSENSE AGAIN.
|
||||||
|
|
||||||
// Validate required fields are present (notes is optional)
|
// Validate required fields are present (notes is optional)
|
||||||
if (!request || !investigated || !learned || !completed || !next_steps) {
|
// if (!request || !investigated || !learned || !completed || !next_steps) {
|
||||||
logger.warn('PARSER', 'Summary missing required fields', {
|
// logger.warn('PARSER', 'Summary missing required fields', {
|
||||||
sessionId,
|
// sessionId,
|
||||||
hasRequest: !!request,
|
// hasRequest: !!request,
|
||||||
hasInvestigated: !!investigated,
|
// hasInvestigated: !!investigated,
|
||||||
hasLearned: !!learned,
|
// hasLearned: !!learned,
|
||||||
hasCompleted: !!completed,
|
// hasCompleted: !!completed,
|
||||||
hasNextSteps: !!next_steps
|
// hasNextSteps: !!next_steps
|
||||||
});
|
// });
|
||||||
return null;
|
// return null;
|
||||||
}
|
// }
|
||||||
|
|
||||||
return {
|
return {
|
||||||
request,
|
request,
|
||||||
|
|||||||
Reference in New Issue
Block a user