REPOSITORY / ScuroNeko/Laniakea
Compare commits
Compare commits
7 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
5d3199dc21
|
||
|
|
158625c220
|
||
|
|
7901fb659e
|
||
|
|
eda635e72c
|
||
|
|
f0da64c7af
|
||
|
|
3861746a3e
|
||
|
|
401173714e
|
+1
-1
@@ -1,2 +1,2 @@
|
|||||||
.idea/
|
.idea/
|
||||||
test/
|
test/
|
||||||
|
|||||||
@@ -0,0 +1,102 @@
|
|||||||
|
# AGENTS.md
|
||||||
|
|
||||||
|
## Purpose
|
||||||
|
This repository uses Codex for full-project Go code review, not diff-only review.
|
||||||
|
|
||||||
|
When asked to review code, inspect the entire repository and use repository-wide context. Do not limit analysis to the latest commit, pull request diff, or recently changed files.
|
||||||
|
|
||||||
|
## Review priorities
|
||||||
|
Review the codebase with focus on:
|
||||||
|
- correctness and reliability;
|
||||||
|
- maintainability and architecture;
|
||||||
|
- idiomatic Go;
|
||||||
|
- testability;
|
||||||
|
- performance where justified by code evidence;
|
||||||
|
- security;
|
||||||
|
- godoc quality.
|
||||||
|
|
||||||
|
## Scope rules
|
||||||
|
- Always review the whole repository unless the prompt explicitly narrows scope.
|
||||||
|
- Check cross-package interactions, public APIs, package boundaries, and shared patterns.
|
||||||
|
- Prefer concrete fixes over generic advice.
|
||||||
|
- When feasible, make small, high-confidence improvements directly.
|
||||||
|
- When uncertain, state confidence level and evidence.
|
||||||
|
|
||||||
|
## Go review expectations
|
||||||
|
Check for:
|
||||||
|
- bugs, fragile logic, invalid assumptions, nil handling issues, resource leaks;
|
||||||
|
- poor error handling;
|
||||||
|
- misuse of context, cancellation, timeouts, retries, and cleanup;
|
||||||
|
- race risks, deadlocks, blocking hazards, unsafe shared state;
|
||||||
|
- non-idiomatic naming, APIs, interfaces, package structure, and error patterns;
|
||||||
|
- unnecessary complexity, duplication, or weak abstractions;
|
||||||
|
- obvious performance problems supported by the code;
|
||||||
|
- security risks such as unsafe input handling, secret leakage, insecure logging, injection risks, and risky file or network operations.
|
||||||
|
|
||||||
|
## Godoc rules
|
||||||
|
Review comments for all declarations.
|
||||||
|
|
||||||
|
### Exported declarations
|
||||||
|
Exported types, funcs, methods, vars, and consts must have godoc comments.
|
||||||
|
|
||||||
|
Each exported godoc comment must:
|
||||||
|
- start with the identifier name;
|
||||||
|
- explain the purpose or behavior;
|
||||||
|
- be as short as possible without losing important meaning;
|
||||||
|
- avoid repeating the signature mechanically;
|
||||||
|
- stay high-signal and informative.
|
||||||
|
|
||||||
|
### Unexported declarations
|
||||||
|
Unexported types, funcs, methods, vars, and consts should generally not have godoc-style comments unless there is a strong reason.
|
||||||
|
|
||||||
|
### Always report
|
||||||
|
- missing godoc on exported declarations;
|
||||||
|
- unnecessary godoc on unexported declarations;
|
||||||
|
- comments that are too long, vague, redundant, or low-value;
|
||||||
|
- comments that should be shortened or rewritten.
|
||||||
|
|
||||||
|
When feasible, rewrite bad godoc into better versions.
|
||||||
|
|
||||||
|
## Testing expectations
|
||||||
|
Treat tests as a required part of review.
|
||||||
|
|
||||||
|
- Assess existing test quality, not only test presence.
|
||||||
|
- Add or propose as many useful tests as reasonably possible.
|
||||||
|
- Prioritize public APIs, critical flows, edge cases, negative paths, boundary conditions, and concurrency-sensitive logic.
|
||||||
|
- Prefer table-driven tests where appropriate.
|
||||||
|
- Add regression tests for bugs you find.
|
||||||
|
- If a case is hard to test directly, explain the gap and the best test strategy.
|
||||||
|
|
||||||
|
## Commands
|
||||||
|
Before finalizing changes, run the relevant project checks when available:
|
||||||
|
- build
|
||||||
|
- tests
|
||||||
|
- lint
|
||||||
|
- static analysis
|
||||||
|
|
||||||
|
Prefer the repository’s documented commands. If multiple choices exist, use the most standard and least destructive ones first.
|
||||||
|
|
||||||
|
## Output format
|
||||||
|
For repo-wide review tasks, structure the result as:
|
||||||
|
|
||||||
|
1. Overall summary
|
||||||
|
2. Critical findings
|
||||||
|
3. Major findings
|
||||||
|
4. Minor findings
|
||||||
|
5. Godoc issues
|
||||||
|
6. Test gaps and added/proposed tests
|
||||||
|
7. Good decisions worth keeping
|
||||||
|
8. Summary of concrete changes made
|
||||||
|
|
||||||
|
For each finding include:
|
||||||
|
- location;
|
||||||
|
- issue;
|
||||||
|
- why it matters;
|
||||||
|
- recommended fix.
|
||||||
|
|
||||||
|
## Working style
|
||||||
|
- Be direct, specific, and action-oriented.
|
||||||
|
- Do not stop at style-only feedback.
|
||||||
|
- Use full repository context before drawing conclusions.
|
||||||
|
- Prefer minimal, high-confidence patches.
|
||||||
|
- Preserve behavior unless intentionally fixing a bug.
|
||||||
+151
@@ -0,0 +1,151 @@
|
|||||||
|
# Changelog
|
||||||
|
|
||||||
|
## v1.0.0-rc.11
|
||||||
|
|
||||||
|
### Fixed
|
||||||
|
- `chat_boost` update decoding now accepts string `boost_id` values, matching the current Telegram Bot API schema and preventing polling failures on boosted-chat updates.
|
||||||
|
|
||||||
|
## v1.0.0-rc.10
|
||||||
|
|
||||||
|
### Added
|
||||||
|
- `Plugin.AddUpdateHandler` for routing non-command Telegram updates by `tgapi.UpdateType`.
|
||||||
|
- Derived `tgapi.Update.Type` assignment during JSON decoding, plus `tgapi.UpdateTypeUnknown` for unmatched payloads.
|
||||||
|
- `tgapi.API.OpenFileByLink(...)` and `OpenFileByLinkWithContext(...)` for streaming downloads from Telegram's file server.
|
||||||
|
- Regression tests for update dispatch, keyboard builders, localization fallback, runners, rate limiting, parse mode encoding, streaming downloads, and context isolation.
|
||||||
|
- Regression tests for bot single-run enforcement, nil plugin registration, `L10n` concurrent access, `API.Close()` idle-connection cleanup, and `tgapi` worker-pool edge cases.
|
||||||
|
- `SEMVER.md` documenting versioning expectations for the project.
|
||||||
|
|
||||||
|
### Changed
|
||||||
|
- `NewBot` now returns `(*Bot[T], error)` instead of terminating the host process on configuration or startup failures.
|
||||||
|
- `Run` and `RunWithContext` now return errors; `RunWithContext` returns `ErrNoPrefixes` and `ErrNoPlugins` for invalid bot configuration.
|
||||||
|
- Polling retries now use exponential backoff instead of busy-looping on repeated `getUpdates` failures.
|
||||||
|
- `Bot` is now explicitly single-use; repeated `Run()` or `RunWithContext(...)` calls return `ErrBotAlreadyRun`.
|
||||||
|
- Database context wiring now uses `T` consistently instead of forcing `*T`; shared dependencies should typically use pointer types such as `*sql.DB`.
|
||||||
|
- `DatabaseContext`, `GetDBContext`, and `DbLogger` were updated to the new `T`-based dependency model.
|
||||||
|
- `DatabaseContext(...)` now warns once when `T` is a value type, to highlight likely unintended copying of shared dependencies.
|
||||||
|
- `AddDatabaseLoggerWriter(...)` now skips unset and nil database contexts instead of calling the writer with invalid values.
|
||||||
|
- `L10n` is now safe for concurrent use and copies added dictionary entries to avoid external mutation after registration.
|
||||||
|
- Plugin registration now snapshots commands, payloads, middlewares, and update handlers so later mutations of the original `*Plugin` do not leak into the bot.
|
||||||
|
- `AddPlugins(...)` now skips nil plugin pointers instead of panicking.
|
||||||
|
- `GetUpdateTypes()` now returns a copy instead of exposing internal slice state.
|
||||||
|
- Update handling now normalizes `MsgContext` for more Telegram update kinds and routes plugin-level update handlers with isolated context copies.
|
||||||
|
- `message`, `channel_post`, and `callback_query` remain on the command/payload flow; non-command updates can be handled through plugin update handlers.
|
||||||
|
- Command auto-generation now validates Telegram command names with the correct character set and `1..32` length limit, and emits commands in deterministic sorted order.
|
||||||
|
- Builder-style APIs were normalized to value returns for `NewCommandArg`, `NewMiddleware`, `NewRunner`, and `NewCallbackData`.
|
||||||
|
- `MenuButton` replaced `BaseMenuButton`, and `GetChatMenuButton(...)` now returns the renamed type.
|
||||||
|
- Several Telegram DTOs were tightened for optionality and serialization correctness, including `InputPaidMedia`, `MenuButton`, optional gift fields, and message entity slices.
|
||||||
|
- `tgapi.NewRequest(...)`, `NewRequestWithChatID(...)`, `NewUploaderRequest(...)`, and `NewUploaderRequestWithChatID(...)` are now documented as low-level unsafe escape hatches rather than internal helpers.
|
||||||
|
- `tgapi.API.Close()` now closes idle HTTP connections before releasing logger resources.
|
||||||
|
- Multipart form encoding now writes scalar field bytes directly instead of converting through temporary strings.
|
||||||
|
- README, README_RU, package docs, and exported godoc were updated to match the current APIs and concurrency/lifecycle model.
|
||||||
|
- Version constants were bumped to `v1.0.0-rc.10`.
|
||||||
|
|
||||||
|
### Fixed
|
||||||
|
- Required command arguments are now enforced by declared argument index, not only by total required count.
|
||||||
|
- `ParseNone` now omits `parse_mode` from JSON requests instead of serializing `"None"`.
|
||||||
|
- Upload file type detection is now case-insensitive for file extensions.
|
||||||
|
- Draft creation no longer panics when no limiter is configured, and draft flushing now rejects zero chat IDs before sending invalid requests.
|
||||||
|
- Channel posts with `SenderChat` no longer panic in the command path and now preserve the expected `MsgContext` fields.
|
||||||
|
- File logger initialization now falls back to stdout loggers instead of terminating the process on logger setup failures.
|
||||||
|
- `GetChatMenuButton` and `SetChatMenuButton` now serialize `chat_id` correctly when omitted.
|
||||||
|
- Update decoding tests now match the canonical `deleted_business_messages` model and no longer rely on the removed singular alias.
|
||||||
|
|
||||||
|
### Breaking Changes
|
||||||
|
- `NewBot[T](opts)` now returns `(*Bot[T], error)`.
|
||||||
|
- `Run()` now returns `error`.
|
||||||
|
- `RunWithContext(ctx)` now returns `error`.
|
||||||
|
- `Run()` and `RunWithContext(ctx)` are now single-use per bot instance; create a new `Bot` after they return.
|
||||||
|
- Database context handlers now receive `T` instead of `*T`. For shared dependencies, instantiate the bot with a pointer type, for example `Bot[*sql.DB]`.
|
||||||
|
- `DatabaseContext(...)` now takes `T` instead of `*T`.
|
||||||
|
- `GetDBContext()` now returns `T` instead of `*T`.
|
||||||
|
- `DbLogger[T]` now receives `T` instead of `*T`.
|
||||||
|
- `NewCommandArg(...)`, `NewMiddleware(...)`, `NewRunner(...)`, and `NewCallbackData(...)` now return values instead of pointers.
|
||||||
|
- `BaseMenuButton` was renamed to `MenuButton`, and `GetChatMenuButton(...)` now returns `MenuButton`.
|
||||||
|
- `tgapi.Update` no longer exposes the deprecated `DeletedBusinessMessage` alias; use `DeletedBusinessMessages`.
|
||||||
|
|
||||||
|
### Tests
|
||||||
|
- Added coverage for polling backoff helpers, command sorting, database logger safety checks, update handler routing, update-context isolation, channel posts with `SenderChat`, parse mode encoding, streaming downloads, and rate limiter behavior.
|
||||||
|
|
||||||
|
## v1.0.0-rc.7
|
||||||
|
|
||||||
|
### Added
|
||||||
|
- Package-level logger helpers: `utils.CreateLogger(prefix, level)` and `utils.CreateFileLogger(prefix, level, filePath)`.
|
||||||
|
- `MsgContext.Logger`, populated from the matched plugin and falling back to the bot logger.
|
||||||
|
- Plugin lifecycle/configuration APIs: `SetLogger`, `RemoveLogger`, `SetOnClose`, and `Close`.
|
||||||
|
- `Bot.CloseRemote(ctx)` as the explicit wrapper for Telegram Bot API close.
|
||||||
|
|
||||||
|
### Changed
|
||||||
|
- Logger initialization is now unified across `Bot`, `tgapi.API`, and `tgapi.Uploader`.
|
||||||
|
- `Bot.Close()` now performs local resource teardown only and invokes `Plugin.Close()` for registered plugins.
|
||||||
|
- Local `tgapi.API` shutdown was renamed to `Close()`.
|
||||||
|
- Telegram Bot API close wrappers in `tgapi.API` were renamed to `CloseRemote()` and `CloseRemoteWithContext()`.
|
||||||
|
- `Bot.Debug()` now updates log levels for the bot logger, request logger, and already registered plugin loggers.
|
||||||
|
- `Bot.AddPlugins()` now creates a default plugin logger automatically when one is not provided.
|
||||||
|
- `Bot.AddDatabaseLoggerWriter()` now also attaches the writer to already registered plugin loggers.
|
||||||
|
- GoDoc was expanded for the new shutdown and logging APIs, and plugin registration is now documented as a configuration commit point.
|
||||||
|
|
||||||
|
### Breaking Changes
|
||||||
|
- `(*Bot).Close(ctx context.Context)` was replaced with `(*Bot).Close()`.
|
||||||
|
- `(*tgapi.API).CloseApi()` was renamed to `(*tgapi.API).Close()`.
|
||||||
|
- `(*tgapi.API).Close()` was renamed to `(*tgapi.API).CloseRemote()`.
|
||||||
|
- `(*tgapi.API).CloseWithContext()` was renamed to `(*tgapi.API).CloseRemoteWithContext(ctx)`.
|
||||||
|
|
||||||
|
### Migration
|
||||||
|
- Replace `bot.Close(ctx)` with `bot.Close()`.
|
||||||
|
- If you need Telegram Bot API close, use `bot.CloseRemote(ctx)`.
|
||||||
|
- Replace `api.CloseApi()` with `api.Close()`.
|
||||||
|
- Replace `api.Close()` with `api.CloseRemote()`.
|
||||||
|
- Replace `api.CloseWithContext(ctx)` with `api.CloseRemoteWithContext(ctx)`.
|
||||||
|
- Configure plugin loggers and `OnClose` hooks before calling `bot.AddPlugins(...)`.
|
||||||
|
|
||||||
|
### Tests
|
||||||
|
- Updated tests for the new shutdown and logging behavior.
|
||||||
|
|
||||||
|
### Notes
|
||||||
|
- Registering a plugin via `AddPlugins(...)` is a configuration commit point; the plugin should not be mutated through the original `*Plugin` afterward.
|
||||||
|
- If plugin loggers must receive a database writer, call `AddDatabaseLoggerWriter(...)` after registering plugins.
|
||||||
|
|
||||||
|
## v1.0.0-rc.4
|
||||||
|
|
||||||
|
### Added
|
||||||
|
- `WithContext` variants across `tgapi` API and uploader methods so callers can pass cancellation and deadline contexts consistently.
|
||||||
|
- `UploaderCertificateType`, `UploadSetWebhookP`, `Uploader.SetWebhook(...)`, and `Uploader.SetWebhookWithContext(...)` for multipart webhook certificate uploads.
|
||||||
|
- Missing media thumbnail fields where applicable.
|
||||||
|
|
||||||
|
### Changed
|
||||||
|
- GoDoc for context-aware methods was improved, and `See` references now point to method-specific Telegram Bot API anchors.
|
||||||
|
- `EditMessageTextP` now includes `entities` and `link_preview_options`.
|
||||||
|
- `EditMessageCaptionP` now includes `caption_entities` and `show_caption_above_media`.
|
||||||
|
- `StopPollP` now uses `reply_markup` and no longer carries `inline_message_id`.
|
||||||
|
- `SendStickerP` now includes reply and suggested-post related fields.
|
||||||
|
- `SendDocumentP` now includes `disable_content_type_detection`.
|
||||||
|
- `SendInvoiceP` no longer includes unsupported `business_connection_id`.
|
||||||
|
- `SetWebhookP` no longer carries `certificate`; GoDoc now points to uploader-based certificate upload.
|
||||||
|
- Existing non-context methods remain available, and the `Do(...)` call style is preserved.
|
||||||
|
|
||||||
|
### Breaking Changes
|
||||||
|
- Users sending webhook certificates through JSON `SetWebhookP.Certificate` must migrate to `Uploader.SetWebhook(...)`.
|
||||||
|
|
||||||
|
## v1.0.0-rc.3
|
||||||
|
|
||||||
|
### Fixed
|
||||||
|
- The update polling loop no longer logs or retries after `context.Canceled` during shutdown.
|
||||||
|
- Extra retry delay was removed from canceled polling requests so `RunWithContext` can exit immediately while stopping.
|
||||||
|
|
||||||
|
### Changed
|
||||||
|
- Shutdown behavior remains explicit: callers are still responsible for invoking `Close()` after `RunWithContext` returns.
|
||||||
|
|
||||||
|
## v1.0.0-rc.2
|
||||||
|
|
||||||
|
### Fixed
|
||||||
|
- Fixed a shutdown crash caused by `DatabaseWriter` calling `Close()` through an uninitialized embedded logger writer.
|
||||||
|
- Fixed bot shutdown hanging during Telegram long polling by making update polling use a cancelable context.
|
||||||
|
- Reduced the chance of container termination with exit code `137` during shutdown by allowing `getUpdates` to stop promptly on cancellation.
|
||||||
|
|
||||||
|
### Changed
|
||||||
|
- Switched the project to use the local `laniakea` replacement for the shutdown fix.
|
||||||
|
- Documentation now clarifies that `RunWithContext` does not close resources automatically and callers must invoke `Close()` explicitly.
|
||||||
|
- `Updates` documentation now describes context-driven cancellation behavior.
|
||||||
|
|
||||||
|
### Tests
|
||||||
|
- Added regression tests for database logger writer shutdown behavior.
|
||||||
@@ -52,7 +52,7 @@ import (
|
|||||||
// It receives two parameters:
|
// It receives two parameters:
|
||||||
// - ctx: the message context (contains info about the message, sender, chat, etc.)
|
// - ctx: the message context (contains info about the message, sender, chat, etc.)
|
||||||
// - db: your custom database context (here we use NoDB, a placeholder for no database)
|
// - db: your custom database context (here we use NoDB, a placeholder for no database)
|
||||||
func echo(ctx *laniakea.MsgContext, db *laniakea.NoDB) {
|
func echo(ctx *laniakea.MsgContext, db laniakea.NoDB) {
|
||||||
// Answer the user with the text they sent, without any command prefix.
|
// Answer the user with the text they sent, without any command prefix.
|
||||||
// ctx.Text contains the user's message with the command part stripped off.
|
// ctx.Text contains the user's message with the command part stripped off.
|
||||||
ctx.Answer(ctx.Text) // User input WITHOUT command
|
ctx.Answer(ctx.Text) // User input WITHOUT command
|
||||||
@@ -64,7 +64,10 @@ func main() {
|
|||||||
|
|
||||||
// 2. Initialize a new bot instance.
|
// 2. Initialize a new bot instance.
|
||||||
// We use laniakea.NoDB as the database context type (no database needed for this example).
|
// We use laniakea.NoDB as the database context type (no database needed for this example).
|
||||||
bot := laniakea.NewBot[laniakea.NoDB](opts)
|
bot, err := laniakea.NewBot[laniakea.NoDB](opts)
|
||||||
|
if err != nil {
|
||||||
|
log.Fatal(err)
|
||||||
|
}
|
||||||
// Ensure bot resources are cleaned up on exit.
|
// Ensure bot resources are cleaned up on exit.
|
||||||
defer bot.Close()
|
defer bot.Close()
|
||||||
|
|
||||||
@@ -78,7 +81,7 @@ func main() {
|
|||||||
|
|
||||||
// 5. Add another command using an anonymous function (closure).
|
// 5. Add another command using an anonymous function (closure).
|
||||||
// This command simply replies "Pong" when the user sends "/ping".
|
// This command simply replies "Pong" when the user sends "/ping".
|
||||||
p.AddCommand(p.NewCommand(func(ctx *laniakea.MsgContext, db *laniakea.NoDB) {
|
p.AddCommand(p.NewCommand(func(ctx *laniakea.MsgContext, db laniakea.NoDB) {
|
||||||
ctx.Answer("Pong")
|
ctx.Answer("Pong")
|
||||||
}, "ping"))
|
}, "ping"))
|
||||||
|
|
||||||
@@ -94,7 +97,9 @@ func main() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
// 8. Start the bot, listening for updates (long polling).
|
// 8. Start the bot, listening for updates (long polling).
|
||||||
bot.Run()
|
if err := bot.Run(); err != nil {
|
||||||
|
log.Fatal(err)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
```
|
```
|
||||||
|
|
||||||
@@ -105,15 +110,16 @@ func main() {
|
|||||||
4. `AddCommand`: Registers a command. The first argument is the handler function (func(*MsgContext, T)), the second is the command name (without the slash).
|
4. `AddCommand`: Registers a command. The first argument is the handler function (func(*MsgContext, T)), the second is the command name (without the slash).
|
||||||
5. **Handler Functions**: Receive *MsgContext (message details, methods like Answer) and your custom database context T.
|
5. **Handler Functions**: Receive *MsgContext (message details, methods like Answer) and your custom database context T.
|
||||||
6. `ErrorTemplate`: Sets a template for error messages. The %s placeholder is replaced by the actual error.
|
6. `ErrorTemplate`: Sets a template for error messages. The %s placeholder is replaced by the actual error.
|
||||||
7. `AutoGenerateCommands`: Adds built-in commands (/start, /help) and a command that lists all available commands.
|
7. `AutoGenerateCommands`: Registers plugin-defined commands with Telegram across the supported scopes.
|
||||||
8. `Run()`: Starts the bot's update polling loop.
|
8. `Run()`: Starts the bot's update polling loop and returns an error if startup or polling fails.
|
||||||
|
9. A `Bot` instance is single-use. After `Run()` or `RunWithContext()` returns, create a new bot instance for the next session.
|
||||||
|
|
||||||
## 📖 Core Concepts
|
## 📖 Core Concepts
|
||||||
### Plugins
|
### Plugins
|
||||||
|
|
||||||
Plugins are the main way to organize code. A plugin can have multiple commands and middlewares.
|
Plugins are the main way to organize code. A plugin can have multiple commands and middlewares.
|
||||||
```go
|
```go
|
||||||
plugin := laniakea.NewPlugin[MyDB]("admin")
|
plugin := laniakea.NewPlugin[*MyDB]("admin")
|
||||||
plugin.AddCommand(plugin.NewCommand(banUser, "ban"))
|
plugin.AddCommand(plugin.NewCommand(banUser, "ban"))
|
||||||
bot.AddPlugins(plugin)
|
bot.AddPlugins(plugin)
|
||||||
```
|
```
|
||||||
@@ -153,14 +159,20 @@ Provides access to the incoming message and useful reply methods:
|
|||||||
|
|
||||||
This split keeps method intent explicit: JSON-only calls go through `API`, file uploads go through `Uploader`.
|
This split keeps method intent explicit: JSON-only calls go through `API`, file uploads go through `Uploader`.
|
||||||
|
|
||||||
|
For advanced cases, `tgapi.NewRequest(...)` and `tgapi.NewUploaderRequest(...)` remain public as low-level escape hatches. They are intentionally less safe than method-specific helpers: callers must supply the correct Telegram method name and compatible request/response types themselves.
|
||||||
|
|
||||||
### Database Context
|
### Database Context
|
||||||
|
|
||||||
The `T` in `NewBot[T]` is a powerful feature. You can pass any type (like a database connection pool), and it will be available in every command and middleware handler.
|
The `T` in `NewBot[T]` is a powerful feature. You can pass any type, but shared dependencies such as database pools should usually use a pointer type.
|
||||||
|
|
||||||
```go
|
```go
|
||||||
type MyDB struct { /* ... */ }
|
type MyDB struct { /* ... */ }
|
||||||
db := &MyDB{...}
|
db := &MyDB{...}
|
||||||
bot := laniakea.NewBot[*MyDB](opts, db) // Pass db instance
|
bot, err := laniakea.NewBot[*MyDB](opts)
|
||||||
|
if err != nil {
|
||||||
|
log.Fatal(err)
|
||||||
|
}
|
||||||
|
bot.DatabaseContext(db)
|
||||||
```
|
```
|
||||||
|
|
||||||
## 🧩 Middleware
|
## 🧩 Middleware
|
||||||
@@ -177,11 +189,12 @@ func(ctx *MsgContext, db T) bool
|
|||||||
- If it returns false, the execution chain stops immediately (the command will not run).
|
- If it returns false, the execution chain stops immediately (the command will not run).
|
||||||
|
|
||||||
### Adding Middleware
|
### Adding Middleware
|
||||||
Use the Use method of a plugin to add one or more middleware functions. They are executed in the order they are added.
|
Use `AddMiddleware` on a plugin to add one or more shared middleware functions. They are executed in the order they are added.
|
||||||
|
|
||||||
```go
|
```go
|
||||||
plugin := laniakea.NewPlugin[MyDB]("admin")
|
plugin := laniakea.NewPlugin[*MyDB]("admin")
|
||||||
plugin.Use(loggingMiddleware, adminOnlyMiddleware)
|
plugin.AddMiddleware(laniakea.NewMiddleware("logging", loggingMiddleware))
|
||||||
|
plugin.AddMiddleware(laniakea.NewMiddleware("admin-only", adminOnlyMiddleware))
|
||||||
plugin.AddCommand(plugin.NewCommand(banUser, "ban"))
|
plugin.AddCommand(plugin.NewCommand(banUser, "ban"))
|
||||||
```
|
```
|
||||||
|
|
||||||
@@ -212,7 +225,15 @@ func adminOnlyMiddleware(ctx *laniakea.MsgContext, db *MyDB) bool {
|
|||||||
## ⚙️ Advanced Configuration
|
## ⚙️ Advanced Configuration
|
||||||
- **Inline Keyboards**: Build keyboards using `laniakea.NewInlineKeyboardJson`, `laniakea.NewInlineKeyboardBase64`, or `laniakea.NewInlineKeyboard`.
|
- **Inline Keyboards**: Build keyboards using `laniakea.NewInlineKeyboardJson`, `laniakea.NewInlineKeyboardBase64`, or `laniakea.NewInlineKeyboard`.
|
||||||
- **Rate Limiting**: Pass a configured utils.RateLimiter via BotOpts to handle Telegram's rate limits gracefully.
|
- **Rate Limiting**: Pass a configured utils.RateLimiter via BotOpts to handle Telegram's rate limits gracefully.
|
||||||
- **Custom HTTP Client**: Provide your own http.Client in BotOpts for fine-tuned control.
|
- **Localization**: `L10n` is safe for concurrent use once attached to the bot.
|
||||||
|
- **Custom Update Handlers**: Use `plugin.AddUpdateHandler(...)` for Telegram update types that are not part of the command/payload flow.
|
||||||
|
- **Lifecycle**: `RunWithContext(...)` does not call `Close()` for you. Shut the bot down explicitly, and create a fresh `Bot` for the next run.
|
||||||
|
|
||||||
|
## Telegram Update Handling
|
||||||
|
- Commands and payloads are handled through plugins.
|
||||||
|
- Non-command updates can be routed with `plugin.AddUpdateHandler(updateType, handler)`.
|
||||||
|
- `message`, `channel_post`, and `callback_query` stay on the command/payload flow.
|
||||||
|
- `tgapi.Update` exposes a derived `Type` field after JSON unmarshalling so handlers can inspect the effective update kind directly.
|
||||||
|
|
||||||
## 📝 License
|
## 📝 License
|
||||||
|
|
||||||
|
|||||||
+47
-19
@@ -53,7 +53,7 @@ import (
|
|||||||
// Она получает два параметра:
|
// Она получает два параметра:
|
||||||
// - ctx: контекст сообщения (содержит информацию о сообщении, отправителе, чате и т.д.)
|
// - ctx: контекст сообщения (содержит информацию о сообщении, отправителе, чате и т.д.)
|
||||||
// - db: ваш пользовательский контекст базы данных (здесь мы используем NoDB — заглушку)
|
// - db: ваш пользовательский контекст базы данных (здесь мы используем NoDB — заглушку)
|
||||||
func echo(ctx *laniakea.MsgContext, db *laniakea.NoDB) {
|
func echo(ctx *laniakea.MsgContext, db laniakea.NoDB) {
|
||||||
// Отвечаем пользователю текстом, который он прислал, без префикса команды.
|
// Отвечаем пользователю текстом, который он прислал, без префикса команды.
|
||||||
// ctx.Text содержит сообщение пользователя, из которого удалена часть с командой.
|
// ctx.Text содержит сообщение пользователя, из которого удалена часть с командой.
|
||||||
ctx.Answer(ctx.Text) // Ввод пользователя БЕЗ команды
|
ctx.Answer(ctx.Text) // Ввод пользователя БЕЗ команды
|
||||||
@@ -65,7 +65,10 @@ func main() {
|
|||||||
|
|
||||||
// 2. Инициализируем новый экземпляр бота.
|
// 2. Инициализируем новый экземпляр бота.
|
||||||
// Используем laniakea.NoDB как тип контекста базы данных (база не нужна для примера).
|
// Используем laniakea.NoDB как тип контекста базы данных (база не нужна для примера).
|
||||||
bot := laniakea.NewBot[laniakea.NoDB](opts)
|
bot, err := laniakea.NewBot[laniakea.NoDB](opts)
|
||||||
|
if err != nil {
|
||||||
|
log.Fatal(err)
|
||||||
|
}
|
||||||
// Гарантируем освобождение ресурсов бота при выходе.
|
// Гарантируем освобождение ресурсов бота при выходе.
|
||||||
defer bot.Close()
|
defer bot.Close()
|
||||||
|
|
||||||
@@ -79,7 +82,7 @@ func main() {
|
|||||||
|
|
||||||
// 5. Добавляем ещё одну команду, используя анонимную функцию (замыкание).
|
// 5. Добавляем ещё одну команду, используя анонимную функцию (замыкание).
|
||||||
// Эта команда просто отвечает "Pong", когда пользователь отправляет "/ping".
|
// Эта команда просто отвечает "Pong", когда пользователь отправляет "/ping".
|
||||||
p.AddCommand(p.NewCommand(func(ctx *laniakea.MsgContext, db *laniakea.NoDB) {
|
p.AddCommand(p.NewCommand(func(ctx *laniakea.MsgContext, db laniakea.NoDB) {
|
||||||
ctx.Answer("Pong")
|
ctx.Answer("Pong")
|
||||||
}, "ping"))
|
}, "ping"))
|
||||||
|
|
||||||
@@ -95,7 +98,9 @@ func main() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
// 8. Запускаем бота, начиная прослушивание обновлений (long polling).
|
// 8. Запускаем бота, начиная прослушивание обновлений (long polling).
|
||||||
bot.Run()
|
if err := bot.Run(); err != nil {
|
||||||
|
log.Fatal(err)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
```
|
```
|
||||||
|
|
||||||
@@ -106,15 +111,16 @@ func main() {
|
|||||||
4. `AddCommand`: Регистрирует команду. Первый аргумент — функция-обработчик (func(*MsgContext, T)), второй — имя команды (без слеша).
|
4. `AddCommand`: Регистрирует команду. Первый аргумент — функция-обработчик (func(*MsgContext, T)), второй — имя команды (без слеша).
|
||||||
5. **Функции-обработчики**: Получают *MsgContext (детали сообщения, методы типа Answer) и ваш контекст базы данных T.
|
5. **Функции-обработчики**: Получают *MsgContext (детали сообщения, методы типа Answer) и ваш контекст базы данных T.
|
||||||
6. `ErrorTemplate`: Устанавливает шаблон для сообщений об ошибках. Плейсхолдер %s заменяется на текст ошибки.
|
6. `ErrorTemplate`: Устанавливает шаблон для сообщений об ошибках. Плейсхолдер %s заменяется на текст ошибки.
|
||||||
7. `AutoGenerateCommands`: Добавляет встроенные команды (/start, /help) и команду, показывающую список всех доступных команд.
|
7. `AutoGenerateCommands`: Регистрирует команды из плагинов в Telegram для поддерживаемых scope.
|
||||||
8. `Run()`: Запускает цикл опроса обновлений бота.
|
8. `Run()`: Запускает цикл опроса обновлений бота и возвращает ошибку, если старт или polling завершился неуспешно.
|
||||||
|
9. Экземпляр `Bot` одноразовый. После завершения `Run()` или `RunWithContext()` для следующего запуска создавайте новый бот.
|
||||||
|
|
||||||
## 📖 Основные концепции
|
## 📖 Основные концепции
|
||||||
### Плагины (Plugins)
|
### Плагины (Plugins)
|
||||||
Плагины — основной способ организации кода. Плагин может содержать несколько команд и Middleware.
|
Плагины — основной способ организации кода. Плагин может содержать несколько команд и Middleware.
|
||||||
|
|
||||||
```go
|
```go
|
||||||
plugin := laniakea.NewPlugin[MyDB]("admin")
|
plugin := laniakea.NewPlugin[*MyDB]("admin")
|
||||||
plugin.AddCommand(plugin.NewCommand(banUser, "ban"))
|
plugin.AddCommand(plugin.NewCommand(banUser, "ban"))
|
||||||
bot.AddPlugins(plugin)
|
bot.AddPlugins(plugin)
|
||||||
```
|
```
|
||||||
@@ -138,21 +144,34 @@ func myHandler(ctx *laniakea.MsgContext, db *MyDB) {
|
|||||||
- `KeyboardMarkdown(text string, keyboard *InlineKeyboard) *AnswerMessage`: Отправляет сообщение, отформатированное MarkdownV2 (экранирование на вашей стороне), и Inline клавиатурой.
|
- `KeyboardMarkdown(text string, keyboard *InlineKeyboard) *AnswerMessage`: Отправляет сообщение, отформатированное MarkdownV2 (экранирование на вашей стороне), и Inline клавиатурой.
|
||||||
- `AnswerPhoto(photoId, text string) *AnswerMessage`: Отправляет фотографию с подписью и parse_mode none.
|
- `AnswerPhoto(photoId, text string) *AnswerMessage`: Отправляет фотографию с подписью и parse_mode none.
|
||||||
- `AnswerPhotoMarkdown(photoId, text string) *AnswerMessage`: Отправляет фотографию с подписью, отформатированной MarkdownV2 (экранирование на вашей стороне).
|
- `AnswerPhotoMarkdown(photoId, text string) *AnswerMessage`: Отправляет фотографию с подписью, отформатированной MarkdownV2 (экранирование на вашей стороне).
|
||||||
- `EditCallback(text string)`: Редактирует сообщение, форматируя его в MarkdownV2 (экранирование на вашей стороне), после нажатия Inline кнопки.
|
- `EditCallback(text string)`: Редактирует сообщение с `parse_mode` none после нажатия inline-кнопки.
|
||||||
- `EditCallbackMarkdown(text string)`: Редактирует сообщение с parse_mode none после нажатия Inline кнопки.
|
- `EditCallbackMarkdown(text string)`: Редактирует сообщение в формате MarkdownV2 (экранирование на вашей стороне) после нажатия inline-кнопки.
|
||||||
- `SendChatAction(action string)`: Отправляет действие "печатает", "загружает фото" и т.д.
|
- `SendAction(action tgapi.ChatActionType)`: Отправляет действие "печатает", "загружает фото" и т.д.
|
||||||
- Поля: `Text`, `Args`, `From`, `Chat`, `Msg` и другие.
|
- Поля: `Text`, `Args`, `From`, `FromID`, `Msg`, `InlineMsgId`, `CallbackQueryId` и другие.
|
||||||
- И много других методов и полей!
|
- И много других методов и полей!
|
||||||
|
|
||||||
### Контекст базы данных (Database Context)
|
### Контекст базы данных (Database Context)
|
||||||
Параметр типа `T` в `NewBot[T]` — мощная функция. Вы можете передать любой тип (например, пул соединений с БД), и он будет доступен в каждом обработчике команды и中间件.
|
Параметр типа `T` в `NewBot[T]` — мощная функция. Вы можете передать любой тип, но для разделяемых зависимостей вроде пула соединений с БД обычно стоит использовать pointer type.
|
||||||
|
|
||||||
```go
|
```go
|
||||||
type MyDB struct { /* ... */ }
|
type MyDB struct { /* ... */ }
|
||||||
db := &MyDB{...}
|
db := &MyDB{...}
|
||||||
bot := laniakea.NewBot[*MyDB](opts, db) // Передаём экземпляр db
|
bot, err := laniakea.NewBot[*MyDB](opts)
|
||||||
|
if err != nil {
|
||||||
|
log.Fatal(err)
|
||||||
|
}
|
||||||
|
bot.DatabaseContext(db)
|
||||||
```
|
```
|
||||||
|
|
||||||
|
### tgapi: API и Uploader
|
||||||
|
|
||||||
|
В `tgapi` есть два клиента:
|
||||||
|
|
||||||
|
- `API` для JSON-запросов (`SendMessage`, `EditMessageText`, методы с `file_id`/URL).
|
||||||
|
- `Uploader` для multipart-загрузок (`SendPhoto`, `SendDocument`, `SendVideo` с бинарными файлами).
|
||||||
|
|
||||||
|
Для продвинутых сценариев `tgapi.NewRequest(...)` и `tgapi.NewUploaderRequest(...)` остаются публичными low-level escape hatch API. Они менее безопасны, чем типизированные helper-методы: вызывающая сторона сама отвечает за корректное имя Telegram-метода и совместимые типы параметров/ответа.
|
||||||
|
|
||||||
## 🧩 Промежуточные слои (Middleware)
|
## 🧩 Промежуточные слои (Middleware)
|
||||||
Middleware — это функции, которые выполняются перед обработчиком команды. Они идеально подходят для сквозных задач, таких как логирование, контроль доступа, ограничение скорости запросов или модификация контекста.
|
Middleware — это функции, которые выполняются перед обработчиком команды. Они идеально подходят для сквозных задач, таких как логирование, контроль доступа, ограничение скорости запросов или модификация контекста.
|
||||||
|
|
||||||
@@ -167,11 +186,12 @@ func(ctx *MsgContext, db T) bool
|
|||||||
- Если возвращается false, цепочка выполнения немедленно прерывается (команда не запускается).
|
- Если возвращается false, цепочка выполнения немедленно прерывается (команда не запускается).
|
||||||
|
|
||||||
### Добавление middleware
|
### Добавление middleware
|
||||||
Используйте метод Use плагина для добавления одной или нескольких функций middleware. Они выполняются в порядке добавления.
|
Используйте метод `AddMiddleware` плагина для добавления одной или нескольких функций middleware. Они выполняются в порядке добавления.
|
||||||
|
|
||||||
```go
|
```go
|
||||||
plugin := laniakea.NewPlugin[MyDB]("admin")
|
plugin := laniakea.NewPlugin[*MyDB]("admin")
|
||||||
plugin.Use(loggingMiddleware, adminOnlyMiddleware)
|
plugin.AddMiddleware(laniakea.NewMiddleware("logging", loggingMiddleware))
|
||||||
|
plugin.AddMiddleware(laniakea.NewMiddleware("admin-only", adminOnlyMiddleware))
|
||||||
plugin.AddCommand(plugin.NewCommand(banUser, "ban"))
|
plugin.AddCommand(plugin.NewCommand(banUser, "ban"))
|
||||||
```
|
```
|
||||||
|
|
||||||
@@ -200,9 +220,17 @@ func adminOnlyMiddleware(ctx *laniakea.MsgContext, db *MyDB) bool {
|
|||||||
- Middleware может изменять MsgContext (например, добавлять пользовательские поля) перед запуском команды.
|
- Middleware может изменять MsgContext (например, добавлять пользовательские поля) перед запуском команды.
|
||||||
|
|
||||||
## ⚙️ Расширенная настройка
|
## ⚙️ Расширенная настройка
|
||||||
**Инлайн-клавиатуры**: Создавайте клавиатуры с помощью laniakea.NewKeyboard().
|
- **Инлайн-клавиатуры**: Создавайте клавиатуры с помощью `laniakea.NewInlineKeyboardJson`, `laniakea.NewInlineKeyboardBase64` или `laniakea.NewInlineKeyboard`.
|
||||||
**Ограничение запросов**: Передайте настроенный utils.RateLimiter через BotOpts для корректной обработки лимитов Telegram.
|
- **Ограничение запросов**: Передайте настроенный `utils.RateLimiter` через `BotOpts` для корректной обработки лимитов Telegram.
|
||||||
**Пользовательский HTTP-клиент**: Предоставьте свой http.Client в BotOpts для точного контроля.
|
- **Локализация**: `L10n` безопасен для конкурентного использования после подключения к боту.
|
||||||
|
- **Пользовательские update handlers**: Используйте `plugin.AddUpdateHandler(...)` для Telegram update types вне command/payload flow.
|
||||||
|
- **Жизненный цикл**: `RunWithContext(...)` не вызывает `Close()` автоматически. Завершайте бот явно и создавайте новый `Bot` для следующего запуска.
|
||||||
|
|
||||||
|
## Обработка Telegram Updates
|
||||||
|
- Команды и payload-ы обрабатываются через плагины.
|
||||||
|
- Для некомандных update-ов можно зарегистрировать обработчик через `plugin.AddUpdateHandler(updateType, handler)`.
|
||||||
|
- `message`, `channel_post` и `callback_query` остаются в command/payload flow.
|
||||||
|
- После JSON-декодирования `tgapi.Update` заполняет поле `Type`, чтобы обработчики могли явно видеть итоговый вид update.
|
||||||
|
|
||||||
## 📝 Лицензия
|
## 📝 Лицензия
|
||||||
Этот проект лицензирован под GNU General Public License v3.0 - подробности см. в файле [LICENSE](LICENSE).
|
Этот проект лицензирован под GNU General Public License v3.0 - подробности см. в файле [LICENSE](LICENSE).
|
||||||
|
|||||||
@@ -0,0 +1,45 @@
|
|||||||
|
# Semantic Versioning Policy
|
||||||
|
|
||||||
|
This project follows Semantic Versioning with the rules below.
|
||||||
|
|
||||||
|
## Public API Surface
|
||||||
|
|
||||||
|
The public API consists of:
|
||||||
|
- exported identifiers in package `laniakea`
|
||||||
|
- exported identifiers in package `tgapi`
|
||||||
|
- documented behavior in `README.md`, `README_RU.md`, and package godoc
|
||||||
|
|
||||||
|
Anything unexported is internal and may change without notice.
|
||||||
|
|
||||||
|
## Breaking Changes
|
||||||
|
|
||||||
|
A release requires a major version bump when it changes any of the following:
|
||||||
|
- exported function, method, type, field, constant, or variable names
|
||||||
|
- function or method signatures
|
||||||
|
- JSON field names or request/response wire compatibility in `tgapi`
|
||||||
|
- documented behavioral guarantees relied on by callers
|
||||||
|
|
||||||
|
Examples:
|
||||||
|
- removing an exported alias
|
||||||
|
- changing callback payload encoding defaults
|
||||||
|
- changing handler dispatch semantics in a way that breaks existing bots
|
||||||
|
|
||||||
|
## Minor Changes
|
||||||
|
|
||||||
|
A release uses a minor version bump for backward-compatible additions:
|
||||||
|
- new exported types, methods, helpers, or update handlers
|
||||||
|
- support for new Telegram Bot API fields or methods
|
||||||
|
- optional configuration knobs that do not change existing defaults
|
||||||
|
|
||||||
|
## Patch Changes
|
||||||
|
|
||||||
|
A release uses a patch version bump for backward-compatible fixes:
|
||||||
|
- bug fixes
|
||||||
|
- test-only changes
|
||||||
|
- godoc and README clarifications
|
||||||
|
- internal refactors with no public behavior change
|
||||||
|
|
||||||
|
## Pre-Releases
|
||||||
|
|
||||||
|
`-rc.N` builds may still adjust API details before `v1.0.0`.
|
||||||
|
Once `v1.0.0` is released, breaking changes require a new major version.
|
||||||
@@ -4,9 +4,11 @@ import (
|
|||||||
"context"
|
"context"
|
||||||
"errors"
|
"errors"
|
||||||
"fmt"
|
"fmt"
|
||||||
|
"reflect"
|
||||||
"sort"
|
"sort"
|
||||||
"strings"
|
"strings"
|
||||||
"sync"
|
"sync"
|
||||||
|
"time"
|
||||||
|
|
||||||
"git.nix13.pw/scuroneko/extypes"
|
"git.nix13.pw/scuroneko/extypes"
|
||||||
"git.nix13.pw/scuroneko/laniakea/tgapi"
|
"git.nix13.pw/scuroneko/laniakea/tgapi"
|
||||||
@@ -15,13 +17,18 @@ import (
|
|||||||
"github.com/alitto/pond/v2"
|
"github.com/alitto/pond/v2"
|
||||||
)
|
)
|
||||||
|
|
||||||
// DbContext is an interface representing the application's database context.
|
// DbContext is the generic dependency type injected into bots, plugins, and handlers.
|
||||||
// It is injected into plugins and middleware via Bot.DatabaseContext().
|
// Use it for shared application state such as database handles or service containers.
|
||||||
//
|
//
|
||||||
// Example:
|
// Example:
|
||||||
//
|
//
|
||||||
// type MyDB struct { ... }
|
// type MyDB struct { ... }
|
||||||
// bot := NewBot[MyDB](opts).DatabaseContext(&myDB)
|
// myDB := &MyDB{}
|
||||||
|
// bot, err := NewBot[*MyDB](opts)
|
||||||
|
// if err != nil {
|
||||||
|
// return err
|
||||||
|
// }
|
||||||
|
// bot.DatabaseContext(myDB)
|
||||||
//
|
//
|
||||||
// Use NoDB if no database is needed.
|
// Use NoDB if no database is needed.
|
||||||
type DbContext any
|
type DbContext any
|
||||||
@@ -32,7 +39,7 @@ type NoDB struct{ DbContext }
|
|||||||
|
|
||||||
// DbLogger is a function type that returns a slog.LoggerWriter for database logging.
|
// DbLogger is a function type that returns a slog.LoggerWriter for database logging.
|
||||||
// Used to inject database-specific log output (e.g., SQL queries, ORM events).
|
// Used to inject database-specific log output (e.g., SQL queries, ORM events).
|
||||||
type DbLogger[T DbContext] func(db *T) slog.LoggerWriter
|
type DbLogger[T DbContext] func(db T) slog.LoggerWriter
|
||||||
|
|
||||||
// BotPayloadType defines the serialization format for callback data payloads.
|
// BotPayloadType defines the serialization format for callback data payloads.
|
||||||
type BotPayloadType string
|
type BotPayloadType string
|
||||||
@@ -44,6 +51,20 @@ var (
|
|||||||
BotPayloadJson BotPayloadType = "json"
|
BotPayloadJson BotPayloadType = "json"
|
||||||
)
|
)
|
||||||
|
|
||||||
|
var (
|
||||||
|
// ErrNoPrefixes reports that the bot was started without any command prefixes.
|
||||||
|
ErrNoPrefixes = errors.New("no prefixes defined")
|
||||||
|
// ErrNoPlugins reports that the bot was started without any registered plugins.
|
||||||
|
ErrNoPlugins = errors.New("no plugins defined")
|
||||||
|
// ErrBotAlreadyRun reports that Run or RunWithContext was called more than once.
|
||||||
|
ErrBotAlreadyRun = errors.New("bot can only be run once")
|
||||||
|
|
||||||
|
// ErrTokenRequired reports that BotOpts.Token was empty.
|
||||||
|
ErrTokenRequired = errors.New("token required")
|
||||||
|
// ErrOptsIsNil reports that NewBot was called with a nil BotOpts pointer.
|
||||||
|
ErrOptsIsNil = errors.New("opts is nil")
|
||||||
|
)
|
||||||
|
|
||||||
// Bot is the core Telegram bot instance.
|
// Bot is the core Telegram bot instance.
|
||||||
//
|
//
|
||||||
// Manages:
|
// Manages:
|
||||||
@@ -53,7 +74,8 @@ var (
|
|||||||
// - Logging and rate limiting
|
// - Logging and rate limiting
|
||||||
// - Localization and draft message support
|
// - Localization and draft message support
|
||||||
//
|
//
|
||||||
// All methods are safe for concurrent use. Direct field access is not recommended.
|
// Runtime accessors are safe for concurrent use. Configure the bot before Run.
|
||||||
|
// A Bot is single-use: after Run or RunWithContext returns, create a new Bot for the next session.
|
||||||
type Bot[T DbContext] struct {
|
type Bot[T DbContext] struct {
|
||||||
token string
|
token string
|
||||||
debug bool
|
debug bool
|
||||||
@@ -73,9 +95,11 @@ type Bot[T DbContext] struct {
|
|||||||
|
|
||||||
api *tgapi.API // Telegram API client
|
api *tgapi.API // Telegram API client
|
||||||
uploader *tgapi.Uploader // File uploader
|
uploader *tgapi.Uploader // File uploader
|
||||||
dbContext *T // Injected database context
|
dbContext T // Injected database context
|
||||||
l10n *L10n // Localization manager
|
hasDBContext bool
|
||||||
draftProvider *DraftProvider // Draft message builder
|
warnedValueDB bool
|
||||||
|
l10n *L10n // Localization manager
|
||||||
|
draftProvider *DraftProvider // Draft message builder
|
||||||
|
|
||||||
updateOffsetMu sync.Mutex
|
updateOffsetMu sync.Mutex
|
||||||
updateOffset int // Last processed update ID
|
updateOffset int // Last processed update ID
|
||||||
@@ -83,6 +107,9 @@ type Bot[T DbContext] struct {
|
|||||||
updateQueue chan *tgapi.Update // Internal queue for processing updates
|
updateQueue chan *tgapi.Update // Internal queue for processing updates
|
||||||
runnerOnceWG sync.WaitGroup // Tracks one-time async runners
|
runnerOnceWG sync.WaitGroup // Tracks one-time async runners
|
||||||
runnerBgWG sync.WaitGroup // Tracks background async runners
|
runnerBgWG sync.WaitGroup // Tracks background async runners
|
||||||
|
runStateMu sync.Mutex
|
||||||
|
running bool
|
||||||
|
ran bool
|
||||||
}
|
}
|
||||||
|
|
||||||
// NewBot creates and initializes a new Bot instance using the provided BotOpts.
|
// NewBot creates and initializes a new Bot instance using the provided BotOpts.
|
||||||
@@ -93,13 +120,12 @@ type Bot[T DbContext] struct {
|
|||||||
// - Fetches bot username via GetMe()
|
// - Fetches bot username via GetMe()
|
||||||
// - Sets up DraftProvider with random IDs
|
// - Sets up DraftProvider with random IDs
|
||||||
// - Adds API and Uploader loggers to extraLoggers
|
// - Adds API and Uploader loggers to extraLoggers
|
||||||
//
|
func NewBot[T any](opts *BotOpts) (*Bot[T], error) {
|
||||||
// Panics if:
|
if opts == nil {
|
||||||
// - Token is empty
|
return nil, ErrOptsIsNil
|
||||||
// - GetMe() fails (invalid token or network error)
|
}
|
||||||
func NewBot[T any](opts *BotOpts) *Bot[T] {
|
|
||||||
if opts.Token == "" {
|
if opts.Token == "" {
|
||||||
panic("laniakea: BotOpts.Token is required")
|
return nil, ErrTokenRequired
|
||||||
}
|
}
|
||||||
|
|
||||||
updateQueue := make(chan *tgapi.Update, 512)
|
updateQueue := make(chan *tgapi.Update, 512)
|
||||||
@@ -163,7 +189,7 @@ func NewBot[T any](opts *BotOpts) *Bot[T] {
|
|||||||
u, err := api.GetMe()
|
u, err := api.GetMe()
|
||||||
if err != nil {
|
if err != nil {
|
||||||
_ = bot.Close()
|
_ = bot.Close()
|
||||||
bot.logger.Fatal(err)
|
return nil, err
|
||||||
}
|
}
|
||||||
bot.username = Val(u.Username, "")
|
bot.username = Val(u.Username, "")
|
||||||
if bot.username == "" {
|
if bot.username == "" {
|
||||||
@@ -171,7 +197,7 @@ func NewBot[T any](opts *BotOpts) *Bot[T] {
|
|||||||
}
|
}
|
||||||
bot.logger.Infoln(fmt.Sprintf("Authorized as %s (@%s)", u.FirstName, Val(u.Username, "unknown")))
|
bot.logger.Infoln(fmt.Sprintf("Authorized as %s (@%s)", u.FirstName, Val(u.Username, "unknown")))
|
||||||
|
|
||||||
return bot
|
return bot, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// Close gracefully shuts down bot-owned resources.
|
// Close gracefully shuts down bot-owned resources.
|
||||||
@@ -226,11 +252,7 @@ func (bot *Bot[T]) CloseRemote(ctx context.Context) error {
|
|||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// initLoggers configures the main and optional request loggers.
|
// Internal logger setup for the bot and optional request logger.
|
||||||
//
|
|
||||||
// Uses DEBUG flag to set log level (DEBUG if true, FATAL otherwise).
|
|
||||||
// Writes to stdout in JSON format by default.
|
|
||||||
// If WriteToFile is true, writes to main.log and requests.log in LoggerBasePath.
|
|
||||||
func (bot *Bot[T]) initLoggers(opts *BotOpts) {
|
func (bot *Bot[T]) initLoggers(opts *BotOpts) {
|
||||||
level := slog.FATAL
|
level := slog.FATAL
|
||||||
if opts.Debug {
|
if opts.Debug {
|
||||||
@@ -242,9 +264,10 @@ func (bot *Bot[T]) initLoggers(opts *BotOpts) {
|
|||||||
path := fmt.Sprintf("%s/main.log", strings.TrimRight(opts.LoggerBasePath, "/"))
|
path := fmt.Sprintf("%s/main.log", strings.TrimRight(opts.LoggerBasePath, "/"))
|
||||||
logger, err := utils.CreateFileLogger("BOT", level, path)
|
logger, err := utils.CreateFileLogger("BOT", level, path)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
bot.logger.Fatal(err)
|
bot.logger.Errorln(err)
|
||||||
|
} else {
|
||||||
|
bot.logger = logger
|
||||||
}
|
}
|
||||||
bot.logger = logger
|
|
||||||
}
|
}
|
||||||
|
|
||||||
if opts.UseRequestLogger {
|
if opts.UseRequestLogger {
|
||||||
@@ -253,9 +276,10 @@ func (bot *Bot[T]) initLoggers(opts *BotOpts) {
|
|||||||
path := fmt.Sprintf("%s/requests.log", strings.TrimRight(opts.LoggerBasePath, "/"))
|
path := fmt.Sprintf("%s/requests.log", strings.TrimRight(opts.LoggerBasePath, "/"))
|
||||||
logger, err := utils.CreateFileLogger("REQUESTS", level, path)
|
logger, err := utils.CreateFileLogger("REQUESTS", level, path)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
bot.logger.Fatal(err)
|
bot.logger.Errorln(err)
|
||||||
|
} else {
|
||||||
|
bot.RequestLogger = logger
|
||||||
}
|
}
|
||||||
bot.RequestLogger = logger
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -275,14 +299,16 @@ func (bot *Bot[T]) SetUpdateOffset(offset int) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
// GetUpdateTypes returns the list of update types the bot is configured to receive.
|
// GetUpdateTypes returns the list of update types the bot is configured to receive.
|
||||||
func (bot *Bot[T]) GetUpdateTypes() []tgapi.UpdateType { return bot.updateTypes }
|
func (bot *Bot[T]) GetUpdateTypes() []tgapi.UpdateType {
|
||||||
|
return append([]tgapi.UpdateType(nil), bot.updateTypes...)
|
||||||
|
}
|
||||||
|
|
||||||
// GetLogger returns the main bot logger.
|
// GetLogger returns the main bot logger.
|
||||||
func (bot *Bot[T]) GetLogger() *slog.Logger { return bot.logger }
|
func (bot *Bot[T]) GetLogger() *slog.Logger { return bot.logger }
|
||||||
|
|
||||||
// GetDBContext returns the injected database context.
|
// GetDBContext returns the injected database context.
|
||||||
// Returns nil if not set via DatabaseContext().
|
// If DatabaseContext was not called, it returns the zero value of T.
|
||||||
func (bot *Bot[T]) GetDBContext() *T { return bot.dbContext }
|
func (bot *Bot[T]) GetDBContext() T { return bot.dbContext }
|
||||||
|
|
||||||
// GetLoggerLevel returns the effective log level derived from the bot's debug
|
// GetLoggerLevel returns the effective log level derived from the bot's debug
|
||||||
// flag.
|
// flag.
|
||||||
@@ -309,8 +335,16 @@ func (bot *Bot[T]) SetDraftProvider(p *DraftProvider) *Bot[T] {
|
|||||||
|
|
||||||
// DatabaseContext injects a database context into the bot.
|
// DatabaseContext injects a database context into the bot.
|
||||||
// This context is accessible to plugins and middleware via GetDBContext().
|
// This context is accessible to plugins and middleware via GetDBContext().
|
||||||
func (bot *Bot[T]) DatabaseContext(ctx *T) *Bot[T] {
|
// For shared dependencies such as *sql.DB, prefer using a pointer type as T.
|
||||||
|
// Value-typed contexts are supported, but the bot warns once because handlers
|
||||||
|
// receive T by value.
|
||||||
|
func (bot *Bot[T]) DatabaseContext(ctx T) *Bot[T] {
|
||||||
|
if !bot.warnedValueDB && shouldWarnOnValueDBContext[T]() && bot.logger != nil {
|
||||||
|
bot.logger.Warnln("database context uses a value type; shared dependencies should usually use a pointer type as T")
|
||||||
|
bot.warnedValueDB = true
|
||||||
|
}
|
||||||
bot.dbContext = ctx
|
bot.dbContext = ctx
|
||||||
|
bot.hasDBContext = true
|
||||||
return bot
|
return bot
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -383,12 +417,20 @@ func (bot *Bot[T]) Debug(debug bool) *Bot[T] {
|
|||||||
func (bot *Bot[T]) AddPlugins(plugin ...*Plugin[T]) *Bot[T] {
|
func (bot *Bot[T]) AddPlugins(plugin ...*Plugin[T]) *Bot[T] {
|
||||||
level := bot.GetLoggerLevel()
|
level := bot.GetLoggerLevel()
|
||||||
for _, p := range plugin {
|
for _, p := range plugin {
|
||||||
if p.logger == nil {
|
if p == nil {
|
||||||
logger := utils.CreateLogger(p.name, level)
|
if bot.logger != nil {
|
||||||
p.SetLogger(logger)
|
bot.logger.Warn("nil plugin skipped")
|
||||||
|
}
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
cloned := clonePlugin(p)
|
||||||
|
if cloned.logger == nil {
|
||||||
|
cloned.logger = utils.CreateLogger(cloned.name, level)
|
||||||
|
}
|
||||||
|
bot.plugins = append(bot.plugins, cloned)
|
||||||
|
if bot.logger != nil {
|
||||||
|
bot.logger.Debugln(fmt.Sprintf("plugins with name \"%s\" registered", cloned.name))
|
||||||
}
|
}
|
||||||
bot.plugins = append(bot.plugins, *p)
|
|
||||||
bot.logger.Debugln(fmt.Sprintf("plugins with name \"%s\" registered", p.name))
|
|
||||||
}
|
}
|
||||||
return bot
|
return bot
|
||||||
}
|
}
|
||||||
@@ -405,13 +447,14 @@ func (bot *Bot[T]) AddPlugins(plugin ...*Plugin[T]) *Bot[T] {
|
|||||||
//
|
//
|
||||||
// Example:
|
// Example:
|
||||||
//
|
//
|
||||||
// bot.AddMiddleware(&authMiddleware, &rateLimitMiddleware)
|
// bot.AddMiddleware(authMiddleware, rateLimitMiddleware)
|
||||||
//
|
//
|
||||||
// Panics if any middleware has a nil name.
|
// Middleware with an empty name are skipped with a warning.
|
||||||
func (bot *Bot[T]) AddMiddleware(middleware ...Middleware[T]) *Bot[T] {
|
func (bot *Bot[T]) AddMiddleware(middleware ...Middleware[T]) *Bot[T] {
|
||||||
for _, m := range middleware {
|
for _, m := range middleware {
|
||||||
if m.name == "" {
|
if m.name == "" {
|
||||||
panic("laniakea: middleware must have a non-empty name")
|
bot.logger.Warnln("middleware must have a non-empty name")
|
||||||
|
continue
|
||||||
}
|
}
|
||||||
bot.middlewares = append(bot.middlewares, m)
|
bot.middlewares = append(bot.middlewares, m)
|
||||||
bot.logger.Debugln(fmt.Sprintf("middleware with name \"%s\" registered", m.name))
|
bot.logger.Debugln(fmt.Sprintf("middleware with name \"%s\" registered", m.name))
|
||||||
@@ -442,12 +485,13 @@ func (bot *Bot[T]) AddMiddleware(middleware ...Middleware[T]) *Bot[T] {
|
|||||||
//
|
//
|
||||||
// Example:
|
// Example:
|
||||||
//
|
//
|
||||||
// bot.AddRunner(&cleanupRunner)
|
// bot.AddRunner(cleanupRunner)
|
||||||
//
|
//
|
||||||
// Panics if runner has a nil name.
|
// Runners with an empty name are skipped with a warning.
|
||||||
func (bot *Bot[T]) AddRunner(runner Runner[T]) *Bot[T] {
|
func (bot *Bot[T]) AddRunner(runner Runner[T]) *Bot[T] {
|
||||||
if runner.name == "" {
|
if runner.name == "" {
|
||||||
panic("laniakea: runner must have a non-empty name")
|
bot.logger.Warnln("runner must have a non-empty name")
|
||||||
|
return bot
|
||||||
}
|
}
|
||||||
bot.runners = append(bot.runners, runner)
|
bot.runners = append(bot.runners, runner)
|
||||||
bot.logger.Debugln(fmt.Sprintf("runner with name \"%s\" registered", runner.name))
|
bot.logger.Debugln(fmt.Sprintf("runner with name \"%s\" registered", runner.name))
|
||||||
@@ -494,6 +538,14 @@ func (bot *Bot[T]) AddL10n(l *L10n) *Bot[T] {
|
|||||||
// return db.QueryLogger()
|
// return db.QueryLogger()
|
||||||
// })
|
// })
|
||||||
func (bot *Bot[T]) AddDatabaseLoggerWriter(writer DbLogger[T]) *Bot[T] {
|
func (bot *Bot[T]) AddDatabaseLoggerWriter(writer DbLogger[T]) *Bot[T] {
|
||||||
|
if !bot.hasDBContext {
|
||||||
|
bot.logger.Warnln("database context is not set; skipping database logger writer")
|
||||||
|
return bot
|
||||||
|
}
|
||||||
|
if isNilValue(bot.dbContext) {
|
||||||
|
bot.logger.Warnln("database context is nil; skipping database logger writer")
|
||||||
|
return bot
|
||||||
|
}
|
||||||
w := writer(bot.dbContext)
|
w := writer(bot.dbContext)
|
||||||
bot.logger.AddWriter(w)
|
bot.logger.AddWriter(w)
|
||||||
if bot.RequestLogger != nil {
|
if bot.RequestLogger != nil {
|
||||||
@@ -532,17 +584,21 @@ func (bot *Bot[T]) AddDatabaseLoggerWriter(writer DbLogger[T]) *Bot[T] {
|
|||||||
// go bot.RunWithContext(ctx)
|
// go bot.RunWithContext(ctx)
|
||||||
// // ... later ...
|
// // ... later ...
|
||||||
// cancel() // triggers graceful shutdown
|
// cancel() // triggers graceful shutdown
|
||||||
// _ = bot.Close(context.Background())
|
// _ = bot.Close()
|
||||||
func (bot *Bot[T]) RunWithContext(ctx context.Context) {
|
//
|
||||||
|
// A Bot is single-use. After RunWithContext returns, later calls return ErrBotAlreadyRun.
|
||||||
|
func (bot *Bot[T]) RunWithContext(ctx context.Context) error {
|
||||||
if len(bot.prefixes) == 0 {
|
if len(bot.prefixes) == 0 {
|
||||||
bot.logger.Fatalln("no prefixes defined")
|
return ErrNoPrefixes
|
||||||
return
|
|
||||||
}
|
}
|
||||||
|
|
||||||
if len(bot.plugins) == 0 {
|
if len(bot.plugins) == 0 {
|
||||||
bot.logger.Fatalln("no plugins defined")
|
return ErrNoPlugins
|
||||||
return
|
|
||||||
}
|
}
|
||||||
|
if err := bot.beginRun(); err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
defer bot.finishRun()
|
||||||
|
|
||||||
bot.ExecRunners(ctx)
|
bot.ExecRunners(ctx)
|
||||||
|
|
||||||
@@ -556,6 +612,7 @@ func (bot *Bot[T]) RunWithContext(ctx context.Context) {
|
|||||||
}
|
}
|
||||||
close(bot.updateQueue)
|
close(bot.updateQueue)
|
||||||
}()
|
}()
|
||||||
|
retryDelay := time.Duration(0)
|
||||||
for {
|
for {
|
||||||
select {
|
select {
|
||||||
case <-ctx.Done():
|
case <-ctx.Done():
|
||||||
@@ -567,8 +624,19 @@ func (bot *Bot[T]) RunWithContext(ctx context.Context) {
|
|||||||
return
|
return
|
||||||
}
|
}
|
||||||
bot.logger.Errorln("failed to fetch updates:", err)
|
bot.logger.Errorln("failed to fetch updates:", err)
|
||||||
|
retryDelay = nextPollRetryDelay(retryDelay)
|
||||||
|
timer := time.NewTimer(retryDelay)
|
||||||
|
select {
|
||||||
|
case <-ctx.Done():
|
||||||
|
if !timer.Stop() {
|
||||||
|
<-timer.C
|
||||||
|
}
|
||||||
|
return
|
||||||
|
case <-timer.C:
|
||||||
|
}
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
|
retryDelay = 0
|
||||||
|
|
||||||
for _, update := range updates {
|
for _, update := range updates {
|
||||||
u := update // copy loop variable to avoid race condition
|
u := update // copy loop variable to avoid race condition
|
||||||
@@ -593,6 +661,7 @@ func (bot *Bot[T]) RunWithContext(ctx context.Context) {
|
|||||||
pool.Stop() // Wait for all tasks to complete and stop the pool
|
pool.Stop() // Wait for all tasks to complete and stop the pool
|
||||||
bot.runnerOnceWG.Wait()
|
bot.runnerOnceWG.Wait()
|
||||||
bot.runnerBgWG.Wait()
|
bot.runnerBgWG.Wait()
|
||||||
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// Run starts the bot using a background context.
|
// Run starts the bot using a background context.
|
||||||
@@ -601,6 +670,96 @@ func (bot *Bot[T]) RunWithContext(ctx context.Context) {
|
|||||||
// Use this for simple bots where graceful shutdown is not required.
|
// Use this for simple bots where graceful shutdown is not required.
|
||||||
//
|
//
|
||||||
// For production use, prefer RunWithContext to handle SIGINT/SIGTERM gracefully.
|
// For production use, prefer RunWithContext to handle SIGINT/SIGTERM gracefully.
|
||||||
func (bot *Bot[T]) Run() {
|
func (bot *Bot[T]) Run() error {
|
||||||
bot.RunWithContext(context.Background())
|
return bot.RunWithContext(context.Background())
|
||||||
|
}
|
||||||
|
|
||||||
|
func (bot *Bot[T]) beginRun() error {
|
||||||
|
bot.runStateMu.Lock()
|
||||||
|
defer bot.runStateMu.Unlock()
|
||||||
|
if bot.running || bot.ran {
|
||||||
|
return ErrBotAlreadyRun
|
||||||
|
}
|
||||||
|
bot.running = true
|
||||||
|
bot.ran = true
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
|
||||||
|
func (bot *Bot[T]) finishRun() {
|
||||||
|
bot.runStateMu.Lock()
|
||||||
|
bot.running = false
|
||||||
|
bot.runStateMu.Unlock()
|
||||||
|
}
|
||||||
|
|
||||||
|
func nextPollRetryDelay(prev time.Duration) time.Duration {
|
||||||
|
if prev <= 0 {
|
||||||
|
return time.Second
|
||||||
|
}
|
||||||
|
next := prev * 2
|
||||||
|
if next > 30*time.Second {
|
||||||
|
return 30 * time.Second
|
||||||
|
}
|
||||||
|
return next
|
||||||
|
}
|
||||||
|
|
||||||
|
func isNilValue[T any](v T) bool {
|
||||||
|
rv := reflect.ValueOf(v)
|
||||||
|
if !rv.IsValid() {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
switch rv.Kind() {
|
||||||
|
case reflect.Chan, reflect.Func, reflect.Interface, reflect.Map, reflect.Pointer, reflect.Slice:
|
||||||
|
return rv.IsNil()
|
||||||
|
default:
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func shouldWarnOnValueDBContext[T any]() bool {
|
||||||
|
t := reflect.TypeFor[T]()
|
||||||
|
if t == reflect.TypeFor[NoDB]() {
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
switch t.Kind() {
|
||||||
|
case reflect.Pointer, reflect.Interface, reflect.Map, reflect.Slice, reflect.Func, reflect.Chan:
|
||||||
|
return false
|
||||||
|
default:
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func clonePlugin[T DbContext](p *Plugin[T]) Plugin[T] {
|
||||||
|
cloned := Plugin[T]{
|
||||||
|
name: p.name,
|
||||||
|
commands: make(map[string]*Command[T], len(p.commands)),
|
||||||
|
payloads: make(map[string]*Command[T], len(p.payloads)),
|
||||||
|
middlewares: append(extypes.Slice[Middleware[T]](nil), p.middlewares...),
|
||||||
|
skipAutoCmd: p.skipAutoCmd,
|
||||||
|
logger: p.logger,
|
||||||
|
handlers: make(map[tgapi.UpdateType]CommandExecutor[T]),
|
||||||
|
onClose: p.onClose,
|
||||||
|
}
|
||||||
|
|
||||||
|
for name, command := range p.commands {
|
||||||
|
cloned.commands[name] = cloneCommand(command)
|
||||||
|
}
|
||||||
|
for name, command := range p.payloads {
|
||||||
|
cloned.payloads[name] = cloneCommand(command)
|
||||||
|
}
|
||||||
|
for t, handler := range p.handlers {
|
||||||
|
cloned.handlers[t] = handler
|
||||||
|
}
|
||||||
|
|
||||||
|
return cloned
|
||||||
|
}
|
||||||
|
|
||||||
|
func cloneCommand[T DbContext](command *Command[T]) *Command[T] {
|
||||||
|
if command == nil {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
|
||||||
|
cloned := *command
|
||||||
|
cloned.args = append(extypes.Slice[CommandArg](nil), command.args...)
|
||||||
|
cloned.middlewares = append(extypes.Slice[Middleware[T]](nil), command.middlewares...)
|
||||||
|
return &cloned
|
||||||
}
|
}
|
||||||
|
|||||||
+19
-7
@@ -11,7 +11,7 @@ import (
|
|||||||
// BotOpts holds configuration options for initializing a Bot.
|
// BotOpts holds configuration options for initializing a Bot.
|
||||||
//
|
//
|
||||||
// Values are loaded from environment variables via LoadOptsFromEnv().
|
// Values are loaded from environment variables via LoadOptsFromEnv().
|
||||||
// Use NewOpts() to create a zero-value struct and set fields manually.
|
// Use &BotOpts{} to create a value and set fields manually.
|
||||||
type BotOpts struct {
|
type BotOpts struct {
|
||||||
// Token is the Telegram bot token (required).
|
// Token is the Telegram bot token (required).
|
||||||
Token string
|
Token string
|
||||||
@@ -56,7 +56,7 @@ type BotOpts struct {
|
|||||||
// Use this to prioritize responsiveness over reliability.
|
// Use this to prioritize responsiveness over reliability.
|
||||||
DropRLOverflow bool
|
DropRLOverflow bool
|
||||||
|
|
||||||
// MaxWorkers is the maximum number of concurrency running update handlers.
|
// MaxWorkers is the maximum number of update handlers that may run concurrently.
|
||||||
MaxWorkers int
|
MaxWorkers int
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -75,20 +75,30 @@ type BotOpts struct {
|
|||||||
// - API_URL: custom API endpoint
|
// - API_URL: custom API endpoint
|
||||||
// - RATE_LIMIT: max requests per second (default: 30)
|
// - RATE_LIMIT: max requests per second (default: 30)
|
||||||
// - DROP_RL_OVERFLOW: "true" to drop updates on rate limit overflow
|
// - DROP_RL_OVERFLOW: "true" to drop updates on rate limit overflow
|
||||||
|
// - MAX_WORKERS: maximum number of concurrent update handlers (default: 32)
|
||||||
//
|
//
|
||||||
// Returns a populated BotOpts. If TG_TOKEN is missing, behavior is undefined.
|
// Returns a populated BotOpts.
|
||||||
|
// NewBot validates required fields and returns ErrTokenRequired when TG_TOKEN is missing.
|
||||||
func LoadOptsFromEnv() *BotOpts {
|
func LoadOptsFromEnv() *BotOpts {
|
||||||
rateLimit := 30
|
rateLimit := 30
|
||||||
|
maxWorkers := 32
|
||||||
|
|
||||||
|
stringUpdateTypes := splitEnvList(os.Getenv("UPDATE_TYPES"))
|
||||||
|
updateTypes := make([]tgapi.UpdateType, 0, len(stringUpdateTypes))
|
||||||
|
for _, updateType := range stringUpdateTypes {
|
||||||
|
updateTypes = append(updateTypes, tgapi.UpdateType(updateType))
|
||||||
|
}
|
||||||
|
|
||||||
if rl := os.Getenv("RATE_LIMIT"); rl != "" {
|
if rl := os.Getenv("RATE_LIMIT"); rl != "" {
|
||||||
if n, err := strconv.Atoi(rl); err == nil {
|
if n, err := strconv.Atoi(rl); err == nil {
|
||||||
rateLimit = n
|
rateLimit = n
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
stringUpdateTypes := splitEnvList(os.Getenv("UPDATE_TYPES"))
|
if mw := os.Getenv("MAX_WORKERS"); mw != "" {
|
||||||
updateTypes := make([]tgapi.UpdateType, 0, len(stringUpdateTypes))
|
if n, err := strconv.Atoi(os.Getenv("MAX_WORKERS")); err == nil {
|
||||||
for _, updateType := range stringUpdateTypes {
|
maxWorkers = n
|
||||||
updateTypes = append(updateTypes, tgapi.UpdateType(updateType))
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
return &BotOpts{
|
return &BotOpts{
|
||||||
@@ -108,6 +118,8 @@ func LoadOptsFromEnv() *BotOpts {
|
|||||||
|
|
||||||
RateLimit: rateLimit,
|
RateLimit: rateLimit,
|
||||||
DropRLOverflow: os.Getenv("DROP_RL_OVERFLOW") == "true",
|
DropRLOverflow: os.Getenv("DROP_RL_OVERFLOW") == "true",
|
||||||
|
|
||||||
|
MaxWorkers: maxWorkers,
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
+201
@@ -0,0 +1,201 @@
|
|||||||
|
package laniakea
|
||||||
|
|
||||||
|
import (
|
||||||
|
"context"
|
||||||
|
"errors"
|
||||||
|
"path/filepath"
|
||||||
|
"reflect"
|
||||||
|
"testing"
|
||||||
|
"time"
|
||||||
|
|
||||||
|
"git.nix13.pw/scuroneko/laniakea/tgapi"
|
||||||
|
"git.nix13.pw/scuroneko/slog"
|
||||||
|
)
|
||||||
|
|
||||||
|
func TestGetUpdateTypesReturnsCopy(t *testing.T) {
|
||||||
|
bot := &Bot[NoDB]{updateTypes: []tgapi.UpdateType{tgapi.UpdateTypeMessage}}
|
||||||
|
|
||||||
|
got := bot.GetUpdateTypes()
|
||||||
|
got[0] = tgapi.UpdateTypeCallbackQuery
|
||||||
|
|
||||||
|
if want := []tgapi.UpdateType{tgapi.UpdateTypeMessage}; !reflect.DeepEqual(bot.updateTypes, want) {
|
||||||
|
t.Fatalf("GetUpdateTypes exposed internal slice: got %v want %v", bot.updateTypes, want)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestAddPluginsSnapshotsConfiguration(t *testing.T) {
|
||||||
|
bot := &Bot[NoDB]{logger: slog.CreateLogger()}
|
||||||
|
plugin := NewPlugin[NoDB]("demo")
|
||||||
|
|
||||||
|
cmd := plugin.NewCommand(func(ctx *MsgContext, db NoDB) {}, "start")
|
||||||
|
plugin.AddMiddleware(NewMiddleware("base", func(ctx *MsgContext, db NoDB) bool { return true }))
|
||||||
|
|
||||||
|
bot.AddPlugins(plugin)
|
||||||
|
|
||||||
|
cmd.SetDescription("mutated after registration")
|
||||||
|
plugin.NewCommand(func(ctx *MsgContext, db NoDB) {}, "late")
|
||||||
|
plugin.AddMiddleware(NewMiddleware("late", func(ctx *MsgContext, db NoDB) bool { return true }))
|
||||||
|
|
||||||
|
registered := bot.plugins[0]
|
||||||
|
if _, exists := registered.commands["late"]; exists {
|
||||||
|
t.Fatal("late command leaked into registered plugin snapshot")
|
||||||
|
}
|
||||||
|
if registered.commands["start"].description != "" {
|
||||||
|
t.Fatalf("registered command description unexpectedly mutated: %q", registered.commands["start"].description)
|
||||||
|
}
|
||||||
|
if len(registered.middlewares) != 1 {
|
||||||
|
t.Fatalf("registered middlewares unexpectedly mutated: got %d want 1", len(registered.middlewares))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestAddPluginsSkipsNilPlugin(t *testing.T) {
|
||||||
|
bot := &Bot[NoDB]{logger: slog.CreateLogger()}
|
||||||
|
plugin := NewPlugin[NoDB]("demo")
|
||||||
|
|
||||||
|
bot.AddPlugins(nil, plugin)
|
||||||
|
|
||||||
|
if len(bot.plugins) != 1 {
|
||||||
|
t.Fatalf("expected exactly one registered plugin, got %d", len(bot.plugins))
|
||||||
|
}
|
||||||
|
if bot.plugins[0].name != "demo" {
|
||||||
|
t.Fatalf("unexpected plugin name: %q", bot.plugins[0].name)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestInitLoggersFallsBackToStdoutLoggerOnFileError(t *testing.T) {
|
||||||
|
bot := &Bot[NoDB]{}
|
||||||
|
|
||||||
|
bot.initLoggers(&BotOpts{
|
||||||
|
Debug: true,
|
||||||
|
WriteToFile: true,
|
||||||
|
UseRequestLogger: true,
|
||||||
|
LoggerBasePath: filepath.Join(t.TempDir(), "missing", "nested"),
|
||||||
|
})
|
||||||
|
|
||||||
|
if bot.logger == nil {
|
||||||
|
t.Fatal("expected main logger fallback")
|
||||||
|
}
|
||||||
|
if bot.RequestLogger == nil {
|
||||||
|
t.Fatal("expected request logger fallback")
|
||||||
|
}
|
||||||
|
if err := bot.RequestLogger.Close(); err != nil {
|
||||||
|
t.Fatalf("failed to close request logger: %v", err)
|
||||||
|
}
|
||||||
|
if err := bot.logger.Close(); err != nil {
|
||||||
|
t.Fatalf("failed to close main logger: %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestNextPollRetryDelay(t *testing.T) {
|
||||||
|
tests := []struct {
|
||||||
|
name string
|
||||||
|
prev time.Duration
|
||||||
|
want time.Duration
|
||||||
|
}{
|
||||||
|
{name: "initial", prev: 0, want: time.Second},
|
||||||
|
{name: "double", prev: 2 * time.Second, want: 4 * time.Second},
|
||||||
|
{name: "cap", prev: 20 * time.Second, want: 30 * time.Second},
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, tt := range tests {
|
||||||
|
t.Run(tt.name, func(t *testing.T) {
|
||||||
|
if got := nextPollRetryDelay(tt.prev); got != tt.want {
|
||||||
|
t.Fatalf("nextPollRetryDelay(%s) = %s, want %s", tt.prev, got, tt.want)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestAddDatabaseLoggerWriterSkipsWhenDBContextIsUnset(t *testing.T) {
|
||||||
|
bot := &Bot[NoDB]{logger: slog.CreateLogger()}
|
||||||
|
called := false
|
||||||
|
|
||||||
|
bot.AddDatabaseLoggerWriter(func(db NoDB) slog.LoggerWriter {
|
||||||
|
called = true
|
||||||
|
return nil
|
||||||
|
})
|
||||||
|
|
||||||
|
if called {
|
||||||
|
t.Fatal("expected database logger writer to be skipped when db context is unset")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestAddDatabaseLoggerWriterSkipsWhenDBContextIsNil(t *testing.T) {
|
||||||
|
type testDB struct{}
|
||||||
|
|
||||||
|
bot := &Bot[*testDB]{logger: slog.CreateLogger()}
|
||||||
|
var db *testDB
|
||||||
|
bot.DatabaseContext(db)
|
||||||
|
|
||||||
|
called := false
|
||||||
|
bot.AddDatabaseLoggerWriter(func(db *testDB) slog.LoggerWriter {
|
||||||
|
called = true
|
||||||
|
return nil
|
||||||
|
})
|
||||||
|
|
||||||
|
if called {
|
||||||
|
t.Fatal("expected database logger writer to be skipped when db context is nil")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestShouldWarnOnValueDBContext(t *testing.T) {
|
||||||
|
type testDB struct{}
|
||||||
|
type dbIface interface{ Ping() error }
|
||||||
|
|
||||||
|
tests := []struct {
|
||||||
|
name string
|
||||||
|
got bool
|
||||||
|
want bool
|
||||||
|
}{
|
||||||
|
{name: "NoDB", got: shouldWarnOnValueDBContext[NoDB](), want: false},
|
||||||
|
{name: "pointer", got: shouldWarnOnValueDBContext[*testDB](), want: false},
|
||||||
|
{name: "interface", got: shouldWarnOnValueDBContext[dbIface](), want: false},
|
||||||
|
{name: "map", got: shouldWarnOnValueDBContext[map[string]int](), want: false},
|
||||||
|
{name: "struct", got: shouldWarnOnValueDBContext[testDB](), want: true},
|
||||||
|
{name: "int", got: shouldWarnOnValueDBContext[int](), want: true},
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, tt := range tests {
|
||||||
|
t.Run(tt.name, func(t *testing.T) {
|
||||||
|
if tt.got != tt.want {
|
||||||
|
t.Fatalf("shouldWarnOnValueDBContext = %v, want %v", tt.got, tt.want)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestDatabaseContextMarksValueWarningOnce(t *testing.T) {
|
||||||
|
type testDB struct{}
|
||||||
|
|
||||||
|
bot := &Bot[testDB]{logger: slog.CreateLogger()}
|
||||||
|
bot.DatabaseContext(testDB{})
|
||||||
|
if !bot.warnedValueDB {
|
||||||
|
t.Fatal("expected value-typed database context to mark warning state")
|
||||||
|
}
|
||||||
|
|
||||||
|
ptrBot := &Bot[*testDB]{logger: slog.CreateLogger()}
|
||||||
|
ptrBot.DatabaseContext(&testDB{})
|
||||||
|
if ptrBot.warnedValueDB {
|
||||||
|
t.Fatal("did not expect pointer-typed database context to mark warning state")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestRunWithContextRejectsSecondRun(t *testing.T) {
|
||||||
|
ctx, cancel := context.WithCancel(context.Background())
|
||||||
|
cancel()
|
||||||
|
|
||||||
|
bot := &Bot[NoDB]{
|
||||||
|
logger: slog.CreateLogger(),
|
||||||
|
prefixes: []string{"/"},
|
||||||
|
plugins: []Plugin[NoDB]{{name: "demo"}},
|
||||||
|
updateQueue: make(chan *tgapi.Update, 1),
|
||||||
|
maxWorkers: 1,
|
||||||
|
}
|
||||||
|
|
||||||
|
if err := bot.RunWithContext(ctx); err != nil {
|
||||||
|
t.Fatalf("first RunWithContext returned error: %v", err)
|
||||||
|
}
|
||||||
|
if err := bot.RunWithContext(ctx); !errors.Is(err, ErrBotAlreadyRun) {
|
||||||
|
t.Fatalf("expected ErrBotAlreadyRun on second run, got %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
+14
-8
@@ -4,13 +4,14 @@ import (
|
|||||||
"errors"
|
"errors"
|
||||||
"fmt"
|
"fmt"
|
||||||
"regexp"
|
"regexp"
|
||||||
|
"sort"
|
||||||
"strings"
|
"strings"
|
||||||
|
|
||||||
"git.nix13.pw/scuroneko/laniakea/tgapi"
|
"git.nix13.pw/scuroneko/laniakea/tgapi"
|
||||||
)
|
)
|
||||||
|
|
||||||
// CmdRegexp matches command names allowed for Telegram command registration.
|
// CmdRegexp matches command names allowed for Telegram command registration.
|
||||||
var CmdRegexp = regexp.MustCompile("^[a-zA-Z0-9]+$")
|
var CmdRegexp = regexp.MustCompile("^[_a-z0-9]{1,32}$")
|
||||||
|
|
||||||
// ErrTooManyCommands is returned when the total number of registered commands
|
// ErrTooManyCommands is returned when the total number of registered commands
|
||||||
// exceeds Telegram's limit of 100 bot commands per bot.
|
// exceeds Telegram's limit of 100 bot commands per bot.
|
||||||
@@ -20,7 +21,7 @@ var CmdRegexp = regexp.MustCompile("^[a-zA-Z0-9]+$")
|
|||||||
// bot initialization.
|
// bot initialization.
|
||||||
var ErrTooManyCommands = errors.New("too many commands. max 100")
|
var ErrTooManyCommands = errors.New("too many commands. max 100")
|
||||||
|
|
||||||
// generateBotCommand builds a BotCommand description with generated usage text.
|
// Internal helper to build a BotCommand description with generated usage text.
|
||||||
func generateBotCommand[T any](cmd *Command[T]) tgapi.BotCommand {
|
func generateBotCommand[T any](cmd *Command[T]) tgapi.BotCommand {
|
||||||
desc := ""
|
desc := ""
|
||||||
if len(cmd.description) > 0 {
|
if len(cmd.description) > 0 {
|
||||||
@@ -44,13 +45,20 @@ func generateBotCommand[T any](cmd *Command[T]) tgapi.BotCommand {
|
|||||||
return tgapi.BotCommand{Command: cmd.command, Description: usage}
|
return tgapi.BotCommand{Command: cmd.command, Description: usage}
|
||||||
}
|
}
|
||||||
|
|
||||||
// checkCmdRegex reports whether cmd matches CmdRegexp.
|
// Internal helper to validate Telegram command names.
|
||||||
func checkCmdRegex(cmd string) bool { return CmdRegexp.MatchString(cmd) }
|
func checkCmdRegex(cmd string) bool { return CmdRegexp.MatchString(cmd) }
|
||||||
|
|
||||||
// gatherCommandsForPlugin collects non-skipped, valid commands from one plugin.
|
// Internal helper to collect non-skipped, valid commands from one plugin.
|
||||||
func gatherCommandsForPlugin[T any](pl Plugin[T]) []tgapi.BotCommand {
|
func gatherCommandsForPlugin[T any](pl Plugin[T]) []tgapi.BotCommand {
|
||||||
commands := make([]tgapi.BotCommand, 0)
|
commands := make([]tgapi.BotCommand, 0)
|
||||||
for _, cmd := range pl.commands {
|
names := make([]string, 0, len(pl.commands))
|
||||||
|
for name := range pl.commands {
|
||||||
|
names = append(names, name)
|
||||||
|
}
|
||||||
|
sort.Strings(names)
|
||||||
|
|
||||||
|
for _, name := range names {
|
||||||
|
cmd := pl.commands[name]
|
||||||
if cmd.skipAutoCmd {
|
if cmd.skipAutoCmd {
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
@@ -62,9 +70,7 @@ func gatherCommandsForPlugin[T any](pl Plugin[T]) []tgapi.BotCommand {
|
|||||||
return commands
|
return commands
|
||||||
}
|
}
|
||||||
|
|
||||||
// gatherCommands collects all commands from all plugins
|
// Internal helper to collect all auto-generated commands from registered plugins.
|
||||||
// and converts them into tgapi.BotCommand objects.
|
|
||||||
// See gatherCommandsForPlugin.
|
|
||||||
func gatherCommands[T any](bot *Bot[T]) []tgapi.BotCommand {
|
func gatherCommands[T any](bot *Bot[T]) []tgapi.BotCommand {
|
||||||
commands := make([]tgapi.BotCommand, 0)
|
commands := make([]tgapi.BotCommand, 0)
|
||||||
for _, pl := range bot.plugins {
|
for _, pl := range bot.plugins {
|
||||||
|
|||||||
+22
-1
@@ -4,6 +4,7 @@ import (
|
|||||||
"errors"
|
"errors"
|
||||||
"io"
|
"io"
|
||||||
"net/http"
|
"net/http"
|
||||||
|
"reflect"
|
||||||
"strconv"
|
"strconv"
|
||||||
"strings"
|
"strings"
|
||||||
"sync/atomic"
|
"sync/atomic"
|
||||||
@@ -43,7 +44,7 @@ func TestAutoGenerateCommandsChecksLimitBeforeDelete(t *testing.T) {
|
|||||||
}()
|
}()
|
||||||
|
|
||||||
plugin := NewPlugin[NoDB]("overflow")
|
plugin := NewPlugin[NoDB]("overflow")
|
||||||
exec := func(ctx *MsgContext, db *NoDB) {}
|
exec := func(ctx *MsgContext, db NoDB) {}
|
||||||
for i := 0; i < 101; i++ {
|
for i := 0; i < 101; i++ {
|
||||||
plugin.AddCommand(NewCommand(exec, "cmd"+strconv.Itoa(i)))
|
plugin.AddCommand(NewCommand(exec, "cmd"+strconv.Itoa(i)))
|
||||||
}
|
}
|
||||||
@@ -62,3 +63,23 @@ func TestAutoGenerateCommandsChecksLimitBeforeDelete(t *testing.T) {
|
|||||||
t.Fatalf("expected no HTTP calls before limit validation, got %d", calls.Load())
|
t.Fatalf("expected no HTTP calls before limit validation, got %d", calls.Load())
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestGatherCommandsForPluginReturnsSortedCommands(t *testing.T) {
|
||||||
|
plugin := NewPlugin[NoDB]("sorted")
|
||||||
|
exec := func(ctx *MsgContext, db NoDB) {}
|
||||||
|
|
||||||
|
plugin.AddCommand(NewCommand(exec, "zeta"))
|
||||||
|
plugin.AddCommand(NewCommand(exec, "alpha"))
|
||||||
|
plugin.AddCommand(NewCommand(exec, "mid"))
|
||||||
|
|
||||||
|
commands := gatherCommandsForPlugin(*plugin)
|
||||||
|
got := make([]string, 0, len(commands))
|
||||||
|
for _, cmd := range commands {
|
||||||
|
got = append(got, cmd.Command)
|
||||||
|
}
|
||||||
|
|
||||||
|
want := []string{"alpha", "mid", "zeta"}
|
||||||
|
if !reflect.DeepEqual(got, want) {
|
||||||
|
t.Fatalf("unexpected command order: got %v want %v", got, want)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
@@ -1,55 +1,33 @@
|
|||||||
/*
|
/*
|
||||||
Package laniakea provides a modular, extensible framework for building scalable Telegram bots.
|
Package laniakea provides a modular, extensible framework for building scalable Telegram bots.
|
||||||
|
|
||||||
It offers a fluent API for configuration and separates concerns through several core concepts:
|
Core concepts:
|
||||||
|
|
||||||
- Bot: The central instance managing API communication, update processing, logging,
|
- Bot manages Telegram API access, update processing, logging, rate limiting, and dependency injection.
|
||||||
rate limiting, and dependency injection. Created via NewBot[T].
|
- Plugins group commands, payloads, and non-command update handlers behind shared middleware.
|
||||||
|
- MsgContext provides access to the current update and reply/edit/delete helpers.
|
||||||
- Plugins: Organize commands and payloads into reusable units.
|
- InlineKeyboard builds callback-driven keyboards and structured payloads.
|
||||||
A plugin can have multiple commands and shared middlewares.
|
- DraftProvider accumulates multi-step replies before sending them.
|
||||||
|
- L10n stores key-based translations with fallback behavior.
|
||||||
- Commands: Named bot commands with descriptions, argument validation, and
|
- Runners execute startup or background tasks alongside the polling loop.
|
||||||
execution logic. Automatically registrable across different chat scopes.
|
|
||||||
|
|
||||||
- Middleware: Functions that intercept and modify updates before they reach plugins.
|
|
||||||
Useful for authentication, logging, validation, etc. Return false to stop processing.
|
|
||||||
|
|
||||||
- MsgContext: Provides access to the incoming update and convenient methods for
|
|
||||||
responding, editing, deleting, and translating messages. Includes built-in rate limiting
|
|
||||||
and error handling. ⚠️ MarkdownV2 methods require manual escaping via EscapeMarkdownV2().
|
|
||||||
|
|
||||||
- InlineKeyboard: A fluent builder for constructing inline keyboards with styled buttons,
|
|
||||||
icons, URLs, and structured callback data (JSON or Base64).
|
|
||||||
|
|
||||||
- DraftProvider: Manages ephemeral, multi-step message drafts with automatic ID generation
|
|
||||||
(random or linear). Drafts can be built incrementally and flushed atomically.
|
|
||||||
|
|
||||||
- L10n: Simple key-based localization system with fallback language support.
|
|
||||||
|
|
||||||
- Runners: Background goroutines for periodic tasks or one‑off initialization,
|
|
||||||
with configurable timeouts and async execution.
|
|
||||||
|
|
||||||
- RateLimiting & Logging: Built‑in rate limiter (respects Telegram's retry_after)
|
|
||||||
and structured logging (JSON stdout + optional file output) with request‑level tracing.
|
|
||||||
|
|
||||||
- Dependency Injection: Pass any custom database context (e.g., *sql.DB) to all handlers
|
|
||||||
via the type parameter T in Bot[T].
|
|
||||||
|
|
||||||
Example usage:
|
Example usage:
|
||||||
|
|
||||||
bot := laniakea.NewBot[mydb.DBContext](laniakea.LoadOptsFromEnv()).
|
bot, err := laniakea.NewBot[*mydb.DBContext](laniakea.LoadOptsFromEnv())
|
||||||
DatabaseContext(&myDB).
|
if err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
bot.DatabaseContext(myDB).
|
||||||
AddUpdateType(tgapi.UpdateTypeMessage).
|
AddUpdateType(tgapi.UpdateTypeMessage).
|
||||||
AddPrefixes("/", "!").
|
AddPrefixes("/", "!").
|
||||||
AddPlugins(&startPlugin, &helpPlugin).
|
AddPlugins(&startPlugin, &helpPlugin).
|
||||||
AddMiddleware(&authMiddleware, &logMiddleware).
|
AddMiddleware(authMiddleware, logMiddleware).
|
||||||
AddRunner(&cleanupRunner).
|
AddRunner(cleanupRunner).
|
||||||
AddL10n(l10n.New())
|
AddL10n(l10n.New())
|
||||||
|
|
||||||
bot.Run()
|
return bot.Run()
|
||||||
|
|
||||||
All public methods are safe for concurrent use unless stated otherwise.
|
Configure bots, plugins, and localization before starting Run or RunWithContext.
|
||||||
Direct field access is not recommended; use provided accessors (e.g., GetDBContext, SetUpdateOffset).
|
Runtime accessors are safe for concurrent use unless stated otherwise.
|
||||||
*/
|
*/
|
||||||
package laniakea
|
package laniakea
|
||||||
|
|||||||
@@ -12,7 +12,7 @@ import (
|
|||||||
// ErrDraftChatIDZero is returned when a draft is used without setting a chat ID.
|
// ErrDraftChatIDZero is returned when a draft is used without setting a chat ID.
|
||||||
var ErrDraftChatIDZero = errors.New("zero draft chat ID")
|
var ErrDraftChatIDZero = errors.New("zero draft chat ID")
|
||||||
|
|
||||||
// draftIdGenerator defines an interface for generating unique draft IDs.
|
// Interface for generating unique draft IDs.
|
||||||
type draftIdGenerator interface {
|
type draftIdGenerator interface {
|
||||||
// Next returns the next unique draft ID.
|
// Next returns the next unique draft ID.
|
||||||
Next() uint64
|
Next() uint64
|
||||||
@@ -38,12 +38,9 @@ func (g *LinearDraftIdGenerator) Next() uint64 {
|
|||||||
return g.lastId.Add(1)
|
return g.lastId.Add(1)
|
||||||
}
|
}
|
||||||
|
|
||||||
// DraftProvider manages a collection of Drafts and provides methods to create and
|
// DraftProvider manages a collection of Drafts and a shared draft ID generator.
|
||||||
// configure them. It holds shared configuration (chat, parse mode, entities) and
|
|
||||||
// a draft ID generator.
|
|
||||||
//
|
//
|
||||||
// DraftProvider is NOT thread-safe. Concurrent access from multiple goroutines
|
// DraftProvider is safe for concurrent use.
|
||||||
// requires external synchronization.
|
|
||||||
type DraftProvider struct {
|
type DraftProvider struct {
|
||||||
mu sync.RWMutex
|
mu sync.RWMutex
|
||||||
api *tgapi.API
|
api *tgapi.API
|
||||||
@@ -133,10 +130,7 @@ type Draft struct {
|
|||||||
|
|
||||||
// NewDraft creates a new draft with the provided parse mode.
|
// NewDraft creates a new draft with the provided parse mode.
|
||||||
//
|
//
|
||||||
// The draft inherits the provider's chatID, messageThreadID, and entities.
|
// The caller must set a chat with SetChat before Push or Flush.
|
||||||
// If parseMode is zero, the provider's default parseMode is used.
|
|
||||||
//
|
|
||||||
// Panics if chatID is zero — call SetChat() on the provider first.
|
|
||||||
func (p *DraftProvider) NewDraft(parseMode tgapi.ParseMode) *Draft {
|
func (p *DraftProvider) NewDraft(parseMode tgapi.ParseMode) *Draft {
|
||||||
id := p.generator.Next()
|
id := p.generator.Next()
|
||||||
draft := &Draft{
|
draft := &Draft{
|
||||||
@@ -224,6 +218,9 @@ func (d *Draft) Flush() error {
|
|||||||
if d.Message == "" {
|
if d.Message == "" {
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
if d.chatID == 0 {
|
||||||
|
return ErrDraftChatIDZero
|
||||||
|
}
|
||||||
|
|
||||||
params := tgapi.SendMessageP{
|
params := tgapi.SendMessageP{
|
||||||
ChatID: d.chatID,
|
ChatID: d.chatID,
|
||||||
@@ -242,7 +239,7 @@ func (d *Draft) Flush() error {
|
|||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
|
|
||||||
// push is the internal helper for Push(). It updates the server draft via SendMessageDraft.
|
// Internal helper for Push that updates the server-side draft.
|
||||||
func (d *Draft) push(text string) error {
|
func (d *Draft) push(text string) error {
|
||||||
if d.chatID == 0 {
|
if d.chatID == 0 {
|
||||||
return ErrDraftChatIDZero
|
return ErrDraftChatIDZero
|
||||||
|
|||||||
@@ -0,0 +1,36 @@
|
|||||||
|
package laniakea
|
||||||
|
|
||||||
|
import (
|
||||||
|
"testing"
|
||||||
|
|
||||||
|
"git.nix13.pw/scuroneko/laniakea/tgapi"
|
||||||
|
"git.nix13.pw/scuroneko/slog"
|
||||||
|
)
|
||||||
|
|
||||||
|
func TestDraftFlushRequiresChatID(t *testing.T) {
|
||||||
|
draft := NewRandomDraftProvider(&tgapi.API{}).NewDraft(tgapi.ParseNone)
|
||||||
|
draft.Message = "hello"
|
||||||
|
|
||||||
|
if err := draft.Flush(); err != ErrDraftChatIDZero {
|
||||||
|
t.Fatalf("expected ErrDraftChatIDZero, got %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestMsgContextNewDraftWorksWithoutLimiter(t *testing.T) {
|
||||||
|
ctx := &MsgContext{
|
||||||
|
Api: &tgapi.API{},
|
||||||
|
Msg: &tgapi.Message{
|
||||||
|
Chat: &tgapi.Chat{ID: 42, Type: string(tgapi.ChatTypePrivate)},
|
||||||
|
},
|
||||||
|
Logger: slog.CreateLogger(),
|
||||||
|
draftProvider: NewRandomDraftProvider(&tgapi.API{}),
|
||||||
|
}
|
||||||
|
|
||||||
|
draft := ctx.NewDraft()
|
||||||
|
if draft == nil {
|
||||||
|
t.Fatal("expected draft")
|
||||||
|
}
|
||||||
|
if draft.chatID != 42 {
|
||||||
|
t.Fatalf("unexpected chat id: %d", draft.chatID)
|
||||||
|
}
|
||||||
|
}
|
||||||
+158
-33
@@ -22,38 +22,47 @@ func (bot *Bot[T]) handle(u *tgapi.Update) {
|
|||||||
|
|
||||||
ctx := &MsgContext{
|
ctx := &MsgContext{
|
||||||
Update: *u, Api: bot.api,
|
Update: *u, Api: bot.api,
|
||||||
botLogger: bot.logger,
|
Logger: bot.logger,
|
||||||
errorTemplate: bot.errorTemplate,
|
errorTemplate: bot.errorTemplate,
|
||||||
l10n: bot.l10n,
|
l10n: bot.l10n,
|
||||||
draftProvider: bot.draftProvider,
|
draftProvider: bot.draftProvider,
|
||||||
payloadType: bot.payloadType,
|
payloadType: bot.payloadType,
|
||||||
}
|
}
|
||||||
|
bot.prepareUpdateCtx(u, ctx)
|
||||||
|
|
||||||
for _, middleware := range bot.middlewares {
|
for _, middleware := range bot.middlewares {
|
||||||
if !middleware.Execute(ctx, bot.dbContext) {
|
if !middleware.Execute(ctx, bot.dbContext) {
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if u.CallbackQuery != nil {
|
switch u.Type {
|
||||||
bot.handleCallback(u, ctx)
|
case tgapi.UpdateTypeMessage, tgapi.UpdateTypeChannelPost:
|
||||||
} else {
|
|
||||||
bot.handleMessage(u, ctx)
|
bot.handleMessage(u, ctx)
|
||||||
|
case tgapi.UpdateTypeCallbackQuery:
|
||||||
|
bot.handleCallback(u, ctx)
|
||||||
|
default:
|
||||||
|
bot.handleUpdate(u, ctx)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
func (bot *Bot[T]) handleMessage(update *tgapi.Update, ctx *MsgContext) {
|
func (bot *Bot[T]) handleMessage(update *tgapi.Update, ctx *MsgContext) {
|
||||||
if update.Message == nil {
|
var msg *tgapi.Message
|
||||||
return
|
if update.Message != nil {
|
||||||
}
|
msg = update.Message
|
||||||
if update.Message.From == nil {
|
} else if update.ChannelPost != nil {
|
||||||
|
msg = update.ChannelPost
|
||||||
|
} else {
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
var text string
|
var text string
|
||||||
if len(update.Message.Text) > 0 {
|
if len(msg.Text) > 0 {
|
||||||
text = update.Message.Text
|
text = msg.Text
|
||||||
|
} else if len(msg.Caption) > 0 {
|
||||||
|
text = msg.Caption
|
||||||
} else {
|
} else {
|
||||||
text = update.Message.Caption
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
text = strings.TrimSpace(text)
|
text = strings.TrimSpace(text)
|
||||||
@@ -61,10 +70,9 @@ func (bot *Bot[T]) handleMessage(update *tgapi.Update, ctx *MsgContext) {
|
|||||||
if !hasPrefix {
|
if !hasPrefix {
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
ctx.Prefix = prefix
|
ctx.Prefix = prefix
|
||||||
ctx.FromID = update.Message.From.ID
|
ctx.Update = *update
|
||||||
ctx.From = update.Message.From
|
|
||||||
ctx.Msg = update.Message
|
|
||||||
|
|
||||||
// Убираем префикс
|
// Убираем префикс
|
||||||
text = strings.TrimSpace(text[len(prefix):])
|
text = strings.TrimSpace(text[len(prefix):])
|
||||||
@@ -94,14 +102,13 @@ func (bot *Bot[T]) handleMessage(update *tgapi.Update, ctx *MsgContext) {
|
|||||||
if _, exists := plugin.commands[cmd]; exists {
|
if _, exists := plugin.commands[cmd]; exists {
|
||||||
ctx.Text = args
|
ctx.Text = args
|
||||||
ctx.Args = strings.Fields(args) // Убирает лишние пробелы
|
ctx.Args = strings.Fields(args) // Убирает лишние пробелы
|
||||||
|
|
||||||
|
if plugin.logger != nil {
|
||||||
|
ctx.Logger = plugin.logger
|
||||||
|
}
|
||||||
if !plugin.executeMiddlewares(ctx, bot.dbContext) {
|
if !plugin.executeMiddlewares(ctx, bot.dbContext) {
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
ctx.Logger = plugin.logger
|
|
||||||
if ctx.Logger == nil {
|
|
||||||
ctx.Logger = ctx.botLogger
|
|
||||||
}
|
|
||||||
plugin.executeCmd(cmd, ctx, bot.dbContext)
|
plugin.executeCmd(cmd, ctx, bot.dbContext)
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
@@ -115,16 +122,6 @@ func (bot *Bot[T]) handleCallback(update *tgapi.Update, ctx *MsgContext) {
|
|||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
ctx.FromID = update.CallbackQuery.From.ID
|
|
||||||
ctx.From = &update.CallbackQuery.From
|
|
||||||
if update.CallbackQuery.Message != nil {
|
|
||||||
ctx.Msg = update.CallbackQuery.Message
|
|
||||||
ctx.CallbackMsgId = update.CallbackQuery.Message.MessageID
|
|
||||||
}
|
|
||||||
if update.CallbackQuery.InlineMessageID != nil {
|
|
||||||
ctx.InlineMsgId = *update.CallbackQuery.InlineMessageID
|
|
||||||
}
|
|
||||||
ctx.CallbackQueryId = update.CallbackQuery.ID
|
|
||||||
ctx.Args = data.Args
|
ctx.Args = data.Args
|
||||||
|
|
||||||
for _, plugin := range bot.plugins {
|
for _, plugin := range bot.plugins {
|
||||||
@@ -133,18 +130,146 @@ func (bot *Bot[T]) handleCallback(update *tgapi.Update, ctx *MsgContext) {
|
|||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
|
|
||||||
if !plugin.executeMiddlewares(ctx, bot.dbContext) {
|
|
||||||
return
|
|
||||||
}
|
|
||||||
ctx.Logger = plugin.logger
|
ctx.Logger = plugin.logger
|
||||||
if ctx.Logger == nil {
|
if ctx.Logger == nil {
|
||||||
ctx.Logger = ctx.botLogger
|
ctx.Logger = bot.logger
|
||||||
|
}
|
||||||
|
if !plugin.executeMiddlewares(ctx, bot.dbContext) {
|
||||||
|
return
|
||||||
}
|
}
|
||||||
plugin.executePayload(data.Command, ctx, bot.dbContext)
|
plugin.executePayload(data.Command, ctx, bot.dbContext)
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func (bot *Bot[T]) handleUpdate(u *tgapi.Update, ctx *MsgContext) {
|
||||||
|
for _, plugin := range bot.plugins {
|
||||||
|
handler, ok := plugin.handlers[u.Type]
|
||||||
|
if !ok {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
|
||||||
|
pluginCtx := cloneMsgContext(ctx)
|
||||||
|
if plugin.logger != nil {
|
||||||
|
pluginCtx.Logger = plugin.logger
|
||||||
|
}
|
||||||
|
if !plugin.executeMiddlewares(pluginCtx, bot.dbContext) {
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
handler(pluginCtx, bot.dbContext)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func cloneMsgContext(src *MsgContext) *MsgContext {
|
||||||
|
cloned := *src
|
||||||
|
if src.Args != nil {
|
||||||
|
cloned.Args = append([]string(nil), src.Args...)
|
||||||
|
}
|
||||||
|
return &cloned
|
||||||
|
}
|
||||||
|
|
||||||
|
func (bot *Bot[T]) prepareUpdateCtx(u *tgapi.Update, ctx *MsgContext) {
|
||||||
|
var from *tgapi.User
|
||||||
|
switch u.Type {
|
||||||
|
case tgapi.UpdateTypeMessage:
|
||||||
|
if u.Message != nil {
|
||||||
|
ctx.Msg = u.Message
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeEditedMessage:
|
||||||
|
if u.EditedMessage != nil {
|
||||||
|
ctx.Msg = u.EditedMessage
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeChannelPost:
|
||||||
|
if u.ChannelPost != nil {
|
||||||
|
ctx.Msg = u.ChannelPost
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeEditedChannelPost:
|
||||||
|
if u.EditedChannelPost != nil {
|
||||||
|
ctx.Msg = u.EditedChannelPost
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeBusinessMessage:
|
||||||
|
if u.BusinessMessage != nil {
|
||||||
|
ctx.Msg = u.BusinessMessage
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeEditedBusinessMessage:
|
||||||
|
if u.EditedBusinessMessage != nil {
|
||||||
|
ctx.Msg = u.EditedBusinessMessage
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeInlineQuery:
|
||||||
|
if u.InlineQuery != nil {
|
||||||
|
from = &u.InlineQuery.From
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeChosenInlineResult:
|
||||||
|
if u.ChosenInlineResult != nil {
|
||||||
|
from = &u.ChosenInlineResult.From
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeCallbackQuery:
|
||||||
|
if u.CallbackQuery != nil {
|
||||||
|
if u.CallbackQuery.Message != nil {
|
||||||
|
ctx.Msg = u.CallbackQuery.Message
|
||||||
|
ctx.CallbackMsgId = u.CallbackQuery.Message.MessageID
|
||||||
|
}
|
||||||
|
if u.CallbackQuery.InlineMessageID != nil {
|
||||||
|
ctx.InlineMsgId = *u.CallbackQuery.InlineMessageID
|
||||||
|
}
|
||||||
|
ctx.CallbackQueryId = u.CallbackQuery.ID
|
||||||
|
from = &u.CallbackQuery.From
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeShippingQuery:
|
||||||
|
if u.ShippingQuery != nil {
|
||||||
|
from = &u.ShippingQuery.From
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypePreCheckoutQuery:
|
||||||
|
if u.PreCheckoutQuery != nil {
|
||||||
|
from = &u.PreCheckoutQuery.From
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypePurchasedPaidMedia:
|
||||||
|
if u.PurchasedPaidMedia != nil {
|
||||||
|
from = &u.PurchasedPaidMedia.From
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeMyChatMember:
|
||||||
|
if u.MyChatMember != nil {
|
||||||
|
from = &u.MyChatMember.From
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeChatMember:
|
||||||
|
if u.ChatMember != nil {
|
||||||
|
from = &u.ChatMember.From
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeChatJoinRequest:
|
||||||
|
if u.ChatJoinRequest != nil {
|
||||||
|
from = &u.ChatJoinRequest.From
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeBusinessConnection:
|
||||||
|
if u.BusinessConnection != nil {
|
||||||
|
from = &u.BusinessConnection.User
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypePollAnswer:
|
||||||
|
if u.PollAnswer != nil {
|
||||||
|
from = &u.PollAnswer.User
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeMessageReaction:
|
||||||
|
if u.MessageReaction != nil {
|
||||||
|
from = u.MessageReaction.User
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeChatBoost:
|
||||||
|
if u.ChatBoost != nil {
|
||||||
|
from = &u.ChatBoost.Boost.Source.User
|
||||||
|
}
|
||||||
|
case tgapi.UpdateTypeRemovedChatBoost:
|
||||||
|
if u.RemovedChatBoost != nil {
|
||||||
|
from = &u.RemovedChatBoost.Source.User
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if ctx.Msg != nil && from == nil {
|
||||||
|
from = ctx.Msg.From
|
||||||
|
}
|
||||||
|
if from != nil {
|
||||||
|
ctx.From = from
|
||||||
|
ctx.FromID = from.ID
|
||||||
|
}
|
||||||
|
ctx.Update = *u
|
||||||
|
}
|
||||||
|
|
||||||
func (bot *Bot[T]) checkPrefixes(text string) (string, bool) {
|
func (bot *Bot[T]) checkPrefixes(text string) (string, bool) {
|
||||||
for _, prefix := range bot.prefixes {
|
for _, prefix := range bot.prefixes {
|
||||||
if prefix == "" {
|
if prefix == "" {
|
||||||
|
|||||||
+225
-1
@@ -1,6 +1,11 @@
|
|||||||
package laniakea
|
package laniakea
|
||||||
|
|
||||||
import "testing"
|
import (
|
||||||
|
"testing"
|
||||||
|
|
||||||
|
"git.nix13.pw/scuroneko/laniakea/tgapi"
|
||||||
|
"git.nix13.pw/scuroneko/slog"
|
||||||
|
)
|
||||||
|
|
||||||
func TestCheckPrefixesSkipsEmptyPrefixes(t *testing.T) {
|
func TestCheckPrefixesSkipsEmptyPrefixes(t *testing.T) {
|
||||||
bot := &Bot[NoDB]{prefixes: []string{"", "/"}}
|
bot := &Bot[NoDB]{prefixes: []string{"", "/"}}
|
||||||
@@ -12,3 +17,222 @@ func TestCheckPrefixesSkipsEmptyPrefixes(t *testing.T) {
|
|||||||
t.Fatalf("unexpected prefix result: prefix=%q ok=%v", prefix, ok)
|
t.Fatalf("unexpected prefix result: prefix=%q ok=%v", prefix, ok)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestBotMiddlewareReceivesLogger(t *testing.T) {
|
||||||
|
logger := slog.CreateLogger()
|
||||||
|
called := false
|
||||||
|
|
||||||
|
bot := &Bot[NoDB]{
|
||||||
|
logger: logger,
|
||||||
|
middlewares: []Middleware[NoDB]{
|
||||||
|
NewMiddleware("logger-check", func(ctx *MsgContext, db NoDB) bool {
|
||||||
|
called = true
|
||||||
|
if ctx.Logger != logger {
|
||||||
|
t.Fatalf("expected bot logger in middleware context, got %#v", ctx.Logger)
|
||||||
|
}
|
||||||
|
return true
|
||||||
|
}),
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
bot.handle(&tgapi.Update{
|
||||||
|
UpdateID: 1,
|
||||||
|
Type: tgapi.UpdateTypePoll,
|
||||||
|
Poll: &tgapi.Poll{
|
||||||
|
ID: "poll",
|
||||||
|
Question: "question",
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
if !called {
|
||||||
|
t.Fatal("expected bot middleware to be called")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestAddUpdateHandlerRejectsReservedUpdateTypes(t *testing.T) {
|
||||||
|
plugin := NewPlugin[NoDB]("test")
|
||||||
|
handler := func(ctx *MsgContext, db NoDB) {}
|
||||||
|
|
||||||
|
for _, updateType := range []tgapi.UpdateType{
|
||||||
|
tgapi.UpdateTypeMessage,
|
||||||
|
tgapi.UpdateTypeChannelPost,
|
||||||
|
tgapi.UpdateTypeCallbackQuery,
|
||||||
|
} {
|
||||||
|
func() {
|
||||||
|
defer func() {
|
||||||
|
if r := recover(); r != nil {
|
||||||
|
t.Fatalf("AddUpdateHandler(%q) panicked: %v", updateType, r)
|
||||||
|
}
|
||||||
|
}()
|
||||||
|
plugin.AddUpdateHandler(updateType, handler)
|
||||||
|
}()
|
||||||
|
|
||||||
|
if _, ok := plugin.handlers[updateType]; ok {
|
||||||
|
t.Fatalf("reserved update type %q must not be registered", updateType)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestHandleUpdateHandlersPopulateFromContext(t *testing.T) {
|
||||||
|
tests := []struct {
|
||||||
|
name string
|
||||||
|
update *tgapi.Update
|
||||||
|
wantID int64
|
||||||
|
}{
|
||||||
|
{
|
||||||
|
name: "inline query",
|
||||||
|
update: &tgapi.Update{
|
||||||
|
UpdateID: 1,
|
||||||
|
Type: tgapi.UpdateTypeInlineQuery,
|
||||||
|
InlineQuery: &tgapi.InlineQuery{
|
||||||
|
ID: "iq",
|
||||||
|
From: tgapi.User{ID: 41},
|
||||||
|
Query: "ping",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
wantID: 41,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "chosen inline result",
|
||||||
|
update: &tgapi.Update{
|
||||||
|
UpdateID: 2,
|
||||||
|
Type: tgapi.UpdateTypeChosenInlineResult,
|
||||||
|
ChosenInlineResult: &tgapi.ChosenInlineResult{
|
||||||
|
ResultID: "res",
|
||||||
|
From: tgapi.User{ID: 77},
|
||||||
|
Query: "pong",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
wantID: 77,
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, tt := range tests {
|
||||||
|
t.Run(tt.name, func(t *testing.T) {
|
||||||
|
called := false
|
||||||
|
plugin := NewPlugin[NoDB]("test").AddUpdateHandler(tt.update.Type, func(ctx *MsgContext, db NoDB) {
|
||||||
|
called = true
|
||||||
|
if ctx.Update.UpdateID != tt.update.UpdateID {
|
||||||
|
t.Fatalf("unexpected update in context: got %d want %d", ctx.Update.UpdateID, tt.update.UpdateID)
|
||||||
|
}
|
||||||
|
if ctx.From == nil {
|
||||||
|
t.Fatal("expected ctx.From to be populated")
|
||||||
|
}
|
||||||
|
if ctx.FromID != tt.wantID {
|
||||||
|
t.Fatalf("unexpected FromID: got %d want %d", ctx.FromID, tt.wantID)
|
||||||
|
}
|
||||||
|
if ctx.From.ID != tt.wantID {
|
||||||
|
t.Fatalf("unexpected ctx.From.ID: got %d want %d", ctx.From.ID, tt.wantID)
|
||||||
|
}
|
||||||
|
if ctx.Msg != nil {
|
||||||
|
t.Fatalf("did not expect message context for %s", tt.name)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
|
||||||
|
bot := &Bot[NoDB]{
|
||||||
|
logger: slog.CreateLogger(),
|
||||||
|
plugins: []Plugin[NoDB]{clonePlugin(plugin)},
|
||||||
|
}
|
||||||
|
|
||||||
|
bot.handle(tt.update)
|
||||||
|
|
||||||
|
if !called {
|
||||||
|
t.Fatalf("expected update handler for %s to be called", tt.name)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestHandleUpdateHandlersReceiveIsolatedContexts(t *testing.T) {
|
||||||
|
firstCalled := false
|
||||||
|
secondCalled := false
|
||||||
|
|
||||||
|
first := NewPlugin[NoDB]("first").AddUpdateHandler(tgapi.UpdateTypeInlineQuery, func(ctx *MsgContext, db NoDB) {
|
||||||
|
firstCalled = true
|
||||||
|
if ctx.FromID != 41 {
|
||||||
|
t.Fatalf("unexpected FromID in first handler: got %d want 41", ctx.FromID)
|
||||||
|
}
|
||||||
|
ctx.From = nil
|
||||||
|
ctx.FromID = 999
|
||||||
|
ctx.Text = "mutated"
|
||||||
|
ctx.Args = []string{"mutated"}
|
||||||
|
})
|
||||||
|
second := NewPlugin[NoDB]("second").AddUpdateHandler(tgapi.UpdateTypeInlineQuery, func(ctx *MsgContext, db NoDB) {
|
||||||
|
secondCalled = true
|
||||||
|
if ctx.From == nil {
|
||||||
|
t.Fatal("expected ctx.From to remain populated for second handler")
|
||||||
|
}
|
||||||
|
if ctx.FromID != 41 {
|
||||||
|
t.Fatalf("unexpected FromID in second handler: got %d want 41", ctx.FromID)
|
||||||
|
}
|
||||||
|
if ctx.Text != "" {
|
||||||
|
t.Fatalf("unexpected leaked Text in second handler: %q", ctx.Text)
|
||||||
|
}
|
||||||
|
if len(ctx.Args) != 0 {
|
||||||
|
t.Fatalf("unexpected leaked Args in second handler: %v", ctx.Args)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
|
||||||
|
bot := &Bot[NoDB]{
|
||||||
|
logger: slog.CreateLogger(),
|
||||||
|
plugins: []Plugin[NoDB]{
|
||||||
|
clonePlugin(first),
|
||||||
|
clonePlugin(second),
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
bot.handle(&tgapi.Update{
|
||||||
|
UpdateID: 3,
|
||||||
|
Type: tgapi.UpdateTypeInlineQuery,
|
||||||
|
InlineQuery: &tgapi.InlineQuery{
|
||||||
|
ID: "iq",
|
||||||
|
From: tgapi.User{ID: 41},
|
||||||
|
Query: "ping",
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
if !firstCalled || !secondCalled {
|
||||||
|
t.Fatalf("expected both handlers to be called, got first=%v second=%v", firstCalled, secondCalled)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestHandleChannelPostCommandWithSenderChat(t *testing.T) {
|
||||||
|
called := false
|
||||||
|
plugin := NewPlugin[NoDB]("test")
|
||||||
|
plugin.NewCommand(func(ctx *MsgContext, db NoDB) {
|
||||||
|
called = true
|
||||||
|
if ctx.Msg == nil {
|
||||||
|
t.Fatal("expected message context")
|
||||||
|
}
|
||||||
|
if ctx.Msg.Chat == nil || ctx.Msg.Chat.ID != -1001 {
|
||||||
|
t.Fatalf("unexpected chat context: %#v", ctx.Msg.Chat)
|
||||||
|
}
|
||||||
|
if ctx.From != nil {
|
||||||
|
t.Fatalf("expected ctx.From to stay nil for sender_chat updates, got %#v", ctx.From)
|
||||||
|
}
|
||||||
|
if ctx.FromID != 0 {
|
||||||
|
t.Fatalf("expected zero FromID for sender_chat updates, got %d", ctx.FromID)
|
||||||
|
}
|
||||||
|
}, "ping")
|
||||||
|
|
||||||
|
bot := &Bot[NoDB]{
|
||||||
|
logger: slog.CreateLogger(),
|
||||||
|
prefixes: []string{"/"},
|
||||||
|
plugins: []Plugin[NoDB]{clonePlugin(plugin)},
|
||||||
|
}
|
||||||
|
|
||||||
|
bot.handle(&tgapi.Update{
|
||||||
|
UpdateID: 10,
|
||||||
|
Type: tgapi.UpdateTypeChannelPost,
|
||||||
|
ChannelPost: &tgapi.Message{
|
||||||
|
MessageID: 55,
|
||||||
|
Text: "/ping",
|
||||||
|
SenderChat: &tgapi.Chat{ID: -1001, Type: string(tgapi.ChatTypeChannel)},
|
||||||
|
Chat: &tgapi.Chat{ID: -1001, Type: string(tgapi.ChatTypeChannel)},
|
||||||
|
},
|
||||||
|
})
|
||||||
|
|
||||||
|
if !called {
|
||||||
|
t.Fatal("expected channel post command handler to be called")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
+13
-16
@@ -7,13 +7,12 @@ import (
|
|||||||
"git.nix13.pw/scuroneko/laniakea/tgapi"
|
"git.nix13.pw/scuroneko/laniakea/tgapi"
|
||||||
)
|
)
|
||||||
|
|
||||||
// ButtonStyleDanger, ButtonStyleSuccess, ButtonStylePrimary are predefined
|
|
||||||
// Telegram keyboard button styles for visual feedback.
|
|
||||||
//
|
|
||||||
// These values map directly to Telegram Bot API's InlineKeyboardButton style field.
|
|
||||||
const (
|
const (
|
||||||
ButtonStyleDanger tgapi.KeyboardButtonStyle = "danger"
|
// ButtonStyleDanger marks a destructive inline keyboard action.
|
||||||
|
ButtonStyleDanger tgapi.KeyboardButtonStyle = "danger"
|
||||||
|
// ButtonStyleSuccess marks a confirmatory inline keyboard action.
|
||||||
ButtonStyleSuccess tgapi.KeyboardButtonStyle = "success"
|
ButtonStyleSuccess tgapi.KeyboardButtonStyle = "success"
|
||||||
|
// ButtonStylePrimary marks a primary inline keyboard action.
|
||||||
ButtonStylePrimary tgapi.KeyboardButtonStyle = "primary"
|
ButtonStylePrimary tgapi.KeyboardButtonStyle = "primary"
|
||||||
)
|
)
|
||||||
|
|
||||||
@@ -83,8 +82,7 @@ func (b InlineKbButtonBuilder) SetCallbackDataBase64(cmd string, args ...any) In
|
|||||||
return b
|
return b
|
||||||
}
|
}
|
||||||
|
|
||||||
// build converts the builder state into a tgapi.InlineKeyboardButton.
|
// Internal helper that converts the builder state into a Telegram button.
|
||||||
// This method is typically called internally by InlineKeyboard.AddButton().
|
|
||||||
func (b InlineKbButtonBuilder) build() tgapi.InlineKeyboardButton {
|
func (b InlineKbButtonBuilder) build() tgapi.InlineKeyboardButton {
|
||||||
return tgapi.InlineKeyboardButton{
|
return tgapi.InlineKeyboardButton{
|
||||||
Text: b.text,
|
Text: b.text,
|
||||||
@@ -146,8 +144,7 @@ func (in *InlineKeyboard) SetPayloadType(t BotPayloadType) *InlineKeyboard {
|
|||||||
return in
|
return in
|
||||||
}
|
}
|
||||||
|
|
||||||
// append adds a button to the current line. If the line is full, it auto-flushes.
|
// Internal helper that appends a button and auto-flushes a full row.
|
||||||
// This is an internal helper used by other builder methods.
|
|
||||||
func (in *InlineKeyboard) append(button tgapi.InlineKeyboardButton) *InlineKeyboard {
|
func (in *InlineKeyboard) append(button tgapi.InlineKeyboardButton) *InlineKeyboard {
|
||||||
if in.CurrentLine.Len() == in.maxRow {
|
if in.CurrentLine.Len() == in.maxRow {
|
||||||
in.AddLine()
|
in.AddLine()
|
||||||
@@ -235,12 +232,12 @@ type CallbackData struct {
|
|||||||
// (int, string, bool, float64) but may not serialize complex structs meaningfully.
|
// (int, string, bool, float64) but may not serialize complex structs meaningfully.
|
||||||
//
|
//
|
||||||
// Use this to build callback payloads for bot command routing.
|
// Use this to build callback payloads for bot command routing.
|
||||||
func NewCallbackData(command string, args ...any) *CallbackData {
|
func NewCallbackData(command string, args ...any) CallbackData {
|
||||||
stringArgs := make([]string, len(args))
|
stringArgs := make([]string, len(args))
|
||||||
for i, arg := range args {
|
for i, arg := range args {
|
||||||
stringArgs[i] = fmt.Sprint(arg)
|
stringArgs[i] = fmt.Sprint(arg)
|
||||||
}
|
}
|
||||||
return &CallbackData{
|
return CallbackData{
|
||||||
Command: command,
|
Command: command,
|
||||||
Args: stringArgs,
|
Args: stringArgs,
|
||||||
}
|
}
|
||||||
@@ -253,8 +250,8 @@ func NewCallbackData(command string, args ...any) *CallbackData {
|
|||||||
//
|
//
|
||||||
// This fallback ensures the bot receives a valid JSON payload even if internal
|
// This fallback ensures the bot receives a valid JSON payload even if internal
|
||||||
// errors occur — avoiding "invalid callback_data" errors from Telegram.
|
// errors occur — avoiding "invalid callback_data" errors from Telegram.
|
||||||
func (d *CallbackData) ToJson() string {
|
func (d CallbackData) ToJson() string {
|
||||||
data, err := encodeJsonPayload(*d)
|
data, err := encodeJsonPayload(d)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
// Fallback: return minimal valid JSON to avoid Telegram API rejection
|
// Fallback: return minimal valid JSON to avoid Telegram API rejection
|
||||||
return `{"cmd":""}`
|
return `{"cmd":""}`
|
||||||
@@ -264,8 +261,8 @@ func (d *CallbackData) ToJson() string {
|
|||||||
|
|
||||||
// ToBase64 serializes the CallbackData to a JSON string and then encodes it as Base64.
|
// ToBase64 serializes the CallbackData to a JSON string and then encodes it as Base64.
|
||||||
// Returns an empty string if serialization or encoding fails.
|
// Returns an empty string if serialization or encoding fails.
|
||||||
func (d *CallbackData) ToBase64() string {
|
func (d CallbackData) ToBase64() string {
|
||||||
s, err := encodeBase64Payload(*d)
|
s, err := encodeBase64Payload(d)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return ``
|
return ``
|
||||||
}
|
}
|
||||||
@@ -275,7 +272,7 @@ func (d *CallbackData) ToBase64() string {
|
|||||||
// Encode serializes the CallbackData according to the specified payload type.
|
// Encode serializes the CallbackData according to the specified payload type.
|
||||||
// Supported types: BotPayloadJson and BotPayloadBase64.
|
// Supported types: BotPayloadJson and BotPayloadBase64.
|
||||||
// For unknown types, returns an empty string.
|
// For unknown types, returns an empty string.
|
||||||
func (d *CallbackData) Encode(t BotPayloadType) string {
|
func (d CallbackData) Encode(t BotPayloadType) string {
|
||||||
switch t {
|
switch t {
|
||||||
case BotPayloadBase64:
|
case BotPayloadBase64:
|
||||||
return d.ToBase64()
|
return d.ToBase64()
|
||||||
|
|||||||
@@ -0,0 +1,44 @@
|
|||||||
|
package laniakea
|
||||||
|
|
||||||
|
import (
|
||||||
|
"strings"
|
||||||
|
"testing"
|
||||||
|
)
|
||||||
|
|
||||||
|
func TestInlineKeyboardWrapsRowsAndEncodesJSONPayloads(t *testing.T) {
|
||||||
|
kb := NewInlineKeyboardJson(2).
|
||||||
|
AddCallbackButton("A", "cmd", 1).
|
||||||
|
AddCallbackButton("B", "cmd", 2).
|
||||||
|
AddCallbackButton("C", "cmd", 3)
|
||||||
|
|
||||||
|
markup := kb.Get()
|
||||||
|
if got := len(markup.InlineKeyboard); got != 2 {
|
||||||
|
t.Fatalf("unexpected row count: %d", got)
|
||||||
|
}
|
||||||
|
if got := len(markup.InlineKeyboard[0]); got != 2 {
|
||||||
|
t.Fatalf("unexpected first row size: %d", got)
|
||||||
|
}
|
||||||
|
if got := len(markup.InlineKeyboard[1]); got != 1 {
|
||||||
|
t.Fatalf("unexpected second row size: %d", got)
|
||||||
|
}
|
||||||
|
if !strings.Contains(markup.InlineKeyboard[0][0].CallbackData, `"cmd":"cmd"`) {
|
||||||
|
t.Fatalf("expected JSON callback payload, got %q", markup.InlineKeyboard[0][0].CallbackData)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestInlineKeyboardBuilderPreservesConfiguredButtonFields(t *testing.T) {
|
||||||
|
kb := NewInlineKeyboardBase64(3).
|
||||||
|
AddButton(
|
||||||
|
NewInlineKbButton("Docs").
|
||||||
|
SetStyle(ButtonStylePrimary).
|
||||||
|
SetUrl("https://example.test"),
|
||||||
|
)
|
||||||
|
|
||||||
|
button := kb.Get().InlineKeyboard[0][0]
|
||||||
|
if button.Style != ButtonStylePrimary {
|
||||||
|
t.Fatalf("unexpected style: %q", button.Style)
|
||||||
|
}
|
||||||
|
if button.URL != "https://example.test" {
|
||||||
|
t.Fatalf("unexpected url: %q", button.URL)
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -1,21 +1,18 @@
|
|||||||
package laniakea
|
package laniakea
|
||||||
|
|
||||||
// DictEntry represents a single localized entry with language-to-text mappings.
|
import "sync"
|
||||||
// Example: {"ru": "Привет", "en": "Hello"}.
|
|
||||||
|
// DictEntry maps language codes to translated strings.
|
||||||
type DictEntry map[string]string
|
type DictEntry map[string]string
|
||||||
|
|
||||||
// L10n is a localization manager that maps keys to language-specific strings.
|
// L10n stores translations with a configurable fallback language and is safe for concurrent use.
|
||||||
type L10n struct {
|
type L10n struct {
|
||||||
entries map[string]DictEntry // Map of translation keys to language dictionaries
|
mu sync.RWMutex
|
||||||
fallbackLang string // Language code to use when requested language is missing
|
entries map[string]DictEntry
|
||||||
|
fallbackLang string
|
||||||
}
|
}
|
||||||
|
|
||||||
// NewL10n creates a new L10n instance with the specified fallback language.
|
// NewL10n creates a localization store with the given fallback language.
|
||||||
// The fallback language is used when a requested language is not available
|
|
||||||
// for a given key.
|
|
||||||
//
|
|
||||||
// Example: NewL10n("en") will return "Hello" for key "greeting" if "ru" is requested
|
|
||||||
// but no "ru" entry exists.
|
|
||||||
func NewL10n(fallbackLanguage string) *L10n {
|
func NewL10n(fallbackLanguage string) *L10n {
|
||||||
return &L10n{
|
return &L10n{
|
||||||
entries: make(map[string]DictEntry),
|
entries: make(map[string]DictEntry),
|
||||||
@@ -23,54 +20,52 @@ func NewL10n(fallbackLanguage string) *L10n {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// AddDictEntry adds a new translation entry for the given key.
|
// AddDictEntry stores translations for key.
|
||||||
// The value must be a DictEntry mapping language codes (e.g., "en", "ru") to their translated strings.
|
|
||||||
//
|
|
||||||
// If a key already exists, it is overwritten.
|
|
||||||
//
|
|
||||||
// Returns the L10n instance for method chaining.
|
|
||||||
func (l *L10n) AddDictEntry(key string, value DictEntry) *L10n {
|
func (l *L10n) AddDictEntry(key string, value DictEntry) *L10n {
|
||||||
l.entries[key] = value
|
l.mu.Lock()
|
||||||
|
defer l.mu.Unlock()
|
||||||
|
if l.entries == nil {
|
||||||
|
l.entries = make(map[string]DictEntry)
|
||||||
|
}
|
||||||
|
l.entries[key] = cloneDictEntry(value)
|
||||||
return l
|
return l
|
||||||
}
|
}
|
||||||
|
|
||||||
// GetFallbackLanguage returns the currently configured fallback language code.
|
// GetFallbackLanguage returns the currently configured fallback language code.
|
||||||
func (l *L10n) GetFallbackLanguage() string {
|
func (l *L10n) GetFallbackLanguage() string {
|
||||||
|
l.mu.RLock()
|
||||||
|
defer l.mu.RUnlock()
|
||||||
return l.fallbackLang
|
return l.fallbackLang
|
||||||
}
|
}
|
||||||
|
|
||||||
// Translate retrieves the translation for the given key and language.
|
// Translate returns the translation for key in lang, falling back to the configured language or the key itself.
|
||||||
//
|
|
||||||
// Behavior:
|
|
||||||
// - If the key exists and the language has a translation → returns the translation
|
|
||||||
// - If the key exists but the language is missing → returns the fallback language's value
|
|
||||||
// - If the key does not exist → returns the key string itself (as fallback)
|
|
||||||
//
|
|
||||||
// Example:
|
|
||||||
//
|
|
||||||
// l.AddDictEntry("greeting", DictEntry{"en": "Hello", "ru": "Привет"})
|
|
||||||
// l.Translate("en", "greeting") → "Hello"
|
|
||||||
// l.Translate("es", "greeting") → "Hello" (fallback to "en")
|
|
||||||
// l.Translate("en", "unknown") → "unknown" (key not found)
|
|
||||||
//
|
|
||||||
// This behavior ensures that missing translations do not break UI or logs —
|
|
||||||
// instead, the original key is displayed, making it easy to identify gaps.
|
|
||||||
func (l *L10n) Translate(lang, key string) string {
|
func (l *L10n) Translate(lang, key string) string {
|
||||||
|
l.mu.RLock()
|
||||||
|
defer l.mu.RUnlock()
|
||||||
|
|
||||||
entries, exists := l.entries[key]
|
entries, exists := l.entries[key]
|
||||||
if !exists {
|
if !exists {
|
||||||
return key // Return key as fallback when translation is missing
|
return key
|
||||||
}
|
}
|
||||||
|
|
||||||
// Try requested language
|
|
||||||
if translation, ok := entries[lang]; ok {
|
if translation, ok := entries[lang]; ok {
|
||||||
return translation
|
return translation
|
||||||
}
|
}
|
||||||
|
|
||||||
// Fall back to configured fallback language
|
|
||||||
if fallback, ok := entries[l.fallbackLang]; ok {
|
if fallback, ok := entries[l.fallbackLang]; ok {
|
||||||
return fallback
|
return fallback
|
||||||
}
|
}
|
||||||
|
|
||||||
// If fallback language is also missing, return the key
|
|
||||||
return key
|
return key
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func cloneDictEntry(src DictEntry) DictEntry {
|
||||||
|
if src == nil {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
cloned := make(DictEntry, len(src))
|
||||||
|
for lang, text := range src {
|
||||||
|
cloned[lang] = text
|
||||||
|
}
|
||||||
|
return cloned
|
||||||
|
}
|
||||||
|
|||||||
@@ -0,0 +1,77 @@
|
|||||||
|
package laniakea
|
||||||
|
|
||||||
|
import (
|
||||||
|
"fmt"
|
||||||
|
"sync"
|
||||||
|
"testing"
|
||||||
|
)
|
||||||
|
|
||||||
|
func TestL10nTranslateUsesFallbackAndKey(t *testing.T) {
|
||||||
|
l10n := NewL10n("en").
|
||||||
|
AddDictEntry("greeting", DictEntry{"en": "Hello", "ru": "Privet"}).
|
||||||
|
AddDictEntry("partial", DictEntry{"ru": "Tolko ru"})
|
||||||
|
|
||||||
|
tests := []struct {
|
||||||
|
name string
|
||||||
|
lang string
|
||||||
|
key string
|
||||||
|
want string
|
||||||
|
}{
|
||||||
|
{name: "exact match", lang: "ru", key: "greeting", want: "Privet"},
|
||||||
|
{name: "fallback language", lang: "es", key: "greeting", want: "Hello"},
|
||||||
|
{name: "missing fallback returns key", lang: "en", key: "partial", want: "partial"},
|
||||||
|
{name: "unknown key returns key", lang: "en", key: "unknown", want: "unknown"},
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, tt := range tests {
|
||||||
|
t.Run(tt.name, func(t *testing.T) {
|
||||||
|
if got := l10n.Translate(tt.lang, tt.key); got != tt.want {
|
||||||
|
t.Fatalf("unexpected translation: got %q want %q", got, tt.want)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestL10nAddDictEntryCopiesInput(t *testing.T) {
|
||||||
|
l10n := NewL10n("en")
|
||||||
|
entry := DictEntry{"en": "Hello"}
|
||||||
|
|
||||||
|
l10n.AddDictEntry("greeting", entry)
|
||||||
|
entry["en"] = "Mutated"
|
||||||
|
|
||||||
|
if got := l10n.Translate("en", "greeting"); got != "Hello" {
|
||||||
|
t.Fatalf("unexpected translation after external mutation: got %q", got)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestL10nZeroValueIsUsable(t *testing.T) {
|
||||||
|
var l10n L10n
|
||||||
|
|
||||||
|
l10n.AddDictEntry("greeting", DictEntry{"en": "Hello"})
|
||||||
|
|
||||||
|
if got := l10n.Translate("en", "greeting"); got != "Hello" {
|
||||||
|
t.Fatalf("unexpected translation from zero-value l10n: got %q", got)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestL10nConcurrentAccess(t *testing.T) {
|
||||||
|
l10n := NewL10n("en")
|
||||||
|
l10n.AddDictEntry("base", DictEntry{"en": "Hello"})
|
||||||
|
|
||||||
|
var wg sync.WaitGroup
|
||||||
|
for i := 0; i < 8; i++ {
|
||||||
|
wg.Add(1)
|
||||||
|
go func(i int) {
|
||||||
|
defer wg.Done()
|
||||||
|
for j := 0; j < 100; j++ {
|
||||||
|
l10n.AddDictEntry(fmt.Sprintf("key-%d-%d", i, j), DictEntry{"en": "value"})
|
||||||
|
_ = l10n.Translate("en", "base")
|
||||||
|
}
|
||||||
|
}(i)
|
||||||
|
}
|
||||||
|
wg.Wait()
|
||||||
|
|
||||||
|
if got := l10n.Translate("en", "base"); got != "Hello" {
|
||||||
|
t.Fatalf("unexpected translation after concurrent access: got %q", got)
|
||||||
|
}
|
||||||
|
}
|
||||||
+41
-40
@@ -32,7 +32,6 @@ type MsgContext struct {
|
|||||||
Args []string
|
Args []string
|
||||||
|
|
||||||
errorTemplate string
|
errorTemplate string
|
||||||
botLogger *slog.Logger
|
|
||||||
l10n *L10n
|
l10n *L10n
|
||||||
draftProvider *DraftProvider
|
draftProvider *DraftProvider
|
||||||
payloadType BotPayloadType
|
payloadType BotPayloadType
|
||||||
@@ -47,8 +46,7 @@ type AnswerMessage struct {
|
|||||||
ctx *MsgContext // internal back-reference
|
ctx *MsgContext // internal back-reference
|
||||||
}
|
}
|
||||||
|
|
||||||
// edit is an internal helper to edit a message's text with optional keyboard and parse mode.
|
// Internal helper for text edits with optional keyboard and parse mode.
|
||||||
// Used by Edit, EditMarkdown, EditCallback, etc.
|
|
||||||
func (ctx *MsgContext) edit(messageId int, text string, keyboard *InlineKeyboard, parseMode tgapi.ParseMode) *AnswerMessage {
|
func (ctx *MsgContext) edit(messageId int, text string, keyboard *InlineKeyboard, parseMode tgapi.ParseMode) *AnswerMessage {
|
||||||
params := tgapi.EditMessageTextP{
|
params := tgapi.EditMessageTextP{
|
||||||
Text: text,
|
Text: text,
|
||||||
@@ -61,7 +59,7 @@ func (ctx *MsgContext) edit(messageId int, text string, keyboard *InlineKeyboard
|
|||||||
case ctx.InlineMsgId != "":
|
case ctx.InlineMsgId != "":
|
||||||
params.InlineMessageID = ctx.InlineMsgId
|
params.InlineMessageID = ctx.InlineMsgId
|
||||||
default:
|
default:
|
||||||
ctx.botLogger.Errorln("Can't edit message: no valid message target")
|
ctx.Logger.Errorln("Can't edit message: no valid message target")
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
if keyboard != nil {
|
if keyboard != nil {
|
||||||
@@ -69,7 +67,7 @@ func (ctx *MsgContext) edit(messageId int, text string, keyboard *InlineKeyboard
|
|||||||
}
|
}
|
||||||
msg, _, err := ctx.Api.EditMessageText(params)
|
msg, _, err := ctx.Api.EditMessageText(params)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
ctx.botLogger.Errorln(err)
|
ctx.Logger.Errorln(err)
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
resultMessageID := messageId
|
resultMessageID := messageId
|
||||||
@@ -95,11 +93,10 @@ func (m *AnswerMessage) EditMarkdown(text string) *AnswerMessage {
|
|||||||
return m.ctx.edit(m.MessageID, text, nil, tgapi.ParseMDV2)
|
return m.ctx.edit(m.MessageID, text, nil, tgapi.ParseMDV2)
|
||||||
}
|
}
|
||||||
|
|
||||||
// editCallback is an internal helper to edit the message associated with a callback query.
|
// Internal helper for editing callback-linked messages.
|
||||||
// Supports both regular callback messages and inline callback messages.
|
|
||||||
func (ctx *MsgContext) editCallback(text string, keyboard *InlineKeyboard, parseMode tgapi.ParseMode) *AnswerMessage {
|
func (ctx *MsgContext) editCallback(text string, keyboard *InlineKeyboard, parseMode tgapi.ParseMode) *AnswerMessage {
|
||||||
if ctx.CallbackMsgId == 0 && ctx.InlineMsgId == "" {
|
if ctx.CallbackMsgId == 0 && ctx.InlineMsgId == "" {
|
||||||
ctx.botLogger.Errorln("Can't edit non-callback update message")
|
ctx.Logger.Errorln("Can't edit non-callback update message")
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
return ctx.edit(ctx.CallbackMsgId, text, keyboard, parseMode)
|
return ctx.edit(ctx.CallbackMsgId, text, keyboard, parseMode)
|
||||||
@@ -129,8 +126,7 @@ func (ctx *MsgContext) EditCallbackfMarkdown(format string, keyboard *InlineKeyb
|
|||||||
return ctx.editCallback(fmt.Sprintf(format, args...), keyboard, tgapi.ParseMDV2)
|
return ctx.editCallback(fmt.Sprintf(format, args...), keyboard, tgapi.ParseMDV2)
|
||||||
}
|
}
|
||||||
|
|
||||||
// editPhotoText edits the caption of a photo/video message.
|
// Internal helper for media-caption edits.
|
||||||
// Returns nil when no valid edit target is available for the current context.
|
|
||||||
func (ctx *MsgContext) editPhotoText(messageId int, text string, kb *InlineKeyboard, parseMode tgapi.ParseMode) *AnswerMessage {
|
func (ctx *MsgContext) editPhotoText(messageId int, text string, kb *InlineKeyboard, parseMode tgapi.ParseMode) *AnswerMessage {
|
||||||
params := tgapi.EditMessageCaptionP{
|
params := tgapi.EditMessageCaptionP{
|
||||||
Caption: text,
|
Caption: text,
|
||||||
@@ -143,7 +139,7 @@ func (ctx *MsgContext) editPhotoText(messageId int, text string, kb *InlineKeybo
|
|||||||
case ctx.InlineMsgId != "":
|
case ctx.InlineMsgId != "":
|
||||||
params.InlineMessageID = ctx.InlineMsgId
|
params.InlineMessageID = ctx.InlineMsgId
|
||||||
default:
|
default:
|
||||||
ctx.botLogger.Errorln("Can't edit caption: no valid message target")
|
ctx.Logger.Errorln("Can't edit caption: no valid message target")
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
if kb != nil {
|
if kb != nil {
|
||||||
@@ -152,7 +148,7 @@ func (ctx *MsgContext) editPhotoText(messageId int, text string, kb *InlineKeybo
|
|||||||
|
|
||||||
msg, _, err := ctx.Api.EditMessageCaption(params)
|
msg, _, err := ctx.Api.EditMessageCaption(params)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
ctx.botLogger.Errorln(err)
|
ctx.Logger.Errorln(err)
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
resultMessageID := messageId
|
resultMessageID := messageId
|
||||||
@@ -188,11 +184,10 @@ func (m *AnswerMessage) EditCaptionKeyboardMarkdown(text string, kb *InlineKeybo
|
|||||||
return m.ctx.editPhotoText(m.MessageID, text, kb, tgapi.ParseMDV2)
|
return m.ctx.editPhotoText(m.MessageID, text, kb, tgapi.ParseMDV2)
|
||||||
}
|
}
|
||||||
|
|
||||||
// answer sends a new message with optional keyboard and parse mode.
|
// Internal helper for message replies with optional keyboard and parse mode.
|
||||||
// Uses API limiter to respect Telegram rate limits per chat.
|
|
||||||
func (ctx *MsgContext) answer(text string, keyboard *InlineKeyboard, parseMode tgapi.ParseMode) *AnswerMessage {
|
func (ctx *MsgContext) answer(text string, keyboard *InlineKeyboard, parseMode tgapi.ParseMode) *AnswerMessage {
|
||||||
if ctx.Msg == nil {
|
if ctx.Msg == nil {
|
||||||
ctx.botLogger.Errorln("Can't answer message without a message")
|
ctx.Logger.Errorln("Can't answer message without a message")
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
params := tgapi.SendMessageP{
|
params := tgapi.SendMessageP{
|
||||||
@@ -212,7 +207,7 @@ func (ctx *MsgContext) answer(text string, keyboard *InlineKeyboard, parseMode t
|
|||||||
|
|
||||||
msg, err := ctx.Api.SendMessage(params)
|
msg, err := ctx.Api.SendMessage(params)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
ctx.botLogger.Errorln(err)
|
ctx.Logger.Errorln(err)
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
return &AnswerMessage{
|
return &AnswerMessage{
|
||||||
@@ -256,10 +251,10 @@ func (ctx *MsgContext) KeyboardMarkdown(text string, keyboard *InlineKeyboard) *
|
|||||||
return ctx.answer(text, keyboard, tgapi.ParseMDV2)
|
return ctx.answer(text, keyboard, tgapi.ParseMDV2)
|
||||||
}
|
}
|
||||||
|
|
||||||
// answerPhoto sends a photo with optional caption and keyboard.
|
// Internal helper for photo replies with optional caption and keyboard.
|
||||||
func (ctx *MsgContext) answerPhoto(photoId, text string, kb *InlineKeyboard, parseMode tgapi.ParseMode) *AnswerMessage {
|
func (ctx *MsgContext) answerPhoto(photoId, text string, kb *InlineKeyboard, parseMode tgapi.ParseMode) *AnswerMessage {
|
||||||
if ctx.Msg == nil {
|
if ctx.Msg == nil {
|
||||||
ctx.botLogger.Errorln("Can't answer message without a message")
|
ctx.Logger.Errorln("Can't answer message without a message")
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
params := tgapi.SendPhotoP{
|
params := tgapi.SendPhotoP{
|
||||||
@@ -280,7 +275,7 @@ func (ctx *MsgContext) answerPhoto(photoId, text string, kb *InlineKeyboard, par
|
|||||||
|
|
||||||
msg, err := ctx.Api.SendPhoto(params)
|
msg, err := ctx.Api.SendPhoto(params)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
ctx.botLogger.Errorln(err)
|
ctx.Logger.Errorln(err)
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
return &AnswerMessage{
|
return &AnswerMessage{
|
||||||
@@ -324,14 +319,14 @@ func (ctx *MsgContext) AnswerPhotofMarkdown(photoId, template string, args ...an
|
|||||||
return ctx.answerPhoto(photoId, fmt.Sprintf(template, args...), nil, tgapi.ParseMDV2)
|
return ctx.answerPhoto(photoId, fmt.Sprintf(template, args...), nil, tgapi.ParseMDV2)
|
||||||
}
|
}
|
||||||
|
|
||||||
// delete removes a message by ID.
|
// Internal helper that deletes a message by ID.
|
||||||
func (ctx *MsgContext) delete(messageId int) {
|
func (ctx *MsgContext) delete(messageId int) {
|
||||||
if messageId == 0 {
|
if messageId == 0 {
|
||||||
ctx.botLogger.Errorln("Can't delete message: message ID zero")
|
ctx.Logger.Errorln("Can't delete message: message ID zero")
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
if ctx.Msg == nil {
|
if ctx.Msg == nil {
|
||||||
ctx.botLogger.Errorln("Can't delete message: no chat message context")
|
ctx.Logger.Errorln("Can't delete message: no chat message context")
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
_, err := ctx.Api.DeleteMessage(tgapi.DeleteMessageP{
|
_, err := ctx.Api.DeleteMessage(tgapi.DeleteMessageP{
|
||||||
@@ -339,7 +334,7 @@ func (ctx *MsgContext) delete(messageId int) {
|
|||||||
MessageID: messageId,
|
MessageID: messageId,
|
||||||
})
|
})
|
||||||
if err != nil {
|
if err != nil {
|
||||||
ctx.botLogger.Errorln(err)
|
ctx.Logger.Errorln(err)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -349,14 +344,13 @@ func (m *AnswerMessage) Delete() { m.ctx.delete(m.MessageID) }
|
|||||||
// CallbackDelete deletes the message that triggered the callback query.
|
// CallbackDelete deletes the message that triggered the callback query.
|
||||||
func (ctx *MsgContext) CallbackDelete() {
|
func (ctx *MsgContext) CallbackDelete() {
|
||||||
if ctx.CallbackMsgId == 0 {
|
if ctx.CallbackMsgId == 0 {
|
||||||
ctx.botLogger.Errorln("Can't delete callback message: no callback message ID")
|
ctx.Logger.Errorln("Can't delete callback message: no callback message ID")
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
ctx.delete(ctx.CallbackMsgId)
|
ctx.delete(ctx.CallbackMsgId)
|
||||||
}
|
}
|
||||||
|
|
||||||
// answerCallbackQuery sends a response to a callback query (optional text/alert/url).
|
// Internal helper that answers a callback query with optional text, alert, or URL.
|
||||||
// Does nothing if CallbackQueryId is empty.
|
|
||||||
func (ctx *MsgContext) answerCallbackQuery(url, text string, showAlert bool) {
|
func (ctx *MsgContext) answerCallbackQuery(url, text string, showAlert bool) {
|
||||||
if len(ctx.CallbackQueryId) == 0 {
|
if len(ctx.CallbackQueryId) == 0 {
|
||||||
return
|
return
|
||||||
@@ -366,7 +360,7 @@ func (ctx *MsgContext) answerCallbackQuery(url, text string, showAlert bool) {
|
|||||||
Text: text, ShowAlert: showAlert, URL: url,
|
Text: text, ShowAlert: showAlert, URL: url,
|
||||||
})
|
})
|
||||||
if err != nil {
|
if err != nil {
|
||||||
ctx.botLogger.Errorln(err)
|
ctx.Logger.Errorln(err)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -385,7 +379,7 @@ func (ctx *MsgContext) AnswerCbQueryUrl(u string) { ctx.answerCallbackQuery(u, "
|
|||||||
// SendAction sends a chat action (typing, uploading_photo, etc.) to indicate bot activity.
|
// SendAction sends a chat action (typing, uploading_photo, etc.) to indicate bot activity.
|
||||||
func (ctx *MsgContext) SendAction(action tgapi.ChatActionType) {
|
func (ctx *MsgContext) SendAction(action tgapi.ChatActionType) {
|
||||||
if ctx.Msg == nil {
|
if ctx.Msg == nil {
|
||||||
ctx.botLogger.Errorln("Can't send action without chat message context")
|
ctx.Logger.Errorln("Can't send action without chat message context")
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
params := tgapi.SendChatActionP{
|
params := tgapi.SendChatActionP{
|
||||||
@@ -396,14 +390,11 @@ func (ctx *MsgContext) SendAction(action tgapi.ChatActionType) {
|
|||||||
}
|
}
|
||||||
_, err := ctx.Api.SendChatAction(params)
|
_, err := ctx.Api.SendChatAction(params)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
ctx.botLogger.Errorln(err)
|
ctx.Logger.Errorln(err)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// error sends an error message to the user and logs it.
|
// Internal helper that formats, sends, and logs an error.
|
||||||
// Uses errorTemplate to format the message.
|
|
||||||
// For callbacks: sends as callback answer (no alert).
|
|
||||||
// For regular messages: sends as plain text.
|
|
||||||
func (ctx *MsgContext) error(err error) {
|
func (ctx *MsgContext) error(err error) {
|
||||||
text := fmt.Sprintf(ctx.errorTemplate, err.Error())
|
text := fmt.Sprintf(ctx.errorTemplate, err.Error())
|
||||||
|
|
||||||
@@ -412,7 +403,7 @@ func (ctx *MsgContext) error(err error) {
|
|||||||
} else {
|
} else {
|
||||||
ctx.answer(text, nil, tgapi.ParseNone)
|
ctx.answer(text, nil, tgapi.ParseNone)
|
||||||
}
|
}
|
||||||
ctx.botLogger.Errorln(err)
|
ctx.Logger.Errorln(err)
|
||||||
}
|
}
|
||||||
|
|
||||||
// Error is an alias for error().
|
// Error is an alias for error().
|
||||||
@@ -420,15 +411,25 @@ func (ctx *MsgContext) Error(err error) { ctx.error(err) }
|
|||||||
|
|
||||||
func (ctx *MsgContext) newDraft(parseMode tgapi.ParseMode) *Draft {
|
func (ctx *MsgContext) newDraft(parseMode tgapi.ParseMode) *Draft {
|
||||||
if ctx.Msg == nil {
|
if ctx.Msg == nil {
|
||||||
ctx.botLogger.Errorln("can't create draft: ctx.Msg is nil")
|
ctx.Logger.Errorln("can't create draft: ctx.Msg is nil")
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
if ctx.Api == nil {
|
||||||
|
ctx.Logger.Errorln("can't create draft: ctx.Api is nil")
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
if ctx.draftProvider == nil {
|
||||||
|
ctx.Logger.Errorln("can't create draft: ctx.draftProvider is nil")
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
c, cancel := context.WithTimeout(context.Background(), 5*time.Second)
|
if ctx.Api.Limiter != nil {
|
||||||
defer cancel()
|
c, cancel := context.WithTimeout(context.Background(), 5*time.Second)
|
||||||
if err := ctx.Api.Limiter.Wait(c, ctx.Msg.Chat.ID); err != nil {
|
defer cancel()
|
||||||
ctx.botLogger.Errorln(err)
|
if err := ctx.Api.Limiter.Wait(c, ctx.Msg.Chat.ID); err != nil {
|
||||||
return nil
|
ctx.Logger.Errorln(err)
|
||||||
|
return nil
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
draft := ctx.draftProvider.NewDraft(parseMode).SetChat(ctx.Msg.Chat.ID, ctx.Msg.MessageThreadID)
|
draft := ctx.draftProvider.NewDraft(parseMode).SetChat(ctx.Msg.Chat.ID, ctx.Msg.MessageThreadID)
|
||||||
|
|||||||
+1
-1
@@ -48,7 +48,7 @@ func TestAnswerPhotoIncludesDirectMessagesTopicID(t *testing.T) {
|
|||||||
Chat: &tgapi.Chat{ID: 42, Type: string(tgapi.ChatTypePrivate)},
|
Chat: &tgapi.Chat{ID: 42, Type: string(tgapi.ChatTypePrivate)},
|
||||||
DirectMessageTopic: &tgapi.DirectMessageTopic{TopicID: 77},
|
DirectMessageTopic: &tgapi.DirectMessageTopic{TopicID: 77},
|
||||||
},
|
},
|
||||||
botLogger: slog.CreateLogger(),
|
Logger: slog.CreateLogger(),
|
||||||
}
|
}
|
||||||
|
|
||||||
answer := ctx.AnswerPhoto("photo-id", "caption")
|
answer := ctx.AnswerPhoto("photo-id", "caption")
|
||||||
|
|||||||
+50
-34
@@ -5,10 +5,12 @@ import (
|
|||||||
"regexp"
|
"regexp"
|
||||||
|
|
||||||
"git.nix13.pw/scuroneko/extypes"
|
"git.nix13.pw/scuroneko/extypes"
|
||||||
|
"git.nix13.pw/scuroneko/laniakea/tgapi"
|
||||||
|
"git.nix13.pw/scuroneko/laniakea/utils"
|
||||||
"git.nix13.pw/scuroneko/slog"
|
"git.nix13.pw/scuroneko/slog"
|
||||||
)
|
)
|
||||||
|
|
||||||
// CommandValueType defines the expected type of a command argument.
|
// CommandValueType defines the expected type of command argument.
|
||||||
type CommandValueType string
|
type CommandValueType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
@@ -50,12 +52,12 @@ type CommandArg struct {
|
|||||||
// NewCommandArg creates a new CommandArg with the given text and type.
|
// NewCommandArg creates a new CommandArg with the given text and type.
|
||||||
// Uses a default regex based on the type (string or int).
|
// Uses a default regex based on the type (string or int).
|
||||||
// For CommandValueAnyType, no validation is performed.
|
// For CommandValueAnyType, no validation is performed.
|
||||||
func NewCommandArg(text string) *CommandArg {
|
func NewCommandArg(text string) CommandArg {
|
||||||
return &CommandArg{CommandValueAnyType, text, CommandRegexString, false}
|
return CommandArg{CommandValueAnyType, text, CommandRegexString, false}
|
||||||
}
|
}
|
||||||
|
|
||||||
// SetValueType sets expected value type and switches built-in validation regexp.
|
// SetValueType sets expected value type and switches built-in validation regexp.
|
||||||
func (c *CommandArg) SetValueType(t CommandValueType) *CommandArg {
|
func (c CommandArg) SetValueType(t CommandValueType) CommandArg {
|
||||||
regex := CommandRegexString
|
regex := CommandRegexString
|
||||||
switch t {
|
switch t {
|
||||||
case CommandValueIntType:
|
case CommandValueIntType:
|
||||||
@@ -72,14 +74,14 @@ func (c *CommandArg) SetValueType(t CommandValueType) *CommandArg {
|
|||||||
|
|
||||||
// SetRequired marks this argument as required.
|
// SetRequired marks this argument as required.
|
||||||
// Returns the receiver for method chaining.
|
// Returns the receiver for method chaining.
|
||||||
func (c *CommandArg) SetRequired() *CommandArg {
|
func (c CommandArg) SetRequired() CommandArg {
|
||||||
c.required = true
|
c.required = true
|
||||||
return c
|
return c
|
||||||
}
|
}
|
||||||
|
|
||||||
// CommandExecutor is the function type that executes a command.
|
// CommandExecutor is the function type that executes a command.
|
||||||
// It receives the message context and a database context (generic).
|
// It receives the message context and a database context (generic).
|
||||||
type CommandExecutor[T DbContext] func(ctx *MsgContext, dbContext *T)
|
type CommandExecutor[T DbContext] func(ctx *MsgContext, dbContext T)
|
||||||
|
|
||||||
// Command represents a bot command with arguments, description, and executor.
|
// Command represents a bot command with arguments, description, and executor.
|
||||||
// Can be registered in a Plugin and optionally skipped from auto-generation.
|
// Can be registered in a Plugin and optionally skipped from auto-generation.
|
||||||
@@ -123,14 +125,12 @@ func (c *Command[T]) SkipCommandAutoGen() *Command[T] {
|
|||||||
return c
|
return c
|
||||||
}
|
}
|
||||||
|
|
||||||
// validateArgs checks if the provided arguments match the command's requirements.
|
// Internal helper that validates provided command arguments.
|
||||||
// Returns ErrCmdArgCountMismatch if too few arguments are provided.
|
|
||||||
// Returns ErrCmdArgRegexpMismatch if any argument fails regex validation.
|
|
||||||
func (c *Command[T]) validateArgs(args []string) error {
|
func (c *Command[T]) validateArgs(args []string) error {
|
||||||
// Count required args
|
for i := range c.args.Len() {
|
||||||
requiredCount := c.args.Filter(func(a CommandArg) bool { return a.required }).Len()
|
if i >= len(args) && c.args.Get(i).required {
|
||||||
if len(args) < requiredCount {
|
return ErrCmdArgCountMismatch
|
||||||
return ErrCmdArgCountMismatch
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// Validate each argument against its regex
|
// Validate each argument against its regex
|
||||||
@@ -164,6 +164,8 @@ type Plugin[T DbContext] struct {
|
|||||||
skipAutoCmd bool // If true, all commands in this plugin are excluded from auto-help
|
skipAutoCmd bool // If true, all commands in this plugin are excluded from auto-help
|
||||||
logger *slog.Logger
|
logger *slog.Logger
|
||||||
|
|
||||||
|
handlers map[tgapi.UpdateType]CommandExecutor[T]
|
||||||
|
|
||||||
onClose func() error
|
onClose func() error
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -176,6 +178,7 @@ func NewPlugin[T DbContext](name string) *Plugin[T] {
|
|||||||
middlewares: make(extypes.Slice[Middleware[T]], 0),
|
middlewares: make(extypes.Slice[Middleware[T]], 0),
|
||||||
skipAutoCmd: false,
|
skipAutoCmd: false,
|
||||||
logger: nil,
|
logger: nil,
|
||||||
|
handlers: make(map[tgapi.UpdateType]CommandExecutor[T]),
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -209,6 +212,24 @@ func (p *Plugin[T]) NewPayload(exec CommandExecutor[T], command string, args ...
|
|||||||
return cmd
|
return cmd
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// AddUpdateHandler registers a handler for a non-command update type.
|
||||||
|
// Message, channel post, and callback query updates stay on the command/payload flow.
|
||||||
|
func (p *Plugin[T]) AddUpdateHandler(t tgapi.UpdateType, handler CommandExecutor[T]) *Plugin[T] {
|
||||||
|
switch t {
|
||||||
|
case tgapi.UpdateTypeMessage, tgapi.UpdateTypeChannelPost, tgapi.UpdateTypeCallbackQuery:
|
||||||
|
if p.logger == nil {
|
||||||
|
logger := utils.CreateLogger(p.name, utils.GetLoggerLevel())
|
||||||
|
logger.Warnf("%s can't be registred through AddUpdateHandler. Use AddPayload/NewPayload or AddCommand/NewCommand", t)
|
||||||
|
_ = logger.Close()
|
||||||
|
return p
|
||||||
|
}
|
||||||
|
p.logger.Warnf("%s can't be registred through AddUpdateHandler. Use AddPayload/NewPayload or AddCommand/NewCommand", t)
|
||||||
|
return p
|
||||||
|
}
|
||||||
|
p.handlers[t] = handler
|
||||||
|
return p
|
||||||
|
}
|
||||||
|
|
||||||
// AddMiddleware adds a middleware to the plugin's global middleware chain.
|
// AddMiddleware adds a middleware to the plugin's global middleware chain.
|
||||||
// Middlewares are executed before any command or payload.
|
// Middlewares are executed before any command or payload.
|
||||||
func (p *Plugin[T]) AddMiddleware(middleware Middleware[T]) *Plugin[T] {
|
func (p *Plugin[T]) AddMiddleware(middleware Middleware[T]) *Plugin[T] {
|
||||||
@@ -267,10 +288,8 @@ func (p *Plugin[T]) Close() error {
|
|||||||
return errors.Join(e...)
|
return errors.Join(e...)
|
||||||
}
|
}
|
||||||
|
|
||||||
// executeCmd finds and executes a command by its trigger string.
|
// Internal helper that validates and executes a command handler.
|
||||||
// Validates arguments and runs middlewares before executor.
|
func (p *Plugin[T]) executeCmd(cmd string, ctx *MsgContext, db T) {
|
||||||
// On error, sends an error message to the user via ctx.error().
|
|
||||||
func (p *Plugin[T]) executeCmd(cmd string, ctx *MsgContext, dbContext *T) {
|
|
||||||
command, exists := p.commands[cmd]
|
command, exists := p.commands[cmd]
|
||||||
if !exists {
|
if !exists {
|
||||||
ctx.error(errors.New("command not found"))
|
ctx.error(errors.New("command not found"))
|
||||||
@@ -284,19 +303,17 @@ func (p *Plugin[T]) executeCmd(cmd string, ctx *MsgContext, dbContext *T) {
|
|||||||
|
|
||||||
// Run command-specific middlewares
|
// Run command-specific middlewares
|
||||||
for _, m := range command.middlewares {
|
for _, m := range command.middlewares {
|
||||||
if !m.Execute(ctx, dbContext) {
|
if !m.Execute(ctx, db) {
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// Execute command
|
// Execute command
|
||||||
command.exec(ctx, dbContext)
|
command.exec(ctx, db)
|
||||||
}
|
}
|
||||||
|
|
||||||
// executePayload finds and executes a payload by its callback_data string.
|
// Internal helper that validates and executes a payload handler.
|
||||||
// Validates arguments and runs middlewares before executor.
|
func (p *Plugin[T]) executePayload(payload string, ctx *MsgContext, db T) {
|
||||||
// On error, sends an error message to the user via ctx.error().
|
|
||||||
func (p *Plugin[T]) executePayload(payload string, ctx *MsgContext, dbContext *T) {
|
|
||||||
command, exists := p.payloads[payload]
|
command, exists := p.payloads[payload]
|
||||||
if !exists {
|
if !exists {
|
||||||
ctx.error(errors.New("payload not found"))
|
ctx.error(errors.New("payload not found"))
|
||||||
@@ -310,18 +327,17 @@ func (p *Plugin[T]) executePayload(payload string, ctx *MsgContext, dbContext *T
|
|||||||
|
|
||||||
// Run command-specific middlewares
|
// Run command-specific middlewares
|
||||||
for _, m := range command.middlewares {
|
for _, m := range command.middlewares {
|
||||||
if !m.Execute(ctx, dbContext) {
|
if !m.Execute(ctx, db) {
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// Execute payload
|
// Execute payload
|
||||||
command.exec(ctx, dbContext)
|
command.exec(ctx, db)
|
||||||
}
|
}
|
||||||
|
|
||||||
// executeMiddlewares runs all plugin middlewares in order.
|
// Internal helper that runs plugin middlewares in order.
|
||||||
// Returns false if any middleware returns false (blocks execution).
|
func (p *Plugin[T]) executeMiddlewares(ctx *MsgContext, db T) bool {
|
||||||
func (p *Plugin[T]) executeMiddlewares(ctx *MsgContext, db *T) bool {
|
|
||||||
for _, m := range p.middlewares {
|
for _, m := range p.middlewares {
|
||||||
if !m.Execute(ctx, db) {
|
if !m.Execute(ctx, db) {
|
||||||
return false
|
return false
|
||||||
@@ -333,7 +349,7 @@ func (p *Plugin[T]) executeMiddlewares(ctx *MsgContext, db *T) bool {
|
|||||||
// MiddlewareExecutor is the function type for middleware logic.
|
// MiddlewareExecutor is the function type for middleware logic.
|
||||||
// Returns true to continue execution, false to block it.
|
// Returns true to continue execution, false to block it.
|
||||||
// If async, return value is ignored.
|
// If async, return value is ignored.
|
||||||
type MiddlewareExecutor[T DbContext] func(ctx *MsgContext, db *T) bool
|
type MiddlewareExecutor[T DbContext] func(ctx *MsgContext, db T) bool
|
||||||
|
|
||||||
// Middleware represents a reusable execution interceptor.
|
// Middleware represents a reusable execution interceptor.
|
||||||
// Can be synchronous (blocking) or asynchronous (non-blocking).
|
// Can be synchronous (blocking) or asynchronous (non-blocking).
|
||||||
@@ -345,19 +361,19 @@ type Middleware[T DbContext] struct {
|
|||||||
}
|
}
|
||||||
|
|
||||||
// NewMiddleware creates a new synchronous middleware.
|
// NewMiddleware creates a new synchronous middleware.
|
||||||
func NewMiddleware[T DbContext](name string, executor MiddlewareExecutor[T]) *Middleware[T] {
|
func NewMiddleware[T DbContext](name string, executor MiddlewareExecutor[T]) Middleware[T] {
|
||||||
return &Middleware[T]{name, executor, 0, false}
|
return Middleware[T]{name, executor, 0, false}
|
||||||
}
|
}
|
||||||
|
|
||||||
// SetOrder sets the execution order (currently ignored).
|
// SetOrder sets the execution order (currently ignored).
|
||||||
func (m *Middleware[T]) SetOrder(order int) *Middleware[T] {
|
func (m Middleware[T]) SetOrder(order int) Middleware[T] {
|
||||||
m.order = order
|
m.order = order
|
||||||
return m
|
return m
|
||||||
}
|
}
|
||||||
|
|
||||||
// SetAsync marks the middleware to run asynchronously.
|
// SetAsync marks the middleware to run asynchronously.
|
||||||
// Execution continues regardless of its return value.
|
// Execution continues regardless of its return value.
|
||||||
func (m *Middleware[T]) SetAsync(async bool) *Middleware[T] {
|
func (m Middleware[T]) SetAsync(async bool) Middleware[T] {
|
||||||
m.async = async
|
m.async = async
|
||||||
return m
|
return m
|
||||||
}
|
}
|
||||||
@@ -365,7 +381,7 @@ func (m *Middleware[T]) SetAsync(async bool) *Middleware[T] {
|
|||||||
// Execute runs the middleware.
|
// Execute runs the middleware.
|
||||||
// If async, runs in a goroutine and returns true immediately.
|
// If async, runs in a goroutine and returns true immediately.
|
||||||
// Otherwise, returns the result of the executor.
|
// Otherwise, returns the result of the executor.
|
||||||
func (m *Middleware[T]) Execute(ctx *MsgContext, db *T) bool {
|
func (m Middleware[T]) Execute(ctx *MsgContext, db T) bool {
|
||||||
if m.async {
|
if m.async {
|
||||||
ctx := *ctx // copy context to avoid race condition
|
ctx := *ctx // copy context to avoid race condition
|
||||||
go func(ctx MsgContext) {
|
go func(ctx MsgContext) {
|
||||||
|
|||||||
+18
-2
@@ -6,7 +6,7 @@ import (
|
|||||||
)
|
)
|
||||||
|
|
||||||
func TestValidateArgsRequiresFullMatch(t *testing.T) {
|
func TestValidateArgsRequiresFullMatch(t *testing.T) {
|
||||||
intCmd := NewCommand[NoDB](func(ctx *MsgContext, db *NoDB) {}, "int", *NewCommandArg("n").SetValueType(CommandValueIntType).SetRequired())
|
intCmd := NewCommand[NoDB](func(ctx *MsgContext, db NoDB) {}, "int", NewCommandArg("n").SetValueType(CommandValueIntType).SetRequired())
|
||||||
if err := intCmd.validateArgs([]string{"123"}); err != nil {
|
if err := intCmd.validateArgs([]string{"123"}); err != nil {
|
||||||
t.Fatalf("expected valid integer argument, got %v", err)
|
t.Fatalf("expected valid integer argument, got %v", err)
|
||||||
}
|
}
|
||||||
@@ -14,7 +14,7 @@ func TestValidateArgsRequiresFullMatch(t *testing.T) {
|
|||||||
t.Fatalf("expected ErrCmdArgRegexpMismatch for partial int match, got %v", err)
|
t.Fatalf("expected ErrCmdArgRegexpMismatch for partial int match, got %v", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
boolCmd := NewCommand[NoDB](func(ctx *MsgContext, db *NoDB) {}, "bool", *NewCommandArg("flag").SetValueType(CommandValueBoolType).SetRequired())
|
boolCmd := NewCommand[NoDB](func(ctx *MsgContext, db NoDB) {}, "bool", NewCommandArg("flag").SetValueType(CommandValueBoolType).SetRequired())
|
||||||
if err := boolCmd.validateArgs([]string{"false"}); err != nil {
|
if err := boolCmd.validateArgs([]string{"false"}); err != nil {
|
||||||
t.Fatalf("expected valid bool argument, got %v", err)
|
t.Fatalf("expected valid bool argument, got %v", err)
|
||||||
}
|
}
|
||||||
@@ -22,3 +22,19 @@ func TestValidateArgsRequiresFullMatch(t *testing.T) {
|
|||||||
t.Fatalf("expected ErrCmdArgRegexpMismatch for partial bool match, got %v", err)
|
t.Fatalf("expected ErrCmdArgRegexpMismatch for partial bool match, got %v", err)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestValidateArgsEnforcesRequiredArgIndex(t *testing.T) {
|
||||||
|
cmd := NewCommand[NoDB](
|
||||||
|
func(ctx *MsgContext, db NoDB) {},
|
||||||
|
"mixed",
|
||||||
|
NewCommandArg("optional"),
|
||||||
|
NewCommandArg("required").SetRequired(),
|
||||||
|
)
|
||||||
|
|
||||||
|
if err := cmd.validateArgs([]string{"only-optional"}); !errors.Is(err, ErrCmdArgCountMismatch) {
|
||||||
|
t.Fatalf("expected ErrCmdArgCountMismatch when required second arg is missing, got %v", err)
|
||||||
|
}
|
||||||
|
if err := cmd.validateArgs([]string{"optional", "required"}); err != nil {
|
||||||
|
t.Fatalf("expected both args to validate, got %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
+5
-5
@@ -33,8 +33,8 @@ type Runner[T DbContext] struct {
|
|||||||
//
|
//
|
||||||
// Builder methods (Onetime, Async, Timeout) can be chained to customize behavior.
|
// Builder methods (Onetime, Async, Timeout) can be chained to customize behavior.
|
||||||
// DO NOT call builder methods concurrently or after Execute().
|
// DO NOT call builder methods concurrently or after Execute().
|
||||||
func NewRunner[T DbContext](name string, fn RunnerFn[T]) *Runner[T] {
|
func NewRunner[T DbContext](name string, fn RunnerFn[T]) Runner[T] {
|
||||||
return &Runner[T]{
|
return Runner[T]{
|
||||||
name: name,
|
name: name,
|
||||||
fn: fn,
|
fn: fn,
|
||||||
async: true, // Default: run asynchronously
|
async: true, // Default: run asynchronously
|
||||||
@@ -45,7 +45,7 @@ func NewRunner[T DbContext](name string, fn RunnerFn[T]) *Runner[T] {
|
|||||||
// Onetime sets whether the runner executes once or repeatedly.
|
// Onetime sets whether the runner executes once or repeatedly.
|
||||||
// If true, the runner runs only once.
|
// If true, the runner runs only once.
|
||||||
// If false, the runner runs in a loop with the configured timeout.
|
// If false, the runner runs in a loop with the configured timeout.
|
||||||
func (r *Runner[T]) Onetime(onetime bool) *Runner[T] {
|
func (r Runner[T]) Onetime(onetime bool) Runner[T] {
|
||||||
r.onetime = onetime
|
r.onetime = onetime
|
||||||
return r
|
return r
|
||||||
}
|
}
|
||||||
@@ -55,7 +55,7 @@ func (r *Runner[T]) Onetime(onetime bool) *Runner[T] {
|
|||||||
// If false, the runner blocks the caller during execution.
|
// If false, the runner blocks the caller during execution.
|
||||||
//
|
//
|
||||||
// Note: If onetime=false and async=false, the runner will be skipped with a warning.
|
// Note: If onetime=false and async=false, the runner will be skipped with a warning.
|
||||||
func (r *Runner[T]) Async(async bool) *Runner[T] {
|
func (r Runner[T]) Async(async bool) Runner[T] {
|
||||||
r.async = async
|
r.async = async
|
||||||
return r
|
return r
|
||||||
}
|
}
|
||||||
@@ -69,7 +69,7 @@ func (r *Runner[T]) Async(async bool) *Runner[T] {
|
|||||||
//
|
//
|
||||||
// A zero value (time.Duration(0)) is allowed but may trigger a warning
|
// A zero value (time.Duration(0)) is allowed but may trigger a warning
|
||||||
// if used with a background (non-onetime) async runner.
|
// if used with a background (non-onetime) async runner.
|
||||||
func (r *Runner[T]) Timeout(timeout time.Duration) *Runner[T] {
|
func (r Runner[T]) Timeout(timeout time.Duration) Runner[T] {
|
||||||
r.timeout = timeout
|
r.timeout = timeout
|
||||||
return r
|
return r
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1,62 @@
|
|||||||
|
package laniakea
|
||||||
|
|
||||||
|
import (
|
||||||
|
"context"
|
||||||
|
"sync/atomic"
|
||||||
|
"testing"
|
||||||
|
"time"
|
||||||
|
|
||||||
|
"git.nix13.pw/scuroneko/slog"
|
||||||
|
)
|
||||||
|
|
||||||
|
func TestExecRunnersRunsOnetimeSyncRunner(t *testing.T) {
|
||||||
|
var calls atomic.Int32
|
||||||
|
bot := &Bot[NoDB]{
|
||||||
|
logger: slog.CreateLogger(),
|
||||||
|
runners: []Runner[NoDB]{
|
||||||
|
NewRunner("sync-once", func(*Bot[NoDB]) error {
|
||||||
|
calls.Add(1)
|
||||||
|
return nil
|
||||||
|
}).Onetime(true).Async(false),
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
bot.ExecRunners(context.Background())
|
||||||
|
|
||||||
|
if got := calls.Load(); got != 1 {
|
||||||
|
t.Fatalf("unexpected sync runner call count: %d", got)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestExecRunnersStopsBackgroundRunnerOnCancel(t *testing.T) {
|
||||||
|
var calls atomic.Int32
|
||||||
|
triggered := make(chan struct{}, 1)
|
||||||
|
ctx, cancel := context.WithCancel(context.Background())
|
||||||
|
|
||||||
|
bot := &Bot[NoDB]{
|
||||||
|
logger: slog.CreateLogger(),
|
||||||
|
runners: []Runner[NoDB]{
|
||||||
|
NewRunner("background", func(*Bot[NoDB]) error {
|
||||||
|
if calls.Add(1) == 1 {
|
||||||
|
triggered <- struct{}{}
|
||||||
|
}
|
||||||
|
return nil
|
||||||
|
}).Timeout(5 * time.Millisecond),
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
bot.ExecRunners(ctx)
|
||||||
|
|
||||||
|
select {
|
||||||
|
case <-triggered:
|
||||||
|
case <-time.After(time.Second):
|
||||||
|
t.Fatal("background runner did not execute")
|
||||||
|
}
|
||||||
|
|
||||||
|
cancel()
|
||||||
|
bot.runnerBgWG.Wait()
|
||||||
|
|
||||||
|
if calls.Load() == 0 {
|
||||||
|
t.Fatal("expected background runner to be called at least once")
|
||||||
|
}
|
||||||
|
}
|
||||||
+12
-19
@@ -124,6 +124,9 @@ func NewAPI(opts *APIOpts) *API {
|
|||||||
// See https://core.telegram.org/bots/api
|
// See https://core.telegram.org/bots/api
|
||||||
func (api *API) Close() error {
|
func (api *API) Close() error {
|
||||||
api.pool.stop()
|
api.pool.stop()
|
||||||
|
if api.client != nil {
|
||||||
|
api.client.CloseIdleConnections()
|
||||||
|
}
|
||||||
return api.logger.Close()
|
return api.logger.Close()
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -149,37 +152,29 @@ type ApiResponse[R any] struct {
|
|||||||
Parameters *ResponseParameters `json:"parameters,omitempty"`
|
Parameters *ResponseParameters `json:"parameters,omitempty"`
|
||||||
}
|
}
|
||||||
|
|
||||||
// TelegramRequest is an internal helper struct.
|
// TelegramRequest is a low-level Telegram API request wrapper.
|
||||||
// DO NOT USE NewRequest or NewRequestWithChatID — they are unsafe and discouraged.
|
|
||||||
// Instead, use explicit methods like SendMessage, GetUpdates, etc.
|
|
||||||
//
|
//
|
||||||
// Why? Because using generics with arbitrary types P and R leads to:
|
// Prefer method-specific helpers such as SendMessage or GetUpdates. TelegramRequest
|
||||||
// - No compile-time validation of parameters
|
// bypasses method-specific parameter types and convenience helpers, so callers are
|
||||||
// - No IDE autocompletion
|
// responsible for using the correct method name and compatible request and response types.
|
||||||
// - Runtime panics on malformed JSON
|
// In that sense it is an unsafe escape hatch compared with the typed API surface.
|
||||||
// - Hard-to-debug errors
|
|
||||||
//
|
|
||||||
// Recommended: Define specific methods for each Telegram method (see below).
|
|
||||||
type TelegramRequest[R, P any] struct {
|
type TelegramRequest[R, P any] struct {
|
||||||
method string
|
method string
|
||||||
params P
|
params P
|
||||||
chatId int64
|
chatId int64
|
||||||
}
|
}
|
||||||
|
|
||||||
// NewRequest creates an untyped TelegramRequest for the given method and params with no chat ID.
|
// NewRequest creates a low-level TelegramRequest with no associated chat ID.
|
||||||
func NewRequest[R, P any](method string, params P) TelegramRequest[R, P] {
|
func NewRequest[R, P any](method string, params P) TelegramRequest[R, P] {
|
||||||
return TelegramRequest[R, P]{method, params, 0}
|
return TelegramRequest[R, P]{method, params, 0}
|
||||||
}
|
}
|
||||||
|
|
||||||
// NewRequestWithChatID creates an untyped TelegramRequest with an associated chat ID.
|
// NewRequestWithChatID creates a low-level TelegramRequest with an associated chat ID.
|
||||||
// The chat ID is used for per-chat rate limiting.
|
// The chat ID is used for per-chat rate limiting.
|
||||||
func NewRequestWithChatID[R, P any](method string, params P, chatId int64) TelegramRequest[R, P] {
|
func NewRequestWithChatID[R, P any](method string, params P, chatId int64) TelegramRequest[R, P] {
|
||||||
return TelegramRequest[R, P]{method, params, chatId}
|
return TelegramRequest[R, P]{method, params, chatId}
|
||||||
}
|
}
|
||||||
|
|
||||||
// doRequest performs a single HTTP request to Telegram API.
|
|
||||||
// Handles rate limiting, retries on 429, and parses responses.
|
|
||||||
// Must be called within a worker pool context if using DoWithContext.
|
|
||||||
func (r TelegramRequest[R, P]) doRequest(ctx context.Context, api *API) (R, error) {
|
func (r TelegramRequest[R, P]) doRequest(ctx context.Context, api *API) (R, error) {
|
||||||
var zero R
|
var zero R
|
||||||
reqData, err := json.Marshal(r.params)
|
reqData, err := json.Marshal(r.params)
|
||||||
@@ -296,15 +291,13 @@ func (r TelegramRequest[R, P]) Do(api *API) (R, error) {
|
|||||||
return r.DoWithContext(context.Background(), api)
|
return r.DoWithContext(context.Background(), api)
|
||||||
}
|
}
|
||||||
|
|
||||||
// readBody reads and limits response body to prevent memory exhaustion.
|
// Internal helper that reads and caps a Telegram response body.
|
||||||
// Telegram responses are typically small (<1MB), but we cap at 10MB.
|
|
||||||
func readBody(body io.ReadCloser) ([]byte, error) {
|
func readBody(body io.ReadCloser) ([]byte, error) {
|
||||||
reader := io.LimitReader(body, 10<<20) // 10 MB
|
reader := io.LimitReader(body, 10<<20) // 10 MB
|
||||||
return io.ReadAll(reader)
|
return io.ReadAll(reader)
|
||||||
}
|
}
|
||||||
|
|
||||||
// parseBody unmarshals a Telegram API response into a typed ApiResponse.
|
// Internal helper that parses a typed Telegram API response body.
|
||||||
// Only returns an error on malformed JSON; non-OK responses are left for the caller to handle.
|
|
||||||
func parseBody[R any](data []byte) (ApiResponse[R], error) {
|
func parseBody[R any](data []byte) (ApiResponse[R], error) {
|
||||||
var resp ApiResponse[R]
|
var resp ApiResponse[R]
|
||||||
err := json.Unmarshal(data, &resp)
|
err := json.Unmarshal(data, &resp)
|
||||||
|
|||||||
@@ -13,6 +13,15 @@ func (fn roundTripFunc) RoundTrip(req *http.Request) (*http.Response, error) {
|
|||||||
return fn(req)
|
return fn(req)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
type closingTransport struct {
|
||||||
|
roundTripFunc
|
||||||
|
closed bool
|
||||||
|
}
|
||||||
|
|
||||||
|
func (t *closingTransport) CloseIdleConnections() {
|
||||||
|
t.closed = true
|
||||||
|
}
|
||||||
|
|
||||||
func TestAPILeavesAcceptEncodingToHTTPTransport(t *testing.T) {
|
func TestAPILeavesAcceptEncodingToHTTPTransport(t *testing.T) {
|
||||||
var gotPath string
|
var gotPath string
|
||||||
var gotAcceptEncoding string
|
var gotAcceptEncoding string
|
||||||
@@ -54,3 +63,28 @@ func TestAPILeavesAcceptEncodingToHTTPTransport(t *testing.T) {
|
|||||||
t.Fatalf("expected empty Accept-Encoding header, got %q", gotAcceptEncoding)
|
t.Fatalf("expected empty Accept-Encoding header, got %q", gotAcceptEncoding)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestAPICloseClosesIdleConnections(t *testing.T) {
|
||||||
|
transport := &closingTransport{
|
||||||
|
roundTripFunc: func(req *http.Request) (*http.Response, error) {
|
||||||
|
return &http.Response{
|
||||||
|
StatusCode: http.StatusOK,
|
||||||
|
Header: http.Header{"Content-Type": []string{"application/json"}},
|
||||||
|
Body: io.NopCloser(strings.NewReader(`{"ok":true,"result":{"id":1,"is_bot":true,"first_name":"Test"}}`)),
|
||||||
|
}, nil
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
api := NewAPI(
|
||||||
|
NewAPIOpts("token").
|
||||||
|
SetAPIUrl("https://example.test").
|
||||||
|
SetHTTPClient(&http.Client{Transport: transport}),
|
||||||
|
)
|
||||||
|
|
||||||
|
if err := api.Close(); err != nil {
|
||||||
|
t.Fatalf("Close returned error: %v", err)
|
||||||
|
}
|
||||||
|
if !transport.closed {
|
||||||
|
t.Fatal("expected Close to close idle HTTP connections")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
@@ -241,7 +241,7 @@ type SendVoiceP struct {
|
|||||||
|
|
||||||
// SendVoice sends a voice note.
|
// SendVoice sends a voice note.
|
||||||
// See https://core.telegram.org/bots/api#sendvoice
|
// See https://core.telegram.org/bots/api#sendvoice
|
||||||
func (api *API) SendVoice(params *SendVoiceP) (Message, error) {
|
func (api *API) SendVoice(params SendVoiceP) (Message, error) {
|
||||||
req := NewRequestWithChatID[Message]("sendVoice", params, params.ChatID)
|
req := NewRequestWithChatID[Message]("sendVoice", params, params.ChatID)
|
||||||
return req.Do(api)
|
return req.Do(api)
|
||||||
}
|
}
|
||||||
@@ -249,7 +249,7 @@ func (api *API) SendVoice(params *SendVoiceP) (Message, error) {
|
|||||||
// SendVoiceWithContext is the context-aware variant of SendVoice.
|
// SendVoiceWithContext is the context-aware variant of SendVoice.
|
||||||
// It executes the same request but uses ctx for cancellation and deadlines.
|
// It executes the same request but uses ctx for cancellation and deadlines.
|
||||||
// See https://core.telegram.org/bots/api#sendvoice
|
// See https://core.telegram.org/bots/api#sendvoice
|
||||||
func (api *API) SendVoiceWithContext(ctx context.Context, params *SendVoiceP) (Message, error) {
|
func (api *API) SendVoiceWithContext(ctx context.Context, params SendVoiceP) (Message, error) {
|
||||||
req := NewRequestWithChatID[Message]("sendVoice", params, params.ChatID)
|
req := NewRequestWithChatID[Message]("sendVoice", params, params.ChatID)
|
||||||
return req.DoWithContext(ctx, api)
|
return req.DoWithContext(ctx, api)
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -55,12 +55,12 @@ type InputPaidMedia struct {
|
|||||||
Type InputPaidMediaType `json:"type"`
|
Type InputPaidMediaType `json:"type"`
|
||||||
Media string `json:"media"`
|
Media string `json:"media"`
|
||||||
|
|
||||||
Cover string `json:"cover"`
|
Cover *string `json:"cover,omitempty"`
|
||||||
StartTimestamp int64 `json:"start_timestamp"`
|
StartTimestamp *int64 `json:"start_timestamp,omitempty"`
|
||||||
Width int `json:"width"`
|
Width *int `json:"width,omitempty"`
|
||||||
Height int `json:"height"`
|
Height *int `json:"height,omitempty"`
|
||||||
Duration int `json:"duration"`
|
Duration *int `json:"duration,omitempty"`
|
||||||
SupportsStreaming bool `json:"supports_streaming"`
|
SupportsStreaming *bool `json:"supports_streaming,omitempty"`
|
||||||
}
|
}
|
||||||
|
|
||||||
// PhotoSize represents one size of a photo or a file/sticker thumbnail.
|
// PhotoSize represents one size of a photo or a file/sticker thumbnail.
|
||||||
|
|||||||
@@ -244,7 +244,7 @@ func (api *API) RemoveMyProfilePhotoWithContext(ctx context.Context) (bool, erro
|
|||||||
// SetChatMenuButtonP holds parameters for the setChatMenuButton method.
|
// SetChatMenuButtonP holds parameters for the setChatMenuButton method.
|
||||||
// See https://core.telegram.org/bots/api#setchatmenubutton
|
// See https://core.telegram.org/bots/api#setchatmenubutton
|
||||||
type SetChatMenuButtonP struct {
|
type SetChatMenuButtonP struct {
|
||||||
ChatID int64 `json:"chat_id"`
|
ChatID int64 `json:"chat_id,omitempty"`
|
||||||
MenuButton MenuButtonType `json:"menu_button"`
|
MenuButton MenuButtonType `json:"menu_button"`
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -267,21 +267,21 @@ func (api *API) SetChatMenuButtonWithContext(ctx context.Context, params SetChat
|
|||||||
// GetChatMenuButtonP holds parameters for the getChatMenuButton method.
|
// GetChatMenuButtonP holds parameters for the getChatMenuButton method.
|
||||||
// See https://core.telegram.org/bots/api#getchatmenubutton
|
// See https://core.telegram.org/bots/api#getchatmenubutton
|
||||||
type GetChatMenuButtonP struct {
|
type GetChatMenuButtonP struct {
|
||||||
ChatID int64 `json:"chat_id"`
|
ChatID int64 `json:"chat_id,omitempty"`
|
||||||
}
|
}
|
||||||
|
|
||||||
// GetChatMenuButton returns the current menu button for the given chat.
|
// GetChatMenuButton returns the current menu button for the given chat.
|
||||||
// See https://core.telegram.org/bots/api#getchatmenubutton
|
// See https://core.telegram.org/bots/api#getchatmenubutton
|
||||||
func (api *API) GetChatMenuButton(params GetChatMenuButtonP) (BaseMenuButton, error) {
|
func (api *API) GetChatMenuButton(params GetChatMenuButtonP) (MenuButton, error) {
|
||||||
req := NewRequest[BaseMenuButton]("getChatMenuButton", params)
|
req := NewRequest[MenuButton]("getChatMenuButton", params)
|
||||||
return req.Do(api)
|
return req.Do(api)
|
||||||
}
|
}
|
||||||
|
|
||||||
// GetChatMenuButtonWithContext is the context-aware variant of GetChatMenuButton.
|
// GetChatMenuButtonWithContext is the context-aware variant of GetChatMenuButton.
|
||||||
// It executes the same request but uses ctx for cancellation and deadlines.
|
// It executes the same request but uses ctx for cancellation and deadlines.
|
||||||
// See https://core.telegram.org/bots/api#getchatmenubutton
|
// See https://core.telegram.org/bots/api#getchatmenubutton
|
||||||
func (api *API) GetChatMenuButtonWithContext(ctx context.Context, params GetChatMenuButtonP) (BaseMenuButton, error) {
|
func (api *API) GetChatMenuButtonWithContext(ctx context.Context, params GetChatMenuButtonP) (MenuButton, error) {
|
||||||
req := NewRequest[BaseMenuButton]("getChatMenuButton", params)
|
req := NewRequest[MenuButton]("getChatMenuButton", params)
|
||||||
return req.DoWithContext(ctx, api)
|
return req.DoWithContext(ctx, api)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
+12
-7
@@ -54,7 +54,9 @@ type BotShortDescription struct {
|
|||||||
type InputProfilePhotoType string
|
type InputProfilePhotoType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
InputProfilePhotoStaticType InputProfilePhotoType = "static"
|
// InputProfilePhotoStaticType identifies a static profile photo input.
|
||||||
|
InputProfilePhotoStaticType InputProfilePhotoType = "static"
|
||||||
|
// InputProfilePhotoAnimatedType identifies an animated profile photo input.
|
||||||
InputProfilePhotoAnimatedType InputProfilePhotoType = "animated"
|
InputProfilePhotoAnimatedType InputProfilePhotoType = "animated"
|
||||||
)
|
)
|
||||||
|
|
||||||
@@ -75,17 +77,20 @@ type InputProfilePhoto struct {
|
|||||||
type MenuButtonType string
|
type MenuButtonType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
|
// MenuButtonCommandsType identifies a commands menu button.
|
||||||
MenuButtonCommandsType MenuButtonType = "commands"
|
MenuButtonCommandsType MenuButtonType = "commands"
|
||||||
MenuButtonWebAppType MenuButtonType = "web_app"
|
// MenuButtonWebAppType identifies a web app menu button.
|
||||||
MenuButtonDefaultType MenuButtonType = "default"
|
MenuButtonWebAppType MenuButtonType = "web_app"
|
||||||
|
// MenuButtonDefaultType identifies Telegram's default menu button.
|
||||||
|
MenuButtonDefaultType MenuButtonType = "default"
|
||||||
)
|
)
|
||||||
|
|
||||||
// BaseMenuButton represents a menu button.
|
// MenuButton represents a menu button.
|
||||||
// See https://core.telegram.org/bots/api#menubutton
|
// See https://core.telegram.org/bots/api#menubutton
|
||||||
type BaseMenuButton struct {
|
type MenuButton struct {
|
||||||
Type MenuButtonType `json:"type"`
|
Type MenuButtonType `json:"type"`
|
||||||
|
|
||||||
// WebApp fields (for web_app button)
|
// WebApp fields (for web_app button)
|
||||||
Text string `json:"text"`
|
Text *string `json:"text"`
|
||||||
WebApp WebAppInfo `json:"web_app"`
|
WebApp *WebAppInfo `json:"web_app"`
|
||||||
}
|
}
|
||||||
|
|||||||
+11
-4
@@ -72,7 +72,9 @@ type BusinessMessagesDeleted struct {
|
|||||||
type InputStoryContentType string
|
type InputStoryContentType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
|
// InputStoryContentPhotoType identifies photo story content.
|
||||||
InputStoryContentPhotoType InputStoryContentType = "photo"
|
InputStoryContentPhotoType InputStoryContentType = "photo"
|
||||||
|
// InputStoryContentVideoType identifies video story content.
|
||||||
InputStoryContentVideoType InputStoryContentType = "video"
|
InputStoryContentVideoType InputStoryContentType = "video"
|
||||||
)
|
)
|
||||||
|
|
||||||
@@ -106,10 +108,15 @@ type StoryAreaPosition struct {
|
|||||||
type StoryAreaTypeType string
|
type StoryAreaTypeType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
StoryAreaTypeLocationType StoryAreaTypeType = "location"
|
// StoryAreaTypeLocationType identifies a location story area.
|
||||||
StoryAreaTypeReactionType StoryAreaTypeType = "suggested_reaction"
|
StoryAreaTypeLocationType StoryAreaTypeType = "location"
|
||||||
StoryAreaTypeLinkType StoryAreaTypeType = "link"
|
// StoryAreaTypeReactionType identifies a suggested reaction story area.
|
||||||
StoryAreaTypeWeatherType StoryAreaTypeType = "weather"
|
StoryAreaTypeReactionType StoryAreaTypeType = "suggested_reaction"
|
||||||
|
// StoryAreaTypeLinkType identifies a link story area.
|
||||||
|
StoryAreaTypeLinkType StoryAreaTypeType = "link"
|
||||||
|
// StoryAreaTypeWeatherType identifies a weather story area.
|
||||||
|
StoryAreaTypeWeatherType StoryAreaTypeType = "weather"
|
||||||
|
// StoryAreaTypeUniqueGiftType identifies a unique gift story area.
|
||||||
StoryAreaTypeUniqueGiftType StoryAreaTypeType = "unique_gift"
|
StoryAreaTypeUniqueGiftType StoryAreaTypeType = "unique_gift"
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|||||||
+19
-9
@@ -17,10 +17,14 @@ type Chat struct {
|
|||||||
type ChatType string
|
type ChatType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
ChatTypePrivate ChatType = "private"
|
// ChatTypePrivate identifies a private chat.
|
||||||
ChatTypeGroup ChatType = "group"
|
ChatTypePrivate ChatType = "private"
|
||||||
|
// ChatTypeGroup identifies a basic group chat.
|
||||||
|
ChatTypeGroup ChatType = "group"
|
||||||
|
// ChatTypeSupergroup identifies a supergroup chat.
|
||||||
ChatTypeSupergroup ChatType = "supergroup"
|
ChatTypeSupergroup ChatType = "supergroup"
|
||||||
ChatTypeChannel ChatType = "channel"
|
// ChatTypeChannel identifies a channel chat.
|
||||||
|
ChatTypeChannel ChatType = "channel"
|
||||||
)
|
)
|
||||||
|
|
||||||
// ChatFullInfo contains full information about a chat.
|
// ChatFullInfo contains full information about a chat.
|
||||||
@@ -143,12 +147,18 @@ type ChatInviteLink struct {
|
|||||||
type ChatMemberStatusType string
|
type ChatMemberStatusType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
ChatMemberStatusOwner ChatMemberStatusType = "owner"
|
// ChatMemberStatusOwner identifies a chat owner.
|
||||||
|
ChatMemberStatusOwner ChatMemberStatusType = "owner"
|
||||||
|
// ChatMemberStatusAdministrator identifies a chat administrator.
|
||||||
ChatMemberStatusAdministrator ChatMemberStatusType = "administrator"
|
ChatMemberStatusAdministrator ChatMemberStatusType = "administrator"
|
||||||
ChatMemberStatusMember ChatMemberStatusType = "member"
|
// ChatMemberStatusMember identifies a regular member.
|
||||||
ChatMemberStatusRestricted ChatMemberStatusType = "restricted"
|
ChatMemberStatusMember ChatMemberStatusType = "member"
|
||||||
ChatMemberStatusLeft ChatMemberStatusType = "left"
|
// ChatMemberStatusRestricted identifies a restricted member.
|
||||||
ChatMemberStatusBanned ChatMemberStatusType = "kicked"
|
ChatMemberStatusRestricted ChatMemberStatusType = "restricted"
|
||||||
|
// ChatMemberStatusLeft identifies a user who left the chat.
|
||||||
|
ChatMemberStatusLeft ChatMemberStatusType = "left"
|
||||||
|
// ChatMemberStatusBanned identifies a banned user.
|
||||||
|
ChatMemberStatusBanned ChatMemberStatusType = "kicked"
|
||||||
)
|
)
|
||||||
|
|
||||||
// ChatMember contains information about one member of a chat.
|
// ChatMember contains information about one member of a chat.
|
||||||
@@ -214,7 +224,7 @@ type ChatBoostSource struct {
|
|||||||
// ChatBoost represents a boost added to a chat.
|
// ChatBoost represents a boost added to a chat.
|
||||||
// See https://core.telegram.org/bots/api#chatboost
|
// See https://core.telegram.org/bots/api#chatboost
|
||||||
type ChatBoost struct {
|
type ChatBoost struct {
|
||||||
BoostID int `json:"boost_id"`
|
BoostID string `json:"boost_id"`
|
||||||
AddDate int `json:"add_date"`
|
AddDate int `json:"add_date"`
|
||||||
ExpirationDate int `json:"expiration_date"`
|
ExpirationDate int `json:"expiration_date"`
|
||||||
Source ChatBoostSource `json:"source"`
|
Source ChatBoostSource `json:"source"`
|
||||||
|
|||||||
@@ -2,7 +2,14 @@ package tgapi
|
|||||||
|
|
||||||
import "errors"
|
import "errors"
|
||||||
|
|
||||||
|
// ErrRateLimit reports that a request exceeded the configured rate limiter.
|
||||||
var ErrRateLimit = errors.New("rate limit exceeded")
|
var ErrRateLimit = errors.New("rate limit exceeded")
|
||||||
|
|
||||||
|
// ErrPoolUnexpected reports an unexpected result type returned from the worker pool.
|
||||||
var ErrPoolUnexpected = errors.New("unexpected response from pool")
|
var ErrPoolUnexpected = errors.New("unexpected response from pool")
|
||||||
|
|
||||||
|
// ErrPoolQueueFull reports that the internal request queue is full.
|
||||||
var ErrPoolQueueFull = errors.New("worker pool queue full")
|
var ErrPoolQueueFull = errors.New("worker pool queue full")
|
||||||
|
|
||||||
|
// ErrPoolStopped reports that a request was submitted after the worker pool stopped.
|
||||||
var ErrPoolStopped = errors.New("worker pool stopped")
|
var ErrPoolStopped = errors.New("worker pool stopped")
|
||||||
|
|||||||
+79
-45
@@ -45,9 +45,9 @@ type Message struct {
|
|||||||
|
|
||||||
Text string `json:"text"`
|
Text string `json:"text"`
|
||||||
|
|
||||||
Photo extypes.Slice[*PhotoSize] `json:"photo,omitempty"`
|
Photo extypes.Slice[PhotoSize] `json:"photo,omitempty"`
|
||||||
Caption string `json:"caption,omitempty"`
|
Caption string `json:"caption,omitempty"`
|
||||||
CaptionEntities []MessageEntity `json:"caption_entities,omitempty"`
|
CaptionEntities []MessageEntity `json:"caption_entities,omitempty"`
|
||||||
|
|
||||||
Date int `json:"date"`
|
Date int `json:"date"`
|
||||||
EditDate int `json:"edit_date"`
|
EditDate int `json:"edit_date"`
|
||||||
@@ -77,26 +77,46 @@ type MaybeInaccessibleMessage interface{ Message | InaccessibleMessage }
|
|||||||
type MessageEntityType string
|
type MessageEntityType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
MessageEntityMention MessageEntityType = "mention"
|
// MessageEntityMention identifies an @mention entity.
|
||||||
MessageEntityHashtag MessageEntityType = "hashtag"
|
MessageEntityMention MessageEntityType = "mention"
|
||||||
MessageEntityCashtag MessageEntityType = "cashtag"
|
// MessageEntityHashtag identifies a hashtag entity.
|
||||||
MessageEntityBotCommand MessageEntityType = "bot_command"
|
MessageEntityHashtag MessageEntityType = "hashtag"
|
||||||
MessageEntityUrl MessageEntityType = "url"
|
// MessageEntityCashtag identifies a cashtag entity.
|
||||||
MessageEntityEmail MessageEntityType = "email"
|
MessageEntityCashtag MessageEntityType = "cashtag"
|
||||||
MessageEntityPhoneNumber MessageEntityType = "phone_number"
|
// MessageEntityBotCommand identifies a bot command entity.
|
||||||
MessageEntityBold MessageEntityType = "bold"
|
MessageEntityBotCommand MessageEntityType = "bot_command"
|
||||||
MessageEntityItalic MessageEntityType = "italic"
|
// MessageEntityUrl identifies a URL entity.
|
||||||
MessageEntityUnderline MessageEntityType = "underline"
|
MessageEntityUrl MessageEntityType = "url"
|
||||||
MessageEntityStrike MessageEntityType = "strikethrough"
|
// MessageEntityEmail identifies an email entity.
|
||||||
MessageEntitySpoiler MessageEntityType = "spoiler"
|
MessageEntityEmail MessageEntityType = "email"
|
||||||
MessageEntityBlockquote MessageEntityType = "blockquote"
|
// MessageEntityPhoneNumber identifies a phone number entity.
|
||||||
|
MessageEntityPhoneNumber MessageEntityType = "phone_number"
|
||||||
|
// MessageEntityBold identifies bold text.
|
||||||
|
MessageEntityBold MessageEntityType = "bold"
|
||||||
|
// MessageEntityItalic identifies italic text.
|
||||||
|
MessageEntityItalic MessageEntityType = "italic"
|
||||||
|
// MessageEntityUnderline identifies underlined text.
|
||||||
|
MessageEntityUnderline MessageEntityType = "underline"
|
||||||
|
// MessageEntityStrike identifies strikethrough text.
|
||||||
|
MessageEntityStrike MessageEntityType = "strikethrough"
|
||||||
|
// MessageEntitySpoiler identifies spoiler text.
|
||||||
|
MessageEntitySpoiler MessageEntityType = "spoiler"
|
||||||
|
// MessageEntityBlockquote identifies a blockquote entity.
|
||||||
|
MessageEntityBlockquote MessageEntityType = "blockquote"
|
||||||
|
// MessageEntityExpandableBlockquote identifies an expandable blockquote entity.
|
||||||
MessageEntityExpandableBlockquote MessageEntityType = "expandable_blockquote"
|
MessageEntityExpandableBlockquote MessageEntityType = "expandable_blockquote"
|
||||||
MessageEntityCode MessageEntityType = "code"
|
// MessageEntityCode identifies inline code.
|
||||||
MessageEntityPre MessageEntityType = "pre"
|
MessageEntityCode MessageEntityType = "code"
|
||||||
MessageEntityTextLink MessageEntityType = "text_link"
|
// MessageEntityPre identifies a preformatted block.
|
||||||
MessageEntityTextMention MessageEntityType = "text_mention"
|
MessageEntityPre MessageEntityType = "pre"
|
||||||
MessageEntityCustomEmoji MessageEntityType = "custom_emoji"
|
// MessageEntityTextLink identifies linked text.
|
||||||
MessageEntityDateTime MessageEntityType = "date_time"
|
MessageEntityTextLink MessageEntityType = "text_link"
|
||||||
|
// MessageEntityTextMention identifies a text mention.
|
||||||
|
MessageEntityTextMention MessageEntityType = "text_mention"
|
||||||
|
// MessageEntityCustomEmoji identifies a custom emoji entity.
|
||||||
|
MessageEntityCustomEmoji MessageEntityType = "custom_emoji"
|
||||||
|
// MessageEntityDateTime identifies a date-time entity.
|
||||||
|
MessageEntityDateTime MessageEntityType = "date_time"
|
||||||
)
|
)
|
||||||
|
|
||||||
// MessageEntity represents one special entity in a text message.
|
// MessageEntity represents one special entity in a text message.
|
||||||
@@ -121,12 +141,12 @@ type ReplyParameters struct {
|
|||||||
MessageID int `json:"message_id"`
|
MessageID int `json:"message_id"`
|
||||||
ChatID int64 `json:"chat_id,omitempty"`
|
ChatID int64 `json:"chat_id,omitempty"`
|
||||||
|
|
||||||
AllowSendingWithoutReply bool `json:"allow_sending_without_reply,omitempty"`
|
AllowSendingWithoutReply bool `json:"allow_sending_without_reply,omitempty"`
|
||||||
Quote string `json:"quote,omitempty"`
|
Quote string `json:"quote,omitempty"`
|
||||||
QuoteParsingMode string `json:"quote_parsing_mode,omitempty"`
|
QuoteParsingMode string `json:"quote_parsing_mode,omitempty"`
|
||||||
QuoteEntities []*MessageEntity `json:"quote_entities,omitempty"`
|
QuoteEntities []MessageEntity `json:"quote_entities,omitempty"`
|
||||||
QuotePosition int `json:"quote_position,omitempty"`
|
QuotePosition int `json:"quote_position,omitempty"`
|
||||||
ChecklistTaskID int `json:"checklist_task_id,omitempty"`
|
ChecklistTaskID int `json:"checklist_task_id,omitempty"`
|
||||||
}
|
}
|
||||||
|
|
||||||
// LinkPreviewOptions describes the options used for link preview generation.
|
// LinkPreviewOptions describes the options used for link preview generation.
|
||||||
@@ -166,8 +186,11 @@ type InlineKeyboardMarkup struct {
|
|||||||
type KeyboardButtonStyle string
|
type KeyboardButtonStyle string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
KeyboardButtonStyleDanger KeyboardButtonStyle = "danger"
|
// KeyboardButtonStyleDanger marks a destructive keyboard button.
|
||||||
|
KeyboardButtonStyleDanger KeyboardButtonStyle = "danger"
|
||||||
|
// KeyboardButtonStyleSuccess marks a confirmatory keyboard button.
|
||||||
KeyboardButtonStyleSuccess KeyboardButtonStyle = "success"
|
KeyboardButtonStyleSuccess KeyboardButtonStyle = "success"
|
||||||
|
// KeyboardButtonStylePrimary marks a primary keyboard button.
|
||||||
KeyboardButtonStylePrimary KeyboardButtonStyle = "primary"
|
KeyboardButtonStylePrimary KeyboardButtonStyle = "primary"
|
||||||
)
|
)
|
||||||
|
|
||||||
@@ -255,32 +278,34 @@ type CallbackQuery struct {
|
|||||||
// InputPollOption contains information about one answer option in a poll to be sent.
|
// InputPollOption contains information about one answer option in a poll to be sent.
|
||||||
// See https://core.telegram.org/bots/api#inputpolloption
|
// See https://core.telegram.org/bots/api#inputpolloption
|
||||||
type InputPollOption struct {
|
type InputPollOption struct {
|
||||||
Text string `json:"text"`
|
Text string `json:"text"`
|
||||||
TextParseMode ParseMode `json:"text_parse_mode,omitempty"`
|
TextParseMode ParseMode `json:"text_parse_mode,omitempty"`
|
||||||
TextEntities []*MessageEntity `json:"text_entities,omitempty"`
|
TextEntities []MessageEntity `json:"text_entities,omitempty"`
|
||||||
}
|
}
|
||||||
|
|
||||||
// PollType represents the type of a poll.
|
// PollType represents the type of a poll.
|
||||||
type PollType string
|
type PollType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
|
// PollTypeRegular identifies a regular poll.
|
||||||
PollTypeRegular PollType = "regular"
|
PollTypeRegular PollType = "regular"
|
||||||
PollTypeQuiz PollType = "quiz"
|
// PollTypeQuiz identifies a quiz poll.
|
||||||
|
PollTypeQuiz PollType = "quiz"
|
||||||
)
|
)
|
||||||
|
|
||||||
// InputChecklistTask describes a task in a checklist.
|
// InputChecklistTask describes a task in a checklist.
|
||||||
type InputChecklistTask struct {
|
type InputChecklistTask struct {
|
||||||
ID int `json:"id"`
|
ID int `json:"id"`
|
||||||
Text string `json:"text"`
|
Text string `json:"text"`
|
||||||
ParseMode ParseMode `json:"parse_mode,omitempty"`
|
ParseMode ParseMode `json:"parse_mode,omitempty"`
|
||||||
TextEntities []*MessageEntity `json:"text_entities,omitempty"`
|
TextEntities []MessageEntity `json:"text_entities,omitempty"`
|
||||||
}
|
}
|
||||||
|
|
||||||
// InputChecklist represents a checklist to be sent.
|
// InputChecklist represents a checklist to be sent.
|
||||||
type InputChecklist struct {
|
type InputChecklist struct {
|
||||||
Title string `json:"title"`
|
Title string `json:"title"`
|
||||||
ParseMode ParseMode `json:"parse_mode,omitempty"`
|
ParseMode ParseMode `json:"parse_mode,omitempty"`
|
||||||
TitleEntities []*MessageEntity `json:"title_entities,omitempty"`
|
TitleEntities []MessageEntity `json:"title_entities,omitempty"`
|
||||||
Tasks []InputChecklistTask `json:"tasks"`
|
Tasks []InputChecklistTask `json:"tasks"`
|
||||||
OtherCanAddTasks bool `json:"other_can_add_tasks,omitempty"`
|
OtherCanAddTasks bool `json:"other_can_add_tasks,omitempty"`
|
||||||
OtherCanMarkTasksAsDone bool `json:"other_can_mark_tasks_as_done,omitempty"`
|
OtherCanMarkTasksAsDone bool `json:"other_can_mark_tasks_as_done,omitempty"`
|
||||||
@@ -290,14 +315,23 @@ type InputChecklist struct {
|
|||||||
type ChatActionType string
|
type ChatActionType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
ChatActionTyping ChatActionType = "typing"
|
// ChatActionTyping tells Telegram the bot is typing.
|
||||||
ChatActionUploadPhoto ChatActionType = "upload_photo"
|
ChatActionTyping ChatActionType = "typing"
|
||||||
ChatActionUploadVideo ChatActionType = "upload_video"
|
// ChatActionUploadPhoto tells Telegram the bot is uploading a photo.
|
||||||
ChatActionUploadVoice ChatActionType = "upload_voice"
|
ChatActionUploadPhoto ChatActionType = "upload_photo"
|
||||||
ChatActionUploadDocument ChatActionType = "upload_document"
|
// ChatActionUploadVideo tells Telegram the bot is uploading a video.
|
||||||
ChatActionChooseSticker ChatActionType = "choose_sticker"
|
ChatActionUploadVideo ChatActionType = "upload_video"
|
||||||
ChatActionFindLocation ChatActionType = "find_location"
|
// ChatActionUploadVoice tells Telegram the bot is uploading a voice message.
|
||||||
|
ChatActionUploadVoice ChatActionType = "upload_voice"
|
||||||
|
// ChatActionUploadDocument tells Telegram the bot is uploading a document.
|
||||||
|
ChatActionUploadDocument ChatActionType = "upload_document"
|
||||||
|
// ChatActionChooseSticker tells Telegram the bot is choosing a sticker.
|
||||||
|
ChatActionChooseSticker ChatActionType = "choose_sticker"
|
||||||
|
// ChatActionFindLocation tells Telegram the bot is finding a location.
|
||||||
|
ChatActionFindLocation ChatActionType = "find_location"
|
||||||
|
// ChatActionUploadVideoNote tells Telegram the bot is uploading a video note.
|
||||||
ChatActionUploadVideoNote ChatActionType = "upload_video_note"
|
ChatActionUploadVideoNote ChatActionType = "upload_video_note"
|
||||||
|
// ChatActionUploadVideoNone is a deprecated alias for ChatActionUploadVideoNote.
|
||||||
ChatActionUploadVideoNone ChatActionType = ChatActionUploadVideoNote
|
ChatActionUploadVideoNone ChatActionType = ChatActionUploadVideoNote
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|||||||
+31
-4
@@ -170,6 +170,7 @@ func (api *API) GetFileWithContext(ctx context.Context, params GetFileP) (File,
|
|||||||
|
|
||||||
// GetFileByLink downloads a file from Telegram's file server using the provided file link.
|
// GetFileByLink downloads a file from Telegram's file server using the provided file link.
|
||||||
// The link is usually obtained from File.FilePath.
|
// The link is usually obtained from File.FilePath.
|
||||||
|
// For large files, prefer OpenFileByLink or OpenFileByLinkWithContext to stream the response body.
|
||||||
// See https://core.telegram.org/bots/api#file
|
// See https://core.telegram.org/bots/api#file
|
||||||
func (api *API) GetFileByLink(link string) ([]byte, error) {
|
func (api *API) GetFileByLink(link string) ([]byte, error) {
|
||||||
return api.getFileByLink(context.Background(), link)
|
return api.getFileByLink(context.Background(), link)
|
||||||
@@ -177,12 +178,38 @@ func (api *API) GetFileByLink(link string) ([]byte, error) {
|
|||||||
|
|
||||||
// GetFileByLinkWithContext is the context-aware variant of GetFileByLink.
|
// GetFileByLinkWithContext is the context-aware variant of GetFileByLink.
|
||||||
// It executes the same request but uses ctx for cancellation and deadlines.
|
// It executes the same request but uses ctx for cancellation and deadlines.
|
||||||
|
// For large files, prefer OpenFileByLinkWithContext to stream the response body.
|
||||||
// See https://core.telegram.org/bots/api#file
|
// See https://core.telegram.org/bots/api#file
|
||||||
func (api *API) GetFileByLinkWithContext(ctx context.Context, link string) ([]byte, error) {
|
func (api *API) GetFileByLinkWithContext(ctx context.Context, link string) ([]byte, error) {
|
||||||
return api.getFileByLink(ctx, link)
|
return api.getFileByLink(ctx, link)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// OpenFileByLink opens a streaming response body for a file hosted on Telegram's file server.
|
||||||
|
// The caller must close the returned ReadCloser.
|
||||||
|
// See https://core.telegram.org/bots/api#file
|
||||||
|
func (api *API) OpenFileByLink(link string) (io.ReadCloser, error) {
|
||||||
|
return api.openFileByLink(context.Background(), link)
|
||||||
|
}
|
||||||
|
|
||||||
|
// OpenFileByLinkWithContext is the context-aware variant of OpenFileByLink.
|
||||||
|
// The caller must close the returned ReadCloser.
|
||||||
|
// See https://core.telegram.org/bots/api#file
|
||||||
|
func (api *API) OpenFileByLinkWithContext(ctx context.Context, link string) (io.ReadCloser, error) {
|
||||||
|
return api.openFileByLink(ctx, link)
|
||||||
|
}
|
||||||
|
|
||||||
func (api *API) getFileByLink(ctx context.Context, link string) ([]byte, error) {
|
func (api *API) getFileByLink(ctx context.Context, link string) ([]byte, error) {
|
||||||
|
body, err := api.openFileByLink(ctx, link)
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
defer func() {
|
||||||
|
_ = body.Close()
|
||||||
|
}()
|
||||||
|
return io.ReadAll(body)
|
||||||
|
}
|
||||||
|
|
||||||
|
func (api *API) openFileByLink(ctx context.Context, link string) (io.ReadCloser, error) {
|
||||||
methodPrefix := ""
|
methodPrefix := ""
|
||||||
if api.useTestServer {
|
if api.useTestServer {
|
||||||
methodPrefix = "/test"
|
methodPrefix = "/test"
|
||||||
@@ -199,15 +226,15 @@ func (api *API) getFileByLink(ctx context.Context, link string) ([]byte, error)
|
|||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
defer func() {
|
|
||||||
_ = res.Body.Close()
|
|
||||||
}()
|
|
||||||
if res.StatusCode < http.StatusOK || res.StatusCode >= http.StatusMultipleChoices {
|
if res.StatusCode < http.StatusOK || res.StatusCode >= http.StatusMultipleChoices {
|
||||||
|
defer func() {
|
||||||
|
_ = res.Body.Close()
|
||||||
|
}()
|
||||||
body, readErr := io.ReadAll(io.LimitReader(res.Body, 4<<10))
|
body, readErr := io.ReadAll(io.LimitReader(res.Body, 4<<10))
|
||||||
if readErr != nil {
|
if readErr != nil {
|
||||||
return nil, fmt.Errorf("unexpected status %d", res.StatusCode)
|
return nil, fmt.Errorf("unexpected status %d", res.StatusCode)
|
||||||
}
|
}
|
||||||
return nil, fmt.Errorf("unexpected status %d: %s", res.StatusCode, string(body))
|
return nil, fmt.Errorf("unexpected status %d: %s", res.StatusCode, string(body))
|
||||||
}
|
}
|
||||||
return io.ReadAll(res.Body)
|
return res.Body, nil
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -44,6 +44,44 @@ func TestGetFileByLinkUsesConfiguredAPIURL(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestOpenFileByLinkStreamsResponseBody(t *testing.T) {
|
||||||
|
api := NewAPI(
|
||||||
|
NewAPIOpts("token").
|
||||||
|
SetAPIUrl("https://example.test").
|
||||||
|
SetHTTPClient(&http.Client{
|
||||||
|
Transport: roundTripFunc(func(req *http.Request) (*http.Response, error) {
|
||||||
|
return &http.Response{
|
||||||
|
StatusCode: http.StatusOK,
|
||||||
|
Body: io.NopCloser(strings.NewReader("streamed payload")),
|
||||||
|
}, nil
|
||||||
|
}),
|
||||||
|
}),
|
||||||
|
)
|
||||||
|
defer func() {
|
||||||
|
if err := api.Close(); err != nil {
|
||||||
|
t.Fatalf("Close returned error: %v", err)
|
||||||
|
}
|
||||||
|
}()
|
||||||
|
|
||||||
|
body, err := api.OpenFileByLink("files/report.txt")
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("OpenFileByLink returned error: %v", err)
|
||||||
|
}
|
||||||
|
defer func() {
|
||||||
|
if err := body.Close(); err != nil {
|
||||||
|
t.Fatalf("Close returned error: %v", err)
|
||||||
|
}
|
||||||
|
}()
|
||||||
|
|
||||||
|
data, err := io.ReadAll(body)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("failed to read body: %v", err)
|
||||||
|
}
|
||||||
|
if string(data) != "streamed payload" {
|
||||||
|
t.Fatalf("unexpected payload: %q", string(data))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestGetFileByLinkReturnsHTTPStatusError(t *testing.T) {
|
func TestGetFileByLinkReturnsHTTPStatusError(t *testing.T) {
|
||||||
client := &http.Client{
|
client := &http.Client{
|
||||||
Transport: roundTripFunc(func(req *http.Request) (*http.Response, error) {
|
Transport: roundTripFunc(func(req *http.Request) (*http.Response, error) {
|
||||||
|
|||||||
@@ -10,8 +10,8 @@ const (
|
|||||||
ParseHTML ParseMode = "HTML"
|
ParseHTML ParseMode = "HTML"
|
||||||
// ParseMD enables legacy Markdown style parsing.
|
// ParseMD enables legacy Markdown style parsing.
|
||||||
ParseMD ParseMode = "Markdown"
|
ParseMD ParseMode = "Markdown"
|
||||||
// ParseNone disables any parsing.
|
// ParseNone disables parse_mode and leaves plain-text requests unannotated.
|
||||||
ParseNone ParseMode = "None"
|
ParseNone ParseMode = ""
|
||||||
)
|
)
|
||||||
|
|
||||||
// EmptyParams is a placeholder for methods that take no parameters.
|
// EmptyParams is a placeholder for methods that take no parameters.
|
||||||
|
|||||||
@@ -0,0 +1,37 @@
|
|||||||
|
package tgapi
|
||||||
|
|
||||||
|
import (
|
||||||
|
"encoding/json"
|
||||||
|
"strings"
|
||||||
|
"testing"
|
||||||
|
)
|
||||||
|
|
||||||
|
func TestParseNoneOmitsParseModeInJSON(t *testing.T) {
|
||||||
|
data, err := json.Marshal(SendMessageP{
|
||||||
|
ChatID: 42,
|
||||||
|
Text: "hello",
|
||||||
|
ParseMode: ParseNone,
|
||||||
|
})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("Marshal returned error: %v", err)
|
||||||
|
}
|
||||||
|
|
||||||
|
if strings.Contains(string(data), `"parse_mode"`) {
|
||||||
|
t.Fatalf("expected parse_mode to be omitted, got %s", string(data))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestParseModeStillSerializesExplicitModes(t *testing.T) {
|
||||||
|
data, err := json.Marshal(SendMessageP{
|
||||||
|
ChatID: 42,
|
||||||
|
Text: "hello",
|
||||||
|
ParseMode: ParseMDV2,
|
||||||
|
})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("Marshal returned error: %v", err)
|
||||||
|
}
|
||||||
|
|
||||||
|
if !strings.Contains(string(data), `"parse_mode":"MarkdownV2"`) {
|
||||||
|
t.Fatalf("expected MarkdownV2 parse_mode, got %s", string(data))
|
||||||
|
}
|
||||||
|
}
|
||||||
+22
-62
@@ -5,44 +5,35 @@ import (
|
|||||||
"sync"
|
"sync"
|
||||||
)
|
)
|
||||||
|
|
||||||
// workerPool — приватная структура, управляющая пулом воркеров.
|
|
||||||
// Внешний код не может создавать или напрямую взаимодействовать с этой структурой.
|
|
||||||
// Используется только через экспортируемые методы newWorkerPool, start, stop, submit.
|
|
||||||
type workerPool struct {
|
type workerPool struct {
|
||||||
taskCh chan requestEnvelope // канал для принятия задач (буферизованный)
|
taskCh chan requestEnvelope
|
||||||
queueSize int // максимальный размер очереди
|
queueSize int
|
||||||
workers int // количество воркеров (горутин)
|
workers int
|
||||||
wg sync.WaitGroup // синхронизирует завершение всех воркеров при остановке
|
wg sync.WaitGroup
|
||||||
quit chan struct{} // канал для сигнала остановки
|
quit chan struct{}
|
||||||
stopOnce sync.Once // гарантирует идемпотентную остановку пула
|
stopOnce sync.Once
|
||||||
started bool // флаг, указывающий, запущен ли пул
|
started bool
|
||||||
stopped bool // флаг, указывающий, что пул остановлен
|
stopped bool
|
||||||
startedMu sync.Mutex // мьютекс для безопасного доступа к started
|
startedMu sync.Mutex
|
||||||
}
|
}
|
||||||
|
|
||||||
// requestEnvelope — приватная структура, инкапсулирующая задачу и канал для результата.
|
|
||||||
// Используется только внутри пакета для передачи задач воркерам.
|
|
||||||
type requestEnvelope struct {
|
type requestEnvelope struct {
|
||||||
ctx context.Context // контекст конкретной задачи
|
ctx context.Context
|
||||||
doFunc func(context.Context) (any, error) // функция, выполняющая запрос
|
doFunc func(context.Context) (any, error)
|
||||||
resultCh chan requestResult // канал, через который воркер вернёт результат
|
resultCh chan requestResult
|
||||||
}
|
}
|
||||||
|
|
||||||
// requestResult — приватная структура, представляющая результат выполнения задачи.
|
|
||||||
// Внешний код получает его через канал, но не знает структуры — только через <-chan requestResult.
|
|
||||||
type requestResult struct {
|
type requestResult struct {
|
||||||
value any // значение, возвращённое задачей
|
value any
|
||||||
err error // ошибка, если возникла
|
err error
|
||||||
}
|
}
|
||||||
|
|
||||||
// newWorkerPool создаёт новый пул воркеров с заданным количеством горутин и размером очереди.
|
|
||||||
// Это единственный способ создать workerPool — внешний код не может создать его напрямую.
|
|
||||||
func newWorkerPool(workers int, queueSize int) *workerPool {
|
func newWorkerPool(workers int, queueSize int) *workerPool {
|
||||||
if workers <= 0 {
|
if workers <= 0 {
|
||||||
workers = 1 // защита от некорректных значений
|
workers = 1
|
||||||
}
|
}
|
||||||
if queueSize <= 0 {
|
if queueSize <= 0 {
|
||||||
queueSize = 100 // разумный дефолт
|
queueSize = 100
|
||||||
}
|
}
|
||||||
|
|
||||||
return &workerPool{
|
return &workerPool{
|
||||||
@@ -53,43 +44,32 @@ func newWorkerPool(workers int, queueSize int) *workerPool {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// start запускает воркеры (горутины), которые будут обрабатывать задачи из очереди.
|
|
||||||
// Метод идемпотентен: если пул уже запущен — ничего не делает.
|
|
||||||
// Должен вызываться перед первым вызовом submit.
|
|
||||||
func (p *workerPool) start() {
|
func (p *workerPool) start() {
|
||||||
p.startedMu.Lock()
|
p.startedMu.Lock()
|
||||||
defer p.startedMu.Unlock()
|
defer p.startedMu.Unlock()
|
||||||
if p.started {
|
if p.started {
|
||||||
return // уже запущен — ничего не делаем
|
return
|
||||||
}
|
}
|
||||||
p.started = true
|
p.started = true
|
||||||
|
|
||||||
// Запускаем воркеры — каждый будет обрабатывать задачи в бесконечном цикле
|
|
||||||
for i := 0; i < p.workers; i++ {
|
for i := 0; i < p.workers; i++ {
|
||||||
p.wg.Add(1)
|
p.wg.Add(1)
|
||||||
go p.worker() // запускаем горутину
|
go p.worker()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// stop останавливает пул воркеров.
|
|
||||||
// Отправляет сигнал остановки через quit-канал и ждёт завершения всех активных задач.
|
|
||||||
// Безопасно вызывать многократно — после остановки повторные вызовы не имеют эффекта.
|
|
||||||
func (p *workerPool) stop() {
|
func (p *workerPool) stop() {
|
||||||
p.stopOnce.Do(func() {
|
p.stopOnce.Do(func() {
|
||||||
p.startedMu.Lock()
|
p.startedMu.Lock()
|
||||||
p.stopped = true
|
p.stopped = true
|
||||||
p.started = false
|
p.started = false
|
||||||
close(p.quit) // сигнал для всех воркеров — выйти из цикла
|
close(p.quit)
|
||||||
p.startedMu.Unlock()
|
p.startedMu.Unlock()
|
||||||
|
|
||||||
p.wg.Wait() // ждём, пока все воркеры завершатся
|
p.wg.Wait()
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
// submit отправляет задачу в очередь и возвращает канал, через который будет получен результат.
|
|
||||||
// Если очередь переполнена — возвращает ErrPoolQueueFull.
|
|
||||||
// Канал результата имеет буфер 1, чтобы не блокировать воркера при записи.
|
|
||||||
// Контекст используется для отмены задачи, если клиент отменил запрос до отправки.
|
|
||||||
func (p *workerPool) submit(ctx context.Context, do func(context.Context) (any, error)) (<-chan requestResult, error) {
|
func (p *workerPool) submit(ctx context.Context, do func(context.Context) (any, error)) (<-chan requestResult, error) {
|
||||||
p.startedMu.Lock()
|
p.startedMu.Lock()
|
||||||
if p.stopped || !p.started {
|
if p.stopped || !p.started {
|
||||||
@@ -97,55 +77,39 @@ func (p *workerPool) submit(ctx context.Context, do func(context.Context) (any,
|
|||||||
return nil, ErrPoolStopped
|
return nil, ErrPoolStopped
|
||||||
}
|
}
|
||||||
|
|
||||||
// Проверяем, не превышена ли очередь
|
|
||||||
if len(p.taskCh) >= p.queueSize {
|
if len(p.taskCh) >= p.queueSize {
|
||||||
p.startedMu.Unlock()
|
p.startedMu.Unlock()
|
||||||
return nil, ErrPoolQueueFull
|
return nil, ErrPoolQueueFull
|
||||||
}
|
}
|
||||||
|
|
||||||
// Создаём канал для результата — буферизованный, чтобы не блокировать воркера
|
|
||||||
resultCh := make(chan requestResult, 1)
|
resultCh := make(chan requestResult, 1)
|
||||||
|
|
||||||
// Создаём обёртку задачи
|
|
||||||
envelope := requestEnvelope{
|
envelope := requestEnvelope{
|
||||||
ctx: ctx,
|
ctx: ctx,
|
||||||
doFunc: do,
|
doFunc: do,
|
||||||
resultCh: resultCh,
|
resultCh: resultCh,
|
||||||
}
|
}
|
||||||
|
|
||||||
// Пытаемся отправить задачу в очередь
|
|
||||||
select {
|
select {
|
||||||
case <-ctx.Done():
|
case <-ctx.Done():
|
||||||
p.startedMu.Unlock()
|
p.startedMu.Unlock()
|
||||||
// Клиент отменил операцию до отправки — возвращаем ошибку отмены
|
|
||||||
return nil, ctx.Err()
|
return nil, ctx.Err()
|
||||||
case p.taskCh <- envelope:
|
case p.taskCh <- envelope:
|
||||||
p.startedMu.Unlock()
|
p.startedMu.Unlock()
|
||||||
// Успешно отправлено — возвращаем канал для чтения результата
|
|
||||||
return resultCh, nil
|
return resultCh, nil
|
||||||
default:
|
default:
|
||||||
p.startedMu.Unlock()
|
p.startedMu.Unlock()
|
||||||
// Очередь переполнена — не должно происходить при проверке len(p.taskCh), но на всякий случай
|
|
||||||
return nil, ErrPoolQueueFull
|
return nil, ErrPoolQueueFull
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// worker — приватная горутина, выполняющая задачи из очереди.
|
|
||||||
// Каждый воркер работает в бесконечном цикле, пока не получит сигнал остановки.
|
|
||||||
// При получении задачи:
|
|
||||||
// - вызывает doFunc с контекстом
|
|
||||||
// - записывает результат в resultCh
|
|
||||||
// - закрывает канал, чтобы клиент мог прочитать и завершить
|
|
||||||
//
|
|
||||||
// После закрытия quit-канала — воркер завершает работу.
|
|
||||||
func (p *workerPool) worker() {
|
func (p *workerPool) worker() {
|
||||||
defer p.wg.Done() // уменьшаем WaitGroup при завершении горутины
|
defer p.wg.Done()
|
||||||
|
|
||||||
for {
|
for {
|
||||||
select {
|
select {
|
||||||
case <-p.quit:
|
case <-p.quit:
|
||||||
// Получен сигнал остановки — дренируем очередь и выходим.
|
// Drain queued work after stop. No new tasks are accepted.
|
||||||
// После stop() новые задачи не принимаются.
|
|
||||||
for {
|
for {
|
||||||
select {
|
select {
|
||||||
case envelope := <-p.taskCh:
|
case envelope := <-p.taskCh:
|
||||||
@@ -162,14 +126,10 @@ func (p *workerPool) worker() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
func (p *workerPool) executeEnvelope(envelope requestEnvelope) {
|
func (p *workerPool) executeEnvelope(envelope requestEnvelope) {
|
||||||
// Выполняем задачу с переданным контекстом (клиентский или общий)
|
|
||||||
value, err := envelope.doFunc(envelope.ctx)
|
value, err := envelope.doFunc(envelope.ctx)
|
||||||
|
|
||||||
// Записываем результат в канал — не блокируем, т.к. буфер 1
|
|
||||||
envelope.resultCh <- requestResult{
|
envelope.resultCh <- requestResult{
|
||||||
value: value,
|
value: value,
|
||||||
err: err,
|
err: err,
|
||||||
}
|
}
|
||||||
// Закрываем канал — клиент знает, что результат пришёл и больше не будет
|
|
||||||
close(envelope.resultCh)
|
close(envelope.resultCh)
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1,62 @@
|
|||||||
|
package tgapi
|
||||||
|
|
||||||
|
import (
|
||||||
|
"context"
|
||||||
|
"errors"
|
||||||
|
"testing"
|
||||||
|
)
|
||||||
|
|
||||||
|
func TestWorkerPoolSubmitAfterStop(t *testing.T) {
|
||||||
|
pool := newWorkerPool(1, 1)
|
||||||
|
pool.start()
|
||||||
|
pool.stop()
|
||||||
|
|
||||||
|
if _, err := pool.submit(context.Background(), func(context.Context) (any, error) {
|
||||||
|
return nil, nil
|
||||||
|
}); !errors.Is(err, ErrPoolStopped) {
|
||||||
|
t.Fatalf("expected ErrPoolStopped, got %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestWorkerPoolQueueFull(t *testing.T) {
|
||||||
|
pool := newWorkerPool(1, 1)
|
||||||
|
pool.start()
|
||||||
|
defer pool.stop()
|
||||||
|
|
||||||
|
started := make(chan struct{})
|
||||||
|
release := make(chan struct{})
|
||||||
|
|
||||||
|
firstResult, err := pool.submit(context.Background(), func(context.Context) (any, error) {
|
||||||
|
close(started)
|
||||||
|
<-release
|
||||||
|
return "first", nil
|
||||||
|
})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("first submit returned error: %v", err)
|
||||||
|
}
|
||||||
|
<-started
|
||||||
|
|
||||||
|
secondResult, err := pool.submit(context.Background(), func(context.Context) (any, error) {
|
||||||
|
return "second", nil
|
||||||
|
})
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("second submit returned error: %v", err)
|
||||||
|
}
|
||||||
|
|
||||||
|
if _, err := pool.submit(context.Background(), func(context.Context) (any, error) {
|
||||||
|
return "third", nil
|
||||||
|
}); !errors.Is(err, ErrPoolQueueFull) {
|
||||||
|
t.Fatalf("expected ErrPoolQueueFull, got %v", err)
|
||||||
|
}
|
||||||
|
|
||||||
|
close(release)
|
||||||
|
|
||||||
|
first := <-firstResult
|
||||||
|
if first.err != nil || first.value != "first" {
|
||||||
|
t.Fatalf("unexpected first result: %+v", first)
|
||||||
|
}
|
||||||
|
second := <-secondResult
|
||||||
|
if second.err != nil || second.value != "second" {
|
||||||
|
t.Fatalf("unexpected second result: %+v", second)
|
||||||
|
}
|
||||||
|
}
|
||||||
+83
-40
@@ -6,6 +6,9 @@ import "encoding/json"
|
|||||||
type UpdateType string
|
type UpdateType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
|
// UpdateTypeUnknown marks an update whose payload does not match a known Telegram update kind.
|
||||||
|
UpdateTypeUnknown UpdateType = "unknown"
|
||||||
|
|
||||||
// UpdateTypeMessage is a regular message update.
|
// UpdateTypeMessage is a regular message update.
|
||||||
UpdateTypeMessage UpdateType = "message"
|
UpdateTypeMessage UpdateType = "message"
|
||||||
// UpdateTypeEditedMessage is an edited message update.
|
// UpdateTypeEditedMessage is an edited message update.
|
||||||
@@ -27,8 +30,6 @@ const (
|
|||||||
UpdateTypeEditedBusinessMessage UpdateType = "edited_business_message"
|
UpdateTypeEditedBusinessMessage UpdateType = "edited_business_message"
|
||||||
// UpdateTypeDeletedBusinessMessages is a deleted business messages update.
|
// UpdateTypeDeletedBusinessMessages is a deleted business messages update.
|
||||||
UpdateTypeDeletedBusinessMessages UpdateType = "deleted_business_messages"
|
UpdateTypeDeletedBusinessMessages UpdateType = "deleted_business_messages"
|
||||||
// UpdateTypeDeletedBusinessMessage is kept as a backward-compatible alias.
|
|
||||||
UpdateTypeDeletedBusinessMessage UpdateType = UpdateTypeDeletedBusinessMessages
|
|
||||||
|
|
||||||
// UpdateTypeInlineQuery is an inline query update.
|
// UpdateTypeInlineQuery is an inline query update.
|
||||||
UpdateTypeInlineQuery UpdateType = "inline_query"
|
UpdateTypeInlineQuery UpdateType = "inline_query"
|
||||||
@@ -61,6 +62,8 @@ const (
|
|||||||
// Update represents an incoming update from Telegram.
|
// Update represents an incoming update from Telegram.
|
||||||
// See https://core.telegram.org/bots/api#update
|
// See https://core.telegram.org/bots/api#update
|
||||||
type Update struct {
|
type Update struct {
|
||||||
|
Type UpdateType `json:"-"`
|
||||||
|
|
||||||
UpdateID int `json:"update_id"`
|
UpdateID int `json:"update_id"`
|
||||||
Message *Message `json:"message,omitempty"`
|
Message *Message `json:"message,omitempty"`
|
||||||
EditedMessage *Message `json:"edited_message,omitempty"`
|
EditedMessage *Message `json:"edited_message,omitempty"`
|
||||||
@@ -71,7 +74,6 @@ type Update struct {
|
|||||||
BusinessMessage *Message `json:"business_message,omitempty"`
|
BusinessMessage *Message `json:"business_message,omitempty"`
|
||||||
EditedBusinessMessage *Message `json:"edited_business_message,omitempty"`
|
EditedBusinessMessage *Message `json:"edited_business_message,omitempty"`
|
||||||
DeletedBusinessMessages *BusinessMessagesDeleted `json:"deleted_business_messages,omitempty"`
|
DeletedBusinessMessages *BusinessMessagesDeleted `json:"deleted_business_messages,omitempty"`
|
||||||
DeletedBusinessMessage *BusinessMessagesDeleted `json:"-"`
|
|
||||||
MessageReaction *MessageReactionUpdated `json:"message_reaction,omitempty"`
|
MessageReaction *MessageReactionUpdated `json:"message_reaction,omitempty"`
|
||||||
MessageReactionCount *MessageReactionCountUpdated `json:"message_reaction_count,omitempty"`
|
MessageReactionCount *MessageReactionCountUpdated `json:"message_reaction_count,omitempty"`
|
||||||
|
|
||||||
@@ -91,33 +93,72 @@ type Update struct {
|
|||||||
RemovedChatBoost *ChatBoostRemoved `json:"removed_chat_boost,omitempty"`
|
RemovedChatBoost *ChatBoostRemoved `json:"removed_chat_boost,omitempty"`
|
||||||
}
|
}
|
||||||
|
|
||||||
func (u *Update) syncDeletedBusinessMessages() {
|
// UnmarshalJSON decodes an update and derives its Type from the populated payload field.
|
||||||
if u.DeletedBusinessMessages != nil {
|
|
||||||
u.DeletedBusinessMessage = u.DeletedBusinessMessages
|
|
||||||
return
|
|
||||||
}
|
|
||||||
if u.DeletedBusinessMessage != nil {
|
|
||||||
u.DeletedBusinessMessages = u.DeletedBusinessMessage
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// UnmarshalJSON keeps the deprecated DeletedBusinessMessage alias in sync.
|
|
||||||
func (u *Update) UnmarshalJSON(data []byte) error {
|
func (u *Update) UnmarshalJSON(data []byte) error {
|
||||||
type alias Update
|
type Alias Update
|
||||||
var aux alias
|
|
||||||
|
var aux Alias
|
||||||
if err := json.Unmarshal(data, &aux); err != nil {
|
if err := json.Unmarshal(data, &aux); err != nil {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
*u = Update(aux)
|
|
||||||
u.syncDeletedBusinessMessages()
|
|
||||||
return nil
|
|
||||||
}
|
|
||||||
|
|
||||||
// MarshalJSON emits the canonical deleted_business_messages field.
|
*u = Update(aux)
|
||||||
func (u Update) MarshalJSON() ([]byte, error) {
|
|
||||||
u.syncDeletedBusinessMessages()
|
switch {
|
||||||
type alias Update
|
case u.Message != nil:
|
||||||
return json.Marshal(alias(u))
|
u.Type = UpdateTypeMessage
|
||||||
|
case u.EditedMessage != nil:
|
||||||
|
u.Type = UpdateTypeEditedMessage
|
||||||
|
case u.ChannelPost != nil:
|
||||||
|
u.Type = UpdateTypeChannelPost
|
||||||
|
case u.EditedChannelPost != nil:
|
||||||
|
u.Type = UpdateTypeEditedChannelPost
|
||||||
|
|
||||||
|
case u.BusinessConnection != nil:
|
||||||
|
u.Type = UpdateTypeBusinessConnection
|
||||||
|
case u.BusinessMessage != nil:
|
||||||
|
u.Type = UpdateTypeBusinessMessage
|
||||||
|
case u.EditedBusinessMessage != nil:
|
||||||
|
u.Type = UpdateTypeEditedBusinessMessage
|
||||||
|
case u.DeletedBusinessMessages != nil:
|
||||||
|
u.Type = UpdateTypeDeletedBusinessMessages
|
||||||
|
case u.MessageReaction != nil:
|
||||||
|
u.Type = UpdateTypeMessageReaction
|
||||||
|
case u.MessageReactionCount != nil:
|
||||||
|
u.Type = UpdateTypeMessageReactionCount
|
||||||
|
|
||||||
|
case u.InlineQuery != nil:
|
||||||
|
u.Type = UpdateTypeInlineQuery
|
||||||
|
case u.ChosenInlineResult != nil:
|
||||||
|
u.Type = UpdateTypeChosenInlineResult
|
||||||
|
case u.CallbackQuery != nil:
|
||||||
|
u.Type = UpdateTypeCallbackQuery
|
||||||
|
case u.ShippingQuery != nil:
|
||||||
|
u.Type = UpdateTypeShippingQuery
|
||||||
|
case u.PreCheckoutQuery != nil:
|
||||||
|
u.Type = UpdateTypePreCheckoutQuery
|
||||||
|
case u.PurchasedPaidMedia != nil:
|
||||||
|
u.Type = UpdateTypePurchasedPaidMedia
|
||||||
|
|
||||||
|
case u.Poll != nil:
|
||||||
|
u.Type = UpdateTypePoll
|
||||||
|
case u.PollAnswer != nil:
|
||||||
|
u.Type = UpdateTypePollAnswer
|
||||||
|
case u.MyChatMember != nil:
|
||||||
|
u.Type = UpdateTypeMyChatMember
|
||||||
|
case u.ChatMember != nil:
|
||||||
|
u.Type = UpdateTypeChatMember
|
||||||
|
case u.ChatJoinRequest != nil:
|
||||||
|
u.Type = UpdateTypeChatJoinRequest
|
||||||
|
case u.ChatBoost != nil:
|
||||||
|
u.Type = UpdateTypeChatBoost
|
||||||
|
case u.RemovedChatBoost != nil:
|
||||||
|
u.Type = UpdateTypeRemovedChatBoost
|
||||||
|
default:
|
||||||
|
u.Type = UpdateTypeUnknown
|
||||||
|
}
|
||||||
|
|
||||||
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// InlineQuery represents an incoming inline query.
|
// InlineQuery represents an incoming inline query.
|
||||||
@@ -351,19 +392,19 @@ type GiftBackground struct {
|
|||||||
|
|
||||||
// Gift represents a gift that can be sent.
|
// Gift represents a gift that can be sent.
|
||||||
type Gift struct {
|
type Gift struct {
|
||||||
ID string `json:"id"`
|
ID string `json:"id"`
|
||||||
Sticker Sticker `json:"sticker"`
|
Sticker Sticker `json:"sticker"`
|
||||||
StarCount int `json:"star_count"`
|
StarCount int `json:"star_count"`
|
||||||
UpdateStarCount *int `json:"update_star_count,omitempty"`
|
UpdateStarCount *int `json:"update_star_count,omitempty"`
|
||||||
IsPremium *bool `json:"is_premium,omitempty"`
|
IsPremium *bool `json:"is_premium,omitempty"`
|
||||||
HasColors *bool `json:"has_colors,omitempty"`
|
HasColors *bool `json:"has_colors,omitempty"`
|
||||||
TotalCount *int `json:"total_count,omitempty"`
|
TotalCount *int `json:"total_count,omitempty"`
|
||||||
RemainingCount *int `json:"remaining_count,omitempty"`
|
RemainingCount *int `json:"remaining_count,omitempty"`
|
||||||
PersonalTotalCount *int `json:"personal_total_count,omitempty"`
|
PersonalTotalCount *int `json:"personal_total_count,omitempty"`
|
||||||
PersonalRemainingCount *int `json:"personal_remaining_count,omitempty"`
|
PersonalRemainingCount *int `json:"personal_remaining_count,omitempty"`
|
||||||
Background GiftBackground `json:"background,omitempty"`
|
Background *GiftBackground `json:"background,omitempty"`
|
||||||
UniqueGiftVariantColor *int `json:"unique_gift_variant_color,omitempty"`
|
UniqueGiftVariantColor *int `json:"unique_gift_variant_color,omitempty"`
|
||||||
PublisherChat *Chat `json:"publisher_chat,omitempty"`
|
PublisherChat *Chat `json:"publisher_chat,omitempty"`
|
||||||
}
|
}
|
||||||
|
|
||||||
// Gifts represents a list of gifts.
|
// Gifts represents a list of gifts.
|
||||||
@@ -375,8 +416,10 @@ type Gifts struct {
|
|||||||
type OwnedGiftType string
|
type OwnedGiftType string
|
||||||
|
|
||||||
const (
|
const (
|
||||||
|
// OwnedGiftRegularType identifies a regular owned gift.
|
||||||
OwnedGiftRegularType OwnedGiftType = "regular"
|
OwnedGiftRegularType OwnedGiftType = "regular"
|
||||||
OwnedGiftUniqueType OwnedGiftType = "unique"
|
// OwnedGiftUniqueType identifies a unique owned gift.
|
||||||
|
OwnedGiftUniqueType OwnedGiftType = "unique"
|
||||||
)
|
)
|
||||||
|
|
||||||
// OwnedGift represents a gift owned by a user or chat.
|
// OwnedGift represents a gift owned by a user or chat.
|
||||||
@@ -388,7 +431,7 @@ type OwnedGift struct {
|
|||||||
|
|
||||||
// Fields specific to "regular" type
|
// Fields specific to "regular" type
|
||||||
Gift Gift `json:"gift"`
|
Gift Gift `json:"gift"`
|
||||||
SenderUser User `json:"sender_user,omitempty"`
|
SenderUser *User `json:"sender_user,omitempty"`
|
||||||
Text string `json:"text,omitempty"`
|
Text string `json:"text,omitempty"`
|
||||||
Entities []MessageEntity `json:"entities,omitempty"`
|
Entities []MessageEntity `json:"entities,omitempty"`
|
||||||
IsPrivate *bool `json:"is_private,omitempty"`
|
IsPrivate *bool `json:"is_private,omitempty"`
|
||||||
|
|||||||
+80
-33
@@ -6,41 +6,88 @@ import (
|
|||||||
"testing"
|
"testing"
|
||||||
)
|
)
|
||||||
|
|
||||||
func TestUpdateDeletedBusinessMessagesUnmarshalSetsAlias(t *testing.T) {
|
func TestUpdateUnmarshalSetsType(t *testing.T) {
|
||||||
var update Update
|
tests := []struct {
|
||||||
err := json.Unmarshal([]byte(`{
|
name string
|
||||||
"update_id": 1,
|
body string
|
||||||
"deleted_business_messages": {
|
want UpdateType
|
||||||
"business_connection_id": "conn",
|
}{
|
||||||
"chat": {"id": 42, "type": "private"},
|
{
|
||||||
"message_ids": [3, 5]
|
name: "deleted business messages",
|
||||||
}
|
body: `{
|
||||||
}`), &update)
|
"update_id": 1,
|
||||||
if err != nil {
|
"deleted_business_messages": {
|
||||||
t.Fatalf("Unmarshal returned error: %v", err)
|
"business_connection_id": "conn",
|
||||||
|
"chat": {"id": 42, "type": "private"},
|
||||||
|
"message_ids": [3, 5]
|
||||||
|
}
|
||||||
|
}`,
|
||||||
|
want: UpdateTypeDeletedBusinessMessages,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "callback query",
|
||||||
|
body: `{
|
||||||
|
"update_id": 2,
|
||||||
|
"callback_query": {
|
||||||
|
"id": "cb",
|
||||||
|
"from": {"id": 1, "is_bot": false, "first_name": "Test"},
|
||||||
|
"chat_instance": "instance",
|
||||||
|
"data": "payload"
|
||||||
|
}
|
||||||
|
}`,
|
||||||
|
want: UpdateTypeCallbackQuery,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "chat boost",
|
||||||
|
body: `{
|
||||||
|
"update_id": 3,
|
||||||
|
"chat_boost": {
|
||||||
|
"chat": {"id": -1001, "type": "supergroup", "title": "Boosted"},
|
||||||
|
"boost": {
|
||||||
|
"boost_id": "boost-1",
|
||||||
|
"add_date": 1735689600,
|
||||||
|
"expiration_date": 1738291600,
|
||||||
|
"source": {
|
||||||
|
"source": "premium",
|
||||||
|
"user": {"id": 1, "is_bot": false, "first_name": "Test"}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}`,
|
||||||
|
want: UpdateTypeChatBoost,
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "unknown",
|
||||||
|
body: `{"update_id":4}`,
|
||||||
|
want: UpdateTypeUnknown,
|
||||||
|
},
|
||||||
}
|
}
|
||||||
|
|
||||||
if update.DeletedBusinessMessages == nil {
|
for _, tt := range tests {
|
||||||
t.Fatal("expected DeletedBusinessMessages to be populated")
|
t.Run(tt.name, func(t *testing.T) {
|
||||||
}
|
var update Update
|
||||||
if update.DeletedBusinessMessage == nil {
|
if err := json.Unmarshal([]byte(tt.body), &update); err != nil {
|
||||||
t.Fatal("expected deprecated DeletedBusinessMessage alias to be populated")
|
t.Fatalf("Unmarshal returned error: %v", err)
|
||||||
}
|
}
|
||||||
if update.DeletedBusinessMessages != update.DeletedBusinessMessage {
|
if update.Type != tt.want {
|
||||||
t.Fatal("expected deleted business message fields to share the same payload")
|
t.Fatalf("unexpected update type: got %q want %q", update.Type, tt.want)
|
||||||
}
|
}
|
||||||
if got := update.DeletedBusinessMessages.MessageIDs; len(got) != 2 || got[0] != 3 || got[1] != 5 {
|
if tt.want == UpdateTypeChatBoost && update.ChatBoost.Boost.BoostID != "boost-1" {
|
||||||
t.Fatalf("unexpected message ids: %v", got)
|
t.Fatalf("unexpected boost id: got %q want %q", update.ChatBoost.Boost.BoostID, "boost-1")
|
||||||
|
}
|
||||||
|
})
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestUpdateMarshalUsesCanonicalDeletedBusinessMessagesField(t *testing.T) {
|
func TestUpdateMarshalOmitsSyntheticTypeField(t *testing.T) {
|
||||||
update := Update{
|
update := Update{
|
||||||
UpdateID: 1,
|
UpdateID: 1,
|
||||||
DeletedBusinessMessage: &BusinessMessagesDeleted{
|
Type: UpdateTypeCallbackQuery,
|
||||||
BusinessConnectionID: "conn",
|
CallbackQuery: &CallbackQuery{
|
||||||
Chat: Chat{ID: 42, Type: string(ChatTypePrivate)},
|
ID: "cb",
|
||||||
MessageIDs: []int{7},
|
From: User{ID: 1, FirstName: "Test"},
|
||||||
|
ChatInstance: "instance",
|
||||||
|
Data: "payload",
|
||||||
},
|
},
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -50,11 +97,8 @@ func TestUpdateMarshalUsesCanonicalDeletedBusinessMessagesField(t *testing.T) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
got := string(data)
|
got := string(data)
|
||||||
if !strings.Contains(got, `"deleted_business_messages"`) {
|
if strings.Contains(got, `"type"`) {
|
||||||
t.Fatalf("expected canonical deleted_business_messages field, got %s", got)
|
t.Fatalf("unexpected synthetic type field, got %s", got)
|
||||||
}
|
|
||||||
if strings.Contains(got, `"deleted_business_message"`) {
|
|
||||||
t.Fatalf("unexpected singular deleted_business_message field, got %s", got)
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -66,4 +110,7 @@ func TestUpdateShippingQueryIsNilWhenAbsent(t *testing.T) {
|
|||||||
if update.ShippingQuery != nil {
|
if update.ShippingQuery != nil {
|
||||||
t.Fatalf("expected ShippingQuery to be nil, got %+v", update.ShippingQuery)
|
t.Fatalf("expected ShippingQuery to be nil, got %+v", update.ShippingQuery)
|
||||||
}
|
}
|
||||||
|
if update.Type != UpdateTypeUnknown {
|
||||||
|
t.Fatalf("expected UpdateTypeUnknown, got %q", update.Type)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
+13
-9
@@ -7,6 +7,7 @@ import (
|
|||||||
"mime/multipart"
|
"mime/multipart"
|
||||||
"net/http"
|
"net/http"
|
||||||
"path/filepath"
|
"path/filepath"
|
||||||
|
"strings"
|
||||||
"time"
|
"time"
|
||||||
|
|
||||||
"git.nix13.pw/scuroneko/laniakea/utils"
|
"git.nix13.pw/scuroneko/laniakea/utils"
|
||||||
@@ -81,8 +82,12 @@ func (u *Uploader) Close() error { return u.logger.Close() }
|
|||||||
// See https://core.telegram.org/bots/api
|
// See https://core.telegram.org/bots/api
|
||||||
func (u *Uploader) GetLogger() *slog.Logger { return u.logger }
|
func (u *Uploader) GetLogger() *slog.Logger { return u.logger }
|
||||||
|
|
||||||
// UploaderRequest is a multipart file upload request to the Telegram API.
|
// UploaderRequest is a low-level multipart upload request wrapper.
|
||||||
// Use NewUploaderRequest or NewUploaderRequestWithChatID to construct one.
|
//
|
||||||
|
// Prefer method-specific helpers such as SendPhoto or SetWebhook. UploaderRequest
|
||||||
|
// is intended for advanced use cases where callers manage the method name, files,
|
||||||
|
// and request/response types themselves. In that sense it is an unsafe escape
|
||||||
|
// hatch compared with the typed uploader API.
|
||||||
type UploaderRequest[R, P any] struct {
|
type UploaderRequest[R, P any] struct {
|
||||||
method string
|
method string
|
||||||
files []UploaderFile
|
files []UploaderFile
|
||||||
@@ -90,16 +95,17 @@ type UploaderRequest[R, P any] struct {
|
|||||||
chatId int64
|
chatId int64
|
||||||
}
|
}
|
||||||
|
|
||||||
// NewUploaderRequest creates a new multipart upload request with no associated chat ID.
|
// NewUploaderRequest creates a low-level multipart upload request with no associated chat ID.
|
||||||
func NewUploaderRequest[R, P any](method string, params P, files ...UploaderFile) UploaderRequest[R, P] {
|
func NewUploaderRequest[R, P any](method string, params P, files ...UploaderFile) UploaderRequest[R, P] {
|
||||||
return UploaderRequest[R, P]{method: method, files: files, params: params, chatId: 0}
|
return UploaderRequest[R, P]{method: method, files: files, params: params, chatId: 0}
|
||||||
}
|
}
|
||||||
|
|
||||||
// NewUploaderRequestWithChatID creates a new multipart upload request with an associated chat ID.
|
// NewUploaderRequestWithChatID creates a low-level multipart upload request with an associated chat ID.
|
||||||
// The chat ID is used for per-chat rate limiting.
|
// The chat ID is used for per-chat rate limiting.
|
||||||
func NewUploaderRequestWithChatID[R, P any](method string, params P, chatId int64, files ...UploaderFile) UploaderRequest[R, P] {
|
func NewUploaderRequestWithChatID[R, P any](method string, params P, chatId int64, files ...UploaderFile) UploaderRequest[R, P] {
|
||||||
return UploaderRequest[R, P]{method: method, files: files, params: params, chatId: chatId}
|
return UploaderRequest[R, P]{method: method, files: files, params: params, chatId: chatId}
|
||||||
}
|
}
|
||||||
|
|
||||||
func (r UploaderRequest[R, P]) doRequest(ctx context.Context, up *Uploader) (R, error) {
|
func (r UploaderRequest[R, P]) doRequest(ctx context.Context, up *Uploader) (R, error) {
|
||||||
var zero R
|
var zero R
|
||||||
|
|
||||||
@@ -204,8 +210,7 @@ func (r UploaderRequest[R, P]) Do(up *Uploader) (R, error) {
|
|||||||
return r.DoWithContext(context.Background(), up)
|
return r.DoWithContext(context.Background(), up)
|
||||||
}
|
}
|
||||||
|
|
||||||
// prepareMultipart builds a multipart form body from the given files and params.
|
// Internal helper that builds a finalized multipart body from files and params.
|
||||||
// Params are encoded via utils.Encode. The writer boundary is finalized before returning.
|
|
||||||
func prepareMultipart[P any](files []UploaderFile, params P) (*bytes.Buffer, string, error) {
|
func prepareMultipart[P any](files []UploaderFile, params P) (*bytes.Buffer, string, error) {
|
||||||
buf := bytes.NewBuffer(nil)
|
buf := bytes.NewBuffer(nil)
|
||||||
w := multipart.NewWriter(buf)
|
w := multipart.NewWriter(buf)
|
||||||
@@ -238,10 +243,9 @@ func prepareMultipart[P any](files []UploaderFile, params P) (*bytes.Buffer, str
|
|||||||
return buf, w.FormDataContentType(), nil
|
return buf, w.FormDataContentType(), nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// uploaderTypeByExt infers the Telegram upload field name from a file extension.
|
// Internal helper that infers an upload field name from a file extension.
|
||||||
// Falls back to UploaderDocumentType for unrecognized extensions.
|
|
||||||
func uploaderTypeByExt(filename string) UploaderFileType {
|
func uploaderTypeByExt(filename string) UploaderFileType {
|
||||||
ext := filepath.Ext(filename)
|
ext := strings.ToLower(filepath.Ext(filename))
|
||||||
switch ext {
|
switch ext {
|
||||||
case ".jpg", ".jpeg", ".png", ".webp", ".bmp":
|
case ".jpg", ".jpeg", ".png", ".webp", ".bmp":
|
||||||
return UploaderPhotoType
|
return UploaderPhotoType
|
||||||
|
|||||||
@@ -104,6 +104,27 @@ func TestUploaderEncodesJSONFieldsAndLeavesAcceptEncodingToHTTPTransport(t *test
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestNewUploaderFileDetectsFileTypeCaseInsensitively(t *testing.T) {
|
||||||
|
tests := []struct {
|
||||||
|
name string
|
||||||
|
filename string
|
||||||
|
want UploaderFileType
|
||||||
|
}{
|
||||||
|
{name: "uppercase photo", filename: "PHOTO.JPG", want: UploaderPhotoType},
|
||||||
|
{name: "uppercase voice", filename: "voice.OGG", want: UploaderVoiceType},
|
||||||
|
{name: "unknown defaults to document", filename: "archive.BIN", want: UploaderDocumentType},
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, tt := range tests {
|
||||||
|
t.Run(tt.name, func(t *testing.T) {
|
||||||
|
file := NewUploaderFile(tt.filename, []byte("x"))
|
||||||
|
if file.field != tt.want {
|
||||||
|
t.Fatalf("unexpected uploader field: got %q want %q", file.field, tt.want)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func readMultipartRequest(req *http.Request) (map[string]string, string, []byte, error) {
|
func readMultipartRequest(req *http.Request) (map[string]string, string, []byte, error) {
|
||||||
_, params, err := mime.ParseMediaType(req.Header.Get("Content-Type"))
|
_, params, err := mime.ParseMediaType(req.Header.Get("Content-Type"))
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
|||||||
@@ -53,11 +53,15 @@ func EscapePunctuation(s string) string {
|
|||||||
return s
|
return s
|
||||||
}
|
}
|
||||||
|
|
||||||
// Version constants mirror values from the internal utils/version package.
|
|
||||||
const (
|
const (
|
||||||
|
// VersionString re-exports the module version string.
|
||||||
VersionString = utils.VersionString
|
VersionString = utils.VersionString
|
||||||
VersionMajor = utils.VersionMajor
|
// VersionMajor re-exports the module major version.
|
||||||
VersionMinor = utils.VersionMinor
|
VersionMajor = utils.VersionMajor
|
||||||
VersionPatch = utils.VersionPatch
|
// VersionMinor re-exports the module minor version.
|
||||||
VersionBeta = utils.VersionBeta
|
VersionMinor = utils.VersionMinor
|
||||||
|
// VersionPatch re-exports the module patch version.
|
||||||
|
VersionPatch = utils.VersionPatch
|
||||||
|
// VersionBeta re-exports the module prerelease counter.
|
||||||
|
VersionBeta = utils.VersionBeta
|
||||||
)
|
)
|
||||||
|
|||||||
+5
-7
@@ -9,6 +9,7 @@ import (
|
|||||||
"golang.org/x/time/rate"
|
"golang.org/x/time/rate"
|
||||||
)
|
)
|
||||||
|
|
||||||
|
// ErrDropOverflow is returned when drop mode rejects a rate-limited request.
|
||||||
var ErrDropOverflow = errors.New("drop overflow limit")
|
var ErrDropOverflow = errors.New("drop overflow limit")
|
||||||
|
|
||||||
// RateLimiter implements per-chat and global rate limiting with optional blocking.
|
// RateLimiter implements per-chat and global rate limiting with optional blocking.
|
||||||
@@ -102,7 +103,7 @@ func (rl *RateLimiter) Wait(ctx context.Context, chatID int64) error {
|
|||||||
return chatLimiter.Wait(ctx)
|
return chatLimiter.Wait(ctx)
|
||||||
}
|
}
|
||||||
|
|
||||||
// getGlobalLimiter returns the global limiter safely under read lock.
|
// Internal helper that returns the global limiter under read lock.
|
||||||
func (rl *RateLimiter) getGlobalLimiter() *rate.Limiter {
|
func (rl *RateLimiter) getGlobalLimiter() *rate.Limiter {
|
||||||
rl.globalMu.RLock()
|
rl.globalMu.RLock()
|
||||||
defer rl.globalMu.RUnlock()
|
defer rl.globalMu.RUnlock()
|
||||||
@@ -190,8 +191,7 @@ func (rl *RateLimiter) Check(ctx context.Context, dropOverflow bool, chatID int6
|
|||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// waitForGlobalUnlock blocks until global cooldown expires or context is done.
|
// Internal helper that waits for the global cooldown to expire.
|
||||||
// Does not check token bucket — only cooldown.
|
|
||||||
func (rl *RateLimiter) waitForGlobalUnlock(ctx context.Context) error {
|
func (rl *RateLimiter) waitForGlobalUnlock(ctx context.Context) error {
|
||||||
rl.globalMu.RLock()
|
rl.globalMu.RLock()
|
||||||
until := rl.globalLockUntil
|
until := rl.globalLockUntil
|
||||||
@@ -209,8 +209,7 @@ func (rl *RateLimiter) waitForGlobalUnlock(ctx context.Context) error {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// waitForChatUnlock blocks until the specified chat's cooldown expires or context is done.
|
// Internal helper that waits for a chat-specific cooldown to expire.
|
||||||
// Does not check token bucket — only cooldown.
|
|
||||||
func (rl *RateLimiter) waitForChatUnlock(ctx context.Context, chatID int64) error {
|
func (rl *RateLimiter) waitForChatUnlock(ctx context.Context, chatID int64) error {
|
||||||
rl.chatMu.RLock()
|
rl.chatMu.RLock()
|
||||||
until, ok := rl.chatLocks[chatID]
|
until, ok := rl.chatLocks[chatID]
|
||||||
@@ -228,8 +227,7 @@ func (rl *RateLimiter) waitForChatUnlock(ctx context.Context, chatID int64) erro
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// getChatLimiter returns the rate limiter for the given chat, creating it if needed.
|
// Internal helper that returns or creates a per-chat limiter.
|
||||||
// Uses 1 request per second with burst of 1 — conservative for per-user limits.
|
|
||||||
func (rl *RateLimiter) getChatLimiter(chatID int64) *rate.Limiter {
|
func (rl *RateLimiter) getChatLimiter(chatID int64) *rate.Limiter {
|
||||||
rl.chatMu.Lock()
|
rl.chatMu.Lock()
|
||||||
defer rl.chatMu.Unlock()
|
defer rl.chatMu.Unlock()
|
||||||
|
|||||||
@@ -0,0 +1,41 @@
|
|||||||
|
package utils
|
||||||
|
|
||||||
|
import (
|
||||||
|
"context"
|
||||||
|
"errors"
|
||||||
|
"testing"
|
||||||
|
"time"
|
||||||
|
)
|
||||||
|
|
||||||
|
func TestRateLimiterCheckDropOverflowHonorsGlobalLock(t *testing.T) {
|
||||||
|
rl := NewRateLimiter()
|
||||||
|
rl.SetGlobalLock(1)
|
||||||
|
|
||||||
|
if err := rl.Check(context.Background(), true, 0); !errors.Is(err, ErrDropOverflow) {
|
||||||
|
t.Fatalf("expected ErrDropOverflow, got %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestRateLimiterChatLocksAreScopedPerChat(t *testing.T) {
|
||||||
|
rl := NewRateLimiter()
|
||||||
|
rl.SetChatLock(42, 1)
|
||||||
|
|
||||||
|
if rl.Allow(42) {
|
||||||
|
t.Fatal("expected locked chat to be rejected")
|
||||||
|
}
|
||||||
|
if !rl.Allow(7) {
|
||||||
|
t.Fatal("expected unrelated chat to remain allowed")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestRateLimiterGlobalWaitRespectsContextCancellation(t *testing.T) {
|
||||||
|
rl := NewRateLimiter()
|
||||||
|
rl.SetGlobalLock(1)
|
||||||
|
|
||||||
|
ctx, cancel := context.WithTimeout(context.Background(), 10*time.Millisecond)
|
||||||
|
defer cancel()
|
||||||
|
|
||||||
|
if err := rl.GlobalWait(ctx); !errors.Is(err, context.DeadlineExceeded) {
|
||||||
|
t.Fatalf("expected DeadlineExceeded, got %v", err)
|
||||||
|
}
|
||||||
|
}
|
||||||
+1
-2
@@ -3,7 +3,6 @@ package utils
|
|||||||
import (
|
import (
|
||||||
"encoding/json"
|
"encoding/json"
|
||||||
"fmt"
|
"fmt"
|
||||||
"io"
|
|
||||||
"mime/multipart"
|
"mime/multipart"
|
||||||
"reflect"
|
"reflect"
|
||||||
"slices"
|
"slices"
|
||||||
@@ -110,6 +109,6 @@ func writeMultipartValue(w *multipart.Writer, fieldName string, value []byte) er
|
|||||||
if err != nil {
|
if err != nil {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
_, err = io.Copy(fw, strings.NewReader(string(value)))
|
_, err = fw.Write(value)
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
|
|||||||
+10
-5
@@ -1,9 +1,14 @@
|
|||||||
package utils
|
package utils
|
||||||
|
|
||||||
const (
|
const (
|
||||||
VersionString = "1.0.0-rc.7"
|
// VersionString is the module version string.
|
||||||
VersionMajor = 1
|
VersionString = "1.0.0-rc.11"
|
||||||
VersionMinor = 0
|
// VersionMajor is the module major version.
|
||||||
VersionPatch = 0
|
VersionMajor = 1
|
||||||
VersionBeta = 7
|
// VersionMinor is the module minor version.
|
||||||
|
VersionMinor = 0
|
||||||
|
// VersionPatch is the module patch version.
|
||||||
|
VersionPatch = 0
|
||||||
|
// VersionBeta is the prerelease counter for the current version.
|
||||||
|
VersionBeta = 11
|
||||||
)
|
)
|
||||||
|
|||||||
Reference in New Issue
Block a user