Step 652: local telemetry model

editor/CMakeLists.txt

@@ -4747,4 +4747,13 @@ target_link_libraries(step651_test PRIVATE
     tree_sitter_javascript tree_sitter_typescript
     tree_sitter_java tree_sitter_rust tree_sitter_go)
 
+add_executable(step652_test tests/step652_test.cpp)
+target_include_directories(step652_test PRIVATE src)
+target_link_libraries(step652_test PRIVATE
+    nlohmann_json::nlohmann_json
+    unofficial::tree-sitter::tree-sitter
+    tree_sitter_python tree_sitter_cpp tree_sitter_elisp
+    tree_sitter_javascript tree_sitter_typescript
+    tree_sitter_java tree_sitter_rust tree_sitter_go)
+
 # Step 12: Dear ImGui shell scaffolding created (main.cpp exists but not built due to dependencies)

editor/src/LocalTelemetryModel.h

@@ -0,0 +1,41 @@
+#pragma once
+// Step 652: Privacy-preserving local telemetry
+
+#include <map>
+#include <string>
+
+struct TelemetryCounters {
+    int sessionMinutes = 0;
+    int toolCalls = 0;
+    int errorCount = 0;
+    std::map<std::string, int> generatorUsage;
+};
+
+class LocalTelemetryModel {
+public:
+    static void recordSessionMinutes(TelemetryCounters* c, int minutes) {
+        if (!c || minutes < 0) return;
+        c->sessionMinutes += minutes;
+    }
+
+    static void recordToolCall(TelemetryCounters* c) {
+        if (!c) return;
+        ++c->toolCalls;
+    }
+
+    static void recordError(TelemetryCounters* c) {
+        if (!c) return;
+        ++c->errorCount;
+    }
+
+    static void recordGeneratorUse(TelemetryCounters* c, const std::string& generatorName) {
+        if (!c || generatorName.empty()) return;
+        ++c->generatorUsage[generatorName];
+    }
+
+    static std::string weeklyReport(const TelemetryCounters& c) {
+        return "session_minutes=" + std::to_string(c.sessionMinutes) +
+               ";tool_calls=" + std::to_string(c.toolCalls) +
+               ";errors=" + std::to_string(c.errorCount);
+    }
+};

editor/tests/step652_test.cpp

@@ -0,0 +1,24 @@
+// Step 652: Privacy-preserving local telemetry (12 tests)
+
+#include "LocalTelemetryModel.h"
+#include <iostream>
+static int p=0,f=0;
+#define T(n) { std::cout<<"  "<<#n<<"... "; }
+#define P() { std::cout<<"PASS\n"; ++p; }
+#define F(m) { std::cout<<"FAIL: "<<m<<"\n"; ++f; }
+#define C(c,m) if(!(c)){F(m);return;}
+
+void t1(){T(record_session_minutes_adds_value);TelemetryCounters c;LocalTelemetryModel::recordSessionMinutes(&c,30);C(c.sessionMinutes==30,"minutes");P();}
+void t2(){T(record_session_minutes_ignores_negative);TelemetryCounters c;LocalTelemetryModel::recordSessionMinutes(&c,-1);C(c.sessionMinutes==0,"negative ignored");P();}
+void t3(){T(record_tool_call_increments_counter);TelemetryCounters c;LocalTelemetryModel::recordToolCall(&c);C(c.toolCalls==1,"tool calls");P();}
+void t4(){T(record_error_increments_counter);TelemetryCounters c;LocalTelemetryModel::recordError(&c);C(c.errorCount==1,"errors");P();}
+void t5(){T(record_generator_use_tracks_name);TelemetryCounters c;LocalTelemetryModel::recordGeneratorUse(&c,"RustGenerator");C(c.generatorUsage["RustGenerator"]==1,"usage");P();}
+void t6(){T(record_generator_use_accumulates);TelemetryCounters c;LocalTelemetryModel::recordGeneratorUse(&c,"RustGenerator");LocalTelemetryModel::recordGeneratorUse(&c,"RustGenerator");C(c.generatorUsage["RustGenerator"]==2,"accumulate");P();}
+void t7(){T(record_generator_use_ignores_empty_name);TelemetryCounters c;LocalTelemetryModel::recordGeneratorUse(&c,"");C(c.generatorUsage.empty(),"empty ignored");P();}
+void t8(){T(weekly_report_contains_minutes);TelemetryCounters c;c.sessionMinutes=42;C(LocalTelemetryModel::weeklyReport(c).find("session_minutes=42")!=std::string::npos,"report minutes");P();}
+void t9(){T(weekly_report_contains_tool_calls);TelemetryCounters c;c.toolCalls=9;C(LocalTelemetryModel::weeklyReport(c).find("tool_calls=9")!=std::string::npos,"report tools");P();}
+void t10(){T(weekly_report_contains_errors);TelemetryCounters c;c.errorCount=3;C(LocalTelemetryModel::weeklyReport(c).find("errors=3")!=std::string::npos,"report errors");P();}
+void t11(){T(null_counter_is_noop_for_recorders);LocalTelemetryModel::recordToolCall(nullptr);LocalTelemetryModel::recordError(nullptr);LocalTelemetryModel::recordSessionMinutes(nullptr,1);P();}
+void t12(){T(report_is_local_summary_only);TelemetryCounters c;auto r=LocalTelemetryModel::weeklyReport(c);C(r.find("http")==std::string::npos,"must not include remote endpoints");P();}
+
+int main(){std::cout<<"Step 652: Local telemetry\n";t1();t2();t3();t4();t5();t6();t7();t8();t9();t10();t11();t12();std::cout<<"\nResults: "<<p<<"/"<<(p+f)<<" passed\n";return f?1:0;}

progress.md

@@ -13373,3 +13373,27 @@ language generator extensions (`whetstone-plugin-*.so`) and factory symbol namin
 - `editor/src/LanguageGeneratorPluginManifest.h` (`30` <= `600`)
 - `editor/tests/step651_test.cpp` within test-file size guidance (`26` lines)
 - Header-only architecture and naming conventions remain aligned with `ARCHITECTURE.md`
+
+### Step 652: Privacy-preserving local telemetry
+**Status:** PASS (12/12 tests)
+
+Adds local-only telemetry counters for session/tool/error/generator usage with
+weekly summary formatting and no remote endpoint usage.
+
+**Files added:**
+- `editor/src/LocalTelemetryModel.h` - local telemetry counter model
+- `editor/tests/step652_test.cpp` - 12 tests for recording/reporting behavior
+
+**Files modified:**
+- `editor/CMakeLists.txt` - `step652_test` target
+
+**Verification run:**
+- `cmake -S editor -B editor/build-native` - PASS
+- `cmake --build editor/build-native --target step652_test step651_test` - PASS
+- `./editor/build-native/step652_test` - PASS (12/12)
+- `./editor/build-native/step651_test` - PASS (12/12) regression coverage
+
+**Architecture gate check:**
+- `editor/src/LocalTelemetryModel.h` (`41` <= `600`)
+- `editor/tests/step652_test.cpp` within test-file size guidance (`24` lines)
+- Header-only architecture and naming conventions remain aligned with `ARCHITECTURE.md`