You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Scope note: the title ("add xAI OAuth route") under-describes this PR. The diff spans 10 files and is dominantly a profile-scoped path-resolution fix (#40677) — resolving auth.json, the skills dir, and the modal/singularity snapshot stores at call time so long-lived multi-profile runtimes (Dashboard/TUI/Desktop backend, cron) respect the live profile-scoped HERMES_HOME, plus hermes_cli/web_server.py_profile_scope simplification (#60180). It also bundles an unrelated bounded Discord image download (memory guard, plugins/platforms/discord/adapter.py) and owner-only perms on snapshot writes (tools/environments/base.py). Completes merged #59837 and shares the xAI-OAuth auxiliary mechanism with open #34241 (related, not duplicate). Reviewers may want the profile-scope work split from the Discord download bound for cleaner review.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
area/authAuthentication, OAuth, credential poolsbackend/modalModal.com cloud executionbackend/singularitySingularity container executioncomp/agentCore agent runtime: loop, agent_init, prompt builder, context-compression, responses endpointcomp/dashboardWeb dashboard / control panel UI (dashboard/, landing)P2Medium — degraded but workaround existsplatform/discordDiscord bot adaptertool/skillsSkills system (list, view, manage)type/bugSomething isn't working
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
alt-glitch
added
type/bug
What does this PR do?
_auth_refresh_provider_for_route()maps auto-routed base URLs to providers (#59837) but missed xAI OAuth. Addsapi.x.ai->xai-oauthmapping matching the sibling_resolve_provider_from_client().Related Issue
Fixes sibling path of #59837 (#20832).
Type of Change
Changes Made
agent/auxiliary_client.py: Add xAI OAuth route to_auth_refresh_provider_for_route()