Skip to content

ci(tests): add privacy-llm checker and redactor to pytest matrix#59

Merged
potiuk merged 1 commit into
apache:mainfrom
andreahlert:fix/add-privacy-llm-to-ci
May 5, 2026
Merged

ci(tests): add privacy-llm checker and redactor to pytest matrix#59
potiuk merged 1 commit into
apache:mainfrom
andreahlert:fix/add-privacy-llm-to-ci

Conversation

@andreahlert
Copy link
Copy Markdown
Collaborator

@andreahlert andreahlert commented May 5, 2026

Why

The CI pytest matrix in .github/workflows/tests.yml currently exercises three Python projects:

  • tools/gmail/oauth-draft
  • tools/vulnogram/generate-cve-json
  • tools/skill-validator

Two additional projects under tools/privacy-llm/ have full test suites but are not in the matrix:

Project Tests Status
checker 33 passing locally
redactor 48 passing locally

This means regressions in privacy-llm code would not be caught by the visible CI signal (the tests.yml workflow). The prek workflow does exercise pytest as a hook, but the per-project matrix in tests.yml is the triage-friendly surface that shows green/red per package in the PR checks list.

What changes

  • .github/workflows/tests.yml — adds two entries to the matrix.project list:
    • name: privacy-llm-checker, path: tools/privacy-llm/checker
    • name: privacy-llm-redactor, path: tools/privacy-llm/redactor

Verification

  • Both suites pass locally:
    • uv run --directory tools/privacy-llm/checker --group dev pytest --color=yes → 33 passed
    • uv run --directory tools/privacy-llm/redactor --group dev pytest --color=yes → 48 passed
  • prek run --all-files passes with the workflow YAML change (markdownlint + yamllint implicit via prek hooks).

Impact

  • CI now surfaces independent pass/fail badges for privacy-llm-checker and privacy-llm-redactor, matching the actual test surface.
  • No changes to source code, test code, or lockfiles — purely a CI coverage fix.

The two privacy-llm Python packages (checker: 33 tests, redactor:
48 tests) were omitted from the CI pytest matrix despite having
passing test suites and their own pyproject.toml + uv.lock. This
brings the visible CI signal in line with the actual test surface.

Signed-off-by: André Ahlert <andre@aex.partners>
@andreahlert andreahlert requested a review from potiuk May 5, 2026 13:59
@potiuk
Copy link
Copy Markdown
Member

potiuk commented May 5, 2026

😱

@potiuk potiuk merged commit ab469b2 into apache:main May 5, 2026
11 checks passed
@andreahlert andreahlert added the mode:platform Substrate / infra — not a mode (sandbox, CI, validators) label May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mode:platform Substrate / infra — not a mode (sandbox, CI, validators)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants