fix: can not find the variable that be created in plugin.

[bestlyg]

What: clear cache when the type of Declaration be returned.

Why:
I want to define a variable in plugin. It will insert @x: 33px to current scope.

module.exports = class LessPluginBestMixin {
    install(less, pluginManager, functions) {
        functions.add("x", () => {
            return new less.tree.Declaration(
                "@x",
                new less.tree.Value(new less.tree.Keyword("33px"))
            );
        });
        functions.add("define_var", (key) => {});
    }
};

When I write the code to a less file:

@plugin "the-plugin-path";

div {
    x();
    width: @x;
}

It works and return the css code:

div {
    width: 33px;
}

But when I write the code to a less file:

@plugin "the-plugin-path";

@a: 1px;

div {
    height: @a;
    x();
    width: @x;
}

It not works and return a error cause can not find the variable of @x.

I had tried debug the source code of less and I found the code:

The height need read this variable and execute the function of variable and be cached.
It lead to the function of variable can not run again when the width read the @x.

I changed the code that will clear the cache when the function return a Declaration.

I added a issue. URL

Please check this PR.

Checklist:

• Documentation
• Added/updated unit tests
• Code complete

[bestlyg]

@matthew-dean hi can you help check this PR? I need use it in my project. I had added unit test.

[iChenLei]

@matthew-dean hi can you help check this PR? I need use it in my project. I had added unit test.

If you're in a hurry, you don't need to wait for code merging and the release of a new version. You can use libraries like patch-package(or pnpm patch) to apply patches to less npm package, temporarily resolving your issue.

[bestlyg]

@matthew-dean hi can you help check this PR? I need use it in my project. I had added unit test.

If you're in a hurry, you don't need to wait for code merging and the release of a new version. You can use libraries like patch-package(or pnpm patch) to apply patches to less npm package, temporarily resolving your issue.

Thank you. It was able to solve my problem temporarily. Please check this PR when you have time.

[matthew-dean]

Looks fine, thanks!

[bestlyg]

Hi, I found the pr is approved. May I ask when the pr merge in main branch. Because now I need use the feature in other repo.And I can’t use pnpm patch in each repo.Thanks.

[bestlyg]

@iChenLei @matthew-dean May I ask when the pr merge in main branch. I need use it in other repo.