Add comments, Remove unnecessary code

Author: tharun571

.github/workflows/main.yml

@@ -74,12 +74,6 @@ jobs:
         run: |
           micromamba shell hook -s cmd.exe -p C:\Users\runneradmin\micromamba-root
 
-      - name: activate environment
-        if: ${{ runner.os != 'windows' }}
-        shell: bash -l {0}
-        run: |
-          micromamba activate xeus-cpp
-
       - name: install cxx compiler
         if: ${{ runner.os != 'windows' }}
         shell: bash -l {0}
@@ -124,17 +118,13 @@ jobs:
           cd build
           make install -j ${{ env.ncpus }}
 
-      # - name: Setup tmate session
-      #   if: ${{ runner.os != 'windows' }}
-      #   uses: mxschmitt/action-tmate@v3
-
       - name: Test xeus-cpp C++ Unix Systems
         if: ${{ runner.os != 'windows' }}
         shell: bash -l {0}
         run: |
           cd build/test
           ./test_xeus_cpp
-        timeout-minutes: 30
+        timeout-minutes: 4
 
       - name: Test xeus-cpp C++ Windows Systems
         if: ${{ runner.os == 'windows' }}

CMakeLists.txt

@@ -312,6 +312,7 @@ macro(xeus_cpp_create_target target_name linkage output_name)
         set(XEUS_CPP_XEUS_TARGET xeus-static)
     endif ()
 
+    #This is a workaround for the issue with the libcurl target on Windows specifically for xassist
     if(WIN32)
         # Set the MSVC runtime library
         set(CMAKE_MSVC_RUNTIME_LIBRARY "MultiThreaded$<$<CONFIG:Debug>:Debug>DLL")
@@ -338,6 +339,7 @@ macro(xeus_cpp_create_target target_name linkage output_name)
             target_compile_options(${target_name} PRIVATE "/MD$<$<CONFIG:Debug>:d>")
         endif()
     else()
+        # Curl initialised specifically for xassist
         target_link_libraries(${target_name} PUBLIC ${XEUS_CPP_XEUS_TARGET} clangCppInterOp pugixml argparse::argparse curl)
     endif()
 

environment-wasm-build.yml

@@ -4,5 +4,4 @@ channels:
 dependencies:
   - cmake
   - emsdk >=3.1.11
-  - empack >=2.0.1
-  - curl
+  - empack >=2.0.1

src/xmagics/xassist.cpp

@@ -19,6 +19,8 @@
 
 using json = nlohmann::json;
 
+// TODO: Implement xplugin to seperate the magics from the main code.
+// TODO: Add support for open-source models.
 namespace xcpp
 {
     class APIKeyManager