#23772: fix(ci): unblock repo-wide check gate type regressions
channel: telegram
agents
size: S
## Summary
- replace value construction of `AssistantMessageEventStream` in the OpenRouter cache-control test with `createAssistantMessageEventStream`
- normalize Blob construction in skills download safety tests to use an `ArrayBuffer` blob part
- fix pinned dispatcher test lookup typing by using `node:dns` lookup signature
- harden telegram monitor webhook abort-signal handling and align runner test mocks (`isRunning`) with runtime typing
- add explicit `Mock` annotations in shared skills-install test mocks to avoid non-portable inferred vitest spy types
## Validation
- `pnpm vitest run src/agents/pi-embedded-runner/extra-params.openrouter-cache-control.test.ts src/agents/skills-install.download.test.ts src/infra/net/ssrf.dispatcher.test.ts src/telegram/monitor.test.ts`
- `pnpm vitest run src/agents/skills-install.test.ts src/agents/skills-install-fallback.test.ts`
- `pnpm check`
<!-- greptile_comment -->
<h3>Greptile Summary</h3>
This PR fixes several TypeScript type-checking issues across test files to unblock repo-wide type checking:
- Replaced `new AssistantMessageEventStream()` constructor usage with `createAssistantMessageEventStream()` factory function in OpenRouter cache-control tests
- Normalized Blob construction in skills download safety tests to use `ArrayBuffer` blob parts instead of `Uint8Array` directly
- Fixed dispatcher test typing by using `dnsLookup` function instead of a mock for the `lookup` signature
- Hardened telegram monitor webhook abort-signal handling by extracting signal to local variable
- Added explicit `Mock` type annotations to shared skills-install test mocks to avoid non-portable inferred vitest spy types
All changes are test-only fixes that improve type safety without affecting runtime behavior.
<h3>Confidence Score: 5/5</h3>
- Safe to merge - test-only type fixes with passing CI validation
- All changes are confined to test files, address TypeScript type checking issues, and have been validated by the test suite (27 passing tests shown in PR description). The changes improve type safety without altering runtime behavior.
- No files require special attention
<sub>Last reviewed commit: 4541a9a</sub>
<!-- greptile_other_comments_section -->
<!-- /greptile_comment -->
Most Similar PRs
#21027: fix(tests): resolve TypeScript type errors in CLI test files
by kittipond2365 · 2026-02-19
79.3%
#23594: test(tui): fix mock typing in command handler tests
by Sashamine · 2026-02-22
79.1%
#17393: fix(ci): resolve TS2742 type error blocking all PRs
by miloudbelarebia · 2026-02-15
78.7%
#22990: test: fix readonly typing regressions in CI check baseline
by bmendonca3 · 2026-02-21
77.3%
#20496: test(utils): add comprehensive unit tests for utility functions
by masifislamm · 2026-02-19
76.7%
#23139: test: fix flaky auth tests when OPENCLAW_GATEWAY_TOKEN is present
by Imccccc · 2026-02-22
76.3%
#4878: fix: string/type handling and API fixes (#4537, #4380, #4373, #4547...
by lailoo · 2026-01-30
75.9%
#23749: fix some issues
by tronpis · 2026-02-22
75.6%
#21804: fix: improve type safety and use project runtime logging
by hobostay · 2026-02-20
75.3%
#22178: test(web): allow fixture roots in media local file tests
by Kansodata · 2026-02-20
75.1%