Skip to content

Remove get_reputation and release 2.7.0 - #96

Merged
vvillait88 merged 1 commit into
mainfrom
sweep/2026-10-02
Oct 2, 2026
Merged

vvillait88 merged 1 commit into
mainfrom
sweep/2026-10-02

Conversation

@vvillait88

Copy link
Copy Markdown
Contributor

Summary

GET /v1/reputation was retired on 2026-09-30 along with the transaction pipeline, and the API now answers 404 on it. This removes get_reputation() / aget_reputation() and the types only they returned, and releases 2.7.0.

  • get_reputation() and aget_reputation() are removed.
  • The reputation response types are removed from agentscore.types and the package exports: ReputationResponse, Reputation, Score, ChainScore, ChainEntry, Classification, RedactedClassification, Subject, Identity, Activity, EvidenceSummary, OperatorScore, AgentSummary, Grade, EntityType, ReputationStatus. AssessResponse is unchanged.
  • The create_session docstrings no longer call it "an assessment session for deferred scoring".
  • README, .claude/CLAUDE.md and the package keywords no longer describe a reputation lookup.
  • Dependencies: ruff 0.16.10, python-dotenv 1.2.4, lefthook 2.1.16. Workflows move to uv 0.12.22.

Versioned as a minor on purpose: the method has returned 404 since the endpoint was retired, so no working call changes behavior. Code that still references it raises AttributeError after upgrading.

Type of change

  • Bug fix (no breaking change)
  • New feature (no breaking change)
  • Breaking change (existing callers must update)
  • Docs, tests, or internal maintenance only

Public API

AgentScore.get_reputation(), AgentScore.aget_reputation() and the reputation types listed above are removed. Migration: drop the call. The endpoint behind it no longer exists, so there is no replacement; assess() / aassess() remain the identity and compliance check.

Test plan

  • The generic request tests (error mapping, context managers, concurrency, retries) used get_reputation as their vehicle and now run through list_credentials() / alist_credentials(), so that coverage is kept.
  • The assess and session-poll fixtures now carry the shapes the API returns, not score and chain bodies. A policy test that sent a nonexistent min_score key now sends require_kyc.
  • A new test pins that both methods are absent from the client.
  • uv run pytest (177 passed, 2 integration skipped without credentials, 99.8% coverage), ruff check, ruff format --check, ty check and vulture all pass locally. The OSV scan of uv.lock is clean.

Worked with Varun.

Checklist

  • Tests cover the new behavior, and the suite passes locally
  • Lint, format, and type checks pass
  • Docs and README examples updated if the public surface changed
  • No secrets, credentials, or personal data in the diff or the tests

GET /v1/reputation was retired with the transaction pipeline and now
answers 404, so the sync and async methods and the score, grade, chain
and reputation types that only they returned come out. Test fixtures
now carry the shapes the API returns.
@vvillait88
vvillait88 merged commit 0a66175 into main Oct 2, 2026
12 checks passed
@vvillait88
vvillait88 deleted the sweep/2026-10-02 branch October 2, 2026 18:18
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