-
Notifications
You must be signed in to change notification settings - Fork 42
Remove inline mode and externalize all scripts via setup action #6912
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove inline mode and externalize all scripts via setup action #6912
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot you most attach the GitHub builtin objects to global (core, GitHub, exec, io) |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot infer action mode from the tag stored in the cli binary |
|
@copilot in action mode local, you must checkout the actions folder (sparse) in the activation job |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Fixed in 6ccc3d3. Action mode now inferred from binary version tag:
|
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
✅ Dev Hawk Report - SuccessWorkflow: #3215
Dev workflow completed successfully! 🎉 Workflow SummaryAll 5 jobs completed successfully:
Key Changes ValidatedThis run validated the latest fix for
The workflow demonstrates that the externalized script pattern is working correctly with the setup action.
|
|
@copilot merge main, format, lint, fix tests |
…-action-setup-activation # Conflicts: # .github/workflows/playground-org-project-update-issue.lock.yml
- Successfully merged origin/main (resolved conflict in playground-org-project-update-issue.lock.yml) - Formatted all code with make fmt (all files unchanged) - Linted all code with make lint (0 issues, all validations passed) - Rebuilt binary with latest changes (version 340474d-dirty) - Rebuilt all actions with make actions-build (117 scripts in setup action) - Recompiled all 123 workflows successfully (123/123) - Total: 747 jobs, 9767 steps, 3446 scripts (36.8 MB) - All workflows use local action path ./actions/setup in dev mode - Note: Some workflow tests need updates for new require() pattern (expected breaking change from inline mode removal) Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Completed in b92ebf3. Successfully merged origin/main, formatted code (make fmt), linted code (make lint - 0 issues), and recompiled all 123 workflows (123/123). Note: Some workflow tests are failing because they check for inline scripts, which have been removed. These tests need to be updated to validate the new require() pattern. The failing tests are:
These failures are expected as part of the breaking change to remove inline mode. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot merge main, recompile, fix tests |
…-action-setup-activation # Conflicts: # .github/workflows/dev.lock.yml # .github/workflows/playground-org-project-update-issue.lock.yml
Fixed in c51d14f. Successfully merged origin/main, recompiled all 123 workflows, and verified all tests pass. The full test suite now completes without any failures - all workflow tests correctly validate the new require() pattern. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Smoke Test ResultsLast 2 Merged PRs:
Test Results:
Status: PASS (4/4 core tests)
|
Smoke Test Results: Copilot Safe Inputs✅ File Writing Test - Created Status: PARTIAL PASS (2/3) Note:
|
|
PRs: Reduce CI cleaner frequency from hourly to 2-3 hour intervals; Document network ecosystem identifiers for package registries and CDNs
|
Smoke Test Results (Run 20469540121)✅ Playwright MCP: Successfully navigated to GitHub and verified page title Overall Status: PARTIAL PASS (2/3 tests passed) Note:
|
Smoke Test Results (Claude Engine)Recent PRs:
Test Results:
Status: PASS (5/6 tests passed)
|
|
Recent merged PRs: Reduce CI cleaner frequency from hourly to 2-3 hour intervals; Document network ecosystem identifiers for package registries and CDNs
|
Implementation Complete ✅
BREAKING CHANGE: Removed inline script mode and renamed action from
setup-activationtosetup. All workflows now use the setup action to externalize JavaScript scripts viarequire()calls.Latest Update
✅ Merged main, recompiled, and all tests pass (commit c51d14f)
Test Status
✅ All tests passing - No test failures
Verification Summary
✅ All externalized scripts use absolute require paths
/tmp/gh-aw/actions/load_agent_output.cjs✅ Setup action uses file copying instead of embedding
actions/setup/js/directorycpcommands to copy files fromjs/to/tmp/gh-aw/actions/✅ All scripts correctly export main without calling it
{ main }viamodule.exportsawait main()calls in the source filesconst { main } = require(...); await main();✅ Build process properly copies JavaScript files
/tmp/gh-aw/actions/at runtime by setup.sh✅ Lock file validation passed
./actions/setup) in dev modeSummary
This PR successfully removes inline mode and externalizes all JavaScript scripts via the setup action. All workflows now use a consistent pattern with the setup action loading scripts that are then executed via
require()calls. The implementation is complete, tested, and ready for use.Key Benefits:
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.