mirror of
https://github.com/TheFunny/TelegramTwitterMediaBot.git
synced 2026-09-23 23:32:05 +00:00
Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
ec65c3ce74
|
||
|
|
893ab7a1e0
|
||
|
|
bd032e3d68
|
||
|
|
dbda6ec1c2
|
||
|
|
0a9ff58a69
|
||
|
|
c2d7c8406e
|
||
|
|
abdc27ed5e
|
||
|
|
3fb8421c3a
|
||
|
|
475cfd18f9
|
||
|
|
0a577600fd
|
||
|
|
32254fa807
|
||
|
|
11c04b66dc
|
||
|
|
2f741e5f4b
|
||
|
|
89c4642e1c
|
@@ -26,6 +26,10 @@
|
|||||||
*.db
|
*.db
|
||||||
LICENSE
|
LICENSE
|
||||||
README.md
|
README.md
|
||||||
|
# Documentation and scratch files: the build only ever reads the manifests,
|
||||||
|
# `crates/` and the entrypoint script.
|
||||||
|
docs/
|
||||||
|
*.md
|
||||||
data/
|
data/
|
||||||
cert/
|
cert/
|
||||||
nginx-certs/
|
nginx-certs/
|
||||||
|
|||||||
@@ -0,0 +1,32 @@
|
|||||||
|
version: 2
|
||||||
|
|
||||||
|
# Pairs with the `actions-rust-lang/audit` gate in ci.yml: the gate reports
|
||||||
|
# advisories in Cargo.lock, this is what actually moves the dependencies.
|
||||||
|
# Patch bumps are batched into one PR; minor/major stay separate so they get
|
||||||
|
# reviewed and tested individually.
|
||||||
|
updates:
|
||||||
|
- package-ecosystem: cargo
|
||||||
|
directory: /
|
||||||
|
schedule:
|
||||||
|
interval: weekly
|
||||||
|
open-pull-requests-limit: 5
|
||||||
|
groups:
|
||||||
|
cargo-patch:
|
||||||
|
applies-to: version-updates
|
||||||
|
patterns: ['*']
|
||||||
|
update-types: ['patch']
|
||||||
|
|
||||||
|
# The workflow actions are pinned to commit SHAs; that pin is what makes
|
||||||
|
# bumping them a manual chore, so let the bot do it.
|
||||||
|
- package-ecosystem: github-actions
|
||||||
|
directory: /
|
||||||
|
schedule:
|
||||||
|
interval: weekly
|
||||||
|
open-pull-requests-limit: 5
|
||||||
|
|
||||||
|
# The Dockerfile's base images (rust:1-bookworm, debian:bookworm-slim).
|
||||||
|
- package-ecosystem: docker
|
||||||
|
directory: /
|
||||||
|
schedule:
|
||||||
|
interval: monthly
|
||||||
|
open-pull-requests-limit: 3
|
||||||
+49
-14
@@ -4,15 +4,19 @@ name: CI
|
|||||||
# job that exercises the real source sites and the token-gated pixiv tests.
|
# job that exercises the real source sites and the token-gated pixiv tests.
|
||||||
#
|
#
|
||||||
# Layering:
|
# Layering:
|
||||||
# test — fmt + clippy + the full offline unit suite + cargo-audit
|
# test — fmt + clippy + the full offline unit suite + a release-profile
|
||||||
# dependency gate. Runs on every push and PR, including forks
|
# build + cargo-audit dependency gate. Runs on every push and PR,
|
||||||
# (it needs no secrets).
|
# including forks (it needs no secrets).
|
||||||
# live — the #[ignore]d live-network tests plus the pixiv tests that are
|
# live — the #[ignore]d live-network tests plus the pixiv tests that are
|
||||||
# gated on PIXIV_REFRESH_TOKEN. Runs on schedule / manual dispatch
|
# gated on PIXIV_REFRESH_TOKEN. Runs on schedule / manual dispatch
|
||||||
# / tag pushes only, because pull requests from forks cannot read
|
# / tag pushes only, because pull requests from forks cannot read
|
||||||
# repository secrets. continue-on-error keeps a flaky external site
|
# repository secrets. continue-on-error keeps a flaky external site
|
||||||
# from blocking, while the run still records the outcome.
|
# from blocking, while the run still records the outcome.
|
||||||
#
|
#
|
||||||
|
# Every action is pinned to a commit SHA (Dependabot keeps the pins current);
|
||||||
|
# `dtolnay/rust-toolchain` deliberately stays on its channel ref, because the
|
||||||
|
# ref itself is what selects the toolchain (`@stable` = install stable).
|
||||||
|
#
|
||||||
# Test gating convention (keep in sync with AGENTS.md "Testing & QA"):
|
# Test gating convention (keep in sync with AGENTS.md "Testing & QA"):
|
||||||
# - pure unit tests: plain #[test] / #[tokio::test], always run.
|
# - pure unit tests: plain #[test] / #[tokio::test], always run.
|
||||||
# - live-network tests: #[ignore = "live network: ..."], only run here.
|
# - live-network tests: #[ignore = "live network: ..."], only run here.
|
||||||
@@ -28,44 +32,75 @@ on:
|
|||||||
- cron: '0 3 * * 1'
|
- cron: '0 3 * * 1'
|
||||||
workflow_dispatch:
|
workflow_dispatch:
|
||||||
|
|
||||||
|
permissions:
|
||||||
|
contents: read
|
||||||
|
|
||||||
|
# A newer push to the same ref supersedes the older run; without this every
|
||||||
|
# intermediate commit of a PR branch keeps a runner busy to completion.
|
||||||
|
concurrency:
|
||||||
|
group: ci-${{ github.ref }}
|
||||||
|
cancel-in-progress: true
|
||||||
|
|
||||||
|
env:
|
||||||
|
# Panicking tests print their backtrace; free when nothing fails.
|
||||||
|
RUST_BACKTRACE: 1
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
test:
|
test:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
# Generous on purpose: the release-profile build below is cold on the very
|
||||||
|
# first run (thin LTO + codegen-units = 1 across every dependency), and a
|
||||||
|
# timeout there would kill the job *before* rust-cache saves its cache —
|
||||||
|
# leaving every later run cold again.
|
||||||
|
timeout-minutes: 45
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v7
|
- uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
|
||||||
- uses: dtolnay/rust-toolchain@stable
|
- uses: dtolnay/rust-toolchain@stable
|
||||||
with:
|
with:
|
||||||
components: clippy, rustfmt
|
components: clippy, rustfmt
|
||||||
- uses: Swatinem/rust-cache@v2
|
- uses: Swatinem/rust-cache@6323deb102c322ba6fcbdcafc7e3dddab59af2b6 # v2
|
||||||
|
# `--locked` on every cargo invocation: the version bump edits
|
||||||
|
# Cargo.lock by hand (AGENTS.md), so a stale lock must fail here instead
|
||||||
|
# of being silently re-resolved — otherwise CI tests a different
|
||||||
|
# dependency set than the one committed, and than the one the released
|
||||||
|
# image is built from.
|
||||||
- name: Check formatting
|
- name: Check formatting
|
||||||
run: cargo fmt --check
|
run: cargo fmt --check
|
||||||
- name: Lint (deny warnings)
|
- name: Lint (deny warnings)
|
||||||
run: cargo clippy --workspace --all-targets -- -D warnings
|
run: cargo clippy --workspace --all-targets --locked -- -D warnings
|
||||||
- name: Run offline tests
|
- name: Run offline tests
|
||||||
run: cargo test --workspace
|
run: cargo test --workspace --locked
|
||||||
|
# The release profile (lto/strip/codegen-units=1, overflow checks off)
|
||||||
|
# was otherwise only exercised by the Docker build on master/tag. Same
|
||||||
|
# package the Dockerfile builds; the cache keeps it cheap after the
|
||||||
|
# first run.
|
||||||
|
- name: Build release profile
|
||||||
|
run: cargo build --release --locked -p xmedia-bot
|
||||||
# Dependency vulnerability gate: fails the build when a crate in
|
# Dependency vulnerability gate: fails the build when a crate in
|
||||||
# Cargo.lock has an unfixed security advisory. Unmaintained/unsound
|
# Cargo.lock has an unfixed security advisory. Unmaintained/unsound
|
||||||
# *warnings* (dotenv, proc-macro-error2, anyhow transitive) do not fail
|
# *warnings* (dotenv, proc-macro-error2, anyhow transitive) do not fail
|
||||||
# the build by default; the advisory DB is cached across runs.
|
# the build by default; the advisory DB is cached across runs.
|
||||||
- name: Audit dependencies
|
- name: Audit dependencies
|
||||||
uses: actions-rust-lang/audit@v1
|
uses: actions-rust-lang/audit@72c09e02f132669d52284a3323acdb503cfc1a24 # v1
|
||||||
|
|
||||||
live:
|
live:
|
||||||
needs: test
|
needs: test
|
||||||
if: github.event_name == 'schedule' || github.event_name == 'workflow_dispatch' || startsWith(github.ref, 'refs/tags/v')
|
if: github.event_name == 'schedule' || github.event_name == 'workflow_dispatch' || startsWith(github.ref, 'refs/tags/v')
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
timeout-minutes: 30
|
||||||
continue-on-error: true
|
continue-on-error: true
|
||||||
env:
|
env:
|
||||||
PIXIV_REFRESH_TOKEN: ${{ secrets.PIXIV_REFRESH_TOKEN }}
|
PIXIV_REFRESH_TOKEN: ${{ secrets.PIXIV_REFRESH_TOKEN }}
|
||||||
TWITTER_AUTH_TOKEN: ${{ secrets.TWITTER_AUTH_TOKEN }}
|
TWITTER_AUTH_TOKEN: ${{ secrets.TWITTER_AUTH_TOKEN }}
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v7
|
- uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
|
||||||
- uses: dtolnay/rust-toolchain@stable
|
- uses: dtolnay/rust-toolchain@stable
|
||||||
- uses: Swatinem/rust-cache@v2
|
- uses: Swatinem/rust-cache@6323deb102c322ba6fcbdcafc7e3dddab59af2b6 # v2
|
||||||
# Full suite: with the secret present, the pixiv token-gated tests run;
|
# Everything network- or secret-gated lives in x-media, and the bot
|
||||||
# without it they skip themselves. Live tests stay #[ignore]d here.
|
# crate's suite (MockSender + tempdir stores, no network) already ran in
|
||||||
|
# the `test` job — rebuilding it here bought nothing.
|
||||||
- name: Run token-gated tests
|
- name: Run token-gated tests
|
||||||
run: cargo test --workspace
|
run: cargo test -p x-media --locked
|
||||||
# The live-network tests, by the "live" name filter (all #[ignore]d).
|
# The live-network tests, by the "live" name filter (all #[ignore]d).
|
||||||
- name: Run live-network tests
|
- name: Run live-network tests
|
||||||
run: cargo test --workspace -- --ignored live
|
run: cargo test -p x-media --locked -- --ignored live
|
||||||
|
|||||||
@@ -1,28 +1,75 @@
|
|||||||
name: Build Docker Image
|
name: Build Docker Image
|
||||||
|
|
||||||
|
# Release builds (master / v* tags) plus a build-only check on pull requests
|
||||||
|
# that touch anything the image depends on — the Dockerfile's stub-source
|
||||||
|
# machinery, the ffmpeg download and the entrypoint are exactly the parts that
|
||||||
|
# would otherwise break only at release time.
|
||||||
|
#
|
||||||
|
# Actions are pinned to commit SHAs (Dependabot keeps the pins current).
|
||||||
|
|
||||||
on:
|
on:
|
||||||
push:
|
push:
|
||||||
tags:
|
tags:
|
||||||
- v*
|
- v*
|
||||||
branches:
|
branches:
|
||||||
- master
|
- master
|
||||||
|
pull_request:
|
||||||
|
paths:
|
||||||
|
- Dockerfile
|
||||||
|
- docker-entrypoint.sh
|
||||||
|
- .dockerignore
|
||||||
|
- Cargo.toml
|
||||||
|
- Cargo.lock
|
||||||
|
- .github/workflows/docker.yml
|
||||||
|
- 'crates/**/Cargo.toml'
|
||||||
|
|
||||||
env:
|
env:
|
||||||
APP_NAME: telegram-twitter-media-bot
|
APP_NAME: telegram-twitter-media-bot
|
||||||
DOCKERHUB_REPO: yoursfunny/telegram-twitter-media-bot
|
DOCKERHUB_REPO: yoursfunny/telegram-twitter-media-bot
|
||||||
|
|
||||||
|
permissions:
|
||||||
|
contents: read
|
||||||
|
|
||||||
|
# Serialize runs per ref. Never cancel in progress: a killed run would drop a
|
||||||
|
# half-finished image push.
|
||||||
|
concurrency:
|
||||||
|
group: docker-${{ github.ref }}
|
||||||
|
cancel-in-progress: false
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
# A tag push and a branch push to the same commit fire two workflow runs;
|
# A tag push and a branch push to the same commit fire two workflow runs;
|
||||||
# build only once. Tag runs always build; master runs build only when the
|
# build only once. Tag runs always build; master runs build only when the
|
||||||
# pushed commit is not already tagged (the tag run covers it).
|
# pushed commit is not already tagged (the tag run covers it).
|
||||||
should-build:
|
should-build:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
timeout-minutes: 10
|
||||||
outputs:
|
outputs:
|
||||||
build: ${{ steps.check.outputs.build }}
|
build: ${{ steps.check.outputs.build }}
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v7
|
- uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
|
||||||
with:
|
with:
|
||||||
fetch-depth: 0
|
fetch-depth: 0
|
||||||
|
# A release tag is the version claim: the manifests are bumped by hand,
|
||||||
|
# so `v1.5.1` with `Cargo.toml` still at 1.5.0 would publish an image
|
||||||
|
# whose tag lies about what is inside it (the binary carries no version).
|
||||||
|
- name: Verify the tag matches both crate versions
|
||||||
|
if: startsWith(github.ref, 'refs/tags/v')
|
||||||
|
shell: bash
|
||||||
|
run: |
|
||||||
|
tag="${GITHUB_REF_NAME#v}"
|
||||||
|
status=0
|
||||||
|
for manifest in crates/x-media/Cargo.toml crates/xmedia-bot/Cargo.toml; do
|
||||||
|
# tr -d '\r': a CRLF checkout (core.autocrlf on Windows) would
|
||||||
|
# otherwise yield "1.5.0\r" and false-fail every tag.
|
||||||
|
version="$(sed -n 's/^version = "\(.*\)"/\1/p' "$manifest" | head -1 | tr -d '\r')"
|
||||||
|
if [ "$version" != "$tag" ]; then
|
||||||
|
echo "::error file=$manifest::$manifest is at $version but the tag is v$tag"
|
||||||
|
status=1
|
||||||
|
else
|
||||||
|
echo "$manifest: $version matches v$tag"
|
||||||
|
fi
|
||||||
|
done
|
||||||
|
exit "$status"
|
||||||
- id: check
|
- id: check
|
||||||
shell: bash
|
shell: bash
|
||||||
run: |
|
run: |
|
||||||
@@ -37,10 +84,11 @@ jobs:
|
|||||||
needs: should-build
|
needs: should-build
|
||||||
if: needs.should-build.outputs.build == 'true'
|
if: needs.should-build.outputs.build == 'true'
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
timeout-minutes: 60
|
||||||
steps:
|
steps:
|
||||||
- name: Docker meta
|
- name: Docker meta
|
||||||
id: meta
|
id: meta
|
||||||
uses: docker/metadata-action@v6
|
uses: docker/metadata-action@dc802804100637a589fabce1cb79ff13a1411302 # v6
|
||||||
with:
|
with:
|
||||||
images: ${{ env.DOCKERHUB_REPO }}
|
images: ${{ env.DOCKERHUB_REPO }}
|
||||||
tags: |
|
tags: |
|
||||||
@@ -49,15 +97,15 @@ jobs:
|
|||||||
type=semver,pattern={{major}}.{{minor}}
|
type=semver,pattern={{major}}.{{minor}}
|
||||||
type=semver,pattern={{major}}
|
type=semver,pattern={{major}}
|
||||||
type=sha
|
type=sha
|
||||||
-
|
|
||||||
name: Set up QEMU
|
|
||||||
uses: docker/setup-qemu-action@v4
|
|
||||||
-
|
-
|
||||||
name: Set up Docker Buildx
|
name: Set up Docker Buildx
|
||||||
uses: docker/setup-buildx-action@v4
|
uses: docker/setup-buildx-action@f87e5991a6d7451dcb8d9637bfbc97413f497069 # v4
|
||||||
|
# Pull requests build the image to prove the Dockerfile still works, but
|
||||||
|
# must not read registry credentials (fork PRs have none).
|
||||||
-
|
-
|
||||||
name: Login to Docker Hub
|
name: Login to Docker Hub
|
||||||
uses: docker/login-action@v4
|
if: github.event_name != 'pull_request'
|
||||||
|
uses: docker/login-action@dbcb813823bdd20940b903addbd779551569679f # v4
|
||||||
with:
|
with:
|
||||||
username: ${{ secrets.DOCKERHUB_USERNAME }}
|
username: ${{ secrets.DOCKERHUB_USERNAME }}
|
||||||
password: ${{ secrets.DOCKERHUB_TOKEN }}
|
password: ${{ secrets.DOCKERHUB_TOKEN }}
|
||||||
@@ -66,15 +114,26 @@ jobs:
|
|||||||
# stage's layers so the cargo-deps and ffmpeg layers are restored
|
# stage's layers so the cargo-deps and ffmpeg layers are restored
|
||||||
# instead of re-downloaded/recompiled. The scope must be pinned to a
|
# instead of re-downloaded/recompiled. The scope must be pinned to a
|
||||||
# fixed string: the gha backend defaults to the current git ref, which
|
# fixed string: the gha backend defaults to the current git ref, which
|
||||||
# would give every new tag a cold cache on release builds.
|
# would give every new tag a cold cache on release builds. PR runs only
|
||||||
|
# read it (cache-to is empty) so they cannot evict the release cache.
|
||||||
|
#
|
||||||
|
# FFMPEG_URL/FFMPEG_SHA256 come from repository variables when set, so a
|
||||||
|
# release can pin an exact ffmpeg build (the Dockerfile default follows
|
||||||
|
# the project's `/redirect/latest/` URL, which has no sha256 sidecar).
|
||||||
|
#
|
||||||
|
# Single-arch (amd64) on purpose: adding arm64 means re-adding
|
||||||
|
# `docker/setup-qemu-action`, `platforms: linux/amd64,linux/arm64`, and
|
||||||
|
# parameterizing FFMPEG_URL by $TARGETARCH in the Dockerfile.
|
||||||
-
|
-
|
||||||
name: Build and push
|
name: Build and push
|
||||||
uses: docker/build-push-action@v7
|
uses: docker/build-push-action@c3c9e263c25d99ce0380d002d59b67737d91b0dc # v7
|
||||||
with:
|
with:
|
||||||
push: true
|
push: ${{ github.event_name != 'pull_request' }}
|
||||||
build-args: |
|
build-args: |
|
||||||
APP_NAME=${{ env.APP_NAME }}
|
APP_NAME=${{ env.APP_NAME }}
|
||||||
|
FFMPEG_URL=${{ vars.FFMPEG_URL || 'https://ffmpeg.martin-riedl.de/redirect/latest/linux/amd64/release/ffmpeg.zip' }}
|
||||||
|
FFMPEG_SHA256=${{ vars.FFMPEG_SHA256 }}
|
||||||
tags: ${{ steps.meta.outputs.tags }}
|
tags: ${{ steps.meta.outputs.tags }}
|
||||||
labels: ${{ steps.meta.outputs.labels }}
|
labels: ${{ steps.meta.outputs.labels }}
|
||||||
cache-from: type=gha,scope=tgxmb-build
|
cache-from: type=gha,scope=tgxmb-build
|
||||||
cache-to: type=gha,mode=max,scope=tgxmb-build
|
cache-to: ${{ github.event_name != 'pull_request' && 'type=gha,mode=max,scope=tgxmb-build' || '' }}
|
||||||
|
|||||||
@@ -2,11 +2,11 @@
|
|||||||
|
|
||||||
## Project Overview
|
## Project Overview
|
||||||
|
|
||||||
Telegram bot (teloxide) that turns post links from X/Twitter, Pixiv, and Bluesky into media messages (images, video, GIF) with the post's title, author, and tags. It supports batch media splitting, retry with persistence, inline queries, forward-channel rebinding with caption templates, and Pixiv ugoira→MP4 transcoding. README is in Chinese; user-facing bot strings are in English. The project is a Rust port of a Python predecessor (see `queue.rs` comments referencing `utils/task_queue.py`).
|
Telegram bot (teloxide) that turns post links from X/Twitter, Pixiv, Bluesky, and Misskey (misskey.io) into media messages (images, video, GIF) with the post's title, author, and tags. It supports batch media splitting, retry with persistence, inline queries, forward-channel rebinding with caption templates, and Pixiv ugoira→MP4 transcoding. README is in Chinese; user-facing bot strings are in English. The project is a Rust port of a Python predecessor (see `queue.rs` comments referencing `utils/task_queue.py`).
|
||||||
|
|
||||||
Two-crate Cargo workspace (both v1.4.0, edition 2024, resolver 3):
|
Two-crate Cargo workspace (both v1.6.0, edition 2024, resolver 3):
|
||||||
|
|
||||||
- **`crates/x-media`** — library that fetches and normalizes media from the three sites. Pure, no Telegram knowledge.
|
- **`crates/x-media`** — library that fetches and normalizes media from the four sites. Pure, no Telegram knowledge.
|
||||||
- **`crates/xmedia-bot`** — the bot binary: teloxide dispatcher, SQLite-backed chat state, persistent task queue.
|
- **`crates/xmedia-bot`** — the bot binary: teloxide dispatcher, SQLite-backed chat state, persistent task queue.
|
||||||
|
|
||||||
## Architecture & Data Flow
|
## Architecture & Data Flow
|
||||||
@@ -20,25 +20,28 @@ Telegram update → Dispatcher (polling or axum webhook) → dptree branches
|
|||||||
|
|
||||||
Message flow: `message_handler` extracts URLs (from `url`/`text_link` entities, text + caption, deduped) → `x_media::site::fetch(url)` → `Fetched` → builds a `Task` → `send::send_media_sequence` (media groups ≤ 9, caption on first item) or `send::send_animation`. On Telegram URL-fetch failure or size error (`send_batch_via_upload`): download via `x_media::site::download_media` to a temp file (≤ 10 MiB), sniff magic bytes (`sniff_ext`), upload via multipart; oversized items fall back to `fallback_url`. On failure: `enqueue_retry` persists resume-state `Task` into the SQLite queue → workers lease (120 s lock TTL) → retry with exponential backoff (≤ 30 s, `MAX_RETRIES = 2`) → dead-letter → `notify_failure`. Success → `post_send_actions`: edit-before-forward prompt with inline buttons, or `copy_messages` to the bound forward channel.
|
Message flow: `message_handler` extracts URLs (from `url`/`text_link` entities, text + caption, deduped) → `x_media::site::fetch(url)` → `Fetched` → builds a `Task` → `send::send_media_sequence` (media groups ≤ 9, caption on first item) or `send::send_animation`. On Telegram URL-fetch failure or size error (`send_batch_via_upload`): download via `x_media::site::download_media` to a temp file (≤ 10 MiB), sniff magic bytes (`sniff_ext`), upload via multipart; oversized items fall back to `fallback_url`. On failure: `enqueue_retry` persists resume-state `Task` into the SQLite queue → workers lease (120 s lock TTL) → retry with exponential backoff (≤ 30 s, `MAX_RETRIES = 2`) → dead-letter → `notify_failure`. Success → `post_send_actions`: edit-before-forward prompt with inline buttons, or `copy_messages` to the bound forward channel.
|
||||||
|
|
||||||
Debug command: `/test <url>` runs the same `x_media::site::fetch` and replies with `test_parse_report` (`handlers/commands.rs`) — site id, normalized cache key, source URL, title/author/tags, sensitive flag, caption and the media list — nothing is sent, cached or forwarded; the report is capped at 4000 chars and sent with HTML parse mode: raw fields are escaped, and the caption is wrapped in a `<blockquote>` so it renders exactly like the sent media caption (escaped text and links included). It uses a custom `parse_test_arg` parser (whole remainder, trimmed) because teloxide's built-in `split` parser takes exactly one space-separated token.
|
Debug command: `/debug <url>` runs the same `x_media::site::fetch` and replies with `debug_report` (`handlers/commands.rs`) — site id, normalized cache key, source URL, title/author/tags, sensitive flag, caption and the media list — nothing is sent, cached or forwarded; the report is capped at 4000 chars and sent with HTML parse mode: raw fields are escaped, and the caption is wrapped in a `<blockquote>` so it renders exactly like the sent media caption (escaped text and links included).
|
||||||
|
|
||||||
The `x-media` library: `site::fetch(url)` dispatches through the `SITES` registry (per-site `impl Site`, in order twitter → bsky → pixiv) and returns `Ok(None)` for unmatched URLs. `Fetched { source_url, caption, title, media: Vec<Media>, sensitive, site_id, … }`; `caption_with(format)` substitutes `{url} {author} {author_url} {title} {tags}`.
|
The `/test <url>` command runs the ordinary link pipeline (`urls::url_media`) with `PostSend::Suppressed`: the media is sent and cached like any other link, but the chat's `forward_channel_id`/`edit_before_forward` are ignored, so a test never forwards to the channel and never opens the edit prompt (retries and dead-letter notifications behave as usual). Both commands use a custom `parse_arg_remainder` parser (whole remainder, trimmed) because teloxide's built-in `split` parser takes exactly one space-separated token.
|
||||||
|
|
||||||
|
The `x-media` library: `site::fetch(url)` dispatches through the `SITES` registry (per-site `impl Site`, in order twitter → bsky → misskey → pixiv) and returns `Ok(None)` for unmatched URLs. `Fetched { source_url, caption, title, media: Vec<Media>, sensitive, site_id, … }`; `caption_with(format)` substitutes `{url} {author} {author_url} {title} {tags}`.
|
||||||
|
|
||||||
## Key Directories
|
## Key Directories
|
||||||
|
|
||||||
| Path | Purpose |
|
| Path | Purpose |
|
||||||
|---|---|
|
|---|---|
|
||||||
| `crates/x-media/src/` | Fetch library. `site/mod.rs` = dispatcher + `Fetched`/`FetchError`/`download_media`/`media_size`; `media.rs` = `Media` enum; `examples/fetch.rs` = end-to-end usage sample |
|
| `crates/x-media/src/` | Fetch library. `site/mod.rs` = dispatcher + `Fetched`/`FetchError`/`download_media`/`media_size`; `media.rs` = `Media` enum; `examples/fetch.rs` = end-to-end usage sample |
|
||||||
| `crates/x-media/src/site/<twitter\|pixiv\|bsky>/` | One directory per site: `mod.rs` (re-exports), `interface.rs` (PATTERN, `enabled()`, `fetch_from_url()`, `cache_key`/`is_retryable`/`media_headers`, unit struct `<Name>Site` implementing `site::Site`, `From<SiteStruct> for Fetched`), `model.rs` (serde DTOs). Pixiv adds `api.rs` (auth + transport); twitter adds `auth.rs` (logged-in GraphQL `TweetDetail` fallback for NSFW tweets, gated on `TWITTER_AUTH_TOKEN`). Twitter's `from_syndication_json` HTML-decodes the API text — syndication and GraphQL `full_text` both arrive pre-escaped (`>` `<` `&` `'`) — so the stored text is raw and the caption escapes exactly once |
|
| `crates/x-media/src/site/<twitter\|pixiv\|bsky\|misskey>/` | One directory per site: `mod.rs` (re-exports), `interface.rs` (PATTERN, `enabled()`, `fetch_from_url()`, `cache_key`/`is_retryable`/`media_headers`, unit struct `<Name>Site` implementing `site::Site`, `From<SiteStruct> for Fetched`), `model.rs` (serde DTOs). Pixiv adds `api.rs` (auth + transport); twitter adds `auth.rs` (logged-in GraphQL `TweetDetail` fallback for NSFW tweets, gated on `TWITTER_AUTH_TOKEN`). Misskey targets misskey.io only (`POST /api/notes/show`, 400+`NO_SUCH_NOTE` → NotFound). Twitter's `from_syndication_json` HTML-decodes the API text — syndication and GraphQL `full_text` both arrive pre-escaped (`>` `<` `&` `'`) — so the stored text is raw and the caption escapes exactly once |
|
||||||
| `crates/xmedia-bot/src/main.rs` | Entry point: env/log init, command registration (`register_commands`), shared `send::BOT` force-init, queue worker start, site login validation (`site::validate_all`), 300 s edit-expiry sweep, dptree handler tree, webhook vs polling dispatch |
|
| `crates/xmedia-bot/src/main.rs` | Entry point: env/log init, command registration (`register_commands`), shared `send::BOT` force-init, queue worker start, site login validation (`site::validate_all`), 300 s edit-expiry sweep, dptree handler tree, webhook vs polling dispatch |
|
||||||
| `crates/xmedia-bot/src/config.rs` | Manual env parsing into `Config` |
|
| `crates/xmedia-bot/src/config.rs` | Manual env parsing into `Config` |
|
||||||
| `crates/xmedia-bot/src/db.rs` | `DbPool`: per-store SQLite connection pool (`POOL_SIZE = 4`, WAL, busy_timeout) over `$DATA_DIR/task_queue.db` (default `data/`); `open_store` creates file + schema, `with_conn` runs all rusqlite I/O in `spawn_blocking` |
|
| `crates/xmedia-bot/src/db.rs` | `DbPool`: one shared SQLite connection pool (`POOL_SIZE = 4`, WAL, busy_timeout) for all three tables over `$DATA_DIR/task_queue.db` (default `data/`) — the three stores share it; `open_store` creates file + schema, `with_conn` runs all rusqlite I/O in `spawn_blocking` |
|
||||||
| `crates/xmedia-bot/src/handlers/` | Handler modules: `mod.rs` (message entry point, `reply`, `log_key`), `commands.rs` (teloxide `BotCommands` enum + command executor, incl. the `/test <url>` parse-only debug command), `urls.rs` (URL extraction + bounded job channel (256) drained by `URL_WORKERS = 8` workers (`start_url_workers`) — backpressure instead of unbounded spawns; teloxide's per-chat workers are sequential — batch-forwards need concurrency), `inline.rs`/`callback.rs` (inline queries / edit-before-forward buttons), `statics.rs` (global statics) |
|
| `crates/xmedia-bot/src/handlers/` | Handler modules: `mod.rs` (message entry point, `reply`, `log_key`), `commands.rs` (teloxide `BotCommands` enum + command executor, incl. `/test <url>` (send-only) / `/debug <url>` (parse-only) and the admin-only `/bot_dict` state dump), `urls.rs` (URL extraction + bounded job channel (256) drained by `URL_WORKERS = 8` workers (`start_url_workers`) — backpressure instead of unbounded spawns; teloxide's per-chat workers are sequential — batch-forwards need concurrency), `inline.rs`/`callback.rs` (inline queries / edit-before-forward buttons), `statics.rs` (global statics) |
|
||||||
| `crates/xmedia-bot/src/state.rs` | `ChatStore`: parking_lot `Mutex<HashMap>` cache + SQLite write-through (`chat_state` table) |
|
| `crates/xmedia-bot/src/state.rs` | `ChatStore`: parking_lot `Mutex<HashMap>` cache + SQLite write-through (`chat_state` table) |
|
||||||
| `crates/xmedia-bot/src/link_cache.rs` | `LinkCache`: SQLite-backed cache (`link_cache` table) of successfully sent posts — raw caption fields + Telegram `file_id`s; repeat links re-send locally (no fetch/upload), TTL + prune, invalidated on permanent send failure |
|
| `crates/xmedia-bot/src/link_cache.rs` | `LinkCache`: SQLite-backed cache (`link_cache` table) of successfully sent posts — raw caption fields + Telegram `file_id`s; repeat links re-send locally (no fetch/upload), TTL + prune, invalidated on permanent send failure |
|
||||||
| `crates/xmedia-bot/src/queue.rs` | `PersistentTaskQueue`: SQLite-backed queue (`tasks` table), `QUEUE_WORKERS = 4` concurrent workers (lease via `BEGIN IMMEDIATE` + `locked_until` TTL), retry→dead-letter, `Notify::notify_waiters` wakeup, `busy_timeout` on all connections |
|
| `crates/xmedia-bot/src/queue.rs` | `PersistentTaskQueue`: SQLite-backed queue (`tasks` table), `QUEUE_WORKERS = 4` concurrent workers (lease via `BEGIN IMMEDIATE` + `locked_until` TTL), retry→dead-letter, `notify_one` worker wakeup plus a separate `Notify` for the 30 s lease-expiry sweep (a shared one let the sweep steal the workers' wakeup permit), `busy_timeout` on all connections |
|
||||||
| `crates/xmedia-bot/src/send.rs` | Media senders, upload fallback, error classification, queue task handlers |
|
| `crates/xmedia-bot/src/ctx.rs` | `AppContext`: the injected collaborators (`sender` + `ChatStore`/`PersistentTaskQueue`/`LinkCache`/`Config`), `from_statics` for production and the `CONTEXT` static the worker closures hold. `test_support::TestStores` backs handler tests with a tempdir store set |
|
||||||
| `crates/xmedia-bot/src/media_sender.rs` | `MediaSender` trait: the send surface (`send_media_group`/`send_animation`/`copy_messages`/`send_message`/`send_chat_action`) implemented by teloxide `Bot` (per-chat rate-limited) and by a scripted `MockSender` in tests |
|
| `crates/xmedia-bot/src/send/` | `send/mod.rs`: `Task`/`MediaItemPayload` payloads, `SendError`/`Classification`, `send_media_sequence`/`send_animation`/`forward_messages`; `send/input_media.rs`: payload → `InputFile`/`InputMedia` + `build_media_group` (caption on the first item only); `send/upload.rs`: the download-and-reupload fallback (`prepare_upload_item`/`send_batch_via_upload`, photo downscale handoff); `send/post_send.rs`: link-cache write, `KEEP_ALIVE` registry, `settle_task`, `post_send_actions`, `handle_task`/`dead_letter_notify` |
|
||||||
|
| `crates/xmedia-bot/src/media_sender.rs` | `MediaSender` trait: the user-flow surface (`send_media_group`/`send_animation`/`copy_messages`/`send_message`/`answer_callback_query`/`edit_message_caption`/`delete_message`/`send_chat_action`) implemented by teloxide `Bot` (per-chat rate-limited) and by a recording `MockSender` in tests. Admin/setup APIs (`get_chat`, `set_my_commands`, …) stay on the concrete `Bot` |
|
||||||
| `crates/xmedia-bot/src/rate_limit.rs` | Per-chat token bucket (`CAPACITY = 20`, ~20 msg/min refill) paced before sends reach the API so batch forwards don't trip flood control |
|
| `crates/xmedia-bot/src/rate_limit.rs` | Per-chat token bucket (`CAPACITY = 20`, ~20 msg/min refill) paced before sends reach the API so batch forwards don't trip flood control |
|
||||||
|
|
||||||
## Development Commands
|
## Development Commands
|
||||||
@@ -64,7 +67,7 @@ Docker: `docker build -t tgxmb .` then `docker run --rm -d --name tgxmb --env-fi
|
|||||||
- **Site adapter convention**: each site module exports `PATTERN: LazyLock<Regex>`, `enabled() -> bool`, `fetch_from_url(url) -> Result<Fetched, FetchError>`, plus `cache_key`/`is_retryable`/`media_headers`, and a unit struct `<Name>Site` implementing `site::Site`; the central dispatcher (`site/mod.rs`) only iterates the `SITES` registry. Adding a site = new `site/<name>/{mod.rs,interface.rs,model.rs}` + one `Box::new(...)` entry in `SITES` — the bot crate never lists sites (SetFormat whitelist, cache-key site lookup and startup validation all derive from the registry). Async trait methods return `SiteFuture` (a boxed `Pin<Box<dyn Future + Send>>`) because `async fn` in traits is not dyn-compatible.
|
- **Site adapter convention**: each site module exports `PATTERN: LazyLock<Regex>`, `enabled() -> bool`, `fetch_from_url(url) -> Result<Fetched, FetchError>`, plus `cache_key`/`is_retryable`/`media_headers`, and a unit struct `<Name>Site` implementing `site::Site`; the central dispatcher (`site/mod.rs`) only iterates the `SITES` registry. Adding a site = new `site/<name>/{mod.rs,interface.rs,model.rs}` + one `Box::new(...)` entry in `SITES` — the bot crate never lists sites (SetFormat whitelist, cache-key site lookup and startup validation all derive from the registry). Async trait methods return `SiteFuture` (a boxed `Pin<Box<dyn Future + Send>>`) because `async fn` in traits is not dyn-compatible.
|
||||||
- **Serde**: per-site `model.rs` are pure `Deserialize` DTOs mirroring API JSON; site structs in `interface.rs` have private fields, a `caption()` builder, and `impl From<SiteStruct> for Fetched`. Persisted payloads use internally-tagged enums (`#[serde(tag = "kind")]` / `type`).
|
- **Serde**: per-site `model.rs` are pure `Deserialize` DTOs mirroring API JSON; site structs in `interface.rs` have private fields, a `caption()` builder, and `impl From<SiteStruct> for Fetched`. Persisted payloads use internally-tagged enums (`#[serde(tag = "kind")]` / `type`).
|
||||||
- **Naming**: module-per-concern, snake_case files, `CamelCase` types, `snake_case` fns. `//!` module docs and `///` docs on non-obvious logic (syndication token, ugoira encoding, `display_text_range`).
|
- **Naming**: module-per-concern, snake_case files, `CamelCase` types, `snake_case` fns. `//!` module docs and `///` docs on non-obvious logic (syndication token, ugoira encoding, `display_text_range`).
|
||||||
- **Retries**: only `x-media::site::fetch` retries (3 attempts, `1 << attempt` backoff, HTTP errors only). Queue retries are explicit `QueueError::Retryable` with computed delay (`retry_delay_seconds`).
|
- **Retries**: only `x-media::site::fetch` retries (3 attempts, `1 << attempt` backoff, HTTP errors only); `site::fetch_once` is the same code path with a single attempt, used by inline queries whose answer window is shorter than the backoff. Queue retries are explicit `QueueError::Retryable` with computed delay (`retry_delay_seconds`).
|
||||||
- Logging via `log` macros (`pretty_env_logger`, level from `RUST_LOG`). Level convention: `info` = lifecycle + per-post business results (`sent`/`forwarded`/`copied`), admin/operator actions and anomalies (fallback, retry enqueue, dead-letter is `error`); `debug` = per-request detail (message/command/URL extraction, `fetching`/`fetched`, batch sends, queue processing, photo processing, inline queries). Full user-submitted URLs and message text only appear at `debug`; at `info` and above links are printed via the normalized cache key (`handlers::log_key`, e.g. `[key=twitter:123...]`) so logs stay short and do not echo user data.
|
- Logging via `log` macros (`pretty_env_logger`, level from `RUST_LOG`). Level convention: `info` = lifecycle + per-post business results (`sent`/`forwarded`/`copied`), admin/operator actions and anomalies (fallback, retry enqueue, dead-letter is `error`); `debug` = per-request detail (message/command/URL extraction, `fetching`/`fetched`, batch sends, queue processing, photo processing, inline queries). Full user-submitted URLs and message text only appear at `debug`; at `info` and above links are printed via the normalized cache key (`handlers::log_key`, e.g. `[key=twitter:123...]`) so logs stay short and do not echo user data.
|
||||||
|
|
||||||
## Important Files
|
## Important Files
|
||||||
@@ -72,15 +75,15 @@ Docker: `docker build -t tgxmb .` then `docker run --rm -d --name tgxmb --env-fi
|
|||||||
| File | Why it matters |
|
| File | Why it matters |
|
||||||
|---|---|
|
|---|---|
|
||||||
| `crates/xmedia-bot/src/main.rs` | Startup sequence, webhook vs polling, graceful shutdown (SIGINT via teloxide ctrlc / SIGTERM via `stop_token` for docker, → sweep stop → admin msg → queue stop) |
|
| `crates/xmedia-bot/src/main.rs` | Startup sequence, webhook vs polling, graceful shutdown (SIGINT via teloxide ctrlc / SIGTERM via `stop_token` for docker, → sweep stop → admin msg → queue stop) |
|
||||||
| `crates/xmedia-bot/src/handlers/` | `statics.rs` = `CHAT_STORE`/`TASK_QUEUE`/`CONFIG` singletons (open `$DATA_DIR/task_queue.db`, default `data/` **relative to CWD**, dir auto-created); `commands.rs` = command dispatch (incl. the `/test <url>` parse-only debug command); `urls.rs` = URL extraction + retry enqueue; `inline.rs`/`callback.rs` = inline queries / edit-before-forward buttons |
|
| `crates/xmedia-bot/src/handlers/` | `statics.rs` = `CHAT_STORE`/`TASK_QUEUE`/`CONFIG` singletons (open `$DATA_DIR/task_queue.db`, default `data/` **relative to CWD**, dir auto-created); `commands.rs` = command dispatch (incl. `/test <url>` send-only, `/debug <url>` parse-only, and the admin-only `/bot_dict` state dump); `urls.rs` = URL extraction + the per-URL pipeline (`url_media` takes a `PostSend` mode: chat settings vs `/test`'s suppressed actions); `inline.rs` = debounced inline queries; `callback.rs` = edit-before-forward buttons (dptree entry + testable `handle_callback` core) |
|
||||||
| `crates/xmedia-bot/src/send.rs` | Constants `MAX_MEDIA_GROUP = 9`; fallback chain; `classify_request_error`; download-and-reupload fallback triggered only by Telegram API errors (`is_media_fetch_failure` / `is_size_error`) |
|
| `crates/xmedia-bot/src/send/` | `mod.rs`: constants `MAX_MEDIA_GROUP = 9`; `classify_request_error`; the senders. `upload.rs`: download-and-reupload fallback triggered only by Telegram API errors (`is_media_fetch_failure` / `is_size_error`). `post_send.rs`: settlement (`settle_task`), cache write, post-send actions, queue handlers. `input_media.rs`: payload → `InputMedia` |
|
||||||
| `crates/xmedia-bot/src/photo.rs` | Pure-Rust photo processing (no ffmpeg): `png` (image-png) decode/encode + `zune-jpeg` decode + `fast_image_resize` Lanczos3 downscale + `jpeg-encoder`. Photos over Telegram's limits (width + height > 10000 px → `PHOTO_INVALID_DIMENSIONS`; bytes > 10 MiB) are decoded, downscaled keeping the format, PNG bit depth > 24 (RGBA 32-bit / 16-bit per channel) reduced to 24-bit RGB with alpha flattened white (≤24-bit untouched, never upconverted), and transcoded to JPEG only if still over the cap; memory budget guarded, otherwise the item's smaller fallback URL |
|
| `crates/xmedia-bot/src/photo.rs` | Pure-Rust photo processing (no ffmpeg): `png` (image-png) decode/encode + `zune-jpeg` decode + `fast_image_resize` Lanczos3 downscale + `jpeg-encoder`. Photos over Telegram's limits (width + height > 10000 px → `PHOTO_INVALID_DIMENSIONS`; bytes > 10 MiB) are decoded, downscaled keeping the format, PNG bit depth > 24 (RGBA 32-bit / 16-bit per channel) reduced to 24-bit RGB with alpha flattened white (≤24-bit untouched, never upconverted), and transcoded to JPEG only if still over the cap; memory budget guarded, otherwise the item's smaller fallback URL |
|
||||||
| `crates/x-media/src/site/mod.rs` | Dispatcher, `Fetched`/`FetchError`, shared `CLIENT`, `download_media` (adds `Referer: https://www.pixiv.net/` for `pximg.net` hotlink protection) |
|
| `crates/x-media/src/site/mod.rs` | Dispatcher, `Fetched`/`FetchError`, shared `CLIENT`, `download_media` (adds `Referer: https://www.pixiv.net/` for `pximg.net` hotlink protection) |
|
||||||
| `crates/x-media/src/site/pixiv/api.rs` | OAuth token exchange (hardcoded app client id/secret), access-token cache, ugoira zip→MP4 via ffmpeg in `spawn_blocking` |
|
| `crates/x-media/src/site/pixiv/api.rs` | OAuth token exchange (hardcoded app client id/secret), access-token cache, ugoira zip→MP4 via ffmpeg in `spawn_blocking` |
|
||||||
| `Dockerfile` | Multi-stage: cached dep layer via stub sources + `touch *.rs` mtime bump (cargo's freshness is mtime-based and `cargo clean -p` removes 0 files — the touch is what forces the real sources to rebuild while deps stay cached), static ffmpeg from ffmpeg.martin-riedl.de (`FFMPEG_URL` arg, optional `FFMPEG_SHA256` checksum, `unzip -t` integrity check), `debian:bookworm-slim` runtime, entrypoint. Runtime ships **no libssl/libcrypto/CA bundle** — rustls webpki-roots handles all TLS, and the static ffmpeg only processes local files (downloads go through reqwest) |
|
| `Dockerfile` | Multi-stage: cached dep layer via stub sources + `touch *.rs` mtime bump (cargo's freshness is mtime-based and `cargo clean -p` removes 0 files — the touch is what forces the real sources to rebuild while deps stay cached), static ffmpeg from ffmpeg.martin-riedl.de (`FFMPEG_URL` arg, optional `FFMPEG_SHA256` checksum, `unzip -t` integrity check), `debian:bookworm-slim` runtime, entrypoint. Runtime ships **no libssl/libcrypto/CA bundle** — rustls webpki-roots handles all TLS, and the static ffmpeg only processes local files (downloads go through reqwest) |
|
||||||
| `docker-entrypoint.sh` | Privilege drop: `useradd` with `LOCAL_USER_ID` (default 9001) + `setpriv` (no gosu on bookworm-slim) |
|
| `docker-entrypoint.sh` | Privilege drop: `useradd` with `LOCAL_USER_ID` (default 9001) + `setpriv` (no gosu on bookworm-slim) |
|
||||||
| `docker-compose.yml.example` | Deployment env reference (real `docker-compose.yml` is gitignored). Ships nginx-proxy + acme-companion: webhook mode needs TLS termination in front (teloxide's axum listener is HTTP-only; `WEBHOOK_CERT` only feeds `set_webhook`), bot exposes `VIRTUAL_HOST`/`VIRTUAL_PORT` on the shared `proxy` network, no host port; container names `nginx-proxy`/`acme-companion`/`tgxmb`, start order via `depends_on` (proxy → acme → bot) |
|
| `docker-compose.yml.example` | Deployment env reference (real `docker-compose.yml` is gitignored). Ships nginx-proxy + acme-companion: webhook mode needs TLS termination in front (teloxide's axum listener is HTTP-only; `WEBHOOK_CERT` only feeds `set_webhook`), bot exposes `VIRTUAL_HOST`/`VIRTUAL_PORT` on the shared `proxy` network, no host port; container names `nginx-proxy`/`acme-companion`/`tgxmb`, start order via `depends_on` (proxy → acme → bot) |
|
||||||
| `.github/workflows/docker.yml` | CI: build+push to Docker Hub on tag `v*`/master; **no test step**; buildx gha cache (`cache-from`/`cache-to`, scope `tgxmb-build`, `mode=max`) so cargo deps + ffmpeg layers are restored across runs |
|
| `.github/workflows/docker.yml` | CI: build+push to Docker Hub on tag `v*`/master, plus a build-only check on PRs touching the build inputs; **no test step**; verifies a release tag matches both crate versions; buildx gha cache (`cache-from` always, `cache-to` except on PRs, scope `tgxmb-build`, `mode=max`) so cargo deps + ffmpeg layers are restored across runs; `FFMPEG_URL`/`FFMPEG_SHA256` come from repo variables when set |
|
||||||
| `README.md` | Feature docs + command table (Chinese) |
|
| `README.md` | Feature docs + command table (Chinese) |
|
||||||
|
|
||||||
## Runtime/Tooling Preferences
|
## Runtime/Tooling Preferences
|
||||||
@@ -88,7 +91,7 @@ Docker: `docker build -t tgxmb .` then `docker run --rm -d --name tgxmb --env-fi
|
|||||||
- **Rust, stable, edition 2024**, workspace resolver 3. No `rust-version`/MSRV pin, no `rust-toolchain.toml` — recent stable is assumed. No nightly features.
|
- **Rust, stable, edition 2024**, workspace resolver 3. No `rust-version`/MSRV pin, no `rust-toolchain.toml` — recent stable is assumed. No nightly features.
|
||||||
- Package manager: **Cargo** (workspace with path dep `x-media` ← `xmedia-bot`). No `[workspace.package]`/shared deps — each crate lists deps independently.
|
- Package manager: **Cargo** (workspace with path dep `x-media` ← `xmedia-bot`). No `[workspace.package]`/shared deps — each crate lists deps independently.
|
||||||
- **TLS is rustls end-to-end** (no native-tls/openssl in the tree, no libssl in the Docker runtime image): `teloxide` is declared `default-features = false` with `["webhooks-axum", "macros", "rustls", "ctrlc_handler"]` (the removed `default` also carried `native-tls` and `ctrlc_handler` — the latter must stay); x-media's reqwest is `default-features = false` with `["json", "rustls-tls"]` (webpki-roots baked in, so the image ships no CA bundle). One reqwest 0.12.28 in the lock.
|
- **TLS is rustls end-to-end** (no native-tls/openssl in the tree, no libssl in the Docker runtime image): `teloxide` is declared `default-features = false` with `["webhooks-axum", "macros", "rustls", "ctrlc_handler"]` (the removed `default` also carried `native-tls` and `ctrlc_handler` — the latter must stay); x-media's reqwest is `default-features = false` with `["json", "rustls-tls"]` (webpki-roots baked in, so the image ships no CA bundle). One reqwest 0.12.28 in the lock.
|
||||||
- **Versioning**: bump the version in all three places (`crates/x-media/Cargo.toml`, `crates/xmedia-bot/Cargo.toml`, `Cargo.lock`) and **keep `README.md`, `README.en.md` and `AGENTS.md` in sync with the code on every bump**, then commit (`chore: bump version to X.Y.Z`), create an annotated tag `vX.Y.Z`, and push branch + tag (the tag push triggers the Docker Hub build).
|
- **Versioning**: bump the version in all three places (`crates/x-media/Cargo.toml`, `crates/xmedia-bot/Cargo.toml`, `Cargo.lock`) and **keep `README.md`, `README.en.md` and `AGENTS.md` in sync with the code on every bump**, then commit (`chore: bump version to X.Y.Z`), create an annotated tag `vX.Y.Z`, and push branch + tag (the tag push triggers the Docker Hub build). The tag must equal both crate versions: `.github/workflows/docker.yml` verifies that before building, and `--locked` verifies the lock file.
|
||||||
- Config is **environment-variable driven** (dotenv loads `.env`, gitignored; no `.env.example` exists). Key vars: `TELOXIDE_TOKEN` (required), `PIXIV_REFRESH_TOKEN`, `TWITTER_AUTH_TOKEN` (optional; x.com `auth_token` cookie — enables the logged-in GraphQL fallback that fetches NSFW tweets syndication withholds), `BOT_ADMIN` (comma-separated ids), `EDIT_MESSAGE_TTL_SECONDS` (default 86400), `LINK_CACHE_TTL_SECONDS` (default 604800), `DATA_DIR` (default `data`, CWD-relative; the SQLite dir, auto-created), `WEBHOOK`/`WEBHOOK_URL`/`WEBHOOK_LISTEN`/`WEBHOOK_PORT`/`WEBHOOK_CERT`/`WEBHOOK_SECRET_TOKEN` (webhook mode requires URL/listen/port, `.expect`ed; `WEBHOOK_CERT` is Telegram-facing self-signed validation only — TLS must be terminated by a reverse proxy), `RUST_LOG`, `TELOXIDE_PROXY`, `LOCAL_USER_ID` (entrypoint only).
|
- Config is **environment-variable driven** (dotenv loads `.env`, gitignored; no `.env.example` exists). Key vars: `TELOXIDE_TOKEN` (required), `PIXIV_REFRESH_TOKEN`, `TWITTER_AUTH_TOKEN` (optional; x.com `auth_token` cookie — enables the logged-in GraphQL fallback that fetches NSFW tweets syndication withholds), `BOT_ADMIN` (comma-separated ids), `EDIT_MESSAGE_TTL_SECONDS` (default 86400), `LINK_CACHE_TTL_SECONDS` (default 604800), `DATA_DIR` (default `data`, CWD-relative; the SQLite dir, auto-created), `WEBHOOK`/`WEBHOOK_URL`/`WEBHOOK_LISTEN`/`WEBHOOK_PORT`/`WEBHOOK_CERT`/`WEBHOOK_SECRET_TOKEN` (webhook mode requires URL/listen/port, `.expect`ed; `WEBHOOK_CERT` is Telegram-facing self-signed validation only — TLS must be terminated by a reverse proxy), `RUST_LOG`, `TELOXIDE_PROXY`, `LOCAL_USER_ID` (entrypoint only).
|
||||||
- SQLite via `rusqlite` with `bundled` feature (no system libsqlite needed). DB file `$DATA_DIR/task_queue.db` (default `data/task_queue.db`, CWD-relative — run from the workspace root, or `/app` in Docker; set `DATA_DIR` to pin state anywhere). Mount `./data` and `./cert` volumes.
|
- SQLite via `rusqlite` with `bundled` feature (no system libsqlite needed). DB file `$DATA_DIR/task_queue.db` (default `data/task_queue.db`, CWD-relative — run from the workspace root, or `/app` in Docker; set `DATA_DIR` to pin state anywhere). Mount `./data` and `./cert` volumes.
|
||||||
- `.gitattributes` enforces LF for `*.sh` (CRLF breaks shebangs in containers). `.gitignore`: `.env`, `data/`, `cert/`, `docker-compose.yml`, `/target`, `.idea/`.
|
- `.gitattributes` enforces LF for `*.sh` (CRLF breaks shebangs in containers). `.gitignore`: `.env`, `data/`, `cert/`, `docker-compose.yml`, `/target`, `.idea/`.
|
||||||
@@ -96,10 +99,10 @@ Docker: `docker build -t tgxmb .` then `docker run --rm -d --name tgxmb --env-fi
|
|||||||
|
|
||||||
## Testing & QA
|
## Testing & QA
|
||||||
|
|
||||||
- **~115 tests, all inline `#[cfg(test)] mod tests`** — no `tests/` integration directories. Framework: built-in Rust test + `#[tokio::test]` (dev-deps only in `x-media`: tokio macros/rt-multi-thread, dotenv).
|
- **~135 tests, all inline `#[cfg(test)] mod tests`** — no `tests/` integration directories. Framework: built-in Rust test + `#[tokio::test]` (dev-deps only in `x-media`: tokio macros/rt-multi-thread, dotenv).
|
||||||
- No mocking framework anywhere (no mockito/wiremock/mockall). Conventions: pure-function units (regex parsing, serde round-trips, chunking, retry math) tested synchronously; async tests use real dependencies — file-backed SQLite via `tempfile` (`queue.rs::new_queue()` helper), live network fetches.
|
- No mocking framework anywhere (no mockito/wiremock/mockall). Conventions: pure-function units (regex parsing, serde round-trips, chunking, retry math) tested synchronously; async tests use real dependencies — file-backed SQLite via `tempfile` (`queue.rs::new_queue()` helper), live network fetches.
|
||||||
- Live-network tests exist in `site/twitter/interface.rs` (5), `site/bsky/interface.rs` (2), `site/pixiv/api.rs` (1); `photo.rs` adds one `#[ignore = "heavy: …"]` test. Test gating convention (enforced by `.github/workflows/ci.yml`): pure unit tests always run; live-network tests carry `#[ignore = "live network: ..."]` (run via `cargo test --workspace -- --ignored live`); token-gated pixiv tests early-return when `PIXIV_REFRESH_TOKEN` is absent **or empty** (an unset GitHub secret arrives as `""` — `is_err()` alone would run them tokenless and fail). Run the full offline suite with `cargo test --workspace`.
|
- Live-network tests exist in `site/twitter/interface.rs` (5), `site/bsky/interface.rs` (2), `site/misskey/interface.rs` (1), `site/pixiv/api.rs` (1); `photo.rs` adds one `#[ignore = "heavy: …"]` test. `site/mod.rs` also has a **token-gated but not `#[ignore]`d** pixiv download test (`download_media_pixiv_original_with_referer`): it hits `i.pximg.net` whenever `PIXIV_REFRESH_TOKEN` is set, so a local `cargo test --workspace` is not fully offline and can flake on a pixiv CDN body timeout. Test gating convention (enforced by `.github/workflows/ci.yml`): pure unit tests always run; live-network tests carry `#[ignore = "live network: ..."]` (run via `cargo test --workspace -- --ignored live`); token-gated pixiv tests early-return when `PIXIV_REFRESH_TOKEN` is absent **or empty** (an unset GitHub secret arrives as `""` — `is_err()` alone would run them tokenless and fail). Run the full offline suite with `cargo test --workspace`.
|
||||||
- Fixtures are inline `serde_json::json!` builder fns (`fixture()`, `thread_json()`, `illust_json()`), not files. The shared `CLIENT` sets `pool_max_idle_per_host(0)` under `#[cfg(test)]` to avoid cross-runtime `DispatchGone`.
|
- Fixtures are inline `serde_json::json!` builder fns (`fixture()`, `thread_json()`, `illust_json()`), not files. The shared `CLIENT` sets `pool_max_idle_per_host(0)` under `#[cfg(test)]` to avoid cross-runtime `DispatchGone`.
|
||||||
- **CI** — `.github/workflows/ci.yml` runs `cargo fmt --check` + `cargo clippy --workspace --all-targets -- -D warnings` + `cargo test --workspace` + a `actions-rust-lang/audit` dependency-vulnerability gate (offline, no secrets, on every push/PR) and a `live` job (schedule/manual/tag only, `PIXIV_REFRESH_TOKEN`/`TWITTER_AUTH_TOKEN` from secrets, `continue-on-error`) for the `#[ignore]`d live + token tests. `.github/workflows/docker.yml` builds/pushes the image only.
|
- **CI** — `.github/workflows/ci.yml` (actions pinned to commit SHAs, `--locked` on every cargo invocation, `concurrency` cancels superseded runs, `RUST_BACKTRACE=1`) runs `cargo fmt --check` + `cargo clippy --workspace --all-targets --locked -- -D warnings` + `cargo test --workspace --locked` + a release-profile `cargo build --release --locked` + an `actions-rust-lang/audit` dependency-vulnerability gate (offline, no secrets, on every push/PR) and a `live` job (schedule/manual/tag only, `-p x-media` since every network/secret-gated test lives there, `continue-on-error`) for the `#[ignore]`d live + token tests. `.github/workflows/docker.yml` builds and pushes the image on master/tag and runs a **build-only check on pull requests touching the build inputs** (`Dockerfile`, entrypoint, manifests, `.dockerignore`); a release tag must match both crate versions or the build stops, and `FFMPEG_URL`/`FFMPEG_SHA256` are taken from repository variables when set (a release can pin an exact ffmpeg build). `.github/dependabot.yml` keeps crates, the pinned actions and the Docker base images current.
|
||||||
- Untested and hard to test without a mock seam: `handlers.rs` (depends directly on teloxide `Bot`); `main.rs`, `config.rs`, `state.rs`; `media.rs`, `lib.rs`, all `model.rs`.
|
- Untested and hard to test without a mock seam: `main.rs`, `config.rs`, `db.rs`, `handlers/statics.rs`, `media_sender.rs` (holds the `MockSender` itself); in `x-media`: `media.rs`, `lib.rs`, all `model.rs`. The `commands.rs` *executor* needs a real `Bot` (only its pure report builder is tested). Everything else — `handlers/{mod,callback,inline,urls}.rs`, `send/*`, `ctx.rs`, `state.rs`, `queue.rs`, `link_cache.rs`, `rate_limit.rs` — is driven through `TestStores`/`ctx::test_support` and the scripted `MockSender`.
|
||||||
- No coverage tracking.
|
- No coverage tracking.
|
||||||
|
|||||||
Generated
+453
-513
File diff suppressed because it is too large
Load Diff
+4
-2
@@ -11,6 +11,8 @@ ARG APP_NAME=telegram-twitter-media-bot
|
|||||||
# runners. `/redirect/latest/` floats to the newest release build; each build
|
# runners. `/redirect/latest/` floats to the newest release build; each build
|
||||||
# also ships a .sha256. Swap `amd64` for `arm64` when building arm64 images.
|
# also ships a .sha256. Swap `amd64` for `arm64` when building arm64 images.
|
||||||
ARG FFMPEG_URL=https://ffmpeg.martin-riedl.de/redirect/latest/linux/amd64/release/ffmpeg.zip
|
ARG FFMPEG_URL=https://ffmpeg.martin-riedl.de/redirect/latest/linux/amd64/release/ffmpeg.zip
|
||||||
|
# Arm64 images need this URL swapped for the `linux/arm64` build (currently
|
||||||
|
# hardcoded amd64; the workflow builds amd64 only — see docker.yml).
|
||||||
# Optional sha256 of ffmpeg.zip (pinned releases only): set to verify the
|
# Optional sha256 of ffmpeg.zip (pinned releases only): set to verify the
|
||||||
# download. The mirror publishes .sha256 sidecars next to pinned builds, e.g.
|
# download. The mirror publishes .sha256 sidecars next to pinned builds, e.g.
|
||||||
# https://ffmpeg.martin-riedl.de/download/linux/amd64/<id>_9.0/ffmpeg.zip.sha256
|
# https://ffmpeg.martin-riedl.de/download/linux/amd64/<id>_9.0/ffmpeg.zip.sha256
|
||||||
@@ -28,7 +30,7 @@ COPY crates/xmedia-bot/Cargo.toml crates/xmedia-bot/Cargo.toml
|
|||||||
RUN mkdir -p crates/x-media/src crates/xmedia-bot/src \
|
RUN mkdir -p crates/x-media/src crates/xmedia-bot/src \
|
||||||
&& printf 'fn main() {}\n' > crates/xmedia-bot/src/main.rs \
|
&& printf 'fn main() {}\n' > crates/xmedia-bot/src/main.rs \
|
||||||
&& : > crates/x-media/src/lib.rs \
|
&& : > crates/x-media/src/lib.rs \
|
||||||
&& cargo build --release -p xmedia-bot
|
&& cargo build --release --locked -p xmedia-bot
|
||||||
|
|
||||||
# 2. Static ffmpeg next (cached unless FFMPEG_URL changes), so source edits
|
# 2. Static ffmpeg next (cached unless FFMPEG_URL changes), so source edits
|
||||||
# never re-download it. The zip contains a single `ffmpeg` binary at the
|
# never re-download it. The zip contains a single `ffmpeg` binary at the
|
||||||
@@ -51,7 +53,7 @@ RUN wget -q -O /tmp/ffmpeg.zip "$FFMPEG_URL" \
|
|||||||
# removes 0 files and the stub binary silently ships.)
|
# removes 0 files and the stub binary silently ships.)
|
||||||
COPY crates/ ./crates/
|
COPY crates/ ./crates/
|
||||||
RUN find crates -type f -name '*.rs' -exec touch {} + \
|
RUN find crates -type f -name '*.rs' -exec touch {} + \
|
||||||
&& cargo build --release -p xmedia-bot
|
&& cargo build --release --locked -p xmedia-bot
|
||||||
|
|
||||||
# ---------- runtime stage ----------
|
# ---------- runtime stage ----------
|
||||||
FROM debian:bookworm-slim
|
FROM debian:bookworm-slim
|
||||||
|
|||||||
+5
-4
@@ -1,6 +1,6 @@
|
|||||||
# TelegramXMediaBot
|
# TelegramXMediaBot
|
||||||
|
|
||||||
A Telegram bot that turns post links from X / Twitter, Pixiv, and Bluesky into media messages (images, video, GIF) with the post's title, author, and tags.
|
A Telegram bot that turns post links from X / Twitter, Pixiv, Bluesky, and Misskey (misskey.io) into media messages (images, video, GIF) with the post's title, author, and tags.
|
||||||
|
|
||||||
## Features
|
## Features
|
||||||
|
|
||||||
@@ -111,10 +111,11 @@ Telegram only accepts ports 443/80/88/8443.
|
|||||||
| `/remove_forward_channel` | Remove the forward channel |
|
| `/remove_forward_channel` | Remove the forward channel |
|
||||||
| `/edit_before_forward` | Toggle "edit before forward": when enabled, the bot posts a prompt after forwarding; replying to it edits the first forwarded message's caption (or taps a template button to apply one) |
|
| `/edit_before_forward` | Toggle "edit before forward": when enabled, the bot posts a prompt after forwarding; replying to it edits the first forwarded message's caption (or taps a template button to apply one) |
|
||||||
| `/set_template <name>` | Reply to a message containing `[]` to save it as a named template; `[]` is replaced by the original post link when forwarding (used with "edit before forward") |
|
| `/set_template <name>` | Reply to a message containing `[]` to save it as a named template; `[]` is replaced by the original post link when forwarding (used with "edit before forward") |
|
||||||
| `/set_format <site> <format>` | Customize the caption format for one site. Sites: `twitter` / `bsky` / `pixiv`. Placeholders: `{url}` `{author}` `{author_url}` `{title}` `{tags}` |
|
| `/set_format <site> <format>` | Customize the caption format for one site. Sites: `twitter` / `bsky` / `pixiv` / `misskey`. Placeholders: `{url}` `{author}` `{author_url}` `{title}` `{tags}` |
|
||||||
| `/clear_cache [link]` | Clear the link cache (admin only); with a link only that entry, otherwise everything |
|
| `/clear_cache [link]` | Clear the link cache (admin only); with a link only that entry, otherwise everything |
|
||||||
| `/bot_dict` | Show the current chat state (debugging) |
|
| `/bot_dict` | Show the current chat state (debugging; admin only) |
|
||||||
| `/test <link>` | Debug: parse a link and report the parse result only (site, title, author, tags, media list) — no media is sent |
|
| `/test <link>` | Parse a link and send its media; no channel forward, no edit-before-forward prompt (send only) |
|
||||||
|
| `/debug <link>` | Debug: parse a link and report the parse result only (site, title, author, tags, media list) — no media is sent |
|
||||||
|
|
||||||
Link processing works only in private chats; commands work in any chat.
|
Link processing works only in private chats; commands work in any chat.
|
||||||
|
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
# TelegramXMediaBot
|
# TelegramXMediaBot
|
||||||
|
|
||||||
Telegram 机器人,将 X / Twitter、Pixiv、Bluesky 的帖子链接转换为媒体消息发送,附带帖子标题、作者与标签。
|
Telegram 机器人,将 X / Twitter、Pixiv、Bluesky、Misskey (misskey.io) 的帖子链接转换为媒体消息发送,附带帖子标题、作者与标签。
|
||||||
|
|
||||||
## 功能
|
## 功能
|
||||||
|
|
||||||
@@ -111,10 +111,11 @@ Telegram 只接受 443/80/88/8443 端口。
|
|||||||
| `/remove_forward_channel` | 取消转发频道 |
|
| `/remove_forward_channel` | 取消转发频道 |
|
||||||
| `/edit_before_forward` | 开关「转发前编辑」:开启后,转发成功后 bot 会发一条提示消息,回复它可修改第一条转发消息的 caption(或点击模板按钮套用模板) |
|
| `/edit_before_forward` | 开关「转发前编辑」:开启后,转发成功后 bot 会发一条提示消息,回复它可修改第一条转发消息的 caption(或点击模板按钮套用模板) |
|
||||||
| `/set_template <名称>` | 回复一条含 `[]` 的消息,将其保存为命名模板;转发时 `[]` 会被替换为原帖链接(配合「转发前编辑」使用) |
|
| `/set_template <名称>` | 回复一条含 `[]` 的消息,将其保存为命名模板;转发时 `[]` 会被替换为原帖链接(配合「转发前编辑」使用) |
|
||||||
| `/set_format <站点> <格式>` | 自定义某站点的 caption 格式。站点:`twitter` / `bsky` / `pixiv`。占位符:`{url}` `{author}` `{author_url}` `{title}` `{tags}` |
|
| `/set_format <站点> <格式>` | 自定义某站点的 caption 格式。站点:`twitter` / `bsky` / `pixiv` / `misskey`。占位符:`{url}` `{author}` `{author_url}` `{title}` `{tags}` |
|
||||||
| `/clear_cache [链接]` | 清空链接缓存(仅管理员);带链接只清该条,否则清空全部 |
|
| `/clear_cache [链接]` | 清空链接缓存(仅管理员);带链接只清该条,否则清空全部 |
|
||||||
| `/bot_dict` | 查看当前聊天状态(调试用) |
|
| `/bot_dict` | 查看当前聊天状态(调试用;仅管理员) |
|
||||||
| `/test <链接>` | 调试:只解析链接并返回解析结果(站点、标题、作者、标签、媒体列表),不发送任何媒体 |
|
| `/test <链接>` | 解析链接并发送媒体;不转发到频道、不弹转发前编辑提示(仅发送) |
|
||||||
|
| `/debug <链接>` | 调试:只解析链接并返回解析结果(站点、标题、作者、标签、媒体列表),不发送任何媒体 |
|
||||||
|
|
||||||
链接处理仅限私聊;命令在任意聊天可用。
|
链接处理仅限私聊;命令在任意聊天可用。
|
||||||
|
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
[package]
|
[package]
|
||||||
name = "x-media"
|
name = "x-media"
|
||||||
version = "1.4.0"
|
version = "1.6.0"
|
||||||
edition = "2024"
|
edition = "2024"
|
||||||
|
|
||||||
[dependencies]
|
[dependencies]
|
||||||
|
|||||||
@@ -0,0 +1,384 @@
|
|||||||
|
//! Site adapter for misskey.io notes: URL pattern, API fetch and
|
||||||
|
//! normalization into [`Fetched`] (see [`crate::site::Site`]).
|
||||||
|
|
||||||
|
use super::model;
|
||||||
|
use crate::media::Media;
|
||||||
|
use crate::site::{FetchError, Fetched, RenderData, Site, SiteFuture};
|
||||||
|
use html_escape::{encode_double_quoted_attribute, encode_text};
|
||||||
|
use regex::Regex;
|
||||||
|
use std::sync::LazyLock;
|
||||||
|
|
||||||
|
const API_URL: &str = "https://misskey.io/api/notes/show";
|
||||||
|
|
||||||
|
/// Registry entry for the misskey.io adapter (see [`crate::site::Site`]).
|
||||||
|
pub struct MisskeySite;
|
||||||
|
|
||||||
|
impl Site for MisskeySite {
|
||||||
|
fn id(&self) -> &'static str {
|
||||||
|
"misskey"
|
||||||
|
}
|
||||||
|
|
||||||
|
fn pattern(&self) -> &'static Regex {
|
||||||
|
&PATTERN
|
||||||
|
}
|
||||||
|
|
||||||
|
fn cache_key(&self, url: &str) -> Option<String> {
|
||||||
|
cache_key(url)
|
||||||
|
}
|
||||||
|
|
||||||
|
fn fetch_from_url<'a>(&'a self, url: &'a str) -> SiteFuture<'a, Fetched> {
|
||||||
|
Box::pin(async move { fetch_from_url(url).await })
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
pub static PATTERN: LazyLock<Regex> =
|
||||||
|
LazyLock::new(|| Regex::new(r"^(?:https?://)?misskey\.io/notes/([\w.\-~]+)").unwrap());
|
||||||
|
|
||||||
|
pub fn enabled() -> bool {
|
||||||
|
true
|
||||||
|
}
|
||||||
|
|
||||||
|
pub async fn fetch_from_url(url: &str) -> Result<Fetched, FetchError> {
|
||||||
|
let caps = PATTERN.captures(url).ok_or(FetchError::NotFound)?;
|
||||||
|
let note_id = caps.get(1).ok_or(FetchError::NotFound)?.as_str();
|
||||||
|
let note = fetch(note_id).await?;
|
||||||
|
Ok(note.into())
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Cache key for a misskey URL: `"misskey:<note id>"`. The prefix is the
|
||||||
|
/// site id used for caption-format lookup and link-cache keys.
|
||||||
|
pub fn cache_key(url: &str) -> Option<String> {
|
||||||
|
PATTERN
|
||||||
|
.captures(url)
|
||||||
|
.map(|caps| format!("misskey:{}", &caps[1]))
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Misskey's fetch-retry policy: transient classes only. Not-found, blocked
|
||||||
|
/// and parse failures are permanent.
|
||||||
|
pub fn is_retryable(err: &FetchError) -> bool {
|
||||||
|
matches!(err, FetchError::Http(_) | FetchError::Transient(_))
|
||||||
|
}
|
||||||
|
|
||||||
|
/// misskey.io media hosts need no extra headers (verified: direct GET works).
|
||||||
|
pub fn media_headers(_url: &str) -> Option<Vec<(&'static str, String)>> {
|
||||||
|
None
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Fetches a note from misskey.io by id. The API answers client failures
|
||||||
|
/// with HTTP 400 + `{"error":{"code":...}}` (NO_SUCH_NOTE → NotFound);
|
||||||
|
/// everything else non-success is transient and retried by [`crate::site::fetch`].
|
||||||
|
pub async fn fetch(note_id: &str) -> Result<model::Note, FetchError> {
|
||||||
|
let response = crate::site::CLIENT
|
||||||
|
.post(API_URL)
|
||||||
|
.json(&serde_json::json!({ "noteId": note_id }))
|
||||||
|
.send()
|
||||||
|
.await?;
|
||||||
|
let status = response.status();
|
||||||
|
if !status.is_success() {
|
||||||
|
return Err(match status.as_u16() {
|
||||||
|
400 => not_found_or_invalid(response).await,
|
||||||
|
_ => FetchError::Transient(format!("misskey status {status}")),
|
||||||
|
});
|
||||||
|
}
|
||||||
|
response.json().await.map_err(|e| FetchError::Site {
|
||||||
|
site: "misskey",
|
||||||
|
error: Box::new(e),
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Maps a 400 response: NO_SUCH_NOTE is permanent NotFound, any other 400 is
|
||||||
|
/// a site error (permanent — retrying a rejected request cannot succeed).
|
||||||
|
async fn not_found_or_invalid(response: reqwest::Response) -> FetchError {
|
||||||
|
match response.json::<serde_json::Value>().await {
|
||||||
|
Ok(v) if v["error"]["code"] == "NO_SUCH_NOTE" => FetchError::NotFound,
|
||||||
|
_ => FetchError::Site {
|
||||||
|
site: "misskey",
|
||||||
|
error: "note rejected (invalid param or private note)".into(),
|
||||||
|
},
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// The note whose content matters: a renote shell has no text/files of its
|
||||||
|
/// own — the embedded renote carries them.
|
||||||
|
fn effective(note: &model::Note) -> &model::Note {
|
||||||
|
match ¬e.renote {
|
||||||
|
Some(renote) if note.files.is_empty() => renote,
|
||||||
|
_ => note,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
impl From<model::Note> for Fetched {
|
||||||
|
fn from(note: model::Note) -> Self {
|
||||||
|
let note = ¬e;
|
||||||
|
let content = effective(note);
|
||||||
|
let url = format!("https://misskey.io/notes/{}", note.id);
|
||||||
|
let author = content
|
||||||
|
.user
|
||||||
|
.name
|
||||||
|
.as_deref()
|
||||||
|
.filter(|n| !n.is_empty())
|
||||||
|
.unwrap_or(&content.user.username)
|
||||||
|
.to_string();
|
||||||
|
let author_url = format!("https://misskey.io/@{}", content.user.username);
|
||||||
|
let cw = content.cw.as_deref().unwrap_or_default();
|
||||||
|
// Notes carry hashtags inline in the text (no structured tags array);
|
||||||
|
// a CW note gets the marker prefixed so recipients see the spoiler.
|
||||||
|
let mut title = cw.to_string();
|
||||||
|
if !cw.is_empty() && !title.ends_with(' ') {
|
||||||
|
title.push(' ');
|
||||||
|
}
|
||||||
|
title.push_str(content.text.as_deref().unwrap_or_default().trim());
|
||||||
|
let title = title.trim().to_string();
|
||||||
|
|
||||||
|
let caption = caption(&url, &author_url, &author, &title);
|
||||||
|
let sensitive = content.cw.is_some() || content.files.iter().any(|f| f.is_sensitive);
|
||||||
|
let media: Vec<Media> = content.files.iter().filter_map(media_from_file).collect();
|
||||||
|
|
||||||
|
Fetched {
|
||||||
|
source_url: url.clone(),
|
||||||
|
caption,
|
||||||
|
title: title.clone(),
|
||||||
|
media,
|
||||||
|
sensitive,
|
||||||
|
site_id: "misskey",
|
||||||
|
render_data: Some(RenderData {
|
||||||
|
url,
|
||||||
|
author: encode_text(&author).into_owned(),
|
||||||
|
author_url: author_url.clone(),
|
||||||
|
title: encode_text(&title).into_owned(),
|
||||||
|
tags: String::new(),
|
||||||
|
}),
|
||||||
|
_keep_alive: None,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
fn caption(url: &str, author_url: &str, author: &str, text: &str) -> String {
|
||||||
|
let url = encode_double_quoted_attribute(url);
|
||||||
|
let author_url = encode_double_quoted_attribute(author_url);
|
||||||
|
let author = encode_text(author);
|
||||||
|
if text.is_empty() {
|
||||||
|
return format!("{url}\n<a href=\"{author_url}\">{author}</a>");
|
||||||
|
}
|
||||||
|
format!(
|
||||||
|
"{url}\n<a href=\"{author_url}\">{author}</a>: {text}",
|
||||||
|
text = encode_text(text),
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Maps a Misskey DriveFile to a [`Media`] item; unknown/audio/other types
|
||||||
|
/// are skipped (twitter's `_ => {}` precedent). GIF must be matched before
|
||||||
|
/// the generic image arm.
|
||||||
|
fn media_from_file(file: &model::DriveFile) -> Option<Media> {
|
||||||
|
let title = file.name.clone();
|
||||||
|
match file.mime_type.as_str() {
|
||||||
|
"image/gif" => Some(Media::Animated {
|
||||||
|
title,
|
||||||
|
url: file.url.clone(),
|
||||||
|
thumbnail_url: file.thumbnail_url.clone().unwrap_or_default(),
|
||||||
|
}),
|
||||||
|
mime if mime.starts_with("image/") => Some(Media::Illustration {
|
||||||
|
title,
|
||||||
|
url: file.url.clone(),
|
||||||
|
thumbnail_url: file.thumbnail_url.clone(),
|
||||||
|
fallback_url: None,
|
||||||
|
}),
|
||||||
|
mime if mime.starts_with("video/") => Some(Media::Video {
|
||||||
|
title,
|
||||||
|
url: file.url.clone(),
|
||||||
|
thumbnail_url: file.thumbnail_url.clone().unwrap_or_default(),
|
||||||
|
}),
|
||||||
|
_ => None,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
#[cfg(test)]
|
||||||
|
mod tests {
|
||||||
|
use super::*;
|
||||||
|
|
||||||
|
fn note_json(json: serde_json::Value) -> model::Note {
|
||||||
|
serde_json::from_value(json).unwrap()
|
||||||
|
}
|
||||||
|
|
||||||
|
fn base_note() -> serde_json::Value {
|
||||||
|
serde_json::json!({
|
||||||
|
"id": "aotihl10lqrs015s",
|
||||||
|
"text": "hello",
|
||||||
|
"user": { "name": "ミロン", "username": "donyan47897", "host": null },
|
||||||
|
"files": []
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn pattern_matches_misskey_note_urls() {
|
||||||
|
for url in [
|
||||||
|
"https://misskey.io/notes/aotihl10lqrs015s",
|
||||||
|
"http://misskey.io/notes/aotihl10lqrs015s",
|
||||||
|
"misskey.io/notes/aotihl10lqrs015s",
|
||||||
|
] {
|
||||||
|
assert!(PATTERN.is_match(url), "{url}");
|
||||||
|
}
|
||||||
|
for url in [
|
||||||
|
"https://misskey.io/",
|
||||||
|
"https://misskey.io/@user",
|
||||||
|
"https://misskey.io/notes/",
|
||||||
|
"https://x.com/user/status/123",
|
||||||
|
] {
|
||||||
|
assert!(!PATTERN.is_match(url), "{url}");
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn cache_key_normalizes_variants() {
|
||||||
|
assert_eq!(
|
||||||
|
cache_key("https://misskey.io/notes/aotihl10lqrs015s"),
|
||||||
|
Some("misskey:aotihl10lqrs015s".to_string())
|
||||||
|
);
|
||||||
|
assert_eq!(x_media_site_id("misskey:abc"), "misskey");
|
||||||
|
}
|
||||||
|
|
||||||
|
fn x_media_site_id(key: &str) -> &'static str {
|
||||||
|
crate::site::site_id_from_key(key)
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn from_json_image_file() {
|
||||||
|
let mut note = base_note();
|
||||||
|
note["files"] = serde_json::json!([{
|
||||||
|
"type": "image/webp",
|
||||||
|
"url": "https://media.misskeyusercontent.jp/io/a.webp",
|
||||||
|
"thumbnailUrl": "https://media.misskeyusercontent.jp/io/t.webp",
|
||||||
|
"isSensitive": true,
|
||||||
|
"name": "pic.webp"
|
||||||
|
}]);
|
||||||
|
let fetched: Fetched = note_json(note).into();
|
||||||
|
assert_eq!(
|
||||||
|
fetched.source_url,
|
||||||
|
"https://misskey.io/notes/aotihl10lqrs015s"
|
||||||
|
);
|
||||||
|
assert_eq!(fetched.site_id, "misskey");
|
||||||
|
assert_eq!(fetched.title, "hello");
|
||||||
|
assert!(fetched.sensitive);
|
||||||
|
assert_eq!(fetched.media.len(), 1);
|
||||||
|
match &fetched.media[0] {
|
||||||
|
Media::Illustration {
|
||||||
|
title,
|
||||||
|
url,
|
||||||
|
thumbnail_url,
|
||||||
|
fallback_url,
|
||||||
|
} => {
|
||||||
|
assert_eq!(title.as_deref(), Some("pic.webp"));
|
||||||
|
assert_eq!(url, "https://media.misskeyusercontent.jp/io/a.webp");
|
||||||
|
assert_eq!(
|
||||||
|
thumbnail_url.as_deref(),
|
||||||
|
Some("https://media.misskeyusercontent.jp/io/t.webp")
|
||||||
|
);
|
||||||
|
assert!(fallback_url.is_none());
|
||||||
|
}
|
||||||
|
other => panic!("expected illustration, got {other:?}"),
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn from_json_gif_video_and_skip_audio() {
|
||||||
|
let mut note = base_note();
|
||||||
|
note["files"] = serde_json::json!([
|
||||||
|
{ "type": "audio/mpeg", "url": "https://m/a.mp3", "isSensitive": false },
|
||||||
|
{ "type": "image/gif", "url": "https://m/a.gif", "isSensitive": false },
|
||||||
|
{ "type": "video/webm", "url": "https://m/a.webm", "isSensitive": false }
|
||||||
|
]);
|
||||||
|
let fetched: Fetched = note_json(note).into();
|
||||||
|
assert_eq!(fetched.media.len(), 2);
|
||||||
|
assert!(
|
||||||
|
matches!(&fetched.media[0], Media::Animated { url, .. } if url == "https://m/a.gif")
|
||||||
|
);
|
||||||
|
assert!(matches!(&fetched.media[1], Media::Video { url, .. } if url == "https://m/a.webm"));
|
||||||
|
// No thumbnailUrl → empty string, not a broken URL.
|
||||||
|
match &fetched.media[1] {
|
||||||
|
Media::Video { thumbnail_url, .. } => assert_eq!(thumbnail_url, ""),
|
||||||
|
other => panic!("expected video, got {other:?}"),
|
||||||
|
}
|
||||||
|
assert!(!fetched.sensitive);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn from_json_cw_marks_sensitive_and_prefixes_title() {
|
||||||
|
let mut note = base_note();
|
||||||
|
note["cw"] = serde_json::json!("spoiler");
|
||||||
|
note["text"] = serde_json::json!("body");
|
||||||
|
let fetched: Fetched = note_json(note).into();
|
||||||
|
assert!(fetched.sensitive);
|
||||||
|
assert_eq!(fetched.title, "spoiler body");
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn from_json_author_falls_back_to_username() {
|
||||||
|
let mut note = base_note();
|
||||||
|
note["user"] = serde_json::json!({ "name": null, "username": "donyan47897", "host": null });
|
||||||
|
let fetched: Fetched = note_json(note).into();
|
||||||
|
assert!(
|
||||||
|
fetched.caption.contains("donyan47897"),
|
||||||
|
"{}",
|
||||||
|
fetched.caption
|
||||||
|
);
|
||||||
|
assert!(fetched.caption.contains("https://misskey.io/@donyan47897"));
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn from_json_renote_uses_embedded_content() {
|
||||||
|
let note = serde_json::json!({
|
||||||
|
"id": "shell0000000000",
|
||||||
|
"text": null,
|
||||||
|
"user": { "name": "shell", "username": "shelluser", "host": null },
|
||||||
|
"files": [],
|
||||||
|
"renote": {
|
||||||
|
"id": "inner000000000",
|
||||||
|
"text": "inner text",
|
||||||
|
"user": { "name": "inner", "username": "inneruser", "host": null },
|
||||||
|
"files": [
|
||||||
|
{ "type": "image/png", "url": "https://m/i.png", "isSensitive": false }
|
||||||
|
]
|
||||||
|
}
|
||||||
|
});
|
||||||
|
let fetched: Fetched = note_json(note).into();
|
||||||
|
assert_eq!(fetched.title, "inner text");
|
||||||
|
assert_eq!(fetched.media.len(), 1);
|
||||||
|
// The source URL still points at the renote shell the user posted.
|
||||||
|
assert_eq!(
|
||||||
|
fetched.source_url,
|
||||||
|
"https://misskey.io/notes/shell0000000000"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn caption_layout_matches_bsky() {
|
||||||
|
let fetched: Fetched = note_json(base_note()).into();
|
||||||
|
assert_eq!(
|
||||||
|
fetched.caption,
|
||||||
|
"https://misskey.io/notes/aotihl10lqrs015s\n<a href=\"https://misskey.io/@donyan47897\">ミロン</a>: hello"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn caption_without_text_has_no_dangling_colon() {
|
||||||
|
let mut note = base_note();
|
||||||
|
note["text"] = serde_json::json!(null);
|
||||||
|
let fetched: Fetched = note_json(note).into();
|
||||||
|
assert_eq!(
|
||||||
|
fetched.caption,
|
||||||
|
"https://misskey.io/notes/aotihl10lqrs015s\n<a href=\"https://misskey.io/@donyan47897\">ミロン</a>"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
#[ignore = "live network: requires outbound HTTPS to misskey.io"]
|
||||||
|
async fn live_fetch_reference_note() {
|
||||||
|
let fetched = fetch_from_url("https://misskey.io/notes/aotihl10lqrs015s")
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
assert_eq!(fetched.site_id, "misskey");
|
||||||
|
assert_eq!(fetched.media.len(), 1);
|
||||||
|
assert!(fetched.sensitive);
|
||||||
|
assert!(!fetched.caption.is_empty());
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -0,0 +1,6 @@
|
|||||||
|
mod interface;
|
||||||
|
mod model;
|
||||||
|
|
||||||
|
pub use interface::{
|
||||||
|
MisskeySite, PATTERN, cache_key, enabled, fetch_from_url, is_retryable, media_headers,
|
||||||
|
};
|
||||||
@@ -0,0 +1,35 @@
|
|||||||
|
use serde::Deserialize;
|
||||||
|
|
||||||
|
#[derive(Deserialize, Debug)]
|
||||||
|
pub(crate) struct Note {
|
||||||
|
pub(crate) id: String,
|
||||||
|
pub(crate) text: Option<String>,
|
||||||
|
#[serde(default)]
|
||||||
|
pub(crate) cw: Option<String>,
|
||||||
|
pub(crate) user: User,
|
||||||
|
#[serde(default)]
|
||||||
|
pub(crate) files: Vec<DriveFile>,
|
||||||
|
/// Embedded original note when this note is a renote; the shell's own
|
||||||
|
/// text/files are usually empty and the content lives here.
|
||||||
|
#[serde(default)]
|
||||||
|
pub(crate) renote: Option<Box<Note>>,
|
||||||
|
}
|
||||||
|
|
||||||
|
#[derive(Deserialize, Debug)]
|
||||||
|
pub(crate) struct User {
|
||||||
|
pub(crate) name: Option<String>,
|
||||||
|
pub(crate) username: String,
|
||||||
|
}
|
||||||
|
|
||||||
|
#[derive(Deserialize, Debug)]
|
||||||
|
pub(crate) struct DriveFile {
|
||||||
|
#[serde(rename = "type")]
|
||||||
|
pub(crate) mime_type: String,
|
||||||
|
pub(crate) url: String,
|
||||||
|
#[serde(default, rename = "thumbnailUrl")]
|
||||||
|
pub(crate) thumbnail_url: Option<String>,
|
||||||
|
#[serde(default, rename = "isSensitive")]
|
||||||
|
pub(crate) is_sensitive: bool,
|
||||||
|
#[serde(default)]
|
||||||
|
pub(crate) name: Option<String>,
|
||||||
|
}
|
||||||
@@ -1,8 +1,8 @@
|
|||||||
//! Site fetching dispatcher and unified result types.
|
//! Site fetching dispatcher and unified result types.
|
||||||
//!
|
//!
|
||||||
//! Dispatch order: twitter → bsky → pixiv. Each site module exports a
|
//! Dispatch order: twitter → bsky → misskey → pixiv. Each site module
|
||||||
//! `PATTERN`, `enabled()` and `fetch_from_url()`; a future site plugs in by
|
//! exports a `PATTERN`, `enabled()` and `fetch_from_url()`; a future site
|
||||||
//! adding one guarded entry in [`fetch_once`].
|
//! plugs in by adding one guarded entry in `SITES`.
|
||||||
|
|
||||||
use std::future::Future;
|
use std::future::Future;
|
||||||
use std::pin::Pin;
|
use std::pin::Pin;
|
||||||
@@ -14,6 +14,7 @@ use regex::Regex;
|
|||||||
use thiserror::Error;
|
use thiserror::Error;
|
||||||
|
|
||||||
pub mod bsky;
|
pub mod bsky;
|
||||||
|
pub mod misskey;
|
||||||
pub mod pixiv;
|
pub mod pixiv;
|
||||||
pub mod twitter;
|
pub mod twitter;
|
||||||
|
|
||||||
@@ -23,9 +24,9 @@ pub use pixiv::PixivError;
|
|||||||
/// media list and spoiler flag. Produced by [`fetch`].
|
/// media list and spoiler flag. Produced by [`fetch`].
|
||||||
#[derive(Debug)]
|
#[derive(Debug)]
|
||||||
pub struct Fetched {
|
pub struct Fetched {
|
||||||
/// Canonical URL: x.com/{author}/status/{id} |
|
/// Canonical URL: `x.com/{author}/status/{id}` |
|
||||||
/// https://www.pixiv.net/artworks/{id} |
|
/// `https://www.pixiv.net/artworks/{id}` |
|
||||||
/// https://bsky.app/profile/{handle}/post/{rkey}
|
/// `https://bsky.app/profile/{handle}/post/{rkey}`
|
||||||
pub source_url: String,
|
pub source_url: String,
|
||||||
/// The exact HTML produced by the site's caption().
|
/// The exact HTML produced by the site's caption().
|
||||||
pub caption: String,
|
pub caption: String,
|
||||||
@@ -45,8 +46,15 @@ pub struct Fetched {
|
|||||||
pub(crate) _keep_alive: Option<tempfile::TempDir>,
|
pub(crate) _keep_alive: Option<tempfile::TempDir>,
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Pre-escaped values for `{url} {author} {author_url} {title} {tags}`
|
/// Values for the `{url} {author} {author_url} {title} {tags}` placeholders in
|
||||||
/// placeholders in user-supplied caption formats.
|
/// user-supplied caption formats, substituted by [`caption_from_fields`] as
|
||||||
|
/// HTML text (never as an attribute value).
|
||||||
|
///
|
||||||
|
/// `author`, `title` and `tags` come from the site API (post text, display
|
||||||
|
/// names) and are HTML-escaped at construction. `url` and `author_url` stay
|
||||||
|
/// raw: they are canonical URLs the adapter builds from numeric ids and
|
||||||
|
/// API-constrained handles/DIDs, so they carry no escapable character — the
|
||||||
|
/// bot's `/test` report relies on that when it embeds them.
|
||||||
#[derive(Debug)]
|
#[derive(Debug)]
|
||||||
pub(crate) struct RenderData {
|
pub(crate) struct RenderData {
|
||||||
pub url: String,
|
pub url: String,
|
||||||
@@ -165,7 +173,7 @@ pub fn caption_from_fields(
|
|||||||
/// Stable per-post cache key derived from any supported URL, so variant
|
/// Stable per-post cache key derived from any supported URL, so variant
|
||||||
/// domains (x.com / twitter.com / fxtwitter.com, mobile, `/photo/N`
|
/// domains (x.com / twitter.com / fxtwitter.com, mobile, `/photo/N`
|
||||||
/// suffixes) map to the same post. Delegates to each registered site's
|
/// suffixes) map to the same post. Delegates to each registered site's
|
||||||
/// `cache_key` (dispatch order twitter → bsky → pixiv).
|
/// `cache_key` (in registry order).
|
||||||
pub fn cache_key(url: &str) -> Option<String> {
|
pub fn cache_key(url: &str) -> Option<String> {
|
||||||
SITES.iter().find_map(|site| site.cache_key(url))
|
SITES.iter().find_map(|site| site.cache_key(url))
|
||||||
}
|
}
|
||||||
@@ -274,20 +282,21 @@ pub(crate) fn log_once_ffmpeg_missing() {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Site adapter: one impl per supported site (twitter / bsky / pixiv),
|
/// Site adapter: one impl per supported site (twitter / bsky / misskey /
|
||||||
/// registered in [`SITES`]. All site-specific knowledge — URL pattern,
|
/// pixiv), registered in `SITES`. All site-specific knowledge — URL pattern,
|
||||||
/// cache-key format, fetch, retry policy, media-host headers, startup
|
/// cache-key format, fetch, retry policy, media-host headers, startup
|
||||||
/// validation — lives in the site module; the central dispatcher only
|
/// validation — lives in the site module; the central dispatcher only
|
||||||
/// iterates the registry.
|
/// iterates the registry.
|
||||||
///
|
///
|
||||||
/// Async methods return a boxed future (see [`SiteFuture`]): `async fn` /
|
/// Async methods return a boxed future (see `SiteFuture`): `async fn` /
|
||||||
/// RPITIT in traits are not dyn-compatible (verified on rustc 1.95), and
|
/// RPITIT in traits are not dyn-compatible (verified on rustc 1.95), and
|
||||||
/// `+ Send` is required since URL/queue workers spawn these futures. The
|
/// `+ Send` is required since URL/queue workers spawn these futures. The
|
||||||
/// site structs are stateless unit structs, so the boxed futures never
|
/// site structs are stateless unit structs, so the boxed futures never
|
||||||
/// borrow from `self` beyond the call's scope.
|
/// borrow from `self` beyond the call's scope.
|
||||||
pub trait Site: Send + Sync {
|
pub trait Site: Send + Sync {
|
||||||
/// Stable site id (`"twitter"` / `"bsky"` / `"pixiv"`): caption-format
|
/// Stable site id (`"twitter"` / `"bsky"` / `"misskey"` / `"pixiv"`):
|
||||||
/// lookup, cache-key prefixes and the SetFormat whitelist derive from it.
|
/// caption-format lookup, cache-key prefixes and the SetFormat whitelist
|
||||||
|
/// derive from it.
|
||||||
fn id(&self) -> &'static str;
|
fn id(&self) -> &'static str;
|
||||||
/// URL pattern; the dispatcher's first match wins (dispatch order).
|
/// URL pattern; the dispatcher's first match wins (dispatch order).
|
||||||
fn pattern(&self) -> &'static Regex;
|
fn pattern(&self) -> &'static Regex;
|
||||||
@@ -323,12 +332,13 @@ pub trait Site: Send + Sync {
|
|||||||
type SiteFuture<'a, T, E = FetchError> = Pin<Box<dyn Future<Output = Result<T, E>> + Send + 'a>>;
|
type SiteFuture<'a, T, E = FetchError> = Pin<Box<dyn Future<Output = Result<T, E>> + Send + 'a>>;
|
||||||
|
|
||||||
/// The one registry of supported sites, in dispatch order (twitter → bsky →
|
/// The one registry of supported sites, in dispatch order (twitter → bsky →
|
||||||
/// pixiv). Adding a site = new module + one `Box::new(...)` entry here; the
|
/// misskey → pixiv). Adding a site = new module + one `Box::new(...)` entry
|
||||||
/// bot crate never lists sites itself.
|
/// here; the bot crate never lists sites itself.
|
||||||
static SITES: LazyLock<Vec<Box<dyn Site>>> = LazyLock::new(|| {
|
static SITES: LazyLock<Vec<Box<dyn Site>>> = LazyLock::new(|| {
|
||||||
vec![
|
vec![
|
||||||
Box::new(twitter::TwitterSite),
|
Box::new(twitter::TwitterSite),
|
||||||
Box::new(bsky::BskySite),
|
Box::new(bsky::BskySite),
|
||||||
|
Box::new(misskey::MisskeySite),
|
||||||
Box::new(pixiv::PixivSite),
|
Box::new(pixiv::PixivSite),
|
||||||
]
|
]
|
||||||
});
|
});
|
||||||
@@ -373,10 +383,24 @@ pub async fn validate_all() -> Vec<(&'static str, String)> {
|
|||||||
/// are returned immediately; retrying them only wastes attempts against the
|
/// are returned immediately; retrying them only wastes attempts against the
|
||||||
/// source site.
|
/// source site.
|
||||||
pub async fn fetch(url: &str) -> Result<Option<Fetched>, FetchError> {
|
pub async fn fetch(url: &str) -> Result<Option<Fetched>, FetchError> {
|
||||||
|
fetch_with_attempts(url, MAX_FETCH_ATTEMPTS).await
|
||||||
|
}
|
||||||
|
|
||||||
|
/// [`fetch`] without the retry backoff (one attempt). For callers with a
|
||||||
|
/// short deadline: an inline query's answer window is measured in seconds, so
|
||||||
|
/// the 1s + 2s retry sleeps would outlast the query the answer belongs to.
|
||||||
|
pub async fn fetch_once(url: &str) -> Result<Option<Fetched>, FetchError> {
|
||||||
|
fetch_with_attempts(url, 1).await
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Total attempts of the retried [`fetch`] (3: the initial try plus two).
|
||||||
|
const MAX_FETCH_ATTEMPTS: u32 = 3;
|
||||||
|
|
||||||
|
async fn fetch_with_attempts(url: &str, attempts: u32) -> Result<Option<Fetched>, FetchError> {
|
||||||
let Some(site) = find_site(url) else {
|
let Some(site) = find_site(url) else {
|
||||||
return Ok(None);
|
return Ok(None);
|
||||||
};
|
};
|
||||||
for attempt in 0..3u32 {
|
for attempt in 0..attempts.max(1) {
|
||||||
match site.fetch_from_url(url).await {
|
match site.fetch_from_url(url).await {
|
||||||
Ok(fetched) => {
|
Ok(fetched) => {
|
||||||
// Per-request detail: debug only, keyed by the post id.
|
// Per-request detail: debug only, keyed by the post id.
|
||||||
@@ -389,7 +413,7 @@ pub async fn fetch(url: &str) -> Result<Option<Fetched>, FetchError> {
|
|||||||
return Ok(Some(fetched));
|
return Ok(Some(fetched));
|
||||||
}
|
}
|
||||||
Err(err) => {
|
Err(err) => {
|
||||||
if site.is_retryable(&err) && attempt < 2 {
|
if site.is_retryable(&err) && attempt + 1 < attempts {
|
||||||
tokio::time::sleep(Duration::from_secs(1 << attempt)).await;
|
tokio::time::sleep(Duration::from_secs(1 << attempt)).await;
|
||||||
} else {
|
} else {
|
||||||
return Err(err);
|
return Err(err);
|
||||||
@@ -531,10 +555,10 @@ mod tests {
|
|||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn registry_lists_all_sites_in_dispatch_order() {
|
fn registry_lists_all_sites_in_dispatch_order() {
|
||||||
assert_eq!(site_ids(), vec!["twitter", "bsky", "pixiv"]);
|
assert_eq!(site_ids(), vec!["twitter", "bsky", "misskey", "pixiv"]);
|
||||||
// Enabled sites dispatch; unsupported URLs never match.
|
// Enabled sites dispatch; unsupported URLs never match.
|
||||||
assert!(find_site("https://x.com/u/status/1").is_some());
|
assert!(find_site("https://x.com/u/status/1").is_some());
|
||||||
assert!(find_site("https://bsky.app/profile/u/post/3x").is_some());
|
assert!(find_site("https://misskey.io/notes/abc").is_some());
|
||||||
assert!(find_site("https://example.com/x").is_none());
|
assert!(find_site("https://example.com/x").is_none());
|
||||||
// Cache keys are pattern-driven, independent of the enabled() gate
|
// Cache keys are pattern-driven, independent of the enabled() gate
|
||||||
// (pixiv is disabled in tests without PIXIV_REFRESH_TOKEN).
|
// (pixiv is disabled in tests without PIXIV_REFRESH_TOKEN).
|
||||||
|
|||||||
@@ -1,11 +1,11 @@
|
|||||||
[package]
|
[package]
|
||||||
name = "xmedia-bot"
|
name = "xmedia-bot"
|
||||||
version = "1.4.0"
|
version = "1.6.0"
|
||||||
edition = "2024"
|
edition = "2024"
|
||||||
|
|
||||||
[dependencies]
|
[dependencies]
|
||||||
teloxide = { version = "0.17", default-features = false, features = ["webhooks-axum", "macros", "rustls", "ctrlc_handler"] }
|
teloxide = { version = "0.17", default-features = false, features = ["webhooks-axum", "macros", "rustls", "ctrlc_handler"] }
|
||||||
tokio = { version = "1.40", features = ["rt-multi-thread", "macros", "time"] }
|
tokio = { version = "1.40", features = ["rt-multi-thread", "macros", "time", "sync"] }
|
||||||
serde = { version = "1", features = ["derive"] }
|
serde = { version = "1", features = ["derive"] }
|
||||||
serde_json = "1"
|
serde_json = "1"
|
||||||
log = "0.4"
|
log = "0.4"
|
||||||
|
|||||||
@@ -0,0 +1,118 @@
|
|||||||
|
//! Runtime context: the collaborators a handler needs, injected as one struct
|
||||||
|
//! so tests can substitute a scripted sender and tempdir-backed stores.
|
||||||
|
//!
|
||||||
|
//! The production context is assembled from the process-wide statics
|
||||||
|
//! ([`AppContext::from_statics`]); the spawned worker closures hold
|
||||||
|
//! [`CONTEXT`], which is `'static` for that reason.
|
||||||
|
|
||||||
|
use crate::config::Config;
|
||||||
|
use crate::handlers::{CHAT_STORE, CONFIG, LINK_CACHE, TASK_QUEUE};
|
||||||
|
use crate::link_cache::LinkCache;
|
||||||
|
use crate::media_sender::MediaSender;
|
||||||
|
use crate::queue::PersistentTaskQueue;
|
||||||
|
use crate::send::BOT;
|
||||||
|
use crate::state::ChatStore;
|
||||||
|
use std::sync::LazyLock;
|
||||||
|
|
||||||
|
pub struct AppContext<'a> {
|
||||||
|
pub sender: &'a dyn MediaSender,
|
||||||
|
pub chat_store: &'a ChatStore,
|
||||||
|
pub task_queue: &'a PersistentTaskQueue,
|
||||||
|
pub link_cache: &'a LinkCache,
|
||||||
|
pub config: &'a Config,
|
||||||
|
}
|
||||||
|
|
||||||
|
impl<'a> AppContext<'a> {
|
||||||
|
/// The stores are the process-wide statics; `sender` is whatever the caller
|
||||||
|
/// was handed (the dispatcher's `Bot` clone for update handlers, the shared
|
||||||
|
/// queue `Bot` for the worker loops). Update handlers build their own
|
||||||
|
/// context from the `Bot` they received so the same code path works with an
|
||||||
|
/// injected mock in tests.
|
||||||
|
pub fn from_statics(sender: &'a dyn MediaSender) -> AppContext<'a> {
|
||||||
|
AppContext {
|
||||||
|
sender,
|
||||||
|
chat_store: &CHAT_STORE,
|
||||||
|
task_queue: &TASK_QUEUE,
|
||||||
|
link_cache: &LINK_CACHE,
|
||||||
|
config: &CONFIG,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// The URL/queue workers' context: `'static` because `tokio::spawn`ed closures
|
||||||
|
/// and the queue's handler type require it.
|
||||||
|
pub static CONTEXT: LazyLock<AppContext<'static>> =
|
||||||
|
LazyLock::new(|| AppContext::from_statics(&*BOT));
|
||||||
|
|
||||||
|
/// Test support: a tempdir-backed set of stores plus the context borrowing
|
||||||
|
/// them, so a handler test needs one line of setup.
|
||||||
|
#[cfg(test)]
|
||||||
|
pub(crate) mod test_support {
|
||||||
|
use super::*;
|
||||||
|
use std::sync::Arc;
|
||||||
|
|
||||||
|
pub(crate) struct TestStores {
|
||||||
|
_dir: tempfile::TempDir,
|
||||||
|
pool: Arc<crate::db::DbPool>,
|
||||||
|
chat_store: ChatStore,
|
||||||
|
task_queue: PersistentTaskQueue,
|
||||||
|
link_cache: LinkCache,
|
||||||
|
config: Config,
|
||||||
|
}
|
||||||
|
|
||||||
|
impl TestStores {
|
||||||
|
pub(crate) fn new() -> Self {
|
||||||
|
let dir = tempfile::tempdir().unwrap();
|
||||||
|
let pool = crate::db::open_store(dir.path().join("ctx.db").to_str().unwrap()).unwrap();
|
||||||
|
TestStores {
|
||||||
|
_dir: dir,
|
||||||
|
chat_store: ChatStore::new(Arc::clone(&pool)),
|
||||||
|
task_queue: PersistentTaskQueue::new(Arc::clone(&pool)),
|
||||||
|
link_cache: LinkCache::new(Arc::clone(&pool)),
|
||||||
|
config: Config::load(),
|
||||||
|
pool,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
pub(crate) fn ctx<'a>(&'a self, sender: &'a dyn MediaSender) -> AppContext<'a> {
|
||||||
|
AppContext {
|
||||||
|
sender,
|
||||||
|
chat_store: &self.chat_store,
|
||||||
|
task_queue: &self.task_queue,
|
||||||
|
link_cache: &self.link_cache,
|
||||||
|
config: &self.config,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
pub(crate) fn chat_store(&self) -> &ChatStore {
|
||||||
|
&self.chat_store
|
||||||
|
}
|
||||||
|
|
||||||
|
pub(crate) fn link_cache(&self) -> &LinkCache {
|
||||||
|
&self.link_cache
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Rows persisted in the task queue: what "queued for retry" looks like
|
||||||
|
/// from the outside.
|
||||||
|
pub(crate) async fn queued_tasks(&self) -> i64 {
|
||||||
|
let pool = Arc::clone(&self.pool);
|
||||||
|
pool.with_conn(|conn| {
|
||||||
|
conn.query_row("SELECT COUNT(*) FROM tasks", [], |row| row.get(0))
|
||||||
|
})
|
||||||
|
.await
|
||||||
|
.unwrap()
|
||||||
|
}
|
||||||
|
|
||||||
|
/// The single queued task payload, for asserting what was rescheduled.
|
||||||
|
pub(crate) async fn queued_payload(&self) -> serde_json::Value {
|
||||||
|
let pool = Arc::clone(&self.pool);
|
||||||
|
let payload: String = pool
|
||||||
|
.with_conn(|conn| {
|
||||||
|
conn.query_row("SELECT payload FROM tasks LIMIT 1", [], |row| row.get(0))
|
||||||
|
})
|
||||||
|
.await
|
||||||
|
.unwrap();
|
||||||
|
serde_json::from_str(&payload).unwrap()
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -160,3 +160,9 @@ pub fn now_f64() -> f64 {
|
|||||||
.map(|d| d.as_secs_f64())
|
.map(|d| d.as_secs_f64())
|
||||||
.unwrap_or(0.0)
|
.unwrap_or(0.0)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// Unix timestamp in whole seconds. Same clock as [`now_f64`], for fields
|
||||||
|
/// that store integer seconds (chat-state expiry, edit prompts).
|
||||||
|
pub fn unix_now() -> i64 {
|
||||||
|
now_f64() as i64
|
||||||
|
}
|
||||||
|
|||||||
@@ -1,56 +1,76 @@
|
|||||||
//! Callback query handling: the edit-before-forward prompt's "forward" and
|
//! Callback query handling: the edit-before-forward prompt's `"forward"` and
|
||||||
//! "template|<name>" buttons.
|
//! `"template|<name>"` buttons.
|
||||||
|
//!
|
||||||
|
//! [`callback_query_handler`] is the dptree entry; it only pulls the plain
|
||||||
|
//! values out of the teloxide update and hands them to [`handle_callback`],
|
||||||
|
//! which holds the button logic and is driven directly by tests.
|
||||||
|
|
||||||
use super::urls::enqueue_retry;
|
use crate::ctx::AppContext;
|
||||||
use super::{CHAT_STORE, CONFIG, TASK_QUEUE};
|
use crate::db::unix_now;
|
||||||
use crate::send::{self, Task};
|
use crate::send::{self, Task};
|
||||||
use crate::state::unix_now;
|
|
||||||
use teloxide::RequestError;
|
use teloxide::RequestError;
|
||||||
use teloxide::prelude::*;
|
use teloxide::prelude::*;
|
||||||
use teloxide::types::{CallbackQuery, ChatId, MessageId, ParseMode};
|
use teloxide::types::{CallbackQuery, CallbackQueryId, MessageId};
|
||||||
|
|
||||||
|
/// The `"forward"` button's data.
|
||||||
|
const FORWARD: &str = "forward";
|
||||||
|
/// Prefix of a template button's data: `"template|<name>"`.
|
||||||
|
const TEMPLATE_PREFIX: &str = "template|";
|
||||||
|
|
||||||
pub async fn callback_query_handler(bot: Bot, query: CallbackQuery) -> Result<(), RequestError> {
|
pub async fn callback_query_handler(bot: Bot, query: CallbackQuery) -> Result<(), RequestError> {
|
||||||
let callback_query_id = query.id;
|
|
||||||
let data = query.data.clone();
|
|
||||||
let Some(message) = &query.message else {
|
let Some(message) = &query.message else {
|
||||||
return respond(());
|
return respond(());
|
||||||
};
|
};
|
||||||
let chat_id = message.chat().id.0;
|
let Some(data) = query.data.clone() else {
|
||||||
let prompt_message_id = message.id().0 as i64;
|
return respond(());
|
||||||
let ttl_secs = CONFIG.edit_message_ttl.as_secs() as i64;
|
};
|
||||||
let chat_data = CHAT_STORE.get(chat_id).await;
|
let ctx = AppContext::from_statics(&bot);
|
||||||
|
handle_callback(
|
||||||
|
&ctx,
|
||||||
|
query.id.clone(),
|
||||||
|
message.chat().id.0,
|
||||||
|
message.id().0 as i64,
|
||||||
|
&data,
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
respond(())
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Handles one button press on the edit-before-forward prompt.
|
||||||
|
async fn handle_callback(
|
||||||
|
ctx: &AppContext<'_>,
|
||||||
|
callback_query_id: CallbackQueryId,
|
||||||
|
chat_id: i64,
|
||||||
|
prompt_message_id: i64,
|
||||||
|
data: &str,
|
||||||
|
) {
|
||||||
|
let ttl_secs = ctx.config.edit_message_ttl.as_secs() as i64;
|
||||||
|
let chat_data = ctx.chat_store.get(chat_id).await;
|
||||||
let edit = chat_data.edit_message.get(&prompt_message_id).cloned();
|
let edit = chat_data.edit_message.get(&prompt_message_id).cloned();
|
||||||
let Some(edit) = edit else {
|
let Some(edit) = edit else {
|
||||||
log::debug!(
|
log::debug!("callback from {chat_id}: no edit record for prompt {prompt_message_id}");
|
||||||
"callback from {}: no edit record for prompt {prompt_message_id}",
|
let _ = ctx
|
||||||
chat_id
|
.sender
|
||||||
);
|
.answer_callback_query(callback_query_id, Some("Expired".to_string()))
|
||||||
bot.answer_callback_query(callback_query_id)
|
.await;
|
||||||
.text("Expired")
|
return;
|
||||||
.await?;
|
|
||||||
return respond(());
|
|
||||||
};
|
};
|
||||||
// Lazy expiry: a stale record (past the TTL, not yet swept) is dropped.
|
// Lazy expiry: a stale record (past the TTL, not yet swept) is dropped.
|
||||||
if edit.created_at + ttl_secs <= unix_now() {
|
if edit.created_at + ttl_secs <= unix_now() {
|
||||||
CHAT_STORE
|
ctx.chat_store
|
||||||
.update(chat_id, |data| {
|
.update(chat_id, |data| {
|
||||||
data.edit_message.remove(&prompt_message_id);
|
data.edit_message.remove(&prompt_message_id);
|
||||||
})
|
})
|
||||||
.await;
|
.await;
|
||||||
bot.answer_callback_query(callback_query_id)
|
let _ = ctx
|
||||||
.text("Expired")
|
.sender
|
||||||
.await?;
|
.answer_callback_query(callback_query_id, Some("Expired".to_string()))
|
||||||
return respond(());
|
.await;
|
||||||
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
let Some(data) = data else {
|
log::info!("callback from {chat_id} on prompt {prompt_message_id}: {data}");
|
||||||
return respond(());
|
if data == FORWARD {
|
||||||
};
|
|
||||||
log::info!(
|
|
||||||
"callback from {} on prompt {prompt_message_id}: {data}",
|
|
||||||
chat_id
|
|
||||||
);
|
|
||||||
if data == "forward" {
|
|
||||||
match chat_data.forward_channel_id {
|
match chat_data.forward_channel_id {
|
||||||
Some(channel_id) => {
|
Some(channel_id) => {
|
||||||
let forward_task = Task::ForwardMessages {
|
let forward_task = Task::ForwardMessages {
|
||||||
@@ -60,62 +80,72 @@ pub async fn callback_query_handler(bot: Bot, query: CallbackQuery) -> Result<()
|
|||||||
notify_chat_id: Some(chat_id),
|
notify_chat_id: Some(chat_id),
|
||||||
notify_message_id: Some(prompt_message_id),
|
notify_message_id: Some(prompt_message_id),
|
||||||
};
|
};
|
||||||
match send::forward_messages(&bot, &forward_task).await {
|
let (answer, settled) = match send::forward_messages(ctx, &forward_task).await {
|
||||||
Ok(()) => {
|
Ok(()) => {
|
||||||
log::info!(
|
log::info!(
|
||||||
"forwarded {} message(s) to channel {channel_id}",
|
"forwarded {} message(s) to channel {channel_id}",
|
||||||
edit.forward_message_ids.len()
|
edit.forward_message_ids.len()
|
||||||
);
|
);
|
||||||
bot.answer_callback_query(callback_query_id)
|
("✅ Forwarded".to_string(), true)
|
||||||
.text("✅ Forwarded")
|
|
||||||
.await?;
|
|
||||||
let _ = bot
|
|
||||||
.delete_message(ChatId(chat_id), MessageId(prompt_message_id as i32))
|
|
||||||
.await;
|
|
||||||
CHAT_STORE
|
|
||||||
.update(chat_id, |data| {
|
|
||||||
data.edit_message.remove(&prompt_message_id);
|
|
||||||
})
|
|
||||||
.await;
|
|
||||||
}
|
}
|
||||||
Err(send::SendError::Retryable {
|
Err(send::SendError::Retryable {
|
||||||
delay_seconds,
|
delay_seconds,
|
||||||
task,
|
task,
|
||||||
}) => {
|
}) => {
|
||||||
log::info!("forward queued for retry in {delay_seconds:.1}s");
|
log::info!("forward queued for retry in {delay_seconds:.1}s");
|
||||||
enqueue_retry(&TASK_QUEUE, task, delay_seconds).await;
|
send::enqueue_retry(ctx.task_queue, *task, delay_seconds).await;
|
||||||
bot.answer_callback_query(callback_query_id)
|
("Forward queued for retry.".to_string(), false)
|
||||||
.text("Forward queued for retry.")
|
|
||||||
.await?;
|
|
||||||
}
|
}
|
||||||
Err(send::SendError::Permanent { message, .. }) => {
|
Err(send::SendError::Permanent { message, .. }) => {
|
||||||
log::error!("forward failed permanently: {message}");
|
log::error!("forward failed permanently: {message}");
|
||||||
bot.answer_callback_query(callback_query_id)
|
(format!("Forward failed: {message}"), false)
|
||||||
.text(format!("Forward failed: {message}"))
|
|
||||||
.await?;
|
|
||||||
}
|
}
|
||||||
|
};
|
||||||
|
if settled {
|
||||||
|
// The prompt is done: drop it and its record.
|
||||||
|
let _ = ctx
|
||||||
|
.sender
|
||||||
|
.delete_message(ChatId(chat_id), MessageId(prompt_message_id as i32))
|
||||||
|
.await;
|
||||||
|
ctx.chat_store
|
||||||
|
.update(chat_id, |data| {
|
||||||
|
data.edit_message.remove(&prompt_message_id);
|
||||||
|
})
|
||||||
|
.await;
|
||||||
}
|
}
|
||||||
|
let _ = ctx
|
||||||
|
.sender
|
||||||
|
.answer_callback_query(callback_query_id, Some(answer))
|
||||||
|
.await;
|
||||||
}
|
}
|
||||||
None => {
|
None => {
|
||||||
log::debug!("forward callback without a forward channel set");
|
log::debug!("forward callback without a forward channel set");
|
||||||
bot.answer_callback_query(callback_query_id)
|
let _ = ctx
|
||||||
.text("No forward channel set.")
|
.sender
|
||||||
.await?;
|
.answer_callback_query(
|
||||||
|
callback_query_id,
|
||||||
|
Some("No forward channel set.".to_string()),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return respond(());
|
return;
|
||||||
}
|
}
|
||||||
if let Some(name) = data.strip_prefix("template|") {
|
|
||||||
|
if let Some(name) = data.strip_prefix(TEMPLATE_PREFIX) {
|
||||||
if let Some(template_html) = chat_data.template.get(name).cloned()
|
if let Some(template_html) = chat_data.template.get(name).cloned()
|
||||||
&& let Some(first_forward_id) = edit.forward_message_ids.first().copied()
|
&& let Some(first_forward_id) = edit.forward_message_ids.first().copied()
|
||||||
{
|
{
|
||||||
// Raw template including the [] placeholder (Python parity).
|
// Raw template including the [] placeholder (Python parity).
|
||||||
let _ = bot
|
let _ = ctx
|
||||||
.edit_message_caption(ChatId(chat_id), MessageId(first_forward_id as i32))
|
.sender
|
||||||
.caption(template_html)
|
.edit_message_caption(
|
||||||
.parse_mode(ParseMode::Html)
|
ChatId(chat_id),
|
||||||
|
MessageId(first_forward_id as i32),
|
||||||
|
template_html,
|
||||||
|
)
|
||||||
.await;
|
.await;
|
||||||
CHAT_STORE
|
ctx.chat_store
|
||||||
.update(chat_id, |data| {
|
.update(chat_id, |data| {
|
||||||
if let Some(entry) = data.edit_message.get_mut(&prompt_message_id) {
|
if let Some(entry) = data.edit_message.get_mut(&prompt_message_id) {
|
||||||
entry.template = name.to_string();
|
entry.template = name.to_string();
|
||||||
@@ -124,7 +154,168 @@ pub async fn callback_query_handler(bot: Bot, query: CallbackQuery) -> Result<()
|
|||||||
.await;
|
.await;
|
||||||
log::info!("template '{name}' applied to prompt {prompt_message_id}");
|
log::info!("template '{name}' applied to prompt {prompt_message_id}");
|
||||||
}
|
}
|
||||||
bot.answer_callback_query(callback_query_id).await?;
|
let _ = ctx
|
||||||
|
.sender
|
||||||
|
.answer_callback_query(callback_query_id, None)
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
#[cfg(test)]
|
||||||
|
mod tests {
|
||||||
|
use super::*;
|
||||||
|
use crate::ctx::test_support::TestStores;
|
||||||
|
use crate::media_sender::test_support::{MockSender, Outcome};
|
||||||
|
use crate::state::EditMessage;
|
||||||
|
use teloxide::ApiError;
|
||||||
|
|
||||||
|
/// The edit-before-forward prompt's message id in these tests.
|
||||||
|
const PROMPT_ID: i64 = 7;
|
||||||
|
/// The message the prompt refers to (the one whose caption is swapped).
|
||||||
|
const FORWARDED_ID: i64 = 9;
|
||||||
|
|
||||||
|
fn api_error() -> RequestError {
|
||||||
|
RequestError::Api(ApiError::Unknown("Bad Request: chat not found".into()))
|
||||||
|
}
|
||||||
|
|
||||||
|
fn callback_id() -> CallbackQueryId {
|
||||||
|
CallbackQueryId("cb-1".to_string())
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Seeds a live prompt record plus a forward channel and a template;
|
||||||
|
/// `created_at` backdates the record for the expiry cases.
|
||||||
|
async fn seed_prompt(ctx: &AppContext<'_>, created_at: i64) {
|
||||||
|
ctx.chat_store
|
||||||
|
.update(1, |data| {
|
||||||
|
data.forward_channel_id = Some(2);
|
||||||
|
data.template
|
||||||
|
.insert("tpl".to_string(), "<b>[]</b>".to_string());
|
||||||
|
data.edit_message.insert(
|
||||||
|
PROMPT_ID,
|
||||||
|
EditMessage {
|
||||||
|
url: "https://x.com/u/status/1".into(),
|
||||||
|
chat_id: 1,
|
||||||
|
forward_message_ids: vec![FORWARDED_ID],
|
||||||
|
template: String::new(),
|
||||||
|
created_at,
|
||||||
|
},
|
||||||
|
);
|
||||||
|
})
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn template_button_swaps_the_caption_and_records_the_choice() {
|
||||||
|
let sender = MockSender::scripted(vec![Outcome::EditOk], api_error);
|
||||||
|
let stores = TestStores::new();
|
||||||
|
let ctx = stores.ctx(&sender);
|
||||||
|
seed_prompt(&ctx, crate::db::unix_now()).await;
|
||||||
|
|
||||||
|
handle_callback(&ctx, callback_id(), 1, PROMPT_ID, "template|tpl").await;
|
||||||
|
|
||||||
|
assert_eq!(
|
||||||
|
sender.calls(),
|
||||||
|
vec!["edit_message_caption", "answer_callback_query"]
|
||||||
|
);
|
||||||
|
// The raw template, including the [] the user edits into.
|
||||||
|
assert_eq!(sender.captions(), vec!["<b>[]</b>"]);
|
||||||
|
assert_eq!(sender.answers(), vec![None]);
|
||||||
|
let data = ctx.chat_store.get(1).await;
|
||||||
|
assert_eq!(data.edit_message[&PROMPT_ID].template, "tpl");
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn forward_button_copies_then_clears_the_prompt() {
|
||||||
|
let sender = MockSender::scripted(vec![Outcome::CopyOk], api_error);
|
||||||
|
let stores = TestStores::new();
|
||||||
|
let ctx = stores.ctx(&sender);
|
||||||
|
seed_prompt(&ctx, crate::db::unix_now()).await;
|
||||||
|
|
||||||
|
handle_callback(&ctx, callback_id(), 1, PROMPT_ID, "forward").await;
|
||||||
|
|
||||||
|
assert_eq!(
|
||||||
|
sender.calls(),
|
||||||
|
vec!["copy_messages", "delete_message", "answer_callback_query"]
|
||||||
|
);
|
||||||
|
assert_eq!(sender.answers(), vec![Some("✅ Forwarded".to_string())]);
|
||||||
|
assert!(
|
||||||
|
ctx.chat_store.get(1).await.edit_message.is_empty(),
|
||||||
|
"a settled prompt must drop its record"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn forward_without_a_channel_is_reported() {
|
||||||
|
let sender = MockSender::scripted(vec![], api_error);
|
||||||
|
let stores = TestStores::new();
|
||||||
|
let ctx = stores.ctx(&sender);
|
||||||
|
seed_prompt(&ctx, crate::db::unix_now()).await;
|
||||||
|
ctx.chat_store
|
||||||
|
.update(1, |data| data.forward_channel_id = None)
|
||||||
|
.await;
|
||||||
|
|
||||||
|
handle_callback(&ctx, callback_id(), 1, PROMPT_ID, "forward").await;
|
||||||
|
|
||||||
|
assert_eq!(sender.calls(), vec!["answer_callback_query"]);
|
||||||
|
assert_eq!(
|
||||||
|
sender.answers(),
|
||||||
|
vec![Some("No forward channel set.".to_string())]
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn retryable_forward_is_queued_and_keeps_the_prompt() {
|
||||||
|
use teloxide::types::Seconds;
|
||||||
|
let sender = MockSender::scripted(vec![Outcome::CopyErr], || {
|
||||||
|
RequestError::RetryAfter(Seconds::from_seconds(7))
|
||||||
|
});
|
||||||
|
let stores = TestStores::new();
|
||||||
|
let ctx = stores.ctx(&sender);
|
||||||
|
seed_prompt(&ctx, crate::db::unix_now()).await;
|
||||||
|
|
||||||
|
handle_callback(&ctx, callback_id(), 1, PROMPT_ID, "forward").await;
|
||||||
|
|
||||||
|
assert_eq!(
|
||||||
|
sender.calls(),
|
||||||
|
vec!["copy_messages", "answer_callback_query"]
|
||||||
|
);
|
||||||
|
assert_eq!(
|
||||||
|
sender.answers(),
|
||||||
|
vec![Some("Forward queued for retry.".to_string())]
|
||||||
|
);
|
||||||
|
assert_eq!(stores.queued_tasks().await, 1);
|
||||||
|
// The prompt is not settled: the queued retry still needs the record.
|
||||||
|
assert!(
|
||||||
|
ctx.chat_store
|
||||||
|
.get(1)
|
||||||
|
.await
|
||||||
|
.edit_message
|
||||||
|
.contains_key(&PROMPT_ID)
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn unknown_and_expired_prompts_answer_expired() {
|
||||||
|
let sender = MockSender::scripted(vec![], api_error);
|
||||||
|
let stores = TestStores::new();
|
||||||
|
let ctx = stores.ctx(&sender);
|
||||||
|
|
||||||
|
// No record at all.
|
||||||
|
handle_callback(&ctx, callback_id(), 1, PROMPT_ID, "forward").await;
|
||||||
|
assert_eq!(sender.answers(), vec![Some("Expired".to_string())]);
|
||||||
|
|
||||||
|
// A record past its TTL (nothing swept it yet) is dropped on use.
|
||||||
|
let stale = crate::db::unix_now() - ctx.config.edit_message_ttl.as_secs() as i64 - 1;
|
||||||
|
seed_prompt(&ctx, stale).await;
|
||||||
|
handle_callback(&ctx, callback_id(), 1, PROMPT_ID, "forward").await;
|
||||||
|
assert_eq!(
|
||||||
|
sender.answers(),
|
||||||
|
vec![Some("Expired".to_string()), Some("Expired".to_string())]
|
||||||
|
);
|
||||||
|
assert!(
|
||||||
|
ctx.chat_store.get(1).await.edit_message.is_empty(),
|
||||||
|
"the expired record must be dropped"
|
||||||
|
);
|
||||||
|
assert_eq!(sender.calls(), vec!["answer_callback_query"; 2]);
|
||||||
}
|
}
|
||||||
respond(())
|
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,7 +1,9 @@
|
|||||||
//! Bot command parsing, the `/`-command executor and `setMyCommands`
|
//! Bot command parsing, the `/`-command executor and `setMyCommands`
|
||||||
//! registration. URL/inline/callback flows live in their own modules.
|
//! registration. URL/inline/callback flows live in their own modules.
|
||||||
|
|
||||||
|
use super::urls::{PostSend, url_media};
|
||||||
use super::{CHAT_STORE, CONFIG, LINK_CACHE, log_key, reply, reply_html};
|
use super::{CHAT_STORE, CONFIG, LINK_CACHE, log_key, reply, reply_html};
|
||||||
|
use crate::ctx::AppContext;
|
||||||
use teloxide::RequestError;
|
use teloxide::RequestError;
|
||||||
use teloxide::prelude::*;
|
use teloxide::prelude::*;
|
||||||
use teloxide::types::{ChatId, Message, Recipient};
|
use teloxide::types::{ChatId, Message, Recipient};
|
||||||
@@ -31,7 +33,7 @@ pub(crate) enum Command {
|
|||||||
parse_with = "split"
|
parse_with = "split"
|
||||||
)]
|
)]
|
||||||
SetTemplate(String),
|
SetTemplate(String),
|
||||||
#[command(description = "Show chat state (debug)")]
|
#[command(description = "Show chat state (debug; admin only)")]
|
||||||
BotDict,
|
BotDict,
|
||||||
#[command(description = "Set site caption format", parse_with = "split")]
|
#[command(description = "Set site caption format", parse_with = "split")]
|
||||||
SetFormat(String),
|
SetFormat(String),
|
||||||
@@ -41,17 +43,22 @@ pub(crate) enum Command {
|
|||||||
)]
|
)]
|
||||||
ClearCache(String),
|
ClearCache(String),
|
||||||
#[command(
|
#[command(
|
||||||
description = "Test link parsing (debug; no media sent)",
|
description = "Send a link's media (no forwarding)",
|
||||||
parse_with = parse_test_arg
|
parse_with = parse_arg_remainder
|
||||||
)]
|
)]
|
||||||
Test(String),
|
Test(String),
|
||||||
|
#[command(
|
||||||
|
description = "Parse a link and report it (debug; nothing sent)",
|
||||||
|
parse_with = parse_arg_remainder
|
||||||
|
)]
|
||||||
|
Debug(String),
|
||||||
}
|
}
|
||||||
|
|
||||||
/// `/test` argument parser: the whole remainder after the command name,
|
/// `/test` and `/debug` argument parser: the whole remainder after the command
|
||||||
/// trimmed. The built-in `split` parser takes exactly one space-separated
|
/// name, trimmed. The built-in `split` parser takes exactly one space-separated
|
||||||
/// token and rejects the rest, so a URL followed by a trailing space (or
|
/// token and rejects the rest, so a URL followed by a trailing space (or
|
||||||
/// pasted text) would silently fall through to the URL flow instead.
|
/// pasted text) would silently fall through to the URL flow instead.
|
||||||
fn parse_test_arg(s: String) -> Result<(String,), ParseError> {
|
fn parse_arg_remainder(s: String) -> Result<(String,), ParseError> {
|
||||||
Ok((s.trim().to_string(),))
|
Ok((s.trim().to_string(),))
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -226,9 +233,28 @@ pub(crate) async fn execute_command(
|
|||||||
reply(bot, message.chat.id.0, message.id, text).await?;
|
reply(bot, message.chat.id.0, message.id, text).await?;
|
||||||
}
|
}
|
||||||
Command::BotDict => {
|
Command::BotDict => {
|
||||||
|
// Debug dump of the chat's persisted state: admin only (it echoes
|
||||||
|
// forward-channel ids and templates to whoever asks).
|
||||||
|
let sender_id = message
|
||||||
|
.from
|
||||||
|
.as_ref()
|
||||||
|
.map(|user| user.id.0 as i64)
|
||||||
|
.unwrap_or(-1);
|
||||||
|
if !CONFIG.admin_ids.contains(&sender_id) {
|
||||||
|
reply(bot, message.chat.id.0, message.id, "Admin only.").await?;
|
||||||
|
return Ok(());
|
||||||
|
}
|
||||||
let chat_data = CHAT_STORE.get(message.chat.id.0).await;
|
let chat_data = CHAT_STORE.get(message.chat.id.0).await;
|
||||||
let debug = format!("{chat_data:?}");
|
let debug = html_escape::encode_text(&format!("{chat_data:?}")).into_owned();
|
||||||
let text = html_escape::encode_text(&debug).into_owned();
|
// A chat with many templates/edit records exceeds Telegram's 4096
|
||||||
|
// char message limit; the dump is plain text (no parse mode), so a
|
||||||
|
// plain byte-boundary cut is safe.
|
||||||
|
let end = debug.floor_char_boundary(MAX_DEBUG_DUMP_CHARS.min(debug.len()));
|
||||||
|
let text = if end < debug.len() {
|
||||||
|
format!("{}…", &debug[..end])
|
||||||
|
} else {
|
||||||
|
debug
|
||||||
|
};
|
||||||
reply(bot, message.chat.id.0, message.id, text).await?;
|
reply(bot, message.chat.id.0, message.id, text).await?;
|
||||||
}
|
}
|
||||||
Command::SetFormat(arg) => {
|
Command::SetFormat(arg) => {
|
||||||
@@ -253,7 +279,7 @@ pub(crate) async fn execute_command(
|
|||||||
bot,
|
bot,
|
||||||
message.chat.id.0,
|
message.chat.id.0,
|
||||||
message.id,
|
message.id,
|
||||||
"Unknown site. Use twitter, bsky or pixiv.",
|
"Unknown site. Use twitter, bsky, pixiv or misskey.",
|
||||||
)
|
)
|
||||||
.await?;
|
.await?;
|
||||||
return Ok(());
|
return Ok(());
|
||||||
@@ -294,7 +320,7 @@ pub(crate) async fn execute_command(
|
|||||||
bot,
|
bot,
|
||||||
message.chat.id.0,
|
message.chat.id.0,
|
||||||
message.id,
|
message.id,
|
||||||
"Unrecognized link. Use a twitter/x, pixiv or bsky post URL.",
|
"Unrecognized link. Use a twitter/x, pixiv, bsky or misskey post URL.",
|
||||||
)
|
)
|
||||||
.await?;
|
.await?;
|
||||||
return Ok(());
|
return Ok(());
|
||||||
@@ -327,17 +353,54 @@ pub(crate) async fn execute_command(
|
|||||||
.await?;
|
.await?;
|
||||||
return Ok(());
|
return Ok(());
|
||||||
}
|
}
|
||||||
|
if x_media::site::cache_key(url).is_none() {
|
||||||
|
reply(
|
||||||
|
bot,
|
||||||
|
message.chat.id.0,
|
||||||
|
message.id,
|
||||||
|
"No enabled site matches this link (twitter/x, pixiv, bsky or misskey).",
|
||||||
|
)
|
||||||
|
.await?;
|
||||||
|
return Ok(());
|
||||||
|
}
|
||||||
|
// The ordinary link pipeline with the chat's post-send actions
|
||||||
|
// suppressed: the media is sent (and cached) like a normal link,
|
||||||
|
// but nothing is forwarded to the channel and no
|
||||||
|
// edit-before-forward prompt opens. Info level echoes the
|
||||||
|
// normalized key (never the raw URL) per the logging convention.
|
||||||
|
log::info!("test: sending [key={}]", log_key(url));
|
||||||
|
let ctx = AppContext::from_statics(bot);
|
||||||
|
url_media(
|
||||||
|
&ctx,
|
||||||
|
message.chat.id.0,
|
||||||
|
message.id.0 as i64,
|
||||||
|
url,
|
||||||
|
PostSend::Suppressed,
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
Command::Debug(arg) => {
|
||||||
|
let url = arg.trim();
|
||||||
|
if url.is_empty() {
|
||||||
|
reply(
|
||||||
|
bot,
|
||||||
|
message.chat.id.0,
|
||||||
|
message.id,
|
||||||
|
"Usage: /debug <post url>",
|
||||||
|
)
|
||||||
|
.await?;
|
||||||
|
return Ok(());
|
||||||
|
}
|
||||||
// Debug tool: report the parse result only — nothing is sent,
|
// Debug tool: report the parse result only — nothing is sent,
|
||||||
// cached or forwarded. Info level echoes the normalized key
|
// cached or forwarded.
|
||||||
// (never the raw URL) per the logging convention.
|
log::info!("debug: parsing [key={}]", log_key(url));
|
||||||
log::info!("test: parsing [key={}]", log_key(url));
|
|
||||||
match x_media::site::fetch(url).await {
|
match x_media::site::fetch(url).await {
|
||||||
Ok(None) => {
|
Ok(None) => {
|
||||||
reply(
|
reply(
|
||||||
bot,
|
bot,
|
||||||
message.chat.id.0,
|
message.chat.id.0,
|
||||||
message.id,
|
message.id,
|
||||||
"No enabled site matches this link (twitter/x, pixiv or bsky).",
|
"No enabled site matches this link (twitter/x, pixiv, bsky or misskey).",
|
||||||
)
|
)
|
||||||
.await?;
|
.await?;
|
||||||
}
|
}
|
||||||
@@ -351,7 +414,7 @@ pub(crate) async fn execute_command(
|
|||||||
.await?;
|
.await?;
|
||||||
}
|
}
|
||||||
Ok(Some(fetched)) => {
|
Ok(Some(fetched)) => {
|
||||||
let report = test_parse_report(
|
let report = debug_report(
|
||||||
url,
|
url,
|
||||||
fetched.site_name(),
|
fetched.site_name(),
|
||||||
&fetched.source_url,
|
&fetched.source_url,
|
||||||
@@ -387,9 +450,13 @@ pub async fn register_commands(bot: &Bot) -> Result<(), RequestError> {
|
|||||||
|
|
||||||
/// Telegram's plain-text message limit is 4096 chars; the report stays under
|
/// Telegram's plain-text message limit is 4096 chars; the report stays under
|
||||||
/// it even for very large threads (many media lines + a long caption).
|
/// it even for very large threads (many media lines + a long caption).
|
||||||
const MAX_TEST_REPORT_CHARS: usize = 4000;
|
const MAX_DEBUG_REPORT_CHARS: usize = 4000;
|
||||||
|
|
||||||
/// Builds the HTML report for the `/test` command: what the parser produced
|
/// Cap for the `/bot_dict` debug dump: the state is echoed as one plain-text
|
||||||
|
/// message, so it must stay under Telegram's 4096-char limit.
|
||||||
|
const MAX_DEBUG_DUMP_CHARS: usize = 3500;
|
||||||
|
|
||||||
|
/// Builds the HTML report for the `/debug` command: what the parser produced
|
||||||
/// for a link (site, canonical URL, title/author/tags, caption and the media
|
/// for a link (site, canonical URL, title/author/tags, caption and the media
|
||||||
/// list) — no media is sent and nothing is cached or forwarded. Sent with
|
/// list) — no media is sent and nothing is cached or forwarded. Sent with
|
||||||
/// HTML parse mode: raw fields are escaped, the pre-escaped render fields are
|
/// HTML parse mode: raw fields are escaped, the pre-escaped render fields are
|
||||||
@@ -399,7 +466,7 @@ const MAX_TEST_REPORT_CHARS: usize = 4000;
|
|||||||
/// constructing a `Fetched` (its render fields are `pub(crate)` to the
|
/// constructing a `Fetched` (its render fields are `pub(crate)` to the
|
||||||
/// x-media crate).
|
/// x-media crate).
|
||||||
#[allow(clippy::too_many_arguments)]
|
#[allow(clippy::too_many_arguments)]
|
||||||
fn test_parse_report(
|
fn debug_report(
|
||||||
url: &str,
|
url: &str,
|
||||||
site_id: &str,
|
site_id: &str,
|
||||||
source_url: &str,
|
source_url: &str,
|
||||||
@@ -460,8 +527,8 @@ fn test_parse_report(
|
|||||||
"
|
"
|
||||||
",
|
",
|
||||||
);
|
);
|
||||||
if out.chars().count() > MAX_TEST_REPORT_CHARS {
|
if out.chars().count() > MAX_DEBUG_REPORT_CHARS {
|
||||||
let end = out.floor_char_boundary(MAX_TEST_REPORT_CHARS - 1);
|
let end = out.floor_char_boundary(MAX_DEBUG_REPORT_CHARS - 1);
|
||||||
out = format!("{}…", &out[..end]);
|
out = format!("{}…", &out[..end]);
|
||||||
}
|
}
|
||||||
out
|
out
|
||||||
@@ -469,11 +536,11 @@ fn test_parse_report(
|
|||||||
|
|
||||||
#[cfg(test)]
|
#[cfg(test)]
|
||||||
mod tests {
|
mod tests {
|
||||||
use super::{MAX_TEST_REPORT_CHARS, test_parse_report};
|
use super::{MAX_DEBUG_REPORT_CHARS, debug_report};
|
||||||
use x_media::media::Media;
|
use x_media::media::Media;
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn test_parse_report_renders_fields_and_media() {
|
fn debug_report_renders_fields_and_media() {
|
||||||
let media = vec![
|
let media = vec![
|
||||||
Media::Illustration {
|
Media::Illustration {
|
||||||
title: None,
|
title: None,
|
||||||
@@ -487,7 +554,7 @@ mod tests {
|
|||||||
thumbnail_url: "https://cdn.example/2.jpg".into(),
|
thumbnail_url: "https://cdn.example/2.jpg".into(),
|
||||||
},
|
},
|
||||||
];
|
];
|
||||||
let report = test_parse_report(
|
let report = debug_report(
|
||||||
"https://x.com/u/status/1",
|
"https://x.com/u/status/1",
|
||||||
"twitter",
|
"twitter",
|
||||||
"https://x.com/u/status/1",
|
"https://x.com/u/status/1",
|
||||||
@@ -516,20 +583,20 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn test_parse_report_without_render_data_and_no_media() {
|
fn debug_report_without_render_data_and_no_media() {
|
||||||
let report = test_parse_report("u", "pixiv", "s", "t", None, true, "c", &[]);
|
let report = debug_report("u", "pixiv", "s", "t", None, true, "c", &[]);
|
||||||
assert!(!report.contains("author:"), "{report}");
|
assert!(!report.contains("author:"), "{report}");
|
||||||
assert!(report.contains("sensitive: true"), "{report}");
|
assert!(report.contains("sensitive: true"), "{report}");
|
||||||
assert!(report.contains("media (0):"), "{report}");
|
assert!(report.contains("media (0):"), "{report}");
|
||||||
}
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn test_parse_report_wraps_caption_in_blockquote() {
|
fn debug_report_wraps_caption_in_blockquote() {
|
||||||
// The report is an HTML message: raw fields are escaped, pre-escaped
|
// The report is an HTML message: raw fields are escaped, pre-escaped
|
||||||
// render fields are embedded as-is, and the caption is wrapped in a
|
// render fields are embedded as-is, and the caption is wrapped in a
|
||||||
// <blockquote> so it shows exactly as it will render in the sent
|
// <blockquote> so it shows exactly as it will render in the sent
|
||||||
// media caption (escaped text and links included).
|
// media caption (escaped text and links included).
|
||||||
let report = test_parse_report(
|
let report = debug_report(
|
||||||
"https://x.com/u/status/1",
|
"https://x.com/u/status/1",
|
||||||
"twitter",
|
"twitter",
|
||||||
"https://x.com/u/status/1",
|
"https://x.com/u/status/1",
|
||||||
@@ -563,7 +630,7 @@ mod tests {
|
|||||||
}
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn test_parse_report_is_capped() {
|
fn debug_report_is_capped() {
|
||||||
// 200 media lines ≈ 8 KB, comfortably over the cap.
|
// 200 media lines ≈ 8 KB, comfortably over the cap.
|
||||||
let media: Vec<Media> = (0..200)
|
let media: Vec<Media> = (0..200)
|
||||||
.map(|i| Media::Illustration {
|
.map(|i| Media::Illustration {
|
||||||
@@ -573,8 +640,8 @@ mod tests {
|
|||||||
fallback_url: None,
|
fallback_url: None,
|
||||||
})
|
})
|
||||||
.collect();
|
.collect();
|
||||||
let report = test_parse_report("u", "twitter", "s", "t", None, false, "c", &media);
|
let report = debug_report("u", "twitter", "s", "t", None, false, "c", &media);
|
||||||
assert!(report.chars().count() <= MAX_TEST_REPORT_CHARS, "{report}");
|
assert!(report.chars().count() <= MAX_DEBUG_REPORT_CHARS, "{report}");
|
||||||
assert!(report.ends_with('…'), "{report}");
|
assert!(report.ends_with('…'), "{report}");
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -3,6 +3,7 @@
|
|||||||
//! inline cache instead of re-fetching.
|
//! inline cache instead of re-fetching.
|
||||||
|
|
||||||
use super::log_key;
|
use super::log_key;
|
||||||
|
use std::collections::HashMap;
|
||||||
use std::sync::LazyLock;
|
use std::sync::LazyLock;
|
||||||
use teloxide::RequestError;
|
use teloxide::RequestError;
|
||||||
use teloxide::prelude::*;
|
use teloxide::prelude::*;
|
||||||
@@ -19,16 +20,65 @@ use x_media::media::Media;
|
|||||||
/// post id. Only answer once the query has been stable for this long.
|
/// post id. Only answer once the query has been stable for this long.
|
||||||
const INLINE_DEBOUNCE: std::time::Duration = std::time::Duration::from_millis(800);
|
const INLINE_DEBOUNCE: std::time::Duration = std::time::Duration::from_millis(800);
|
||||||
|
|
||||||
/// Last seen inline query and whether it was already answered. Guards the
|
/// Last seen inline query per user and whether it was already answered.
|
||||||
/// debounce timer: a repeat of an answered query is served by Telegram's
|
/// Guards the debounce timer: a repeat of an answered query is served by
|
||||||
/// inline cache (see `cache_time`), not by another fetch.
|
/// Telegram's inline cache (see `cache_time`), not by another fetch. Keyed by
|
||||||
|
/// user id — a single shared slot would let one user's typing burst (or a
|
||||||
|
/// different user's query) cancel another user's pending answer.
|
||||||
struct InlineDebounceState {
|
struct InlineDebounceState {
|
||||||
query: String,
|
query: String,
|
||||||
answered: bool,
|
answered: bool,
|
||||||
}
|
}
|
||||||
|
|
||||||
static INLINE_DEBOUNCE_STATE: LazyLock<parking_lot::Mutex<Option<InlineDebounceState>>> =
|
#[derive(Default)]
|
||||||
LazyLock::new(|| parking_lot::Mutex::new(None));
|
struct DebounceStates(HashMap<u64, InlineDebounceState>);
|
||||||
|
|
||||||
|
impl DebounceStates {
|
||||||
|
/// Records `query` as the user's newest query. Returns false when it is a
|
||||||
|
/// repeat whose answer already went out (Telegram's inline cache serves
|
||||||
|
/// it; re-fetching would only hit the source site again).
|
||||||
|
fn note(&mut self, user_id: u64, query: &str) -> bool {
|
||||||
|
if let Some(prev) = self.0.get(&user_id)
|
||||||
|
&& prev.query == query
|
||||||
|
&& prev.answered
|
||||||
|
{
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
self.0.insert(
|
||||||
|
user_id,
|
||||||
|
InlineDebounceState {
|
||||||
|
query: query.to_string(),
|
||||||
|
answered: false,
|
||||||
|
},
|
||||||
|
);
|
||||||
|
true
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Claims the answer for the user's newest query; false when a newer query
|
||||||
|
/// superseded it or the answer was already claimed.
|
||||||
|
fn claim(&mut self, user_id: u64, query: &str) -> bool {
|
||||||
|
let Some(state) = self.0.get_mut(&user_id) else {
|
||||||
|
return false;
|
||||||
|
};
|
||||||
|
if state.query != query || state.answered {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
state.answered = true;
|
||||||
|
true
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Releases a claimed-but-unsent answer so a repeat can retry the fetch.
|
||||||
|
fn release(&mut self, user_id: u64, query: &str) {
|
||||||
|
if let Some(state) = self.0.get_mut(&user_id)
|
||||||
|
&& state.query == query
|
||||||
|
{
|
||||||
|
state.answered = false;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
static INLINE_DEBOUNCE_STATE: LazyLock<parking_lot::Mutex<DebounceStates>> =
|
||||||
|
LazyLock::new(|| parking_lot::Mutex::new(DebounceStates::default()));
|
||||||
|
|
||||||
pub async fn inline_query_handler(bot: Bot, query: InlineQuery) -> Result<(), RequestError> {
|
pub async fn inline_query_handler(bot: Bot, query: InlineQuery) -> Result<(), RequestError> {
|
||||||
if query.query.is_empty() {
|
if query.query.is_empty() {
|
||||||
@@ -41,48 +91,23 @@ pub async fn inline_query_handler(bot: Bot, query: InlineQuery) -> Result<(), Re
|
|||||||
// Debounce: record the query and answer only after it has been stable for
|
// Debounce: record the query and answer only after it has been stable for
|
||||||
// INLINE_DEBOUNCE (the timer below). An already-answered repeat of the
|
// INLINE_DEBOUNCE (the timer below). An already-answered repeat of the
|
||||||
// same query is left to Telegram's inline cache instead of re-fetching.
|
// same query is left to Telegram's inline cache instead of re-fetching.
|
||||||
{
|
let user_id = query.from.id.0;
|
||||||
let mut state = INLINE_DEBOUNCE_STATE.lock();
|
if !INLINE_DEBOUNCE_STATE.lock().note(user_id, &query.query) {
|
||||||
if let Some(prev) = state.as_ref()
|
return respond(());
|
||||||
&& prev.query == query.query
|
|
||||||
&& prev.answered
|
|
||||||
{
|
|
||||||
return respond(());
|
|
||||||
}
|
|
||||||
*state = Some(InlineDebounceState {
|
|
||||||
query: query.query.clone(),
|
|
||||||
answered: false,
|
|
||||||
});
|
|
||||||
}
|
}
|
||||||
let query_text = query.query.clone();
|
let query_text = query.query.clone();
|
||||||
tokio::spawn(async move {
|
tokio::spawn(async move {
|
||||||
tokio::time::sleep(INLINE_DEBOUNCE).await;
|
tokio::time::sleep(INLINE_DEBOUNCE).await;
|
||||||
// Only the last query of a typing burst survives: earlier timers see
|
// Only the user's last query of a typing burst survives: earlier
|
||||||
// the query changed and give up without answering.
|
// timers see the query changed and give up without answering.
|
||||||
{
|
if !INLINE_DEBOUNCE_STATE.lock().claim(user_id, &query_text) {
|
||||||
let mut state = INLINE_DEBOUNCE_STATE.lock();
|
return;
|
||||||
let Some(state) = state.as_mut() else {
|
|
||||||
return;
|
|
||||||
};
|
|
||||||
if state.query != query_text || state.answered {
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
// Claim the answer so a repeat of the same query cannot start a
|
|
||||||
// second fetch; reset below when no answer was produced.
|
|
||||||
state.answered = true;
|
|
||||||
}
|
}
|
||||||
match answer_inline_query(bot, query).await {
|
match answer_inline_query(bot, query).await {
|
||||||
Ok(true) => {}
|
Ok(true) => {}
|
||||||
// No results produced (or nothing to answer): let a repeat of the
|
// No results produced (or nothing to answer): let a repeat of the
|
||||||
// same query retry the fetch.
|
// same query retry the fetch.
|
||||||
Ok(false) | Err(_) => {
|
Ok(false) | Err(_) => INLINE_DEBOUNCE_STATE.lock().release(user_id, &query_text),
|
||||||
let mut state = INLINE_DEBOUNCE_STATE.lock();
|
|
||||||
if let Some(state) = state.as_mut()
|
|
||||||
&& state.query == query_text
|
|
||||||
{
|
|
||||||
state.answered = false;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
respond(())
|
respond(())
|
||||||
@@ -96,7 +121,9 @@ async fn answer_inline_query(bot: Bot, query: InlineQuery) -> Result<bool, Reque
|
|||||||
query.query,
|
query.query,
|
||||||
log_key(&query.query)
|
log_key(&query.query)
|
||||||
);
|
);
|
||||||
match x_media::site::fetch(&query.query).await {
|
// No retries: the debounce plus a 1s/2s backoff would outlast the inline
|
||||||
|
// query the answer belongs to.
|
||||||
|
match x_media::site::fetch_once(&query.query).await {
|
||||||
Ok(Some(fetched)) => {
|
Ok(Some(fetched)) => {
|
||||||
let mut results: Vec<InlineQueryResult> = Vec::new();
|
let mut results: Vec<InlineQueryResult> = Vec::new();
|
||||||
// Inline results have the same 1024-char caption limit as regular
|
// Inline results have the same 1024-char caption limit as regular
|
||||||
@@ -159,3 +186,49 @@ async fn answer_inline_query(bot: Bot, query: InlineQuery) -> Result<bool, Reque
|
|||||||
}
|
}
|
||||||
Ok(false)
|
Ok(false)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[cfg(test)]
|
||||||
|
mod tests {
|
||||||
|
use super::DebounceStates;
|
||||||
|
|
||||||
|
const URL_A: &str = "https://x.com/a/status/1";
|
||||||
|
const URL_B: &str = "https://x.com/b/status/2";
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn debounce_state_is_per_user() {
|
||||||
|
let mut states = DebounceStates::default();
|
||||||
|
// Two users query different links: both proceed, and neither timer
|
||||||
|
// cancels the other (a single shared slot dropped one of them).
|
||||||
|
assert!(states.note(1, URL_A));
|
||||||
|
assert!(states.note(2, URL_B));
|
||||||
|
assert!(states.claim(1, URL_A), "user 1's answer was cancelled");
|
||||||
|
assert!(states.claim(2, URL_B), "user 2's answer was cancelled");
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn answered_query_is_suppressed_per_user_only() {
|
||||||
|
let mut states = DebounceStates::default();
|
||||||
|
assert!(states.note(1, URL_A));
|
||||||
|
assert!(states.claim(1, URL_A));
|
||||||
|
// A repeat of the answered query by the same user is left to
|
||||||
|
// Telegram's inline cache.
|
||||||
|
assert!(!states.note(1, URL_A));
|
||||||
|
// Another user pasting the same link still gets an answer.
|
||||||
|
assert!(states.note(2, URL_A));
|
||||||
|
assert!(states.claim(2, URL_A));
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn newer_query_supersedes_and_failed_answer_is_released() {
|
||||||
|
let mut states = DebounceStates::default();
|
||||||
|
assert!(states.note(1, URL_A));
|
||||||
|
assert!(states.note(1, URL_B));
|
||||||
|
// The stale timer for the half-typed query gives up…
|
||||||
|
assert!(!states.claim(1, URL_A));
|
||||||
|
// …and the newest one answers.
|
||||||
|
assert!(states.claim(1, URL_B));
|
||||||
|
// No results → release so a repeat may retry the fetch.
|
||||||
|
states.release(1, URL_B);
|
||||||
|
assert!(states.claim(1, URL_B));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
@@ -18,6 +18,7 @@ pub use inline::inline_query_handler;
|
|||||||
pub use statics::{CHAT_STORE, CONFIG, LINK_CACHE, TASK_QUEUE};
|
pub use statics::{CHAT_STORE, CONFIG, LINK_CACHE, TASK_QUEUE};
|
||||||
pub use urls::{start_url_workers, stop_url_workers};
|
pub use urls::{start_url_workers, stop_url_workers};
|
||||||
|
|
||||||
|
use crate::ctx::AppContext;
|
||||||
use crate::media_sender::MediaSender;
|
use crate::media_sender::MediaSender;
|
||||||
use commands::{Command, execute_command};
|
use commands::{Command, execute_command};
|
||||||
use teloxide::RequestError;
|
use teloxide::RequestError;
|
||||||
@@ -27,16 +28,13 @@ use teloxide::utils::command::BotCommands;
|
|||||||
use urls::{URL_JOBS, extract_urls};
|
use urls::{URL_JOBS, extract_urls};
|
||||||
|
|
||||||
/// Reply to a message by id, keeping the reply decoration even if the
|
/// Reply to a message by id, keeping the reply decoration even if the
|
||||||
/// original was already deleted.
|
/// original was already deleted. Returns the reply's message id.
|
||||||
pub(crate) async fn reply<T>(
|
pub(crate) async fn reply(
|
||||||
sender: &dyn MediaSender,
|
sender: &dyn MediaSender,
|
||||||
chat_id: i64,
|
chat_id: i64,
|
||||||
reply_to: MessageId,
|
reply_to: MessageId,
|
||||||
text: T,
|
text: impl Into<String>,
|
||||||
) -> Result<Message, RequestError>
|
) -> Result<i64, RequestError> {
|
||||||
where
|
|
||||||
T: Into<String>,
|
|
||||||
{
|
|
||||||
sender
|
sender
|
||||||
.send_message(ChatId(chat_id), text.into(), Some(reply_to), None)
|
.send_message(ChatId(chat_id), text.into(), Some(reply_to), None)
|
||||||
.await
|
.await
|
||||||
@@ -50,13 +48,14 @@ pub(crate) async fn reply_html(
|
|||||||
chat_id: i64,
|
chat_id: i64,
|
||||||
reply_to: MessageId,
|
reply_to: MessageId,
|
||||||
text: String,
|
text: String,
|
||||||
) -> Result<Message, RequestError> {
|
) -> Result<i64, RequestError> {
|
||||||
// `<Bot as Requester>::` disambiguates from the MediaSender trait's
|
// `<Bot as Requester>::` disambiguates from the MediaSender trait's
|
||||||
// same-named method (see media_sender.rs).
|
// same-named method (see media_sender.rs).
|
||||||
<Bot as Requester>::send_message(bot, ChatId(chat_id), text)
|
<Bot as Requester>::send_message(bot, ChatId(chat_id), text)
|
||||||
.parse_mode(ParseMode::Html)
|
.parse_mode(ParseMode::Html)
|
||||||
.reply_parameters(ReplyParameters::new(reply_to).allow_sending_without_reply())
|
.reply_parameters(ReplyParameters::new(reply_to).allow_sending_without_reply())
|
||||||
.await
|
.await
|
||||||
|
.map(|message| message.id.0 as i64)
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Log prefix tying the whole lifecycle of one link (fetch → send → cache →
|
/// Log prefix tying the whole lifecycle of one link (fetch → send → cache →
|
||||||
@@ -69,16 +68,16 @@ pub fn log_key(url: &str) -> String {
|
|||||||
|
|
||||||
/// Edit-before-forward: a reply to the prompt swaps the caption of the first
|
/// Edit-before-forward: a reply to the prompt swaps the caption of the first
|
||||||
/// forwarded message. Returns true when the message was consumed as an edit.
|
/// forwarded message. Returns true when the message was consumed as an edit.
|
||||||
async fn edit_message_handler(bot: &Bot, message: &Message) -> bool {
|
/// Body of [`message_handler`]'s edit branch, without teloxide update types so
|
||||||
let Some(reply) = message.reply_to_message() else {
|
/// it can be driven by tests.
|
||||||
return false;
|
async fn edit_message_handler(
|
||||||
};
|
ctx: &AppContext<'_>,
|
||||||
let chat_id = message.chat.id.0;
|
chat_id: i64,
|
||||||
let Some(text) = message.text() else {
|
reply_to_message_id: i64,
|
||||||
return false;
|
text: &str,
|
||||||
};
|
) -> bool {
|
||||||
let chat_data = CHAT_STORE.get(chat_id).await;
|
let chat_data = ctx.chat_store.get(chat_id).await;
|
||||||
let Some(edit) = chat_data.edit_message.get(&(reply.id.0 as i64)) else {
|
let Some(edit) = chat_data.edit_message.get(&reply_to_message_id) else {
|
||||||
return false;
|
return false;
|
||||||
};
|
};
|
||||||
let Some(first_forward_id) = edit.forward_message_ids.first() else {
|
let Some(first_forward_id) = edit.forward_message_ids.first() else {
|
||||||
@@ -98,15 +97,17 @@ async fn edit_message_handler(bot: &Bot, message: &Message) -> bool {
|
|||||||
.map(|template| template.replace("[]", &link))
|
.map(|template| template.replace("[]", &link))
|
||||||
.unwrap_or(link)
|
.unwrap_or(link)
|
||||||
};
|
};
|
||||||
let result = bot
|
match ctx
|
||||||
.edit_message_caption(ChatId(chat_id), MessageId(*first_forward_id as i32))
|
.sender
|
||||||
.caption(new_text)
|
.edit_message_caption(
|
||||||
.parse_mode(ParseMode::Html)
|
ChatId(chat_id),
|
||||||
.await;
|
MessageId(*first_forward_id as i32),
|
||||||
match result {
|
new_text,
|
||||||
Ok(_) => log::info!(
|
)
|
||||||
"edit-before-forward: caption swapped on message {first_forward_id} for prompt {}",
|
.await
|
||||||
reply.id.0
|
{
|
||||||
|
Ok(()) => log::info!(
|
||||||
|
"edit-before-forward: caption swapped on message {first_forward_id} for prompt {reply_to_message_id}"
|
||||||
),
|
),
|
||||||
Err(e) => log::error!("edit_message_caption failed: {e}"),
|
Err(e) => log::error!("edit_message_caption failed: {e}"),
|
||||||
}
|
}
|
||||||
@@ -133,7 +134,17 @@ pub async fn message_handler(bot: Bot, message: Message) -> Result<(), RequestEr
|
|||||||
message.chat.id
|
message.chat.id
|
||||||
);
|
);
|
||||||
// URL/edit flows only run in private chats; commands run in any chat.
|
// URL/edit flows only run in private chats; commands run in any chat.
|
||||||
if is_private && edit_message_handler(&bot, &message).await {
|
if is_private
|
||||||
|
&& let Some(reply) = message.reply_to_message()
|
||||||
|
&& let Some(text) = message.text()
|
||||||
|
&& edit_message_handler(
|
||||||
|
&AppContext::from_statics(&bot),
|
||||||
|
message.chat.id.0,
|
||||||
|
reply.id.0 as i64,
|
||||||
|
text,
|
||||||
|
)
|
||||||
|
.await
|
||||||
|
{
|
||||||
return respond(());
|
return respond(());
|
||||||
}
|
}
|
||||||
if let Some(text) = message.text()
|
if let Some(text) = message.text()
|
||||||
@@ -157,8 +168,107 @@ pub async fn message_handler(bot: Bot, message: Message) -> Result<(), RequestEr
|
|||||||
log::warn!("url workers not started; dropping link");
|
log::warn!("url workers not started; dropping link");
|
||||||
break;
|
break;
|
||||||
};
|
};
|
||||||
let _ = tx.send((message.clone(), url)).await;
|
// A closed channel means the workers are stopping (shutdown):
|
||||||
|
// report the dropped link instead of losing it silently.
|
||||||
|
if tx.send((message.clone(), url)).await.is_err() {
|
||||||
|
log::warn!("url workers stopped; dropping link");
|
||||||
|
break;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
respond(())
|
respond(())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[cfg(test)]
|
||||||
|
mod tests {
|
||||||
|
use super::*;
|
||||||
|
use crate::ctx::test_support::TestStores;
|
||||||
|
use crate::media_sender::test_support::{MockSender, Outcome};
|
||||||
|
use crate::state::EditMessage;
|
||||||
|
use teloxide::ApiError;
|
||||||
|
|
||||||
|
const PROMPT_ID: i64 = 7;
|
||||||
|
const FORWARDED_ID: i64 = 9;
|
||||||
|
|
||||||
|
fn api_error() -> RequestError {
|
||||||
|
RequestError::Api(ApiError::Unknown("Bad Request: message not found".into()))
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Seeds a prompt record; `template` names the chat template used for it
|
||||||
|
/// (empty = none, the caption gets the bare link).
|
||||||
|
async fn seed_prompt(ctx: &AppContext<'_>, template: &str) {
|
||||||
|
ctx.chat_store
|
||||||
|
.update(1, |data| {
|
||||||
|
data.template
|
||||||
|
.insert("tpl".to_string(), "<b>[]</b>".to_string());
|
||||||
|
data.edit_message.insert(
|
||||||
|
PROMPT_ID,
|
||||||
|
EditMessage {
|
||||||
|
url: "https://x.com/u/status/1".into(),
|
||||||
|
chat_id: 1,
|
||||||
|
forward_message_ids: vec![FORWARDED_ID],
|
||||||
|
template: template.to_string(),
|
||||||
|
created_at: crate::db::unix_now(),
|
||||||
|
},
|
||||||
|
);
|
||||||
|
})
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn reply_to_a_prompt_swaps_the_caption_through_its_template() {
|
||||||
|
let sender = MockSender::scripted(vec![Outcome::EditOk], api_error);
|
||||||
|
let stores = TestStores::new();
|
||||||
|
let ctx = stores.ctx(&sender);
|
||||||
|
seed_prompt(&ctx, "tpl").await;
|
||||||
|
|
||||||
|
let consumed = edit_message_handler(&ctx, 1, PROMPT_ID, "new caption").await;
|
||||||
|
|
||||||
|
assert!(consumed, "a reply to the prompt must be consumed");
|
||||||
|
assert_eq!(
|
||||||
|
sender.captions(),
|
||||||
|
vec!["<b><a href=\"https://x.com/u/status/1\">new caption</a></b>"]
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn reply_text_and_url_are_escaped_into_the_caption() {
|
||||||
|
let sender = MockSender::scripted(vec![Outcome::EditOk], api_error);
|
||||||
|
let stores = TestStores::new();
|
||||||
|
let ctx = stores.ctx(&sender);
|
||||||
|
seed_prompt(&ctx, "").await;
|
||||||
|
|
||||||
|
edit_message_handler(&ctx, 1, PROMPT_ID, "<script>alert(1)</script>").await;
|
||||||
|
|
||||||
|
// No raw markup from user text may reach the HTML caption.
|
||||||
|
assert_eq!(
|
||||||
|
sender.captions(),
|
||||||
|
vec!["<a href=\"https://x.com/u/status/1\"><script>alert(1)</script></a>"]
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn a_failed_caption_swap_still_consumes_the_reply() {
|
||||||
|
let sender = MockSender::scripted(vec![Outcome::EditErr], api_error);
|
||||||
|
let stores = TestStores::new();
|
||||||
|
let ctx = stores.ctx(&sender);
|
||||||
|
seed_prompt(&ctx, "tpl").await;
|
||||||
|
|
||||||
|
// The edit failed (message deleted etc.); the reply must still be
|
||||||
|
// swallowed instead of being treated as a link to fetch.
|
||||||
|
assert!(edit_message_handler(&ctx, 1, PROMPT_ID, "new caption").await);
|
||||||
|
assert_eq!(sender.calls(), vec!["edit_message_caption"]);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn reply_to_an_unrelated_message_is_not_consumed() {
|
||||||
|
let sender = MockSender::scripted(vec![], api_error);
|
||||||
|
let stores = TestStores::new();
|
||||||
|
let ctx = stores.ctx(&sender);
|
||||||
|
|
||||||
|
// No prompt record for that message id → the reply runs the normal
|
||||||
|
// (URL/command) path instead.
|
||||||
|
assert!(!edit_message_handler(&ctx, 1, PROMPT_ID, "hello").await);
|
||||||
|
assert!(sender.calls().is_empty());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
@@ -1,14 +1,11 @@
|
|||||||
//! URL extraction and the per-URL media pipeline: bounded job channel +
|
//! URL extraction and the per-URL media pipeline: bounded job channel +
|
||||||
//! worker pool, link-cache fast path, fetch, task build and send dispatch.
|
//! worker pool, link-cache fast path, fetch, task build and send dispatch.
|
||||||
|
|
||||||
use super::{CHAT_STORE, CONFIG, LINK_CACHE, TASK_QUEUE, log_key, reply};
|
use super::{log_key, reply};
|
||||||
use crate::config::Config;
|
use crate::ctx::{AppContext, CONTEXT};
|
||||||
use crate::db::now_f64;
|
use crate::link_cache::{CachedMediaKind, CachedPost};
|
||||||
use crate::link_cache::{CachedMediaKind, CachedPost, LinkCache};
|
|
||||||
use crate::media_sender::MediaSender;
|
|
||||||
use crate::queue::PersistentTaskQueue;
|
|
||||||
use crate::send::{self, MediaItemPayload, Task};
|
use crate::send::{self, MediaItemPayload, Task};
|
||||||
use crate::state::{ChatData, ChatStore};
|
use crate::state::ChatData;
|
||||||
use std::collections::HashSet;
|
use std::collections::HashSet;
|
||||||
use std::sync::LazyLock;
|
use std::sync::LazyLock;
|
||||||
use teloxide::types::{ChatAction, ChatId, Message, MessageEntityKind, MessageId};
|
use teloxide::types::{ChatAction, ChatId, Message, MessageEntityKind, MessageId};
|
||||||
@@ -35,27 +32,6 @@ static URL_WORKER_HANDLES: LazyLock<parking_lot::Mutex<Option<Vec<tokio::task::J
|
|||||||
/// while bounding how many jobs can be queued at all.
|
/// while bounding how many jobs can be queued at all.
|
||||||
const URL_WORKERS: usize = 8;
|
const URL_WORKERS: usize = 8;
|
||||||
|
|
||||||
/// Dependencies of the per-URL pipeline, injected so tests can substitute a
|
|
||||||
/// mock sender and tempdir-backed stores.
|
|
||||||
pub(crate) struct AppContext<'a> {
|
|
||||||
pub sender: &'a dyn MediaSender,
|
|
||||||
pub chat_store: &'a ChatStore,
|
|
||||||
pub task_queue: &'a PersistentTaskQueue,
|
|
||||||
pub link_cache: &'a LinkCache,
|
|
||||||
pub config: &'a Config,
|
|
||||||
}
|
|
||||||
|
|
||||||
/// Assembles the production context from the process-wide statics.
|
|
||||||
fn app_context() -> AppContext<'static> {
|
|
||||||
AppContext {
|
|
||||||
sender: &*crate::send::BOT,
|
|
||||||
chat_store: &CHAT_STORE,
|
|
||||||
task_queue: &TASK_QUEUE,
|
|
||||||
link_cache: &LINK_CACHE,
|
|
||||||
config: &CONFIG,
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
/// Starts the URL job workers (called once from main after the queue starts).
|
/// Starts the URL job workers (called once from main after the queue starts).
|
||||||
/// teloxide dispatches updates to a per-chat worker that handles them
|
/// teloxide dispatches updates to a per-chat worker that handles them
|
||||||
/// sequentially, so a batch-forward of many messages would otherwise be
|
/// sequentially, so a batch-forward of many messages would otherwise be
|
||||||
@@ -70,12 +46,18 @@ pub async fn start_url_workers() {
|
|||||||
for _ in 0..URL_WORKERS {
|
for _ in 0..URL_WORKERS {
|
||||||
let rx = std::sync::Arc::clone(&rx);
|
let rx = std::sync::Arc::clone(&rx);
|
||||||
handles.push(tokio::spawn(async move {
|
handles.push(tokio::spawn(async move {
|
||||||
let ctx = app_context();
|
|
||||||
while !URL_STOP.load(std::sync::atomic::Ordering::Relaxed) {
|
while !URL_STOP.load(std::sync::atomic::Ordering::Relaxed) {
|
||||||
let job = rx.lock().await.recv().await;
|
let job = rx.lock().await.recv().await;
|
||||||
match job {
|
match job {
|
||||||
Some((message, url)) => {
|
Some((message, url)) => {
|
||||||
url_media(&ctx, message.chat.id.0, message.id.0 as i64, &url).await
|
url_media(
|
||||||
|
&CONTEXT,
|
||||||
|
message.chat.id.0,
|
||||||
|
message.id.0 as i64,
|
||||||
|
&url,
|
||||||
|
PostSend::FromChat,
|
||||||
|
)
|
||||||
|
.await
|
||||||
}
|
}
|
||||||
None => break,
|
None => break,
|
||||||
}
|
}
|
||||||
@@ -135,7 +117,12 @@ pub fn extract_urls(message: &Message) -> Vec<String> {
|
|||||||
fn thumbnail_for(media: &Media) -> Option<String> {
|
fn thumbnail_for(media: &Media) -> Option<String> {
|
||||||
let url = media.url();
|
let url = media.url();
|
||||||
if url.starts_with("http://") || url.starts_with("https://") {
|
if url.starts_with("http://") || url.starts_with("https://") {
|
||||||
media.thumbnail_url().map(str::to_string)
|
// An empty thumbnail string (misskey video/gif files without a
|
||||||
|
// thumbnailUrl) must not reach Telegram; let it generate its own.
|
||||||
|
media
|
||||||
|
.thumbnail_url()
|
||||||
|
.map(str::to_string)
|
||||||
|
.filter(|t| !t.is_empty())
|
||||||
} else {
|
} else {
|
||||||
None
|
None
|
||||||
}
|
}
|
||||||
@@ -169,14 +156,6 @@ fn media_to_payload(media: &Media, sensitive: bool) -> MediaItemPayload {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
pub(crate) async fn enqueue_retry(queue: &PersistentTaskQueue, task: Task, delay_seconds: f64) {
|
|
||||||
let payload = serde_json::to_value(task).expect("task serializes");
|
|
||||||
let run_after = now_f64() + delay_seconds;
|
|
||||||
if let Err(e) = queue.enqueue(payload, run_after).await {
|
|
||||||
log::error!("failed to enqueue retry: {e}");
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
/// Sends a task and handles the outcome: post-send actions on success, retry
|
/// Sends a task and handles the outcome: post-send actions on success, retry
|
||||||
/// enqueue on retryable failure, reply + link-cache invalidation on
|
/// enqueue on retryable failure, reply + link-cache invalidation on
|
||||||
/// permanent failure (a stale cached file id must not repeat forever).
|
/// permanent failure (a stale cached file id must not repeat forever).
|
||||||
@@ -188,8 +167,8 @@ async fn dispatch_send(
|
|||||||
url: &str,
|
url: &str,
|
||||||
) {
|
) {
|
||||||
let result = match task {
|
let result = match task {
|
||||||
Task::SendAnimation { .. } => send::send_animation(ctx.sender, task).await,
|
Task::SendAnimation { .. } => send::send_animation(ctx, task).await,
|
||||||
Task::SendMediaSequence { .. } => send::send_media_sequence(ctx.sender, task).await,
|
Task::SendMediaSequence { .. } => send::send_media_sequence(ctx, task).await,
|
||||||
Task::ForwardMessages { .. } => unreachable!(),
|
Task::ForwardMessages { .. } => unreachable!(),
|
||||||
};
|
};
|
||||||
match result {
|
match result {
|
||||||
@@ -199,9 +178,8 @@ async fn dispatch_send(
|
|||||||
message_ids.len(),
|
message_ids.len(),
|
||||||
log_key(url)
|
log_key(url)
|
||||||
);
|
);
|
||||||
send::post_send_actions(ctx.sender, task, message_ids).await;
|
send::post_send_actions(ctx, task, message_ids).await;
|
||||||
// The task settled: drop any keep-alive temp media.
|
send::settle_task(ctx, task, send::Settled::Sent).await;
|
||||||
send::release_keep_alive(task);
|
|
||||||
}
|
}
|
||||||
Err(send::SendError::Retryable {
|
Err(send::SendError::Retryable {
|
||||||
delay_seconds,
|
delay_seconds,
|
||||||
@@ -211,7 +189,7 @@ async fn dispatch_send(
|
|||||||
"send for [key={}] failed, queued for retry in {delay_seconds:.1}s",
|
"send for [key={}] failed, queued for retry in {delay_seconds:.1}s",
|
||||||
log_key(url)
|
log_key(url)
|
||||||
);
|
);
|
||||||
enqueue_retry(ctx.task_queue, task, delay_seconds).await;
|
send::enqueue_retry(ctx.task_queue, *task, delay_seconds).await;
|
||||||
let _ = reply(
|
let _ = reply(
|
||||||
ctx.sender,
|
ctx.sender,
|
||||||
chat_id,
|
chat_id,
|
||||||
@@ -224,8 +202,7 @@ async fn dispatch_send(
|
|||||||
message: err_message,
|
message: err_message,
|
||||||
task,
|
task,
|
||||||
}) => {
|
}) => {
|
||||||
send::invalidate_cache_with(ctx.link_cache, &task).await;
|
send::settle_task(ctx, &task, send::Settled::Failed).await;
|
||||||
send::release_keep_alive(&task);
|
|
||||||
log::error!("send for {url} failed permanently: {err_message}");
|
log::error!("send for {url} failed permanently: {err_message}");
|
||||||
let _ = reply(
|
let _ = reply(
|
||||||
ctx.sender,
|
ctx.sender,
|
||||||
@@ -238,8 +215,20 @@ async fn dispatch_send(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// Whether a send also runs the chat's post-send actions. `/test` sends with
|
||||||
|
/// them suppressed so a test can never forward to the channel or open the
|
||||||
|
/// edit-before-forward prompt; a normal link uses whatever the chat is
|
||||||
|
/// configured with.
|
||||||
|
#[derive(Clone, Copy, PartialEq, Eq, Debug)]
|
||||||
|
pub(crate) enum PostSend {
|
||||||
|
/// Apply the chat's `forward_channel_id` / `edit_before_forward`.
|
||||||
|
FromChat,
|
||||||
|
/// Send only: no channel forward, no edit prompt.
|
||||||
|
Suppressed,
|
||||||
|
}
|
||||||
|
|
||||||
/// Builds the send task from ready-made items, sharing the payload shape
|
/// Builds the send task from ready-made items, sharing the payload shape
|
||||||
/// between the fresh-fetch and link-cache paths.
|
/// between the fresh-fetch, link-cache and `/test` paths.
|
||||||
#[allow(clippy::too_many_arguments)]
|
#[allow(clippy::too_many_arguments)]
|
||||||
fn build_send_task(
|
fn build_send_task(
|
||||||
chat_data: &ChatData,
|
chat_data: &ChatData,
|
||||||
@@ -249,7 +238,14 @@ fn build_send_task(
|
|||||||
caption: String,
|
caption: String,
|
||||||
items: Vec<MediaItemPayload>,
|
items: Vec<MediaItemPayload>,
|
||||||
cache_data: Option<CachedPost>,
|
cache_data: Option<CachedPost>,
|
||||||
|
post_send: PostSend,
|
||||||
) -> Task {
|
) -> Task {
|
||||||
|
// Notification ids stay set in both modes: a queued retry that
|
||||||
|
// dead-letters should still tell the chat.
|
||||||
|
let (edit_before_forward, forward_channel_id) = match post_send {
|
||||||
|
PostSend::FromChat => (chat_data.edit_before_forward, chat_data.forward_channel_id),
|
||||||
|
PostSend::Suppressed => (false, None),
|
||||||
|
};
|
||||||
if items.len() == 1 && matches!(items[0], MediaItemPayload::Animation { .. }) {
|
if items.len() == 1 && matches!(items[0], MediaItemPayload::Animation { .. }) {
|
||||||
Task::SendAnimation {
|
Task::SendAnimation {
|
||||||
chat_id,
|
chat_id,
|
||||||
@@ -257,8 +253,8 @@ fn build_send_task(
|
|||||||
caption,
|
caption,
|
||||||
animation: items.into_iter().next().unwrap(),
|
animation: items.into_iter().next().unwrap(),
|
||||||
source_url,
|
source_url,
|
||||||
edit_before_forward: chat_data.edit_before_forward,
|
edit_before_forward,
|
||||||
forward_channel_id: chat_data.forward_channel_id,
|
forward_channel_id,
|
||||||
notify_chat_id: Some(chat_id),
|
notify_chat_id: Some(chat_id),
|
||||||
notify_message_id: Some(reply_to_message_id),
|
notify_message_id: Some(reply_to_message_id),
|
||||||
cache_data,
|
cache_data,
|
||||||
@@ -274,8 +270,8 @@ fn build_send_task(
|
|||||||
batch_index: 0,
|
batch_index: 0,
|
||||||
sent_message_ids: vec![],
|
sent_message_ids: vec![],
|
||||||
source_url,
|
source_url,
|
||||||
edit_before_forward: chat_data.edit_before_forward,
|
edit_before_forward,
|
||||||
forward_channel_id: chat_data.forward_channel_id,
|
forward_channel_id,
|
||||||
notify_chat_id: Some(chat_id),
|
notify_chat_id: Some(chat_id),
|
||||||
notify_message_id: Some(reply_to_message_id),
|
notify_message_id: Some(reply_to_message_id),
|
||||||
cache_data,
|
cache_data,
|
||||||
@@ -283,7 +279,19 @@ fn build_send_task(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
async fn url_media(ctx: &AppContext<'_>, chat_id: i64, reply_to_message_id: i64, url: &str) {
|
/// The per-URL pipeline: link cache → fetch → build → send → post-send.
|
||||||
|
///
|
||||||
|
/// `post_send` selects whether the chat's forward/edit settings apply: the URL
|
||||||
|
/// workers pass [`PostSend::FromChat`], the `/test` command
|
||||||
|
/// [`PostSend::Suppressed`]. Everything else (cache write, retry enqueue,
|
||||||
|
/// dead-letter notification) is identical.
|
||||||
|
pub(crate) async fn url_media(
|
||||||
|
ctx: &AppContext<'_>,
|
||||||
|
chat_id: i64,
|
||||||
|
reply_to_message_id: i64,
|
||||||
|
url: &str,
|
||||||
|
post_send: PostSend,
|
||||||
|
) {
|
||||||
let reply_to = MessageId(reply_to_message_id as i32);
|
let reply_to = MessageId(reply_to_message_id as i32);
|
||||||
if let Err(e) = ctx
|
if let Err(e) = ctx
|
||||||
.sender
|
.sender
|
||||||
@@ -354,6 +362,7 @@ async fn url_media(ctx: &AppContext<'_>, chat_id: i64, reply_to_message_id: i64,
|
|||||||
caption,
|
caption,
|
||||||
items,
|
items,
|
||||||
Some(cached),
|
Some(cached),
|
||||||
|
post_send,
|
||||||
);
|
);
|
||||||
dispatch_send(ctx, chat_id, reply_to, &task, url).await;
|
dispatch_send(ctx, chat_id, reply_to, &task, url).await;
|
||||||
return;
|
return;
|
||||||
@@ -422,6 +431,7 @@ async fn url_media(ctx: &AppContext<'_>, chat_id: i64, reply_to_message_id: i64,
|
|||||||
caption,
|
caption,
|
||||||
items,
|
items,
|
||||||
cache_data,
|
cache_data,
|
||||||
|
post_send,
|
||||||
);
|
);
|
||||||
// Hand the keep-alive temp dir (ugoira / bsky remux MP4) to the
|
// Hand the keep-alive temp dir (ugoira / bsky remux MP4) to the
|
||||||
// retry registry: a queued retry runs after this function returns
|
// retry registry: a queued retry runs after this function returns
|
||||||
@@ -438,10 +448,9 @@ async fn url_media(ctx: &AppContext<'_>, chat_id: i64, reply_to_message_id: i64,
|
|||||||
#[cfg(test)]
|
#[cfg(test)]
|
||||||
mod tests {
|
mod tests {
|
||||||
use super::*;
|
use super::*;
|
||||||
use crate::db;
|
use crate::ctx::test_support::TestStores;
|
||||||
use crate::link_cache::CachedMedia;
|
use crate::link_cache::CachedMedia;
|
||||||
use crate::media_sender::test_support::{MockSender, Outcome};
|
use crate::media_sender::test_support::{MockSender, Outcome};
|
||||||
use std::sync::Arc;
|
|
||||||
use std::time::Duration;
|
use std::time::Duration;
|
||||||
use teloxide::{ApiError, RequestError};
|
use teloxide::{ApiError, RequestError};
|
||||||
|
|
||||||
@@ -469,26 +478,18 @@ mod tests {
|
|||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
async fn cache_hit_sends_file_ids_and_invalidates_on_permanent_failure() {
|
async fn cache_hit_sends_file_ids_and_invalidates_on_permanent_failure() {
|
||||||
let dir = tempfile::tempdir().unwrap();
|
let stores = TestStores::new();
|
||||||
let pool = db::open_store(dir.path().join("t.db").to_str().unwrap()).unwrap();
|
|
||||||
let chat_store = ChatStore::new(Arc::clone(&pool));
|
|
||||||
let task_queue = PersistentTaskQueue::new(Arc::clone(&pool));
|
|
||||||
let link_cache = LinkCache::new(Arc::clone(&pool));
|
|
||||||
let config = Config::load();
|
|
||||||
let sender = MockSender::scripted(
|
let sender = MockSender::scripted(
|
||||||
vec![Outcome::GroupErr, Outcome::MessageErr],
|
vec![Outcome::GroupErr, Outcome::MessageErr],
|
||||||
permanent_error,
|
permanent_error,
|
||||||
);
|
);
|
||||||
let ctx = AppContext {
|
let ctx = stores.ctx(&sender);
|
||||||
sender: &sender,
|
stores
|
||||||
chat_store: &chat_store,
|
.link_cache()
|
||||||
task_queue: &task_queue,
|
.put("twitter:1", &cached_photo_entry())
|
||||||
link_cache: &link_cache,
|
.await;
|
||||||
config: &config,
|
|
||||||
};
|
|
||||||
link_cache.put("twitter:1", &cached_photo_entry()).await;
|
|
||||||
|
|
||||||
url_media(&ctx, 1, 2, "https://x.com/u/status/1").await;
|
url_media(&ctx, 1, 2, "https://x.com/u/status/1", PostSend::FromChat).await;
|
||||||
|
|
||||||
// The cached file id went out as a group send; the permanent failure
|
// The cached file id went out as a group send; the permanent failure
|
||||||
// then triggered the fire-and-forget reply (its mock error is fine).
|
// then triggered the fire-and-forget reply (its mock error is fine).
|
||||||
@@ -498,7 +499,8 @@ mod tests {
|
|||||||
);
|
);
|
||||||
// The stale cache entry was invalidated so the next request re-fetches.
|
// The stale cache entry was invalidated so the next request re-fetches.
|
||||||
assert!(
|
assert!(
|
||||||
link_cache
|
stores
|
||||||
|
.link_cache()
|
||||||
.get("twitter:1", Duration::from_secs(3600))
|
.get("twitter:1", Duration::from_secs(3600))
|
||||||
.await
|
.await
|
||||||
.is_none()
|
.is_none()
|
||||||
@@ -507,28 +509,21 @@ mod tests {
|
|||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
async fn cache_hit_success_keeps_the_cache_entry() {
|
async fn cache_hit_success_keeps_the_cache_entry() {
|
||||||
let dir = tempfile::tempdir().unwrap();
|
let stores = TestStores::new();
|
||||||
let pool = db::open_store(dir.path().join("t.db").to_str().unwrap()).unwrap();
|
|
||||||
let chat_store = ChatStore::new(Arc::clone(&pool));
|
|
||||||
let task_queue = PersistentTaskQueue::new(Arc::clone(&pool));
|
|
||||||
let link_cache = LinkCache::new(Arc::clone(&pool));
|
|
||||||
let config = Config::load();
|
|
||||||
let sender = MockSender::scripted(vec![Outcome::GroupOk], permanent_error);
|
let sender = MockSender::scripted(vec![Outcome::GroupOk], permanent_error);
|
||||||
let ctx = AppContext {
|
let ctx = stores.ctx(&sender);
|
||||||
sender: &sender,
|
stores
|
||||||
chat_store: &chat_store,
|
.link_cache()
|
||||||
task_queue: &task_queue,
|
.put("twitter:1", &cached_photo_entry())
|
||||||
link_cache: &link_cache,
|
.await;
|
||||||
config: &config,
|
|
||||||
};
|
|
||||||
link_cache.put("twitter:1", &cached_photo_entry()).await;
|
|
||||||
|
|
||||||
url_media(&ctx, 1, 2, "https://x.com/u/status/1").await;
|
url_media(&ctx, 1, 2, "https://x.com/u/status/1", PostSend::FromChat).await;
|
||||||
|
|
||||||
assert_eq!(sender.calls(), vec!["send_chat_action", "send_media_group"]);
|
assert_eq!(sender.calls(), vec!["send_chat_action", "send_media_group"]);
|
||||||
// Success must not evict the entry.
|
// Success must not evict the entry.
|
||||||
assert!(
|
assert!(
|
||||||
link_cache
|
stores
|
||||||
|
.link_cache()
|
||||||
.get("twitter:1", Duration::from_secs(3600))
|
.get("twitter:1", Duration::from_secs(3600))
|
||||||
.await
|
.await
|
||||||
.is_some()
|
.is_some()
|
||||||
@@ -537,24 +532,134 @@ mod tests {
|
|||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
async fn unsupported_url_is_ignored_silently() {
|
async fn unsupported_url_is_ignored_silently() {
|
||||||
let dir = tempfile::tempdir().unwrap();
|
let stores = TestStores::new();
|
||||||
let pool = db::open_store(dir.path().join("t.db").to_str().unwrap()).unwrap();
|
|
||||||
let chat_store = ChatStore::new(Arc::clone(&pool));
|
|
||||||
let task_queue = PersistentTaskQueue::new(Arc::clone(&pool));
|
|
||||||
let link_cache = LinkCache::new(Arc::clone(&pool));
|
|
||||||
let config = Config::load();
|
|
||||||
let sender = MockSender::scripted(vec![], permanent_error);
|
let sender = MockSender::scripted(vec![], permanent_error);
|
||||||
let ctx = AppContext {
|
let ctx = stores.ctx(&sender);
|
||||||
sender: &sender,
|
|
||||||
chat_store: &chat_store,
|
|
||||||
task_queue: &task_queue,
|
|
||||||
link_cache: &link_cache,
|
|
||||||
config: &config,
|
|
||||||
};
|
|
||||||
|
|
||||||
// No cache key → the fetch dispatcher returns Ok(None) without any
|
// No cache key → the fetch dispatcher returns Ok(None) without any
|
||||||
// network; nothing is sent or replied.
|
// network; nothing is sent or replied.
|
||||||
url_media(&ctx, 1, 2, "https://example.com/not-a-post").await;
|
url_media(
|
||||||
|
&ctx,
|
||||||
|
1,
|
||||||
|
2,
|
||||||
|
"https://example.com/not-a-post",
|
||||||
|
PostSend::FromChat,
|
||||||
|
)
|
||||||
|
.await;
|
||||||
assert_eq!(sender.calls(), vec!["send_chat_action"]);
|
assert_eq!(sender.calls(), vec!["send_chat_action"]);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// ── Send modes: the URL flow vs `/test` ─────────────────────────────
|
||||||
|
|
||||||
|
/// A chat that has both post-send actions configured.
|
||||||
|
async fn seed_post_send_settings(ctx: &AppContext<'_>) {
|
||||||
|
ctx.chat_store
|
||||||
|
.update(1, |data| {
|
||||||
|
data.forward_channel_id = Some(2);
|
||||||
|
data.edit_before_forward = true;
|
||||||
|
})
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn chat_settings_apply_to_the_normal_link_flow() {
|
||||||
|
let stores = TestStores::new();
|
||||||
|
let sender =
|
||||||
|
MockSender::scripted(vec![Outcome::GroupOk, Outcome::MessageOk], permanent_error);
|
||||||
|
let ctx = stores.ctx(&sender);
|
||||||
|
stores
|
||||||
|
.link_cache()
|
||||||
|
.put("twitter:1", &cached_photo_entry())
|
||||||
|
.await;
|
||||||
|
seed_post_send_settings(&ctx).await;
|
||||||
|
|
||||||
|
url_media(&ctx, 1, 2, "https://x.com/u/status/1", PostSend::FromChat).await;
|
||||||
|
|
||||||
|
// Media group, then the edit prompt (edit-before-forward wins over the
|
||||||
|
// channel forward, which only runs once the prompt is confirmed).
|
||||||
|
assert_eq!(
|
||||||
|
sender.calls(),
|
||||||
|
vec!["send_chat_action", "send_media_group", "send_message"]
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn test_mode_sends_the_media_without_forwarding_or_editing() {
|
||||||
|
let stores = TestStores::new();
|
||||||
|
// Only the group send is scripted: any forward (copy_messages) or edit
|
||||||
|
// prompt (send_message) would panic with "unexpected outcome".
|
||||||
|
let sender = MockSender::scripted(vec![Outcome::GroupOk], permanent_error);
|
||||||
|
let ctx = stores.ctx(&sender);
|
||||||
|
stores
|
||||||
|
.link_cache()
|
||||||
|
.put("twitter:1", &cached_photo_entry())
|
||||||
|
.await;
|
||||||
|
seed_post_send_settings(&ctx).await;
|
||||||
|
|
||||||
|
url_media(&ctx, 1, 2, "https://x.com/u/status/1", PostSend::Suppressed).await;
|
||||||
|
|
||||||
|
assert_eq!(sender.calls(), vec!["send_chat_action", "send_media_group"]);
|
||||||
|
// The send is otherwise ordinary: the post stays cached.
|
||||||
|
assert!(
|
||||||
|
stores
|
||||||
|
.link_cache()
|
||||||
|
.get("twitter:1", Duration::from_secs(3600))
|
||||||
|
.await
|
||||||
|
.is_some()
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn send_mode_decides_whether_chat_actions_ride_along() {
|
||||||
|
let chat = ChatData {
|
||||||
|
forward_channel_id: Some(2),
|
||||||
|
edit_before_forward: true,
|
||||||
|
..ChatData::default()
|
||||||
|
};
|
||||||
|
|
||||||
|
let with_chat = build_send_task(
|
||||||
|
&chat,
|
||||||
|
1,
|
||||||
|
2,
|
||||||
|
"https://x.com/u/status/1".into(),
|
||||||
|
"cap".into(),
|
||||||
|
vec![],
|
||||||
|
None,
|
||||||
|
PostSend::FromChat,
|
||||||
|
);
|
||||||
|
let Task::SendMediaSequence {
|
||||||
|
edit_before_forward,
|
||||||
|
forward_channel_id,
|
||||||
|
..
|
||||||
|
} = with_chat
|
||||||
|
else {
|
||||||
|
panic!("expected a media sequence task");
|
||||||
|
};
|
||||||
|
assert!(edit_before_forward);
|
||||||
|
assert_eq!(forward_channel_id, Some(2));
|
||||||
|
|
||||||
|
let suppressed = build_send_task(
|
||||||
|
&chat,
|
||||||
|
1,
|
||||||
|
2,
|
||||||
|
"https://x.com/u/status/1".into(),
|
||||||
|
"cap".into(),
|
||||||
|
vec![],
|
||||||
|
None,
|
||||||
|
PostSend::Suppressed,
|
||||||
|
);
|
||||||
|
let Task::SendMediaSequence {
|
||||||
|
edit_before_forward,
|
||||||
|
forward_channel_id,
|
||||||
|
notify_chat_id,
|
||||||
|
..
|
||||||
|
} = suppressed
|
||||||
|
else {
|
||||||
|
panic!("expected a media sequence task");
|
||||||
|
};
|
||||||
|
assert!(!edit_before_forward, "`/test` must not open an edit prompt");
|
||||||
|
assert_eq!(forward_channel_id, None, "`/test` must not forward");
|
||||||
|
// Dead-letter notification still reaches the chat that asked.
|
||||||
|
assert_eq!(notify_chat_id, Some(1));
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -5,7 +5,7 @@
|
|||||||
//! key]. A repeated link is then answered entirely from local state — no
|
//! key]. A repeated link is then answered entirely from local state — no
|
||||||
//! re-fetch of the source site, no re-upload — and no media file is stored
|
//! re-fetch of the source site, no re-upload — and no media file is stored
|
||||||
//! on disk (the file ids point at Telegram's servers). Entries expire after
|
//! on disk (the file ids point at Telegram's servers). Entries expire after
|
||||||
//! [`Config::link_cache_ttl`]; a stale entry is dropped lazily on read and
|
//! `Config::link_cache_ttl`; a stale entry is dropped lazily on read and
|
||||||
//! by the periodic prune in `main`.
|
//! by the periodic prune in `main`.
|
||||||
|
|
||||||
use crate::db::now_f64;
|
use crate::db::now_f64;
|
||||||
@@ -78,9 +78,15 @@ impl LinkCache {
|
|||||||
conn.execute("DELETE FROM link_cache WHERE url = ?1", params![key])?;
|
conn.execute("DELETE FROM link_cache WHERE url = ?1", params![key])?;
|
||||||
return Ok(None);
|
return Ok(None);
|
||||||
}
|
}
|
||||||
Ok(Some(serde_json::from_str::<CachedPost>(&payload).map_err(
|
match serde_json::from_str::<CachedPost>(&payload) {
|
||||||
|e| rusqlite::Error::ToSqlConversionFailure(Box::new(e)),
|
Ok(post) => Ok(Some(post)),
|
||||||
)?))
|
Err(e) => {
|
||||||
|
// Unreadable payload (e.g. an older schema): drop it
|
||||||
|
// instead of re-failing the parse on every later hit.
|
||||||
|
conn.execute("DELETE FROM link_cache WHERE url = ?1", params![key])?;
|
||||||
|
Err(rusqlite::Error::ToSqlConversionFailure(Box::new(e)))
|
||||||
|
}
|
||||||
|
}
|
||||||
})
|
})
|
||||||
.await;
|
.await;
|
||||||
match result {
|
match result {
|
||||||
@@ -228,6 +234,32 @@ mod tests {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn unreadable_entry_is_dropped_on_read() {
|
||||||
|
// A payload from an older schema must not be re-parsed on every hit:
|
||||||
|
// the row is removed and the read reports a miss.
|
||||||
|
let dir = tempfile::tempdir().unwrap();
|
||||||
|
let db_path = dir.path().join("c.db");
|
||||||
|
let cache = LinkCache::new(crate::db::open_store(db_path.to_str().unwrap()).unwrap());
|
||||||
|
{
|
||||||
|
let conn = rusqlite::Connection::open(&db_path).unwrap();
|
||||||
|
conn.execute(
|
||||||
|
"INSERT INTO link_cache (url, payload, created_at) VALUES (?1, ?2, ?3)",
|
||||||
|
params!["twitter:1", "{not json", now_f64()],
|
||||||
|
)
|
||||||
|
.unwrap();
|
||||||
|
}
|
||||||
|
|
||||||
|
assert!(
|
||||||
|
cache
|
||||||
|
.get("twitter:1", Duration::from_secs(3600))
|
||||||
|
.await
|
||||||
|
.is_none()
|
||||||
|
);
|
||||||
|
// Dropped, not left behind for the next hit.
|
||||||
|
assert_eq!(cache.clear(None).await, 0, "corrupted row still present");
|
||||||
|
}
|
||||||
|
|
||||||
#[tokio::test]
|
#[tokio::test]
|
||||||
async fn remove_and_prune() {
|
async fn remove_and_prune() {
|
||||||
let dir = tempfile::tempdir().unwrap();
|
let dir = tempfile::tempdir().unwrap();
|
||||||
|
|||||||
@@ -8,6 +8,7 @@ use tokio::sync::watch;
|
|||||||
use x_media::site;
|
use x_media::site;
|
||||||
|
|
||||||
mod config;
|
mod config;
|
||||||
|
mod ctx;
|
||||||
mod db;
|
mod db;
|
||||||
mod handlers;
|
mod handlers;
|
||||||
mod link_cache;
|
mod link_cache;
|
||||||
@@ -18,6 +19,7 @@ mod rate_limit;
|
|||||||
mod send;
|
mod send;
|
||||||
mod state;
|
mod state;
|
||||||
|
|
||||||
|
use ctx::CONTEXT;
|
||||||
use handlers::{CHAT_STORE, CONFIG, LINK_CACHE, TASK_QUEUE};
|
use handlers::{CHAT_STORE, CONFIG, LINK_CACHE, TASK_QUEUE};
|
||||||
|
|
||||||
/// Docker `stop` / `compose down` delivers SIGTERM, which teloxide's ctrlc
|
/// Docker `stop` / `compose down` delivers SIGTERM, which teloxide's ctrlc
|
||||||
@@ -61,9 +63,13 @@ async fn main() {
|
|||||||
);
|
);
|
||||||
|
|
||||||
// Queue worker: handles typed tasks, dead-letters failed sends to the
|
// Queue worker: handles typed tasks, dead-letters failed sends to the
|
||||||
// task's chat.
|
// task's chat. Both closures use the shared context (the queue requires
|
||||||
|
// 'static handlers, and the statics are process-wide anyway).
|
||||||
TASK_QUEUE
|
TASK_QUEUE
|
||||||
.start(send::handle_task, send::dead_letter_notify)
|
.start(
|
||||||
|
|payload| send::handle_task(&CONTEXT, payload),
|
||||||
|
|payload, message| send::dead_letter_notify(&CONTEXT, payload, message),
|
||||||
|
)
|
||||||
.await;
|
.await;
|
||||||
log::info!("task queue worker started");
|
log::info!("task queue worker started");
|
||||||
|
|
||||||
@@ -108,6 +114,10 @@ async fn main() {
|
|||||||
if pruned > 0 {
|
if pruned > 0 {
|
||||||
log::info!("link cache: pruned {pruned} expired entr(ies)");
|
log::info!("link cache: pruned {pruned} expired entr(ies)");
|
||||||
}
|
}
|
||||||
|
let idle_limiters = crate::rate_limit::prune_idle();
|
||||||
|
if idle_limiters > 0 {
|
||||||
|
log::debug!("rate limiter: dropped {idle_limiters} idle bucket(s)");
|
||||||
|
}
|
||||||
for (chat_id, prompt_message_id) in removed {
|
for (chat_id, prompt_message_id) in removed {
|
||||||
// If the prompt was already deleted, this fails with a
|
// If the prompt was already deleted, this fails with a
|
||||||
// 400 "message to edit not found" — log and ignore.
|
// 400 "message to edit not found" — log and ignore.
|
||||||
|
|||||||
@@ -8,8 +8,8 @@ use teloxide::RequestError;
|
|||||||
use teloxide::prelude::Requester;
|
use teloxide::prelude::Requester;
|
||||||
use teloxide::prelude::*;
|
use teloxide::prelude::*;
|
||||||
use teloxide::types::{
|
use teloxide::types::{
|
||||||
ChatAction, ChatId, InlineKeyboardMarkup, InputFile, InputMedia, Message, MessageId, ParseMode,
|
CallbackQueryId, ChatAction, ChatId, InlineKeyboardMarkup, InputFile, InputMedia, Message,
|
||||||
ReplyParameters,
|
MessageId, ParseMode, ReplyParameters,
|
||||||
};
|
};
|
||||||
|
|
||||||
/// Boxed, `Send` future returned by a [`MediaSender`] method (`async fn` in
|
/// Boxed, `Send` future returned by a [`MediaSender`] method (`async fn` in
|
||||||
@@ -48,14 +48,42 @@ pub trait MediaSender: Send + Sync {
|
|||||||
) -> BoxFuture<'_, Result<Vec<MessageId>, RequestError>>;
|
) -> BoxFuture<'_, Result<Vec<MessageId>, RequestError>>;
|
||||||
|
|
||||||
/// Sends a plain text message, optionally replying to `reply_to` and
|
/// Sends a plain text message, optionally replying to `reply_to` and
|
||||||
/// attaching `reply_markup`.
|
/// attaching `reply_markup`. Returns the sent message's id: the bot only
|
||||||
|
/// ever needs that (the edit-before-forward prompt's record is keyed by
|
||||||
|
/// it), and returning the whole `Message` would force every test mock to
|
||||||
|
/// construct one.
|
||||||
fn send_message(
|
fn send_message(
|
||||||
&self,
|
&self,
|
||||||
chat_id: ChatId,
|
chat_id: ChatId,
|
||||||
text: String,
|
text: String,
|
||||||
reply_to: Option<MessageId>,
|
reply_to: Option<MessageId>,
|
||||||
reply_markup: Option<InlineKeyboardMarkup>,
|
reply_markup: Option<InlineKeyboardMarkup>,
|
||||||
) -> BoxFuture<'_, Result<Message, RequestError>>;
|
) -> BoxFuture<'_, Result<i64, RequestError>>;
|
||||||
|
|
||||||
|
/// Answers a callback query, optionally with a toast `text` shown to the
|
||||||
|
/// user who pressed the button.
|
||||||
|
fn answer_callback_query(
|
||||||
|
&self,
|
||||||
|
id: CallbackQueryId,
|
||||||
|
text: Option<String>,
|
||||||
|
) -> BoxFuture<'_, Result<(), RequestError>>;
|
||||||
|
|
||||||
|
/// Rewrites a message's caption, always with HTML parse mode (every caller
|
||||||
|
/// in this bot renders escaped HTML: templates and edit-before-forward
|
||||||
|
/// links).
|
||||||
|
fn edit_message_caption(
|
||||||
|
&self,
|
||||||
|
chat_id: ChatId,
|
||||||
|
message_id: MessageId,
|
||||||
|
caption: String,
|
||||||
|
) -> BoxFuture<'_, Result<(), RequestError>>;
|
||||||
|
|
||||||
|
/// Deletes a message (the edit-before-forward prompt after a forward).
|
||||||
|
fn delete_message(
|
||||||
|
&self,
|
||||||
|
chat_id: ChatId,
|
||||||
|
message_id: MessageId,
|
||||||
|
) -> BoxFuture<'_, Result<(), RequestError>>;
|
||||||
|
|
||||||
/// Sets the chat's "typing / uploading …" indicator (cosmetic).
|
/// Sets the chat's "typing / uploading …" indicator (cosmetic).
|
||||||
fn send_chat_action(
|
fn send_chat_action(
|
||||||
@@ -129,7 +157,7 @@ impl MediaSender for Bot {
|
|||||||
text: String,
|
text: String,
|
||||||
reply_to: Option<MessageId>,
|
reply_to: Option<MessageId>,
|
||||||
reply_markup: Option<InlineKeyboardMarkup>,
|
reply_markup: Option<InlineKeyboardMarkup>,
|
||||||
) -> BoxFuture<'_, Result<Message, RequestError>> {
|
) -> BoxFuture<'_, Result<i64, RequestError>> {
|
||||||
Box::pin(async move {
|
Box::pin(async move {
|
||||||
let mut request = <Bot as Requester>::send_message(self, chat_id, text);
|
let mut request = <Bot as Requester>::send_message(self, chat_id, text);
|
||||||
if let Some(reply_to) = reply_to {
|
if let Some(reply_to) = reply_to {
|
||||||
@@ -139,7 +167,48 @@ impl MediaSender for Bot {
|
|||||||
if let Some(markup) = reply_markup {
|
if let Some(markup) = reply_markup {
|
||||||
request = request.reply_markup(markup);
|
request = request.reply_markup(markup);
|
||||||
}
|
}
|
||||||
request.await
|
request.await.map(|message| message.id.0 as i64)
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
fn answer_callback_query(
|
||||||
|
&self,
|
||||||
|
id: CallbackQueryId,
|
||||||
|
text: Option<String>,
|
||||||
|
) -> BoxFuture<'_, Result<(), RequestError>> {
|
||||||
|
Box::pin(async move {
|
||||||
|
let mut request = <Bot as Requester>::answer_callback_query(self, id);
|
||||||
|
if let Some(text) = text {
|
||||||
|
request = request.text(text);
|
||||||
|
}
|
||||||
|
request.await.map(|_| ())
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
fn edit_message_caption(
|
||||||
|
&self,
|
||||||
|
chat_id: ChatId,
|
||||||
|
message_id: MessageId,
|
||||||
|
caption: String,
|
||||||
|
) -> BoxFuture<'_, Result<(), RequestError>> {
|
||||||
|
Box::pin(async move {
|
||||||
|
<Bot as Requester>::edit_message_caption(self, chat_id, message_id)
|
||||||
|
.caption(caption)
|
||||||
|
.parse_mode(ParseMode::Html)
|
||||||
|
.await
|
||||||
|
.map(|_| ())
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
fn delete_message(
|
||||||
|
&self,
|
||||||
|
chat_id: ChatId,
|
||||||
|
message_id: MessageId,
|
||||||
|
) -> BoxFuture<'_, Result<(), RequestError>> {
|
||||||
|
Box::pin(async move {
|
||||||
|
<Bot as Requester>::delete_message(self, chat_id, message_id)
|
||||||
|
.await
|
||||||
|
.map(|_| ())
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -162,7 +231,7 @@ impl MediaSender for Bot {
|
|||||||
#[cfg(test)]
|
#[cfg(test)]
|
||||||
pub(crate) mod test_support {
|
pub(crate) mod test_support {
|
||||||
use super::*;
|
use super::*;
|
||||||
use std::sync::Mutex;
|
use parking_lot::Mutex;
|
||||||
|
|
||||||
/// One scripted outcome, consumed front-to-back; the last entry repeats
|
/// One scripted outcome, consumed front-to-back; the last entry repeats
|
||||||
/// for further calls of the same method kind.
|
/// for further calls of the same method kind.
|
||||||
@@ -176,19 +245,30 @@ pub(crate) mod test_support {
|
|||||||
/// An error from `send_message` (replies are fire-and-forget, so an
|
/// An error from `send_message` (replies are fire-and-forget, so an
|
||||||
/// error is fine for tests).
|
/// error is fine for tests).
|
||||||
MessageErr,
|
MessageErr,
|
||||||
|
/// A successful `send_message`, returning message id [`MockSender::SENT_ID`].
|
||||||
|
MessageOk,
|
||||||
|
EditOk,
|
||||||
|
EditErr,
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Replays a script and records the method names that were called.
|
/// Replays a script and records what was sent, so tests can assert the
|
||||||
|
/// user-visible text a path produced.
|
||||||
pub(crate) struct MockSender {
|
pub(crate) struct MockSender {
|
||||||
script: Mutex<Vec<Outcome>>,
|
script: Mutex<Vec<Outcome>>,
|
||||||
cursor: Mutex<usize>,
|
cursor: Mutex<usize>,
|
||||||
calls: Mutex<Vec<&'static str>>,
|
calls: Mutex<Vec<&'static str>>,
|
||||||
|
messages: Mutex<Vec<String>>,
|
||||||
|
captions: Mutex<Vec<String>>,
|
||||||
|
answers: Mutex<Vec<Option<String>>>,
|
||||||
/// Builds the error every `*Err` outcome returns (RequestError is not
|
/// Builds the error every `*Err` outcome returns (RequestError is not
|
||||||
/// cloneable, so the factory recreates it per call).
|
/// cloneable, so the factory recreates it per call).
|
||||||
error: Box<dyn Fn() -> RequestError + Send + Sync>,
|
error: Box<dyn Fn() -> RequestError + Send + Sync>,
|
||||||
}
|
}
|
||||||
|
|
||||||
impl MockSender {
|
impl MockSender {
|
||||||
|
/// The message id a successful `send_message` reports.
|
||||||
|
pub(crate) const SENT_ID: i64 = 1;
|
||||||
|
|
||||||
pub(crate) fn scripted(
|
pub(crate) fn scripted(
|
||||||
script: Vec<Outcome>,
|
script: Vec<Outcome>,
|
||||||
error: impl Fn() -> RequestError + Send + Sync + 'static,
|
error: impl Fn() -> RequestError + Send + Sync + 'static,
|
||||||
@@ -197,6 +277,9 @@ pub(crate) mod test_support {
|
|||||||
script: Mutex::new(script),
|
script: Mutex::new(script),
|
||||||
cursor: Mutex::new(0),
|
cursor: Mutex::new(0),
|
||||||
calls: Mutex::new(Vec::new()),
|
calls: Mutex::new(Vec::new()),
|
||||||
|
messages: Mutex::new(Vec::new()),
|
||||||
|
captions: Mutex::new(Vec::new()),
|
||||||
|
answers: Mutex::new(Vec::new()),
|
||||||
error: Box::new(error),
|
error: Box::new(error),
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -204,13 +287,28 @@ pub(crate) mod test_support {
|
|||||||
/// Method names in call order (e.g. `["send_media_group",
|
/// Method names in call order (e.g. `["send_media_group",
|
||||||
/// "send_media_group"]` proves the fallback re-sent).
|
/// "send_media_group"]` proves the fallback re-sent).
|
||||||
pub(crate) fn calls(&self) -> Vec<&'static str> {
|
pub(crate) fn calls(&self) -> Vec<&'static str> {
|
||||||
self.calls.lock().unwrap().clone()
|
self.calls.lock().clone()
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Texts of the plain messages sent, in order.
|
||||||
|
pub(crate) fn messages(&self) -> Vec<String> {
|
||||||
|
self.messages.lock().clone()
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Captions passed to `edit_message_caption`, in order.
|
||||||
|
pub(crate) fn captions(&self) -> Vec<String> {
|
||||||
|
self.captions.lock().clone()
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Toast texts of the answered callback queries, in order.
|
||||||
|
pub(crate) fn answers(&self) -> Vec<Option<String>> {
|
||||||
|
self.answers.lock().clone()
|
||||||
}
|
}
|
||||||
|
|
||||||
fn next(&self, kind: &'static str) -> Outcome {
|
fn next(&self, kind: &'static str) -> Outcome {
|
||||||
self.calls.lock().unwrap().push(kind);
|
self.calls.lock().push(kind);
|
||||||
let script = self.script.lock().unwrap();
|
let script = self.script.lock();
|
||||||
let mut cursor = self.cursor.lock().unwrap();
|
let mut cursor = self.cursor.lock();
|
||||||
if script.is_empty() {
|
if script.is_empty() {
|
||||||
panic!("mock script exhausted: {kind}");
|
panic!("mock script exhausted: {kind}");
|
||||||
}
|
}
|
||||||
@@ -274,25 +372,69 @@ pub(crate) mod test_support {
|
|||||||
fn send_message(
|
fn send_message(
|
||||||
&self,
|
&self,
|
||||||
_chat_id: ChatId,
|
_chat_id: ChatId,
|
||||||
_text: String,
|
text: String,
|
||||||
_reply_to: Option<MessageId>,
|
_reply_to: Option<MessageId>,
|
||||||
_reply_markup: Option<InlineKeyboardMarkup>,
|
_reply_markup: Option<InlineKeyboardMarkup>,
|
||||||
) -> BoxFuture<'_, Result<Message, RequestError>> {
|
) -> BoxFuture<'_, Result<i64, RequestError>> {
|
||||||
Box::pin(async move {
|
Box::pin(async move {
|
||||||
|
self.messages.lock().push(text);
|
||||||
match self.next("send_message") {
|
match self.next("send_message") {
|
||||||
|
Outcome::MessageOk => Ok(MockSender::SENT_ID),
|
||||||
Outcome::MessageErr => Err(self.error()),
|
Outcome::MessageErr => Err(self.error()),
|
||||||
other => panic!("unexpected outcome {other:?} for send_message"),
|
other => panic!("unexpected outcome {other:?} for send_message"),
|
||||||
}
|
}
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn answer_callback_query(
|
||||||
|
&self,
|
||||||
|
_id: CallbackQueryId,
|
||||||
|
text: Option<String>,
|
||||||
|
) -> BoxFuture<'_, Result<(), RequestError>> {
|
||||||
|
// Always succeeds: the toast is cosmetic, so the script stays
|
||||||
|
// focused on the outcomes a test cares about.
|
||||||
|
Box::pin(async move {
|
||||||
|
self.calls.lock().push("answer_callback_query");
|
||||||
|
self.answers.lock().push(text);
|
||||||
|
Ok(())
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
fn edit_message_caption(
|
||||||
|
&self,
|
||||||
|
_chat_id: ChatId,
|
||||||
|
_message_id: MessageId,
|
||||||
|
caption: String,
|
||||||
|
) -> BoxFuture<'_, Result<(), RequestError>> {
|
||||||
|
Box::pin(async move {
|
||||||
|
self.captions.lock().push(caption);
|
||||||
|
match self.next("edit_message_caption") {
|
||||||
|
Outcome::EditOk => Ok(()),
|
||||||
|
Outcome::EditErr => Err(self.error()),
|
||||||
|
other => panic!("unexpected outcome {other:?} for edit_message_caption"),
|
||||||
|
}
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
fn delete_message(
|
||||||
|
&self,
|
||||||
|
_chat_id: ChatId,
|
||||||
|
_message_id: MessageId,
|
||||||
|
) -> BoxFuture<'_, Result<(), RequestError>> {
|
||||||
|
// Deletion is fire-and-forget in every caller; always succeeds.
|
||||||
|
Box::pin(async move {
|
||||||
|
self.calls.lock().push("delete_message");
|
||||||
|
Ok(())
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
fn send_chat_action(
|
fn send_chat_action(
|
||||||
&self,
|
&self,
|
||||||
_chat_id: ChatId,
|
_chat_id: ChatId,
|
||||||
_action: ChatAction,
|
_action: ChatAction,
|
||||||
) -> BoxFuture<'_, Result<(), RequestError>> {
|
) -> BoxFuture<'_, Result<(), RequestError>> {
|
||||||
Box::pin(async move {
|
Box::pin(async move {
|
||||||
self.calls.lock().unwrap().push("send_chat_action");
|
self.calls.lock().push("send_chat_action");
|
||||||
Ok(())
|
Ok(())
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -122,7 +122,7 @@ fn output_channels(color: png::ColorType) -> usize {
|
|||||||
/// white; 16-bit per channel was already stripped to 8-bit at decode.
|
/// white; 16-bit per channel was already stripped to 8-bit at decode.
|
||||||
fn flatten_rgba_to_rgb(rgba: &[u8]) -> Vec<u8> {
|
fn flatten_rgba_to_rgb(rgba: &[u8]) -> Vec<u8> {
|
||||||
let mut rgb = Vec::with_capacity(rgba.len() / 4 * 3);
|
let mut rgb = Vec::with_capacity(rgba.len() / 4 * 3);
|
||||||
for px in rgba.chunks_exact(4) {
|
for px in rgba.as_chunks::<4>().0 {
|
||||||
let a = px[3] as u32;
|
let a = px[3] as u32;
|
||||||
for v in &px[..3] {
|
for v in &px[..3] {
|
||||||
// Over white: C = C*a/255 + 255*(1 - a/255).
|
// Over white: C = C*a/255 + 255*(1 - a/255).
|
||||||
@@ -178,7 +178,9 @@ fn encode_jpeg(pix: &PixBuf, w: u32, h: u32) -> Result<Vec<u8>, String> {
|
|||||||
PixBuf::GrayAlpha(v) => {
|
PixBuf::GrayAlpha(v) => {
|
||||||
// JPEG has no alpha: composite onto white, output as gray.
|
// JPEG has no alpha: composite onto white, output as gray.
|
||||||
let gray: Vec<u8> = v
|
let gray: Vec<u8> = v
|
||||||
.chunks_exact(2)
|
.as_chunks::<2>()
|
||||||
|
.0
|
||||||
|
.iter()
|
||||||
.map(|px| {
|
.map(|px| {
|
||||||
let (g, a) = (px[0] as u32, px[1] as u32);
|
let (g, a) = (px[0] as u32, px[1] as u32);
|
||||||
((g * a + 255 * (255 - a)) / 255).min(255) as u8
|
((g * a + 255 * (255 - a)) / 255).min(255) as u8
|
||||||
|
|||||||
@@ -41,7 +41,13 @@ type DeadLetter = dyn Fn(Value, String) -> BoxFuture<'static, ()> + Send + Sync;
|
|||||||
|
|
||||||
pub struct PersistentTaskQueue {
|
pub struct PersistentTaskQueue {
|
||||||
pool: std::sync::Arc<crate::db::DbPool>,
|
pool: std::sync::Arc<crate::db::DbPool>,
|
||||||
|
/// Wakes the workers when a row becomes leasable. `notify_one` stores a
|
||||||
|
/// permit, so nothing else may share it: a waiter that is not a worker
|
||||||
|
/// (the sweep) can consume the permit and leave the due row pending until
|
||||||
|
/// the next enqueue.
|
||||||
notify: Arc<Notify>,
|
notify: Arc<Notify>,
|
||||||
|
/// Wakes the lease-expiry sweep; `stop` is the only producer.
|
||||||
|
sweep_notify: Arc<Notify>,
|
||||||
stop: Arc<AtomicBool>,
|
stop: Arc<AtomicBool>,
|
||||||
worker: Mutex<Vec<JoinHandle<()>>>,
|
worker: Mutex<Vec<JoinHandle<()>>>,
|
||||||
counter: AtomicU64,
|
counter: AtomicU64,
|
||||||
@@ -88,6 +94,7 @@ impl PersistentTaskQueue {
|
|||||||
Self {
|
Self {
|
||||||
pool,
|
pool,
|
||||||
notify: Arc::new(Notify::new()),
|
notify: Arc::new(Notify::new()),
|
||||||
|
sweep_notify: Arc::new(Notify::new()),
|
||||||
stop: Arc::new(AtomicBool::new(false)),
|
stop: Arc::new(AtomicBool::new(false)),
|
||||||
worker: Mutex::new(Vec::new()),
|
worker: Mutex::new(Vec::new()),
|
||||||
counter: AtomicU64::new(0),
|
counter: AtomicU64::new(0),
|
||||||
@@ -119,12 +126,12 @@ impl PersistentTaskQueue {
|
|||||||
handles.push(tokio::spawn(worker.run_loop_supervised()));
|
handles.push(tokio::spawn(worker.run_loop_supervised()));
|
||||||
}
|
}
|
||||||
// Periodic lease-expiry sweep: recovers rows a crashed/panicked
|
// Periodic lease-expiry sweep: recovers rows a crashed/panicked
|
||||||
// worker left `in_progress` (the lock TTL bounds the wait). Woken by
|
// worker left `in_progress` (the lock TTL bounds the wait). Its own
|
||||||
// the same notify as the workers, so enqueue and stop interrupt the
|
// notify (not the workers'): sharing that one let this task consume a
|
||||||
// sleep; the first interval tick fires immediately (harmless extra
|
// `notify_one` permit meant for a worker, which then slept through a
|
||||||
// recovery at startup).
|
// due row until some later event. Only `stop` wakes it.
|
||||||
let sweep_pool = std::sync::Arc::clone(&self.pool);
|
let sweep_pool = std::sync::Arc::clone(&self.pool);
|
||||||
let sweep_notify = Arc::clone(&self.notify);
|
let sweep_notify = Arc::clone(&self.sweep_notify);
|
||||||
let sweep_stop = Arc::clone(&self.stop);
|
let sweep_stop = Arc::clone(&self.stop);
|
||||||
handles.push(tokio::spawn(async move {
|
handles.push(tokio::spawn(async move {
|
||||||
let mut interval = tokio::time::interval(Duration::from_secs(30));
|
let mut interval = tokio::time::interval(Duration::from_secs(30));
|
||||||
@@ -150,6 +157,7 @@ impl PersistentTaskQueue {
|
|||||||
pub async fn stop(&self) {
|
pub async fn stop(&self) {
|
||||||
self.stop.store(true, Ordering::Relaxed);
|
self.stop.store(true, Ordering::Relaxed);
|
||||||
self.notify.notify_waiters();
|
self.notify.notify_waiters();
|
||||||
|
self.sweep_notify.notify_waiters();
|
||||||
let handles = std::mem::take(&mut *self.worker.lock());
|
let handles = std::mem::take(&mut *self.worker.lock());
|
||||||
for handle in handles {
|
for handle in handles {
|
||||||
let _ = handle.await;
|
let _ = handle.await;
|
||||||
|
|||||||
@@ -48,6 +48,19 @@ impl TokenBucket {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// Applies the elapsed refill to `state`. Shared by [`Self::acquire`] and
|
||||||
|
/// the idle check so the two cannot drift apart.
|
||||||
|
fn refill(&self, state: &mut State) {
|
||||||
|
let now = tokio::time::Instant::now();
|
||||||
|
let elapsed = now
|
||||||
|
.saturating_duration_since(state.last_refill)
|
||||||
|
.as_secs_f64();
|
||||||
|
// Refill up to the capacity; a debt (negative balance) is repaid
|
||||||
|
// before any surplus accumulates.
|
||||||
|
state.tokens = (state.tokens + elapsed * self.refill_per_sec).min(self.capacity);
|
||||||
|
state.last_refill = now;
|
||||||
|
}
|
||||||
|
|
||||||
/// Waits until `n` tokens are available, consuming them. The wait is
|
/// Waits until `n` tokens are available, consuming them. The wait is
|
||||||
/// bounded: the deficit is committed as debt and repaid over time, so a
|
/// bounded: the deficit is committed as debt and repaid over time, so a
|
||||||
/// large acquire returns once its share of the refill budget has passed.
|
/// large acquire returns once its share of the refill budget has passed.
|
||||||
@@ -57,14 +70,7 @@ impl TokenBucket {
|
|||||||
// would make the future !Send).
|
// would make the future !Send).
|
||||||
let wait = {
|
let wait = {
|
||||||
let mut state = self.state.lock();
|
let mut state = self.state.lock();
|
||||||
let now = tokio::time::Instant::now();
|
self.refill(&mut state);
|
||||||
let elapsed = now
|
|
||||||
.saturating_duration_since(state.last_refill)
|
|
||||||
.as_secs_f64();
|
|
||||||
// Refill up to the capacity; a debt (negative balance) is repaid
|
|
||||||
// before any surplus accumulates.
|
|
||||||
state.tokens = (state.tokens + elapsed * self.refill_per_sec).min(self.capacity);
|
|
||||||
state.last_refill = now;
|
|
||||||
if state.tokens >= n {
|
if state.tokens >= n {
|
||||||
state.tokens -= n;
|
state.tokens -= n;
|
||||||
return;
|
return;
|
||||||
@@ -77,6 +83,14 @@ impl TokenBucket {
|
|||||||
};
|
};
|
||||||
tokio::time::sleep(Duration::from_secs_f64(wait)).await;
|
tokio::time::sleep(Duration::from_secs_f64(wait)).await;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// True when the bucket has refilled to capacity: no debt outstanding, so
|
||||||
|
/// the chat has not sent anything recently.
|
||||||
|
fn is_idle(&self) -> bool {
|
||||||
|
let mut state = self.state.lock();
|
||||||
|
self.refill(&mut state);
|
||||||
|
state.tokens >= self.capacity
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/// One limiter per chat, created on first use. Per-chat so one chat's burst
|
/// One limiter per chat, created on first use. Per-chat so one chat's burst
|
||||||
@@ -93,6 +107,18 @@ pub fn limiter_for(chat_id: i64) -> Arc<TokenBucket> {
|
|||||||
.clone()
|
.clone()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// Drops limiters that are idle (refilled to capacity, so the chat has not
|
||||||
|
/// sent recently) and are not still held by an in-flight sender. The map
|
||||||
|
/// would otherwise keep one bucket per chat that ever sent media, forever.
|
||||||
|
/// Called from the periodic sweep; returns how many were dropped.
|
||||||
|
pub fn prune_idle() -> usize {
|
||||||
|
let mut limiters = LIMITERS.lock();
|
||||||
|
let before = limiters.len();
|
||||||
|
// Lock order map → bucket, the only order taken anywhere.
|
||||||
|
limiters.retain(|_, bucket| Arc::strong_count(bucket) > 1 || !bucket.is_idle());
|
||||||
|
before - limiters.len()
|
||||||
|
}
|
||||||
|
|
||||||
#[cfg(test)]
|
#[cfg(test)]
|
||||||
mod tests {
|
mod tests {
|
||||||
use super::*;
|
use super::*;
|
||||||
@@ -133,4 +159,26 @@ mod tests {
|
|||||||
start.elapsed()
|
start.elapsed()
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[tokio::test(start_paused = true)]
|
||||||
|
async fn prune_idle_drops_full_unheld_buckets_only() {
|
||||||
|
// Held by this task: kept even at full capacity, a sender has it.
|
||||||
|
let held = limiter_for(9_001);
|
||||||
|
assert!(held.is_idle(), "a fresh bucket is full");
|
||||||
|
// Only the map holds this one and it is full → dropped.
|
||||||
|
limiter_for(9_002);
|
||||||
|
// Mid-debt (an acquire larger than the capacity): kept.
|
||||||
|
{
|
||||||
|
let bucket = Arc::new(TokenBucket::new(CAPACITY, REFILL_PER_SEC));
|
||||||
|
bucket.state.lock().tokens = -1.0;
|
||||||
|
LIMITERS.lock().insert(9_003, bucket);
|
||||||
|
}
|
||||||
|
|
||||||
|
assert!(prune_idle() >= 1);
|
||||||
|
|
||||||
|
let limiters = LIMITERS.lock();
|
||||||
|
assert!(limiters.contains_key(&9_001), "held bucket pruned");
|
||||||
|
assert!(!limiters.contains_key(&9_002), "idle unheld bucket kept");
|
||||||
|
assert!(limiters.contains_key(&9_003), "indebted bucket pruned");
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1,128 @@
|
|||||||
|
//! Payload → Telegram input types: `InputFile` selection (cached file id /
|
||||||
|
//! URL / local path), the per-kind `InputMedia` builders and the media-group
|
||||||
|
//! assembly with its caption rule.
|
||||||
|
|
||||||
|
use super::MediaItemPayload;
|
||||||
|
use teloxide::types::{
|
||||||
|
InputFile, InputMedia, InputMediaAnimation, InputMediaPhoto, InputMediaVideo, ParseMode,
|
||||||
|
};
|
||||||
|
|
||||||
|
fn parse_media_url(s: &str) -> Result<url::Url, String> {
|
||||||
|
url::Url::parse(s).map_err(|e| format!("invalid media URL: {e}"))
|
||||||
|
}
|
||||||
|
|
||||||
|
pub(super) fn item_url(item: &MediaItemPayload) -> &str {
|
||||||
|
match item {
|
||||||
|
MediaItemPayload::Photo { media, .. }
|
||||||
|
| MediaItemPayload::Video { media, .. }
|
||||||
|
| MediaItemPayload::Animation { media, .. } => media,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Remote http(s) URLs are handed to Telegram to fetch; everything else
|
||||||
|
/// (e.g. a locally encoded ugoira MP4) is uploaded directly.
|
||||||
|
pub(super) fn input_file_for(media: &str) -> Result<InputFile, String> {
|
||||||
|
if media.starts_with("http://") || media.starts_with("https://") {
|
||||||
|
Ok(InputFile::url(parse_media_url(media)?))
|
||||||
|
} else if !std::path::Path::new(media).exists() {
|
||||||
|
// A retried task may reference a temp file the original send's
|
||||||
|
// TempDir already cleaned up; fail fast and permanent instead of
|
||||||
|
// burning retries on a file that can never come back.
|
||||||
|
Err(format!("local media file missing: {media}"))
|
||||||
|
} else {
|
||||||
|
Ok(InputFile::file(media))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
impl MediaItemPayload {
|
||||||
|
/// The input for a send: a cached file id goes out as `InputFile::file_id`
|
||||||
|
/// (no fetch, no upload), URLs go to Telegram, anything else is a local
|
||||||
|
/// path (transient upload fallback).
|
||||||
|
fn input_file(&self) -> Result<InputFile, String> {
|
||||||
|
match self {
|
||||||
|
MediaItemPayload::Photo {
|
||||||
|
media,
|
||||||
|
file_id: true,
|
||||||
|
..
|
||||||
|
}
|
||||||
|
| MediaItemPayload::Video {
|
||||||
|
media,
|
||||||
|
file_id: true,
|
||||||
|
..
|
||||||
|
}
|
||||||
|
| MediaItemPayload::Animation {
|
||||||
|
media,
|
||||||
|
file_id: true,
|
||||||
|
..
|
||||||
|
} => Ok(InputFile::file_id(media.clone().into())),
|
||||||
|
_ => input_file_for(item_url(self)),
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
pub(super) fn photo_media(file: InputFile, caption: Option<&str>, spoiler: bool) -> InputMedia {
|
||||||
|
let mut photo = InputMediaPhoto::new(file).parse_mode(ParseMode::Html);
|
||||||
|
if let Some(caption) = caption {
|
||||||
|
photo = photo.caption(caption);
|
||||||
|
}
|
||||||
|
if spoiler {
|
||||||
|
photo = photo.spoiler();
|
||||||
|
}
|
||||||
|
InputMedia::Photo(photo)
|
||||||
|
}
|
||||||
|
|
||||||
|
pub(super) fn video_media(file: InputFile, caption: Option<&str>, spoiler: bool) -> InputMedia {
|
||||||
|
let mut video = InputMediaVideo::new(file).parse_mode(ParseMode::Html);
|
||||||
|
if let Some(caption) = caption {
|
||||||
|
video = video.caption(caption);
|
||||||
|
}
|
||||||
|
if spoiler {
|
||||||
|
video = video.spoiler();
|
||||||
|
}
|
||||||
|
InputMedia::Video(video)
|
||||||
|
}
|
||||||
|
|
||||||
|
pub(super) fn animation_media(file: InputFile, caption: Option<&str>, spoiler: bool) -> InputMedia {
|
||||||
|
let mut animation = InputMediaAnimation::new(file).parse_mode(ParseMode::Html);
|
||||||
|
if let Some(caption) = caption {
|
||||||
|
animation = animation.caption(caption);
|
||||||
|
}
|
||||||
|
if spoiler {
|
||||||
|
animation = animation.spoiler();
|
||||||
|
}
|
||||||
|
InputMedia::Animation(animation)
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Builds a media group from payloads; only the first item of the batch gets
|
||||||
|
/// the caption (Telegram rejects captions on later items).
|
||||||
|
pub(super) fn build_media_group(
|
||||||
|
batch: &[MediaItemPayload],
|
||||||
|
caption: Option<&str>,
|
||||||
|
) -> Result<Vec<InputMedia>, String> {
|
||||||
|
batch
|
||||||
|
.iter()
|
||||||
|
.enumerate()
|
||||||
|
.map(|(i, item)| {
|
||||||
|
let item_caption = if i == 0 { caption } else { None };
|
||||||
|
Ok(match item {
|
||||||
|
MediaItemPayload::Photo { has_spoiler, .. } => {
|
||||||
|
photo_media(item.input_file()?, item_caption, *has_spoiler)
|
||||||
|
}
|
||||||
|
MediaItemPayload::Video {
|
||||||
|
has_spoiler,
|
||||||
|
thumbnail,
|
||||||
|
..
|
||||||
|
} => {
|
||||||
|
let mut video = video_media(item.input_file()?, item_caption, *has_spoiler);
|
||||||
|
if let (Some(thumb), InputMedia::Video(v)) = (thumbnail, &mut video) {
|
||||||
|
*v = v.clone().thumbnail(input_file_for(thumb)?);
|
||||||
|
}
|
||||||
|
video
|
||||||
|
}
|
||||||
|
MediaItemPayload::Animation { has_spoiler, .. } => {
|
||||||
|
animation_media(item.input_file()?, item_caption, *has_spoiler)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
})
|
||||||
|
.collect()
|
||||||
|
}
|
||||||
File diff suppressed because it is too large
Load Diff
@@ -0,0 +1,366 @@
|
|||||||
|
//! Everything around a send: the link-cache write that follows one, the
|
||||||
|
//! keep-alive registry for locally produced media, task settlement, the
|
||||||
|
//! post-send actions (edit prompt / channel forward) and the queue entry
|
||||||
|
//! points.
|
||||||
|
|
||||||
|
use super::{SendError, Task, forward_messages, send_animation, send_media_sequence};
|
||||||
|
use crate::ctx::AppContext;
|
||||||
|
use crate::db::{now_f64, unix_now};
|
||||||
|
use crate::handlers::log_key;
|
||||||
|
use crate::link_cache::{CachedMedia, CachedMediaKind, LinkCache};
|
||||||
|
use crate::media_sender::MediaSender;
|
||||||
|
use crate::queue::{PersistentTaskQueue, QueueError};
|
||||||
|
use crate::state::EditMessage;
|
||||||
|
use std::collections::HashMap;
|
||||||
|
use std::sync::LazyLock;
|
||||||
|
use teloxide::types::{ChatId, InlineKeyboardButton, InlineKeyboardMarkup, Message, MessageId};
|
||||||
|
|
||||||
|
/// Persists a successful send under the post's cache key. Only runs for a
|
||||||
|
/// fresh (non-resumed) task that carried raw cache data with no file ids yet.
|
||||||
|
pub(super) async fn cache_sent_task(ctx: &AppContext<'_>, task: &Task, media: Vec<CachedMedia>) {
|
||||||
|
let Some(cache_data) = task.cache_data() else {
|
||||||
|
return;
|
||||||
|
};
|
||||||
|
if !cache_data.media.is_empty() || media.is_empty() {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
let mut post = cache_data.clone();
|
||||||
|
post.media = media;
|
||||||
|
if let Some(key) = x_media::site::cache_key(&post.url) {
|
||||||
|
ctx.link_cache.put(&key, &post).await;
|
||||||
|
log::debug!("cached send for [key={}]", log_key(&post.url));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Persists a lone animation send under the post's cache key.
|
||||||
|
pub(super) async fn cache_animation_send(ctx: &AppContext<'_>, task: &Task, message: &Message) {
|
||||||
|
if let Some(file_id) = message.animation().map(|a| a.file.id.to_string()) {
|
||||||
|
cache_sent_task(
|
||||||
|
ctx,
|
||||||
|
task,
|
||||||
|
vec![CachedMedia {
|
||||||
|
kind: CachedMediaKind::Animation,
|
||||||
|
file_id,
|
||||||
|
}],
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// How a task ended. The two states differ only in whether a link-cache entry
|
||||||
|
/// may still be holding the (now unusable) media.
|
||||||
|
pub(crate) enum Settled {
|
||||||
|
Sent,
|
||||||
|
Failed,
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Every path that ends a task's life — sent, permanently failed, or
|
||||||
|
/// dead-lettered after the last retry — funnels through here, so the cleanup a
|
||||||
|
/// settled task owes cannot be forgotten by a new path: release the keep-alive
|
||||||
|
/// temp media (retryable tasks keep it, they will be resent) and drop the
|
||||||
|
/// link-cache entry that a failed send's stale file ids would keep poisoning.
|
||||||
|
pub(crate) async fn settle_task(ctx: &AppContext<'_>, task: &Task, outcome: Settled) {
|
||||||
|
if matches!(outcome, Settled::Failed) {
|
||||||
|
invalidate_cache(ctx.link_cache, task).await;
|
||||||
|
}
|
||||||
|
release_keep_alive(task);
|
||||||
|
}
|
||||||
|
|
||||||
|
/// A cached Telegram file id failed permanently (stale/expired); drop the
|
||||||
|
/// cache entry so the next request re-fetches instead of repeating it.
|
||||||
|
async fn invalidate_cache(cache: &LinkCache, task: &Task) {
|
||||||
|
if task.is_cached_send()
|
||||||
|
&& let Some(url) = task.source_url()
|
||||||
|
&& let Some(key) = x_media::site::cache_key(url)
|
||||||
|
{
|
||||||
|
log::debug!("removing stale link cache entry for [key={}]", log_key(url));
|
||||||
|
cache.remove(&key).await;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Locally produced media files (ugoira MP4, bsky remux MP4) whose temp dirs
|
||||||
|
/// must stay alive while their task may be retried by the queue. The fetch
|
||||||
|
/// pipeline hands ownership here via
|
||||||
|
/// [`x_media::site::Fetched::take_keep_alive`] before that
|
||||||
|
/// [`x_media::site::Fetched`] is dropped; a queued retry runs after that drop,
|
||||||
|
/// so without this the local file would be gone by the time the retry sends
|
||||||
|
/// it. Entries are removed when the task settles (see [`release_keep_alive`]).
|
||||||
|
pub(crate) static KEEP_ALIVE: LazyLock<parking_lot::Mutex<Vec<tempfile::TempDir>>> =
|
||||||
|
LazyLock::new(|| parking_lot::Mutex::new(Vec::new()));
|
||||||
|
|
||||||
|
/// Drops the keep-alive temp dirs holding media referenced by `task` (matched
|
||||||
|
/// by path prefix). Called once a task settles — sent or permanently failed —
|
||||||
|
/// so retry-only temp files do not leak; retryable tasks keep them alive.
|
||||||
|
pub(crate) fn release_keep_alive(task: &Task) {
|
||||||
|
let paths = task.local_media_paths();
|
||||||
|
if paths.is_empty() {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
let mut alive = KEEP_ALIVE.lock();
|
||||||
|
alive.retain(|dir| {
|
||||||
|
let dir_path = dir.path();
|
||||||
|
!paths.iter().any(|p| p.starts_with(dir_path))
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
/// One button per template name (column layout), then the confirm button.
|
||||||
|
/// Sorted by name: the templates live in a `HashMap`, so an unsorted walk
|
||||||
|
/// would reshuffle the buttons between prompts.
|
||||||
|
pub(super) fn build_edit_markup(templates: &HashMap<String, String>) -> InlineKeyboardMarkup {
|
||||||
|
let mut names: Vec<&String> = templates.keys().collect();
|
||||||
|
names.sort();
|
||||||
|
let mut rows = Vec::with_capacity(names.len() + 1);
|
||||||
|
for name in names {
|
||||||
|
rows.push(vec![InlineKeyboardButton::callback(
|
||||||
|
name.clone(),
|
||||||
|
format!("template|{name}"),
|
||||||
|
)]);
|
||||||
|
}
|
||||||
|
rows.push(vec![InlineKeyboardButton::callback(
|
||||||
|
"↩️ Confirm",
|
||||||
|
"forward",
|
||||||
|
)]);
|
||||||
|
InlineKeyboardMarkup::new(rows)
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Notifies a chat about a dead-lettered task (skips when `notify_chat_id` is
|
||||||
|
/// absent).
|
||||||
|
pub(super) async fn notify_failure(
|
||||||
|
sender: &dyn MediaSender,
|
||||||
|
chat_id: Option<i64>,
|
||||||
|
message_id: Option<i64>,
|
||||||
|
message: &str,
|
||||||
|
) {
|
||||||
|
let Some(chat_id) = chat_id else { return };
|
||||||
|
let reply_to = message_id.map(|id| MessageId(id as i32));
|
||||||
|
if let Err(e) = sender
|
||||||
|
.send_message(ChatId(chat_id), message.to_string(), reply_to, None)
|
||||||
|
.await
|
||||||
|
{
|
||||||
|
log::error!("failed to notify about failed task: {e}");
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// After a successful send: either open the edit-before-forward prompt or
|
||||||
|
/// forward to the configured channel (with retry/queue handling).
|
||||||
|
pub(crate) async fn post_send_actions(ctx: &AppContext<'_>, task: &Task, message_ids: Vec<i64>) {
|
||||||
|
let (
|
||||||
|
chat_id,
|
||||||
|
reply_to,
|
||||||
|
source_url,
|
||||||
|
edit_before_forward,
|
||||||
|
forward_channel_id,
|
||||||
|
notify_chat_id,
|
||||||
|
notify_message_id,
|
||||||
|
) = match task {
|
||||||
|
Task::SendMediaSequence {
|
||||||
|
chat_id,
|
||||||
|
reply_to_message_id,
|
||||||
|
source_url,
|
||||||
|
edit_before_forward,
|
||||||
|
forward_channel_id,
|
||||||
|
notify_chat_id,
|
||||||
|
notify_message_id,
|
||||||
|
..
|
||||||
|
}
|
||||||
|
| Task::SendAnimation {
|
||||||
|
chat_id,
|
||||||
|
reply_to_message_id,
|
||||||
|
source_url,
|
||||||
|
edit_before_forward,
|
||||||
|
forward_channel_id,
|
||||||
|
notify_chat_id,
|
||||||
|
notify_message_id,
|
||||||
|
..
|
||||||
|
} => (
|
||||||
|
*chat_id,
|
||||||
|
*reply_to_message_id,
|
||||||
|
source_url.clone(),
|
||||||
|
*edit_before_forward,
|
||||||
|
*forward_channel_id,
|
||||||
|
*notify_chat_id,
|
||||||
|
*notify_message_id,
|
||||||
|
),
|
||||||
|
Task::ForwardMessages { .. } => return,
|
||||||
|
};
|
||||||
|
|
||||||
|
if edit_before_forward {
|
||||||
|
let keyboard = build_edit_markup(&ctx.chat_store.get(chat_id).await.template);
|
||||||
|
let prompt = ctx
|
||||||
|
.sender
|
||||||
|
.send_message(
|
||||||
|
ChatId(chat_id),
|
||||||
|
"Reply to edit message.".to_string(),
|
||||||
|
Some(MessageId(reply_to as i32)),
|
||||||
|
Some(keyboard),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
match prompt {
|
||||||
|
Ok(prompt_id) => {
|
||||||
|
log::info!(
|
||||||
|
"edit-before-forward prompt {prompt_id} opened for {} message(s)",
|
||||||
|
message_ids.len()
|
||||||
|
);
|
||||||
|
let source_url = source_url.clone();
|
||||||
|
ctx.chat_store
|
||||||
|
.update(chat_id, move |data| {
|
||||||
|
data.edit_message.insert(
|
||||||
|
prompt_id,
|
||||||
|
EditMessage {
|
||||||
|
url: source_url,
|
||||||
|
chat_id,
|
||||||
|
forward_message_ids: message_ids,
|
||||||
|
template: String::new(),
|
||||||
|
created_at: unix_now(),
|
||||||
|
},
|
||||||
|
);
|
||||||
|
})
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
Err(e) => log::error!("failed to send edit prompt: {e}"),
|
||||||
|
}
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if let Some(channel_id) = forward_channel_id {
|
||||||
|
log::info!(
|
||||||
|
"forwarding {} message(s) to channel {channel_id}",
|
||||||
|
message_ids.len()
|
||||||
|
);
|
||||||
|
let forward_task = Task::ForwardMessages {
|
||||||
|
from_chat_id: chat_id,
|
||||||
|
to_chat_id: channel_id,
|
||||||
|
message_ids,
|
||||||
|
notify_chat_id,
|
||||||
|
notify_message_id,
|
||||||
|
};
|
||||||
|
match forward_messages(ctx, &forward_task).await {
|
||||||
|
Ok(()) => {}
|
||||||
|
Err(SendError::Retryable {
|
||||||
|
delay_seconds,
|
||||||
|
task,
|
||||||
|
}) => {
|
||||||
|
enqueue_retry(ctx.task_queue, *task, delay_seconds).await;
|
||||||
|
}
|
||||||
|
Err(SendError::Permanent { message, .. }) => {
|
||||||
|
notify_failure(
|
||||||
|
ctx.sender,
|
||||||
|
notify_chat_id,
|
||||||
|
notify_message_id,
|
||||||
|
&format!("Task failed after retries: {message}"),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Enqueues a task for a later attempt (retry / forward resume). When the
|
||||||
|
/// enqueue itself fails the task can never be sent again, so its keep-alive
|
||||||
|
/// temp media is released instead of leaking until process exit.
|
||||||
|
pub(crate) async fn enqueue_retry(queue: &PersistentTaskQueue, task: Task, delay_seconds: f64) {
|
||||||
|
let payload = serde_json::to_value(&task).expect("task serializes");
|
||||||
|
let run_after = now_f64() + delay_seconds;
|
||||||
|
if let Err(e) = queue.enqueue(payload, run_after).await {
|
||||||
|
log::error!("failed to enqueue retry: {e}");
|
||||||
|
release_keep_alive(&task);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Queue entry point: parses the stored task and dispatches.
|
||||||
|
pub(crate) async fn handle_task(
|
||||||
|
ctx: &AppContext<'_>,
|
||||||
|
payload: serde_json::Value,
|
||||||
|
) -> Result<(), QueueError> {
|
||||||
|
let task: Task = match serde_json::from_value(payload.clone()) {
|
||||||
|
Ok(task) => task,
|
||||||
|
Err(e) => {
|
||||||
|
return Err(QueueError::Permanent {
|
||||||
|
message: format!("invalid task payload: {e}"),
|
||||||
|
payload,
|
||||||
|
});
|
||||||
|
}
|
||||||
|
};
|
||||||
|
match task {
|
||||||
|
Task::SendMediaSequence { .. } | Task::SendAnimation { .. } => {
|
||||||
|
let message_ids = match send_media_or_animation(ctx, &task).await {
|
||||||
|
Ok(ids) => ids,
|
||||||
|
Err(SendError::Retryable {
|
||||||
|
delay_seconds,
|
||||||
|
task,
|
||||||
|
}) => {
|
||||||
|
return Err(QueueError::Retryable {
|
||||||
|
delay_seconds,
|
||||||
|
payload: serde_json::to_value(task).expect("task serializes"),
|
||||||
|
});
|
||||||
|
}
|
||||||
|
Err(SendError::Permanent { message, task }) => {
|
||||||
|
settle_task(ctx, &task, Settled::Failed).await;
|
||||||
|
return Err(QueueError::Permanent {
|
||||||
|
message,
|
||||||
|
payload: serde_json::to_value(task).expect("task serializes"),
|
||||||
|
});
|
||||||
|
}
|
||||||
|
};
|
||||||
|
// A task only reaches the queue after a failed send, so this
|
||||||
|
// successful run is the first time post_send_actions can fire —
|
||||||
|
// the fresh attempt failed before it ever got here. Run it
|
||||||
|
// unconditionally: `post_send_actions` executes once, after the
|
||||||
|
// whole sequence (every batch) completed, so the channel forward
|
||||||
|
// and the edit-before-forward prompt must not be lost just
|
||||||
|
// because the send needed a retry.
|
||||||
|
post_send_actions(ctx, &task, message_ids).await;
|
||||||
|
settle_task(ctx, &task, Settled::Sent).await;
|
||||||
|
Ok(())
|
||||||
|
}
|
||||||
|
Task::ForwardMessages { .. } => match forward_messages(ctx, &task).await {
|
||||||
|
Ok(()) => Ok(()),
|
||||||
|
Err(SendError::Retryable {
|
||||||
|
delay_seconds,
|
||||||
|
task,
|
||||||
|
}) => Err(QueueError::Retryable {
|
||||||
|
delay_seconds,
|
||||||
|
payload: serde_json::to_value(task).expect("task serializes"),
|
||||||
|
}),
|
||||||
|
Err(SendError::Permanent { message, task }) => {
|
||||||
|
settle_task(ctx, &task, Settled::Failed).await;
|
||||||
|
Err(QueueError::Permanent {
|
||||||
|
message,
|
||||||
|
payload: serde_json::to_value(task).expect("task serializes"),
|
||||||
|
})
|
||||||
|
}
|
||||||
|
},
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
async fn send_media_or_animation(ctx: &AppContext<'_>, task: &Task) -> Result<Vec<i64>, SendError> {
|
||||||
|
match task {
|
||||||
|
Task::SendMediaSequence { .. } => send_media_sequence(ctx, task).await,
|
||||||
|
Task::SendAnimation { .. } => send_animation(ctx, task).await,
|
||||||
|
Task::ForwardMessages { .. } => unreachable!(),
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Dead-letter callback wired to the queue in main: settles the task and
|
||||||
|
/// notifies its chat.
|
||||||
|
pub(crate) async fn dead_letter_notify(
|
||||||
|
ctx: &AppContext<'_>,
|
||||||
|
payload: serde_json::Value,
|
||||||
|
message: String,
|
||||||
|
) {
|
||||||
|
// A dead-lettered task never runs again, and the queue dead-letters retry
|
||||||
|
// exhaustion itself (the handler is not called again), so this is the only
|
||||||
|
// place that sees the final payload.
|
||||||
|
if let Ok(task) = serde_json::from_value::<Task>(payload.clone()) {
|
||||||
|
settle_task(ctx, &task, Settled::Failed).await;
|
||||||
|
}
|
||||||
|
let notify_chat_id = payload.get("notify_chat_id").and_then(|v| v.as_i64());
|
||||||
|
let notify_message_id = payload.get("notify_message_id").and_then(|v| v.as_i64());
|
||||||
|
notify_failure(
|
||||||
|
ctx.sender,
|
||||||
|
notify_chat_id,
|
||||||
|
notify_message_id,
|
||||||
|
&format!("Task failed after retries: {message}"),
|
||||||
|
)
|
||||||
|
.await;
|
||||||
|
}
|
||||||
@@ -0,0 +1,345 @@
|
|||||||
|
//! Download-and-reupload fallback: when Telegram cannot fetch a media URL
|
||||||
|
//! itself (hotlink protection), the bot downloads the file, shrinks photos
|
||||||
|
//! that exceed Telegram's limits and uploads the batch via multipart.
|
||||||
|
|
||||||
|
use super::input_media::{animation_media, input_file_for, item_url, photo_media, video_media};
|
||||||
|
use super::{MediaItemPayload, SendError, Task, classify_to_send_error, retry_delay_seconds};
|
||||||
|
use crate::media_sender::MediaSender;
|
||||||
|
use crate::photo::{self, MAX_UPLOAD_BYTES, PhotoPrep};
|
||||||
|
use teloxide::prelude::*;
|
||||||
|
use teloxide::types::{ChatId, InputFile, InputMedia, MessageId};
|
||||||
|
use tempfile::NamedTempFile;
|
||||||
|
use x_media::site::FetchError;
|
||||||
|
|
||||||
|
/// Infers a file extension from magic bytes so Telegram detects the mime type
|
||||||
|
/// on multipart uploads.
|
||||||
|
pub(super) fn sniff_ext(bytes: &[u8]) -> &'static str {
|
||||||
|
if bytes.starts_with(&[0xFF, 0xD8]) {
|
||||||
|
"jpg"
|
||||||
|
} else if bytes.starts_with(b"\x89PNG") {
|
||||||
|
"png"
|
||||||
|
} else if bytes.starts_with(b"RIFF") && bytes.len() >= 12 && &bytes[8..12] == b"WEBP" {
|
||||||
|
"webp"
|
||||||
|
} else if bytes.starts_with(b"GIF8") {
|
||||||
|
"gif"
|
||||||
|
} else if bytes.len() >= 12 && &bytes[4..8] == b"ftyp" {
|
||||||
|
"mp4"
|
||||||
|
} else {
|
||||||
|
"bin"
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
pub(super) enum FallbackError {
|
||||||
|
Retryable {
|
||||||
|
delay_seconds: f64,
|
||||||
|
},
|
||||||
|
Permanent {
|
||||||
|
message: String,
|
||||||
|
},
|
||||||
|
/// The downloaded file exceeds the upload cap; the caller falls back to
|
||||||
|
/// the item's smaller URL.
|
||||||
|
MediaTooLarge,
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Brings a downloaded photo within Telegram's limits via the pure-Rust
|
||||||
|
/// chain in [`crate::photo`] (no ffmpeg): dimension cap / upload cap
|
||||||
|
/// exceeded photos are decoded, downscaled with Lanczos3, PNG bit depth
|
||||||
|
/// reduced (>24-bit → 24-bit RGB, ≤24-bit untouched) and transcoded to JPEG
|
||||||
|
/// only if still too big. Anything that cannot be fixed falls back to the
|
||||||
|
/// item's smaller URL.
|
||||||
|
///
|
||||||
|
/// Downloads one media item to a temp file (deleted on drop), returning the
|
||||||
|
/// file plus the downloaded bytes (photos keep the bytes for
|
||||||
|
/// [`photo::prepare_photo`] — re-reading the file would double the I/O).
|
||||||
|
/// Network errors are retryable; size over the upload cap and other download
|
||||||
|
/// errors are not.
|
||||||
|
async fn download_to_temp(
|
||||||
|
item: &MediaItemPayload,
|
||||||
|
) -> Result<(NamedTempFile, bytes::Bytes), FallbackError> {
|
||||||
|
let media_url = match item {
|
||||||
|
MediaItemPayload::Photo { media, .. }
|
||||||
|
| MediaItemPayload::Video { media, .. }
|
||||||
|
| MediaItemPayload::Animation { media, .. } => media,
|
||||||
|
};
|
||||||
|
// Photos are downloaded even over the upload cap so `prepare_photo` can
|
||||||
|
// downscale / transcode them (cap = decode budget); videos/animations
|
||||||
|
// abort as soon as the upload cap is crossed mid-stream.
|
||||||
|
let limit = if matches!(item, MediaItemPayload::Photo { .. }) {
|
||||||
|
photo::MAX_DECODE_BYTES
|
||||||
|
} else {
|
||||||
|
MAX_UPLOAD_BYTES + 1
|
||||||
|
};
|
||||||
|
let bytes = match x_media::site::download_media_limited(media_url, limit).await {
|
||||||
|
Ok(bytes) => bytes,
|
||||||
|
Err(FetchError::Http(_)) => {
|
||||||
|
return Err(FallbackError::Retryable {
|
||||||
|
delay_seconds: retry_delay_seconds(0),
|
||||||
|
});
|
||||||
|
}
|
||||||
|
Err(FetchError::TooLarge) => {
|
||||||
|
return Err(FallbackError::MediaTooLarge);
|
||||||
|
}
|
||||||
|
Err(e) => {
|
||||||
|
return Err(FallbackError::Permanent {
|
||||||
|
message: format!("download failed: {e}"),
|
||||||
|
});
|
||||||
|
}
|
||||||
|
};
|
||||||
|
let ext = sniff_ext(&bytes);
|
||||||
|
let mut file = tempfile::Builder::new()
|
||||||
|
.suffix(&format!(".{ext}"))
|
||||||
|
.tempfile()
|
||||||
|
.map_err(|e| FallbackError::Permanent {
|
||||||
|
message: format!("temp file failed: {e}"),
|
||||||
|
})?;
|
||||||
|
use std::io::Write;
|
||||||
|
file.as_file_mut()
|
||||||
|
.write_all(&bytes)
|
||||||
|
.map_err(|e| FallbackError::Permanent {
|
||||||
|
message: format!("temp file write failed: {e}"),
|
||||||
|
})?;
|
||||||
|
Ok((file, bytes))
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Builds the media group item from an uploaded file.
|
||||||
|
fn media_from_file(
|
||||||
|
item: &MediaItemPayload,
|
||||||
|
path: std::path::PathBuf,
|
||||||
|
caption: Option<&str>,
|
||||||
|
thumbnail: Option<&str>,
|
||||||
|
) -> Result<InputMedia, String> {
|
||||||
|
let mut media = match item {
|
||||||
|
MediaItemPayload::Photo { has_spoiler, .. } => {
|
||||||
|
photo_media(InputFile::file(path), caption, *has_spoiler)
|
||||||
|
}
|
||||||
|
MediaItemPayload::Video { has_spoiler, .. } => {
|
||||||
|
video_media(InputFile::file(path), caption, *has_spoiler)
|
||||||
|
}
|
||||||
|
MediaItemPayload::Animation { has_spoiler, .. } => {
|
||||||
|
animation_media(InputFile::file(path), caption, *has_spoiler)
|
||||||
|
}
|
||||||
|
};
|
||||||
|
if let (Some(thumb), InputMedia::Video(v)) = (thumbnail, &mut media) {
|
||||||
|
*v = v.clone().thumbnail(input_file_for(thumb)?);
|
||||||
|
}
|
||||||
|
Ok(media)
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Builds the media group item from a (smaller) URL.
|
||||||
|
fn media_from_url(
|
||||||
|
item: &MediaItemPayload,
|
||||||
|
url: &str,
|
||||||
|
caption: Option<&str>,
|
||||||
|
thumbnail: Option<&str>,
|
||||||
|
) -> Result<InputMedia, String> {
|
||||||
|
let mut media = match item {
|
||||||
|
MediaItemPayload::Photo { has_spoiler, .. } => {
|
||||||
|
photo_media(input_file_for(url)?, caption, *has_spoiler)
|
||||||
|
}
|
||||||
|
MediaItemPayload::Video { has_spoiler, .. } => {
|
||||||
|
video_media(input_file_for(url)?, caption, *has_spoiler)
|
||||||
|
}
|
||||||
|
MediaItemPayload::Animation { has_spoiler, .. } => {
|
||||||
|
animation_media(input_file_for(url)?, caption, *has_spoiler)
|
||||||
|
}
|
||||||
|
};
|
||||||
|
if let (Some(thumb), InputMedia::Video(v)) = (thumbnail, &mut media) {
|
||||||
|
*v = v.clone().thumbnail(input_file_for(thumb)?);
|
||||||
|
}
|
||||||
|
Ok(media)
|
||||||
|
}
|
||||||
|
|
||||||
|
/// One item prepared for the upload fallback: the ready-to-send media plus
|
||||||
|
/// the temp file that must stay on disk until the group request completes.
|
||||||
|
pub(super) struct PreparedItem {
|
||||||
|
/// Original position in the batch (concurrent prep completes out of order).
|
||||||
|
pub(super) index: usize,
|
||||||
|
pub(super) media: InputMedia,
|
||||||
|
pub(super) keep_alive: Option<NamedTempFile>,
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Downloads / processes one media item for the upload fallback (see
|
||||||
|
/// [`send_batch_via_upload`]). Local files are uploaded directly; oversized
|
||||||
|
/// items fall back to their smaller URL; photos are downscaled/transcoded.
|
||||||
|
pub(super) async fn prepare_upload_item(
|
||||||
|
item: MediaItemPayload,
|
||||||
|
index: usize,
|
||||||
|
caption: Option<&str>,
|
||||||
|
) -> Result<PreparedItem, FallbackError> {
|
||||||
|
// Locally produced files (ugoira / bsky remux MP4): nothing to download
|
||||||
|
// or shrink — upload the file directly. The send is a multipart upload,
|
||||||
|
// so the only remaining failure is an upload-cap error, which is
|
||||||
|
// permanent (a video cannot be re-encoded here).
|
||||||
|
let media_url = item_url(&item);
|
||||||
|
if !media_url.starts_with("http://") && !media_url.starts_with("https://") {
|
||||||
|
let media = media_from_file(
|
||||||
|
&item,
|
||||||
|
std::path::PathBuf::from(media_url),
|
||||||
|
caption,
|
||||||
|
item.thumbnail_url(),
|
||||||
|
)
|
||||||
|
.map_err(|message| FallbackError::Permanent { message })?;
|
||||||
|
return Ok(PreparedItem {
|
||||||
|
index,
|
||||||
|
media,
|
||||||
|
keep_alive: None,
|
||||||
|
});
|
||||||
|
}
|
||||||
|
// Size check before downloading/uploading: over the cap, use the
|
||||||
|
// smaller URL instead of the file. Photos are exempt — they are
|
||||||
|
// downloaded and processed (downscale / PNG→JPEG) before uploading.
|
||||||
|
let too_large = match x_media::site::media_size(media_url).await {
|
||||||
|
Ok(Some(size)) => size > MAX_UPLOAD_BYTES,
|
||||||
|
_ => false,
|
||||||
|
};
|
||||||
|
let too_large = too_large && !matches!(item, MediaItemPayload::Photo { .. });
|
||||||
|
if too_large {
|
||||||
|
let url = item
|
||||||
|
.fallback_url()
|
||||||
|
.ok_or_else(|| FallbackError::Permanent {
|
||||||
|
message: "media too large".into(),
|
||||||
|
})?;
|
||||||
|
let media = media_from_url(&item, url, caption, item.thumbnail_url())
|
||||||
|
.map_err(|message| FallbackError::Permanent { message })?;
|
||||||
|
return Ok(PreparedItem {
|
||||||
|
index,
|
||||||
|
media,
|
||||||
|
keep_alive: None,
|
||||||
|
});
|
||||||
|
}
|
||||||
|
match download_to_temp(&item).await {
|
||||||
|
Ok((file, bytes)) => {
|
||||||
|
if matches!(item, MediaItemPayload::Photo { .. }) {
|
||||||
|
// Telegram rejects photos wider+taller than 10000 px combined
|
||||||
|
// (PHOTO_INVALID_DIMENSIONS): downscale the downloaded file
|
||||||
|
// before uploading; photos that cannot be brought within the
|
||||||
|
// limits degrade to the smaller URL. CPU-heavy work runs off
|
||||||
|
// the async executor thread.
|
||||||
|
let prep = tokio::task::spawn_blocking(move || photo::prepare_photo(file, &bytes))
|
||||||
|
.await
|
||||||
|
.map_err(|e| FallbackError::Permanent {
|
||||||
|
message: format!("photo worker panicked: {e}"),
|
||||||
|
})?
|
||||||
|
.map_err(|message| FallbackError::Permanent { message })?;
|
||||||
|
match prep {
|
||||||
|
PhotoPrep::Upload(upload) => {
|
||||||
|
let path = upload.path().to_path_buf();
|
||||||
|
let media = media_from_file(&item, path, caption, item.thumbnail_url())
|
||||||
|
.map_err(|message| FallbackError::Permanent { message })?;
|
||||||
|
Ok(PreparedItem {
|
||||||
|
index,
|
||||||
|
media,
|
||||||
|
keep_alive: Some(upload),
|
||||||
|
})
|
||||||
|
}
|
||||||
|
PhotoPrep::UseFallback => {
|
||||||
|
let url = item.fallback_url().ok_or_else(|| FallbackError::Permanent {
|
||||||
|
message: "photo dimensions exceed Telegram limits and no smaller variant is available"
|
||||||
|
.into(),
|
||||||
|
})?;
|
||||||
|
let media = media_from_url(&item, url, caption, item.thumbnail_url())
|
||||||
|
.map_err(|message| FallbackError::Permanent { message })?;
|
||||||
|
Ok(PreparedItem {
|
||||||
|
index,
|
||||||
|
media,
|
||||||
|
keep_alive: None,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
let path = file.path().to_path_buf();
|
||||||
|
let media = media_from_file(&item, path, caption, item.thumbnail_url())
|
||||||
|
.map_err(|message| FallbackError::Permanent { message })?;
|
||||||
|
Ok(PreparedItem {
|
||||||
|
index,
|
||||||
|
media,
|
||||||
|
keep_alive: Some(file),
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
Err(FallbackError::MediaTooLarge) => {
|
||||||
|
let url = item
|
||||||
|
.fallback_url()
|
||||||
|
.ok_or_else(|| FallbackError::Permanent {
|
||||||
|
message: "media too large".into(),
|
||||||
|
})?;
|
||||||
|
let media = media_from_url(&item, url, caption, item.thumbnail_url())
|
||||||
|
.map_err(|message| FallbackError::Permanent { message })?;
|
||||||
|
Ok(PreparedItem {
|
||||||
|
index,
|
||||||
|
media,
|
||||||
|
keep_alive: None,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
Err(e) => Err(e),
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/// Download-and-reupload fallback for one media batch. Files over the upload
|
||||||
|
/// cap are not downloaded/uploaded; the item falls back to its smaller URL
|
||||||
|
/// (which Telegram fetches itself). Items are prepared concurrently (bounded)
|
||||||
|
/// because the downloads are network-bound; the batch is then uploaded in its
|
||||||
|
/// original order. Returns the fallback-error without the task attached;
|
||||||
|
/// callers wrap it with the updated task state.
|
||||||
|
pub(super) async fn send_batch_via_upload(
|
||||||
|
sender: &dyn MediaSender,
|
||||||
|
chat_id: i64,
|
||||||
|
reply_to: i64,
|
||||||
|
batch: &[MediaItemPayload],
|
||||||
|
caption: Option<&str>,
|
||||||
|
task: Task,
|
||||||
|
) -> Result<Vec<Message>, SendError> {
|
||||||
|
let sem = std::sync::Arc::new(tokio::sync::Semaphore::new(3));
|
||||||
|
let mut set = tokio::task::JoinSet::new();
|
||||||
|
for (i, item) in batch.iter().enumerate() {
|
||||||
|
let item_caption = if i == 0 {
|
||||||
|
caption.map(str::to_string)
|
||||||
|
} else {
|
||||||
|
None
|
||||||
|
};
|
||||||
|
let item = item.clone();
|
||||||
|
let sem = std::sync::Arc::clone(&sem);
|
||||||
|
set.spawn(async move {
|
||||||
|
let _permit = sem.acquire().await.expect("upload semaphore closed");
|
||||||
|
prepare_upload_item(item, i, item_caption.as_deref()).await
|
||||||
|
});
|
||||||
|
}
|
||||||
|
let mut prepared: Vec<Option<InputMedia>> = (0..batch.len()).map(|_| None).collect();
|
||||||
|
let mut keep_alive: Vec<NamedTempFile> = Vec::new();
|
||||||
|
while let Some(joined) = set.join_next().await {
|
||||||
|
let item = match joined {
|
||||||
|
Ok(Ok(item)) => item,
|
||||||
|
// Dropping the JoinSet aborts the remaining prep tasks; their
|
||||||
|
// temp files are cleaned up on drop (short-circuit like before).
|
||||||
|
Ok(Err(e)) => return Err(SendError::from_fallback(e, task.clone())),
|
||||||
|
Err(e) => {
|
||||||
|
return Err(SendError::Permanent {
|
||||||
|
message: format!("upload worker panicked: {e}"),
|
||||||
|
task: Box::new(task),
|
||||||
|
});
|
||||||
|
}
|
||||||
|
};
|
||||||
|
let PreparedItem {
|
||||||
|
index,
|
||||||
|
media,
|
||||||
|
keep_alive: file_opt,
|
||||||
|
} = item;
|
||||||
|
if let Some(file) = file_opt {
|
||||||
|
keep_alive.push(file);
|
||||||
|
}
|
||||||
|
prepared[index] = Some(media);
|
||||||
|
}
|
||||||
|
let items: Vec<InputMedia> = prepared
|
||||||
|
.into_iter()
|
||||||
|
.map(|m| m.expect("every upload item was prepared"))
|
||||||
|
.collect();
|
||||||
|
// `keep_alive` holds the temp files until the group request completes.
|
||||||
|
let result = sender
|
||||||
|
.send_media_group(ChatId(chat_id), MessageId(reply_to as i32), items)
|
||||||
|
.await;
|
||||||
|
drop(keep_alive);
|
||||||
|
match result {
|
||||||
|
Ok(messages) => Ok(messages),
|
||||||
|
Err(e) => Err(classify_to_send_error(&e, task, "upload failed")),
|
||||||
|
}
|
||||||
|
}
|
||||||
+114
-53
@@ -1,12 +1,13 @@
|
|||||||
//! Per-chat state with SQLite persistence (table `chat_state` in
|
//! Per-chat state with SQLite persistence (table `chat_state` in
|
||||||
//! `data/task_queue.db`, shared with the task queue).
|
//! `data/task_queue.db`, shared with the task queue).
|
||||||
|
|
||||||
|
use crate::db::unix_now;
|
||||||
use parking_lot::Mutex;
|
use parking_lot::Mutex;
|
||||||
use rusqlite::params;
|
use rusqlite::params;
|
||||||
use serde::{Deserialize, Serialize};
|
use serde::{Deserialize, Serialize};
|
||||||
use std::collections::HashMap;
|
use std::collections::HashMap;
|
||||||
use std::sync::Arc;
|
use std::sync::Arc;
|
||||||
use std::time::{Duration, SystemTime, UNIX_EPOCH};
|
use std::time::Duration;
|
||||||
|
|
||||||
#[derive(Serialize, Deserialize, Default, Clone, Debug)]
|
#[derive(Serialize, Deserialize, Default, Clone, Debug)]
|
||||||
pub struct ChatData {
|
pub struct ChatData {
|
||||||
@@ -16,8 +17,8 @@ pub struct ChatData {
|
|||||||
pub edit_message: HashMap<i64, EditMessage>,
|
pub edit_message: HashMap<i64, EditMessage>,
|
||||||
/// name -> HTML template containing "[]"
|
/// name -> HTML template containing "[]"
|
||||||
pub template: HashMap<String, String>,
|
pub template: HashMap<String, String>,
|
||||||
/// site name (twitter/bsky/pixiv) -> user-supplied caption format with
|
/// site name (twitter/bsky/misskey/pixiv) -> user-supplied caption format
|
||||||
/// {url} {author} {author_url} {title} {tags} placeholders.
|
/// with {url} {author} {author_url} {title} {tags} placeholders.
|
||||||
pub message_format: HashMap<String, String>,
|
pub message_format: HashMap<String, String>,
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -40,13 +41,6 @@ pub struct ChatStore {
|
|||||||
pool: Arc<crate::db::DbPool>,
|
pool: Arc<crate::db::DbPool>,
|
||||||
}
|
}
|
||||||
|
|
||||||
pub fn unix_now() -> i64 {
|
|
||||||
SystemTime::now()
|
|
||||||
.duration_since(UNIX_EPOCH)
|
|
||||||
.map(|d| d.as_secs() as i64)
|
|
||||||
.unwrap_or(0)
|
|
||||||
}
|
|
||||||
|
|
||||||
impl ChatStore {
|
impl ChatStore {
|
||||||
/// Wraps the shared DB pool (schema initialized once by
|
/// Wraps the shared DB pool (schema initialized once by
|
||||||
/// [`crate::db::open_store`]; the `chat_state` table lives in the merged
|
/// [`crate::db::open_store`]; the `chat_state` table lives in the merged
|
||||||
@@ -108,19 +102,22 @@ impl ChatStore {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// The per-chat async lock serializing get→mutate→set cycles.
|
||||||
|
fn lock_for(&self, chat_id: i64) -> Arc<tokio::sync::Mutex<()>> {
|
||||||
|
self.locks
|
||||||
|
.lock()
|
||||||
|
.entry(chat_id)
|
||||||
|
.or_insert_with(|| Arc::new(tokio::sync::Mutex::new(())))
|
||||||
|
.clone()
|
||||||
|
}
|
||||||
|
|
||||||
/// Serializes a get→mutate→set cycle per chat: concurrent handler tasks
|
/// Serializes a get→mutate→set cycle per chat: concurrent handler tasks
|
||||||
/// (the batch-forward design spawns several per chat) each snapshot the
|
/// (the batch-forward design spawns several per chat) each snapshot the
|
||||||
/// same `ChatData` and last-writer-wins would silently drop mutations,
|
/// same `ChatData` and last-writer-wins would silently drop mutations,
|
||||||
/// e.g. a second `edit_message` record. The per-chat lock makes the
|
/// e.g. a second `edit_message` record. The per-chat lock makes the
|
||||||
/// cycle atomic. Returns the closure's result.
|
/// cycle atomic. Returns the closure's result.
|
||||||
pub async fn update<R>(&self, chat_id: i64, f: impl FnOnce(&mut ChatData) -> R) -> R {
|
pub async fn update<R>(&self, chat_id: i64, f: impl FnOnce(&mut ChatData) -> R) -> R {
|
||||||
let lock = {
|
let lock = self.lock_for(chat_id);
|
||||||
let mut locks = self.locks.lock();
|
|
||||||
locks
|
|
||||||
.entry(chat_id)
|
|
||||||
.or_insert_with(|| Arc::new(tokio::sync::Mutex::new(())))
|
|
||||||
.clone()
|
|
||||||
};
|
|
||||||
let _guard = lock.lock().await;
|
let _guard = lock.lock().await;
|
||||||
let mut data = self.get(chat_id).await;
|
let mut data = self.get(chat_id).await;
|
||||||
let r = f(&mut data);
|
let r = f(&mut data);
|
||||||
@@ -134,43 +131,45 @@ impl ChatStore {
|
|||||||
pub async fn prune_expired(&self, ttl: Duration) -> Vec<(i64, i64)> {
|
pub async fn prune_expired(&self, ttl: Duration) -> Vec<(i64, i64)> {
|
||||||
let now = unix_now();
|
let now = unix_now();
|
||||||
let ttl_secs = ttl.as_secs() as i64;
|
let ttl_secs = ttl.as_secs() as i64;
|
||||||
let mut removed = Vec::new();
|
// Chats that may have an expired record, from a cache snapshot; the
|
||||||
// Chats with no live edit records: evicted from the cache (and their
|
// pruning itself re-reads and writes under the per-chat lock below
|
||||||
// per-chat lock) so the cache stays bounded to active prompts. The DB
|
// (see the eviction note). Takes no lock of its own, so a chat
|
||||||
// keeps the row; the next get() reloads it.
|
// appearing later is simply picked up by the next sweep.
|
||||||
let mut evicted_chats = Vec::new();
|
let candidates: Vec<i64> = {
|
||||||
let changed: Vec<(i64, ChatData)> = {
|
let cache = self.cache.lock();
|
||||||
let mut cache = self.cache.lock();
|
cache
|
||||||
let mut out = Vec::new();
|
.iter()
|
||||||
for (chat_id, data) in cache.iter_mut() {
|
.filter(|(_, data)| {
|
||||||
let keys: Vec<i64> = data.edit_message.keys().copied().collect();
|
data.edit_message
|
||||||
let mut kept = HashMap::new();
|
.values()
|
||||||
for key in keys {
|
.any(|entry| entry.created_at + ttl_secs <= now)
|
||||||
if let Some(entry) = data.edit_message.get(&key) {
|
})
|
||||||
if entry.created_at + ttl_secs > now {
|
.map(|(chat_id, _)| *chat_id)
|
||||||
kept.insert(key, entry.clone());
|
.collect()
|
||||||
} else {
|
|
||||||
removed.push((*chat_id, key));
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
if kept.len() != data.edit_message.len() {
|
|
||||||
// Persist the pruned row (removes expired records from
|
|
||||||
// the DB too, not just the cache).
|
|
||||||
data.edit_message = kept;
|
|
||||||
out.push((*chat_id, data.clone()));
|
|
||||||
}
|
|
||||||
if data.edit_message.is_empty() {
|
|
||||||
evicted_chats.push(*chat_id);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
// Lock order: update() takes the per-chat lock before the cache
|
|
||||||
// lock, so prune must not hold the cache lock while taking locks.
|
|
||||||
drop(cache);
|
|
||||||
out
|
|
||||||
};
|
};
|
||||||
for (chat_id, data) in changed {
|
let mut removed = Vec::new();
|
||||||
self.set(chat_id, &data).await;
|
let mut evicted_chats = Vec::new();
|
||||||
|
for chat_id in candidates {
|
||||||
|
let lock = self.lock_for(chat_id);
|
||||||
|
let _guard = lock.lock().await;
|
||||||
|
let mut data = self.get(chat_id).await;
|
||||||
|
let before = data.edit_message.len();
|
||||||
|
data.edit_message.retain(|key, entry| {
|
||||||
|
if entry.created_at + ttl_secs > now {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
removed.push((chat_id, *key));
|
||||||
|
false
|
||||||
|
});
|
||||||
|
if data.edit_message.len() != before {
|
||||||
|
self.set(chat_id, &data).await;
|
||||||
|
}
|
||||||
|
// Chats with no live edit records: evicted from the cache (and
|
||||||
|
// their per-chat lock) so the cache stays bounded to active
|
||||||
|
// prompts. The DB keeps the row; the next get() reloads it.
|
||||||
|
if data.edit_message.is_empty() {
|
||||||
|
evicted_chats.push(chat_id);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
if !evicted_chats.is_empty() {
|
if !evicted_chats.is_empty() {
|
||||||
let mut cache = self.cache.lock();
|
let mut cache = self.cache.lock();
|
||||||
@@ -229,4 +228,66 @@ mod tests {
|
|||||||
"concurrent get→mutate→set must not drop records"
|
"concurrent get→mutate→set must not drop records"
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn edit_entry(chat_id: i64, created_at: i64) -> EditMessage {
|
||||||
|
EditMessage {
|
||||||
|
url: "https://x.com/u/status/1".into(),
|
||||||
|
chat_id,
|
||||||
|
forward_message_ids: vec![9],
|
||||||
|
template: String::new(),
|
||||||
|
created_at,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn prune_removes_only_expired_records() {
|
||||||
|
let dir = tempfile::tempdir().unwrap();
|
||||||
|
let pool = crate::db::open_store(dir.path().join("p.db").to_str().unwrap()).unwrap();
|
||||||
|
let store = ChatStore::new(pool);
|
||||||
|
let now = unix_now();
|
||||||
|
store
|
||||||
|
.update(7, |data| {
|
||||||
|
data.template.insert("t".into(), "[]".into());
|
||||||
|
data.edit_message.insert(1, edit_entry(7, now - 3600));
|
||||||
|
data.edit_message.insert(2, edit_entry(7, now));
|
||||||
|
})
|
||||||
|
.await;
|
||||||
|
|
||||||
|
let removed = store.prune_expired(Duration::from_secs(60)).await;
|
||||||
|
|
||||||
|
assert_eq!(removed, vec![(7, 1)]);
|
||||||
|
let data = store.get(7).await;
|
||||||
|
assert!(data.edit_message.contains_key(&2), "live record pruned");
|
||||||
|
assert_eq!(
|
||||||
|
data.template.get("t").map(String::as_str),
|
||||||
|
Some("[]"),
|
||||||
|
"unrelated state lost by the prune"
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
#[tokio::test]
|
||||||
|
async fn prune_eviction_keeps_the_persisted_state() {
|
||||||
|
// Every record expires → the chat is evicted from the cache; the
|
||||||
|
// pruned state must already be in the DB when that happens.
|
||||||
|
let dir = tempfile::tempdir().unwrap();
|
||||||
|
let pool = crate::db::open_store(dir.path().join("p.db").to_str().unwrap()).unwrap();
|
||||||
|
let store = ChatStore::new(pool);
|
||||||
|
store
|
||||||
|
.update(8, |data| {
|
||||||
|
data.template.insert("keep".into(), "[]".into());
|
||||||
|
data.edit_message.insert(1, edit_entry(8, 0));
|
||||||
|
})
|
||||||
|
.await;
|
||||||
|
|
||||||
|
let removed = store.prune_expired(Duration::from_secs(60)).await;
|
||||||
|
|
||||||
|
assert_eq!(removed, vec![(8, 1)]);
|
||||||
|
let data = store.get(8).await;
|
||||||
|
assert!(data.edit_message.is_empty());
|
||||||
|
assert_eq!(
|
||||||
|
data.template.get("keep").map(String::as_str),
|
||||||
|
Some("[]"),
|
||||||
|
"eviction dropped state the DB never received"
|
||||||
|
);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -10,6 +10,10 @@
|
|||||||
|
|
||||||
## 1. 现状摩擦清单
|
## 1. 现状摩擦清单
|
||||||
|
|
||||||
|
> ⚠️ 本节记录的是**重构前**的现状:其中的行号、以及 `site/mod.rs` 里的
|
||||||
|
> `fetch_once` 分派函数(当时的实现)都已不存在,仅作历史记录。当前形态见
|
||||||
|
> `site/mod.rs` 的 `SITES` 注册表——新增站点 = 新模块 + 注册一行。
|
||||||
|
|
||||||
以现有三站(twitter / bsky / pixiv)为基线,新增第 4 个站点(代号 `example`)
|
以现有三站(twitter / bsky / pixiv)为基线,新增第 4 个站点(代号 `example`)
|
||||||
今天需要触碰的位置:
|
今天需要触碰的位置:
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user