Add zapier-workflows skill, consolidated from agent-skills#20
Draft
BenMcGit wants to merge 1 commit into
Draft
Conversation
Durable workflow guidance was split across the zapier-sdk skill's references/workflows.md and six separate skills in zapier/agent-skills (install, doctor, create, list, history, modify) that each repeated the same compatibility-gate boilerplate. Per agentskills.io's guidance on skill scoping, building/testing/deploying/listing/modifying a workflow is one coherent task, not six. zapier-workflows replaces all of that with a single skill: a SKILL.md under the 500-line budget that routes to four reference files (setup, build-and-deploy, list-and-history, modify) loaded on demand. Dropped the workflows-doctor SDK-CLI-drift/freshness-check apparatus in favor of zapier-sdk's existing pattern: trust the live `--help` output over hardcoded version assumptions. zapier-sdk's own workflows.md and durable example skeleton are removed in favor of a pointer to the new skill, eliminating the duplication.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Replaces the fragmented durable-workflow skill guidance with a single
zapier-workflowsskill, following agentskills.io's scoping guidance.Today, durable workflow guidance lives in two places that duplicate and drift from each other:
zapier-sdk's ownreferences/workflows.mdzapier/agent-skills(workflows-install,-doctor,-create,-list,-history,-modify), each repeating the same "Compatibility Gate" boilerplatePer agentskills.io: "a skill for querying a database and formatting results may be one coherent unit... skills scoped too narrowly force multiple skills to load for a single task, risking overhead and conflicting instructions." Building, testing, deploying, listing, and modifying a workflow is one coherent task a user does in a single sitting — not six.
What changed
skills/zapier-workflows/— one skill,SKILL.md(77 lines, well under the 500-line/~5k-token budget) that routes to four reference files loaded on demand:references/setup.md— install/auth/EA-access flow (fromworkflows-install)references/build-and-deploy.md— the 7-phase build/test/deploy flow (fromworkflows-create, folding in the parts ofworkflows.mdthat weren't duplicated: idempotency-key table, naming convention, catch-hook triggers, escape hatches, explorer-subagent pointer)references/list-and-history.md— merged fromworkflows-list+workflows-historyreferences/modify.md— fromworkflows-modifyskills/zapier-sdk/references/workflows.mdand the durable-workflow skeleton inexamples.md— both replaced with a pointer to the new skillREADME.md,AGENTS.md,llms.txt,skills.sh.json,.codex-plugin/plugin.jsonto register the new skillDeliberate design choice: dropped the
workflows-doctormechanismworkflows-doctordid two things: (1) a throttled daily freshness check that re-pulled skill content across 6 independently-drifting files, and (2) a hand-maintainedsdk_cli_min/sdk_cli_validatedcompatibility contract between the skill and the SDK CLI. Both existed because the CLI and the skill lived in separate repos with separate release cadences. Now that this is one skill in the same repo as the CLI it documents, I replaced both withzapier-sdk's existing lighter pattern: tell the agent to trust the livezapier-sdk --experimental <cmd> --helpoutput over hardcoded assumptions, rather than maintaining a version-drift detector. Flagging this explicitly in case there's a reason to keep the heavier mechanism I'm not seeing.Not in this PR
#team-agent-discoveryraised this consolidation with the team; Chris McLachlan-Lee'scosub-bencheval harness currently points at theagent-skillslayout for the install/create/modify skills. That harness needs to be repointed at this new location before/alongside merging — flagging so it isn't dropped.zapier/agent-skills: left as a follow-up PR in that repo once this lands, rather than bundling a cross-repo change here.Test plan
zapier-sdk --experimental --helpcommand surface spot-checked against what's documented inbuild-and-deploy.md/setup.md/modify.md(content carried over verbatim from the source skills, not re-verified against a live CLI in this pass)skills.sh.jsonschema still validatescosub-benchcan be repointed atsdk/skills/zapier-workflows