Feat: Support include/exclude in github action (#135)

.github/workflows/test.yml

@@ -20,6 +20,29 @@ jobs:
         uses: ./
         with:
           project: happy-rat-64
-          root: examples
+          root: action/tests
           entrypoint: hello.ts
           import-map: ./import_map.json
+      - name: Deploy with single include
+        uses: ./
+        with:
+          project: happy-rat-64
+          root: action/tests
+          entrypoint: include_exclude.ts
+          include: include_exclude.ts
+      - name: Deploy with comma-separated exclude 
+        uses: ./
+        with:
+          project: happy-rat-64
+          root: action/tests
+          entrypoint: include_exclude.ts
+          exclude: import_bomb1,import_bomb2
+      - name: Deploy with multiline exclude 
+        uses: ./
+        with:
+          project: happy-rat-64
+          root: action/tests
+          entrypoint: include_exclude.ts
+          exclude: |
+            import_bomb1
+            import_bomb2

action.yml

@@ -16,6 +16,12 @@ inputs:
   import-map:
     description: The path or URL to an import map file
     required: false
+  include:
+    description: Only upload files that match this pattern (multiline and/or comma-separated)
+    required: false
+  exclude:
+    description: Exclude files that match this pattern (multiline and/or comma-separated)
+    required: false
   root:
     description: The path to the directory containing the code and assets to upload
     required: false

action/deps.js

@@ -4006,10 +4006,10 @@ async function calculateGitSha1(bytes) {
     return hashHex;
 }
 function include(path, include, exclude) {
-    if (include && !include.some((pattern)=>path.startsWith(pattern))) {
+    if (include.length && !include.some((pattern)=>path.startsWith(pattern))) {
         return false;
     }
-    if (exclude && exclude.some((pattern)=>path.startsWith(pattern))) {
+    if (exclude.length && exclude.some((pattern)=>path.startsWith(pattern))) {
         return false;
     }
     return true;
@@ -4054,5 +4054,5 @@ async function walk(cwd, dir, files, options) {
 export { parseEntrypoint as parseEntrypoint };
 export { API as API, APIError as APIError };
 export { walk as walk };
-export { fromFileUrl2 as fromFileUrl, resolve2 as resolve };
+export { fromFileUrl2 as fromFileUrl, resolve2 as resolve, normalize3 as normalize };
 

action/index.js

@@ -5,6 +5,7 @@ import {
   API,
   APIError,
   fromFileUrl,
+  normalize,
   parseEntrypoint,
   resolve,
   walk,
@@ -17,6 +18,8 @@ async function main() {
   const projectId = core.getInput("project", { required: true });
   const entrypoint = core.getInput("entrypoint", { required: true });
   const importMap = core.getInput("import-map", {});
+  const include = core.getMultilineInput("include", {});
+  const exclude = core.getMultilineInput("exclude", {});
   const cwd = resolve(process.cwd(), core.getInput("root", {}));
 
   if (github.context.eventName === "pull_request") {
@@ -72,8 +75,12 @@ async function main() {
   core.debug(`Discovering assets in "${cwd}"`);
   const assets = new Map();
   const entries = await walk(cwd, cwd, assets, {
-    include: undefined,
-    exclude: undefined,
+    include: include.flatMap((i) => i.split(",")).map((pattern) =>
+      normalize(pattern)
+    ),
+    exclude: exclude.flatMap((e) => e.split(",")).map((pattern) =>
+      normalize(pattern)
+    ),
   });
   core.debug(`Discovered ${assets.size} assets`);
 

action/tests/hello.ts

@@ -0,0 +1,11 @@
+import { serve } from "std/http/server.ts";
+
+async function handler(_req: Request) {
+  const text = await Deno.readTextFile(new URL(import.meta.url));
+  return new Response(text, {
+    headers: { "content-type": "text/plain; charset=utf8" },
+  });
+}
+
+console.log("Listening on http://localhost:8000");
+serve(handler);

action/tests/import_map.json

@@ -0,0 +1,5 @@
+{
+  "imports": {
+    "std/": "https://deno.land/[email protected]/"
+  }
+}

action/tests/include_exclude.ts

@@ -0,0 +1,17 @@
+try {
+  await Deno.lstat("import_bomb1");
+  throw new Error("BOOM!");
+} catch (e) {
+  if (!(e instanceof Deno.errors.NotFound)) {
+    throw e;
+  }
+}
+try {
+  await Deno.lstat("import_bomb2");
+  throw new Error("BOOM!");
+} catch (e) {
+  if (!(e instanceof Deno.errors.NotFound)) {
+    throw e;
+  }
+}
+Deno.serve(() => new Response("Hello World"));