Skip to content

Conversation

@seer-by-sentry
Copy link
Contributor

Fixes LAUNCHPAD-6D. The issue was that: ProtoXmlUtils expects protobuf XmlAttribute objects, but receives dataclass XmlAttribute objects lacking the 'compiled_item' field.

  • Introduced a new internal dictionary _attrs_to_pb_attrs to map the id of converted attribute lists back to their original protobuf attribute objects.
  • Modified _convert_pb_element_to_xml_node to store this mapping after converting protobuf attributes.
  • Updated _get_attr_value to retrieve and use the original protobuf attributes when calling ProtoXmlUtils methods for resource lookup, ensuring access to compiled_item and other protobuf-specific data.
  • Added a fallback mechanism in _get_attr_value to use the converted attributes if the original protobuf attributes cannot be found, along with a warning.

This fix was generated by Seer in Sentry, triggered by Hector Dearman. 👁️ Run ID: 8532303

Not quite right? Click here to continue debugging with Seer.

@chromy chromy closed this Jan 13, 2026
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