Skip to content

debug: add logging to interpretWriteTurn pipeline#79

Merged
MaxLinCode merged 1 commit into
mainfrom
claude/interpret-logging
Apr 5, 2026
Merged

debug: add logging to interpretWriteTurn pipeline#79
MaxLinCode merged 1 commit into
mainfrom
claude/interpret-logging

Conversation

@MaxLinCode
Copy link
Copy Markdown
Owner

Summary

  • Log interpret_write_turn_error and interpret_write_turn_parse_failed instead of silently falling back to empty fields
  • Add interpret_write_turn_gate log to confirm whether the WRITE_INTERPRETING_TURN_TYPES check passes before the call

Context

interpretWriteTurn failures were silently caught, returning a fallback with empty fields: {}. This caused applyWriteCommit to report missing schedule fields even when the user provided them (e.g., "Tuesday 10am"), routing to ask_clarification incorrectly.

Test plan

  • Typecheck passes
  • Existing interpret-write-turn tests pass
  • Deploy and reproduce the issue — check logs for gate/error output

🤖 Generated with Claude Code

Logs whether WRITE_INTERPRETING_TURN_TYPES check passes, to confirm
whether the function is actually being reached.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
atlas-web Ignored Ignored Apr 5, 2026 6:31pm

@MaxLinCode MaxLinCode merged commit 6113467 into main Apr 5, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant