Update for cleanmodels Go v4 rewrite

.github/workflows/codeql-analysis.yml

@@ -1,72 +1,43 @@
-# For most projects, this workflow file will not need changing; you simply need
-# to commit it to your repository.
-#
-# You may wish to alter this file to override the set of languages analyzed,
-# or to provide custom queries or build logic.
-#
-# ******** NOTE ********
-# We have attempted to detect the languages in your repository. Please check
-# the `language` matrix defined below to confirm you have the correct set of
-# supported CodeQL languages.
-#
-name: "CodeQL"
+name: CodeQL
 
 on:
   push:
-    branches: [ main ]
+    branches: [main]
   pull_request:
-    # The branches below must be a subset of the branches above
-    branches: [ main ]
+    branches: [main]
   schedule:
-    - cron: '20 19 * * 3'
+    - cron: "20 19 * * 3"
+
+permissions:
+  security-events: write
 
 jobs:
   analyze:
     name: Analyze
     runs-on: ubuntu-latest
-
-    strategy:
-      fail-fast: false
-      matrix:
-        language: [ 'cpp' ]
-        # CodeQL supports [ 'cpp', 'csharp', 'go', 'java', 'javascript', 'python' ]
-        # Learn more:
-        # https://docs.github.com/en/free-pro-team@latest/github/finding-security-vulnerabilities-and-errors-in-your-code/configuring-code-scanning#changing-the-languages-that-are-analyzed
-
     steps:
-    - name: Checkout repository
-      uses: actions/checkout@v2
-
-    # Initializes the CodeQL tools for scanning.
-    - name: Initialize CodeQL
-      uses: github/codeql-action/init@v1
-      with:
-        languages: ${{ matrix.language }}
-        # If you wish to specify custom queries, you can do so here or in a config file.
-        # By default, queries listed here will override any specified in a config file.
-        # Prefix the list here with "+" to use these queries and those in the config file.
-        # queries: ./path/to/local/query, your-org/your-repo/queries@main
-
-    - name: Qt Dependencies
-      run: |
-        sudo apt-get update
-        sudo apt-get install qtbase5-dev
-
-    # Autobuild attempts to build any compiled languages  (C/C++, C#, or Java).
-    # If this step fails, then you should remove it and run the build manually (see below)
-    - name: Autobuild
-      uses: github/codeql-action/autobuild@v1
-
-    # ℹ️ Command-line programs to run using the OS shell.
-    # 📚 https://git.io/JvXDl
-
-    # ✏️ If the Autobuild fails above, remove it and uncomment the following three lines
-    #    and modify them (or add more) to build your code if your project
-    #    uses a compiled language
-
-    #- run: |
-    #   make bootstrap
-    #   make release
-
-    - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@v1
+      - uses: actions/checkout@v6
+
+      - name: Install Qt6
+        uses: jurplel/install-qt-action@v4
+        with:
+          version: "6.7.*"
+          cache: true
+
+      - name: Install OpenGL
+        run: |
+          sudo apt-get update
+          sudo apt-get install -y libgl1-mesa-dev libglu1-mesa-dev
+
+      - name: Initialize CodeQL
+        uses: github/codeql-action/init@v3
+        with:
+          languages: cpp
+
+      - name: Configure and build
+        run: |
+          cmake -B build -DCMAKE_BUILD_TYPE=Debug
+          cmake --build build --parallel
+
+      - name: Perform CodeQL Analysis
+        uses: github/codeql-action/analyze@v3

.gitignore

@@ -29,6 +29,7 @@ ui_*.h
 *.jsc
 Makefile*
 *build-*
+build/
 *.qm
 *.prl
 release

CLAUDE.md

