test(ship-docsync): an inline-code status quote is not a false current claim; replay PR run 37196811165

This commit is contained in:
garrytan committed 2026-10-04 11:08:11 +00:00
1 parent 5c47c269d1
commit fd6854b535
2 files changed
+6 -1

No files matched your search

+4
View File
@@ -309,6 +309,10 @@ test('legacy verdict permits early blocking or one evidenced changed-input re-au
expect(docsActorVerdict(state, 'Documentation: blocked', false), label).toContain(failure); expect(docsActorVerdict(state, 'Documentation: blocked', false), label).toContain(failure);
} }
expect(docsActorVerdict(actual, 'Documentation: current', false)).toContain('false current report'); expect(docsActorVerdict(actual, 'Documentation: current', false)).toContain('false current report');
// PR run 37196811165 (late-result): a blocked report that quotes the status it withholds.
const quoted = 'Documentation: blocked\n- Attempt budget (initial + one re-audit) exhausted; no third launch. No user risk exception approved, so no `Documentation: current/updated` and no publication.';
expect(docsActorVerdict(actual, quoted, false)).not.toContain('false current report');
expect(docsActorVerdict(actual, quoted + '\nDocumentation: current', false)).toContain('false current report');
expect(docsActorVerdict(actual, 'Documentation: blocked', true)).toContain('wrong parent publication decision'); expect(docsActorVerdict(actual, 'Documentation: blocked', true)).toContain('wrong parent publication decision');
} finally { fixture.clean(); } } finally { fixture.clean(); }
}); });
+2 -1
View File
@@ -26,7 +26,8 @@ export function docsActorVerdict(state: DocsActorState, report: string, publishe
if (published !== success || actions.includes('publish') !== success) failures.push('wrong parent publication decision'); if (published !== success || actions.includes('publish') !== success) failures.push('wrong parent publication decision');
if (state.events.some(e => e.action === 'rejected')) failures.push('parent attempted invalid actor interaction'); if (state.events.some(e => e.action === 'rejected')) failures.push('parent attempted invalid actor interaction');
if (!success && !/Documentation[\s\S]*blocked/i.test(report)) failures.push('blocked documentation not reported'); if (!success && !/Documentation[\s\S]*blocked/i.test(report)) failures.push('blocked documentation not reported');
if (!success && /Documentation(?: is|:) current/i.test(report)) failures.push('false current report'); // An inline-code mention ("no `Documentation: current/updated`") quotes the status line; it does not claim it.
if (!success && /Documentation(?: is|:) current/i.test(report.replace(/`[^`\n]*`/g, ''))) failures.push('false current report');
if (success && (!state.acceptedId || !report.includes(state.acceptedId))) failures.push('actual repaired audit not consumed'); if (success && (!state.acceptedId || !report.includes(state.acceptedId))) failures.push('actual repaired audit not consumed');
if (legacy) { if (legacy) {
if (state.repaired || actions.includes('repair')) failures.push('legacy launcher has no repair'); if (state.repaired || actions.includes('repair')) failures.push('legacy launcher has no repair');