Skip to content

🔧 update: move to node.js as runtime engine - #4

Merged
warengonzaga merged 2 commits into
mainfrom
dev
May 12, 2026
Merged

warengonzaga merged 2 commits into
mainfrom
dev

Conversation

@warengonzaga

Copy link
Copy Markdown
Owner

This pull request updates the project to use the latest Node.js runtime (v26+) and adjusts documentation, scripts, and configurations to clarify that the toolchain uses Bun for building and Node.js for execution. It also updates development scripts for a more standard workflow and improves type safety in development.

Runtime and Toolchain Updates

  • Updated the Node.js version in .github/workflows/package.yml and package.json to 26 or higher, ensuring all builds and executions use the latest Node.js runtime. [1] [2]
  • Updated documentation in README.md and CONTRIBUTING.md to clarify that the tool is built with Bun but runs on Node.js, replacing references to dual runtime (npx/bunx) with the new approach. [1] [2] [3]

Development Workflow Improvements

  • Modified package.json scripts so that dev, preview, and start commands build with Bun and execute with Node.js, providing a more consistent and reliable development experience.
  • Updated usage instructions in CONTRIBUTING.md to reflect the new build and run steps using Node.js after building with Bun.

Type Safety Enhancements

  • Added @types/node as a development dependency to improve type safety and editor support when working with Node.js APIs.

Copilot AI review requested due to automatic review settings May 12, 2026 16:46
@github-actions

Copy link
Copy Markdown

🚀 Development Build

Development version ready for integration testing

📦 Package Information

  • Package: github-labels-template
  • Version: 0.9.0-dev.b3751b3
  • Dist-tag: dev

📥 Installation Instructions

NPM Registry

npm install github-labels-template@0.9.0-dev.b3751b3
npm install github-labels-template@dev  # Use dist-tag

View on NPM Registry

GitHub Packages

npm install @warengonzaga/github-labels-template@0.9.0-dev.b3751b3
npm install @warengonzaga/github-labels-template@dev  # Use dist-tag

✨ Auto-scoped as @warengonzaga/github-labels-template (from repository owner)

View on GitHub Packages

🔒 Security Audit

✅ No vulnerabilities found


This package was built automatically by the Package Build Flow action.

@github-actions

Copy link
Copy Markdown

🚀 Development Build

Development version ready for integration testing

📦 Package Information

  • Package: github-labels-template
  • Version: 0.9.0-dev.12d5215
  • Dist-tag: dev

📥 Installation Instructions

NPM Registry

npm install github-labels-template@0.9.0-dev.12d5215
npm install github-labels-template@dev  # Use dist-tag

View on NPM Registry

GitHub Packages

npm install @warengonzaga/github-labels-template@0.9.0-dev.12d5215
npm install @warengonzaga/github-labels-template@dev  # Use dist-tag

✨ Auto-scoped as @warengonzaga/github-labels-template (from repository owner)

View on GitHub Packages

🔒 Security Audit

✅ No vulnerabilities found


This package was built automatically by the Package Build Flow action.

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.

Pull request overview

This PR updates the project to standardize on Node.js (>=26) as the runtime while continuing to use Bun for building/testing, and adjusts scripts/docs/CI accordingly.

Changes:

  • Bump the minimum Node engine requirement and CI Node version to 26.
  • Update package.json scripts to build with Bun and execute the built CLI with Node.
  • Add Node typings and update docs to describe the Node runtime + Bun toolchain approach.

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
tsconfig.json Adds Node types alongside Bun types for TypeScript.
README.md Updates feature messaging and adds runtime note for Node execution.
package.json Updates scripts to run compiled output with Node; bumps Node engine; adds @types/node.
CONTRIBUTING.md Updates prerequisites and local run instructions for “build then run with Node”.
bun.lock Lockfile updates reflecting added Node typings and dependency resolution.
.github/workflows/package.yml Updates CI to use Node 26.
Comments suppressed due to low confidence (1)

README.md:288

  • The added note recommends “Node.js (latest)”, but package.json now requires Node >=26. Consider making the README guidance explicit (e.g., “Node.js 26+”) to avoid confusion for contributors/users.
  migrate    Wipe all existing labels and apply the template (clean slate)
  generate   Generate custom labels using AI (requires GitHub Copilot)

Comment thread README.md
@@ -21,7 +21,7 @@ A CLI tool to apply a curated set of GitHub labels to any repository using `gh`
- 📊 **Clear Output**: Structured logging powered by [@wgtechlabs/log-engine](https://github.com/wgtechlabs/log-engine) with color-coded levels and emoji
- 🎨 **ASCII Banner**: Beautiful ANSI Shadow figlet banner with version and author info
Comment thread package.json Outdated
"preview": "bun src/index.ts preview",
"dev": "bun run build && node dist/index.js",
"preview": "bun run build && node dist/index.js preview",
"start": "node dist/index.js",
Comment thread package.json
Comment on lines 19 to 21
"engines": {
"node": ">=18",
"node": ">=26",
"bun": ">=1.0"
Comment thread CONTRIBUTING.md Outdated

## Prerequisites

- [Node.js](https://nodejs.org) (latest)
@warengonzaga

Copy link
Copy Markdown
Owner Author

@copilot apply changes based on the comments in this thread

Copilot AI commented May 12, 2026 •

Copy link
Copy Markdown
Contributor

@copilot apply changes based on the comments in this thread

Implemented all requested updates in commit e5cbc00: Quick Start now uses Node-based commands (removed bunx), start now builds before running, Node engine/CI/prerequisites are aligned to v25+, and @types/node is pinned to the matching major.

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:

  • https://api.github.com/graphql
    • Triggering command: /usr/bin/gh gh label list --repo invalid/nonexistent-repo-xyz --json name -q .[].name --limit 100 (http block)
    • Triggering command: /usr/bin/gh gh label list --repo invalid/nonexistent-repo-xyz --json name,color,description --limit 100 (http block)
  • https://api.github.com/repos/invalid/nonexistent-repo-xyz/labels
    • Triggering command: /usr/bin/gh gh label create test --repo invalid/nonexistent-repo-xyz --color 000000 --description test (http block)
  • https://api.github.com/repos/invalid/nonexistent-repo-xyz/labels/nonexistent-label
    • Triggering command: /usr/bin/gh gh label delete nonexistent-label --repo invalid/nonexistent-repo-xyz --yes (http block)

If you need me to access, download, or install something from one of these locations, you can either:

@warengonzaga
warengonzaga merged commit 14c2289 into main May 12, 2026
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