plugins/flow/skills/merge-and-release/SKILL.md
Reference document describing merge prerequisites (approval, CI checks, mergeable, conversations resolved, stale approval), release versioning (semantic semver), and changelog generation. Explains why Tier 3 confirmation is structural: merge and release cost is borne by downstream people. Reference only (`disable-model-invocation: true`); consumed by `/flow:merge` and `/flow:release`.
npx skillsauth add synaptiai/synapti-marketplace merge-and-releaseInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
Reference document for merge and release policy. The executable bash lives in plugins/flow/commands/merge.md and plugins/flow/commands/release.md. This skill describes what those commands enforce and why.
Both merge and release are Tier 3 — they always require explicit human confirmation, even in autonomous mode. This is non-negotiable.
MERGE IS IRREVERSIBLE IN PRACTICE. Treat every merge as permanent. Every prerequisite must be verified with fresh evidence, not memory.
Reverting a merge is technically possible in git but disruptive in practice — downstream branches rebase off the merge, deploys propagate it, and history grows confusing. Prevention is the only reliable strategy. The Tier 3 confirmation requirement is the structural expression of that fact.
Before a merge proceeds, all five checks must pass. The runnable verification is in plugins/flow/commands/merge.md Phase 1.
| # | Check | Requirement | What it protects against |
|---|-------|-------------|--------------------------|
| 1 | Approval | At least one approval, no outstanding requested changes | Merging code that no human has signed off on |
| 2 | CI Checks | All status checks pass (statusCheckRollup all success) | Merging code that fails the project's automated tests |
| 3 | Mergeable | No merge conflicts (mergeable == "MERGEABLE") | Merging a PR whose content cannot cleanly land on the base branch |
| 4 | Conversations | All review threads resolved (unresolved count == 0) | Closing reviewer concerns by ignoring them |
| 5 | Stale approval | No commits after the last approval timestamp | Merging code that was approved before the latest changes existed |
The conversation-resolution check requires GitHub's GraphQL API (the REST reviewThreads field is incomplete). The exact GraphQL query used is in plugins/flow/commands/merge.md.
When any of these conditions is detected, the command stops without asking "merge anyway?":
pr-lifecycle skill for details)There is no "override the gate" option in autonomous mode. Overrides require the human user to take the action themselves outside the command.
A "stale" approval is one that predates the most recent commit on the PR. The check compares:
submittedAt timestamp among reviews where state == "APPROVED"committedDate of the most recent commit on the PRIf commits exist after the last approval, the command warns: "Approval may be stale." Stale approvals are a stop condition because the approver has not actually seen what is being merged.
After all prerequisites pass, the command displays a Merge Assessment table and explicitly asks the user to confirm. Only after a clear "yes" does the merge proceed:
squash | merge | rebase, default squash)true)gh pr merge invocation is in plugins/flow/commands/merge.md Phase 3The user-visible assessment lists every prerequisite's status, the chosen strategy, and the branch-delete decision so the human can verify before approving.
Releases follow the same Tier 3 confirmation discipline as merges. The runnable bash is in plugins/flow/commands/release.md.
| Bump type | Format | When to use |
|-----------|--------|-------------|
| patch | 0.0.X+1 | Bug fixes only, no behavior change |
| minor | 0.X+1.0 | New features, no breaking changes |
| major | X+1.0.0 | Breaking changes |
The first release defaults to v1.0.0 if no prior tag exists.
The changelog is built from merged PRs since the last release tag, categorized by conventional commit prefix:
feat: PRsfix: PRsdocs:, chore:, etc.Each entry includes the PR number and author. A "Full Changelog" link compares the previous tag to the new one.
After the changelog and version are displayed, the command asks the user to confirm. Only after explicit confirmation:
git tag -a "$TAG" -m "Release $TAG"git push origin "$TAG"gh release create "$TAG" --title "$TAG" --notes "$CHANGELOG"These three operations together are the release. They are not split across multiple agent turns — the user confirms once and the command performs them in sequence.
After either operation completes, the command verifies state:
gh pr view $PR_NUM --json state should return MERGEDgh release view $TAG should succeedThe command then suggests cleanup: switch to the default branch, pull latest, and (for releases) update plugin version files if applicable.
Merges and releases produce visible, durable artifacts: a merge commit on main, a published tag, a GitHub release page that subscribers may receive notifications for. The cost of an unwanted merge or release is paid by everyone downstream — other developers rebasing, users seeing a notification for a release that was a mistake, package managers fetching the new version.
Tier 3 is not gatekeeping for its own sake. It is the structural expression of: the cost of an unwanted action is borne by people who are not in this conversation. The human in the loop is the only person who can speak for those downstream people.
This skill is reference-only. The runnable implementations are:
plugins/flow/commands/merge.mdplugins/flow/commands/release.mdWhen the policy needs to evolve — a new prerequisite, a new release strategy, a different versioning scheme — update the command and update this reference together. The bash is the runtime; this document is the rationale.
tools
Validate a FlowWorkflow YAML at `plugins/flow/workflows/<id>.workflow.yaml` against `schemas/v1/workflow.schema.json` AND cross-reference the referenced skills/agents exist + every Tier 3 action is confirm-gated + no native /goal or /loop dependency is declared. Use when /flow:workflow validate is invoked, when CI runs the workflow schema gates, or when a new workflow is being authored. This skill MUST be consulted because schema validation alone catches shape errors; cross-reference validation catches the silent-correctness failures (typo'd skill name, Tier 3 escape, /goal dependency) that would otherwise ship to users.
tools
Verify UI-facing changes by running a screenshot-analyze-verify loop across configured viewports, with a browser-tool priority cascade (Playwright MCP → Chrome DevTools MCP → CLI fallback → external skill fallback) and bounded iteration. Use after build/runtime verification passes and the diff includes `.tsx`/`.jsx`/`.vue`/`.html`/`.css`/`.scss`/`.svelte` files OR the acceptance criteria mention UI/page/render/display/visual. This skill MUST be consulted because UI changes that pass build and unit tests can still ship blank pages, render-blocking console errors, or broken responsive layouts that no other verification phase catches.
data-ai
Coordinate agent teams for adversarial review (paired skeptic/verifier per facet, challenge round with disposition vocabulary, consolidated findings with confidence) or parallel implementation (task sizing 5-6 per teammate, non-overlapping files). Enforces independent analysis before shared conclusions. Reference only (`disable-model-invocation: true`); loaded only when `agentTeams: true` in settings.
development
Conduct two-stage code review: Stage 1 verifies spec compliance (criterion-to-code mapping), Stage 2 evaluates security, correctness, performance, and maintainability across 6 parallel facets with P1/P2/P3 synthesis and deduplication by file:line. Use when reviewing code changes or pull requests. This skill MUST be consulted because reviewing quality on broken logic is wasted effort, and unmet acceptance criteria must block merge.