refactor: trim pi helper exports

steipete · steipete · commit 06fe78e4c4f5 · 2026-05-01T21:39:44.000+01:00
diff --git a/src/agents/pi-auth-credentials.ts b/src/agents/pi-auth-credentials.ts
@@ -2,8 +2,8 @@ import { normalizeOptionalString } from "../shared/string-coerce.js";
 import type { AuthProfileCredential, AuthProfileStore } from "./auth-profiles.js";
 import { normalizeProviderId } from "./provider-id.js";
 
-export type PiApiKeyCredential = { type: "api_key"; key: string };
-export type PiOAuthCredential = {
+type PiApiKeyCredential = { type: "api_key"; key: string };
+type PiOAuthCredential = {
   type: "oauth";
   access: string;
   refresh: string;
@@ -13,7 +13,7 @@ export type PiOAuthCredential = {
 export type PiCredential = PiApiKeyCredential | PiOAuthCredential;
 export type PiCredentialMap = Record<string, PiCredential>;
 
-export function convertAuthProfileCredentialToPi(cred: AuthProfileCredential): PiCredential | null {
+function convertAuthProfileCredentialToPi(cred: AuthProfileCredential): PiCredential | null {
   if (cred.type === "api_key") {
     const key = normalizeOptionalString(cred.key) ?? "";
     if (!key) {
diff --git a/src/agents/pi-bundle-mcp-names.ts b/src/agents/pi-bundle-mcp-names.ts
@@ -30,7 +30,7 @@ export function sanitizeServerName(raw: string, usedNames: Set<string>): string
   return candidate;
 }
 
-export function sanitizeToolName(raw: string): string {
+function sanitizeToolName(raw: string): string {
   return sanitizeToolFragment(raw, "tool");
 }
 
diff --git a/src/agents/pi-embedded-subscribe.tools.ts b/src/agents/pi-embedded-subscribe.tools.ts
@@ -325,7 +325,7 @@ export function filterToolResultMediaUrls(
  * returns base64 image data but no file path; those need a different delivery
  * path like saving to a temp file).
  */
-export type ToolResultMediaArtifact = {
+type ToolResultMediaArtifact = {
   mediaUrls: string[];
   audioAsVoice?: boolean;
   trustedLocalMedia?: boolean;
diff --git a/src/agents/pi-settings.ts b/src/agents/pi-settings.ts
@@ -123,9 +123,7 @@ export function applyPiCompactionSettingsFromConfig(params: {
 }
 
 /** Decide whether Pi's internal auto-compaction should be disabled for this run. */
-export function shouldDisablePiAutoCompaction(params: {
-  contextEngineInfo?: ContextEngineInfo;
-}): boolean {
+function shouldDisablePiAutoCompaction(params: { contextEngineInfo?: ContextEngineInfo }): boolean {
   return params.contextEngineInfo?.ownsCompaction === true;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ export function sanitizeServerName(raw: string, usedNames: Set<string>): string`
`30`	`30`	`return candidate;`
`31`	`31`	`}`
`32`	`32`
`33`		`-export function sanitizeToolName(raw: string): string {`
	`33`	`+function sanitizeToolName(raw: string): string {`
`34`	`34`	`return sanitizeToolFragment(raw, "tool");`
`35`	`35`	`}`
`36`	`36`
Original file line number	Diff line number	Diff line change
`@@ -123,9 +123,7 @@ export function applyPiCompactionSettingsFromConfig(params: {`
`123`	`123`	`}`
`124`	`124`
`125`	`125`	`/** Decide whether Pi's internal auto-compaction should be disabled for this run. */`
`126`		`-export function shouldDisablePiAutoCompaction(params: {`
`127`		`- contextEngineInfo?: ContextEngineInfo;`
`128`		`-}): boolean {`
	`126`	`+function shouldDisablePiAutoCompaction(params: { contextEngineInfo?: ContextEngineInfo }): boolean {`
`129`	`127`	`return params.contextEngineInfo?.ownsCompaction === true;`
`130`	`128`	`}`
`131`	`129`