Skip to content

Conversation

@ballPointPenguin
Copy link
Member

@ballPointPenguin ballPointPenguin commented Aug 8, 2025

DB MIGRATION REQUIRED

tevko and others added 3 commits August 8, 2025 13:52
# Configurable Topic-Informed Comment Routing for Alpha Client (and more)

## Client Participation Alpha

- centralize JWT handling
- convert net.js to typescript
- fetch NextComment upon hydration for personalization (ie don't show voted on comment)

## Server

- dedicated NextComment module
- checks for TOPICAL_COMMENT_RATIO (float)
- if ^ is set, blends NextComment selection between Topic-Informed comment pool and traditional prioritization
- graceful handling of empty-ish pca objects when there are comments but zero votes
- include DynamoDB helpers in integration tests
- integration tests for NextComment routing

## E2E

- test comment csv upload
- test report generation and viewing for "empty-ish" pca with comments but zero votes

## Client Report

- graceful handling of empty-ish pca objects (has comments, zero votes)

## Math

- enhanced Postgres connection pooling
- reduced risk of exhausting postgres connections and crashing
- reduced risk of "zombie" postgres connections that don't get cleaned up (mainly in test runs)

## Postgres

- *Migration* 000012 creates topic_agenda_selections table
- Improved stability in test & dev docker setups

## Misc

- consolidated Authentication docs
- improved testing stability overall
- update example.env files
Bumps [astro](https://github.com/withastro/astro/tree/HEAD/packages/astro) from 5.11.0 to 5.12.8.
- [Release notes](https://github.com/withastro/astro/releases)
- [Changelog](https://github.com/withastro/astro/blob/main/packages/astro/CHANGELOG.md)
- [Commits](https://github.com/withastro/astro/commits/astro@5.12.8/packages/astro)

---
updated-dependencies:
- dependency-name: astro
  dependency-version: 5.12.8
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@tevko tevko merged commit 1995694 into stable Aug 9, 2025
4 of 9 checks passed
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.

3 participants