eslint-plugin-github-actions-2 1.0.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +7 -0
- package/LICENSE +21 -0
- package/README.md +164 -0
- package/dist/_internal/case-police-dictionary.d.ts +10 -0
- package/dist/_internal/case-police-dictionary.d.ts.map +1 -0
- package/dist/_internal/case-police-dictionary.js +486 -0
- package/dist/_internal/case-police-dictionary.js.map +1 -0
- package/dist/_internal/casing.d.ts +17 -0
- package/dist/_internal/casing.d.ts.map +1 -0
- package/dist/_internal/casing.js +197 -0
- package/dist/_internal/casing.js.map +1 -0
- package/dist/_internal/github-actions-config-references.d.ts +16 -0
- package/dist/_internal/github-actions-config-references.d.ts.map +1 -0
- package/dist/_internal/github-actions-config-references.js +76 -0
- package/dist/_internal/github-actions-config-references.js.map +1 -0
- package/dist/_internal/github-actions-trigger-events.d.ts +13 -0
- package/dist/_internal/github-actions-trigger-events.d.ts.map +1 -0
- package/dist/_internal/github-actions-trigger-events.js +47 -0
- package/dist/_internal/github-actions-trigger-events.js.map +1 -0
- package/dist/_internal/github-expressions.d.ts +9 -0
- package/dist/_internal/github-expressions.d.ts.map +1 -0
- package/dist/_internal/github-expressions.js +87 -0
- package/dist/_internal/github-expressions.js.map +1 -0
- package/dist/_internal/lint-targets.d.ts +21 -0
- package/dist/_internal/lint-targets.d.ts.map +1 -0
- package/dist/_internal/lint-targets.js +60 -0
- package/dist/_internal/lint-targets.js.map +1 -0
- package/dist/_internal/rule-docs.d.ts +16 -0
- package/dist/_internal/rule-docs.d.ts.map +1 -0
- package/dist/_internal/rule-docs.js +2 -0
- package/dist/_internal/rule-docs.js.map +1 -0
- package/dist/_internal/rules-registry.d.ts +145 -0
- package/dist/_internal/rules-registry.d.ts.map +1 -0
- package/dist/_internal/rules-registry.js +144 -0
- package/dist/_internal/rules-registry.js.map +1 -0
- package/dist/_internal/workflow-template-properties.d.ts +20 -0
- package/dist/_internal/workflow-template-properties.d.ts.map +1 -0
- package/dist/_internal/workflow-template-properties.js +44 -0
- package/dist/_internal/workflow-template-properties.js.map +1 -0
- package/dist/_internal/workflow-yaml.d.ts +47 -0
- package/dist/_internal/workflow-yaml.d.ts.map +1 -0
- package/dist/_internal/workflow-yaml.js +140 -0
- package/dist/_internal/workflow-yaml.js.map +1 -0
- package/dist/_internal/yaml-traversal.d.ts +10 -0
- package/dist/_internal/yaml-traversal.d.ts.map +1 -0
- package/dist/_internal/yaml-traversal.js +38 -0
- package/dist/_internal/yaml-traversal.js.map +1 -0
- package/dist/plugin.cjs +7107 -0
- package/dist/plugin.cjs.map +7 -0
- package/dist/plugin.d.cts +34 -0
- package/dist/plugin.d.ts +34 -0
- package/dist/plugin.d.ts.map +1 -0
- package/dist/plugin.js +122 -0
- package/dist/plugin.js.map +1 -0
- package/dist/rules/action-name-casing.d.ts +9 -0
- package/dist/rules/action-name-casing.d.ts.map +1 -0
- package/dist/rules/action-name-casing.js +141 -0
- package/dist/rules/action-name-casing.js.map +1 -0
- package/dist/rules/job-id-casing.d.ts +9 -0
- package/dist/rules/job-id-casing.d.ts.map +1 -0
- package/dist/rules/job-id-casing.js +123 -0
- package/dist/rules/job-id-casing.js.map +1 -0
- package/dist/rules/max-jobs-per-action.d.ts +9 -0
- package/dist/rules/max-jobs-per-action.d.ts.map +1 -0
- package/dist/rules/max-jobs-per-action.js +61 -0
- package/dist/rules/max-jobs-per-action.js.map +1 -0
- package/dist/rules/no-case-insensitive-input-id-collision.d.ts +9 -0
- package/dist/rules/no-case-insensitive-input-id-collision.d.ts.map +1 -0
- package/dist/rules/no-case-insensitive-input-id-collision.js +66 -0
- package/dist/rules/no-case-insensitive-input-id-collision.js.map +1 -0
- package/dist/rules/no-composite-input-env-access.d.ts +9 -0
- package/dist/rules/no-composite-input-env-access.d.ts.map +1 -0
- package/dist/rules/no-composite-input-env-access.js +62 -0
- package/dist/rules/no-composite-input-env-access.js.map +1 -0
- package/dist/rules/no-deprecated-node-runtime.d.ts +9 -0
- package/dist/rules/no-deprecated-node-runtime.d.ts.map +1 -0
- package/dist/rules/no-deprecated-node-runtime.js +59 -0
- package/dist/rules/no-deprecated-node-runtime.js.map +1 -0
- package/dist/rules/no-duplicate-composite-step-id.d.ts +9 -0
- package/dist/rules/no-duplicate-composite-step-id.d.ts.map +1 -0
- package/dist/rules/no-duplicate-composite-step-id.js +74 -0
- package/dist/rules/no-duplicate-composite-step-id.js.map +1 -0
- package/dist/rules/no-empty-template-file-pattern.d.ts +9 -0
- package/dist/rules/no-empty-template-file-pattern.d.ts.map +1 -0
- package/dist/rules/no-empty-template-file-pattern.js +49 -0
- package/dist/rules/no-empty-template-file-pattern.js.map +1 -0
- package/dist/rules/no-external-job.d.ts +9 -0
- package/dist/rules/no-external-job.d.ts.map +1 -0
- package/dist/rules/no-external-job.js +49 -0
- package/dist/rules/no-external-job.js.map +1 -0
- package/dist/rules/no-hardcoded-default-branch-in-template.d.ts +9 -0
- package/dist/rules/no-hardcoded-default-branch-in-template.d.ts.map +1 -0
- package/dist/rules/no-hardcoded-default-branch-in-template.js +54 -0
- package/dist/rules/no-hardcoded-default-branch-in-template.js.map +1 -0
- package/dist/rules/no-icon-file-extension-in-template-icon-name.d.ts +9 -0
- package/dist/rules/no-icon-file-extension-in-template-icon-name.d.ts.map +1 -0
- package/dist/rules/no-icon-file-extension-in-template-icon-name.js +54 -0
- package/dist/rules/no-icon-file-extension-in-template-icon-name.js.map +1 -0
- package/dist/rules/no-inherit-secrets.d.ts +9 -0
- package/dist/rules/no-inherit-secrets.d.ts.map +1 -0
- package/dist/rules/no-inherit-secrets.js +57 -0
- package/dist/rules/no-inherit-secrets.js.map +1 -0
- package/dist/rules/no-invalid-concurrency-context.d.ts +9 -0
- package/dist/rules/no-invalid-concurrency-context.d.ts.map +1 -0
- package/dist/rules/no-invalid-concurrency-context.js +141 -0
- package/dist/rules/no-invalid-concurrency-context.js.map +1 -0
- package/dist/rules/no-invalid-key.d.ts +9 -0
- package/dist/rules/no-invalid-key.d.ts.map +1 -0
- package/dist/rules/no-invalid-key.js +152 -0
- package/dist/rules/no-invalid-key.js.map +1 -0
- package/dist/rules/no-invalid-reusable-workflow-job-key.d.ts +9 -0
- package/dist/rules/no-invalid-reusable-workflow-job-key.d.ts.map +1 -0
- package/dist/rules/no-invalid-reusable-workflow-job-key.js +73 -0
- package/dist/rules/no-invalid-reusable-workflow-job-key.js.map +1 -0
- package/dist/rules/no-invalid-template-file-pattern-regex.d.ts +9 -0
- package/dist/rules/no-invalid-template-file-pattern-regex.d.ts.map +1 -0
- package/dist/rules/no-invalid-template-file-pattern-regex.js +58 -0
- package/dist/rules/no-invalid-template-file-pattern-regex.js.map +1 -0
- package/dist/rules/no-invalid-workflow-call-output-value.d.ts +9 -0
- package/dist/rules/no-invalid-workflow-call-output-value.d.ts.map +1 -0
- package/dist/rules/no-invalid-workflow-call-output-value.js +99 -0
- package/dist/rules/no-invalid-workflow-call-output-value.js.map +1 -0
- package/dist/rules/no-path-separators-in-template-icon-name.d.ts +9 -0
- package/dist/rules/no-path-separators-in-template-icon-name.d.ts.map +1 -0
- package/dist/rules/no-path-separators-in-template-icon-name.js +52 -0
- package/dist/rules/no-path-separators-in-template-icon-name.js.map +1 -0
- package/dist/rules/no-post-if-without-post.d.ts +9 -0
- package/dist/rules/no-post-if-without-post.d.ts.map +1 -0
- package/dist/rules/no-post-if-without-post.js +52 -0
- package/dist/rules/no-post-if-without-post.js.map +1 -0
- package/dist/rules/no-pr-head-checkout-in-pull-request-target.d.ts +12 -0
- package/dist/rules/no-pr-head-checkout-in-pull-request-target.d.ts.map +1 -0
- package/dist/rules/no-pr-head-checkout-in-pull-request-target.js +89 -0
- package/dist/rules/no-pr-head-checkout-in-pull-request-target.js.map +1 -0
- package/dist/rules/no-pre-if-without-pre.d.ts +9 -0
- package/dist/rules/no-pre-if-without-pre.d.ts.map +1 -0
- package/dist/rules/no-pre-if-without-pre.js +51 -0
- package/dist/rules/no-pre-if-without-pre.js.map +1 -0
- package/dist/rules/no-required-input-with-default.d.ts +9 -0
- package/dist/rules/no-required-input-with-default.d.ts.map +1 -0
- package/dist/rules/no-required-input-with-default.js +68 -0
- package/dist/rules/no-required-input-with-default.js.map +1 -0
- package/dist/rules/no-secrets-in-if.d.ts +9 -0
- package/dist/rules/no-secrets-in-if.d.ts.map +1 -0
- package/dist/rules/no-secrets-in-if.js +84 -0
- package/dist/rules/no-secrets-in-if.js.map +1 -0
- package/dist/rules/no-self-hosted-runner-on-fork-pr-events.d.ts +9 -0
- package/dist/rules/no-self-hosted-runner-on-fork-pr-events.d.ts.map +1 -0
- package/dist/rules/no-self-hosted-runner-on-fork-pr-events.js +93 -0
- package/dist/rules/no-self-hosted-runner-on-fork-pr-events.js.map +1 -0
- package/dist/rules/no-subdirectory-template-file-pattern.d.ts +9 -0
- package/dist/rules/no-subdirectory-template-file-pattern.d.ts.map +1 -0
- package/dist/rules/no-subdirectory-template-file-pattern.js +52 -0
- package/dist/rules/no-subdirectory-template-file-pattern.js.map +1 -0
- package/dist/rules/no-template-placeholder-in-non-template-workflow.d.ts +9 -0
- package/dist/rules/no-template-placeholder-in-non-template-workflow.d.ts.map +1 -0
- package/dist/rules/no-template-placeholder-in-non-template-workflow.js +55 -0
- package/dist/rules/no-template-placeholder-in-non-template-workflow.js.map +1 -0
- package/dist/rules/no-top-level-env.d.ts +9 -0
- package/dist/rules/no-top-level-env.d.ts.map +1 -0
- package/dist/rules/no-top-level-env.js +42 -0
- package/dist/rules/no-top-level-env.js.map +1 -0
- package/dist/rules/no-top-level-permissions.d.ts +9 -0
- package/dist/rules/no-top-level-permissions.d.ts.map +1 -0
- package/dist/rules/no-top-level-permissions.js +39 -0
- package/dist/rules/no-top-level-permissions.js.map +1 -0
- package/dist/rules/no-universal-template-file-pattern.d.ts +9 -0
- package/dist/rules/no-universal-template-file-pattern.d.ts.map +1 -0
- package/dist/rules/no-universal-template-file-pattern.js +60 -0
- package/dist/rules/no-universal-template-file-pattern.js.map +1 -0
- package/dist/rules/no-unknown-input-reference-in-composite.d.ts +9 -0
- package/dist/rules/no-unknown-input-reference-in-composite.d.ts.map +1 -0
- package/dist/rules/no-unknown-input-reference-in-composite.js +80 -0
- package/dist/rules/no-unknown-input-reference-in-composite.js.map +1 -0
- package/dist/rules/no-unknown-job-output-reference.d.ts +9 -0
- package/dist/rules/no-unknown-job-output-reference.d.ts.map +1 -0
- package/dist/rules/no-unknown-job-output-reference.js +229 -0
- package/dist/rules/no-unknown-job-output-reference.js.map +1 -0
- package/dist/rules/no-unknown-step-reference.d.ts +9 -0
- package/dist/rules/no-unknown-step-reference.d.ts.map +1 -0
- package/dist/rules/no-unknown-step-reference.js +133 -0
- package/dist/rules/no-unknown-step-reference.js.map +1 -0
- package/dist/rules/no-untrusted-input-in-run.d.ts +9 -0
- package/dist/rules/no-untrusted-input-in-run.d.ts.map +1 -0
- package/dist/rules/no-untrusted-input-in-run.js +115 -0
- package/dist/rules/no-untrusted-input-in-run.js.map +1 -0
- package/dist/rules/no-unused-input-in-composite.d.ts +9 -0
- package/dist/rules/no-unused-input-in-composite.d.ts.map +1 -0
- package/dist/rules/no-unused-input-in-composite.js +78 -0
- package/dist/rules/no-unused-input-in-composite.js.map +1 -0
- package/dist/rules/no-write-all-permissions.d.ts +9 -0
- package/dist/rules/no-write-all-permissions.d.ts.map +1 -0
- package/dist/rules/no-write-all-permissions.js +61 -0
- package/dist/rules/no-write-all-permissions.js.map +1 -0
- package/dist/rules/pin-action-shas.d.ts +12 -0
- package/dist/rules/pin-action-shas.d.ts.map +1 -0
- package/dist/rules/pin-action-shas.js +100 -0
- package/dist/rules/pin-action-shas.js.map +1 -0
- package/dist/rules/prefer-action-yml.d.ts +9 -0
- package/dist/rules/prefer-action-yml.d.ts.map +1 -0
- package/dist/rules/prefer-action-yml.js +37 -0
- package/dist/rules/prefer-action-yml.js.map +1 -0
- package/dist/rules/prefer-fail-fast.d.ts +9 -0
- package/dist/rules/prefer-fail-fast.d.ts.map +1 -0
- package/dist/rules/prefer-fail-fast.js +61 -0
- package/dist/rules/prefer-fail-fast.js.map +1 -0
- package/dist/rules/prefer-file-extension.d.ts +9 -0
- package/dist/rules/prefer-file-extension.d.ts.map +1 -0
- package/dist/rules/prefer-file-extension.js +104 -0
- package/dist/rules/prefer-file-extension.js.map +1 -0
- package/dist/rules/prefer-inputs-context.d.ts +9 -0
- package/dist/rules/prefer-inputs-context.d.ts.map +1 -0
- package/dist/rules/prefer-inputs-context.js +91 -0
- package/dist/rules/prefer-inputs-context.js.map +1 -0
- package/dist/rules/prefer-step-uses-style.d.ts +9 -0
- package/dist/rules/prefer-step-uses-style.d.ts.map +1 -0
- package/dist/rules/prefer-step-uses-style.js +209 -0
- package/dist/rules/prefer-step-uses-style.js.map +1 -0
- package/dist/rules/prefer-template-yml-extension.d.ts +9 -0
- package/dist/rules/prefer-template-yml-extension.d.ts.map +1 -0
- package/dist/rules/prefer-template-yml-extension.js +41 -0
- package/dist/rules/prefer-template-yml-extension.js.map +1 -0
- package/dist/rules/require-action-name.d.ts +9 -0
- package/dist/rules/require-action-name.d.ts.map +1 -0
- package/dist/rules/require-action-name.js +57 -0
- package/dist/rules/require-action-name.js.map +1 -0
- package/dist/rules/require-action-run-name.d.ts +9 -0
- package/dist/rules/require-action-run-name.d.ts.map +1 -0
- package/dist/rules/require-action-run-name.js +56 -0
- package/dist/rules/require-action-run-name.js.map +1 -0
- package/dist/rules/require-checkout-before-local-action.d.ts +9 -0
- package/dist/rules/require-checkout-before-local-action.d.ts.map +1 -0
- package/dist/rules/require-checkout-before-local-action.js +74 -0
- package/dist/rules/require-checkout-before-local-action.js.map +1 -0
- package/dist/rules/require-composite-step-name.d.ts +9 -0
- package/dist/rules/require-composite-step-name.d.ts.map +1 -0
- package/dist/rules/require-composite-step-name.js +67 -0
- package/dist/rules/require-composite-step-name.js.map +1 -0
- package/dist/rules/require-job-name.d.ts +9 -0
- package/dist/rules/require-job-name.d.ts.map +1 -0
- package/dist/rules/require-job-name.js +70 -0
- package/dist/rules/require-job-name.js.map +1 -0
- package/dist/rules/require-job-step-name.d.ts +9 -0
- package/dist/rules/require-job-step-name.d.ts.map +1 -0
- package/dist/rules/require-job-step-name.js +71 -0
- package/dist/rules/require-job-step-name.js.map +1 -0
- package/dist/rules/require-job-timeout-minutes.d.ts +9 -0
- package/dist/rules/require-job-timeout-minutes.d.ts.map +1 -0
- package/dist/rules/require-job-timeout-minutes.js +99 -0
- package/dist/rules/require-job-timeout-minutes.js.map +1 -0
- package/dist/rules/require-merge-group-trigger.d.ts +9 -0
- package/dist/rules/require-merge-group-trigger.d.ts.map +1 -0
- package/dist/rules/require-merge-group-trigger.js +65 -0
- package/dist/rules/require-merge-group-trigger.js.map +1 -0
- package/dist/rules/require-pull-request-target-branches.d.ts +9 -0
- package/dist/rules/require-pull-request-target-branches.d.ts.map +1 -0
- package/dist/rules/require-pull-request-target-branches.js +102 -0
- package/dist/rules/require-pull-request-target-branches.js.map +1 -0
- package/dist/rules/require-run-step-shell.d.ts +9 -0
- package/dist/rules/require-run-step-shell.d.ts.map +1 -0
- package/dist/rules/require-run-step-shell.js +114 -0
- package/dist/rules/require-run-step-shell.js.map +1 -0
- package/dist/rules/require-template-categories.d.ts +9 -0
- package/dist/rules/require-template-categories.d.ts.map +1 -0
- package/dist/rules/require-template-categories.js +55 -0
- package/dist/rules/require-template-categories.js.map +1 -0
- package/dist/rules/require-template-file-patterns.d.ts +9 -0
- package/dist/rules/require-template-file-patterns.d.ts.map +1 -0
- package/dist/rules/require-template-file-patterns.js +55 -0
- package/dist/rules/require-template-file-patterns.js.map +1 -0
- package/dist/rules/require-template-icon-file-exists.d.ts +9 -0
- package/dist/rules/require-template-icon-file-exists.d.ts.map +1 -0
- package/dist/rules/require-template-icon-file-exists.js +61 -0
- package/dist/rules/require-template-icon-file-exists.js.map +1 -0
- package/dist/rules/require-template-icon-name.d.ts +9 -0
- package/dist/rules/require-template-icon-name.d.ts.map +1 -0
- package/dist/rules/require-template-icon-name.js +48 -0
- package/dist/rules/require-template-icon-name.js.map +1 -0
- package/dist/rules/require-template-workflow-name.d.ts +9 -0
- package/dist/rules/require-template-workflow-name.d.ts.map +1 -0
- package/dist/rules/require-template-workflow-name.js +48 -0
- package/dist/rules/require-template-workflow-name.js.map +1 -0
- package/dist/rules/require-trigger-types.d.ts +9 -0
- package/dist/rules/require-trigger-types.d.ts.map +1 -0
- package/dist/rules/require-trigger-types.js +126 -0
- package/dist/rules/require-trigger-types.js.map +1 -0
- package/dist/rules/require-workflow-call-input-type.d.ts +9 -0
- package/dist/rules/require-workflow-call-input-type.d.ts.map +1 -0
- package/dist/rules/require-workflow-call-input-type.js +89 -0
- package/dist/rules/require-workflow-call-input-type.js.map +1 -0
- package/dist/rules/require-workflow-call-output-value.d.ts +9 -0
- package/dist/rules/require-workflow-call-output-value.d.ts.map +1 -0
- package/dist/rules/require-workflow-call-output-value.js +79 -0
- package/dist/rules/require-workflow-call-output-value.js.map +1 -0
- package/dist/rules/require-workflow-concurrency.d.ts +9 -0
- package/dist/rules/require-workflow-concurrency.d.ts.map +1 -0
- package/dist/rules/require-workflow-concurrency.js +157 -0
- package/dist/rules/require-workflow-concurrency.js.map +1 -0
- package/dist/rules/require-workflow-dispatch-input-type.d.ts +9 -0
- package/dist/rules/require-workflow-dispatch-input-type.d.ts.map +1 -0
- package/dist/rules/require-workflow-dispatch-input-type.js +80 -0
- package/dist/rules/require-workflow-dispatch-input-type.js.map +1 -0
- package/dist/rules/require-workflow-interface-description.d.ts +9 -0
- package/dist/rules/require-workflow-interface-description.d.ts.map +1 -0
- package/dist/rules/require-workflow-interface-description.js +98 -0
- package/dist/rules/require-workflow-interface-description.js.map +1 -0
- package/dist/rules/require-workflow-permissions.d.ts +9 -0
- package/dist/rules/require-workflow-permissions.d.ts.map +1 -0
- package/dist/rules/require-workflow-permissions.js +83 -0
- package/dist/rules/require-workflow-permissions.js.map +1 -0
- package/dist/rules/require-workflow-run-branches.d.ts +9 -0
- package/dist/rules/require-workflow-run-branches.d.ts.map +1 -0
- package/dist/rules/require-workflow-run-branches.js +84 -0
- package/dist/rules/require-workflow-run-branches.js.map +1 -0
- package/dist/rules/require-workflow-template-pair.d.ts +9 -0
- package/dist/rules/require-workflow-template-pair.d.ts.map +1 -0
- package/dist/rules/require-workflow-template-pair.js +47 -0
- package/dist/rules/require-workflow-template-pair.js.map +1 -0
- package/dist/rules/require-workflow-template-properties-pair.d.ts +9 -0
- package/dist/rules/require-workflow-template-properties-pair.d.ts.map +1 -0
- package/dist/rules/require-workflow-template-properties-pair.js +49 -0
- package/dist/rules/require-workflow-template-properties-pair.js.map +1 -0
- package/dist/rules/valid-timeout-minutes.d.ts +9 -0
- package/dist/rules/valid-timeout-minutes.d.ts.map +1 -0
- package/dist/rules/valid-timeout-minutes.js +255 -0
- package/dist/rules/valid-timeout-minutes.js.map +1 -0
- package/dist/rules/valid-trigger-events.d.ts +9 -0
- package/dist/rules/valid-trigger-events.d.ts.map +1 -0
- package/dist/rules/valid-trigger-events.js +100 -0
- package/dist/rules/valid-trigger-events.js.map +1 -0
- package/docs/rules/action-name-casing.md +64 -0
- package/docs/rules/getting-started.md +46 -0
- package/docs/rules/job-id-casing.md +73 -0
- package/docs/rules/max-jobs-per-action.md +79 -0
- package/docs/rules/no-case-insensitive-input-id-collision.md +63 -0
- package/docs/rules/no-composite-input-env-access.md +66 -0
- package/docs/rules/no-deprecated-node-runtime.md +61 -0
- package/docs/rules/no-duplicate-composite-step-id.md +73 -0
- package/docs/rules/no-empty-template-file-pattern.md +57 -0
- package/docs/rules/no-external-job.md +66 -0
- package/docs/rules/no-hardcoded-default-branch-in-template.md +63 -0
- package/docs/rules/no-icon-file-extension-in-template-icon-name.md +57 -0
- package/docs/rules/no-inherit-secrets.md +65 -0
- package/docs/rules/no-invalid-concurrency-context.md +101 -0
- package/docs/rules/no-invalid-key.md +86 -0
- package/docs/rules/no-invalid-reusable-workflow-job-key.md +74 -0
- package/docs/rules/no-invalid-template-file-pattern-regex.md +57 -0
- package/docs/rules/no-invalid-workflow-call-output-value.md +80 -0
- package/docs/rules/no-path-separators-in-template-icon-name.md +57 -0
- package/docs/rules/no-post-if-without-post.md +64 -0
- package/docs/rules/no-pr-head-checkout-in-pull-request-target.md +83 -0
- package/docs/rules/no-pre-if-without-pre.md +64 -0
- package/docs/rules/no-required-input-with-default.md +64 -0
- package/docs/rules/no-secrets-in-if.md +70 -0
- package/docs/rules/no-self-hosted-runner-on-fork-pr-events.md +75 -0
- package/docs/rules/no-subdirectory-template-file-pattern.md +57 -0
- package/docs/rules/no-template-placeholder-in-non-template-workflow.md +63 -0
- package/docs/rules/no-top-level-env.md +64 -0
- package/docs/rules/no-top-level-permissions.md +64 -0
- package/docs/rules/no-universal-template-file-pattern.md +57 -0
- package/docs/rules/no-unknown-input-reference-in-composite.md +71 -0
- package/docs/rules/no-unknown-job-output-reference.md +88 -0
- package/docs/rules/no-unknown-step-reference.md +73 -0
- package/docs/rules/no-untrusted-input-in-run.md +74 -0
- package/docs/rules/no-unused-input-in-composite.md +71 -0
- package/docs/rules/no-write-all-permissions.md +60 -0
- package/docs/rules/overview.md +91 -0
- package/docs/rules/pin-action-shas.md +65 -0
- package/docs/rules/prefer-action-yml.md +57 -0
- package/docs/rules/prefer-fail-fast.md +72 -0
- package/docs/rules/prefer-file-extension.md +77 -0
- package/docs/rules/prefer-inputs-context.md +84 -0
- package/docs/rules/prefer-step-uses-style.md +70 -0
- package/docs/rules/prefer-template-yml-extension.md +57 -0
- package/docs/rules/presets/action-metadata.md +18 -0
- package/docs/rules/presets/all.md +76 -0
- package/docs/rules/presets/index.md +100 -0
- package/docs/rules/presets/recommended.md +26 -0
- package/docs/rules/presets/security.md +16 -0
- package/docs/rules/presets/strict.md +48 -0
- package/docs/rules/presets/workflow-template-properties.md +18 -0
- package/docs/rules/presets/workflow-templates.md +22 -0
- package/docs/rules/require-action-name.md +61 -0
- package/docs/rules/require-action-run-name.md +63 -0
- package/docs/rules/require-checkout-before-local-action.md +66 -0
- package/docs/rules/require-composite-step-name.md +66 -0
- package/docs/rules/require-job-name.md +63 -0
- package/docs/rules/require-job-step-name.md +69 -0
- package/docs/rules/require-job-timeout-minutes.md +76 -0
- package/docs/rules/require-merge-group-trigger.md +67 -0
- package/docs/rules/require-pull-request-target-branches.md +79 -0
- package/docs/rules/require-run-step-shell.md +85 -0
- package/docs/rules/require-template-categories.md +62 -0
- package/docs/rules/require-template-file-patterns.md +63 -0
- package/docs/rules/require-template-icon-file-exists.md +61 -0
- package/docs/rules/require-template-icon-name.md +57 -0
- package/docs/rules/require-template-workflow-name.md +60 -0
- package/docs/rules/require-trigger-types.md +76 -0
- package/docs/rules/require-workflow-call-input-type.md +70 -0
- package/docs/rules/require-workflow-call-output-value.md +67 -0
- package/docs/rules/require-workflow-concurrency.md +73 -0
- package/docs/rules/require-workflow-dispatch-input-type.md +70 -0
- package/docs/rules/require-workflow-interface-description.md +96 -0
- package/docs/rules/require-workflow-permissions.md +75 -0
- package/docs/rules/require-workflow-run-branches.md +66 -0
- package/docs/rules/require-workflow-template-pair.md +58 -0
- package/docs/rules/require-workflow-template-properties-pair.md +58 -0
- package/docs/rules/valid-timeout-minutes.md +74 -0
- package/docs/rules/valid-trigger-events.md +62 -0
- package/package.json +497 -0
|
@@ -0,0 +1,99 @@
|
|
|
1
|
+
import { getGithubExpressionBodies, getReferencedContextRoots, } from "../_internal/github-expressions.js";
|
|
2
|
+
import { getMappingPair, getMappingValueAsMapping, getScalarStringValue, getWorkflowRoot, unwrapYamlValue, } from "../_internal/workflow-yaml.js";
|
|
3
|
+
/** Match reusable workflow output mappings to job outputs. */
|
|
4
|
+
const jobsOutputReferencePattern = /jobs\.[A-Z_a-z][\w-]*\.outputs\.[A-Z_a-z][\w-]*/;
|
|
5
|
+
/** Allowed expression contexts for `on.workflow_call.outputs.*.value`. */
|
|
6
|
+
const allowedWorkflowCallOutputContexts = new Set([
|
|
7
|
+
"github",
|
|
8
|
+
"inputs",
|
|
9
|
+
"jobs",
|
|
10
|
+
"vars",
|
|
11
|
+
]);
|
|
12
|
+
/** Determine whether a scalar value reads at least one declared job output. */
|
|
13
|
+
const hasJobsOutputReference = (value) => getGithubExpressionBodies(value).some((expression) => jobsOutputReferencePattern.test(expression));
|
|
14
|
+
/** Rule implementation for validating reusable workflow output values. */
|
|
15
|
+
const rule = {
|
|
16
|
+
create(context) {
|
|
17
|
+
return {
|
|
18
|
+
Program() {
|
|
19
|
+
const root = getWorkflowRoot(context);
|
|
20
|
+
if (root === null) {
|
|
21
|
+
return;
|
|
22
|
+
}
|
|
23
|
+
const onMapping = getMappingValueAsMapping(root, "on");
|
|
24
|
+
if (onMapping === null) {
|
|
25
|
+
return;
|
|
26
|
+
}
|
|
27
|
+
const workflowCallMapping = unwrapYamlValue(getMappingPair(onMapping, "workflow_call")?.value ?? null);
|
|
28
|
+
if (workflowCallMapping?.type !== "YAMLMapping") {
|
|
29
|
+
return;
|
|
30
|
+
}
|
|
31
|
+
const outputsMapping = getMappingValueAsMapping(workflowCallMapping, "outputs");
|
|
32
|
+
if (outputsMapping === null) {
|
|
33
|
+
return;
|
|
34
|
+
}
|
|
35
|
+
for (const pair of outputsMapping.pairs) {
|
|
36
|
+
const outputId = getScalarStringValue(pair.key);
|
|
37
|
+
const outputMapping = unwrapYamlValue(pair.value);
|
|
38
|
+
if (outputId === null ||
|
|
39
|
+
outputMapping?.type !== "YAMLMapping") {
|
|
40
|
+
continue;
|
|
41
|
+
}
|
|
42
|
+
const valuePair = getMappingPair(outputMapping, "value");
|
|
43
|
+
const value = getScalarStringValue(valuePair?.value ?? null);
|
|
44
|
+
if (valuePair === null ||
|
|
45
|
+
value === null ||
|
|
46
|
+
value.trim().length === 0) {
|
|
47
|
+
continue;
|
|
48
|
+
}
|
|
49
|
+
const disallowedContexts = getReferencedContextRoots(value).filter((contextRoot) => !allowedWorkflowCallOutputContexts.has(contextRoot));
|
|
50
|
+
if (disallowedContexts.length > 0) {
|
|
51
|
+
context.report({
|
|
52
|
+
data: {
|
|
53
|
+
contexts: disallowedContexts.join(", "),
|
|
54
|
+
outputId,
|
|
55
|
+
},
|
|
56
|
+
messageId: "invalidContext",
|
|
57
|
+
node: (valuePair.value ??
|
|
58
|
+
valuePair),
|
|
59
|
+
});
|
|
60
|
+
continue;
|
|
61
|
+
}
|
|
62
|
+
if (!hasJobsOutputReference(value)) {
|
|
63
|
+
context.report({
|
|
64
|
+
data: {
|
|
65
|
+
outputId,
|
|
66
|
+
},
|
|
67
|
+
messageId: "missingJobOutputReference",
|
|
68
|
+
node: (valuePair.value ??
|
|
69
|
+
valuePair),
|
|
70
|
+
});
|
|
71
|
+
}
|
|
72
|
+
}
|
|
73
|
+
},
|
|
74
|
+
};
|
|
75
|
+
},
|
|
76
|
+
meta: {
|
|
77
|
+
docs: {
|
|
78
|
+
configs: [
|
|
79
|
+
"github-actions.configs.all",
|
|
80
|
+
"github-actions.configs.recommended",
|
|
81
|
+
"github-actions.configs.strict",
|
|
82
|
+
],
|
|
83
|
+
description: "disallow `workflow_call` output values that use unavailable contexts or fail to map from `jobs.<job_id>.outputs.<output_name>`.",
|
|
84
|
+
recommended: true,
|
|
85
|
+
requiresTypeChecking: false,
|
|
86
|
+
ruleId: "R040",
|
|
87
|
+
ruleNumber: 40,
|
|
88
|
+
url: "https://nick2bad4u.github.io/eslint-plugin-github-actions-2/docs/rules/no-invalid-workflow-call-output-value",
|
|
89
|
+
},
|
|
90
|
+
messages: {
|
|
91
|
+
invalidContext: "`workflow_call` output '{{outputId}}' uses unsupported context access in `value` ({{contexts}}). Reusable workflow output values may only reference `github`, `jobs`, `vars`, and `inputs`.",
|
|
92
|
+
missingJobOutputReference: "`workflow_call` output '{{outputId}}' must map `value` from a job output such as `jobs.build.outputs.artifact`.",
|
|
93
|
+
},
|
|
94
|
+
schema: [],
|
|
95
|
+
type: "problem",
|
|
96
|
+
},
|
|
97
|
+
};
|
|
98
|
+
export default rule;
|
|
99
|
+
//# sourceMappingURL=no-invalid-workflow-call-output-value.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-invalid-workflow-call-output-value.js","sourceRoot":"","sources":["../../src/rules/no-invalid-workflow-call-output-value.ts"],"names":[],"mappings":"AAMA,OAAO,EACH,yBAAyB,EACzB,yBAAyB,GAC5B,MAAM,oCAAoC,CAAC;AAC5C,OAAO,EACH,cAAc,EACd,wBAAwB,EACxB,oBAAoB,EACpB,eAAe,EACf,eAAe,GAClB,MAAM,+BAA+B,CAAC;AAEvC,8DAA8D;AAC9D,MAAM,0BAA0B,GAC5B,iDAAiD,CAAC;AAEtD,0EAA0E;AAC1E,MAAM,iCAAiC,GAAwB,IAAI,GAAG,CAAC;IACnE,QAAQ;IACR,QAAQ;IACR,MAAM;IACN,MAAM;CACT,CAAC,CAAC;AAEH,+EAA+E;AAC/E,MAAM,sBAAsB,GAAG,CAAC,KAAa,EAAW,EAAE,CACtD,yBAAyB,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,UAAU,EAAE,EAAE,CACjD,0BAA0B,CAAC,IAAI,CAAC,UAAU,CAAC,CAC9C,CAAC;AAEN,0EAA0E;AAC1E,MAAM,IAAI,GAAoB;IAC1B,MAAM,CAAC,OAAO;QACV,OAAO;YACH,OAAO;gBACH,MAAM,IAAI,GAAG,eAAe,CAAC,OAAO,CAAC,CAAC;gBAEtC,IAAI,IAAI,KAAK,IAAI,EAAE,CAAC;oBAChB,OAAO;gBACX,CAAC;gBAED,MAAM,SAAS,GAAG,wBAAwB,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC;gBAEvD,IAAI,SAAS,KAAK,IAAI,EAAE,CAAC;oBACrB,OAAO;gBACX,CAAC;gBAED,MAAM,mBAAmB,GAAG,eAAe,CACvC,cAAc,CAAC,SAAS,EAAE,eAAe,CAAC,EAAE,KAAK,IAAI,IAAI,CAC5D,CAAC;gBAEF,IAAI,mBAAmB,EAAE,IAAI,KAAK,aAAa,EAAE,CAAC;oBAC9C,OAAO;gBACX,CAAC;gBAED,MAAM,cAAc,GAAG,wBAAwB,CAC3C,mBAAmB,EACnB,SAAS,CACZ,CAAC;gBAEF,IAAI,cAAc,KAAK,IAAI,EAAE,CAAC;oBAC1B,OAAO;gBACX,CAAC;gBAED,KAAK,MAAM,IAAI,IAAI,cAAc,CAAC,KAAK,EAAE,CAAC;oBACtC,MAAM,QAAQ,GAAG,oBAAoB,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;oBAChD,MAAM,aAAa,GAAG,eAAe,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;oBAElD,IACI,QAAQ,KAAK,IAAI;wBACjB,aAAa,EAAE,IAAI,KAAK,aAAa,EACvC,CAAC;wBACC,SAAS;oBACb,CAAC;oBAED,MAAM,SAAS,GAAG,cAAc,CAAC,aAAa,EAAE,OAAO,CAAC,CAAC;oBACzD,MAAM,KAAK,GAAG,oBAAoB,CAC9B,SAAS,EAAE,KAAK,IAAI,IAAI,CAC3B,CAAC;oBAEF,IACI,SAAS,KAAK,IAAI;wBAClB,KAAK,KAAK,IAAI;wBACd,KAAK,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC,EAC3B,CAAC;wBACC,SAAS;oBACb,CAAC;oBAED,MAAM,kBAAkB,GAAG,yBAAyB,CAChD,KAAK,CACR,CAAC,MAAM,CACJ,CAAC,WAAW,EAAE,EAAE,CACZ,CAAC,iCAAiC,CAAC,GAAG,CAAC,WAAW,CAAC,CAC1D,CAAC;oBAEF,IAAI,kBAAkB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;wBAChC,OAAO,CAAC,MAAM,CAAC;4BACX,IAAI,EAAE;gCACF,QAAQ,EAAE,kBAAkB,CAAC,IAAI,CAAC,IAAI,CAAC;gCACvC,QAAQ;6BACX;4BACD,SAAS,EAAE,gBAAgB;4BAC3B,IAAI,EAAE,CAAC,SAAS,CAAC,KAAK;gCAClB,SAAS,CAAyB;yBACzC,CAAC,CAAC;wBAEH,SAAS;oBACb,CAAC;oBAED,IAAI,CAAC,sBAAsB,CAAC,KAAK,CAAC,EAAE,CAAC;wBACjC,OAAO,CAAC,MAAM,CAAC;4BACX,IAAI,EAAE;gCACF,QAAQ;6BACX;4BACD,SAAS,EAAE,2BAA2B;4BACtC,IAAI,EAAE,CAAC,SAAS,CAAC,KAAK;gCAClB,SAAS,CAAyB;yBACzC,CAAC,CAAC;oBACP,CAAC;gBACL,CAAC;YACL,CAAC;SACJ,CAAC;IACN,CAAC;IACD,IAAI,EAAE;QACF,IAAI,EAAE;YACF,OAAO,EAAE;gBACL,4BAA4B;gBAC5B,oCAAoC;gBACpC,+BAA+B;aAClC;YACD,WAAW,EACP,iIAAiI;YACrI,WAAW,EAAE,IAAI;YACjB,oBAAoB,EAAE,KAAK;YAC3B,MAAM,EAAE,MAAM;YACd,UAAU,EAAE,EAAE;YACd,GAAG,EAAE,8GAA8G;SACtH;QACD,QAAQ,EAAE;YACN,cAAc,EACV,6LAA6L;YACjM,yBAAyB,EACrB,iHAAiH;SACxH;QACD,MAAM,EAAE,EAAE;QACV,IAAI,EAAE,SAAS;KACG;CACzB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,9 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* @packageDocumentation
|
|
3
|
+
* Disallow path separators in workflow-template `iconName` values.
|
|
4
|
+
*/
|
|
5
|
+
import type { Rule } from "eslint";
|
|
6
|
+
/** Rule implementation for icon path-separator checks. */
|
|
7
|
+
declare const rule: Rule.RuleModule;
|
|
8
|
+
export default rule;
|
|
9
|
+
//# sourceMappingURL=no-path-separators-in-template-icon-name.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-path-separators-in-template-icon-name.d.ts","sourceRoot":"","sources":["../../src/rules/no-path-separators-in-template-icon-name.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAQnC,0DAA0D;AAC1D,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,UA0DhB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,52 @@
|
|
|
1
|
+
import { isWorkflowTemplatePropertiesFile } from "../_internal/lint-targets.js";
|
|
2
|
+
import { getWorkflowTemplatePropertiesRoot, getWorkflowTemplateStringProperty, } from "../_internal/workflow-template-properties.js";
|
|
3
|
+
/** Rule implementation for icon path-separator checks. */
|
|
4
|
+
const rule = {
|
|
5
|
+
create(context) {
|
|
6
|
+
return {
|
|
7
|
+
Program(node) {
|
|
8
|
+
if (!isWorkflowTemplatePropertiesFile(context.filename)) {
|
|
9
|
+
return;
|
|
10
|
+
}
|
|
11
|
+
const root = getWorkflowTemplatePropertiesRoot(context);
|
|
12
|
+
if (root === null) {
|
|
13
|
+
return;
|
|
14
|
+
}
|
|
15
|
+
const iconName = getWorkflowTemplateStringProperty(root, "iconName");
|
|
16
|
+
if (iconName === null ||
|
|
17
|
+
(!iconName.includes("/") && !iconName.includes("\\"))) {
|
|
18
|
+
return;
|
|
19
|
+
}
|
|
20
|
+
context.report({
|
|
21
|
+
data: {
|
|
22
|
+
iconName,
|
|
23
|
+
},
|
|
24
|
+
messageId: "iconNameContainsPathSeparator",
|
|
25
|
+
node: node,
|
|
26
|
+
});
|
|
27
|
+
},
|
|
28
|
+
};
|
|
29
|
+
},
|
|
30
|
+
meta: {
|
|
31
|
+
docs: {
|
|
32
|
+
configs: [
|
|
33
|
+
"github-actions.configs.workflowTemplateProperties",
|
|
34
|
+
"github-actions.configs.workflowTemplates",
|
|
35
|
+
"github-actions.configs.all",
|
|
36
|
+
],
|
|
37
|
+
description: "disallow path separators in workflow-template `iconName` values.",
|
|
38
|
+
recommended: true,
|
|
39
|
+
requiresTypeChecking: false,
|
|
40
|
+
ruleId: "R064",
|
|
41
|
+
ruleNumber: 64,
|
|
42
|
+
url: "https://nick2bad4u.github.io/eslint-plugin-github-actions-2/docs/rules/no-path-separators-in-template-icon-name",
|
|
43
|
+
},
|
|
44
|
+
messages: {
|
|
45
|
+
iconNameContainsPathSeparator: "Template `iconName` '{{iconName}}' should be a plain icon token, not a path.",
|
|
46
|
+
},
|
|
47
|
+
schema: [],
|
|
48
|
+
type: "problem",
|
|
49
|
+
},
|
|
50
|
+
};
|
|
51
|
+
export default rule;
|
|
52
|
+
//# sourceMappingURL=no-path-separators-in-template-icon-name.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-path-separators-in-template-icon-name.js","sourceRoot":"","sources":["../../src/rules/no-path-separators-in-template-icon-name.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,gCAAgC,EAAE,MAAM,8BAA8B,CAAC;AAChF,OAAO,EACH,iCAAiC,EACjC,iCAAiC,GACpC,MAAM,8CAA8C,CAAC;AAEtD,0DAA0D;AAC1D,MAAM,IAAI,GAAoB;IAC1B,MAAM,CAAC,OAAO;QACV,OAAO;YACH,OAAO,CAAC,IAAI;gBACR,IAAI,CAAC,gCAAgC,CAAC,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAC;oBACtD,OAAO;gBACX,CAAC;gBAED,MAAM,IAAI,GAAG,iCAAiC,CAAC,OAAO,CAAC,CAAC;gBAExD,IAAI,IAAI,KAAK,IAAI,EAAE,CAAC;oBAChB,OAAO;gBACX,CAAC;gBAED,MAAM,QAAQ,GAAG,iCAAiC,CAC9C,IAAI,EACJ,UAAU,CACb,CAAC;gBAEF,IACI,QAAQ,KAAK,IAAI;oBACjB,CAAC,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,EACvD,CAAC;oBACC,OAAO;gBACX,CAAC;gBAED,OAAO,CAAC,MAAM,CAAC;oBACX,IAAI,EAAE;wBACF,QAAQ;qBACX;oBACD,SAAS,EAAE,+BAA+B;oBAC1C,IAAI,EAAE,IAA4B;iBACrC,CAAC,CAAC;YACP,CAAC;SACJ,CAAC;IACN,CAAC;IACD,IAAI,EAAE;QACF,IAAI,EAAE;YACF,OAAO,EAAE;gBACL,mDAAmD;gBACnD,0CAA0C;gBAC1C,4BAA4B;aAC/B;YACD,WAAW,EACP,kEAAkE;YACtE,WAAW,EAAE,IAAI;YACjB,oBAAoB,EAAE,KAAK;YAC3B,MAAM,EAAE,MAAM;YACd,UAAU,EAAE,EAAE;YACd,GAAG,EAAE,iHAAiH;SACzH;QACD,QAAQ,EAAE;YACN,6BAA6B,EACzB,8EAA8E;SACrF;QACD,MAAM,EAAE,EAAE;QACV,IAAI,EAAE,SAAS;KACG;CACzB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,9 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* @packageDocumentation
|
|
3
|
+
* Disallow `runs.post-if` when `runs.post` is not declared.
|
|
4
|
+
*/
|
|
5
|
+
import type { Rule } from "eslint";
|
|
6
|
+
/** Rule implementation for invalid `runs.post-if` usage. */
|
|
7
|
+
declare const rule: Rule.RuleModule;
|
|
8
|
+
export default rule;
|
|
9
|
+
//# sourceMappingURL=no-post-if-without-post.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-post-if-without-post.d.ts","sourceRoot":"","sources":["../../src/rules/no-post-if-without-post.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AASnC,4DAA4D;AAC5D,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,UAwDhB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,52 @@
|
|
|
1
|
+
import { isActionMetadataFile } from "../_internal/lint-targets.js";
|
|
2
|
+
import { getMappingPair, getMappingValueAsMapping, getWorkflowRoot, } from "../_internal/workflow-yaml.js";
|
|
3
|
+
/** Rule implementation for invalid `runs.post-if` usage. */
|
|
4
|
+
const rule = {
|
|
5
|
+
create(context) {
|
|
6
|
+
return {
|
|
7
|
+
Program() {
|
|
8
|
+
if (!isActionMetadataFile(context.filename)) {
|
|
9
|
+
return;
|
|
10
|
+
}
|
|
11
|
+
const root = getWorkflowRoot(context);
|
|
12
|
+
const runsMapping = root === null
|
|
13
|
+
? null
|
|
14
|
+
: getMappingValueAsMapping(root, "runs");
|
|
15
|
+
if (runsMapping === null) {
|
|
16
|
+
return;
|
|
17
|
+
}
|
|
18
|
+
const postIfPair = getMappingPair(runsMapping, "post-if");
|
|
19
|
+
if (postIfPair === null ||
|
|
20
|
+
getMappingPair(runsMapping, "post")) {
|
|
21
|
+
return;
|
|
22
|
+
}
|
|
23
|
+
context.report({
|
|
24
|
+
messageId: "postIfWithoutPost",
|
|
25
|
+
node: (postIfPair.value ??
|
|
26
|
+
postIfPair),
|
|
27
|
+
});
|
|
28
|
+
},
|
|
29
|
+
};
|
|
30
|
+
},
|
|
31
|
+
meta: {
|
|
32
|
+
docs: {
|
|
33
|
+
configs: [
|
|
34
|
+
"github-actions.configs.actionMetadata",
|
|
35
|
+
"github-actions.configs.all",
|
|
36
|
+
],
|
|
37
|
+
description: "disallow `runs.post-if` when `runs.post` is not configured in action metadata.",
|
|
38
|
+
recommended: true,
|
|
39
|
+
requiresTypeChecking: false,
|
|
40
|
+
ruleId: "R046",
|
|
41
|
+
ruleNumber: 46,
|
|
42
|
+
url: "https://nick2bad4u.github.io/eslint-plugin-github-actions-2/docs/rules/no-post-if-without-post",
|
|
43
|
+
},
|
|
44
|
+
messages: {
|
|
45
|
+
postIfWithoutPost: "`runs.post-if` is set, but `runs.post` is missing. Remove `post-if` or add `post`.",
|
|
46
|
+
},
|
|
47
|
+
schema: [],
|
|
48
|
+
type: "problem",
|
|
49
|
+
},
|
|
50
|
+
};
|
|
51
|
+
export default rule;
|
|
52
|
+
//# sourceMappingURL=no-post-if-without-post.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-post-if-without-post.js","sourceRoot":"","sources":["../../src/rules/no-post-if-without-post.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,oBAAoB,EAAE,MAAM,8BAA8B,CAAC;AACpE,OAAO,EACH,cAAc,EACd,wBAAwB,EACxB,eAAe,GAClB,MAAM,+BAA+B,CAAC;AAEvC,4DAA4D;AAC5D,MAAM,IAAI,GAAoB;IAC1B,MAAM,CAAC,OAAO;QACV,OAAO;YACH,OAAO;gBACH,IAAI,CAAC,oBAAoB,CAAC,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAC;oBAC1C,OAAO;gBACX,CAAC;gBAED,MAAM,IAAI,GAAG,eAAe,CAAC,OAAO,CAAC,CAAC;gBACtC,MAAM,WAAW,GACb,IAAI,KAAK,IAAI;oBACT,CAAC,CAAC,IAAI;oBACN,CAAC,CAAC,wBAAwB,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;gBAEjD,IAAI,WAAW,KAAK,IAAI,EAAE,CAAC;oBACvB,OAAO;gBACX,CAAC;gBAED,MAAM,UAAU,GAAG,cAAc,CAAC,WAAW,EAAE,SAAS,CAAC,CAAC;gBAE1D,IACI,UAAU,KAAK,IAAI;oBACnB,cAAc,CAAC,WAAW,EAAE,MAAM,CAAC,EACrC,CAAC;oBACC,OAAO;gBACX,CAAC;gBAED,OAAO,CAAC,MAAM,CAAC;oBACX,SAAS,EAAE,mBAAmB;oBAC9B,IAAI,EAAE,CAAC,UAAU,CAAC,KAAK;wBACnB,UAAU,CAAyB;iBAC1C,CAAC,CAAC;YACP,CAAC;SACJ,CAAC;IACN,CAAC;IACD,IAAI,EAAE;QACF,IAAI,EAAE;YACF,OAAO,EAAE;gBACL,uCAAuC;gBACvC,4BAA4B;aAC/B;YACD,WAAW,EACP,gFAAgF;YACpF,WAAW,EAAE,IAAI;YACjB,oBAAoB,EAAE,KAAK;YAC3B,MAAM,EAAE,MAAM;YACd,UAAU,EAAE,EAAE;YACd,GAAG,EAAE,gGAAgG;SACxG;QACD,QAAQ,EAAE;YACN,iBAAiB,EACb,oFAAoF;SAC3F;QACD,MAAM,EAAE,EAAE;QACV,IAAI,EAAE,SAAS;KACG;CACzB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,12 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* @packageDocumentation
|
|
3
|
+
* Disallow checking out pull-request head content in `pull_request_target` workflows.
|
|
4
|
+
*/
|
|
5
|
+
import type { Rule } from "eslint";
|
|
6
|
+
/**
|
|
7
|
+
* Rule implementation for disallowing pull-request head checkouts in
|
|
8
|
+
* pull_request_target.
|
|
9
|
+
*/
|
|
10
|
+
declare const rule: Rule.RuleModule;
|
|
11
|
+
export default rule;
|
|
12
|
+
//# sourceMappingURL=no-pr-head-checkout-in-pull-request-target.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-pr-head-checkout-in-pull-request-target.d.ts","sourceRoot":"","sources":["../../src/rules/no-pr-head-checkout-in-pull-request-target.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAsBnC;;;GAGG;AACH,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,UA0GhB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,89 @@
|
|
|
1
|
+
import { getMappingPair, getMappingValueAsMapping, getMappingValueAsSequence, getScalarStringValue, getWorkflowEventNames, getWorkflowJobs, getWorkflowRoot, unwrapYamlValue, } from "../_internal/workflow-yaml.js";
|
|
2
|
+
/** Detect `actions/checkout` references. */
|
|
3
|
+
const isCheckoutActionReference = (reference) => reference.startsWith("actions/checkout@");
|
|
4
|
+
/** Detect risky PR-head references in checkout parameters. */
|
|
5
|
+
const hasPullRequestHeadReference = (value) => /\bgithub\.head_ref\b/u.test(value) ||
|
|
6
|
+
/\bgithub\.event\.pull_request\.head\./u.test(value);
|
|
7
|
+
/**
|
|
8
|
+
* Rule implementation for disallowing pull-request head checkouts in
|
|
9
|
+
* pull_request_target.
|
|
10
|
+
*/
|
|
11
|
+
const rule = {
|
|
12
|
+
create(context) {
|
|
13
|
+
return {
|
|
14
|
+
Program() {
|
|
15
|
+
const root = getWorkflowRoot(context);
|
|
16
|
+
if (root === null) {
|
|
17
|
+
return;
|
|
18
|
+
}
|
|
19
|
+
const eventNames = getWorkflowEventNames(root);
|
|
20
|
+
if (!eventNames.has("pull_request_target")) {
|
|
21
|
+
return;
|
|
22
|
+
}
|
|
23
|
+
for (const job of getWorkflowJobs(root)) {
|
|
24
|
+
const stepsSequence = getMappingValueAsSequence(job.mapping, "steps");
|
|
25
|
+
if (stepsSequence === null) {
|
|
26
|
+
continue;
|
|
27
|
+
}
|
|
28
|
+
for (const entry of stepsSequence.entries) {
|
|
29
|
+
const stepMapping = unwrapYamlValue(entry);
|
|
30
|
+
if (stepMapping?.type !== "YAMLMapping") {
|
|
31
|
+
continue;
|
|
32
|
+
}
|
|
33
|
+
const usesPair = getMappingPair(stepMapping, "uses");
|
|
34
|
+
const usesReference = getScalarStringValue(usesPair?.value ?? null);
|
|
35
|
+
if (usesPair === null ||
|
|
36
|
+
usesReference === null ||
|
|
37
|
+
!isCheckoutActionReference(usesReference)) {
|
|
38
|
+
continue;
|
|
39
|
+
}
|
|
40
|
+
const withMapping = getMappingValueAsMapping(stepMapping, "with");
|
|
41
|
+
if (withMapping === null) {
|
|
42
|
+
continue;
|
|
43
|
+
}
|
|
44
|
+
for (const key of ["ref", "repository"]) {
|
|
45
|
+
const optionPair = getMappingPair(withMapping, key);
|
|
46
|
+
const optionValue = getScalarStringValue(optionPair?.value ?? null);
|
|
47
|
+
if (optionPair === null ||
|
|
48
|
+
optionValue === null ||
|
|
49
|
+
!hasPullRequestHeadReference(optionValue)) {
|
|
50
|
+
continue;
|
|
51
|
+
}
|
|
52
|
+
context.report({
|
|
53
|
+
data: {
|
|
54
|
+
jobId: job.id,
|
|
55
|
+
key,
|
|
56
|
+
},
|
|
57
|
+
messageId: "pullRequestHeadCheckout",
|
|
58
|
+
node: (optionPair.value ??
|
|
59
|
+
optionPair),
|
|
60
|
+
});
|
|
61
|
+
}
|
|
62
|
+
}
|
|
63
|
+
}
|
|
64
|
+
},
|
|
65
|
+
};
|
|
66
|
+
},
|
|
67
|
+
meta: {
|
|
68
|
+
docs: {
|
|
69
|
+
configs: [
|
|
70
|
+
"github-actions.configs.all",
|
|
71
|
+
"github-actions.configs.security",
|
|
72
|
+
"github-actions.configs.strict",
|
|
73
|
+
],
|
|
74
|
+
description: "disallow `actions/checkout` configurations in `pull_request_target` workflows that fetch pull request head refs, SHAs, or repositories into a privileged run.",
|
|
75
|
+
recommended: false,
|
|
76
|
+
requiresTypeChecking: false,
|
|
77
|
+
ruleId: "R030",
|
|
78
|
+
ruleNumber: 30,
|
|
79
|
+
url: "https://nick2bad4u.github.io/eslint-plugin-github-actions-2/docs/rules/no-pr-head-checkout-in-pull-request-target",
|
|
80
|
+
},
|
|
81
|
+
messages: {
|
|
82
|
+
pullRequestHeadCheckout: "Job '{{jobId}}' uses `actions/checkout` with `with.{{key}}` pointing at pull request head content inside a `pull_request_target` workflow. Avoid checking out untrusted PR head code in this privileged trigger.",
|
|
83
|
+
},
|
|
84
|
+
schema: [],
|
|
85
|
+
type: "problem",
|
|
86
|
+
},
|
|
87
|
+
};
|
|
88
|
+
export default rule;
|
|
89
|
+
//# sourceMappingURL=no-pr-head-checkout-in-pull-request-target.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-pr-head-checkout-in-pull-request-target.js","sourceRoot":"","sources":["../../src/rules/no-pr-head-checkout-in-pull-request-target.ts"],"names":[],"mappings":"AAMA,OAAO,EACH,cAAc,EACd,wBAAwB,EACxB,yBAAyB,EACzB,oBAAoB,EACpB,qBAAqB,EACrB,eAAe,EACf,eAAe,EACf,eAAe,GAClB,MAAM,+BAA+B,CAAC;AAEvC,4CAA4C;AAC5C,MAAM,yBAAyB,GAAG,CAAC,SAAiB,EAAW,EAAE,CAC7D,SAAS,CAAC,UAAU,CAAC,mBAAmB,CAAC,CAAC;AAE9C,8DAA8D;AAC9D,MAAM,2BAA2B,GAAG,CAAC,KAAa,EAAW,EAAE,CAC3D,uBAAuB,CAAC,IAAI,CAAC,KAAK,CAAC;IACnC,wCAAwC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;AAEzD;;;GAGG;AACH,MAAM,IAAI,GAAoB;IAC1B,MAAM,CAAC,OAAO;QACV,OAAO;YACH,OAAO;gBACH,MAAM,IAAI,GAAG,eAAe,CAAC,OAAO,CAAC,CAAC;gBAEtC,IAAI,IAAI,KAAK,IAAI,EAAE,CAAC;oBAChB,OAAO;gBACX,CAAC;gBAED,MAAM,UAAU,GAAG,qBAAqB,CAAC,IAAI,CAAC,CAAC;gBAE/C,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,qBAAqB,CAAC,EAAE,CAAC;oBACzC,OAAO;gBACX,CAAC;gBAED,KAAK,MAAM,GAAG,IAAI,eAAe,CAAC,IAAI,CAAC,EAAE,CAAC;oBACtC,MAAM,aAAa,GAAG,yBAAyB,CAC3C,GAAG,CAAC,OAAO,EACX,OAAO,CACV,CAAC;oBAEF,IAAI,aAAa,KAAK,IAAI,EAAE,CAAC;wBACzB,SAAS;oBACb,CAAC;oBAED,KAAK,MAAM,KAAK,IAAI,aAAa,CAAC,OAAO,EAAE,CAAC;wBACxC,MAAM,WAAW,GAAG,eAAe,CAAC,KAAK,CAAC,CAAC;wBAE3C,IAAI,WAAW,EAAE,IAAI,KAAK,aAAa,EAAE,CAAC;4BACtC,SAAS;wBACb,CAAC;wBAED,MAAM,QAAQ,GAAG,cAAc,CAAC,WAAW,EAAE,MAAM,CAAC,CAAC;wBACrD,MAAM,aAAa,GAAG,oBAAoB,CACtC,QAAQ,EAAE,KAAK,IAAI,IAAI,CAC1B,CAAC;wBAEF,IACI,QAAQ,KAAK,IAAI;4BACjB,aAAa,KAAK,IAAI;4BACtB,CAAC,yBAAyB,CAAC,aAAa,CAAC,EAC3C,CAAC;4BACC,SAAS;wBACb,CAAC;wBAED,MAAM,WAAW,GAAG,wBAAwB,CACxC,WAAW,EACX,MAAM,CACT,CAAC;wBAEF,IAAI,WAAW,KAAK,IAAI,EAAE,CAAC;4BACvB,SAAS;wBACb,CAAC;wBAED,KAAK,MAAM,GAAG,IAAI,CAAC,KAAK,EAAE,YAAY,CAAU,EAAE,CAAC;4BAC/C,MAAM,UAAU,GAAG,cAAc,CAAC,WAAW,EAAE,GAAG,CAAC,CAAC;4BACpD,MAAM,WAAW,GAAG,oBAAoB,CACpC,UAAU,EAAE,KAAK,IAAI,IAAI,CAC5B,CAAC;4BAEF,IACI,UAAU,KAAK,IAAI;gCACnB,WAAW,KAAK,IAAI;gCACpB,CAAC,2BAA2B,CAAC,WAAW,CAAC,EAC3C,CAAC;gCACC,SAAS;4BACb,CAAC;4BAED,OAAO,CAAC,MAAM,CAAC;gCACX,IAAI,EAAE;oCACF,KAAK,EAAE,GAAG,CAAC,EAAE;oCACb,GAAG;iCACN;gCACD,SAAS,EAAE,yBAAyB;gCACpC,IAAI,EAAE,CAAC,UAAU,CAAC,KAAK;oCACnB,UAAU,CAAyB;6BAC1C,CAAC,CAAC;wBACP,CAAC;oBACL,CAAC;gBACL,CAAC;YACL,CAAC;SACJ,CAAC;IACN,CAAC;IACD,IAAI,EAAE;QACF,IAAI,EAAE;YACF,OAAO,EAAE;gBACL,4BAA4B;gBAC5B,iCAAiC;gBACjC,+BAA+B;aAClC;YACD,WAAW,EACP,+JAA+J;YACnK,WAAW,EAAE,KAAK;YAClB,oBAAoB,EAAE,KAAK;YAC3B,MAAM,EAAE,MAAM;YACd,UAAU,EAAE,EAAE;YACd,GAAG,EAAE,mHAAmH;SAC3H;QACD,QAAQ,EAAE;YACN,uBAAuB,EACnB,kNAAkN;SACzN;QACD,MAAM,EAAE,EAAE;QACV,IAAI,EAAE,SAAS;KACG;CACzB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,9 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* @packageDocumentation
|
|
3
|
+
* Disallow `runs.pre-if` when `runs.pre` is not declared.
|
|
4
|
+
*/
|
|
5
|
+
import type { Rule } from "eslint";
|
|
6
|
+
/** Rule implementation for invalid `runs.pre-if` usage. */
|
|
7
|
+
declare const rule: Rule.RuleModule;
|
|
8
|
+
export default rule;
|
|
9
|
+
//# sourceMappingURL=no-pre-if-without-pre.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-pre-if-without-pre.d.ts","sourceRoot":"","sources":["../../src/rules/no-pre-if-without-pre.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AASnC,2DAA2D;AAC3D,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,UAqDhB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,51 @@
|
|
|
1
|
+
import { isActionMetadataFile } from "../_internal/lint-targets.js";
|
|
2
|
+
import { getMappingPair, getMappingValueAsMapping, getWorkflowRoot, } from "../_internal/workflow-yaml.js";
|
|
3
|
+
/** Rule implementation for invalid `runs.pre-if` usage. */
|
|
4
|
+
const rule = {
|
|
5
|
+
create(context) {
|
|
6
|
+
return {
|
|
7
|
+
Program() {
|
|
8
|
+
if (!isActionMetadataFile(context.filename)) {
|
|
9
|
+
return;
|
|
10
|
+
}
|
|
11
|
+
const root = getWorkflowRoot(context);
|
|
12
|
+
const runsMapping = root === null
|
|
13
|
+
? null
|
|
14
|
+
: getMappingValueAsMapping(root, "runs");
|
|
15
|
+
if (runsMapping === null) {
|
|
16
|
+
return;
|
|
17
|
+
}
|
|
18
|
+
const preIfPair = getMappingPair(runsMapping, "pre-if");
|
|
19
|
+
if (preIfPair === null || getMappingPair(runsMapping, "pre")) {
|
|
20
|
+
return;
|
|
21
|
+
}
|
|
22
|
+
context.report({
|
|
23
|
+
messageId: "preIfWithoutPre",
|
|
24
|
+
node: (preIfPair.value ??
|
|
25
|
+
preIfPair),
|
|
26
|
+
});
|
|
27
|
+
},
|
|
28
|
+
};
|
|
29
|
+
},
|
|
30
|
+
meta: {
|
|
31
|
+
docs: {
|
|
32
|
+
configs: [
|
|
33
|
+
"github-actions.configs.actionMetadata",
|
|
34
|
+
"github-actions.configs.all",
|
|
35
|
+
],
|
|
36
|
+
description: "disallow `runs.pre-if` when `runs.pre` is not configured in action metadata.",
|
|
37
|
+
recommended: true,
|
|
38
|
+
requiresTypeChecking: false,
|
|
39
|
+
ruleId: "R045",
|
|
40
|
+
ruleNumber: 45,
|
|
41
|
+
url: "https://nick2bad4u.github.io/eslint-plugin-github-actions-2/docs/rules/no-pre-if-without-pre",
|
|
42
|
+
},
|
|
43
|
+
messages: {
|
|
44
|
+
preIfWithoutPre: "`runs.pre-if` is set, but `runs.pre` is missing. Remove `pre-if` or add `pre`.",
|
|
45
|
+
},
|
|
46
|
+
schema: [],
|
|
47
|
+
type: "problem",
|
|
48
|
+
},
|
|
49
|
+
};
|
|
50
|
+
export default rule;
|
|
51
|
+
//# sourceMappingURL=no-pre-if-without-pre.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-pre-if-without-pre.js","sourceRoot":"","sources":["../../src/rules/no-pre-if-without-pre.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,oBAAoB,EAAE,MAAM,8BAA8B,CAAC;AACpE,OAAO,EACH,cAAc,EACd,wBAAwB,EACxB,eAAe,GAClB,MAAM,+BAA+B,CAAC;AAEvC,2DAA2D;AAC3D,MAAM,IAAI,GAAoB;IAC1B,MAAM,CAAC,OAAO;QACV,OAAO;YACH,OAAO;gBACH,IAAI,CAAC,oBAAoB,CAAC,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAC;oBAC1C,OAAO;gBACX,CAAC;gBAED,MAAM,IAAI,GAAG,eAAe,CAAC,OAAO,CAAC,CAAC;gBACtC,MAAM,WAAW,GACb,IAAI,KAAK,IAAI;oBACT,CAAC,CAAC,IAAI;oBACN,CAAC,CAAC,wBAAwB,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;gBAEjD,IAAI,WAAW,KAAK,IAAI,EAAE,CAAC;oBACvB,OAAO;gBACX,CAAC;gBAED,MAAM,SAAS,GAAG,cAAc,CAAC,WAAW,EAAE,QAAQ,CAAC,CAAC;gBAExD,IAAI,SAAS,KAAK,IAAI,IAAI,cAAc,CAAC,WAAW,EAAE,KAAK,CAAC,EAAE,CAAC;oBAC3D,OAAO;gBACX,CAAC;gBAED,OAAO,CAAC,MAAM,CAAC;oBACX,SAAS,EAAE,iBAAiB;oBAC5B,IAAI,EAAE,CAAC,SAAS,CAAC,KAAK;wBAClB,SAAS,CAAyB;iBACzC,CAAC,CAAC;YACP,CAAC;SACJ,CAAC;IACN,CAAC;IACD,IAAI,EAAE;QACF,IAAI,EAAE;YACF,OAAO,EAAE;gBACL,uCAAuC;gBACvC,4BAA4B;aAC/B;YACD,WAAW,EACP,8EAA8E;YAClF,WAAW,EAAE,IAAI;YACjB,oBAAoB,EAAE,KAAK;YAC3B,MAAM,EAAE,MAAM;YACd,UAAU,EAAE,EAAE;YACd,GAAG,EAAE,8FAA8F;SACtG;QACD,QAAQ,EAAE;YACN,eAAe,EACX,gFAAgF;SACvF;QACD,MAAM,EAAE,EAAE;QACV,IAAI,EAAE,SAAS;KACG;CACzB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,9 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* @packageDocumentation
|
|
3
|
+
* Disallow action inputs that are both required and defaulted.
|
|
4
|
+
*/
|
|
5
|
+
import type { Rule } from "eslint";
|
|
6
|
+
/** Rule implementation for contradictory action input definitions. */
|
|
7
|
+
declare const rule: Rule.RuleModule;
|
|
8
|
+
export default rule;
|
|
9
|
+
//# sourceMappingURL=no-required-input-with-default.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-required-input-with-default.d.ts","sourceRoot":"","sources":["../../src/rules/no-required-input-with-default.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAWnC,sEAAsE;AACtE,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,UAiFhB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,68 @@
|
|
|
1
|
+
import { isActionMetadataFile } from "../_internal/lint-targets.js";
|
|
2
|
+
import { getMappingPair, getMappingValueAsMapping, getScalarStringValue, getWorkflowRoot, unwrapYamlValue, } from "../_internal/workflow-yaml.js";
|
|
3
|
+
/** Rule implementation for contradictory action input definitions. */
|
|
4
|
+
const rule = {
|
|
5
|
+
create(context) {
|
|
6
|
+
return {
|
|
7
|
+
Program() {
|
|
8
|
+
if (!isActionMetadataFile(context.filename)) {
|
|
9
|
+
return;
|
|
10
|
+
}
|
|
11
|
+
const root = getWorkflowRoot(context);
|
|
12
|
+
const inputsMapping = root === null
|
|
13
|
+
? null
|
|
14
|
+
: getMappingValueAsMapping(root, "inputs");
|
|
15
|
+
if (inputsMapping === null) {
|
|
16
|
+
return;
|
|
17
|
+
}
|
|
18
|
+
for (const pair of inputsMapping.pairs) {
|
|
19
|
+
const inputId = getScalarStringValue(pair.key);
|
|
20
|
+
const inputMapping = unwrapYamlValue(pair.value);
|
|
21
|
+
if (inputId === null ||
|
|
22
|
+
inputMapping?.type !== "YAMLMapping") {
|
|
23
|
+
continue;
|
|
24
|
+
}
|
|
25
|
+
const requiredPair = getMappingPair(inputMapping, "required");
|
|
26
|
+
const defaultPair = getMappingPair(inputMapping, "default");
|
|
27
|
+
if (requiredPair === null || defaultPair === null) {
|
|
28
|
+
continue;
|
|
29
|
+
}
|
|
30
|
+
const requiredValue = unwrapYamlValue(requiredPair.value);
|
|
31
|
+
if (requiredValue?.type !== "YAMLScalar" ||
|
|
32
|
+
requiredValue.value !== true) {
|
|
33
|
+
continue;
|
|
34
|
+
}
|
|
35
|
+
context.report({
|
|
36
|
+
data: {
|
|
37
|
+
inputId,
|
|
38
|
+
},
|
|
39
|
+
messageId: "requiredInputWithDefault",
|
|
40
|
+
node: (defaultPair.value ??
|
|
41
|
+
defaultPair),
|
|
42
|
+
});
|
|
43
|
+
}
|
|
44
|
+
},
|
|
45
|
+
};
|
|
46
|
+
},
|
|
47
|
+
meta: {
|
|
48
|
+
docs: {
|
|
49
|
+
configs: [
|
|
50
|
+
"github-actions.configs.actionMetadata",
|
|
51
|
+
"github-actions.configs.all",
|
|
52
|
+
],
|
|
53
|
+
description: "disallow action inputs that set both `required: true` and `default`.",
|
|
54
|
+
recommended: true,
|
|
55
|
+
requiresTypeChecking: false,
|
|
56
|
+
ruleId: "R047",
|
|
57
|
+
ruleNumber: 47,
|
|
58
|
+
url: "https://nick2bad4u.github.io/eslint-plugin-github-actions-2/docs/rules/no-required-input-with-default",
|
|
59
|
+
},
|
|
60
|
+
messages: {
|
|
61
|
+
requiredInputWithDefault: "Input '{{inputId}}' sets both `required: true` and `default`, which is contradictory for action callers.",
|
|
62
|
+
},
|
|
63
|
+
schema: [],
|
|
64
|
+
type: "problem",
|
|
65
|
+
},
|
|
66
|
+
};
|
|
67
|
+
export default rule;
|
|
68
|
+
//# sourceMappingURL=no-required-input-with-default.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-required-input-with-default.js","sourceRoot":"","sources":["../../src/rules/no-required-input-with-default.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,oBAAoB,EAAE,MAAM,8BAA8B,CAAC;AACpE,OAAO,EACH,cAAc,EACd,wBAAwB,EACxB,oBAAoB,EACpB,eAAe,EACf,eAAe,GAClB,MAAM,+BAA+B,CAAC;AAEvC,sEAAsE;AACtE,MAAM,IAAI,GAAoB;IAC1B,MAAM,CAAC,OAAO;QACV,OAAO;YACH,OAAO;gBACH,IAAI,CAAC,oBAAoB,CAAC,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAC;oBAC1C,OAAO;gBACX,CAAC;gBAED,MAAM,IAAI,GAAG,eAAe,CAAC,OAAO,CAAC,CAAC;gBACtC,MAAM,aAAa,GACf,IAAI,KAAK,IAAI;oBACT,CAAC,CAAC,IAAI;oBACN,CAAC,CAAC,wBAAwB,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC;gBAEnD,IAAI,aAAa,KAAK,IAAI,EAAE,CAAC;oBACzB,OAAO;gBACX,CAAC;gBAED,KAAK,MAAM,IAAI,IAAI,aAAa,CAAC,KAAK,EAAE,CAAC;oBACrC,MAAM,OAAO,GAAG,oBAAoB,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;oBAC/C,MAAM,YAAY,GAAG,eAAe,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;oBAEjD,IACI,OAAO,KAAK,IAAI;wBAChB,YAAY,EAAE,IAAI,KAAK,aAAa,EACtC,CAAC;wBACC,SAAS;oBACb,CAAC;oBAED,MAAM,YAAY,GAAG,cAAc,CAC/B,YAAY,EACZ,UAAU,CACb,CAAC;oBACF,MAAM,WAAW,GAAG,cAAc,CAAC,YAAY,EAAE,SAAS,CAAC,CAAC;oBAE5D,IAAI,YAAY,KAAK,IAAI,IAAI,WAAW,KAAK,IAAI,EAAE,CAAC;wBAChD,SAAS;oBACb,CAAC;oBAED,MAAM,aAAa,GAAG,eAAe,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC;oBAE1D,IACI,aAAa,EAAE,IAAI,KAAK,YAAY;wBACpC,aAAa,CAAC,KAAK,KAAK,IAAI,EAC9B,CAAC;wBACC,SAAS;oBACb,CAAC;oBAED,OAAO,CAAC,MAAM,CAAC;wBACX,IAAI,EAAE;4BACF,OAAO;yBACV;wBACD,SAAS,EAAE,0BAA0B;wBACrC,IAAI,EAAE,CAAC,WAAW,CAAC,KAAK;4BACpB,WAAW,CAAyB;qBAC3C,CAAC,CAAC;gBACP,CAAC;YACL,CAAC;SACJ,CAAC;IACN,CAAC;IACD,IAAI,EAAE;QACF,IAAI,EAAE;YACF,OAAO,EAAE;gBACL,uCAAuC;gBACvC,4BAA4B;aAC/B;YACD,WAAW,EACP,sEAAsE;YAC1E,WAAW,EAAE,IAAI;YACjB,oBAAoB,EAAE,KAAK;YAC3B,MAAM,EAAE,MAAM;YACd,UAAU,EAAE,EAAE;YACd,GAAG,EAAE,uGAAuG;SAC/G;QACD,QAAQ,EAAE;YACN,wBAAwB,EACpB,0GAA0G;SACjH;QACD,MAAM,EAAE,EAAE;QACV,IAAI,EAAE,SAAS;KACG;CACzB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,9 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* @packageDocumentation
|
|
3
|
+
* Disallow direct secret references in `if` conditionals.
|
|
4
|
+
*/
|
|
5
|
+
import type { Rule } from "eslint";
|
|
6
|
+
/** Rule implementation for disallowing direct secret usage in `if` conditionals. */
|
|
7
|
+
declare const rule: Rule.RuleModule;
|
|
8
|
+
export default rule;
|
|
9
|
+
//# sourceMappingURL=no-secrets-in-if.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-secrets-in-if.d.ts","sourceRoot":"","sources":["../../src/rules/no-secrets-in-if.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,QAAQ,CAAC;AAwCnC,oFAAoF;AACpF,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,UA0FhB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,84 @@
|
|
|
1
|
+
import { getMappingPair, getMappingValueAsSequence, getScalarStringValue, getWorkflowJobs, getWorkflowRoot, unwrapYamlValue, } from "../_internal/workflow-yaml.js";
|
|
2
|
+
/** Direct secret-access patterns that GitHub discourages in `if` conditionals. */
|
|
3
|
+
const directSecretsPatterns = [
|
|
4
|
+
/\bsecrets\.\w+\b/u,
|
|
5
|
+
/\bsecrets\[["'][^"'\]]+["']\]/u,
|
|
6
|
+
];
|
|
7
|
+
/**
|
|
8
|
+
* Determine whether an `if` expression directly references the `secrets`
|
|
9
|
+
* context.
|
|
10
|
+
*/
|
|
11
|
+
const hasDirectSecretsReference = (expression) => directSecretsPatterns.some((pattern) => pattern.test(expression));
|
|
12
|
+
/** Report a direct-secrets conditional reference. */
|
|
13
|
+
const reportDirectSecretsConditional = (context, ifPair, scope) => {
|
|
14
|
+
context.report({
|
|
15
|
+
data: {
|
|
16
|
+
scope,
|
|
17
|
+
},
|
|
18
|
+
messageId: "directSecretsInIf",
|
|
19
|
+
node: ifPair.value,
|
|
20
|
+
});
|
|
21
|
+
};
|
|
22
|
+
/** Rule implementation for disallowing direct secret usage in `if` conditionals. */
|
|
23
|
+
const rule = {
|
|
24
|
+
create(context) {
|
|
25
|
+
return {
|
|
26
|
+
Program() {
|
|
27
|
+
const root = getWorkflowRoot(context);
|
|
28
|
+
if (root === null) {
|
|
29
|
+
return;
|
|
30
|
+
}
|
|
31
|
+
for (const job of getWorkflowJobs(root)) {
|
|
32
|
+
const jobIfPair = getMappingPair(job.mapping, "if");
|
|
33
|
+
const jobIfExpression = getScalarStringValue(jobIfPair?.value ?? null);
|
|
34
|
+
if (jobIfPair !== null &&
|
|
35
|
+
jobIfExpression !== null &&
|
|
36
|
+
hasDirectSecretsReference(jobIfExpression)) {
|
|
37
|
+
reportDirectSecretsConditional(context, jobIfPair, `job '${job.id}'`);
|
|
38
|
+
}
|
|
39
|
+
const stepsSequence = getMappingValueAsSequence(job.mapping, "steps");
|
|
40
|
+
if (stepsSequence === null) {
|
|
41
|
+
continue;
|
|
42
|
+
}
|
|
43
|
+
for (const entry of stepsSequence.entries) {
|
|
44
|
+
const stepMapping = unwrapYamlValue(entry);
|
|
45
|
+
if (stepMapping?.type !== "YAMLMapping") {
|
|
46
|
+
continue;
|
|
47
|
+
}
|
|
48
|
+
const stepIfPair = getMappingPair(stepMapping, "if");
|
|
49
|
+
const stepIfExpression = getScalarStringValue(stepIfPair?.value ?? null);
|
|
50
|
+
if (stepIfPair === null ||
|
|
51
|
+
stepIfExpression === null ||
|
|
52
|
+
!hasDirectSecretsReference(stepIfExpression)) {
|
|
53
|
+
continue;
|
|
54
|
+
}
|
|
55
|
+
reportDirectSecretsConditional(context, stepIfPair, `a step in job '${job.id}'`);
|
|
56
|
+
}
|
|
57
|
+
}
|
|
58
|
+
},
|
|
59
|
+
};
|
|
60
|
+
},
|
|
61
|
+
meta: {
|
|
62
|
+
docs: {
|
|
63
|
+
configs: [
|
|
64
|
+
"github-actions.configs.all",
|
|
65
|
+
"github-actions.configs.recommended",
|
|
66
|
+
"github-actions.configs.security",
|
|
67
|
+
"github-actions.configs.strict",
|
|
68
|
+
],
|
|
69
|
+
description: "disallow direct `secrets.*` references in job and step `if` conditionals; move secret values into `env` first and condition on that instead.",
|
|
70
|
+
recommended: true,
|
|
71
|
+
requiresTypeChecking: false,
|
|
72
|
+
ruleId: "R027",
|
|
73
|
+
ruleNumber: 27,
|
|
74
|
+
url: "https://nick2bad4u.github.io/eslint-plugin-github-actions-2/docs/rules/no-secrets-in-if",
|
|
75
|
+
},
|
|
76
|
+
messages: {
|
|
77
|
+
directSecretsInIf: "Do not reference `secrets` directly in {{scope}} `if` conditionals. Assign the secret to `env` first, then check the environment variable.",
|
|
78
|
+
},
|
|
79
|
+
schema: [],
|
|
80
|
+
type: "problem",
|
|
81
|
+
},
|
|
82
|
+
};
|
|
83
|
+
export default rule;
|
|
84
|
+
//# sourceMappingURL=no-secrets-in-if.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"no-secrets-in-if.js","sourceRoot":"","sources":["../../src/rules/no-secrets-in-if.ts"],"names":[],"mappings":"AAOA,OAAO,EACH,cAAc,EACd,yBAAyB,EACzB,oBAAoB,EACpB,eAAe,EACf,eAAe,EACf,eAAe,GAClB,MAAM,+BAA+B,CAAC;AAEvC,kFAAkF;AAClF,MAAM,qBAAqB,GAAG;IAC1B,mBAAmB;IACnB,gCAAgC;CAC1B,CAAC;AAEX;;;GAGG;AACH,MAAM,yBAAyB,GAAG,CAAC,UAAkB,EAAW,EAAE,CAC9D,qBAAqB,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC,CAAC;AAEtE,qDAAqD;AACrD,MAAM,8BAA8B,GAAG,CACnC,OAAmC,EACnC,MAA8B,EAC9B,KAAa,EACT,EAAE;IACN,OAAO,CAAC,MAAM,CAAC;QACX,IAAI,EAAE;YACF,KAAK;SACR;QACD,SAAS,EAAE,mBAAmB;QAC9B,IAAI,EAAE,MAAM,CAAC,KAA6B;KAC7C,CAAC,CAAC;AACP,CAAC,CAAC;AAEF,oFAAoF;AACpF,MAAM,IAAI,GAAoB;IAC1B,MAAM,CAAC,OAAO;QACV,OAAO;YACH,OAAO;gBACH,MAAM,IAAI,GAAG,eAAe,CAAC,OAAO,CAAC,CAAC;gBAEtC,IAAI,IAAI,KAAK,IAAI,EAAE,CAAC;oBAChB,OAAO;gBACX,CAAC;gBAED,KAAK,MAAM,GAAG,IAAI,eAAe,CAAC,IAAI,CAAC,EAAE,CAAC;oBACtC,MAAM,SAAS,GAAG,cAAc,CAAC,GAAG,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC;oBACpD,MAAM,eAAe,GAAG,oBAAoB,CACxC,SAAS,EAAE,KAAK,IAAI,IAAI,CAC3B,CAAC;oBAEF,IACI,SAAS,KAAK,IAAI;wBAClB,eAAe,KAAK,IAAI;wBACxB,yBAAyB,CAAC,eAAe,CAAC,EAC5C,CAAC;wBACC,8BAA8B,CAC1B,OAAO,EACP,SAAS,EACT,QAAQ,GAAG,CAAC,EAAE,GAAG,CACpB,CAAC;oBACN,CAAC;oBAED,MAAM,aAAa,GAAG,yBAAyB,CAC3C,GAAG,CAAC,OAAO,EACX,OAAO,CACV,CAAC;oBAEF,IAAI,aAAa,KAAK,IAAI,EAAE,CAAC;wBACzB,SAAS;oBACb,CAAC;oBAED,KAAK,MAAM,KAAK,IAAI,aAAa,CAAC,OAAO,EAAE,CAAC;wBACxC,MAAM,WAAW,GAAG,eAAe,CAAC,KAAK,CAAC,CAAC;wBAE3C,IAAI,WAAW,EAAE,IAAI,KAAK,aAAa,EAAE,CAAC;4BACtC,SAAS;wBACb,CAAC;wBAED,MAAM,UAAU,GAAG,cAAc,CAAC,WAAW,EAAE,IAAI,CAAC,CAAC;wBACrD,MAAM,gBAAgB,GAAG,oBAAoB,CACzC,UAAU,EAAE,KAAK,IAAI,IAAI,CAC5B,CAAC;wBAEF,IACI,UAAU,KAAK,IAAI;4BACnB,gBAAgB,KAAK,IAAI;4BACzB,CAAC,yBAAyB,CAAC,gBAAgB,CAAC,EAC9C,CAAC;4BACC,SAAS;wBACb,CAAC;wBAED,8BAA8B,CAC1B,OAAO,EACP,UAAU,EACV,kBAAkB,GAAG,CAAC,EAAE,GAAG,CAC9B,CAAC;oBACN,CAAC;gBACL,CAAC;YACL,CAAC;SACJ,CAAC;IACN,CAAC;IACD,IAAI,EAAE;QACF,IAAI,EAAE;YACF,OAAO,EAAE;gBACL,4BAA4B;gBAC5B,oCAAoC;gBACpC,iCAAiC;gBACjC,+BAA+B;aAClC;YACD,WAAW,EACP,8IAA8I;YAClJ,WAAW,EAAE,IAAI;YACjB,oBAAoB,EAAE,KAAK;YAC3B,MAAM,EAAE,MAAM;YACd,UAAU,EAAE,EAAE;YACd,GAAG,EAAE,yFAAyF;SACjG;QACD,QAAQ,EAAE;YACN,iBAAiB,EACb,4IAA4I;SACnJ;QACD,MAAM,EAAE,EAAE;QACV,IAAI,EAAE,SAAS;KACG;CACzB,CAAC;AAEF,eAAe,IAAI,CAAC"}
|
|
@@ -0,0 +1,9 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* @packageDocumentation
|
|
3
|
+
* Disallow self-hosted runners on fork-capable pull request events.
|
|
4
|
+
*/
|
|
5
|
+
import type { Rule } from "eslint";
|
|
6
|
+
/** Rule implementation for blocking self-hosted runners on fork PR events. */
|
|
7
|
+
declare const rule: Rule.RuleModule;
|
|
8
|
+
export default rule;
|
|
9
|
+
//# sourceMappingURL=no-self-hosted-runner-on-fork-pr-events.d.ts.map
|