Skip to content

Commit

Permalink
fix(github): correctly convert file: URL to path (#131)
Browse files Browse the repository at this point in the history
* fix(github): correctly convert `file:` URL to path

* Apply suggestions from code review

---------

Co-authored-by: Moshe Atlow <[email protected]>
  • Loading branch information
aduh95 and MoLow authored Dec 10, 2024
1 parent a421b9a commit 31843fc
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions packages/github/index.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
'use strict';

const path = require('node:path');
const { fileURLToPath } = require('node:url');
const util = require('node:util');
const { EOL } = require('node:os');
const core = require('@actions/core');
Expand All @@ -12,11 +13,14 @@ const stack = new StackUtils({ cwd: WORKSPACE, internals: StackUtils.nodeInterna

const isFile = (name) => name?.startsWith(WORKSPACE);

const getRelativeFilePath = (name) => (isFile(name) ? path.relative(WORKSPACE, require.resolve(name) ?? '') : null);
const getRelativeFilePath = (name) => (isFile(name) ? path.relative(WORKSPACE, name) : null);

function getFilePath(fileName) {
if (fileName.startsWith('file://')) {
return getRelativeFilePath(new URL(fileName).pathname);
return getRelativeFilePath(fileURLToPath(fileName));
}
if (!path.isAbsolute(fileName)) {
return getRelativeFilePath(path.resolve(fileName) ?? '');
}
return getRelativeFilePath(fileName);
}
Expand Down

0 comments on commit 31843fc

Please sign in to comment.