Conversation
260ca91 to
dfe94bd
Compare
Co-authored-by: Codex <noreply@openai.com>
| } | ||
|
|
||
| impl HookEventName { | ||
| pub fn analytics_name(self) -> &'static str { |
There was a problem hiding this comment.
we could also give the enum #[strum(serialize_all = "pascal_case")] but this felt more explicit
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bc67544123
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bc67544123
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Why
Add product analytics for hook handler executions so we can understand which hooks are running, where they came from, and whether they completed, failed, stopped, or blocked work.
What
codex_hook_runanalytics event and payload plumbing incodex-rs/analyticscodex-rs/coresystem,user,project, orunknown