Skip to content

fix(speakerdeck-knowledge): 画像読解のコンテキスト消費を抑える - #102

Merged
9uiLe merged 1 commit into
masterfrom
fix/speakerdeck-image-context
Sep 14, 2026
Merged

9uiLe merged 1 commit into
masterfrom
fix/speakerdeck-image-context

Conversation

@9uiLe

@9uiLe 9uiLe commented Sep 11, 2026

Copy link
Copy Markdown
Owner

概要

SpeakerDeck の読解で全ページの画像確認を促す指示があり、画像の寸法や再読の基準もなかったため、AI セッションへ必要以上の画像が入る可能性がありました。本文から確認対象を選び、縮小画像・必要領域の切り出し・少数ページの俯瞰画像を使って根拠を確認する流れに変更します。

対象プラグイン

  • speakerdeck-knowledge(0.1.0 → 0.1.1)
  • リポジトリの不要ドキュメント整理

変更内容

  • 本文を先に読み、画像は未解決の問いに必要なページだけを開く。本文欠損や画像主体の資料は少数ページずつ確認し、未確認範囲を明記する。
  • prepare_images.py を追加。既定で長辺1280px以下のプレビュー、元画像からの切り出し、最大6ページのコンタクトシートを生成する。ページ指定は必須で、標準出力にはパス・ページ番号・寸法だけを返す。
  • 確認メモによって執筆中の再読を抑え、同じ元画像・変換条件の生成物と取得済み画像を再利用する。原画像と既存の取得状態を保持する。
  • HTML の画像埋め込みをローカルで処理し、base64 や埋め込み後の HTML 全文をセッションへ出力しない。表示確認は必要なビューポート・領域を対象にする。
  • 画像生成用の Pillow 依存を案内し、取得・画像生成の回帰テストを CI に追加する。Claude / Codex のマニフェストと marketplace のバージョンを揃える。
  • 利用者が削除した旧 ADR・表示用アセット・旧ハンドオフ指示書を同じコミットに含める。削除に伴う現行ドキュメントの参照も整理する。

動作確認

  • python3 -m unittest discover -s plugins/speakerdeck-knowledge/tests -v: 20件成功。画像の寸法、ページ順と分割、切り出し、元画像の保持、キャッシュ再利用、透明画像、失敗時の挙動を確認。
  • bash scripts/verify-versions.sh: 成功。
  • bash scripts/tests/verify-versions-completeness.test.sh: 6件成功。
  • quick_validate.py: スキル検証成功。
  • 合成した図入りスライド7ページで画像生成を実行し、6ページのシートを目視確認。シートは852×747pxでページ番号・図の順序を保持し、7ページ目は別画像になることを確認。
  • CI YAML の構文確認、git diff --check: 成功。

チェックリスト

  • 元画像・ページ別根拠・未確認範囲を保持する
  • Claude / Codex のマニフェストと marketplace のバージョンを更新した
  • README・読解手順・CHANGELOG を更新した

補足

実際のモデルによる一連の知識抽出と token 使用量の比較、ローカル marketplace への再インストールは未実施です。画像の枚数・寸法・再読を抑えるための変更であり、固定の token 削減率は示しません。

@9uiLe 9uiLe self-assigned this Sep 14, 2026
@9uiLe
9uiLe merged commit a719228 into master Sep 14, 2026
5 checks passed
@9uiLe
9uiLe deleted the fix/speakerdeck-image-context branch September 14, 2026 01:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant