Purge remnants of extensions v0 from Product

[gigincg]

Proposed Changes

• Removed the validate_extensions class method and its @field_validator decorator from BaseProductSpec, which previously validated the extensions field using a JSON schema.

Merge Checklist

• Tests added/fixed
• Update docs in /docs
• Linting Complete
• Any other necessary step

Only PR's with test cases included and passing lint and test pipelines will be reviewed

@ohcnetwork/care-backend-maintainers @ohcnetwork/care-backend-admins

Summary by CodeRabbit

• Bug Fixes
  • Simplified product metadata handling by removing overly restrictive validation rules, enabling better data input flexibility across product inventory management.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b45422f and 5120547.

📒 Files selected for processing (1)

• care/emr/resources/inventory/product/spec.py

💤 Files with no reviewable changes (1)

• care/emr/resources/inventory/product/spec.py

---

📝 Walkthrough

Walkthrough

The validate_extensions field validator was removed from the BaseProductSpec class in the product specification module. This validator previously enforced schema validation for the extensions field during assignment; the removal eliminates this runtime validation step.

Changes

Cohort / File(s)	Summary
Extensions field validator removal care/emr/resources/inventory/product/spec.py	Removed the validate_extensions field validator decorated with @field_validator("extensions") from BaseProductSpec. This validator previously validated extensions against settings.PRODUCT_EXTENSIONS_JSON_SCHEMA and raised ValueError on validation failure. Extensions field now accepts values without runtime validation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: removing the validate_extensions method, which was part of the 'extensions v0' implementation being purged from the Product specification.
Description check	✅ Passed	The description includes the proposed changes section with specifics about what was removed, but is missing the 'Associated Issue' section and does not mention whether this is cleanup work or addresses a specific issue.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch cleanup_product_extensions

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.24%. Comparing base (b45422f) to head (5120547).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3548      +/-   ##
===========================================
+ Coverage    76.23%   76.24%   +0.01%     
===========================================
  Files          473      473              
  Lines        22117    22109       -8     
  Branches      2309     2309              
===========================================
- Hits         16860    16857       -3     
+ Misses        4740     4735       -5     
  Partials       517      517              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[vigneshhari]

This would have been overridden and worked correctly, but yeah we can clean it up.

[vigneshhari]

Linting Changes Required