remove {data} namespace from result

README.md

@@ -13,7 +13,7 @@ Send a simple query
 
 ```js
 const graphql = require('@octokit/graphql')
-const { data } = await graphql(`{
+const { repository } = await graphql(`{
   repository(owner:"octokit", name:"graphql.js") {
     issues(last:3) {
       edges {
@@ -34,7 +34,7 @@ const { data } = await graphql(`{
 
 ```js
 const graphql = require('@octokit/graphql')
-const { data } = await graphql(`query lastIssues($owner: String!, $repo: String!, $num: Int = 3) {
+const { lastIssues } = await graphql(`query lastIssues($owner: String!, $repo: String!, $num: Int = 3) {
     repository(owner:$owner, name:$repo) {
       issues(last:$num) {
         edges {
@@ -62,7 +62,7 @@ const graphql = require('@octokit/graphql').defaults({
     authorization: `token secret123`
   }
 })
-const { data } = await graphql(`{
+const { repository } = await graphql(`{
   repository(owner:"octokit", name:"graphql.js") {
     issues(last:3) {
       edges {
@@ -79,7 +79,7 @@ Pass query together with headers and variables
 
 ```js
 const graphql = require('@octokit/graphql')
-const { data } = await graphql({
+const { lastIssues } = await graphql({
   query: `query lastIssues($owner: String!, $repo: String!, $num: Int = 3) {
     repository(owner:$owner, name:$repo) {
       issues(last:$num) {
@@ -108,7 +108,7 @@ const graphql = require('@octokit/graphql').defaults({
     authorization: `token secret123`
   }
 })
-const { data } = await graphql(`{
+const { repository } = await graphql(`{
   repository(owner:"acme-project", name:"acme-repo") {
     issues(last:3) {
       edges {
@@ -138,7 +138,7 @@ const query = `{
 }`
 
 try {
-  const { data } = await graphql(query)
+  const result = await graphql(query)
 } catch (error) {
   // server responds with
   // {

lib/error.js

@@ -4,6 +4,7 @@ module.exports = class GraphqlError extends Error {
     super(message)
 
     Object.assign(this, response.data)
+    this.name = 'GraphqlError'
     this.request = request
 
     // Maintains proper stack trace (only available on V8)

lib/graphql.js

@@ -27,7 +27,7 @@ function graphql (request, query, options) {
   return request(requestOptions)
     .then(response => {
       if (response.data.data) {
-        return response.data
+        return response.data.data
       }
 
       throw new GraphqlError(requestOptions, response)

test/defaults-test.js

@@ -59,8 +59,8 @@ describe('graphql.defaults()', () => {
         }
       }
     }`)
-      .then(response => {
-        expect(response.data).to.deep.equal(mockData)
+      .then(result => {
+        expect(result).to.deep.equal(mockData)
       })
   })
 
@@ -115,8 +115,8 @@ describe('graphql.defaults()', () => {
         }
       }
     }`)
-      .then(response => {
-        expect(response.data).to.deep.equal(mockData)
+      .then(result => {
+        expect(result).to.deep.equal(mockData)
       })
   })
 })

test/error-test.js

@@ -41,7 +41,7 @@ describe('errors', () => {
       }
     })
 
-      .then(response => {
+      .then(result => {
         throw new Error('Should not resolve')
       })
 

test/graphql-test.js

@@ -69,8 +69,8 @@ describe('graphql()', () => {
       }
     })
 
-      .then(response => {
-        expect(response.data).to.deep.equal(mockData)
+      .then(result => {
+        expect(result).to.deep.equal(mockData)
       })
   })