Skip to content

dune skill: document the matview command group#11

Open
ivpusic wants to merge 1 commit into
mainfrom
ivan/matview-skill-docs
Open

dune skill: document the matview command group#11
ivpusic wants to merge 1 commit into
mainfrom
ivan/matview-skill-docs

Conversation

@ivpusic

@ivpusic ivpusic commented Jun 12, 2026

Copy link
Copy Markdown
Member

Documents the new dune matview (alias mv) CLI commands in the dune skill.

Changes

  • New skills/dune/references/matview-management.md — reference doc for create, get, list, update, refresh, delete, matching the style of the existing reference docs. Covers the bare-vs-fully-qualified name distinction, refresh-schedule semantics (cron min 15 min), the read-modify-write update behavior (schedule preserved unless changed), create-or-replace semantics, and a full lifecycle workflow.
  • SKILL.md — added matview to the frontmatter description/triggers, the command overview table, the write-command confirmation list (noting create/update/refresh consume credits and delete is irreversible), and the reference-documents table.

Context

Follows the matview rollout: duneapi REST endpoints (merged), duneapi-client-go v0.5.0 (matview client methods), and the CLI dune matview command group (duneanalytics/cli#61).

Documents the new `dune matview` (alias `mv`) CLI commands in the dune skill:

- New references/matview-management.md covering create/get/list/update/refresh/delete, the bare-vs-fully-qualified name distinction, refresh-schedule semantics, the read-modify-write update behavior, and a full lifecycle workflow.
- SKILL.md: added matview to the description/triggers, the command overview table, the write-command confirmation list (noting create/update/refresh consume credits and delete is irreversible), and the reference-documents table.

Follows the duneapi-client-go v0.5.0 matview methods and the CLI matview command group (duneanalytics/cli#61).
@cursor

cursor Bot commented Jun 12, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Documentation-only changes to skill markdown; no runtime, auth, or application code.

Overview
Extends the dune agent skill so assistants can discover and safely use the new dune matview (mv) command group for materialized views.

SKILL.md now mentions matviews in the skill description and trigger phrases, lists six matview subcommands in the command overview, requires user confirmation before matview write operations (with notes that create/update/refresh spend credits and delete is irreversible), and links to the new reference doc.

references/matview-management.md is a full command reference aligned with existing skill references: bare vs fully-qualified naming, cron refresh rules (15-minute minimum), create-or-replace on create, read-modify-write update (schedule preserved unless changed), flags/output for create, get, list, update, refresh, and delete, plus an end-to-end lifecycle workflow.

Reviewed by Cursor Bugbot for commit 151f871. Configure here.

@charisra

Copy link
Copy Markdown

bugbot run

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 151f871. Configure here.

@charisra charisra left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving with a few comments

### Output

- **text**: `Refreshing materialized view <fully-qualified-name>` + the execution ID and a results hint.
- **json**: `{"sql_id": "<fully-qualified-name>", "execution_id": "<id>"}`

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: the JSON key is sql_id here, not name as in create/update — use .sql_id when capturing the name for subsequent calls

### Tips

- `create` is **create-or-replace**: if a matview with this name already exists for the same query it is re-run. To change settings on an existing matview, use `dune matview update`.
- Matviews are public by default; `--private` requires a supporting plan.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Duplicates the --private flag description above — remove this line.


### Tips

- `create` is **create-or-replace**: if a matview with this name already exists for the same query it is re-run. To change settings on an existing matview, use `dune matview update`.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What happens if the name exists but with a different --query-id? "for the same query" implies different behavior in that case but it's not documented — either clarify or drop the qualifier.

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.

3 participants