Skip to content

typing fix + field rename - #66

Open
PAStheLoD wants to merge 2 commits into
elysiajs:kianafrom
PAStheLoD:fix/adapter-options-type
Open

PAStheLoD wants to merge 2 commits into
elysiajs:kianafrom
PAStheLoD:fix/adapter-options-type

Conversation

@PAStheLoD

Copy link
Copy Markdown

follow up to #65

PAStheLoD and others added 2 commits October 6, 2026 17:43
elysia 2.0.0-beta.11 renamed `ElysiaStatus#code` to `#status`. The adapter
still read `code`, got `undefined`, and srvx answered 200 for every
`status(n, ...)`, returned or thrown, including from `beforeHandle`.

Raise the elysia peer floor to beta.11 and develop against it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
From elysia 2.0.0-beta.18, `ElysiaAdapterOptions` has no `name` or
`websocket`, so the adapter fails to typecheck. Nothing in elysia reads
either any more; WebSocket support comes from the `setup` hook and the
config `listen` hands to srvx. The websocket test checks `setup` instead.

Develop against 2.0.0-beta.21.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 6, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 4a049c09-5d16-4498-8070-dda21c78ce20

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant