feat: enhance readiness reporting with review gate status and isolati… #142
lhy0718Annotations
6 errors and 1 warning
|
build-and-test
Process completed with exit code 1.
|
|
tests/doctorHarnessIntegration.test.ts > runDoctorReport > surfaces required networked execution as a stronger warning with explicit highlight guidance:
tests/doctorHarnessIntegration.test.ts#L199
AssertionError: expected true to be false // Object.is equality
- Expected
+ Received
- false
+ true
❯ tests/doctorHarnessIntegration.test.ts:199:38
|
|
tests/doctorHarnessIntegration.test.ts > runDoctorReport > downgrades declared networked execution to a warning instead of a hard failure:
tests/doctorHarnessIntegration.test.ts#L162
AssertionError: expected true to be false // Object.is equality
- Expected
+ Received
- false
+ true
❯ tests/doctorHarnessIntegration.test.ts:162:38
|
|
tests/doctorHarnessIntegration.test.ts > runDoctorReport > treats local snapshot isolation plus disabled network as ready for code execution:
tests/doctorHarnessIntegration.test.ts#L125
AssertionError: expected true to be false // Object.is equality
- Expected
+ Received
- false
+ true
❯ tests/doctorHarnessIntegration.test.ts:125:38
|
|
tests/doctorHarnessIntegration.test.ts > runDoctorReport > includes the latest compiled paper page-budget check when available:
tests/doctorHarnessIntegration.test.ts#L72
AssertionError: expected [ …(3) ] to deeply equal [ …(2) ]
- Expected
+ Received
[
"[ATTN] profile: llm=unknown, pdf=unknown, dependency=local, isolation=not-configured.",
- StringContaining "[ATTN] paper page budget:",
+ "[ATTN] provider/runtime blockers: latex.",
+ "[ATTN] paper page budget: Latest compiled paper page-budget check for run run-1: warn. pages=3, minimum_main_pages=8, target_main_pages=8. Compiled PDF is only 3 pages, below the configured minimum_main_pages of 8.",
]
❯ tests/doctorHarnessIntegration.test.ts:72:47
|
|
tests/doctorHarnessIntegration.test.ts > runDoctorReport > includes harness diagnostics for current workspace runs:
tests/doctorHarnessIntegration.test.ts#L40
AssertionError: expected true to be false // Object.is equality
- Expected
+ Received
- false
+ true
❯ tests/doctorHarnessIntegration.test.ts:40:38
|
|
build-and-test
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/setup-node@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|