Remove support for schemas that are loaded from a URL (breaking)

[ATorrise]

What It Does
Removes support for loading a JSON schema from a remote http(s):// URL via a config file's $schema property.

• validate_config_json (used during profile loading) now raises a ValueError if $schema is an http(s):// URL, telling the user to use a local file path instead.
• ConfigFile.schema_list (used by get_env) now returns an empty list and logs/warns instead of making a network request for a remote $schema URL.

Only local schema files (absolute path, relative path, or file://) are supported now.

How to Test
run the new tests and make sure they pass

Review Checklist
I certify that I have:

• updated the changelog
• manually tested my changes
• added/updated automated unit/integration tests
• created/ran system tests (provide build number if applicable)
• followed the contribution guidelines

Additional Comments
Breaking! Cases where $schema points at a remote url now fail instead of silently succeeding

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.49%. Comparing base (0c48fbe) to head (32cf533).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #412      +/-   ##
==========================================
+ Coverage   85.20%   85.49%   +0.28%     
==========================================
  Files          49       49              
  Lines        3002     2993       -9     
==========================================
+ Hits         2558     2559       +1     
+ Misses        444      434      -10     

Flag	Coverage Δ
unittests	85.49% <100.00%> (+0.28%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 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.

[zFernand0]

LGTM! 😋

I do have one small request about raising the error.
Feel free to get other developer's opinion on this.

[zFernand0]

LGTM! 😋

dismiss self-review

[CBforZ]

i think this test is still expecting a warning instead of an error:

            with self.assertWarns(UserWarning):
>               result = config_file.schema_list()

tests/unit/core/test_profile_manager.py:307: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

[traeok]

Seeing a desync between the test and latest code changes, plan to approve once resolved 😋

test fixed 🙏

[traeok]

LGTM Thanks @zFernand0 for the quick resolution 😋