pi-free 1.0.2 → 1.0.4

package/CHANGELOG.md

@@ -7,10 +7,37 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.0.4] - 2025-04-03
+
+### Fixed
+- **All tests now passing** (127/127)
+  - Fixed mock paths in kilo.test.ts, zen.test.ts, ollama.test.ts
+  - Fixed createCtxReRegister mocks in zen.test.ts and openrouter.test.ts
+  - Fixed cline.test.ts to test actual provider re-registration behavior
+  - Added missing DEFAULT_MIN_SIZE_B constant to openrouter mock
+
+### Changed
+- **Code quality improvements**
+  - Refactored usage modules to break circular dependency (limits.ts ↔ formatters.ts)
+  - Created usage/types.ts with shared interfaces (FreeTierLimit, FreeTierUsage)
+  - Bumped version to 1.0.4
+
+## [1.0.3] - 2025-04-03
+
+### Changed
+- Updated package.json metadata (name, description, keywords, repository URL)
+- Updated .npmignore for cleaner publishes
+
+## [1.0.0] - 2024-03-28
+
 ### Added
-- Structured logging with namespaced logger (lib/logger.ts)
-- LICENSE file (MIT)
-- CHANGELOG.md
+- Initial release with 6 providers: Kilo, Zen, OpenRouter, NVIDIA, Cline, Fireworks
+- Free tier usage tracking across all sessions
+- Provider failover with model hopping
+- Autocompact integration for rate limit recovery
+- Usage widget with glimpseui
+- Command toggles for free/all model filtering
+- Hardcoded benchmark data from Artificial Analysis
 
 ### Changed
 - **Major refactoring**: Split free-tier-limits.ts into usage/* modules
@@ -46,14 +73,3 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - fetchWithRetry() now properly throws after exhausting retries
 - Auth error pattern matching now handles more message variants
 - Test isolation for free-tier-limits tests
-
-## [1.0.0] - 2024-03-28
-
-### Added
-- Initial release with 6 providers: Kilo, Zen, OpenRouter, NVIDIA, Cline, Fireworks
-- Free tier usage tracking across all sessions
-- Provider failover with model hopping
-- Autocompact integration for rate limit recovery
-- Usage widget with glimpseui
-- Command toggles for free/all model filtering
-- Hardcoded benchmark data from Artificial Analysis

package/package.json

@@ -1,8 +1,8 @@
 {
 	"name": "pi-free",
-	"version": "1.0.2",
+	"version": "1.0.4",
 	"type": "module",
-	"description": "AIO Free AI models for Pi",
+	"description": "AIO Free AI models for Pi - Access free models from Kilo, Zen, OpenRouter, NVIDIA, Cline, Mistral, and Ollama",
 	"keywords": [
 		"pi-package",
 		"pi-extension",
@@ -10,7 +10,6 @@
 		"ai-providers",
 		"openrouter",
 		"nvidia-nim",
-		"opencode",
 		"kilo",
 		"cline",
 		"ollama",
@@ -23,11 +22,36 @@
 	"bugs": {
 		"url": "https://github.com/apmantza/pi-free/issues"
 	},
+	"repository": {
+		"type": "git",
+		"url": "git+https://github.com/apmantza/pi-free.git"
+	},
+	"engines": {
+		"node": ">=20.0.0"
+	},
+	"files": [
+		"providers/**/*.ts",
+		"lib/**/*.ts",
+		"usage/**/*.ts",
+		"config.ts",
+		"constants.ts",
+		"provider-factory.ts",
+		"provider-helper.ts",
+		"README.md",
+		"LICENSE",
+		"CHANGELOG.md"
+	],
 	"scripts": {
 		"test": "vitest",
 		"test:ui": "vitest --ui",
 		"test:run": "vitest run"
 	},
