WIP: Migrate to yarn

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [typescript](https://www.typescriptlang.org/) ([source](https://github.com/Microsoft/TypeScript)) | devDependencies | minor | [`5.4.5` -> `5.5.2`](https://renovatebot.com/diffs/npm/typescript/5.4.5/5.5.2) |

---

### Release Notes

<details>
<summary>Microsoft/TypeScript (typescript)</summary>

### [`v5.5.2`](https://github.com/microsoft/TypeScript/releases/tag/v5.5.2): TypeScript 5.5

[Compare Source](https://github.com/Microsoft/TypeScript/compare/v5.4.5...v5.5.2)

For release notes, check out the [release announcement](https://devblogs.microsoft.com/typescript/announcing-typescript-5-5/).

For the complete list of fixed issues, check out the

-   [fixed issues query for TypeScript v5.5.2 (Stable)](https://github.com/Microsoft/TypeScript/issues?utf8=%E2%9C%93\&q=is%3Aissue+milestone%3A%22TypeScript+5.5.2%22+is%3Aclosed+).
-   [fixed issues query for TypeScript v5.5.1 (RC)](https://github.com/Microsoft/TypeScript/issues?utf8=%E2%9C%93\&q=is%3Aissue+milestone%3A%22TypeScript+5.5.1%22+is%3Aclosed+).
-   [fixed issues query for TypeScript v5.5.0 (Beta)](https://github.com/Microsoft/TypeScript/issues?utf8=%E2%9C%93\&q=is%3Aissue+milestone%3A%22TypeScript+5.5.0%22+is%3Aclosed+).

Downloads are available on:

-   [npm](https://www.npmjs.com/package/typescript)
-   [NuGet package](https://www.nuget.org/packages/Microsoft.TypeScript.MSBuild)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4wLjAiLCJ1cGRhdGVkSW5WZXIiOiIzNy4wLjAiLCJ0YXJnZXRCcmFuY2giOiJkZXZlbG9wIn0=-->

Reviewed-on: #440
Reviewed-by: Vylpes <ethan@vylpes.com>
Co-authored-by: Renovate Bot <renovate@vylpes.com>
Co-committed-by: Renovate Bot <renovate@vylpes.com>

.forgejo/workflows/production.yml

@@ -14,12 +14,12 @@ jobs:
     steps:
     - uses: actions/checkout@v4
     - name: Use Node.js
-      uses: actions/setup-node@v1
+      uses: actions/setup-node@v4
       with:
         node-version: 18.x
-    - run: npm ci
-    - run: npm run build
-    - run: npm test
+    - run: yarn install --frozen-lockfile
+    - run: yarn build
+    - run: yarn test
 
     - name: "Copy files over to location"
       run: cp -r . ${{ secrets.PROD_REPO_PATH }}
@@ -63,5 +63,5 @@ jobs:
           && (pm2 delete vylbot_prod || true) \
           && docker compose up -d \
           && sleep 10 \
-          && npm run db:up \
-          && pm2 start --name vylbot_prod dist/vylbot.js
+          && yarn db:up \
+          && pm2 start --name vylbot_prod dist/vylbot.js

.forgejo/workflows/stage.yml

@@ -14,12 +14,12 @@ jobs:
     steps:
     - uses: actions/checkout@v4
     - name: Use Node.js
-      uses: actions/setup-node@v1
+      uses: actions/setup-node@v4
       with:
         node-version: 18.x
-    - run: npm ci
-    - run: npm run build
-    - run: npm test
+    - run: yarn install --frozen-lockfile
+    - run: yarn build
+    - run: yarn test
 
     - name: "Copy files over to location"
       run: cp -r . ${{ secrets.STAGE_REPO_PATH }}
@@ -63,5 +63,5 @@ jobs:
           && (pm2 delete vylbot_stage || true) \
           && docker compose up -d \
           && sleep 10 \
-          && npm run db:up \
-          && pm2 start --name vylbot_stage dist/vylbot.js
+          && yarn db:up \
+          && pm2 start --name vylbot_stage dist/vylbot.js

.forgejo/workflows/test.yml

@@ -16,9 +16,9 @@ jobs:
     steps:
     - uses: actions/checkout@v4
     - name: Use Node.js
-      uses: actions/setup-node@v1
+      uses: actions/setup-node@v4
       with:
         node-version: 18.x
-    - run: npm ci
-    - run: npm run build
-    - run: npm test
+    - run: yarn install --frozen-lockfile
+    - run: yarn build
+    - run: yarn test 

src/commands/audits.ts

@@ -92,7 +92,7 @@ export default class Audits extends Command {
     private async SendAuditForUser(interaction: CommandInteraction) {
         if (!interaction.guildId) return;
 
-        const user = interaction.options.getUser('target');
+        const user = interaction.options.get('target', true).user!;
 
         if (!user) {
             await interaction.reply("User not found.");
@@ -190,7 +190,7 @@ export default class Audits extends Command {
     private async AddAudit(interaction: CommandInteraction) {
         if (!interaction.guildId) return;
 
-        const user = interaction.options.getUser('target');
+        const user = interaction.options.get('target', true).user!;
         const auditType = interaction.options.get('type');
         const reasonInput = interaction.options.get('reason');
 
@@ -208,4 +208,4 @@ export default class Audits extends Command {
 
         await interaction.reply(`Created new audit with ID \`${audit.AuditId}\``);
     }
-}
+}

tests/commands/audits.test.ts

@@ -130,7 +130,9 @@ describe("user", () => {
             isChatInputCommand: jest.fn().mockReturnValue(true),
             options: {
                 getSubcommand: jest.fn().mockReturnValue("user"),
-                getUser: jest.fn().mockReturnValue(user),
+                get: jest.fn().mockReturnValue({
+                    user: user
+                }),
             },
             guildId: "guildId",
             reply: jest.fn().mockImplementation((options) => {
@@ -150,8 +152,8 @@ describe("user", () => {
         expect(AuditTools.TypeToFriendlyText).toHaveBeenCalledTimes(1);
         expect(AuditTools.TypeToFriendlyText).toHaveBeenCalledWith(AuditType.Warn);
 
-        expect(interaction.options.getUser).toHaveBeenCalledTimes(1);
-        expect(interaction.options.getUser).toHaveBeenCalledWith("target");
+        expect(interaction.options.get).toHaveBeenCalledTimes(1);
+        expect(interaction.options.get).toHaveBeenCalledWith("target", true);
 
         expect(interaction.reply).toHaveBeenCalledTimes(1);
         expect(repliedWith).toBeDefined();
@@ -193,7 +195,9 @@ describe("user", () => {
             isChatInputCommand: jest.fn().mockReturnValue(true),
             options: {
                 getSubcommand: jest.fn().mockReturnValue("user"),
-                getUser: jest.fn().mockReturnValue(null),
+                get: jest.fn().mockReturnValue({
+                    user: null
+                }),
             },
             guildId: "guildId",
             reply: jest.fn(),
@@ -202,7 +206,7 @@ describe("user", () => {
         const audits = new Audits();
         await audits.execute(interaction);
 
-        expect(interaction.options.getUser).toHaveBeenCalledTimes(1);
+        expect(interaction.options.get).toHaveBeenCalledTimes(1);
 
         expect(interaction.reply).toHaveBeenCalledTimes(1);
         expect(interaction.reply).toHaveBeenCalledWith("User not found.");
@@ -213,7 +217,9 @@ describe("user", () => {
             isChatInputCommand: jest.fn().mockReturnValue(true),
             options: {
                 getSubcommand: jest.fn().mockReturnValue("user"),
-                getUser: jest.fn().mockReturnValue({}),
+                get: jest.fn().mockReturnValue({
+                    user: {},
+                }),
             },
             guildId: "guildId",
             reply: jest.fn(),
@@ -226,7 +232,7 @@ describe("user", () => {
 
         expect(Audit.FetchAuditsByUserId).toHaveBeenCalledTimes(1);
 
-        expect(interaction.options.getUser).toHaveBeenCalledTimes(1);
+        expect(interaction.options.get).toHaveBeenCalledTimes(1);
 
         expect(interaction.reply).toHaveBeenCalledTimes(1);
         expect(interaction.reply).toHaveBeenCalledWith("There are no audits for this user.");
@@ -237,7 +243,9 @@ describe("user", () => {
             isChatInputCommand: jest.fn().mockReturnValue(true),
             options: {
                 getSubcommand: jest.fn().mockReturnValue("user"),
-                getUser: jest.fn().mockReturnValue({}),
+                get: jest.fn().mockReturnValue({
+                    user: {},
+                }),
             },
             guildId: "guildId",
             reply: jest.fn(),
@@ -250,7 +258,7 @@ describe("user", () => {
 
         expect(Audit.FetchAuditsByUserId).toHaveBeenCalledTimes(1);
 
-        expect(interaction.options.getUser).toHaveBeenCalledTimes(1);
+        expect(interaction.options.get).toHaveBeenCalledTimes(1);
 
         expect(interaction.reply).toHaveBeenCalledTimes(1);
         expect(interaction.reply).toHaveBeenCalledWith("There are no audits for this user.");
@@ -593,8 +601,9 @@ describe("add", () => {
             isChatInputCommand: jest.fn().mockReturnValue(true),
             options: {
                 getSubcommand: jest.fn().mockReturnValue("add"),
-                getUser: jest.fn().mockReturnValue(user),
-                get: jest.fn().mockReturnValueOnce(type)
+                get: jest.fn()
+                    .mockReturnValueOnce({ user })
+                    .mockReturnValueOnce(type)
                     .mockReturnValue(reason),
             },
             user: {
@@ -637,8 +646,9 @@ describe("add", () => {
             isChatInputCommand: jest.fn().mockReturnValue(true),
             options: {
                 getSubcommand: jest.fn().mockReturnValue("add"),
-                getUser: jest.fn().mockReturnValue(null),
-                get: jest.fn().mockReturnValue({}),
+                get: jest.fn()
+                    .mockReturnValueOnce({ user: null })
+                    .mockReturnValue({}),
             },
             reply: jest.fn(),
         } as unknown as CommandInteraction;
@@ -656,8 +666,9 @@ describe("add", () => {
             isChatInputCommand: jest.fn().mockReturnValue(true),
             options: {
                 getSubcommand: jest.fn().mockReturnValue("add"),
-                getUser: jest.fn().mockReturnValue(null),
-                get: jest.fn().mockReturnValueOnce(null)
+                get: jest.fn()
+                    .mockReturnValueOnce({ user: null })
+                    .mockReturnValueOnce(null)
                     .mockReturnValue({}),
             },
             reply: jest.fn(),
@@ -676,10 +687,11 @@ describe("add", () => {
             isChatInputCommand: jest.fn().mockReturnValue(true),
             options: {
                 getSubcommand: jest.fn().mockReturnValue("add"),
-                getUser: jest.fn().mockReturnValue(null),
-                get: jest.fn().mockReturnValueOnce({
-                    value: undefined
-                })
+                get: jest.fn()
+                    .mockReturnValueOnce({ user: null })
+                    .mockReturnValueOnce({
+                        value: undefined
+                    })
                     .mockReturnValue({}),
             },
             reply: jest.fn(),
@@ -708,8 +720,9 @@ describe("add", () => {
             isChatInputCommand: jest.fn().mockReturnValue(true),
             options: {
                 getSubcommand: jest.fn().mockReturnValue("add"),
-                getUser: jest.fn().mockReturnValue(user),
-                get: jest.fn().mockReturnValueOnce(type)
+                get: jest.fn()
+                    .mockReturnValueOnce({ user })
+                    .mockReturnValueOnce(type)
                     .mockReturnValue(null),
             },
             user: {
@@ -752,8 +765,9 @@ describe("add", () => {
             isChatInputCommand: jest.fn().mockReturnValue(true),
             options: {
                 getSubcommand: jest.fn().mockReturnValue("add"),
-                getUser: jest.fn().mockReturnValue(user),
-                get: jest.fn().mockReturnValueOnce(type)
+                get: jest.fn()
+                    .mockReturnValueOnce({ user })
+                    .mockReturnValueOnce(type)
                     .mockReturnValue(reason),
             },
             user: {
@@ -775,4 +789,4 @@ describe("add", () => {
         expect(savedAudit).toBeDefined();
         expect(savedAudit!.Reason).toBe("");
     });
-});
+});