fix: keep codex approval wording role-based
This commit is contained in:
@@ -70,4 +70,4 @@ Implementation, commits, and pushes require explicit agreement from both you and
|
|||||||
|
|
||||||
- Take the lead on implementation, debugging, and command execution
|
- Take the lead on implementation, debugging, and command execution
|
||||||
- Ship only after consensus is reached
|
- Ship only after consensus is reached
|
||||||
- When you spot a flaw in Claude's review or test plan, call it out directly
|
- When you spot a flaw in the owner-side paired agent's review or test plan, call it out directly
|
||||||
|
|||||||
@@ -1,6 +1,7 @@
|
|||||||
import fs from 'fs';
|
import fs from 'fs';
|
||||||
import os from 'os';
|
import os from 'os';
|
||||||
import path from 'path';
|
import path from 'path';
|
||||||
|
import { fileURLToPath } from 'url';
|
||||||
|
|
||||||
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
|
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
|
||||||
|
|
||||||
@@ -55,4 +56,24 @@ describe('platform-prompts', () => {
|
|||||||
);
|
);
|
||||||
expect(readPairedRoomPrompt('claude-code')).toBe('Claude paired prompt');
|
expect(readPairedRoomPrompt('claude-code')).toBe('Claude paired prompt');
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('keeps codex approval wording role-based while preserving failover identity wording', () => {
|
||||||
|
const repoRoot = path.resolve(
|
||||||
|
path.dirname(fileURLToPath(import.meta.url)),
|
||||||
|
'..',
|
||||||
|
);
|
||||||
|
|
||||||
|
const codexPairedPrompt = fs.readFileSync(
|
||||||
|
path.join(repoRoot, 'prompts', 'codex-paired-room.md'),
|
||||||
|
'utf-8',
|
||||||
|
);
|
||||||
|
expect(codexPairedPrompt).toContain('owner-side paired agent');
|
||||||
|
expect(codexPairedPrompt).not.toContain("Claude's review or test plan");
|
||||||
|
|
||||||
|
const failoverPlatformPrompt = fs.readFileSync(
|
||||||
|
path.join(repoRoot, 'prompts', 'codex-review-failover-platform.md'),
|
||||||
|
'utf-8',
|
||||||
|
);
|
||||||
|
expect(failoverPlatformPrompt).toContain('acting as `클코`');
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user