[Cache Components] Remove unstable prefix from unstable_cacheTag

`cacheLife` is now stable and does not require an unstable prefix

Author: gnoff

crates/next-custom-transforms/src/transforms/react_server_components.rs

@@ -11,19 +11,19 @@ use regex::Regex;
 use rustc_hash::FxHashMap;
 use serde::Deserialize;
 use swc_core::{
-    atoms::{atom, Atom},
+    atoms::{Atom, atom},
     common::{
+        DUMMY_SP, FileName, Span, Spanned,
         comments::{Comment, CommentKind, Comments},
         errors::HANDLER,
         util::take::Take,
-        FileName, Span, Spanned, DUMMY_SP,
     },
     ecma::{
         ast::*,
-        utils::{prepend_stmts, quote_ident, quote_str, ExprFactory},
+        utils::{ExprFactory, prepend_stmts, quote_ident, quote_str},
         visit::{
-            noop_visit_mut_type, noop_visit_type, visit_mut_pass, Visit, VisitMut, VisitMutWith,
-            VisitWith,
+            Visit, VisitMut, VisitMutWith, VisitWith, noop_visit_mut_type, noop_visit_type,
+            visit_mut_pass,
         },
     },
 };
@@ -653,7 +653,7 @@ impl ReactServerComponentValidator {
                         "revalidateTag",
                         // "unstable_cache", // useless in client, but doesn't technically error
                         "cacheLife",
-                        "unstable_cacheTag",
+                        "cacheTag",
                         // "unstable_noStore" // no-op in client, but allowed for legacy reasons
                     ],
                 ),

docs/01-app/03-api-reference/04-functions/cacheTag.mdx

@@ -43,7 +43,7 @@ export default nextConfig
 The `cacheTag` function takes one or more string values.
 
 ```tsx filename="app/data.ts" switcher
-import { unstable_cacheTag as cacheTag } from 'next/cache'
+import { cacheTag } from 'next/cache'
 
 export async function getData() {
   'use cache'
@@ -54,7 +54,7 @@ export async function getData() {
 ```
 
 ```jsx filename="app/data.js" switcher
-import { unstable_cacheTag as cacheTag } from 'next/cache'
+import { cacheTag } from 'next/cache'
 
 export async function getData() {
   'use cache'
@@ -104,7 +104,7 @@ cacheTag('tag-one', 'tag-two')
 Tag your cached data by calling `cacheTag` within a cached function or component:
 
 ```tsx filename="app/components/bookings.tsx" switcher
