Skip to content

[Tracking] v0.1.34 prod UX review backlog #702

@memtomem

Description

@memtomem

Summary

Backlog tracker for the v0.1.34 prod UX review (Playwright, run on 2026-05-02).

Source report: docs/reports/mm-web-prod-v0.1.34-playwright-review.md

Each child issue is independently mergeable. This issue stays open until everything below is either shipped or explicitly declined.

Status

# Finding Issue Severity Status
1 Indexing creates duplicate chunks within a single run #691 (pre-existing) P0 ✅ shipped via #705
2 BM25 FTS5 query sanitization gaps + silent degradation #697 P1 open — PR #715 in review (external contributor; CLA + ruff format pending)
3 Korean i18n guard only scans showToast/showConfirm #698 P1 ✅ closed
4 First search silent for 30s+ during model warm-up #696 (pre-existing) P1 ✅ shipped via #703 / #734
5 Artifact sync 3/3 synced ignores missing_target #692 (pre-existing) P1 ✅ shipped via #704 / #734
6 Tab hidden attribute not re-applied on deactivation #699 P2 (downgraded from P0) ✅ shipped via #737
7 List-row a11y: search/sources/timeline rows lack role #700 P2 ✅ shipped via #808 + #809 + #810 (split by surface)
8 Settings Runtime group expands to nothing in prod #701 P2 ✅ shipped via #736

Remaining: #697 only (waiting on external contributor response for CLA + ruff format).

Verification adjustments to the original report

The review was reverified against source on 2026-05-02. Two adjustments worth noting:

  • P0 "Cross-tab stale content leaks into the accessibility tree" — reframed and downgraded to P2 (Tab hidden attribute not re-applied on deactivation in activateTab() #699). Modern browsers exclude display: none from the a11y tree, so screen readers are not actually misled. The underlying DOM-state hygiene gap (the hidden attribute not being re-applied) is real and worth fixing, but the original "screen readers receive misleading page context" framing is incorrect.
  • P0 "Duplicate chunks" — confirmed at the data layer via SQL (matched (source_file, start_line, end_line, content_hash, namespace) rows ~1 second apart, concentrated in claude-plans namespace). Indexing creates duplicate chunks within a single run (no UNIQUE on content_hash) #691 already tracks this — the fix should land at the insert layer (UNIQUE constraint or INSERT OR IGNORE), not at the presentation layer the original review suggested.

Items intentionally NOT filed as issues

  • Long path / namespace truncation (P2 in review) — current right-side text-overflow: ellipsis is in place across .source-group-dir, .source-name, .result-filename, etc. The review's recommendation (middle-ellipsis, basename-first) is a UI preference, not a correctness bug. Reopen if user feedback shows scanability is actually painful.
  • Prod includes dev/harness scripts (P2 in review, reclassified P3) — settings-harness.js and settings-hooks-watchdog.js are loaded unconditionally from index.html in prod, but their UI targets are all data-ui-tier="dev" so they do not activate. The cost is a few KB of payload and parser time. Will revisit if/when prod payload becomes a measured concern.

Re-verification scheduled

A remote agent is scheduled for 2026-05-30 (4 weeks after the review) to re-check each finding against the then-current release and post a status comment here. (Routine: trig_018GFJqCAhYux4h1SLpjbKFQ.)

Metadata

Metadata

Assignees

No one assigned

    Labels

    do-not-staleManually exempt from stale sweepenhancementNew feature or request

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions