Home Original page

fix(sql-drizzle): allow $count to be yielded as an Effect by vinicius507 · Pull Request #6068 · Effect-TS/effect

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@vinicius507

Copy link

Type

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

Patches drizzle's *CountBuilder so $count can be yielded as an Effect.

Related

@changeset-bot

Copy link

changeset-bot bot commented

Feb 16, 2026

•

edited

Loading

🦋 Changeset detected

Latest commit: 1a6c87d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@effect/sql-drizzle Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vinicius507

Copy link

Author

There seems to be an issue with MySql. I'm turning this into a draft until I fix this.

@vinicius507 vinicius507 marked this pull request as draft

February 16, 2026 20:20 
Always wrap raw statement results in an array for the execute
callback, fixing MySqlSession.count() which expects res[0][0]
nesting.

@vinicius507 vinicius507 marked this pull request as ready for review

February 16, 2026 20:30

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

Reviewers

@tim-smart tim-smart Awaiting requested review from tim-smart tim-smart is a code owner

Assignees

No one assigned

Labels

None yet

Projects

PR Backlog

Status: Discussion Ongoing

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

@effect/sql-drizzle: cannot yield $count, value is not iterable

1 participant

@vinicius507