fix(editor): remove unnecessary gitloader from editable docs loader (#4718)

Fixes FER-

## Short description of the changes made

## What was the motivation & context behind this PR?

## How has this PR been tested?

Author: sbawabe

packages/commons/docs-loader/src/editable-docs-loader.ts

@@ -15,10 +15,7 @@ class EditableDocsLoader implements DocsLoader {
     domain: string;
     fern_token: string | undefined;
 
-    constructor(
-        docsLoader: DocsLoader,
-        private gitLoader?: GitLoader
-    ) {
+    constructor(docsLoader: DocsLoader) {
         this.readOnlyDocsLoader = docsLoader;
         this.domain = docsLoader.domain;
         this.fern_token = docsLoader.fern_token;
@@ -180,14 +177,12 @@ export const createEditableDocsLoader = async ({
     host,
     encodedDocsUrl,
     fernToken,
-    gitLoader,
     forceRevalidate,
     branchName
 }: {
     host: string;
     encodedDocsUrl: string;
     fernToken?: string;
-    gitLoader?: GitLoader;
     forceRevalidate?: boolean;
     branchName?: string;
 }) => {
@@ -201,5 +196,5 @@ export const createEditableDocsLoader = async ({
         skipAuth: true
     });
 
-    return new EditableDocsLoader(docsLoader, gitLoader);
+    return new EditableDocsLoader(docsLoader);
 };

packages/fern-dashboard/src/app/[orgName]/(visual-editor)/editor/[docsUrl]/[branch]/[...slug]/@headertabs/page.tsx

@@ -27,7 +27,7 @@ export default async function HeaderTabsPage({
         docsUrl: parseDocsUrlParam({ docsUrl })
     });
     // Use cached loader - this will reuse the loader created in layout.tsx
-    const loader = await getCachedEditableDocsLoader(host, docsUrl, session.accessToken, branch, githubUrl);
+    const loader = await getCachedEditableDocsLoader(host, docsUrl, session.accessToken, branch);
     const layout = await loader.getLayout();
 
     if (layout.tabsPlacement !== "HEADER") {

packages/fern-dashboard/src/app/[orgName]/(visual-editor)/editor/[docsUrl]/[branch]/[...slug]/layout.tsx

@@ -60,7 +60,7 @@ export default async function VisualEditorPreviewLayout({
     const host = await getHostFromHeaders();
 
     // Use cached loader to prevent duplicate creation across parallel routes
-    const loader = await getCachedEditableDocsLoader(host, docsUrl, session.accessToken, branch, githubUrl);
+    const loader = await getCachedEditableDocsLoader(host, docsUrl, session.accessToken, branch);
 
     const [colors, layout, fonts, config, root, unsafe_fullRoot, files, { basePath }] = await Promise.all([
         loader.getColors(),

packages/fern-dashboard/src/app/[orgName]/(visual-editor)/editor/[docsUrl]/[branch]/[...slug]/page.tsx

@@ -40,7 +40,7 @@ export default async function Page({
     });
 
     // Use cached loader - this will reuse the loader created in layout.tsx
-    const loader = await getCachedEditableDocsLoader(host, docsUrl, session.accessToken, branch, githubUrl);
+    const loader = await getCachedEditableDocsLoader(host, docsUrl, session.accessToken, branch);
 
     const root = await loader.getRoot();
     const navigationSlug = getRootAliasAwareNavigationSlug(slugjoin(slug), root);

packages/fern-dashboard/src/app/services/docs-loader/cachedEditableDocsLoader.ts

@@ -2,7 +2,6 @@ import { createEditableDocsLoader } from "@fern-api/docs-loader";
 import type { DocsLoader } from "@fern-api/docs-server/docs-loader";
 import { cache } from "react";
 import type { EncodedDocsUrl } from "@/utils/types";
-import { GitHubLoader } from "../github/github-loader";
 
 /**
  * Cached version of createEditableDocsLoader to prevent duplicate loader creation
@@ -14,14 +13,12 @@ export const getCachedEditableDocsLoader = cache(
         encodedDocsUrl: EncodedDocsUrl,
         fernToken: string,
         branchName?: string,
-        githubUrl?: string,
         forceRevalidate?: boolean
     ) => {
         return createEditableDocsLoader({
             host,
             encodedDocsUrl,
             fernToken,
-            gitLoader: githubUrl ? new GitHubLoader(githubUrl) : undefined,
             branchName,
             forceRevalidate
         });

packages/fern-dashboard/src/app/services/docs-loader/preloadEditorData.ts

@@ -28,7 +28,6 @@ export default async function preloadEditorData(request: {
             request.docsUrl,
             session.accessToken,
             request.branch,
-            undefined,
             true // force revalidate when preloading
         );