-import { unstable_cacheTag as cacheTag } from 'next/cache'
+import { cacheTag } from 'next/cache'
 
 interface BookingsProps {
   type: string
@@ -124,7 +124,7 @@ export async function Bookings({ type = 'haircut' }: BookingsProps) {
 ```
 
 ```jsx filename="app/components/bookings.js" switcher
-import { unstable_cacheTag as cacheTag } from 'next/cache'
+import { cacheTag } from 'next/cache'
 
 export async function Bookings({ type = 'haircut' }) {
   'use cache'
@@ -144,7 +144,7 @@ export async function Bookings({ type = 'haircut' }) {
 You can use the data returned from an async function to tag the cache entry.
 
 ```tsx filename="app/components/bookings.tsx" switcher
-import { unstable_cacheTag as cacheTag } from 'next/cache'
+import { cacheTag } from 'next/cache'
 
 interface BookingsProps {
   type: string
@@ -162,7 +162,7 @@ export async function Bookings({ type = 'haircut' }: BookingsProps) {
 ```
 
 ```jsx filename="app/components/bookings.js" switcher
-import { unstable_cacheTag as cacheTag } from 'next/cache'
+import { cacheTag } from 'next/cache'
 
 export async function Bookings({ type = 'haircut' }) {
   async function getBookingsData() {

errors/next-prerender-missing-suspense.mdx

@@ -34,7 +34,7 @@ export default async function Page() {
 After:
 
 ```jsx filename="app/page.js"
-import { unstable_cacheTag as cacheTag, cacheLife } from 'next/cache'
+import { cacheTag, cacheLife } from 'next/cache'
 
 async function getRecentArticles() {
   "use cache"

packages/next/cache.d.ts

@@ -9,7 +9,7 @@ export {
 
 export { unstable_noStore } from 'next/dist/server/web/spec-extension/unstable-no-store'
 
-export { cacheTag as unstable_cacheTag } from 'next/dist/server/use-cache/cache-tag'
+export { cacheTag } from 'next/dist/server/use-cache/cache-tag'
 
 /**
  * Cache this `"use cache"` for a timespan defined by the `"default"` profile.

packages/next/cache.js

@@ -16,7 +16,7 @@ const cacheExports = {
     require('next/dist/server/web/spec-extension/unstable-no-store')
       .unstable_noStore,
   cacheLife: require('next/dist/server/use-cache/cache-life').cacheLife,
-  unstable_cacheTag: require('next/dist/server/use-cache/cache-tag').cacheTag,
+  cacheTag: require('next/dist/server/use-cache/cache-tag').cacheTag,
 }
 
 // https://nodejs.org/api/esm.html#commonjs-namespaces
@@ -30,5 +30,5 @@ exports.revalidateTag = cacheExports.revalidateTag
 exports.updateTag = cacheExports.updateTag
 exports.unstable_noStore = cacheExports.unstable_noStore
 exports.cacheLife = cacheExports.cacheLife
-exports.unstable_cacheTag = cacheExports.unstable_cacheTag
+exports.cacheTag = cacheExports.cacheTag
 exports.refresh = cacheExports.refresh

packages/next/src/build/webpack/plugins/next-types-plugin/index.ts

@@ -538,7 +538,7 @@ declare module 'next/cache' {
 
   ${overloads}
 
-  export { cacheTag as unstable_cacheTag } from 'next/dist/server/use-cache/cache-tag'
+  export { cacheTag } from 'next/dist/server/use-cache/cache-tag'
 }
 `
 }

test/development/acceptance-app/fixtures/rsc-build-errors/app/server-with-errors/next-cache-in-client/cachetag/page.js

@@ -0,0 +1,8 @@
+'use client'
+import { cacheTag } from 'next/cache'
+
+console.log({ cacheTag })
+
+export default function Page() {
+  return null
+}

test/development/acceptance-app/fixtures/rsc-build-errors/app/server-with-errors/next-cache-in-client/unstable_cachetag/page.js

@@ -279,23 +279,21 @@ describe('Error overlay - RSC build errors', () => {
   })
 
   describe("importing 'next/cache' APIs in a client component", () => {
-    test.each([
-      'revalidatePath',
-      'revalidateTag',
-      'cacheLife',
-      'unstable_cacheTag',
-    ])('%s is not allowed', async (api) => {
-      await using sandbox = await createSandbox(
-        next,
-        undefined,
-        `/server-with-errors/next-cache-in-client/${api.toLowerCase()}`
-      )
-      const { session } = sandbox
-      await session.assertHasRedbox()
-      expect(await session.getRedboxSource()).toInclude(
-        `You're importing a component that needs "${api}". That only works in a Server Component but one of its parents is marked with "use client", so it's a Client Component.`
-      )
-    })
+    test.each(['revalidatePath', 'revalidateTag', 'cacheLife', 'cacheTag'])(
+      '%s is not allowed',
+      async (api) => {
+        await using sandbox = await createSandbox(
+          next,
+          undefined,
+          `/server-with-errors/next-cache-in-client/${api.toLowerCase()}`
+        )
+        const { session } = sandbox
+        await session.assertHasRedbox()
+        expect(await session.getRedboxSource()).toInclude(
+          `You're importing a component that needs "${api}". That only works in a Server Component but one of its parents is marked with "use client", so it's a Client Component.`
+        )
+      }
+    )
 
     test.each([
       'unstable_cache', // useless in client, but doesn't technically error

test/development/acceptance-app/rsc-build-errors.test.ts

@@ -385,7 +385,7 @@ describe('Error Overlay for server components compiler errors in pages', () => {
       'revalidatePath',
       'revalidateTag',
       'cacheLife',
-      'unstable_cacheTag',
+      'cacheTag',
       'revalidatePath',
       'revalidateTag',
     ])('%s is not allowed', async (api) => {