mirror of
https://github.com/thedotmack/claude-mem
synced 2026-04-25 17:15:04 +02:00
cbeddef2d7
- Update test: parseSummary with <observation> tags now returns salvaged summary object instead of null, matching the intentional #1908 behavior - Fix docstring: salvageSummaryFromObservationTags extracts from all observations, not just the first - Add explanatory comment in test linking #1360 → #1908 behavior change Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
65 lines
2.8 KiB
TypeScript
65 lines
2.8 KiB
TypeScript
/**
|
|
* Tests for parseSummary (fix for #1360)
|
|
*
|
|
* Validates that false-positive summary matches (no sub-tags) are rejected
|
|
* while real summaries — even with some missing fields — are still saved.
|
|
*/
|
|
import { describe, it, expect } from 'bun:test';
|
|
import { parseSummary } from '../../src/sdk/parser.js';
|
|
|
|
describe('parseSummary', () => {
|
|
// Intentional behavior change from #1908: parseSummary now salvages data from
|
|
// <observation> tags when no <summary> tags are present, rather than discarding
|
|
// the output. Previously this returned null (see #1360), but #1908 determined
|
|
// that discarding observation-tagged output loses valuable session data.
|
|
it('salvages summary from <observation> tags when no <summary> tag present (#1908)', () => {
|
|
const result = parseSummary('<observation><title>foo</title></observation>');
|
|
expect(result).not.toBeNull();
|
|
expect(result?.request).toBe('foo');
|
|
expect(result?.investigated).toBeNull();
|
|
expect(result?.learned).toBeNull();
|
|
expect(result?.completed).toBeNull();
|
|
expect(result?.next_steps).toBeNull();
|
|
expect(result?.notes).toBeNull();
|
|
});
|
|
|
|
it('returns null when <summary> has no sub-tags (false positive — fix for #1360)', () => {
|
|
// This is the bug: observation response accidentally contains <summary>some text</summary>
|
|
expect(parseSummary('<observation>done <summary>some content here</summary></observation>')).toBeNull();
|
|
});
|
|
|
|
it('returns null for bare <summary> with only plain text, no sub-tags', () => {
|
|
expect(parseSummary('<summary>This session was productive.</summary>')).toBeNull();
|
|
});
|
|
|
|
it('returns summary when at least one sub-tag is present (respects maintainer note)', () => {
|
|
const text = `<summary><request>Fix the bug</request></summary>`;
|
|
const result = parseSummary(text);
|
|
expect(result).not.toBeNull();
|
|
expect(result?.request).toBe('Fix the bug');
|
|
expect(result?.investigated).toBeNull();
|
|
expect(result?.learned).toBeNull();
|
|
});
|
|
|
|
it('returns full summary when all fields are present', () => {
|
|
const text = `<summary>
|
|
<request>Fix login bug</request>
|
|
<investigated>Auth flow and JWT expiry</investigated>
|
|
<learned>Token was expiring too soon</learned>
|
|
<completed>Extended token TTL to 24h</completed>
|
|
<next_steps>Monitor error rates</next_steps>
|
|
</summary>`;
|
|
const result = parseSummary(text);
|
|
expect(result).not.toBeNull();
|
|
expect(result?.request).toBe('Fix login bug');
|
|
expect(result?.investigated).toBe('Auth flow and JWT expiry');
|
|
expect(result?.learned).toBe('Token was expiring too soon');
|
|
expect(result?.completed).toBe('Extended token TTL to 24h');
|
|
expect(result?.next_steps).toBe('Monitor error rates');
|
|
});
|
|
|
|
it('returns null when skip_summary tag is present', () => {
|
|
expect(parseSummary('<skip_summary reason="no work done"/>')).toBeNull();
|
|
});
|
|
});
|