Scheduler service #977

Fiery-132 · 2025-11-12T21:15:23Z

Implements #928

Draft for now - not done. Will update with more info later.

Copilot

Pull request overview

This PR implements a scheduler service to enable scheduling of news articles for future publishing. It introduces a new Go-based microservice that handles task scheduling and execution, integrates it with the existing SvelteKit application, and updates the authentication flow to support token refresh.

Key Changes

New Go scheduler microservice with REST API for scheduling and executing tasks
Enhanced JWT authentication flow with refresh token support
Article creation form extended with publish date scheduling
Frontend display of scheduled articles for authorized users

Reviewed changes

Copilot reviewed 20 out of 21 changed files in this pull request and generated 32 comments.

Show a summary per file

File	Description
scheduler-service/main.go	Entry point for scheduler service, handles initialization and startup
scheduler-service/taskHandler.go	Task execution logic including scheduling and HTTP callbacks
scheduler-service/requestHandler.go	API endpoints for creating and querying scheduled tasks
scheduler-service/authMiddleware.go	JWT authentication middleware with JWKS caching
scheduler-service/rateLimitMiddleware.go	IP-based rate limiting for API requests
scheduler-service/databaseHandler.go	PostgreSQL database connection setup
scheduler-service/go.mod	Go module dependencies (contains invalid Go version)
scheduler-service/.env	Environment configuration (should not be committed)
src/lib/news/schema.ts	Added publishTime field to article schema
src/lib/news/server/actions.ts	Updated article creation to support scheduling via external service
src/lib/news/server/notifications.ts	Extracted notification logic for reusability
src/routes/(app)/api/schedule/news/+server.ts	Callback endpoint for scheduler to create articles
src/routes/(app)/news/ArticleForm.svelte	Added date input for scheduling articles
src/routes/(app)/news/+page.svelte	Display list of scheduled articles
src/routes/(app)/news/+page.server.ts	Fetch scheduled tasks from scheduler service
src/lib/server/getDecryptedJWT.ts	Utility to decrypt JWT from request cookies
src/hooks.server.ts	Enhanced JWT callback with refresh token support
src/translations/en.json	English translations for scheduling features
src/translations/sv.json	Swedish translations for scheduling features
.env	Added scheduler endpoint and password configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/lib/news/server/actions.ts

scheduler-service/taskHandler.go

.env

src/lib/news/server/actions.ts

scheduler-service/taskHandler.go

Copilot · 2025-11-25T15:40:24Z

src/routes/(app)/news/+page.server.ts

+type ScheduledTaskRaw = {
+  ID: string;
+  RunTimestamp: string;
+  Body: string;
+};
+
+type ScheduledTaskParsed = {
+  ID: string;
+  RunTimestamp: string;
+  Body: NewsArticleData;
+};
+
+type NewsArticleData = {
+  author: {
+    connect:
+      | {
+          id: string;
+        }
+      | undefined;
+    create:
+      | {
+          member: {
+            connect: {
+              studentId: string | undefined;
+            };
+          };
+          mandate:
+            | {
+                connect: {
+                  member: {
+                    studentId: string | undefined;
+                  };
+                  id: string;
+                };
+              }
+            | undefined;
+          customAuthor:
+            | {
+                connect: {
+                  id: string;
+                };
+              }
+            | undefined;
+        }
+      | undefined;
+  };
+  tags: {
+    connect: Array<{
+      id: string;
+    }>;
+  };
+  publishedAt: Date;
+  imageUrl?: string | null | undefined;
+  imageUrls?: string[] | undefined;
+  youtubeUrl?: string | null | undefined;
+  slug: string;
+  headerSv: string;
+  headerEn: string | null;
+  bodySv: string;
+  bodyEn: string | null;
+};


The NewsArticleData type is duplicated and defined inline. This complex type definition (lines 77-125) should be extracted to a shared types file or imported from the schema definition to maintain a single source of truth and avoid type drift between the scheduler request and the article creation.

Copilot · 2025-11-25T15:40:25Z

src/lib/server/getDecryptedJWT.ts

@@ -0,0 +1,6 @@
+import { env } from "$env/dynamic/private";
+import { getToken, type JWT } from "@auth/core/jwt";
+


This utility function lacks documentation explaining its purpose, when it should be used, and what it returns. Add a JSDoc comment describing that it decrypts and returns the JWT from the request cookie, and explain when it might return null (e.g., user not authenticated).

Suggested change

/**

* Decrypts and returns the JWT from the request cookie.

*

* Use this function when you need to access the authenticated user's JWT from a request.

* Returns the JWT object if the user is authenticated, or `null` if the user is not authenticated

* or if the JWT is missing or invalid.

*

* @param req - The incoming HTTP request containing the JWT cookie.

* @returns The decrypted JWT object, or `null` if not authenticated.

*/

Copilot · 2025-11-25T15:40:25Z

src/routes/(app)/news/ArticleForm.svelte

+  <FormDateInput
+    {superform}
+    field="publishTime"
+    label="Schemalägg publicering"


Hard-coded Swedish text "Schemalägg publicering" should use the translation system instead. This should be replaced with a translation key like {m.news_schedulePublishing()} to support multiple languages consistently with the rest of the application.

