Add docs and make capitalization/naming of props consistent.

src/lib/cloud-provider.js

@@ -2,9 +2,19 @@ import log from './log.js';
 
 
 class CloudProvider {
-    constructor (cloudHost, vm, user, projectId) {
+    /**
+     * A cloud data provider which creates and manages a web socket connection
+     * to the Scratch cloud data server. This provider is responsible for
+     * interfacing with the VM's cloud io device.
+     * @param {string} cloudHost The url for the cloud data server
+     * @param {VirtualMachine} vm The Scratch virtual machine to interface with
+     * @param {string} username The username to associate cloud data updates with
+     * @param {string} projectId The id associated with the project containing
+     * cloud data.
+     */
+    constructor (cloudHost, vm, username, projectId) {
         this.vm = vm;
-        this.username = user;
+        this.username = username;
         this.projectId = projectId;
 
         // Open a websocket connection to the clouddata server
@@ -36,9 +46,7 @@ class CloudProvider {
             const message = JSON.parse(messageString);
             if (message.method === 'set') {
                 const varData = {
-                    projectId: this.projectId,
                     varUpdate: {
-                        projectId: message.project_id,
                         name: message.name,
                         value: message.value
                     }

src/lib/vm-manager-hoc.jsx

@@ -74,7 +74,7 @@ const vmManagerHOC = function (WrappedComponent) {
                 onLoadedProject: onLoadedProjectProp,
                 projectData,
                 projectId,
-                userName,
+                username,
                 /* eslint-enable no-unused-vars */
                 isLoadingWithId: isLoadingWithIdProp,
                 vm,
@@ -100,7 +100,7 @@ const vmManagerHOC = function (WrappedComponent) {
         onLoadedProject: PropTypes.func,
         projectData: PropTypes.oneOfType([PropTypes.object, PropTypes.string]),
         projectId: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
-        userName: PropTypes.string,
+        username: PropTypes.string,
         vm: PropTypes.instanceOf(VM).isRequired
     };