Refactor: Move all state from TimeSeriesType/Input/Output interfaces …

[hhenson]

…to Base classes

• Created BaseTimeSeriesInput and BaseTimeSeriesOutput classes containing all state and concrete implementations
• Made TimeSeriesType, TimeSeriesInput, and TimeSeriesOutput pure virtual interfaces with zero state
• Updated all derived classes to inherit from Base classes instead of interfaces
• Fixed nanobind registrations: methods defined on interfaces, hierarchy only on Base classes
• Updated all qualified base class calls from TimeSeriesInput:: to BaseTimeSeriesInput::
• Moved protected virtual methods (do_bind_output, notify_parent) to Base classes
• Replaced all reinterpret_cast with static_cast for type safety
• Fixed TimeSeriesT concept to use std::is_base_of_v instead of exact type matching
• Fixed key_set() to return TimeSeriesSetOutput/Input (not templated version)
• Updated all concrete type nanobind registrations to inherit from Base classes
• Added missing headers to CMakeLists.txt (base_time_series.h, observer headers)

All 1309 tests pass with both Python and C++ runtimes (HGRAPH_USE_CPP=1)

---

Note

Introduce BaseTimeSeriesInput/Output holding all state and behavior, convert TimeSeriesType/Input/Output to pure interfaces, and update types, bindings, and build to match.

• Core Type System:
  • Introduce BaseTimeSeriesInput and BaseTimeSeriesOutput with all state and concrete behavior moved from interfaces.
  • Make TimeSeriesType, TimeSeriesInput, and TimeSeriesOutput pure virtual interfaces; adjust method surfaces accordingly.
  • Update TimeSeriesT concept to use std::is_base_of_v.
• Derived Types & Collections:
  • Migrate all concrete types (ts, tsw, tss, tsd, ts_indexed, ref, ts_signal, tsb) to inherit from Base classes; replace calls like TimeSeriesInput:: with BaseTimeSeriesInput::.
  • Fix key_set() to return TimeSeriesSetOutput/TimeSeriesSetInput and related APIs.
  • Replace unsafe casts with static_cast where applicable; tighten type checks.
• Bindings (nanobind):
  • Keep methods on interfaces; register Base classes for hierarchy; update all class bindings to derive from Base types.
  • Export new Base classes in _hgraph_types.cpp.
• Build/Config:
  • Add new headers and source (types/base_time_series.*) and observer headers to CMake lists.
  • Forward declarations updated to include Base classes and new concept.
• Observers:
  • InspectionObserver constructor drops _track_recent_performance storage; parameter marked unused.
• Docs:
  • CLAUDE.md updated with feature flag usage (HGRAPH_USE_CPP=1) and dev/testing guidance.

^{Written by Cursor Bugbot for commit ab2e994. This will update automatically on new commits. Configure here.}

[hhenson]

Step 1 to clean up existing code (preparation of pure virtual base classes for TimeSeriesInput / Output / Type.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[hhenson]

OK, need to try and keep a clean separation of the required API's from the implementation specific logic. This cleans the code up slightly and should not introduce too much overhead. I need to prepare the code for the new logic in a few smaller steps to reduce the complexity of the change. This is step 1. and should be mostly harmless.