release/0.3.3-alpha

.github/actions/code-style/trailing-whitespace/action.yml

@@ -29,16 +29,14 @@ runs:
               sed -i 's/\r$//' "$file"
             else
               # Report trailing whitespace
-              while IFS= read -r entry; do
-                path="${entry%%:*}"
-                num="${entry#*:}"
+              while IFS=: read -r file_num; do
                 if [[ "${{ inputs.mode }}" == "soft-check" ]]; then
-                  echo "::warning file=$path,line=$num::Trailing whitespace found"
+                  echo "::warning file=$file,line=$file_num::Trailing whitespace found in $file (line $file_num)"
                 else
-                  echo "::error file=$path,line=$num::Trailing whitespace found"
+                  echo "::error file=$file,line=$file_num::Trailing whitespace found in $file (line $file_num)"
                   ERR=1
                 fi
-              done < <(grep -nE '[[:blank:]]$' "$file" || true)
+              done < <(grep -nE '[^[:space:]]+[[:space:]]+$' "$file" | cut -d: -f1,3- || true)
               # # Report CRLF
               # while IFS= read -r entry; do
               #   path="${entry%%:*}"

.github/labels.yml

@@ -115,6 +115,9 @@
 - name: "provider: OpenAI"
   color: "000"
   description: "Issues related to the OpenAI provider"
+- name: "provider: DeepSeek"
+  color: "000"
+  description: "Issues related to the DeepSeek provider"
 
 # Feature Labels
 - name: "feature: Settings"

.github/workflows/github-issue-labels-on-close.yml

@@ -28,6 +28,27 @@ jobs:
           script: |
             const issue = context.payload.issue;
             const currentLabels = issue.labels.map(label => label.name);
+
+            // Remove all status labels
+            const statusLabels = currentLabels.filter(label => label.toLowerCase().startsWith('status:'));
+
+            if (statusLabels.length > 0) {
+              for (const label of statusLabels) {
+                await github.rest.issues.removeLabel({
+                  owner: context.repo.owner,
+                  repo: context.repo.repo,
+                  issue_number: issue.number,
+                  name: label
+                });
+              }
+              console.log(`Removed status labels: ${statusLabels.join(', ')}`);
+            }
+
+            // Skip close label mapping if issue is completed
+            if (issue.state_reason === 'completed') {
+              console.log('Skipping close-label mapping for completed issue');
+              return;
+            }
 
             // Define label mappings (status label -> close label)
             const labelMappings = {
@@ -55,18 +76,3 @@ jobs:
               });
               console.log(`Added labels: ${labelsToAdd.join(', ')}`);
             }
-
-            // Remove all status labels
-            const statusLabels = currentLabels.filter(label => label.toLowerCase().startsWith('status:'));
-
-            if (statusLabels.length > 0) {
-              for (const label of statusLabels) {
-                await github.rest.issues.removeLabel({
-                  owner: context.repo.owner,
-                  repo: context.repo.repo,
-                  issue_number: issue.number,
-                  name: label
-                });
-              }
-              console.log(`Removed status labels: ${statusLabels.join(', ')}`);
-            }

.github/workflows/pr-delete-auto-branches.yml

@@ -31,8 +31,10 @@ jobs:
       startsWith(github.event.pull_request.head.ref, 'ci/') ||
       startsWith(github.event.pull_request.head.ref, 'release/') ||
       startsWith(github.event.pull_request.head.ref, 'feature/') ||
+      startsWith(github.event.pull_request.head.ref, 'feat/') ||
       startsWith(github.event.pull_request.head.ref, 'hotfix/') ||
-      startsWith(github.event.pull_request.head.ref, 'bugfix/')
+      startsWith(github.event.pull_request.head.ref, 'bugfix/') ||
+      startsWith(github.event.pull_request.head.ref, 'fix/')
     runs-on: ubuntu-latest
     steps:
       - name: Delete branch

.windsurf/rules/ai-provider-conventions.md

