@adhdev/daemon-core 1.0.6-rc.5 → 1.0.7-rc.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adhdev/daemon-core",
-  "version": "1.0.6-rc.5",
+  "version": "1.0.7-rc.1",
   "description": "ADHDev daemon core — CDP, IDE detection, providers, command execution",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -47,8 +47,8 @@
   "author": "vilmire",
   "license": "AGPL-3.0-or-later",
   "dependencies": {
-    "@adhdev/mesh-shared": "1.0.6-rc.5",
-    "@adhdev/session-host-core": "1.0.6-rc.5",
+    "@adhdev/mesh-shared": "1.0.7-rc.1",
+    "@adhdev/session-host-core": "1.0.7-rc.1",
     "@agentclientprotocol/sdk": "^0.16.1",
     "ajv": "^8.20.0",
     "ajv-formats": "^3.0.1",

package/src/providers/provider-loader.ts

@@ -1706,12 +1706,26 @@ export class ProviderLoader {
       }
     } catch { }
 
- // Minimum 30-minute interval (prevent excessive checks)
+ // Minimum 30-minute interval (prevent excessive checks). BUT the cooldown must
+ // never strand a clean machine with zero providers: fetchLatest() stamps the
+ // timestamp even on a failed/ETag-unchanged attempt (to avoid retry storms), so
+ // if the very first attempt hiccups the upstream dir stays empty yet every later
+ // boot within 30min is skipped — leaving "Total: 0 providers" forever. When the
+ // upstream currently has NO providers we bypass the cooldown and force a fetch;
+ // the normal 30min throttle still applies once at least one provider is present.
     const MIN_INTERVAL_MS = 30 * 60 * 1000;
-    if (prevTimestamp && (Date.now() - prevTimestamp) < MIN_INTERVAL_MS) {
+    const upstreamProviderCount = this.countProviders(this.upstreamDir);
+    if (
+      upstreamProviderCount > 0 &&
+      prevTimestamp &&
+      (Date.now() - prevTimestamp) < MIN_INTERVAL_MS
+    ) {
       this.log('Upstream check skipped (last check < 30min ago)');
       return { updated: false };
     }
+    if (upstreamProviderCount === 0 && prevTimestamp && (Date.now() - prevTimestamp) < MIN_INTERVAL_MS) {
+      this.log('Upstream empty (0 providers) — forcing fetch despite <30min cooldown');
+    }
 
     // Resolve the tarball target (config → env → vendor default) once so the
     // HEAD probe and the download below hit the same (possibly self-hosted) URL.
@@ -1753,8 +1767,11 @@ export class ProviderLoader {
         req.end();
       });
 
- // Compare ETag — skip if unchanged
-      if (etag && etag === prevEtag) {
+ // Compare ETag — skip if unchanged, but only when providers are actually on
+ // disk. A stale .meta.json etag can match while .upstream is empty (first-boot
+ // hiccup, or the dir was cleared under a persisted meta); short-circuiting then
+ // would leave the machine at 0 providers, so fall through to a real download.
+      if (etag && etag === prevEtag && upstreamProviderCount > 0) {
  // Update timestamp only
         this.writeMeta(metaPath, prevEtag, Date.now());
         this.log('Upstream unchanged (ETag match)');