Skip to content

Conversation

@koutst
Copy link
Contributor

@koutst koutst commented Dec 22, 2025

  • Extracts X metrics mapping logic into mapper.
  • Stops storing posts in external_metadata.
  • Adds XPosts table and starts storing posts in there.

@koutst koutst force-pushed the start-storing-x-posts branch from a719885 to c181594 Compare December 29, 2025 15:15
@koutst koutst marked this pull request as ready for review December 29, 2025 15:22
@koutst koutst requested a review from a team as a code owner December 29, 2025 15:22
@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@codecov
Copy link

codecov bot commented Dec 29, 2025

Codecov Report

❌ Patch coverage is 92.55319% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.66%. Comparing base (d194f9d) to head (edb4a73).

Files with missing lines Patch % Lines
src/paper/related_models/x_post_model.py 88.00% 3 Missing ⚠️
src/paper/ingestion/clients/enrichment/x.py 88.23% 2 Missing ⚠️
src/paper/ingestion/mappers/enrichment/x.py 94.11% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2979      +/-   ##
==========================================
+ Coverage   75.64%   75.66%   +0.02%     
==========================================
  Files         533      535       +2     
  Lines       29270    29320      +50     
==========================================
+ Hits        22142    22186      +44     
- Misses       7128     7134       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@koutst koutst changed the title Start storing x posts Start storing X posts in own table Dec 29, 2025
@koutst koutst changed the title Start storing X posts in own table Start storing X posts in separate table Dec 29, 2025
@koutst koutst marked this pull request as draft January 2, 2026 02:22
@koutst koutst marked this pull request as ready for review January 5, 2026 15:07
@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@koutst koutst force-pushed the start-storing-x-posts branch from f552052 to edb4a73 Compare January 5, 2026 15:08
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 5, 2026

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.

2 participants