Repository navigation
fix(oci-cloud): dispatch pagination by SDK contract and preserve native responses - #490
Open
dustin-sale wants to merge 2 commits into
Conversation
…ve responses Implemented contract-aware OCI pagination dispatch, safe scalar/aggregate serialization, native collection aggregation with SCIM continuation, retained page metadata and cap completeness, plus public FastMCP regressions using real OCI SDK models. Package tests, root lint, and diff hygiene pass locally. Refs: oracle#477
dustin-sale
marked this pull request as ready for review
October 13, 2026 20:19
dustin-sale
requested review from
AlaaShaker,
gebhardtr and
krisrice
as code owners
October 13, 2026 20:19
…by-sdk-contract-and-preserve-native-57494b66c889
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.
Description
Addresses #477 in the local worktree based on f99eeca09b42987a71a4568605f7efc611932269 (including the new test_native_pagination.py file).
Pagination selection now uses SDK continuation arguments rather than list_/summarize_ prefixes or limit alone. Scalar responses serialize normally. Native SDK envelopes with a single declared record array retain their shape and aggregate records; Identity Domains resources support OPC page tokens and SCIM start_index/count traversal. Original per-page non-record metadata is exposed in result_meta.page_metadata. Scalar and ambiguous multi-array paged aggregates remain intact as single-page payloads with explicit next_page continuation rather than speculative metric merging.
Existing raw-list, items, DNS, Logging Search results, and Object Storage behavior remains covered. Capped results report has_more_results in full and summary modes, expose continuation only at lossless page boundaries, and reject repeated/no-progress traversal. SCIM count=0 remains a metadata-only query. Updated README and changelog. No dependency changes.
Type of change
How Has This Been Tested?
Regression tests exercise production public-method validation, dispatch, serialization, full/summary result shaping, real SDK envelopes, token and SCIM traversal, truncation, metadata-only queries, and no-progress failures. Execution is mocked; no live OCI calls were made.
Test Configuration:
Checklist:
This is a validation proposal only, not a published PR. OCA/signoff and publication prerequisites have not been verified.