Skip to content

Remediate ClickPipes and organization OpenAPI drift - #1022

Open
sdairs wants to merge 2 commits into
codex/bump-is-ai-agent-0.6.0from
codex/997-openapi-drift
Open

sdairs wants to merge 2 commits into
codex/bump-is-ai-agent-0.6.0from
codex/997-openapi-drift

Conversation

@sdairs

@sdairs sdairs commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

The live Cloud OpenAPI document exposed fields the Rust library dropped or could not send: organization snapshot eligibility, Kinesis AWS Glue schema registries, and Kafka tombstone handling. Refresh the snapshot and model all nine drift findings.

  • Add optional organization capabilities and typed Kafka tombstone modes.
  • Add strict Kinesis Glue registry requests, tolerant responses, and a fallible response-to-request conversion that reports missing required wire fields.
  • Cover serialization, null/missing fields, unknown enum values, write-back, and client requests/responses; update the library migration notes and CI path mappings and fixtures.

Existing CLI builders leave the new request options unset. This PR adds library support; CLI flags can follow separately.

Fixes #997.

Linked with gh stack link 1019 1022 into native stack #1019, directly above #1021 (codex/bump-is-ai-agent-0.6.0).

Validation:

  • Fresh live-spec dry run: 0 actionable drift, with the existing 3 acknowledged enum constraints unchanged.
  • API/analyzer tests: 705 passed, 26 live/environment-dependent tests ignored.
  • cargo fmt --all; API/analyzer Clippy; both CLI Clippy configurations; telemetry-disabled check; workspace all-features check: passed.
  • Python suite: 95 passed.
  • Local CLI tests: 2,034 passed, 5 ignored, 1 environment-dependent failure. config_and_argument_failures_carry_their_full_human_detail_in_json also fails in isolation because an existing local ClickHouse uses ports 8123/9000 and adds a port-selection notice. Its test and server implementation are unchanged by this PR. Remaining test binaries passed.

The complete CLI CI workflow passed on a clean runner, including all CLI tests and the install path mapping check. All four install platforms also pass. The API-library CI workflow and combined Unit Tests job also pass. The current-head Cloud integration run remains queued; it was requested with run-cloud-integration.

@sdairs
sdairs added this pull request to stack #1019 September 24, 2026 13:02
@sdairs sdairs added the run-cloud-integration Run live Cloud integration tests for this PR head label Sep 24, 2026
@sdairs
sdairs deployed to cloud-integration September 24, 2026 13:02 — with GitHub Actions Active
@sdairs sdairs added run-cloud-integration Run live Cloud integration tests for this PR head and removed run-cloud-integration Run live Cloud integration tests for this PR head labels Sep 24, 2026
@sdairs
sdairs deployed to cloud-integration September 24, 2026 13:05 — with GitHub Actions Active
@sdairs
sdairs marked this pull request as ready for review September 24, 2026 13:13
@sdairs
sdairs requested a review from iskakaushik as a code owner September 24, 2026 13:13

@iskakaushik iskakaushik left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

This branch was successfully deployed

1 active deployment
cloud-integration — 330e7eda Deployed Sep 24, 2026 by sdairs via Cloud integration tests #1206
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-cloud-integration Run live Cloud integration tests for this PR head

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OpenAPI drift: 33 gaps between live spec and library

2 participants