fix(cordova): handle source-file with framework attribute

[jcesarmobile]

don't try to read source-file tags with framework attribute as they might be directories

closes #2506

[rfeague]

Unfortunately the fix above doesn't work when providers of plugins sometimes use to indicate what is actually a folder, but is not part of a framework. Example: https://cmbdn.cognex.com/files/download_latest/cmbSDK_Cordova
has this line in plugin.xml:

but ic_blank.imageset is actually a directory. As a result, npx cap sync (among other commands) fails at
@capacitor/cli/dist/ios/update.js:282:40
on the file
ios/capacitor-cordova-ios-plugins/sources/CmbsdkCordova/ic_blank.imageset
which has $ values:
{
src: 'src/ios/CMBScanner/SDK/cmbsdk-ios-binary/Resources/MWBScannerImages.xcassets/ic_blank.imageset',
'target-dir': '../../Images.xcassets'
}
Note that this passes the added conditional test: if (!codeFile.$.framework) {
Resulting in this error:
✖ update ios: EISDIR: illegal operation on a directory, read
[error] Error running update: Error: EISDIR: illegal operation on a directory, read
at Object.readSync (fs.js:577:3)
at tryReadSync (fs.js:353:20)
at Object.readFileSync (fs.js:390:19)
at /Users/roy.feague/proj/cmta-app/node_modules/@capacitor/cli/dist/ios/update.js:282:40
at Array.map ()
at /Users/roy.feague/proj/cmta-app/node_modules/@capacitor/cli/dist/ios/update.js:266:19
at Array.map ()
at copyPluginsNativeFiles (/Users/roy.feague/proj/cmta-app/node_modules/@capacitor/cli/dist/ios/update.js:254:20)
at Object.updateIOS (/Users/roy.feague/proj/cmta-app/node_modules/@capacitor/cli/dist/ios/update.js:27:9)
at async /Users/roy.feague/proj/cmta-app/node_modules/@capacitor/cli/dist/tasks/update.js:53:17 {
errno: -21,
syscall: 'read',
code: 'EISDIR'
}
In light of the fact that some providers are using source-file for directories, maybe we could just try/catch this read or explicitly verify that the object is a file and not a directory?

[jcesarmobile]

Can you report it as an issue?