Skip to content

Fix newsletter not following distilled prompts and Instead giving Hacker News related updates #143

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
satwikkansal merged 5 commits into from
Jan 12, 2026
Merged

Fix newsletter not following distilled prompts and Instead giving Hacker News related updates #143

satwikkansal merged 5 commits into from
Jan 12, 2026

Conversation

@BhaveshKukreja29
Copy link
Collaborator

@BhaveshKukreja29 BhaveshKukreja29 commented Jan 9, 2026 •
edited
Loading

3 fixes:

  1. We were not receiving the actual newsletter after the sample.
  2. The LLM was not following the user specific needs like geography and was only following the generic prompt.
  3. Fix DB engine to be global and not create a new one on every request.

Checklist

  • I have created an issue for this change (not mandatory for small changes)
  • My changes are to-the-point
  • Code is styled as the rest of the codebase and linting passes
  • I have tested my changes locally and they work as expected
  • I have updated the documentation (if needed)
  • I have reviewed the code myself once and I don't see any issues

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 10, 2026 •
edited
Loading

Deploying mxtoai-backend with  Cloudflare Pages  Cloudflare Pages

Latest commit: a518e78
Status: ✅  Deploy successful!
Preview URL: https://57774538.mxtoai-backend.pages.dev
Branch Preview URL: https://news-endpoint-fix.mxtoai-backend.pages.dev

View logs

satwikkansal
satwikkansal reviewed Jan 10, 2026
View reviewed changes
satwikkansal
satwikkansal previously approved these changes Jan 10, 2026
View reviewed changes
Copy link
Collaborator

@satwikkansal satwikkansal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just one comment, looks good otherwise.

@satwikkansal satwikkansal merged commit 5eeaa6b into master Jan 12, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@satwikkansal satwikkansal satwikkansal approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@BhaveshKukreja29 @satwikkansal