Skip to content

Comments

Auto update schemaorg#369

Open
NikkiBytes wants to merge 5 commits intomainfrom
auto-update-schemaorg
Open

Auto update schemaorg#369
NikkiBytes wants to merge 5 commits intomainfrom
auto-update-schemaorg

Conversation

@NikkiBytes
Copy link
Contributor

@NikkiBytes NikkiBytes commented Feb 11, 2026

Adds automated monthly updates for schema.org to keep the registry in sync with the latest schema.org releases.

New Features
• Monthly schema.org update routine (discovery/utils/update.py)
◦ Compares current stored version against latest schema.org version
◦ Performs dry-run validation before applying updates
◦ Comprehensive error handling for RegistryError and AttributeError with detailed logging
• Scheduled execution (index.py)
◦ Runs monthly on the 1st at 2 AM (cron: 0 2 1 * *)
◦ Executes in a separate thread to avoid blocking

Modified
_add_schema_class – New schema_org_version and dryrun parameters for version-specific validation
add_core – Now accepts version parameter and stores version after update

Function renames across the codebase — This is a significant API change:
get_schema_org_version()get_stored_schema_org_version() (in schemas.py, clarifies it reads from ES metadata)
get_schema_org_version()get_latest_schema_org_version() (in adapters.py, clarifies it fetches from biothings_schema)
◦ These renames are updated through schema.py handler, schemas.py, adapters.py, and all tests

@NikkiBytes NikkiBytes marked this pull request as ready for review February 11, 2026 21:10
@marcodarko
Copy link
Collaborator

@NikkiBytes is this ready to be merged to dev and tested?

@NikkiBytes
Copy link
Contributor Author

@marcodarko ready for a final review from @newgene and then we can merge. I added notes to the summary above about additional name convention changes.

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.

2 participants