local values referencing this inside Mappings are included in entry count when binary encoded #324

New Issue

adam · 2025-12-30T01:23:32+01:00

adam commented

2025-12-30 01:23:32 +01:00

Originally created by @sin-ack on GitHub (Jul 25, 2025).

Reproduced on 0.29.0.

Reproducer:

foo = new Mapping {
    local self = this
    ["foo"] = new Dynamic { name = "foo" }
    ["bar"] = new Dynamic { name = self["foo"].name + "bar" }
}

Evaluate foo with something that communicates with pkl server. The returned value will be (in JSON repr):

[3, (map size: 3) {"foo": [0x3, (map size: 1) {"name": "foo"}], "bar": [3, (map size: 1) {"name": "foobar"}]}]

Notice that it returns a map of size 3. However, the MessagePack data only contains 2 items. This causes a decode error on my side.

Originally created by @sin-ack on GitHub (Jul 25, 2025). Reproduced on 0.29.0. Reproducer: ```pkl foo = new Mapping { local self = this ["foo"] = new Dynamic { name = "foo" } ["bar"] = new Dynamic { name = self["foo"].name + "bar" } } ``` Evaluate `foo` with something that communicates with `pkl server`. The returned value will be (in JSON repr): ```js [3, (map size: 3) {"foo": [0x3, (map size: 1) {"name": "foo"}], "bar": [3, (map size: 1) {"name": "foobar"}]}] ``` Notice that it returns a map of size 3. However, the MessagePack data only contains 2 items. This causes a decode error on my side.

adam closed this issue

2025-12-30 01:23:32 +01:00

adam commented

2025-12-30 01:23:33 +01:00

@bioball commented on GitHub (Jul 25, 2025):

Thanks for the easily reproducible bug report!

@bioball commented on GitHub (Jul 25, 2025): Thanks for the easily reproducible bug report!

adam referenced this issue

2025-12-30 01:24:58 +01:00

[PR #324] [MERGED] Enable Gradle build caching #500

Sign in to join this conversation.