Skip to content

fix: sort App Store sections above news - #22

Merged
chapati23 merged 2 commits into
mainfrom
fix/app-store-before-news
Apr 17, 2026
Merged

fix: sort App Store sections above news#22
chapati23 merged 2 commits into
mainfrom
fix/app-store-before-news

Conversation

@gisk0

@gisk0 gisk0 commented Apr 17, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • sort briefing sections using source priority directly
  • preserve correct ordering for sources that emit multiple titled sections
  • add a regression test covering App Store sections above Crypto News
  • override vulnerable transitive basic-ftp so repo security checks pass

Testing

  • bun run typecheck
  • trunk check --fix src/orchestrator.ts tests/orchestrator.test.ts package.json bun.lock
  • bun test tests/orchestrator.test.ts tests/e2e/briefing.test.ts
  • bun test

@chapati23
chapati23 merged commit f6913d7 into main Apr 17, 2026
2 checks passed
@chapati23
chapati23 deleted the fix/app-store-before-news branch April 17, 2026 08:45
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