Skip to content

Modernize action packaging/runtime and Dependabot policy - #4

Merged
GalvinPython merged 3 commits into
mainfrom
copilot/improve-repo-structure
Sep 23, 2026
Merged

GalvinPython merged 3 commits into
mainfrom
copilot/improve-repo-structure

Conversation

Copilot AI commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

This PR restructures the action to follow GitHub’s expected JavaScript action distribution model (bundled artifact committed, node_modules not committed). It also upgrades Node runtime usage to Node 24 and tightens dependency update cadence/policy in Dependabot.

  • Action packaging + runtime contract

    • Switched action entrypoint from source to bundled artifact (dist/index.js).
    • Updated action.yml runtime from node20 to node24.
    • Added bundling workflow via @vercel/ncc and package scripts to produce distributable output.
  • Repository hygiene

    • Added .gitignore rules for node_modules/ and npm debug logs.
    • Removed tracked node_modules content from the repository to keep source control clean and deterministic.
  • Dependency updates

    • Upgraded action toolkit dependency (@actions/core) to current compatible major for this codebase/runtime.
    • Refreshed lockfile to reflect the updated dependency graph.
  • Docs and consumer workflow alignment

    • Updated README examples to current ecosystem expectations:
      • Node 24
      • actions/checkout@v4
      • actions/setup-node@v4
  • Dependabot policy

    • Changed npm update schedule from weekly to monthly.
    • Configured cooldown to require package versions be at least 3 days old before version-update PRs.
# action.yml
runs:
  using: 'node24'
  main: 'dist/index.js'

Copilot AI and others added 2 commits September 23, 2026 21:30
Co-authored-by: GalvinPython <77013913+GalvinPython@users.noreply.github.com>
Co-authored-by: GalvinPython <77013913+GalvinPython@users.noreply.github.com>
Co-authored-by: GalvinPython <77013913+GalvinPython@users.noreply.github.com>

@GalvinPython GalvinPython left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah i think this is fine - yolo

@GalvinPython
GalvinPython marked this pull request as ready for review September 23, 2026 21:43
Copilot AI lite review requested due to automatic review settings September 23, 2026 21:43
@GalvinPython
GalvinPython merged commit 06d5797 into main Sep 23, 2026
1 check passed
@GalvinPython
GalvinPython deleted the copilot/improve-repo-structure branch September 23, 2026 21:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

One or more issues must be addressed before approval.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 High severity

Open (1)
What changed in this PR

Modernizes the JavaScript action packaging for Node 24 and removes committed dependencies.

Changes:

  • Switches action execution to bundled dist/index.js.
  • Adds ncc build scripts and updates dependencies.
  • Updates documentation and Dependabot cadence.
