Skip to content

Commit 49c6114

Browse files
committed
docs(skill): address review — purge the last no-defaults statements
CodeRabbit caught two passages my phrase-sweep missed (the checklist's field-types line still banning arrays/multi-select, and the worked example's summary still claiming no prefilled defaults); a re-read caught a third in the worked example's intro. All three now state the shipped dialect.
1 parent dde995c commit 49c6114

2 files changed

Lines changed: 9 additions & 7 deletions

File tree

.agents/skills/write-template-playbooks/references/prompting-checklist.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,9 @@ builder-reliability findings, or design reasoning).
6161
trigger) so the user can veto. Commit and trigger are approval gates; pre-state what is
6262
coming.
6363

64-
Field types are `string`, `number`, `integer`, `boolean` only. No arrays, no nested objects,
65-
no multi-select. Formats: `date`, `date-time`, `email`, `uri`, `multiline`.
64+
Field types are `string`, `number`, `integer`, `boolean`, and the multi-pick array
65+
`{type: "array", items: {type: "string", enum: [...]}}`. No nested objects. Formats: `date`,
66+
`date-time`, `email`, `uri`, `multiline` (date fields ignore defaults).
6667

6768
## Verification (against our runtime)
6869

.agents/skills/write-template-playbooks/references/worked-example.md

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22

33
Copy from this. It is the seed playbook, already corrected against the runtime: `test_run`
44
explores uncommitted tools through its delta, the trigger test is the Lightning "Test event"
5-
button (Play "Run" for schedules), and no field is prefilled because `request_input` has no
6-
default. Notice how each section is short, names exact tools, and ends the instructions on the
7-
terminal side effect.
5+
button (Play "Run" for schedules), and proposals ride each field's `default` so the form is
6+
one-click acceptable. Notice how each section is short, names exact tools, and ends the
7+
instructions on the terminal side effect.
88

99
The fenced block below is the file that ships as `references/agent-templates/changelog-writer.md`.
1010

@@ -80,8 +80,9 @@ authorize).
8080
- **Required vs researchable is explicit.** The repo and publish target are required (the agent
8181
cannot proceed without them). The release process is researchable, so its enum leads with
8282
"Use your best judgment."
83-
- **No prefilled defaults.** The repo guess lives in the field description, not a default field.
84-
The recommended enum choice is listed first.
83+
- **Proposals ride the `default` field.** The repo guess becomes the field `default` when a
84+
prior read surfaces one; the recommended enum choice is set as the `default`, and the
85+
built-in Other… covers off-list answers.
8586
- **The priors table proposes a setup** per release process, so the agent does not interrogate
8687
the user about the trigger and tools.
8788
- **The instructions template ends on the publish tool** and names each tool in order. It does

0 commit comments

Comments
 (0)