7b63ba859d
Prevent duplicate CI completion notifications in paired rooms by checking for existing active watchers with the same channel+provider+metadata before creating a new one. Remove Task ID from watcher prompt construction to avoid router secret redaction (task- IDs contain sk- pattern), passing EJCLAW_RUNTIME_TASK_ID via env var instead and making cancel_task auto-resolve when task_id is omitted. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
59 lines
2.0 KiB
TypeScript
59 lines
2.0 KiB
TypeScript
import { describe, expect, it } from 'vitest';
|
|
|
|
import {
|
|
buildCiWatchPrompt,
|
|
DEFAULT_WATCH_CI_CONTEXT_MODE,
|
|
DEFAULT_GITHUB_WATCH_CI_INTERVAL_SECONDS,
|
|
normalizeWatchCiIntervalSeconds,
|
|
} from '../src/watch-ci.js';
|
|
|
|
describe('watch-ci helpers', () => {
|
|
it('builds a self-cancelling CI watch prompt', () => {
|
|
const prompt = buildCiWatchPrompt({
|
|
target: 'PR #42 checks',
|
|
checkInstructions:
|
|
'Use gh pr checks 42 and summarize only terminal results.',
|
|
});
|
|
|
|
expect(prompt).toContain('PR #42 checks');
|
|
expect(prompt).not.toContain('Task ID:');
|
|
expect(prompt).toContain('cancel_task');
|
|
expect(prompt).toContain('send_message');
|
|
expect(prompt).toContain('gh pr checks 42');
|
|
expect(prompt).toContain('CI 완료: <target>');
|
|
expect(prompt).toContain('판정: <one-line conclusion>');
|
|
expect(prompt).toContain(
|
|
'Use the watch target and check instructions in this prompt as the source of truth',
|
|
);
|
|
});
|
|
|
|
it('defaults CI watchers to isolated context', () => {
|
|
expect(DEFAULT_WATCH_CI_CONTEXT_MODE).toBe('isolated');
|
|
});
|
|
|
|
it('normalizes valid poll intervals', () => {
|
|
expect(normalizeWatchCiIntervalSeconds()).toBe(60);
|
|
expect(normalizeWatchCiIntervalSeconds(30)).toBe(30);
|
|
expect(normalizeWatchCiIntervalSeconds(600)).toBe(600);
|
|
expect(
|
|
normalizeWatchCiIntervalSeconds(undefined, { ciProvider: 'github' }),
|
|
).toBe(DEFAULT_GITHUB_WATCH_CI_INTERVAL_SECONDS);
|
|
expect(normalizeWatchCiIntervalSeconds(10, { ciProvider: 'github' })).toBe(
|
|
10,
|
|
);
|
|
});
|
|
|
|
it('rejects invalid poll intervals', () => {
|
|
expect(() => normalizeWatchCiIntervalSeconds(29)).toThrow(
|
|
/between 30 and 3600/i,
|
|
);
|
|
expect(() => normalizeWatchCiIntervalSeconds(3601)).toThrow(
|
|
/between 30 and 3600/i,
|
|
);
|
|
expect(() => normalizeWatchCiIntervalSeconds(30.5)).toThrow(/integer/i);
|
|
expect(() =>
|
|
normalizeWatchCiIntervalSeconds(9, { ciProvider: 'github' }),
|
|
).toThrow(/between 10 and 3600/i);
|
|
});
|
|
});
|