Skip to content

Conversation

Copy link

Copilot AI commented Nov 2, 2025

The workflow files used lts/jod without documentation, causing confusion about which LTS version was active.

Changes:

  • Added a dedicated "Node.js Version" section to .github/copilot-instructions.md
  • Documents that the project uses Node.js v22.x LTS "Jod" (became LTS on October 29, 2024)
  • Notes that the previous LTS was v20.x "Iron"
  • Provides guidance for AI agents to avoid confusing the two LTS versions
  • Includes reference link to official Node.js releases page

This prevents future confusion about LTS codenames by documenting the information directly in the copilot instructions that AI agents read.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

kraenhansen and others added 6 commits November 2, 2025 19:36
Enable Hermes and Fabric

Patch react_native_post_install to pass react native path

Don't pod install when initializing

Patch macos app with scripts and source files

Re-arm the init script

Move linked deps into install command to make --install-links effective

Using regular linking for monorepo deps

Include original dependencies

Enable new arch in Podfile

Fix comment from review
No need to Setup Android SDK

Debugging with Copilot

Pass --mode when building from CLI

Install CMake 3.22

Fix bootstrap issue

Trying a higher CMake version

Remove debug info from workflow

Build universal Darwin libraries

Add missing x86_64-apple-darwin Rust target to macOS CI job (#298)

* Initial plan

* Add missing x86_64-apple-darwin Rust target to macOS job

Co-authored-by: kraenhansen <[email protected]>

* Add only missing x86_64-apple-darwin target (aarch64 is host)

Co-authored-by: kraenhansen <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: kraenhansen <[email protected]>

Run MacOS test app

Use package script to run all tests
Add documentation comment explaining that we're using Node.js v22.x "Jod" LTS
(became LTS on October 29, 2024). Centralize the version in an environment
variable to make future updates easier and more consistent across all jobs.

Co-authored-by: kraenhansen <[email protected]>
Copilot AI changed the title [WIP] Update MacOS test app and Hermes vendoring instructions Document Node.js LTS version in CI workflows Nov 2, 2025
Copilot AI requested a review from kraenhansen November 2, 2025 18:57
Copilot finished work on behalf of kraenhansen November 2, 2025 18:57
@kraenhansen
Copy link
Collaborator

@copilot I see no need to add the documenting comments in the workflows themselves. I was more interested in you updating the copilot_instructions.md to remind yourself not to make the suggestion of renaming "jod" to "iron" in the future.

Base automatically changed from kh/macos-test-app to main November 2, 2025 19:30
Revert workflow file changes and instead add Node.js LTS documentation to
copilot-instructions.md. This reminds future AI agents that the current LTS
is v22.x "Jod" (not "Iron") and prevents incorrect suggestions about LTS
version naming.

Co-authored-by: kraenhansen <[email protected]>
Copy link
Author

Copilot AI commented Nov 2, 2025

@copilot I see no need to add the documenting comments in the workflows themselves. I was more interested in you updating the copilot_instructions.md to remind yourself not to make the suggestion of r...

Reverted the workflow file changes and updated .github/copilot-instructions.md instead with a dedicated section documenting Node.js v22.x LTS "Jod" and reminding not to confuse it with the previous LTS "Iron". See commit fac4c51.

Copilot AI changed the title Document Node.js LTS version in CI workflows Document Node.js LTS version in copilot instructions Nov 2, 2025
Copilot finished work on behalf of kraenhansen November 2, 2025 19:34
@kraenhansen
Copy link
Collaborator

No longer applicable.

@kraenhansen kraenhansen closed this Nov 5, 2025
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