@@ -5,4 +5,6 @@ globs: **/SmartHopper.Providers.*/*.cs
 
 # AI Provider conventions
 - Use SmartHopper.Providers.Template as a starting point
-- Implement IAIProviderFactory & IAIProviderSettings
+- Implement IAIProviderFactory & IAIProviderSettings
+- AI providers must wrap any chain-of-thought output or reasoning summary in `<think></think>` tags so that `ChatResourceManager.CreateMessageHtml` can detect and render reasoning.
+- UI settings are created in _controlFactories in SettingsDialog.cs by parsing SettingsDescriptor.

.windsurf/rules/general-guidelines.md

@@ -6,4 +6,5 @@ trigger: always_on
 - Use native Grasshopper types & methods when possible
 - Refer to https://developer.rhino3d.com/ as the official documentation
 - Use English only
-- Only change code lines directly needed to implement the request; avoid unrelated refactors
+- Only change code lines directly needed to implement the request; avoid unrelated refactors
+- Prefer copy/pasting, renaming, and removing files via PowerShell commands, rather than direct edits

.windsurf/rules/using-references-sorting.md

@@ -0,0 +1,5 @@
+---
+trigger: always_on
+---
+
+Place System references first, sorted alphabetically. Then sort other using references in alphabetic order.

.windsurf/rules/webchat-cdn.md

@@ -0,0 +1,6 @@
+---
+trigger: glob
+globs: src/SmartHopper.Core/AI/Chat/*
+---
+
+Always host third-party WebChat JS/CSS dependencies locally rather than using external CDNs (e.g. MathJax).

.windsurf/workflows/changelog-review.md

@@ -0,0 +1,13 @@
+---
+description: Review the changelog and last commits for consistency
+---
+
+Omit this workflow if current branch is main or dev.
+
+1. Get a list of all commits in the current branch that are not in dev branch.
+
+2. Get the messages for all retrieved commits.
+
+3. Compare the list of commits against the [Unreleased] section in CHANGELOG.md.
+
+4. Suggest the user for missing mentions in CHANGELOG

.windsurf/workflows/code-style.md

@@ -0,0 +1,28 @@
+---
+description: Clean code style
+---
+
+1. Ask the user which files they want to clean the code style, if not provided.
+
+2. With the given list of files, perform de following cleaning in *.cs files:
+  - Add docstrings for all elements
+  - When using `/// <inheritdoc/>`, ensure the parent has docstring
+  - There must be a blank line between elements
+  - Code should not contain multiple whitespace characters in a row, such as:
+      Min       = 1, -> Min = 1,
+  - Use trailing comma in multi-line initializers
+  - Single-line comment must be preceded by blank line
+  - Remove multiple consecutive blank lines
+  - Closing brace should be followed by blank line
+  - Add omitted braces in if, for, while, try...
+  - Using statements should be sorted alphabetically, keeping the system ones at the top
+  - The heading of the file should be as follows, replacing YYYY with the creation year of the file:
+/*
+ * SmartHopper - AI-powered Grasshopper Plugin
+ * Copyright (C) YYYY Marc Roca Musach
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 3 of the License, or (at your option) any later version.
+ */

.windsurf/workflows/commit-message.md

@@ -0,0 +1,11 @@
+---
+description: Write a commit message for the currently staged/uncommited changes
+---
+
+The aim is to return a semantic commit message "<type>(<scope>): <subject>" for the current staged or uncommited changes.
+
+1. Check for staged changes or, if none, all uncommited changes
+
+2. Analyze them
+
+3. Return a semantinc commit message

.windsurf/workflows/init.md

@@ -0,0 +1,13 @@
+---
+description: Initialize today's work session
+---
+
+1. Update dev date to today in Solution.props. It must follow X.X.X-dev.YYMMDD format. If it currently is not a -dev version, increment the patch number and add the -dev.YYMMDD part. Examples:
+  - 0.1.0-beta -> 0.1.1-dev.YYMMDD
+  - 0.1.0-dev -> 0.1.0-dev.YYMMDD
+  - 0.1.0 -> 0.1.1-dev.YYMMDD
+  - 0.1.0-dev.250101-> 0.1.0-dev.YYMMDD
+
+2. Update the version badge in README.md. Follow the logic in .github/actions/update-badges/action.yml.
+
+3. Ensure the top section in CHANGELOG.md is [Unreleased].

