Skip to content

[codex] Extract Platform consumer kit helpers#81

Merged
haasonsaas merged 3 commits intomainfrom
codex/platform-consumer-kit
Apr 30, 2026
Merged

[codex] Extract Platform consumer kit helpers#81
haasonsaas merged 3 commits intomainfrom
codex/platform-consumer-kit

Conversation

@haasonsaas
Copy link
Copy Markdown
Collaborator

Summary

  • expose Platform Consumer Kit helpers from the published JS SDK
  • add service endpoint/header builders and a reusable consumer conformance scorecard
  • update Kestrel consumer bootstrap to use the SDK helpers for Platform headers and service base URLs

Related

Validation

  • npm --prefix sdk/js run build
  • npm run build
  • npm --prefix sdk/js run pack:dry

Notes

  • npm run test was rerun after npm run contextkit:build; contextkit pipeline and meeting detection passed, but the existing audio data size test failed on local WAV duration reporting (Duration > 3s (got 0.0s)) despite mic buffers and WAV byte size. This appears unrelated to the TypeScript consumer-kit path.

@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 30, 2026

PR Summary

Medium Risk
Moderate risk because it centralizes and changes how Platform request headers and service base URLs are built, which can affect auth scoping and routing if defaults/mappings are wrong.

Overview
Adds a Platform Consumer Kit to the JS SDK, including typed endpoint/config builders (e.g. buildEvalOpsConsumerClientConfig, buildEvalOpsServiceBaseUrls, buildEvalOpsConsumerHeaders) plus a reusable conformance scorecard via scoreEvalOpsConsumerConformance and EVALOPS_CONSUMER_CONFORMANCE_CHECKS.

Updates the Kestrel main-process EvalOps client bootstrap to use these shared builders (including adding agent/trace header support) and expands tsconfig.node.json to compile the SDK TS sources. Documentation is updated with usage examples for the new helpers.

Reviewed by Cursor Bugbot for commit 7bec0bb. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Token cleaning incorrectly strips trailing slashes
    • The SDK now trims tokens with a dedicated string cleaner so trailing slashes are preserved while URL cleanup remains unchanged.

You can send follow-ups to the cloud agent here.

Reviewed by Cursor Bugbot for commit 6a47786. Configure here.

Comment thread sdk/js/src/index.ts Outdated
@haasonsaas haasonsaas marked this pull request as ready for review April 30, 2026 18:47
@haasonsaas haasonsaas merged commit f347a90 into main Apr 30, 2026
5 checks passed
@haasonsaas haasonsaas deleted the codex/platform-consumer-kit branch April 30, 2026 18:51
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.

2 participants