fix: add pnpm patch for pi-ai to support LiteLLM providerType#23
Closed
tiagoefreitas wants to merge 1 commit intoopenclaw:mainfrom
Closed
fix: add pnpm patch for pi-ai to support LiteLLM providerType#23tiagoefreitas wants to merge 1 commit intoopenclaw:mainfrom
tiagoefreitas wants to merge 1 commit intoopenclaw:mainfrom
Conversation
When using Claude models via LiteLLM proxy, the bundled pi-ai sends OpenAI-specific parameters (store, max_completion_tokens) that cause errors with non-OpenAI backends like AWS Bedrock. This patch adds providerType support to openai-completions.js: - Check model.providerType to determine if model is OpenAI-native - Only send OpenAI-specific params to actual OpenAI models - Use max_tokens instead of max_completion_tokens for proxy models Users can now configure custom models in ~/.pi/agent/models.json with providerType: "anthropic" (or "google", etc.) to properly route requests through LiteLLM to Claude and other non-OpenAI models. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Contributor
|
Interesting addition! You can maintain your own fork of pi-ai like I do in the config: Also try upstreaming your changes to pi-ai! That's the best long-term solution. |
Pastorsimon1798
pushed a commit
to Pastorsimon1798/openclaw
that referenced
this pull request
Feb 1, 2026
- Create analyze.py (profile, describe, groupby, pivot, filter) - Create excel.py (read, write, list-sheets, merge/VLOOKUP) - Create visualize.py (bar, line, scatter, heatmap, histogram, pie, box) - Update SKILL.md with venv instructions - Mark openclaw#14 Blogwatcher RESOLVED (installed) - Mark openclaw#15 Data Analytics RESOLVED - Mark openclaw#17 Calendar CANCELLED (using GOG) - Mark openclaw#18 Edison PAUSED - Mark openclaw#23 Network Fallback PAUSED
dgarson
referenced
this pull request
in dgarson/clawdbot
Feb 2, 2026
feat(ui): add skeleton loading screens for overseer and cron views
alexprime1889-prog
pushed a commit
to alexprime1889-prog/moltbot
that referenced
this pull request
Feb 8, 2026
…t entrypoint script
slathrop
referenced
this pull request
in slathrop/openclaw-js
Feb 11, 2026
Tasks completed: 2/2 - Port PR/issue guides + Install sharpening (commits #15, #17, #23) - Port PR sign-off + guide revisions + markdownlint fixes (commits #24, #26, #35) SUMMARY: .planning/phases/17-docs/17-03-SUMMARY.md
frodo-harborbot
added a commit
to harborworks/openclaw
that referenced
this pull request
Feb 16, 2026
…openclaw#23) The database service's environment block used ${DATABASE_PASSWORD:-postgres} which is interpolated by docker compose from the host shell — where the var is never set, so it always defaults to 'postgres'. The backend reads .env.prod and gets the real password, causing a mismatch. Fix: remove the environment block from the database service and pass POSTGRES_DB, POSTGRES_USER, POSTGRES_PASSWORD directly via .env.prod. The deploy workflow now writes these vars into .env.prod alongside the existing DATABASE_* vars.
|
@claude please review this PR |
ivanuser
added a commit
to ivanuser/cortex
that referenced
this pull request
Feb 23, 2026
…claw#22, openclaw#23) Issue openclaw#22 — QR Code Pairing: - QRPairModal component: generates QR with encoded gateway URL + invite code + role - Nodes page: QR Pair button in header toolbar - SetupWizard: detects ?qr= URL parameter, auto-fills gateway URL and invite code - Uses invite.create RPC to generate one-time invite codes (falls back if unavailable) - QR codes auto-expire after 5 minutes with regenerate prompt Issue openclaw#23 — LAN Auto-Approve: - Added gateway.security.lanAutoApprove config option (default: false) - Added gateway.security.lanAutoApproveRole config option (default: 'operator') - Config types (GatewaySecurityConfig) and zod schema validation - Message handler checks isPrivateOrLoopbackAddress for LAN clients - LAN auto-approved devices get securityRole from config - Audit logged as device.pair.lan-auto-approve - approveReason field added to PairedDevice type - Nodes page shows 'LAN auto-approved' badge on auto-approved devices Also includes invite/pairing code validation in connect handshake.
emmalone
added a commit
to emmalone/openclaw
that referenced
this pull request
Mar 1, 2026
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Piboonsak
added a commit
to Piboonsak/openclaw_github
that referenced
this pull request
Mar 3, 2026
Config changes (openclaw#33): - Switch primary model to anthropic/claude-sonnet-4-5 (Claude Sonnet 4.6) - Add model to deploy.sh post-deploy config, update fallback chain Nginx changes (openclaw#32, openclaw#21): - Increase /line/ proxy_read_timeout from 120s to 300s for complex agent responses Docker build (openclaw#39): - Enable OPENCLAW_INSTALL_BROWSER=1 build-arg to include Chromium in image Deploy pipeline fix: - deploy-vps.yml now actually applies nginx config (was only staging to .new) - Added backup + rollback safety for nginx config changes - Removed broken nginx path from deploy.sh (was referencing wrong VPS path) Issues addressed: openclaw#9, openclaw#20, openclaw#21, openclaw#22, openclaw#23, openclaw#32, openclaw#33, openclaw#37, openclaw#39, openclaw#42, openclaw#45, openclaw#46
benieralexis-sudo
pushed a commit
to benieralexis-sudo/openclaw
that referenced
this pull request
Mar 6, 2026
MOYENS (16): - openclaw#1 generateReactiveFollowUp maxLength string vs number — aligne sur string - openclaw#2 Multi-threading mort — filtre sur mappedLeads (avec score) au lieu de result.leads - openclaw#3 _checkEmailSpecificity null safety — (prospectIntel || '').match() - openclaw#4 totalContacts pre-dedup — utilise addedCount - openclaw#5 Bounces tous hard_bounce — distinction soft/hard, soft = retry pas blacklist - openclaw#8 _isSystemEmail includes() trop large — exact match + prefix patterns - openclaw#10 sendBatch bypass warmup — recordSend dans domain-manager apres chaque envoi - openclaw#11 Domain manager parsing ':' dans password — split limite a 4 segments - openclaw#12 prospectDomains jamais nettoye — cleanup LRU max 2000 entries - openclaw#13 trackNicheEvent('clicked') ignore — ajout compteur clicked - openclaw#14 apStorage.getLeads() inexistant — utilise automailer storage.getEmails() - openclaw#25 Interval HITL drafts jamais cleared — clearInterval dans gracefulShutdown FAIBLES (9): - openclaw#17 Retry skip prompt 25-35 mots → aligne sur 50-65 mots 4-blocs - openclaw#18 _scoreAndFilter gate 60 mots → 80 mots (coherent avec prompts) - openclaw#19 "curieux d'avoir ton retour" autorise dans follow-ups → remplace - openclaw#20 seniorities priorite inversee → brain > config - openclaw#21 data.growth jamais active → detecte via employee count + multi-country - openclaw#23 Fallback plan non-pondere → utilise icp-loader.getNicheForCycle() - campaign-engine: double updateEmailStatus pour opened → supprime appel redondant Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
elliot-ylambda
pushed a commit
to elliot-ylambda/magister-openclaw
that referenced
this pull request
Mar 7, 2026
Release: footer contact email + waitlist email improvements
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
@mariozechner/pi-aito supportproviderTypefield for custom modelsstore,max_completion_tokens) that cause errors with non-OpenAI backends like AWS Bedrockmodel.providerTypeto determine if the model is OpenAI-native before sending OpenAI-specific paramsProblem
When configuring custom models in
~/.pi/agent/models.jsonthat route through LiteLLM to Claude (e.g. AWS Bedrock), the bundled pi-ai would send:store: false(OpenAI-specific)max_completion_tokensinstead ofmax_tokensThis caused errors like:
Solution
The patch adds
providerTypesupport toopenai-completions.js:model.providerType === "openai"or the baseUrl indicates OpenAI, send OpenAI-specific paramsmax_tokens)Users can now configure custom models with
providerType: "anthropic"(or"google", etc.) to properly route requests.Test plan
🤖 Generated with Claude Code