Avoid bursting cache

Author: gregberge

packages/gitbook/src/lib/api.ts

@@ -394,7 +394,7 @@ const getAPIContextId = async () => {
  * Get a revision by its ID.
  */
 export const getRevision = cache({
-    name: 'api.getRevision.v3',
+    name: 'api.getRevision.v2',
     tag: (spaceId, revisionId, fetchOptions) =>
         // Temporary hack to make it work with OpenAPI on v1
         fetchOptions.tags?.[0] ??
@@ -427,7 +427,7 @@ export const getRevision = cache({
         return cacheResponse(response, {
             ...(fetchOptions.metadata ? cacheTtl_7days : cacheTtl_1day),
             data: {
-                revision: response.data,
+                ...response.data,
                 tags: getResponseCacheTags(response),
             },
         });
@@ -439,7 +439,7 @@ export const getRevision = cache({
  * Get all the pages in a revision of a space.
  */
 export const getRevisionPages = cache({
-    name: 'api.getRevisionPages.v5',
+    name: 'api.getRevisionPages.v4',
     tag: (spaceId, revisionId, fetchOptions) =>
         // Temporary hack to make it work with OpenAPI on v1
         fetchOptions.tags?.[0] ??
@@ -471,7 +471,7 @@ export const getRevisionPages = cache({
 
         return cacheResponse(response, {
             ...(fetchOptions.metadata ? cacheTtl_7days : cacheTtl_1day),
-            data: { pages: response.data.pages, tags: getResponseCacheTags(response) },
+            data: { ...response.data, tags: getResponseCacheTags(response) },
         });
     },
     getKeyArgs: (args) => [args[0], args[1]],
@@ -663,7 +663,7 @@ export const getRevisionFile = batch<[string, string, string], RevisionFile | nu
             let files: Record<string, RevisionFile> = {};
 
             if (hasRevisionInMemory) {
-                const { revision } = await getRevision(spaceId, revisionId, { metadata: false });
+                const revision = await getRevision(spaceId, revisionId, { metadata: false });
                 files = {};
                 revision.files.forEach((file) => {
                     files[file.id] = file;
@@ -709,7 +709,7 @@ export const getReusableContent = async (
     });
 
     if (hasRevisionInMemory) {
-        const { revision } = await getRevision(spaceId, revisionId, { metadata: false });
+        const revision = await getRevision(spaceId, revisionId, { metadata: false });
         return (
             revision.reusableContents.find(
                 (reusableContent) => reusableContent.id === reusableContentId

packages/gitbook/src/lib/v1.ts

@@ -135,7 +135,7 @@ async function getDataFetcherV1(): Promise<GitBookDataFetcher> {
 
         getRevision(params) {
             return wrapDataFetcherError(async () => {
-                const { revision, tags } = await getRevision(params.spaceId, params.revisionId, {
+                const { tags, ...revision } = await getRevision(params.spaceId, params.revisionId, {
                     metadata: params.metadata,
                     computed: false,
                 });
@@ -145,11 +145,15 @@ async function getDataFetcherV1(): Promise<GitBookDataFetcher> {
                         (page) => page.type === RevisionPageType.Computed
                     )
                 ) {
-                    const { revision } = await getRevision(params.spaceId, params.revisionId, {
-                        metadata: params.metadata,
-                        computed: true,
-                        tags,
-                    });
+                    const { tags: _tags, ...revision } = await getRevision(
+                        params.spaceId,
+                        params.revisionId,
+                        {
+                            metadata: params.metadata,
+                            computed: true,
+                            tags,
+                        }
+                    );
                     return revision;
                 }