Skip to content

Conversation

@JasonXuDeveloper
Copy link
Owner

@JasonXuDeveloper JasonXuDeveloper commented Jan 25, 2026

Summary

  • Fix CodeQL failure by using buildless mode (build-mode: none)
  • Remove unnecessary .NET setup and autobuild steps

Problem

CodeQL's autobuild failed with:

Error: Could not auto-detect a suitable build method

Unity projects don't have standard .NET solution files (.sln, .csproj), so CodeQL's autobuild can't figure out how to build them.

Solution

Use CodeQL's buildless mode which analyzes C# source code directly without requiring a build. This is the recommended approach for:

  • Unity projects
  • Script-only codebases
  • Projects without standard build systems

Changes

  • Added build-mode: none to CodeQL init step
  • Removed unnecessary Setup .NET step
  • Removed Autobuild step

Test plan

  • Verify CodeQL workflow passes after merge
  • Check that security analysis results are generated
  • Confirm only JEngine code is scanned (per config)

🤖 Generated with Claude Code

Unity projects don't have standard .NET solution files, so CodeQL's
autobuild fails. Switch to buildless mode (build-mode: none) which
analyzes C# source code directly without requiring a build.

This is the recommended approach for Unity projects and other
non-standard C# codebases.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
@JasonXuDeveloper JasonXuDeveloper enabled auto-merge (squash) January 25, 2026 05:10
@claude
Copy link

claude bot commented Jan 25, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@JasonXuDeveloper JasonXuDeveloper merged commit 783e071 into master Jan 25, 2026
9 checks passed
@JasonXuDeveloper JasonXuDeveloper deleted the fix/codeql-buildless-mode branch January 25, 2026 05:13
@github-actions
Copy link

Unity Test Results

EditMode: All tests passed
PlayMode: All tests passed

Unity Version: 2022.3.55f1
Project Path: UnityProject

✅ All tests passed! The PR is ready for review.

View workflow run

Click here to view the full workflow run

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.

2 participants