Skip to content

refactor(backend): consolidate duplicate venice ai client into single service - #530

Merged
devJaja merged 2 commits into
Epta-Node:mainfrom
AngelAnselmoRS:refactor/consolidate-venice-client-1890
Sep 3, 2026
Merged

refactor(backend): consolidate duplicate venice ai client into single service#530
devJaja merged 2 commits into
Epta-Node:mainfrom
AngelAnselmoRS:refactor/consolidate-venice-client-1890

Conversation

@AngelAnselmoRS

@AngelAnselmoRS AngelAnselmoRS commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Summary

Resolves #1890

Consolidates duplicate Venice AI client implementations into a single canonical service under backend/src/services/venice/.

Changes

  • Moved circuitBreaker.ts and errors.ts to backend/src/services/venice/.
  • Re-exported client modules in backend/src/services/venice/index.ts.
  • Updated import paths across BaseAgent.ts, client.ts, types.ts, and test files.
  • Removed obsolete directory backend/src/venice/.
  • Added regression test singleClient.test.ts to enforce single service guard.

Verification & Testing

  • npm test (Backend/Frontend)
  • cargo test (Smart Contracts)
  • cargo fmt --check && cargo clippy (Contract Linting)

Contributor Checklist

  • PR title follows Conventional Commits (feat:, fix:, docs:, etc.)
  • All CI checks pass locally
  • Documentation updated in docs/ or README.md if applicable
  • No secrets or credentials committed

@vercel

vercel Bot commented Sep 2, 2026

Copy link
Copy Markdown

@AngelAnselmoRS is attempting to deploy a commit to the Jaja's projects Team on Vercel.

A member of the Team first needs to authorize it.

@devJaja
devJaja merged commit 2df3e3b into Epta-Node:main Sep 3, 2026
1 check failed
@AngelAnselmoRS

Copy link
Copy Markdown
Contributor Author

Clarification: This merged PR addresses and resolves issue #491 (Consolidate duplicate Venice AI client implementations into single canonical service).

@AngelAnselmoRS

Copy link
Copy Markdown
Contributor Author

Hi @devJaja! GrantFox support mentioned that because the PR was merged before manual assignment, it can't be updated through Discord support. Could you please check on your GrantFox maintainer dashboard if you can trigger the assignment/reward distribution for issue #491? Thank you!

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