Skip to content

refactor(AI): remove addAi customization#1781

Open
EnkiP wants to merge 2 commits into
mainfrom
feature/prd-805-remove-self-hosted-ai
Open

refactor(AI): remove addAi customization#1781
EnkiP wants to merge 2 commits into
mainfrom
feature/prd-805-remove-self-hosted-ai

Conversation

@EnkiP

@EnkiP EnkiP commented Jul 24, 2026

Copy link
Copy Markdown
Member

Definition of Done

fixes PRD-805

General

  • Write an explicit title for the Pull Request, following Conventional Commits specification
  • Test manually the implemented changes
  • Validate the code quality (indentation, syntax, style, simplicity, readability)

Security

  • Consider the security impact of the changes made

Note

Remove addAi customization and AI proxy routes from the agent

  • Removes the addAi() method from Agent, eliminating AI provider registration, AI proxy route mounting, and ai_llms from generated schema metadata.
  • Deletes AiProxyRoute and the createAiProvider factory from @forestadmin/ai-proxy, along with the AiProviderDefinition/AiRouter/AiProviderMeta interfaces from agent-toolkit.
  • Behavioral Change: Any agent configured with addAi() will break; schema metadata no longer includes the ai_llms field sent to Forest Admin.

Macroscope summarized aa1720a.

@linear-code

linear-code Bot commented Jul 24, 2026

Copy link
Copy Markdown

PRD-805

@qltysh

qltysh Bot commented Jul 24, 2026

Copy link
Copy Markdown

Qlty


Coverage Impact

This PR will not change total coverage.

Modified Files with Diff Coverage (1)

RatingFile% DiffUncovered Line #s
Coverage rating: A Coverage rating: A
packages/agent/src/agent.ts100.0%
Total100.0%
🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

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