Skip to content

Test codeguard pr#3

Open
m1el wants to merge 10 commits intomasterfrom
test_codeguard_pr
Open

Test codeguard pr#3
m1el wants to merge 10 commits intomasterfrom
test_codeguard_pr

Conversation

@m1el
Copy link
Copy Markdown
Owner

@m1el m1el commented Feb 16, 2026

No description provided.

m1el added 2 commits February 13, 2026 18:55
- Configure codeguard-action with custom rubric
- Upload evidence bundle as artifact
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 16, 2026

[!!] GuardSpine: CONDITIONAL - Reviewer action needed

Policy: standard | Findings: 12

Reviewer Action Required (max 2)

  1. HIGH [auth] (.github/workflows/codeguard.yml:24): Sensitive auth code modified

    Review sensitive-auth

  2. HIGH [None] (.codeguard/rubrics/clarity-mine.yaml:126): Complex boolean expressions are hard to reason about

    Review CLR-006

Advisory (10 items)
  • [medium] Sensitive infra code modified
  • [medium] Sensitive config code modified
  • [medium] Sensitive config code modified
  • [medium] Sensitive config code modified
  • [low] FOOBAR should not be present in the codebase
  • [medium] Chains of negations are confusing
  • [low] FOOBAR should not be present in the codebase
  • [medium] AI minority concern: Hardcoded API key reference in workflow file (.github/workflows/codeguard.yml) uses secrets.OPENROUTER_API_KEY which appears to be a new external dependency with no validation of the key format or scope
  • [medium] AI minority concern: The FOOBAR test function in src/nemo-ggml.cpp appears to be debug/test code that should not be committed to production
  • [medium] AI minority concern: The trivial print statement change in src/transcribe_stream.cpp ("this is a change that is not that important but it is a change") appears to be test/debug code

GuardSpine Decision Engine | Removing reviewer decisions, not just effort

@m1el m1el had a problem deploying to codeguard-check March 9, 2026 16:33 — with GitHub Actions Failure
@m1el m1el had a problem deploying to codeguard-check March 9, 2026 16:36 — with GitHub Actions Failure
@m1el m1el force-pushed the test_codeguard_pr branch from 877eb6b to f794c7c Compare March 9, 2026 18:33
@m1el m1el temporarily deployed to codeguard-check March 9, 2026 20:19 — with GitHub Actions Inactive
@m1el m1el temporarily deployed to codeguard-check March 9, 2026 20:50 — with GitHub Actions Inactive
@m1el m1el temporarily deployed to codeguard-check March 9, 2026 20:53 — with GitHub Actions Inactive
@m1el m1el temporarily deployed to codeguard-check March 10, 2026 00:22 — with GitHub Actions Inactive
@m1el m1el temporarily deployed to codeguard-check March 10, 2026 00:39 — with GitHub Actions Inactive
@m1el m1el temporarily deployed to codeguard-check March 10, 2026 00:56 — with GitHub Actions Inactive
@m1el m1el temporarily deployed to codeguard-check March 10, 2026 01:24 — with GitHub Actions Inactive
@m1el m1el deployed to codeguard-check March 10, 2026 02:07 — with GitHub Actions Active
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.

1 participant