Skip to content

feat: read-only recommended.json consumer (mqlaunch) — PR1#16

Merged
MCamner merged 1 commit into
mainfrom
feat/recommendations-consumer-pr1
Jun 24, 2026
Merged

feat: read-only recommended.json consumer (mqlaunch) — PR1#16
MCamner merged 1 commit into
mainfrom
feat/recommendations-consumer-pr1

Conversation

@MCamner

@MCamner MCamner commented Jun 24, 2026

Copy link
Copy Markdown
Owner

Standalone, read-only consumer of the producer's command-recommendations.v1 output (recommended.json) in mqlaunch: resolve → validate → list → show → doctor. No ranking, no menu-wire, no execution.

What it does

  • recommendations-list.sh — rank-ordered, default-visible patterns (mutating hidden).
  • recommendations-show.sh <id> — full detail; renders command_template for show/copy, never executes.
  • recommendations-doctor.sh — source health check.
  • lib/recommendations/errors, resolve (reuses lib/mqobsidian/resolve.sh for vault root), parse, render, doctor.

Contract honored

Reads the file's own contract inline: allowed_actions: ["show","copy"], default_visible_risk: ["read-only"] — so mutating templates are hidden by default and never run. Producer (mqobsidian) owns ranking/curation; this only reads, filters, displays.

Base for the stacked PR2 (show/copy + menu + launcher-robustness fixes).

🤖 Generated with Claude Code

Consume the producer's command-recommendations.v1 output (recommended.json)
in mqlaunch as a standalone, read-only adapter — list, show, doctor. No
ranking, no menu-wire, no execution: honors the file's own contract
(allowed_actions: ["show","copy"], default_visible_risk: ["read-only"]) so
mutating templates are hidden by default and never run. Reuses
lib/mqobsidian/resolve.sh for vault-root resolution (no path sprawl).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@MCamner MCamner merged commit b3df009 into main Jun 24, 2026
1 check passed
@MCamner MCamner deleted the feat/recommendations-consumer-pr1 branch June 24, 2026 01:40
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