fix: remove repository link display on missing param errors (#4560)

* fix: remove repository link display on missing param errors

* fix tests

---------

Co-authored-by: Alexandr <qwerty541zxc@gmail.com>

backend/api-renamed/gist.js

@@ -12,7 +12,10 @@ import {
   setErrorCacheHeaders,
 } from "../src/common/cache.js";
 import { guardAccess } from "../src/common/access.js";
-import { retrieveSecondaryMessage } from "../src/common/error.js";
+import {
+  MissingParamError,
+  retrieveSecondaryMessage,
+} from "../src/common/error.js";
 
 // @ts-ignore
 export default async (req, res) => {
@@ -104,6 +107,7 @@ export default async (req, res) => {
             bg_color,
             border_color,
             theme,
+            show_repo_link: !(err instanceof MissingParamError),
           },
         }),
       );

backend/api-renamed/pin.js

@@ -8,7 +8,10 @@ import {
   setCacheHeaders,
   setErrorCacheHeaders,
 } from "../src/common/cache.js";
-import { retrieveSecondaryMessage } from "../src/common/error.js";
+import {
+  MissingParamError,
+  retrieveSecondaryMessage,
+} from "../src/common/error.js";
 import { parseArray, parseBoolean, renderError } from "../src/common/utils.js";
 import { fetchRepo } from "../src/fetchers/repo.js";
 import { isLocaleAvailable } from "../src/translations.js";
@@ -151,6 +154,7 @@ export default async (req, res) => {
             bg_color,
             border_color,
             theme,
+            show_repo_link: !(err instanceof MissingParamError),
           },
         }),
       );

backend/api-renamed/top-langs.js

@@ -8,7 +8,10 @@ import {
   setCacheHeaders,
   setErrorCacheHeaders,
 } from "../src/common/cache.js";
-import { retrieveSecondaryMessage } from "../src/common/error.js";
+import {
+  MissingParamError,
+  retrieveSecondaryMessage,
+} from "../src/common/error.js";
 import { parseArray, parseBoolean, renderError } from "../src/common/utils.js";
 import { fetchTopLanguages } from "../src/fetchers/top-languages.js";
 import { isLocaleAvailable } from "../src/translations.js";
@@ -167,6 +170,7 @@ export default async (req, res) => {
             bg_color,
             border_color,
             theme,
+            show_repo_link: !(err instanceof MissingParamError),
           },
         }),
       );

backend/api-renamed/wakatime.js

@@ -12,7 +12,10 @@ import {
   setErrorCacheHeaders,
 } from "../src/common/cache.js";
 import { guardAccess } from "../src/common/access.js";
-import { retrieveSecondaryMessage } from "../src/common/error.js";
+import {
+  MissingParamError,
+  retrieveSecondaryMessage,
+} from "../src/common/error.js";
 
 // @ts-ignore
 export default async (req, res) => {
@@ -123,6 +126,7 @@ export default async (req, res) => {
             bg_color,
             border_color,
             theme,
+            show_repo_link: !(err instanceof MissingParamError),
           },
         }),
       );

backend/tests/gist.test.js

@@ -129,6 +129,7 @@ describe("Test /api/gist", () => {
       renderError({
         message: 'Missing params "id" make sure you pass the parameters in URL',
         secondaryMessage: "/api/gist?id=GIST_ID",
+        renderOptions: { show_repo_link: false },
       }),
     );
   });

backend/tests/pin.test.js

@@ -212,6 +212,7 @@ describe("Test /api/pin", () => {
         message:
           'Missing params "username", "repo" make sure you pass the parameters in URL',
         secondaryMessage: "/api/pin?username=USERNAME&repo=REPO_NAME",
+        renderOptions: { show_repo_link: false },
       }),
     );
   });