Skip to content

fix: harden sdk public contract verification - #3

Merged
agentictiger merged 1 commit into
Agentics-Rising:mainfrom
fas89:gh-36-sdk-verification-hardening
Apr 4, 2026
Merged

fix: harden sdk public contract verification#3
agentictiger merged 1 commit into
Agentics-Rising:mainfrom
fas89:gh-36-sdk-verification-hardening

Conversation

@fas89

@fas89 fas89 commented Apr 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Harden forge-cli-sdk verification around its public contract instead of relying on import/build smoke checks alone.

This PR adds direct tests for:

  • top-level public exports and version stability
  • fluid_provider_sdk.testing exports and bundled fixtures
  • action/type/helper behavior (ProviderAction, ApplyResult, ProviderCapabilities, ProviderMetadata, hooks)
  • ContractHelper parsing across the bundled sample contracts
  • a minimal provider passing the bundled ProviderTestHarness

It also includes the two smallest runtime fixes the new tests exposed:

  • ProviderCapabilities is now a real Mapping[str, bool]
  • BaseProvider.get_provider_info() now returns a deterministic non-empty default description

Related PRs / Tracker

Verification

  • python3 -m pytest -q
  • python3 -m ruff check src tests
  • python3 -m black --check src tests
  • python3 -m build --sdist --wheel
  • Verified built wheel and sdist include fluid_provider_sdk/py.typed
  • Verified built wheel and sdist include fluid_provider_sdk/testing
  • Installed the built wheel into a clean venv and imported both fluid_provider_sdk and fluid_provider_sdk.testing

@agentictiger
agentictiger merged commit 6b54e3b into Agentics-Rising:main Apr 4, 2026
6 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

Development

Successfully merging this pull request may close these issues.

2 participants