Skip to content

docs: reconcile release model to gated single-branch publish (delete PUBLISH_ON_MERGE) - #275

Merged
ptr727 merged 4 commits into
developfrom
feature/release-model-docs
Jul 12, 2026
Merged

docs: reconcile release model to gated single-branch publish (delete PUBLISH_ON_MERGE)#275
ptr727 merged 4 commits into
developfrom
feature/release-model-docs

Conversation

@ptr727

Copy link
Copy Markdown
Owner

Reconciles the governance docs to the release model agreed with the maintainer, and clears two pre-existing inconsistencies.

Model (now documented)

A human PR merge never auto-publishes. A release is:

  • a workflow_dispatch of main/develop (human-initiated), or
  • a code-affecting bot push to main — the codegen App merges every Dependabot/codegen PR, so github.actor is the gate; a human merge/promotion to main skips, and an Actions-only bump is dropped by the paths filter, or
  • a weekly schedule (Docker only, base-image refresh).

The decision is single-sourced in the plan job; every publish job gates on its output.

Changes

  • AGENTS.md lines 13/29/52 (+ 54/56/224): the gated model; delete every PUBLISH_ON_MERGE reference (it was implemented in zero repos — pure removal).
  • WORKFLOW.md §105, D4.1, D8.3, S5/S6/S7/S11: same model; reference publish-plan-task.
  • Cleared the stale "branch matrix" language in AGENTS.md — WORKFLOW.md §105 already says the publisher builds the single trigger branch ("no branch matrix"); AGENTS.md hadn't caught up. (Flagged for your veto — this was the one item I wasn't 100% sure was stale; verified against every repo's publish-release.yml, which all build one branch.)

markdownlint + spec/validate.py clean; CRLF preserved. Codifies the rollout PRs (LanguageTags #255, Utilities #388, aiopurpleair #26, NxWitness #494, MediaTools #8, ESPHome-NonRoot #138) + the snippet #274.

🤖 Generated with Claude Code

A human PR merge never auto-publishes: a release is a workflow_dispatch, a
code-affecting bot push to main (gated to the codegen App / Dependabot
actor by the plan job), or the Docker weekly schedule. Rewrite AGENTS.md
(lines 13/29/52) and WORKFLOW.md (§105, D4.1, D8.3, S5/S6/S7/S11) to this
model, reference publish-plan-task, delete every PUBLISH_ON_MERGE mention
(implemented in zero repos), and clear the stale "branch matrix" language
(the publisher builds the single trigger branch, per WORKFLOW.md §105).
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
CopilotAI review requested due to automatic review settings July 10, 2026 14:50

CopilotAI 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 repo's governance documentation to describe a gated, single-branch publish model (human merges do not auto-publish) and removes/updates legacy language tied to the prior release model.

Changes:

  • Rewrites the documented release/publish triggers to a gated model (dispatch + bot push to main + scheduled Docker refresh).
  • Removes PUBLISH_ON_MERGE references and updates surrounding rationale to match the new model.
  • Updates upstream-version tracker and trace-scenario wording to align with the new gating rules.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 7 comments.

FileDescription
AGENTS.mdUpdates foundational and release-model governance text to the new gated publish model; removes legacy PUBLISH_ON_MERGE language.
WORKFLOW.mdUpdates the release-model contract and scenarios to the gated single-branch publish model and references a planned single-source publish decision.

Comment threadAGENTS.md
Comment threadWORKFLOW.md
Comment threadWORKFLOW.md Outdated
Comment threadWORKFLOW.md Outdated
Comment threadAGENTS.md
Comment threadAGENTS.md Outdated
Comment threadWORKFLOW.md
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
CopilotAI review requested due to automatic review settings July 10, 2026 14:57

CopilotAI 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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

Comment threadAGENTS.md
Comment threadWORKFLOW.md
Comment threadWORKFLOW.md
Comment threadAGENTS.md
The publisher builds one branch per run (the trigger ref), not both main and develop;
the branch input is still required so a fallback can't mislabel the build. Matches the
reconciled single-branch release model in this PR.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
CopilotAI review requested due to automatic review settings July 12, 2026 04:51

CopilotAI 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

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

Comment threadAGENTS.md
Comment threadWORKFLOW.md
Comment threadWORKFLOW.md
Comment threadWORKFLOW.md
CopilotAI review requested due to automatic review settings July 12, 2026 04:55

CopilotAI 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

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

@ptr727
ptr727 merged commit 1d409b3 into developJul 12, 2026
7 checks passed
@ptr727
ptr727 deleted the feature/release-model-docs branch July 12, 2026 04:58
ptr727 added a commit that referenced this pull request Jul 12, 2026
…rkflow comments (#286)
Forward promotion of #274 + #275 + #277 (and prior develop work). No
release.
Sign up for freeto 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.

2 participants

@ptr727