Skip to content

feat: own M6 learning shortcuts - #16

Merged
yuezengwu merged 1 commit into
mainfrom
codex/m6-owned-adapters
Aug 14, 2026
Merged

feat: own M6 learning shortcuts#16
yuezengwu merged 1 commit into
mainfrom
codex/m6-owned-adapters

Conversation

@yuezengwu

@yuezengwu yuezengwu commented Aug 14, 2026

Copy link
Copy Markdown
Owner

What changed

  • register Explain-owned selection and finalized-answer shortcuts on first-party DSH slots
  • create editable /explain --selection and exact-turn /explain --answer drafts without automatic submission
  • keep optional selection-chat, suggested-replies, and Advisor plugins unmodified and independent
  • preserve legacy --suggested input and persisted origin: suggested history as read compatibility
  • update the M6 product, architecture, acceptance, and release documentation

Why

M6 previously placed integration changes in consumer plugins. Keeping both adapters inside Explain gives the feature one owner, removes cross-repository release coupling, and still allows users to select visible content produced by optional plugins.

Validation

  • pnpm run typecheck
  • pnpm test (64 tests)
  • DSH_SOURCE_DIR=/tmp/dsh-official-latest.CUnRHD/source pnpm run test:web (4 tests)
  • DSH_SOURCE_DIR=/tmp/dsh-official-latest.CUnRHD/source pnpm run test:m6 (3 tests)
  • DSH_SOURCE_DIR=/tmp/dsh-official-latest.CUnRHD/source pnpm run dsh:link:check
  • npm pack --dry-run --json
  • git diff --check

Real-model GUI evidence

Explain-owned answer and selection shortcuts

  • Explain commit: c64a76224ba77ef650be8cad9572281e7f57f4ba
  • DSH commit: 47f943859bef60e4160492346772ded9b24f765a
  • Server: built production Web profile from those exact clean trees, with Explain installed from the recorded checkout
  • State: fresh DSH_HOME, DSH_AGENTS_HOME, workspace, real Session, and isolated browser context
  • Models: one real DeepSeek main-agent round created the source answer; one real deepseek-chat Explain round created the learning card
  • Recording: repository-declared Playwright in isolated headless Chromium; this was the documented fallback because the in-app browser could not drive a real text drag reliably
  • No fixture query, mock transport, synthetic selection, or test-only hook was used
  • GIF SHA-256: 6c20446187f3ca09a701d28e50a81c4b285a0aeafa2f180a61ce63201d874f33

@yuezengwu
yuezengwu marked this pull request as ready for review August 14, 2026 10:26
@yuezengwu
yuezengwu merged commit f2cd85b into main Aug 14, 2026
@yuezengwu
yuezengwu deleted the codex/m6-owned-adapters branch August 14, 2026 10: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