It doesn't look like we're following the Inspector.json syntax properly for return values.  Here's an example of their structure:

```javascript
            {
                "name": "getProcessMemoryDistribution",
                "parameters": [
                    { "name": "reportGraph", "type": "boolean", "optional": true, "description": "Whether native memory graph should be reported in addition to aggregated statistics." }
                ],
                "returns": [
                    { "name": "distribution", "$ref": "MemoryBlock", "description": "An object describing all memory allocated by the process"}
                ]
            }
```

What we're doing instead of roughly like this:

```javascript
            {
                "getProcessMemoryDistribution",
                ......,
                [
                    { "name": "reportGraph", "type": "boolean", "optional": true, "description": "Whether native memory graph should be reported in addition to aggregated statistics." }
                ],
                {
                    "distribution": { "$ref": "MemoryBlock", "description": "An object describing all memory allocated by the process"}
                }
            }
```

It seems like at the least we should change the return object properties into an array of objects with 'name' members, rather than have an object where each key is the name of one property.  And as NJ said, this would document only the success arg -- we'd gloss over the fact that this isn't technically a return value, and the error arg to the callback would go unmentioned.  @dangoor does that sound ok to you too?