Skip to content

Remove the unused identity-token minting - #136

Merged
vvillait88 merged 1 commit into
mainfrom
remove-aip
Oct 8, 2026
Merged

vvillait88 merged 1 commit into
mainfrom
remove-aip

Conversation

@vvillait88

Copy link
Copy Markdown
Contributor

Summary

Removes the unused identity-token code from the CLI:

  • the minting, request-signing and presenting modules;
  • the agent key path, and the request-descriptor helper only they used;
  • their tests;
  • the knip exclusion that kept them as a dormant surface.

No command exposed them. Takes @agent-score/sdk ^2.9.0. Version 0.7.0: a minor.

Worked with Varun, who asked for the feature to be removed everywhere.

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

No CLI command, flag or MCP tool changes.

Test plan

lint, typecheck, knip (now with nothing excluded), test (547 passed, 2 skipped) and build, all green locally.

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

The token-based identity path is retired, so the minting, request signing and
presenting modules, the agent key path, and their tests are removed. Takes
@agent-score/sdk ^2.9.0. Minor bump: no command used it.
@vvillait88
vvillait88 merged commit 411410c into main Oct 8, 2026
6 checks passed
@vvillait88
vvillait88 deleted the remove-aip branch October 8, 2026 03:35
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