Skip to content

Add missing Solana config properties to BotConfig interface#2

Merged
overstepping merged 1 commit into
mainfrom
railway/code-change-R7mn98
May 11, 2026
Merged

Add missing Solana config properties to BotConfig interface#2
overstepping merged 1 commit into
mainfrom
railway/code-change-R7mn98

Conversation

@railway-app
Copy link
Copy Markdown
Contributor

@railway-app railway-app Bot commented May 11, 2026

Problem

The build fails with 7 TypeScript errors in packages/claim/src/rpc-monitor.ts because RpcClaimMonitor accesses solanaRpcUrl, solanaWsUrl, and pollIntervalSeconds on BotConfig, but none of these properties were declared in the interface in types.ts.

Solution

Added the three missing optional properties to the BotConfig interface in packages/claim/src/types.ts: solanaRpcUrl?: string, solanaWsUrl?: string, and pollIntervalSeconds?: number. All three are marked optional to match their usage in rpc-monitor.ts (non-null assertions, truthiness checks, and nullish coalescing with a default), which resolves all 7 compilation errors.

Changes

  • Modified packages/claim/src/types.ts

Generated by Railway

@vercel
Copy link
Copy Markdown

vercel Bot commented May 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pumpkit Ready Ready Preview, Comment May 11, 2026 9:45am

Request Review

@railway-app railway-app Bot force-pushed the railway/code-change-R7mn98 branch from 47dc569 to 8805902 Compare May 11, 2026 09:44
@overstepping overstepping merged commit a7beb9e into main May 11, 2026
3 of 6 checks passed
@overstepping overstepping deleted the railway/code-change-R7mn98 branch May 11, 2026 21:33
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.

1 participant