Skip to content

[WIP] Fix tlog report access error#6

Merged
waymbers merged 1 commit intomainfrom
copilot/fix-tlog-report-error-again
Mar 16, 2026
Merged

[WIP] Fix tlog report access error#6
waymbers merged 1 commit intomainfrom
copilot/fix-tlog-report-error-again

Conversation

Copy link
Contributor

Copilot AI commented Mar 3, 2026

  • Add MAX_CRITICAL_EVENTS (50), MAX_MODE_CHANGES (20), and UNKNOWN_SEVERITY (999) constants in js/ai.js
  • In buildSummary, sort critical events by severity (most critical first: ARM/DISARM, then Emergency→Warning) and truncate to MAX_CRITICAL_EVENTS
  • In buildSummary, truncate modeChanges to MAX_MODE_CHANGES
  • Address code review comments (named constant for sentinel, comment on _sev strip)
  • Pass CodeQL security scan (0 alerts)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@waymbers waymbers merged commit 93a8f9a into main Mar 16, 2026
1 check failed
@waymbers waymbers deleted the copilot/fix-tlog-report-error-again branch March 16, 2026 00:34
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