Adding Chat demo (Not stream) (#11)

* Chat demo

* Typo

* Comments

* Adding openAi dependency

* Getting ready for openai changes

* OpenAI tools

* Adding final tools conversion for openai

* Adding openai demo

* Adding hint code

* lint, mostly

* fix json data being spread over several chunks received from stdio

* update readme for sandbox

* updating branch with stream changes

* Solving problem for env

* Update MCPClient/Sources/stdioTransport/DataChannel+StdioProcess.swift

Co-authored-by: Guillaume Sabran <[email protected]>

* Update MCPClient/Sources/stdioTransport/DataChannel+StdioProcess.swift

Co-authored-by: Guillaume Sabran <[email protected]>

* pr feedback

* Update MCPInterface/Sources/helpers/JSON+Streaming.swift

Co-authored-by: Guillaume Sabran <[email protected]>

* lint

* Avoding print rules

* lint

* More linting

---------

Co-authored-by: Gui Sabran <gsabran@www.com>
Co-authored-by: Guillaume Sabran <[email protected]>

Author: 3 people

MCPClient/Sources/DataChannel+StdioProcess.swift renamed to MCPClient/Sources/stdioTransport/DataChannel+StdioProcess.swift

@@ -1,6 +1,7 @@
 
 import Foundation
 import JSONRPC
+import MCPInterface
 import OSLog
 
 private let logger = Logger(
@@ -42,17 +43,18 @@ extension JSONRPCSetupError: LocalizedError {
   }
 }
 
-extension DataChannel {
+extension Transport {
 
   // MARK: Public
 
+  /// Creates a new `Transport` by launching the given executable with the specified arguments and attaching to its standard IO.
   public static func stdioProcess(
     _ executable: String,
     args: [String] = [],
     cwd: String? = nil,
     env: [String: String]? = nil,
     verbose: Bool = false)
-    throws -> DataChannel
+    throws -> Transport
   {
     if verbose {
       let command = "\(executable) \(args.joined(separator: " "))"
@@ -103,10 +105,11 @@ extension DataChannel {
     return try stdioProcess(unlaunchedProcess: process, verbose: verbose)
   }
 
+  /// Creates a new `Transport` by launching the given process and attaching to its standard IO.
   public static func stdioProcess(
     unlaunchedProcess process: Process,
     verbose: Bool = false)
-    throws -> DataChannel
+    throws -> Transport
   {
     guard
       let stdin = process.standardInput as? Pipe,
@@ -119,7 +122,6 @@ extension DataChannel {
     // Run the process
     var stdoutData = Data()
     var stderrData = Data()
-
     let outStream: AsyncStream<Data>
     if verbose {
       // As we are both reading stdout here in this function, and want to make the stream readable to the caller,
@@ -131,7 +133,7 @@ extension DataChannel {
       }
 
       Task {
-        for await data in stdout.fileHandleForReading.dataStream {
+        for await data in stdout.fileHandleForReading.dataStream.jsonStream {
           stdoutData.append(data)
           outContinuation?.yield(data)
 
@@ -150,10 +152,10 @@ extension DataChannel {
       }
     } else {
       // If we are not in verbose mode, we are not reading from stdout internally, so we can just return the stream directly.
-      outStream = stdout.fileHandleForReading.dataStream
+      outStream = stdout.fileHandleForReading.dataStream.jsonStream
     }
 
-    // Ensures that the process is terminated when the DataChannel is de-referenced.
+    // Ensures that the process is terminated when the Transport is de-referenced.
     let lifetime = Lifetime {
       if process.isRunning {
         process.terminate()
@@ -177,7 +179,7 @@ extension DataChannel {
       throw error
     }
 
-    let writeHandler: DataChannel.WriteHandler = { [lifetime] data in
+    let writeHandler: Transport.WriteHandler = { [lifetime] data in
       _ = lifetime
       if verbose {
         logger.log("Sending data:\n\(String(data: data, encoding: .utf8) ?? "nil")")
@@ -188,7 +190,7 @@ extension DataChannel {
       stdin.fileHandleForWriting.write(Data("\n".utf8))
     }
 
-    return DataChannel(writeHandler: writeHandler, dataSequence: outStream)
+    return Transport(writeHandler: writeHandler, dataSequence: outStream)
   }
 
   // MARK: Private
@@ -213,10 +215,12 @@ extension DataChannel {
   private static func loadZshEnvironment() throws -> [String: String] {
     let process = Process()
     process.launchPath = "/bin/zsh"
-    process.arguments = ["-c", "source ~/.zshrc && printenv"]
+    // Those are loaded for interactive login shell by zsh:
+    // https://www.freecodecamp.org/news/how-do-zsh-configuration-files-work/
+    process.arguments = ["-c", "source ~/.zshenv; source ~/.zprofile; source ~/.zshrc; source ~/.zshrc; printenv"]
     let env = try getProcessStdout(process: process)
 
-    if let path = env?.split(separator: "\n").filter({ $0.starts(with: "PATH=") }).first {
+    if let path = env?.split(separator: "\n").filter({ $0.starts(with: "PATH=") }).last {
       return ["PATH": String(path.dropFirst("PATH=".count))]
     } else {
       return ProcessInfo.processInfo.environment