.windsurf/workflows/pr-description.md

@@ -0,0 +1,17 @@
+---
+description: PR title and description generation
+---
+
+The aim is to return the title and the description for a PR, following the rules in CONTRIBUTING.md.
+
+1. Analyze CONTRIBUTING.md
+
+2. Read the [Unreleased] section in CHANGELOG.md
+
+3. Check for an active PR in the current branch
+
+4. Analyze the changes in the active PR, or the changes between the current branch and the dev branch if no PR
+
+5. Return the PR title in a codeblock as plain text
+
+6. Return the PR description in a seperate codeblock in markdown format

CHANGELOG.md

@@ -7,6 +7,60 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.3.3-alpha] - 2025-06-23
+
+### WIP
+
+- Adding LaTeX support in chat UI with the MathJax library.
+
+### Added
+
+- Added DeepSeek provider ([#222](https://github.com/architects-toolkit/SmartHopper/issues/222)).
+- Added temperature parameter support for MistralAI, OpenAI, and DeepSeek providers.
+- Added slider UI control in settings dialog for numeric parameters.
+- Added reasoning support:
+  - Render reasoning panels for `<think>` tags in chat UI as collapsible `<details>` blocks.
+  - Exclude reasoning from copy-paste (`mdContent`) and include in HTML display (`htmlContent`).
+  - Added configurable `reasoning_effort` setting (low, medium, high) for OpenAI o-series models.
+  - New `StripThinkTags` method in `Config.Utils.AI`.
+  - Set up OpenAI and DeepSeek to return reasoning in the response.
+
+### Changed
+
+- Updated default OpenAI model to gpt-4.1-mini.
+- Mention `DeepSeek` as available provider in the About dialog.
+- Settings dialog improvements:
+  - Added dropdown support for provider settings when a list of allowed values is provided.
+  - Increased max tokens for OpenAI, MistralAI and DeepSeek providers to 100000.
+  - Improved descriptions.
+  - Setting values that are empty or whitespace will be removed from the settings file on `UpdateProviderSettings`.
+- AI providers updates:
+  - Updated deprecated OpenAI max_tokens parameter to max_completion_tokens.
+  - Refactored OpenAI, MistralAI, DeepSeek and TemplateProvider settings validation to use centralized validation methods.
+  - Renamed `OpenAI` to `OpenAIProvider`.
+  - Renamed `OpenAISettings` to `OpenAIProviderSettings`.
+  - Renamed `MistralAI` to `MistralAIProvider`.
+  - Renamed `MistralAISettings` to `MistralAIProviderSettings`.
+  - OpenAI, MisralAI and DeepSeek now remove `<think>` tags from messages before sending them to the API, using the `StripThinkTags` method from `Config.Utils.AI`.
+  - Reorganized providers settings and moved them from `AIProvider` to `AIProviderSettings`.
+
+### Removed
+
+- Provider settings cleanup:
+  - Removed `private LoadSettings` method from `OpenAISettings`.
+  - Removed `private LoadSettings` method from `TemplateProviderSettings`.
+  - Removed `CreateSettingsControl`, `GetSettings` and `LoadSettings` method from `IAIProviderSettings` and all implementations.
+- Removed `ConcatenateItemsToJsonList` method from `ParsingTools` since it was not used.
+
+### Fixed
+
+- Fixed "AI List Filter might not be working as expected" ([#220](https://github.com/architects-toolkit/SmartHopper/issues/220)).
+- Fixes "Accepted feature request: Add compatibility to Magistral thinking (by MistralAI)" ([#223](https://github.com/architects-toolkit/SmartHopper/issues/223)).
+- Fixed limit to 4096 tokens for AI providers in settings dialog.
+- Fixed errors in validation methods of providers.
+- Fixed failing settings integrity check during initialization because providers were not loaded yet.
+- Fixed settings dialog dropdown values not being saved and restored from storage.
+
 ## [0.3.2-alpha] - 2025-06-15
 
 ### Added