Skip to content

Conversation

@petetan
Copy link
Contributor

@petetan petetan commented Aug 12, 2025

🚨 Critical Fix: GitHub Action Distribution Files + CHANGELOG

This PR fixes the broken GitHub Action by committing the required distribution files and establishes proper changelog maintenance.

Problem

Users were getting this error when trying to use the action:

Error: File not found: '/home/runner/work/_actions/synctree/releasebot/v1/dist/index.js'

Root Cause

  • The dist/ directory was included in .gitignore
  • GitHub Actions require pre-built JavaScript files to be committed to the repository
  • When users reference @v1, GitHub downloads the repository and expects to find dist/index.js

Solution

  • ✅ Removed dist/ from .gitignore
  • ✅ Committed all compiled distribution files (dist/index.js, type definitions, source maps)
  • ✅ Added comprehensive CHANGELOG.md following Keep a Changelog format
  • ✅ This allows GitHub Actions to find and execute the pre-built JavaScript

Files Changed

  • .gitignore - Removed dist/ exclusion
  • dist/ - Added all 23 compiled distribution files (38,658 lines)
  • CHANGELOG.md - Added changelog documenting v1.0.1 fixes and v1.0.0 initial release

Package Management Note

This PR properly uses pnpm-lock.yaml (not package-lock.json) since we use pnpm as our package manager.

Impact

  • 🔧 Fixes broken action - Users can now successfully use synctree/releasebot@v1
  • 📦 Standard GitHub Actions practice - Most TypeScript-based actions commit their dist/ files
  • 📚 Establishes changelog maintenance - Professional release documentation going forward
  • 🚀 No breaking changes - Existing workflows will work immediately after merge

Testing

After merge, the action should work correctly when referenced as:

- uses: synctree/releasebot@v1

Related Issues & PRs


Priority: Critical - This restores functionality for all users of the action and establishes proper release documentation.

petetan and others added 2 commits August 11, 2025 18:12
- Remove dist/ from .gitignore to allow distribution files
- Add all compiled JavaScript and TypeScript declaration files
- This fixes the 'File not found: index.js' error in GitHub Actions
- GitHub Actions require pre-built distribution files to be committed

Fixes the broken action deployment issue.
- Add comprehensive CHANGELOG.md following Keep a Changelog format
- Document both git symbolic-ref fix (#3) and dist files fix (#7)
- Include proper dates and versioning information
- This establishes changelog maintenance going forward

Co-authored-by: Copilot <copilot@github.com>
@petetan petetan changed the title 🚨 Fix: Commit dist files to fix broken GitHub Action 🚨 Fix: Commit dist files + Add CHANGELOG for GitHub Action Aug 12, 2025
@petetan petetan merged commit 0ab7a40 into main Aug 12, 2025
1 check passed
@petetan petetan deleted the feature/commit-dist-files branch August 12, 2025 03:06
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.

2 participants