Repository navigation
Conversation
drk-ram
left a comment
There was a problem hiding this comment.
Reviewed both changed files at a8161038f1ed4f08cc90cc1feba15e22d9b6ac32. Approved.
src/controllers/web_controller.py:58-63 now sources both status version fields from the same existing metadata-derived constant used elsewhere in the generator. This removes two divergent hard-coded values without changing the route, response keys, or operational-status value.
Validation: python -m pytest tests\test_web_controller.py -k status -q --no-cov passed. I additionally mounted the actual router, substituted the distinctive version 9.8.7-review, and verified that /status returned that value in both fields with HTTP 200; the direct coroutine result agreed.
Non-blocking test improvement: consider retaining that sentinel-version substitution in the regression test. It makes the synchronization contract explicit even when the ordinary project version later changes.
Summary: Source both status version fields from project metadata instead of stale hard-coded values and add endpoint coverage. Testing: web-controller tests pass.