@viniciosrab/pi-claude-bridge 0.9.0 → 0.9.1-fork.1

package/README.md

@@ -1,6 +1,6 @@
 # pi-claude-bridge
 
-> **Fork note.** `@viniciosrab/pi-claude-bridge` is an automated mirror of [`pi-claude-bridge`](https://www.npmjs.com/package/pi-claude-bridge) by Eli Dickinson ([elidickinson/pi-claude-bridge](https://github.com/elidickinson/pi-claude-bridge)), rebuilt for each upstream release with one addition: the `provider.loadClaudeSettings` option ([upstream PR #142](https://github.com/elidickinson/pi-claude-bridge/pull/142)). Versions match upstream. Licensed MIT; original copyright and credit belong to Eli Dickinson. Fork: [viniciosrab/pi-claude-bridge](https://github.com/viniciosrab/pi-claude-bridge).
+> **Fork note.** `@viniciosrab/pi-claude-bridge` is an automated mirror of [`pi-claude-bridge`](https://www.npmjs.com/package/pi-claude-bridge) by Eli Dickinson ([elidickinson/pi-claude-bridge](https://github.com/elidickinson/pi-claude-bridge)), rebuilt for each upstream release with fork-only additions: the `provider.loadClaudeSettings` option ([upstream PR #142](https://github.com/elidickinson/pi-claude-bridge/pull/142)) and the `provider.rateLimitWarnings` option. Versions match upstream; a `X.Y.(Z+1)-fork.N` version is upstream `X.Y.Z` with newer fork-only changes. Licensed MIT; original copyright and credit belong to Eli Dickinson. Fork: [viniciosrab/pi-claude-bridge](https://github.com/viniciosrab/pi-claude-bridge).
 
 [![npm version](https://img.shields.io/npm/v/@viniciosrab/pi-claude-bridge)](https://www.npmjs.com/package/@viniciosrab/pi-claude-bridge)
 
@@ -92,6 +92,7 @@ Config: `~/.pi/agent/claude-bridge.json` (global) or the project Pi config direc
 - `strictMcpConfig` — block MCP servers from `~/.claude.json` / `.mcp.json` (default `true`). Cloud MCP (Gmail/Drive via claude.ai OAuth) is always blocked.
 - `autoMemoryEnabled` — enable Claude Code's auto-memory system (default `false`)
 - `loadClaudeSettings` — load Claude Code's user/project/local settings in the provider's Claude Code subprocess (default `true`). Set `false` to skip them: settings-sourced hooks and plugins no longer run on every turn, which can noticeably cut tokens and latency when you have many configured. It also drops settings-sourced `env` and `apiKeyHelper`, so keep the default if you rely on those (e.g. Bedrock/Vertex setup in `settings.json`). OAuth login, pi tools and extensions are unaffected. Applies only to the provider path; AskClaude always loads settings.
+- `rateLimitWarnings` — show Claude rate-limit notices in pi (default `true`): a usage warning at each 5% step once Claude Code reports `allowed_warning`, and a "rate limited" notice with the reset time when a request is rejected. Set `false` to hide both. A rejected request's failure is still named as a rate limit (so fallback chains keep working).
 - `pathToClaudeCodeExecutable` — path to the `claude` binary. Useful if your OS/filesystem has the SDK's bundled musl/glibc binaries in a place where they can't run. For example, with Nix you can set the binary to e.g. `"/home/you/.nix-profile/bin/claude"`.
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@viniciosrab/pi-claude-bridge",
-  "version": "0.9.0",
+  "version": "0.9.1-fork.1",
   "private": false,
   "description": "Pi extension that uses Claude Code (via Agent SDK) as a model provider and adds an AskClaude tool.",
   "keywords": [

package/src/config.ts

@@ -28,6 +28,9 @@ export interface Config {
 		// Set to false to stop the provider's Claude Code subprocess from loading
 		// user/project/local settings (hooks, plugins, env, apiKeyHelper).
 		loadClaudeSettings?: boolean;
+		// Set to false to hide the Claude rate-limit notices (usage warnings and
+		// "rate limited") shown in pi. Failures are still named as rate limits.
+		rateLimitWarnings?: boolean;
 		pathToClaudeCodeExecutable?: string;
 		// Subscription plan tier. Setting to "max" enables Opus 4.6 at 1M context
 		plan?: "pro" | "max";
@@ -61,6 +64,12 @@ export function providerSettingSourcesOption(provider: Config["provider"] = {}):
 	return provider.loadClaudeSettings === false ? { settingSources: [] } : {};
 }
 
+/** Whether to show the Claude rate-limit notices in pi. Absent keeps them on;
+ *  `rateLimitWarnings: false` hides them. */
+export function rateLimitNoticesEnabled(provider: Config["provider"] = {}): boolean {
+	return provider.rateLimitWarnings !== false;
+}
+
 export function globalConfigPath(): string {
 	return join(getAgentDir(), "claude-bridge.json");
 }

package/src/index.ts

@@ -15,7 +15,7 @@ import { verifyWrittenSession as _verifyWrittenSession } from "./session-verify.
 import { extractAllToolResults as _extractAllToolResults, type McpResult } from "./extract-tool-results.js";
 import { QueryContext, ctx } from "./query-state.js";
 import { makePromptStream, userMessage, type PromptStream } from "./prompt-stream.js";
-import { claudeCodeSettings, loadConfig, markStartupNoticeShown, providerSettingSourcesOption, type Config } from "./config.js";
+import { claudeCodeSettings, loadConfig, markStartupNoticeShown, providerSettingSourcesOption, rateLimitNoticesEnabled, type Config } from "./config.js";
 import {
 	collectPromptSkills,
 	projectPromptCapture,
@@ -1485,6 +1485,8 @@ async function consumeQuery(
 		if (message.type === "rate_limit_event") {
 			const info = (message as any).rate_limit_info;
 			debug("consumeQuery: rate_limit_event", JSON.stringify(info).slice(0, 300));
+			// provider.rateLimitWarnings: false hides the notices; the bookkeeping below still runs.
+			const showNotices = rateLimitNoticesEnabled(providerSettings);
 			if (info?.status === "rejected") {
 				// Held so the failure Claude Code sends next can be named as a rate limit.
 				queryCtx.rateLimitRejection = info;
@@ -1494,7 +1496,9 @@ async function consumeQuery(
 				queryCtx.lastRateLimitWarnThreshold = undefined;
 				// resetsAt is Unix seconds, not milliseconds.
 				const resetsAt = info.resetsAt ? new Date(info.resetsAt * 1000).toLocaleTimeString() : "unknown";
-				piUI?.notify(`Claude rate limited (${info.rateLimitType ?? "unknown"}) — resets at ${resetsAt}`, "warning");
+				if (showNotices) {
+					piUI?.notify(`Claude rate limited (${info.rateLimitType ?? "unknown"}) — resets at ${resetsAt}`, "warning");
+				}
 			} else if (info?.status === "allowed") {
 				// Back under the threshold (window reset) — re-arm the warning dedupe.
 				queryCtx.lastRateLimitWarnStep = null;
@@ -1509,7 +1513,9 @@ async function consumeQuery(
 				if (rose || info.surpassedThreshold !== queryCtx.lastRateLimitWarnThreshold) {
 					queryCtx.lastRateLimitWarnStep = step;
 					queryCtx.lastRateLimitWarnThreshold = info.surpassedThreshold;
-					piUI?.notify(`Claude rate limit warning: ${percent}% used (${info.rateLimitType ?? ""})`, "warning");
+					if (showNotices) {
+						piUI?.notify(`Claude rate limit warning: ${percent}% used (${info.rateLimitType ?? ""})`, "warning");
+					}
 				}
 			}
 			continue;