Skip to content

Decoupled ScenarioMetadata and InitializerMetadata from Identity by introducing RegistryEntry#1370

Merged
bashirpartovi merged 2 commits intoAzure:mainfrom
bashirpartovi:dev/bashirpartovi/ident-impl
Feb 17, 2026
Merged

Decoupled ScenarioMetadata and InitializerMetadata from Identity by introducing RegistryEntry#1370
bashirpartovi merged 2 commits intoAzure:mainfrom
bashirpartovi:dev/bashirpartovi/ident-impl

Conversation

@bashirpartovi
Copy link
Contributor

Summary

Replaces Identifier with a new lightweight RegistryEntry dataclass as the base for ScenarioMetadata and InitializerMetadata. Registry metadata no longer carries Identifier-specific fields like hash, identifier_type, unique_name, or pyrit_version, keeping the registry layer focused on class discovery and lookup.

Motivation

Identifier includes hash computation, versioning, and storage-related metadata that are irrelevant for class registry entries. Using a minimal RegistryEntry base:

  • Reduces coupling between the registry and identity systems
  • Removes unnecessary fields from metadata objects
  • Makes the registry metadata contract explicit and minimal

Context

Please see #1369

Copy link
Contributor

@rlundeen2 rlundeen2 left a comment

Choose a reason for hiding this comment

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

Looks great! TY for thinking about this!

@bashirpartovi bashirpartovi merged commit 2484292 into Azure:main Feb 17, 2026
29 checks passed
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.

3 participants