Skip to content

fix(nextcloud-talk): restore replay test typecheck#107808

Merged
steipete merged 1 commit into
mainfrom
codex/fix-nextcloud-test-response-type
Jul 14, 2026
Merged

fix(nextcloud-talk): restore replay test typecheck#107808
steipete merged 1 commit into
mainfrom
codex/fix-nextcloud-test-response-type

Conversation

@steipete

Copy link
Copy Markdown
Contributor

What Problem This Solves

Resolves a problem where the full test-types CI shard fails on current main while type-checking the Nextcloud Talk replay response mock.

Why This Change Was Made

The response mock was asserted as a complete ServerResponse at its declaration, which erased the object literal's contextual this type. The assertion now sits only at the listener boundary, preserving the partial mock's inferred method receiver while keeping the production listener contract unchanged.

User Impact

No user-visible behavior change. CI can type-check the test suite again.

Evidence

  • Failure reproduced in check-test-types: Property 'headersSent' does not exist on type '{}'.
  • Testbox tbx_01kxhb58jezv7et2ttkc0w9yga: pnpm test extensions/nextcloud-talk/src/monitor.replay.test.ts — 10/10 passed.
  • Same Testbox: pnpm check:test-types — passed.
  • Fresh autoreview: clean, confidence 0.99.
  • Test-only repair; no docs or changelog change.

@openclaw-barnacle openclaw-barnacle Bot added channel: nextcloud-talk Channel integration: nextcloud-talk size: XS maintainer Maintainer-authored PR labels Jul 14, 2026
@steipete

Copy link
Copy Markdown
Contributor Author

Land-ready.

@steipete
steipete merged commit 352473c into main Jul 14, 2026
107 of 111 checks passed
@steipete
steipete deleted the codex/fix-nextcloud-test-response-type branch July 14, 2026 22:26
@steipete

Copy link
Copy Markdown
Contributor Author

Merged via squash.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

channel: nextcloud-talk Channel integration: nextcloud-talk maintainer Maintainer-authored PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant