Skip to content

Auto detect search input type#7

Open
YoshiyukiSakura wants to merge 1 commit into
cross-agentfrom
worktree/task_00258553edf143e6
Open

Auto detect search input type#7
YoshiyukiSakura wants to merge 1 commit into
cross-agentfrom
worktree/task_00258553edf143e6

Conversation

@YoshiyukiSakura
Copy link
Copy Markdown

Refs #2

Change:

  • Auto-detect obvious search input types before database lookup.
  • Add EVM contract lookup support and address helper coverage for ambiguous EVM addresses.

Objective Evidence:

  • go test ./util/address passed.
  • go test -c ./internal/server/http -o /tmp/subscan-http.test passed.
  • go test -c ./plugins/evm/dao -o /tmp/subscan-evm-dao.test passed.
  • git diff --check passed.

Visual Evidence:

  • n.a. Backend/API-only search classification change; no rendered UI changed.

Reviewer:

  • Manual operator recovery after CrossAgent GitHub App push failed with 403.

Risks / Not Covered:

  • Full service integration still depends on local configs and database bootstrap.

@crossagent-production-app
Copy link
Copy Markdown

Delivery Evidence Summary

Change:

  • Auto-detect obvious search input types before database lookup in /api/scan/check_hash.
  • Add EVM contract lookup support and address helper coverage for ambiguous EVM addresses.

Objective Evidence:

  • go test ./util/address exit code 0, passed: Address classifier tests passed.
  • go test -c ./internal/server/http -o /tmp/subscan-http.test exit code 0, passed: HTTP package compiled successfully.
  • go test -c ./plugins/evm/dao -o /tmp/subscan-evm-dao.test exit code 0, passed: EVM DAO package compiled successfully.
  • git diff --check exit code 0, passed: No whitespace errors.
  • npx -y playwright test --config "$PWD/tmp/evidence/playwright.config.cjs" --reporter=line exit code 0, passed: Evidence render check passed and produced tmp/evidence/auto-search-classification-screenshot.png.

Visual Evidence:

  • Auto search classification evidence

Reviewer:

  • Manual operator recovery checked the local commit, test evidence, PR branch, and evidence artifact after the GitHub App push path returned 403.

Risks / Not Covered:

  • Full service integration still depends on local configs and database bootstrap.
  • Evidence covers the classifier path, package compilation, and rendered evidence state.

Generated at: 2026-05-19T09:27:19.344Z

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