patternlab-node icon indicating copy to clipboard operation
patternlab-node copied to clipboard

Patternlab engine-twig watcher doesn't build correctly

Open ray87 opened this issue 3 years ago • 3 comments

I am using Pattern Lab Node v5.14.2 on Windows / Mac / Dockerized, tested with Node v12.22, v14.x, v15.x, using a Gulp or Vanilla Edition.

Expected Behavior

After changing pattern, the watcher should build all components and create new rendered versions for UI. The process doesn't throw any errors or warning, everything seems to run as it should and refreshes the browser. The UI then should show updated patterns.

Actual Behavior

After browser updates, it still shows old rendered version, but updated twig-file. After running patternlab build from console and refreshing browser manually, it shows updated versions.

Steps to Reproduce

Install @pattern-lab/[email protected], create an dummy pattern with just text in it. Run pl:serve, change pattern, check browser.

ray87 avatar Apr 12 '21 07:04 ray87

I experience a similar issue with watchers

SaroGFX avatar Apr 15 '21 06:04 SaroGFX

I am using the engine-twig-php and for me this happens too occasionally.

Think in this case it has something to do with @basalt/twig-renderer, but i am not 100% sure. Thats why i am posting here.

Once there was an error message with cannot connect to server....

A few minutes ago it was "failing silently", everything looked good no errors etc. but the pattern did not get updated. I checked lsof -i -P | grep -i "listen" and saw that there was a php process listening at port 53420. Killed that and now it is working again. All without restarting my watcher.

It would be great if someone can have a look at that issue. Its very time consuming especially if no errors happen.

Maybe @EvanLovely can say something about that?

hadl avatar May 18 '21 12:05 hadl

A behaviour I can reproduce is, that it actually works when I make a change to the JSON file specific to the twig file I updated. So, I'm often forced to quickly save the JSON file (even though I didn't make a change or only added a space), this way I can work around the issue.

(Interesting enough, the change to the json file alone builds match faster in watch mode, seems incremental, while a change to a twig file seems to build everything from scratch, yet doesn't show the change in the UI).

renestalder avatar Oct 25 '21 14:10 renestalder