fix(jargons-editor): login issues by changing the cookie name using appropriate naming convention (#110)

### Description
Fixed login issues by renaming `jargons.dev:token` to `jargondevsToken`
ensuring the access token gets the appropriate cookie value enabling the
authentication to work seamlessly.

### Related Issue
Fixes #109

Author: ShyneADL

src/lib/actions/do-auth.js

@@ -1,37 +1,46 @@
 import app from "../octokit/app.js";
 import { decrypt, encrypt } from "../utils/crypto.js";
 import { GET as getAuthorization } from "../../pages/api/github/oauth/authorize.js";
-import { isObjectEmpty as isStateEmpty, resolveCookieExpiryDate } from "../utils/index.js";
+import {
+  isObjectEmpty as isStateEmpty,
+  resolveCookieExpiryDate,
+} from "../utils/index.js";
 
 /**
  * Authentication action with GitHub OAuth
- * @param {import("astro").AstroGlobal} astroGlobal 
+ * @param {import("astro").AstroGlobal} astroGlobal
  */
 export default async function doAuth(astroGlobal) {
-  const { url: { searchParams }, cookies } = astroGlobal;
+  const {
+    url: { searchParams },
+    cookies,
+  } = astroGlobal;
   const code = searchParams.get("code");
-  const accessToken = cookies.get("jargons.dev:token", {
-    decode: value => decrypt(value)
+  const accessToken = cookies.get("jargonsdevToken", {
+    decode: (value) => decrypt(value),
   });
 
   /**
    * Generate OAuth Url to start authorization flow
    * @todo improvement: store `state` in cookie for later retrieval/comparison with auth `state` in `github/oauth/callback`
-   * @param {{ path: string }} state 
+   * @param {{ path: string }} state
    */
   function getAuthUrl(state) {
     let parsedState = "";
 
-    if (!isStateEmpty(state)){
+    if (!isStateEmpty(state)) {
       if (state.path) parsedState += `path:${state.path}`;
-      const otherStates = String(Object.keys(state)
-        .filter(key => key !== "path")
-        .map(key => key + ":" + state[key]).join("|"));
+      const otherStates = String(
+        Object.keys(state)
+          .filter((key) => key !== "path")
+          .map((key) => key + ":" + state[key])
+          .join("|")
+      );
       if (otherStates.length > 0) parsedState += `|${otherStates}`;
     }
 
     const { url } = app.oauth.getWebFlowAuthorizationUrl({
-      state: parsedState
+      state: parsedState,
     });
 
     return url;
@@ -41,29 +50,29 @@ export default async function doAuth(astroGlobal) {
     if (code) {
       const response = await getAuthorization(astroGlobal);
       const auth = await response.json();
-  
+
       if (auth.accessToken) {
-        cookies.set("jargons.dev:token", auth.accessToken, {
+        cookies.set("jargonsdevToken", auth.accessToken, {
           expires: resolveCookieExpiryDate(28800),
           path: "/",
-          encode: value => encrypt(value)
+          encode: (value) => encrypt(value),
         });
       }
     }
-  
+
     const userOctokit = app.getUserOctokit({ token: accessToken.value });
     const { data } = await userOctokit.request("GET /user");
-  
+
     return {
       getAuthUrl,
       isAuthed: true,
-      authedData: data
-    }
+      authedData: data,
+    };
   } catch (error) {
     return {
       getAuthUrl,
       isAuthed: false,
-      authedData: null
-    }
+      authedData: null,
+    };
   }
-}
+}

src/lib/actions/do-contribution-stats.js

@@ -5,17 +5,19 @@ import { PROJECT_REPO_DETAILS } from "../../../constants.js";
 
 /**
  * Get some jargons contribution stats for current user on the Jargons Editor
- * @param {import("astro").AstroGlobal} astroGlobal 
+ * @param {import("astro").AstroGlobal} astroGlobal
  */
 export default async function doContributionStats(astroGlobal) {
   const { cookies } = astroGlobal;
-  const { repoFullname, repoMainBranchRef } = PROJECT_REPO_DETAILS; 
+  const { repoFullname, repoMainBranchRef } = PROJECT_REPO_DETAILS;
 
-  const accessToken = cookies.get("jargons.dev:token", { decode: value => decrypt(value) });
+  const accessToken = cookies.get("jargonsdevToken", {
+    decode: (value) => decrypt(value),
+  });
   const userOctokit = app.getUserOctokit({ token: accessToken.value });
 
   /**
-   * @todo Implement narrowed search to project's main branch 
+   * @todo Implement narrowed search to project's main branch
    */
   const baseQuery = `repo:${repoFullname} is:pull-request type:pr author:@me`;
   const baseStatsUrlQuery = `is:pr author:@me`;
@@ -25,27 +27,39 @@ export default async function doContributionStats(astroGlobal) {
    * [potential bottleneck]: no way to know whether a PR is "merged" considering that "closed" doesn't mean merged
    */
   const { data: newType } = await userOctokit.request("GET /search/issues", {
-    q: `${baseQuery} label:":book: new word" is:merged is:closed`
+    q: `${baseQuery} label:":book: new word" is:merged is:closed`,
   });
   const { data: editType } = await userOctokit.request("GET /search/issues", {
-    q: `${baseQuery} label:":book: edit word" is:merged is:closed`
-  });
-  const { data: pendingType } = await userOctokit.request("GET /search/issues", {
-    q: `${baseQuery} label:":book: edit word",":book: new word" is:unmerged is:open`
+    q: `${baseQuery} label:":book: edit word" is:merged is:closed`,
   });
+  const { data: pendingType } = await userOctokit.request(
+    "GET /search/issues",
+    {
+      q: `${baseQuery} label:":book: edit word",":book: new word" is:unmerged is:open`,
+    }
+  );
 
   return {
     newWords: {
       count: newType.total_count,
-      url: buildStatsUrl(repoFullname, `${baseStatsUrlQuery} is:merged is:closed label:":book: new word"`)
+      url: buildStatsUrl(
+        repoFullname,
+        `${baseStatsUrlQuery} is:merged is:closed label:":book: new word"`
+      ),
     },
     editedWords: {
       count: editType.total_count,
-      url: buildStatsUrl(repoFullname, `${baseStatsUrlQuery} is:merged is:closed label:":book: edit word"`)
+      url: buildStatsUrl(
+        repoFullname,
+        `${baseStatsUrlQuery} is:merged is:closed label:":book: edit word"`
+      ),
     },
     pendingWords: {
       count: pendingType.total_count,
-      url: buildStatsUrl(repoFullname, `${baseStatsUrlQuery} is:unmerged is:open`)
-    }
-  }
-}
+      url: buildStatsUrl(
+        repoFullname,
+        `${baseStatsUrlQuery} is:unmerged is:open`
+      ),
+    },
+  };
+}

src/lib/actions/do-edit-word.js

@@ -4,15 +4,24 @@ import { PROJECT_REPO_DETAILS } from "../../../constants.js";
 
 /**
  * Edit Word action - meant to be executed on `editor/edit/[word]` route
- * @param {import("astro").AstroGlobal} astroGlobal 
+ * @param {import("astro").AstroGlobal} astroGlobal
  */
 export default async function doEditWord(astroGlobal) {
-  const { cookies, params: { word } } = astroGlobal;
+  const {
+    cookies,
+    params: { word },
+  } = astroGlobal;
 
-  const accessToken = cookies.get("jargons.dev:token", { decode: value => decrypt(value) });
+  const accessToken = cookies.get("jargonsdevToken", {
+    decode: (value) => decrypt(value),
+  });
   const userOctokit = app.getUserOctokit({ token: accessToken.value });
 
-  const response = await getExistingWord(userOctokit, PROJECT_REPO_DETAILS, word);
+  const response = await getExistingWord(
+    userOctokit,
+    PROJECT_REPO_DETAILS,
+    word
+  );
 
   return response;
-}
+}