作品投稿時のSlack複数チャンネル投稿を並列化

[utgwkk]

概要

作品投稿時に複数のSlackチャンネルへ直列で投稿していた処理を、ThreadPoolExecutor を使って並列化しました。

変更の背景

複数チャンネルを指定して投稿する場合、チャンネル数に比例してレスポンスタイムが増加していました。各チャンネルへの投稿は互いに依存しないため、並列化によって待ち時間を短縮できます。

変更内容

api/src/goduploader/graphql/type/mutation/upload_artwork.py

• concurrent.futures.ThreadPoolExecutor をインポート
• 直列の for ループを ThreadPoolExecutor + executor.map() に置き換え

# 変更前
for ch_id in channel_ids:
    try:
        share_to_slack(artwork, top_illust.image_path("full"), share_option, ch_id)
    except Exception as e:
        logging.error(e)

# 変更後
image_path = top_illust.image_path("full")

def _post_to_channel(ch_id):
    try:
        share_to_slack(artwork, image_path, share_option, ch_id)
    except Exception as e:
        logging.error(e)

with ThreadPoolExecutor() as executor:
    executor.map(_post_to_channel, channel_ids)

注意点

• executor.map() はすべてのスレッドの完了を待ってからリターンするため、後続の Twitter 投稿処理への影響はありません
• チャンネルごとのエラーハンドリング（ログ出力・例外握り潰し）は従来と同様に維持しています
• SHARE_TO_SLACK_WITH_IMAGE の場合、各チャンネルが個別に Gyazo へ画像をアップロードする挙動は変更前と同じです（並列になるだけ）

https://claude.ai/code/session_013vQq3dTD72krdJ1jDuzjQz

---

Generated by Claude Code

[utgwkk]

さすがにあんまりなので、share_to_slack に artwork そのものではなく必要な情報を列挙した引数 (あるいはDTO) を渡すような形にしてほしいナー

[utgwkk]

あとは、そもそも並列化する前の関数の呼び出し元でデータを予め用意しておく形になるのであれば、Gyazoへの画像のアップロードも最初の1回だけにするのが収まりよさそう