Skip to content

feat: major bug fixes and formatting improvement#5

Merged
bisug merged 2 commits into
masterfrom
dev
Jul 11, 2026
Merged

feat: major bug fixes and formatting improvement#5
bisug merged 2 commits into
masterfrom
dev

Conversation

@bisug

@bisug bisug commented Jul 11, 2026

Copy link
Copy Markdown
Owner

No description provided.

bisug and others added 2 commits July 11, 2026 07:36
- Cache GitHub clients per access token to stop rebuilding the oauth2 transport on every API call
- Deduplicate webhook deliveries via X-GitHub-Delivery; enforce MaxBytesReader before signature validation
- Replace stringly-typed "unauthorized" comparisons with errors.Is(ErrUnauthorized)
- Make the OAuth callback asynchronous: return 200 fast, run GitHub exchange/DB write in a goroutine
- Add /healthz endpoint and a background TTL cache sweep to stop unbounded cache growth
- Debounce per-chat upserts; escape Telegram usernames in the connect success message
- Deduplicate repo pagination and compact-button helpers into the ui package
- Add a UpsertUser BSON-shape unit test (validates _id lives under $setOnInsert)

Co-Authored-By: Claude <noreply@anthropic.com>
FormatTextWithMarkdown protected links but left the anchor text unescaped.
An anchor containing MarkdownV2 special chars (e.g. "release.v2") made
Telegram reject the message, so the bot fell back to plain text and dropped
all formatting. Link anchors are now escaped; code spans stay verbatim.

Also cap the formatted message at Telegram's 4096-rune limit on the primary
send path (only the plain-text fallback truncated before, so long events
could be lost outright).

Co-Authored-By: Claude <noreply@anthropic.com>
@bisug bisug merged commit a31d6d3 into master Jul 11, 2026
1 check failed
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.

1 participant