[codex] Clarify reviewer verification evidence (#141)
* add gated codex goals support * sync README SDK versions * bump claude agent sdk * add codex goals settings toggle * reuse status dashboard message and simplify settings actions * refine settings page UX * fix settings nav hash routing * add dashboard UX verification * refine dashboard settings and inbox UX * remove inbox top-level navigation * remove dashboard health top-level navigation * fix: allow runtime image attachment paths * feat: configure attachment allowlist dirs * fix: clean duplicate dashboard status messages * fix: poll dashboard duplicate cleanup between status updates * fix: delete dashboard duplicates on create * Refine settings IA with tabbed sections * Add read-only runtime inventory settings * Fix runtime inventory MCP JSON parsing * Add room skill settings inventory * Add room skill setting mutations * Apply room skill overrides to runner spawn * Refine scheduled task dashboard UX * Remove reviewer final prompt reinjection * Stabilize pnpm verification fixture * Remove previous owner final prompt carryover * Clear role-scoped sessions * Tighten reviewer prompt output scope * Clarify reviewer verification evidence * Require reviewer status first line
This commit is contained in:
@@ -73,6 +73,7 @@ describe('platform-prompts', () => {
|
||||
|
||||
const codexPairedPrompt = readPairedRoomPrompt('codex', repoRoot);
|
||||
expect(codexPairedPrompt).toContain('reviewer');
|
||||
expect(codexPairedPrompt).toContain('the output is invalid');
|
||||
expect(codexPairedPrompt).toContain('EJCLAW_WORK_DIR');
|
||||
expect(codexPairedPrompt).toContain(
|
||||
'canonical verification root for this turn',
|
||||
@@ -83,6 +84,9 @@ describe('platform-prompts', () => {
|
||||
expect(codexPairedPrompt).toContain(
|
||||
'Separate correctness issues from improvement ideas',
|
||||
);
|
||||
expect(codexPairedPrompt).toContain(
|
||||
'Do not present static analysis as completed verification',
|
||||
);
|
||||
expect(codexPairedPrompt).toContain('Keep reviewer output owner-facing');
|
||||
expect(codexPairedPrompt).toContain('prefer 3-6 lines');
|
||||
expect(codexPairedPrompt).not.toContain('owner-side paired agent');
|
||||
|
||||
Reference in New Issue
Block a user