fix: Changes in node_modules are always skipped

Currently all changes in `node_modules` are skipped as we do not check for changes in case `--syncAllFiles` is not passed.
In fact the meaning of this option is for LiveSync operations and by default CLI does not watch for changes in node_modules. However, the separate commands like prepare, build, test and even the initial LiveSync must check the node_modules and prepare the correct files.
In latest versions (4.2.x) we were always checking all `node_modules` even during change of a single file during LiveSync. After that we've applied a change in the current master branch to skip node_modules check in case specific option is passed to the projectChangesService. The problem with latest approach is that we've missed this option is always passed based on syncAllFiles.
This led to the problem that once you have node_modules setup, CLI from master branch never checks for changes in these files and does not move them to platforms dir.

With the current changes, node_modules will be checked on all commands. Once LiveSync starts, all consecutive changes in project files will not check node_modules unless `syncAllFiles` option is passed to CLI.

Author: rosen-vladimirov

lib/commands/appstore-upload.ts

@@ -60,7 +60,6 @@ export class PublishIOS implements ICommand {
 			const platformInfo: IPreparePlatformInfo = {
 				platform,
 				appFilesUpdaterOptions,
-				skipModulesNativeCheck: !this.$options.syncAllFiles,
 				platformTemplate: this.$options.platformTemplate,
 				projectData: this.$projectData,
 				config: this.$options,

lib/commands/build.ts

@@ -17,7 +17,6 @@ export class BuildCommandBase {
 		const platformInfo: IPreparePlatformInfo = {
 			platform,
 			appFilesUpdaterOptions,
-			skipModulesNativeCheck: !this.$options.syncAllFiles,
 			platformTemplate: this.$options.platformTemplate,
 			projectData: this.$projectData,
 			config: this.$options,

lib/commands/prepare.ts

@@ -14,7 +14,6 @@ export class PrepareCommand implements ICommand {
 		const platformInfo: IPreparePlatformInfo = {
 			platform: args[0],
 			appFilesUpdaterOptions,
-			skipModulesNativeCheck: !this.$options.syncAllFiles,
 			platformTemplate: this.$options.platformTemplate,
 			projectData: this.$projectData,
 			config: this.$options,

lib/services/livesync/livesync-service.ts

@@ -469,7 +469,6 @@ export class LiveSyncService extends EventEmitter implements IDebugLiveSyncServi
 					deviceBuildInfoDescriptor,
 					liveSyncData,
 					settings,
-					skipModulesNativeCheck: !liveSyncData.watchAllFiles,
 					bundle: liveSyncData.bundle,
 					release: liveSyncData.release,
 					env: liveSyncData.env

lib/services/test-execution-service.ts

@@ -59,7 +59,6 @@ class TestExecutionService implements ITestExecutionService {
 					const preparePlatformInfo: IPreparePlatformInfo = {
 						platform,
 						appFilesUpdaterOptions,
-						skipModulesNativeCheck: !this.$options.syncAllFiles,
 						platformTemplate: this.$options.platformTemplate,
 						projectData,
 						config: this.$options,
@@ -187,7 +186,6 @@ class TestExecutionService implements ITestExecutionService {
 				const preparePlatformInfo: IPreparePlatformInfo = {
 					platform,
 					appFilesUpdaterOptions,
-					skipModulesNativeCheck: !this.$options.syncAllFiles,
 					platformTemplate: this.$options.platformTemplate,
 					projectData,
 					config: this.$options,