Merge pull request #8 from mudlabs/dev

[dev] Improve path

Author: mudlabs

Diff for: .github/npm.worker.config.yml renamed to .github/npm.worker.config.yaml

@@ -2,6 +2,8 @@ name: Test Diff
 
 on:
   push:
+    paths: 
+      - '*.md'
 
 jobs:
   test:
@@ -14,8 +16,8 @@ jobs:
         id: diff
         uses: ./
         with:
-          path: README.md
           strict: false
+          path: 'READ*.md'
       - run: |
           echo "Added? ${{ steps.diff.outputs.added }}"
           echo "Modified? ${{ steps.diff.outputs.modified }}"

Diff for: .github/workflows/test.yaml

@@ -19,15 +19,15 @@ Very simple, you provide the action with a `path` to a _file_ or _folder_, and i
 ## Usage
 ```yaml
 - name: Simple Diff
-  uses: mudlabs/simple-diff@v1.1.0
+  uses: mudlabs/simple-diff@v1.2.0
   with:
     path: path/to/file   
 ```
 
 ## Inputs
 | Input | Description | Default |
 | --- | :--- | --- |
-| `path` | Specifies a path from _root_ to the file or folder you want to check. | |
+| `path` | Specifies a path from the _root_ of your repository to the file or folder you want to check. If not provided the action will try to find a path specification from the workflow file itself. The path can be a `glob` string. | |
 | `strict` | Specifies the action should fail if the `path` is not in the commits diff tree. | `true` |
 
 ## Outputs
@@ -42,6 +42,7 @@ Very simple, you provide the action with a `path` to a _file_ or _folder_, and i
 
 ## Example Case
 You have a workflow that only runs on a push event to a file path. But you don't want it to run if the file was `removed` _(deleted)_.
+  - _Note:_ In this example we do not specify the path property. If your workflow is conditioned to only run when changes to a given path occure, you don't need to provide the action with the file path. _(assuming that's the file path you want to check)_.
 
 ```yaml
 name: My File Workflow
@@ -57,10 +58,8 @@ jobs:
     steps:
       - uses: actions/checkout@v2
       - name: Simple Diff
+        uses: mudlabs/[email protected]
         id: diff
-        uses: mudlabs/[email protected]
-        with:
-          path: path/to/my/file.ext
       - run: exit 1
         if: steps.diff.outputs.removed == true
   

Diff for: README.md

@@ -1,4 +1,7 @@
+const fs = require("fs");
+const yaml = require("js-yaml");
 const core = require("@actions/core");
+const minimatch = require("minimatch");
 const github = require("@actions/github");
 
 const unsupportedEvent = name => name !== "pull_request" && name !== "push" ? true : false;
@@ -7,9 +10,40 @@ const getHead = name => data => name === "pull_request" ? data.pull_request.head
 const normalise = path => path.split("/").filter(item => item !== "" && item !== ".").join("/");
 const toBoolean = value => value.toLowerCase() == "true";
 
+const setFromPath = owner => repo => async octokit => {
+  try {
+    const input_path = core.getInput("path");
+    if (input_path) return normalise(input_path);
+    
+    console.warn("No path provided. Attempting to discern a path from the workflow file.");
+    
+    const event_name = github.context.eventName;
+    const workflows = await octokit.request("GET /repos/:owner/:repo/actions/workflows", { owner, repo });
+    const workflow = workflows.data.workflows.find(workflow => workflow.name === process.env.GITHUB_WORKFLOW);
+    const file = await fs.promises.readFile(workflow.path);
+    const data = yaml.safeLoad(file);
+    const path = data.on[event_name] && data.on[event_name].paths ? data.on[event_name].paths[0] : undefined;
+    
+    if (!path) throw new Error(`No path specified within the workflow file for this (${event_name}) event.`);
+    
+    console.log(`Using the specified path (${path}), for this ${event_name} event`);
+    return normalise(path);
+  } catch(error) {
+    console.error(error.message);
+    return undefined;
+  }
+}
+
+const contentsUrlDoesMatch = file => target => {
+  const contents_url = decodeURIComponent(file.contents_url);
+  const start = contents_url.indexOf("contents/");
+  const end = contents_url.indexOf("?ref=");
+  const contents_path = contents_url.substring(start, end);
+  return minimatch(contents_path, `contents/${target}`);
+};
+
 (async function(){
   try {
-    const path = core.getInput("path");
     const token = core.getInput("token");
     const repo = github.context.repo.repo;
     const owner = github.context.repo.owner;
@@ -26,9 +60,9 @@ const toBoolean = value => value.toLowerCase() == "true";
     if (response.status !== 200) throw `The API request for this ${github.context.eventName} event returned ${response.status}, expected 200.`;
     if (response.data.status !== "ahead") throw `The head commit for this ${github.context.eventName} event is not ahead of the base commit.`;
 
-    const target = normalise(path);
     const files = response.data.files;
-    const file = files.find(file => decodeURIComponent(file.contents_url).indexOf(`contents/${target}`) !== -1);
+    const target = await setFromPath(owner)(repo)(octokit);
+    const file = files.find(file => contentsUrlDoesMatch(file)(target));
 
     core.setOutput("name", file ? file.filename : target);
     core.setOutput("added", file ? file.status === "added" : false);
@@ -38,8 +72,8 @@ const toBoolean = value => value.toLowerCase() == "true";
     core.setOutput("previous", file ? file.previous_filename || file.filename : target);
 
     if (file) return;
-    if (strict) throw `None of the files in this commits diff tree match the provided file (${path}).`;
-    console.log(`None of the files in this commits diff tree match the provided file (${path}).`);
+    if (strict) throw `None of the files in this commits diff tree match the provided file (${target}).`;
+    console.log(`None of the files in this commits diff tree match the provided file (${target}).`);
 
   } catch (error) {
     core.setFailed(error);