Merge pull request #842 from ioBroker/copilot/update-isnewadapter-setup

Set isNewAdapter before config.updateConfig via new getLatestRepo preprocessing step

Author: mcm1957

README.md

@@ -26,6 +26,7 @@ Example:
 	### **WORK IN PROGRESS**
 -->
 ### **WORK IN PROGRESS**
+- (@copilot) Fixed adapter detection to ensure the `isNewAdapter` flag is set before configuration logging. Added a new `getLatestRepo` preprocessing step so that the flag is correctly visible in the environment log output.
 - (@copilot) Added German translation of OBJECTDUMP.md as OBJECTDUMP_de.md.
 
 ### 5.10.4 (2026-04-12)

index.js

@@ -148,6 +148,7 @@ function check(request, ctx, callback) {
         .then(context => M0000_PackageJson.getPackageJson(context))
         .then(context => M1000_IOPackageJson.getIOPackageJson(context))
         .then(context => M2000_Npm.getNpm(context))
+        .then(context => M4000_Repository.getLatestRepo(context))
         .then(context => config.updateConfig(context))
         .then(context => config.logEnvironment(context))
         .then(context => M0000_PackageJson.checkPackageJson(context))

lib/M4000_Repository.js

@@ -21,32 +21,54 @@ const JSON5 = require('json5');
 const common = require('./common.js');
 const config = require('./config.js');
 
+async function getLatestRepo(context) {
+    console.log('[init] getting latest repository info');
+    try {
+        const response = await axios(
+            'https://raw.githubusercontent.com/ioBroker/ioBroker.repositories/master/sources-dist.json',
+        );
+        const body = response.data;
+        if (body) {
+            context.latestRepo = body;
+            if (!context.latestRepo[context.adapterName]) {
+                context.cfg.isNewAdapter = true;
+                common.info(
+                    `isNewAdapter is set: adapter "${context.adapterName}" is not listed in the latest repository`,
+                );
+            }
+        }
+    } catch (e) {
+        common.error(`Cannot download latest repository info: ${e}`);
+    }
+    return context;
+}
+
 async function checkRepository(context) {
     console.log('\n[E4000 - E4999] checking repository');
 
     if (context.ioPackageJson && context.ioPackageJson.common && context.ioPackageJson.common.type) {
         /* TODO: why check type here ? */
-        // download latest repo
-        const response = await axios(
-            'https://raw.githubusercontent.com/ioBroker/ioBroker.repositories/master/sources-dist.json',
-        );
-        let body = response.data;
-        if (!body) {
-            context.errors.push(
-                '[E4000] Cannot download https://raw.githubusercontent.com/ioBroker/ioBroker.repositories/master/sources-dist.json',
+        let body;
+        // download latest repo (if not already retrieved by preprocessing)
+        if (!context.latestRepo) {
+            const response = await axios(
+                'https://raw.githubusercontent.com/ioBroker/ioBroker.repositories/master/sources-dist.json',
             );
-        } else {
-            context.latestRepo = body;
+            body = response.data;
+            if (!body) {
+                context.errors.push(
+                    '[E4000] Cannot download https://raw.githubusercontent.com/ioBroker/ioBroker.repositories/master/sources-dist.json',
+                );
+            } else {
+                context.latestRepo = body;
+            }
+        }
+        if (context.latestRepo) {
             if (!context.latestRepo[context.adapterName]) {
                 context.warnings.push(
                     `[W4001] Cannot find "${context.adapterName}" in latest repository. Please ignore if PR to add adapter to repositories already exists.`,
                 );
 
-                context.cfg.isNewAdapter = true;
-                common.info(
-                    `isNewAdapter is set: adapter "${context.adapterName}" is not listed in the latest repository`,
-                );
-
                 if (context.adapterName.includes('_')) {
                     context.errors.push('[E4021] Adapter name should use "-" instead of "_"');
                 } else {
@@ -489,6 +511,7 @@ async function checkVSCodeSchemas(context) {
     }
 }
 
+exports.getLatestRepo = getLatestRepo;
 exports.checkRepository = checkRepository;
 
 // List of error and warnings used at this module