From bb8c2031e454083e49696bd0f6f1b428f5c05243 Mon Sep 17 00:00:00 2001 From: yan Date: Mon, 29 Apr 2024 22:32:43 -0700 Subject: [PATCH] WIP: run linter on action.yml script using https://github.com/actions/github-script?tab=readme-ov-file#run-a-separate-file-with-an-async-function --- action.js | 128 ++++++++++++++++++++++++++++++++++++++++++++++++++++ action.yml | 130 ++--------------------------------------------------- 2 files changed, 131 insertions(+), 127 deletions(-) create mode 100644 action.js diff --git a/action.js b/action.js new file mode 100644 index 0000000..e87002c --- /dev/null +++ b/action.js @@ -0,0 +1,128 @@ +module.exports = async ({github, context, inputs}) => { + const { default: filterdiff } = await import('${{ github.action_path }}/src/filterdiff.js') + const { default: getConfig } = await import('${{ github.action_path }}/src/getConfig.js') + const { default: getProperties } = await import('${{ github.action_path }}/src/getProperties.js') + + // delete if empty string in inputs value + Object.keys(inputs).forEach(key => inputs[key] === '' && delete inputs[key]) + + let debug = process.env.DEBUG == 'true' + if (debug) console.log('Initializing puLL-Merge') + + const config = await getConfig({ owner: context.repo.owner, repo: context.repo.repo, path: '.github/pull-merge.json', debug, github }) + const properties = await getProperties({ owner: context.repo.owner, repo: context.repo.repo, debug, github }) + + const options = Object.assign({ + debounce_time: '6', + amplification: '4', + filterdiff_args: '--exclude=**/package-lock.json --exclude=**/yarn.lock --exclude=**/*.js.map --exclude=**/*.svg --exclude=**/test/data/**/* --exclude=**/docs/**/* --exclude=**/deploy/**/* --exclude=**/.htpasswd', + openai_models: 'gpt-4-turbo-2024-04-09 gpt-3.5-turbo-0125', + anthropic_models: 'claude-3-opus-20240229', + bedrock_models: 'anthropic.claude-3-opus-20240229-v1:0', + owner: context.repo.owner, + repo: context.repo.repo, + prnum: context.issue.number, + max_tokens: '2048', + subtle_mode: 'false' + }, config, properties, inputs) + + // convert to numbers some options + options.debounce_time = parseFloat(options.debounce_time, 10) + options.amplification = parseFloat(options.amplification, 10) + options.prnum = parseFloat(options.prnum, 10) + options.max_tokens = parseFloat(options.max_tokens, 10) + options.subtle_mode = options.subtle_mode === 'true' + + const { default: explainPatch } = + options.bedrock_aws_iam_role_arn + ? await import('${{ github.action_path }}/src/bedrockExplainPatch.js') + : options.anthropic_api_key + ? await import('${{ github.action_path }}/src/anthropicExplainPatch.js') + : await import('${{ github.action_path }}/src/openaiExplainPatch.js') + + const { default: submitReview } = options.subtle_mode + ? await import('${{ github.action_path }}/src/subtleSubmitReview.js') + : await import('${{ github.action_path }}/src/submitReview.js') + + const { default: getPatch } = (context.payload.pull_request && context.payload.pull_request.user.login === 'renovate[bot]') || context.actor === 'renovate[bot]' + ? await import('${{ github.action_path }}/src/getRenovatePatch.js') + : (context.payload.pull_request && context.payload.pull_request.user.login === 'dependabot[bot]') || context.actor === 'dependabot[bot]' + ? await import('${{ github.action_path }}/src/getDependabotPatch.js') + : await import('${{ github.action_path }}/src/getPatch.js') + + options.key = options.anthropic_api_key || options.openai_api_key + options.models = options.bedrock_aws_iam_role_arn + ? options.bedrock_models + : options.anthropic_api_key + ? options.anthropic_models + : options.openai_models + + debug = options.debug ? (options.debug == 'true') : debug + + if (debug) { + console.log(`Using options: ${JSON.stringify(options)}`) + console.log(`Using config: ${JSON.stringify(config)}`) + console.log(`Using properties: ${JSON.stringify(properties)}`) + console.log(`Using inputs: ${JSON.stringify(inputs)}`) + } + + try { + const patch = await getPatch({ + owner: options.owner, + repo: options.repo, + prnum: options.prnum, + debug, + runIfPrivate: options.run_if_private, + github + }) + + const filteredPatch = await filterdiff({ + content: patch.body, + args: options.filterdiff_args, + debug + }) + + const explainPatchCb = async () => await explainPatch({ + apiKey: options.key, + patchBody: filteredPatch, + owner: patch.owner, + repo: patch.repo, + debug, + models: options.models.split(' '), + amplification: options.amplification, + max_tokens: options.max_tokens, + region: options.region + }) + + let watermark = patch.watermark + + if (debug) { + watermark = options.bedrock_aws_iam_role_arn + ? `bedrock debug - ${watermark}` + : options.anthropic_api_key + ? `anthropic debug - ${watermark}` + : `openai debug - ${watermark}` + } + + await submitReview({ + owner: options.owner, + repo: options.repo, + prnum: options.prnum, + watermark, + explainPatch: explainPatchCb, + debounceTime: options.debounce_time, + debug, + github + }) + + await github.rest.issues.addLabels({ + owner: options.owner, + repo: options.repo, + issue_number: options.prnum, + labels: ['puLL-Merge'] + }) + } catch (error) { + console.log(error) + if (debug) throw error + } +} diff --git a/action.yml b/action.yml index 1048989..b6b64c8 100644 --- a/action.yml +++ b/action.yml @@ -87,130 +87,6 @@ runs: DEBUG: ${{ (inputs.debug == 'true' || runner.debug) && 'true' || 'false'}} with: script: | - const { default: filterdiff } = await import('${{ github.action_path }}/src/filterdiff.js') - const { default: getConfig } = await import('${{ github.action_path }}/src/getConfig.js') - const { default: getProperties } = await import('${{ github.action_path }}/src/getProperties.js') - - const inputs = ${{ toJson(inputs) }}; - // delete if empty string in inputs value - Object.keys(inputs).forEach(key => inputs[key] === '' && delete inputs[key]) - - let debug = process.env.DEBUG == 'true' - if (debug) console.log('Initializing puLL-Merge') - - const config = await getConfig({ owner: context.repo.owner, repo: context.repo.repo, path: '.github/pull-merge.json', debug, github }) - const properties = await getProperties({ owner: context.repo.owner, repo: context.repo.repo, debug, github }) - - const options = Object.assign({ - debounce_time: '6', - amplification: '4', - filterdiff_args: '--exclude=**/package-lock.json --exclude=**/yarn.lock --exclude=**/*.js.map --exclude=**/*.svg --exclude=**/test/data/**/* --exclude=**/docs/**/* --exclude=**/deploy/**/* --exclude=**/.htpasswd', - openai_models: 'gpt-4-turbo-2024-04-09 gpt-3.5-turbo-0125', - anthropic_models: 'claude-3-opus-20240229', - bedrock_models: 'anthropic.claude-3-opus-20240229-v1:0', - owner: context.repo.owner, - repo: context.repo.repo, - prnum: context.issue.number, - max_tokens: '2048', - subtle_mode: 'false' - }, config, properties, inputs) - - // convert to numbers some options - options.debounce_time = parseFloat(options.debounce_time, 10) - options.amplification = parseFloat(options.amplification, 10) - options.prnum = parseFloat(options.prnum, 10) - options.max_tokens = parseFloat(options.max_tokens, 10) - options.subtle_mode = options.subtle_mode === 'true' - - const { default: explainPatch } = - options.bedrock_aws_iam_role_arn - ? await import('${{ github.action_path }}/src/bedrockExplainPatch.js') - : options.anthropic_api_key - ? await import('${{ github.action_path }}/src/anthropicExplainPatch.js') - : await import('${{ github.action_path }}/src/openaiExplainPatch.js') - - const { default: submitReview } = options.subtle_mode - ? await import('${{ github.action_path }}/src/subtleSubmitReview.js') - : await import('${{ github.action_path }}/src/submitReview.js') - - const { default: getPatch } = (context.payload.pull_request && context.payload.pull_request.user.login === 'renovate[bot]') || context.actor === 'renovate[bot]' - ? await import('${{ github.action_path }}/src/getRenovatePatch.js') - : (context.payload.pull_request && context.payload.pull_request.user.login === 'dependabot[bot]') || context.actor === 'dependabot[bot]' - ? await import('${{ github.action_path }}/src/getDependabotPatch.js') - : await import('${{ github.action_path }}/src/getPatch.js') - - options.key = options.anthropic_api_key || options.openai_api_key - options.models = options.bedrock_aws_iam_role_arn - ? options.bedrock_models - : options.anthropic_api_key - ? options.anthropic_models - : options.openai_models - - debug = options.debug ? (options.debug == 'true') : debug - - if (debug) { - console.log(`Using options: ${JSON.stringify(options)}`) - console.log(`Using config: ${JSON.stringify(config)}`) - console.log(`Using properties: ${JSON.stringify(properties)}`) - console.log(`Using inputs: ${JSON.stringify(inputs)}`) - } - - try { - const patch = await getPatch({ - owner: options.owner, - repo: options.repo, - prnum: options.prnum, - debug, - runIfPrivate: options.run_if_private, - github - }) - - const filteredPatch = await filterdiff({ - content: patch.body, - args: options.filterdiff_args, - debug - }) - - const explainPatchCb = async () => await explainPatch({ - apiKey: options.key, - patchBody: filteredPatch, - owner: patch.owner, - repo: patch.repo, - debug, - models: options.models.split(' '), - amplification: options.amplification, - max_tokens: options.max_tokens, - region: options.region - }) - - let watermark = patch.watermark - - if (debug) { - watermark = options.bedrock_aws_iam_role_arn - ? `bedrock debug - ${watermark}` - : options.anthropic_api_key - ? `anthropic debug - ${watermark}` - : `openai debug - ${watermark}` - } - - await submitReview({ - owner: options.owner, - repo: options.repo, - prnum: options.prnum, - watermark, - explainPatch: explainPatchCb, - debounceTime: options.debounce_time, - debug, - github - }) - - await github.rest.issues.addLabels({ - owner: options.owner, - repo: options.repo, - issue_number: options.prnum, - labels: ['puLL-Merge'] - }) - } catch (error) { - console.log(error) - if (debug) throw error - } + const script = require('./action.js') + const inputs = ${{ toJson(inputs) }} + await script({github, context, inputs})