ENG-2516: Add shared clean_version utility for version string sanitization

[NevilleS]

Ticket ENG-2516

Description Of Changes

In this PR https://github.com/ethyca/fidesplus/pull/3047 we added a clean_version() function to fidesplus to clean up the sometimes-messy version numbers for displaying in the API. This refactors that approach by moving that clean_version() function to fides.common.utils, which enables fidesplus to import and reuse this utility rather than duplicating the code.

Code Changes

• Added clean_version() function to src/fides/common/utils.py that strips:
  • The dirty suffix (.dirty or -dirty) added when there are uncommitted changes
  • The +0.gXXXXXX suffix when exactly on a git tag (zero commits past)
• Updated src/fides/__init__.py to import clean_version from common.utils instead of defining it locally
• Added comprehensive unit tests in tests/lib/test_version.py

Steps to Confirm

1. Verify clean_version can be imported from fides.common.utils
2. Run nox -s "pytest(lib)" -- tests/lib/test_version.py to verify tests pass
3. Confirm version display remains clean in health endpoints

Pre-Merge Checklist

• Issue requirements met
• All CI pipelines succeeded
• CHANGELOG.md updated
  • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
  • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
  • Updates unreleased work already in Changelog, no new entry necessary
• UX feedback:
  • No UX review needed
• Followup issues:
  • Followup issues created
  • No followup issues
• Database migrations:
  • No migrations
• Documentation:
  • No documentation updates required

Made with Cursor

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
fides-plus-nightly	Ready	Preview, Comment	Feb 12, 2026 5:36pm

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
fides-privacy-center	Ignored		Feb 12, 2026 5:36pm

[greptile-apps]

Greptile Overview

Greptile Summary

Refactored version string cleaning by extracting the clean_version() function from local implementation into a shared utility in fides.common.utils. This enables code reuse across the fides and fidesplus repositories.

• Added clean_version() function to src/fides/common/utils.py that strips dirty suffixes and zero-distance git tags
• Updated src/fides/__init__.py to import and use the shared utility
• Added comprehensive test coverage with 87 test cases covering various version string formats
• Minor issue: test function name on line 26 has a typo (testclean_versions_unchanged should be test_clean_versions_unchanged)

Confidence Score: 4/5

• Safe to merge after fixing the test function name typo
• Clean refactoring with comprehensive test coverage. The only issue is a minor typo in the test function name that needs correction before merging to ensure tests run properly
• Pay attention to tests/lib/test_version.py - fix the function name typo on line 26

Important Files Changed

Filename	Overview
src/fides/init.py	Refactored to use imported clean_version() from common.utils instead of defining inline
src/fides/common/utils.py	Added new clean_version() utility with regex-based version string sanitization
tests/lib/test_version.py	Comprehensive test coverage for version cleaning, includes typo in test function name on line 26

[greptile-apps]

_{4 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[adamsachs]

nice followup. i assume you'll want a fidesplus PR to switch to using this util, right?

[NevilleS]

nice followup. i assume you'll want a fidesplus PR to switch to using this util, right?

Yeah, that one goes here: https://github.com/ethyca/fidesplus/pull/3082

I'm just going to let the dust settle on our current release before merging those two in.