@sweny-ai/core 0.1.56 → 0.1.58

package/dist/cli/config.js

@@ -8,7 +8,7 @@ export function registerTriageCommand(program) {
         .description("Run the SWEny triage workflow")
         .option("--agent <provider>", "Coding agent: claude (default), codex, gemini")
         .option("--coding-agent-provider <provider>", "Coding agent provider (alias for --agent)")
-        .option("--observability-provider <provider>", "Observability provider (default: datadog)")
+        .option("--observability-provider <provider>", "Observability provider (default: none)")
         .option("--issue-tracker-provider <provider>", "Issue tracker provider (default: github-issues)")
         .option("--source-control-provider <provider>", "Source control provider (default: github)")
         .option("--notification-provider <provider>", "Notification provider: console, github-summary, slack, teams, discord, webhook, email, file (default: console)")
@@ -86,7 +86,7 @@ export function parseCliInputs(options, fileConfig = {}) {
             return [fileVal];
         return [];
     };
-    const obsProvider = options.observabilityProvider || f("observability-provider") || "datadog";
+    const obsProvider = options.observabilityProvider || f("observability-provider") || "none";
     return {
         codingAgentProvider: options.agent || options.codingAgentProvider || f("coding-agent-provider") || "claude",
         // Secrets: env only — never from config file
@@ -220,6 +220,8 @@ export function validateInputs(config) {
     }
     // Observability credentials by provider
     switch (config.observabilityProvider) {
+        case "none":
+            break; // No observability provider configured — nothing to validate
         case "datadog":
             if (!config.observabilityCredentials.apiKey)
                 errors.push("Missing: DD_API_KEY — find API keys at https://app.datadoghq.com/organization-settings/api-keys");
@@ -331,7 +333,7 @@ export function validateInputs(config) {
                 errors.push("Missing: RENDER_SERVICE_ID — find it in your service's Settings page (srv-...)");
             break;
         default:
-            errors.push(`Unknown --observability-provider "${config.observabilityProvider}". Valid values: datadog, sentry, cloudwatch, splunk, elastic, newrelic, loki, prometheus, pagerduty, honeycomb, heroku, opsgenie, axiom, betterstack, vercel, supabase, netlify, fly, render, file`);
+            errors.push(`Unknown --observability-provider "${config.observabilityProvider}". Valid values: none, datadog, sentry, cloudwatch, splunk, elastic, newrelic, loki, prometheus, pagerduty, honeycomb, heroku, opsgenie, axiom, betterstack, vercel, supabase, netlify, fly, render, file`);
     }
     // Issue tracker credentials by provider
     switch (config.issueTrackerProvider) {

package/dist/cli/config.test.js

@@ -112,6 +112,27 @@ describe("validateInputs — notification provider", () => {
         expect(err).toContain("console");
     });
 });
+describe("validateInputs — observability provider", () => {
+    it("accepts 'none' without requiring any credentials", () => {
+        const errors = validateInputs(baseConfig({ observabilityProvider: "none", observabilityCredentials: {} }));
+        expect(errors.filter((e) => e.includes("DD_") || e.includes("observability"))).toEqual([]);
+    });
+    it("requires DD keys when explicitly set to datadog", () => {
+        const errors = validateInputs(baseConfig({ observabilityProvider: "datadog", observabilityCredentials: {} }));
+        expect(errors.some((e) => e.includes("DD_API_KEY"))).toBe(true);
+        expect(errors.some((e) => e.includes("DD_APP_KEY"))).toBe(true);
+    });
+});
+describe("parseCliInputs — observability provider default", () => {
+    it("defaults to 'none' when no provider is configured", () => {
+        const config = parseCliInputs({}, {});
+        expect(config.observabilityProvider).toBe("none");
+    });
+    it("respects explicit datadog provider", () => {
+        const config = parseCliInputs({ observabilityProvider: "datadog" }, {});
+        expect(config.observabilityProvider).toBe("datadog");
+    });
+});
 describe("fetch.auth + offline parsing", () => {
     it("parses offline flag", () => {
         const config = parseCliInputs({ offline: true }, {});

package/dist/sources.validation.test.js

@@ -10,11 +10,12 @@
  *   - Security: no auth token leakage, URL validation
  */
 import { mkdirSync, writeFileSync, rmSync } from "node:fs";
+import { randomBytes } from "node:crypto";
 import { tmpdir } from "node:os";
 import * as path from "node:path";
 import { describe, it, expect, beforeEach, vi } from "vitest";
 import { classifySource, sourceZ, hashContent, resolveSource, resolveSources, } from "./sources.js";
-const fileTmp = path.join(tmpdir(), "sweny-sources-validation-test");
+const fileTmp = path.join(tmpdir(), `sweny-sources-validation-test-${randomBytes(4).toString("hex")}`);
 const baseCtx = () => ({
     cwd: "/tmp",
     env: {},
@@ -253,6 +254,24 @@ describe("error messages include actionable information", () => {
             expect(e.message).toContain("--offline");
         }
     });
+    it("SOURCE_FILE_READ_FAILED on permission denied", async () => {
+        const unreadable = path.join(fileTmp, "unreadable.md");
+        writeFileSync(unreadable, "secret");
+        const { chmodSync } = await import("node:fs");
+        chmodSync(unreadable, 0o000);
+        try {
+            await resolveSource("./unreadable.md", "context[0]", { ...baseCtx(), cwd: fileTmp });
+            expect.unreachable("should have thrown");
+        }
+        catch (e) {
+            expect(e.message).toContain("SOURCE_FILE_READ_FAILED");
+            expect(e.message).toContain("unreadable.md");
+            expect(e.message).toContain("context[0]");
+        }
+        finally {
+            chmodSync(unreadable, 0o644);
+        }
+    });
     it("SOURCE_OFFLINE_REQUIRES_FETCH includes the URL and field path", async () => {
         try {
             await resolveSource("https://example.com/rules.md", "rules[1]", { ...baseCtx(), offline: true });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sweny-ai/core",
-  "version": "0.1.56",
+  "version": "0.1.58",
   "type": "module",
   "bin": {
     "sweny": "./dist/cli/main.js"