@camstack/system 1.2.17 → 1.2.18

package/dist/builtins/loki-logging/index.js

@@ -118,10 +118,32 @@ var LokiDestination = class {
 	}
 	async initialize(config) {
 		this.config = config ?? null;
-		if (!this.isActive()) return;
+		this.startTimerIfActive();
+	}
+	/**
+	* Re-apply configuration to an already-initialized destination after a
+	* settings change, without dropping the queued tail. Stops the current flush
+	* timer, swaps the config in, and starts a fresh timer only when the new
+	* config is active. This is what makes the addon's "enable without a restart"
+	* contract real: `initialize()` starts a timer, so calling it twice would leak
+	* the first one — `reconfigure()` clears it first.
+	*/
+	async reconfigure(config) {
+		this.stopTimer();
+		this.config = config;
+		this.startTimerIfActive();
+	}
+	stopTimer() {
+		if (this.timer !== void 0) {
+			this.clearIntervalFn(this.timer);
+			this.timer = void 0;
+		}
+	}
+	startTimerIfActive() {
+		if (!this.isActive() || this.config === null) return;
 		this.timer = this.setIntervalFn(() => {
 			this.flush();
-		}, Math.max(250, config?.batchIntervalMs ?? 5e3));
+		}, Math.max(250, this.config.batchIntervalMs));
 		const timer = this.timer;
 		if (typeof timer === "object" && timer !== null && "unref" in timer) timer.unref();
 	}
@@ -254,10 +276,7 @@ var LokiDestination = class {
 		return [];
 	}
 	async shutdown() {
-		if (this.timer !== void 0) {
-			this.clearIntervalFn(this.timer);
-			this.timer = void 0;
-		}
+		this.stopTimer();
 		await this.flush();
 		this.config = null;
 	}
@@ -296,6 +315,17 @@ var LokiLoggingAddon = class extends require_dist.BaseAddon {
 		this.destination = null;
 	}
 	/**
+	* React to a settings change (enable/disable, URL, level, batching) without a
+	* restart — the contract this addon advertises. `BaseAddon` has already
+	* reloaded `this.config` by the time this runs; re-apply it to the live
+	* destination so toggling `enabled` on actually starts pushing (and off stops
+	* it). Without this the destination keeps its boot-time config forever.
+	*/
+	async onConfigChanged() {
+		await this.destination?.reconfigure(this.destinationConfig());
+		this.ctx.logger.info(this.config.enabled && this.config.url.length > 0 ? `Loki logging reconfigured → ${this.config.url}` : "Loki logging now inactive (disabled or no URL)");
+	}
+	/**
 	* `node` is a Loki LABEL, so it must be the node's real identity and must not
 	* be operator-editable — a typo would fork the stream and split a node's history
 	* in two. `kernel.localNodeId` is the Moleculer node id (`'hub'` on the hub, the

package/dist/builtins/loki-logging/index.mjs

@@ -113,10 +113,32 @@ var LokiDestination = class {
 	}
 	async initialize(config) {
 		this.config = config ?? null;
-		if (!this.isActive()) return;
+		this.startTimerIfActive();
+	}
+	/**
+	* Re-apply configuration to an already-initialized destination after a
+	* settings change, without dropping the queued tail. Stops the current flush
+	* timer, swaps the config in, and starts a fresh timer only when the new
+	* config is active. This is what makes the addon's "enable without a restart"
+	* contract real: `initialize()` starts a timer, so calling it twice would leak
+	* the first one — `reconfigure()` clears it first.
+	*/
+	async reconfigure(config) {
+		this.stopTimer();
+		this.config = config;
+		this.startTimerIfActive();
+	}
+	stopTimer() {
+		if (this.timer !== void 0) {
+			this.clearIntervalFn(this.timer);
+			this.timer = void 0;
+		}
+	}
+	startTimerIfActive() {
+		if (!this.isActive() || this.config === null) return;
 		this.timer = this.setIntervalFn(() => {
 			this.flush();
-		}, Math.max(250, config?.batchIntervalMs ?? 5e3));
+		}, Math.max(250, this.config.batchIntervalMs));
 		const timer = this.timer;
 		if (typeof timer === "object" && timer !== null && "unref" in timer) timer.unref();
 	}
@@ -249,10 +271,7 @@ var LokiDestination = class {
 		return [];
 	}
 	async shutdown() {
-		if (this.timer !== void 0) {
-			this.clearIntervalFn(this.timer);
-			this.timer = void 0;
-		}
+		this.stopTimer();
 		await this.flush();
 		this.config = null;
 	}
@@ -291,6 +310,17 @@ var LokiLoggingAddon = class extends BaseAddon {
 		this.destination = null;
 	}
 	/**
+	* React to a settings change (enable/disable, URL, level, batching) without a
+	* restart — the contract this addon advertises. `BaseAddon` has already
+	* reloaded `this.config` by the time this runs; re-apply it to the live
+	* destination so toggling `enabled` on actually starts pushing (and off stops
+	* it). Without this the destination keeps its boot-time config forever.
+	*/
+	async onConfigChanged() {
+		await this.destination?.reconfigure(this.destinationConfig());
+		this.ctx.logger.info(this.config.enabled && this.config.url.length > 0 ? `Loki logging reconfigured → ${this.config.url}` : "Loki logging now inactive (disabled or no URL)");
+	}
+	/**
 	* `node` is a Loki LABEL, so it must be the node's real identity and must not
 	* be operator-editable — a typo would fork the stream and split a node's history
 	* in two. `kernel.localNodeId` is the Moleculer node id (`'hub'` on the hub, the

package/dist/builtins/loki-logging/loki-destination.d.ts

@@ -60,6 +60,17 @@ export declare class LokiDestination implements ILogDestination {
     private readonly clearIntervalFn;
     constructor(deps?: LokiDestinationDeps);
     initialize(config?: LokiConfig): Promise<void>;
+    /**
+     * Re-apply configuration to an already-initialized destination after a
+     * settings change, without dropping the queued tail. Stops the current flush
+     * timer, swaps the config in, and starts a fresh timer only when the new
+     * config is active. This is what makes the addon's "enable without a restart"
+     * contract real: `initialize()` starts a timer, so calling it twice would leak
+     * the first one — `reconfigure()` clears it first.
+     */
+    reconfigure(config: LokiConfig): Promise<void>;
+    private stopTimer;
+    private startTimerIfActive;
     /** Enabled AND pointed somewhere. An enabled destination with no URL is inert. */
     private isActive;
     write(entry: LogEntry): void;

package/dist/builtins/loki-logging/loki-logging.addon.d.ts

@@ -18,6 +18,14 @@ export declare class LokiLoggingAddon extends BaseAddon<LokiAddonConfig> {
     constructor();
     protected onInitialize(): Promise<ProviderRegistration[]>;
     protected onShutdown(): Promise<void>;
+    /**
+     * React to a settings change (enable/disable, URL, level, batching) without a
+     * restart — the contract this addon advertises. `BaseAddon` has already
+     * reloaded `this.config` by the time this runs; re-apply it to the live
+     * destination so toggling `enabled` on actually starts pushing (and off stops
+     * it). Without this the destination keeps its boot-time config forever.
+     */
+    protected onConfigChanged(): Promise<void>;
     /**
      * `node` is a Loki LABEL, so it must be the node's real identity and must not
      * be operator-editable — a typo would fork the stream and split a node's history

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camstack/system",
-  "version": "1.2.17",
+  "version": "1.2.18",
   "description": "Core addon for CamStack — builtins, pipeline, process management, auth, logging, events",
   "keywords": [
     "camstack",