Skip to content

Commit 3f0d774

Browse files
crazyrabbitLTCclaude
crazyrabbitLTC
and
claude
committed
fix: update TwitterClient import paths and TypeScript parameter types
- Changed import from '../twitterClient.js' to '../client/twitter.js' in: - src/handlers/user.handlers.ts - src/handlers/search.handlers.ts - Added proper TypeScript parameter type annotations: - Fixed anonymous function parameter types in user.handlers.ts (response: any) - Fixed find callback parameter type in search.handlers.ts (u: UserV2) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent a0879f3 commit 3f0d774

File tree

2 files changed

+69
-30
lines changed

2 files changed

+69
-30
lines changed

‎src/handlers/search.handlers.ts‎

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
1-
import { TwitterClient } from '../twitterClient.js';
1+
import { TwitterClient } from '../client/twitter.js';
22
import { HandlerResponse, TwitterHandler } from '../types/handlers.js';
33
import { createResponse } from '../utils/response.js';
4+
import { createMissingTwitterApiKeyResponse, formatTwitterError } from '../utils/twitter-response.js';
45
import { TweetV2, TwitterApiReadOnly, UserV2, TweetSearchRecentV2Paginator } from 'twitter-api-v2';
56

67
interface SearchTweetsArgs {
@@ -20,9 +21,12 @@ interface TweetWithAuthor extends TweetV2 {
2021
}
2122

2223
export const handleSearchTweets: TwitterHandler<SearchTweetsArgs> = async (
23-
client: TwitterClient,
24+
client: TwitterClient | null,
2425
{ query, maxResults = 10, tweetFields }: SearchTweetsArgs
2526
): Promise<HandlerResponse> => {
27+
if (!client) {
28+
return createMissingTwitterApiKeyResponse('searchTweets');
29+
}
2630
try {
2731
const searchResult = await client.v2.search(query, {
2832
max_results: maxResults,
@@ -38,7 +42,7 @@ export const handleSearchTweets: TwitterHandler<SearchTweetsArgs> = async (
3842

3943
const formattedTweets = tweets.map((tweet: TweetV2): TweetWithAuthor => ({
4044
...tweet,
41-
author: searchResult.includes?.users?.find(u => u.id === tweet.author_id)
45+
author: searchResult.includes?.users?.find((u: UserV2) => u.id === tweet.author_id)
4246
}));
4347

4448
return createResponse(`Search results: ${JSON.stringify(formattedTweets, null, 2)}`);
@@ -47,16 +51,19 @@ export const handleSearchTweets: TwitterHandler<SearchTweetsArgs> = async (
4751
if (error.message.includes('400') && error.message.includes('Invalid Request')) {
4852
throw new Error(`Search functionality requires Pro tier access ($5,000/month) or higher. Current Basic tier ($200/month) does not include recent search API access. Consider upgrading at https://developer.x.com/en/portal/products/pro or use alternative data sources.`);
4953
}
50-
throw new Error(`Failed to search tweets: ${error.message}`);
54+
throw new Error(formatTwitterError(error, 'searching tweets'));
5155
}
5256
throw error;
5357
}
5458
};
5559

5660
export const handleHashtagAnalytics: TwitterHandler<HashtagAnalyticsArgs> = async (
57-
client: TwitterClient,
61+
client: TwitterClient | null,
5862
{ hashtag, startTime, endTime }: HashtagAnalyticsArgs
5963
): Promise<HandlerResponse> => {
64+
if (!client) {
65+
return createMissingTwitterApiKeyResponse('hashtagAnalytics');
66+
}
6067
try {
6168
const query = `#${hashtag.replace(/^#/, '')}`;
6269
const searchResult = await client.v2.search(query, {
@@ -87,7 +94,7 @@ export const handleHashtagAnalytics: TwitterHandler<HashtagAnalyticsArgs> = asyn
8794
if (error.message.includes('400') && error.message.includes('Invalid Request')) {
8895
throw new Error(`Hashtag analytics requires Pro tier access ($5,000/month) or higher for search functionality. Current Basic tier ($200/month) does not include recent search API access. Consider upgrading at https://developer.x.com/en/portal/products/pro or use alternative analytics sources.`);
8996
}
90-
throw new Error(`Failed to get hashtag analytics: ${error.message}`);
97+
throw new Error(formatTwitterError(error, 'getting hashtag analytics'));
9198
}
9299
throw error;
93100
}

‎src/handlers/user.handlers.ts‎

Lines changed: 56 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { TwitterClient } from '../twitterClient.js';
1+
import { TwitterClient } from '../client/twitter.js';
22
import { UserV2, TTweetv2UserField } from 'twitter-api-v2';
33
import {
44
HandlerResponse,
@@ -7,6 +7,7 @@ import {
77
GetAuthenticatedUserArgs
88
} from '../types/handlers.js';
99
import { createResponse } from '../utils/response.js';
10+
import { createMissingTwitterApiKeyResponse, formatTwitterError } from '../utils/twitter-response.js';
1011

1112
interface GetUserInfoArgs extends UserHandlerArgs {
1213
fields?: TTweetv2UserField[];
@@ -28,29 +29,44 @@ interface GetFollowingArgs extends UserHandlerArgs {
2829
}
2930

3031
export const handleGetUserInfo: TwitterHandler<GetUserInfoArgs> = async (
31-
client: TwitterClient,
32+
client: TwitterClient | null,
3233
{ username, fields }: GetUserInfoArgs
3334
): Promise<HandlerResponse> => {
34-
const user = await client.v2.userByUsername(
35-
username,
36-
{
37-
'user.fields': fields || ['description', 'public_metrics', 'profile_image_url', 'verified'] as TTweetv2UserField[]
38-
}
39-
);
40-
41-
if (!user.data) {
42-
throw new Error(`User not found: ${username}`);
35+
if (!client) {
36+
return createMissingTwitterApiKeyResponse('getUserInfo');
4337
}
4438

45-
return createResponse(`User info: ${JSON.stringify(user.data, null, 2)}`);
39+
try {
40+
const user = await client.v2.userByUsername(
41+
username,
42+
{
43+
'user.fields': fields || ['description', 'public_metrics', 'profile_image_url', 'verified'] as TTweetv2UserField[]
44+
}
45+
);
46+
47+
if (!user.data) {
48+
throw new Error(`User not found: ${username}`);
49+
}
50+
51+
return createResponse(`User info: ${JSON.stringify(user.data, null, 2)}`);
52+
} catch (error) {
53+
if (error instanceof Error) {
54+
throw new Error(formatTwitterError(error, 'getting user info'));
55+
}
56+
throw error;
57+
}
4658
};
4759

4860
export const handleFollowUser: TwitterHandler<UserHandlerArgs> = async (
49-
client: TwitterClient,
61+
client: TwitterClient | null,
5062
{ username }: UserHandlerArgs
5163
): Promise<HandlerResponse> => {
64+
if (!client) {
65+
return createMissingTwitterApiKeyResponse('followUser');
66+
}
67+
5268
try {
53-
const userId = await client.v2.me().then(response => response.data.id);
69+
const userId = await client.v2.me().then((response: any) => response.data.id);
5470
const targetUser = await client.v2.userByUsername(username);
5571

5672
if (!targetUser.data) {
@@ -61,18 +77,22 @@ export const handleFollowUser: TwitterHandler<UserHandlerArgs> = async (
6177
return createResponse(`Successfully followed user: ${username}`);
6278
} catch (error) {
6379
if (error instanceof Error) {
64-
throw new Error(`Failed to follow user: ${error.message}`);
80+
throw new Error(formatTwitterError(error, 'following user'));
6581
}
6682
throw error;
6783
}
6884
};
6985

7086
export const handleUnfollowUser: TwitterHandler<UserHandlerArgs> = async (
71-
client: TwitterClient,
87+
client: TwitterClient | null,
7288
{ username }: UserHandlerArgs
7389
): Promise<HandlerResponse> => {
90+
if (!client) {
91+
return createMissingTwitterApiKeyResponse('unfollowUser');
92+
}
93+
7494
try {
75-
const userId = await client.v2.me().then(response => response.data.id);
95+
const userId = await client.v2.me().then((response: any) => response.data.id);
7696
const targetUser = await client.v2.userByUsername(username);
7797

7898
if (!targetUser.data) {
@@ -83,16 +103,20 @@ export const handleUnfollowUser: TwitterHandler<UserHandlerArgs> = async (
83103
return createResponse(`Successfully unfollowed user: ${username}`);
84104
} catch (error) {
85105
if (error instanceof Error) {
86-
throw new Error(`Failed to unfollow user: ${error.message}`);
106+
throw new Error(formatTwitterError(error, 'unfollowing user'));
87107
}
88108
throw error;
89109
}
90110
};
91111

92112
export const handleGetFollowers: TwitterHandler<GetFollowersArgs> = async (
93-
client: TwitterClient,
113+
client: TwitterClient | null,
94114
{ username, maxResults, userFields }: GetFollowersArgs
95115
): Promise<HandlerResponse> => {
116+
if (!client) {
117+
return createMissingTwitterApiKeyResponse('getFollowers');
118+
}
119+
96120
try {
97121
const user = await client.v2.userByUsername(username);
98122
if (!user.data) {
@@ -119,16 +143,20 @@ export const handleGetFollowers: TwitterHandler<GetFollowersArgs> = async (
119143
if (error.message.includes('403')) {
120144
throw new Error(`Get followers functionality requires elevated permissions. This endpoint may require Pro tier access ($5,000/month) or special permission approval from X. Current Basic tier ($200/month) has limited access to follower data for privacy reasons. Contact X Developer Support or consider upgrading at https://developer.x.com/en/portal/products/pro`);
121145
}
122-
throw new Error(`Failed to get followers: ${error.message}`);
146+
throw new Error(formatTwitterError(error, 'getting followers'));
123147
}
124148
throw error;
125149
}
126150
};
127151

128152
export const handleGetFollowing: TwitterHandler<GetFollowingArgs> = async (
129-
client: TwitterClient,
153+
client: TwitterClient | null,
130154
{ username, maxResults, userFields }: GetFollowingArgs
131155
): Promise<HandlerResponse> => {
156+
if (!client) {
157+
return createMissingTwitterApiKeyResponse('getFollowing');
158+
}
159+
132160
try {
133161
const user = await client.v2.userByUsername(username);
134162
if (!user.data) {
@@ -155,7 +183,7 @@ export const handleGetFollowing: TwitterHandler<GetFollowingArgs> = async (
155183
if (error.message.includes('403')) {
156184
throw new Error(`Get following functionality requires elevated permissions. This endpoint may require Pro tier access ($5,000/month) or special permission approval from X. Current Basic tier ($200/month) has limited access to following data for privacy reasons. Contact X Developer Support or consider upgrading at https://developer.x.com/en/portal/products/pro`);
157185
}
158-
throw new Error(`Failed to get following: ${error.message}`);
186+
throw new Error(formatTwitterError(error, 'getting following'));
159187
}
160188
throw error;
161189
}
@@ -165,9 +193,13 @@ export const handleGetFollowing: TwitterHandler<GetFollowingArgs> = async (
165193
* Get the authenticated user's own profile information
166194
*/
167195
export const handleGetAuthenticatedUser: TwitterHandler<GetAuthenticatedUserArgs> = async (
168-
client: TwitterClient,
196+
client: TwitterClient | null,
169197
{ userFields }: GetAuthenticatedUserArgs
170198
): Promise<HandlerResponse> => {
199+
if (!client) {
200+
return createMissingTwitterApiKeyResponse('getAuthenticatedUser');
201+
}
202+
171203
try {
172204
const me = await client.v2.me({
173205
'user.fields': (userFields as TTweetv2UserField[]) || ['id', 'username', 'name', 'description', 'public_metrics', 'verified', 'profile_image_url', 'created_at'] as TTweetv2UserField[]
@@ -186,7 +218,7 @@ export const handleGetAuthenticatedUser: TwitterHandler<GetAuthenticatedUserArgs
186218
if (error.message.includes('429')) {
187219
throw new Error(`Rate limit exceeded. Please wait before making another request.`);
188220
}
189-
throw new Error(`Failed to get authenticated user: ${error.message}`);
221+
throw new Error(formatTwitterError(error, 'getting authenticated user'));
190222
}
191223
throw error;
192224
}

0 commit comments

Comments
 (0)