Tidy and add a changelog entry.

Author: jaydenseric

changelog.md

@@ -8,6 +8,10 @@
 - Use the TypeScript v5.5+ JSDoc tag `@import` to import types in modules.
 - Updated dev dependencies, some of which require newer Node.js versions than previously supported.
 
+### Minor
+
+- Added a new function `createUploadLink` option `includeUnusedVariables` defaulting to `false` to toggle including unused GraphQL variables in the request (similar to the Apollo `BaseHttpLink` option [`includeUnusedVariables`](https://www.apollographql.com/docs/react/api/link/apollo-link-base-http#options-includeunusedvariables)), via [#348](https://github.com/jaydenseric/apollo-upload-client/pull/348).
+
 ### Patch
 
 - Updated the package scripts:

createUploadLink.mjs

@@ -70,8 +70,8 @@ import isExtractableFile from "./isExtractableFile.mjs";
  *   {@linkcode fetchOptions}.
  * @param {boolean} [options.includeExtensions] Toggles sending `extensions`
  *   fields to the GraphQL server. Defaults to `false`.
- * @param {boolean} [options.includeUnusedVariables] * If set to true, the default behavior of stripping unused variables
- * from the request will be disabled. Defaults to `false`.
+ * @param {boolean} [options.includeUnusedVariables] Toggles including unused
+ *   GraphQL variables in the request. Defaults to `false`.
  * @returns A [terminating Apollo Link](https://www.apollographql.com/docs/react/api/link/introduction/#the-terminating-link).
  * @example
  * A basic Apollo Client setup:
@@ -146,12 +146,11 @@ export default function createUploadLink({
       contextConfig,
     );
 
-    if (body.variables && !includeUnusedVariables) {
+    if (body.variables && !includeUnusedVariables)
       body.variables = filterOperationVariables(
         body.variables,
         operation.query,
       );
-    }
 
     const { clone, files } = extractFiles(body, customIsExtractableFile, "");
 

createUploadLink.test.mjs

@@ -314,7 +314,7 @@ describe("Function `createUploadLink`.", { concurrency: true }, () => {
     deepStrictEqual(nextData, payload);
   });
 
-  it("Option `includeUnusedVariables`, set to false", async () => {
+  it("Option `includeUnusedVariables`, set to false.", async () => {
     /** @type {unknown} */
     let fetchInput;
 
@@ -325,7 +325,7 @@ describe("Function `createUploadLink`.", { concurrency: true }, () => {
     let nextData;
 
     const query = "query ($a: Boolean) {\n  a(a: $a)\n}";
-    const payload = { data: { a: true, b: true } };
+    const payload = { data: { a: true } };
 
     await timeLimitPromise(
       /** @type {Promise<void>} */ (
@@ -384,7 +384,7 @@ describe("Function `createUploadLink`.", { concurrency: true }, () => {
     deepStrictEqual(nextData, payload);
   });
 
-  it("Option `includeUnusedVariables`, set to true", async () => {
+  it("Option `includeUnusedVariables`, set to true.", async () => {
     /** @type {unknown} */
     let fetchInput;
 
@@ -395,7 +395,7 @@ describe("Function `createUploadLink`.", { concurrency: true }, () => {
     let nextData;
 
     const query = "query ($a: Boolean) {\n  a(a: $a)\n}";
-    const payload = { data: { a: true, b: true } };
+    const payload = { data: { a: true } };
 
     await timeLimitPromise(
       /** @type {Promise<void>} */ (