Skip to content

chore: log read / write errors when persisting the telemetry state file#2016

Open
yulunz wants to merge 1 commit intomainfrom
chore/log-file-persistence-errors
Open

chore: log read / write errors when persisting the telemetry state file#2016
yulunz wants to merge 1 commit intomainfrom
chore/log-file-persistence-errors

Conversation

@yulunz
Copy link
Copy Markdown
Collaborator

@yulunz yulunz commented May 8, 2026

This adds logging to the read & write operations to the telemetry state file. In particular,

  • for read, no error is logged if the state file doesn't exist, which is expected to happen when the user is new. An error is logged otherwise (e.g. file format error, errors when reading an existing file like permission issue).
  • for write, any error in the path will be logged.

This commit also puts the instantiation of the Persistence object out of the ClearcutLogger object to avoid circular dependency.

@yulunz yulunz requested a review from OrKoN May 8, 2026 19:40
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