Skip to content

fix(designer): add missing safe output types and frontmatter fields to designer SKILL.md#43118

Merged
pelikhan merged 3 commits into
mainfrom
copilot/fix-designer-drift-report
Jul 3, 2026
Merged

fix(designer): add missing safe output types and frontmatter fields to designer SKILL.md#43118
pelikhan merged 3 commits into
mainfrom
copilot/fix-designer-drift-report

Conversation

Copilot AI commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

2026-07-03 Designer Drift Audit identified two gaps in agentic-workflow-designer/SKILL.md: the Safe Output mapping table was missing three error-signaling system types, and the Generation Template omitted six frontmatter fields documented in syntax-agentic.md.

Safe Output Mapping — added 3 system error-signaling types

User intent Maps to
report missing data needed for the task missing-data (system type, auto-enabled)
report unavailable or missing tool/permission missing-tool (system type, auto-enabled)
signal task could not be completed due to infrastructure failure report-incomplete (system type, auto-enabled)

Added a callout below the table clarifying that system types are auto-available in every workflow, require no safe-outputs: declaration, and can be explicitly disabled (e.g. missing-tool: false).

Generation Template — added 6 missing frontmatter fields

strict: true
on:
  <trigger config>
max-turns: <integer or omit>
max-ai-credits: <integer or omit for default 1000>
max-daily-ai-credits: <integer or omit to leave disabled>
user-rate-limit: <object or omit>
models: <object or omit>

strict: true is placed near the top as a strongly-recommended default. The remaining five fields are exposed as optional placeholders so generated workflows surface cost/rate guardrails as first-class considerations.

Copilot AI and others added 2 commits July 3, 2026 06:02
… Mapping and add 6 missing frontmatter fields to Generation Template in designer SKILL.md

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…output types

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix missing safe output types and frontmatter fields in drift report fix(designer): add missing safe output types and frontmatter fields to designer SKILL.md Jul 3, 2026
Copilot AI requested a review from pelikhan July 3, 2026 06:04
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Great work on this drift audit fix! 🎉 The changes to agentic-workflow-designer/SKILL.md are clean, well-scoped, and the PR description clearly explains both the gap found and how it was addressed.

  • The three new system error-signaling types (missing-data, missing-tool, report-incomplete) fill a real gap in the Safe Output mapping table, and the callout clarifying their auto-enabled nature is a nice usability improvement.
  • The six added frontmatter fields in the Generation Template (strict, max-turns, max-ai-credits, max-daily-ai-credits, user-rate-limit, models) bring the template in sync with syntax-agentic.md and make cost/rate guardrails visible by default.

This looks ready for review. 🚀

Generated by ✅ Contribution Check · 105.3 AIC · ⌖ 10.5 AIC · ⊞ 6.3K ·

@pelikhan pelikhan marked this pull request as ready for review July 3, 2026 06:27
Copilot AI review requested due to automatic review settings July 3, 2026 06:27
@pelikhan pelikhan merged commit e3c7a10 into main Jul 3, 2026
@pelikhan pelikhan deleted the copilot/fix-designer-drift-report branch July 3, 2026 06:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Agentic Workflow Designer skill documentation to better reflect current safe-output capabilities and workflow frontmatter options, based on a drift audit.

Changes:

  • Adds missing error-signaling safe-output mappings (missing-data, missing-tool, report-incomplete) to the Safe Output mapping table.
  • Extends the generation template frontmatter to surface strict and cost/rate/model guardrail fields.
Show a summary per file
File Description
.github/skills/agentic-workflow-designer/SKILL.md Expands safe-output mapping guidance and updates the workflow generation template with additional frontmatter fields.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 3
  • Review effort level: Low

Comment on lines +155 to +157
| "report missing data needed for the task" | `missing-data` (system type, auto-enabled) |
| "report unavailable or missing tool/permission" | `missing-tool` (system type, auto-enabled) |
| "signal task could not be completed due to infrastructure failure" | `report-incomplete` (system type, auto-enabled) |
| "signal task could not be completed due to infrastructure failure" | `report-incomplete` (system type, auto-enabled) |
| "nothing visible", "just analyze" | no safe outputs required |

> **System types** (`missing-data`, `missing-tool`, `report-incomplete`) are error-signaling outputs that are automatically available in every workflow without being declared in `safe-outputs:`. They emit structured infrastructure signals (not task results) and can be disabled explicitly (e.g. `missing-tool: false`) but should rarely be suppressed.
<trigger config>
max-turns: <integer or omit>
max-ai-credits: <integer or omit for default 1000>
max-daily-ai-credits: <integer or omit to leave disabled>
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.

Designer Drift AuditDesigner Drift Report — 2026-07-03

3 participants