Local API - findByID works in beta.107 but is broken in beta.108

[martynasj]

Describe the Bug

I believe that beta.108 broke the payload.findByID api in some way.
This is not an issue in beta.107

*However I was not able to reproduce this issue with the forked payload repo, where I checked out from a beta.108 tag.

Link to the code that reproduces this issue

https://github.com/martynasj/payload/blob/bug/test/_community/int.spec.ts#L51

Reproduction Steps

test('findByID api', async () => {
  const location = await payload.create({
    collection: 'locations',
    data: {
      name: 'New York',
    },
  })

  const allLocations = await payload.find({ collection: 'locations' })
  console.log(allLocations.docs)
  // finds array of these:
  // [
  //   {
  //     id: "671904875c999c26f3964f4a",
  //     name: "New York",
  //     createdAt: "2024-10-23T14:13:27.877Z",
  //     updatedAt: "2024-10-23T14:13:27.877Z",
  //   }
  // ]

  let singleLocation = await payload.db.collections['locations'].findById(location.id)
  console.log(singleLocation)
  // returns object like this:
  // {
  //   coordinates: {},
  //   _id: new ObjectId("671904875c999c26f3964f4a"),
  //   name: "Vilnius International Airport",
  //   createdAt: 2024-10-23T14:13:27.877Z,
  //   updatedAt: 2024-10-23T14:13:27.877Z,
  //   __v: 0,
  // }

  // throws NotFound error
  const singleLoc = await payload.findByID({ collection: 'locations', id: location.id })
})

Which area(s) are affected? (Select all that apply)

db-mongodb, area: core

Environment Info

Binaries:
Node: 20.17.0
npm: 10.8.2
Yarn: N/A
pnpm: 9.11.0
Relevant Packages:
payload: 3.0.0-beta.108
next: 15.0.0-canary.173
@payloadcms/db-mongodb: 3.0.0-beta.108
@payloadcms/email-nodemailer: 3.0.0-beta.108
@payloadcms/graphql: 3.0.0-beta.108
@payloadcms/next/utilities: 3.0.0-beta.108
@payloadcms/plugin-cloud: 3.0.0-beta.108
@payloadcms/richtext-lexical: 3.0.0-beta.108
@payloadcms/translations: 3.0.0-beta.108
@payloadcms/ui/shared: 3.0.0-beta.108
react: 19.0.0-rc-2d16326d-20240930
react-dom: 19.0.0-rc-2d16326d-20240930

[paulpopus]

I tested the provided test in the latest beta and it passes successfully.

In your own project where you're seeing this can you test on the latest beta please? Or at least like beta.116

If it still happens, can you then test it with a fresh database please?

[martynasj]

@paulpopus thanks for the response! This too seems really weird to me. I replicated the exact same config in the forked payload repo as I'm running and indeed it does work there as opposed in my own repo. Then I noticed that the only difference is the test runner. I'm running my tests with bun test.

So, in the forked repo:

• running a test with pnpm test:int does PASS
• running a test with bun test does FAIL

It does fail with this error:

....payload/packages/payload/src/collections/operations/findByID.ts:101:15

 97 |     let result: DataFromCollectionSlug<TSlug> = await req.payload.db.findOne(findOneArgs)
 98 | 
 99 |     if (!result) {
100 |       if (!disableErrors) {
101 |         throw new NotFound(req.t)

I tried to back trace the version of payload when the bun test command started failing and I found that it is beta.108

I do understand that bun is not something that you support, but maybe if you had any idea what was introduced in beta.108 that might've broken the findByID operation in bun environment?

[enricobuehler]

Having the same problem on beta.118, also using bun. Find function works fine.

[paulpopus]

Sorry guys but Bun is not officially supported right now, at best using it as a package manager is still experimental.

[github-actions]

This issue has been automatically locked.
Please open a new issue if this issue persists with any additional detail.