Skip to content

IEBH-353: Sync with the latest changes#9

Merged
vmoshynskyi merged 2 commits intomainfrom
IEBH-353
Nov 24, 2025
Merged

IEBH-353: Sync with the latest changes#9
vmoshynskyi merged 2 commits intomainfrom
IEBH-353

Conversation

@vmoshynskyi
Copy link
Member

Add the latest changes from the pilot repo.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR synchronizes the codebase with the latest changes from the pilot repository, modernizing Python type annotations to leverage Python 3.10+ features and updating key dependencies. The changes eliminate deprecated typing imports in favor of built-in type syntax (PEP 585/604), add a new network_origin field to activity tracking, and upgrade the FastAPI framework to a newer version.

  • Modernize type annotations to use Python 3.10+ built-in types (type[T], X | Y, collections.abc imports)
  • Add network_origin field to item and dataset activity models for enhanced tracking
  • Update dependencies: FastAPI 0.88.0 → 0.115.13, pilot-platform-common 0.3.0 → 0.8.1, testcontainers 3.5.4 → 3.6.1

Reviewed changes

Copilot reviewed 24 out of 25 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tests/fixtures/jq.py Replace Type import with built-in type syntax; convert ellipsis method stubs to pass
tests/fixtures/components/metadata_item.py Add AsyncGenerator import from collections.abc; change fixture from yield to return; add refresh parameter documentation
tests/fixtures/components/item_activity.py Add AsyncGenerator import; add network_origin parameter to factory methods; change fixture from yield to return
tests/fixtures/components/dataset_activity.py Add AsyncGenerator import; add network_origin parameter; add new SHARING_REQUEST_UPDATE activity type; change fixture from yield to return
tests/fixtures/app.py Import Callable from collections.abc instead of typing
tests/components/test_parameters.py Use inspect.get_annotations() instead of inspect.signature() for cleaner annotation access
search/components/schemas.py Remove __future__ annotations import; add string quotes to forward reference in return type
search/components/project_files/views.py Remove Union import; use built-in union syntax (str | None)
search/components/project_files/parameters.py Replace deprecated regex parameter with pattern in Query validators
search/components/parameters.py Remove __future__ annotations and Type import; use built-in type syntax with string quotes for forward reference
search/components/models.py Remove __future__ annotations import; add string quotes to forward reference
search/components/metadata_item/filtering.py Remove Union import; use built-in union syntax
search/components/item_activity/schemas.py Replace ItemActivityChange model with generic dict[str, Any]; add network_origin field
search/components/item_activity/models.py Remove ItemActivityChange class; use dict[str, Any] for changes; add network_origin field with default
search/components/item_activity/index.py Add network_origin field to Elasticsearch index mapping
search/components/exceptions.py Import Sequence from collections.abc instead of typing
search/components/dataset_activity/schemas.py Remove DatasetActivityChangeSchema class; use dict[str, Any] for changes; add network_origin field
search/components/dataset_activity/models.py Remove DatasetActivityChange class; add SHARING_REQUEST_UPDATE activity type; add network_origin field with default
search/components/dataset_activity/index.py Add network_origin field to Elasticsearch index mapping
pyproject.toml Update version to 2.2.10; constrain Python to 3.10.x; update FastAPI to ^0.115.8 and pilot-platform-common to 0.8.1; change dev-dependencies to group.dev.dependencies
poetry.lock Update lock file with new dependency versions (FastAPI 0.115.13, pilot-platform-common 0.8.1, testcontainers 3.6.1); replace aioredis with redis; update async-timeout, starlette, typing-extensions
migrations/index_checker.py Remove Type import; use built-in type syntax
Dockerfile Update base image from Python 3.10.12 to 3.10.14
.pre-commit-config.yaml Uncomment and update docformatter hook from v1.7.5 to v1.7.7
.github/workflows/hdc-pipeline.yml Add new CI/CD pipeline configuration for HDC deployment workflow

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py00100% 
__main__.py050%7, 9, 11–13
app.py0789%112, 121–122, 124–125, 127, 131
config.py00100% 
logger.py020%7, 9
components
   __init__.py00100% 
   crud.py0198%38
   encoders.py00100% 
   exceptions.py0784%26, 33, 40, 45, 56, 60, 64
   filtering.py0190%29
   index.py00100% 
   models.py00100% 
   pagination.py00100% 
   parameters.py0196%61
   schemas.py00100% 
   search_query.py00100% 
   sorting.py00100% 
   types.py00100% 
components/dataset_activity
   __init__.py00100% 
   crud.py00100% 
   dependencies.py00100% 
   filtering.py00100% 
   index.py00100% 
   models.py00100% 
   parameters.py00100% 
   schemas.py00100% 
   views.py00100% 
components/item_activity
   __init__.py00100% 
   dependencies.py00100% 
   filtering.py00100% 
   index.py00100% 
   models.py00100% 
   parameters.py00100% 
   schemas.py00100% 
   views.py00100% 
components/item_activity/crud
   __init__.py00100% 
   crud.py0393%47, 65–66
   file_activity.py00100% 
components/metadata_item
   __init__.py00100% 
   dependencies.py00100% 
   filtering.py00100% 
   index.py00100% 
   models.py00100% 
   parameters.py00100% 
   schemas.py00100% 
   views.py00100% 
components/metadata_item/crud
   __init__.py00100% 
   crud.py0197%56
   size_usage.py00100% 
components/project_files
   __init__.py00100% 
   parameters.py00100% 
   schemas.py00100% 
   views.py00100% 
dependencies
   __init__.py00100% 
   elasticsearch.py00100% 
TOTAL12082897% 

@vmoshynskyi vmoshynskyi merged commit 0cfb300 into main Nov 24, 2025
7 checks passed
@vmoshynskyi vmoshynskyi deleted the IEBH-353 branch November 24, 2025 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant