Skip to content

feat(messaging): add 2000-char limit warning to chat input#331

Merged
cbmahoney merged 1 commit into
mainfrom
feature/chat-message-char-limit-warning
Mar 29, 2026
Merged

feat(messaging): add 2000-char limit warning to chat input#331
cbmahoney merged 1 commit into
mainfrom
feature/chat-message-char-limit-warning

Conversation

@pkouadio001

Copy link
Copy Markdown
Contributor

o Enforced a hard limit of 2000 characters on chat input.
o Added a live character count indicator when users approach the limit (starting at 1800).
o Applied warning colors as the user nears/reaches the limit:
o neutral near threshold
o warning close to limit
o error at the limit
o Disabled the send action when input is invalid (over limit / blocked).
o Added a guard in send logic so over-limit text cannot be sent even if triggered programmatically.

This prevents oversized messages, gives clear user feedback before reaching the limit, and avoids failed/invalid message submissions.

@pkouadio001 pkouadio001 requested review from cbmahoney and Copilot and removed request for Copilot March 28, 2026 08:06
@pkouadio001 pkouadio001 assigned parthavd and unassigned parthavd Mar 28, 2026
@github-actions

Copy link
Copy Markdown

CareConnect Quality Gate Report

BLOCKED — One or more required checks failed. Fix the issues below before merging.

CareConnect — Security & Quality Analysis Report

Report Header

Field Value
Generated (UTC) 2026-03-28 08:13:55 UTC
Pipeline Run #628
Trigger pull_request
Scan Root .

All timestamps are reported in Coordinated Universal Time (UTC).

Pull Request

Field Value
PR Number #331
PR Author @pkouadio001
Source Branch feature/chat-message-char-limit-warning
Target Branch main

Commit Details

Field Value
Commit SHA a8779fe (full)

Legend

Status Meaning
SUCCESS Tool ran and found no violations
FAILURE Tool found one or more violations
DISABLED Tool is not yet configured
Enforced Violations from this tool will block the merge
Advisory Violations are reported but will not block the merge

Tool Results Summary

Tool Category Status Role Findings
trufflehog Secrets Scan FAILURE Enforced 44 finding(s)
gitleaks Secrets Scan FAILURE Enforced 20 finding(s)
flutter_analyze SAST — Flutter FAILURE Enforced 1481 finding(s)
checkstyle SAST — Java FAILURE Enforced 29571 finding(s)
pmd SAST — Java FAILURE Enforced 11251 finding(s)
spotbugs SAST — Java FAILURE Enforced 710 finding(s)
semgrep SAST — Multi FAILURE Enforced 53 finding(s)
pylint SAST — Python FAILURE Enforced 180 finding(s)
bandit SAST — Python FAILURE Enforced 39 finding(s)
htmlhint SAST — Web FAILURE Enforced 623 finding(s)
stylelint SAST — Web FAILURE Enforced 15 finding(s)
dependency_check SCA — Multi SUCCESS Enforced
trivy SCA — Container FAILURE Enforced 39 finding(s)

Full artifact bundle available in the workflow run artifacts.

@cbmahoney cbmahoney merged commit 17e843c into main Mar 29, 2026
4 of 5 checks passed
@willywil548 willywil548 deleted the feature/chat-message-char-limit-warning branch June 6, 2026 17:40
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