Skip to content

Add and validate CodeQL scanning - #148

Merged
konard merged 6 commits into
mainfrom
issue-139-6bab94f7
Sep 20, 2026
Merged

konard merged 6 commits into
mainfrom
issue-139-6bab94f7

Conversation

@konard

@konard konard commented Sep 12, 2025 •

Copy link
Copy Markdown
Member

Summary

  • add a CodeQL status badge to the README
  • add scheduled, push, and pull-request CodeQL analysis for C/C++ and C#
  • use current action runtimes and build-free extraction, leaving compilation and tests to the dedicated language workflows
  • add policy tests that guard the workflow's language identifiers, action versions, runner pin, and least-privilege checkout configuration

Fixes #139.

CI investigation

A fresh run reproduced the existing C/C++ failure: CMake could not find GTest and therefore could not resolve GTest::gtest. The workflow also used deprecated action/runtime versions and a manual build that duplicated the repository's language CI.

The final workflow uses CodeQL v4 with build-mode: none. This removes the failing duplicate build while still producing complete CodeQL databases for both supported languages.

Verification

  • node --test .github/scripts/*.test.mjs — 30 tests passed
  • actionlint .github/workflows/codeql.yml — passed
  • node .github/scripts/check-readme-badges.mjs — passed
  • strict C# validation — build completed with 0 warnings/errors, 2 tests passed, package contents validated
  • local C++ build and test — 7 tests passed
  • pull-request CI — CodeQL C/C++, CodeQL C#, C#, documentation, README badges, and Codacy passed
  • uploaded CodeQL analyses contain 179 C/C++ rules and 164 C# rules, with no alerts

No package version change is needed because this change only affects CI and documentation.

Adding CLAUDE.md with task information for AI processing.
This file will be removed when the task is complete.

Issue: #139
@konard konard self-assigned this Sep 12, 2025
This commit adds:
- CodeQL security analysis workflow for C# and C++ languages
- CodeQL badge to README.md displaying security analysis status
- Scheduled weekly scans and analysis on pull requests

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@konard konard changed the title [WIP] Add CodeQL Badge Add CodeQL Badge Sep 12, 2025
@konard
konard marked this pull request as ready for review September 12, 2025 17:33
@github-advanced-security

Copy link
Copy Markdown

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@konard

konard commented Sep 20, 2026

Copy link
Copy Markdown
Member Author

We also need to make sure CI/CD fill pass by fixing all issues.

@konard
konard marked this pull request as draft September 20, 2026 07:46
@konard

konard commented Sep 20, 2026

Copy link
Copy Markdown
Member Author

🤖 AI Work Session Started

Starting automated work session at 2026-09-20T07:46:53.614Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

Runtime: solve v2.29.0 · tool codex · model gpt-5.6-sol · task image konard/hive-mind-dind:2.29.0@sha256:11a236d03854cbaea86fdd7ca06265c912cf0b67d1fcc9b3eb60bd84c43ff749

@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@konard konard changed the title Add CodeQL Badge Add and validate CodeQL scanning Sep 20, 2026
@konard
konard marked this pull request as ready for review September 20, 2026 08:07
@konard

konard commented Sep 20, 2026

Copy link
Copy Markdown
Member Author

Working session summary

Implemented and finalized PR #148.

  • Added the CodeQL badge and C/C++ plus C# scanning.
  • Fixed the failing C++ analysis by using CodeQL v4 build-free extraction.
  • Added workflow regression tests.
  • Merged the latest main.
  • All required CI checks pass; CodeQL analyzed 179 C/C++ and 164 C# rules with no alerts.
  • Updated the PR title/description and marked it ready for review.
  • Branch is clean, pushed, current, and mergeable.

This summary was automatically extracted from the AI working session output.

@konard

konard commented Sep 20, 2026

Copy link
Copy Markdown
Member Author

🤖 Solution Draft Log

This log file contains the complete execution trace of the AI solution draft process.

💰 Cost estimation:

  • Model: GPT-5.6 Sol
  • Provider: OpenAI
  • Public pricing estimate: $6.987995

📊 Context and tokens usage:

  • 171.6K / 200K (86%) input tokens, 27.1K / 128K (21%) output tokens

Total: (171.6K + 6.0M cached) input tokens, 27.1K output tokens, $6.987995 cost

🤖 Models used:

  • Tool: OpenAI Codex
  • Requested: gpt-5.6-sol
  • Thinking level: xhigh (~31999 tokens)
  • Model: GPT-5.6 Sol (gpt-5.6-sol)

📎 Log file uploaded as Gist (2570KB)


Now working session is ended, feel free to review and add any feedback on the solution draft.

@konard

konard commented Sep 20, 2026

Copy link
Copy Markdown
Member Author

✅ Ready to merge

This pull request is now ready to be merged:

  • All CI checks have passed
  • No merge conflicts
  • No pending changes

Monitored by hive-mind with --auto-restart-until-mergeable flag

@konard
konard merged commit 71f8576 into main Sep 20, 2026
16 checks passed
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.

Add CodeQL Badge

2 participants