feat: remove previews

.github/workflows/test.yml

@@ -13,9 +13,8 @@ jobs:
     strategy:
       matrix:
         node_version:
-          - 14
-          - 16
           - 18
+          - 20
     steps:
       - uses: actions/checkout@v3
       - name: Use Node.js ${{ matrix.node_version }}

README.md

@@ -145,41 +145,6 @@ const octokit = new Octokit({
 });
 ```
 
-</td></tr>
-    <tr>
-      <th>
-        <code>options.previews</code>
-      </th>
-      <td>
-        <code>Array of Strings</code>
-      </td>
-      <td>
-
-Some REST API endpoints require preview headers to be set, or enable
-additional features. Preview headers can be set on a per-request basis, e.g.
-
-```js
-octokit.request("POST /repos/{owner}/{repo}/pulls", {
-  mediaType: {
-    previews: ["shadow-cat"],
-  },
-  owner,
-  repo,
-  title: "My pull request",
-  base: "main",
-  head: "my-feature",
-  draft: true,
-});
-```
-
-You can also set previews globally, by setting the `options.previews` option on the constructor. Example:
-
-```js
-const octokit = new Octokit({
-  previews: ["shadow-cat"],
-});
-```
-
 </td></tr>
     <tr>
       <th>

package.json

@@ -96,6 +96,6 @@
     ]
   },
   "engines": {
-    "node": ">= 14"
+    "node": ">= 18"
   }
 }

scripts/build.mjs

@@ -46,7 +46,7 @@ async function main() {
       outdir: "pkg/dist-node",
       bundle: true,
       platform: "node",
-      target: "node14",
+      target: "node18",
       format: "cjs",
       ...sharedOptions,
     }),

src/index.ts

@@ -80,7 +80,6 @@ export class Octokit {
         hook: hook.bind(null, "request"),
       }),
       mediaType: {
-        previews: [],
         format: "",
       },
     };
@@ -97,10 +96,6 @@ export class Octokit {
       requestDefaults.baseUrl = options.baseUrl;
     }
 
-    if (options.previews) {
-      requestDefaults.mediaType.previews = options.previews;
-    }
-
     if (options.timeZone) {
       requestDefaults.headers["time-zone"] = options.timeZone;
     }

src/types.ts

@@ -11,7 +11,6 @@ export interface OctokitOptions {
   authStrategy?: any;
   auth?: any;
   userAgent?: string;
-  previews?: string[];
   baseUrl?: string;
   log?: {
     debug: (message: string) => unknown;

test/auth.test.ts

@@ -231,7 +231,6 @@ describe("Authentication", () => {
         { id: 123 },
         {
           headers: {
-            accept: "application/vnd.github.machine-man-preview+json",
             "user-agent": userAgent,
             authorization: `bearer ${BEARER}`,
           },
@@ -253,11 +252,7 @@ describe("Authentication", () => {
     await octokit.request("GET /repos/octocat/hello-world");
     await octokit.request("GET /repos/octocat/hello-world");
 
-    await octokit.request("GET /app", {
-      mediaType: {
-        previews: ["machine-man"],
-      },
-    });
+    await octokit.request("GET /app");
 
     expect(mock.done()).toBe(true);
   });

test/constructor.test.ts

@@ -2,32 +2,6 @@ import { Octokit } from "../src";
 import fetchMock from "fetch-mock";
 
 describe("Smoke test", () => {
-  it("previews option", () => {
-    const mock = fetchMock.sandbox().getOnce(
-      "https://api.github.com/",
-      { ok: true },
-      {
-        headers: {
-          accept:
-            "application/vnd.github.jean-grey-preview+json,application/vnd.github.symmetra-preview+json",
-        },
-      }
-    );
-
-    const octokit = new Octokit({
-      previews: [
-        // test with & without -preview suffix
-        "jean-grey-preview",
-        "symmetra",
-      ],
-      request: {
-        fetch: mock,
-      },
-    });
-
-    return octokit.request("/");
-  });
-
   it("timeZone option", () => {
     const mock = fetchMock.sandbox().getOnce(
       "https://api.github.com/",

test/hook.test.ts

@@ -54,7 +54,6 @@ describe("octokit.hook", () => {
           "x-foo": "bar",
         },
         mediaType: {
-          previews: ["octicon"],
           format: "rad",
         },
         bar: "daz",
@@ -77,7 +76,6 @@ describe("octokit.hook", () => {
         "x-foo": "bar",
       },
       mediaType: {
-        previews: ["octicon"],
         format: "rad",
       },
     });
@@ -104,7 +102,6 @@ describe("octokit.hook", () => {
           "user-agent": userAgent,
         },
         mediaType: {
-          previews: [],
           format: "",
         },
         request: {
@@ -145,7 +142,6 @@ describe("octokit.hook", () => {
           "user-agent": userAgent,
         },
         mediaType: {
-          previews: [],
           format: "",
         },
         request: {
@@ -178,7 +174,6 @@ describe("octokit.hook", () => {
           "user-agent": userAgent,
         },
         mediaType: {
-          previews: [],
           format: "",
         },
         request: {

test/request.test.ts

@@ -94,49 +94,6 @@ describe("octokit.request()", () => {
     return octokit.request("GET /");
   });
 
-  it("previews", async () => {
-    const mock = fetchMock
-      .sandbox()
-      .getOnce(
-        "https://api.github.com/",
-        {},
-        {
-          headers: {
-            accept:
-              "application/vnd.github.foo-preview+json,application/vnd.github.bar-preview+json",
-            "user-agent": userAgent,
-          },
-        }
-      )
-      .getOnce(
-        "https://api.github.com/",
-        {},
-        {
-          headers: {
-            accept:
-              "application/vnd.github.foo-preview.raw,application/vnd.github.bar-preview.raw,application/vnd.github.baz-preview.raw",
-            "user-agent": userAgent,
-          },
-          overwriteRoutes: false,
-        }
-      );
-
-    const octokit = new Octokit({
-      previews: ["foo", "bar-preview"],
-      request: {
-        fetch: mock,
-      },
-    });
-
-    await octokit.request("/");
-    await octokit.request("/", {
-      mediaType: {
-        previews: ["bar", "baz-preview"],
-        format: "raw",
-      },
-    });
-  });
-
   it('octokit.request.endpoint("GET /")', () => {
     const octokit = new Octokit();
     const requestOptions = octokit.request.endpoint("GET /");