Skip to content

chore: add anonymizer skill publication artifacts#214

Draft
binaryaaron wants to merge 2 commits into
mainfrom
binaryaaron/chore/publish-anonymizer-skill
Draft

chore: add anonymizer skill publication artifacts#214
binaryaaron wants to merge 2 commits into
mainfrom
binaryaaron/chore/publish-anonymizer-skill

Conversation

@binaryaaron

@binaryaaron binaryaaron commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

Related Issue

No linked issue required: maintainer-owned NVSkills publication preparation.

Plan Document

No plan document required: this ports the landed Safe-Synthesizer NVSkills publication pattern to the existing Anonymizer skill package.

Summary

  • Add the Anonymizer NVSkills skill card, benchmark scaffold, and six-case evaluation dataset.
  • Make copied skill installs portable with published documentation fallbacks.
  • Point fallbacks at the deployed /dev/ docs and include the evaluation guide added in docs: add docs for anonymizer replace evaluation #181.
  • Move the interactive workflow into references/, add structural tests, and make the README agent-neutral.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Refactoring
  • CI, release, or contributor workflow update

Contributor Checklist

  • PR title follows Conventional Commits, for example fix: handle empty entity list
  • Related issue is linked, or a maintainer-owned no-issue reason is documented above
  • For non-trivial changes, a plan document is linked above, or the no-plan reason is documented above
  • Public API impact checked; skills/anonymizer/SKILL.md updated if needed
  • No real PII added to tests, docs, notebooks, fixtures, or artifacts
  • No API keys, service tokens, private keys, credentials, or real endpoint secrets added

Validation

  • Commands run:
    • uv run pytest tests/skills/test_anonymizer_skill.py -q
    • uv run --with pyyaml python /root/.codex/skills/.system/skill-creator/scripts/quick_validate.py skills/anonymizer
    • uv run pre-commit run check-json --files skills/anonymizer/evals/evals.json
    • make check
    • make docs-build
    • Verified that all six published /dev/ fallback URLs return HTTP 200.
  • Skipped checks or known failures:
    • /nvskills-ci, measured benchmark results, and signing require the external NVSkills publication environment.
    • make check passes; its advisory typecheck continues to report existing repository diagnostics.

Documentation and Artifacts

  • Docs updated, or not needed
  • If docs changed: make docs-build passes locally
  • If tutorial sources changed: notebooks regenerated with make convert-notebooks
  • If e2e, benchmark, or model-provider behavior changed: relevant validation is listed above

Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
Signed-off-by: Aaron Gonzales <aagonzales@nvidia.com>
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