Use npm's semver rules for plugin's nativescript key

[rosen-vladimirov]

Problem with plugins verification with runtimes

Currently each NativeScript plugin must have nativescript key in it's package.json and define the minimum required version of each supported runtime. The structure is :

"name": "my-plugin",
"version": "1.0.0",
"nativescript": { 
    "platforms": {
        "ios": "1.5.0",
        "android": "1.5.0",
    }
}

When tns plugin add my-plugin is executed, CLI checks the values against current runtimes versions and informs the user if a plugin cannot be used with current version.
However CLI uses the specified version in the plugin as minimum required. So for example in case the project uses tns-android 1.6.0, 1.6.2 or even 2.0.0 for example, it can use my-plugin.
This makes it really hard to control the versions against which the plugin is tested and verified by the plugin author.

Suggested solution

Do not reinvent the wheel!
Use semver rules for the nativescript key in package.json, for example:

"name": "my-plugin",
"version": "1.0.0",
"nativescript": { 
    "platforms": {
        "ios": "^1.5.0",
        "android": "~1.5.0",
    }
}

This way in case the project targets android 1.5.2, the plugin can be installed, but in case it targets 1.6.0, the plugin will not be available. More information for the version ranges:
https://docs.npmjs.com/getting-started/semantic-versioning#semver-for-consumers

Breaking change

In case we implement this idea, all plugins, which currently has nativescript key inside their package.json, will not work, as the values: "1.5.0" will mean - can work with this version only.

[rosen-vladimirov]

I would like to hear the opinion of plugins authors. What do you think @NathanaelA, @enchev , @maknz

[rosen-vladimirov]

Ping @tjvantoll , @EddyVerbruggen , @sitefinitysteve, @bradmartin, @AntonioCuevaUrraco, @hdeshev , @vakrilov , @PeterStaev, @alexziskind1, @emiloberg, @PanayotCankov , @TobiasHennig
What do you think guys?

[sitefinitysteve]

In favor... Better now than later

[enchev]

👍

[PeterStaev]

👍

[tjvantoll]

I’m in favor, however, I think we’ll need to have a grace period where the CLI interprets a specific version number—e.g. "1.6.0"—the old way. This will give plugin authors time to migrate, and also prevent having a CLI release that instantaneously breaks all plugins.

[NathanaelA]

This would definitely solve some issues.

[NathanaelA]

@sitefinitysteve @enchev @PeterStaev @tjvantoll part of this issue is raised because of issue (NativeScript/android#93) any of you want to weigh in on the ability to get a runtimeVersion at runtime? We are debating the best ways to handle certain issues.

[teobugslayer]

I second @tjvantoll that we need a grace period. I have another idea - to introduce a flag to explicitly say which behavior do you need - the old or the new. We clearly communicate that this flag will be supported for say 2 versions and then leave only the new code.

[PeterStaev]

@NathanaelA although I see your point in the other issue about detecting runtime version, I think I would never want to add conditional code by version in the plugin code. Otherwise it will become rather messy to manage. I would rather update the plugin to work with the current runtime version. Then it is up to users, utilizing that plugin to know which version of the plugin they want to install and which is compatible with their runtime.
Another idea would be to change the tns plugin add command to check the published versions of a plugin, read their config and find the best suited version for the project's runtime version.