File Description
README.md Updated as part of this pull request.
package.json Updated as part of this pull request.
node_modules/​uuid/​wrapper.mjs Updated as part of this pull request.
node_modules/​uuid/​LICENSE.md Updated as part of this pull request.
node_modules/​uuid/​dist/​version.js Updated as part of this pull request.
node_modules/​uuid/​dist/​validate.js Updated as part of this pull request.
node_modules/​uuid/​dist/​v5.js Updated as part of this pull request.
node_modules/​uuid/​dist/​v4.js Updated as part of this pull request.
node_modules/​uuid/​dist/​v35.js Updated as part of this pull request.
node_modules/​uuid/​dist/​v3.js Updated as part of this pull request.
node_modules/​uuid/​dist/​uuid-bin.js Updated as part of this pull request.
node_modules/​uuid/​dist/​umd/​uuidVersion.min.js Updated as part of this pull request.
node_modules/​uuid/​dist/​umd/​uuidValidate.min.js Updated as part of this pull request.
node_modules/​uuid/​dist/​umd/​uuidv4.min.js Updated as part of this pull request.
node_modules/​uuid/​dist/​umd/​uuidv1.min.js Updated as part of this pull request.
node_modules/​uuid/​dist/​umd/​uuidStringify.min.js Updated as part of this pull request.
node_modules/​uuid/​dist/​umd/​uuidParse.min.js Updated as part of this pull request.
node_modules/​uuid/​dist/​umd/​uuidNIL.min.js Updated as part of this pull request.
node_modules/​uuid/​dist/​stringify.js Updated as part of this pull request.
node_modules/​uuid/​dist/​sha1.js Updated as part of this pull request.
node_modules/​uuid/​dist/​rng.js Updated as part of this pull request.
node_modules/​uuid/​dist/​rng-browser.js Updated as part of this pull request.
node_modules/​uuid/​dist/​regex.js Updated as part of this pull request.
node_modules/​uuid/​dist/​parse.js Updated as part of this pull request.
node_modules/​uuid/​dist/​nil.js Updated as part of this pull request.
node_modules/​uuid/​dist/​md5.js Updated as part of this pull request.
node_modules/​uuid/​dist/​index.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​version.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​validate.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​v5.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​v4.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​v35.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​v3.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​stringify.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​sha1.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​rng.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​regex.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​parse.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​nil.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​md5.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-node/​index.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-browser/​version.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-browser/​validate.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-browser/​v5.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-browser/​v4.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-browser/​v35.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-browser/​v3.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-browser/​stringify.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-browser/​rng.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-browser/​regex.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-browser/​parse.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-browser/​nil.js Updated as part of this pull request.
node_modules/​uuid/​dist/​esm-browser/​index.js Updated as part of this pull request.
node_modules/​uuid/​dist/​bin/​uuid Updated as part of this pull request.
node_modules/​uuid/​CONTRIBUTING.md Updated as part of this pull request.
node_modules/​undici/​types/​websocket.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​retry-handler.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​README.md Updated as part of this pull request.
node_modules/​undici/​types/​readable.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​proxy-agent.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​pool.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​pool-stats.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​patch.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​mock-pool.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​mock-interceptor.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​mock-errors.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​mock-client.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​mock-agent.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​interceptors.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​index.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​header.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​handlers.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​global-origin.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​global-dispatcher.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​filereader.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​file.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​errors.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​diagnostics-channel.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​cookies.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​content-type.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​connector.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​cache.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​balanced-pool.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​api.d.ts Updated as part of this pull request.
node_modules/​undici/​types/​agent.d.ts Updated as part of this pull request.
node_modules/​undici/​LICENSE Updated as part of this pull request.
node_modules/​undici/​lib/​websocket/​symbols.js Updated as part of this pull request.
node_modules/​undici/​lib/​websocket/​frame.js Updated as part of this pull request.
node_modules/​undici/​lib/​websocket/​constants.js Updated as part of this pull request.
node_modules/​undici/​lib/​timers.js Updated as part of this pull request.
node_modules/​undici/​lib/​pool.js Updated as part of this pull request.
node_modules/​undici/​lib/​pool-stats.js Updated as part of this pull request.
node_modules/​undici/​lib/​mock/​pluralizer.js Updated as part of this pull request.
node_modules/​undici/​lib/​mock/​pending-interceptors-formatter.js Updated as part of this pull request.
node_modules/​undici/​lib/​mock/​mock-symbols.js Updated as part of this pull request.
node_modules/​undici/​lib/​mock/​mock-pool.js Updated as part of this pull request.
node_modules/​undici/​lib/​mock/​mock-errors.js Updated as part of this pull request.
node_modules/​undici/​lib/​mock/​mock-client.js Updated as part of this pull request.
node_modules/​undici/​lib/​llhttp/​wasm_build_env.txt Updated as part of this pull request.
node_modules/​undici/​lib/​llhttp/​utils.js.map Updated as part of this pull request.
node_modules/​undici/​lib/​llhttp/​utils.js Updated as part of this pull request.
node_modules/​undici/​lib/​llhttp/​utils.d.ts Updated as part of this pull request.
node_modules/​undici/​lib/​interceptor/​redirectInterceptor.js Updated as part of this pull request.
node_modules/​undici/​lib/​handler/​DecoratorHandler.js Updated as part of this pull request.
node_modules/​undici/​lib/​global.js Updated as part of this pull request.
node_modules/​undici/​lib/​fileapi/​symbols.js Updated as part of this pull request.
node_modules/​undici/​lib/​fileapi/​progressevent.js Updated as part of this pull request.
node_modules/​undici/​lib/​fetch/​symbols.js Updated as part of this pull request.
node_modules/​undici/​lib/​fetch/​LICENSE Updated as part of this pull request.
node_modules/​undici/​lib/​fetch/​global.js Updated as part of this pull request.
node_modules/​undici/​lib/​dispatcher.js Updated as part of this pull request.
node_modules/​undici/​lib/​core/​symbols.js Updated as part of this pull request.
node_modules/​undici/​lib/​core/​constants.js Updated as part of this pull request.
node_modules/​undici/​lib/​cookies/​constants.js Updated as part of this pull request.
node_modules/​undici/​lib/​compat/​dispatcher-weakref.js Updated as part of this pull request.
node_modules/​undici/​lib/​cache/​util.js Updated as part of this pull request.
node_modules/​undici/​lib/​cache/​symbols.js Updated as part of this pull request.
node_modules/​undici/​lib/​api/​util.js Updated as part of this pull request.
node_modules/​undici/​lib/​api/​index.js Updated as part of this pull request.
node_modules/​undici/​lib/​api/​api-upgrade.js Updated as part of this pull request.
node_modules/​undici/​lib/​api/​api-connect.js Updated as part of this pull request.
node_modules/​undici/​lib/​api/​abort-signal.js Updated as part of this pull request.
node_modules/​undici/​index.d.ts Updated as part of this pull request.
node_modules/​undici/​index-fetch.js Updated as part of this pull request.
node_modules/​undici/​docs/​best-practices/​writing-tests.md Updated as part of this pull request.
node_modules/​undici/​docs/​best-practices/​proxy.md Updated as part of this pull request.
node_modules/​undici/​docs/​best-practices/​client-certificate.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​WebSocket.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​RetryHandler.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​PoolStats.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​Pool.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​MockErrors.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​MockClient.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​Fetch.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​Errors.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​DispatchInterceptor.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​Cookies.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​ContentType.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​Connector.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​CacheStorage.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​BalancedPool.md Updated as part of this pull request.
node_modules/​undici/​docs/​api/​Agent.md Updated as part of this pull request.
node_modules/​tunnel/​package.json Updated as part of this pull request.
node_modules/​tunnel/​LICENSE Updated as part of this pull request.
node_modules/​tunnel/​index.js Updated as part of this pull request.
node_modules/​tunnel/​CHANGELOG.md Updated as part of this pull request.
node_modules/​tunnel/​.travis.yml Updated as part of this pull request.
node_modules/​tunnel/​.idea/​vcs.xml Updated as part of this pull request.
node_modules/​tunnel/​.idea/​node-tunnel.iml Updated as part of this pull request.
node_modules/​tunnel/​.idea/​modules.xml Updated as part of this pull request.
node_modules/​tunnel/​.idea/​encodings.xml Updated as part of this pull request.
node_modules/​@fastify/​busboy/​package.json Updated as part of this pull request.
node_modules/​@fastify/​busboy/​LICENSE Updated as part of this pull request.
node_modules/​@fastify/​busboy/​lib/​utils/​getLimit.js Updated as part of this pull request.
node_modules/​@fastify/​busboy/​lib/​utils/​decodeText.js Updated as part of this pull request.
node_modules/​@fastify/​busboy/​lib/​utils/​Decoder.js Updated as part of this pull request.
node_modules/​@fastify/​busboy/​lib/​utils/​basename.js Updated as part of this pull request.
node_modules/​@fastify/​busboy/​lib/​main.js Updated as part of this pull request.
node_modules/​@fastify/​busboy/​deps/​dicer/​LICENSE Updated as part of this pull request.
node_modules/​@fastify/​busboy/​deps/​dicer/​lib/​PartStream.js Updated as part of this pull request.
node_modules/​@fastify/​busboy/​deps/​dicer/​lib/​HeaderParser.js Updated as part of this pull request.
node_modules/​@actions/​http-client/​README.md Updated as part of this pull request.
node_modules/​@actions/​http-client/​package.json Updated as part of this pull request.
node_modules/​@actions/​http-client/​LICENSE Updated as part of this pull request.
node_modules/​@actions/​http-client/​lib/​proxy.js Updated as part of this pull request.
node_modules/​@actions/​http-client/​lib/​proxy.d.ts Updated as part of this pull request.
node_modules/​@actions/​http-client/​lib/​interfaces.js.map Updated as part of this pull request.
node_modules/​@actions/​http-client/​lib/​interfaces.js Updated as part of this pull request.
node_modules/​@actions/​http-client/​lib/​interfaces.d.ts Updated as part of this pull request.
node_modules/​@actions/​http-client/​lib/​auth.js.map Updated as part of this pull request.
node_modules/​@actions/​http-client/​lib/​auth.js Updated as part of this pull request.
node_modules/​@actions/​http-client/​lib/​auth.d.ts Updated as part of this pull request.
node_modules/​@actions/​core/​package.json Updated as part of this pull request.
node_modules/​@actions/​core/​LICENSE.md Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​utils.js.map Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​utils.js Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​utils.d.ts Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​path-utils.js.map Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​path-utils.js Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​path-utils.d.ts Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​oidc-utils.js.map Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​oidc-utils.js Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​oidc-utils.d.ts Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​file-command.js.map Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​file-command.js Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​file-command.d.ts Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​command.js.map Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​command.js Updated as part of this pull request.
node_modules/​@actions/​core/​lib/​command.d.ts Updated as part of this pull request.
node_modules/​.package-lock.json Updated as part of this pull request.
node_modules/​.bin/​uuid.ps1 Updated as part of this pull request.
node_modules/​.bin/​uuid.cmd Updated as part of this pull request.
node_modules/​.bin/​uuid Updated as part of this pull request.
action.yml Updated as part of this pull request.
.gitignore Updated as part of this pull request.
.github/​dependabot.yml Updated as part of this pull request.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread package.json
"description": "",
"main": "index.js",
"scripts": {
"test": "echo \"Error: no test specified\" && exit 1"
"test": "echo \"Error: no test specified\" && exit 1",
"build": "ncc build index.js -o dist",
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.

3 participants