Skip to content

feat(mcq): optional image/images preview in Gtk dialog - #31

Merged
ajlennon merged 1 commit into
mainfrom
feat/mcq-with-image
Aug 2, 2026
Merged

feat(mcq): optional image/images preview in Gtk dialog#31
ajlennon merged 1 commit into
mainfrom
feat/mcq-with-image

Conversation

@ajlennon

@ajlennon ajlennon commented Aug 2, 2026

Copy link
Copy Markdown
Member

Summary

  • Add optional image / images args on ask_multiple_choice so agents can show local PNG/JPEG stills in the Linux Gtk MCQ (scaled preview above the question).
  • Normalize path/file:// URIs via mcq_images.normalize_mcq_images (dedupe, cap 4, skip missing/unsupported).
  • Document in docs/AGENTS.md + skill; unit test scripts/test_mcq_images.py. Pattern: mcq-with-image.

Hosts / platforms touched

  • Cursor (Linux Gtk)
  • Claude Code
  • Claude Desktop / other stdio host
  • Windows Phase 1 (tkinter text-only)
  • Docs only (no behaviour change)

Test plan

CI must stay green (test, secrets-hygiene). Also say what you ran:

Check Result Notes
uv run python scripts/test_match_transcript.py Pass
uv run python scripts/test_doctor.py Pass
uv run python scripts/test_acks.py Pass
uv run python scripts/test_install.py Skip
uv run python scripts/test_dialog_ergonomics.py Skip
uv run python scripts/test_windows_backend.py Skip
uv run python scripts/test_contracts.py Pass
uv run python scripts/test_mcq_images.py Pass
check_setup in a real host Skip
ask_multiple_choice dialog click Pass Cursor MCP; Alex chose create PR
Image preview in Gtk dialog (image/images) Pass eth-rear-io.png smoke; Alex did not report broken
Keyboard 1–8 / Enter / Esc Skip
Freeform / Something else Skip
Speak / duck (if claiming voice) Skip text-only prefs

Environment: Ubuntu Linux, Cursor host, local DISPLAY :0, MCP dir /data_drive/dd/ask-question-mcp-mcq-image.

Behaviour contract

  • No removal/rename of MCP tools without a migration note (ask_multiple_choice, check_setup, setup_guide, record_platform_feedback)
  • Cursor absolute-uv docs kept (GUI thin PATH)
  • No lab IPs / home paths / secrets in tracked files
  • If adding a host: update Tested platforms table (Verified vs Not yet reported must match your results)

AI assistance

  • Human-authored
  • AI-assisted (Cursor) — verified unit tests locally; Alex confirmed smoke MCQ with image and approved this PR create

Made with Cursor

Let agents pass a local path or file:// URI so stills appear in the
desktop MCQ, not only in chat. Pattern: mcq-with-image.

Co-authored-by: Cursor <cursoragent@cursor.com>
@ajlennon
ajlennon requested a review from jackghx as a code owner August 2, 2026 22:16
@ajlennon
ajlennon merged commit 1ca80e2 into main Aug 2, 2026
4 checks passed
@ajlennon
ajlennon deleted the feat/mcq-with-image branch August 2, 2026 22:19
@ajlennon
ajlennon restored the feat/mcq-with-image branch August 2, 2026 22:19
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