fix: map error_message and failed_step in telemetry edge function#769
Open
sunnnybala wants to merge 1 commit intogarrytan:mainfrom
Open
fix: map error_message and failed_step in telemetry edge function#769sunnnybala wants to merge 1 commit intogarrytan:mainfrom
sunnnybala wants to merge 1 commit intogarrytan:mainfrom
Conversation
These fields are captured locally in JSONL and the Supabase columns exist (added in 002_tighten_rls.sql), but the edge function's TelemetryEvent interface and rows.push() mapping silently dropped them during remote sync. Also updates the preamble epilogue to pass --error-message and --failed-step flags so skills actually report error context on failures. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
error_messageandfailed_stepfields are captured locally in JSONL bygstack-telemetry-logand the Supabase columns exist (added in002_tighten_rls.sql), but the edge function'sTelemetryEventinterface androws.push()mapping silently dropped them during remote sync — resulting in perpetually NULL columns.error_class(optional,String()coercion,.slice()truncation).--error-messageand--failed-stepflags togstack-telemetry-logso skills report error context on failures.Privacy note
Error messages could contain file paths or repo names. The
gstack-telemetry-logbinary already truncates to 200 chars, and the edge function truncates to 500 chars. Maintainers may want to add redaction in the telemetry-log binary if this is a concern relative to the "no file paths or repo names" telemetry promise.Test plan
bun testpasses (static validation)error_messageandfailed_stepto non-NULL values when present in payload""is falsy, shell[ -n "" ]is false)🤖 Generated with Claude Code