@plainconceptsplatform/loop-task 2.6.0 → 2.8.0

package/dist/daemon/telemetry/open-telemetry-adapter.js

@@ -11,6 +11,10 @@ import { OTLPMetricExporter } from "@opentelemetry/exporter-metrics-otlp-http";
 import { OTLPMetricExporter as OTLPMetricExporterGrpc } from "@opentelemetry/exporter-metrics-otlp-grpc";
 import { PeriodicExportingMetricReader } from "@opentelemetry/sdk-metrics";
 import { trace, context, propagation, metrics, SpanStatusCode, SpanKind } from "@opentelemetry/api";
+import { logs } from "@opentelemetry/api-logs";
+import { LoggerProvider, SimpleLogRecordProcessor } from "@opentelemetry/sdk-logs";
+import { OTLPLogExporter } from "@opentelemetry/exporter-logs-otlp-http";
+import { OTLPLogExporter as OTLPLogExporterGrpc } from "@opentelemetry/exporter-logs-otlp-grpc";
 import { daemonLog } from "../daemon-log.js";
 class OtelSpan {
     constructor(span, ctx, onEnd) {
@@ -79,9 +83,11 @@ export class OpenTelemetryAdapter {
     constructor(settings) {
         this.sdk = null;
         this.tracerProvider = null;
+        this.loggerProvider = null;
         this.settings = settings;
         this.tracer = trace.getTracer("loop-task");
         this.meter = metrics.getMeter("loop-task");
+        this.logger = logs.getLogger("loop-task");
         this.status = {
             enabled: settings.telemetryEnabled,
             exporterConfigured: !!settings.telemetryEndpoint,
@@ -93,7 +99,12 @@ export class OpenTelemetryAdapter {
             captureCommandOutput: settings.telemetryCaptureCommandOutput,
             exporterState: this.resolveExporterState(settings),
         };
-        // Initialize metric instruments (always created — no-op when no SDK)
+        if (settings.telemetryEnabled && settings.telemetryEndpoint) {
+            this.initializeSdk(settings);
+        }
+        // Initialize metric instruments AFTER SDK startup so they bind to the
+        // real MeterProvider, not the default no-op one. When the SDK is not
+        // initialized, these still work — they bind to the API-level no-op meter.
         this.runCounter = this.meter.createCounter(METRIC_NAMES.RUNS);
         this.runDurationHistogram = this.meter.createHistogram(METRIC_NAMES.RUN_DURATION);
         this.taskCounter = this.meter.createCounter(METRIC_NAMES.TASKS);
@@ -109,9 +120,6 @@ export class OpenTelemetryAdapter {
         this.agentCacheWriteTokensCounter = this.meter.createCounter(METRIC_NAMES.AGENT_CACHE_WRITE_TOKENS);
         this.agentCostCounter = this.meter.createCounter(METRIC_NAMES.AGENT_COST);
         this.failureCounter = this.meter.createCounter(METRIC_NAMES.FAILURES);
-        if (settings.telemetryEnabled && settings.telemetryEndpoint) {
-            this.initializeSdk(settings);
-        }
     }
     resolveExporterState(settings) {
         if (!settings.telemetryEnabled)
@@ -162,7 +170,24 @@ export class OpenTelemetryAdapter {
                 instrumentations: [],
             });
             this.sdk.start();
+            // Re-obtain tracer and meter from the global API after SDK startup.
+            // The API now resolves to the real providers registered by NodeSDK.
             this.tracer = trace.getTracer(settings.telemetryServiceName);
+            this.meter = metrics.getMeter(settings.telemetryServiceName);
+            // Initialize OTLP log exporter
+            const logsUrl = endpoint.endsWith("/v1/logs")
+                ? endpoint
+                : `${endpoint}/v1/logs`;
+            const logExporter = isGrpc
+                ? new OTLPLogExporterGrpc({ url: settings.telemetryEndpoint })
+                : new OTLPLogExporter({ url: logsUrl, headers });
+            const loggerProvider = new LoggerProvider({
+                resource,
+                processors: [new SimpleLogRecordProcessor({ exporter: logExporter })],
+            });
+            logs.setGlobalLoggerProvider(loggerProvider);
+            this.loggerProvider = loggerProvider;
+            this.logger = logs.getLogger(settings.telemetryServiceName);
             this.status.exporterState = "configured";
             daemonLog(`telemetry: SDK initialized, endpoint=${settings.telemetryEndpoint}`);
         }
@@ -197,6 +222,24 @@ export class OpenTelemetryAdapter {
             return "http/protobuf";
         return this.settings.telemetryProtocol;
     }
+    /**
+     * Check if any agent serve sidecar is alive.
+     * When serve is alive, static OTEL config lives in the serve process
+     * and we skip per-task injection.
+     */
+    checkServeAlive() {
+        try {
+            const integrations = getAgentIntegrations();
+            for (const integration of integrations) {
+                if (integration.isServeAlive?.())
+                    return true;
+            }
+            return false;
+        }
+        catch {
+            return false;
+        }
+    }
     startLoop(input) {
         const span = this.tracer.startSpan(SPAN_NAMES.LOOP_RUN, { kind: SpanKind.SERVER });
         span.setAttributes({
@@ -324,6 +367,20 @@ export class OpenTelemetryAdapter {
             activeSpan.setAttribute("gen_ai.request.model", input.model);
         }
     }
+    logEvent(level, message, attributes) {
+        try {
+            const record = this.logger.emit({
+                body: message,
+                severityText: level,
+                attributes: attributes ?? {},
+            });
+            // Ensure the log record is flushed in a timely manner
+            void record;
+        }
+        catch (err) {
+            daemonLog(`telemetry: log emit failed: ${String(err)}`);
+        }
+    }
     prepareChildProcess(invocation, childContext, integrationOverride) {
         const env = {};
         const endpoint = this.resolveEndpoint();
@@ -337,25 +394,34 @@ export class OpenTelemetryAdapter {
                 env.TRACESTATE = childContext.traceState;
             }
         }
-        env.OTEL_EXPORTER_OTLP_ENDPOINT = endpoint;
-        env.OTEL_EXPORTER_OTLP_TRACES_ENDPOINT = endpoint;
-        env.OTEL_EXPORTER_OTLP_PROTOCOL = protocol;
-        env.OTEL_TRACES_EXPORTER = "otlp";
-        const authHeaders = process.env.OTEL_EXPORTER_OTLP_HEADERS
-            ?? process.env.OTEL_EXPORTER_OTLP_TRACES_HEADERS;
-        if (authHeaders) {
-            env.OTEL_EXPORTER_OTLP_HEADERS = authHeaders;
+        // Check if an agent serve sidecar is alive — if so, static OTEL config
+        // lives in the serve process env and we skip per-task injection.
+        const serveAlive = this.checkServeAlive();
+        if (!serveAlive) {
+            env.OTEL_EXPORTER_OTLP_ENDPOINT = endpoint;
+            env.OTEL_EXPORTER_OTLP_TRACES_ENDPOINT = endpoint;
+            env.OTEL_EXPORTER_OTLP_METRICS_ENDPOINT = endpoint;
+            env.OTEL_EXPORTER_OTLP_LOGS_ENDPOINT = endpoint;
+            env.OTEL_EXPORTER_OTLP_PROTOCOL = protocol;
+            env.OTEL_TRACES_EXPORTER = "otlp";
+            env.OTEL_METRICS_EXPORTER = "otlp";
+            env.OTEL_LOGS_EXPORTER = "otlp";
+            const authHeaders = process.env.OTEL_EXPORTER_OTLP_HEADERS
+                ?? process.env.OTEL_EXPORTER_OTLP_TRACES_HEADERS;
+            if (authHeaders) {
+                env.OTEL_EXPORTER_OTLP_HEADERS = authHeaders;
+            }
+            const correlationAttrs = {
+                [CORRELATION_KEYS.RUN_ID]: childContext.runId,
+                [CORRELATION_KEYS.LOOP_ID]: childContext.loopId,
+            };
+            if (childContext.taskId)
+                correlationAttrs[CORRELATION_KEYS.TASK_ID] = childContext.taskId;
+            if (childContext.projectId)
+                correlationAttrs[CORRELATION_KEYS.PROJECT_ID] = childContext.projectId;
+            const mergedResourceAttrs = this.mergeResourceAttributes(process.env.OTEL_RESOURCE_ATTRIBUTES, correlationAttrs);
+            env.OTEL_RESOURCE_ATTRIBUTES = mergedResourceAttrs;
         }
-        const correlationAttrs = {
-            [CORRELATION_KEYS.RUN_ID]: childContext.runId,
-            [CORRELATION_KEYS.LOOP_ID]: childContext.loopId,
-        };
-        if (childContext.taskId)
-            correlationAttrs[CORRELATION_KEYS.TASK_ID] = childContext.taskId;
-        if (childContext.projectId)
-            correlationAttrs[CORRELATION_KEYS.PROJECT_ID] = childContext.projectId;
-        const mergedResourceAttrs = this.mergeResourceAttributes(process.env.OTEL_RESOURCE_ATTRIBUTES, correlationAttrs);
-        env.OTEL_RESOURCE_ATTRIBUTES = mergedResourceAttrs;
         // Apply agent-specific integrations when auto-instrumentation is enabled
         let integrationId;
         if (this.settings.telemetryAutoInstrumentAgents || integrationOverride) {
@@ -419,6 +485,9 @@ export class OpenTelemetryAdapter {
             if (this.tracerProvider) {
                 flushes.push(this.tracerProvider.forceFlush());
             }
+            if (this.loggerProvider) {
+                flushes.push(this.loggerProvider.forceFlush());
+            }
             const sdkInternals = this.sdk;
             if (typeof sdkInternals._meterProvider?.forceFlush === "function") {
                 flushes.push(sdkInternals._meterProvider.forceFlush());
@@ -450,6 +519,16 @@ export class OpenTelemetryAdapter {
             }
             this.tracerProvider = null;
         }
+        if (this.loggerProvider) {
+            try {
+                await this.loggerProvider.forceFlush();
+                await this.loggerProvider.shutdown();
+            }
+            catch {
+                // best effort
+            }
+            this.loggerProvider = null;
+        }
         if (!this.sdk)
             return;
         try {

package/dist/shared/i18n/en.json

@@ -460,7 +460,7 @@
     "project.wizard.directoryPrompt": "Working directory? (optional)",
     "project.wizard.directoryHint": "Default directory for loops in this project. Leave blank to inherit",
     "project.wizard.githubSourcePrompt": "GitHub Source? (optional)",
-    "project.wizard.githubSourceHint": "Repository in owner/repo format, e.g. CKGrafico/loop-task",
+    "project.wizard.githubSourceHint": "Repository in owner/repo format, e.g. PlainConceptsPlatform/loop-task",
     "project.error.updateFailed": "Failed to update project",
     "project.error.deleteFailed": "Failed to delete project",
     "project.toastCreated": "Project \"{name}\" created",

package/package.json

@@ -1,119 +1,119 @@
-{
-  "name": "@plainconceptsplatform/loop-task",
-  "version": "2.6.0",
-  "description": "Loop engineering toolkit. Run any command on a cadence, in the background, managed from a terminal board. Schedule tests, builds, syncs, or agent prompts.",
-  "type": "module",
-  "bin": {
-    "loop-task": "dist/entry.js"
-  },
-  "main": "dist/entry.js",
-  "files": [
-    "dist",
-    "README.md",
-    "LICENSE"
-  ],
-  "scripts": {
-    "build": "node -e \"require('fs').rmSync('dist',{recursive:true,force:true})\" && tsc -p tsconfig.build.json && node -e \"require('fs').copyFileSync('src/entry.js','dist/entry.js'); require('fs').copyFileSync('src/esm-loader.js','dist/esm-loader.js')\"",
-    "prepublishOnly": "npm run build",
-    "start": "node dist/entry.js",
-    "dev": "tsx src/cli.ts",
-    "dev:watch": "tsx --watch src/cli.ts",
-    "board": "node dist/entry.js",
-    "test": "vitest run --exclude '**/background-cli.test.ts'",
-    "test:all": "vitest run",
-    "test:watch": "vitest",
-    "test:coverage": "vitest run --coverage --exclude '**/background-cli.test.ts'",
-    "lint": "eslint src/ tests/",
-    "typecheck": "tsc --noEmit",
-    "release:dry": "npm publish --dry-run",
-    "release": "npm publish",
-    "visual-evidence": "tsx src/visual-evidence/cli.ts",
-    "visual-evidence:publish": "tsx src/visual-evidence/publish.ts",
-    "skills:mark-internal": "node scripts/mark-agent-skills-internal.mjs",
-    "prepare": "husky"
-  },
-  "keywords": [
-    "cli",
-    "loop",
-    "loop-engineering",
-    "repeat",
-    "interval",
-    "schedule",
-    "timer",
-    "cron",
-    "automation",
-    "automations",
-    "devops",
-    "agent",
-    "ai-agent",
-    "coding-agent",
-    "agent-automation",
-    "claude-code",
-    "codex",
-    "opencode",
-    "background-tasks",
-    "scheduler"
-  ],
-  "author": "Quique Fdez Guerra",
-  "license": "MIT",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/plainconceptsplatform/loop-task.git"
-  },
-  "homepage": "https://github.com/plainconceptsplatform/loop-task",
-  "bugs": {
-    "url": "https://github.com/plainconceptsplatform/loop-task/issues"
-  },
-  "engines": {
-    "node": ">=20.0.0"
-  },
-  "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.29.0",
-    "@opentelemetry/api": "^1.9.1",
-    "@opentelemetry/exporter-metrics-otlp-grpc": "^0.221.0",
-    "@opentelemetry/exporter-metrics-otlp-http": "^0.221.0",
-    "@opentelemetry/exporter-trace-otlp-grpc": "^0.221.0",
-    "@opentelemetry/exporter-trace-otlp-http": "^0.221.0",
-    "@opentelemetry/resources": "^2.10.0",
-    "@opentelemetry/sdk-metrics": "^2.10.0",
-    "@opentelemetry/sdk-node": "^0.221.0",
-    "@opentelemetry/sdk-trace-base": "^2.10.0",
-    "@opentelemetry/sdk-trace-node": "^2.10.0",
-    "@opentelemetry/semantic-conventions": "^1.43.0",
-    "commander": "^13.1.0",
-    "execa": "^9.6.0",
-    "ink": "^7.1.0",
-    "ink-combobox": "^0.2.0",
-    "ink-scroll-list": "^0.4.1",
-    "ink-select-input": "^6.2.0",
-    "ink-spinner": "^5.0.0",
-    "ink-text-input": "^6.0.0",
-    "inversify": "^8.1.1",
-    "inversify-hooks": "^4.0.0",
-    "js-yaml": "^5.2.2",
-    "ms": "^2.1.3",
-    "react": "^19.2.7",
-    "yaml": "^2.9.0",
-    "zod": "^4.4.3"
-  },
-  "devDependencies": {
-    "@types/js-yaml": "^4.0.9",
-    "@types/ms": "^2.1.0",
-    "@types/node": "^22.15.0",
-    "@types/react": "^19.2.17",
-    "@vitest/coverage-v8": "^3.1.0",
-    "eslint": "^9.25.0",
-    "husky": "^9.1.7",
-    "ink-testing-library": "^4.0.0",
-    "tsx": "^4.19.0",
-    "typescript": "^5.8.0",
-    "typescript-eslint": "^8.30.0",
-    "vitest": "^3.1.0"
-  },
-  "packageManager": "pnpm@10.30.3",
-  "pnpm": {
-    "overrides": {
-      "picocolors": "1.1.0"
-    }
-  }
+{
+  "name": "@plainconceptsplatform/loop-task",
+  "version": "2.8.0",
+  "description": "Loop engineering toolkit. Run any command on a cadence, in the background, managed from a terminal board. Schedule tests, builds, syncs, or agent prompts.",
+  "type": "module",
+  "bin": {
+    "loop-task": "dist/entry.js"
+  },
+  "main": "dist/entry.js",
+  "files": [
+    "dist",
+    "README.md",
+    "LICENSE"
+  ],
+  "keywords": [
+    "cli",
+    "loop",
+    "loop-engineering",
+    "repeat",
+    "interval",
+    "schedule",
+    "timer",
+    "cron",
+    "automation",
+    "automations",
+    "devops",
+    "agent",
+    "ai-agent",
+    "coding-agent",
+    "agent-automation",
+    "claude-code",
+    "codex",
+    "opencode",
+    "background-tasks",
+    "scheduler"
+  ],
+  "author": "Plain Concepts",
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/PlainConceptsPlatform/loop-task.git"
+  },
+  "homepage": "https://github.com/PlainConceptsPlatform/loop-task",
+  "bugs": {
+    "url": "https://github.com/PlainConceptsPlatform/loop-task/issues"
+  },
+  "publishConfig": {
+    "access": "public",
+    "registry": "https://registry.npmjs.org/"
+  },
+  "engines": {
+    "node": ">=20.0.0"
+  },
+  "dependencies": {
+    "@modelcontextprotocol/sdk": "^1.29.0",
+    "@opentelemetry/api": "^1.9.1",
+    "@opentelemetry/api-logs": "^0.221.0",
+    "@opentelemetry/exporter-logs-otlp-grpc": "^0.221.0",
+    "@opentelemetry/exporter-logs-otlp-http": "^0.221.0",
+    "@opentelemetry/exporter-metrics-otlp-grpc": "^0.221.0",
+    "@opentelemetry/exporter-metrics-otlp-http": "^0.221.0",
+    "@opentelemetry/exporter-trace-otlp-grpc": "^0.221.0",
+    "@opentelemetry/exporter-trace-otlp-http": "^0.221.0",
+    "@opentelemetry/resources": "^2.10.0",
+    "@opentelemetry/sdk-logs": "^0.221.0",
+    "@opentelemetry/sdk-metrics": "^2.10.0",
+    "@opentelemetry/sdk-node": "^0.221.0",
+    "@opentelemetry/sdk-trace-base": "^2.10.0",
+    "@opentelemetry/sdk-trace-node": "^2.10.0",
+    "@opentelemetry/semantic-conventions": "^1.43.0",
+    "commander": "^13.1.0",
+    "execa": "^9.6.0",
+    "ink": "^7.1.0",
+    "ink-combobox": "^0.2.0",
+    "ink-scroll-list": "^0.4.1",
+    "ink-select-input": "^6.2.0",
+    "ink-spinner": "^5.0.0",
+    "ink-text-input": "^6.0.0",
+    "inversify": "^8.1.1",
+    "inversify-hooks": "^4.0.0",
+    "js-yaml": "^5.2.2",
+    "ms": "^2.1.3",
+    "react": "^19.2.7",
+    "yaml": "^2.9.0",
+    "zod": "^4.4.3"
+  },
+  "devDependencies": {
+    "@types/js-yaml": "^4.0.9",
+    "@types/ms": "^2.1.0",
+    "@types/node": "^22.15.0",
+    "@types/react": "^19.2.17",
+    "@vitest/coverage-v8": "^3.1.0",
+    "eslint": "^9.25.0",
+    "husky": "^9.1.7",
+    "ink-testing-library": "^4.0.0",
+    "tsx": "^4.19.0",
+    "typescript": "^5.8.0",
+    "typescript-eslint": "^8.30.0",
+    "vitest": "^3.1.0"
+  },
+  "scripts": {
+    "build": "node -e \"require('fs').rmSync('dist',{recursive:true,force:true})\" && tsc -p tsconfig.build.json && node -e \"require('fs').copyFileSync('src/entry.js','dist/entry.js'); require('fs').copyFileSync('src/esm-loader.js','dist/esm-loader.js')\"",
+    "start": "node dist/entry.js",
+    "dev": "tsx src/cli.ts",
+    "dev:watch": "tsx --watch src/cli.ts",
+    "board": "node dist/entry.js",
+    "test": "vitest run --exclude '**/background-cli.test.ts'",
+    "test:all": "vitest run",
+    "test:watch": "vitest",
+    "test:coverage": "vitest run --coverage --exclude '**/background-cli.test.ts'",
+    "lint": "eslint src/ tests/",
+    "typecheck": "tsc --noEmit",
+    "release:dry": "npm publish --dry-run --access public",
+    "release": "npm publish --access public",
+    "visual-evidence": "tsx src/visual-evidence/cli.ts",
+    "visual-evidence:publish": "tsx src/visual-evidence/publish.ts",
+    "skills:mark-internal": "node scripts/mark-agent-skills-internal.mjs"
+  }
 }