Skip to content
This repository was archived by the owner on Feb 26, 2026. It is now read-only.

REST API controller for the scoring service#246

Closed
metlos wants to merge 10 commits intometlos/scoring-servicefrom
metlos/scoring-controller
Closed

REST API controller for the scoring service#246
metlos wants to merge 10 commits intometlos/scoring-servicefrom
metlos/scoring-controller

Conversation

@metlos
Copy link
Collaborator

@metlos metlos commented Jan 28, 2026

This PR stacks on top of #245 and introduces a REST API for the scoring service.

Note that websockets are not supported at the moment intentionally so that we don't overcomplicate the impl from the very start.

Part of the PR stack:

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 28, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch metlos/scoring-controller

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@metlos metlos force-pushed the metlos/scoring-service branch from b36d6fa to fc8a9a9 Compare January 29, 2026 12:55
@metlos metlos force-pushed the metlos/scoring-controller branch from 1bb3267 to 1a5c51a Compare January 29, 2026 12:55
@metlos metlos force-pushed the metlos/scoring-service branch from fc8a9a9 to 8a27218 Compare January 29, 2026 12:58
@metlos metlos force-pushed the metlos/scoring-controller branch from 1a5c51a to 4ba5a4c Compare January 29, 2026 12:58
@metlos metlos force-pushed the metlos/scoring-service branch from 8a27218 to a0cd1d1 Compare January 30, 2026 19:26
@metlos metlos force-pushed the metlos/scoring-controller branch 2 times, most recently from 251599b to 12e956c Compare January 30, 2026 22:13
@metlos metlos force-pushed the metlos/scoring-service branch from a0cd1d1 to 22adfda Compare January 30, 2026 22:13
@metlos metlos force-pushed the metlos/scoring-controller branch from 12e956c to cded324 Compare February 2, 2026 12:26
@metlos metlos force-pushed the metlos/scoring-service branch 2 times, most recently from 10f7fe3 to 8acd64b Compare February 2, 2026 13:28
@metlos metlos force-pushed the metlos/scoring-controller branch from cded324 to 487717a Compare February 2, 2026 13:28
@metlos metlos force-pushed the metlos/scoring-service branch from 8acd64b to cb61021 Compare February 2, 2026 20:11
@metlos metlos force-pushed the metlos/scoring-controller branch from 487717a to 9a62aaa Compare February 2, 2026 20:11
@metlos metlos force-pushed the metlos/scoring-controller branch from 9a62aaa to 5774fb1 Compare February 3, 2026 10:28
@metlos metlos force-pushed the metlos/scoring-service branch from cb61021 to 1a0b260 Compare February 3, 2026 10:28
metlos and others added 7 commits February 3, 2026 12:52
* Implement expandable executive summary in AlertListItem component

- Added functionality to display an expandable executive summary for alerts in the AlertListItem component.
- Introduced state management for summary expansion and collapse.
- Updated the backend to include final_analysis_summary in session data for quick access.
- Enhanced the SessionOverview model to support the new summary field.

This improves user experience by allowing quick previews of session summaries without needing to fetch full session details.

Assisted-by: Cursor

* Dashboard

Assisted-by: Cursor

* Tests

Assisted-by: Cursor
@metlos metlos marked this pull request as draft February 13, 2026 15:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants