feat(result): Initial OrtResult code - Analyzer#15
Merged
heliocastro merged 5 commits intomainfrom Jan 26, 2026
Merged
Conversation
Signed-off-by: Helio Chissini de Castro <heliocastro@gmail.com>
Signed-off-by: Helio Chissini de Castro <dev@heliocastro.info>
Signed-off-by: Helio Chissini de Castro <dev@heliocastro.info>
Signed-off-by: Helio Chissini de Castro <dev@heliocastro.info>
Signed-off-by: Helio Chissini de Castro <dev@heliocastro.info>
e29048a to
01fd4aa
Compare
There was a problem hiding this comment.
Pull request overview
This PR introduces the initial implementation of OrtResult code for the Analyzer component, representing a significant expansion of the python-ort library's data model capabilities. The PR adds comprehensive support for parsing and serializing ORT (OSS Review Toolkit) analyzer results.
Changes:
- Adds complete analyzer result data models including OrtResult, AnalyzerRun, AnalyzerResult, Project, Package, DependencyGraph, and supporting classes
- Introduces test data files demonstrating the expected format for analyzer results and ORT configuration
- Updates VCS models to handle empty strings and improve serialization
- Removes deprecated ort_configuration.py file
- Updates dependencies and development tooling to newer versions
- Adds example scripts for parsing ORT result and repository configuration files
Reviewed changes
Copilot reviewed 35 out of 38 changed files in this pull request and generated 17 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/data/analyzer-result.yml | Large test data file (1832 lines) demonstrating analyzer result format with package dependencies |
| tests/data/ort_configuration_reference.yml | Reference configuration file (393 lines) showing all ORT configuration options |
| src/ort/utils/processed_declared_license.py | New utility model for processed license declarations |
| src/ort/utils/environment.py | New model for ORT execution environment information |
| src/ort/severity.py | Enum for issue severity levels (HINT, WARNING, ERROR) |
| src/ort/models/vcstype.py | Updated to serialize as plain string; added model_serializer |
| src/ort/models/vcsinfo.py | Modified url field to accept empty strings; changed type from strict AnyUrl |
| src/ort/models/source_code_origin.py | Added docstring documentation |
| src/ort/models/scope.py | New model for dependency scopes |
| src/ort/models/root_dependency_index.py | New model for root dependency indexing |
| src/ort/models/repository.py | New model for repository information |
| src/ort/models/remote_artifact.py | New model for remote artifact references |
| src/ort/models/project.py | New model for project information with custom hash/equality |
| src/ort/models/package_reference.py | New model for package references in dependency trees |
| src/ort/models/package_linkage.py | New enum for package linkage types |
| src/ort/models/package.py | New model for package information with custom hash/equality |
| src/ort/models/ort_result.py | Main ORT result container model |
| src/ort/models/issue.py | New model for ORT issues |
| src/ort/models/identifier.py | Made frozen/immutable and added str method |
| src/ort/models/dependency_*.py | New models for dependency graph representation (node, edge, reference, graph) |
| src/ort/models/config/analyzer_configuration.py | Updated docstring for clarity |
| src/ort/models/analyzer_*.py | New models for analyzer run and result |
| src/ort/models/ort_configuration.py | Deleted entire file (breaking change) |
| examples/*.py | New example scripts for parsing ORT data |
| pyproject.toml | Version bump to 0.5.0, updated dependencies |
| .pre-commit-config.yaml | Updated tool versions and hook configurations |
| .ort.yml | New ORT configuration file for the project |
| .gitignore | Added patterns for ORT output and requirements.txt |
| .vscode/settings.json | Deleted file |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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.
No description provided.