Skip to content

refactor: rename trigger-website-update to dispatch-with-fallback - #15

Merged
baxyz merged 1 commit into
mainfrom
refactor/genericize-trigger-website-update
Jul 29, 2026
Merged

refactor: rename trigger-website-update to dispatch-with-fallback#15
baxyz merged 1 commit into
mainfrom
refactor/genericize-trigger-website-update

Conversation

@baxyz

@baxyz baxyz commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Per discussion: trigger-website-update was too helpers4-specific for a repo whose stated purpose is publishing generic actions to the GitHub Marketplace. Its actual mechanics were already 100% generic (mint a GitHub App token → repository-dispatch, retry with a fallback identity on failure) — only the name and the target-owner: helpers4/target-repo: website defaults were org-specific.
  • Renamed trigger-website-updatedispatch-with-fallback, made target-owner/target-repo required (no defaults) instead of defaulting to helpers4/website
  • Verified the other 3 actions (conventional-commits, setup-pnpm, pr-status-comment) have no comparable org-specific defaults — scopes.json/.vscode/settings.json are generic filename conventions, not helpers4-branded
  • Updated this repo's own usage (release.yml) to pass target-owner: helpers4 / target-repo: website explicitly
  • Follow-up PRs needed in devcontainer, typescript, .github to update their call sites to the new name + explicit target inputs — opening those next, this is deliberately first so the new path exists on main before they reference it

Test plan

  • CI: shellcheck, manifest validation, existing smoke tests for the other 2 actions unaffected
  • Confirm dispatch-with-fallback isn't referenced anywhere by its old name after merge

🤖 Generated with Claude Code

…llback

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

✅ PR Validation Passed

All checks passed!


📋 Pipeline Status

Job Status
🧾 Conventional Commits passing
🐚 ShellCheck passing
🧪 Action Tests passing
📄 Manifest Validation passing
📦 setup-pnpm Tests passing
💬 pr-status-comment Tests passing

🤖 Generated by @helpers4 CI • 2026-07-29

@baxyz
baxyz merged commit c66ddc3 into main Jul 29, 2026
10 checks passed
@baxyz
baxyz deleted the refactor/genericize-trigger-website-update branch July 29, 2026 13:30
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