Instrumentation loads as client component, even in Node runtime

[bbangert]

Link to the code that reproduces this issue

https://github.com/bbangert/nextjs-instrumentation-bug

To Reproduce

1. Create Next.js app
2. Add instrumentation.ts file.
3. Include code to toggle loading of instrumentation-node file per Documentation
4. Put "import 'server-only';" in instrumentation-node file.
5. Build app

Current vs. Expected behavior

Current behavior is that Nextjs errors as instrumentation.ts is a client component, and cannot load server-only code.

Error:

ReactServerComponentsError:

You're importing a component that needs server-only. That only works in a Server Component but one of its parents is marked with "use client", so it's a Client Component.
Learn more: https://nextjs.org/docs/getting-started/react-essentials

   ╭─[/home/user/Programming/instrumentation-bug/instrumentation-node.ts:1:1]
 1 │ import 'server-only';
   · ─────────────────────
 2 │ 
 3 │ export const Nothing = 0;
   ╰────

One of these is marked as a client entry with "use client":
  ./instrumentation-node.ts
  ./instrumentation.ts

Expected behavior is that code targeted to run in Node runtime only should be capable of being marked as server-only code.

Verify canary release

• I verified that the issue exists in the latest Next.js canary release

Provide environment information

Operating System:
  Platform: linux
  Arch: x64
  Version: #96-Ubuntu SMP Wed Sep 20 08:23:49 UTC 2023
Binaries:
  Node: 18.18.2
  npm: 9.8.1
  Yarn: 1.22.19
  pnpm: N/A
Relevant Packages:
  next: 14.0.0
  eslint-config-next: 14.0.0
  react: 18.2.0
  react-dom: 18.2.0
  typescript: 5.2.2
Next.js Config:
  output: N/A

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

Not sure

Additional context

No response

_NEXT-2181

[ariesclark]

Still an issue in 14.0.2.

[paschaldev]

Can confirm this issue exists in v13.5 also

[olivierr91]

I have this issue as well

[AntonAndreevichMoroz]

I'm experiencing the same problem on Next.js version 14.0.5-canary.44.
The main problem is that even in NEXT_RUNTIME === "nodejs" I can't import functions that are marked 'use server' or import 'server-only'

[khuezy]

Should instrumentation also apply the experimental.serverComponentsExternalPackages too? It looks like it ignores that causing some external package issues.

[github-actions]

This closed issue has been automatically locked because it had no new activity for 2 weeks. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.