@@ -0,0 +1,33 @@
+# cleanmodels-qt
+
+Qt6/C++17 desktop GUI for [cleanmodels](https://github.com/plenarius/cleanmodels), a tool to fix and clean NWN:EE MDL model files. Launches the Go CLI as a subprocess and consumes its `--json-lines` output.
+
+## Architecture
+- **mainwindow.cpp** — UI construction (programmatic, not Designer-driven), settings, directory management
+- **mainwindow_clean.cpp** — QProcess lifecycle, JSON-lines parsing, file table + debug log updates
+- **modelviewport.cpp** — QOpenGLWidget subclass for 3D model preview (OpenGL 3.3 Core)
+- **renderer.cpp** — Scene graph traversal, shader compilation, mesh/texture upload
+- **mdlscene.cpp** — Lightweight ASCII MDL parser (preview-only, not the authoritative parser)
+- **gpumesh.cpp / gputexture.cpp** — GPU resource wrappers
+- **camera.cpp** — Orbit camera with mouse interaction
+- **fsmodel.cpp** — QFileSystemModel wrapper for directory completion
+
+## Design Context
+
+### Users
+NWN community modders who build, fix, and maintain 3D model files (.mdl) for Neverwinter Nights: Enhanced Edition. These users range from experienced 3D artists to module builders with limited technical background. They work in a pipeline that includes 3D editors (NWMax/Blender), the game's toolset, and cleanmodels as the quality gate before assets ship.
+
+### Brand Personality
+**Fast, precise, flexible.** The tool should feel like a sharp instrument — quick to produce correct results, trustworthy in its output, and accommodating of the wide variety of models found in the NWN community (old, malformed, hand-edited, tool-exported). It never corrupts data and always explains what it changed.
+
+### Aesthetic Direction
+- **Qt GUI**: Native OS look — blend in with the platform rather than imposing a custom theme. The interface should feel like a well-made professional utility, not a game launcher or a toy. Polished and intentional, but never flashy.
+- **3D Viewport**: Critical to the workflow. Users rely on it to visually verify fixes. It should be clear, responsive, and show geometry accurately. Grid, wireframe, and reference model overlay are first-class features.
+- **Anti-references**: Not Blender-dark, not game-fantasy-themed, not web-app-styled. Think: native file manager meets engineering tool.
+
+### Design Principles
+1. **Speed is a feature** — The legacy Prolog tool was too slow. Every operation should feel instant on single models, and batch processing should saturate I/O, not CPU. Never block the UI.
+2. **Correctness over convenience** — Binary output must match the game's own compiler. Checks should catch real issues, not generate noise. When in doubt, preserve the original data.
+3. **Approachable complexity** — The tool has deep options (pivot repair, tilefade slicing, walkmesh material remapping) but non-technical modders should be able to hit "All Fixes" and get a good result. Progressive disclosure: simple defaults, advanced options collapsed.
+4. **Explain what changed** — Every fix, warning, and repair should produce a human-readable message. The debug log and per-file fix counts are the user's audit trail.
+5. **Stay focused** — NWN MDL files only. No scope creep into other formats, game engines, or unrelated tooling.

CMakeLists.txt

@@ -1,33 +1,41 @@
-cmake_minimum_required(VERSION 3.10)
-project(cleanmodels-qt)
+cmake_minimum_required(VERSION 3.16)
+project(cleanmodels-qt LANGUAGES CXX)
 
 set(CMAKE_CXX_STANDARD 17)
+set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_INCLUDE_CURRENT_DIR ON)
-set(QT_COMPONENTS Core Widgets Gui)
-
-if(DEFINED ENV{STATIC_BUILD} AND NOT $ENV{STATIC_BUILD} STREQUAL "")
-    if(DEFINED ENV{QT_STATIC_PATH} AND NOT $ENV{QT_STATIC_PATH} STREQUAL "")
-        set(CMAKE_PREFIX_PATH "$ENV{QT_STATIC_PATH}/lib/cmake/Qt5")
-        set(QT_ROOT "$ENV{QT_STATIC_PATH}/lib/cmake/")
-        include("./CMakeModules/Qt.cmake")
-    endif()
-endif()
-
-find_package(Qt5 COMPONENTS ${QT_COMPONENTS} REQUIRED)
 
 set(CMAKE_AUTOMOC ON)
 set(CMAKE_AUTORCC ON)
 set(CMAKE_AUTOUIC ON)
 
-add_executable(${PROJECT_NAME} main.cpp mainwindow.cpp mainwindow_clean.cpp fsmodel.cpp icons.qrc prolog_files.qrc mainwindow.ui)
+find_package(Qt6 REQUIRED COMPONENTS Core Widgets Gui OpenGL OpenGLWidgets)
+
+add_executable(${PROJECT_NAME}
+    main.cpp
+    mainwindow.cpp
+    mainwindow_clean.cpp
+    fsmodel.cpp
+    camera.cpp
+    mdlscene.cpp
+    mdlanimationplayer.cpp
+    gpumesh.cpp
+    gputexture.cpp
+    renderer.cpp
+    modelviewport.cpp
+    constants.h
+    icons.qrc
+    mainwindow.ui
+)
 
-include_directories(BEFORE ${CMAKE_CURRENT_BINARY_DIR})
-target_link_libraries(${PROJECT_NAME} Qt5::Core Qt5::Widgets Qt5::Gui)
+target_link_libraries(${PROJECT_NAME} PRIVATE
+    Qt6::Core Qt6::Widgets Qt6::Gui Qt6::OpenGL Qt6::OpenGLWidgets)
+
+if(APPLE)
+    set_target_properties(${PROJECT_NAME} PROPERTIES MACOSX_BUNDLE TRUE)
+endif()
+if(WIN32)
+    set_target_properties(${PROJECT_NAME} PROPERTIES WIN32_EXECUTABLE TRUE)
+endif()
 
-if(DEFINED ENV{STATIC_BUILD} AND NOT $ENV{STATIC_BUILD} STREQUAL "")
-    if(DEFINED ENV{QT_STATIC_PATH} AND NOT $ENV{QT_STATIC_PATH} STREQUAL "")
-    qt5_import_plugins(${PROJECT_NAME}
-            EXCLUDE_BY_TYPE imageformats
-            )
-    endif()
-endif()
+install(TARGETS ${PROJECT_NAME} DESTINATION bin)