Skip to content

[WIP] Fix failing GitHub Actions job Create GitHub Release - #368

Merged
data-douser merged 3 commits into
mainfrom
copilot/fix-create-github-release-job
Sep 18, 2026
Merged

data-douser merged 3 commits into
mainfrom
copilot/fix-create-github-release-job

Conversation

Copilot AI commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Fix the failing GitHub Actions job "Create GitHub Release"
Analyze the Actions logs, identify the root cause of the failure, and implement a fix.
Check run ID: 105638448915
Job URL: https://github.com/advanced-security/codeql-development-mcp-server/actions/runs/34808716205/job/105638448915

Copilot AI lite review requested due to automatic review settings September 18, 2026 14:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Co-authored-by: data-douser <70299490+data-douser@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 18, 2026 14:40
Copilot AI requested a review from data-douser September 18, 2026 14:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

The release artifact excludes .node-version, so Node.js setup will fail unless hidden files are included or the repository is checked out.

Review details

Suppressed comments (1)

.github/workflows/release.yml:219

  • This job does not check out the repository, so this file must come from the downloaded release-build artifact. However, the producer uploads .node-version without include-hidden-files: true, and upload-artifact@v7 excludes hidden files by default; setup-node will therefore fail to resolve this file before npm runs. Update the artifact upload to include the hidden file (or check out the tag before this step) so the Node setup actually fixes the failing install.
          node-version-file: '.node-version'
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails
actions/actions/setup-node 820762786026740c76f36085b0efc47a31fe5020 🟢 6.4
Details
CheckScoreReason
Code-Review🟢 10all changesets reviewed
Maintained🟢 1021 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 10
Binary-Artifacts🟢 9binaries present in source code
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Packaging⚠️ -1packaging workflow not detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Pinned-Dependencies🟢 5dependency not pinned by hash detected -- score normalized to 5
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Security-Policy🟢 9security policy file detected
Branch-Protection⚠️ 0branch protection not enabled on development/release branches
SAST🟢 10SAST tool is run on all commits

Scanned Files

  • .github/workflows/release.yml

@data-douser

Copy link
Copy Markdown
Collaborator

🔵 Needs a closer look

The release artifact excludes .node-version, so Node.js setup will fail unless hidden files are included or the repository is checked out.

Review details

Suppressed comments (1)

.github/workflows/release.yml:219

  • This job does not check out the repository, so this file must come from the downloaded release-build artifact. However, the producer uploads .node-version without include-hidden-files: true, and upload-artifact@v7 excludes hidden files by default; setup-node will therefore fail to resolve this file before npm runs. Update the artifact upload to include the hidden file (or check out the tag before this step) so the Node setup actually fixes the failing install.
          node-version-file: '.node-version'
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@copilot Improve htis

Co-authored-by: data-douser <70299490+data-douser@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 18, 2026 15:19

Copilot AI commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

🔵 Needs a closer look

The release artifact excludes .node-version, so Node.js setup will fail unless hidden files are included or the repository is checked out.

...

Fixed in c218a28: enabled include-hidden-files: true on the scoped release-build upload so .node-version reaches Node setup. Formatting, secret scan, CodeQL, and fallback review passed. Build/tests and lint were blocked by sandbox Go 1.25.0 (project requires 1.25.5).

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The release workflow fixes are complete with no unresolved issues.

Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@data-douser
data-douser marked this pull request as ready for review September 18, 2026 16:18
@data-douser
data-douser requested a review from a team as a code owner September 18, 2026 16:18
@data-douser
data-douser requested a balanced review from Copilot September 18, 2026 16:20

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The coordinated workflow changes directly address the runtime mismatch shown in the failed job.

Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

@data-douser
data-douser merged commit d0dfff2 into main Sep 18, 2026
17 checks passed
@data-douser
data-douser deleted the copilot/fix-create-github-release-job branch September 18, 2026 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants