Skip to content

Comments

Update API URL to use 'platform' instead of 'api'; Rebuilt to fix source-type error#2

Merged
tode-rl merged 3 commits intomainfrom
tode-rl-patch-1
Nov 17, 2025
Merged

Update API URL to use 'platform' instead of 'api'; Rebuilt to fix source-type error#2
tode-rl merged 3 commits intomainfrom
tode-rl-patch-1

Conversation

@tode-rl
Copy link
Contributor

@tode-rl tode-rl commented Nov 17, 2025

@tode-rl tode-rl requested a review from wall-rl November 17, 2025 22:28
tode-rl and others added 2 commits November 17, 2025 14:53
This fixes the issue where the action would fail on GitHub Actions cloud
with "Input required and not supplied: source-type" while working fine
with act locally.

Root cause:
The composite action structure with runtime dependency installation and
bash-based environment variable passing had issues on GitHub Actions cloud.
The environment variables were not being properly scoped when running
"cd ${{ github.action_path }} && node dist/index.js" in the bash step.

Solution:
- Bundle all dependencies using @vercel/ncc into a single dist/index.js
- Switch from composite action to direct node20 action
- This allows GitHub Actions to natively handle INPUT_* env vars
- Eliminates runtime dependency installation (faster execution)
- More reliable and follows GitHub Actions best practices

Changes:
- Added @vercel/ncc for dependency bundling
- Updated build script to use ncc instead of tsc
- Changed action.yml to use 'using: node20' instead of 'composite'
- Updated README with ncc build instructions and act testing info
- Fixed all example workflows to use runloopai/deploy-agent@main
- Bumped version to 1.0.1
- Added .gitattributes to mark dist folder as generated

The bundled dist/index.js is now ~2MB and includes all dependencies,
making the action fully self-contained and compatible with both GitHub
Actions cloud and act.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@tode-rl tode-rl changed the title Update API URL to use 'platform' instead of 'api' Update API URL to use 'platform' instead of 'api'; Rebuilt to fix source-type error Nov 17, 2025
@tode-rl tode-rl merged commit aa7177b into main Nov 17, 2025
1 check passed
@tode-rl tode-rl deleted the tode-rl-patch-1 branch November 17, 2025 22:56
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