+	"peerDependencies": {
+		"@mariozechner/pi-ai": "*",
+		"@mariozechner/pi-coding-agent": "*",
+		"@mariozechner/pi-tui": "*",
+		"@sinclair/typebox": "*"
+	},
 	"devDependencies": {
 		"vitest": "^1.0.0",
 		"@vitest/ui": "^1.0.0",
@@ -44,9 +68,5 @@
 			"./providers/mistral.ts",
 			"./providers/ollama.ts"
 		]
-	},
-	"repository": {
-		"type": "git",
-		"url": "git+https://github.com/apmantza/pi-free.git"
 	}
 }

package/usage/formatters.ts

@@ -3,25 +3,9 @@
  */
 
 import type { CumulativeUsageReport } from "./cumulative.ts";
-import {
-	type FreeTierLimit,
-	getFreeTierUsage,
-	getLimitWarning,
-} from "./limits.ts";
+import { getFreeTierUsage, getLimitWarning } from "./limits.ts";
 import type { SessionUsageReport } from "./tracking.ts";
-
-export interface FreeTierUsage {
-	provider: string;
-	requestsToday: number;
-	requestsThisHour: number;
-	requestsThisMonth?: number;
-	limit: FreeTierLimit;
-	remainingToday?: number;
-	remainingThisHour?: number;
-	remainingThisMonth?: number;
-	percentUsed: number;
-	status: "ok" | "warning" | "critical" | "unknown";
-}
+// Types imported via limits.ts (which re-exports from types.ts)
 
 export function formatSessionUsage(report: SessionUsageReport): string {
 	if (report.providers.length === 0) {

package/usage/limits.ts

@@ -9,6 +9,7 @@
 
 import { createLogger } from "../lib/logger.ts";
 import { getDailyRequestCount } from "./metrics.ts";
+import type { FreeTierLimit, FreeTierUsage } from "./types.ts";
 
 export {
 	type CumulativeUsageReport,
@@ -30,6 +31,8 @@ export {
 	resetUsageStats,
 	type SessionUsageReport,
 } from "./tracking.ts";
+// Re-export types for consumers
+export type { FreeTierLimit, FreeTierUsage } from "./types.ts";
 
 const _logger = createLogger("free-tier");
 
@@ -37,14 +40,6 @@ const _logger = createLogger("free-tier");
 // Free Tier Limits Configuration
 // =============================================================================
 
-export interface FreeTierLimit {
-	provider: string;
-	requestsPerDay?: number;
-	requestsPerHour?: number;
-	requestsPerMonth?: number;
-	description: string;
-}
-
 export const FREE_TIER_LIMITS: Record<string, FreeTierLimit> = {
 	kilo: {
 		provider: "kilo",
@@ -80,19 +75,6 @@ export const FREE_TIER_LIMITS: Record<string, FreeTierLimit> = {
 // Usage Status and Warnings
 // =============================================================================
 
-export interface FreeTierUsage {
-	provider: string;
-	requestsToday: number;
-	requestsThisHour: number;
-	requestsThisMonth?: number;
-	limit: FreeTierLimit;
-	remainingToday?: number;
-	remainingThisHour?: number;
-	remainingThisMonth?: number;
-	percentUsed: number;
-	status: "ok" | "warning" | "critical" | "unknown";
-}
-
 export function getFreeTierUsage(provider: string): FreeTierUsage {
 	const limit = FREE_TIER_LIMITS[provider];
 	if (!limit) {

package/usage/types.ts

@@ -0,0 +1,26 @@
+/**
+ * Shared types for usage tracking modules
+ *
+ * Extracted to break circular dependency between limits.ts and formatters.ts
+ */
+
+export interface FreeTierLimit {
+	provider: string;
+	requestsPerDay?: number;
+	requestsPerHour?: number;
+	requestsPerMonth?: number;
+	description: string;
+}
+
+export interface FreeTierUsage {
+	provider: string;
+	requestsToday: number;
+	requestsThisHour: number;
+	requestsThisMonth?: number;
+	limit: FreeTierLimit;
+	remainingToday?: number;
+	remainingThisHour?: number;
+	remainingThisMonth?: number;
+	percentUsed: number;
+	status: "ok" | "warning" | "critical" | "unknown";
+}

package/provider-failover/errors.ts

@@ -1,275 +0,0 @@
-/**
- * Error classification for provider failover
- * Detects 429 rate limits, capacity errors, and other provider-specific errors
- */
-
-import { createLogger } from "../lib/logger.ts";
-import { getFreeTierUsage, getLimitWarning } from "../usage/limits.ts";
-
-const _logger = createLogger("failover");
-
-export type ErrorType =
-	| "rate_limit" // 429, quota exceeded
-	| "capacity" // No capacity, overloaded
-	| "auth" // Invalid key, unauthorized
-	| "network" // Timeout, connection error
-	| "unknown"; // Unclassified
-
-export interface ClassifiedError {
-	type: ErrorType;
-	provider?: string;
-	model?: string;
-	statusCode?: number;
-	message: string;
-	retryable: boolean;
-	retryAfterMs?: number; // Server-suggested retry delay
-}
-
-// Pattern matching for various provider error messages
-const RATE_LIMIT_PATTERNS = [
-	/429/i,
-	/rate.?limit/i,
-	/too.?many.?requests/i,
-	/quota.*exceeded/i,
-	/insufficient.*quota/i,
-	/billing.*quota/i,
-	/limit.*exceeded/i,
-	/throttled/i,
-	/ratelimit/i,
-];
-
-const CAPACITY_PATTERNS = [
-	/no.*capacity/i,
-	/overloaded/i,
-	/engine.*overloaded/i,
-	/temporarily.*unavailable/i,
-	/service.*unavailable/i,
-	/503/i,
-	/529/i, // Cloudflare origin is overloaded
-	/busy/i,
-];
-
-const AUTH_PATTERNS = [
-	/401/i,
-	/403/i,
-	/unauthorized/i,
-	/invalid.*key/i,
-	/invalid.*token/i,
-	/authentication/i,
-	/api.*key.*invalid/i,
-	/key.*not.*valid/i,
-	/invalid.*api.*key/i,
-	/invalid.*auth/i,
-];
-
-const NETWORK_PATTERNS = [
-	/timeout/i,
-	/etimedout/i,
-	/enetunreach/i,
-	/econnreset/i,
-	/connection.*refused/i,
-	/fetch.*failed/i,
-	/network.*error/i,
-	/abort/i,
-	/signal/i,
-];
-
-/**
- * Extract HTTP status code from error object or message
- */
-function extractStatusCode(error: unknown): number | undefined {
-	// Check for statusCode property
-	if (
-		typeof error === "object" &&
-		error !== null &&
-		"statusCode" in error &&
-		typeof error.statusCode === "number"
-	) {
-		return error.statusCode;
-	}
-
-	// Check for status property
-	if (
-		typeof error === "object" &&
-		error !== null &&
-		"status" in error &&
-		typeof error.status === "number"
-	) {
-		return error.status;
-	}
-
-	// Extract from message
-	const message = String(error);
-	const match = message.match(/\b(\d{3})\b/);
-	if (match) {
-		const code = Number.parseInt(match[1], 10);
-		if (code >= 400 && code < 600) return code;
-	}
-
-	return undefined;
-}
-
-/**
- * Extract retry-after hint from error
- */
-function extractRetryAfter(error: unknown): number | undefined {
-	const message = String(error);
-
-	// Look for "retry after X seconds/minutes"
-	const secondsMatch = message.match(/retry.?after\s+(\d+)\s*s/i);
-	if (secondsMatch) {
-		return Number.parseInt(secondsMatch[1], 10) * 1000;
-	}
-
-	const minutesMatch = message.match(/retry.?after\s+(\d+)\s*m/i);
-	if (minutesMatch) {
-		return Number.parseInt(minutesMatch[1], 10) * 60 * 1000;
-	}
-
-	// Check for retry_after property
-	if (
-		typeof error === "object" &&
-		error !== null &&
-		"retry_after" in error &&
-		typeof error.retry_after === "number"
-	) {
-		return error.retry_after * 1000;
-	}
-
-	return undefined;
-}
-
-/**
- * Classify an error to determine if it's a 429/capacity issue
- */
-export function classifyError(error: unknown): ClassifiedError {
-	const message = String(error);
-	const statusCode = extractStatusCode(error);
-	const retryAfterMs = extractRetryAfter(error);
-
-	// Check status code first
-	if (statusCode === 429) {
-		return {
-			type: "rate_limit",
-			statusCode,
-			message,
-			retryable: true,
-			retryAfterMs: retryAfterMs ?? 60000, // Default 1 min
-		};
-	}
-
-	if (statusCode === 503 || statusCode === 529) {
-		return {
-			type: "capacity",
-			statusCode,
-			message,
-			retryable: true,
-			retryAfterMs: retryAfterMs ?? 30000, // Default 30 sec
-		};
-	}
-
-	if (statusCode === 401 || statusCode === 403) {
-		return {
-			type: "auth",
-			statusCode,
-			message,
-			retryable: false,
-		};
-	}
-
-	// Check patterns in message
-	if (RATE_LIMIT_PATTERNS.some((p) => p.test(message))) {
-		return {
-			type: "rate_limit",
-			statusCode,
-			message,
-			retryable: true,
-			retryAfterMs: retryAfterMs ?? 60000,
-		};
-	}
-
-	if (CAPACITY_PATTERNS.some((p) => p.test(message))) {
-		return {
-			type: "capacity",
-			statusCode,
-			message,
-			retryable: true,
-			retryAfterMs: retryAfterMs ?? 30000,
-		};
-	}
-
-	if (AUTH_PATTERNS.some((p) => p.test(message))) {
-		return {
-			type: "auth",
-			statusCode,
-			message,
-			retryable: false,
-		};
-	}
-
-	if (NETWORK_PATTERNS.some((p) => p.test(message))) {
-		return {
-			type: "network",
-			statusCode,
-			message,
-			retryable: true,
-			retryAfterMs: 5000, // Short retry for network
-		};
-	}
-
-	// Unknown error - assume retryable but with caution
-	return {
-		type: "unknown",
-		statusCode,
-		message,
-		retryable: statusCode ? statusCode >= 500 : true,
-		retryAfterMs: 10000,
-	};
-}
-
-/**
- * Check if error is specifically a rate limit (429)
- */
-export function isRateLimit(error: unknown): boolean {
-	return classifyError(error).type === "rate_limit";
-}
-
-/**
- * Check if error is capacity-related (provider overloaded)
- */
-export function isCapacityError(error: unknown): boolean {
-	return classifyError(error).type === "capacity";
-}
-
-/**
- * Log error classification for debugging
- */
-export function logErrorClassification(
-	_error: unknown,
-	classified: ClassifiedError,
-): void {
-	_logger.info(`Error classified: ${classified.type}`, {
-		statusCode: classified.statusCode,
-		retryable: classified.retryable,
-		retryAfterMs: classified.retryAfterMs,
-		message: classified.message.slice(0, 100),
-	});
-}
-
-/**
- * Log free tier usage when rate limit occurs
- * Helps users understand their quota consumption
- */
-export function logFreeTierUsage(provider: string): void {
-	const usage = getFreeTierUsage(provider);
-	const warning = getLimitWarning(provider);
-
-	if (warning) {
-		_logger.warn(`Free tier warning: ${warning}`, { provider });
-	} else {
-		_logger.info(`${provider} usage`, {
-			requestsToday: usage.requestsToday,
-			limit: usage.limit.description,
-		});
-	}
-}