Skip to content

Remove node-fetch alternative from HTTP client libraries recommendation#51

Merged
DutchmanNL merged 3 commits intomainfrom
copilot/fix-45
Sep 19, 2025
Merged

Remove node-fetch alternative from HTTP client libraries recommendation#51
DutchmanNL merged 3 commits intomainfrom
copilot/fix-45

Conversation

Copy link
Contributor

Copilot AI commented Sep 19, 2025

This PR removes the outdated node-fetch alternative recommendation from the HTTP Client Libraries section in the template, as it's no longer needed for modern ioBroker adapter development.

Context

The template currently recommends node-fetch as an alternative for older Node.js versions:

### HTTP Client Libraries
- **Preferred:** Use native `fetch` API (available in Node.js 18+)
- **Alternative:** Use `node-fetch` for older Node.js versions
- **Avoid:** `axios` unless specific features are required (reduces bundle size)

However, ioBroker adapters now require Node.js 20+ as minimum (with Node.js 22 recommended), and the native fetch API has been available since Node.js 18+, making the node-fetch alternative obsolete.

Changes

  • Removed the line recommending node-fetch for older Node.js versions
  • Updated the preferred option description to clarify Node.js version requirements: "available in Node.js 18+, required Node.js 20+ for adapters"
  • Maintained all existing code examples and other recommendations

Impact

This change provides clearer guidance to adapter developers by removing confusion about which HTTP client to use, aligning the template with current ioBroker ecosystem requirements.

Fixes #45.


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

Co-authored-by: DutchmanNL <7318445+DutchmanNL@users.noreply.github.com>
@DutchmanNL DutchmanNL marked this pull request as ready for review September 19, 2025 09:29
Copilot AI review requested due to automatic review settings September 19, 2025 09:29
Copilot AI changed the title [WIP] [DOCS/AUTO] remove node-fetch alternative from repository template Remove node-fetch alternative from HTTP client libraries recommendation Sep 19, 2025
Copilot AI requested a review from DutchmanNL September 19, 2025 09:29
Copy link
Contributor

Copilot AI left a comment

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 documentation to align with the minimum Node.js 20+ requirement by removing the outdated node-fetch alternative and clarifying the recommended HTTP client.

  • Remove node-fetch as an alternative
  • Clarify that adapters require Node.js 20+ while fetch is available since Node.js 18

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@DutchmanNL DutchmanNL merged commit 9169631 into main Sep 19, 2025
2 checks passed
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.

[DOCS/AUTO] remove node-fetch alternative from repository template

3 participants