fix(twitter): refresh TweetDetail query ids
This commit is contained in:
@@ -2,6 +2,6 @@
|
|||||||
"CreateTweet": "2BA2DIWTRVxHgKc4Xh2uKQ",
|
"CreateTweet": "2BA2DIWTRVxHgKc4Xh2uKQ",
|
||||||
"CreateRetweet": "Uf3io9zVp1DsYxrmL5FJ7g",
|
"CreateRetweet": "Uf3io9zVp1DsYxrmL5FJ7g",
|
||||||
"FavoriteTweet": "kuFUYP9eV1FPoEy4N-pi7w",
|
"FavoriteTweet": "kuFUYP9eV1FPoEy4N-pi7w",
|
||||||
"TweetDetail": "aFvUsJm2c-oDkJV75blV6g",
|
"TweetDetail": "97JF30KziU00483E_8elBA",
|
||||||
"SearchTimeline": "M1jEez78PEfVfbQLvlWMvQ"
|
"SearchTimeline": "5h0kNbk3ii97rmfY6CdgAA"
|
||||||
}
|
}
|
||||||
|
|||||||
+58
-28
@@ -14,7 +14,7 @@ const FALLBACK_QUERY_IDS = {
|
|||||||
CreateTweet: 'TAJw1rBsjAtdNgTdlo2oeg',
|
CreateTweet: 'TAJw1rBsjAtdNgTdlo2oeg',
|
||||||
CreateRetweet: 'ojPdsZsimiJrUGLR1sjUtA',
|
CreateRetweet: 'ojPdsZsimiJrUGLR1sjUtA',
|
||||||
FavoriteTweet: 'lI07N6Otwv1PhnEgXILM7A',
|
FavoriteTweet: 'lI07N6Otwv1PhnEgXILM7A',
|
||||||
TweetDetail: 'nBS-WpgA6ZG0CyNHD517JQ',
|
TweetDetail: '97JF30KziU00483E_8elBA',
|
||||||
SearchTimeline: 'M1jEez78PEfVfbQLvlWMvQ',
|
SearchTimeline: 'M1jEez78PEfVfbQLvlWMvQ',
|
||||||
UserArticlesTweets: '8zBy9h4L90aDL02RsBcCFg',
|
UserArticlesTweets: '8zBy9h4L90aDL02RsBcCFg',
|
||||||
} as const;
|
} as const;
|
||||||
@@ -26,6 +26,10 @@ const QUERY_IDS: Record<OperationName, string> = {
|
|||||||
...(queryIds as Partial<Record<OperationName, string>>),
|
...(queryIds as Partial<Record<OperationName, string>>),
|
||||||
};
|
};
|
||||||
|
|
||||||
|
const TWEET_DETAIL_QUERY_IDS = Array.from(
|
||||||
|
new Set([QUERY_IDS.TweetDetail, '97JF30KziU00483E_8elBA', 'aFvUsJm2c-oDkJV75blV6g']),
|
||||||
|
);
|
||||||
|
|
||||||
const SEARCH_TIMELINE_QUERY_IDS = Array.from(
|
const SEARCH_TIMELINE_QUERY_IDS = Array.from(
|
||||||
new Set([QUERY_IDS.SearchTimeline, 'M1jEez78PEfVfbQLvlWMvQ', '5h0kNbk3ii97rmfY6CdgAA', 'Tp1sewRU1AsZpBWhqCZicQ']),
|
new Set([QUERY_IDS.SearchTimeline, 'M1jEez78PEfVfbQLvlWMvQ', '5h0kNbk3ii97rmfY6CdgAA', 'Tp1sewRU1AsZpBWhqCZicQ']),
|
||||||
);
|
);
|
||||||
@@ -636,6 +640,23 @@ export class TwitterClient {
|
|||||||
};
|
};
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private buildTweetDetailFeatures(): Record<string, boolean> {
|
||||||
|
return {
|
||||||
|
...this.buildArticleFeatures(),
|
||||||
|
responsive_web_graphql_exclude_directive_enabled: true,
|
||||||
|
communities_web_enable_tweet_community_results_fetch: true,
|
||||||
|
responsive_web_twitter_article_plain_text_enabled: true,
|
||||||
|
responsive_web_twitter_article_seed_tweet_detail_enabled: true,
|
||||||
|
responsive_web_twitter_article_seed_tweet_summary_enabled: true,
|
||||||
|
longform_notetweets_rich_text_read_enabled: true,
|
||||||
|
longform_notetweets_inline_media_enabled: true,
|
||||||
|
responsive_web_edit_tweet_api_enabled: true,
|
||||||
|
tweet_awards_web_tipping_enabled: false,
|
||||||
|
creator_subscriptions_quote_tweet_preview_enabled: false,
|
||||||
|
verified_phone_label_enabled: false,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
private buildArticleFieldToggles(): Record<string, boolean> {
|
private buildArticleFieldToggles(): Record<string, boolean> {
|
||||||
return {
|
return {
|
||||||
withPayments: false,
|
withPayments: false,
|
||||||
@@ -798,33 +819,19 @@ export class TwitterClient {
|
|||||||
};
|
};
|
||||||
|
|
||||||
const features = {
|
const features = {
|
||||||
rweb_tipjar_consumption_enabled: true,
|
...this.buildTweetDetailFeatures(),
|
||||||
responsive_web_graphql_exclude_directive_enabled: true,
|
|
||||||
verified_phone_label_enabled: false,
|
|
||||||
creator_subscriptions_tweet_preview_api_enabled: true,
|
|
||||||
responsive_web_graphql_timeline_navigation_enabled: true,
|
|
||||||
responsive_web_graphql_skip_user_profile_image_extensions_enabled: false,
|
|
||||||
communities_web_enable_tweet_community_results_fetch: true,
|
|
||||||
c9s_tweet_anatomy_moderator_badge_enabled: true,
|
|
||||||
articles_preview_enabled: true,
|
articles_preview_enabled: true,
|
||||||
articles_rest_api_enabled: true,
|
articles_rest_api_enabled: true,
|
||||||
responsive_web_edit_tweet_api_enabled: true,
|
responsive_web_graphql_skip_user_profile_image_extensions_enabled: false,
|
||||||
|
creator_subscriptions_tweet_preview_api_enabled: true,
|
||||||
graphql_is_translatable_rweb_tweet_is_translatable_enabled: true,
|
graphql_is_translatable_rweb_tweet_is_translatable_enabled: true,
|
||||||
view_counts_everywhere_api_enabled: true,
|
view_counts_everywhere_api_enabled: true,
|
||||||
longform_notetweets_consumption_enabled: true,
|
longform_notetweets_consumption_enabled: true,
|
||||||
responsive_web_twitter_article_tweet_consumption_enabled: true,
|
responsive_web_twitter_article_tweet_consumption_enabled: true,
|
||||||
responsive_web_twitter_article_plain_text_enabled: true,
|
|
||||||
responsive_web_twitter_article_seed_tweet_detail_enabled: true,
|
|
||||||
responsive_web_twitter_article_seed_tweet_summary_enabled: true,
|
|
||||||
tweet_awards_web_tipping_enabled: false,
|
|
||||||
creator_subscriptions_quote_tweet_preview_enabled: false,
|
|
||||||
freedom_of_speech_not_reach_fetch_enabled: true,
|
freedom_of_speech_not_reach_fetch_enabled: true,
|
||||||
standardized_nudges_misinfo: true,
|
standardized_nudges_misinfo: true,
|
||||||
tweet_with_visibility_results_prefer_gql_limited_actions_policy_enabled: true,
|
tweet_with_visibility_results_prefer_gql_limited_actions_policy_enabled: true,
|
||||||
rweb_video_timestamps_enabled: true,
|
rweb_video_timestamps_enabled: true,
|
||||||
longform_notetweets_rich_text_read_enabled: true,
|
|
||||||
longform_notetweets_inline_media_enabled: true,
|
|
||||||
responsive_web_enhance_cards_enabled: false,
|
|
||||||
};
|
};
|
||||||
|
|
||||||
const params = new URLSearchParams({
|
const params = new URLSearchParams({
|
||||||
@@ -832,17 +839,11 @@ export class TwitterClient {
|
|||||||
features: JSON.stringify(features),
|
features: JSON.stringify(features),
|
||||||
});
|
});
|
||||||
|
|
||||||
const url = `${TWITTER_API_BASE}/${QUERY_IDS.TweetDetail}/TweetDetail?${params}`;
|
|
||||||
|
|
||||||
try {
|
try {
|
||||||
const response = await this.fetchWithTimeout(url, {
|
const parseResponse = async (response: Response) => {
|
||||||
method: 'GET',
|
|
||||||
headers: this.getHeaders(),
|
|
||||||
});
|
|
||||||
|
|
||||||
if (!response.ok) {
|
if (!response.ok) {
|
||||||
const text = await response.text();
|
const text = await response.text();
|
||||||
return { success: false, error: `HTTP ${response.status}: ${text.slice(0, 200)}` };
|
return { success: false as const, error: `HTTP ${response.status}: ${text.slice(0, 200)}` };
|
||||||
}
|
}
|
||||||
|
|
||||||
const data = (await response.json()) as {
|
const data = (await response.json()) as {
|
||||||
@@ -866,10 +867,39 @@ export class TwitterClient {
|
|||||||
};
|
};
|
||||||
|
|
||||||
if (data.errors && data.errors.length > 0) {
|
if (data.errors && data.errors.length > 0) {
|
||||||
return { success: false, error: data.errors.map((e) => e.message).join(', ') };
|
return { success: false as const, error: data.errors.map((e) => e.message).join(', ') };
|
||||||
}
|
}
|
||||||
|
|
||||||
return { success: true, data: data.data ?? {} };
|
return { success: true as const, data: data.data ?? {} };
|
||||||
|
};
|
||||||
|
|
||||||
|
let lastError: string | undefined;
|
||||||
|
|
||||||
|
for (const queryId of TWEET_DETAIL_QUERY_IDS) {
|
||||||
|
const url = `${TWITTER_API_BASE}/${queryId}/TweetDetail?${params}`;
|
||||||
|
const response = await this.fetchWithTimeout(url, {
|
||||||
|
method: 'GET',
|
||||||
|
headers: this.getHeaders(),
|
||||||
|
});
|
||||||
|
|
||||||
|
if (response.status !== 404) {
|
||||||
|
return await parseResponse(response);
|
||||||
|
}
|
||||||
|
|
||||||
|
const postResponse = await this.fetchWithTimeout(`${TWITTER_API_BASE}/${queryId}/TweetDetail`, {
|
||||||
|
method: 'POST',
|
||||||
|
headers: this.getHeaders(),
|
||||||
|
body: JSON.stringify({ variables, features, queryId }),
|
||||||
|
});
|
||||||
|
|
||||||
|
if (postResponse.status !== 404) {
|
||||||
|
return await parseResponse(postResponse);
|
||||||
|
}
|
||||||
|
|
||||||
|
lastError = 'HTTP 404';
|
||||||
|
}
|
||||||
|
|
||||||
|
return { success: false, error: lastError ?? 'Unknown error fetching tweet detail' };
|
||||||
} catch (error) {
|
} catch (error) {
|
||||||
return { success: false, error: error instanceof Error ? error.message : String(error) };
|
return { success: false, error: error instanceof Error ? error.message : String(error) };
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -283,7 +283,7 @@ describe('TwitterClient', () => {
|
|||||||
});
|
});
|
||||||
|
|
||||||
it('should report HTTP errors from getTweet', async () => {
|
it('should report HTTP errors from getTweet', async () => {
|
||||||
mockFetch.mockResolvedValueOnce({
|
mockFetch.mockResolvedValue({
|
||||||
ok: false,
|
ok: false,
|
||||||
status: 404,
|
status: 404,
|
||||||
text: async () => 'Not Found',
|
text: async () => 'Not Found',
|
||||||
@@ -474,6 +474,38 @@ describe('TwitterClient', () => {
|
|||||||
expect(result.success).toBe(true);
|
expect(result.success).toBe(true);
|
||||||
expect(result.tweet?.text).toBe('Long form note content.');
|
expect(result.tweet?.text).toBe('Long form note content.');
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('retries TweetDetail query id on 404', async () => {
|
||||||
|
const payload = {
|
||||||
|
data: {
|
||||||
|
tweetResult: {
|
||||||
|
result: {
|
||||||
|
rest_id: '1',
|
||||||
|
legacy: {
|
||||||
|
full_text: 'hello',
|
||||||
|
created_at: '2024-01-01T00:00:00Z',
|
||||||
|
reply_count: 0,
|
||||||
|
retweet_count: 0,
|
||||||
|
favorite_count: 0,
|
||||||
|
},
|
||||||
|
core: { user_results: { result: { legacy: { screen_name: 'root', name: 'Root' } } } },
|
||||||
|
},
|
||||||
|
},
|
||||||
|
},
|
||||||
|
};
|
||||||
|
|
||||||
|
mockFetch
|
||||||
|
.mockResolvedValueOnce({ ok: false, status: 404, text: async () => '' })
|
||||||
|
.mockResolvedValueOnce({ ok: false, status: 404, text: async () => '' })
|
||||||
|
.mockResolvedValueOnce({ ok: true, status: 200, json: async () => payload });
|
||||||
|
|
||||||
|
const client = new TwitterClient({ cookies: validCookies });
|
||||||
|
const result = await client.getTweet('1');
|
||||||
|
|
||||||
|
expect(result.success).toBe(true);
|
||||||
|
expect(result.tweet?.id).toBe('1');
|
||||||
|
expect(mockFetch).toHaveBeenCalledTimes(3);
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
describe('getCurrentUser', () => {
|
describe('getCurrentUser', () => {
|
||||||
|
|||||||
Reference in New Issue
Block a user