[rig-tasks] Daily rig evaluation — 2026-09-01 — 10/10 passed

Comienza con la tabla de decisiones de alta frecuencia y la sección de referencia rápida de SKILL.md, y luego compáralas con los requisitos de metadatos del workflow y los hallazgos sobre prompt-helper de este issue. Documenta los requisitos de metadatos del workflow, el patrón body({ call }), los prompts de agente no-undefined y p.readOptional con un fallback. Se considera terminado cuando las carencias enumeradas están cubiertas con ejemplos concisos y la referencia existente sigue siendo coherente.

Summary

Problems encountered

Task 6 — NPM script dep workflow (initial failures)

What the code tried to do: A workflow() with two sequential agent calls passing structured output from scriptGraphBuilder to cycleDetector.

Errors:

error TS2305: Module '"rig"' has no exported member 'call'.
error TS2769: Property 'description' is missing in type '{ name: string; }' but required in type 'WorkflowMeta'.
error TS2345: Argument of type 'undefined' is not assignable to parameter of type 'string | PromptIntent | PromptBuilder'.

Root cause: Three distinct mistakes:

1. call is not exported from "rig" — it is destructured from the body({ call }) parameter or imported from "rig/globals".
2. workflow({ meta: { name } }) requires a description field in WorkflowMeta.
3. call(agent, undefined) is not valid — agents always receive a string prompt or typed input.

Fix: Used body: async ({ call }) => destructuring, added description to meta, passed a string prompt to agents.

Task 7 — HTML form field extractor (initial failure)

What the code tried to do: Push objects with optional name/id fields by assigning undefined values.

Errors:

error TS2379: exactOptionalPropertyTypes: 'string | undefined' is not assignable to type 'string'.

Root cause: With exactOptionalPropertyTypes: true, assigning undefined to an optional field that expects only string is rejected. The fix is to conditionally include the property only when the value is defined.

Fix: Build the entry object without optional fields, then conditionally assign them.

Task 10 — Multi-stage artifact pipeline (initial failures)

Same call import and meta.description errors as task 6. Fixed identically.

Improvement opportunities

Missing or undiscoverable schema helpers (s.*)

No missing helpers this run. The existing set (s.object, s.record, s.array, s.enum, s.optional, s.path, s.int, s.number, s.boolean, s.string) covered all needs.

Missing or undiscoverable prompt helpers (p.*)

p.readOptional with an empty-string fallback (task 8) is not clearly documented in SKILL.md's quick reference table. Consider adding it as a distinct row: p.readOptional(path, fallback?) — currently documented only in the focused reference.

Error message quality

The exactOptionalPropertyTypes error (TS2379) is confusing because it mentions a parameter type mismatch rather than naming the root cause directly. The fix (conditional assignment) is non-obvious from the error text alone.

The call import error (TS2305: Module '"rig"' has no exported member 'call') is clear, but nothing in the error guides toward the correct body({ call }) destructuring pattern. A short comment in SKILL.md would prevent this.

API ergonomics

1. call discoverability: The most common workflow mistake is import { call } from "rig". SKILL.md's composition rule could add a note: "call is available via body({ call }) destructuring — do not import it from "rig"."
2. workflow meta requirement: description being required in WorkflowMeta is not called out in the high-frequency decisions table. A brief reminder alongside the workflow() row would prevent the error.
3. call(agent, undefined) vs call(agent, string): It is unclear from the table that agents always require a non-null prompt. Noting "pass a string prompt or typed input; never undefined" would help.

Candidate lint rules

Rule: no-rig-call-import

• Invalid: import { agent, call } from "rig";
• Valid: body: async ({ call }) => { ... } or import { call } from "rig/globals";
• Why model-confusing: call looks like a plain function export given how agent and workflow are imported. The destructuring pattern is non-obvious without reading the full reference.
• Safe autofix: possible — remove call from the "rig" import and add import { call } from "rig/globals" as a fallback (with a comment to prefer destructuring inside body).

Documentation gaps

• SKILL.md high-frequency decisions table should add: workflow meta → must include both name and description.
• SKILL.md should note that call inside workflow.body is always obtained via destructuring, not via import.
• p.readOptional deserves a row in the quick-reference table with an example fallback value.

Tasks run today

• (reused) Dockerfile ENV instruction parser
• (reused) NPM script command prefix analyzer
• (reused) INI config file parser
• (reused) Git reflog inspector
• (reused) Source file line length auditor
• (reused) NPM script dependency workflow
• (new) HTML form field extractor
• (new) Dotenv template validator
• (new) TypeScript namespace usage reporter
• (new) Multi-stage build artifact pipeline

Generated by Daily Rig Task Generator · sonnet46 164.3 AIC · ⌖ 9.36 AIC · ⊞ 6.8K · ◷