feat: support prefix in VariableScope

This change allows us to use the current VariableScope
with VaultVariables which have a fixed 'vault' prefix

lib/collection/variable-scope.js

@@ -111,6 +111,10 @@ _.inherit((
         if (mutations) {
             this.mutations = new MutationTracker(mutations);
         }
+
+        if (definition && definition.prefix) {
+            this._postman_scopePrefix = definition.prefix;
+        }
     }), Property);
 
 /**
@@ -136,6 +140,16 @@ _.assign(VariableScope.prototype, /** @lends VariableScope.prototype */ {
      */
     _postman_propertyRequiresId: true,
 
+    /**
+     * Defines the prefix associated with the VariableList variables for internal use.
+     *
+     * @private
+     * @type {String}
+     *
+     */
+    _postman_scopePrefix: '',
+
+
     /**
      * @private
      * @deprecated discontinued in v4.0
@@ -176,7 +190,9 @@ _.assign(VariableScope.prototype, /** @lends VariableScope.prototype */ {
      *                      false otherwise
      */
     has: function (key) {
-        var variable = this.values.oneNormalizedVariable(key),
+        const keyWithPrefix = this._postman_scopePrefix + key;
+
+        var variable = this.values.oneNormalizedVariable(keyWithPrefix),
             i,
             ii;
 
@@ -199,7 +215,9 @@ _.assign(VariableScope.prototype, /** @lends VariableScope.prototype */ {
      * @returns {*} The value of the specified variable across scopes.
      */
     get: function (key) {
-        var variable = this.values.oneNormalizedVariable(key),
+        const keyWithPrefix = this._postman_scopePrefix + key;
+
+        var variable = this.values.oneNormalizedVariable(keyWithPrefix),
             i,
             ii;
 
@@ -222,10 +240,12 @@ _.assign(VariableScope.prototype, /** @lends VariableScope.prototype */ {
      * @param {Variable.types} [type] - Optionally, the value of the variable can be set to a type
      */
     set: function (key, value, type) {
-        var variable = this.values.oneNormalizedVariable(key),
+        const keyWithPrefix = this._postman_scopePrefix + key;
+
+        var variable = this.values.oneNormalizedVariable(keyWithPrefix),
 
             // create an object that will be used as setter
-            update = { key, value };
+            update = { key: keyWithPrefix, value: value };
 
         _.isString(type) && (update.type = type);
 
@@ -248,11 +268,13 @@ _.assign(VariableScope.prototype, /** @lends VariableScope.prototype */ {
      * @param {String} key -
      */
     unset: function (key) {
+        const keyWithPrefix = this._postman_scopePrefix + key;
+
         var lastDisabledVariable;
 
         this.values.remove(function (variable) {
             // bail out if variable name didn't match
-            if (variable.key !== key) {
+            if (variable.key !== keyWithPrefix) {
                 return false;
             }
 
@@ -269,7 +291,7 @@ _.assign(VariableScope.prototype, /** @lends VariableScope.prototype */ {
 
         // restore the reference with the last disabled variable
         if (lastDisabledVariable) {
-            this.values.reference[key] = lastDisabledVariable;
+            this.values.reference[keyWithPrefix] = lastDisabledVariable;
         }
 
         // track the change if mutation tracking is enabled