Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
76 changes: 0 additions & 76 deletions .github/workflows/codeql.yml

This file was deleted.

34 changes: 0 additions & 34 deletions .github/workflows/devskim.yml

This file was deleted.

49 changes: 34 additions & 15 deletions .github/workflows/dotnet.yml
Original file line number Diff line number Diff line change
@@ -1,13 +1,12 @@
# This workflow will build a .NET project
# For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-net
# This workflow will build and test .NET projects

name: .NET

on:
push:
branches: [ "main" ]
branches: ["main"]
pull_request:
branches: [ "main" ]
branches: ["main"]

jobs:
build-api-client:
Expand All @@ -18,14 +17,34 @@ jobs:
working-directory: ./src/adguard-api-dotnet

steps:
- uses: actions/checkout@v4
- name: Setup .NET
uses: actions/setup-dotnet@v4
with:
dotnet-version: 10.0.x
- name: Restore dependencies
run: dotnet restore AdGuard.ApiClient.slnx
- name: Build
run: dotnet build AdGuard.ApiClient.slnx --no-restore
- name: Test
run: dotnet test AdGuard.ApiClient.slnx --no-build --verbosity normal
- uses: actions/checkout@v4
- name: Setup .NET
uses: actions/setup-dotnet@v4
with:
dotnet-version: 10.0.x
- name: Restore dependencies
run: dotnet restore AdGuard.ApiClient.slnx
- name: Build
run: dotnet build AdGuard.ApiClient.slnx --no-restore
- name: Test
run: dotnet test AdGuard.ApiClient.slnx --no-build --verbosity normal

build-rules-compiler:
name: Build Rules Compiler
runs-on: ubuntu-latest
defaults:
run:
working-directory: ./src/rules-compiler-dotnet

steps:
- uses: actions/checkout@v4
- name: Setup .NET
uses: actions/setup-dotnet@v4
with:
dotnet-version: 10.0.x
- name: Restore dependencies
run: dotnet restore RulesCompiler.slnx
- name: Build
run: dotnet build RulesCompiler.slnx --no-restore
- name: Test
run: dotnet test RulesCompiler.slnx --no-build --verbosity normal
49 changes: 0 additions & 49 deletions .github/workflows/powershell.yml

This file was deleted.

90 changes: 90 additions & 0 deletions .github/workflows/security.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,90 @@
# Consolidated security scanning workflow
# Combines CodeQL, DevSkim, and PSScriptAnalyzer

name: Security

on:
push:
branches: ["main"]
pull_request:
branches: ["main"]
schedule:
- cron: "28 16 * * 6" # Weekly on Saturday

permissions:
contents: read

jobs:
codeql:
name: CodeQL Analysis
runs-on: ubuntu-latest
permissions:
actions: read
contents: read
security-events: write

strategy:
fail-fast: false
matrix:
language: ["csharp", "javascript"]

steps:
- name: Checkout repository
uses: actions/checkout@v4

- name: Initialize CodeQL
uses: github/codeql-action/init@v3
with:
languages: ${{ matrix.language }}

- name: Autobuild
uses: github/codeql-action/autobuild@v3

- name: Perform CodeQL Analysis
uses: github/codeql-action/analyze@v3
with:
category: "/language:${{matrix.language}}"

devskim:
name: DevSkim Analysis
runs-on: ubuntu-latest
permissions:
actions: read
contents: read
security-events: write

steps:
- name: Checkout code
uses: actions/checkout@v4

- name: Run DevSkim scanner
uses: microsoft/DevSkim-Action@v1

- name: Upload DevSkim scan results
uses: github/codeql-action/upload-sarif@v3
with:
sarif_file: devskim-results.sarif

psscriptanalyzer:
name: PSScriptAnalyzer
runs-on: ubuntu-latest
permissions:
contents: read
security-events: write
actions: read

steps:
- uses: actions/checkout@v4

- name: Run PSScriptAnalyzer
uses: microsoft/psscriptanalyzer-action@6b2948b1944407914a58661c49941824d149734f
with:
path: .\
recurse: true
includeRule: '"PSAvoidGlobalAliases", "PSAvoidUsingConvertToSecureStringWithPlainText"'
output: results.sarif

- name: Upload SARIF results
uses: github/codeql-action/upload-sarif@v3
with:
sarif_file: results.sarif
42 changes: 0 additions & 42 deletions .github/workflows/static.yml

This file was deleted.

13 changes: 6 additions & 7 deletions CLAUDE.md
Original file line number Diff line number Diff line change
Expand Up @@ -420,14 +420,13 @@ RemoveComments, Compress, RemoveModifiers, Validate, ValidateAllowIp, Deduplicat
## CI/CD Alignment

GitHub Actions workflows validate:
- `.github/workflows/dotnet.yml` - Builds/tests .NET projects with .NET 8
- `.github/workflows/typescript.yml` - Node 20, tsc --noEmit, eslint for rules-compiler-typescript
- `.github/workflows/dotnet.yml` - Builds/tests .NET projects (API client and rules compiler) with .NET 10
- `.github/workflows/typescript.yml` - Node 20, tsc --noEmit, eslint for rules-compiler-typescript and website
- `.github/workflows/gatsby.yml` - Builds website and deploys to GitHub Pages
- `.github/workflows/powershell.yml` - PSScriptAnalyzer on PowerShell scripts
- `.github/workflows/codeql.yml` - CodeQL security scanning
- `.github/workflows/devskim.yml` - DevSkim security analysis
- `.github/workflows/claude.yml` - Claude AI integration
- `.github/workflows/claude-code-review.yml` - Automated code review
- `.github/workflows/security.yml` - Consolidated security scanning (CodeQL, DevSkim, PSScriptAnalyzer)
- `.github/workflows/release.yml` - Builds and publishes release binaries (.NET, Rust, Python)
- `.github/workflows/claude.yml` - Claude AI integration for @claude mentions
- `.github/workflows/claude-code-review.yml` - Automated PR code review
Comment on lines +423 to +429
Copy link

Copilot AI Dec 24, 2025

Choose a reason for hiding this comment

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

The workflow documentation has been updated correctly in CLAUDE.md, but similar outdated references exist in other documentation files that also need updating:

  1. README.md (lines 5, 85-89, 971-977) - Still references powershell.yml badge and lists codeql.yml, devskim.yml, powershell.yml as separate workflows
  2. .github/copilot-instructions.md (lines 277, 299, 362-366) - References the old individual workflow files
  3. custom-instructions/repo/.github/copilot-instructions.md (lines 277, 299, 362-366) - Same outdated references
  4. docs/LINEAR_DOCUMENTATION.md (lines 214-217) - Still lists all old workflows including static.yml

All these files should be updated to reflect the consolidation into security.yml and the removal of static.yml.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

@copilot Make these changes


## Prerequisites

Expand Down
Loading
Loading