style: スタイルを修正

detekt.yml

@@ -32,6 +32,15 @@ style:
   ForbiddenComment:
     active: false
 
+  ThrowsCount:
+    active: false
+
+  UseCheckOrError:
+    active: false
+
+  UseRequire:
+    active: false
+
 complexity:
   CognitiveComplexMethod:
     active: true

src/main/kotlin/dev/usbharu/hideout/Application.kt

@@ -100,12 +100,12 @@ fun Application.parent() {
         inject<JwkProvider>().value,
     )
     configureRouting(
-        inject<HttpSignatureVerifyService>().value,
+        httpSignatureVerifyService = inject<HttpSignatureVerifyService>().value,
-        inject<ActivityPubService>().value,
+        activityPubService = inject<ActivityPubService>().value,
-        inject<IUserService>().value,
+        userService = inject<IUserService>().value,
-        inject<ActivityPubUserService>().value,
+        activityPubUserService = inject<ActivityPubUserService>().value,
-        inject<IPostService>().value,
+        postService = inject<IPostService>().value,
-        inject<IUserApiService>().value,
+        userApiService = inject<IUserApiService>().value,
     )
 }
 

src/main/kotlin/dev/usbharu/hideout/domain/model/ap/Key.kt

@@ -36,7 +36,5 @@ open class Key : Object {
         return result
     }
 
-    override fun toString(): String {
+    override fun toString(): String = "Key(owner=$owner, publicKeyPem=$publicKeyPem) ${super.toString()}"
-        return "Key(owner=$owner, publicKeyPem=$publicKeyPem) ${super.toString()}"
-    }
 }

src/main/kotlin/dev/usbharu/hideout/domain/model/ap/Object.kt

@@ -40,9 +40,7 @@ open class Object : JsonLd {
         return result
     }
 
