ProgressNS / sidekick-feedback

This repository is for feedback regarding NativeScript Sidekick. Use the issues system here to submit feature requests or vote for existing ones.
45 stars 9 forks source link

Hot Module Reload takes almost 2 minutes! #483

Open baermathias opened 4 years ago

baermathias commented 4 years ago

Environment tns info --> √ Component nativescript has 6.5.0 version and is up to date.

Describe the bug I changed the button color and saved the file at 13:37:35.187 (start time) and the whole process ended almost two minutes later at 13:39:11.787 (end time)

To Reproduce Create a nativescript-vue project with the CLI and at least two single file components. Then make some changes and save them.

Expected behavior Display changes immediately like Vue.js for Web

Additional context I am using Sidekick and Android Emulator with Galaxy_Nexus_API_29:5554

Here is the complete Sidekick console output, after a change is detected:

[20-04-08 13:37:35.187] (CLI) File change detected. Starting incremental webpack compilation...
[20-04-08 13:37:35.824] (CLI) Hash: ff649f4f10a09fc09018
[20-04-08 13:37:35.824] (CLI) Version: webpack 4.27.1
[20-04-08 13:37:35.824] (CLI) Time: 623ms
[20-04-08 13:37:35.824] (CLI) Built at: 2020-04-08 13:37:35
[20-04-08 13:37:35.824] (CLI)                                     Asset      Size   Chunks             Chunk Names
[20-04-08 13:37:35.824] (CLI)      8ecd807330132aa55e4b.hot-update.json  48 bytes           [emitted]  
[20-04-08 13:37:35.824] (CLI) bundle.8ecd807330132aa55e4b.hot-update.js  3.22 KiB   bundle  [emitted]  bundle
[20-04-08 13:37:35.824] (CLI)                                 bundle.js   372 KiB   bundle  [emitted]  bundle
[20-04-08 13:37:35.824] (CLI)                                runtime.js  71.4 KiB  runtime  [emitted]  runtime
[20-04-08 13:37:35.824] (CLI)  + 1 hidden asset
[20-04-08 13:37:35.824] (CLI) Entrypoint bundle = runtime.js vendor.js bundle.js bundle.8ecd807330132aa55e4b.hot-update.js
[20-04-08 13:37:35.824] (CLI) [./ sync ^\.\/app\.(css|scss|less|sass)$] . sync nonrecursive ^\.\/app\.(css|scss|less|sass)$ 175 bytes {bundle} [built]
[20-04-08 13:37:35.824] (CLI) [./ sync recursive (?<!\bApp_Resources\b.*)(?<!\.\/\btests\b\/.*?)\.(xml|css|js|kt|(?<!\.d\.)ts|(?<!\b_[\w-]*\.)scss)$] . sync (?<!\bApp_Resources\b.*)(?<!\.\/\btests\b\/.*?)\.(xml|css|js|kt|(?<!\.d\.)ts|(?<!\b_[\w-]*\.)scss)$ 188 bytes {bundle} [built]
[20-04-08 13:37:35.824] (CLI)     + 344 hidden modules
[20-04-08 13:37:35.846] (CLI) Webpack compilation complete. Watching for file changes.
[20-04-08 13:37:35.848] (CLI) Webpack build done!
[20-04-08 13:38:51.523] (CLI) Successfully transferred bundle.8ecd807330132aa55e4b.hot-update.js on device emulator-5554.
[20-04-08 13:38:51.527] (CLI) Successfully transferred 8ecd807330132aa55e4b.hot-update.json on device emulator-5554.
[20-04-08 13:39:11.540] (CLI) Refreshing application on device emulator-5554...
[20-04-08 13:39:11.787] (CLI) Successfully synced application org.nativescript.test3 on device emulator-5554.
NickIliev commented 4 years ago

@baermathias while using local CLI build via this test app I am unable to reproduce the issue. It could be an issue related to Sidekick and if that is the case report the issue here

baermathias commented 4 years ago

With tns preview on my physical smartphone it works well. But with Sidekick + Emulator it takes years. Could you move this issue to Sidekick?