ioBroker.devices icon indicating copy to clipboard operation
ioBroker.devices copied to clipboard

Fix ioBroker repository checker warnings and suggestions

Open Copilot opened this issue 5 months ago • 2 comments

This PR addresses all warnings and suggestions identified by the ioBroker Check and Service Bot to ensure the adapter follows current best practices and requirements.

Changes Made

Fixed Warnings (W132, W156)

  • Reduced news entries: Removed the oldest news entry (1.1.5) from io-package.json to comply with the 7-entry limit enforced by the repository builder
  • Updated admin dependency: Upgraded the minimum admin version requirement from >=7.4.10 to >=7.6.17 as recommended

Addressed Suggestions (S191, S523, S526)

  • Added package-lock.json: Removed package-lock.json from .gitignore and committed it to the repository for reproducible builds
  • Kept admin dependency: Retained the admin dependency as it's required for the tab-only adapter functionality, despite having "adminUI": {"config": "none"}

Validation

  • All package validation tests pass
  • Build process completes successfully
  • Linting passes with zero warnings
  • No breaking changes to existing functionality

These minimal changes ensure the adapter meets current ioBroker standards while maintaining full compatibility with existing installations.

Fixes #444.


💡 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.

Copilot avatar Sep 11 '25 21:09 Copilot

@copilot do not add @alcalzone/release-script-plugin-manual-review, but fix other points

GermanBluefox avatar Sep 11 '25 21:09 GermanBluefox

@copilot do not add @alcalzone/release-script-plugin-manual-review, but fix other points

Done! Removed the @alcalzone/release-script-plugin-manual-review from both .releaseconfig.json and package.json while keeping all other fixes (news entry removal, admin version update, package-lock.json gitignore change). All tests pass. (7bbbe0d)

Copilot avatar Sep 11 '25 21:09 Copilot