-    override fun toString(): String {
+    override fun toString(): String = "Object(type=$type, name=$name, actor=$actor, id=$id) ${super.toString()}"
-        return "Object(type=$type, name=$name, actor=$actor, id=$id) ${super.toString()}"
-    }
 
     companion object {
         @JvmStatic

src/main/kotlin/dev/usbharu/hideout/domain/model/hideout/dto/UserResponse.kt

@@ -14,13 +14,13 @@ data class UserResponse(
     companion object {
         fun from(user: User): UserResponse {
             return UserResponse(
-                user.id,
+                id = user.id,
-                user.name,
+                name = user.name,
-                user.domain,
+                domain = user.domain,
-                user.screenName,
+                screenName = user.screenName,
-                user.description,
+                description = user.description,
-                user.url,
+                url = user.url,
-                user.createdAt.toEpochMilli()
+                createdAt = user.createdAt.toEpochMilli()
             )
         }
     }

src/main/kotlin/dev/usbharu/hideout/plugins/Routing.kt

@@ -17,6 +17,7 @@ import io.ktor.server.application.*
 import io.ktor.server.plugins.autohead.*
 import io.ktor.server.routing.*
 
+@Suppress("LongParameterList")
 fun Application.configureRouting(
     httpSignatureVerifyService: HttpSignatureVerifyService,
     activityPubService: ActivityPubService,

src/main/kotlin/dev/usbharu/hideout/plugins/Security.kt

@@ -76,8 +76,8 @@ fun Application.configureSecurity(
         }
         authenticate(TOKEN_AUTH) {
             get("/auth-check") {
-                val principal = call.principal<JWTPrincipal>()
+                val principal = call.principal<JWTPrincipal>() ?: throw IllegalStateException("no principal")
-                val username = principal!!.payload.getClaim("uid")
+                val username = principal.payload.getClaim("uid")
                 call.respondText("Hello $username")
             }
         }

src/main/kotlin/dev/usbharu/hideout/routing/activitypub/UserRouting.kt

@@ -26,7 +26,9 @@ fun Routing.usersAP(activityPubUserService: ActivityPubUserService, userService:
             )
         }
         get {
-            val userEntity = userService.findByNameLocalUser(call.parameters["name"]!!)
+            val userEntity = userService.findByNameLocalUser(
+                call.parameters["name"] ?: throw ParameterNotExistException("Parameter(name='name') does not exist.")
+            )
             call.respondText(userEntity.toString() + "\n" + userService.findFollowersById(userEntity.id))
         }
     }

src/main/kotlin/dev/usbharu/hideout/routing/api/internal/v1/Posts.kt

@@ -17,21 +17,22 @@ import io.ktor.server.request.*
 import io.ktor.server.response.*
 import io.ktor.server.routing.*
 
+@Suppress("LongMethod")
 fun Route.posts(postService: IPostService) {
     route("/posts") {
         authenticate(TOKEN_AUTH) {
             post {
-                val principal = call.principal<JWTPrincipal>() ?: throw RuntimeException("no principal")
+                val principal = call.principal<JWTPrincipal>() ?: throw IllegalStateException("no principal")
                 val userId = principal.payload.getClaim("uid").asLong()
 
                 val receive = call.receive<Post>()
                 val postCreateDto = PostCreateDto(
-                    receive.text,
+                    text = receive.text,
-                    receive.overview,
+                    overview = receive.overview,
-                    receive.visibility,
+                    visibility = receive.visibility,
-                    receive.repostId,
+                    repostId = receive.repostId,
-                    receive.replyId,
+                    repolyId = receive.replyId,
-                    userId
+                    userId = userId
                 )
                 val create = postService.create(postCreateDto)
                 call.response.header("Location", create.url)

src/main/kotlin/dev/usbharu/hideout/routing/api/internal/v1/Users.kt

@@ -16,6 +16,7 @@ import io.ktor.server.request.*
 import io.ktor.server.response.*
 import io.ktor.server.routing.*
 
+@Suppress("LongMethod")
 fun Route.users(userService: IUserService, userApiService: IUserApiService) {
     route("/users") {
         get {

src/main/kotlin/dev/usbharu/hideout/routing/api/mastodon/v1/Statuses.kt

@@ -3,6 +3,7 @@ package dev.usbharu.hideout.routing.api.mastodon.v1
 import dev.usbharu.hideout.service.IPostService
 import io.ktor.server.routing.*
 
+@Suppress("UnusedPrivateMember")
 fun Route.statuses(postService: IPostService) {
 //    route("/statuses") {
 //        post {

src/main/kotlin/dev/usbharu/hideout/service/IPostService.kt

@@ -5,6 +5,7 @@ import dev.usbharu.hideout.domain.model.hideout.dto.PostCreateDto
 import dev.usbharu.hideout.domain.model.hideout.entity.Post
 import java.time.Instant
 
+@Suppress("LongParameterList")
 interface IPostService {
     suspend fun create(post: Post): Post
     suspend fun create(post: PostCreateDto): Post

src/main/kotlin/dev/usbharu/hideout/service/ServerInitialiseServiceImpl.kt

@@ -25,7 +25,7 @@ class ServerInitialiseServiceImpl(private val metaRepository: IMetaRepository) :
             return
         }
 
-        if (isVersionChanged(savedMeta!!)) {
+        if (isVersionChanged(requireNotNull(savedMeta))) {
             logger.info("Version changed!! (${savedMeta.version} -> $implementationVersion)")
             updateVersion(savedMeta, implementationVersion)
         }

src/main/kotlin/dev/usbharu/hideout/service/impl/PostService.kt

@@ -41,8 +41,15 @@ class PostService(
         val user = userService.findById(post.userId)
         val id = postRepository.generateId()
         val postEntity = Post(
-            id, user.id, null, post.text,
+            id = id,
-            Instant.now().toEpochMilli(), Visibility.PUBLIC, "${user.url}/posts/$id", null, null
+            userId = user.id,
+            overview = null,
+            text = post.text,
+            createdAt = Instant.now().toEpochMilli(),
+            visibility = Visibility.PUBLIC,
+            url = "${user.url}/posts/$id",
+            repostId = null,
+            replyId = null
         )
         postRepository.save(postEntity)
         return postEntity

src/main/kotlin/dev/usbharu/hideout/service/impl/UserService.kt

@@ -3,7 +3,6 @@ package dev.usbharu.hideout.service.impl
 import dev.usbharu.hideout.config.Config
 import dev.usbharu.hideout.domain.model.hideout.dto.RemoteUserCreateDto
 import dev.usbharu.hideout.domain.model.hideout.dto.UserCreateDto
-import dev.usbharu.hideout.domain.model.hideout.dto.UserResponse
 import dev.usbharu.hideout.domain.model.hideout.entity.User
 import dev.usbharu.hideout.exception.UserNotFoundException
 import dev.usbharu.hideout.repository.IUserRepository
@@ -24,10 +23,6 @@ class UserService(private val userRepository: IUserRepository, private val userA
         )
     }
 
-    suspend fun findAllForUser(limit: Int?, offset: Long?): List<UserResponse> {
-        TODO("Not yet implemented")
-    }
-
     override suspend fun findById(id: Long): User =
         userRepository.findById(id) ?: throw UserNotFoundException("$id was not found.")