Suggested change

label="Schemalägg publicering"

label={m.news_schedulePublishing()}

scheduler-service/databaseHandler.go

src/routes/(app)/api/schedule/news/+server.ts

- Added `publishTime` field to article schema and form for scheduling article publication - Implemented backend logic to send data to the scheduler service for later publishing - Created API endpoint `/api/schedule/news` to handle scheduled article creation and notifications

github-actions · 2026-01-17T03:06:46Z

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 14 days.

github-project-automation bot added this to CPU Nov 12, 2025

github-project-automation bot moved this to 🆕 New in CPU Nov 12, 2025

github-actions bot assigned Fiery-132 Nov 12, 2025

Fiery-132 moved this from 🆕 New to 🏗 In Progress in CPU Nov 12, 2025

Fiery-132 linked an issue Nov 12, 2025 that may be closed by this pull request

Create a scheduling service #928

Open

Fiery-132 force-pushed the scheduler branch 3 times, most recently from 0da74a4 to 007905d Compare November 15, 2025 00:54

Isak-Kallini moved this from 🏗 In Progress to 🆕 New in CPU Nov 24, 2025

Fiery-132 linked an issue Nov 24, 2025 that may be closed by this pull request

Implement handling of refresh tokens #842

Closed

Fiery-132 added 16 commits November 25, 2025 16:12

initial scheduling service

feeccd3

rename executed field to has_executed

e139a3c

added password auth to scheduler

776e159

don't expose password in scheduler service logs

4a49c9b

add rate limiting to scheduler service

f827c46

add proof of concept for user auth via jwt in scheduler service

85c90b8

also store id_token in session token for later decryption

a649bff

move code out of main.go file in scheduler service

cd67184

read port from .env when starting scheduler service

a91cab9

remove global context variable in scheduler

a9fce91

add function for retrieving decrypted jwt

79ed9d0

store user identifier with scheduled tasks

da6bc38

add GET endpoint to scheduler for retrieving tasks per user

34891b6

use query params for scheduler GET auth

d46ddeb

remove hardcoded values from scheduler auth

bcc48f7

improve JWT validation and subject handling in scheduler

60adb77

Fiery-132 force-pushed the scheduler branch 2 times, most recently from b9ba9ff to 3020a4f Compare November 25, 2025 15:25

Fiery-132 requested a review from Copilot November 25, 2025 15:29

Copilot started reviewing on behalf of Fiery-132 November 25, 2025 15:30 View session

Copilot finished reviewing on behalf of Fiery-132 November 25, 2025 15:35

Copilot AI reviewed Nov 25, 2025

View reviewed changes

Fiery-132 added 3 commits December 1, 2025 23:10

add refresh token logic to website auth

6cb187c

fix uninitialised global vars in scheduler auth

fe24c4a

Fiery-132 force-pushed the scheduler branch from a6cfad4 to 741fc33 Compare December 1, 2025 22:11

Fiery-132 added 12 commits December 2, 2025 00:22

fix jwt subject logic in scheduler

aa13dc0

add scheduled news view to news page

276dd7a

only fetch non-executed tasks on GET requests

b7fec5c

improve error handling for fetching scheduled news

fbc3bbf

prevent marking failed scheduled tasks as executed

23a29c2

redirect to news page on succesful scheduling

d60f40d

extract news scheduling logic to generic function

e20a8f4

add cleanup logic to scheduler rate limiters

1e49a53

add error handling when fetching user's scheduled news

b08acb8

extend JWT and Session types

0ad97b0

remove unused public env var

eba5a5a

remove fatal log when reading env in scheduler

e58f57f

Fiery-132 force-pushed the scheduler branch from 741fc33 to e58f57f Compare December 1, 2025 23:22

Fiery-132 force-pushed the scheduler branch 2 times, most recently from 57667bf to 8e4fe3d Compare December 17, 2025 09:42

log out user if tokens are invalid

9c56b77

Fiery-132 force-pushed the scheduler branch from 8e4fe3d to 9c56b77 Compare December 17, 2025 09:42

github-actions bot added the stale Has not been updated for 30 days label Jan 17, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Scheduler service #977

Scheduler service #977

Uh oh!

Fiery-132 commented Nov 12, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Nov 25, 2025

Uh oh!

Copilot AI Nov 25, 2025

Uh oh!

Copilot AI Nov 25, 2025

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Jan 17, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

		@@ -0,0 +1,6 @@
		import { env } from "$env/dynamic/private";
		import { getToken, type JWT } from "@auth/core/jwt";

+/**
+ * Decrypts and returns the JWT from the request cookie.
+ *
+ * Use this function when you need to access the authenticated user's JWT from a request.
+ * Returns the JWT object if the user is authenticated, or `null` if the user is not authenticated
+ * or if the JWT is missing or invalid.
+ *
+ * @param req - The incoming HTTP request containing the JWT cookie.
+ * @returns The decrypted JWT object, or `null` if not authenticated.
+ */

	label="Schemalägg publicering"
	label={m.news_schedulePublishing()}

Scheduler service #977

Are you sure you want to change the base?

Scheduler service #977

Uh oh!

Conversation

Fiery-132 commented Nov 12, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Key Changes

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Jan 17, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant