source/build.ts

@@ -35,10 +35,9 @@ const sourceDir = toAbsolutePath("../source");
 await fsp.mkdir(outDir, {recursive: true});
 
 // Write the WebExtension manifest file.
-const manifest = createManifest(browser);
 await fsp.writeFile(
   path.join(outDir, "manifest.json"),
-  JSON.stringify(manifest),
+  JSON.stringify(createManifest(browser)),
 );
 
 // Write the web-ext configuration file.
@@ -74,7 +73,6 @@ const options: esbuild.BuildOptions = {
     $browser: JSON.stringify(browser),
     $dev: JSON.stringify(dev),
     $test: JSON.stringify(test),
-    $version: JSON.stringify(manifest.version),
   },
   entryPoints: [
     path.join(sourceDir, "background/setup.ts"),

source/content-scripts/features/group-list-subscribe-button.tsx

@@ -1,10 +1,5 @@
 import {Component, render} from "preact";
-import {
-  log,
-  makeIntercoolerRequest,
-  pluralize,
-  querySelectorAll,
-} from "../../utilities/exports.js";
+import {log, pluralize, querySelectorAll} from "../../utilities/exports.js";
 
 export function runGroupListSubscribeButtonFeature(): void {
   const count = addSubscribeButtonsToGroupList();
@@ -19,6 +14,14 @@ function addSubscribeButtonsToGroupList(): number {
     return 0;
   }
 
+  const csrfToken = document.querySelector<HTMLMetaElement>(
+    'meta[name="csrftoken"]',
+  )?.content;
+  if (csrfToken === undefined) {
+    log("No CSRF token found", true);
+    return 0;
+  }
+
   let count = 0;
   for (const listItem of querySelectorAll<HTMLLIElement>(
     ".group-list li:not(.trx-group-list-subscribe-button)",
@@ -31,7 +34,14 @@ function addSubscribeButtonsToGroupList(): number {
     }
 
     const button = document.createDocumentFragment();
-    render(<SubscribeButton group={group} listItem={listItem} />, button);
+    render(
+      <SubscribeButton
+        csrfToken={csrfToken}
+        group={group}
+        listItem={listItem}
+      />,
+      button,
+    );
 
     const activity =
       listItem.querySelector(".group-list-activity") ?? undefined;
@@ -49,6 +59,7 @@ function addSubscribeButtonsToGroupList(): number {
 }
 
 type Props = {
+  csrfToken: string;
   listItem: HTMLLIElement;
   group: string;
 };
@@ -69,15 +80,21 @@ class SubscribeButton extends Component<Props, State> {
   }
 
   clickHandler = async () => {
-    const {group} = this.props;
+    const {csrfToken, group} = this.props;
     const {isSubscribed} = this.state;
 
-    const response = await makeIntercoolerRequest(
+    const response = await window.fetch(
       `https://tildes.net/api/web/group/${group}/subscribe`,
       {
+        headers: {
+          "X-CSRF-Token": csrfToken,
+          "X-IC-Request": "true",
+        },
         method: isSubscribed ? "DELETE" : "PUT",
+        referrer: "https://tildes.net",
       },
     );
+
     if (response.status !== 200) {
       log(`Unexpected status code: ${response.status}`, true);
       return;

source/types.d.ts

@@ -12,5 +12,4 @@ declare global {
   const $browser: "chromium" | "firefox";
   const $dev: boolean;
   const $test: boolean;
-  const $version: string;
 }

source/utilities/exports.ts

@@ -3,7 +3,6 @@ export * from "./components/link.js";
 export * from "./elements.js";
 export * from "./globals.js";
 export * from "./groups.js";
-export * from "./http.js";
 export * from "./logging.js";
 export * from "./query-selectors.js";
 export * from "./report-a-bug.js";

source/utilities/http.ts

@@ -1,51 +0,0 @@
-import {log} from "./logging.js";
-
-/**
- * Make an HTTP request to the Tildes API that is normally used by Intercooler.
- * This should only be used when using HTML elements from Tildes itself isn't
- * feasible.
- * @param url The API URL to call.
- * @param request Any extra request details, note that some of these values will
- * be overridden by the ones required to make a proper Intercooler request.
- */
-export async function makeIntercoolerRequest(
-  url: string,
-  request: RequestInit,
-): Promise<Response> {
-  if (!url.startsWith("https://tildes.net")) {
-    throw new Error(`Can't make Intercooler request to non-Tildes URL: ${url}`);
-  }
-
-  const csrfToken = document.querySelector<HTMLMetaElement>(
-    'meta[name="csrftoken"]',
-  )!.content;
-
-  const ic: RequestInit = {
-    headers: {
-      // Change the user agent and add an extra header so it's clear this isn't
-      // a request actually sent by Intercooler but by Tildes ReExtended.
-      "User-Agent": `Tildes ReExtended (${$browser}, version ${$version})`,
-      "X-CSRF-Token": csrfToken,
-      "X-IC-Request": "true",
-      "X-TRX-Request": "true",
-    },
-    referrer: "https://tildes.net",
-  };
-
-  request.headers =
-    request.headers === undefined
-      ? ic.headers
-      : {
-          ...request.headers,
-          // Apply the Intercooler headers last so they can't be overridden.
-          ...ic.headers,
-        };
-
-  request.referrer = request.referrer ?? ic.referrer;
-
-  // Explicitly log the request because content script HTTP calls don't show up
-  // in the Network DevTools.
-  log("Making Intercooler request:");
-  log(request);
-  return window.fetch(url, request);
-}