Make sure normalizeCSSVarsValue runs for more cases

[Zn4rK]

const color = createVar('color');

const x = style({
   background: `rgba(${color}, 0.15)`,
});

Will produce this css:

.a1 {
   background: rgba(--color, 0.15);
}

Instead of this:

.a1 {
   background: rgba(var(--color), 0.15);
}

Source:
https://github.com/eagerpatch/navita/blob/main/packages/engine/src/helpers/normalizeCSSVarsValue.ts#L3

At first glance, it looks like the regex is still valid, but we need to expand the test cases, and modify or remove the if-statement that decides if we should normalize CSS var values.

[vinayakhegde1]

@Zn4rK Although I'm just a beginner. I would like to work on this but I don't have rust on my system do I need to install it? also how do i run this on my local machine if there are any contributing guidelines do let me know :)

[Zn4rK]

Hello @vinayakhegde1! Nice! You should for sure try!

To build the whole project, you'll need to have rust installed. But the @navita/engine does not require the rust package.

Right now, to not have to install the rust package, while still maintaining the correct relationship between the packages, would be to potentially use a .pnpmfile.cjs:

module.exports = {
  hooks: {
    readPackage(packageJson) {
      if (packageJson.name === '@navita/swc') {
        packageJson.version = 'latest';
      }
      return packageJson;
    },
  },
};

(This will affect the lock file incorrectly though, so dependencies would need to be re-installed before publishing, but you should be able to run it locally without having to install rust)

I'll try to compose some contributing guidelines later this week, and also think about ways to contribute without having to install rust.