Merge pull request #6 from vtex/bugfix/error-on-export

Replace graphql client with http POST

.github/workflows/ci.yml

@@ -14,7 +14,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1
         with:
-            node-version: 12
+            node-version: 20
       - run: yarn install --ignore-scripts
       - run: yarn run ci:prettier-check
       - run: yarn run lint
@@ -25,7 +25,7 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        node-version: [12]
+        node-version: [20]
         os: [ubuntu-latest, windows-latest, macOS-latest]
 
     steps:

.github/workflows/plugin-documentation.yml

@@ -16,7 +16,7 @@ jobs:
       - name: Set up Node.js
         uses: actions/setup-node@v1
         with:
-          node-version: 12.x
+          node-version: 20.x
       - name: Install dependencies
         run: yarn install
       - name: Update README

.github/workflows/publish-prerelease-npm.yml

@@ -14,7 +14,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1
         with:
-            node-version: 12
+            node-version: 20
             registry-url: https://registry.npmjs.org/
       - run: yarn install --ignore-scripts
       - run: yarn build

.github/workflows/publish-stable-npm.yml

@@ -14,7 +14,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v1
         with:
-            node-version: 12
+            node-version: 20
             registry-url: https://registry.npmjs.org/
       - run: yarn install --ignore-scripts
       - run: yarn build

CHANGELOG.md

@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- Use http POST instead of graphql client to list redirects
+
 ## [1.0.0] - 2021-07-12
 
 ### Fixed

README.md

@@ -31,7 +31,7 @@ $ npm install -g @vtex/cli-plugin-redirects
 $ oclif-example COMMAND
 running command...
 $ oclif-example (-v|--version|version)
-@vtex/cli-plugin-redirects/1.0.0 linux-x64 node-v12.22.1
+@vtex/cli-plugin-redirects/1.0.0 linux-x64 node-v20.17.0
 $ oclif-example --help [COMMAND]
 USAGE
   $ oclif-example COMMAND
@@ -58,9 +58,9 @@ ARGUMENTS
   CSVPATH  CSV file containing the URL paths to delete.
 
 OPTIONS
-  -h, --help     show CLI help
-  -v, --verbose  Show debug level logs
-  --trace        Ensure all requests to VTEX IO are traced
+  -h, --help     Shows this help message.
+  -v, --verbose  Shows debug level logs.
+  --trace        Ensures all requests to VTEX IO are traced.
 
 EXAMPLE
   vtex redirects delete csvPath
@@ -80,9 +80,9 @@ ARGUMENTS
   CSVPATH  Name of the CSV file.
 
 OPTIONS
-  -h, --help     show CLI help
-  -v, --verbose  Show debug level logs
-  --trace        Ensure all requests to VTEX IO are traced
+  -h, --help     Shows this help message.
+  -v, --verbose  Shows debug level logs.
+  --trace        Ensures all requests to VTEX IO are traced.
 
 EXAMPLE
   vtex redirects export csvPath
@@ -102,10 +102,10 @@ ARGUMENTS
   CSVPATH  Name of the CSV file.
 
 OPTIONS
-  -h, --help     show CLI help
+  -h, --help     Shows this help message.
   -r, --reset    Removes all redirects previously defined.
-  -v, --verbose  Show debug level logs
-  --trace        Ensure all requests to VTEX IO are traced
+  -v, --verbose  Shows debug level logs.
+  --trace        Ensures all requests to VTEX IO are traced.
 
 EXAMPLE
   vtex redirects import csvPath

package.json

@@ -41,7 +41,7 @@
     "ts-jest": "^25.2.1",
     "ts-node": "^8",
     "typescript": "^3.9.7",
-    "vtex": "^2.128.0"
+    "vtex": "4.1.0"
   },
   "engines": {
     "node": ">=8.0.0"

src/clients/apps/Rewriter.ts

@@ -1,4 +1,5 @@
-import { AppGraphQLClient, InstanceOptions, IOContext } from '@vtex/api'
+import type { InstanceOptions, IOContext } from '@vtex/api'
+import { AppGraphQLClient } from '@vtex/api'
 import { IOClientFactory } from 'vtex'
 
 export interface RedirectInput {
@@ -40,32 +41,30 @@ export class Rewriter extends AppGraphQLClient {
   }
 
   public exportRedirects = (next?: string): Promise<ExportResponse> =>
-    this.graphql
-      .query<{ redirect: { listRedirects: ExportResponse } }, { next?: string }>(
+    this.http
+      .post<{ data: { redirect: { listRedirects: ExportResponse } } }>(
+        '',
         {
-          query: `
-      query ListRedirects($next: String) {
-        redirect {
-          listRedirects(next: $next) {
-            next
-            routes {
-              binding
-              from
-              to
-              type
-              endDate
-            }
-          }
-        }
-      }
-      `,
+          query: `query ListRedirects($next: String) {
+                    redirect {
+                      listRedirects(next: $next) {
+                        next
+                        routes {
+                          from
+                          to
+                          type
+                          endDate
+                        }
+                      }
+                    }
+                  }`,
           variables: { next },
         },
         {
           metric: 'rewriter-get-redirects',
         }
       )
-      .then((res) => res.data?.redirect?.listRedirects) as Promise<ExportResponse>
+      .then((res) => res.data.redirect?.listRedirects) as Promise<ExportResponse>
 
   public importRedirects = (routes: RedirectInput[]): Promise<boolean> =>
     this.graphql

src/modules/rewriter/export.ts

@@ -3,7 +3,8 @@ import { readJson, writeFile } from 'fs-extra'
 import { Parser } from 'json2csv'
 import ora from 'ora'
 import { createInterface } from 'readline'
-import { Redirect, Rewriter } from '../../clients/apps/Rewriter'
+import type { Redirect } from '../../clients/apps/Rewriter'
+import { Rewriter } from '../../clients/apps/Rewriter'
 import { SessionManager, logger, isVerbose } from 'vtex'
 import {
   deleteMetainfo,