Telemetry is displaying `$LastModule` in Results page 'Default Workbench' section

[luzpaz]

[luzpaz]

If the configuration path (BaseApp/Preferences/General/AutoloadModule) returns $LastModule, it indicates that:

• This is the stored value in the user's configuration.
• No fallback logic exists to replace placeholders like $LastModule.

Here's a diff for FreeCAD main that ensures $LastModule isn't returned.

diff --git a/src/Gui/StartupProcess.cpp b/src/Gui/StartupProcess.cpp
index 1234567..89abcde 100644
--- a/src/Gui/StartupProcess.cpp
+++ b/src/Gui/StartupProcess.cpp
@@ -474,8 +474,11 @@ void StartupPostProcess::execute() {
     if ("$LastModule" == autoload) {
         start = App::GetApplication()
                     .GetParameterGroupByPath("User parameter:BaseApp/Preferences/General")
                     ->GetASCII("LastModule", start.c_str());
+        if (start.empty()) {
+            Base::Console().Log("Warning: LastModule was not set. Falling back to StartWorkbench.\n");
+            start = "StartWorkbench";  // Default fallback workbench
+        }
     }
     else {
         start = autoload;
     }

@chennes is this the correct solution ?

[luzpaz]

Base::Console().Log("Warning: LastModule was not set. Falling back to StartWorkbench.\n");
start = "StartWorkbench";  // Default fallback workbench

This is probably not the right solution because some people opt-out of Start being their default wb. What would the solution then be ?

[chennes]

The code you wrote basically replaces the existing default with itself, actually, so I expect it to have no effect (it's already defaulting to the Start workbench). Actually, displaying $lastModule is completely correct, and tells devs exactly what we want to know.

[luzpaz]

Oh...my bad

Telemetry is also for the community, so maybe 1$LastModule` can be represented differently or (if there isn't consensus how to do that then) could we add an asterisk explaining its significance?