Simplify polyfill setup + prepare for temporal release (#585)

* Move temporal polyfill to be global

* Remove temporal imports

* Fix types of polyfill

* Use tsgo language service

* Add polyfills to entrypoint

* Remove types that are now shipped in ts

Author: holzmaster

.vscode/settings.json

@@ -7,8 +7,7 @@
         "*.ts": "$(capture).test.ts,$(capture).d.ts"
     },
     "oxc.typeAware": true,
-    "js/ts.experimental.useTsgo": false,
-    "js/ts.tsdk.path": "node_modules/typescript/lib",
+    "js/ts.experimental.useTsgo": true,
 
     "cSpell.language": "de,en",
     "editor.formatOnSaveMode": "modificationsIfAvailable",

Dockerfile

@@ -40,4 +40,4 @@ FROM node:alpine
     RUN echo "RELEASE_IDENTIFIER=${RELEASE_IDENTIFIER:-debug}" >> /app/.env && \
         echo "BUILD_NUMBER=${BUILD_NUMBER:-0}" >> /app/.env
 
-    ENTRYPOINT ["node", "--env-file", "/app/.env", "src/app.ts"]
+    ENTRYPOINT ["node", "--env-file", "/app/.env", "--require", "./src/polyfills.ts", "src/app.ts"]

package-lock.json

@@ -61,8 +61,7 @@
     "lefthook": "^2.1.3",
     "oxfmt": "^0.36.0",
     "oxlint": "^1.51.0",
-    "oxlint-tsgolint": "^0.16.0",
-    "typescript": "^6.0.1-rc"
+    "oxlint-tsgolint": "^0.16.0"
   },
   "engines": {
     "node": ">=25.6.1"

package.json

@@ -4,9 +4,6 @@ import * as sentry from "@sentry/node";
 import { readConfig, databasePath, args } from "#/service/config.ts";
 import log from "#log";
 
-import { Temporal } from "@js-temporal/polyfill";
-import "#/polyfills.ts";
-
 import * as kysely from "#/storage/db/db.ts";
 
 import type { ReactionHandler } from "#/handler/ReactionHandler.ts";
@@ -58,16 +55,6 @@ log.info(`Bot starting up...${release ? ` (release: ${release})` : ""}`);
 
 const config = await readConfig();
 
-// This polyfill requires that Temporal is already available in the runtime
-// We cannot add it in the polyfills.ts because that file is also used as --require argument for ts-node
-// TODO: Remove this once temporal is available in Node.js, see: https://github.com/nodejs/node/issues/57127
-if (typeof Date.prototype.toTemporalInstant !== "function") {
-    // biome-ignore lint/suspicious/noExplicitAny: hack
-    (Date.prototype as any).toTemporalInstant = function () {
-        return Temporal.Instant.fromEpochMilliseconds(this.getTime());
-    };
-}
-
 if (config.sentry?.dsn) {
     sentry.init({
         dsn: config.sentry.dsn,

src/app.ts

@@ -16,7 +16,6 @@ import type { ApplicationCommand } from "#/commands/command.ts";
 import type { BotContext } from "#/context.ts";
 import assertNever from "#/utils/assertNever.ts";
 import { getPlaybackStats, setUserRegistration, type TrackStat } from "#/service/lauscher.ts";
-import { Temporal } from "@js-temporal/polyfill";
 import { truncateToLength } from "#/utils/stringUtils.ts";
 import { chunkArray } from "#/utils/arrayUtils.ts";
 import { Vec2 } from "#/utils/math.ts";

src/commands/lauscher.ts

@@ -1,7 +1,6 @@
 import { parseArgs, type ParseArgsConfig } from "node:util";
 
 import { cleanContent } from "discord.js";
-import { Temporal } from "@js-temporal/polyfill";
 
 import type { MessageCommand } from "#/commands/command.ts";
 import type { BotContext } from "#/context.ts";

src/commands/vote.ts

@@ -1,5 +1,3 @@
-import { Temporal } from "@js-temporal/polyfill"; // TODO: Remove once Node.js ships temporal
-
 import {
     type CommandInteraction,
     type Message,

src/commands/woisvote.ts

@@ -15,7 +15,6 @@ import type {
 } from "discord.js";
 import { ChannelType } from "discord.js";
 import { SpotifyApi } from "@spotify/web-api-ts-sdk";
-import { Temporal } from "@js-temporal/polyfill";
 
 import type { UserMapEntry } from "#/commands/aoc.ts";
 import { readConfig } from "#/service/config.ts";

src/context.ts

@@ -9,7 +9,6 @@ import {
     type GuildTextBasedChannel,
     hyperlink,
 } from "discord.js";
-import { Temporal } from "@js-temporal/polyfill";
 
 import type { BotContext, QuoteConfig } from "#/context.ts";
 import type { ReactionHandler } from "../ReactionHandler.ts";