feature add sentry for catch crash on win (#50)

* feature add sentry for soft catch dmp

* feature add cli upload pdb to sentry server

* feature add flag for control sentry to use

* fix  flag not effect on src/makefile

* feature unzip for dmp file to upload sentry server

* feature remove test code

* feature add api for function report log to server

* feature update the soft version

.github/workflows/build_all.yml

@@ -151,7 +151,7 @@ jobs:
         if-no-files-found: error
 
     - name: Deploy Flatpak to nightly release
-      if: ${{github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.0'}}
+      if: ${{github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.1'}}
       uses: WebFreak001/deploy-nightly@v3.2.0
       with:
         upload_url: https://uploads.github.com/repos/Snapmaker/OrcaSlicer/releases/169912305/assets{?name,label}

.github/workflows/build_deps.yml

@@ -150,3 +150,14 @@ jobs:
       os: ${{ inputs.os }}
       arch: ${{ inputs.arch }}
     secrets: inherit
+
+  upload_symbols:
+    name: Upload Debug Symbols to Sentry
+    needs: [build_orca]
+    if: ${{ !cancelled() && needs.build_orca.result == 'success' }}
+    uses: ./.github/workflows/sentry_cli.yml
+    with:
+      os: ${{ inputs.os }}
+      pdb-artifact-name: PDB
+      release: ${{ github.sha }}
+    secrets: inherit

.github/workflows/build_orca.yml

@@ -18,6 +18,8 @@ jobs:
   build_orca:
     name: Build Snapmaker_Orca
     runs-on: ${{ inputs.os }}
+    outputs:
+      release: ${{ steps.set_release.outputs.release || steps.set_release_win.outputs.release }}
     env:
       date:
       ver:
@@ -77,6 +79,21 @@ jobs:
           echo "date: ${{ env.date }} version: ${{ env.ver }}"
         shell: pwsh
 
+      - name: Set release output (non-Windows)
+        id: set_release
+        if: inputs.os != 'windows-latest'
+        run: |
+          echo "release=$ver" >> $GITHUB_OUTPUT
+        shell: bash
+
+      - name: Set release output (Windows)
+        id: set_release_win
+        if: inputs.os == 'windows-latest'
+        run: |
+          $release = $env:ver
+          Write-Output ("release=$release") | Out-File -Append -FilePath $env:GITHUB_OUTPUT -Encoding utf8
+        shell: pwsh
+
 #   Mac
       - name: Install tools mac
         if: inputs.os == 'macos-14'
@@ -289,7 +306,7 @@ jobs:
           path: ${{ github.workspace }}/build/src/Release/Snapmaker_Orca_profile_validator.exe
 
       - name: Deploy Windows release portable
-        if: (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.0') && inputs.os == 'windows-latest'
+        if: (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.1') && inputs.os == 'windows-latest'
         uses: WebFreak001/deploy-nightly@v3.2.0
         with:
           upload_url: https://uploads.github.com/repos/Snapmaker/OrcaSlicer/releases/169912305/assets{?name,label}
@@ -300,7 +317,7 @@ jobs:
           max_releases: 1
 
       - name: Deploy Windows release installer
-        if: (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.0') && inputs.os == 'windows-latest'
+        if: (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.1') && inputs.os == 'windows-latest'
         uses: WebFreak001/deploy-nightly@v3.2.0
         with:
           upload_url: https://uploads.github.com/repos/Snapmaker/OrcaSlicer/releases/169912305/assets{?name,label}
@@ -311,7 +328,7 @@ jobs:
           max_releases: 1
 
       - name: Deploy Windows Snapmaker_Orca_profile_validator release
-        if: (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.0') && inputs.os == 'windows-latest'
+        if: (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.1') && inputs.os == 'windows-latest'
         uses: WebFreak001/deploy-nightly@v3.2.0
         with:
           upload_url: https://uploads.github.com/repos/Snapmaker/OrcaSlicer/releases/169912305/assets{?name,label}
@@ -385,7 +402,7 @@ jobs:
           path: './build/src/Release/Snapmaker_Orca_profile_validator'
 
       - name: Deploy Ubuntu release
-        if: ${{ ! env.ACT && (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.0') && (inputs.os == 'ubuntu-20.04' || inputs.os == 'ubuntu-24.04') }}
+        if: ${{ ! env.ACT && (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.1') && (inputs.os == 'ubuntu-20.04' || inputs.os == 'ubuntu-24.04') }}
         env:
           ubuntu-ver-str: ${{ (inputs.os == 'ubuntu-24.04' && '_Ubuntu2404') || '' }}
         uses: WebFreak001/deploy-nightly@v3.2.0
@@ -406,7 +423,7 @@ jobs:
           message: "nightly-builds"
 
       - name: Deploy Ubuntu Snapmaker_Orca_profile_validator release
-        if: ${{ ! env.ACT && (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.0') && (inputs.os == 'ubuntu-20.04' || inputs.os == 'ubuntu-24.04') }}
+        if: ${{ ! env.ACT && (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.1') && (inputs.os == 'ubuntu-20.04' || inputs.os == 'ubuntu-24.04') }}
         env:
           ubuntu-ver-str: ${{ (inputs.os == 'ubuntu-24.04' && '_Ubuntu2404') || '' }}
         uses: WebFreak001/deploy-nightly@v3.2.0
@@ -419,7 +436,7 @@ jobs:
           max_releases: 1
 
       - name: Deploy orca_custom_preset_tests
-        if: ${{ ! env.ACT && (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.0') && inputs.os == 'ubuntu-24.04' }}
+        if: ${{ ! env.ACT && (github.ref == 'refs/heads/main' || github.ref == 'refs/heads/2.2.1') && inputs.os == 'ubuntu-24.04' }}
         uses: WebFreak001/deploy-nightly@v3.2.0
         with:
           upload_url: https://uploads.github.com/repos/Snapmaker/OrcaSlicer/releases/169912305/assets{?name,label}

.github/workflows/sentry_cli.yml

@@ -0,0 +1,92 @@
+name: Upload Debug Symbols to Sentry
+
+on:
+  workflow_call:
+    inputs:
+      os:
+        required: true
+        type: string
+        description: "Target OS: windows-latest, macos-14, ubuntu-20.04, ubuntu-24.04"
+      pdb-artifact-name:
+        required: false
+        type: string
+        description: "Artifact name for Windows PDB archive (e.g., 'PDB')"
+      release:
+        required: true
+        type: string
+        description: "Release version/tag"
+
+jobs:
+  upload_symbols:
+    name: Upload Debug Symbols to Sentry
+    runs-on: ${{ inputs.os }}
+    steps:
+      # ==================== Windows ====================
+      - name: "[Windows] Install sentry-cli via choco"
+        if: inputs.os == 'windows-latest'
+        shell: pwsh
+        run: |
+          choco install sentry-cli -y -y 2>&1 | Out-Null
+
+      - name: "[Windows] Download PDB artifact"
+        if: inputs.os == 'windows-latest'
+        uses: actions/download-artifact@v4
+        with:
+          name: ${{ inputs.pdb-artifact-name }}
+          path: ./symbols
+
+      - name: "[Windows] Extract PDB archive"
+        if: inputs.os == 'windows-latest'
+        shell: pwsh
+        run: |
+          $archive = Get-ChildItem -Path ./symbols -Filter "*.7z" -File | Select-Object -First 1
+          if ($archive) {
+            Write-Host "Found archive: $($archive.FullName)"
+            Write-Host "Extracting to ./symbols/extracted ..."
+            7z x "$($archive.FullName)" -o"./symbols/extracted" -y
+            if ($LASTEXITCODE -ne 0) {
+              Write-Host "::error::Failed to extract archive with exit code $LASTEXITCODE"
+              exit $LASTEXITCODE
+            }
+            Write-Host "Extraction complete. Contents:"
+            Get-ChildItem -Path ./symbols/extracted -Recurse | ForEach-Object { Write-Host "  $($_.FullName)" }
+          } else {
+            Write-Host "No .7z archive found, assuming PDB files are already extracted"
+            Get-ChildItem -Path ./symbols -Recurse | ForEach-Object { Write-Host "  Found: $($_.FullName)" }
+          }
+
+      - name: "[Windows] Upload PDB to Sentry (sentry-cli)"
+        if: inputs.os == 'windows-latest'
+        shell: pwsh
+        env:
+          SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
+          SENTRY_LOG_LEVEL: debug
+        run: |
+          # Determine upload path - prefer extracted folder if it exists
+          $uploadPath = "./symbols"
+          if (Test-Path "./symbols/extracted") {
+            $uploadPath = "./symbols/extracted"
+          }
+          
+          $pdbFiles = Get-ChildItem -Path $uploadPath -Filter "*.pdb" -File -Recurse
+          if ($pdbFiles.Count -gt 0) {
+            Write-Host "Found $($pdbFiles.Count) PDB file(s) to upload from $uploadPath :"
+            $pdbFiles | ForEach-Object { Write-Host "  - $($_.FullName)" }
+            Write-Host ""
+            Write-Host "Starting Sentry upload with debug logging..."
+            sentry-cli.exe --log-level=debug --auth-token $env:SENTRY_AUTH_TOKEN upload-dif --org "${{ secrets.SENTRY_ORG }}" --project "${{ secrets.SENTRY_PROJECT }}" $uploadPath 2>&1 | Out-Host
+            if ($LASTEXITCODE -ne 0) {
+              Write-Host "::error::Sentry upload failed with exit code $LASTEXITCODE"
+              exit $LASTEXITCODE
+            }
+          } else {
+            Write-Host "::error::No PDB files found in $uploadPath"
+            Get-ChildItem -Path ./symbols -Recurse | ForEach-Object { Write-Host "  Found: $($_.FullName)" }
+            exit 1
+          }
+
+      # ==================== macOS ====================
+
+
+      # ==================== Linux ====================
+

CMakeLists.txt

@@ -99,6 +99,14 @@ option(SLIC3R_MSVC_COMPILE_PARALLEL "Compile on Visual Studio in parallel" 1)
 option(SLIC3R_MSVC_PDB          "Generate PDB files on MSVC in Release mode" 1)
 option(SLIC3R_PERL_XS           "Compile XS Perl module and enable Perl unit and integration tests" 0)
 option(SLIC3R_ASAN              "Enable ASan on Clang and GCC" 0)
+
+# Sentry crash reporting - enabled only on Windows by default
+if (WIN32)
+    set(SLIC3R_SENTRY_DEFAULT ON)
+else()
+    set(SLIC3R_SENTRY_DEFAULT OFF)
+endif()
+option(SLIC3R_SENTRY            "Enable Sentry crash reporting SDK" ${SLIC3R_SENTRY_DEFAULT})
 # If SLIC3R_FHS is 1 -> SLIC3R_DESKTOP_INTEGRATION is always 0, othrewise variable.
 CMAKE_DEPENDENT_OPTION(SLIC3R_DESKTOP_INTEGRATION "Allow perfoming desktop integration during runtime" 1 "NOT SLIC3R_FHS" 0)
 
@@ -534,6 +542,9 @@ find_package(TBB REQUIRED)
 
 find_package(OpenSSL REQUIRED)
 find_package(CURL REQUIRED)
+if (SLIC3R_SENTRY)
+    find_package(Sentry REQUIRED)
+endif()
 find_package(Freetype REQUIRED)
 
 
@@ -543,6 +554,24 @@ target_link_libraries(libcurl INTERFACE CURL::libcurl)
 find_package(ZLIB REQUIRED)
 target_link_libraries(libcurl INTERFACE ZLIB::ZLIB)
 
+if (SLIC3R_SENTRY)
+    add_library(sentry INTERFACE)
+    target_link_libraries(sentry INTERFACE sentry::sentry)
+
+    # Add sentry library directory and link sentry.lib explicitly for Windows
+    if (WIN32 AND CMAKE_PREFIX_PATH)
+        # Try to find sentry.lib in the install prefix
+        file(GLOB _sentry_lib "${CMAKE_PREFIX_PATH}/lib/sentry.lib")
+        if (_sentry_lib)
+            target_link_directories(sentry INTERFACE "${CMAKE_PREFIX_PATH}/lib")
+            target_link_libraries(sentry INTERFACE "sentry.lib")
+            message(STATUS "Found sentry.lib at: ${CMAKE_PREFIX_PATH}/lib")
+        else()
+            message(WARNING "sentry.lib not found in ${CMAKE_PREFIX_PATH}/lib. Sentry SDK may not have been built yet.")
+        endif()
+    endif()
+endif()
+
 # Fixing curl's cmake config script bugs
 if (NOT WIN32)
     # Required by libcurl
@@ -727,6 +756,14 @@ function(Snapmaker_Orca_copy_dlls target config postfix output_dlls)
             ${TOP_LEVEL_PROJECT_DIR}/deps/WebView2/lib/win-${_arch}/WebView2Loader.dll
         DESTINATION ${_out_dir})
 
+    # Copy Sentry and Crashpad artifacts if they were built by deps
+    if (SLIC3R_SENTRY AND EXISTS "${CMAKE_PREFIX_PATH}/bin/sentry.dll")
+        file(COPY "${CMAKE_PREFIX_PATH}/bin/sentry.dll"
+                  "${CMAKE_PREFIX_PATH}/bin/crashpad_handler.exe"
+            DESTINATION ${_out_dir}
+        )
+    endif()
+
     file(COPY ${CMAKE_PREFIX_PATH}/bin/occt/TKBO.dll
             ${CMAKE_PREFIX_PATH}/bin/occt/TKBRep.dll
             ${CMAKE_PREFIX_PATH}/bin/occt/TKCAF.dll
@@ -793,6 +830,16 @@ function(Snapmaker_Orca_copy_dlls target config postfix output_dlls)
         PARENT_SCOPE
     )
 
+    # Add Sentry DLLs to the output list if Sentry is enabled
+    if (SLIC3R_SENTRY)
+        set(${output_dlls}
+            ${${output_dlls}}
+            ${_out_dir}/sentry.dll
+            ${_out_dir}/crashpad_handler.exe
+            PARENT_SCOPE
+        )
+    endif()
+
 endfunction()
 
 

deps/CMakeLists.txt

@@ -62,6 +62,14 @@ if(CMAKE_SYSTEM_NAME STREQUAL "Linux")
     option(DEP_WX_GTK3 "Build wxWidgets against GTK3" OFF)
 endif()
 
+# Sentry crash reporting - enabled only on Windows by default
+if (WIN32)
+    set(SLIC3R_SENTRY_DEFAULT ON)
+else()
+    set(SLIC3R_SENTRY_DEFAULT OFF)
+endif()
+option(SLIC3R_SENTRY "Enable Sentry crash reporting SDK" ${SLIC3R_SENTRY_DEFAULT})
+
 set(IS_CROSS_COMPILE FALSE)
 
 if (APPLE)
@@ -314,6 +322,12 @@ if (NOT OPENSSL_FOUND OR NOT CURL_FOUND)
     set(CURL_PKG dep_CURL)
 endif ()
 
+set(SENTRY_PKG "")
+if (SLIC3R_SENTRY AND NOT SENTRY_FOUND)
+    include(Sentry/Sentry.cmake)
+    set(SENTRY_PKG dep_Sentry)
+endif()
+
 set(JPEG_PKG "")
 if (NOT JPEG_FOUND)
     include(JPEG/JPEG.cmake)
@@ -355,6 +369,7 @@ set(_dep_list
     dep_TBB
     ${OPENSSL_PKG}
     ${CURL_PKG}
+    ${SENTRY_PKG}
     ${WXWIDGETS_PKG}
     dep_Cereal
     dep_NLopt

deps/Sentry/Sentry.cmake

@@ -0,0 +1,63 @@
+set(_sentry_platform_flags
+  -DCMAKE_POSITION_INDEPENDENT_CODE=ON
+  -DSENTRY_BUILD_TESTS=OFF
+  -DSENTRY_EXAMPLES=OFF
+  -DSENTRY_CRASHPAD_BACKEND=ON
+  -DSENTRY_ENABLE_INSTALL=ON
+)
+
+# Platform-specific CMake generator and flags
+set(_sentry_cmake_generator "")
+set(_sentry_build_config "Release")
+
+if (WIN32)
+  # Windows: build shared libs so we get sentry.dll
+  set(_sentry_platform_flags  ${_sentry_platform_flags}
+    -DSENTRY_TRANSPORT_WINHTTP=ON
+    -DSENTRY_BUILD_SHARED_LIBS=ON
+    -DCMAKE_BUILD_TYPE:STRING=RelWithDebInfo
+    -DCMAKE_C_FLAGS_RELWITHDEBINFO:STRING=/Zi /O2
+    -DCMAKE_CXX_FLAGS_RELWITHDEBINFO:STRING=/Zi /O2
+    -DCMAKE_EXE_LINKER_FLAGS:STRING=/DEBUG
+    -DCMAKE_SHARED_LINKER_FLAGS:STRING=/DEBUG
+  )
+  if (MSVC)
+    set(_sentry_cmake_generator -G "Visual Studio 17 2022")
+  endif()
+elseif (APPLE)
+  # macOS: Use Unix Makefiles (install will put libs in ${DESTDIR}/bin or lib)
+  set(_sentry_platform_flags 
+    ${_sentry_platform_flags}
+    -DSENTRY_TRANSPORT_CURL=ON
+    -DSENTRY_BUILD_SHARED_LIBS=OFF
+    -DCMAKE_BUILD_TYPE:STRING=RelWithDebInfo
+    -DCMAKE_C_FLAGS:STRING=-g -O2
+    -DCMAKE_CXX_FLAGS:STRING=-g -O2
+  )
+  set(_sentry_cmake_generator -G "Unix Makefiles")
+elseif(CMAKE_SYSTEM_NAME STREQUAL "Linux")
+  # Linux: Use Unix Makefiles
+  set(_sentry_platform_flags 
+    ${_sentry_platform_flags}
+    -DSENTRY_TRANSPORT_CURL=ON
+    -DSENTRY_BUILD_SHARED_LIBS=OFF
+    -DCMAKE_BUILD_TYPE:STRING=RelWithDebInfo
+    -DCMAKE_C_FLAGS:STRING=-g -O2
+    -DCMAKE_CXX_FLAGS:STRING=-g -O2
+  )
+  set(_sentry_cmake_generator -G "Unix Makefiles")
+endif ()
+
+Snapmaker_Orca_add_cmake_project(Sentry
+  GIT_REPOSITORY      https://github.com/getsentry/sentry-native.git
+  GIT_TAG             0.12.1
+  PATCH_COMMAND       ${GIT_EXECUTABLE} submodule update --init --recursive && ${CMAKE_COMMAND} -S external/crashpad -B external/crashpad/build ${_sentry_cmake_generator} -DCMAKE_BUILD_TYPE=Release && ${CMAKE_COMMAND} --build external/crashpad/build --config Release
+  CMAKE_ARGS
+    ${_sentry_cmake_generator}
+    -DCMAKE_INSTALL_DATADIR:STRING=share
+    ${_sentry_platform_flags}
+)
+
+if (MSVC)
+    add_debug_dep(dep_Sentry)
+endif ()

src/CMakeLists.txt

@@ -115,7 +115,7 @@ configure_file(${CMAKE_CURRENT_SOURCE_DIR}/dev-utils/platform/msw/Snapmaker_Orca
 configure_file(${CMAKE_CURRENT_SOURCE_DIR}/dev-utils/platform/msw/Snapmaker_Orca.manifest.in ${CMAKE_CURRENT_BINARY_DIR}/Snapmaker_Orca.manifest @ONLY)
 configure_file(${CMAKE_CURRENT_SOURCE_DIR}/dev-utils/platform/osx/Info.plist.in ${CMAKE_CURRENT_BINARY_DIR}/Info.plist @ONLY)
 if (WIN32)
-    add_library(Snapmaker_Orca SHARED Snapmaker_Orca.cpp Snapmaker_Orca.hpp dev-utils/BaseException.cpp dev-utils/BaseException.h dev-utils/StackWalker.cpp dev-utils/StackWalker.h)
+    add_library(Snapmaker_Orca SHARED Snapmaker_Orca.cpp Snapmaker_Orca.hpp dev-utils/BaseException.cpp dev-utils/BaseException.h dev-utils/StackWalker.cpp dev-utils/StackWalker.h sentry_wrapper/SentryWrapper.hpp sentry_wrapper/SentryWrapper.cpp)
 else ()
     add_executable(Snapmaker_Orca Snapmaker_Orca.cpp Snapmaker_Orca.hpp)
 endif ()
@@ -168,6 +168,12 @@ target_link_libraries(Snapmaker_Orca libslic3r_gui)
     #endif ()
 endif ()
 
+# Sentry crash reporting integration
+if (SLIC3R_SENTRY)
+    target_compile_definitions(Snapmaker_Orca PUBLIC SLIC3R_SENTRY)
+    target_link_libraries(Snapmaker_Orca sentry::sentry)
+endif()
+
 # On Windows, a shim application is required to produce a console / non console version of the Slic3r application.
 # Also the shim may load the Mesa software OpenGL renderer if the default renderer does not support OpenGL 2.0 and higher.
 if (WIN32)
@@ -175,7 +181,7 @@ if (WIN32)
         set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -municode")
     endif()
 
-    add_executable(Snapmaker_Orca_app_gui WIN32 Snapmaker_Orca_app_msvc.cpp ${CMAKE_CURRENT_BINARY_DIR}/Snapmaker_Orca.rc)
+    add_executable(Snapmaker_Orca_app_gui WIN32 Snapmaker_Orca_app_msvc.cpp sentry_wrapper/SentryWrapper.cpp ${CMAKE_CURRENT_BINARY_DIR}/Snapmaker_Orca.rc)
     # Generate debug symbols even in release mode.
     if(MSVC)
         target_link_options(Snapmaker_Orca_app_gui PUBLIC "$<$<CONFIG:RELEASE>:/DEBUG>")
@@ -183,8 +189,15 @@ if (WIN32)
     target_compile_definitions(Snapmaker_Orca_app_gui PRIVATE -DSLIC3R_WRAPPER_NOCONSOLE)
     add_dependencies(Snapmaker_Orca_app_gui Snapmaker_Orca)
     set_target_properties(Snapmaker_Orca_app_gui PROPERTIES OUTPUT_NAME "snapmaker-orca")
+    if (SLIC3R_SENTRY)
+        target_link_libraries(Snapmaker_Orca_app_gui PRIVATE boost_headeronly sentry::sentry)
+    else()
         target_link_libraries(Snapmaker_Orca_app_gui PRIVATE boost_headeronly)
     endif()
+    if (SLIC3R_SENTRY)
+        target_compile_definitions(Snapmaker_Orca_app_gui PRIVATE SLIC3R_SENTRY)
+    endif()
+endif ()
 
 # Link the resources dir to where Slic3r GUI expects it
 set(output_dlls_Release "")

src/Snapmaker_Orca.cpp

@@ -6268,63 +6268,6 @@ std::string CLI::output_filepath(const ModelObject &object, unsigned int index,
 }
 
 
-//BBS: dump stack debug codes, don't delete currently
-//#include <dbghelp.h>
-//#pragma comment(lib, "version.lib")
-//#pragma comment( lib, "dbghelp.lib" )
-/*DWORD main_thread_id;
-std::string TraceStack()
-{
-    static const int MAX_STACK_FRAMES = 16;
-
-    void* pStack[MAX_STACK_FRAMES];
-
-    HANDLE process = GetCurrentProcess();
-    SymInitialize(process, NULL, TRUE);
-    WORD frames = CaptureStackBackTrace(0, MAX_STACK_FRAMES, pStack, NULL);
-
-    std::ostringstream oss;
-    oss << "stack traceback: frames="<< frames << std::endl;
-    for (WORD i = 0; i < frames; ++i) {
-        DWORD64 address = (DWORD64)(pStack[i]);
-
-        DWORD64 displacementSym = 0;
-        char buffer[sizeof(SYMBOL_INFO) + MAX_SYM_NAME * sizeof(TCHAR)];
-        PSYMBOL_INFO pSymbol = (PSYMBOL_INFO)buffer;
-        pSymbol->SizeOfStruct = sizeof(SYMBOL_INFO);
-        pSymbol->MaxNameLen = MAX_SYM_NAME;
-
-        DWORD displacementLine = 0;
-        IMAGEHLP_LINE64 line;
-        //SymSetOptions(SYMOPT_LOAD_LINES);
-        line.SizeOfStruct = sizeof(IMAGEHLP_LINE64);
-
-        if (SymFromAddr(process, address, &displacementSym, pSymbol)
-            && SymGetLineFromAddr64(process, address, &displacementLine, &line)) {
-            oss << "\t" << pSymbol->Name << " at " << line.FileName << ":" << line.LineNumber << "(0x" << std::hex << pSymbol->Address << std::dec << ")" << std::endl;
-        }
-        else {
-            oss << "\terror: " << GetLastError() << std::endl;
-        }
-    }
-    return oss.str();
-}
-
-LONG WINAPI VectoredExceptionHandler(PEXCEPTION_POINTERS pExceptionInfo)
-{
-    std::ofstream f;
-
-    DWORD cur_thread_id = GetCurrentThreadId();
-    f.open("VectoredExceptionHandler.txt", std::ios::out | std::ios::app);
-    f << "main thread id="<<main_thread_id<<", current thread_id="<< cur_thread_id << std::endl;
-    f << std::hex << pExceptionInfo->ExceptionRecord->ExceptionCode << std::endl;
-    f << TraceStack();
-    f.flush();
-    f.close();
-
-    return EXCEPTION_CONTINUE_SEARCH;
-}*/
-
 #if defined(_MSC_VER) || defined(__MINGW32__)
 extern "C" {
     __declspec(dllexport) int __stdcall Snapmaker_Orca_main(int argc, wchar_t **argv)
@@ -6337,13 +6280,6 @@ extern "C" {
         for (size_t i = 0; i < argc; ++ i)
             argv_ptrs[i] = argv_narrow[i].data();
 
-//BBS: register default exception handler
-#if BBL_RELEASE_TO_PUBLIC
-        SET_DEFULTER_HANDLER();
-#else
-        //AddVectoredExceptionHandler(1, CBaseException::UnhandledExceptionFilter);
-        SET_DEFULTER_HANDLER();
-#endif
         std::set_new_handler([]() {
             int *a = nullptr;
             *a     = 0;

src/Snapmaker_Orca_app_msvc.cpp

@@ -6,12 +6,11 @@
 #include <Windows.h>
 #include <shellapi.h>
 #include <wchar.h>
-
+#include <shlobj.h>
-
+#include "sentry_wrapper/SentryWrapper.hpp"
 
 #ifdef SLIC3R_GUI
-extern "C"
+extern "C" {
-{
 // Let the NVIDIA and AMD know we want to use their graphics card
 // on a dual graphics card system.
 __declspec(dllexport) DWORD NvOptimusEnablement                  = 0x00000000;
@@ -31,9 +30,11 @@ extern "C"
 
 #include <boost/algorithm/string/split.hpp>
 #include <boost/algorithm/string/classification.hpp>
-
+#include <boost/asio/ip/host_name.hpp>
 #include <stdio.h>
 
+using namespace Slic3r;
+
 #ifdef SLIC3R_GUI
 class OpenGLVersionCheck
 {
@@ -56,7 +57,8 @@ public:
         wc.lpszClassName = L"Snapmaker_Orca_opengl_version_check";
         wc.style         = CS_OWNDC;
         if (RegisterClass(&wc)) {
-            HWND hwnd = CreateWindowW(wc.lpszClassName, L"Snapmaker_Orca_opengl_version_check", WS_OVERLAPPEDWINDOW, 0, 0, 640, 480, 0, 0, wc.hInstance, (LPVOID)this);
+            HWND hwnd = CreateWindowW(wc.lpszClassName, L"Snapmaker_Orca_opengl_version_check", WS_OVERLAPPEDWINDOW, 0, 0, 640, 480, 0, 0,
+                                      wc.hInstance, (LPVOID) this);
             if (hwnd) {
                 message_pump_exit = false;
                 while (GetMessage(&msg, NULL, 0, 0) > 0 && !message_pump_exit)
@@ -130,25 +132,32 @@ protected:
             return;
         }
 
-        PIXELFORMATDESCRIPTOR pfd =
+        PIXELFORMATDESCRIPTOR pfd = {sizeof(PIXELFORMATDESCRIPTOR),
-        {
-            sizeof(PIXELFORMATDESCRIPTOR),
                                      1,
                                      PFD_DRAW_TO_WINDOW | PFD_SUPPORT_OPENGL | PFD_DOUBLEBUFFER,
                                      PFD_TYPE_RGBA, // The kind of framebuffer. RGBA or palette.
                                      32,            // Color depth of the framebuffer.
-            0, 0, 0, 0, 0, 0,
                                      0,
                                      0,
                                      0,
-            0, 0, 0, 0,
+                                     0,
+                                     0,
+                                     0,
+                                     0,
+                                     0,
+                                     0,
+                                     0,
+                                     0,
+                                     0,
+                                     0,
                                      24, // Number of bits for the depthbuffer
                                      8,  // Number of bits for the stencilbuffer
                                      0,  // Number of Aux buffers in the framebuffer.
                                      PFD_MAIN_PLANE,
                                      0,
-            0, 0, 0
+                                     0,
-        };
+                                     0,
+                                     0};
 
         HDC ourWindowHandleToDeviceContext = ::GetDC(hWnd);
         // Gdi32.dll
@@ -180,21 +189,16 @@ protected:
 
     static LRESULT CALLBACK supports_opengl2_wndproc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
     {
-        switch(message)
+        switch (message) {
-        {
+        case WM_CREATE: {
-        case WM_CREATE:
-        {
             CREATESTRUCT*       pCreate  = reinterpret_cast<CREATESTRUCT*>(lParam);
             OpenGLVersionCheck* ogl_data = reinterpret_cast<OpenGLVersionCheck*>(pCreate->lpCreateParams);
             ogl_data->check(hWnd);
             DestroyWindow(hWnd);
             return 0;
         }
-        case WM_NCDESTROY:
+        case WM_NCDESTROY: message_pump_exit = true; return 0;
-            message_pump_exit = true;
+        default: return DefWindowProc(hWnd, message, wParam, lParam);
-            return 0;
-        default:
-            return DefWindowProc(hWnd, message, wParam, lParam);
         }
     }
 };
@@ -222,9 +226,10 @@ int wmain(int argc, wchar_t **argv)
     // the application will be killed even if "Ignore" button is pressed.
     _set_error_mode(_OUT_TO_MSGBOX);
 
+    initSentry();
+
     std::vector<wchar_t*> argv_extended;
     argv_extended.emplace_back(argv[0]);
-
 #ifdef SLIC3R_WRAPPER_GCODEVIEWER
     wchar_t gcodeviewer_param[] = L"--gcodeviewer";
     argv_extended.emplace_back(gcodeviewer_param);
@@ -280,7 +285,6 @@ int wmain(int argc, wchar_t **argv)
     }
 #endif /* SLIC3R_GUI */
 
-
     wchar_t path_to_slic3r[MAX_PATH + 1] = {0};
     wcscpy(path_to_slic3r, path_to_exe);
     wcscat(path_to_slic3r, L"Snapmaker_Orca.dll");
@@ -288,11 +292,13 @@ int wmain(int argc, wchar_t **argv)
     HINSTANCE hInstance_Slic3r = LoadLibraryExW(path_to_slic3r, nullptr, 0);
     if (hInstance_Slic3r == nullptr) {
         printf("Snapmaker_Orca.dll was not loaded, error=%d\n", GetLastError());
+        exitSentry();
         return -1;
     }
 
     // resolve function address here
-    Snapmaker_Orca_main = (Slic3rMainFunc)GetProcAddress(hInstance_Slic3r,
+    Snapmaker_Orca_main = (Slic3rMainFunc)
+        GetProcAddress(hInstance_Slic3r,
 #ifdef _WIN64
                        // there is just a single calling conversion, therefore no mangling of the function name.
                        "Snapmaker_Orca_main"
@@ -302,9 +308,13 @@ int wmain(int argc, wchar_t **argv)
         );
     if (Snapmaker_Orca_main == nullptr) {
         printf("could not locate the function Snapmaker_Orca_main in Snapmaker_Orca.dll\n");
+        exitSentry();
         return -1;
     }
+
     // argc minus the trailing nullptr of the argv
-    return Snapmaker_Orca_main((int)argv_extended.size() - 1, argv_extended.data());
+    auto res = Snapmaker_Orca_main((int) argv_extended.size() - 1, argv_extended.data());
+    exitSentry();
+    return res;
 }
 }

src/dev-utils/BaseException.h

@@ -26,6 +26,4 @@ protected:
 	boost::nowide::ofstream *output_file;
 };
 
-#define SET_DEFULTER_HANDLER() SetUnhandledExceptionFilter(CBaseException::UnhandledExceptionFilter)
-
 #define SET_DEFAUL_EXCEPTION() _set_se_translator(CBaseException::STF)

src/libslic3r/CMakeLists.txt

@@ -511,7 +511,8 @@ endif ()
 encoding_check(libslic3r)
 
 target_compile_definitions(libslic3r PUBLIC -DUSE_TBB -DTBB_USE_CAPTURED_EXCEPTION=0)
-target_include_directories(libslic3r PRIVATE ${CMAKE_CURRENT_SOURCE_DIR} PUBLIC ${CMAKE_CURRENT_BINARY_DIR})
+# Make source directory PUBLIC so other modules (like libslic3r_gui) can include headers like SentryWrapper.hpp
+target_include_directories(libslic3r PUBLIC ${CMAKE_CURRENT_SOURCE_DIR} ${CMAKE_CURRENT_BINARY_DIR})
 target_include_directories(libslic3r SYSTEM PUBLIC ${EXPAT_INCLUDE_DIRS})
 
 # Find the OCCT and related libraries
@@ -610,3 +611,9 @@ endif()
 if (SLIC3R_PCH AND NOT SLIC3R_SYNTAXONLY)
     add_precompiled_header(libslic3r pchheader.hpp FORCEINCLUDE)
 endif ()
+
+# Sentry crash reporting integration
+if (SLIC3R_SENTRY)
+    target_compile_definitions(libslic3r PUBLIC SLIC3R_SENTRY)
+    target_link_libraries(libslic3r PUBLIC sentry::sentry)
+endif()

src/libslic3r/PresetBundle.cpp

@@ -1837,7 +1837,7 @@ void PresetBundle::export_selections(AppConfig &config)
 void PresetBundle::update_num_filaments(unsigned int to_del_filament_id)
 {
     unsigned old_filament_count = this->filament_presets.size();
-    assert(to_del_flament_id < old_filament_count);
+    assert(to_del_filament_id < old_filament_count);
     filament_presets.erase(filament_presets.begin() + to_del_filament_id);
 
     ConfigOptionStrings* filament_color = project_config.option<ConfigOptionStrings>("filament_colour");

src/sentry_wrapper/SentryWrapper.cpp

@@ -0,0 +1,194 @@
+/**
+ * @file SentryWrapper.cpp
+ * @brief Sentry crash reporting wrapper implementation for cross-platform support.
+ * 
+ * This implementation provides a unified API for Sentry integration.
+ * When SLIC3R_SENTRY is not defined, all functions become no-ops.
+ */
+
+#include "SentryWrapper.hpp"
+
+#ifdef SLIC3R_SENTRY
+#include "sentry.h"
+
+#ifdef _WIN32
+#include <Windows.h>
+#include <shlobj.h>
+#endif
+
+#endif
+
+#include <atomic>
+
+namespace Slic3r {
+
+#ifdef SLIC3R_SENTRY
+
+static sentry_value_t on_crash_callback(const sentry_ucontext_t* uctx, sentry_value_t event, void* closure)
+{
+    (void) uctx;
+    (void) closure;
+
+    // tell the backend to retain the event
+    return event;
+}
+
+void initSentryEx()
+{
+    sentry_options_t* options = sentry_options_new();
+    {
+#ifdef WIN32
+        std::string dsn = std::string("https://c74b617c2aedc291444d3a238d23e780@o4508125599563776.ingest.us.sentry.io/4510425163956224");
+
+        sentry_options_set_dsn(options, dsn.c_str());
+
+        wchar_t exeDir[MAX_PATH];
+        ::GetModuleFileNameW(nullptr, exeDir, MAX_PATH);
+        std::wstring wsExeDir(exeDir);
+        int          nPos     = wsExeDir.find_last_of('\\');
+        std::wstring wsDmpDir = wsExeDir.substr(0, nPos + 1);
+
+        std::wstring handlerDir = wsDmpDir + L"crashpad_handler.exe";
+        wsDmpDir += L"dump";
+
+        auto wstringTostring = [](std::wstring wTmpStr) -> std::string {
+            std::string resStr = std::string();
+            int         len    = WideCharToMultiByte(CP_UTF8, 0, wTmpStr.c_str(), -1, nullptr, 0, nullptr, nullptr);
+
+            if (len <= 0)
+                return std::string();
+            std::string desStr(len, 0);
+
+            WideCharToMultiByte(CP_UTF8, 0, wTmpStr.c_str(), -1, &desStr[0], len, nullptr, nullptr);
+
+            resStr = desStr;
+
+            return resStr;
+        };
+
+        std::string desDir = wstringTostring(handlerDir);
+        if (!desDir.empty())
+            sentry_options_set_handler_path(options, desDir.c_str());
+        desDir                        = wstringTostring(wsDmpDir);
+        desDir                        = wstringTostring(wsDmpDir);
+        wchar_t appDataPath[MAX_PATH] = {0};
+        auto    hr                    = SHGetFolderPathW(NULL, CSIDL_LOCAL_APPDATA, NULL, SHGFP_TYPE_CURRENT, appDataPath);
+        char*   path                  = new char[MAX_PATH];
+        size_t  pathLength;
+        wcstombs_s(&pathLength, path, MAX_PATH, appDataPath, MAX_PATH);
+        std::string filePath = path;
+        std::string appName  = "\\" + std::string("Snapmaker_Orca\\");
+        filePath             = filePath + appName;
+
+        if (!filePath.empty())
+            sentry_options_set_database_path(options, filePath.c_str());
+#endif
+        std::string softVersion = "snapmaker_orca_2.2.0_beta2";
+        // Snapmaker_VERSION
+        sentry_options_set_release(options, softVersion.c_str());
+
+#if defined(_DEBUG) || !defined(NDEBUG)
+        sentry_options_set_debug(options, 1);
+#else
+        sentry_options_set_debug(options, 0);
+#endif
+        // release version environment(Testing/production/development/Staging)
+        sentry_options_set_environment(options, "develop");
+        sentry_options_set_auto_session_tracking(options, false);
+        sentry_options_set_symbolize_stacktraces(options, true);
+        sentry_options_set_on_crash(options, on_crash_callback, NULL);
+        // Enable before_send hook for filtering sensitive data
+        sentry_options_set_before_send(options, NULL, NULL);
+
+        // Ensure all events and crashes are captured (sample rate 100%)
+        sentry_options_set_sample_rate(options, 1.0);        // Capture 100% of events
+        sentry_options_set_traces_sample_rate(options, 1.0); // Capture 100% of traces
+
+        sentry_init(options);
+        sentry_start_session();
+    }
+}
+
+void exitSentryEx()
+{ 
+    sentry_close();
+}
+void sentryReportLogEx(SENTRY_LOG_LEVEL   logLevel,
+                         const std::string& logContent,
+                         const std::string& funcModule,
+                         const std::string& logTagKey,
+                         const std::string& logTagValue,
+                         const std::string& logTraceId)
+{
+    sentry_level_t sentry_msg_level;
+    switch (logLevel)
+    {
+    case SENTRY_LOG_TRACE: 
+        sentry_msg_level = SENTRY_LEVEL_TRACE;
+        break;
+    case SENTRY_LOG_DEBUG: 
+        sentry_msg_level = SENTRY_LEVEL_DEBUG; 
+        break;
+    case SENTRY_LOG_INFO: 
+        sentry_msg_level = SENTRY_LEVEL_INFO; 
+        break;
+    case SENTRY_LOG_WARNING: 
+        sentry_msg_level = SENTRY_LEVEL_WARNING; 
+        break;
+    case SENTRY_LOG_ERROR: 
+        sentry_msg_level = SENTRY_LEVEL_ERROR;
+        break;
+    case SENTRY_LOG_FATAL: 
+        sentry_msg_level = SENTRY_LEVEL_FATAL;
+        break;
+    default:
+        return;
+    }
+
+     sentry_value_t event = sentry_value_new_message_event(sentry_msg_level,           
+                                                           funcModule.c_str(), 
+                                                           logContent.c_str()
+    );
+
+    if (!logTraceId.empty())
+        sentry_set_trace(logTraceId.c_str(), "");
+
+    if (!logTagKey.empty())
+        sentry_set_tag(logTagKey.c_str(), logTagValue.c_str());
+
+    sentry_capture_event(event);
+}
+
+
+#else // SLIC3R_SENTRY not defined - provide no-op implementations
+
+
+#endif // SLIC3R_SENTRY
+
+void initSentry()
+{
+#ifdef SLIC3R_SENTRY
+    initSentryEx();
+#endif
+}
+
+void exitSentry()
+{
+#ifdef SLIC3R_SENTRY
+    exitSentryEx();
+#endif
+}
+void sentryReportLog(SENTRY_LOG_LEVEL   logLevel,
+    const std::string& logContent,
+    const std::string& funcModule,
+    const std::string& logTagKey,
+    const std::string& logTagValue,
+    const std::string& logTraceId)
+{
+#ifdef SLIC3R_SENTRY
+    sentryReportLogEx(logLevel, logContent, funcModule, logTagKey, logTagValue, logTraceId);
+#endif
+}
+
+} // namespace Slic3r
+

src/sentry_wrapper/SentryWrapper.hpp

@@ -0,0 +1,30 @@
+#ifndef slic3r_SentryWrapper_hpp_
+#define slic3r_SentryWrapper_hpp_
+
+#include <string>
+
+namespace Slic3r {
+
+	void initSentry();
+
+	void exitSentry();
+
+    typedef enum SENTRY_LOG_LEVEL {
+        SENTRY_LOG_TRACE   = -2,
+        SENTRY_LOG_DEBUG   = -1,
+        SENTRY_LOG_INFO    = 0,
+        SENTRY_LOG_WARNING = 1,
+        SENTRY_LOG_ERROR   = 2,
+        SENTRY_LOG_FATAL   = 3,
+    };
+
+	void sentryReportLog(SENTRY_LOG_LEVEL   logLevel,
+                        const std::string& logContent,
+                         const std::string& funcModule  = "",
+                         const std::string& logTagKey   = "",
+                         const std::string& logTagValue = "",
+                         const std::string& logTraceId = "");  
+    } // namespace Slic3r
+
+#endif // slic3r_SentryWrapper_hpp_
+

src/slic3r/CMakeLists.txt

@@ -737,3 +737,9 @@ endif ()
 
 # Add a definition so that we can tell we are compiling slic3r.
 target_compile_definitions(libslic3r_gui PRIVATE SLIC3R_CURRENTLY_COMPILING_GUI_MODULE)
+
+# Sentry crash reporting integration
+if (SLIC3R_SENTRY)
+    target_compile_definitions(libslic3r_gui PUBLIC SLIC3R_SENTRY)
+    target_link_libraries(libslic3r_gui sentry::sentry)
+endif()

src/slic3r/GUI/MainFrame.cpp

@@ -72,6 +72,7 @@
 #include <dbt.h>
 #include <shlobj.h>
 #include <shellapi.h>
+#include "sentry_wrapper/SentryWrapper.hpp"
 #endif // _WIN32
 #include <slic3r/GUI/CreatePresetsDialog.hpp>