build fresh; url changes; documentation changes; removed some unncess…

…ary logging

README.md

@@ -26,7 +26,7 @@ node . [options]
 node . --help
 
 # An example of running the CLI with a custom config, and a CLI-specified FHIR URL and pointer to a patients.ndjson file
-node . --config config/my-config.json -f https://bulk-match-provider-df228d68a43b.herokuapp.com/fhir/ --resource data/patients-to-match.ndjson
+node . --config config/my-config.json -f https://bulk-match.smarthealthit.org/fhir/ --resource data/patients-to-match.ndjson
 ```
 
 ## Options

build/lib/request.js

@@ -23,8 +23,7 @@ async function augmentedFetch(input, options = {}) {
         // After requests – handle logging and retrying
         .then(async (response) => {
         let body = await response.text();
-        const contentType = response.headers.get("content-type") || "";
-        if (body.length && contentType.match(/\bjson\b/i)) {
+        if (body.length && (0, utils_1.isJsonResponse)(response)) {
             body = JSON.parse(body);
         }
         // Create eventual response now so we can use it in error objects
@@ -61,8 +60,8 @@ async function augmentedFetch(input, options = {}) {
         // downstream by the postprocessing components
         if (options?.context?.interactive && body && (0, utils_1.isJsonResponse)(response)) {
             // @ts-ignore OperationOutcome errors
-            // Parse the body from above into JSON
-            const oo = JSON.parse(body);
+            // Treat the body as an operationOutcome to check for errors
+            const oo = body;
             if (oo.resourceType === "OperationOutcome") {
                 if (oo.issue.every((i) => i.code === "transient")) {
                     const msg = oo.issue
@@ -89,7 +88,6 @@ async function augmentedFetch(input, options = {}) {
     })
         .catch((e) => {
         debug("FAILED fetch: ", e.message);
-        console.error(e);
         throw e;
     }));
 }

build/reporters/cli.js

@@ -41,7 +41,7 @@ class CLIReporter extends reporter_1.default {
         lib_1.Utils.print("There was an error in the matching process: " + JSON.stringify(details)).commit();
     }
     onDownloadStart({ fileUrl, itemType, startTime, }) {
-        lib_1.Utils.print(`Begin ${itemType === "error" ? "error-file " : " "}download for ${fileUrl} at ${lib_1.Utils.formatDatetimeTimestamp(startTime)}...`).commit();
+        lib_1.Utils.print(`Begin ${itemType === "error" ? "error-file " : ""}download for ${fileUrl} at ${lib_1.Utils.formatDatetimeTimestamp(startTime)}...`).commit();
     }
     onDownloadComplete({ fileUrl, duration }) {
         lib_1.Utils.print(`${fileUrl} download complete in ${duration}`).commit();

src/lib/request.ts

@@ -103,9 +103,6 @@ async function augmentedFetch<T>(
             })
             .catch((e: Error) => {
                 debug("FAILED fetch: ", e.message);
-                if (process.env.NODE_ENV !== "test") {
-                    console.error(e);
-                }
                 throw e;
             })
     );

src/reporters/cli.ts

@@ -65,7 +65,7 @@ export default class CLIReporter extends Reporter {
         startTime: number;
     }) {
         Utils.print(
-            `Begin ${itemType === "error" ? "error-file " : " "}download for ${fileUrl} at ${Utils.formatDatetimeTimestamp(startTime)}...`,
+            `Begin ${itemType === "error" ? "error-file " : ""}download for ${fileUrl} at ${Utils.formatDatetimeTimestamp(startTime)}...`,
         ).commit();
     }
     onDownloadComplete({ fileUrl, duration }: { fileUrl: string; duration: string }) {