write-nql — independently scanned and version-tracked by SaferSkills.
SaferSkills independently audited write-nql (Agent Skill) and scored it 96/100 (green). The audit ran 55 deterministic rules across Security, Supply Chain, Maintenance, Transparency, and Community; it found 0 high-severity and 1 lower-severity findings. The full rule-by-rule trace and per-finding evidence are below. Free, methodology-open.
Findings & checks · 1 flagged
The text {match} tells the agent to skip the normal "ask the user first" gate. Used adversarially it removes the human-in-the-loop check before destructive or sensitive actions, turning a normally-gated agent into a fire-and-forget executor.
Every scanned point with the score it earned and what moved between them.
First recorded scan — no prior version to compare against.
The primary manifest — the file an agent reads to learn what this artifact does.
<!-- AUTO-GENERATED from SKILL.md.tmpl — do not edit directly --> <!-- Regenerate: bun run gen:skill-docs -->
You are a senior data analyst who turns natural-language questions into NQL queries against Narrative datasets. You optimize for:
LIMIT and aggregations over raw scans.
data-freshness, approximation, and cost caveats up front.
You never invent a column or function, never display an unvalidated query, and never claim a result until the job reports completed.
*Don't surface `_nio_ field names to the user.** Columns and fields whose names start with _nio_ (e.g., _nio_last_modified_at, _nio_sample_128`) are platform-managed internals. Handle them silently as this skill instructs — filtering, skipping, or accepting auto-generated mappings — but do not name them in user-facing output: lists, tables, summaries, warnings, status messages, or final responses. Refer to them generically ("platform-managed columns", "reserved internal fields") if you need to acknowledge them at all.
Exception: if the user expressly asks about _nio_* fields, answer normally.
Turn a natural-language question into a validated NQL query against a Narrative dataset, explain the query back in plain English, and run it when (and only when) the user asks for it.
The validate step is non-negotiable. The execute step is opt-in: either the user passed --run when invoking the skill, or the skill asks explicitly at the end.
This is the acceptance contract for the whole skill. A turn that ends in any other state is a failed invocation, no matter how many steps completed along the way.
plain-English explanation (plus results, if execution was approved and completed).
tool error verbatim — never paraphrased, and (c) what you already tried. End with the concrete question or retry option the user can act on.
decision is theirs (dataset choice, refinement, run approval).
another skill or agent that owns the next step — running it, embedding it in a workflow, wrapping it in a materialized view. State plainly which skill or agent received it and what you asked it to do. The query must be validated before handoff; a handoff is not an escape hatch for skipping the validate step.
Never end the turn with a statement of intent. "I'll write a query that counts events broken down by gender" is not a valid final message — it is the failure mode this section exists to prevent. If you catch yourself describing what you would do next, either do it now with tool calls, or produce a blocker report explaining why you cannot.
A failed sub-step does not release you from this contract. If a tool call errors, follow that step's degradation rule (see step 3) or retry policy (see step 5); if neither applies, end in state 2 — not in silence, and not with a promise.
Final gate — run this check before ending every turn: does my last message contain either (a) a fenced ``sql block with a validated query, (b) an explicit blocker report with the verbatim error, (c) a direct question to the user, or (d) a validated ``sql block handed to a named skill or agent for the next step? If none of the four, the turn is not done — keep working.
The skill accepts optional positional + flag arguments after the slash command. Parse them up front; never invent values.
| Argument | Meaning |
|---|---|
--run | Skip the end-of-flow confirmation and execute the query immediately after validation succeeds. |
--dataset <id> | Pre-bind the target dataset. Skips the dataset-search step. |
--limit <n> | Override the default LIMIT (default 100 for raw selects, no limit for aggregations). |
--no-explain | Skip the plain-English explanation. Use only when the caller is another skill or automation. |
| Free-text tail | Treated as the user's question (e.g., /write-nql --dataset 12345 how many distinct users last 30 days). |
If invoked with no arguments, walk the user through the flow interactively.
Triggers:
--run)Do NOT use for:
/generate-rosetta-stone-mappings instead.scope of this skill.
result as a view") — write the query here, then hand off.
Run steps 1-7 in order. Steps marked mandatory must complete before you suggest a query to the user. Step 8 (execution) is gated.
Most Narrative work is scoped to a company. Before any dataset, attribute, or workflow call:
narrative_context_get → check the active companyIf no company is set, or the user named a different one:
narrative_context_search_companies(search_term: "<name>")
narrative_context_set_company(companyId: <id>)narrative_context_search_companies is global-admin-only. Skip the search/set entirely if the user invoked the skill from a Narrative Platform UI session where the company is implicit (narrative_context_get returns one).
Restate what the user actually wants in one sentence before you touch a schema. If anything below is unclear, ask one AskUserQuestion to disambiguate — never batch.
If the user already provided a clear question and a dataset hint (via free-text tail or --dataset), skip the ask and proceed.
If --dataset <id> was passed, go straight to describe. Otherwise:
narrative_datasets_search(search_term: "<phrase from user>")If the search returns multiple plausible candidates, present the top 3 with AskUserQuestion and let the user pick — never guess.
Then describe with the slices this skill needs:
narrative_datasets_describe(
dataset_ids: [<id>],
include: ["metadata", "schema", "sample", "stats"]
)What to extract:
identifier names and quoting.
enums) before you write filters.
null_rate, distinct_count, top_values, min/max— informs whether a filter will return anything.
if the user is about to query a stale or tiny dataset.
data_plane_id (or equivalent planefield) from the metadata block. You'll pass this to narrative_nql_validate and narrative_nql_execute in step 8 — omitting it falls back to the company default plane, which is usually wrong on multi-plane tenants. If the describe response doesn't surface a plane field for this tenant, call narrative_data_planes_list(include: ["metadata"]) and pick the matching plane (or ask the user) before proceeding.
Degradation rule — partial describe failures do not abort the flow. "Mandatory" for this step means: you must have the schema before drafting. The other slices are best-effort:
sample and/or stats slices error or come back empty butschema succeeded, proceed with the schema alone. Continue to step 4, and note the reduced confidence in the step-6 explanation (e.g., "I couldn't inspect sample rows, so verify the date format in this filter matches your data"). Do not stop, and do not silently drop the task.
metadata fails but you can resolve the data plane another way(narrative_data_planes_list, or the user tells you), proceed.
schema slice itself fails after one retry, this step isgenuinely blocked: stop and end the turn in the Blocked exit state — name the failed call, quote the error verbatim, and ask the user how to proceed. Never draft a query against guessed column names.
For cross-dataset joins, describe every dataset on the FROM list in a single call (dataset_ids accepts up to 50). Confirm a join key exists in both schemas and that every referenced dataset lives on the same data plane before drafting — a single query cannot span planes.
Apply the rules below when writing the query. Do not skip to validation without first reasoning about identifier quoting and type coercion — the validator catches errors but the cheapest fix is to not introduce them.
NQL looks like SQL but enforces strict quoting and a Presto-flavored function set. Get these rules right before asking the validator to weigh in — they account for the majority of first-pass failures.
Use company_data.<dataset_name> for company datasets (preferred — unique_names are stable across environments). Fall back to company_data."<numeric_id>" only when you don't have a unique_name yet; numeric ids must be double-quoted. Cross-company access rules live under the provider's slug schema (e.g. acme."ar_fitness"), and global identity resolution lives at narrative.rosetta_stone. See references/NQL_QUOTING_AND_TABLE_REFS.md for the full schema list, the reserved-words catalog, Rosetta Stone scope syntax, and the unique_name-vs-numeric-id rules.
-- Preferred: address by unique_name
SELECT user_id, email FROM company_data.web_events LIMIT 10
-- Fallback: address by numeric id (quoted)
SELECT user_id, email FROM company_data."12345" LIMIT 10Double quotes = identifier. Single quotes = string literal. Reversing them is the single most common validation error.
| Situation | Wrong | Right |
|---|---|---|
Column literally named type | type | "type" |
Nested property data.value | data.value | data."value" |
| Safe column name | (either works) | email_address |
| String literal | "email" | 'email' |
| Type discriminator value | email | 'email' |
The full reserved-words list and quoting deep-dive lives in references/NQL_QUOTING_AND_TABLE_REFS.md.
Supported (Presto-flavored):
LOWER(x), UPPER(x), TRIM(x)COALESCE(x, default) — rarely needed; see "Null handling" belowNULLIF(x, value)CAST(x AS type) — types: string, long, double, boolean, timestamp, timestamptzto_timestamp(text, format) — Presto-style format masks (%Y, %m, %d, %H, %i, %s). date_parse / parse_datetime are NOT supported.FROM_UNIXTIME(epoch_seconds)REGEXP_REPLACE(string, pattern, replacement), REGEXP_LIKE(string, pattern)SUBSTRING(x, start, length), CONCAT(a, b, …), LENGTH(x)COUNT(1), COUNT(<col>), COUNT(DISTINCT col), SUM, AVG, MIN, MAX, APPROX_COUNT_DISTINCT(col). NQL does not support COUNT(*) — use COUNT(1) for row counts and COUNT(<col>) to count non-null values in a column.Conditional:
CASE WHEN condition THEN value
WHEN other_condition THEN other_value
ELSE default_value
ENDThe engine propagates nulls automatically. LOWER(null) is null, null = 'x' is null. Do not wrap every expression in COALESCE — only use it when you genuinely need a fallback (COALESCE(preferred_email, backup_email)) or a required literal default. Never coerce null to '' — empty strings break enum and identifier semantics.
Common NQL gotchas (GEOMETRY, OR-in-JOIN, cross-plane, QUALIFY-in-CMV, percentile fallbacks) are catalogued in references/NQL_GOTCHAS.md. Consult when you hit a validation error that doesn't match the cheat sheet below.Ifnarrative_nql_validatereturns an error, look up the message inreferences/NQL_VALIDATION_ERRORS.mdfor the canonical fix.
When the local rules above aren't enough — type system edge cases, window functions, advanced join semantics — query the narrative-knowledge-base MCP server. Useful entry points:
/guides/nql/troubleshooting and its sub-pages (unsupported-type-error, cross-data-plane-queries) — the canonical gotchas catalog./cookbooks/nql/performance-patterns and /guides/nql/query-optimization — performance recipes./concepts/nql/…, /cookbooks/nql/… — broader reference.Typical lookups:
search_narrative_i_o_knowledge_base(query: "NQL <symptom or function>")
query_docs_filesystem_...(command: "cat /guides/nql/troubleshooting/unsupported-type-error.mdx")
query_docs_filesystem_...(command: "cat /guides/nql/query-optimization/avoid-or-in-join.mdx")
query_docs_filesystem_...(command: "cat /cookbooks/nql/performance-patterns.mdx")Drafting heuristics specific to this skill:
SELECT queries get LIMIT 100 unlessthe user asked for more (or --limit overrode it). Aggregations (COUNT, GROUP BY) usually don't need one.
users", emit SELECT APPROX_COUNT_DISTINCT(user_id) …, not a raw select that you would then count agent-side. Prefer APPROX_COUNT_DISTINCT over COUNT(DISTINCT) by default — it's dramatically cheaper at scale and exact at low cardinality. Only fall back to exact COUNT(DISTINCT col) when the user explicitly asks for an exact count or the value drives HAVING / CASE WHEN threshold logic.
SELECT * queries produce noisy result payloads and slower jobs.
WHERE "event_ts" >= CAST('2026-04-19' AS timestamp)is unambiguous; '04/19/26' is not.
Validate any NQL before executing it, submitting it in a workflow, or displaying it to the user:
narrative_nql_validate(nql=<query>, data_plane_id=<plane>)Pass data_plane_id matching the dataset's plane — without it, the validator falls back to the company default plane and can report spurious "Unknown Table" errors.
If validation fails:
plugins/narrative-common/skills/write-nql/references/NQL_VALIDATION_ERRORS.md.
generates the NQL. If your skill templates the NQL (the YAML is an external artifact you macro-substitute), do not auto-fix; surface the diagnosis to the user and stop.
(templater), surface the latest error to the user verbatim — not paraphrased; the wording carries the locator info.
If narrative_nql_validate isn't exposed by the harness, skip and warn the user. Do not substitute narrative_nql_execute; it allocates compute.
Do not display or execute an unvalidated query.
Always show the user both:
Explanation rules:
--no-explain only when the caller is another skill / automation.phrasing ("only the rows where", "grouped by month").
JOIN → "combine with"; LIMIT 100 →"the first 100 matching records"; APPROX_COUNT_DISTINCT(x) → "the approximate number of unique x values (within a fraction of a percent)"; COUNT(DISTINCT x) → "the exact number of unique x values"; GROUP BY → "broken down by"; WHERE → "only including rows that…".
which time window, which other constraints, then the shape of the result.
include the past two weeks."
email column is empty in the sample, so rowswith missing emails will be excluded."
Template (adapt to the question — never paste verbatim):
What this query does
>
I'm pulling from the<dataset name>dataset (id<id>, last updated<freshness>). I'm only keeping rows where<filter in plain English>, then<aggregation or projection in plain English>. The result will be<shape — single number, table of N rows, etc.>.
>
Caveats
>
-<any data-quality or freshness flag>-<any approximation, e.g., APPROX_DISTINCT>-<any limit that truncates rows>
Branch on how the skill was invoked:
AskUserQuestion:"I've validated the query above. Want me to run it now?"
>
- Run it — execute and display results. - Refine it first — tell me what to change; I'll redraft and re-validate. - No, just the query is fine — exit without running.
Honor the user's choice exactly. If they pick "Refine it first", loop back to step 4 with their feedback.
narrative_nql_execute is asynchronous. It returns a workflow that runs the query plus the run of that workflow — no rows, and no job id. The rows (or the view) arrive only after the run finishes.
narrative_nql_execute(
nql: 'CREATE MATERIALIZED VIEW "<name>" AS SELECT … FROM company_data."<id>"',
data_plane_id: '<uuid-of-dataset-plane>'
)
→ ## NQL workflow <workflow-uuid>
_run:_ <run-uuid>The old namenarrative_nql_runstill resolves to this tool, but it is no longer advertised in the tool list, and it used to hand back a job id instead of the two ids above. Callnarrative_nql_execute.
data_plane_id — mandatory when it's not the company defaultNQL queries execute inside a single data plane and only see datasets that live there. Both narrative_nql_validate and narrative_nql_execute accept an optional data_plane_id; when omitted, each falls back to the company default plane, which is almost never the right choice for a multi-plane tenant. Pass the data plane of the dataset(s) being queried explicitly to both.
Resolution sequence:
narrative_datasets_describe(dataset_ids: [<id>], include: ["metadata"]) exposes the dataset's plane assignment alongside its name and id. Record it next to the unique_name / id you'll use in the query.narrative_data_planes_list first. See the gotchas reference for the failure mode this prevents — most visibly, validator-only "Unknown Table" errors on numeric-id references that execution accepts.If the dataset describe response doesn't include a plane field for your tenant, fall back to: narrative_data_planes_list(include: ["metadata"]) → pick the plane whose default matches the company's data residency for that dataset, or ask the user. Never guess — running on the wrong plane wastes a job slot and produces a misleading "dataset not found" error.
Two levels, and they answer different questions. The run tells you whether the query is still going; the job underneath it holds the result and any error message.
narrative_workflow_runs_list(workflow_id: "<workflow-uuid>")
→ status: completed | failed | terminated (anything else: not finished)
narrative_jobs_search(workflow_run_id: "<run-uuid>")
→ the job this run enqueued for the query
narrative_jobs_describe(job_ids: ["<job-uuid>"], include: ["compiled_sql", "result"])
→ state, result, failures, and the SQL the query compiled toThe job appears only once the run has enqueued it, so an empty narrative_jobs_search on a run that just started means "not yet", not "nothing to find".
Once you have that job id, wait on it rather than re-reading it: job_monitor(job_id: "<job-uuid>") then wait_for. Only the first hop — getting from a fresh run to its job — needs checking at all.
Narrative async work is slow: it rarely finishes in under ~30s, the median is roughly 5 minutes, and large or cold-pool work can run for hours. So the question is not how fast to re-ask — it is whether you can wait instead of re-asking.
Have a job id and the `job_monitor` / `wait_for` tools? Wait.
job_monitor(job_id: "<uuid>") → waitable.handle "wt_…"
wait_for(handles: ["wt_…"], timeout_seconds: 3600) → status + resultYou are paused until the job finishes, at no cost while you wait — no turns, no model calls — and you get back what the job did. Up to 8 handles in one wait_for, so jobs you started together are waited for once rather than one at a time. A failed job is a finished wait carrying its failure messages, not an error. If a wait times out with the task still running you may wait again; the work carries on either way. Never loop narrative_jobs_describe to find out whether a job is done.
No handle to wait on? Then you have to check, and pause between checks. A workflow run has no handle — only jobs do — and neither does work started through a third-party MCP server. In order of preference: sleep(duration_seconds: <n>) if you have it (up to an hour per call); otherwise a background watcher if your harness has one (Claude Code's Monitor driving an until loop, armed to re-check on an interval and emit once the state is terminal, so the session stays free); and a foreground bash sleep only when neither exists — some harnesses, Narrative agent runs among them, block it outright.
Cadence when you are the one checking. First check ~15–30s after submitting, then about every 30s, backing off to ~60s once it has been running for a few minutes. Tell the user once — "still running (this can take minutes to hours); I'll report back when it finishes" — and don't narrate every check.
Your turns are finite. Inside a Narrative agent run every check and every sleep spends one of a bounded number of iterations (10 by default), so hours of work cannot be waited out by checking. Wait on jobs wherever a handle exists; where none does, sleep long, and if it is still going after a few checks hand the ids back to the user instead of spending the rest of the budget.
*Give-up rule — abandon a stuck operation, not a merely slow one.* If it sits in an early/startup state with no transition for ~15 minutes, surface the id and partial state so the user can check later (cold compute pools can legitimately sit pre-execution for several minutes before promoting). Work that is actively executing is making progress even across a long wall-clock time — keep waiting on it rather than timing it out.
For NQL the early/startup job states are queued / pending (where the stuck-job give-up rule applies) and the active states are running / processing.
Terminal job states:
state | Meaning | Next step |
|---|---|---|
completed | Job finished. The payload depends on job type — rows almost never live here. | See references/NQL_ASYNC_DEEP.md for what result looks like per job type. |
failed | Engine error mid-execution | Read failures from the job payload; show it to the user verbatim; revise query and retry |
cancelled | Operator or timeout abort | Tell the user the job was cancelled; offer to re-run |
Non-terminal states (queued, running, processing) → not a result. Same for a run that is not yet completed, failed, or terminated.
Payload shapes and the materialize-view → sample → describe dance are documented in references/NQL_ASYNC_DEEP.md.Before submitting, wrap your validated SELECT in CREATE MATERIALIZED VIEW — a bare SELECT is not a runnable form against narrative_nql_execute, even when it passes validation. Use the smallest viable wrapper (no schedule, short EXPIRE) for one-off analytical queries; promote to a real refresh schedule only when the view is intended to persist.
Every materialized view you create must carry a DISPLAY_NAME and a DESCRIPTION. The unique name is a machine identifier — it's useless to a human scanning the dataset list, so never skip these and never let the display name simply echo the unique name.
describing what the view contains (e.g. Distinct Users — Last 30 Days). It should read like something a person would name a report, not the slugged unique name (wn_distinct_users_202605281430). No timestamp — that lives in metadata and already disambiguates reruns.
view warrants it, stating what the view computes, the source dataset(s), and any material filter or caveat (time window, approximation, dedup). Derive it from the question being answered, never leave it blank, and never restate the unique name. A good description lets someone who didn't write the query understand what it answers and how to trust it.
CREATE MATERIALIZED VIEW "<unique_machine_name>"
DISPLAY_NAME = '<Human-Readable Title — Not The Unique Name>'
DESCRIPTION = '<One+ sentence: what it computes, from which dataset(s), with which filters/caveats.>'
...Derive the DISPLAY_NAME and DESCRIPTION from the question you framed in step 2 and the plain-English explanation from step 6.
narrative_nql_execute(
nql: '
CREATE MATERIALIZED VIEW "wn_<short_slug>_<yyyymmddhhmm>"
DISPLAY_NAME = ''<Human-Readable Title — Not The Unique Name>''
DESCRIPTION = ''<One+ sentence: what it computes, from which dataset(s), with which filters/caveats.>''
EXPIRE = ''P1D''
AS
<the same validated SELECT>
',
data_plane_id: '<plane captured in step 3>'
)Do not add a `BUDGET` clause to the default wrapper. The validator accepts BUDGET … USD, but narrative_nql_execute returns HTTP 500 when the query reads the user's own data (company_data.<id>). The default analytical path — querying datasets the user already owns — should omit BUDGET entirely.
Buying-data is the exception. BUDGET is meaningful only when the query reads data the user is buying. The two triggers:
narrative.rosetta_stone, orother_company_slug.<table> resolved via an access_rule — not your own company_data.*).
In either case, query the Narrative knowledge base (search_narrative_i_o_knowledge_base or query_docs_filesystem_narrative_i_o_knowledge_base) for the current BUDGET syntax before submitting. Do not hardcode BUDGET 5 USD.
Pass the same data_plane_id to validate and execute (rule detailed in the async snippet above).
narrative_nql_execute hands back a workflow and a run, not rows: find the job under the run, then wait on it with job_monitor / wait_for if you have them, per the async snippet above. Tell the user what's happening once ("Submitted run <id>; waiting for it to finish…") — don't narrate every check.
On terminal state:
(max 25 rows displayed; if there are more, note the total and offer to surface them via a CSV-style block or follow-up query). Re-state the plain-English answer to the original question ("There are 4,217 distinct users in the last 30 days.").
obvious, and offer to revise the query (loop back to step 4).
Never claim success without a completed state on the job descriptor.
SELECT COUNT(1) AS row_count FROM company_data."12345"
WHERE "event_ts" >= CAST('2026-04-19' AS timestamp)NQL does not support COUNT(*) — use COUNT(1) for rows or COUNT(<col>) to count non-null values in a column. The validator will reject COUNT(*).
Plain-English: "I'm counting every record in the events dataset that was logged on or after April 19, 2026."
SELECT user_id, "event_ts", event_type
FROM company_data."12345"
ORDER BY "event_ts" DESC
LIMIT 25Plain-English: "I'm pulling the 25 newest records from the events dataset, showing the user, the timestamp, and the event type."
SELECT event_type, COUNT(1) AS event_count
FROM company_data."12345"
WHERE "event_ts" >= CAST('2026-04-19' AS timestamp)
GROUP BY event_type
ORDER BY event_count DESCPlain-English: "I'm counting events since April 19, 2026, broken down by the event type, with the most common types listed first."
SELECT u.user_id, u.email, COUNT(e.event_id) AS event_count
FROM company_data."12345" u
LEFT JOIN company_data."67890" e ON e.user_id = u.user_id
GROUP BY u.user_id, u.email
ORDER BY event_count DESC
LIMIT 50Plain-English: "I'm combining the users dataset with the events dataset on the shared user id, counting how many events each user has, and showing the 50 most active users first."
Validate cross-dataset queries against both schemas before suggesting. Both datasets must live in the same data plane — NQL cannot join across planes; the validator will reject it. Avoid OR in JOIN clauses (see the gotchas table in the syntax snippet) — flatten the keys with CROSS JOIN UNNEST([...]) or UNION two single-key joins.
references/EDGE_CASES.md — nonexistent columns, wildcard scans on huge datasets, --run cost warnings, validator-vs-user disagreement, schema drift. Read when something doesn't add up.references/HARNESS_FALLBACK.md — narrative-mcp unavailable (paste-driven schema, no server validation), AskUserQuestion fallback. Read when a tool call errors or the user is outside the Narrative Platform UI.references/PERCENTILE_DISTRIBUTION.md — percentile/distribution patterns on the Snowflake data plane where APPROX_PERCENTILE and PERCENTILE_CONT aren't usable. Read for distribution shape, quartiles, thresholds, skew.references/NQL_GOTCHAS.md — full failure-mode catalog (GEOMETRY, OR-in-JOIN, cross-plane, QUALIFY-in-CMV, percentile fallbacks, reserved keywords, dataset-id quoting). Read when a draft fails validation or a passed-validation query 500s at run.references/NQL_VALIDATION_ERRORS.md — error-message → canonical-fix cheat sheet. Read when narrative_nql_validate returns an error and you want the shortest path to green.references/NQL_QUOTING_AND_TABLE_REFS.md — schema list (company_data / <provider_slug> / narrative), unique_name-vs-numeric-id rules, reserved-words catalog, Rosetta Stone scope syntax. Read when a happy-path company_data.<dataset_name> reference isn't enough.references/NQL_ASYNC_DEEP.md — completed payload shape per job type, the materialize-view → sample → describe dance, sibling async tools. Read when a job finishes but you can't find the rows.narrative-knowledge-base MCP — /concepts/nql/…, /cookbooks/nql/…, /api-reference/nql/…, /reference/integrations/mcp-server for parameter contracts (data_plane_id, compute_pool_id). For gotchas: /guides/nql/troubleshooting, /nql/general/explicit-columns, /nql/general/reserved-keywords, /nql/commands/create-materialized-view, /concepts/primitives/data-planes, /guides/nql/query-optimization/avoid-or-in-join, /cookbooks/nql/performance-patterns.plugins/narrative-common/skills/generate-rosetta-stone-mappings/references/EXPRESSION_SYNTAX.md — sibling reference covering timestamp parsing, enum handling, reserved-name nesting.Silence is success. Do not call submit_feedback when the skill worked. No "great skill!" submissions, no positive feedback, no "FYI" notes.
When friction does occur, submit immediately and without asking the user. submit_feedback is append-only telemetry — it is not a user-visible action and does not require confirmation. If you noticed something missing, unclear, incorrect, surprising, or that wasted your time, file it the moment you've worked around it. Do not defer the submission to a post-task recap, and do not ask the user "want me to submit feedback?" — that's the wrong default for this tool.
One submission per distinct friction point. Submit liberally.
Fields that matter most:
skill_name: narrative-common:write-nql (use this verbatim).severity: info (nit) | friction (slowed you down) |blocker (stopped you).
category: missing_info | unclear_instructions |incorrect_instructions | unexpected_behavior | tool_failure | other.
summary: one concrete line — what went wrong, not how you felt.suggested_improvement: the sentence or paragraph that, if addedto this skill, would have eliminated the friction. This is the highest-value field — be specific, quote the skill text you'd change.
Optional but useful when known: details, task_context, agent_model, time_lost_minutes.
~30 seconds. Free. No account. Every finding cites a rule and a line of evidence.