Reference tokens not correct

[reivaxui]

Hi,
When I generate json tokens, the references are incorrect.

Example :
I have 3 modes "Theme A", "Theme B", "Theme C".
Each value reference take the last mode (theme C)

"value": "{colors.themeC.palette.brand.secondary}"
Reference value must be {colors.themeA.palette.brand.secondary}

  "colors": {
    "themeA": {
      "palette": {
        "brand": {
          "primary": {
            "type": "color",
            "value": "#001944ff",
            "blendMode": "normal"
          }
        }
      },
      "surfaces": {
        "primary": {
          "default": {
            "type": "color",
            "value": "{colors.themeC.palette.brand.secondary}"
          }
        }
      }
    },
    "themeB": {
      "palette": {
        "brand": {
          "primary": {
            "type": "color",
            "value": "#001944ff",
            "blendMode": "normal"
          }
        }
      },
      "surfaces": {
        "primary": {
          "default": {
            "type": "color",
            "value": "{colors.themeC.palette.brand.secondary}"
          }
        }
      }
    },
    "themeC": {
      "palette": {
        "brand": {
          "primary": {
            "type": "color",
            "value": "#001944ff",
            "blendMode": "normal"
          }
        }
      },
      "surfaces": {
        "primary": {
          "default": {
            "type": "color",
            "value": "{colors.themeC.palette.brand.secondary}"
          }
        }
      }
    }
  }

[JackHowa]

I'm seeing this as well

[Franjoo]

Yes, it would be really nice if that could be fixed.

[Franjoo]

Any updates here?

[JulianBissekkou]

I can also reproduce this. This sadly makes the plugin unusable :/

[lukasoppermann]

Sorry I missed this. I will loom into the Pr by @JackHowa to merge it

[lukasoppermann]

Hey, I looked into it but it is not as easy as you made it seem. 🤣

@JackHowa can you please just add the needed test to the PR you made? #289

Only the test please. Happy to look into the other PR you closed as a separate one. But I would love the PR to stay as it is and just add a test so I don't have to re-review and can merge it as soon as the test is there. 🙏

[JackHowa]

here's the tests! https://github.com/lukasoppermann/design-tokens/pull/300/files#diff-e837ef5c10534f796d3fb7ce69273c9fc247852124c1ea52c0eed956756c9281

[JackHowa]

ah I didn't make that pr linked #289 but I can see if I can make a clean pr for ya

[JackHowa]

here's the updated pr #300. I had to break out some utils so they were easier to test

[lukasoppermann]

Okay, I merged and released this now. Can you please test if this fixes it?

[JackHowa]

Okay, I merged and released this now. Can you please test if this fixes it?

Yep works for me!