Skip to content

feat: enhance pi-agent configuration with detailed model settings and fallback options - #45

Merged
gitricko merged 2 commits into
mainfrom
pi-agent-failover
Aug 23, 2026
Merged

gitricko merged 2 commits into
mainfrom
pi-agent-failover

Conversation

@intricko

@intricko intricko commented Aug 23, 2026

Copy link
Copy Markdown
Collaborator

This pull request updates the development container configuration to improve model definitions, enhance fallback behavior, and streamline startup scripts for local AI model routing and failover.

Model configuration improvements:

  • Expanded the models.json configuration for both omniroute and modelrelay providers to include detailed model metadata (such as name, provider, contextWindow, maxTokens, and cost structure), enabled reasoning, and added fallback logic for automatic failover between providers. Removed the obsolete compat section and placeholder API keys. (.devcontainer/pi-config/models.json)
  • Updated settings.json to remove the unused lastChangelogVersion property, keeping only the relevant default provider and model settings. (.devcontainer/pi-config/settings.json)

Startup and installation script enhancements:

  • Modified the post-create-cmd.sh script to start omniroute with the --no-open flag, preventing it from opening a browser window on startup. (.devcontainer/post-create-cmd.sh)
  • Added installation of a Pi-agent failover extension (pi-failover@hermes-impl) to the start-hermes.sh script, ensuring improved resilience for the Hermes agent. (.devcontainer/start-hermes.sh)
  • Minor cleanup in post-create-cmd.sh by removing an extra blank line. (.devcontainer/post-create-cmd.sh)

@greptile-apps

greptile-apps Bot commented Aug 23, 2026

Copy link
Copy Markdown

Greptile Summary

The PR expands Pi’s local model-provider definitions and configures reciprocal failover between OmniRoute and ModelRelay.

  • Adds detailed metadata and fallback settings for both configured models.
  • Records the failover extension in Pi settings and installs it during Hermes startup.
  • Starts OmniRoute with browser opening disabled.

Confidence Score: 5/5

The PR appears safe to merge because no blocking failure remains within the scope of this follow-up review.

No blocking failure remains.

Important Files Changed

Filename Overview
.devcontainer/pi-config/models.json Expands both local-provider model definitions and adds reciprocal failover chains.
.devcontainer/pi-config/settings.json Adds the Pi failover extension to the persisted package configuration.
.devcontainer/post-create-cmd.sh Starts OmniRoute with --no-open and removes an extra blank line.
.devcontainer/start-hermes.sh Installs the configured failover extension after linking Pi’s tracked configuration.

Reviews (2): Last reviewed commit: "feat: update pi-agent settings with chan..." | Re-trigger Greptile

Comment thread .devcontainer/start-hermes.sh
@gitricko
gitricko merged commit 38e2fa6 into main Aug 23, 2026
7 checks passed
@gitricko
gitricko deleted the pi-agent-failover branch August 23, 2026 05:34
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