Typed metadata not handled on commodities

[zacchiro]

Typed metadata are handled like regular metadata (and hence always rendered as string literals) when used on commodities (at least), whereas they work properly on transactions.

For example, the following ledger snippet:

commodity EUR
    note Euro
    format 1000.00 EUR
    ; normal_metadata: 42.0
    ; typed_metadata:: 42.0

2025-01-21 * salary
    ; normal_metadata: 42.0
    ; typed_metadata:: 42.0
    Income:Salary                                               -1000.00 EUR
    Assets:Checking                                              1000.00 EUR

is translated by ledger2beancount to the following beancount snippet:

1970-01-01 commodity EUR
    name: "Euro"
    normal_metadata: "42.0"
    typed_metadata: "42.0"

2025-01-21 * "salary"
    normal_metadata: "42.0"
    typed_metadata: 42.0
    Income:Salary                                               -1000.00 EUR
    Assets:Checking                                              1000.00 EUR

Note the (incorrect) double quotes around typed_metadata value in the EUR commodity, which are (correctly) missing in the transaction case.

[tbm]

Nice catch.

The problem is that we use pp_metadata to print this (line 2625) which doesn't handle typed metadata where for transactions we create a metadata object using create_metadata_from_regex (line 1143) and that is printed correctly.

I need to check how to harmonize/unify this.

I'm away for 10 days starting on Saturday, so this will have to wait until February.