Add first GitHub entities and automatically fetch them for the leaderboard

.gitignore

@@ -1,6 +1,9 @@
 .venv
 .DS_Store
 
+# Idea files
+.idea/
+
 # Byte-compiled / optimized / DLL files
 __pycache__/
 *.py[cod]
@@ -162,4 +165,6 @@ cython_debug/
 #  option (not recommended) you can uncomment the following to ignore the entire idea folder.
 #.idea/
 
-node_modules/
+node_modules/
+
+application-local.yml

server/application-server/HELP.md

@@ -1,38 +1,53 @@
 # Getting Started
 
+### Local development
+
+The Spring Boot Maven Plugin supports running your application with a local development environment. To do this, you can use the `spring-boot:run` goal:
+
+```shell
+mvn spring-boot:run
+```
+
+This will start your application in a local development environment. You can access the application at `http://localhost:8080`.
+
+Setting environment variables works through profile-based `application.yml` files. For local development, create a `application-local.yml` file overwriting the original properties. See [Using the Plugin :: Spring Boot](https://docs.spring.io/spring-boot/maven-plugin/using.html#using.overriding-command-line) for more information.
+
 ### Reference Documentation
+
 For further reference, please consider the following sections:
 
-* [Official Apache Maven documentation](https://maven.apache.org/guides/index.html)
-* [Spring Boot Maven Plugin Reference Guide](https://docs.spring.io/spring-boot/docs/3.3.2/maven-plugin/reference/html/)
-* [Create an OCI image](https://docs.spring.io/spring-boot/docs/3.3.2/maven-plugin/reference/html/#build-image)
-* [Docker Compose Support](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#features.docker-compose)
-* [Spring Boot DevTools](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#using.devtools)
-* [Spring Web](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#web)
-* [Spring Security](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#web.security)
-* [Spring Data JPA](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#data.sql.jpa-and-spring-data)
-* [Spring Boot Actuator](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#actuator)
-* [Liquibase Migration](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#howto.data-initialization.migration-tool.liquibase)
-* [OAuth2 Client](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#web.security.oauth2.client)
-* [OAuth2 Resource Server](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#web.security.oauth2.server)
+-   [Official Apache Maven documentation](https://maven.apache.org/guides/index.html)
+-   [Spring Boot Maven Plugin Reference Guide](https://docs.spring.io/spring-boot/docs/3.3.2/maven-plugin/reference/html/)
+-   [Create an OCI image](https://docs.spring.io/spring-boot/docs/3.3.2/maven-plugin/reference/html/#build-image)
+-   [Docker Compose Support](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#features.docker-compose)
+-   [Spring Boot DevTools](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#using.devtools)
+-   [Spring Web](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#web)
+-   [Spring Security](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#web.security)
+-   [Spring Data JPA](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#data.sql.jpa-and-spring-data)
+-   [Spring Boot Actuator](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#actuator)
+-   [Liquibase Migration](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#howto.data-initialization.migration-tool.liquibase)
+-   [OAuth2 Client](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#web.security.oauth2.client)
+-   [OAuth2 Resource Server](https://docs.spring.io/spring-boot/docs/3.3.2/reference/htmlsingle/index.html#web.security.oauth2.server)
 
 ### Guides
+
 The following guides illustrate how to use some features concretely:
 
-* [Building a RESTful Web Service](https://spring.io/guides/gs/rest-service/)
-* [Serving Web Content with Spring MVC](https://spring.io/guides/gs/serving-web-content/)
-* [Building REST services with Spring](https://spring.io/guides/tutorials/rest/)
-* [Securing a Web Application](https://spring.io/guides/gs/securing-web/)
-* [Spring Boot and OAuth2](https://spring.io/guides/tutorials/spring-boot-oauth2/)
-* [Authenticating a User with LDAP](https://spring.io/guides/gs/authenticating-ldap/)
-* [Accessing Data with JPA](https://spring.io/guides/gs/accessing-data-jpa/)
-* [Building a RESTful Web Service with Spring Boot Actuator](https://spring.io/guides/gs/actuator-service/)
+-   [Building a RESTful Web Service](https://spring.io/guides/gs/rest-service/)
+-   [Serving Web Content with Spring MVC](https://spring.io/guides/gs/serving-web-content/)
+-   [Building REST services with Spring](https://spring.io/guides/tutorials/rest/)
+-   [Securing a Web Application](https://spring.io/guides/gs/securing-web/)
+-   [Spring Boot and OAuth2](https://spring.io/guides/tutorials/spring-boot-oauth2/)
+-   [Authenticating a User with LDAP](https://spring.io/guides/gs/authenticating-ldap/)
+-   [Accessing Data with JPA](https://spring.io/guides/gs/accessing-data-jpa/)
+-   [Building a RESTful Web Service with Spring Boot Actuator](https://spring.io/guides/gs/actuator-service/)
 
 ### Docker Compose support
+
 This project contains a Docker Compose file named `compose.yaml`.
 In this file, the following services have been defined:
 
-* postgres: [`postgres:latest`](https://hub.docker.com/_/postgres)
+-   postgres: [`postgres:latest`](https://hub.docker.com/_/postgres)
 
 Please review the tags of the used images and set them to the same as you're running in production.
 
@@ -42,4 +57,3 @@ Due to Maven's design, elements are inherited from the parent POM to the project
 While most of the inheritance is fine, it also inherits unwanted elements like `<license>` and `<developers>` from the parent.
 To prevent this, the project POM contains empty overrides for these elements.
 If you manually switch to a different parent and actually want the inheritance, you need to remove those overrides.
-

server/application-server/pom.xml

@@ -29,6 +29,7 @@
 	</scm>
 	<properties>
 		<java.version>21</java.version>
+		<app.profiles>local,dev</app.profiles>
 	</properties>
 	<dependencies>
 		<dependency>
@@ -120,6 +121,16 @@
 			<artifactId>spring-modulith-starter-test</artifactId>
 			<scope>test</scope>
 		</dependency>
+		<dependency>
+			<groupId>org.kohsuke</groupId>
+			<artifactId>github-api</artifactId>
+			<version>1.324</version>
+		</dependency>
+		<dependency>
+			<groupId>io.hypersistence</groupId>
+			<artifactId>hypersistence-utils-hibernate-63</artifactId>
+			<version>3.8.2</version>
+		</dependency>
 	</dependencies>
 
 	<dependencyManagement>
@@ -158,6 +169,7 @@
 				<groupId>org.springframework.boot</groupId>
 				<artifactId>spring-boot-maven-plugin</artifactId>
 				<configuration>
+					<profiles>${app.profiles}</profiles>
 					<jvmArguments>-Dspring.application.admin.enabled=true</jvmArguments>
 					<excludes>
 						<exclude>

server/application-server/src/main/java/de/tum/in/www1/hephaestus/Application.java

@@ -11,5 +11,4 @@ public class Application {
 	public static void main(String[] args) {
 		SpringApplication.run(Application.class, args);
 	}
-
 }

server/application-server/src/main/java/de/tum/in/www1/hephaestus/ClassImportIntegratorIntegratorProvider.java

@@ -0,0 +1,29 @@
+package de.tum.in.www1.hephaestus;
+
+import java.util.ArrayList;
+import java.util.List;
+
+import org.hibernate.integrator.spi.Integrator;
+import org.hibernate.jpa.boot.spi.IntegratorProvider;
+
+import de.tum.in.www1.hephaestus.codereview.comment.IssueCommentDTO;
+import de.tum.in.www1.hephaestus.codereview.pullrequest.PullRequestDTO;
+import de.tum.in.www1.hephaestus.codereview.repository.RepositoryDTO;
+import de.tum.in.www1.hephaestus.codereview.user.UserDTO;
+import io.hypersistence.utils.hibernate.type.util.ClassImportIntegrator;
+
+public class ClassImportIntegratorIntegratorProvider implements IntegratorProvider {
+
+    @Override
+    public List<Integrator> getIntegrators() {
+        // Accessible DTOs
+        @SuppressWarnings("rawtypes")
+        List<Class> classes = new ArrayList<>();
+        classes.add(UserDTO.class);
+        classes.add(PullRequestDTO.class);
+        classes.add(IssueCommentDTO.class);
+        classes.add(RepositoryDTO.class);
+
+        return List.of(new ClassImportIntegrator(classes));
+    }
+}

server/application-server/src/main/java/de/tum/in/www1/hephaestus/SecurityConfig.java

@@ -1,6 +1,5 @@
 package de.tum.in.www1.hephaestus;
 
-import java.util.Arrays;
 import java.util.List;
 
 import org.springframework.context.annotation.Bean;
@@ -20,14 +19,13 @@ public class SecurityConfig {
     @Bean
     public SecurityFilterChain securityFilterChain(HttpSecurity http) throws Exception {
         http
-		    .csrf(csrf -> csrf.disable())
-            .cors(cors -> cors.configurationSource(corsConfigurationSource()))
-            .authorizeHttpRequests(authorizeRequests -> authorizeRequests
-                .requestMatchers("/admin/**").authenticated()
-                .anyRequest().permitAll()
-            )
-            .formLogin(Customizer.withDefaults())
-            .logout(Customizer.withDefaults());
+                .csrf(csrf -> csrf.disable())
+                .cors(cors -> cors.configurationSource(corsConfigurationSource()))
+                .authorizeHttpRequests(authorizeRequests -> authorizeRequests
+                        .requestMatchers("/admin/**").authenticated()
+                        .anyRequest().permitAll())
+                .formLogin(Customizer.withDefaults())
+                .logout(Customizer.withDefaults());
 
         return http.build();
     }
@@ -38,7 +36,7 @@ CorsConfigurationSource corsConfigurationSource() {
         configuration.applyPermitDefaultValues();
         configuration.setAllowedMethods(List.of("GET", "POST", "PUT", "DELETE", "OPTIONS", "HEAD"));
         UrlBasedCorsConfigurationSource source = new UrlBasedCorsConfigurationSource();
-        source.registerCorsConfiguration("/**",configuration);
+        source.registerCorsConfiguration("/**", configuration);
         return source;
     }
 }

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/base/BaseGitServiceEntity.java

@@ -0,0 +1,26 @@
+package de.tum.in.www1.hephaestus.codereview.base;
+
+import java.time.OffsetDateTime;
+
+import jakarta.persistence.Id;
+import jakarta.persistence.MappedSuperclass;
+import lombok.AllArgsConstructor;
+import lombok.Getter;
+import lombok.NoArgsConstructor;
+import lombok.Setter;
+import lombok.ToString;
+
+@MappedSuperclass
+@Getter
+@Setter
+@NoArgsConstructor
+@AllArgsConstructor
+@ToString
+public abstract class BaseGitServiceEntity {
+    @Id
+    protected Long id;
+
+    protected OffsetDateTime createdAt;
+
+    protected OffsetDateTime updatedAt;
+}

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/base/BaseGitServiceEntityConverter.java

@@ -0,0 +1,45 @@
+package de.tum.in.www1.hephaestus.codereview.base;
+
+import org.kohsuke.github.GHObject;
+import org.springframework.core.convert.converter.Converter;
+import org.springframework.data.convert.ReadingConverter;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import java.io.IOException;
+import java.time.OffsetDateTime;
+import java.time.ZoneOffset;
+import java.util.Date;
+
+@ReadingConverter
+public abstract class BaseGitServiceEntityConverter<S extends GHObject, T extends BaseGitServiceEntity>
+        implements Converter<S, T> {
+
+    private static final Logger logger = LoggerFactory.getLogger(BaseGitServiceEntityConverter.class);
+
+    protected void convertBaseFields(S source, T target) {
+        if (source == null || target == null) {
+            throw new IllegalArgumentException("Source and target must not be null");
+        }
+
+        // Map common fields
+        target.setId(source.getId());
+
+        try {
+            target.setCreatedAt(convertToOffsetDateTime(source.getCreatedAt()));
+        } catch (IOException e) {
+            logger.error("Failed to convert createdAt field for source {}: {}", source.getId(), e.getMessage());
+            target.setCreatedAt(null);
+        }
+
+        try {
+            target.setUpdatedAt(convertToOffsetDateTime(source.getUpdatedAt()));
+        } catch (IOException e) {
+            logger.error("Failed to convert updatedAt field for source {}: {}", source.getId(), e.getMessage());
+            target.setUpdatedAt(null);
+        }
+    }
+
+    private OffsetDateTime convertToOffsetDateTime(Date date) {
+        return date != null ? date.toInstant().atOffset(ZoneOffset.UTC) : null;
+    }
+}

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/comment/IssueComment.java

@@ -0,0 +1,42 @@
+package de.tum.in.www1.hephaestus.codereview.comment;
+
+import jakarta.persistence.Table;
+
+import org.springframework.lang.NonNull;
+
+import de.tum.in.www1.hephaestus.codereview.base.BaseGitServiceEntity;
+import de.tum.in.www1.hephaestus.codereview.pullrequest.PullRequest;
+import de.tum.in.www1.hephaestus.codereview.user.User;
+import jakarta.persistence.Basic;
+import jakarta.persistence.Entity;
+import jakarta.persistence.FetchType;
+import jakarta.persistence.JoinColumn;
+import jakarta.persistence.Lob;
+import jakarta.persistence.ManyToOne;
+import lombok.Getter;
+import lombok.NoArgsConstructor;
+import lombok.Setter;
+import lombok.ToString;
+
+@Entity
+@Table(name = "issue_comment")
+@Getter
+@Setter
+@NoArgsConstructor
+@ToString(callSuper = true)
+public class IssueComment extends BaseGitServiceEntity {
+    @NonNull
+    @Lob
+    @Basic(fetch = FetchType.EAGER)
+    private String body;
+
+    @ManyToOne(fetch = FetchType.LAZY)
+    @JoinColumn(name = "author_id")
+    @ToString.Exclude
+    private User author;
+
+    @ManyToOne(optional = false)
+    @JoinColumn(name = "pullrequest_id", referencedColumnName = "id")
+    @ToString.Exclude
+    private PullRequest pullRequest;
+}

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/comment/IssueCommentConverter.java

@@ -0,0 +1,20 @@
+package de.tum.in.www1.hephaestus.codereview.comment;
+
+import org.kohsuke.github.GHIssueComment;
+import org.springframework.lang.NonNull;
+import org.springframework.stereotype.Service;
+
+import de.tum.in.www1.hephaestus.codereview.base.BaseGitServiceEntityConverter;
+
+@Service
+public class IssueCommentConverter extends BaseGitServiceEntityConverter<GHIssueComment, IssueComment> {
+
+    @Override
+    public IssueComment convert(@NonNull GHIssueComment source) {
+        IssueComment comment = new IssueComment();
+        convertBaseFields(source, comment);
+        comment.setBody(source.getBody());
+        return comment;
+    }
+
+}

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/comment/IssueCommentDTO.java

@@ -0,0 +1,14 @@
+package de.tum.in.www1.hephaestus.codereview.comment;
+
+import com.fasterxml.jackson.annotation.JsonInclude;
+
+import de.tum.in.www1.hephaestus.codereview.pullrequest.PullRequestDTO;
+import de.tum.in.www1.hephaestus.codereview.user.UserDTO;
+
+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public record IssueCommentDTO(Long id, String body, String createdAt, String updatedAt, UserDTO author,
+        PullRequestDTO pullRequest) {
+    public IssueCommentDTO(Long id, String body, String createdAt, String updatedAt) {
+        this(id, body, createdAt, updatedAt, null, null);
+    }
+}

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/comment/IssueCommentRepository.java

@@ -0,0 +1,7 @@
+package de.tum.in.www1.hephaestus.codereview.comment;
+
+import org.springframework.data.jpa.repository.JpaRepository;
+
+public interface IssueCommentRepository extends JpaRepository<IssueComment, Long> {
+
+}

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/pullrequest/IssueState.java

@@ -0,0 +1,5 @@
+package de.tum.in.www1.hephaestus.codereview.pullrequest;
+
+public enum IssueState {
+    CLOSED, OPEN
+}