refactor(ai): remove obsolete context and model adapters - #794
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryThis PR removes obsolete internal AI adapters and routes their remaining caller directly through the existing website lookup and validated model tier.
Confidence Score: 5/5The PR appears safe to merge; the removed adapters have no remaining callers, and the direct replacements preserve validated model selection and website authorization behavior. No actionable failures remain: accepted model tiers match every configured model key, the removed symbols are unused within the private package, and the website access path retains not-found, organization, scope, and permission checks. Important Files Changed
Reviews (1): Last reviewed commit: "refactor(ai): remove obsolete context an..." | Re-trigger Greptile |
The private AI package still exposed a request/authentication path with no callers, a single-caller website validation wrapper, and a model router that only returned its input. Remove those adapters and their unused types and exports. Current callers use the existing website lookup and validated model tier directly.
Website-not-found errors, organization isolation, API-key scopes, session permissions, tier validation, and the balanced default retain their behavior. The existing authorization test now explicitly checks a missing site before permission evaluation. No new files or dependencies. Net reduction: 177 lines across 8 files.
Validation: root lint, all 33 type/build tasks, all 27 test tasks, 76 focused AI tests, and 11 actual API route tests pass. Independent diff review found no issues.
Scope: internal AI adapters and their direct callers. No dependencies or file overlap with open PR #751 or #790.
Summary by cubic
Removes the unused request/authentication path, website validation wrapper, and model router from the private AI package. Callers now use the existing website lookup and validated model tier directly.
Refactors
Written for commit 5382e98. Summary will update on new commits.