test pipeline by Peu77 · Pull Request #86 · 42core-team/website

Peu77 · 2025-08-09T14:04:28Z

No description provided.

56 update readme and envexample

…calls

…ecific event

…ionality

…od to EventService

…ontroller

…ler with event creation check

…with checks for locked teams

…error handling in leaveTeam method

…tions

Queue

…hod in TeamService

… in MatchController; update Elo scores for QUEUE phase matches

…s with their details

…ore, buchholz just as second factor

…llers; enhance error handling in canUserCreateEvent function

…p phase

Copilot

Pull Request Overview

This pull request titled "test pipeline" appears to be a comprehensive rewrite of the website codebase. It removes several key components while introducing a new wiki system and restructuring the project architecture.

Removes the entire database layer (Prisma schema, migrations, database initialization)
Removes GitHub API integration and related repository/user management features
Introduces a new versioned wiki system with markdown processing and navigation
Removes all Kubernetes deployment infrastructure and Helm charts
Replaces a gRPC-based k8s-service with a new REST API service using OpenAPI and RabbitMQ

Reviewed Changes

Copilot reviewed 213 out of 227 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
rabbitmq message format.md	Adds documentation for RabbitMQ message format used for game communication
frontend/lib/markdown.ts	Implements comprehensive markdown processing with version support and callouts
frontend/lib/wiki-navigation.ts	Provides navigation utilities for switching between wiki versions
k8s-service/*	Complete rewrite from gRPC to REST API with RabbitMQ integration
frontend/package.json	Updates dependencies and adds wiki cloning script
frontend/layouts/*	Updates navigation and layout components for wiki integration

Copilot · 2025-08-09T14:06:14Z

rabbitmq message format.md

+
+### Result Field Descriptions
+
+- `pattern`: Always `"game_result"` for game completion messages


The documentation states the pattern should be "game_result" but the example shows "game_server" on line 49. This inconsistency should be resolved to avoid confusion.

Copilot · 2025-08-09T14:06:14Z

k8s-service/internal/queue/parse.go

+import (
+	"encoding/json"
+
+	"github.com/42core-team/website_relaunch/k8s-service-gen/internal/kube"


The import path uses k8s-service-gen but the module name in go.mod is github.com/42core-team/website_relaunch/k8s-service-gen. This import should be corrected to match the actual module structure.

Copilot · 2025-08-09T14:06:14Z

k8s-service/internal/kube/config.go

+	"os"
+	"path/filepath"
+
+	"github.com/42core-team/website_relaunch/k8s-service-gen/internal/config"


Another incorrect import path. The import uses k8s-service-gen in the path but should match the actual module structure from go.mod.

Suggested change

"github.com/42core-team/website_relaunch/k8s-service-gen/internal/config"

"github.com/42core-team/website_relaunch/k8s-service/internal/config"

Copilot · 2025-08-09T14:06:15Z

k8s-service/cmd/server/main.go

+	"github.com/42core-team/website_relaunch/k8s-service-gen/internal/api/server"
+	"github.com/42core-team/website_relaunch/k8s-service-gen/internal/config"
+	"github.com/42core-team/website_relaunch/k8s-service-gen/internal/kube"
+	"github.com/42core-team/website_relaunch/k8s-service-gen/internal/queue"


Consistent with other files, this import path contains k8s-service-gen which doesn't match the module name in go.mod. All import paths in this service need to be corrected.

Suggested change

"github.com/42core-team/website_relaunch/k8s-service-gen/internal/queue"

"github.com/42core-team/website_relaunch/k8s-service/internal/api"

"github.com/42core-team/website_relaunch/k8s-service/internal/api/server"

"github.com/42core-team/website_relaunch/k8s-service/internal/config"

"github.com/42core-team/website_relaunch/k8s-service/internal/kube"

"github.com/42core-team/website_relaunch/k8s-service/internal/queue"

Copilot · 2025-08-09T14:06:15Z

k8s-service/kubernetes/github-secret.yaml

+metadata:
+  name: github-secret
+stringData:
+  github-token: "git"


The GitHub token is hardcoded to "git" which appears to be a placeholder value. This should use a proper secret reference or environment variable substitution to avoid exposing credentials in the codebase.

Suggested change

github-token: "git"

# Replace the value below with your actual GitHub token, or inject it securely via your deployment pipeline.

github-token: "${GITHUB_TOKEN}"

Copilot · 2025-08-09T14:06:15Z

k8s-service/cmd/new_game/new_game.go

+	"log"
+	"time"
+
+	"github.com/42core-team/website_relaunch/k8s-service-gen/internal/kube"


Same import path issue as other files. The k8s-service-gen in the import path doesn't match the module structure.

Suggested change

"github.com/42core-team/website_relaunch/k8s-service-gen/internal/kube"

"github.com/42core-team/website_relaunch/k8s-service/internal/kube"

k8s-service/internal/api/server/server.go

PaulicStudios and others added 30 commits June 11, 2025 14:42

Merge pull request #64 from 42core-team/56-update-readme-and-envexample

fbb90b0

56 update readme and envexample

merge dev

50475e6

changed openapi spec

2aded3f

refactoring and started adding kubeconfig

2df609e

finished basic setup with connection to cluster and logger

433af60

added yaml file for game deployment

520963c

refactor: enhance authentication flow and add axios instance for API …

f6509d8

…calls

user create/update request

a5cee6c

add api with user creation and update request

96498ee

send user id as header in each request

894593c

feat: add UserId decorator to extract user ID from request headers

b21bda4

remove userId param and only get it from the session

b023bd7

feat: add isEventAdmin endpoint to check if user is an admin for a sp…

346940c

…ecific event

refactor: simplify isEventAdmin function and update related calls

89c7caf

feat: add TeamService and TeamController to manage team-related funct…

22d15f5

…ionality

feat: integrate TeamModule into EventModule and add getAllEvents meth…

231c51d

…od to EventService

feat: implement CreateEventDto and add createEvent endpoint to EventC…

9dac5f9

…ontroller

fix nextauth crash

de34d4f

feat: update date handling in page component and enhance user control…

4689f9d

…ler with event creation check

fix event creation

39065ea

refactor getMyEventTeam

fbce59f

fix: correct template string syntax in getTeamById function

b224a79

remove event type and add githubOrg and encrypted githubOrgSecret

88e3f8c

impl github api and add lock event endpoint

251e075

feat: add CreateTeamDto and endpoints for team creation and management

a755a14

feat: add functionality to remove users from teams and delete teams, …

c797914

…with checks for locked teams

feat: add endpoint to retrieve team members for an event and improve …

d70ca7a

…error handling in leaveTeam method

refactor send invite

8c673ee

feat: add search functionality for invite users by event and query

8858552

feat: add endpoint to retrieve pending invites for a user by event

3765bd5

Peu77 and others added 23 commits August 8, 2025 15:37

add event queue page and queue state component

5f7ee58

add inQueue property to Team interface and implement queue state func…

d25586a

…tions

add queue management functions to TeamService

fc59407

add QueueState interface and integrate MatchModule into TeamModule

97f7460

add inQueue property to Team entity

8da9439

add queue management endpoints to TeamController

68ac968

add cron job to process queue matches in MatchService

42c9897

refactor MatchModule to use forwardRef for Team and Event modules

0f6e4f6

add QUEUE phase to MatchPhase enum and clean up event service logging

cdd1e28

Merge pull request #84 from 42core-team/queue

4088da9

Queue

fix frontend build error

94b1e42

add score and queueScore properties to Team and tournament models

f0c86ab

add queueScore property to Team model and implement setQueueScore met…

a5b7bba

…hod in TeamService

add getQueueMatches method to MatchService and corresponding endpoint…

ec2c8d8

… in MatchController; update Elo scores for QUEUE phase matches

add queueMatches prop to QueueState component and display past matche…

2e6c584

…s with their details

working game_result sending

aa9047b

Better seeding for tournament bracket and correct team fetching by sc…

cc0de97

…ore, buchholz just as second factor

Proper mirrored seeding

e2dc603

implement UserGuard for user authentication in event and match contro…

1f8dcde

…llers; enhance error handling in canUserCreateEvent function

only display tournament and group phase if event state is set to grou…

dedc271

…p phase

Merge pull request #85 from 42core-team/better_elimination_seeding

3b66ff6

unregister game queue

a686aa5

Merge remote-tracking branch 'origin/dev' into dev

b1ae61c

Peu77 requested a review from jojomo96 August 9, 2025 14:04

Peu77 temporarily deployed to dev August 9, 2025 14:04 — with GitHub Actions Inactive

jojomo96 requested a review from Copilot August 9, 2025 14:04

Copilot AI reviewed Aug 9, 2025

View reviewed changes

jojomo96 approved these changes Aug 9, 2025

View reviewed changes

jojomo96 merged commit 3ec994b into main Aug 9, 2025
8 checks passed

Peu77 mentioned this pull request Aug 9, 2025

Revert "test pipeline" #87

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

test pipeline#86

test pipeline#86
jojomo96 merged 262 commits intomainfrom
dev

Peu77 commented Aug 9, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Aug 9, 2025

Uh oh!

Copilot AI Aug 9, 2025

Uh oh!

Copilot AI Aug 9, 2025

Uh oh!

Copilot AI Aug 9, 2025

Uh oh!

Copilot AI Aug 9, 2025

Uh oh!

Copilot AI Aug 9, 2025

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants


		### Result Field Descriptions

		- `pattern`: Always `"game_result"` for game completion messages

	"github.com/42core-team/website_relaunch/k8s-service-gen/internal/config"
	"github.com/42core-team/website_relaunch/k8s-service/internal/config"

-	"github.com/42core-team/website_relaunch/k8s-service-gen/internal/queue"
+	"github.com/42core-team/website_relaunch/k8s-service/internal/api"
+	"github.com/42core-team/website_relaunch/k8s-service/internal/api/server"
+	"github.com/42core-team/website_relaunch/k8s-service/internal/config"
+	"github.com/42core-team/website_relaunch/k8s-service/internal/kube"
+	"github.com/42core-team/website_relaunch/k8s-service/internal/queue"

	github-token: "git"
	# Replace the value below with your actual GitHub token, or inject it securely via your deployment pipeline.
	github-token: "${GITHUB_TOKEN}"

Conversation

Peu77 commented Aug 9, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Aug 9, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 9, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 9, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 9, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 9, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Aug 9, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants