Skip to content

Enable user setting chat auto scroll#393

Merged
LennartSchmidtKern merged 6 commits intodevfrom
use-auto-scroll
Mar 16, 2026
Merged

Enable user setting chat auto scroll#393
LennartSchmidtKern merged 6 commits intodevfrom
use-auto-scroll

Conversation

@LennartSchmidtKern
Copy link
Copy Markdown
Contributor

@LennartSchmidtKern LennartSchmidtKern commented Mar 6, 2026

Copy link
Copy Markdown
Member

@JWittmeyer JWittmeyer left a comment

Choose a reason for hiding this comment

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

Automated review (review-bot)

@JWittmeyer
Copy link
Copy Markdown
Member

🤖 Review Bot Summary

Risk Level: CRITICAL

Count
🔴 Critical 0
🟡 Suggestion 4
🔵 Note 3
❓ Question 0
Total 7

Per-Repo Breakdown

Repo 🔴 🟡 🔵 Total PR
✅ refinery-submodule-model 0 1 1 0 2 #236
✅ refinery-gateway 0 1 0 0 1 #393
✅ cognition-ui 0 2 1 0 3 #210

🔗 Cross-Repo Findings

  • 🔵 [code-kern-ai/refinery-gateway, code-kern-ai/cognition-ui, code-kern-ai/refinery-submodule-model] The provided source snippets do not contain 'use_new_cognition_ui'. Propagation to cognition-ui or refinery-submodule-model cannot be verified from the given evidence; the field may exist elsewhere (e.g. config or feature flags) and was likely grouped with Alembic metadata by mistake.

Inline comments with details are posted on each PR above.

@JWittmeyer
Copy link
Copy Markdown
Member

JWittmeyer commented Mar 13, 2026

Add Note for better default handling for review / agentic work to cursorfiles

  ## Alembic Migrations
  - Column defaults belong in the ORM model (submodule), not as `server_default` in migrations.
    Do not suggest adding `server_default` unless there is a specific need for database-level defaults.
  • resolved
  • discussed

@LennartSchmidtKern LennartSchmidtKern merged commit 0c09bb4 into dev Mar 16, 2026
1 check passed
@JWittmeyer JWittmeyer deleted the use-auto-scroll branch March 16, 2026 14:19
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