Skip to content

feat(workflow-templates): add dependabot-auto-merge caller stub + properties - #7

Merged
twistedmelonman merged 2 commits into
mainfrom
claude/dependabot-auto-merge-caller-stub
Aug 7, 2026
Merged

feat(workflow-templates): add dependabot-auto-merge caller stub + properties#7
twistedmelonman merged 2 commits into
mainfrom
claude/dependabot-auto-merge-caller-stub

Conversation

@twistedmelonman

Copy link
Copy Markdown
Contributor

Summary

Mirrors smartwatermelon/.github#11. Part of Part A2/A4 of the reusable-workflows plan. Replaces this template's self-contained dependabot-auto-merge.yml body with a caller stub pointing at the new reusable workflow being added in smartwatermelon/github-workflows#96. Adds the missing dependabot-auto-merge.properties.json so the workflow-templates picker actually offers this template (Part B's B1 deliverable per the plan — confirmed silently absent from the picker during Q2 planning validation).

Actor guard — confirmed preserved

The reusable workflow (smartwatermelon/github-workflows commit ad3f0ec, PR #96) keeps the same if: github.actor == 'dependabot[bot]' guard, now at the reusable's auto-merge job level instead of this caller's — it was moved, not dropped.

Blocking on tag creation

Temporarily pinned to the github-workflows commit SHA ad3f0ec080ad5b119f2359bf42de6623c4730544 (the tip of PR #96's branch at the time this PR was opened) rather than a tag, since dependabot-auto-merge-v1 doesn't exist until that PR merges and the tag is cut (~1 minute after merge, per that fleet's convention). Caveat: this SHA is real and fetchable today (it's a pushed branch tip), but if PR #96 is squash-merged, the final commit on github-workflows main will have a different SHA than this one. Follow-up PR needed here regardless — either to repoint to @dependabot-auto-merge-v1 once cut, or sooner if this exact SHA stops resolving.

Test plan

  • YAML valid, yamllint clean (one pre-existing-style line-length warning)
  • JSON valid
  • Local code-reviewer + adversarial-reviewer: PASS
  • Pre-push full-diff + whole-codebase review: PASS (flagged a pre-existing, out-of-scope observation that dependabot.yml's github-actions ecosystem entry doesn't scan workflow-templates/, so pins there won't get automated bump PRs — noting for the record, not fixing here)
  • Manual: once merged, confirm the workflow-templates picker offers "Dependabot Auto-Merge" on a fresh nightowlstudiollc repo (Q2 in the plan)
  • Follow-up: repoint SHA pin to @dependabot-auto-merge-v1 once that tag is cut in github-workflows

Claude-Session: https://claude.ai/code/session_0143Nf4sXqnTp2Kx3LNbMvBA

…perties

Mirrors smartwatermelon/.github#11: replaces the self-contained
dependabot-auto-merge.yml body with a caller stub pointing at the new
reusable workflow in smartwatermelon/github-workflows (Part A2 of the
reusable-workflows plan; see smartwatermelon/github-workflows#96). Adds
the missing dependabot-auto-merge.properties.json so the
workflow-templates picker actually offers this template — confirmed
silently absent during Q2 planning validation.

Temporarily pinned to a commit SHA rather than the dependabot-auto-merge-v1
tag, since that tag doesn't exist until github-workflows#96 merges and the
tag is cut (~1 minute after merge, per this fleet's established
convention). Follow-up PR needed once the tag exists.

Claude-Session: https://claude.ai/code/session_0143Nf4sXqnTp2Kx3LNbMvBA
@claude

This comment has been minimized.

Mirrors smartwatermelon/.github#11: the reusable workflow's
dependabot-auto-merge-v1.0.0 tag (and floating dependabot-auto-merge-v1)
are now cut in smartwatermelon/github-workflows following #96's merge.
Replace the temporary branch-tip SHA pin with the real floating tag, and
document explicitly that floating (versus a stricter pin) is an accepted
tradeoff for this template, plus note that the actor gate and merge-scope
defaults now live entirely inside the reusable workflow, invisible from
this caller stub.
@claude

claude Bot commented Aug 7, 2026

Copy link
Copy Markdown

No blocking issues found. The PR converts an inline dependabot-auto-merge job into a caller stub for a reusable workflow, and adds the required .properties.json picker metadata.

The floating-tag pin (@dependabot-auto-merge-v1) under pull_request_target with contents: write / pull-requests: write is a real supply-chain tradeoff, but it is explicitly documented and acknowledged in the file header as a deliberate decision mirroring the existing claude-blocking-review.yml@v3 pattern.

VERDICT: PASS

@twistedmelonman
twistedmelonman merged commit c19466b into main Aug 7, 2026
2 checks passed
@twistedmelonman
twistedmelonman deleted the claude/dependabot-auto-merge-caller-stub branch August 8, 2026 03:02
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.

1 participant