Merge branch 'dev'

.github/README.md

@@ -12,6 +12,7 @@
 <br />
 [![Test](https://img.shields.io/github/actions/workflow/status/5ouma/reproxy/test.yml?label=test&style=flat-square)](https://github.com/5ouma/reproxy/actions/workflows/test.yml)
 [![pre-commit.ci status](https://results.pre-commit.ci/badge/github/5ouma/reproxy/main.svg?style=flat-square)](https://results.pre-commit.ci/latest/github/5ouma/reproxy/main)
+[![codecov](https://codecov.io/github/5ouma/reproxy/graph/badge.svg?token=OQB55KXJIL)](https://codecov.io/github/5ouma/reproxy)
 
 </div>
 

.github/workflows/test.yml

@@ -28,5 +28,19 @@ jobs:
         with:
           deno-version: v1.x
 
-      - name: 🧪 Test Libraries
-        run: deno task test
+      - name: 🧪 Run Tests
+        run: deno task cov
+
+      - name: 🧹 Lint Check
+        run: deno lint
+
+      - name: 📝 Format Check
+        run: deno fmt --check
+
+      - name: 🔍 Type Check
+        run: deno check ./**/*.ts
+
+      - name: ☂️ Upload Coverage
+        uses: codecov/codecov-action@v4
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}

.gitignore

@@ -1 +1,3 @@
 /.env
+/coverage.lcov
+/coverage

deno.json

@@ -6,16 +6,17 @@
     "run": "deno run --env='.env' --allow-env='REPOSITORY_OWNER,REPOSITORY_NAME,REPOSITORY_PATH,GITHUB_TOKEN' --allow-net='0.0.0.0,api.github.com'",
     "start": "deno task run ./src/server.ts",
     "dev": "deno task run --watch ./src/server.ts",
-    "test": "deno test --allow-env='REPOSITORY_OWNER,REPOSITORY_NAME,REPOSITORY_PATH,GITHUB_TOKEN' --allow-net='api.github.com' --parallel --shuffle"
+    "test": "deno test --allow-env='REPOSITORY_OWNER,REPOSITORY_NAME,REPOSITORY_PATH,GITHUB_TOKEN' --allow-net='api.github.com' --parallel --shuffle",
+    "cov": "deno task test --coverage && deno coverage --lcov > coverage.lcov"
   },
   "imports": {
-    "@oak/oak": "jsr:@oak/oak@16.0.0",
+    "@oak/oak": "jsr:@oak/oak@16.1.0",
     "@octokit/rest": "https://esm.sh/@octokit/[email protected]",
     "@std/assert": "jsr:@std/[email protected]",
-    "@std/fmt": "jsr:@std/[email protected].3",
-    "@std/http": "jsr:@std/[email protected].3",
+    "@std/fmt": "jsr:@std/[email protected].4",
+    "@std/http": "jsr:@std/[email protected].4",
     "@std/http/user-agent": "jsr:@std/http@~0.223/user-agent",
-    "@std/path": "jsr:@std/path@0.225.2"
+    "@std/url": "jsr:@std/url@0.224.1"
   },
   "deploy": {
     "project": "reproxy",

src/libs/content.ts

@@ -4,7 +4,7 @@ import { getRepository, githubToken } from "./env.ts";
 
 export async function getContent<R extends string>(
   ctx: RouterContext<R>,
-  ref: string | undefined = undefined
+  ref: string | undefined = undefined,
 ): Promise<void> {
   const octokit = new Octokit({ auth: githubToken });
   const repository = getRepository();

src/libs/env.ts

@@ -7,9 +7,11 @@ export function getRepository(): Repository {
     path: Deno.env.get("REPOSITORY_PATH") ?? "",
   };
 
-  for (const key in repository)
-    if (!repository[key as keyof Repository])
+  for (const key in repository) {
+    if (!repository[key as keyof Repository]) {
       console.error(`🚨 missing: $REPOSITORY_${key.toUpperCase()}`);
+    }
+  }
 
   return repository;
 }

src/libs/redirect.ts

@@ -1,25 +1,23 @@
 import type { RouterContext } from "@oak/oak";
 import { STATUS_CODE } from "@std/http/status";
 import { UserAgent } from "@std/http/user-agent";
-import { join } from "@std/path";
+import { join } from "@std/url";
 import { getRepository } from "./env.ts";
 
 export function redirect<R extends string>(
   ctx: RouterContext<R>,
   userAgent: UserAgent,
-  ref: string = "HEAD"
+  ref: string = "master",
 ): void {
   const repository = getRepository();
-  const url: string =
-    "https://" +
-    join(
-      "github.com",
-      repository.owner,
-      repository.name,
-      "blob",
-      ref,
-      repository.path
-    );
+  const url = join(
+    new URL("https://github.com"),
+    repository.owner,
+    repository.name,
+    "blob",
+    ref,
+    repository.path,
+  );
 
   if (userAgent?.browser.name) {
     ctx.response.status = STATUS_CODE.PermanentRedirect;

src/server.ts

@@ -12,11 +12,13 @@ app.addEventListener(
   "listen",
   ({ secure, hostname, port }: ApplicationListenEvent) => {
     console.log(
-      `🔔 listening: ${yellow(
-        `${secure ? "https" : "http"}://${hostname ?? "localhost"}:${port}`
-      )}`
+      `🔔 listening: ${
+        yellow(
+          `${secure ? "https" : "http"}://${hostname ?? "localhost"}:${port}`,
+        )
+      }`,
     );
-  }
+  },
 );
 
 await app.listen({ port: 8080 });

test/libs/content_test.ts

@@ -1,8 +1,8 @@
-import { testing, type RouterContext } from "@oak/oak";
+import { type RouterContext, testing } from "@oak/oak";
 import { assertEquals, assertStringIncludes } from "@std/assert";
 import { STATUS_CODE } from "@std/http/status";
 import { getContent } from "../../src/libs/content.ts";
-import { testRepo, unknownRepo, testRef, exportRepo } from "../utils.ts";
+import { exportRepo, testRef, testRepo, unknownRepo } from "../utils.ts";
 
 Deno.test("Get Content", async <R extends string>() => {
   const ctx: RouterContext<R> = testing.createMockContext();
@@ -28,6 +28,6 @@ Deno.test("Get Content (Not found)", async <R extends string>() => {
   assertEquals(ctx.response.status, STATUS_CODE.NotFound);
   assertStringIncludes(
     ctx.response.body!.toString(),
-    `⚠️ ${STATUS_CODE.NotFound}:`
+    `⚠️ ${STATUS_CODE.NotFound}:`,
   );
 });

test/libs/env_test.ts

@@ -1,6 +1,6 @@
 import { assertEquals } from "@std/assert";
 import { getRepository } from "../../src/libs/env.ts";
-import { testRepo, exportRepo, clearRepo } from "../utils.ts";
+import { clearRepo, exportRepo, testRepo } from "../utils.ts";
 
 Deno.test("Get Repository Env", () => {
   exportRepo(testRepo);

test/libs/redirect_test.ts

@@ -1,9 +1,9 @@
-import { testing, type RouterContext } from "@oak/oak";
+import { type RouterContext, testing } from "@oak/oak";
 import { assertEquals } from "@std/assert";
 import { STATUS_CODE } from "@std/http/status";
 import { UserAgent } from "@std/http/user-agent";
 import { redirect } from "../../src/libs/redirect.ts";
-import { testRepo, testRef, exportRepo } from "../utils.ts";
+import { exportRepo, testRef, testRepo } from "../utils.ts";
 
 Deno.test("Redirect Detection", <R extends string>() => {
   const ctx: RouterContext<R> = testing.createMockContext();
@@ -13,7 +13,7 @@ Deno.test("Redirect Detection", <R extends string>() => {
   assertEquals(ctx.response.status, STATUS_CODE.PermanentRedirect);
   assertEquals(
     ctx.response.headers.get("location"),
-    `https://github.com/${testRepo.owner}/${testRepo.name}/blob/HEAD/${testRepo.path}`
+    `https://github.com/${testRepo.owner}/${testRepo.name}/blob/master/${testRepo.path}`,
   );
 });
 
@@ -25,6 +25,6 @@ Deno.test("Redirect Detection (With ref)", <R extends string>() => {
   assertEquals(ctx.response.status, STATUS_CODE.PermanentRedirect);
   assertEquals(
     ctx.response.headers.get("location"),
-    `https://github.com/${testRepo.owner}/${testRepo.name}/blob/${testRef}/${testRepo.path}`
+    `https://github.com/${testRepo.owner}/${testRepo.name}/blob/${testRef}/${testRepo.path}`,
   );
 });