fix: adapt FirmwareID to work with origin

[MrKevinWeiss]

Things change and so must we...

[Copilot]

Pull Request Overview

This PR adapts the FirmwareID parsing logic to correctly handle firmware identifiers containing the keyword "origin" in the version string. Key changes include:

• Adding a new test case in tests/test_firmware_id.py to ensure origin-based identifiers are parsed correctly.
• Introducing a new regex (_IDENTIFIER_RE_ORIGIN) in FirmwareID to handle the origin format.
• Modifying post_init to fall back to the origin regex when the default pattern does not match.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
tests/test_firmware_id.py	Adds a test for origin-based firmware identifiers.
src/lob_hlpr/lib_types.py	Updates FirmwareID regex matching to support "origin" formats.

[Copilot]

Consider adding an explicit error check for when both regex matches return None, so that a clear ValueError is raised rather than relying on a later exception. For example, after attempting both matches, check if m is None and then raise ValueError with an informative message.

Suggested change

	m = self._IDENTIFIER_RE_ORIGIN.match(self.id_string)
	m = self._IDENTIFIER_RE_ORIGIN.match(self.id_string)
	if not m:
	raise ValueError(
	f"Invalid firmware identifier string: '{self.id_string}'. "
	"It does not match the expected formats."
	)