Codex analytics connector - #1357
Conversation
There was a problem hiding this comment.
Claude Code Review
Claude Code Review is paused for this repository. To reconnect it, an admin of this repository's GitHub organization (or the account owner, for personal repositories) who can also manage your Claude organization's Code Review settings needs to re-link GitHub in Code Review settings. This is a one-time step.
Tip: disable this comment in your organization's Code Review settings.
There was a problem hiding this comment.
Pull request overview
Adds a new Codex Enterprise Analytics (OpenAI/ChatGPT) connector in beta, including sanitizer rules, example payloads, Terraform connector specs, and end-user documentation.
Changes:
- Register Codex Enterprise Analytics rules in Java prebuilt rule maps and add a Java rules test case.
- Add connector specs and authorization instructions to the Terraform connector-specs module.
- Add mdBook docs + example API responses + YAML rules, and note the connector in
CHANGELOG.md.
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| java/core/src/test/java/co/worklytics/psoxy/rules/chatgpt/CodexEnterpriseAnalyticsTests.java | Adds rules-based test coverage/examples for the new connector (note: test discovery issue flagged in PR comment). |
| java/core/src/main/java/co/worklytics/psoxy/rules/chatgpt/PrebuiltSanitizerRules.java | Registers the new Codex Enterprise Analytics rules and exposes them via the ChatGPT rules map. |
| infra/modules/worklytics-connector-specs/main.tf | Adds the codex-enterprise-analytics connector entry to generated connector specs and example calls. |
| infra/modules/worklytics-connector-specs/docs/chatgpt/codex-enterprise-analytics/instructions.tftpl | Adds end-user instructions for creating/scoping the OpenAI Platform API key and connecting in Worklytics. |
| docs/SUMMARY.md | Adds the connector to the docs navigation. |
| docs/sources/README.md | Adds the connector to the “Sources” index table. |
| docs/sources/codex-enterprise-analytics/README.md | New connector documentation (includes Connector ID + usage/rate-limit notes + examples). |
| docs/sources/codex-enterprise-analytics/example-api-responses/sanitized/usage.json | Sanitized example response for the /usage endpoint. |
| docs/sources/codex-enterprise-analytics/example-api-responses/original/usage.json | Original example response for the /usage endpoint. |
| docs/sources/codex-enterprise-analytics/codex-enterprise-analytics.yaml | New YAML rules: endpoint allowlisting + pseudonymization + response schema filter. |
| CHANGELOG.md | Notes the new beta connector and links to its docs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @Getter | ||
| public class CodexEnterpriseAnalyticsTests extends JavaRulesTestBaseCase { | ||
|
|
# Conflicts: # CHANGELOG.md
There was a problem hiding this comment.
Claude Code Review
Claude Code Review is paused for this repository. To reconnect it, an admin of this repository's GitHub organization (or the account owner, for personal repositories) who can also manage your Claude organization's Code Review settings needs to re-link GitHub in Code Review settings. This is a one-time step.
Tip: disable this comment in your organization's Code Review settings.
Features
Change implications
CHANGELOG.mdanything that will show up interraform plan/applythat isn'tobviously a no-op?
alpha, requires major versionchange