Skip to content

Support installed npm modules and relative require #135

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jclem merged 25 commits into from
Apr 21, 2021
Merged

Support installed npm modules and relative require #135

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
ddba1b1
Use non-Webpack-require in evaluated scripts
jclem Apr 21, 2021
f4e5d39
Use a proxy to support relative requires
jclem Apr 21, 2021
e853490
Run build script
jclem Apr 21, 2021
75e3a5b
Move wrapRequire to its own module
jclem Apr 21, 2021
c416f56
Pass nativeRequire, as well
jclem Apr 21, 2021
83c92d6
Rename nativeRequire to __original_require__
jclem Apr 21, 2021
3ca4cd5
Update documentation for relative require support
jclem Apr 21, 2021
c1c139b
Add a catch to support requiring installed modules
jclem Apr 21, 2021
ade5cea
Use concat for paths
jclem Apr 21, 2021
f57c84a
Try global.module
jclem Apr 21, 2021
1f8fc98
Run build
jclem Apr 21, 2021
d37f92f
Test eval
jclem Apr 21, 2021
7e8659d
Run build
jclem Apr 21, 2021
7e12bd7
Call resolve, then require
jclem Apr 21, 2021
b0e12e7
Add Webpack comment vis-a-vis eval
jclem Apr 21, 2021
01f87b6
Remove caveat about node_modules
jclem Apr 21, 2021
3ede589
Add new integration tests
jclem Apr 21, 2021
19e7914
Add integration test running
jclem Apr 21, 2021
4a93ad9
Add actions/checkout
jclem Apr 21, 2021
fb7e8f7
Add npm ci to integration test
jclem Apr 21, 2021
5ee517d
Workflow syntax error
jclem Apr 21, 2021
256da4e
Update src/wrap-require.ts
jclem Apr 21, 2021
c758586
v4.0.0
jclem Apr 21, 2021
b616178
Merge branch 'wrap-require' of https://github.com/actions/github-scri…
jclem Apr 21, 2021
2923e50
Run build script
jclem Apr 21, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use a proxy to support relative requires
  • Loading branch information
@jclem
jclem committed Apr 21, 2021
commit f4e5d39c2ae1bcb0726aed39da38b49d54fd9661
4 changes: 2 additions & 2 deletions src/async-function.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import * as core from '@actions/core'
import { Context } from '@actions/github/lib/context'
import { GitHub } from '@actions/github/lib/utils'
import {Context} from '@actions/github/lib/context'
import {GitHub} from '@actions/github/lib/utils'
import * as glob from '@actions/glob'
import * as io from '@actions/io'

Expand Down
25 changes: 23 additions & 2 deletions src/main.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,10 @@ import * as core from '@actions/core'
import {context, getOctokit} from '@actions/github'
import * as glob from '@actions/glob'
import * as io from '@actions/io'
import * as path from 'path'
import {callAsyncFunction} from './async-function'

declare const __non_webpack_require__: typeof require
declare const __non_webpack_require__: NodeRequire

process.on('unhandledRejection', handleError)
main().catch(handleError)
Expand All @@ -31,7 +32,14 @@ async function main(): Promise<void> {

// Using property/value shorthand on `require` (e.g. `{require}`) causes compilation errors.
const result = await callAsyncFunction(
{require: __non_webpack_require__, github, context, core, glob, io},
{
require: wrapRequire,
github,
context,
core,
glob,
io
},
script
)

Expand All @@ -54,6 +62,19 @@ async function main(): Promise<void> {
core.setOutput('result', output)
}

const wrapRequire = new Proxy(__non_webpack_require__, {
apply: (target, thisArg, [moduleID]) => {
if (moduleID.startsWith('.')) {
moduleID = path.join(process.cwd(), moduleID)
}
return target.apply(thisArg, [moduleID])
},

get: (target, prop, receiver) => {
Reflect.get(target, prop, receiver)
}
})

// eslint-disable-next-line @typescript-eslint/no-explicit-any
function handleError(err: any): void {
console.error(err)
Expand Down