Properly parse lines like "1-0:0.2.0((ER11))".

[PolarGoose]

Fix issue [dsmr] 2026.1.3 no longer works and prints out Unexpected '(' symbol repeatidly.
The smart meter ISk5\\2M550T-2006 output contains the line 1-0:0.2.0((ER11)). This causes the parser to fail with the error Unexpected '(' symbol.

Changes:

• The parser properly handles this case.
• Change fw version fields to strings instead on integer. Because a version can be an arbitrary string.
• Increase the version to 1.1.0

[Copilot]

Pull request overview

This PR fixes a parser issue where smart meters that output firmware version fields with double brackets (like 1-0:0.2.0((ER11))) would cause the parser to fail with an "Unexpected '(' symbol" error. Additionally, the PR changes firmware version and checksum fields from numeric types to strings to support arbitrary version formats.

Changes:

• Added double-bracket handling logic in both P1Parser (for line detection) and StringParser (for value parsing) to properly parse values like ((ER11)) as (ER11)
• Changed firmware version and checksum fields (fw_core_version, fw_core_checksum, fw_module_version, fw_module_checksum) from FixedValue numeric types to std::string types to support non-numeric version strings
• Added comprehensive test cases covering double-bracket scenarios and various edge cases

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
src/dsmr_parser/parser.h	Added logic to handle double brackets in both line detection (P1Parser) and string value parsing (StringParser)
src/dsmr_parser/fields.h	Changed firmware version and checksum fields from numeric FixedValue to std::string with StringField parser, increased max length to 96 characters
tests/parser_test.cpp	Added test cases for double-bracket parsing scenarios, added firmware fields to test data structure, updated CRC to match new message content

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.