chore: sync with getopenscreen/openscreen upstream (v1.6.0)

.github/CODEOWNERS

@@ -1 +1 @@
-* @siddharthvaddem
+* @EtienneLescot

.github/actions/setup/action.yml

@@ -0,0 +1,11 @@
+name: Setup Node.js
+description: Install Node 22, restore npm cache, run npm ci
+runs:
+  using: composite
+  steps:
+    - uses: actions/setup-node@v4
+      with:
+        node-version: 22
+        cache: npm
+    - run: npm ci
+      shell: bash

.github/pull_request_template.md

@@ -1,43 +1,36 @@
-# Pull Request Template
-
-## Description
-<!-- Briefly describe the purpose of this PR. -->
-
-## Motivation
-<!-- Explain why this change is needed. What problem does it solve? -->
-
-## Type of Change
-- [ ] New Feature
-- [ ] Bug Fix
-- [ ] Refactor / Code Cleanup
-- [ ] Documentation Update
-- [ ] Other (please specify)
-
-## Related Issue(s)
-<!-- Link to any related issue(s) (e.g., #123) -->
-
-## Screenshots / Video
-<!-- Include screenshots or a short video demonstrating the change. If the change adds a new UI feature, attach an image. If it adds functionality best shown via video, embed a video. -->
-
-**Screenshot** (if applicable):
-
-```markdown
-![Screenshot Description](path/to/screenshot.png)
-```
-
-**Video** (if applicable):
-
-```html
-<video src="path/to/video.mp4" controls width="600"></video>
-```
+## Summary
+<!-- Briefly describe the change. -->
+
+## Related issue
+<!-- Use "Fixes #123", "Closes #123", or "Resolves #123" only when this PR fully resolves the issue. -->
+<!-- Use "Refs #123", "Related to #123", or "Part of #123" when this PR is partial and should not close the issue. -->
+
+Fixes #
+
+## Type of change
+- [ ] Bug fix
+- [ ] Feature
+- [ ] Enhancement
+- [ ] Documentation
+- [ ] Refactor / maintenance
+- [ ] Performance
+- [ ] Security
+
+## Release impact
+- [ ] Patch
+- [ ] Minor
+- [ ] Major / breaking change
+- [ ] No release note needed
+
+## Desktop impact
+- [ ] Windows
+- [ ] macOS
+- [ ] Linux
+- [ ] Installer / packaging
+- [ ] Not platform-specific
+
+## Screenshots / video
+<!-- Include screenshots or a short video for UI or visual changes. -->
 
 ## Testing
-<!-- Describe how reviewers can test the changes. Include steps, commands, or environment setup. -->
-
-## Checklist
-- [ ] I have performed a self-review of my code.
-- [ ] I have added any necessary screenshots or videos.
-- [ ] I have linked related issue(s) and updated the changelog if applicable.
-
----
-*Thank you for contributing!*
+<!-- Describe how this was tested. Include commands, steps, or environment setup. -->

.github/scripts/discord-bot-api.mjs

@@ -0,0 +1,51 @@
+import { warning } from "@actions/core";
+
+const API_BASE = "https://discord.com/api/v10";
+const DEFAULT_TIMEOUT_MS = 5_000;
+
+async function callDiscord(botToken, method, path, body, { timeoutMs = DEFAULT_TIMEOUT_MS } = {}) {
+	const controller = new AbortController();
+	const timeout = setTimeout(() => controller.abort(), timeoutMs);
+	let res;
+	try {
+		res = await fetch(`${API_BASE}${path}`, {
+			method,
+			headers: {
+				Authorization: `Bot ${botToken}`,
+				"Content-Type": "application/json",
+			},
+			body: body !== undefined ? JSON.stringify(body) : undefined,
+			signal: controller.signal,
+		});
+	} finally {
+		clearTimeout(timeout);
+	}
+
+	if (res.status === 429) {
+		const txt = await res.text();
+		warning(`Discord rate-limited (429) on ${method} ${path}: ${txt}`);
+		throw new Error(`Discord rate-limited (429) on ${method} ${path}`);
+	}
+
+	if (!res.ok) {
+		const txt = await res.text();
+		throw new Error(`Discord API ${method} ${path} failed ${res.status}: ${txt}`);
+	}
+
+	if (res.status === 204) return null;
+	return res.json();
+}
+
+export async function createForumThread({ botToken, forumChannelId, payload, timeoutMs }) {
+	return callDiscord(botToken, "POST", `/channels/${forumChannelId}/threads`, payload, {
+		timeoutMs,
+	});
+}
+
+export async function postChannelMessage({ botToken, channelId, payload, timeoutMs }) {
+	return callDiscord(botToken, "POST", `/channels/${channelId}/messages`, payload, { timeoutMs });
+}
+
+export async function patchChannel({ botToken, channelId, payload, timeoutMs }) {
+	return callDiscord(botToken, "PATCH", `/channels/${channelId}`, payload, { timeoutMs });
+}

.github/scripts/discord-bot-api.test.mjs

@@ -0,0 +1,95 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { createForumThread, patchChannel, postChannelMessage } from "./discord-bot-api.mjs";
+
+const botToken = "test-token";
+
+beforeEach(() => {
+	vi.stubGlobal("fetch", vi.fn());
+});
+
+afterEach(() => {
+	vi.unstubAllGlobals();
+});
+
+function mockResponse({ status = 200, body = { id: "x" } } = {}) {
+	vi.mocked(fetch).mockResolvedValue({
+		ok: status >= 200 && status < 300,
+		status,
+		text: vi.fn().mockResolvedValue(JSON.stringify(body)),
+		json: vi.fn().mockResolvedValue(body),
+	});
+}
+
+const happyCases = [
+	{
+		name: "createForumThread",
+		call: (args) => createForumThread(args),
+		args: { forumChannelId: "forum-1", payload: { name: "PR #1" } },
+		expectUrl: "https://discord.com/api/v10/channels/forum-1/threads",
+		expectMethod: "POST",
+		expectBody: { name: "PR #1" },
+	},
+	{
+		name: "postChannelMessage",
+		call: (args) => postChannelMessage(args),
+		args: { channelId: "thread-1", payload: { content: "hello" } },
+		expectUrl: "https://discord.com/api/v10/channels/thread-1/messages",
+		expectMethod: "POST",
+		expectBody: { content: "hello" },
+	},
+	{
+		name: "patchChannel",
+		call: (args) => patchChannel(args),
+		args: { channelId: "thread-1", payload: { archived: true } },
+		expectUrl: "https://discord.com/api/v10/channels/thread-1",
+		expectMethod: "PATCH",
+		expectBody: { archived: true },
+	},
+];
+
+describe.each(happyCases)("$name", ({ call, args, expectUrl, expectMethod, expectBody }) => {
+	it("calls Discord with the right URL, method, bot auth, and payload", async () => {
+		mockResponse();
+
+		await call({ ...args, botToken });
+
+		const [url, init] = vi.mocked(fetch).mock.calls[0];
+		expect(url).toBe(expectUrl);
+		expect(init.method).toBe(expectMethod);
+		expect(init.headers.Authorization).toBe(`Bot ${botToken}`);
+		expect(JSON.parse(init.body)).toEqual(expectBody);
+		expect(init.signal).toBeDefined();
+	});
+
+	it("throws on 429 with rate-limit message", async () => {
+		mockResponse({ status: 429, body: { retry_after: 1 } });
+		await expect(call({ ...args, botToken })).rejects.toThrow(/rate-limited \(429\)/);
+	});
+
+	it("throws on non-ok responses with status in message", async () => {
+		mockResponse({ status: 403, body: { message: "Missing Permissions" } });
+		await expect(call({ ...args, botToken })).rejects.toThrow(/failed 403/);
+	});
+});
+
+describe("callDiscord timeout", () => {
+	it("aborts the request after timeoutMs when the fetch hangs", async () => {
+		// fetch that never resolves until aborted
+		vi.mocked(fetch).mockImplementation(
+			(_url, init) =>
+				new Promise((_resolve, reject) => {
+					init.signal?.addEventListener("abort", () => {
+						reject(new DOMException("aborted", "AbortError"));
+					});
+				}),
+		);
+		await expect(
+			postChannelMessage({
+				botToken,
+				channelId: "x",
+				payload: {},
+				timeoutMs: 10,
+			}),
+		).rejects.toThrow();
+	});
+});