Skip to content

feat: erc8004 identity & reputation - #7

Merged
andrew-virtuals merged 8 commits into
mainfrom
feat/erc8004
Apr 24, 2026
Merged

feat: erc8004 identity & reputation#7
andrew-virtuals merged 8 commits into
mainfrom
feat/erc8004

Conversation

@andrew-virtuals

@andrew-virtualsandrew-virtuals commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

Note

Medium Risk
Adds new on-chain signing/transaction flows (ERC-8004 registration and reputation reviews) and temporarily switches the active wallet, which can affect user funds/identity if misused or if chain selection/typed-data signing is wrong.

Overview
Adds ERC-8004 identity support to the CLI via acp agent register-erc8004, including an end-to-end flow that fetches backend-prepared registration data, sends either a transaction or EIP-712 signature, and then confirms registration with the API; agent creation now attempts auto-registration after signer setup.

Adds job reviews/reputation via acp client review, validating rating/text, requesting feedback transaction data from the API, submitting an on-chain call when required (otherwise recording off-chain), then confirming feedback with the API.

Extends API types/endpoints to include erc8004AgentId on agent chains plus new AgentApi methods for ERC-8004 registration and job feedback, and updates docs (README.md, SKILL.md) to document both commands and behaviors.

Reviewed by Cursor Bugbot for commit d3d045e. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursorcursorBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 3 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a089b4b. Configure here.

Comment threadsrc/lib/api/agent.ts
Comment threadsrc/commands/agent.ts Outdated
Comment threadsrc/commands/agent.ts Outdated
@andrew-virtuals
andrew-virtuals merged commit fbf48ee into mainApr 24, 2026
1 check passed
Sign up for freeto 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

@andrew-virtuals@Zuhwa