Skip to content

Align Python workflows with supported endpoints and retired APIs - #191

Merged
maxlastglance merged 4 commits into
mainfrom
fix/python-endpoint-contracts
Nov 7, 2026
Merged

maxlastglance merged 4 commits into
mainfrom
fix/python-endpoint-contracts

Conversation

@maxlastglance

@maxlastglance maxlastglance commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

The Python client still calls removed catalog and CSV analysis APIs, expects the old statistics response, and has no working route for current-user access lookup.

Four separate commits address the live findings:

Closes #175. Closes #176. Closes #177. Closes #178. Closes #147.

Validation: 341 SDK unit tests passed locally. Four standalone Metadata endpoint tests passed for owner/member/missing-grant/anonymous cases. Schema tests cover upload and authenticated analysis, explicit/default/sole-image selection, candidate scores, rejected unsupported switches and routing precedence in both gateway configurations. The live user profile against a separate candidate Metadata instance and internal gateway on s93 reports 104 passed/11 failed, versus 100/15 before this batch. All four newly supported cases pass and no previously passing case fails. Three remaining raw failures are the explicitly retired catalog calls; eight belong to other open findings. Candidate containers and their temporary credentials were removed afterward. All PR checks now pass on the final review head; image publishing is intentionally skipped for pull requests. The earlier missing OpenAPI 403/404 declarations were corrected within the isolated #177 commit, and the complete suite passed on rerun.

The live check used a candidate JAR on the existing runtime image and a candidate gateway config; all 577 bundled core classes match a build from the candidate source. The persistent s93 deployment remains on its existing version. Removed catalog functions must be reported as unsupported, never counted as successful catalog reads. Hold merge until the maintainer has reviewed the four commits.

@maxlastglance
maxlastglance force-pushed the fix/python-endpoint-contracts branch from f63ae02 to 6edd6f4 Compare November 7, 2026 09:48
@maxlastglance

Copy link
Copy Markdown
Contributor Author

The Metadata suite exposed missing 403/404 OpenAPI response declarations on the new current-access endpoint (1074 tests, one assertion failure). The declarations are now included in the isolated #177 commit. The existing response-contract check and the four access tests pass locally after the correction; CI is running again.

The PR still contains four commits. The live-tested Python code and endpoint business logic are unchanged. The current review head is 6edd6f42dc5e7fb152d8eccf5458f858fa9f31db. Merge remains on hold for maintainer approval.

@maxlastglance
maxlastglance merged commit 5346bbf into main Nov 7, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant