Repository navigation
Conversation
Declare the adapter capabilities harness 1.23.0 uses to gate tests: event_uuid_override, bounded_retries, disable_geoip and historical_migration. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…loads, property_layers and fail_open features Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
eli-r-ph
marked this pull request as ready for review
October 28, 2026 00:51
This branch has not been deployed
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.
💡 Motivation and Context
Harness 1.23.0 (posthog-sdk-test-harness#92) brings the
capture_v1andcapture_ai_v1suites up to date with capture v1: 404, 429 and 502 retries, the SDK's own backoff,duplicates_dropped, thewarningresult, caller uuids in lowercase, AI routing and lane isolation. It also gates some tests behind new adapter capabilities.Pin the compliance job to harness
1.23.0.Declare
event_uuid_override,bounded_retries,disable_geoipandhistorical_migration, which this adapter already supports. Without them those tests are skipped.Support the newer harness features (posthog-sdk-test-harness#95) and declare them:
reported_drops(fromon_error),log_capture(theposthoglogger),generated_payloads,property_layers(super_propertiesandbefore_send) andfail_open.Draft until harness 1.23.0 is published. The image tag does not exist until posthog-sdk-test-harness#92 merges and is released; if another release lands first, the version moves.
💚 How did you test it?
Ran every harness suite from the #92 branch locally against this adapter on
v1with these capabilities declared: 235/235 passed, including the new tests. Adapter unit tests pass, with the capability assertion extended.Both v1 suites from the posthog-sdk-test-harness#95 branch, with these features: 223/223.
📝 Checklist
🤖 Agent context
Autonomy: Human-driven (agent-assisted)
Claude Code (Claude Opus 5.5) wrote this change as part of the harness update and ran the compliance suites locally against the adapter.
🤖 Generated with Claude Code