Skip to content

fix: wire optional AI SDK into plugin installation - #69

Merged
Pigbibi merged 1 commit into
mainfrom
fix/ai-qsp-sdk-adoption-20260908
Sep 8, 2026
Merged

fix: wire optional AI SDK into plugin installation#69
Pigbibi merged 1 commit into
mainfrom
fix/ai-qsp-sdk-adoption-20260908

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Purpose

The plugin imports ai_gateway_client but did not declare a way to install it. Add an explicit ai extra pinned to the existing standard-library SDK from AAB #158. Default runtime dependencies and AI enablement remain unchanged.

Scope

Five files: package metadata/lock, existing AI tests, existing CI, README. No plugin source, model/auth API, workflow permissions, business switch or runtime configuration changes. CI explicitly installs test,ai and tests the real SDK, mocking only HTTP/OIDC network boundaries.

Validation

  • Fresh base: actual installed-SDK regressions RED, 3 failed (ModuleNotFoundError).
  • Locked non-editable install with ai extra: successful; SDK Git commit matches declared pin.
  • Network-denied AI tests: 56 passed; full suite: 247 passed.
  • Fresh default installation: SDK absent and AI disabled.
  • Offline lock/dependency checks, Ruff and sdist/wheel build passed.
  • Existing lock drift: base lock had Ruff 0.16.4 while pyproject already required 0.16.6. Lock now matches that existing requirement; all other existing dependency identities unchanged.

No actual model, paid API, provider, deployment, publishing or feedback call. Production consumer adoption, source authorization and remote validation remain separate.

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit 88a1a67 into main Sep 8, 2026
2 checks passed
@Pigbibi
Pigbibi deleted the fix/ai-qsp-sdk-adoption-20260908 branch September 8, 2026 00:49
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