fix(nextjs,shared): Revert "Use `use client` directive to support sha… by nikosdouvlis · Pull Request #1806 · clerk/javascript

nikosdouvlis · 2023-09-29T18:35:21Z

Temporarily resolve

 ⨯ node_modules/@clerk/shared/dist/cjs/hooks/createContextAndHook.js (43:35) @ createContext
 ⨯ createContext only works in Client Components. Add the "use client" directive at the top of the file to use it. Read more: https://nextjs.org/docs/messages/context-in-server-component
    at createContextAndHook (webpack-internal:///(rsc)/./node_modules/@clerk/shared/dist/cjs/hooks/createContextAndHook.js:48:38)
    at eval (webpack-internal:///(rsc)/./node_modules/@clerk/shared/dist/cjs/hooks/contexts.js:53:110)

Description

Checklist

npm test runs as expected.
npm run build runs as expected.
(If applicable) JSDoc comments have been added or updated for any package exports
(If applicable) Documentation has been updated

Type of change

Packages affected

changeset-bot · 2023-09-29T18:35:24Z

🦋 Changeset detected

Latest commit: cecf74d

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

This PR includes changesets to release 11 packages

Name	Type
@clerk/backend	Patch
@clerk/shared	Patch
@clerk/nextjs	Patch
@clerk/fastify	Patch
gatsby-plugin-clerk	Patch
@clerk/remix	Patch
@clerk/clerk-sdk-node	Patch
@clerk/clerk-js	Patch
@clerk/clerk-react	Patch
@clerk/chrome-extension	Patch
@clerk/clerk-expo	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

nikosdouvlis · 2023-09-29T18:35:28Z

!snapshot

clerk-cookie · 2023-09-29T18:38:28Z

Hey @nikosdouvlis - the snapshot version command generated the following package versions:

Package	Version
@clerk/chrome-extension	0.4.4-snapshot.e37df91
@clerk/clerk-js	4.59.1-snapshot.e37df91
@clerk/clerk-expo	0.19.6-snapshot.e37df91
gatsby-plugin-clerk	4.4.12-snapshot.e37df91
@clerk/nextjs	4.25.1-snapshot.e37df91
@clerk/clerk-react	4.26.1-snapshot.e37df91
@clerk/remix	3.0.3-snapshot.e37df91

Tip: use the snippet copy button below to quickly install the required packages.

# @clerk/chrome-extension
npm i @clerk/chrome-extension@0.4.4-snapshot.e37df91

# @clerk/clerk-js
npm i @clerk/clerk-js@4.59.1-snapshot.e37df91

# @clerk/clerk-expo
npm i @clerk/clerk-expo@0.19.6-snapshot.e37df91

# gatsby-plugin-clerk
npm i gatsby-plugin-clerk@4.4.12-snapshot.e37df91

# @clerk/nextjs
npm i @clerk/nextjs@4.25.1-snapshot.e37df91

# @clerk/clerk-react
npm i @clerk/clerk-react@4.26.1-snapshot.e37df91

# @clerk/remix
npm i @clerk/remix@3.0.3-snapshot.e37df91

This reverts commit 6dbaa2f.

…nused" This reverts commit 2f6a6ac

…rts" This reverts commit 932a403

This reverts commit 00d9d42 Revert "chore(backend): Warn about apiKey deprecation" This reverts commit 1a2f12b Revert "chore(backend): Warn about httpOptions deprecation" This reverts commit 838475b. Revert "chore(backend): Warn about InterstitialAPI deprecation" This reverts commit 51a5c39. Revert "chore(backend): Warn about picture/logoUrl/profileImageUrl (image related) deprecations" This reverts commit 077ce3a Revert "chore(backend): Warn about pkgVersion deprecation" This reverts commit 362caaa. Revert "chore(backend): Warn about clockSkewInSeconds deprecation" This reverts commit f500d46. Revert "chore(backend): Warn about backend api return format deprecation" This reverts commit c937990. Revert "chore(backend): Warn about __unstable_options deprecation" This reverts commit 54ba397.

nikosdouvlis · 2023-09-29T19:31:06Z

!snapshot

clerk-cookie · 2023-09-29T19:35:52Z

Hey @nikosdouvlis - the snapshot version command generated the following package versions:

Package	Version
@clerk/chrome-extension	0.4.4-snapshot.aae1150
@clerk/clerk-js	4.59.1-snapshot.aae1150
@clerk/clerk-expo	0.19.6-snapshot.aae1150
gatsby-plugin-clerk	4.4.12-snapshot.aae1150
@clerk/nextjs	4.25.1-snapshot.aae1150
@clerk/clerk-react	4.26.1-snapshot.aae1150
@clerk/remix	3.0.3-snapshot.aae1150

Tip: use the snippet copy button below to quickly install the required packages.

# @clerk/chrome-extension
npm i @clerk/chrome-extension@0.4.4-snapshot.aae1150

# @clerk/clerk-js
npm i @clerk/clerk-js@4.59.1-snapshot.aae1150

# @clerk/clerk-expo
npm i @clerk/clerk-expo@0.19.6-snapshot.aae1150

# gatsby-plugin-clerk
npm i gatsby-plugin-clerk@4.4.12-snapshot.aae1150

# @clerk/nextjs
npm i @clerk/nextjs@4.25.1-snapshot.aae1150

# @clerk/clerk-react
npm i @clerk/clerk-react@4.26.1-snapshot.aae1150

# @clerk/remix
npm i @clerk/remix@3.0.3-snapshot.aae1150

nikosdouvlis · 2023-09-29T19:45:55Z

!snapshot

clerk-cookie · 2023-09-29T19:49:04Z

Hey @nikosdouvlis - the snapshot version command generated the following package versions:

Package	Version
@clerk/backend	0.30.1-snapshot.cecf74d
@clerk/chrome-extension	0.4.4-snapshot.cecf74d
@clerk/clerk-js	4.59.1-snapshot.cecf74d
@clerk/clerk-expo	0.19.6-snapshot.cecf74d
@clerk/fastify	0.6.11-snapshot.cecf74d
gatsby-plugin-clerk	4.4.12-snapshot.cecf74d
@clerk/nextjs	4.25.1-snapshot.cecf74d
@clerk/clerk-react	4.26.1-snapshot.cecf74d
@clerk/remix	3.0.3-snapshot.cecf74d
@clerk/clerk-sdk-node	4.12.10-snapshot.cecf74d
@clerk/shared	0.24.1-snapshot.cecf74d

Tip: use the snippet copy button below to quickly install the required packages.

# @clerk/backend
npm i @clerk/backend@0.30.1-snapshot.cecf74d

# @clerk/chrome-extension
npm i @clerk/chrome-extension@0.4.4-snapshot.cecf74d

# @clerk/clerk-js
npm i @clerk/clerk-js@4.59.1-snapshot.cecf74d

# @clerk/clerk-expo
npm i @clerk/clerk-expo@0.19.6-snapshot.cecf74d

# @clerk/fastify
npm i @clerk/fastify@0.6.11-snapshot.cecf74d

# gatsby-plugin-clerk
npm i gatsby-plugin-clerk@4.4.12-snapshot.cecf74d

# @clerk/nextjs
npm i @clerk/nextjs@4.25.1-snapshot.cecf74d

# @clerk/clerk-react
npm i @clerk/clerk-react@4.26.1-snapshot.cecf74d

# @clerk/remix
npm i @clerk/remix@3.0.3-snapshot.cecf74d

# @clerk/clerk-sdk-node
npm i @clerk/clerk-sdk-node@4.12.10-snapshot.cecf74d

# @clerk/shared
npm i @clerk/shared@0.24.1-snapshot.cecf74d

OwenVey · 2023-10-05T14:29:40Z

@nikosdouvlis I'm still seeing this error using 4.25.2.

 ⨯ node_modules/.pnpm/@clerk+shared@0.24.2_react@18.2.0/node_modules/@clerk/shared/dist/esm/hooks/createContextAndHook.js (9:14) @ React
 ⨯ createContext only works in Client Components. Add the "use client" directive at the top of the file to use it. Read more: https://nextjs.org/docs/messages/context-in-server-component
    at __webpack_require__ (/Users/owenvey/Developer/chowder/.next/server/webpack-runtime.js:33:43)
    at __webpack_require__ (/Users/owenvey/Developer/chowder/.next/server/webpack-runtime.js:33:43)
    at __webpack_require__ (/Users/owenvey/Developer/chowder/.next/server/webpack-runtime.js:33:43)
    at __webpack_require__ (/Users/owenvey/Developer/chowder/.next/server/webpack-runtime.js:33:43)
    at __webpack_require__ (/Users/owenvey/Developer/chowder/.next/server/webpack-runtime.js:33:43)
    at __webpack_require__ (/Users/owenvey/Developer/chowder/.next/server/webpack-runtime.js:33:43)
    at __webpack_require__ (/Users/owenvey/Developer/chowder/.next/server/webpack-runtime.js:33:43)
    at __webpack_require__ (/Users/owenvey/Developer/chowder/.next/server/webpack-runtime.js:33:43)
    at __webpack_require__ (/Users/owenvey/Developer/chowder/.next/server/webpack-runtime.js:33:43)
    at __webpack_require__ (/Users/owenvey/Developer/chowder/.next/server/webpack-runtime.js:33:43)
    at eval (./src/app/actions.ts:7:71)
    at (action-browser)/./src/app/actions.ts (/Users/owenvey/Developer/chowder/.next/server/app/recipes/new/page.js:587:1)
    at Function.__webpack_require__ (/Users/owenvey/Developer/chowder/.next/server/webpack-runtime.js:33:43)

and here is my action.ts file:

'use server';

import { db } from '@/lib/db';
import type { RecipeCreate } from '@/types';
import { auth } from '@clerk/nextjs';
import { redirect } from 'next/navigation';

export async function createRecipe(recipe: Omit<RecipeCreate, 'userId'>) {
  const { userId } = auth();
  if (!userId) {
    throw new Error('You must be signed in to create a new recipe');
  }

  const newRecipe = await db.recipe.create({ data: { ...recipe, userId } });
  redirect(`/recipes/${newRecipe.id}`);
}

nikosdouvlis · 2023-10-05T14:58:39Z

@OwenVey Thank you for reporting this one, forwarding this to the team ASAP

royanger · 2023-10-05T16:06:31Z

@OwenVey, if you import the Server Action into a Server Component and then pass it as a prop to the Client Component does that resolve the issue?

OwenVey · 2023-10-05T16:13:39Z

@royanger yeah, I was the one who posted it on Discord. I think it's definitely a Next.js issue

clerk-cookie · 2024-10-05T00:01:29Z

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

nikosdouvlis requested a review from a team as a code owner September 29, 2023 18:35

clerk-cookie added the nextjs label Sep 29, 2023

nikosdouvlis added 3 commits September 29, 2023 22:12

Revert "chore(shared): Add JSDoc to helpers moved to shared package"

9626959

This reverts commit 6dbaa2f.

Revert "chore(backend): Moved shared module to util/shared and drop u…

0df7e2c

…nused" This reverts commit 2f6a6ac

Revert "chore(backend): Replace shared module with @clerk/shared impo…

01bb8ca

…rts" This reverts commit 932a403

nikosdouvlis force-pushed the revert-1767 branch from e37df91 to 01bb8ca Compare September 29, 2023 19:17

nikosdouvlis requested a review from dimkl as a code owner September 29, 2023 19:17

clerk-cookie added backend playground and removed nextjs labels Sep 29, 2023

chore(repo): Changeset

cecf74d

nikosdouvlis merged commit d2dd6e1 into main Sep 29, 2023

nikosdouvlis deleted the revert-1767 branch September 29, 2023 19:50

clerk-cookie mentioned this pull request Sep 29, 2023

Version Packages #1802

Merged

dimkl mentioned this pull request Sep 29, 2023

fix(nextjs, shared): Restores reverted commits and fixes issue with shared in nextjs #1807

Merged

24 tasks

This was referenced Oct 5, 2023

Version Packages #1831

Merged

Version Packages #1842

Merged

clerk locked as resolved and limited conversation to collaborators Oct 5, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix(nextjs,shared): Revert "Use `use client` directive to support sha… - #1806

fix(nextjs,shared): Revert "Use `use client` directive to support sha…#1806
nikosdouvlis merged 5 commits into
mainfrom
revert-1767

nikosdouvlis commented Sep 29, 2023 •

edited

Loading

Uh oh!

changeset-bot Bot commented Sep 29, 2023 •

edited

Loading

Uh oh!

nikosdouvlis commented Sep 29, 2023

Uh oh!

clerk-cookie commented Sep 29, 2023

Uh oh!

nikosdouvlis commented Sep 29, 2023

Uh oh!

clerk-cookie commented Sep 29, 2023

Uh oh!

nikosdouvlis commented Sep 29, 2023

Uh oh!

clerk-cookie commented Sep 29, 2023

Uh oh!

OwenVey commented Oct 5, 2023

Uh oh!

nikosdouvlis commented Oct 5, 2023

Uh oh!

royanger commented Oct 5, 2023

Uh oh!

OwenVey commented Oct 5, 2023

Uh oh!

clerk-cookie commented Oct 5, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

nikosdouvlis commented Sep 29, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Checklist

Type of change

Packages affected

Uh oh!

changeset-bot Bot commented Sep 29, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

nikosdouvlis commented Sep 29, 2023

Uh oh!

clerk-cookie commented Sep 29, 2023

Uh oh!

nikosdouvlis commented Sep 29, 2023

Uh oh!

clerk-cookie commented Sep 29, 2023

Uh oh!

nikosdouvlis commented Sep 29, 2023

Uh oh!

clerk-cookie commented Sep 29, 2023

Uh oh!

OwenVey commented Oct 5, 2023

Uh oh!

nikosdouvlis commented Oct 5, 2023

Uh oh!

royanger commented Oct 5, 2023

Uh oh!

OwenVey commented Oct 5, 2023

Uh oh!

clerk-cookie commented Oct 5, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

nikosdouvlis commented Sep 29, 2023 •

edited

Loading

changeset-bot Bot commented Sep 29, 2023 •

edited

Loading