fix: Do not log health checks to langsmith

Roman Eisnerrequested to merge
re/rm-langsmith-logging-healthcheck into main

What does this merge request do and why?

Removes logging of health checks to langsmith.

How to set up and validate locally

  1. gdk start duo-workflow-service
  2. check langsmith logs to ensure health checks dont appear
  3. other DAP runs should still show up in langsmith

Before: roman-eisner-testing_-_LangSmith

After: Blank lines dont appear on startup.

Merge request checklist

  • Tests added for new functionality. If not, please raise an issue to follow up.
  • Documentation added/updated, if needed.
  • If this change requires executor implementation: verified that issues/MRs exist for both Go executor and Node executor or confirmed that changes are backward-compatible and don't break existing executor functionality.
Edited by Roman Eisner

Merge request reports