Skip to content

Replace spacy Language.en baseline with on-device model version pinning #268

Closed

Description

@kilodesodiq-arch

Problem Statement. pii_scrubber.py::_build_nlp() constructs a spacy.blank("en")
pipeline with hand-rolled entity-ruler patterns. There is no explicit model version.

Why it matters. Reproducibility. Two machines scrubbing the same text might produce
different results over time.

Technical Context. app/ai-service/services/pii_scrubber.py.

Expected Outcome. A PII_MODEL_VERSION field; tests assert that the model version is
exposed in metrics and that the regression harness outputs include it.

Acceptance Criteria.

  • Metric pii_model_version{version="X"} exported.
  • Regression harness report includes the model version in footer.

Files or modules likely to be affected.
app/ai-service/services/pii_scrubber.py,
app/ai-service/metrics.py.

Difficulty. Medium
Estimated effort. M



Backlog item #60 from `docs/maintainer-issue-backlog.md.

Activity

  1. yunus-dev-codecrafter commented on Jul 15, 2026

    @yunus-dev-codecrafter
    Contributor

    Hi! I've carefully reviewed the entire task description and fully understand the requirements and expected outcome.
    I have experience working on similar tasks and I'm confident I can complete this within about an hour. I'd be happy to take ownership of this issue and submit a clean, well-tested solution promptly.
    If it's available, please assign it to me. Thank you!

  2. grantfox-oss commented on Jul 15, 2026

    @grantfox-oss

    🦊 GrantFox — @yunus-dev-codecrafter has been assigned to this issue as part of the Official Campaign | FWC26 campaign!

    Next steps:

    1. Open a Pull Request referencing this issue (e.g., Closes #268)
    2. Your PR will be reviewed by the ChainForgee maintainers

    Good luck! Track your progress on GrantFox.

  3. grantfox-oss commented on Jul 17, 2026

    @grantfox-oss

    🎉 This issue has been marked as completed on GrantFox as part of the Official Campaign | FWC26 campaign!

    @yunus-dev-codecrafter's PR #360 was approved and merged by @kilodesodiq-arch.

    🏆 @yunus-dev-codecrafter: You earned 40 FoxPoints for this contribution! Your current tier: Builder (2,364 total points). Track your full progress on GrantFox.

    👏 Great work, @yunus-dev-codecrafter! Keep contributing to ChainForgee.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions