MAINT: Update Registry APIs - #2550
Open
Richard Lundeen (rlundeen2) wants to merge 5 commits into
Open
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: c9eafadc-3a00-471b-b61b-68ba4d656577
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: c9eafadc-3a00-471c-a08b-28df698d8109
Type converter file inputs as Path instead of special-casing strings in the backend: AddImageVideoConverter.video_path is now a Path, so the registry describes it as an input file and REST treats it as an upload like every other Path parameter. Drop the per-parameter MIME/signature allowlist. Uploads are stored verbatim because any file type is a legitimate payload; the generated file name means a declared MIME type only picks an extension. Content restrictions now live only in the media route, which serves active document types as neutralized downloads without renaming stored files. Mark the temporary catalog projections consistently so reviewers can see the whole concept is deleted once the UI moves to /types. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: c9eafadc-3a00-471b-b61b-68ba4d656577
Richard Lundeen (rlundeen2)
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/api/converters/typesAPIs, plus the target registry consistency needed by later stack layerspathlib.Pathconstructor metadata and uploaded data-URI persistence without exposing server paths/api/converters/catalogand/api/targets/catalogas temporary compatibility projections for the unchanged main UI; the higher chat-migration layer will remove themValidation
UV_NO_SYNC=1 uv run pytest --quiet tests\unit\models\test_parameter.py tests\unit\registry\test_instance_registry.py tests\unit\registry\test_converter_registry.py tests\unit\registry\test_target_registry.py tests\unit\registry\test_scorer_registry.py tests\unit\backend\test_converter_service.py tests\unit\backend\test_target_service.py tests\unit\backend\test_api_routes.py tests\unit\backend\test_mappers.py tests\unit\backend\test_target_types_concurrency.py tests\unit\setup\test_targets_initializer.py tests\unit\setup\test_scorer_initializer.py(776 passed, 4 skipped)UV_NO_SYNC=1 uv run pre-commit run --all-files