mirror of
https://github.com/Ark0N/Codeman.git
synced 2026-10-08 16:39:42 +02:00
The unified session list only set firstPrompt from the transcript-history view, keyed by the Claude transcript file's UUID. A live/persisted row keyed by its Codeman id only inherited a prompt when that id happened to equal an on-disk transcript filename; when it didn't (stale/wrong claudeSessionId, post-/clear new uuid, resumed/attached/worktree session), the session manager showed "(no prompt captured)" even though a real transcript for that working dir existed under a different UUID. Add a pure firstPrompt backfill pass in mergeUnifiedSessions (after the merge loops, using the already-passed history source): for any row with no firstPrompt, join by claudeSessionId first, then fall back to the newest transcript in the same workingDir. Never overwrites a non-empty prompt, so rows keyed to their own transcript are untouched; rows with genuinely no transcript still show the placeholder. Pure, unit-tested (+5). (cherry picked from commit 1f9f53ec64a61c9fa7f77d29efcbdd1d2794ec38)
322 lines
12 KiB
TypeScript
322 lines
12 KiB
TypeScript
/**
|
|
* @fileoverview Unit tests for the pure unified-session merge/filter service (COD-121).
|
|
*
|
|
* Covers dedup across sources, source precedence, mux-stat merge + meaningfulness
|
|
* floor, sort ordering, and filterAndPaginate (search + paging + clamps).
|
|
* Node env only — no jsdom, no IO.
|
|
*/
|
|
|
|
import { describe, it, expect } from 'vitest';
|
|
import {
|
|
mergeUnifiedSessions,
|
|
filterAndPaginate,
|
|
type UnifiedSessionItem,
|
|
} from '../../src/services/unified-session-service.js';
|
|
|
|
describe('mergeUnifiedSessions', () => {
|
|
it('dedupes the same sessionId across live + persisted into one item', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
live: [{ id: 's1', status: 'working', isWorking: true }],
|
|
persisted: [{ id: 's1', status: 'idle' }],
|
|
history: [{ sessionId: 's1', workingDir: '/w', sizeBytes: 5000, lastModified: '2026-01-01T00:00:00.000Z' }],
|
|
});
|
|
expect(merged).toHaveLength(1);
|
|
const item = merged[0];
|
|
expect(item.sessionId).toBe('s1');
|
|
// sources accumulate from every contributing source (order-insensitive)
|
|
expect([...item.sources].sort()).toEqual(['history', 'live', 'persisted']);
|
|
// live wins for status
|
|
expect(item.status).toBe('working');
|
|
expect(item.isWorking).toBe(true);
|
|
});
|
|
|
|
it('folds a resumed session transcript (claudeSessionId != id) into ONE row', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
live: [{ id: 'cm-1', status: 'working', claudeSessionId: 'uuid-resume' }],
|
|
history: [
|
|
{
|
|
sessionId: 'uuid-resume', // transcript rows are keyed by the conversation UUID
|
|
workingDir: '/w',
|
|
sizeBytes: 7000,
|
|
lastModified: '2026-01-03T00:00:00.000Z',
|
|
firstPrompt: 'resumed prompt',
|
|
},
|
|
],
|
|
});
|
|
expect(merged).toHaveLength(1);
|
|
const item = merged[0];
|
|
expect(item.sessionId).toBe('cm-1');
|
|
expect([...item.sources].sort()).toEqual(['history', 'live']);
|
|
expect(item.firstPrompt).toBe('resumed prompt');
|
|
expect(item.sizeBytes).toBe(7000);
|
|
});
|
|
|
|
it('resolves history rows through a persisted claudeSessionId alias', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
persisted: [{ id: 'cm-2', name: 'Resumed', claudeSessionId: 'uuid-p' }],
|
|
history: [{ sessionId: 'uuid-p', workingDir: '/w', sizeBytes: 4200, lastModified: '2026-01-04T00:00:00.000Z' }],
|
|
});
|
|
expect(merged).toHaveLength(1);
|
|
expect(merged[0].sessionId).toBe('cm-2');
|
|
expect([...merged[0].sources].sort()).toEqual(['history', 'persisted']);
|
|
});
|
|
|
|
it('surfaces the NEWEST lifecycle name/mode (entries arrive newest-first)', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
// query() returns newest-first: the rename must win over the original
|
|
// name — an unconditional overwrite would leave the OLDEST standing.
|
|
lifecycle: [
|
|
{ sessionId: 'del-1', name: 'Renamed', mode: 'claude', ts: 2000, event: 'deleted' },
|
|
{ sessionId: 'del-1', name: 'Original', mode: 'shell', ts: 1000, event: 'created' },
|
|
],
|
|
history: [
|
|
{
|
|
sessionId: 'del-1',
|
|
workingDir: '/w',
|
|
sizeBytes: 5000,
|
|
lastModified: '2026-01-01T00:00:00.000Z',
|
|
firstPrompt: 'hello',
|
|
},
|
|
],
|
|
});
|
|
expect(merged).toHaveLength(1);
|
|
expect(merged[0].name).toBe('Renamed');
|
|
expect(merged[0].mode).toBe('claude');
|
|
});
|
|
|
|
it('lets live status win over persisted (precedence)', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
persisted: [{ id: 's1', status: 'idle', name: 'Persisted Name' }],
|
|
live: [{ id: 's1', status: 'working' }],
|
|
});
|
|
expect(merged).toHaveLength(1);
|
|
expect(merged[0].status).toBe('working');
|
|
// persisted name survives because live did not provide one
|
|
expect(merged[0].name).toBe('Persisted Name');
|
|
});
|
|
|
|
it('merges mux stats onto a live item but drops a mux-only entry with no name', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
live: [{ id: 's1', status: 'working' }],
|
|
mux: [
|
|
{ sessionId: 's1', stats: { memoryMB: 42, cpuPercent: 3.5 }, remote: true },
|
|
{ sessionId: 'noise', stats: { memoryMB: 10, cpuPercent: 1 } },
|
|
],
|
|
});
|
|
expect(merged).toHaveLength(1);
|
|
const item = merged[0];
|
|
expect(item.sessionId).toBe('s1');
|
|
expect(item.stats).toEqual({ memoryMB: 42, cpuPercent: 3.5 });
|
|
expect(item.remote).toBe(true);
|
|
});
|
|
|
|
it('drops a lifecycle-only entry with no name/firstPrompt but keeps a history item with firstPrompt', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
lifecycle: [{ sessionId: 'bare', event: 'created', ts: 1000 }],
|
|
history: [
|
|
{
|
|
sessionId: 'hist',
|
|
workingDir: '/w',
|
|
sizeBytes: 9000,
|
|
lastModified: '2026-01-02T00:00:00.000Z',
|
|
firstPrompt: 'do the thing',
|
|
},
|
|
],
|
|
});
|
|
const ids = merged.map((m) => m.sessionId);
|
|
expect(ids).toContain('hist');
|
|
expect(ids).not.toContain('bare');
|
|
});
|
|
|
|
it('sorts by lastActivityAt desc with undefined last', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
live: [
|
|
{ id: 'a', status: 'idle', lastActivityAt: 100 },
|
|
{ id: 'b', status: 'idle', lastActivityAt: 300 },
|
|
{ id: 'c', status: 'idle' }, // no lastActivityAt → sorts last
|
|
{ id: 'd', status: 'idle', lastActivityAt: 200 },
|
|
],
|
|
});
|
|
expect(merged.map((m) => m.sessionId)).toEqual(['b', 'd', 'a', 'c']);
|
|
});
|
|
|
|
it('derives lastActivityAt from history lastModified when none better exists', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
history: [{ sessionId: 'h', workingDir: '/w', sizeBytes: 5000, lastModified: '2026-01-01T00:00:00.000Z' }],
|
|
});
|
|
expect(merged).toHaveLength(1);
|
|
expect(merged[0].lastActivityAt).toBe(new Date('2026-01-01T00:00:00.000Z').getTime());
|
|
});
|
|
|
|
it('surfaces projectKey from a history input onto the merged item', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
history: [
|
|
{
|
|
sessionId: 'h',
|
|
workingDir: '/w',
|
|
sizeBytes: 5000,
|
|
lastModified: '2026-01-01T00:00:00.000Z',
|
|
projectKey: '-repo-alpha',
|
|
},
|
|
],
|
|
});
|
|
expect(merged).toHaveLength(1);
|
|
expect(merged[0].projectKey).toBe('-repo-alpha');
|
|
});
|
|
|
|
// COD-140: firstPrompt backfill — live sessions whose Codeman id does not match an
|
|
// on-disk transcript UUID still surface a first prompt (by claudeSessionId join, then
|
|
// by newest transcript in the same workingDir).
|
|
it('backfills firstPrompt onto a live session by claudeSessionId join (uuid-join)', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
live: [{ id: 'codeman-1', status: 'working', claudeSessionId: 'uuid-A', workingDir: '/w' }],
|
|
history: [
|
|
{
|
|
sessionId: 'uuid-A',
|
|
workingDir: '/w',
|
|
sizeBytes: 5000,
|
|
lastModified: '2026-01-01T00:00:00.000Z',
|
|
firstPrompt: 'fix the bug',
|
|
},
|
|
],
|
|
});
|
|
const live = merged.find((m) => m.sessionId === 'codeman-1');
|
|
expect(live).toBeDefined();
|
|
expect(live!.firstPrompt).toBe('fix the bug');
|
|
// The upstream unified-service alias map (COD-160/161) folds a history row keyed
|
|
// by the Claude conversation UUID into the owning live session (claudeSessionId
|
|
// join), so it does NOT surface as a separate item — the firstPrompt reaches the
|
|
// live row above rather than a duplicate uuid-A entry.
|
|
const hist = merged.find((m) => m.sessionId === 'uuid-A');
|
|
expect(hist).toBeUndefined();
|
|
});
|
|
|
|
it('falls back to the workingDir transcript when no uuid join exists (workingDir fallback)', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
live: [{ id: 'codeman-2', status: 'working', claudeSessionId: 'uuid-missing', workingDir: '/w2' }],
|
|
history: [
|
|
{
|
|
sessionId: 'uuid-other',
|
|
workingDir: '/w2',
|
|
sizeBytes: 5000,
|
|
lastModified: '2026-01-01T00:00:00.000Z',
|
|
firstPrompt: 'borrowed prompt',
|
|
},
|
|
],
|
|
});
|
|
const live = merged.find((m) => m.sessionId === 'codeman-2');
|
|
expect(live).toBeDefined();
|
|
expect(live!.firstPrompt).toBe('borrowed prompt');
|
|
});
|
|
|
|
it('uses the newest transcript per workingDir for the fallback (newest-wins)', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
live: [{ id: 'codeman-3', status: 'working', claudeSessionId: 'uuid-missing', workingDir: '/w3' }],
|
|
history: [
|
|
{
|
|
sessionId: 'uuid-old',
|
|
workingDir: '/w3',
|
|
sizeBytes: 5000,
|
|
lastModified: '2026-01-01T00:00:00.000Z',
|
|
firstPrompt: 'older prompt',
|
|
},
|
|
{
|
|
sessionId: 'uuid-new',
|
|
workingDir: '/w3',
|
|
sizeBytes: 6000,
|
|
lastModified: '2026-02-01T00:00:00.000Z',
|
|
firstPrompt: 'newer prompt',
|
|
},
|
|
],
|
|
});
|
|
const live = merged.find((m) => m.sessionId === 'codeman-3');
|
|
expect(live).toBeDefined();
|
|
expect(live!.firstPrompt).toBe('newer prompt');
|
|
});
|
|
|
|
it('never overwrites a firstPrompt that already merged from the session own transcript (no overwrite)', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
live: [{ id: 'self-uuid', status: 'working', claudeSessionId: 'self-uuid', workingDir: '/w4' }],
|
|
history: [
|
|
// the session's own transcript (keyed by its id) — provides the real prompt
|
|
{
|
|
sessionId: 'self-uuid',
|
|
workingDir: '/w4',
|
|
sizeBytes: 5000,
|
|
lastModified: '2026-01-01T00:00:00.000Z',
|
|
firstPrompt: 'own prompt',
|
|
},
|
|
// a newer sibling transcript in the same dir that must NOT clobber it
|
|
{
|
|
sessionId: 'sibling-uuid',
|
|
workingDir: '/w4',
|
|
sizeBytes: 6000,
|
|
lastModified: '2026-03-01T00:00:00.000Z',
|
|
firstPrompt: 'sibling prompt',
|
|
},
|
|
],
|
|
});
|
|
const self = merged.find((m) => m.sessionId === 'self-uuid');
|
|
expect(self).toBeDefined();
|
|
expect(self!.firstPrompt).toBe('own prompt');
|
|
});
|
|
|
|
it('leaves firstPrompt undefined when there is no transcript at all (no transcript)', () => {
|
|
const merged = mergeUnifiedSessions({
|
|
live: [{ id: 'codeman-5', status: 'working', claudeSessionId: 'uuid-none', workingDir: '/empty' }],
|
|
});
|
|
const live = merged.find((m) => m.sessionId === 'codeman-5');
|
|
expect(live).toBeDefined();
|
|
expect(live!.firstPrompt).toBeUndefined();
|
|
});
|
|
});
|
|
|
|
describe('filterAndPaginate', () => {
|
|
const items: UnifiedSessionItem[] = [
|
|
{ sessionId: 's1', name: 'Alpha build', sources: ['live'], workingDir: '/repo/alpha' },
|
|
{ sessionId: 's2', name: 'Beta', firstPrompt: 'fix the login bug', sources: ['history'], workingDir: '/repo/beta' },
|
|
{ sessionId: 's3', name: 'Gamma', sources: ['persisted'], workingDir: '/srv/gamma' },
|
|
];
|
|
|
|
it('filters by name (case-insensitive)', () => {
|
|
const r = filterAndPaginate(items, { q: 'alpha' });
|
|
expect(r.total).toBe(1);
|
|
expect(r.sessions[0].sessionId).toBe('s1');
|
|
});
|
|
|
|
it('filters by firstPrompt and workingDir', () => {
|
|
expect(filterAndPaginate(items, { q: 'login bug' }).sessions[0].sessionId).toBe('s2');
|
|
expect(filterAndPaginate(items, { q: '/srv/' }).sessions[0].sessionId).toBe('s3');
|
|
});
|
|
|
|
it('reports total as the pre-page filtered count', () => {
|
|
const r = filterAndPaginate(items, { q: 'repo', limit: 1 });
|
|
// both s1 and s2 have /repo/ workingDir
|
|
expect(r.total).toBe(2);
|
|
expect(r.sessions).toHaveLength(1);
|
|
});
|
|
|
|
it('clamps limit to a max of 500', () => {
|
|
const r = filterAndPaginate(items, { limit: 99999 });
|
|
expect(r.sessions).toHaveLength(items.length);
|
|
// clamp does not throw and returns all 3 (< 500)
|
|
expect(r.total).toBe(3);
|
|
});
|
|
|
|
it('clamps limit to a min of 1', () => {
|
|
const r = filterAndPaginate(items, { limit: 0 });
|
|
expect(r.sessions).toHaveLength(1);
|
|
});
|
|
|
|
it('paginates with disjoint pages via offset', () => {
|
|
const page1 = filterAndPaginate(items, { offset: 0, limit: 2 });
|
|
const page2 = filterAndPaginate(items, { offset: 2, limit: 2 });
|
|
expect(page1.sessions.map((s) => s.sessionId)).toEqual(['s1', 's2']);
|
|
expect(page2.sessions.map((s) => s.sessionId)).toEqual(['s3']);
|
|
const overlap = page1.sessions
|
|
.map((s) => s.sessionId)
|
|
.filter((id) => page2.sessions.map((s2) => s2.sessionId).includes(id));
|
|
expect(overlap).toEqual([]);
|
|
});
|
|
});
|