From 2dcb532e24cb545c45e4757d03f51deed803f29f Mon Sep 17 00:00:00 2001 From: toki Date: Sun, 12 Apr 2026 16:35:39 +0900 Subject: [PATCH] Add Kotlin implementation and improve protocol documentation --- .claude/settings.json | 5 +- PROTOCOL.md | 10 +- README.md | 27 +- go/crosstest/go_dart.go | 2 + go/crosstest/go_kotlin.go | 324 +++++++ go/crosstest/kotlin_go_client/main.go | 205 +++++ kotlin/build.gradle.kts | 64 ++ .../tokilabs/toki_socket/crosstest/Main.kt | 208 +++++ kotlin/crosstest/kotlin_go.kt | 233 +++++ kotlin/gradle/wrapper/gradle-wrapper.jar | Bin 0 -> 43583 bytes .../gradle/wrapper/gradle-wrapper.properties | 7 + kotlin/gradlew | 252 ++++++ kotlin/gradlew.bat | 94 ++ kotlin/settings.gradle.kts | 17 + .../com/tokilabs/toki_socket/BaseClient.kt | 104 +++ .../com/tokilabs/toki_socket/Communicator.kt | 319 +++++++ .../tokilabs/toki_socket/HeartbeatTimer.kt | 31 + .../com/tokilabs/toki_socket/TcpClient.kt | 129 +++ .../com/tokilabs/toki_socket/TcpServer.kt | 77 ++ .../com/tokilabs/toki_socket/WsClient.kt | 228 +++++ .../com/tokilabs/toki_socket/WsServer.kt | 93 ++ kotlin/src/main/proto/message_common.proto | 19 + .../tokilabs/toki_socket/CommunicatorTest.kt | 95 +++ .../com/tokilabs/toki_socket/HeartbeatTest.kt | 107 +++ .../toki_socket/HeartbeatTimerTest.kt | 49 ++ .../com/tokilabs/toki_socket/TcpTest.kt | 114 +++ .../com/tokilabs/toki_socket/TestHelpers.kt | 33 + .../kotlin/com/tokilabs/toki_socket/WsTest.kt | 101 +++ skills/code-review/SKILL.md | 91 +- tasks/kotlin_impl/code_review_0.log | 233 +++++ tasks/kotlin_impl/code_review_1.log | 254 ++++++ tasks/kotlin_impl/complete.log | 31 + tasks/kotlin_impl/plan_0.log | 807 ++++++++++++++++++ tasks/kotlin_impl/plan_1.log | 396 +++++++++ tools/check_proto_sync.sh | 18 + 35 files changed, 4763 insertions(+), 14 deletions(-) create mode 100644 go/crosstest/go_kotlin.go create mode 100644 go/crosstest/kotlin_go_client/main.go create mode 100644 kotlin/build.gradle.kts create mode 100644 kotlin/crosstest/go_kotlin_client/src/main/kotlin/com/tokilabs/toki_socket/crosstest/Main.kt create mode 100644 kotlin/crosstest/kotlin_go.kt create mode 100644 kotlin/gradle/wrapper/gradle-wrapper.jar create mode 100644 kotlin/gradle/wrapper/gradle-wrapper.properties create mode 100755 kotlin/gradlew create mode 100644 kotlin/gradlew.bat create mode 100644 kotlin/settings.gradle.kts create mode 100644 kotlin/src/main/kotlin/com/tokilabs/toki_socket/BaseClient.kt create mode 100644 kotlin/src/main/kotlin/com/tokilabs/toki_socket/Communicator.kt create mode 100644 kotlin/src/main/kotlin/com/tokilabs/toki_socket/HeartbeatTimer.kt create mode 100644 kotlin/src/main/kotlin/com/tokilabs/toki_socket/TcpClient.kt create mode 100644 kotlin/src/main/kotlin/com/tokilabs/toki_socket/TcpServer.kt create mode 100644 kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsClient.kt create mode 100644 kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsServer.kt create mode 100644 kotlin/src/main/proto/message_common.proto create mode 100644 kotlin/src/test/kotlin/com/tokilabs/toki_socket/CommunicatorTest.kt create mode 100644 kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTest.kt create mode 100644 kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTimerTest.kt create mode 100644 kotlin/src/test/kotlin/com/tokilabs/toki_socket/TcpTest.kt create mode 100644 kotlin/src/test/kotlin/com/tokilabs/toki_socket/TestHelpers.kt create mode 100644 kotlin/src/test/kotlin/com/tokilabs/toki_socket/WsTest.kt create mode 100644 tasks/kotlin_impl/code_review_0.log create mode 100644 tasks/kotlin_impl/code_review_1.log create mode 100644 tasks/kotlin_impl/complete.log create mode 100644 tasks/kotlin_impl/plan_0.log create mode 100644 tasks/kotlin_impl/plan_1.log diff --git a/.claude/settings.json b/.claude/settings.json index 0cf2045..712dac1 100644 --- a/.claude/settings.json +++ b/.claude/settings.json @@ -6,7 +6,10 @@ "Bash(sudo apt-get:*)", "Bash(protoc --version)", "Bash(dart analyze:*)", - "Bash(ls *.log)" + "Bash(ls *.log)", + "Bash(tools/check_proto_sync.sh)", + "Bash(mv CODE_REVIEW.md code_review_0.log)", + "Bash(mv PLAN.md plan_0.log)" ], "additionalDirectories": [ "/config/workspace/toki_socket/tasks" diff --git a/PROTOCOL.md b/PROTOCOL.md index 89ee213..7863f53 100644 --- a/PROTOCOL.md +++ b/PROTOCOL.md @@ -33,7 +33,7 @@ This protocol intentionally defines only the transport-level contract shared acr - 4 bytes, big-endian positive integer. Dart reads/writes it as `int32`; Go reads/writes the same 4 bytes as `uint32`. - Value: byte length of the following `PacketBase` protobuf payload - Value of `0`: reserved / no-op, receiver clears buffer -- Implementations may apply safety limits to payload size. The current Dart and Go implementations reject TCP packets larger than 64 MiB. +- Implementations may apply safety limits to payload size. The current Dart, Go, and Kotlin implementations reject TCP packets larger than 64 MiB. ### WebSocket / WSS @@ -111,12 +111,12 @@ On the receive side, use the same value as the registration key. | Dart | `T.toString()` (equals qualified name for top-level proto messages) | | Go | `string(proto.MessageName(m))` via `TypeNameOf(m)` | | C# | `typeof(T).Name` — verify matches proto qualified name | -| Kotlin | `T::class.simpleName` — verify matches | +| Kotlin | `descriptorForType.fullName` via `typeNameOf(m)` | | Swift | `String(describing: T.self)` — verify matches | | Python | `descriptor.name` from `MessageClass.DESCRIPTOR` — verify matches | | Rust | `M::default().descriptor_dyn().name().to_string()` (protobuf crate) — verify matches | -The current packet proto has no `package` declaration, so Dart and Go both use simple names such as `TestData` and `HeartBeat`. If a future proto adds a `package`, `proto.MessageName` may become fully qualified, and all implementations must use the same value. +The current packet proto has no `package` declaration, so Dart, Go, and Kotlin all use simple names such as `TestData` and `HeartBeat`. If a future proto adds a `package`, `proto.MessageName` may become fully qualified, and all implementations must use the same value. **Important**: Verify typeName consistency across languages before connecting heterogeneous clients. @@ -177,7 +177,7 @@ Sending `HeartBeat {}`: |----------|--------|------| | Dart | Available | `dart/` | | C# (Unity) | Planned | `csharp/` | -| Kotlin | Planned | `kotlin/` | +| Kotlin | In progress | `kotlin/` | | Swift | Planned | `swift/` | | Go | Available | `go/` | | Python | Planned | `python/` | @@ -190,4 +190,4 @@ Sending `HeartBeat {}`: `dart/lib/src/packets/message_common.proto` is the canonical packet definition. All language implementations must keep the same message schema and generate bindings from it. -The Go implementation keeps a copy at `go/packets/message_common.proto` because Go generation needs `option go_package`. Keep the message fields in sync with the canonical Dart proto before regenerating `go/packets/message_common.pb.go`. +The Go implementation keeps a copy at `go/packets/message_common.proto` because Go generation needs `option go_package`. The Kotlin implementation keeps a copy at `kotlin/src/main/proto/message_common.proto` for Java/Kotlin generation options. Keep the message fields in sync with the canonical Dart proto before regenerating language bindings. diff --git a/README.md b/README.md index 09a7d94..774ab29 100644 --- a/README.md +++ b/README.md @@ -36,7 +36,7 @@ Protocol compatibility is tracked separately from language package versions. See |----------|--------|------|----------| | Dart | Available | [dart/](dart/) | Flutter, Dart server | | C# | Planned | `csharp/` | Unity, .NET | -| Kotlin | Planned | `kotlin/` | Android | +| Kotlin | In progress | [kotlin/](kotlin/) | Android, JVM | | Swift | Planned | `swift/` | iOS, macOS | | Go | Available | [go/](go/) | Server, tooling, scripting | | Python | Planned | `python/` | Server, tooling, scripting | @@ -91,7 +91,7 @@ tools/generate_proto.sh tools/check_proto_sync.sh ``` -The Go proto copy is allowed to keep only its Go-specific `option go_package` difference. `tools/check_proto_sync.sh` fails with a diff when the message schema drifts. +The Go and Kotlin proto copies are allowed to keep only language-specific options such as `option go_package` or Java package/class options. `tools/check_proto_sync.sh` fails with a diff when the message schema drifts. --- @@ -180,6 +180,29 @@ cd go go test ./... ``` +```bash +cd kotlin +./gradlew test +``` + +Cross-language checks: + +```bash +cd go +go run ./crosstest/go_dart.go +go run ./crosstest/go_kotlin.go +``` + +```bash +cd dart +dart run crosstest/dart_go.dart +``` + +```bash +cd kotlin +./gradlew run -PmainClass=com.tokilabs.toki_socket.crosstest.KotlinGoKt +``` + When proto files change, also run: ```bash diff --git a/go/crosstest/go_dart.go b/go/crosstest/go_dart.go index 5a21af7..a9c56c2 100644 --- a/go/crosstest/go_dart.go +++ b/go/crosstest/go_dart.go @@ -1,3 +1,5 @@ +//go:build ignore + package main import ( diff --git a/go/crosstest/go_kotlin.go b/go/crosstest/go_kotlin.go new file mode 100644 index 0000000..ab77f46 --- /dev/null +++ b/go/crosstest/go_kotlin.go @@ -0,0 +1,324 @@ +//go:build ignore + +package main + +import ( + "bufio" + "context" + "errors" + "fmt" + "io" + "net" + "os" + "os/exec" + "path/filepath" + "regexp" + "runtime" + "sync" + "time" + + "google.golang.org/protobuf/proto" + "nhooyr.io/websocket" + + toki "toki-labs.com/toki_socket/go" + "toki-labs.com/toki_socket/go/packets" +) + +const ( + host = "127.0.0.1" + goKotlinTCPPort = 29290 + goKotlinWSPort = 29292 + wsPath = "/" + processTimeout = 20 * time.Second + serverObservationWindow = 200 * time.Millisecond +) + +func parserMap() toki.ParserMap { + return toki.ParserMap{ + toki.TypeNameOf(&packets.TestData{}): func(b []byte) (proto.Message, error) { + m := &packets.TestData{} + return m, proto.Unmarshal(b, m) + }, + } +} + +func main() { + fmt.Printf("INFO typeName go=%s\n", toki.TypeNameOf(&packets.TestData{})) + if err := run(); err != nil { + fmt.Fprintf(os.Stderr, "FAIL crosstest error=%v\n", err) + os.Exit(1) + } + fmt.Println("PASS all go-server/kotlin-client crosstests passed") +} + +func run() error { + if err := runTCPSendPush(); err != nil { + return err + } + time.Sleep(150 * time.Millisecond) + if err := runTCPRequests(); err != nil { + return err + } + time.Sleep(150 * time.Millisecond) + if err := runWSSendPush(); err != nil { + return err + } + time.Sleep(150 * time.Millisecond) + return runWSRequests() +} + +func runTCPSendPush() error { + ctx, cancel := context.WithCancel(context.Background()) + defer cancel() + + received := make(chan bool, 1) + server := toki.NewTcpServer(host, goKotlinTCPPort, func(conn net.Conn) *toki.TcpClient { + return toki.NewTcpClient(conn, 0, 0, parserMap()) + }) + server.OnClientConnected = func(client *toki.TcpClient) { + toki.AddListenerTyped[*packets.TestData](&client.Communicator, func(data *packets.TestData) { + fmt.Printf("SERVER_RECEIVED index=%d message=%s\n", data.GetIndex(), data.GetMessage()) + valid := data.GetIndex() == 101 && data.GetMessage() == "fire from kotlin client" + select { + case received <- valid: + default: + } + if valid { + _ = client.Send(&packets.TestData{Index: 200, Message: "push from go server"}) + } + }) + } + if err := server.Start(ctx); err != nil { + return err + } + defer server.Stop() + + if err := runKotlinClient("tcp", goKotlinTCPPort, "send-push", map[string]bool{"1": true, "2": true}); err != nil { + return err + } + select { + case ok := <-received: + if !ok { + return errors.New("TCP send-push server received unexpected data") + } + case <-time.After(serverObservationWindow): + return errors.New("TCP send-push server did not receive expected data") + } + return nil +} + +func runTCPRequests() error { + ctx, cancel := context.WithCancel(context.Background()) + defer cancel() + + server := toki.NewTcpServer(host, goKotlinTCPPort, func(conn net.Conn) *toki.TcpClient { + return toki.NewTcpClient(conn, 0, 0, parserMap()) + }) + server.OnClientConnected = func(client *toki.TcpClient) { + toki.AddRequestListenerTyped[*packets.TestData, *packets.TestData](&client.Communicator, func(req *packets.TestData) (*packets.TestData, error) { + return &packets.TestData{ + Index: req.GetIndex() * 2, + Message: "echo: " + req.GetMessage(), + }, nil + }) + } + if err := server.Start(ctx); err != nil { + return err + } + defer server.Stop() + + return runKotlinClient("tcp", goKotlinTCPPort, "requests", map[string]bool{"3": true, "4": true}) +} + +func runWSSendPush() error { + ctx, cancel := context.WithCancel(context.Background()) + defer cancel() + + received := make(chan bool, 1) + server := toki.NewWsServer(host, goKotlinWSPort, wsPath, func(conn *websocket.Conn) *toki.WsClient { + return toki.NewWsClient(conn, 0, 0, parserMap()) + }) + server.OnClientConnected = func(client *toki.WsClient) { + toki.AddListenerTyped[*packets.TestData](&client.Communicator, func(data *packets.TestData) { + fmt.Printf("SERVER_RECEIVED index=%d message=%s\n", data.GetIndex(), data.GetMessage()) + valid := data.GetIndex() == 101 && data.GetMessage() == "fire from kotlin client" + select { + case received <- valid: + default: + } + if valid { + _ = client.Send(&packets.TestData{Index: 200, Message: "push from go server"}) + } + }) + } + if err := server.Start(ctx); err != nil { + return err + } + defer server.Stop() + + if err := runKotlinClient("ws", goKotlinWSPort, "send-push", map[string]bool{"1": true, "2": true}); err != nil { + return err + } + select { + case ok := <-received: + if !ok { + return errors.New("WS send-push server received unexpected data") + } + case <-time.After(serverObservationWindow): + return errors.New("WS send-push server did not receive expected data") + } + return nil +} + +func runWSRequests() error { + ctx, cancel := context.WithCancel(context.Background()) + defer cancel() + + server := toki.NewWsServer(host, goKotlinWSPort, wsPath, func(conn *websocket.Conn) *toki.WsClient { + return toki.NewWsClient(conn, 0, 0, parserMap()) + }) + server.OnClientConnected = func(client *toki.WsClient) { + toki.AddRequestListenerTyped[*packets.TestData, *packets.TestData](&client.Communicator, func(req *packets.TestData) (*packets.TestData, error) { + return &packets.TestData{ + Index: req.GetIndex() * 2, + Message: "echo: " + req.GetMessage(), + }, nil + }) + } + if err := server.Start(ctx); err != nil { + return err + } + defer server.Stop() + + return runKotlinClient("ws", goKotlinWSPort, "requests", map[string]bool{"3": true, "4": true}) +} + +func runKotlinClient(mode string, port int, phase string, expected map[string]bool) error { + ctx, cancel := context.WithTimeout(context.Background(), processTimeout) + defer cancel() + + kotlinDir, err := kotlinPackageDir() + if err != nil { + return err + } + + cmd := exec.CommandContext(ctx, "./gradlew", "run", + "--args=--mode="+mode+" --port="+fmt.Sprint(port)+" --phase="+phase, + ) + cmd.Dir = kotlinDir + + stdoutPipe, err := cmd.StdoutPipe() + if err != nil { + return err + } + stderrPipe, err := cmd.StderrPipe() + if err != nil { + return err + } + if err := cmd.Start(); err != nil { + return err + } + + var mu sync.Mutex + var resultLines []string + var wg sync.WaitGroup + wg.Add(2) + go scanLines(&wg, stdoutPipe, os.Stdout, &mu, &resultLines) + go scanLines(&wg, stderrPipe, os.Stderr, nil, nil) + + waitErr := cmd.Wait() + wg.Wait() + if ctx.Err() == context.DeadlineExceeded { + return fmt.Errorf("kotlin client %s/%s timed out", mode, phase) + } + return validateResultLines("kotlin-client "+mode+"/"+phase, waitErr, resultLines, expected) +} + +func kotlinPackageDir() (string, error) { + candidates := make([]string, 0, 3) + + _, filename, _, ok := runtime.Caller(0) + if ok { + repoRoot := filepath.Dir(filepath.Dir(filepath.Dir(filename))) + candidates = append(candidates, filepath.Join(repoRoot, "kotlin")) + } + if wd, err := os.Getwd(); err == nil { + candidates = append(candidates, findKotlinPackageCandidates(wd)...) + } + if executable, err := os.Executable(); err == nil { + candidates = append(candidates, findKotlinPackageCandidates(filepath.Dir(executable))...) + } + + for _, candidate := range candidates { + if isKotlinPackageDir(candidate) { + return candidate, nil + } + } + return "", fmt.Errorf("cannot resolve kotlin package directory from candidates %v", candidates) +} + +func findKotlinPackageCandidates(start string) []string { + candidates := make([]string, 0) + for dir := start; ; dir = filepath.Dir(dir) { + candidates = append(candidates, filepath.Join(dir, "kotlin")) + if filepath.Base(dir) == "kotlin" { + candidates = append(candidates, dir) + } + parent := filepath.Dir(dir) + if parent == dir { + return candidates + } + } +} + +func isKotlinPackageDir(dir string) bool { + info, err := os.Stat(filepath.Join(dir, "build.gradle.kts")) + return err == nil && !info.IsDir() +} + +func scanLines(wg *sync.WaitGroup, reader io.Reader, writer *os.File, mu *sync.Mutex, resultLines *[]string) { + defer wg.Done() + scanner := bufio.NewScanner(reader) + for scanner.Scan() { + line := scanner.Text() + fmt.Fprintln(writer, line) + if mu != nil && startsWithResult(line) { + mu.Lock() + *resultLines = append(*resultLines, line) + mu.Unlock() + } + } +} + +func validateResultLines(label string, waitErr error, lines []string, expected map[string]bool) error { + failed := make([]string, 0) + passed := make(map[string]bool) + re := regexp.MustCompile(`scenario=([^ ]+)`) + for _, line := range lines { + if len(line) >= 5 && line[:5] == "FAIL " { + failed = append(failed, line) + continue + } + if len(line) >= 5 && line[:5] == "PASS " { + match := re.FindStringSubmatch(line) + if len(match) == 2 { + passed[match[1]] = true + } + } + } + + missing := make([]string, 0) + for scenario := range expected { + if !passed[scenario] { + missing = append(missing, scenario) + } + } + if waitErr != nil || len(failed) > 0 || len(missing) > 0 { + return fmt.Errorf("%s failed waitErr=%v failed=%v missing=%v", label, waitErr, failed, missing) + } + return nil +} + +func startsWithResult(line string) bool { + return (len(line) >= 5 && line[:5] == "PASS ") || (len(line) >= 5 && line[:5] == "FAIL ") +} diff --git a/go/crosstest/kotlin_go_client/main.go b/go/crosstest/kotlin_go_client/main.go new file mode 100644 index 0000000..d513848 --- /dev/null +++ b/go/crosstest/kotlin_go_client/main.go @@ -0,0 +1,205 @@ +package main + +import ( + "context" + "flag" + "fmt" + "os" + "sync" + "time" + + "google.golang.org/protobuf/proto" + + toki "toki-labs.com/toki_socket/go" + "toki-labs.com/toki_socket/go/packets" +) + +const ( + host = "127.0.0.1" + wsPath = "/" + connectWindow = 3 * time.Second + requestWindow = 2 * time.Second +) + +type clientHandle struct { + communicator *toki.Communicator + send func(proto.Message) error + close func() error +} + +func parserMap() toki.ParserMap { + return toki.ParserMap{ + toki.TypeNameOf(&packets.TestData{}): func(b []byte) (proto.Message, error) { + m := &packets.TestData{} + return m, proto.Unmarshal(b, m) + }, + } +} + +func main() { + mode := flag.String("mode", "tcp", "transport mode: tcp or ws") + port := flag.Int("port", 0, "server port") + phase := flag.String("phase", "send-push", "test phase: send-push or requests") + flag.Parse() + + fmt.Printf("INFO typeName go=%s\n", toki.TypeNameOf(&packets.TestData{})) + + if *port == 0 { + fail("setup", "port is required") + os.Exit(1) + } + + client, err := dialWithRetry(*mode, *port) + if err != nil { + fail("setup", err.Error()) + os.Exit(1) + } + defer client.close() + + var ok bool + switch *phase { + case "send-push": + ok = runSendPush(client) + case "requests": + ok = runRequests(client) + default: + fail("setup", fmt.Sprintf("unknown phase %q", *phase)) + ok = false + } + if !ok { + os.Exit(1) + } +} + +func dialWithRetry(mode string, port int) (*clientHandle, error) { + deadline := time.Now().Add(connectWindow) + var lastErr error + for time.Now().Before(deadline) { + ctx, cancel := context.WithTimeout(context.Background(), 300*time.Millisecond) + handle, err := dial(ctx, mode, port) + cancel() + if err == nil { + return handle, nil + } + lastErr = err + time.Sleep(100 * time.Millisecond) + } + return nil, fmt.Errorf("connect %s:%d timed out: %w", mode, port, lastErr) +} + +func dial(ctx context.Context, mode string, port int) (*clientHandle, error) { + switch mode { + case "tcp": + client, err := toki.DialTcp(ctx, host, port, 0, 0, parserMap()) + if err != nil { + return nil, err + } + return &clientHandle{ + communicator: &client.Communicator, + send: client.Send, + close: client.Close, + }, nil + case "ws": + client, err := toki.DialWsWithHeartbeat(ctx, host, port, wsPath, 0, 0, parserMap()) + if err != nil { + return nil, err + } + return &clientHandle{ + communicator: &client.Communicator, + send: client.Send, + close: client.Close, + }, nil + default: + return nil, fmt.Errorf("unknown mode %q", mode) + } +} + +func runSendPush(client *clientHandle) bool { + pushCh := make(chan *packets.TestData, 1) + toki.AddListenerTyped[*packets.TestData](client.communicator, func(msg *packets.TestData) { + pushCh <- msg + }) + + err := client.send(&packets.TestData{ + Index: 101, + Message: "fire from go client", + }) + if err != nil { + fail("1", err.Error()) + return false + } + pass("1", "fire-and-forget sent") + + select { + case msg := <-pushCh: + if msg.GetIndex() != 200 || msg.GetMessage() != "push from kotlin server" { + fail("2", fmt.Sprintf("unexpected push index=%d message=%q", msg.GetIndex(), msg.GetMessage())) + return false + } + pass("2", "received push from kotlin server") + return true + case <-time.After(requestWindow): + fail("2", "timeout waiting for server push") + return false + } +} + +func runRequests(client *clientHandle) bool { + res, err := toki.SendRequestTyped[*packets.TestData, *packets.TestData]( + client.communicator, + &packets.TestData{Index: 21, Message: "single request from go"}, + requestWindow, + ) + if err != nil { + fail("3", err.Error()) + return false + } + if res.GetIndex() != 42 || res.GetMessage() != "echo: single request from go" { + fail("3", fmt.Sprintf("unexpected response index=%d message=%q", res.GetIndex(), res.GetMessage())) + return false + } + pass("3", "single request response matched") + + const count = 5 + var wg sync.WaitGroup + errCh := make(chan error, count) + for i := 0; i < count; i++ { + i := i + wg.Add(1) + go func() { + defer wg.Done() + index := int32(30 + i) + message := fmt.Sprintf("multi request %d from go", i) + res, err := toki.SendRequestTyped[*packets.TestData, *packets.TestData]( + client.communicator, + &packets.TestData{Index: index, Message: message}, + requestWindow, + ) + if err != nil { + errCh <- err + return + } + if res.GetIndex() != index*2 || res.GetMessage() != "echo: "+message { + errCh <- fmt.Errorf("request %d got index=%d message=%q", i, res.GetIndex(), res.GetMessage()) + } + }() + } + wg.Wait() + close(errCh) + for err := range errCh { + if err != nil { + fail("4", err.Error()) + return false + } + } + pass("4", "concurrent request responses matched") + return true +} + +func pass(scenario, detail string) { + fmt.Printf("PASS scenario=%s detail=%s\n", scenario, detail) +} + +func fail(scenario, detail string) { + fmt.Printf("FAIL scenario=%s error=%s\n", scenario, detail) +} diff --git a/kotlin/build.gradle.kts b/kotlin/build.gradle.kts new file mode 100644 index 0000000..82f763b --- /dev/null +++ b/kotlin/build.gradle.kts @@ -0,0 +1,64 @@ +plugins { + kotlin("jvm") version "2.0.0" + application + id("com.google.protobuf") version "0.9.4" +} + +group = "com.tokilabs" +version = "0.1.0" + +kotlin { + jvmToolchain(17) +} + +sourceSets { + create("crosstest") { + kotlin.srcDir("crosstest") + compileClasspath += sourceSets["main"].output + configurations["runtimeClasspath"] + runtimeClasspath += output + compileClasspath + } +} + +dependencies { + implementation("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.8.1") + implementation("com.google.protobuf:protobuf-kotlin:4.27.0") + implementation("com.squareup.okhttp3:okhttp:4.12.0") + implementation("org.java-websocket:Java-WebSocket:1.5.6") + + testImplementation("org.jetbrains.kotlinx:kotlinx-coroutines-test:1.8.1") + testImplementation(kotlin("test")) +} + +protobuf { + protoc { + artifact = "com.google.protobuf:protoc:4.27.0" + } + generateProtoTasks { + all().forEach { task -> + task.builtins { + create("kotlin") + } + } + } +} + +application { + mainClass.set( + (findProperty("mainClass") as String?) + ?: "com.tokilabs.toki_socket.crosstest.MainKt", + ) +} + +val crosstestSourceSet = sourceSets["crosstest"] + +tasks.named("run") { + classpath = crosstestSourceSet.runtimeClasspath + mainClass.set( + (findProperty("mainClass") as String?) + ?: "com.tokilabs.toki_socket.crosstest.MainKt", + ) +} + +tasks.test { + useJUnitPlatform() +} diff --git a/kotlin/crosstest/go_kotlin_client/src/main/kotlin/com/tokilabs/toki_socket/crosstest/Main.kt b/kotlin/crosstest/go_kotlin_client/src/main/kotlin/com/tokilabs/toki_socket/crosstest/Main.kt new file mode 100644 index 0000000..ee8eda1 --- /dev/null +++ b/kotlin/crosstest/go_kotlin_client/src/main/kotlin/com/tokilabs/toki_socket/crosstest/Main.kt @@ -0,0 +1,208 @@ +package com.tokilabs.toki_socket.crosstest + +import com.tokilabs.toki_socket.DialTcp +import com.tokilabs.toki_socket.DialWs +import com.tokilabs.toki_socket.ParserMap +import com.tokilabs.toki_socket.WsClient +import com.tokilabs.toki_socket.TcpClient +import com.tokilabs.toki_socket.addListenerTyped +import com.tokilabs.toki_socket.sendRequestTyped +import com.tokilabs.toki_socket.typeNameOf +import com.tokilabs.toki_socket.packets.TestData +import kotlinx.coroutines.CompletableDeferred +import kotlinx.coroutines.async +import kotlinx.coroutines.coroutineScope +import kotlinx.coroutines.delay +import kotlinx.coroutines.runBlocking +import kotlinx.coroutines.withTimeout +import kotlin.system.exitProcess + +private const val HOST = "127.0.0.1" +private const val WS_PATH = "/" +private const val CONNECT_WINDOW_MS = 3_000L +private const val REQUEST_WINDOW_MS = 2_000L + +private interface ClientHandle { + suspend fun send(data: TestData) + + suspend fun close() + + val communicator: com.tokilabs.toki_socket.Communicator +} + +private class TcpHandle( + private val client: TcpClient, +) : ClientHandle { + override val communicator = client.communicator + + override suspend fun send(data: TestData) { + client.send(data) + } + + override suspend fun close() { + client.close() + } +} + +private class WsHandle( + private val client: WsClient, +) : ClientHandle { + override val communicator = client.communicator + + override suspend fun send(data: TestData) { + client.send(data) + } + + override suspend fun close() { + client.close() + } +} + +fun main(args: Array) = runBlocking { + val mode = argValue(args, "mode") ?: "tcp" + val phase = argValue(args, "phase") ?: "send-push" + val port = argValue(args, "port")?.toIntOrNull() + + println("INFO typeName kotlin=${typeNameOf()}") + + if (port == null) { + fail("setup", "port is required") + exitProcess(1) + } + + val client = try { + connectWithRetry(mode, port) + } catch (error: Throwable) { + fail("setup", error.message ?: error.toString()) + exitProcess(1) + } + + val ok = try { + when (phase) { + "send-push" -> runSendPush(client) + "requests" -> runRequests(client) + else -> { + fail("setup", "unknown phase $phase") + false + } + } + } finally { + client.close() + } + + if (!ok) exitProcess(1) +} + +private suspend fun connectWithRetry(mode: String, port: Int): ClientHandle { + val deadline = System.nanoTime() + CONNECT_WINDOW_MS * 1_000_000L + var lastError: Throwable? = null + while (System.nanoTime() < deadline) { + try { + return when (mode) { + "tcp" -> TcpHandle(DialTcp(HOST, port, 0, 0, parserMap())) + "ws" -> WsHandle(DialWs(HOST, port, WS_PATH, 0, 0, parserMap())) + else -> error("unknown mode $mode") + } + } catch (error: Throwable) { + lastError = error + delay(100) + } + } + error("connect $mode:$port timed out: $lastError") +} + +private suspend fun runSendPush(client: ClientHandle): Boolean { + val push = CompletableDeferred() + addListenerTyped(client.communicator) { + push.complete(it) + } + + return try { + client.send( + TestData.newBuilder() + .setIndex(101) + .setMessage("fire from kotlin client") + .build(), + ) + pass("1", "fire-and-forget sent") + + val msg = withTimeout(REQUEST_WINDOW_MS) { push.await() } + if (msg.index != 200 || msg.message != "push from go server") { + fail("2", "unexpected push index=${msg.index} message=${msg.message}") + false + } else { + pass("2", "received push from go server") + true + } + } catch (error: Throwable) { + fail("2", error.message ?: error.toString()) + false + } +} + +private suspend fun runRequests(client: ClientHandle): Boolean = + try { + val single = sendRequestTyped( + client.communicator, + TestData.newBuilder() + .setIndex(21) + .setMessage("single request from kotlin") + .build(), + REQUEST_WINDOW_MS, + ) + if (single.index != 42 || single.message != "echo: single request from kotlin") { + fail("3", "unexpected response index=${single.index} message=${single.message}") + false + } else { + pass("3", "single request response matched") + runConcurrentRequests(client) + } + } catch (error: Throwable) { + fail("3", error.message ?: error.toString()) + false + } + +private suspend fun runConcurrentRequests(client: ClientHandle): Boolean = + coroutineScope { + val jobs = (0 until 5).map { i -> + async { + val index = 30 + i + val message = "multi request $i from kotlin" + val res = sendRequestTyped( + client.communicator, + TestData.newBuilder() + .setIndex(index) + .setMessage(message) + .build(), + REQUEST_WINDOW_MS, + ) + check(res.index == index * 2 && res.message == "echo: $message") { + "request $i got index=${res.index} message=${res.message}" + } + } + } + try { + jobs.forEach { it.await() } + pass("4", "concurrent request responses matched") + true + } catch (error: Throwable) { + fail("4", error.message ?: error.toString()) + false + } + } + +private fun parserMap(): ParserMap = + mapOf(typeNameOf() to { TestData.parseFrom(it) }) + +private fun argValue(args: Array, name: String): String? { + val prefix = "--$name=" + return args.firstOrNull { it.startsWith(prefix) }?.substring(prefix.length) +} + +private fun pass(scenario: String, detail: String) { + println("PASS scenario=$scenario detail=$detail") +} + +private fun fail(scenario: String, error: String) { + println("FAIL scenario=$scenario error=$error") +} diff --git a/kotlin/crosstest/kotlin_go.kt b/kotlin/crosstest/kotlin_go.kt new file mode 100644 index 0000000..1b5c6de --- /dev/null +++ b/kotlin/crosstest/kotlin_go.kt @@ -0,0 +1,233 @@ +@file:JvmName("KotlinGoKt") + +package com.tokilabs.toki_socket.crosstest + +import com.tokilabs.toki_socket.TcpClient +import com.tokilabs.toki_socket.TcpServer +import com.tokilabs.toki_socket.ParserMap +import com.tokilabs.toki_socket.WsClient +import com.tokilabs.toki_socket.WsServer +import com.tokilabs.toki_socket.addListenerTyped +import com.tokilabs.toki_socket.addRequestListenerTyped +import com.tokilabs.toki_socket.typeNameOf +import com.tokilabs.toki_socket.packets.TestData +import kotlinx.coroutines.CompletableDeferred +import kotlinx.coroutines.coroutineScope +import kotlinx.coroutines.delay +import kotlinx.coroutines.launch +import kotlinx.coroutines.runBlocking +import kotlinx.coroutines.withTimeoutOrNull +import java.io.File +import java.util.concurrent.TimeUnit +import kotlin.system.exitProcess + +private const val HOST = "127.0.0.1" +private const val TCP_PORT = 29390 +private const val WS_PORT = 29392 +private const val WS_PATH = "/" +private const val PROCESS_TIMEOUT_MS = 20_000L +private const val SERVER_OBSERVATION_MS = 200L + +fun main() = runBlocking { + println("INFO typeName kotlin=${typeNameOf()}") + try { + runTcpSendPush() + delay(150) + runTcpRequests() + delay(150) + runWsSendPush() + delay(150) + runWsRequests() + println("PASS all kotlin-server/go-client crosstests passed") + } catch (error: Throwable) { + System.err.println("FAIL crosstest error=${error.message ?: error}") + exitProcess(1) + } +} + +private suspend fun runTcpSendPush() = coroutineScope { + val received = CompletableDeferred() + val server = TcpServer(HOST, TCP_PORT) { socket -> + TcpClient.fromSocket(socket, 0, 0, parserMap()) + } + server.onClientConnected = { client -> + addListenerTyped(client.communicator) { data -> + println("SERVER_RECEIVED index=${data.index} message=${data.message}") + val valid = data.index == 101 && data.message == "fire from go client" + received.complete(valid) + if (valid) { + launch { + client.send( + TestData.newBuilder() + .setIndex(200) + .setMessage("push from kotlin server") + .build(), + ) + } + } + } + } + withServer(server::start, server::stop) { + runGoClient("tcp", TCP_PORT, "send-push", setOf("1", "2")) + val ok = withTimeoutOrNull(SERVER_OBSERVATION_MS) { received.await() } ?: false + check(ok) { "TCP send-push server did not receive expected data" } + } +} + +private suspend fun runTcpRequests() { + val server = TcpServer(HOST, TCP_PORT) { socket -> + TcpClient.fromSocket(socket, 0, 0, parserMap()) + } + server.onClientConnected = { client -> + addRequestListenerTyped(client.communicator) { req -> + TestData.newBuilder() + .setIndex(req.index * 2) + .setMessage("echo: ${req.message}") + .build() + } + } + withServer(server::start, server::stop) { + runGoClient("tcp", TCP_PORT, "requests", setOf("3", "4")) + } +} + +private suspend fun runWsSendPush() = coroutineScope { + val received = CompletableDeferred() + val server = WsServer(HOST, WS_PORT, WS_PATH) { conn -> + WsClient.forServer(conn, 0, 0, parserMap()) + } + server.onClientConnected = { client -> + addListenerTyped(client.communicator) { data -> + println("SERVER_RECEIVED index=${data.index} message=${data.message}") + val valid = data.index == 101 && data.message == "fire from go client" + received.complete(valid) + if (valid) { + launch { + client.send( + TestData.newBuilder() + .setIndex(200) + .setMessage("push from kotlin server") + .build(), + ) + } + } + } + } + withServer(server::start, server::stop) { + runGoClient("ws", WS_PORT, "send-push", setOf("1", "2")) + val ok = withTimeoutOrNull(SERVER_OBSERVATION_MS) { received.await() } ?: false + check(ok) { "WS send-push server did not receive expected data" } + } +} + +private suspend fun runWsRequests() { + val server = WsServer(HOST, WS_PORT, WS_PATH) { conn -> + WsClient.forServer(conn, 0, 0, parserMap()) + } + server.onClientConnected = { client -> + addRequestListenerTyped(client.communicator) { req -> + TestData.newBuilder() + .setIndex(req.index * 2) + .setMessage("echo: ${req.message}") + .build() + } + } + withServer(server::start, server::stop) { + runGoClient("ws", WS_PORT, "requests", setOf("3", "4")) + } +} + +private inline fun withServer( + start: () -> Unit, + stop: () -> Unit, + body: () -> Unit, +) { + start() + try { + body() + } finally { + stop() + } +} + +private fun runGoClient( + mode: String, + port: Int, + phase: String, + expectedScenarios: Set, +) { + val process = ProcessBuilder( + "go", + "run", + "./crosstest/kotlin_go_client", + "--mode=$mode", + "--port=$port", + "--phase=$phase", + ) + .directory(goDir()) + .redirectErrorStream(false) + .start() + + val stdoutLines = mutableListOf() + val stdoutThread = Thread { + process.inputStream.bufferedReader().forEachLine { line -> + println(line) + if (line.startsWith("PASS ") || line.startsWith("FAIL ")) { + stdoutLines.add(line) + } + } + } + val stderrThread = Thread { + process.errorStream.bufferedReader().forEachLine { line -> + System.err.println(line) + } + } + stdoutThread.start() + stderrThread.start() + + val finished = process.waitFor(PROCESS_TIMEOUT_MS, TimeUnit.MILLISECONDS) + if (!finished) { + process.destroyForcibly() + } + stdoutThread.join() + stderrThread.join() + + validateResultLines( + "go-client $mode/$phase", + if (finished) process.exitValue() else -1, + stdoutLines, + expectedScenarios, + ) +} + +private fun validateResultLines( + label: String, + exitCode: Int, + lines: List, + expectedScenarios: Set, +) { + val failed = lines.filter { it.startsWith("FAIL ") } + val passed = lines + .filter { it.startsWith("PASS ") } + .mapNotNull { Regex("""scenario=([^ ]+)""").find(it)?.groupValues?.get(1) } + .toSet() + val missing = expectedScenarios - passed + check(exitCode == 0 && failed.isEmpty() && missing.isEmpty()) { + "$label failed exitCode=$exitCode failed=$failed missing=$missing" + } +} + +private fun parserMap(): ParserMap = + mapOf(typeNameOf() to { TestData.parseFrom(it) }) + +private fun goDir(): File { + var dir = File(System.getProperty("user.dir")).absoluteFile + while (dir.parentFile != null) { + val candidate = File(dir, "../go").canonicalFile + if (File(candidate, "go.mod").isFile) return candidate + val direct = File(dir, "go").canonicalFile + if (File(direct, "go.mod").isFile) return direct + dir = dir.parentFile + } + error("cannot resolve go directory") +} diff --git a/kotlin/gradle/wrapper/gradle-wrapper.jar b/kotlin/gradle/wrapper/gradle-wrapper.jar new file mode 100644 index 0000000000000000000000000000000000000000..a4b76b9530d66f5e68d973ea569d8e19de379189 GIT binary patch literal 43583 zcma&N1CXTcmMvW9vTb(Rwr$&4wr$(C?dmSu>@vG-+vuvg^_??!{yS%8zW-#zn-LkA z5&1^$^{lnmUON?}LBF8_K|(?T0Ra(xUH{($5eN!MR#ZihR#HxkUPe+_R8Cn`RRs(P z_^*#_XlXmGv7!4;*Y%p4nw?{bNp@UZHv1?Um8r6)Fei3p@ClJn0ECfg1hkeuUU@Or zDaPa;U3fE=3L}DooL;8f;P0ipPt0Z~9P0)lbStMS)ag54=uL9ia-Lm3nh|@(Y?B`; zx_#arJIpXH!U{fbCbI^17}6Ri*H<>OLR%c|^mh8+)*h~K8Z!9)DPf zR2h?lbDZQ`p9P;&DQ4F0sur@TMa!Y}S8irn(%d-gi0*WxxCSk*A?3lGh=gcYN?FGl z7D=Js!i~0=u3rox^eO3i@$0=n{K1lPNU zwmfjRVmLOCRfe=seV&P*1Iq=^i`502keY8Uy-WNPwVNNtJFx?IwAyRPZo2Wo1+S(xF37LJZ~%i)kpFQ3Fw=mXfd@>%+)RpYQLnr}B~~zoof(JVm^^&f zxKV^+3D3$A1G;qh4gPVjhrC8e(VYUHv#dy^)(RoUFM?o%W-EHxufuWf(l*@-l+7vt z=l`qmR56K~F|v<^Pd*p~1_y^P0P^aPC##d8+HqX4IR1gu+7w#~TBFphJxF)T$2WEa zxa?H&6=Qe7d(#tha?_1uQys2KtHQ{)Qco)qwGjrdNL7thd^G5i8Os)CHqc>iOidS} z%nFEDdm=GXBw=yXe1W-ShHHFb?Cc70+$W~z_+}nAoHFYI1MV1wZegw*0y^tC*s%3h zhD3tN8b=Gv&rj}!SUM6|ajSPp*58KR7MPpI{oAJCtY~JECm)*m_x>AZEu>DFgUcby z1Qaw8lU4jZpQ_$;*7RME+gq1KySGG#Wql>aL~k9tLrSO()LWn*q&YxHEuzmwd1?aAtI zBJ>P=&$=l1efe1CDU;`Fd+_;&wI07?V0aAIgc(!{a z0Jg6Y=inXc3^n!U0Atk`iCFIQooHqcWhO(qrieUOW8X(x?(RD}iYDLMjSwffH2~tB z)oDgNBLB^AJBM1M^c5HdRx6fBfka`(LD-qrlh5jqH~);#nw|iyp)()xVYak3;Ybik z0j`(+69aK*B>)e_p%=wu8XC&9e{AO4c~O1U`5X9}?0mrd*m$_EUek{R?DNSh(=br# z#Q61gBzEpmy`$pA*6!87 zSDD+=@fTY7<4A?GLqpA?Pb2z$pbCc4B4zL{BeZ?F-8`s$?>*lXXtn*NC61>|*w7J* z$?!iB{6R-0=KFmyp1nnEmLsA-H0a6l+1uaH^g%c(p{iT&YFrbQ$&PRb8Up#X3@Zsk zD^^&LK~111%cqlP%!_gFNa^dTYT?rhkGl}5=fL{a`UViaXWI$k-UcHJwmaH1s=S$4 z%4)PdWJX;hh5UoK?6aWoyLxX&NhNRqKam7tcOkLh{%j3K^4Mgx1@i|Pi&}<^5>hs5 zm8?uOS>%)NzT(%PjVPGa?X%`N2TQCKbeH2l;cTnHiHppPSJ<7y-yEIiC!P*ikl&!B z%+?>VttCOQM@ShFguHVjxX^?mHX^hSaO_;pnyh^v9EumqSZTi+#f&_Vaija0Q-e*| z7ulQj6Fs*bbmsWp{`auM04gGwsYYdNNZcg|ph0OgD>7O}Asn7^Z=eI>`$2*v78;sj-}oMoEj&@)9+ycEOo92xSyY344^ z11Hb8^kdOvbf^GNAK++bYioknrpdN>+u8R?JxG=!2Kd9r=YWCOJYXYuM0cOq^FhEd zBg2puKy__7VT3-r*dG4c62Wgxi52EMCQ`bKgf*#*ou(D4-ZN$+mg&7$u!! z-^+Z%;-3IDwqZ|K=ah85OLwkO zKxNBh+4QHh)u9D?MFtpbl)us}9+V!D%w9jfAMYEb>%$A;u)rrI zuBudh;5PN}_6J_}l55P3l_)&RMlH{m!)ai-i$g)&*M`eN$XQMw{v^r@-125^RRCF0 z^2>|DxhQw(mtNEI2Kj(;KblC7x=JlK$@78`O~>V!`|1Lm-^JR$-5pUANAnb(5}B}JGjBsliK4& zk6y(;$e&h)lh2)L=bvZKbvh@>vLlreBdH8No2>$#%_Wp1U0N7Ank!6$dFSi#xzh|( zRi{Uw%-4W!{IXZ)fWx@XX6;&(m_F%c6~X8hx=BN1&q}*( zoaNjWabE{oUPb!Bt$eyd#$5j9rItB-h*5JiNi(v^e|XKAj*8(k<5-2$&ZBR5fF|JA z9&m4fbzNQnAU}r8ab>fFV%J0z5awe#UZ|bz?Ur)U9bCIKWEzi2%A+5CLqh?}K4JHi z4vtM;+uPsVz{Lfr;78W78gC;z*yTch~4YkLr&m-7%-xc ztw6Mh2d>_iO*$Rd8(-Cr1_V8EO1f*^@wRoSozS) zy1UoC@pruAaC8Z_7~_w4Q6n*&B0AjOmMWa;sIav&gu z|J5&|{=a@vR!~k-OjKEgPFCzcJ>#A1uL&7xTDn;{XBdeM}V=l3B8fE1--DHjSaxoSjNKEM9|U9#m2<3>n{Iuo`r3UZp;>GkT2YBNAh|b z^jTq-hJp(ebZh#Lk8hVBP%qXwv-@vbvoREX$TqRGTgEi$%_F9tZES@z8Bx}$#5eeG zk^UsLBH{bc2VBW)*EdS({yw=?qmevwi?BL6*=12k9zM5gJv1>y#ML4!)iiPzVaH9% zgSImetD@dam~e>{LvVh!phhzpW+iFvWpGT#CVE5TQ40n%F|p(sP5mXxna+Ev7PDwA zamaV4m*^~*xV+&p;W749xhb_X=$|LD;FHuB&JL5?*Y2-oIT(wYY2;73<^#46S~Gx| z^cez%V7x$81}UWqS13Gz80379Rj;6~WdiXWOSsdmzY39L;Hg3MH43o*y8ibNBBH`(av4|u;YPq%{R;IuYow<+GEsf@R?=@tT@!}?#>zIIn0CoyV!hq3mw zHj>OOjfJM3F{RG#6ujzo?y32m^tgSXf@v=J$ELdJ+=5j|=F-~hP$G&}tDZsZE?5rX ztGj`!S>)CFmdkccxM9eGIcGnS2AfK#gXwj%esuIBNJQP1WV~b~+D7PJTmWGTSDrR` zEAu4B8l>NPuhsk5a`rReSya2nfV1EK01+G!x8aBdTs3Io$u5!6n6KX%uv@DxAp3F@{4UYg4SWJtQ-W~0MDb|j-$lwVn znAm*Pl!?Ps&3wO=R115RWKb*JKoexo*)uhhHBncEDMSVa_PyA>k{Zm2(wMQ(5NM3# z)jkza|GoWEQo4^s*wE(gHz?Xsg4`}HUAcs42cM1-qq_=+=!Gk^y710j=66(cSWqUe zklbm8+zB_syQv5A2rj!Vbw8;|$@C!vfNmNV!yJIWDQ>{+2x zKjuFX`~~HKG~^6h5FntRpnnHt=D&rq0>IJ9#F0eM)Y-)GpRjiN7gkA8wvnG#K=q{q z9dBn8_~wm4J<3J_vl|9H{7q6u2A!cW{bp#r*-f{gOV^e=8S{nc1DxMHFwuM$;aVI^ zz6A*}m8N-&x8;aunp1w7_vtB*pa+OYBw=TMc6QK=mbA-|Cf* zvyh8D4LRJImooUaSb7t*fVfih<97Gf@VE0|z>NcBwBQze);Rh!k3K_sfunToZY;f2 z^HmC4KjHRVg+eKYj;PRN^|E0>Gj_zagfRbrki68I^#~6-HaHg3BUW%+clM1xQEdPYt_g<2K+z!$>*$9nQ>; zf9Bei{?zY^-e{q_*|W#2rJG`2fy@{%6u0i_VEWTq$*(ZN37|8lFFFt)nCG({r!q#9 z5VK_kkSJ3?zOH)OezMT{!YkCuSSn!K#-Rhl$uUM(bq*jY? zi1xbMVthJ`E>d>(f3)~fozjg^@eheMF6<)I`oeJYx4*+M&%c9VArn(OM-wp%M<-`x z7sLP1&3^%Nld9Dhm@$3f2}87!quhI@nwd@3~fZl_3LYW-B?Ia>ui`ELg z&Qfe!7m6ze=mZ`Ia9$z|ARSw|IdMpooY4YiPN8K z4B(ts3p%2i(Td=tgEHX z0UQ_>URBtG+-?0E;E7Ld^dyZ;jjw0}XZ(}-QzC6+NN=40oDb2^v!L1g9xRvE#@IBR zO!b-2N7wVfLV;mhEaXQ9XAU+>=XVA6f&T4Z-@AX!leJ8obP^P^wP0aICND?~w&NykJ#54x3_@r7IDMdRNy4Hh;h*!u(Ol(#0bJdwEo$5437-UBjQ+j=Ic>Q2z` zJNDf0yO6@mr6y1#n3)s(W|$iE_i8r@Gd@!DWDqZ7J&~gAm1#~maIGJ1sls^gxL9LLG_NhU!pTGty!TbhzQnu)I*S^54U6Yu%ZeCg`R>Q zhBv$n5j0v%O_j{QYWG!R9W?5_b&67KB$t}&e2LdMvd(PxN6Ir!H4>PNlerpBL>Zvyy!yw z-SOo8caEpDt(}|gKPBd$qND5#a5nju^O>V&;f890?yEOfkSG^HQVmEbM3Ugzu+UtH zC(INPDdraBN?P%kE;*Ae%Wto&sgw(crfZ#Qy(<4nk;S|hD3j{IQRI6Yq|f^basLY; z-HB&Je%Gg}Jt@={_C{L$!RM;$$|iD6vu#3w?v?*;&()uB|I-XqEKqZPS!reW9JkLewLb!70T7n`i!gNtb1%vN- zySZj{8-1>6E%H&=V}LM#xmt`J3XQoaD|@XygXjdZ1+P77-=;=eYpoEQ01B@L*a(uW zrZeZz?HJsw_4g0vhUgkg@VF8<-X$B8pOqCuWAl28uB|@r`19DTUQQsb^pfqB6QtiT z*`_UZ`fT}vtUY#%sq2{rchyfu*pCg;uec2$-$N_xgjZcoumE5vSI{+s@iLWoz^Mf; zuI8kDP{!XY6OP~q5}%1&L}CtfH^N<3o4L@J@zg1-mt{9L`s^z$Vgb|mr{@WiwAqKg zp#t-lhrU>F8o0s1q_9y`gQNf~Vb!F%70f}$>i7o4ho$`uciNf=xgJ>&!gSt0g;M>*x4-`U)ysFW&Vs^Vk6m%?iuWU+o&m(2Jm26Y(3%TL; zA7T)BP{WS!&xmxNw%J=$MPfn(9*^*TV;$JwRy8Zl*yUZi8jWYF>==j~&S|Xinsb%c z2?B+kpet*muEW7@AzjBA^wAJBY8i|#C{WtO_or&Nj2{=6JTTX05}|H>N2B|Wf!*3_ z7hW*j6p3TvpghEc6-wufFiY!%-GvOx*bZrhZu+7?iSrZL5q9}igiF^*R3%DE4aCHZ zqu>xS8LkW+Auv%z-<1Xs92u23R$nk@Pk}MU5!gT|c7vGlEA%G^2th&Q*zfg%-D^=f z&J_}jskj|Q;73NP4<4k*Y%pXPU2Thoqr+5uH1yEYM|VtBPW6lXaetokD0u z9qVek6Q&wk)tFbQ8(^HGf3Wp16gKmr>G;#G(HRBx?F`9AIRboK+;OfHaLJ(P>IP0w zyTbTkx_THEOs%Q&aPrxbZrJlio+hCC_HK<4%f3ZoSAyG7Dn`=X=&h@m*|UYO-4Hq0 z-Bq&+Ie!S##4A6OGoC~>ZW`Y5J)*ouaFl_e9GA*VSL!O_@xGiBw!AF}1{tB)z(w%c zS1Hmrb9OC8>0a_$BzeiN?rkPLc9%&;1CZW*4}CDDNr2gcl_3z+WC15&H1Zc2{o~i) z)LLW=WQ{?ricmC`G1GfJ0Yp4Dy~Ba;j6ZV4r{8xRs`13{dD!xXmr^Aga|C=iSmor% z8hi|pTXH)5Yf&v~exp3o+sY4B^^b*eYkkCYl*T{*=-0HniSA_1F53eCb{x~1k3*`W zr~};p1A`k{1DV9=UPnLDgz{aJH=-LQo<5%+Em!DNN252xwIf*wF_zS^!(XSm(9eoj z=*dXG&n0>)_)N5oc6v!>-bd(2ragD8O=M|wGW z!xJQS<)u70m&6OmrF0WSsr@I%T*c#Qo#Ha4d3COcX+9}hM5!7JIGF>7<~C(Ear^Sn zm^ZFkV6~Ula6+8S?oOROOA6$C&q&dp`>oR-2Ym3(HT@O7Sd5c~+kjrmM)YmgPH*tL zX+znN>`tv;5eOfX?h{AuX^LK~V#gPCu=)Tigtq9&?7Xh$qN|%A$?V*v=&-2F$zTUv z`C#WyIrChS5|Kgm_GeudCFf;)!WH7FI60j^0o#65o6`w*S7R@)88n$1nrgU(oU0M9 zx+EuMkC>(4j1;m6NoGqEkpJYJ?vc|B zOlwT3t&UgL!pX_P*6g36`ZXQ; z9~Cv}ANFnJGp(;ZhS(@FT;3e)0)Kp;h^x;$*xZn*k0U6-&FwI=uOGaODdrsp-!K$Ac32^c{+FhI-HkYd5v=`PGsg%6I`4d9Jy)uW0y%) zm&j^9WBAp*P8#kGJUhB!L?a%h$hJgQrx!6KCB_TRo%9{t0J7KW8!o1B!NC)VGLM5! zpZy5Jc{`r{1e(jd%jsG7k%I+m#CGS*BPA65ZVW~fLYw0dA-H_}O zrkGFL&P1PG9p2(%QiEWm6x;U-U&I#;Em$nx-_I^wtgw3xUPVVu zqSuKnx&dIT-XT+T10p;yjo1Y)z(x1fb8Dzfn8e yu?e%!_ptzGB|8GrCfu%p?(_ zQccdaaVK$5bz;*rnyK{_SQYM>;aES6Qs^lj9lEs6_J+%nIiuQC*fN;z8md>r_~Mfl zU%p5Dt_YT>gQqfr@`cR!$NWr~+`CZb%dn;WtzrAOI>P_JtsB76PYe*<%H(y>qx-`Kq!X_; z<{RpAqYhE=L1r*M)gNF3B8r(<%8mo*SR2hu zccLRZwGARt)Hlo1euqTyM>^!HK*!Q2P;4UYrysje@;(<|$&%vQekbn|0Ruu_Io(w4#%p6ld2Yp7tlA`Y$cciThP zKzNGIMPXX%&Ud0uQh!uQZz|FB`4KGD?3!ND?wQt6!n*f4EmCoJUh&b?;B{|lxs#F- z31~HQ`SF4x$&v00@(P+j1pAaj5!s`)b2RDBp*PB=2IB>oBF!*6vwr7Dp%zpAx*dPr zb@Zjq^XjN?O4QcZ*O+8>)|HlrR>oD*?WQl5ri3R#2?*W6iJ>>kH%KnnME&TT@ZzrHS$Q%LC?n|e>V+D+8D zYc4)QddFz7I8#}y#Wj6>4P%34dZH~OUDb?uP%-E zwjXM(?Sg~1!|wI(RVuxbu)-rH+O=igSho_pDCw(c6b=P zKk4ATlB?bj9+HHlh<_!&z0rx13K3ZrAR8W)!@Y}o`?a*JJsD+twZIv`W)@Y?Amu_u zz``@-e2X}27$i(2=9rvIu5uTUOVhzwu%mNazS|lZb&PT;XE2|B&W1>=B58#*!~D&) zfVmJGg8UdP*fx(>Cj^?yS^zH#o-$Q-*$SnK(ZVFkw+er=>N^7!)FtP3y~Xxnu^nzY zikgB>Nj0%;WOltWIob|}%lo?_C7<``a5hEkx&1ku$|)i>Rh6@3h*`slY=9U}(Ql_< zaNG*J8vb&@zpdhAvv`?{=zDedJ23TD&Zg__snRAH4eh~^oawdYi6A3w8<Ozh@Kw)#bdktM^GVb zrG08?0bG?|NG+w^&JvD*7LAbjED{_Zkc`3H!My>0u5Q}m!+6VokMLXxl`Mkd=g&Xx z-a>m*#G3SLlhbKB!)tnzfWOBV;u;ftU}S!NdD5+YtOjLg?X}dl>7m^gOpihrf1;PY zvll&>dIuUGs{Qnd- zwIR3oIrct8Va^Tm0t#(bJD7c$Z7DO9*7NnRZorrSm`b`cxz>OIC;jSE3DO8`hX955ui`s%||YQtt2 z5DNA&pG-V+4oI2s*x^>-$6J?p=I>C|9wZF8z;VjR??Icg?1w2v5Me+FgAeGGa8(3S z4vg*$>zC-WIVZtJ7}o9{D-7d>zCe|z#<9>CFve-OPAYsneTb^JH!Enaza#j}^mXy1 z+ULn^10+rWLF6j2>Ya@@Kq?26>AqK{A_| zQKb*~F1>sE*=d?A?W7N2j?L09_7n+HGi{VY;MoTGr_)G9)ot$p!-UY5zZ2Xtbm=t z@dpPSGwgH=QtIcEulQNI>S-#ifbnO5EWkI;$A|pxJd885oM+ zGZ0_0gDvG8q2xebj+fbCHYfAXuZStH2j~|d^sBAzo46(K8n59+T6rzBwK)^rfPT+B zyIFw)9YC-V^rhtK`!3jrhmW-sTmM+tPH+;nwjL#-SjQPUZ53L@A>y*rt(#M(qsiB2 zx6B)dI}6Wlsw%bJ8h|(lhkJVogQZA&n{?Vgs6gNSXzuZpEyu*xySy8ro07QZ7Vk1!3tJphN_5V7qOiyK8p z#@jcDD8nmtYi1^l8ml;AF<#IPK?!pqf9D4moYk>d99Im}Jtwj6c#+A;f)CQ*f-hZ< z=p_T86jog%!p)D&5g9taSwYi&eP z#JuEK%+NULWus;0w32-SYFku#i}d~+{Pkho&^{;RxzP&0!RCm3-9K6`>KZpnzS6?L z^H^V*s!8<>x8bomvD%rh>Zp3>Db%kyin;qtl+jAv8Oo~1g~mqGAC&Qi_wy|xEt2iz zWAJEfTV%cl2Cs<1L&DLRVVH05EDq`pH7Oh7sR`NNkL%wi}8n>IXcO40hp+J+sC!W?!krJf!GJNE8uj zg-y~Ns-<~D?yqbzVRB}G>0A^f0!^N7l=$m0OdZuqAOQqLc zX?AEGr1Ht+inZ-Qiwnl@Z0qukd__a!C*CKuGdy5#nD7VUBM^6OCpxCa2A(X;e0&V4 zM&WR8+wErQ7UIc6LY~Q9x%Sn*Tn>>P`^t&idaOEnOd(Ufw#>NoR^1QdhJ8s`h^|R_ zXX`c5*O~Xdvh%q;7L!_!ohf$NfEBmCde|#uVZvEo>OfEq%+Ns7&_f$OR9xsihRpBb z+cjk8LyDm@U{YN>+r46?nn{7Gh(;WhFw6GAxtcKD+YWV?uge>;+q#Xx4!GpRkVZYu zzsF}1)7$?%s9g9CH=Zs+B%M_)+~*j3L0&Q9u7!|+T`^O{xE6qvAP?XWv9_MrZKdo& z%IyU)$Q95AB4!#hT!_dA>4e@zjOBD*Y=XjtMm)V|+IXzjuM;(l+8aA5#Kaz_$rR6! zj>#&^DidYD$nUY(D$mH`9eb|dtV0b{S>H6FBfq>t5`;OxA4Nn{J(+XihF(stSche7$es&~N$epi&PDM_N`As;*9D^L==2Q7Z2zD+CiU(|+-kL*VG+&9!Yb3LgPy?A zm7Z&^qRG_JIxK7-FBzZI3Q<;{`DIxtc48k> zc|0dmX;Z=W$+)qE)~`yn6MdoJ4co;%!`ddy+FV538Y)j(vg}5*k(WK)KWZ3WaOG!8 z!syGn=s{H$odtpqFrT#JGM*utN7B((abXnpDM6w56nhw}OY}0TiTG1#f*VFZr+^-g zbP10`$LPq_;PvrA1XXlyx2uM^mrjTzX}w{yuLo-cOClE8MMk47T25G8M!9Z5ypOSV zAJUBGEg5L2fY)ZGJb^E34R2zJ?}Vf>{~gB!8=5Z) z9y$>5c)=;o0HeHHSuE4U)#vG&KF|I%-cF6f$~pdYJWk_dD}iOA>iA$O$+4%@>JU08 zS`ep)$XLPJ+n0_i@PkF#ri6T8?ZeAot$6JIYHm&P6EB=BiaNY|aA$W0I+nz*zkz_z zkEru!tj!QUffq%)8y0y`T&`fuus-1p>=^hnBiBqD^hXrPs`PY9tU3m0np~rISY09> z`P3s=-kt_cYcxWd{de@}TwSqg*xVhp;E9zCsnXo6z z?f&Sv^U7n4`xr=mXle94HzOdN!2kB~4=%)u&N!+2;z6UYKUDqi-s6AZ!haB;@&B`? z_TRX0%@suz^TRdCb?!vNJYPY8L_}&07uySH9%W^Tc&1pia6y1q#?*Drf}GjGbPjBS zbOPcUY#*$3sL2x4v_i*Y=N7E$mR}J%|GUI(>WEr+28+V z%v5{#e!UF*6~G&%;l*q*$V?&r$Pp^sE^i-0$+RH3ERUUdQ0>rAq2(2QAbG}$y{de( z>{qD~GGuOk559Y@%$?N^1ApVL_a704>8OD%8Y%8B;FCt%AoPu8*D1 zLB5X>b}Syz81pn;xnB}%0FnwazlWfUV)Z-~rZg6~b z6!9J$EcE&sEbzcy?CI~=boWA&eeIa%z(7SE^qgVLz??1Vbc1*aRvc%Mri)AJaAG!p z$X!_9Ds;Zz)f+;%s&dRcJt2==P{^j3bf0M=nJd&xwUGlUFn?H=2W(*2I2Gdu zv!gYCwM10aeus)`RIZSrCK=&oKaO_Ry~D1B5!y0R=%!i2*KfXGYX&gNv_u+n9wiR5 z*e$Zjju&ODRW3phN925%S(jL+bCHv6rZtc?!*`1TyYXT6%Ju=|X;6D@lq$8T zW{Y|e39ioPez(pBH%k)HzFITXHvnD6hw^lIoUMA;qAJ^CU?top1fo@s7xT13Fvn1H z6JWa-6+FJF#x>~+A;D~;VDs26>^oH0EI`IYT2iagy23?nyJ==i{g4%HrAf1-*v zK1)~@&(KkwR7TL}L(A@C_S0G;-GMDy=MJn2$FP5s<%wC)4jC5PXoxrQBFZ_k0P{{s@sz+gX`-!=T8rcB(=7vW}^K6oLWMmp(rwDh}b zwaGGd>yEy6fHv%jM$yJXo5oMAQ>c9j`**}F?MCry;T@47@r?&sKHgVe$MCqk#Z_3S z1GZI~nOEN*P~+UaFGnj{{Jo@16`(qVNtbU>O0Hf57-P>x8Jikp=`s8xWs^dAJ9lCQ z)GFm+=OV%AMVqVATtN@|vp61VVAHRn87}%PC^RAzJ%JngmZTasWBAWsoAqBU+8L8u z4A&Pe?fmTm0?mK-BL9t+{y7o(7jm+RpOhL9KnY#E&qu^}B6=K_dB}*VlSEiC9fn)+V=J;OnN)Ta5v66ic1rG+dGAJ1 z1%Zb_+!$=tQ~lxQrzv3x#CPb?CekEkA}0MYSgx$Jdd}q8+R=ma$|&1a#)TQ=l$1tQ z=tL9&_^vJ)Pk}EDO-va`UCT1m#Uty1{v^A3P~83_#v^ozH}6*9mIjIr;t3Uv%@VeW zGL6(CwCUp)Jq%G0bIG%?{_*Y#5IHf*5M@wPo6A{$Um++Co$wLC=J1aoG93&T7Ho}P z=mGEPP7GbvoG!uD$k(H3A$Z))+i{Hy?QHdk>3xSBXR0j!11O^mEe9RHmw!pvzv?Ua~2_l2Yh~_!s1qS`|0~0)YsbHSz8!mG)WiJE| z2f($6TQtt6L_f~ApQYQKSb=`053LgrQq7G@98#igV>y#i==-nEjQ!XNu9 z~;mE+gtj4IDDNQJ~JVk5Ux6&LCSFL!y=>79kE9=V}J7tD==Ga+IW zX)r7>VZ9dY=V&}DR))xUoV!u(Z|%3ciQi_2jl}3=$Agc(`RPb z8kEBpvY>1FGQ9W$n>Cq=DIpski};nE)`p3IUw1Oz0|wxll^)4dq3;CCY@RyJgFgc# zKouFh!`?Xuo{IMz^xi-h=StCis_M7yq$u) z?XHvw*HP0VgR+KR6wI)jEMX|ssqYvSf*_3W8zVTQzD?3>H!#>InzpSO)@SC8q*ii- z%%h}_#0{4JG;Jm`4zg};BPTGkYamx$Xo#O~lBirRY)q=5M45n{GCfV7h9qwyu1NxOMoP4)jjZMxmT|IQQh0U7C$EbnMN<3)Kk?fFHYq$d|ICu>KbY_hO zTZM+uKHe(cIZfEqyzyYSUBZa8;Fcut-GN!HSA9ius`ltNebF46ZX_BbZNU}}ZOm{M2&nANL9@0qvih15(|`S~z}m&h!u4x~(%MAO$jHRWNfuxWF#B)E&g3ghSQ9|> z(MFaLQj)NE0lowyjvg8z0#m6FIuKE9lDO~Glg}nSb7`~^&#(Lw{}GVOS>U)m8bF}x zVjbXljBm34Cs-yM6TVusr+3kYFjr28STT3g056y3cH5Tmge~ASxBj z%|yb>$eF;WgrcOZf569sDZOVwoo%8>XO>XQOX1OyN9I-SQgrm;U;+#3OI(zrWyow3 zk==|{lt2xrQ%FIXOTejR>;wv(Pb8u8}BUpx?yd(Abh6? zsoO3VYWkeLnF43&@*#MQ9-i-d0t*xN-UEyNKeyNMHw|A(k(_6QKO=nKMCxD(W(Yop zsRQ)QeL4X3Lxp^L%wzi2-WVSsf61dqliPUM7srDB?Wm6Lzn0&{*}|IsKQW;02(Y&| zaTKv|`U(pSzuvR6Rduu$wzK_W-Y-7>7s?G$)U}&uK;<>vU}^^ns@Z!p+9?St1s)dG zK%y6xkPyyS1$~&6v{kl?Md6gwM|>mt6Upm>oa8RLD^8T{0?HC!Z>;(Bob7el(DV6x zi`I)$&E&ngwFS@bi4^xFLAn`=fzTC;aimE^!cMI2n@Vo%Ae-ne`RF((&5y6xsjjAZ zVguVoQ?Z9uk$2ON;ersE%PU*xGO@T*;j1BO5#TuZKEf(mB7|g7pcEA=nYJ{s3vlbg zd4-DUlD{*6o%Gc^N!Nptgay>j6E5;3psI+C3Q!1ZIbeCubW%w4pq9)MSDyB{HLm|k zxv-{$$A*pS@csolri$Ge<4VZ}e~78JOL-EVyrbxKra^d{?|NnPp86!q>t<&IP07?Z z^>~IK^k#OEKgRH+LjllZXk7iA>2cfH6+(e&9ku5poo~6y{GC5>(bRK7hwjiurqAiZ zg*DmtgY}v83IjE&AbiWgMyFbaRUPZ{lYiz$U^&Zt2YjG<%m((&_JUbZcfJ22(>bi5 z!J?<7AySj0JZ&<-qXX;mcV!f~>G=sB0KnjWca4}vrtunD^1TrpfeS^4dvFr!65knK zZh`d;*VOkPs4*-9kL>$GP0`(M!j~B;#x?Ba~&s6CopvO86oM?-? zOw#dIRc;6A6T?B`Qp%^<U5 z19x(ywSH$_N+Io!6;e?`tWaM$`=Db!gzx|lQ${DG!zb1Zl&|{kX0y6xvO1o z220r<-oaS^^R2pEyY;=Qllqpmue|5yI~D|iI!IGt@iod{Opz@*ml^w2bNs)p`M(Io z|E;;m*Xpjd9l)4G#KaWfV(t8YUn@A;nK^#xgv=LtnArX|vWQVuw3}B${h+frU2>9^ z!l6)!Uo4`5k`<<;E(ido7M6lKTgWezNLq>U*=uz&s=cc$1%>VrAeOoUtA|T6gO4>UNqsdK=NF*8|~*sl&wI=x9-EGiq*aqV!(VVXA57 zw9*o6Ir8Lj1npUXvlevtn(_+^X5rzdR>#(}4YcB9O50q97%rW2me5_L=%ffYPUSRc z!vv?Kv>dH994Qi>U(a<0KF6NH5b16enCp+mw^Hb3Xs1^tThFpz!3QuN#}KBbww`(h z7GO)1olDqy6?T$()R7y%NYx*B0k_2IBiZ14&8|JPFxeMF{vW>HF-Vi3+ZOI=+qP}n zw(+!WcTd~4ZJX1!ZM&y!+uyt=&i!+~d(V%GjH;-NsEEv6nS1TERt|RHh!0>W4+4pp z1-*EzAM~i`+1f(VEHI8So`S`akPfPTfq*`l{Fz`hS%k#JS0cjT2mS0#QLGf=J?1`he3W*;m4)ce8*WFq1sdP=~$5RlH1EdWm|~dCvKOi4*I_96{^95p#B<(n!d?B z=o`0{t+&OMwKcxiBECznJcfH!fL(z3OvmxP#oWd48|mMjpE||zdiTBdWelj8&Qosv zZFp@&UgXuvJw5y=q6*28AtxZzo-UUpkRW%ne+Ylf!V-0+uQXBW=5S1o#6LXNtY5!I z%Rkz#(S8Pjz*P7bqB6L|M#Er{|QLae-Y{KA>`^} z@lPjeX>90X|34S-7}ZVXe{wEei1<{*e8T-Nbj8JmD4iwcE+Hg_zhkPVm#=@b$;)h6 z<<6y`nPa`f3I6`!28d@kdM{uJOgM%`EvlQ5B2bL)Sl=|y@YB3KeOzz=9cUW3clPAU z^sYc}xf9{4Oj?L5MOlYxR{+>w=vJjvbyO5}ptT(o6dR|ygO$)nVCvNGnq(6;bHlBd zl?w-|plD8spjDF03g5ip;W3Z z><0{BCq!Dw;h5~#1BuQilq*TwEu)qy50@+BE4bX28+7erX{BD4H)N+7U`AVEuREE8 z;X?~fyhF-x_sRfHIj~6f(+^@H)D=ngP;mwJjxhQUbUdzk8f94Ab%59-eRIq?ZKrwD z(BFI=)xrUlgu(b|hAysqK<}8bslmNNeD=#JW*}^~Nrswn^xw*nL@Tx!49bfJecV&KC2G4q5a!NSv)06A_5N3Y?veAz;Gv+@U3R% z)~UA8-0LvVE{}8LVDOHzp~2twReqf}ODIyXMM6=W>kL|OHcx9P%+aJGYi_Om)b!xe zF40Vntn0+VP>o<$AtP&JANjXBn7$}C@{+@3I@cqlwR2MdwGhVPxlTIcRVu@Ho-wO` z_~Or~IMG)A_`6-p)KPS@cT9mu9RGA>dVh5wY$NM9-^c@N=hcNaw4ITjm;iWSP^ZX| z)_XpaI61<+La+U&&%2a z0za$)-wZP@mwSELo#3!PGTt$uy0C(nTT@9NX*r3Ctw6J~7A(m#8fE)0RBd`TdKfAT zCf@$MAxjP`O(u9s@c0Fd@|}UQ6qp)O5Q5DPCeE6mSIh|Rj{$cAVIWsA=xPKVKxdhg zLzPZ`3CS+KIO;T}0Ip!fAUaNU>++ZJZRk@I(h<)RsJUhZ&Ru9*!4Ptn;gX^~4E8W^TSR&~3BAZc#HquXn)OW|TJ`CTahk+{qe`5+ixON^zA9IFd8)kc%*!AiLu z>`SFoZ5bW-%7}xZ>gpJcx_hpF$2l+533{gW{a7ce^B9sIdmLrI0)4yivZ^(Vh@-1q zFT!NQK$Iz^xu%|EOK=n>ug;(7J4OnS$;yWmq>A;hsD_0oAbLYhW^1Vdt9>;(JIYjf zdb+&f&D4@4AS?!*XpH>8egQvSVX`36jMd>$+RgI|pEg))^djhGSo&#lhS~9%NuWfX zDDH;3T*GzRT@5=7ibO>N-6_XPBYxno@mD_3I#rDD?iADxX`! zh*v8^i*JEMzyN#bGEBz7;UYXki*Xr(9xXax(_1qVW=Ml)kSuvK$coq2A(5ZGhs_pF z$*w}FbN6+QDseuB9=fdp_MTs)nQf!2SlROQ!gBJBCXD&@-VurqHj0wm@LWX-TDmS= z71M__vAok|@!qgi#H&H%Vg-((ZfxPAL8AI{x|VV!9)ZE}_l>iWk8UPTGHs*?u7RfP z5MC&=c6X;XlUzrz5q?(!eO@~* zoh2I*%J7dF!!_!vXoSIn5o|wj1#_>K*&CIn{qSaRc&iFVxt*^20ngCL;QonIS>I5^ zMw8HXm>W0PGd*}Ko)f|~dDd%;Wu_RWI_d;&2g6R3S63Uzjd7dn%Svu-OKpx*o|N>F zZg=-~qLb~VRLpv`k zWSdfHh@?dp=s_X`{yxOlxE$4iuyS;Z-x!*E6eqmEm*j2bE@=ZI0YZ5%Yj29!5+J$4h{s($nakA`xgbO8w zi=*r}PWz#lTL_DSAu1?f%-2OjD}NHXp4pXOsCW;DS@BC3h-q4_l`<))8WgzkdXg3! zs1WMt32kS2E#L0p_|x+x**TFV=gn`m9BWlzF{b%6j-odf4{7a4y4Uaef@YaeuPhU8 zHBvRqN^;$Jizy+ z=zW{E5<>2gp$pH{M@S*!sJVQU)b*J5*bX4h>5VJve#Q6ga}cQ&iL#=(u+KroWrxa%8&~p{WEUF0il=db;-$=A;&9M{Rq`ouZ5m%BHT6%st%saGsD6)fQgLN}x@d3q>FC;=f%O3Cyg=Ke@Gh`XW za@RajqOE9UB6eE=zhG%|dYS)IW)&y&Id2n7r)6p_)vlRP7NJL(x4UbhlcFXWT8?K=%s7;z?Vjts?y2+r|uk8Wt(DM*73^W%pAkZa1Jd zNoE)8FvQA>Z`eR5Z@Ig6kS5?0h;`Y&OL2D&xnnAUzQz{YSdh0k zB3exx%A2TyI)M*EM6htrxSlep!Kk(P(VP`$p0G~f$smld6W1r_Z+o?=IB@^weq>5VYsYZZR@` z&XJFxd5{|KPZmVOSxc@^%71C@;z}}WhbF9p!%yLj3j%YOlPL5s>7I3vj25 z@xmf=*z%Wb4;Va6SDk9cv|r*lhZ`(y_*M@>q;wrn)oQx%B(2A$9(74>;$zmQ!4fN; z>XurIk-7@wZys<+7XL@0Fhe-f%*=(weaQEdR9Eh6>Kl-EcI({qoZqyzziGwpg-GM#251sK_ z=3|kitS!j%;fpc@oWn65SEL73^N&t>Ix37xgs= zYG%eQDJc|rqHFia0!_sm7`@lvcv)gfy(+KXA@E{3t1DaZ$DijWAcA)E0@X?2ziJ{v z&KOYZ|DdkM{}t+@{@*6ge}m%xfjIxi%qh`=^2Rwz@w0cCvZ&Tc#UmCDbVwABrON^x zEBK43FO@weA8s7zggCOWhMvGGE`baZ62cC)VHyy!5Zbt%ieH+XN|OLbAFPZWyC6)p z4P3%8sq9HdS3=ih^0OOlqTPbKuzQ?lBEI{w^ReUO{V?@`ARsL|S*%yOS=Z%sF)>-y z(LAQdhgAcuF6LQjRYfdbD1g4o%tV4EiK&ElLB&^VZHbrV1K>tHTO{#XTo>)2UMm`2 z^t4s;vnMQgf-njU-RVBRw0P0-m#d-u`(kq7NL&2T)TjI_@iKuPAK-@oH(J8?%(e!0Ir$yG32@CGUPn5w4)+9@8c&pGx z+K3GKESI4*`tYlmMHt@br;jBWTei&(a=iYslc^c#RU3Q&sYp zSG){)V<(g7+8W!Wxeb5zJb4XE{I|&Y4UrFWr%LHkdQ;~XU zgy^dH-Z3lmY+0G~?DrC_S4@=>0oM8Isw%g(id10gWkoz2Q%7W$bFk@mIzTCcIB(K8 zc<5h&ZzCdT=9n-D>&a8vl+=ZF*`uTvQviG_bLde*k>{^)&0o*b05x$MO3gVLUx`xZ z43j+>!u?XV)Yp@MmG%Y`+COH2?nQcMrQ%k~6#O%PeD_WvFO~Kct za4XoCM_X!c5vhRkIdV=xUB3xI2NNStK*8_Zl!cFjOvp-AY=D;5{uXj}GV{LK1~IE2 z|KffUiBaStRr;10R~K2VVtf{TzM7FaPm;Y(zQjILn+tIPSrJh&EMf6evaBKIvi42-WYU9Vhj~3< zZSM-B;E`g_o8_XTM9IzEL=9Lb^SPhe(f(-`Yh=X6O7+6ALXnTcUFpI>ekl6v)ZQeNCg2 z^H|{SKXHU*%nBQ@I3It0m^h+6tvI@FS=MYS$ZpBaG7j#V@P2ZuYySbp@hA# ze(kc;P4i_-_UDP?%<6>%tTRih6VBgScKU^BV6Aoeg6Uh(W^#J^V$Xo^4#Ekp ztqQVK^g9gKMTHvV7nb64UU7p~!B?>Y0oFH5T7#BSW#YfSB@5PtE~#SCCg3p^o=NkMk$<8- z6PT*yIKGrvne7+y3}_!AC8NNeI?iTY(&nakN>>U-zT0wzZf-RuyZk^X9H-DT_*wk= z;&0}6LsGtfVa1q)CEUPlx#(ED@-?H<1_FrHU#z5^P3lEB|qsxEyn%FOpjx z3S?~gvoXy~L(Q{Jh6*i~=f%9kM1>RGjBzQh_SaIDfSU_9!<>*Pm>l)cJD@wlyxpBV z4Fmhc2q=R_wHCEK69<*wG%}mgD1=FHi4h!98B-*vMu4ZGW~%IrYSLGU{^TuseqVgV zLP<%wirIL`VLyJv9XG_p8w@Q4HzNt-o;U@Au{7%Ji;53!7V8Rv0^Lu^Vf*sL>R(;c zQG_ZuFl)Mh-xEIkGu}?_(HwkB2jS;HdPLSxVU&Jxy9*XRG~^HY(f0g8Q}iqnVmgjI zfd=``2&8GsycjR?M%(zMjn;tn9agcq;&rR!Hp z$B*gzHsQ~aXw8c|a(L^LW(|`yGc!qOnV(ZjU_Q-4z1&0;jG&vAKuNG=F|H?@m5^N@ zq{E!1n;)kNTJ>|Hb2ODt-7U~-MOIFo%9I)_@7fnX+eMMNh>)V$IXesJpBn|uo8f~#aOFytCT zf9&%MCLf8mp4kwHTcojWmM3LU=#|{3L>E}SKwOd?%{HogCZ_Z1BSA}P#O(%H$;z7XyJ^sjGX;j5 zrzp>|Ud;*&VAU3x#f{CKwY7Vc{%TKKqmB@oTHA9;>?!nvMA;8+Jh=cambHz#J18x~ zs!dF>$*AnsQ{{82r5Aw&^7eRCdvcgyxH?*DV5(I$qXh^zS>us*I66_MbL8y4d3ULj z{S(ipo+T3Ag!+5`NU2sc+@*m{_X|&p#O-SAqF&g_n7ObB82~$p%fXA5GLHMC+#qqL zdt`sJC&6C2)=juQ_!NeD>U8lDVpAOkW*khf7MCcs$A(wiIl#B9HM%~GtQ^}yBPjT@ z+E=|A!Z?A(rwzZ;T}o6pOVqHzTr*i;Wrc%&36kc@jXq~+w8kVrs;%=IFdACoLAcCAmhFNpbP8;s`zG|HC2Gv?I~w4ITy=g$`0qMQdkijLSOtX6xW%Z9Nw<;M- zMN`c7=$QxN00DiSjbVt9Mi6-pjv*j(_8PyV-il8Q-&TwBwH1gz1uoxs6~uU}PrgWB zIAE_I-a1EqlIaGQNbcp@iI8W1sm9fBBNOk(k&iLBe%MCo#?xI$%ZmGA?=)M9D=0t7 zc)Q0LnI)kCy{`jCGy9lYX%mUsDWwsY`;jE(;Us@gmWPqjmXL+Hu#^;k%eT>{nMtzj zsV`Iy6leTA8-PndszF;N^X@CJrTw5IIm!GPeu)H2#FQitR{1p;MasQVAG3*+=9FYK zw*k!HT(YQorfQj+1*mCV458(T5=fH`um$gS38hw(OqVMyunQ;rW5aPbF##A3fGH6h z@W)i9Uff?qz`YbK4c}JzQpuxuE3pcQO)%xBRZp{zJ^-*|oryTxJ-rR+MXJ)!f=+pp z10H|DdGd2exhi+hftcYbM0_}C0ZI-2vh+$fU1acsB-YXid7O|=9L!3e@$H*6?G*Zp z%qFB(sgl=FcC=E4CYGp4CN>=M8#5r!RU!u+FJVlH6=gI5xHVD&k;Ta*M28BsxfMV~ zLz+@6TxnfLhF@5=yQo^1&S}cmTN@m!7*c6z;}~*!hNBjuE>NLVl2EwN!F+)0$R1S! zR|lF%n!9fkZ@gPW|x|B={V6x3`=jS*$Pu0+5OWf?wnIy>Y1MbbGSncpKO0qE(qO=ts z!~@&!N`10S593pVQu4FzpOh!tvg}p%zCU(aV5=~K#bKi zHdJ1>tQSrhW%KOky;iW+O_n;`l9~omqM%sdxdLtI`TrJzN6BQz+7xOl*rM>xVI2~# z)7FJ^Dc{DC<%~VS?@WXzuOG$YPLC;>#vUJ^MmtbSL`_yXtNKa$Hk+l-c!aC7gn(Cg ze?YPYZ(2Jw{SF6MiO5(%_pTo7j@&DHNW`|lD`~{iH+_eSTS&OC*2WTT*a`?|9w1dh zh1nh@$a}T#WE5$7Od~NvSEU)T(W$p$s5fe^GpG+7fdJ9=enRT9$wEk+ZaB>G3$KQO zgq?-rZZnIv!p#>Ty~}c*Lb_jxJg$eGM*XwHUwuQ|o^}b3^T6Bxx{!?va8aC@-xK*H ztJBFvFfsSWu89%@b^l3-B~O!CXs)I6Y}y#0C0U0R0WG zybjroj$io0j}3%P7zADXOwHwafT#uu*zfM!oD$6aJx7+WL%t-@6^rD_a_M?S^>c;z zMK580bZXo1f*L$CuMeM4Mp!;P@}b~$cd(s5*q~FP+NHSq;nw3fbWyH)i2)-;gQl{S zZO!T}A}fC}vUdskGSq&{`oxt~0i?0xhr6I47_tBc`fqaSrMOzR4>0H^;A zF)hX1nfHs)%Zb-(YGX;=#2R6C{BG;k=?FfP?9{_uFLri~-~AJ;jw({4MU7e*d)?P@ zXX*GkNY9ItFjhwgAIWq7Y!ksbMzfqpG)IrqKx9q{zu%Mdl+{Dis#p9q`02pr1LG8R z@As?eG!>IoROgS!@J*to<27coFc1zpkh?w=)h9CbYe%^Q!Ui46Y*HO0mr% zEff-*$ndMNw}H2a5@BsGj5oFfd!T(F&0$<{GO!Qdd?McKkorh=5{EIjDTHU`So>8V zBA-fqVLb2;u7UhDV1xMI?y>fe3~4urv3%PX)lDw+HYa;HFkaLqi4c~VtCm&Ca+9C~ zge+67hp#R9`+Euq59WhHX&7~RlXn=--m8$iZ~~1C8cv^2(qO#X0?vl91gzUKBeR1J z^p4!!&7)3#@@X&2aF2-)1Ffcc^F8r|RtdL2X%HgN&XU-KH2SLCbpw?J5xJ*!F-ypZ zMG%AJ!Pr&}`LW?E!K~=(NJxuSVTRCGJ$2a*Ao=uUDSys!OFYu!Vs2IT;xQ6EubLIl z+?+nMGeQQhh~??0!s4iQ#gm3!BpMpnY?04kK375e((Uc7B3RMj;wE?BCoQGu=UlZt!EZ1Q*auI)dj3Jj{Ujgt zW5hd~-HWBLI_3HuO) zNrb^XzPsTIb=*a69wAAA3J6AAZZ1VsYbIG}a`=d6?PjM)3EPaDpW2YP$|GrBX{q*! z$KBHNif)OKMBCFP5>!1d=DK>8u+Upm-{hj5o|Wn$vh1&K!lVfDB&47lw$tJ?d5|=B z^(_9=(1T3Fte)z^>|3**n}mIX;mMN5v2F#l(q*CvU{Ga`@VMp#%rQkDBy7kYbmb-q z<5!4iuB#Q_lLZ8}h|hPODI^U6`gzLJre9u3k3c#%86IKI*^H-@I48Bi*@avYm4v!n0+v zWu{M{&F8#p9cx+gF0yTB_<2QUrjMPo9*7^-uP#~gGW~y3nfPAoV%amgr>PSyVAd@l)}8#X zR5zV6t*uKJZL}?NYvPVK6J0v4iVpwiN|>+t3aYiZSp;m0!(1`bHO}TEtWR1tY%BPB z(W!0DmXbZAsT$iC13p4f>u*ZAy@JoLAkJhzFf1#4;#1deO8#8d&89}en&z!W&A3++^1(;>0SB1*54d@y&9Pn;^IAf3GiXbfT`_>{R+Xv; zQvgL>+0#8-laO!j#-WB~(I>l0NCMt_;@Gp_f0#^c)t?&#Xh1-7RR0@zPyBz!U#0Av zT?}n({(p?p7!4S2ZBw)#KdCG)uPnZe+U|0{BW!m)9 zi_9$F?m<`2!`JNFv+w8MK_K)qJ^aO@7-Ig>cM4-r0bi=>?B_2mFNJ}aE3<+QCzRr*NA!QjHw# z`1OsvcoD0?%jq{*7b!l|L1+Tw0TTAM4XMq7*ntc-Ived>Sj_ZtS|uVdpfg1_I9knY z2{GM_j5sDC7(W&}#s{jqbybqJWyn?{PW*&cQIU|*v8YGOKKlGl@?c#TCnmnAkAzV- zmK={|1G90zz=YUvC}+fMqts0d4vgA%t6Jhjv?d;(Z}(Ep8fTZfHA9``fdUHkA+z3+ zhh{ohP%Bj?T~{i0sYCQ}uC#5BwN`skI7`|c%kqkyWIQ;!ysvA8H`b-t()n6>GJj6xlYDu~8qX{AFo$Cm3d|XFL=4uvc?Keb zzb0ZmMoXca6Mob>JqkNuoP>B2Z>D`Q(TvrG6m`j}-1rGP!g|qoL=$FVQYxJQjFn33lODt3Wb1j8VR zlR++vIT6^DtYxAv_hxupbLLN3e0%A%a+hWTKDV3!Fjr^cWJ{scsAdfhpI)`Bms^M6 zQG$waKgFr=c|p9Piug=fcJvZ1ThMnNhQvBAg-8~b1?6wL*WyqXhtj^g(Ke}mEfZVM zJuLNTUVh#WsE*a6uqiz`b#9ZYg3+2%=C(6AvZGc=u&<6??!slB1a9K)=VL zY9EL^mfyKnD zSJyYBc_>G;5RRnrNgzJz#Rkn3S1`mZgO`(r5;Hw6MveN(URf_XS-r58Cn80K)ArH4 z#Rrd~LG1W&@ttw85cjp8xV&>$b%nSXH_*W}7Ch2pg$$c0BdEo-HWRTZcxngIBJad> z;C>b{jIXjb_9Jis?NZJsdm^EG}e*pR&DAy0EaSGi3XWTa(>C%tz1n$u?5Fb z1qtl?;_yjYo)(gB^iQq?=jusF%kywm?CJP~zEHi0NbZ);$(H$w(Hy@{i>$wcVRD_X|w-~(0Z9BJyh zhNh;+eQ9BEIs;tPz%jSVnfCP!3L&9YtEP;svoj_bNzeGSQIAjd zBss@A;)R^WAu-37RQrM%{DfBNRx>v!G31Z}8-El9IOJlb_MSoMu2}GDYycNaf>uny z+8xykD-7ONCM!APry_Lw6-yT>5!tR}W;W`C)1>pxSs5o1z#j7%m=&=7O4hz+Lsqm` z*>{+xsabZPr&X=}G@obTb{nPTkccJX8w3CG7X+1+t{JcMabv~UNv+G?txRqXib~c^Mo}`q{$`;EBNJ;#F*{gvS12kV?AZ%O0SFB$^ zn+}!HbmEj}w{Vq(G)OGAzH}R~kS^;(-s&=ectz8vN!_)Yl$$U@HNTI-pV`LSj7Opu zTZ5zZ)-S_{GcEQPIQXLQ#oMS`HPu{`SQiAZ)m1at*Hy%3xma|>o`h%E%8BEbi9p0r zVjcsh<{NBKQ4eKlXU|}@XJ#@uQw*$4BxKn6#W~I4T<^f99~(=}a`&3(ur8R9t+|AQ zWkQx7l}wa48-jO@ft2h+7qn%SJtL%~890FG0s5g*kNbL3I&@brh&f6)TlM`K^(bhr zJWM6N6x3flOw$@|C@kPi7yP&SP?bzP-E|HSXQXG>7gk|R9BTj`e=4de9C6+H7H7n# z#GJeVs1mtHhLDmVO?LkYRQc`DVOJ_vdl8VUihO-j#t=0T3%Fc1f9F73ufJz*adn*p zc%&vi(4NqHu^R>sAT_0EDjVR8bc%wTz#$;%NU-kbDyL_dg0%TFafZwZ?5KZpcuaO54Z9hX zD$u>q!-9`U6-D`E#`W~fIfiIF5_m6{fvM)b1NG3xf4Auw;Go~Fu7cth#DlUn{@~yu z=B;RT*dp?bO}o%4x7k9v{r=Y@^YQ^UUm(Qmliw8brO^=NP+UOohLYiaEB3^DB56&V zK?4jV61B|1Uj_5fBKW;8LdwOFZKWp)g{B%7g1~DgO&N& z#lisxf?R~Z@?3E$Mms$$JK8oe@X`5m98V*aV6Ua}8Xs2#A!{x?IP|N(%nxsH?^c{& z@vY&R1QmQs83BW28qAmJfS7MYi=h(YK??@EhjL-t*5W!p z^gYX!Q6-vBqcv~ruw@oMaU&qp0Fb(dbVzm5xJN%0o_^@fWq$oa3X?9s%+b)x4w-q5Koe(@j6Ez7V@~NRFvd zfBH~)U5!ix3isg`6be__wBJp=1@yfsCMw1C@y+9WYD9_C%{Q~7^0AF2KFryfLlUP# zwrtJEcH)jm48!6tUcxiurAMaiD04C&tPe6DI0#aoqz#Bt0_7_*X*TsF7u*zv(iEfA z;$@?XVu~oX#1YXtceQL{dSneL&*nDug^OW$DSLF0M1Im|sSX8R26&)<0Fbh^*l6!5wfSu8MpMoh=2l z^^0Sr$UpZp*9oqa23fcCfm7`ya2<4wzJ`Axt7e4jJrRFVf?nY~2&tRL* zd;6_njcz01c>$IvN=?K}9ie%Z(BO@JG2J}fT#BJQ+f5LFSgup7i!xWRKw6)iITjZU z%l6hPZia>R!`aZjwCp}I zg)%20;}f+&@t;(%5;RHL>K_&7MH^S+7<|(SZH!u zznW|jz$uA`P9@ZWtJgv$EFp>)K&Gt+4C6#*khZQXS*S~6N%JDT$r`aJDs9|uXWdbg zBwho$phWx}x!qy8&}6y5Vr$G{yGSE*r$^r{}pw zVTZKvikRZ`J_IJrjc=X1uw?estdwm&bEahku&D04HD+0Bm~q#YGS6gp!KLf$A{%Qd z&&yX@Hp>~(wU{|(#U&Bf92+1i&Q*-S+=y=3pSZy$#8Uc$#7oiJUuO{cE6=tsPhwPe| zxQpK>`Dbka`V)$}e6_OXKLB%i76~4N*zA?X+PrhH<&)}prET;kel24kW%+9))G^JI zsq7L{P}^#QsZViX%KgxBvEugr>ZmFqe^oAg?{EI=&_O#e)F3V#rc z8$4}0Zr19qd3tE4#$3_f=Bbx9oV6VO!d3(R===i-7p=Vj`520w0D3W6lQfY48}!D* z&)lZMG;~er2qBoI2gsX+Ts-hnpS~NYRDtPd^FPzn!^&yxRy#CSz(b&E*tL|jIkq|l zf%>)7Dtu>jCf`-7R#*GhGn4FkYf;B$+9IxmqH|lf6$4irg{0ept__%)V*R_OK=T06 zyT_m-o@Kp6U{l5h>W1hGq*X#8*y@<;vsOFqEjTQXFEotR+{3}ODDnj;o0@!bB5x=N z394FojuGOtVKBlVRLtHp%EJv_G5q=AgF)SKyRN5=cGBjDWv4LDn$IL`*=~J7u&Dy5 zrMc83y+w^F&{?X(KOOAl-sWZDb{9X9#jrQtmrEXD?;h-}SYT7yM(X_6qksM=K_a;Z z3u0qT0TtaNvDER_8x*rxXw&C^|h{P1qxK|@pS7vdlZ#P z7PdB7MmC2}%sdzAxt>;WM1s0??`1983O4nFK|hVAbHcZ3x{PzytQLkCVk7hA!Lo` zEJH?4qw|}WH{dc4z%aB=0XqsFW?^p=X}4xnCJXK%c#ItOSjdSO`UXJyuc8bh^Cf}8 z@Ht|vXd^6{Fgai8*tmyRGmD_s_nv~r^Fy7j`Bu`6=G)5H$i7Q7lvQnmea&TGvJp9a|qOrUymZ$6G|Ly z#zOCg++$3iB$!6!>215A4!iryregKuUT344X)jQb3|9qY>c0LO{6Vby05n~VFzd?q zgGZv&FGlkiH*`fTurp>B8v&nSxNz)=5IF$=@rgND4d`!AaaX;_lK~)-U8la_Wa8i?NJC@BURO*sUW)E9oyv3RG^YGfN%BmxzjlT)bp*$<| zX3tt?EAy<&K+bhIuMs-g#=d1}N_?isY)6Ay$mDOKRh z4v1asEGWoAp=srraLW^h&_Uw|6O+r;wns=uwYm=JN4Q!quD8SQRSeEcGh|Eb5Jg8m zOT}u;N|x@aq)=&;wufCc^#)5U^VcZw;d_wwaoh9$p@Xrc{DD6GZUqZ ziC6OT^zSq@-lhbgR8B+e;7_Giv;DK5gn^$bs<6~SUadiosfewWDJu`XsBfOd1|p=q zE>m=zF}!lObA%ePey~gqU8S6h-^J2Y?>7)L2+%8kV}Gp=h`Xm_}rlm)SyUS=`=S7msKu zC|T!gPiI1rWGb1z$Md?0YJQ;%>uPLOXf1Z>N~`~JHJ!^@D5kSXQ4ugnFZ>^`zH8CAiZmp z6Ms|#2gcGsQ{{u7+Nb9sA?U>(0e$5V1|WVwY`Kn)rsnnZ4=1u=7u!4WexZD^IQ1Jk zfF#NLe>W$3m&C^ULjdw+5|)-BSHwpegdyt9NYC{3@QtMfd8GrIWDu`gd0nv-3LpGCh@wgBaG z176tikL!_NXM+Bv#7q^cyn9$XSeZR6#!B4JE@GVH zoobHZN_*RF#@_SVYKkQ_igme-Y5U}cV(hkR#k1c{bQNMji zU7aE`?dHyx=1`kOYZo_8U7?3-7vHOp`Qe%Z*i+FX!s?6huNp0iCEW-Z7E&jRWmUW_ z67j>)Ew!yq)hhG4o?^z}HWH-e=es#xJUhDRc4B51M4~E-l5VZ!&zQq`gWe`?}#b~7w1LH4Xa-UCT5LXkXQWheBa2YJYbyQ zl1pXR%b(KCXMO0OsXgl0P0Og<{(@&z1aokU-Pq`eQq*JYgt8xdFQ6S z6Z3IFSua8W&M#`~*L#r>Jfd6*BzJ?JFdBR#bDv$_0N!_5vnmo@!>vULcDm`MFU823 zpG9pqjqz^FE5zMDoGqhs5OMmC{Y3iVcl>F}5Rs24Y5B^mYQ;1T&ks@pIApHOdrzXF z-SdX}Hf{X;TaSxG_T$0~#RhqKISGKNK47}0*x&nRIPtmdwxc&QT3$8&!3fWu1eZ_P zJveQj^hJL#Sn!*4k`3}(d(aasl&7G0j0-*_2xtAnoX1@9+h zO#c>YQg60Z;o{Bi=3i7S`Ic+ZE>K{(u|#)9y}q*j8uKQ1^>+(BI}m%1v3$=4ojGBc zm+o1*!T&b}-lVvZqIUBc8V}QyFEgm#oyIuC{8WqUNV{Toz`oxhYpP!_p2oHHh5P@iB*NVo~2=GQm+8Yrkm2Xjc_VyHg1c0>+o~@>*Qzo zHVBJS>$$}$_4EniTI;b1WShX<5-p#TPB&!;lP!lBVBbLOOxh6FuYloD%m;n{r|;MU3!q4AVkua~fieeWu2 zQAQ$ue(IklX6+V;F1vCu-&V?I3d42FgWgsb_e^29ol}HYft?{SLf>DrmOp9o!t>I^ zY7fBCk+E8n_|apgM|-;^=#B?6RnFKlN`oR)`e$+;D=yO-(U^jV;rft^G_zl`n7qnM zL z*-Y4Phq+ZI1$j$F-f;`CD#|`-T~OM5Q>x}a>B~Gb3-+9i>Lfr|Ca6S^8g*{*?_5!x zH_N!SoRP=gX1?)q%>QTY!r77e2j9W(I!uAz{T`NdNmPBBUzi2{`XMB^zJGGwFWeA9 z{fk33#*9SO0)DjROug+(M)I-pKA!CX;IY(#gE!UxXVsa)X!UftIN98{pt#4MJHOhY zM$_l}-TJlxY?LS6Nuz1T<44m<4i^8k@D$zuCPrkmz@sdv+{ciyFJG2Zwy&%c7;atIeTdh!a(R^QXnu1Oq1b42*OQFWnyQ zWeQrdvP|w_idy53Wa<{QH^lFmEd+VlJkyiC>6B#s)F;w-{c;aKIm;Kp50HnA-o3lY z9B~F$gJ@yYE#g#X&3ADx&tO+P_@mnQTz9gv30_sTsaGXkfNYXY{$(>*PEN3QL>I!k zp)KibPhrfX3%Z$H6SY`rXGYS~143wZrG2;=FLj50+VM6soI~up_>fU(2Wl@{BRsMi zO%sL3x?2l1cXTF)k&moNsHfQrQ+wu(gBt{sk#CU=UhrvJIncy@tJX5klLjgMn>~h= zg|FR&;@eh|C7`>s_9c~0-{IAPV){l|Ts`i=)AW;d9&KPc3fMeoTS%8@V~D8*h;&(^>yjT84MM}=%#LS7shLAuuj(0VAYoozhWjq z4LEr?wUe2^WGwdTIgWBkDUJa>YP@5d9^Rs$kCXmMRxuF*YMVrn?0NFyPl}>`&dqZb z<5eqR=ZG3>n2{6v6BvJ`YBZeeTtB88TAY(x0a58EWyuf>+^|x8Qa6wA|1Nb_p|nA zWWa}|z8a)--Wj`LqyFk_a3gN2>5{Rl_wbW?#by7&i*^hRknK%jwIH6=dQ8*-_{*x0j^DUfMX0`|K@6C<|1cgZ~D(e5vBFFm;HTZF(!vT8=T$K+|F)x3kqzBV4-=p1V(lzi(s7jdu0>LD#N=$Lk#3HkG!a zIF<7>%B7sRNzJ66KrFV76J<2bdYhxll0y2^_rdG=I%AgW4~)1Nvz=$1UkE^J%BxLo z+lUci`UcU062os*=`-j4IfSQA{w@y|3}Vk?i;&SSdh8n+$iHA#%ERL{;EpXl6u&8@ zzg}?hkEOUOJt?ZL=pWZFJ19mI1@P=$U5*Im1e_8Z${JsM>Ov?nh8Z zP5QvI!{Jy@&BP48%P2{Jr_VgzW;P@7)M9n|lDT|Ep#}7C$&ud&6>C^5ZiwKIg2McPU(4jhM!BD@@L(Gd*Nu$ji(ljZ<{FIeW_1Mmf;76{LU z-ywN~=uNN)Xi6$<12A9y)K%X|(W0p|&>>4OXB?IiYr||WKDOJPxiSe01NSV-h24^L z_>m$;|C+q!Mj**-qQ$L-*++en(g|hw;M!^%_h-iDjFHLo-n3JpB;p?+o2;`*jpvJU zLY^lt)Un4joij^^)O(CKs@7E%*!w>!HA4Q?0}oBJ7Nr8NQ7QmY^4~jvf0-`%waOLn zdNjAPaC0_7c|RVhw)+71NWjRi!y>C+Bl;Z`NiL^zn2*0kmj5gyhCLCxts*cWCdRI| zjsd=sT5BVJc^$GxP~YF$-U{-?kW6r@^vHXB%{CqYzU@1>dzf#3SYedJG-Rm6^RB7s zGM5PR(yKPKR)>?~vpUIeTP7A1sc8-knnJk*9)3t^e%izbdm>Y=W{$wm(cy1RB-19i za#828DMBY+ps#7Y8^6t)=Ea@%Nkt)O6JCx|ybC;Ap}Z@Zw~*}3P>MZLPb4Enxz9Wf zssobT^(R@KuShj8>@!1M7tm|2%-pYYDxz-5`rCbaTCG5{;Uxm z*g=+H1X8{NUvFGzz~wXa%Eo};I;~`37*WrRU&K0dPSB$yk(Z*@K&+mFal^?c zurbqB-+|Kb5|sznT;?Pj!+kgFY1#Dr;_%A(GIQC{3ct|{*Bji%FNa6c-thbpBkA;U zURV!Dr&X{0J}iht#-Qp2=xzuh(fM>zRoiGrYl5ttw2#r34gC41CCOC31m~^UPTK@s z6;A@)7O7_%C)>bnAXerYuAHdE93>j2N}H${zEc6&SbZ|-fiG*-qtGuy-qDelH(|u$ zorf8_T6Zqe#Ub!+e3oSyrskt_HyW_^5lrWt#30l)tHk|j$@YyEkXUOV;6B51L;M@=NIWZXU;GrAa(LGxO%|im%7F<-6N;en0Cr zLH>l*y?pMwt`1*cH~LdBPFY_l;~`N!Clyfr;7w<^X;&(ZiVdF1S5e(+Q%60zgh)s4 zn2yj$+mE=miVERP(g8}G4<85^-5f@qxh2ec?n+$A_`?qN=iyT1?U@t?V6DM~BIlBB z>u~eXm-aE>R0sQy!-I4xtCNi!!qh?R1!kKf6BoH2GG{L4%PAz0{Sh6xpuyI%*~u)s z%rLuFl)uQUCBQAtMyN;%)zFMx4loh7uTfKeB2Xif`lN?2gq6NhWhfz0u5WP9J>=V2 zo{mLtSy&BA!mSzs&CrKWq^y40JF5a&GSXIi2= z{EYb59J4}VwikL4P=>+mc6{($FNE@e=VUwG+KV21;<@lrN`mnz5jYGASyvz7BOG_6(p^eTxD-4O#lROgon;R35=|nj#eHIfJBYPWG>H>`dHKCDZ3`R{-?HO0mE~(5_WYcFmp8sU?wr*UkAQiNDGc6T zA%}GOLXlOWqL?WwfHO8MB#8M8*~Y*gz;1rWWoVSXP&IbKxbQ8+s%4Jnt?kDsq7btI zCDr0PZ)b;B%!lu&CT#RJzm{l{2fq|BcY85`w~3LSK<><@(2EdzFLt9Y_`;WXL6x`0 zDoQ?=?I@Hbr;*VVll1Gmd8*%tiXggMK81a+T(5Gx6;eNb8=uYn z5BG-0g>pP21NPn>$ntBh>`*})Fl|38oC^9Qz>~MAazH%3Q~Qb!ALMf$srexgPZ2@&c~+hxRi1;}+)-06)!#Mq<6GhP z-Q?qmgo${aFBApb5p}$1OJKTClfi8%PpnczyVKkoHw7Ml9e7ikrF0d~UB}i3vizos zXW4DN$SiEV9{faLt5bHy2a>33K%7Td-n5C*N;f&ZqAg#2hIqEb(y<&f4u5BWJ>2^4 z414GosL=Aom#m&=x_v<0-fp1r%oVJ{T-(xnomNJ(Dryv zh?vj+%=II_nV+@NR+(!fZZVM&(W6{6%9cm+o+Z6}KqzLw{(>E86uA1`_K$HqINlb1 zKelh3-jr2I9V?ych`{hta9wQ2c9=MM`2cC{m6^MhlL2{DLv7C^j z$xXBCnDl_;l|bPGMX@*tV)B!c|4oZyftUlP*?$YU9C_eAsuVHJ58?)zpbr30P*C`T z7y#ao`uE-SOG(Pi+`$=e^mle~)pRrdwL5)N;o{gpW21of(QE#U6w%*C~`v-z0QqBML!!5EeYA5IQB0 z^l01c;L6E(iytN!LhL}wfwP7W9PNAkb+)Cst?qg#$n;z41O4&v+8-zPs+XNb-q zIeeBCh#ivnFLUCwfS;p{LC0O7tm+Sf9Jn)~b%uwP{%69;QC)Ok0t%*a5M+=;y8j=v z#!*pp$9@!x;UMIs4~hP#pnfVc!%-D<+wsG@R2+J&%73lK|2G!EQC)O05TCV=&3g)C!lT=czLpZ@Sa%TYuoE?v8T8`V;e$#Zf2_Nj6nvBgh1)2 GZ~q4|mN%#X literal 0 HcmV?d00001 diff --git a/kotlin/gradle/wrapper/gradle-wrapper.properties b/kotlin/gradle/wrapper/gradle-wrapper.properties new file mode 100644 index 0000000..df97d72 --- /dev/null +++ b/kotlin/gradle/wrapper/gradle-wrapper.properties @@ -0,0 +1,7 @@ +distributionBase=GRADLE_USER_HOME +distributionPath=wrapper/dists +distributionUrl=https\://services.gradle.org/distributions/gradle-8.10.2-bin.zip +networkTimeout=10000 +validateDistributionUrl=true +zipStoreBase=GRADLE_USER_HOME +zipStorePath=wrapper/dists diff --git a/kotlin/gradlew b/kotlin/gradlew new file mode 100755 index 0000000..f5feea6 --- /dev/null +++ b/kotlin/gradlew @@ -0,0 +1,252 @@ +#!/bin/sh + +# +# Copyright © 2015-2021 the original authors. +# +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# https://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. +# +# SPDX-License-Identifier: Apache-2.0 +# + +############################################################################## +# +# Gradle start up script for POSIX generated by Gradle. +# +# Important for running: +# +# (1) You need a POSIX-compliant shell to run this script. If your /bin/sh is +# noncompliant, but you have some other compliant shell such as ksh or +# bash, then to run this script, type that shell name before the whole +# command line, like: +# +# ksh Gradle +# +# Busybox and similar reduced shells will NOT work, because this script +# requires all of these POSIX shell features: +# * functions; +# * expansions «$var», «${var}», «${var:-default}», «${var+SET}», +# «${var#prefix}», «${var%suffix}», and «$( cmd )»; +# * compound commands having a testable exit status, especially «case»; +# * various built-in commands including «command», «set», and «ulimit». +# +# Important for patching: +# +# (2) This script targets any POSIX shell, so it avoids extensions provided +# by Bash, Ksh, etc; in particular arrays are avoided. +# +# The "traditional" practice of packing multiple parameters into a +# space-separated string is a well documented source of bugs and security +# problems, so this is (mostly) avoided, by progressively accumulating +# options in "$@", and eventually passing that to Java. +# +# Where the inherited environment variables (DEFAULT_JVM_OPTS, JAVA_OPTS, +# and GRADLE_OPTS) rely on word-splitting, this is performed explicitly; +# see the in-line comments for details. +# +# There are tweaks for specific operating systems such as AIX, CygWin, +# Darwin, MinGW, and NonStop. +# +# (3) This script is generated from the Groovy template +# https://github.com/gradle/gradle/blob/HEAD/platforms/jvm/plugins-application/src/main/resources/org/gradle/api/internal/plugins/unixStartScript.txt +# within the Gradle project. +# +# You can find Gradle at https://github.com/gradle/gradle/. +# +############################################################################## + +# Attempt to set APP_HOME + +# Resolve links: $0 may be a link +app_path=$0 + +# Need this for daisy-chained symlinks. +while + APP_HOME=${app_path%"${app_path##*/}"} # leaves a trailing /; empty if no leading path + [ -h "$app_path" ] +do + ls=$( ls -ld "$app_path" ) + link=${ls#*' -> '} + case $link in #( + /*) app_path=$link ;; #( + *) app_path=$APP_HOME$link ;; + esac +done + +# This is normally unused +# shellcheck disable=SC2034 +APP_BASE_NAME=${0##*/} +# Discard cd standard output in case $CDPATH is set (https://github.com/gradle/gradle/issues/25036) +APP_HOME=$( cd -P "${APP_HOME:-./}" > /dev/null && printf '%s +' "$PWD" ) || exit + +# Use the maximum available, or set MAX_FD != -1 to use that value. +MAX_FD=maximum + +warn () { + echo "$*" +} >&2 + +die () { + echo + echo "$*" + echo + exit 1 +} >&2 + +# OS specific support (must be 'true' or 'false'). +cygwin=false +msys=false +darwin=false +nonstop=false +case "$( uname )" in #( + CYGWIN* ) cygwin=true ;; #( + Darwin* ) darwin=true ;; #( + MSYS* | MINGW* ) msys=true ;; #( + NONSTOP* ) nonstop=true ;; +esac + +CLASSPATH=$APP_HOME/gradle/wrapper/gradle-wrapper.jar + + +# Determine the Java command to use to start the JVM. +if [ -n "$JAVA_HOME" ] ; then + if [ -x "$JAVA_HOME/jre/sh/java" ] ; then + # IBM's JDK on AIX uses strange locations for the executables + JAVACMD=$JAVA_HOME/jre/sh/java + else + JAVACMD=$JAVA_HOME/bin/java + fi + if [ ! -x "$JAVACMD" ] ; then + die "ERROR: JAVA_HOME is set to an invalid directory: $JAVA_HOME + +Please set the JAVA_HOME variable in your environment to match the +location of your Java installation." + fi +else + JAVACMD=java + if ! command -v java >/dev/null 2>&1 + then + die "ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH. + +Please set the JAVA_HOME variable in your environment to match the +location of your Java installation." + fi +fi + +# Increase the maximum file descriptors if we can. +if ! "$cygwin" && ! "$darwin" && ! "$nonstop" ; then + case $MAX_FD in #( + max*) + # In POSIX sh, ulimit -H is undefined. That's why the result is checked to see if it worked. + # shellcheck disable=SC2039,SC3045 + MAX_FD=$( ulimit -H -n ) || + warn "Could not query maximum file descriptor limit" + esac + case $MAX_FD in #( + '' | soft) :;; #( + *) + # In POSIX sh, ulimit -n is undefined. That's why the result is checked to see if it worked. + # shellcheck disable=SC2039,SC3045 + ulimit -n "$MAX_FD" || + warn "Could not set maximum file descriptor limit to $MAX_FD" + esac +fi + +# Collect all arguments for the java command, stacking in reverse order: +# * args from the command line +# * the main class name +# * -classpath +# * -D...appname settings +# * --module-path (only if needed) +# * DEFAULT_JVM_OPTS, JAVA_OPTS, and GRADLE_OPTS environment variables. + +# For Cygwin or MSYS, switch paths to Windows format before running java +if "$cygwin" || "$msys" ; then + APP_HOME=$( cygpath --path --mixed "$APP_HOME" ) + CLASSPATH=$( cygpath --path --mixed "$CLASSPATH" ) + + JAVACMD=$( cygpath --unix "$JAVACMD" ) + + # Now convert the arguments - kludge to limit ourselves to /bin/sh + for arg do + if + case $arg in #( + -*) false ;; # don't mess with options #( + /?*) t=${arg#/} t=/${t%%/*} # looks like a POSIX filepath + [ -e "$t" ] ;; #( + *) false ;; + esac + then + arg=$( cygpath --path --ignore --mixed "$arg" ) + fi + # Roll the args list around exactly as many times as the number of + # args, so each arg winds up back in the position where it started, but + # possibly modified. + # + # NB: a `for` loop captures its iteration list before it begins, so + # changing the positional parameters here affects neither the number of + # iterations, nor the values presented in `arg`. + shift # remove old arg + set -- "$@" "$arg" # push replacement arg + done +fi + + +# Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script. +DEFAULT_JVM_OPTS='"-Xmx64m" "-Xms64m"' + +# Collect all arguments for the java command: +# * DEFAULT_JVM_OPTS, JAVA_OPTS, JAVA_OPTS, and optsEnvironmentVar are not allowed to contain shell fragments, +# and any embedded shellness will be escaped. +# * For example: A user cannot expect ${Hostname} to be expanded, as it is an environment variable and will be +# treated as '${Hostname}' itself on the command line. + +set -- \ + "-Dorg.gradle.appname=$APP_BASE_NAME" \ + -classpath "$CLASSPATH" \ + org.gradle.wrapper.GradleWrapperMain \ + "$@" + +# Stop when "xargs" is not available. +if ! command -v xargs >/dev/null 2>&1 +then + die "xargs is not available" +fi + +# Use "xargs" to parse quoted args. +# +# With -n1 it outputs one arg per line, with the quotes and backslashes removed. +# +# In Bash we could simply go: +# +# readarray ARGS < <( xargs -n1 <<<"$var" ) && +# set -- "${ARGS[@]}" "$@" +# +# but POSIX shell has neither arrays nor command substitution, so instead we +# post-process each arg (as a line of input to sed) to backslash-escape any +# character that might be a shell metacharacter, then use eval to reverse +# that process (while maintaining the separation between arguments), and wrap +# the whole thing up as a single "set" statement. +# +# This will of course break if any of these variables contains a newline or +# an unmatched quote. +# + +eval "set -- $( + printf '%s\n' "$DEFAULT_JVM_OPTS $JAVA_OPTS $GRADLE_OPTS" | + xargs -n1 | + sed ' s~[^-[:alnum:]+,./:=@_]~\\&~g; ' | + tr '\n' ' ' + )" '"$@"' + +exec "$JAVACMD" "$@" diff --git a/kotlin/gradlew.bat b/kotlin/gradlew.bat new file mode 100644 index 0000000..9b42019 --- /dev/null +++ b/kotlin/gradlew.bat @@ -0,0 +1,94 @@ +@rem +@rem Copyright 2015 the original author or authors. +@rem +@rem Licensed under the Apache License, Version 2.0 (the "License"); +@rem you may not use this file except in compliance with the License. +@rem You may obtain a copy of the License at +@rem +@rem https://www.apache.org/licenses/LICENSE-2.0 +@rem +@rem Unless required by applicable law or agreed to in writing, software +@rem distributed under the License is distributed on an "AS IS" BASIS, +@rem WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +@rem See the License for the specific language governing permissions and +@rem limitations under the License. +@rem +@rem SPDX-License-Identifier: Apache-2.0 +@rem + +@if "%DEBUG%"=="" @echo off +@rem ########################################################################## +@rem +@rem Gradle startup script for Windows +@rem +@rem ########################################################################## + +@rem Set local scope for the variables with windows NT shell +if "%OS%"=="Windows_NT" setlocal + +set DIRNAME=%~dp0 +if "%DIRNAME%"=="" set DIRNAME=. +@rem This is normally unused +set APP_BASE_NAME=%~n0 +set APP_HOME=%DIRNAME% + +@rem Resolve any "." and ".." in APP_HOME to make it shorter. +for %%i in ("%APP_HOME%") do set APP_HOME=%%~fi + +@rem Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script. +set DEFAULT_JVM_OPTS="-Xmx64m" "-Xms64m" + +@rem Find java.exe +if defined JAVA_HOME goto findJavaFromJavaHome + +set JAVA_EXE=java.exe +%JAVA_EXE% -version >NUL 2>&1 +if %ERRORLEVEL% equ 0 goto execute + +echo. 1>&2 +echo ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH. 1>&2 +echo. 1>&2 +echo Please set the JAVA_HOME variable in your environment to match the 1>&2 +echo location of your Java installation. 1>&2 + +goto fail + +:findJavaFromJavaHome +set JAVA_HOME=%JAVA_HOME:"=% +set JAVA_EXE=%JAVA_HOME%/bin/java.exe + +if exist "%JAVA_EXE%" goto execute + +echo. 1>&2 +echo ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME% 1>&2 +echo. 1>&2 +echo Please set the JAVA_HOME variable in your environment to match the 1>&2 +echo location of your Java installation. 1>&2 + +goto fail + +:execute +@rem Setup the command line + +set CLASSPATH=%APP_HOME%\gradle\wrapper\gradle-wrapper.jar + + +@rem Execute Gradle +"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %* + +:end +@rem End local scope for the variables with windows NT shell +if %ERRORLEVEL% equ 0 goto mainEnd + +:fail +rem Set variable GRADLE_EXIT_CONSOLE if you need the _script_ return code instead of +rem the _cmd.exe /c_ return code! +set EXIT_CODE=%ERRORLEVEL% +if %EXIT_CODE% equ 0 set EXIT_CODE=1 +if not ""=="%GRADLE_EXIT_CONSOLE%" exit %EXIT_CODE% +exit /b %EXIT_CODE% + +:mainEnd +if "%OS%"=="Windows_NT" endlocal + +:omega diff --git a/kotlin/settings.gradle.kts b/kotlin/settings.gradle.kts new file mode 100644 index 0000000..e21134a --- /dev/null +++ b/kotlin/settings.gradle.kts @@ -0,0 +1,17 @@ +pluginManagement { + repositories { + gradlePluginPortal() + mavenCentral() + google() + } +} + +dependencyResolutionManagement { + repositoriesMode.set(RepositoriesMode.FAIL_ON_PROJECT_REPOS) + repositories { + mavenCentral() + google() + } +} + +rootProject.name = "toki-socket-kotlin" diff --git a/kotlin/src/main/kotlin/com/tokilabs/toki_socket/BaseClient.kt b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/BaseClient.kt new file mode 100644 index 0000000..89eac7e --- /dev/null +++ b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/BaseClient.kt @@ -0,0 +1,104 @@ +package com.tokilabs.toki_socket + +import com.tokilabs.toki_socket.packets.HeartBeat +import kotlinx.coroutines.CoroutineScope +import kotlinx.coroutines.Dispatchers +import kotlinx.coroutines.SupervisorJob +import kotlinx.coroutines.cancel +import kotlinx.coroutines.launch +import kotlinx.coroutines.sync.Mutex +import kotlinx.coroutines.sync.withLock +import java.util.concurrent.CopyOnWriteArrayList +import java.util.concurrent.atomic.AtomicBoolean + +abstract class BaseClient>( + intervalSec: Int, + waitSec: Int, + private val doClose: () -> Unit, + internal val scope: CoroutineScope = CoroutineScope(SupervisorJob() + Dispatchers.IO), +) { + abstract val self: Self + abstract val communicator: Communicator + + private val heartbeatIntervalMs = intervalSec * 1000L + private val heartbeatWaitMs = waitSec * 1000L + private val closedOnce = AtomicBoolean(false) + private val hbMutex = Mutex() + private var hbTimer: HeartbeatTimer? = null + private var waitingHBResponse = false + private val disconnectListeners = CopyOnWriteArrayList<(Self) -> Unit>() + + fun addDisconnectListener(handler: (Self) -> Unit) { + disconnectListeners.add(handler) + } + + fun removeDisconnectListeners() { + disconnectListeners.clear() + } + + open fun close() { + if (!closedOnce.compareAndSet(false, true)) return + communicator.shutdown() + stopHeartbeat() + runCatching { doClose() } + notifyDisconnected() + scope.cancel() + } + + fun sendHeartBeat() { + if (!communicator.isAlive() || heartbeatIntervalMs <= 0) return + scope.launch { + hbMutex.withLock { + hbTimer?.stop() + hbTimer = HeartbeatTimer(scope, heartbeatIntervalMs) { + if (!communicator.isAlive()) return@HeartbeatTimer + hbMutex.withLock { + waitingHBResponse = true + } + runCatching { communicator.send(HeartBeat.getDefaultInstance()) } + hbMutex.withLock { + hbTimer?.stop() + hbTimer = HeartbeatTimer(scope, heartbeatWaitMs) { + if (communicator.isAlive()) onDisconnected() + } + } + } + } + } + } + + fun onHeartBeat() { + scope.launch { + val wasWaiting = hbMutex.withLock { + if (waitingHBResponse) { + waitingHBResponse = false + true + } else { + false + } + } + if (!wasWaiting) { + runCatching { communicator.send(HeartBeat.getDefaultInstance()) } + } + } + } + + fun stopHeartbeat() { + scope.launch { + hbMutex.withLock { + hbTimer?.stop() + hbTimer = null + } + } + } + + fun onDisconnected() { + close() + } + + private fun notifyDisconnected() { + val listeners = disconnectListeners.toList() + disconnectListeners.clear() + listeners.forEach { it(self) } + } +} diff --git a/kotlin/src/main/kotlin/com/tokilabs/toki_socket/Communicator.kt b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/Communicator.kt new file mode 100644 index 0000000..46e8210 --- /dev/null +++ b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/Communicator.kt @@ -0,0 +1,319 @@ +package com.tokilabs.toki_socket + +import com.google.protobuf.Message +import com.google.protobuf.MessageLite +import com.tokilabs.toki_socket.packets.HeartBeat +import com.tokilabs.toki_socket.packets.PacketBase +import kotlinx.coroutines.CompletableDeferred +import kotlinx.coroutines.CoroutineScope +import kotlinx.coroutines.channels.Channel +import kotlinx.coroutines.channels.ClosedSendChannelException +import kotlinx.coroutines.launch +import kotlinx.coroutines.selects.select +import kotlinx.coroutines.withTimeout +import java.util.concurrent.ConcurrentHashMap +import java.util.concurrent.atomic.AtomicBoolean +import java.util.concurrent.atomic.AtomicInteger +import java.util.concurrent.locks.ReentrantReadWriteLock +import kotlin.concurrent.read +import kotlin.concurrent.write +import kotlin.reflect.KClass + +class NotConnectedException : IllegalStateException("not connected") + +interface Transport { + suspend fun writePacket(base: PacketBase) + + fun close() +} + +typealias ParserMap = Map MessageLite> + +private typealias RequestHandler = suspend (MessageLite, Int) -> Unit + +private data class PendingRequest( + val expectedTypeName: String, + val ch: Channel = Channel(capacity = 1), + val errCh: Channel = Channel(capacity = 1), +) + +private data class QueuedPacket( + val base: PacketBase, + val done: CompletableDeferred = CompletableDeferred(), +) + +fun typeNameOf(m: MessageLite): String = + (m as Message).descriptorForType.fullName + +@PublishedApi +internal fun defaultMessage(kClass: KClass): T { + val method = kClass.java.getDeclaredMethod("getDefaultInstance") + @Suppress("UNCHECKED_CAST") + return method.invoke(null) as T +} + +inline fun typeNameOf(): String = + typeNameOf(defaultMessage(T::class)) + +class Communicator( + private val transport: Transport, + parserMap: ParserMap, + private val scope: CoroutineScope, +) { + private val rwLock = ReentrantReadWriteLock() + private val nonce = AtomicInteger(0) + private val isAlive = AtomicBoolean(false) + private val parserMap: MutableMap MessageLite> = ConcurrentHashMap() + private val handlers = mutableMapOf Unit>>() + private val reqHandlers = mutableMapOf() + private val pendingRequests = mutableMapOf() + private val writeQueue = Channel(capacity = 64) + private val closed = CompletableDeferred() + private val shutdownStarted = AtomicBoolean(false) + private var writeErrorHandler: ((Throwable) -> Unit)? = null + + init { + initialize(parserMap) + } + + private fun initialize(parserMap: ParserMap) { + this.parserMap.putAll(parserMap) + this.parserMap[typeNameOf()] = { HeartBeat.parseFrom(it) } + isAlive.set(true) + scope.launch { writeLoop() } + } + + fun isAlive(): Boolean = isAlive.get() + + fun setWriteErrorHandler(fn: (Throwable) -> Unit) { + rwLock.write { writeErrorHandler = fn } + } + + fun nextNonce(): Int = nonce.incrementAndGet() + + fun shutdown() { + if (!shutdownStarted.compareAndSet(false, true)) return + isAlive.set(false) + writeQueue.close() + closed.complete(Unit) + + val pending = rwLock.write { + val snapshot = pendingRequests.values.toList() + pendingRequests.clear() + snapshot + } + pending.forEach { it.errCh.trySend(NotConnectedException()) } + } + + fun close() { + shutdown() + transport.close() + } + + suspend fun queuePacket(base: PacketBase) { + if (!isAlive()) throw NotConnectedException() + val item = QueuedPacket(base) + try { + select { + writeQueue.onSend(item) {} + closed.onAwait { throw NotConnectedException() } + } + } catch (e: ClosedSendChannelException) { + throw NotConnectedException() + } + + val error = select { + item.done.onAwait { it } + closed.onAwait { NotConnectedException() } + } + if (error != null) throw error + } + + suspend fun send(m: MessageLite) { + if (!isAlive()) throw NotConnectedException() + queuePacket( + PacketBase.newBuilder() + .setTypeName(typeNameOf(m)) + .setNonce(nextNonce()) + .setData(m.toByteString()) + .build(), + ) + } + + suspend fun sendRequest( + req: MessageLite, + resTypeName: String, + timeoutMs: Long = 30_000L, + ): MessageLite { + if (!isAlive()) throw NotConnectedException() + val requestNonce = nextNonce() + val pending = PendingRequest(expectedTypeName = resTypeName) + + rwLock.write { pendingRequests[requestNonce] = pending } + + try { + queuePacket( + PacketBase.newBuilder() + .setTypeName(typeNameOf(req)) + .setNonce(requestNonce) + .setData(req.toByteString()) + .build(), + ) + } catch (e: Throwable) { + removePending(requestNonce) + throw e + } + + return try { + withTimeout(if (timeoutMs > 0) timeoutMs else 30_000L) { + select { + pending.ch.onReceive { it } + pending.errCh.onReceive { throw it } + closed.onAwait { throw NotConnectedException() } + } + } + } catch (e: kotlinx.coroutines.TimeoutCancellationException) { + removePending(requestNonce) + throw IllegalStateException("request timeout for nonce $requestNonce", e) + } + } + + fun addListener(typeName: String, fn: (MessageLite) -> Unit) { + rwLock.write { + check(!reqHandlers.containsKey(typeName)) { + "type $typeName is already registered with addRequestListener" + } + handlers.getOrPut(typeName) { mutableListOf() }.add(fn) + } + } + + fun removeListeners(typeName: String) { + rwLock.write { handlers.remove(typeName) } + } + + fun addRequestListener(typeName: String, fn: RequestHandler) { + rwLock.write { + check(handlers[typeName].isNullOrEmpty()) { + "type $typeName is already registered with addListener" + } + check(!reqHandlers.containsKey(typeName)) { + "type $typeName is already registered with addRequestListener" + } + reqHandlers[typeName] = fn + } + } + + fun onReceivedData( + typeName: String, + data: ByteArray, + incomingNonce: Int = 0, + responseNonce: Int = 0, + ) { + if (responseNonce > 0) { + handleResponse(typeName, data, responseNonce) + return + } + + val reqHandler: RequestHandler? + val listeners: List<(MessageLite) -> Unit> + rwLock.read { + reqHandler = reqHandlers[typeName] + listeners = handlers[typeName]?.toList().orEmpty() + } + + if (reqHandler != null) { + val msg = runCatching { parse(typeName, data) }.getOrNull() ?: return + scope.launch { reqHandler(msg, incomingNonce) } + return + } + + if (listeners.isEmpty()) return + val msg = runCatching { parse(typeName, data) }.getOrNull() ?: return + listeners.forEach { it(msg) } + } + + private suspend fun writeLoop() { + for (item in writeQueue) { + val error = runCatching { transport.writePacket(item.base) }.exceptionOrNull() + item.done.complete(error) + if (error != null) { + val handler = rwLock.read { writeErrorHandler } + handler?.invoke(error) + } + } + } + + private fun handleResponse( + typeName: String, + data: ByteArray, + responseNonce: Int, + ) { + val pending = removePending(responseNonce) ?: return + if (typeName != pending.expectedTypeName) { + pending.errCh.trySend( + IllegalStateException( + "response type mismatch for nonce $responseNonce: " + + "expected ${pending.expectedTypeName}, got $typeName", + ), + ) + return + } + val msg = runCatching { parse(typeName, data) } + msg.onSuccess { pending.ch.trySend(it) } + .onFailure { pending.errCh.trySend(it) } + } + + internal fun parse(typeName: String, data: ByteArray): MessageLite { + val parser = parserMap[typeName] + ?: throw IllegalStateException("protobuf parser is not registered for type $typeName") + return parser(data) + } + + private fun removePending(nonce: Int): PendingRequest? = + rwLock.write { pendingRequests.remove(nonce) } +} + +inline fun addListenerTyped( + communicator: Communicator, + noinline fn: (T) -> Unit, +) { + val typeName = typeNameOf() + communicator.addListener(typeName) { message -> + val typed = message as? T + ?: throw IllegalStateException("received ${message::class} for listener $typeName") + fn(typed) + } +} + +inline fun addRequestListenerTyped( + communicator: Communicator, + noinline fn: suspend (Req) -> Res, +) { + val reqTypeName = typeNameOf() + communicator.addRequestListener(reqTypeName) { message, requestNonce -> + val req = message as? Req + if (req != null) { + val res = fn(req) + if (communicator.isAlive()) { + communicator.queuePacket( + PacketBase.newBuilder() + .setTypeName(typeNameOf(res)) + .setNonce(communicator.nextNonce()) + .setResponseNonce(requestNonce) + .setData(res.toByteString()) + .build(), + ) + } + } + } +} + +suspend inline fun sendRequestTyped( + communicator: Communicator, + req: Req, + timeoutMs: Long = 30_000L, +): Res { + val msg = communicator.sendRequest(req, typeNameOf(), timeoutMs) + return msg as? Res + ?: throw IllegalStateException("received ${msg::class}, expected ${Res::class}") +} diff --git a/kotlin/src/main/kotlin/com/tokilabs/toki_socket/HeartbeatTimer.kt b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/HeartbeatTimer.kt new file mode 100644 index 0000000..b2896b5 --- /dev/null +++ b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/HeartbeatTimer.kt @@ -0,0 +1,31 @@ +package com.tokilabs.toki_socket + +import kotlinx.coroutines.CoroutineScope +import kotlinx.coroutines.Job +import kotlinx.coroutines.delay +import kotlinx.coroutines.launch + +class HeartbeatTimer( + private val scope: CoroutineScope, + private val delayMs: Long, + private val callback: suspend () -> Unit, +) { + private var job: Job? = null + + init { + reset() + } + + fun reset(delayMs: Long = this.delayMs) { + job?.cancel() + job = scope.launch { + delay(delayMs) + callback() + } + } + + fun stop() { + job?.cancel() + job = null + } +} diff --git a/kotlin/src/main/kotlin/com/tokilabs/toki_socket/TcpClient.kt b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/TcpClient.kt new file mode 100644 index 0000000..8f247c2 --- /dev/null +++ b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/TcpClient.kt @@ -0,0 +1,129 @@ +package com.tokilabs.toki_socket + +import com.tokilabs.toki_socket.packets.HeartBeat +import com.tokilabs.toki_socket.packets.PacketBase +import kotlinx.coroutines.Dispatchers +import kotlinx.coroutines.launch +import kotlinx.coroutines.runBlocking +import kotlinx.coroutines.sync.Mutex +import kotlinx.coroutines.sync.withLock +import kotlinx.coroutines.withContext +import java.io.DataInputStream +import java.io.IOException +import java.net.InetSocketAddress +import java.net.Socket +import java.nio.ByteBuffer + +class TcpClient private constructor( + private val socket: Socket, + intervalSec: Int, + waitSec: Int, + parserMap: ParserMap, +) : BaseClient( + intervalSec = intervalSec, + waitSec = waitSec, + doClose = { socket.close() }, +), + Transport { + override val self: TcpClient + get() = this + + override val communicator: Communicator = Communicator(this, parserMap, scope) + private val writeMutex = Mutex() + + init { + communicator.setWriteErrorHandler { onDisconnected() } + communicator.addListener(typeNameOf()) { onHeartBeat() } + readLoop() + sendHeartBeat() + } + + override suspend fun writePacket(base: PacketBase) { + val bytes = base.toByteArray() + if (bytes.size > MAX_PACKET_SIZE) { + throw IOException("packet size ${bytes.size} exceeds max $MAX_PACKET_SIZE") + } + val header = ByteBuffer.allocate(4).putInt(bytes.size).array() + writeMutex.withLock { + withContext(Dispatchers.IO) { + val output = socket.getOutputStream() + output.write(header) + output.write(bytes) + output.flush() + } + } + } + + private fun readLoop() { + scope.launch(Dispatchers.IO) { + val input = DataInputStream(socket.getInputStream()) + while (communicator.isAlive()) { + try { + val length = input.readInt() + if (length == 0) continue + if (length < 0 || length > MAX_PACKET_SIZE) { + onDisconnected() + return@launch + } + val bytes = ByteArray(length) + input.readFully(bytes) + val base = PacketBase.parseFrom(bytes) + communicator.onReceivedData( + base.typeName, + base.data.toByteArray(), + base.nonce, + base.responseNonce, + ) + sendHeartBeat() + } catch (_: Exception) { + onDisconnected() + return@launch + } + } + } + } + + suspend fun send(message: com.google.protobuf.MessageLite) { + communicator.send(message) + } + + companion object { + const val MAX_PACKET_SIZE: Int = 64 * 1024 * 1024 + + fun fromSocket( + socket: Socket, + intervalSec: Int = 30, + waitSec: Int = 10, + parserMap: ParserMap, + ): TcpClient = TcpClient(socket, intervalSec, waitSec, parserMap) + } +} + +fun dialTcp( + host: String, + port: Int, + intervalSec: Int = 30, + waitSec: Int = 10, + parserMap: ParserMap, +): TcpClient { + val socket = Socket() + socket.connect(InetSocketAddress(host, port)) + return TcpClient.fromSocket(socket, intervalSec, waitSec, parserMap) +} + +@Suppress("FunctionName") +fun DialTcp( + host: String, + port: Int, + intervalSec: Int = 30, + waitSec: Int = 10, + parserMap: ParserMap, +): TcpClient = dialTcp(host, port, intervalSec, waitSec, parserMap) + +fun TcpClient.closeBlocking() { + close() +} + +fun TcpClient.sendBlocking(message: com.google.protobuf.MessageLite) { + runBlocking { send(message) } +} diff --git a/kotlin/src/main/kotlin/com/tokilabs/toki_socket/TcpServer.kt b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/TcpServer.kt new file mode 100644 index 0000000..f773403 --- /dev/null +++ b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/TcpServer.kt @@ -0,0 +1,77 @@ +package com.tokilabs.toki_socket + +import com.google.protobuf.MessageLite +import kotlinx.coroutines.CoroutineScope +import kotlinx.coroutines.Dispatchers +import kotlinx.coroutines.SupervisorJob +import kotlinx.coroutines.cancel +import kotlinx.coroutines.launch +import kotlinx.coroutines.runBlocking +import java.net.InetSocketAddress +import java.net.ServerSocket +import java.net.SocketException +import java.util.concurrent.CopyOnWriteArrayList +import java.util.concurrent.atomic.AtomicBoolean + +class TcpServer( + private val host: String, + private val port: Int, + private val newClient: (java.net.Socket) -> TcpClient, +) { + private val scope = CoroutineScope(SupervisorJob() + Dispatchers.IO) + private val clients = CopyOnWriteArrayList() + private val started = AtomicBoolean(false) + private var serverSocket: ServerSocket? = null + + var onClientConnected: (TcpClient) -> Unit = {} + + fun started(): Boolean = started.get() + + fun clients(): List = clients.toList() + + fun start() { + if (!started.compareAndSet(false, true)) return + val server = ServerSocket() + server.reuseAddress = true + server.bind(InetSocketAddress(host, port)) + serverSocket = server + scope.launch(Dispatchers.IO) { + acceptLoop(server) + } + } + + fun stop() { + if (!started.compareAndSet(true, false)) return + runCatching { serverSocket?.close() } + val snapshot = clients.toList() + clients.clear() + snapshot.forEach { it.close() } + scope.cancel() + } + + suspend fun broadcast(message: MessageLite) { + clients().forEach { client -> + client.communicator.send(message) + } + } + + private fun acceptLoop(server: ServerSocket) { + while (started.get()) { + try { + val socket = server.accept() + val client = newClient(socket) + client.addDisconnectListener { clients.remove(it) } + clients.add(client) + onClientConnected(client) + } catch (_: SocketException) { + return + } catch (_: Exception) { + if (!started.get()) return + } + } + } +} + +fun TcpServer.broadcastBlocking(message: MessageLite) { + runBlocking { broadcast(message) } +} diff --git a/kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsClient.kt b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsClient.kt new file mode 100644 index 0000000..2d29365 --- /dev/null +++ b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsClient.kt @@ -0,0 +1,228 @@ +package com.tokilabs.toki_socket + +import com.tokilabs.toki_socket.packets.HeartBeat +import com.tokilabs.toki_socket.packets.PacketBase +import kotlinx.coroutines.runBlocking +import okhttp3.OkHttpClient +import okhttp3.Request +import okhttp3.Response +import okhttp3.WebSocketListener +import okio.ByteString +import org.java_websocket.WebSocket +import java.io.IOException +import java.security.KeyStore +import java.util.concurrent.CountDownLatch +import java.util.concurrent.TimeUnit +import java.util.concurrent.atomic.AtomicReference +import javax.net.ssl.SSLContext +import javax.net.ssl.TrustManagerFactory +import javax.net.ssl.X509TrustManager + +class WsClient internal constructor( + private val connection: WsConnection, + intervalSec: Int, + waitSec: Int, + parserMap: ParserMap, +) : BaseClient( + intervalSec = intervalSec, + waitSec = waitSec, + doClose = { connection.close() }, +), + Transport { + override val self: WsClient + get() = this + + override val communicator: Communicator = Communicator(this, parserMap, scope) + + init { + communicator.setWriteErrorHandler { onDisconnected() } + communicator.addListener(typeNameOf()) { onHeartBeat() } + sendHeartBeat() + } + + override suspend fun writePacket(base: PacketBase) { + val bytes = base.toByteArray() + if (!connection.send(bytes)) { + throw IOException("websocket send failed") + } + } + + internal fun receiveBytes(bytes: ByteArray) { + try { + val base = PacketBase.parseFrom(bytes) + communicator.onReceivedData( + base.typeName, + base.data.toByteArray(), + base.nonce, + base.responseNonce, + ) + sendHeartBeat() + } catch (_: Exception) { + onDisconnected() + } + } + + internal fun handleFailure() { + onDisconnected() + } + + suspend fun send(message: com.google.protobuf.MessageLite) { + communicator.send(message) + } + + companion object { + fun forServer( + conn: WebSocket, + intervalSec: Int, + waitSec: Int, + parserMap: ParserMap, + ): WsClient = WsClient(JavaWebSocketConnection(conn), intervalSec, waitSec, parserMap) + } +} + +internal interface WsConnection { + fun send(bytes: ByteArray): Boolean + + fun close() +} + +private class JavaWebSocketConnection( + private val conn: WebSocket, +) : WsConnection { + override fun send(bytes: ByteArray): Boolean = + runCatching { + conn.send(bytes) + true + }.getOrElse { false } + + override fun close() { + runCatching { conn.close(1000) } + } +} + +private class OkHttpWsConnection( + private val client: OkHttpClient, +) : WsConnection { + private val socketRef = AtomicReference() + + fun setSocket(ws: okhttp3.WebSocket) { + socketRef.set(ws) + } + + override fun send(bytes: ByteArray): Boolean = + socketRef.get()?.send(ByteString.of(*bytes)) ?: false + + override fun close() { + runCatching { socketRef.get()?.close(1000, null) } + client.dispatcher.executorService.shutdown() + } +} + +fun dialWs( + host: String, + port: Int, + path: String, + intervalSec: Int = 30, + waitSec: Int = 10, + parserMap: ParserMap, +): WsClient = dialWsUrl("ws://$host:$port$path", null, intervalSec, waitSec, parserMap) + +fun dialWss( + host: String, + port: Int, + path: String, + sslContext: SSLContext?, + intervalSec: Int = 30, + waitSec: Int = 10, + parserMap: ParserMap, +): WsClient = dialWsUrl("wss://$host:$port$path", sslContext, intervalSec, waitSec, parserMap) + +private fun dialWsUrl( + url: String, + sslContext: SSLContext?, + intervalSec: Int, + waitSec: Int, + parserMap: ParserMap, +): WsClient { + val builder = OkHttpClient.Builder() + if (sslContext != null) { + builder.sslSocketFactory(sslContext.socketFactory, defaultTrustManager()) + } + val okHttpClient = builder.build() + val connection = OkHttpWsConnection(okHttpClient) + val opened = CountDownLatch(1) + val failed = AtomicReference() + var client: WsClient? = null + val listener = object : WebSocketListener() { + override fun onOpen(webSocket: okhttp3.WebSocket, response: Response) { + connection.setSocket(webSocket) + opened.countDown() + } + + override fun onMessage(webSocket: okhttp3.WebSocket, bytes: ByteString) { + client?.receiveBytes(bytes.toByteArray()) + } + + override fun onClosing(webSocket: okhttp3.WebSocket, code: Int, reason: String) { + webSocket.close(code, reason) + client?.handleFailure() + } + + override fun onClosed(webSocket: okhttp3.WebSocket, code: Int, reason: String) { + client?.handleFailure() + } + + override fun onFailure(webSocket: okhttp3.WebSocket, t: Throwable, response: Response?) { + failed.set(t) + opened.countDown() + client?.handleFailure() + } + } + + client = WsClient(connection, intervalSec, waitSec, parserMap) + val ws = okHttpClient.newWebSocket(Request.Builder().url(url).build(), listener) + connection.setSocket(ws) + if (!opened.await(5, TimeUnit.SECONDS)) { + client.close() + throw IOException("websocket open timed out") + } + failed.get()?.let { + client.close() + throw IOException("websocket open failed", it) + } + return client +} + +private fun defaultTrustManager(): X509TrustManager { + val factory = TrustManagerFactory.getInstance(TrustManagerFactory.getDefaultAlgorithm()) + factory.init(null as KeyStore?) + return factory.trustManagers + .filterIsInstance() + .singleOrNull() + ?: throw IllegalStateException("expected exactly one default X509TrustManager") +} + +@Suppress("FunctionName") +fun DialWs( + host: String, + port: Int, + path: String, + intervalSec: Int = 30, + waitSec: Int = 10, + parserMap: ParserMap, +): WsClient = dialWs(host, port, path, intervalSec, waitSec, parserMap) + +@Suppress("FunctionName") +fun DialWss( + host: String, + port: Int, + path: String, + sslContext: SSLContext?, + intervalSec: Int = 30, + waitSec: Int = 10, + parserMap: ParserMap, +): WsClient = dialWss(host, port, path, sslContext, intervalSec, waitSec, parserMap) + +fun WsClient.sendBlocking(message: com.google.protobuf.MessageLite) { + runBlocking { send(message) } +} diff --git a/kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsServer.kt b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsServer.kt new file mode 100644 index 0000000..7177784 --- /dev/null +++ b/kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsServer.kt @@ -0,0 +1,93 @@ +package com.tokilabs.toki_socket + +import com.google.protobuf.MessageLite +import kotlinx.coroutines.runBlocking +import org.java_websocket.WebSocket +import org.java_websocket.handshake.ClientHandshake +import org.java_websocket.server.WebSocketServer +import java.net.InetSocketAddress +import java.nio.ByteBuffer +import java.util.concurrent.ConcurrentHashMap +import java.util.concurrent.CopyOnWriteArrayList +import java.util.concurrent.atomic.AtomicBoolean + +class WsServer( + host: String, + port: Int, + private val path: String = "/", + private val newClient: (WebSocket) -> WsClient, +) : WebSocketServer(InetSocketAddress(host, port)) { + private val clientByConn = ConcurrentHashMap() + private val clients = CopyOnWriteArrayList() + private val startedFlag = AtomicBoolean(false) + + var onClientConnected: (WsClient) -> Unit = {} + + fun started(): Boolean = startedFlag.get() + + fun clients(): List = clients.toList() + + override fun start() { + startedFlag.set(true) + super.start() + } + + override fun stop() { + if (!startedFlag.compareAndSet(true, false)) return + val snapshot = clients.toList() + clients.clear() + snapshot.forEach { it.close() } + runCatching { super.stop(1000) } + } + + suspend fun broadcast(message: MessageLite) { + clients().forEach { client -> + client.communicator.send(message) + } + } + + override fun onOpen(conn: WebSocket, handshake: ClientHandshake) { + if (handshake.resourceDescriptor != path) { + conn.close(1008, "unexpected path") + return + } + val client = newClient(conn) + client.addDisconnectListener { + clients.remove(it) + clientByConn.remove(conn) + } + clientByConn[conn] = client + clients.add(client) + onClientConnected(client) + } + + override fun onMessage(conn: WebSocket, message: String) { + } + + override fun onMessage(conn: WebSocket, message: ByteBuffer) { + val bytes = ByteArray(message.remaining()) + message.get(bytes) + clientByConn[conn]?.receiveBytes(bytes) + } + + override fun onClose( + conn: WebSocket, + code: Int, + reason: String, + remote: Boolean, + ) { + clientByConn.remove(conn)?.handleFailure() + } + + override fun onError(conn: WebSocket?, ex: Exception) { + if (conn == null) return + clientByConn[conn]?.handleFailure() + } + + override fun onStart() { + } +} + +fun WsServer.broadcastBlocking(message: MessageLite) { + runBlocking { broadcast(message) } +} diff --git a/kotlin/src/main/proto/message_common.proto b/kotlin/src/main/proto/message_common.proto new file mode 100644 index 0000000..08069f5 --- /dev/null +++ b/kotlin/src/main/proto/message_common.proto @@ -0,0 +1,19 @@ +syntax = "proto3"; + +option java_package = "com.tokilabs.toki_socket.packets"; +option java_outer_classname = "MessageCommonProto"; +option java_multiple_files = true; + +message PacketBase { + string typeName = 1; + int32 nonce = 2; + bytes data = 3; + int32 responseNonce = 4; +} + +message HeartBeat {} + +message TestData { + int32 index = 1; + string message = 2; +} diff --git a/kotlin/src/test/kotlin/com/tokilabs/toki_socket/CommunicatorTest.kt b/kotlin/src/test/kotlin/com/tokilabs/toki_socket/CommunicatorTest.kt new file mode 100644 index 0000000..60b3b75 --- /dev/null +++ b/kotlin/src/test/kotlin/com/tokilabs/toki_socket/CommunicatorTest.kt @@ -0,0 +1,95 @@ +package com.tokilabs.toki_socket + +import com.tokilabs.toki_socket.packets.HeartBeat +import com.tokilabs.toki_socket.packets.PacketBase +import com.tokilabs.toki_socket.packets.TestData +import kotlinx.coroutines.CoroutineScope +import kotlinx.coroutines.Dispatchers +import kotlinx.coroutines.SupervisorJob +import kotlinx.coroutines.async +import kotlinx.coroutines.delay +import kotlinx.coroutines.test.runTest +import java.util.Collections +import kotlin.test.Test +import kotlin.test.assertEquals +import kotlin.test.assertFailsWith +import kotlin.test.assertTrue + +private class FakeTransport : Transport { + val packets: MutableList = Collections.synchronizedList(mutableListOf()) + + override suspend fun writePacket(base: PacketBase) { + packets.add(base) + } + + override fun close() { + } +} + +class CommunicatorTest { + @Test + fun testSendRequestTimeout() = runTest { + val transport = FakeTransport() + val communicator = Communicator(transport, testParserMap(), CoroutineScope(SupervisorJob() + Dispatchers.Default)) + + val err = assertFailsWith { + sendRequestTyped( + communicator, + TestData.newBuilder().setIndex(1).setMessage("no response").build(), + timeoutMs = 25, + ) + } + assertTrue(err.message!!.contains("timeout")) + assertEquals(1, transport.packets.size) + communicator.close() + } + + @Test + fun testSendRequestTypeMismatch() = runTest { + val transport = FakeTransport() + val communicator = Communicator(transport, testParserMap(), CoroutineScope(SupervisorJob() + Dispatchers.Default)) + + val result = async { + assertFailsWith { + sendRequestTyped( + communicator, + TestData.newBuilder().setIndex(1).setMessage("hello").build(), + timeoutMs = 1_000, + ) + } + } + while (transport.packets.isEmpty()) delay(1) + val requestNonce = transport.packets.single().nonce + communicator.onReceivedData( + typeNameOf(), + HeartBeat.getDefaultInstance().toByteArray(), + responseNonce = requestNonce, + ) + assertTrue(result.await().message!!.contains("response type mismatch")) + communicator.close() + } + + @Test + fun testListenerAndRequestListenerConflict() = runTest { + val transport = FakeTransport() + val communicator = Communicator(transport, testParserMap(), CoroutineScope(SupervisorJob() + Dispatchers.Default)) + + addListenerTyped(communicator) {} + assertFailsWith { + addRequestListenerTyped(communicator) { it } + } + communicator.close() + } + + @Test + fun testSendFireAndForget() = runTest { + val transport = FakeTransport() + val communicator = Communicator(transport, testParserMap(), CoroutineScope(SupervisorJob() + Dispatchers.Default)) + + communicator.send(TestData.newBuilder().setIndex(7).setMessage("fire").build()) + + assertEquals(1, transport.packets.size) + assertEquals(typeNameOf(), transport.packets.single().typeName) + communicator.close() + } +} diff --git a/kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTest.kt b/kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTest.kt new file mode 100644 index 0000000..ff2bb76 --- /dev/null +++ b/kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTest.kt @@ -0,0 +1,107 @@ +package com.tokilabs.toki_socket + +import com.tokilabs.toki_socket.packets.HeartBeat +import com.tokilabs.toki_socket.packets.PacketBase +import kotlinx.coroutines.delay +import kotlinx.coroutines.runBlocking +import java.util.Collections +import kotlin.test.Test +import kotlin.test.assertTrue + +private class HeartbeatTransport : Transport { + val packets: MutableList = Collections.synchronizedList(mutableListOf()) + + override suspend fun writePacket(base: PacketBase) { + packets.add(base) + } + + override fun close() { + } +} + +private class HeartbeatClient( + private val transport: HeartbeatTransport, + intervalSec: Int, + waitSec: Int, +) : BaseClient( + intervalSec = intervalSec, + waitSec = waitSec, + doClose = { transport.close() }, +) { + override val self: HeartbeatClient + get() = this + + override val communicator = Communicator(transport, testParserMap(), scope) + + init { + communicator.addListener(typeNameOf()) { onHeartBeat() } + } +} + +class HeartbeatTest { + @Test + fun testHeartbeatSentAfterInactivity() = runBlocking { + val transport = HeartbeatTransport() + val client = HeartbeatClient(transport, 1, 1) + + client.sendHeartBeat() + delay(1_100) + + assertTrue(transport.packets.any { it.typeName == typeNameOf() }) + client.close() + } + + @Test + fun testHeartbeatDisconnectOnNoResponse() = runBlocking { + val transport = HeartbeatTransport() + val client = HeartbeatClient(transport, 1, 1) + var disconnected = false + client.addDisconnectListener { disconnected = true } + + client.sendHeartBeat() + delay(2_300) + + assertTrue(disconnected) + assertTrue(!client.communicator.isAlive()) + } + + @Test + fun testHeartbeatTimerResetBySendHeartBeat() = runBlocking { + val transport = HeartbeatTransport() + val client = HeartbeatClient(transport, 1, 1) + + client.sendHeartBeat() + delay(700) + client.sendHeartBeat() + delay(500) + assertTrue(transport.packets.none { it.typeName == typeNameOf() }) + delay(700) + assertTrue(transport.packets.any { it.typeName == typeNameOf() }) + client.close() + } + + @Test + fun testHeartbeatTimerResetOnReceivedData() = runBlocking { + val transport = HeartbeatTransport() + val client = HeartbeatClient(transport, 1, 1) + var received = false + client.communicator.addListener(typeNameOf(testData())) { + received = true + } + + client.sendHeartBeat() + delay(700) + client.communicator.onReceivedData( + typeNameOf(testData()), + testData().toByteArray(), + incomingNonce = 1, + ) + assertTrue(received) + client.sendHeartBeat() + delay(500) + assertTrue(transport.packets.none { it.typeName == typeNameOf() }) + delay(700) + assertTrue(transport.packets.any { it.typeName == typeNameOf() }) + client.close() + } +} diff --git a/kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTimerTest.kt b/kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTimerTest.kt new file mode 100644 index 0000000..e3fb975 --- /dev/null +++ b/kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTimerTest.kt @@ -0,0 +1,49 @@ +package com.tokilabs.toki_socket + +import kotlinx.coroutines.CompletableDeferred +import kotlinx.coroutines.test.runTest +import kotlin.test.Test +import kotlin.test.assertFalse +import kotlin.test.assertTrue + +class HeartbeatTimerTest { + @Test + fun testCallbackFires() = runTest { + var fired = false + HeartbeatTimer(this, 100) { + fired = true + } + + testScheduler.advanceTimeBy(101) + + assertTrue(fired) + } + + @Test + fun testStopPreventsCallback() = runTest { + var fired = false + val timer = HeartbeatTimer(this, 100) { + fired = true + } + timer.stop() + + testScheduler.advanceTimeBy(150) + + assertFalse(fired) + } + + @Test + fun testResetRestartsTimer() = runTest { + val fired = CompletableDeferred() + val timer = HeartbeatTimer(this, 100) { + fired.complete(Unit) + } + + testScheduler.advanceTimeBy(80) + timer.reset() + testScheduler.advanceTimeBy(80) + assertFalse(fired.isCompleted) + testScheduler.advanceTimeBy(21) + assertTrue(fired.isCompleted) + } +} diff --git a/kotlin/src/test/kotlin/com/tokilabs/toki_socket/TcpTest.kt b/kotlin/src/test/kotlin/com/tokilabs/toki_socket/TcpTest.kt new file mode 100644 index 0000000..5f7cabc --- /dev/null +++ b/kotlin/src/test/kotlin/com/tokilabs/toki_socket/TcpTest.kt @@ -0,0 +1,114 @@ +package com.tokilabs.toki_socket + +import com.tokilabs.toki_socket.packets.TestData +import kotlinx.coroutines.CompletableDeferred +import kotlinx.coroutines.Dispatchers +import kotlinx.coroutines.async +import kotlinx.coroutines.runBlocking +import java.net.Socket +import kotlin.test.Test +import kotlin.test.assertEquals +import kotlin.test.assertFalse + +class TcpTest { + @Test + fun testTcpSendReceive() = runBlocking { + val port = freePort() + val received = CompletableDeferred() + val server = TcpServer("127.0.0.1", port) { socket -> + TcpClient.fromSocket(socket, 0, 0, testParserMap()) + } + server.onClientConnected = { client -> + addListenerTyped(client.communicator) { received.complete(it) } + } + server.start() + val client = DialTcp("127.0.0.1", port, 0, 0, testParserMap()) + + client.send(TestData.newBuilder().setIndex(42).setMessage("hello toki-socket").build()) + + assertEquals(42, received.await().index) + client.close() + server.stop() + } + + @Test + fun testTcpRequestResponse() = runBlocking { + val port = freePort() + val server = TcpServer("127.0.0.1", port) { socket -> + TcpClient.fromSocket(socket, 0, 0, testParserMap()) + } + server.onClientConnected = { client -> + addRequestListenerTyped(client.communicator) { req -> + TestData.newBuilder() + .setIndex(req.index * 2) + .setMessage("echo: ${req.message}") + .build() + } + } + server.start() + val client = DialTcp("127.0.0.1", port, 0, 0, testParserMap()) + + val res = sendRequestTyped( + client.communicator, + TestData.newBuilder().setIndex(21).setMessage("hello").build(), + timeoutMs = 2_000, + ) + + assertEquals(42, res.index) + assertEquals("echo: hello", res.message) + client.close() + server.stop() + } + + @Test + fun testTcpBroadcast() = runBlocking { + val port = freePort() + val server = TcpServer("127.0.0.1", port) { socket -> + TcpClient.fromSocket(socket, 0, 0, testParserMap()) + } + server.start() + val client = DialTcp("127.0.0.1", port, 0, 0, testParserMap()) + val received = CompletableDeferred() + addListenerTyped(client.communicator) { received.complete(it) } + + waitForCondition(message = "client did not connect") { server.clients().size == 1 } + server.broadcast(TestData.newBuilder().setIndex(9).setMessage("broadcast").build()) + + assertEquals("broadcast", received.await().message) + client.close() + server.stop() + } + + @Test + fun testTcpServerStopDisconnectsClients() = runBlocking { + val port = freePort() + val server = TcpServer("127.0.0.1", port) { socket -> + TcpClient.fromSocket(socket, 0, 0, testParserMap()) + } + server.start() + val client = DialTcp("127.0.0.1", port, 0, 0, testParserMap()) + val disconnected = CompletableDeferred() + client.addDisconnectListener { disconnected.complete(Unit) } + + waitForCondition(message = "client did not connect") { server.clients().size == 1 } + server.stop() + + disconnected.await() + assertFalse(client.communicator.isAlive()) + } + + @Test + fun testTcpClientCloseIdempotent() = runBlocking { + val serverSocket = java.net.ServerSocket(0) + val accept = async(Dispatchers.IO) { serverSocket.accept() } + val socket = Socket("127.0.0.1", serverSocket.localPort) + val peer = accept.await() + val client = TcpClient.fromSocket(socket, 0, 0, testParserMap()) + + repeat(3) { client.close() } + + assertFalse(client.communicator.isAlive()) + peer.close() + serverSocket.close() + } +} diff --git a/kotlin/src/test/kotlin/com/tokilabs/toki_socket/TestHelpers.kt b/kotlin/src/test/kotlin/com/tokilabs/toki_socket/TestHelpers.kt new file mode 100644 index 0000000..00e5f92 --- /dev/null +++ b/kotlin/src/test/kotlin/com/tokilabs/toki_socket/TestHelpers.kt @@ -0,0 +1,33 @@ +package com.tokilabs.toki_socket + +import com.tokilabs.toki_socket.packets.TestData +import java.net.ServerSocket +import kotlin.test.fail + +fun testParserMap(): ParserMap = + mapOf( + typeNameOf() to { TestData.parseFrom(it) }, + ) + +fun testData(): TestData = + TestData.newBuilder() + .setIndex(1) + .setMessage("ping") + .build() + +fun freePort(): Int = + ServerSocket(0).use { it.localPort } + +suspend fun waitForCondition( + timeoutMs: Long = 2_000L, + intervalMs: Long = 10L, + message: String, + predicate: () -> Boolean, +) { + val deadline = System.nanoTime() + timeoutMs * 1_000_000L + while (System.nanoTime() < deadline) { + if (predicate()) return + kotlinx.coroutines.delay(intervalMs) + } + fail(message) +} diff --git a/kotlin/src/test/kotlin/com/tokilabs/toki_socket/WsTest.kt b/kotlin/src/test/kotlin/com/tokilabs/toki_socket/WsTest.kt new file mode 100644 index 0000000..4502d9b --- /dev/null +++ b/kotlin/src/test/kotlin/com/tokilabs/toki_socket/WsTest.kt @@ -0,0 +1,101 @@ +package com.tokilabs.toki_socket + +import com.tokilabs.toki_socket.packets.TestData +import kotlinx.coroutines.CompletableDeferred +import kotlinx.coroutines.runBlocking +import kotlin.test.Test +import kotlin.test.assertEquals +import kotlin.test.assertFalse + +class WsTest { + @Test + fun testWsSendReceive() = runBlocking { + val port = freePort() + val received = CompletableDeferred() + val server = WsServer("127.0.0.1", port, "/") { conn -> + WsClient.forServer(conn, 0, 0, testParserMap()) + } + server.onClientConnected = { client -> + addListenerTyped(client.communicator) { received.complete(it) } + } + server.start() + val client = DialWs("127.0.0.1", port, "/", 0, 0, testParserMap()) + + client.send(TestData.newBuilder().setIndex(42).setMessage("hello toki-socket ws").build()) + + assertEquals(42, received.await().index) + client.close() + server.stop() + } + + @Test + fun testWsRequestResponse() = runBlocking { + val port = freePort() + val server = WsServer("127.0.0.1", port, "/") { conn -> + WsClient.forServer(conn, 0, 0, testParserMap()) + } + server.onClientConnected = { client -> + addRequestListenerTyped(client.communicator) { req -> + TestData.newBuilder() + .setIndex(req.index * 2) + .setMessage("echo: ${req.message}") + .build() + } + } + server.start() + val client = DialWs("127.0.0.1", port, "/", 0, 0, testParserMap()) + + val res = sendRequestTyped( + client.communicator, + TestData.newBuilder().setIndex(21).setMessage("hello ws").build(), + timeoutMs = 2_000, + ) + + assertEquals(42, res.index) + assertEquals("echo: hello ws", res.message) + client.close() + server.stop() + } + + @Test + fun testWsBroadcast() = runBlocking { + val port = freePort() + val server = WsServer("127.0.0.1", port, "/") { conn -> + WsClient.forServer(conn, 0, 0, testParserMap()) + } + server.start() + val client1 = DialWs("127.0.0.1", port, "/", 0, 0, testParserMap()) + val client2 = DialWs("127.0.0.1", port, "/", 0, 0, testParserMap()) + val received1 = CompletableDeferred() + val received2 = CompletableDeferred() + addListenerTyped(client1.communicator) { received1.complete(it) } + addListenerTyped(client2.communicator) { received2.complete(it) } + + waitForCondition(message = "clients did not connect") { server.clients().size == 2 } + server.broadcast(TestData.newBuilder().setIndex(9).setMessage("ws broadcast").build()) + + assertEquals("ws broadcast", received1.await().message) + assertEquals("ws broadcast", received2.await().message) + client1.close() + client2.close() + server.stop() + } + + @Test + fun testWsServerStopDisconnectsClients() = runBlocking { + val port = freePort() + val server = WsServer("127.0.0.1", port, "/") { conn -> + WsClient.forServer(conn, 0, 0, testParserMap()) + } + server.start() + val client = DialWs("127.0.0.1", port, "/", 0, 0, testParserMap()) + val disconnected = CompletableDeferred() + client.addDisconnectListener { disconnected.complete(Unit) } + + waitForCondition(message = "client did not connect") { server.clients().size == 1 } + server.stop() + + disconnected.await() + assertFalse(client.communicator.isAlive()) + } +} diff --git a/skills/code-review/SKILL.md b/skills/code-review/SKILL.md index 390f05c..2835000 100644 --- a/skills/code-review/SKILL.md +++ b/skills/code-review/SKILL.md @@ -1,6 +1,6 @@ --- name: code-review -description: Review completed implementation work in the current repository. Reads PLAN.md and CODE_REVIEW.md from the active task, reviews the actual changed source files, appends a verdict to CODE_REVIEW.md, then archives both files as .log. If Required or Suggested issues are found (FAIL or WARN), writes a new PLAN.md and CODE_REVIEW.md stub so the loop continues immediately. Nit-only findings may still PASS. +description: Review completed implementation work in the current repository. Reads PLAN.md and CODE_REVIEW.md from the active task, reviews the actual changed source files, appends a verdict to CODE_REVIEW.md, then archives both files as .log. On PASS, writes complete.log summarising the full loop history. If Required or Suggested issues are found (FAIL or WARN), writes a new PLAN.md and CODE_REVIEW.md stub so the loop continues immediately. Nit-only findings may still PASS. --- # Code Review @@ -24,7 +24,8 @@ Directory states: | State | Meaning | |-------|---------| | `PLAN.md` + `CODE_REVIEW.md` | Ready for review | -| Only `*.log` files | Task complete | +| `complete.log` + `*.log` files | Task complete (PASS) | +| Only `*.log` files (no `complete.log`) | Task terminated mid-loop or abandoned | The implementing agent never archives or deletes active files; archiving is this skill's responsibility. @@ -93,11 +94,20 @@ After archiving, neither active `.md` file remains unless Step 6 writes a follow ## Step 6 - Post-Review Actions -For `PASS`, report: +For `PASS`, write `tasks/{task_name}/complete.log` before reporting: + +Required fields in `complete.log`: +- `완료 일시`: date completed. +- `요약`: one-line task description and loop count. +- `루프 이력`: table of plan/code_review log pairs with their verdict. +- `최종 리뷰 요약`: bullet list of what was implemented. +- `잔여 Nit`: any Nit-only findings recorded but not acted on (omit section if none). + +Then report: - Verdict. - Archive filenames. -- Task complete; only `.log` files remain. +- `complete.log` written; task complete. For `WARN` or `FAIL`, write a new `PLAN.md` and `CODE_REVIEW.md` stub using the plan skill format: @@ -106,7 +116,75 @@ For `WARN` or `FAIL`, write a new `PLAN.md` and `CODE_REVIEW.md` stub using the - `FAIL`: one plan item per Required issue. - `WARN`: one grouped plan item for Suggested issues, plus related Nit issues if useful. - Each plan item needs problem, solution with before/after when non-trivial, checklist, test decision, intermediate verification. -- New `CODE_REVIEW.md` stub must use the same header and list every new plan item unchecked. + +`CODE_REVIEW.md` stub template (fill `{…}` placeholders; everything else is fixed and must not be changed by the implementing agent): + +```markdown + + +# Code Review Reference - {TAG} + +## 개요 + +task={task_name}, plan={N}, tag={TAG} + +## 이 파일을 읽는 리뷰 에이전트에게 + +각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요. +리뷰 완료 후 반드시 아래 순서로 아카이브하세요. + +1. `CODE_REVIEW.md` → `code_review_N.log` (N = 기존 code_review_*.log 수) +2. `PLAN.md` → `plan_M.log` (M = 기존 plan_*.log 수) +3. PASS인 경우 `complete.log` 작성 후 종료. WARN/FAIL인 경우 새 `PLAN.md` + `CODE_REVIEW.md` 스텁 작성. + +--- + +## 구현 항목별 완료 여부 + +| 항목 | 완료 여부 | +|------|---------| +| [{TAG}-1] {item description} | [ ] | +| [{TAG}-2] {item description} | [ ] | + +## 계획 대비 변경 사항 + +_구현 에이전트가 계획과 다르게 구현한 부분을 이유와 함께 기록한다._ + +## 주요 설계 결정 + +_구현 에이전트가 주요 설계 결정 사항을 기록한다._ + +## 리뷰어를 위한 체크포인트 + +{pre-filled from plan — one bullet per review focus area} + +## 검증 결과 + +_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다._ + +### {TAG}-1 중간 검증 +``` +$ {verification command from plan} +(output) +``` + +### 최종 검증 +``` +$ {final verification command from plan} +(output) +``` +``` + +Sections and their ownership: + +| 섹션 | 소유자 | 설명 | +|------|--------|------| +| 헤더 주석, 개요, 리뷰 에이전트 지시 | 스텁 생성 시 고정 | 구현 에이전트가 수정하지 않음 | +| 구현 항목별 완료 여부 (항목명) | 스텁 생성 시 고정 | `[ ]` → `[x]` 체크만 구현 에이전트가 수행 | +| 계획 대비 변경 사항, 주요 설계 결정 | 구현 에이전트가 채움 | placeholder 텍스트를 실제 내용으로 교체 | +| 리뷰어를 위한 체크포인트 | 스텁 생성 시 고정 | 계획에서 추출한 리뷰 포인트 | +| 검증 결과 (섹션 제목 + 명령) | 스텁 생성 시 고정 | 실행 출력만 구현 에이전트가 채움 | +| 코드리뷰 결과 | 리뷰 에이전트가 append | 스텁에 포함하지 않음 | Report Required/Suggested counts, archive names, and the new plan path. @@ -135,4 +213,5 @@ Report Required/Suggested counts, archive names, and the new plan path. - `code_review_N.log` exists with verdict appended. - `plan_M.log` exists. - No active `.md` files remain after PASS. -- WARN/FAIL created new active `PLAN.md` and `CODE_REVIEW.md` with matching headers. +- PASS: `complete.log` written with loop history, implementation summary, and residual Nits. +- WARN/FAIL: new active `PLAN.md` and `CODE_REVIEW.md` created with matching headers; no `complete.log`. diff --git a/tasks/kotlin_impl/code_review_0.log b/tasks/kotlin_impl/code_review_0.log new file mode 100644 index 0000000..d1eda7f --- /dev/null +++ b/tasks/kotlin_impl/code_review_0.log @@ -0,0 +1,233 @@ + + +# Code Review Reference - API + +## 개요 + +task=kotlin_impl, plan=0, tag=API + +## 구현 항목별 완료 여부 + +| 항목 | 완료 여부 | +|------|---------| +| [API-1] Kotlin 프로젝트 구조 및 proto 바인딩 설정 | [x] 구현 완료 / JVM 부재로 Gradle 검증 차단 | +| [API-2] Communicator 구현 | [x] 구현 완료 / JVM 부재로 Gradle 검증 차단 | +| [API-3] HeartbeatTimer 구현 | [x] 구현 완료 / JVM 부재로 Gradle 검증 차단 | +| [API-4] BaseClient 구현 | [x] 구현 완료 / JVM 부재로 Gradle 검증 차단 | +| [API-5] TcpClient / TcpServer 구현 | [x] 구현 완료 / JVM 부재로 Gradle 검증 차단 | +| [API-6] WsClient / WsServer 구현 | [x] 구현 완료 / JVM 부재로 Gradle 검증 차단 | +| [API-7] Heartbeat 통합 테스트 | [x] 작성 완료 / JVM 부재로 Gradle 검증 차단 | +| [API-8] Go ↔ Kotlin 크로스테스트 | [x] 작성 완료 / JVM 부재로 Kotlin 실행 차단 | + +## 계획 대비 변경 사항 + +- `go/crosstest/go_dart.go`와 `go/crosstest/go_kotlin.go`에 `//go:build ignore`를 추가했다. 두 파일 모두 단독 `go run ./crosstest/.go` 오케스트레이터라서 같은 `package main` 안에 둘 경우 `go test ./...`에서 `main` 및 helper symbol이 충돌하기 때문이다. 명시 파일 `go run`은 유지된다. +- `tools/check_proto_sync.sh`가 Kotlin proto copy도 검사하도록 확장했다. Java/Kotlin 생성용 `option java_*`만 canonical diff에서 제외한다. +- `README.md`와 `PROTOCOL.md`의 Kotlin 상태는 `Available`이 아니라 `In progress`로 기록했다. 현재 컨테이너에 Java runtime이 없어 Kotlin 테스트와 크로스테스트를 실제 통과시키지 못했기 때문이다. +- Heartbeat 통합 테스트는 `runTest` 가상 시간이 아니라 `runBlocking` 실제 시간 기반으로 작성했다. `BaseClient`가 `Dispatchers.IO` scope를 소유하는 현재 구조에서는 가상 스케줄러만으로 transport 통합 흐름이 진행되지 않는다. +- `DialWss`는 API 표면을 추가했지만, 주어진 `SSLContext`에서 `X509TrustManager`를 안전하게 복원하는 경로는 아직 보수적으로 제한되어 있다. WSS 실검증은 후속 JVM 환경에서 보강 대상이다. + +## 주요 설계 결정 + +- Kotlin proto는 schema package를 추가하지 않고 Java/Kotlin generation option만 둔다. `typeNameOf`는 `descriptorForType.fullName`을 사용하므로 현재 wire key는 Go/Dart와 같은 `TestData`, `HeartBeat`이다. +- `Communicator`는 `Channel(64)` 기반 단일 write loop로 stream write interleaving을 방지한다. pending request는 `responseNonce`로 제거하며, type mismatch는 waiting caller에 오류로 전달한다. +- `addListener`와 `addRequestListener`는 `ReentrantReadWriteLock` write lock 안에서 상호 배타 조건을 검사한다. +- `BaseClient.close()`는 `AtomicBoolean.compareAndSet(false, true)`로 멱등성을 보장하고, communicator shutdown, heartbeat stop, transport close, disconnect listener notify 순서로 처리한다. +- TCP는 4-byte big-endian length prefix와 64 MiB max packet guard를 적용한다. WebSocket은 binary frame 하나에 `PacketBase` protobuf bytes를 싣는다. +- Go crosstest runner는 Go 서버/Kotlin 클라이언트, Kotlin runner는 Kotlin 서버/Go 클라이언트 방향을 각각 담당한다. + +## 리뷰어를 위한 체크포인트 + +- Communicator의 `addListener` / `addRequestListener` 상호 배타 로직이 race-free한가? +- `close()` 멱등성: `AtomicBoolean.compareAndSet` 패턴이 모든 코드 경로에서 일관되게 적용됐는가? +- writeLoop가 `writeQueue.close()` 후 정상 종료되는가? (pending done 채널에 오류 전달 여부 확인) +- TCP readLoop에서 `length > MAX_PACKET_SIZE` 조건이 실제로 disconnect를 트리거하는가? +- WsClient OkHttp `WebSocketListener.onFailure` → `onDisconnected` 경로가 누락되지 않았는가? +- HeartbeatTimer가 `BaseClient.close()` 이후 callback을 발사하지 않는가? (scope 취소 순서 확인) +- 크로스테스트 포트(29290, 29292, 29390, 29392)가 기존 테스트 포트와 충돌하지 않는가? +- typeName이 `TestData` (단순 이름, no package prefix)로 Go / Dart 측과 일치하는가? + +## 검증 결과 + +_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다._ + +### API-1 중간 검증 +``` +$ cd kotlin && ./gradlew compileKotlin + +ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH. + +Please set the JAVA_HOME variable in your environment to match the +location of your Java installation. +``` + +### API-2 중간 검증 +``` +$ cd kotlin && ./gradlew test --tests "*.CommunicatorTest" + +ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH. + +Please set the JAVA_HOME variable in your environment to match the +location of your Java installation. +``` + +### API-3 중간 검증 +``` +$ cd kotlin && ./gradlew test --tests "*.HeartbeatTimerTest" + +Not rerun separately: same Gradle/JVM blocker as API-2. +``` + +### API-4 중간 검증 +``` +$ cd kotlin && ./gradlew compileKotlin + +Not rerun separately: same Gradle/JVM blocker as API-1. +``` + +### API-5 중간 검증 +``` +$ cd kotlin && ./gradlew test --tests "*.TcpTest" + +Not rerun separately: same Gradle/JVM blocker as API-2. +``` + +### API-6 중간 검증 +``` +$ cd kotlin && ./gradlew test --tests "*.WsTest" + +Not rerun separately: same Gradle/JVM blocker as API-2. +``` + +### API-7 중간 검증 +``` +$ cd kotlin && ./gradlew test --tests "*.HeartbeatTest" + +Not rerun separately: same Gradle/JVM blocker as API-2. +``` + +### API-8 중간 검증 +``` +$ cd go && PATH=/config/go-sdk/go/bin:/config/go/bin:$PATH GOCACHE=/tmp/go-build GOMODCACHE=/tmp/go-mod go run ./crosstest/go_kotlin.go +INFO typeName go=TestData + +ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH. + +Please set the JAVA_HOME variable in your environment to match the +location of your Java installation. + +FAIL crosstest error=kotlin-client tcp/send-push failed waitErr=exit status 1 failed=[] missing=[1 2] +exit status 1 +``` + +### 최종 검증 +``` +$ cd go && PATH=/config/go-sdk/go/bin:/config/go/bin:$PATH GOCACHE=/tmp/go-build GOMODCACHE=/tmp/go-mod go test ./... +? toki-labs.com/toki_socket/go [no test files] +? toki-labs.com/toki_socket/go/crosstest/dart_go_client [no test files] +? toki-labs.com/toki_socket/go/crosstest/kotlin_go_client [no test files] +? toki-labs.com/toki_socket/go/examples/tcp_echo [no test files] +? toki-labs.com/toki_socket/go/examples/ws_echo [no test files] +? toki-labs.com/toki_socket/go/packets [no test files] +ok toki-labs.com/toki_socket/go/test (cached) + +$ ./tools/check_proto_sync.sh +Proto schemas are in sync. + +$ cd kotlin && ./gradlew test + +ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH. + +Please set the JAVA_HOME variable in your environment to match the +location of your Java installation. +``` + +--- + +## 코드리뷰 결과 + +### 종합 판정 + +**WARN** + +--- + +### 차원별 평가 + +| 차원 | 판정 | 비고 | +|------|------|------| +| 정확성 (Correctness) | Pass | 프로토콜 wire format, nonce 단조 증가, close-once, addListener/addRequestListener 상호 배타, pending request 취소 모두 올바름 | +| 완성도 (Completeness) | Pass | 8개 항목 전부 구현됨. proto sync 검증, Go 테스트 회귀 없음 확인 | +| 테스트 커버리지 (Test coverage) | Warn | HeartbeatTest의 reset 테스트가 실제 메시지 수신 경로를 커버하지 않음 | +| API 계약 (API contract) | Warn | `crosstest/` 소스가 main sourceset에 포함되어 라이브러리 JAR에 crosstest 코드가 실려감 | +| 코드 품질 (Code quality) | Warn | `WsServer.stop()`의 `InterruptedException` 미처리, `kotlin_go.kt` 내 `runBlocking` in listener | +| 계획 대비 변경 (Plan deviation) | Pass | 모든 변경이 CODE_REVIEW.md에 기록되고 이유가 명확함 | +| 검증 신뢰도 (Verification trust) | Pass | JVM 환경 부재로 Kotlin 검증 불가, 가능한 Go 검증은 모두 실행됨. 제약이 명확히 기록됨 | + +--- + +### 발견된 문제 + +**Suggested — `WsServer.stop()`: `super.stop(1000)` `InterruptedException` 미처리** +`kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsServer.kt:36-41` + +`WebSocketServer.stop(int)` 는 Java 메서드로 `InterruptedException` 을 선언한다. Kotlin은 checked exception을 컴파일 타임에 강제하지 않지만, `runBlocking` 컨텍스트에서 `InterruptedException` 이 던져지면 현재 coroutine이 취소 신호로 처리해 `stop()` 이후 로직이 실행되지 않는다. + +```kotlin +// 현재 +super.stop(1000) + +// 수정 +runCatching { super.stop(1000) } +``` + +--- + +**Suggested — `crosstest/` 코드가 main sourceset에 포함됨** +`kotlin/build.gradle.kts:14` + +```kotlin +named("main") { + kotlin.srcDir("crosstest") +} +``` + +`kotlin_go.kt` 와 `go_kotlin_client/` 의 `Main.kt` 가 라이브러리 JAR에 함께 포함된다. crosstest 코드는 별도 sourceset(`crosstestMain`) 이나 `application` 전용 소스 경로로 분리해야 한다. + +--- + +**Suggested — `kotlin_go.kt` listener callback 내 `runBlocking` 사용** +`kotlin/crosstest/kotlin_go.kt:56-63`, `104-111` + +```kotlin +addListenerTyped(client.communicator) { data -> + ... + if (valid) { + runBlocking { client.send(...) } // ← Dispatchers.IO 스레드 블로킹 + } +} +``` + +`onReceivedData` 는 `TcpClient.readLoop` 의 `Dispatchers.IO` coroutine 내에서 호출된다. listener callback에서 `runBlocking` 을 호출하면 IO 스레드를 점유한다. `scope.launch { client.send(...) }` 또는 listener를 `suspend` 람다로 받는 패턴으로 교체해야 한다. + +현재 `addListenerTyped` 의 시그니처가 `fn: (T) -> Unit` 이므로 즉시 수정하려면 코드 내부에서 `scope.launch { }` 로 감싸거나, crosstest 코드 한정으로 별도 scope를 사용해야 한다. + +--- + +**Suggested — `HeartbeatTest.testHeartbeatResetOnReceive()`: 수신 경로 미검증** +`kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTest.kt:69-81` + +테스트 이름은 "receive 시 타이머 리셋"이지만 실제로는 `sendHeartBeat()` 를 직접 두 번 호출한다. 실제 경로인 `onReceivedData()` → `sendHeartBeat()` 를 통한 타이머 리셋이 검증되지 않는다. 테스트에서 `TcpClient` 또는 `HeartbeatClient` 의 `communicator.onReceivedData(...)` 를 호출하고 heartbeat가 지연되는지 확인해야 한다. + +--- + +**Nit — `Communicator.parse()` 가 public으로 노출됨** +`kotlin/src/main/kotlin/com/tokilabs/toki_socket/Communicator.kt:266` + +Go 구현에서는 `parse` 가 unexported이다. Kotlin에서도 `internal` 로 제한하는 것이 적절하다. 현재 inline helper (`addRequestListenerTyped`) 는 `queuePacket`, `nextNonce` 만 필요하고 `parse` 를 직접 호출하지 않는다. + +--- + +### 다음 단계 + +WARN: Suggested 항목을 수정하는 후속 PLAN.md 가 작성된다. 수정 완료 후 재리뷰. diff --git a/tasks/kotlin_impl/code_review_1.log b/tasks/kotlin_impl/code_review_1.log new file mode 100644 index 0000000..48134e2 --- /dev/null +++ b/tasks/kotlin_impl/code_review_1.log @@ -0,0 +1,254 @@ + + +# Code Review Reference - REVIEW_API + +## 개요 + +task=kotlin_impl, plan=1, tag=REVIEW_API + +## 구현 항목별 완료 여부 + +| 항목 | 완료 여부 | +|------|---------| +| [REVIEW_API-1] `WsServer.stop()` `InterruptedException` 미처리 수정 | [x] | +| [REVIEW_API-2] `crosstest/` 를 main sourceset에서 분리 | [x] | +| [REVIEW_API-3] `kotlin_go.kt` listener 내 `runBlocking` 제거 | [x] | +| [REVIEW_API-4] `HeartbeatTest.testHeartbeatTimerResetOnReceivedData` 테스트 추가 | [x] | +| [REVIEW_API-5] `Communicator.parse()` visibility `internal` 로 제한 | [x] | + +## 계획 대비 변경 사항 + +- REVIEW_API-3의 listener send는 계획의 `client.scope.launch` 대신 `runTcpSendPush`/`runWsSendPush`를 `coroutineScope`로 감싸고 해당 scope의 `launch`를 사용했다. `crosstest`를 main sourceset에서 분리하면 main의 `internal` 멤버 접근이 별도 compilation에서 막힐 수 있어, crosstest runner가 main 내부 scope에 의존하지 않도록 했다. +- REVIEW_API-3 구현 중 `WsClient.forServer`도 `internal`이면 분리된 crosstest sourceset에서 `kotlin_go.kt`가 접근할 수 없다. 또한 public `WsServer` 생성자에서 기본 server-side `WsClient`를 만들 방법이 필요하므로 `WsClient.forServer`를 public companion factory로 조정했다. +- REVIEW_API-3 검증 중 분리된 crosstest sourceset에서 `parserMap()`의 `TestData.parseFrom(it)` overload 추론이 모호해져 `parserMap(): ParserMap` 반환 타입을 명시했다. +- REVIEW_API-4의 수신 경로 테스트는 `TestData` listener를 등록하고 `received` 플래그를 확인해 `onReceivedData()`가 실제 listener dispatch까지 통과했는지도 검증한다. +- 최종 검증 중 `TcpTest` class 실행이 timeout 된 뒤 메서드 단위로 분리해 확인했다. `testTcpClientCloseIdempotent`의 blocking `ServerSocket.accept()`를 `Dispatchers.IO`에서 실행하도록 바꿔 테스트 스레드 점유 위험을 낮췄고, 이후 `TcpTest`, `WsTest`, 전체 `./gradlew test`가 통과했다. + +## 주요 설계 결정 + +- `crosstest` sourceset은 main 출력과 runtime classpath를 compile/runtime classpath로 갖도록 구성했다. +- `run` task는 `crosstest` runtime classpath에서 기본 crosstest main class를 실행하도록 재구성했다. +- `BaseClient.scope`는 계획대로 `internal`로 제한했지만, crosstest runner는 이 scope를 직접 참조하지 않는다. +- Gradle 검증은 `JAVA_HOME=/config/opt/jdk/jdk-17.0.10+7`, `GRADLE_USER_HOME=/tmp/gradle` 환경으로 실행했다. + +## 리뷰어를 위한 체크포인트 + +- `crosstest` sourceset 분리 후 `./gradlew run` 이 여전히 `MainKt` 를 실행하는가? +- `BaseClient.scope` 를 `internal` 로 노출 시 외부 모듈에서 scope를 직접 조작하는 위험이 없는가? +- `testHeartbeatTimerResetOnReceivedData` 가 `onReceivedData` 경로를 실제로 통과하는가? +- JAR 에 crosstest 클래스가 포함되지 않는가? + +## 검증 결과 + +_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다._ + +### REVIEW_API-1 중간 검증 +``` +$ ./gradlew compileKotlin +> Task :checkKotlinGradlePluginConfigurationErrors SKIPPED +> Task :extractIncludeProto UP-TO-DATE +> Task :extractProto UP-TO-DATE +> Task :generateProto UP-TO-DATE +> Task :compileKotlin UP-TO-DATE + +BUILD SUCCESSFUL in 13s +4 actionable tasks: 4 up-to-date +``` + +### REVIEW_API-2 중간 검증 +``` +$ ./gradlew compileKotlin compileCrosstestKotlin +> Task :checkKotlinGradlePluginConfigurationErrors SKIPPED +> Task :extractIncludeProto UP-TO-DATE +> Task :extractProto UP-TO-DATE +> Task :generateProto UP-TO-DATE +> Task :compileKotlin UP-TO-DATE +> Task :compileJava UP-TO-DATE +> Task :processResources UP-TO-DATE +> Task :classes UP-TO-DATE +> Task :extractCrosstestProto UP-TO-DATE +> Task :extractIncludeCrosstestProto UP-TO-DATE +> Task :generateCrosstestProto NO-SOURCE +> Task :compileCrosstestKotlin + +BUILD SUCCESSFUL in 11s +9 actionable tasks: 1 executed, 8 up-to-date + +$ ./gradlew jar +> Task :checkKotlinGradlePluginConfigurationErrors SKIPPED +> Task :extractIncludeProto UP-TO-DATE +> Task :extractProto UP-TO-DATE +> Task :generateProto UP-TO-DATE +> Task :compileKotlin UP-TO-DATE +> Task :compileJava UP-TO-DATE +> Task :processResources UP-TO-DATE +> Task :classes UP-TO-DATE +> Task :jar + +BUILD SUCCESSFUL in 3s +7 actionable tasks: 1 executed, 6 up-to-date + +$ jar tf build/libs/toki-socket-kotlin-0.1.0.jar | grep crosstest +(no output) +``` + +### REVIEW_API-3 중간 검증 +``` +$ ./gradlew compileCrosstestKotlin +> Task :checkKotlinGradlePluginConfigurationErrors SKIPPED +> Task :extractIncludeProto UP-TO-DATE +> Task :extractProto UP-TO-DATE +> Task :generateProto UP-TO-DATE +> Task :compileKotlin UP-TO-DATE +> Task :compileJava UP-TO-DATE +> Task :processResources UP-TO-DATE +> Task :classes UP-TO-DATE +> Task :extractCrosstestProto UP-TO-DATE +> Task :extractIncludeCrosstestProto UP-TO-DATE +> Task :generateCrosstestProto NO-SOURCE +> Task :compileCrosstestKotlin UP-TO-DATE + +BUILD SUCCESSFUL in 3s +9 actionable tasks: 9 up-to-date +``` + +### REVIEW_API-4 중간 검증 +``` +$ ./gradlew test --tests "*.HeartbeatTest" +> Task :checkKotlinGradlePluginConfigurationErrors SKIPPED +> Task :extractIncludeProto UP-TO-DATE +> Task :extractProto UP-TO-DATE +> Task :generateProto UP-TO-DATE +> Task :compileKotlin UP-TO-DATE +> Task :compileJava UP-TO-DATE +> Task :processResources UP-TO-DATE +> Task :classes UP-TO-DATE +> Task :extractIncludeTestProto UP-TO-DATE +> Task :extractTestProto UP-TO-DATE +> Task :generateTestProto NO-SOURCE +> Task :compileTestKotlin UP-TO-DATE +> Task :compileTestJava NO-SOURCE +> Task :processTestResources NO-SOURCE +> Task :testClasses UP-TO-DATE +> Task :test + +BUILD SUCCESSFUL in 15s +10 actionable tasks: 1 executed, 9 up-to-date +``` + +### REVIEW_API-5 중간 검증 +``` +$ ./gradlew compileKotlin compileTestKotlin +> Task :checkKotlinGradlePluginConfigurationErrors SKIPPED +> Task :extractIncludeProto UP-TO-DATE +> Task :extractProto UP-TO-DATE +> Task :generateProto UP-TO-DATE +> Task :compileKotlin UP-TO-DATE +> Task :compileJava UP-TO-DATE +> Task :processResources UP-TO-DATE +> Task :classes UP-TO-DATE +> Task :extractIncludeTestProto UP-TO-DATE +> Task :extractTestProto UP-TO-DATE +> Task :generateTestProto NO-SOURCE +> Task :compileTestKotlin + +BUILD SUCCESSFUL in 8s +9 actionable tasks: 1 executed, 8 up-to-date +``` + +### 최종 검증 +``` +$ ./gradlew test +> Task :checkKotlinGradlePluginConfigurationErrors SKIPPED +> Task :extractIncludeProto UP-TO-DATE +> Task :extractProto UP-TO-DATE +> Task :generateProto UP-TO-DATE +> Task :compileKotlin UP-TO-DATE +> Task :compileJava UP-TO-DATE +> Task :processResources UP-TO-DATE +> Task :classes UP-TO-DATE +> Task :extractIncludeTestProto UP-TO-DATE +> Task :extractTestProto UP-TO-DATE +> Task :generateTestProto NO-SOURCE +> Task :compileTestKotlin UP-TO-DATE +> Task :compileTestJava NO-SOURCE +> Task :processTestResources NO-SOURCE +> Task :testClasses UP-TO-DATE +> Task :test + +BUILD SUCCESSFUL in 16s +10 actionable tasks: 1 executed, 9 up-to-date + +$ ./gradlew jar +> Task :checkKotlinGradlePluginConfigurationErrors SKIPPED +> Task :extractIncludeProto UP-TO-DATE +> Task :extractProto UP-TO-DATE +> Task :generateProto UP-TO-DATE +> Task :compileKotlin UP-TO-DATE +> Task :compileJava UP-TO-DATE +> Task :processResources UP-TO-DATE +> Task :classes UP-TO-DATE +> Task :jar + +BUILD SUCCESSFUL in 3s +7 actionable tasks: 1 executed, 6 up-to-date + +$ jar tf build/libs/toki-socket-kotlin-0.1.0.jar | grep crosstest +(no output) + +$ /usr/bin/bash -lc 'PATH=/config/go-sdk/go/bin:/config/go/bin:$PATH GOCACHE=/tmp/go-build GOMODCACHE=/tmp/go-mod go test ./...' +? toki-labs.com/toki_socket/go [no test files] +? toki-labs.com/toki_socket/go/crosstest/dart_go_client [no test files] +? toki-labs.com/toki_socket/go/crosstest/kotlin_go_client [no test files] +? toki-labs.com/toki_socket/go/examples/tcp_echo [no test files] +? toki-labs.com/toki_socket/go/examples/ws_echo [no test files] +? toki-labs.com/toki_socket/go/packets [no test files] +ok toki-labs.com/toki_socket/go/test (cached) + +$ bash tools/check_proto_sync.sh +Proto schemas are in sync. +``` + +--- + +## 코드리뷰 결과 + +### 종합 판정 + +**PASS** + +--- + +### 차원별 평가 + +| 차원 | 판정 | 비고 | +|------|------|------| +| 정확성 (Correctness) | Pass | REVIEW_API-1~5 모두 의도대로 구현됨. 멱등성, race 보호, crosstest 분리 정상 | +| 완성도 (Completeness) | Pass | 5개 항목 전부 구현. `./gradlew test`, `jar`, Go 테스트, proto sync 통과 | +| 테스트 커버리지 (Test coverage) | Pass | `testHeartbeatTimerResetOnReceivedData` 추가. listener dispatch 경로 검증 포함 | +| API 계약 (API contract) | Pass | crosstest가 JAR에서 제외됨 확인. `parse()` internal 제한 | +| 코드 품질 (Code quality) | Pass | `runBlocking` in listener 제거, `InterruptedException` 처리 완료 | +| 계획 대비 변경 (Plan deviation) | Pass | 모든 변경이 CODE_REVIEW.md에 이유와 함께 기록됨 | +| 검증 신뢰도 (Verification trust) | Pass | JVM 환경에서 전 항목 실제 실행 확인 | + +--- + +### 발견된 문제 + +**Nit — `testHeartbeatTimerResetOnReceivedData` 테스트 이름과 검증 범위 불일치** +`kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTest.kt:84-106` + +테스트 이름은 "수신 데이터에 의한 타이머 리셋"이지만, 타이머를 실제로 리셋하는 것은 `onReceivedData` 이후 명시적으로 호출하는 `sendHeartBeat()`이다. `onReceivedData` 자체가 타이머를 리셋하는 경로(`TcpClient.readLoop`, `WsClient.receiveBytes`에서 `sendHeartBeat()` 호출)는 `HeartbeatClient`에서 재현되지 않는다. `received` 플래그 확인으로 listener dispatch는 검증되나, 테스트 이름이 과장된다. 블로킹 이슈는 아님. + +--- + +**Info — `build.gradle.kts` `application.mainClass` 중복 설정** +`kotlin/build.gradle.kts:46-60` + +`application { mainClass }` 와 `tasks.named("run") { mainClass }` 에 동일한 값이 중복 설정되어 있다. `tasks.named("run")`이 오버라이드하므로 `application { }` 블록의 설정은 실질적으로 무효이다. 기능상 문제는 없으나 `application { mainClass }` 제거 또는 주석 추가가 명확하다. + +--- + +### 다음 단계 + +PASS: 후속 PLAN.md 없음. kotlin_impl 완료. diff --git a/tasks/kotlin_impl/complete.log b/tasks/kotlin_impl/complete.log new file mode 100644 index 0000000..89dac73 --- /dev/null +++ b/tasks/kotlin_impl/complete.log @@ -0,0 +1,31 @@ +# Task Complete — kotlin_impl + +## 완료 일시 + +2026-04-12 + +## 요약 + +Kotlin 구현 후속 수정 (REVIEW_API) — plan=1, tag=REVIEW_API + +plan-code-review 루프 2회 완료 후 PASS 판정으로 종료. + +## 루프 이력 + +| 회차 | 계획 | 리뷰 | 판정 | +|------|------|------|------| +| 0 | plan_0.log | code_review_0.log | WARN | +| 1 | plan_1.log | code_review_1.log | PASS | + +## 최종 리뷰 요약 (plan=1) + +- REVIEW_API-1: `WsServer.stop()` `InterruptedException` 미처리 → `runCatching` 으로 처리 +- REVIEW_API-2: `crosstest/` 를 main sourceset에서 분리 → JAR에 crosstest 미포함 확인 +- REVIEW_API-3: `kotlin_go.kt` listener 내 `runBlocking` 제거 → `coroutineScope` + `launch` 대체 +- REVIEW_API-4: `testHeartbeatTimerResetOnReceivedData` 테스트 추가 +- REVIEW_API-5: `Communicator.parse()` visibility `internal` 로 제한 + +## 잔여 Nit + +- `testHeartbeatTimerResetOnReceivedData` 테스트 이름이 검증 범위를 과장 (블로킹 아님) +- `build.gradle.kts` `application.mainClass` 중복 설정 (기능 영향 없음) diff --git a/tasks/kotlin_impl/plan_0.log b/tasks/kotlin_impl/plan_0.log new file mode 100644 index 0000000..733256b --- /dev/null +++ b/tasks/kotlin_impl/plan_0.log @@ -0,0 +1,807 @@ + + +# Kotlin 구현체 신규 추가 + +## 이 파일을 읽는 구현 에이전트에게 + +각 항목의 체크리스트를 하나씩 완료 처리하고, 중간 검증 명령을 실제로 실행한 뒤 출력을 `CODE_REVIEW.md`의 `검증 결과` 섹션에 붙여 넣으세요. +계획과 다르게 구현한 부분이 있으면 `계획 대비 변경 사항`에 이유와 함께 기록하세요. +`CODE_REVIEW.md`의 모든 섹션을 실제 구현 내용으로 채운 뒤 코드 리뷰를 요청하세요. + +--- + +## 배경 + +`PROTOCOL.md`에 기술된 Toki Socket 프로토콜의 Kotlin 구현체가 없다. Go/Dart 레퍼런스 구현체를 기준으로 동일한 wire format(TCP 4-byte big-endian framing, WebSocket binary frame), typeName 라우팅, request-response nonce 상관관계, heartbeat 자동 처리를 Kotlin coroutine 관용 패턴으로 구현한다. 구현 완료 기준은 같은 언어 단위 테스트 통과와 Go ↔ Kotlin 양방향 크로스테스트 통과이다. + +--- + +## 의존 관계 및 구현 순서 + +API-1 → API-2 → API-3 → API-4 → API-5 → API-6 → API-7 → API-8 + +--- + +### [API-1] Kotlin 프로젝트 구조 및 protobuf 바인딩 설정 + +#### 문제 + +`kotlin/` 디렉터리가 없고 프로젝트 파일, proto 바인딩이 없다. + +#### 해결 방법 + +Gradle 멀티플랫폼이 아닌 순수 JVM 라이브러리 프로젝트로 설정한다. 빌드 도구는 Gradle Kotlin DSL(`build.gradle.kts`)을 사용한다. + +**디렉터리 구조:** + +``` +kotlin/ + build.gradle.kts + settings.gradle.kts + gradlew (gradle wrapper) + gradlew.bat + gradle/wrapper/ + src/ + main/ + kotlin/com/tokilabs/toki_socket/ + proto/ ← canonical proto 복사본 + test/ + kotlin/com/tokilabs/toki_socket/ + crosstest/ + go_kotlin_client/ ← Kotlin subprocess (Go 서버 ↔ Kotlin 클라이언트) + kotlin_go.kt ← Kotlin 서버 오케스트레이터 (Kotlin 서버 ↔ Go 클라이언트) +``` + +**`build.gradle.kts` 핵심 의존성:** + +```kotlin +plugins { + kotlin("jvm") version "2.0.0" + id("com.google.protobuf") version "0.9.4" +} + +dependencies { + implementation("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.8.1") + implementation("com.google.protobuf:protobuf-kotlin:4.27.0") + implementation("com.squareup.okhttp3:okhttp:4.12.0") // WS client + implementation("org.java-websocket:Java-WebSocket:1.5.6") // WS server + testImplementation("org.jetbrains.kotlinx:kotlinx-coroutines-test:1.8.1") + testImplementation(kotlin("test")) +} + +protobuf { + protoc { artifact = "com.google.protobuf:protoc:4.27.0" } + generateProtoTasks { + all().forEach { task -> + task.builtins { + id("kotlin") + } + } + } +} +``` + +**proto 동기화:** +`dart/lib/src/packets/message_common.proto`에서 `kotlin/src/main/proto/message_common.proto`로 복사. Go 처럼 언어별 option(`option java_package`, `option java_outer_classname`)을 추가할 수 있으나 **message schema는 canonical proto와 동일하게 유지**. `tools/check_proto_sync.sh`가 message 필드 diff를 검출한다. + +#### 수정 파일 및 체크리스트 + +- [ ] `kotlin/settings.gradle.kts` 생성 +- [ ] `kotlin/build.gradle.kts` 생성 (위 의존성 포함) +- [ ] Gradle wrapper 생성 (`gradle wrapper` 실행 또는 수동 배치) +- [ ] `kotlin/src/main/proto/message_common.proto` 생성 (canonical proto 복사 + java options 추가) +- [ ] `./gradlew generateProto` 실행하여 바인딩 생성 확인 + +#### 테스트 작성 + +SKIP — 이 항목은 빌드 인프라 설정이며 동작 검증은 이후 항목의 테스트로 커버된다. + +#### 중간 검증 + +```bash +cd kotlin +./gradlew compileKotlin +# 예상: BUILD SUCCESSFUL (proto 바인딩 포함 컴파일) +``` + +--- + +### [API-2] Communicator 구현 + +#### 문제 + +Kotlin에 `Communicator`, `Transport`, `ParserMap` 타입이 없다. + +#### 해결 방법 + +Go `communicator.go` 구조를 Kotlin coroutine 관용 패턴으로 이식한다. + +**파일:** `kotlin/src/main/kotlin/com/tokilabs/toki_socket/Communicator.kt` + +**핵심 매핑:** + +| Go | Kotlin | +|----|--------| +| `Transport` interface | `interface Transport` | +| `ParserMap` | `typealias ParserMap = Map MessageLite>` | +| `atomic.Int32` (nonce) | `AtomicInteger` | +| `atomic.Bool` (isAlive) | `AtomicBoolean` | +| `sync.RWMutex` | `ReentrantReadWriteLock` | +| `channel queuedPacket (64)` | `Channel(capacity = 64)` | +| `closed chan struct{}` | `Job` (coroutine cancellation) | +| `sync.Once` (closeOnce) | 없음 — `shutdown()`은 `closeOnce` 없이 AtomicBoolean + coroutine cancel | +| goroutine writeLoop | `scope.launch { writeLoop() }` | + +**typeName 추출:** + +```kotlin +fun typeNameOf(m: MessageLite): String = + (m as com.google.protobuf.Message).descriptorForType.fullName +``` + +proto에 `package` 선언이 없으므로 `fullName` = `"TestData"`, `"HeartBeat"` 등 단순 이름. PROTOCOL.md Kotlin 행과 일치. + +**`addListener` / `addRequestListener` 상호 배타:** + +```kotlin +fun addRequestListener(typeName: String, fn: (MessageLite, Int) -> Unit) { + val lock = rwLock.writeLock() + lock.lock() + try { + check(handlers[typeName].isNullOrEmpty()) { + "type $typeName is already registered with addListener" + } + check(!reqHandlers.containsKey(typeName)) { + "type $typeName is already registered with addRequestListener" + } + reqHandlers[typeName] = fn + } finally { lock.unlock() } +} +``` + +위반 시 `IllegalStateException`(Go의 `panic` 대응). + +**`writeLoop`:** + +```kotlin +private suspend fun writeLoop() { + for (item in writeQueue) { + val err = runCatching { transport.writePacket(item.base) } + item.done.complete(err.exceptionOrNull()) + if (err.isFailure) { + writeErrorHandler?.invoke(err.exceptionOrNull()!!) + } + } +} +``` + +**`sendRequest`:** + +```kotlin +suspend fun sendRequest( + req: MessageLite, + resTypeName: String, + timeoutMs: Long = 30_000L +): MessageLite { + ... + withTimeout(timeoutMs) { + select { + pending.ch.onReceive { it } + pending.errCh.onReceive { throw it } + } + } +} +``` + +#### 수정 파일 및 체크리스트 + +- [ ] `Communicator.kt` 생성 + - [ ] `interface Transport { suspend fun writePacket(base: PacketBase); fun close() }` + - [ ] `typealias ParserMap = Map MessageLite>` + - [ ] `class Communicator(transport, parserMap, scope)` 생성자 + - [ ] `initialize()`: HeartBeat 파서 자동 등록, writeLoop launch + - [ ] `isAlive(): Boolean` + - [ ] `nextNonce()`: AtomicInteger.incrementAndGet() + - [ ] `shutdown()`: isAlive=false, writeQueue.close(), scope 내부 채널 정리 + - [ ] `close()`: shutdown() + transport.close() + - [ ] `queuePacket(base)`: writeQueue에 enqueue, done 대기 + - [ ] `send(m)`: marshal → queuePacket + - [ ] `sendRequest(req, resTypeName, timeout)`: nonce 등록 → queuePacket → withTimeout select + - [ ] `addListener(typeName, fn)`: 상호 배타 체크 + - [ ] `removeListeners(typeName)` + - [ ] `addRequestListener(typeName, fn)`: 상호 배타 체크 + - [ ] `onReceivedData(typeName, data, nonce, responseNonce)` + - [ ] `handleResponse(typeName, data, responseNonce)` + - [ ] `parse(typeName, data)` + - [ ] `removePending(nonce)` +- [ ] 타입 헬퍼 함수 (Go의 제네릭 헬퍼 대응): + - [ ] `inline fun addListenerTyped(communicator, fn)` + - [ ] `inline fun addRequestListenerTyped(communicator, fn)` + - [ ] `inline fun sendRequestTyped(communicator, req, timeoutMs)` + +#### 테스트 작성 + +**파일:** `kotlin/src/test/kotlin/com/tokilabs/toki_socket/CommunicatorTest.kt` + +| 테스트명 | 검증 목표 | +|---------|---------| +| `testSendRequestTimeout` | timeout 경과 시 exception 발생 | +| `testSendRequestTypeMismatch` | 응답 typeName 불일치 시 exception 발생 | +| `testListenerAndRequestListenerConflict` | 동일 typeName 이중 등록 시 IllegalStateException | +| `testSendFireAndForget` | send 후 fakeTransport에 패킷 1개 기록됨 | + +#### 중간 검증 + +```bash +cd kotlin +./gradlew test --tests "*.CommunicatorTest" +# 예상: 4개 테스트 PASS +``` + +--- + +### [API-3] HeartbeatTimer 구현 + +#### 문제 + +주기적 heartbeat 발송을 위한 취소 가능한 타이머가 없다. + +#### 해결 방법 + +Go의 `HeartbeatTimer`를 coroutine `delay` 기반으로 이식한다. + +**파일:** `kotlin/src/main/kotlin/com/tokilabs/toki_socket/HeartbeatTimer.kt` + +```kotlin +class HeartbeatTimer( + private val scope: CoroutineScope, + private val delayMs: Long, + private val callback: suspend () -> Unit +) { + private var job: Job? = null + + fun reset(delayMs: Long = this.delayMs) { + job?.cancel() + job = scope.launch { + delay(delayMs) + callback() + } + } + + fun stop() { + job?.cancel() + job = null + } +} +``` + +`scope`는 `BaseClient`에서 생성된 `CoroutineScope(SupervisorJob() + Dispatchers.IO)`를 전달한다. + +#### 수정 파일 및 체크리스트 + +- [ ] `HeartbeatTimer.kt` 생성 + - [ ] `reset(delayMs)`: 기존 job cancel 후 새 delay job launch + - [ ] `stop()`: job cancel + +#### 테스트 작성 + +**파일:** `kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTimerTest.kt` + +| 테스트명 | 검증 목표 | +|---------|---------| +| `testCallbackFires` | delay 후 callback 호출됨 | +| `testStopPreventsCallback` | stop() 후 callback 미호출 | +| `testResetRestartsTimer` | reset() 후 이전 callback 미호출, 새 delay 후 호출 | + +#### 중간 검증 + +```bash +cd kotlin +./gradlew test --tests "*.HeartbeatTimerTest" +# 예상: 3개 테스트 PASS +``` + +--- + +### [API-4] BaseClient 구현 + +#### 문제 + +heartbeat 로직, disconnect 리스너, connCloseOnce(close-once) 공통 구조가 없다. + +#### 해결 방법 + +Go의 `baseClient[Self]`를 `abstract class BaseClient>`로 이식한다. + +**파일:** `kotlin/src/main/kotlin/com/tokilabs/toki_socket/BaseClient.kt` + +**핵심 매핑:** + +| Go | Kotlin | +|----|--------| +| `self Self` | 생성자 파라미터로 `self: Self` 수신 | +| `connCloseOnce sync.Once` | `AtomicBoolean` + `compareAndSet(false, true)` | +| `hbMu sync.Mutex` | `Mutex` (kotlinx.coroutines) | +| `disconnectListeners` | `CopyOnWriteArrayList<(Self) -> Unit>` | +| `doClose func() error` | 생성자 람다 `doClose: suspend () -> Unit` | + +```kotlin +abstract class BaseClient>( + private val self: Self, + intervalSec: Int, + waitSec: Int, + private val doClose: suspend () -> Unit +) { + protected val scope = CoroutineScope(SupervisorJob() + Dispatchers.IO) + abstract val communicator: Communicator + + private val heartbeatIntervalMs = intervalSec * 1000L + private val heartbeatWaitMs = waitSec * 1000L + private val closedOnce = AtomicBoolean(false) + private val hbMutex = Mutex() + private var hbTimer: HeartbeatTimer? = null + private var waitingHBResponse = false + private val disconnectListeners = CopyOnWriteArrayList<(Self) -> Unit>() + + fun addDisconnectListener(handler: (Self) -> Unit) { disconnectListeners.add(handler) } + fun removeDisconnectListeners() { disconnectListeners.clear() } + + suspend fun close() { + if (!closedOnce.compareAndSet(false, true)) return + communicator.shutdown() + stopHeartbeat() + doClose() + notifyDisconnected() + scope.cancel() + } + + // sendHeartBeat, onHeartBeat, stopHeartbeat, onDisconnected, notifyDisconnected + // Go 로직과 동일하게 구현 +} +``` + +`close()`는 `connCloseOnce.compareAndSet(false, true)` 패턴으로 멱등성 보장. Go의 `sync.Once`와 동일 의미. + +#### 수정 파일 및 체크리스트 + +- [ ] `BaseClient.kt` 생성 + - [ ] `scope`: `CoroutineScope(SupervisorJob() + Dispatchers.IO)` + - [ ] `close()`: compareAndSet + communicator.shutdown() + stopHeartbeat() + doClose() + notifyDisconnected() + scope.cancel() + - [ ] `sendHeartBeat()`: hbMutex 잠금, 이전 타이머 정지, 새 interval 타이머 → HeartBeat 전송 → waitingHBResponse=true → wait 타이머 → onDisconnected + - [ ] `onHeartBeat()`: waitingHBResponse 분기 처리 (Go 로직 그대로) + - [ ] `stopHeartbeat()`: hbTimer?.stop() + - [ ] `onDisconnected()`: close() 호출 + - [ ] `notifyDisconnected()`: listener 복사 후 순회 호출 + +#### 테스트 작성 + +SKIP — heartbeat 통합 동작은 API-7의 HeartbeatTest에서 TcpClient를 통해 검증한다. BaseClient 자체는 추상 클래스여서 단독 단위 테스트가 어렵다. + +#### 중간 검증 + +```bash +cd kotlin +./gradlew compileKotlin +# 예상: BUILD SUCCESSFUL (API-5 구현 전이므로 컴파일만 확인) +``` + +--- + +### [API-5] TcpClient / TcpServer 구현 + +#### 문제 + +Kotlin에 TCP transport가 없다. + +#### 해결 방법 + +Go의 `TcpClient`, `TcpServer`를 `java.net.Socket` / `ServerSocket` + coroutine으로 이식한다. + +**파일:** +- `kotlin/src/main/kotlin/com/tokilabs/toki_socket/TcpClient.kt` +- `kotlin/src/main/kotlin/com/tokilabs/toki_socket/TcpServer.kt` + +**TcpClient:** + +```kotlin +class TcpClient private constructor( + private val socket: Socket, + intervalSec: Int, + waitSec: Int, +) : BaseClient( + self = /* 순환 참조 해결: lateinit + apply */ ..., + intervalSec = intervalSec, + waitSec = waitSec, + doClose = { socket.close() } +) { + override val communicator: Communicator = Communicator(this, parserMap, scope) + private val writeMutex = Mutex() + + // TCP Transport 구현 + suspend fun writePacket(base: PacketBase) { + val bytes = base.toByteArray() + val header = ByteBuffer.allocate(4).putInt(bytes.size).array() + writeMutex.withLock { + withContext(Dispatchers.IO) { + socket.getOutputStream().write(header) + socket.getOutputStream().write(bytes) + } + } + } + fun closeTransport() { socket.close() } + + private fun readLoop() { + scope.launch(Dispatchers.IO) { + val input = socket.getInputStream() + val header = ByteArray(4) + while (communicator.isAlive()) { + try { + input.readFully(header) + val length = ByteBuffer.wrap(header).int + if (length == 0) continue + if (length > MAX_PACKET_SIZE) { onDisconnected(); return@launch } + val bytes = ByteArray(length) + input.readFully(bytes) + val base = PacketBase.parseFrom(bytes) + communicator.onReceivedData(base.typeName, base.data.toByteArray(), base.nonce, base.responseNonce) + sendHeartBeat() + } catch (e: Exception) { + onDisconnected(); return@launch + } + } + } + } +} +``` + +`InputStream.readFully`는 `java.io.DataInputStream` wrapping으로 구현 (`readFully` extension 함수 정의). + +**TcpServer:** + +Go의 `TcpServer`와 동일 구조. `ServerSocket.accept()`를 `Dispatchers.IO`에서 loop. + +```kotlin +class TcpServer( + private val host: String, + private val port: Int, + private val newClient: (Socket) -> TcpClient, +) { + private val scope = CoroutineScope(SupervisorJob() + Dispatchers.IO) + private val clients = CopyOnWriteArrayList() + private var serverSocket: ServerSocket? = null + var onClientConnected: (TcpClient) -> Unit = {} + + fun start() { ... } + fun stop() { ... } + fun broadcast(m: MessageLite) { ... } + fun clients(): List { ... } +} +``` + +#### 수정 파일 및 체크리스트 + +- [ ] `TcpClient.kt` 생성 + - [ ] `companion object { const val MAX_PACKET_SIZE = 64 * 1024 * 1024 }` + - [ ] `Transport` 구현: `writePacket` (4-byte BE header + proto bytes), `close` + - [ ] `readLoop()`: header 4바이트 읽기 → length=0 skip → length>MAX reject → bytes 읽기 → PacketBase.parseFrom → onReceivedData → sendHeartBeat + - [ ] `DialTcp(host, port, intervalSec, waitSec, parserMap)` 팩토리 함수 + - [ ] HeartBeat 리스너 등록 (`communicator.addListener(HeartBeat typeName, ::onHeartBeat)`) + - [ ] WriteErrorHandler 등록 (`communicator.setWriteErrorHandler { onDisconnected() }`) +- [ ] `TcpServer.kt` 생성 + - [ ] `start()`: ServerSocket bind → accept loop (Dispatchers.IO) + - [ ] `stop()`: serverSocket.close() → clients 복사 후 전부 close + - [ ] `broadcast(m)`: clients 순회하여 send + - [ ] 클라이언트 disconnect 시 clients 리스트에서 제거 (addDisconnectListener 활용) + +#### 테스트 작성 + +**파일:** `kotlin/src/test/kotlin/com/tokilabs/toki_socket/TcpTest.kt` + +| 테스트명 | 검증 목표 | +|---------|---------| +| `testTcpSendReceive` | client.send → server addListener 수신 | +| `testTcpRequestResponse` | sendRequestTyped → index*2, "echo: msg" | +| `testTcpBroadcast` | server.broadcast → 연결된 클라이언트 수신 | +| `testTcpServerStopDisconnectsClients` | server.stop() → client disconnect 콜백 | +| `testTcpClientCloseIdempotent` | close() 3회 호출 시 오류 없음 | + +#### 중간 검증 + +```bash +cd kotlin +./gradlew test --tests "*.TcpTest" +# 예상: 5개 테스트 PASS +``` + +--- + +### [API-6] WsClient / WsServer 구현 + +#### 문제 + +Kotlin에 WebSocket transport가 없다. + +#### 해결 방법 + +WebSocket 클라이언트는 OkHttp(`okhttp3.WebSocket`), 서버는 `org.java-websocket:Java-WebSocket`(`WebSocketServer`)을 사용한다. Android 호환을 위해 서버 쪽 `java-websocket`은 JVM 테스트/서버 전용으로 명시한다. + +**파일:** +- `kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsClient.kt` +- `kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsServer.kt` + +**WsClient (OkHttp 기반):** + +```kotlin +class WsClient private constructor( + private val ws: okhttp3.WebSocket, + private val closeWs: () -> Unit, + intervalSec: Int, + waitSec: Int, + parserMap: ParserMap, +) : BaseClient(...) { + override val communicator = Communicator(this /* as Transport */, parserMap, scope) + + // Transport 구현 + fun writePacket(base: PacketBase) { + val bytes = base.toByteArray() + ws.send(ByteString.of(*bytes)) // OkHttp binary frame + } + + // OkHttp WebSocketListener (onMessage에서 onReceivedData 호출) +} +``` + +OkHttp `WebSocket.send(ByteString)`은 thread-safe하므로 별도 writeMutex 불필요. + +**WsServer (Java-WebSocket 기반):** + +```kotlin +class WsServer(host: String, port: Int) : org.java_websocket.server.WebSocketServer(...) { + val clients = CopyOnWriteArrayList() + var onClientConnected: (WsClient) -> Unit = {} + + override fun onOpen(conn: WebSocket, handshake: ClientHandshake) { ... } + override fun onMessage(conn: WebSocket, bytes: ByteBuffer) { ... } + override fun onClose(...) { ... } + override fun onError(...) { ... } +} +``` + +`java-websocket`의 `onMessage(conn, ByteBuffer)`로 binary frame을 수신하여 `PacketBase.parseFrom(bytes)` 후 해당 WsClient의 `communicator.onReceivedData(...)` 호출. + +#### 수정 파일 및 체크리스트 + +- [ ] `WsClient.kt` 생성 + - [ ] OkHttpClient + `Request.Builder().url(ws://...)` + `newWebSocket(request, listener)` + - [ ] `WebSocketListener.onMessage(ws, bytes: ByteString)`: PacketBase.parseFrom → onReceivedData → sendHeartBeat + - [ ] `WebSocketListener.onFailure(ws, t, response)`: onDisconnected + - [ ] `Transport.writePacket`: `ws.send(ByteString.of(*bytes))` + - [ ] `Transport.close`: `ws.close(1000, null)` + OkHttpClient.dispatcher.executorService.shutdown() + - [ ] HeartBeat 리스너 등록, WriteErrorHandler 등록 + - [ ] `DialWs(host, port, path, intervalSec, waitSec, parserMap)` 팩토리 + - [ ] `DialWss(host, port, path, sslContext, intervalSec, waitSec, parserMap)` 팩토리 +- [ ] `WsServer.kt` 생성 + - [ ] `WebSocketServer` 상속, binary 프레임 처리 + - [ ] WsClient 인스턴스 생성 및 clients 리스트 관리 + - [ ] `start()` / `stop()` / `broadcast(m)` + - [ ] 클라이언트 disconnect 시 clients 리스트에서 제거 + +#### 테스트 작성 + +**파일:** `kotlin/src/test/kotlin/com/tokilabs/toki_socket/WsTest.kt` + +| 테스트명 | 검증 목표 | +|---------|---------| +| `testWsSendReceive` | client.send → server addListener 수신 | +| `testWsRequestResponse` | sendRequestTyped → index*2, "echo: msg" | +| `testWsBroadcast` | server.broadcast → 연결된 클라이언트 수신 | +| `testWsServerStopDisconnectsClients` | server.stop() → client disconnect 콜백 | + +#### 중간 검증 + +```bash +cd kotlin +./gradlew test --tests "*.WsTest" +# 예상: 4개 테스트 PASS +``` + +--- + +### [API-7] Heartbeat 통합 테스트 + +#### 문제 + +heartbeat 타이머가 비활성 구간 후 올바르게 발송되고, 응답 없을 때 disconnect를 트리거하는지 검증되지 않았다. + +#### 해결 방법 + +Go의 `heartbeat_test.go` 패턴을 Kotlin coroutine 테스트로 이식한다. `runTest` + `TestCoroutineScheduler`로 실제 시간 대기 없이 가상 시간 진행. + +**파일:** `kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTest.kt` + +| 테스트명 | 검증 목표 | +|---------|---------| +| `testHeartbeatSentAfterInactivity` | intervalSec 경과 → HeartBeat 패킷 전송 확인 | +| `testHeartbeatDisconnectOnNoResponse` | waitSec 경과 → onDisconnected 호출 확인 | +| `testHeartbeatResetOnReceive` | 메시지 수신 → heartbeat 타이머 리셋 확인 | + +#### 수정 파일 및 체크리스트 + +- [ ] `HeartbeatTest.kt` 생성 (위 3개 테스트) + +#### 테스트 작성 + +본 항목 자체가 테스트 작성이다. + +#### 중간 검증 + +```bash +cd kotlin +./gradlew test --tests "*.HeartbeatTest" +# 예상: 3개 테스트 PASS +``` + +--- + +### [API-8] Go ↔ Kotlin 크로스테스트 + +#### 문제 + +다른 언어 구현체와의 wire format 호환성이 검증되지 않았다. + +#### 해결 방법 + +`skills/add-toki-socket-crosstest-language/SKILL.md`의 runner 배치 규칙을 따른다. + +**추가 파일:** + +| 파일 | 역할 | +|------|------| +| `go/crosstest/go_kotlin.go` | Go 서버 오케스트레이터 (Kotlin subprocess 실행) | +| `kotlin/crosstest/go_kotlin_client/Main.kt` | Kotlin subprocess (Go 서버에 접속) | +| `kotlin/crosstest/kotlin_go.kt` | Kotlin 서버 오케스트레이터 (Go subprocess 실행) | +| `go/crosstest/kotlin_go_client/main.go` | Go subprocess (Kotlin 서버에 접속) | + +**포트 배정 (기존과 충돌 없음):** + +``` +Go server / Kotlin client TCP: 29290 +Go server / Kotlin client WS: 29292 +Kotlin server / Go client TCP: 29390 +Kotlin server / Go client WS: 29392 +``` + +기존 포트 (충돌 없음 확인): +- 29090 (Dart server / Go client TCP) +- 29092 (Dart server / Go client WS) +- 29190 (Go server / Dart client TCP) +- 29192 (Go server / Dart client WS) + +**시나리오 (TCP + WebSocket 각각):** + +| 시나리오 | 내용 | +|---------|------| +| 1 | 클라이언트 fire-and-forget `TestData(index=101, message="fire from kotlin client")` → 서버 검증 | +| 2 | 서버 push `TestData(index=200, message="push from go server")` → 클라이언트 검증 | +| 3 | 클라이언트 `sendRequest` → 응답 `index=req.index*2`, `message="echo: req.message"` | +| 4 | 동시 5개 `sendRequest` → responseNonce 라우팅 검증 | + +send-push/requests 페이즈 분리 패턴은 Go ↔ Dart 크로스테스트와 동일하게 적용한다. + +**`kotlin/crosstest/go_kotlin_client/Main.kt` 실행 방법:** + +```bash +cd kotlin +./gradlew run --args="--mode=tcp --port=29290 --phase=send-push" +``` + +`build.gradle.kts`에 `application { mainClass.set("com.tokilabs.toki_socket.crosstest.MainKt") }` 추가 필요. + +**`go/crosstest/go_kotlin.go` 실행 방법:** + +```bash +cd go +go run ./crosstest/go_kotlin.go +``` + +#### 수정 파일 및 체크리스트 + +- [ ] `go/crosstest/go_kotlin.go` 생성 + - [ ] TCP send-push (port 29290), TCP requests, WS send-push (port 29292), WS requests + - [ ] Kotlin subprocess 실행: `./gradlew run --args="..."` (kotlin dir 기준) + - [ ] `validateResultLines` 활용 (go_dart.go와 동일 헬퍼) +- [ ] `kotlin/crosstest/go_kotlin_client/Main.kt` 생성 + - [ ] `INFO typeName kotlin=TestData` 출력 + - [ ] `--mode`, `--port`, `--phase` 인수 파싱 + - [ ] send-push: TestData 전송 (시나리오 1) + 서버 push 수신 (시나리오 2) + - [ ] requests: 단일 request (시나리오 3) + 5개 concurrent request (시나리오 4) + - [ ] `PASS scenario=N detail=...` / `FAIL scenario=N error=...` 출력 +- [ ] `kotlin/crosstest/kotlin_go.kt` 생성 (Kotlin 서버 오케스트레이터) + - [ ] TCP 서버 (29390) + WS 서버 (29392) 시작 + - [ ] Go subprocess 실행: `go run ./crosstest/kotlin_go_client` + - [ ] PASS/FAIL 파싱 및 검증 +- [ ] `go/crosstest/kotlin_go_client/main.go` 생성 + - [ ] Kotlin 서버에 접속하는 Go 클라이언트 (dart_go_client/main.go와 동일 구조) + - [ ] `INFO typeName go=TestData` 출력 + - [ ] 동일 4개 시나리오 + +#### 테스트 작성 + +본 항목 자체가 크로스테스트이다. + +#### 중간 검증 + +```bash +# Go 서버 ↔ Kotlin 클라이언트 +cd go && go run ./crosstest/go_kotlin.go +# 예상: PASS all go-server/kotlin-client crosstests passed + +# Kotlin 서버 ↔ Go 클라이언트 +cd kotlin && ./gradlew run -PmainClass=com.tokilabs.toki_socket.crosstest.KotlinGoKt +# 또는 +cd kotlin && kotlinc -script crosstest/kotlin_go.kts +# 예상: PASS all kotlin-server/go-client crosstests passed +``` + +--- + +## 수정 파일 요약 + +| 파일 | 항목 | +|------|------| +| `kotlin/settings.gradle.kts` | API-1 | +| `kotlin/build.gradle.kts` | API-1 | +| `kotlin/src/main/proto/message_common.proto` | API-1 | +| `kotlin/src/main/kotlin/.../Communicator.kt` | API-2 | +| `kotlin/src/test/kotlin/.../CommunicatorTest.kt` | API-2 | +| `kotlin/src/main/kotlin/.../HeartbeatTimer.kt` | API-3 | +| `kotlin/src/test/kotlin/.../HeartbeatTimerTest.kt` | API-3 | +| `kotlin/src/main/kotlin/.../BaseClient.kt` | API-4 | +| `kotlin/src/main/kotlin/.../TcpClient.kt` | API-5 | +| `kotlin/src/main/kotlin/.../TcpServer.kt` | API-5 | +| `kotlin/src/test/kotlin/.../TcpTest.kt` | API-5 | +| `kotlin/src/main/kotlin/.../WsClient.kt` | API-6 | +| `kotlin/src/main/kotlin/.../WsServer.kt` | API-6 | +| `kotlin/src/test/kotlin/.../WsTest.kt` | API-6 | +| `kotlin/src/test/kotlin/.../HeartbeatTest.kt` | API-7 | +| `go/crosstest/go_kotlin.go` | API-8 | +| `kotlin/crosstest/go_kotlin_client/Main.kt` | API-8 | +| `kotlin/crosstest/kotlin_go.kt` | API-8 | +| `go/crosstest/kotlin_go_client/main.go` | API-8 | +| `PROTOCOL.md` | Kotlin 행 status 업데이트 | +| `README.md` | Kotlin 행 status 업데이트 | + +--- + +## 최종 검증 + +```bash +# 1. Kotlin 단위 테스트 전체 +cd kotlin +./gradlew test +# 예상: 모든 테스트 PASS, BUILD SUCCESSFUL + +# 2. Go 단위 테스트 (크로스테스트 헬퍼 추가로 인한 회귀 없음) +cd go +go test ./... +# 예상: ok toki-labs.com/toki_socket/go [no test files changed] + +# 3. Kotlin linter +cd kotlin +./gradlew ktlintCheck # ktlint plugin 추가 시 +# 예상: BUILD SUCCESSFUL + +# 4. Go 서버 ↔ Kotlin 클라이언트 크로스테스트 +cd go +go run ./crosstest/go_kotlin.go +# 예상: PASS all go-server/kotlin-client crosstests passed + +# 5. Kotlin 서버 ↔ Go 클라이언트 크로스테스트 +cd kotlin +./gradlew run -PmainClass=com.tokilabs.toki_socket.crosstest.KotlinGoKt +# 예상: PASS all kotlin-server/go-client crosstests passed + +# 6. proto 동기화 검증 +tools/check_proto_sync.sh +# 예상: no diff +``` diff --git a/tasks/kotlin_impl/plan_1.log b/tasks/kotlin_impl/plan_1.log new file mode 100644 index 0000000..356ba7d --- /dev/null +++ b/tasks/kotlin_impl/plan_1.log @@ -0,0 +1,396 @@ + + +# Kotlin 구현 후속 수정 — WARN 항목 해소 + +## 이 파일을 읽는 구현 에이전트에게 + +각 항목의 체크리스트를 완료 처리하고, 중간 검증 명령을 실제로 실행한 뒤 출력을 `CODE_REVIEW.md`의 `검증 결과` 섹션에 붙여 넣으세요. +계획과 다르게 구현한 부분이 있으면 `계획 대비 변경 사항`에 이유와 함께 기록하세요. + +--- + +## 배경 + +`plan_0.log` 의 WARN 판정에서 발견된 4개 Suggested 항목을 수정한다. Required 항목은 없었고, Nit 1개(`Communicator.parse()` visibility)는 아래 포함한다. JVM 환경 부재로 인한 Gradle 미검증 상태는 이 항목들과 무관하므로 별도 다루지 않는다. + +--- + +## 의존 관계 및 구현 순서 + +REVIEW_API-1 → REVIEW_API-2 (sourceset 변경이 선행돼야 kotlin 빌드 구조에서 REVIEW_API-3, 4 검증 가능) → REVIEW_API-3, REVIEW_API-4 (병렬 가능) + +--- + +### [REVIEW_API-1] `WsServer.stop()` — `InterruptedException` 미처리 수정 + +#### 문제 + +`kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsServer.kt:41` + +`super.stop(1000)` 은 `WebSocketServer.stop(int)` 로, Java 검사 예외 `InterruptedException` 을 선언한다. `runBlocking` 내에서 호출될 때 이 예외가 전파되면 현재 coroutine이 취소 신호로 처리하여 `stop()` 이후 정리 코드가 실행되지 않을 수 있다. + +#### 해결 방법 + +`super.stop(1000)` 을 `runCatching` 으로 감싼다. + +**Before (`WsServer.kt:36-41`):** +```kotlin +override fun stop() { + if (!startedFlag.compareAndSet(true, false)) return + val snapshot = clients.toList() + clients.clear() + snapshot.forEach { it.close() } + super.stop(1000) +} +``` + +**After:** +```kotlin +override fun stop() { + if (!startedFlag.compareAndSet(true, false)) return + val snapshot = clients.toList() + clients.clear() + snapshot.forEach { it.close() } + runCatching { super.stop(1000) } +} +``` + +#### 수정 파일 및 체크리스트 + +- [x] `kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsServer.kt` — `super.stop(1000)` → `runCatching { super.stop(1000) }` + +#### 테스트 작성 + +SKIP — 기존 `WsTest.testWsServerStopDisconnectsClients` 가 stop 경로를 커버한다. `InterruptedException` 을 강제로 발생시키는 단위 테스트는 `java-websocket` 내부 구현 의존성이 높아 추가하지 않는다. + +#### 중간 검증 + +```bash +cd kotlin +./gradlew compileKotlin +# 예상: BUILD SUCCESSFUL +``` + +--- + +### [REVIEW_API-2] `crosstest/` 를 main sourceset에서 분리 + +#### 문제 + +`kotlin/build.gradle.kts:14` + +```kotlin +named("main") { + kotlin.srcDir("crosstest") +} +``` + +`kotlin_go.kt`, `go_kotlin_client/Main.kt` 가 라이브러리 main sourceset에 포함되어 프로덕션 JAR에 crosstest 코드가 실린다. + +#### 해결 방법 + +`build.gradle.kts` 에서 `crosstest` 를 sourceSets.main 에서 제거하고, 대신 별도 `crosstest` sourceset 을 정의한다. `application` 플러그인의 `mainClass` 는 기본값을 크로스테스트 runner로 유지하되 `crosstest` sourceset을 통해 공급한다. + +**Before (`build.gradle.kts:10-17`):** +```kotlin +kotlin { + jvmToolchain(17) + sourceSets { + named("main") { + kotlin.srcDir("crosstest") + } + } +} +``` + +**After:** +```kotlin +kotlin { + jvmToolchain(17) +} + +sourceSets { + create("crosstest") { + kotlin.srcDir("crosstest") + compileClasspath += sourceSets["main"].output + configurations["runtimeClasspath"] + runtimeClasspath += output + compileClasspath + } +} +``` + +`application` 블록의 `mainClass` 는 `crosstest` sourceset의 classpath 에서 로드되도록 `run` task를 재구성한다. + +**Before (`build.gradle.kts:42-47`):** +```kotlin +application { + mainClass.set( + (findProperty("mainClass") as String?) + ?: "com.tokilabs.toki_socket.crosstest.MainKt", + ) +} +``` + +**After:** +```kotlin +val crosstestSourceSet = sourceSets["crosstest"] + +tasks.named("run") { + classpath = crosstestSourceSet.runtimeClasspath + mainClass.set( + (findProperty("mainClass") as String?) + ?: "com.tokilabs.toki_socket.crosstest.MainKt", + ) +} +``` + +#### 수정 파일 및 체크리스트 + +- [x] `kotlin/build.gradle.kts` — `sourceSets.main.kotlin.srcDir("crosstest")` 제거 +- [x] `kotlin/build.gradle.kts` — `crosstest` sourceset 정의 추가 +- [x] `kotlin/build.gradle.kts` — `tasks.named("run")` 블록에서 classpath를 `crosstestSourceSet.runtimeClasspath` 로 지정 +- [x] `kotlin/build.gradle.kts` — `kotlin { sourceSets { named("main") { ... } } }` 블록 제거 또는 빈 상태로 정리 + +#### 테스트 작성 + +SKIP — 빌드 구조 변경이므로 컴파일 성공으로 충분하다. + +#### 중간 검증 + +```bash +cd kotlin +./gradlew compileKotlin compileCrosstestKotlin +# 예상: BUILD SUCCESSFUL +# crosstest 클래스가 main JAR에 포함되지 않는지 확인: +./gradlew jar +jar tf build/libs/toki-socket-kotlin-0.1.0.jar | grep crosstest +# 예상: 출력 없음 (crosstest 클래스 미포함) +``` + +--- + +### [REVIEW_API-3] `kotlin_go.kt` listener 내 `runBlocking` 제거 + +#### 문제 + +`kotlin/crosstest/kotlin_go.kt:56-63`, `104-111` + +`addListenerTyped` 의 callback은 `TcpClient.readLoop` 의 `Dispatchers.IO` coroutine에서 동기 호출된다. 여기서 `runBlocking { client.send(...) }` 를 호출하면 IO 스레드를 블로킹한다. + +#### 해결 방법 + +`runBlocking { ... }` 을 `scope.launch { ... }` 로 교체한다. `kotlin_go.kt` 파일은 `runBlocking` scope 안에서 실행되므로, 서버 scope를 직접 사용하거나 `GlobalScope` 대신 로컬 `CoroutineScope` 를 생성한다. 가장 단순한 방법은 listener에서 별도 coroutine을 시작한다. + +**Before (`kotlin_go.kt:51-63`):** +```kotlin +server.onClientConnected = { client -> + addListenerTyped(client.communicator) { data -> + println("SERVER_RECEIVED index=${data.index} message=${data.message}") + val valid = data.index == 101 && data.message == "fire from go client" + received.complete(valid) + if (valid) { + runBlocking { + client.send( + TestData.newBuilder() + .setIndex(200) + .setMessage("push from kotlin server") + .build(), + ) + } + } + } +} +``` + +**After:** +```kotlin +server.onClientConnected = { client -> + addListenerTyped(client.communicator) { data -> + println("SERVER_RECEIVED index=${data.index} message=${data.message}") + val valid = data.index == 101 && data.message == "fire from go client" + received.complete(valid) + if (valid) { + client.communicator.scope.launch { + client.send( + TestData.newBuilder() + .setIndex(200) + .setMessage("push from kotlin server") + .build(), + ) + } + } + } +} +``` + +단, `Communicator.scope` 가 현재 `private` 이다. `BaseClient.scope` 가 `protected` 이므로, `kotlin_go.kt` 에서 접근 가능한 방법이 필요하다. `TcpClient` 가 `BaseClient` 를 상속하므로 `client.scope` (protected)를 `internal` 또는 직접 접근 가능한 방식으로 노출하거나, `client.communicator` 를 통해 접근하는 대신 `kotlinx.coroutines.GlobalScope` 를 임시 사용한다. + +가장 깔끔한 방법: `BaseClient` 의 `scope` 를 `internal` 로 노출하여 같은 모듈 내 crosstest 코드에서 사용 가능하게 한다. + +**`BaseClient.kt` 변경:** +```kotlin +// Before +protected val scope = CoroutineScope(SupervisorJob() + Dispatchers.IO) + +// After +internal val scope = CoroutineScope(SupervisorJob() + Dispatchers.IO) +``` + +`kotlin_go.kt` 의 `runBlocking { client.send(...) }` 두 곳 (TCP send-push:56-63, WS send-push:104-111) 을 `client.scope.launch { client.send(...) }` 로 교체한다. + +#### 수정 파일 및 체크리스트 + +- [x] `kotlin/src/main/kotlin/com/tokilabs/toki_socket/BaseClient.kt` — `scope` 를 `protected` → `internal` 로 변경 +- [x] `kotlin/crosstest/kotlin_go.kt:56-63` — `runBlocking { client.send(...) }` → `coroutineScope` 의 `launch { client.send(...) }` +- [x] `kotlin/crosstest/kotlin_go.kt:104-111` — 동일 변경 + +#### 테스트 작성 + +SKIP — crosstest 코드 수정이고 기능 변경이 없다. 크로스테스트 자체가 검증이다. + +#### 중간 검증 + +```bash +cd kotlin +./gradlew compileCrosstestKotlin +# 예상: BUILD SUCCESSFUL (runBlocking 제거로 인한 compile error 없음) +``` + +--- + +### [REVIEW_API-4] `HeartbeatTest.testHeartbeatResetOnReceive()` — 수신 경로 검증 추가 + +#### 문제 + +`kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTest.kt:69-81` + +현재 테스트는 `sendHeartBeat()` 를 두 번 직접 호출한다. 실제 경로인 `communicator.onReceivedData()` → `sendHeartBeat()` 를 거쳐 타이머가 리셋되는지 검증하지 않는다. + +#### 해결 방법 + +기존 테스트를 두 개로 분리한다: +1. `testHeartbeatTimerResetBySendHeartBeat` — 기존 로직 유지 (sendHeartBeat 재호출 → 타이머 리셋) +2. `testHeartbeatTimerResetOnReceivedData` — `communicator.onReceivedData()` 를 호출한 뒤 heartbeat가 지연되는지 확인 + +**새 테스트 (`testHeartbeatTimerResetOnReceivedData`):** + +```kotlin +@Test +fun testHeartbeatTimerResetOnReceivedData() = runBlocking { + val transport = HeartbeatTransport() + val client = HeartbeatClient(transport, 1, 1) + var received = false + client.communicator.addListener(typeNameOf(testData())) { + received = true + } + + client.sendHeartBeat() + delay(700) + // 데이터 수신 시뮬레이션 — readLoop가 onReceivedData 후 sendHeartBeat() 호출하는 경로 + client.communicator.onReceivedData(typeNameOf(testData()), testData().toByteArray(), incomingNonce = 1) + assertTrue(received) + client.sendHeartBeat() // readLoop가 호출하는 sendHeartBeat() + delay(500) + assertTrue(transport.packets.none { it.typeName == typeNameOf() }) + delay(700) + assertTrue(transport.packets.any { it.typeName == typeNameOf() }) + client.close() +} +``` + +`testData()` 헬퍼: `TestHelpers.kt` 에 `fun testData() = TestData.newBuilder().setIndex(1).setMessage("ping").build()` 추가. + +`HeartbeatClient` 의 `communicator` parserMap 에 `TestData` 가 포함되도록 `testParserMap()` 을 사용한다. 테스트에서는 `TestData` 리스너를 등록하고 `received` 를 확인해 `onReceivedData()` 가 listener dispatch까지 실제로 진행됐는지 검증한다. + +#### 수정 파일 및 체크리스트 + +- [x] `kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTest.kt` — `testHeartbeatResetOnReceive` 를 `testHeartbeatTimerResetBySendHeartBeat` 로 rename +- [x] `kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTest.kt` — `testHeartbeatTimerResetOnReceivedData` 테스트 추가 +- [x] `kotlin/src/test/kotlin/com/tokilabs/toki_socket/TestHelpers.kt` — `fun testData()` 헬퍼 추가 + +#### 테스트 작성 + +본 항목이 테스트 추가이다. + +#### 중간 검증 + +```bash +cd kotlin +./gradlew test --tests "*.HeartbeatTest" +# 예상: 4개 테스트 PASS (기존 3개 + 신규 1개) +``` + +--- + +### [REVIEW_API-5] `Communicator.parse()` visibility — `internal` 로 제한 (Nit) + +#### 문제 + +`kotlin/src/main/kotlin/com/tokilabs/toki_socket/Communicator.kt:266` + +`parse()` 가 `public` 으로 노출되어 있다. Go 의 `parse` 는 unexported이다. inline helper들은 `parse()` 를 직접 호출하지 않는다. + +#### 해결 방법 + +```kotlin +// Before +fun parse(typeName: String, data: ByteArray): MessageLite { + +// After +internal fun parse(typeName: String, data: ByteArray): MessageLite { +``` + +#### 수정 파일 및 체크리스트 + +- [x] `kotlin/src/main/kotlin/com/tokilabs/toki_socket/Communicator.kt:266` — `fun parse` → `internal fun parse` + +#### 테스트 작성 + +SKIP — `CommunicatorTest` 는 `parse()` 를 직접 호출하지 않는다. 같은 모듈이므로 `internal` 후에도 테스트 접근 가능하다. + +#### 중간 검증 + +```bash +cd kotlin +./gradlew compileKotlin compileTestKotlin +# 예상: BUILD SUCCESSFUL +``` + +--- + +## 수정 파일 요약 + +| 파일 | 항목 | +|------|------| +| `kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsServer.kt` | REVIEW_API-1 | +| `kotlin/build.gradle.kts` | REVIEW_API-2 | +| `kotlin/src/main/kotlin/com/tokilabs/toki_socket/BaseClient.kt` | REVIEW_API-3 | +| `kotlin/crosstest/kotlin_go.kt` | REVIEW_API-3 | +| `kotlin/src/main/kotlin/com/tokilabs/toki_socket/WsClient.kt` | REVIEW_API-3 follow-up: crosstest sourceset에서 server factory 접근 가능하도록 공개 | +| `kotlin/src/test/kotlin/com/tokilabs/toki_socket/HeartbeatTest.kt` | REVIEW_API-4 | +| `kotlin/src/test/kotlin/com/tokilabs/toki_socket/TestHelpers.kt` | REVIEW_API-4 | +| `kotlin/src/test/kotlin/com/tokilabs/toki_socket/TcpTest.kt` | 최종 검증 follow-up: blocking accept를 `Dispatchers.IO`로 이동 | +| `kotlin/src/main/kotlin/com/tokilabs/toki_socket/Communicator.kt` | REVIEW_API-5 | + +--- + +## 최종 검증 + +```bash +# 1. Kotlin 전체 단위 테스트 +cd kotlin +./gradlew test +# 예상: 모든 테스트 PASS (HeartbeatTest 4개 포함) + +# 2. JAR에 crosstest 미포함 확인 +./gradlew jar +jar tf build/libs/toki-socket-kotlin-0.1.0.jar | grep crosstest +# 예상: 출력 없음 + +# 3. Go 단위 테스트 회귀 없음 +cd ../go && go test ./... +# 예상: ok toki-labs.com/toki_socket/go/test + +# 4. proto 동기화 +cd .. && bash tools/check_proto_sync.sh +# 예상: Proto schemas are in sync. +``` diff --git a/tools/check_proto_sync.sh b/tools/check_proto_sync.sh index b9ce646..f3707d7 100755 --- a/tools/check_proto_sync.sh +++ b/tools/check_proto_sync.sh @@ -4,6 +4,7 @@ set -euo pipefail repo_root="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)" dart_proto="$repo_root/dart/lib/src/packets/message_common.proto" go_proto="$repo_root/go/packets/message_common.proto" +kotlin_proto="$repo_root/kotlin/src/main/proto/message_common.proto" if [[ ! -f "$dart_proto" ]]; then echo "Missing canonical Dart proto: $dart_proto" >&2 @@ -15,12 +16,20 @@ if [[ ! -f "$go_proto" ]]; then exit 1 fi +if [[ -d "$repo_root/kotlin" && ! -f "$kotlin_proto" ]]; then + echo "Missing Kotlin proto copy: $kotlin_proto" >&2 + exit 1 +fi + tmp_dir="$(mktemp -d)" trap 'rm -rf "$tmp_dir"' EXIT normalize_proto() { awk ' /^[[:space:]]*option go_package[[:space:]]*=.*;[[:space:]]*$/ { next } + /^[[:space:]]*option java_package[[:space:]]*=.*;[[:space:]]*$/ { next } + /^[[:space:]]*option java_outer_classname[[:space:]]*=.*;[[:space:]]*$/ { next } + /^[[:space:]]*option java_multiple_files[[:space:]]*=.*;[[:space:]]*$/ { next } /^[[:space:]]*$/ { next } { print } ' "$1" @@ -28,6 +37,9 @@ normalize_proto() { normalize_proto "$dart_proto" >"$tmp_dir/dart.proto" normalize_proto "$go_proto" >"$tmp_dir/go.proto" +if [[ -f "$kotlin_proto" ]]; then + normalize_proto "$kotlin_proto" >"$tmp_dir/kotlin.proto" +fi if ! cmp -s "$tmp_dir/dart.proto" "$tmp_dir/go.proto"; then echo "Proto schema mismatch: go/packets/message_common.proto must match the Dart canonical proto except option go_package." >&2 @@ -35,4 +47,10 @@ if ! cmp -s "$tmp_dir/dart.proto" "$tmp_dir/go.proto"; then exit 1 fi +if [[ -f "$tmp_dir/kotlin.proto" ]] && ! cmp -s "$tmp_dir/dart.proto" "$tmp_dir/kotlin.proto"; then + echo "Proto schema mismatch: kotlin/src/main/proto/message_common.proto must match the Dart canonical proto except Java options." >&2 + diff -u "$tmp_dir/dart.proto" "$tmp_dir/kotlin.proto" >&2 + exit 1 +fi + echo "Proto schemas are in sync."