Skip to content

[codex] Refactor api access lifecycle helpers#171

Draft
tkilias wants to merge 3 commits into
pr3/support-num-nodesfrom
pr4/refactor-api-access
Draft

[codex] Refactor api access lifecycle helpers#171
tkilias wants to merge 3 commits into
pr3/support-num-nodesfrom
pr4/refactor-api-access

Conversation

@tkilias

@tkilias tkilias commented Jun 27, 2026

Copy link
Copy Markdown
Collaborator

What changed

This PR applies the structural api_access.py refactor on top of PR 3.

It includes:

  • the _api_access internal package split
  • the slim exasol/saas/client/api_access.py facade
  • the allowed-IP cleanup that accompanies the refactor
  • the unit-test updates needed for the new patch targets and structure

Why it changed

The previous PRs establish the final behavior. This PR keeps that behavior while reorganizing the implementation into smaller internal modules so the lifecycle helpers are easier to reason about and maintain.

Impact

There should be no intended user-facing behavior change relative to PR 3. The impact is internal structure and maintainability.

Validation

  • poetry run pytest -q test/unit/test_api_access.py test/unit/test_ensure_type.py
  • poetry run nox -s format:check
  • poetry run nox -s lint:typing

@tkilias tkilias force-pushed the pr3/support-num-nodes branch from 7482e92 to 95fafc4 Compare June 27, 2026 15:20
@tkilias tkilias force-pushed the pr4/refactor-api-access branch from 965ee38 to 41a1953 Compare June 27, 2026 15:20
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
2 New Code Smells (required ≤ 0)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

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.

1 participant