@hardfin/cli 0.1.0-dev.37 → 0.1.1-dev.39

package/README.md

@@ -432,6 +432,11 @@ Only defects are reported. None of these are:
 - a mistake on the command line or in a settings file
 - being signed out, or a sign in that was cancelled or refused
 - the machine refusing a file, such as a full disk or a missing permission
+- anything from a run with no API key and no stored sign in, except a defect in `hardfin login`
+
+The last rule is there because package scanners install the CLI and run it in sandboxes. They
+never sign in, and their reports would bury the real ones. A defect while signing in is still
+reported, because that is the one thing a signed-out person is doing.
 
 A report carries the error, its stack trace, and what `hardfin status` would print at that
 moment. The command is named, and its flags are listed by name without their values.
@@ -447,7 +452,7 @@ organization appear only as their Hardfin keys.
 | Send reports to another Sentry project            | `HARDFIN_SENTRY_DSN`                       |
 
 `hardfin status` shows which of these is in effect. `hardfin __crash` fails on purpose, to
-check that a report arrives.
+check that a report arrives. It reports only when signed in or given an API key.
 
 ## Local configuration
 

package/dist/cli.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="b1580db0-6822-5a25-b000-03705726eaf6")}catch(e){}}();
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="ee01278c-9a53-5677-8abd-c28911112932")}catch(e){}}();
 import module from "node:module";
 //#region src/cli.ts
 module.enableCompileCache?.();
@@ -14,7 +14,7 @@ process.on("unhandledRejection", (error) => void toCrash(error));
 try {
 	const answer = await toQuickAnswer(process.argv.slice(2));
 	if (answer === void 0) {
-		const { toCli, toParseExitCode, wasReportAttempted } = await import("./program-CjKwnPeY.js");
+		const { toCli, toParseExitCode, wasReportAttempted } = await import("./program-d7dXAbE4.js");
 		try {
 			await toCli().parseAsync(process.argv);
 		} catch (error) {
@@ -33,7 +33,7 @@ async function toCrash(error) {
 	isCrashing = true;
 	process.stderr.write(`error: ${error instanceof Error ? error.message : String(error)}\n`);
 	try {
-		const { toReported } = await import("./program-CjKwnPeY.js");
+		const { toReported } = await import("./program-d7dXAbE4.js");
 		await toReported(error, false, void 0, "fatal");
 	} finally {
 		process.exit(1);
@@ -51,4 +51,4 @@ async function toQuickAnswer(args) {
 export {};
 
 //# sourceMappingURL=cli.js.map
-//# debugId=b1580db0-6822-5a25-b000-03705726eaf6
+//# debugId=ee01278c-9a53-5677-8abd-c28911112932

package/dist/{program-CjKwnPeY.js → program-d7dXAbE4.js}

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="5a591d32-0d31-5357-b447-e33420753351")}catch(e){}}();
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="18ff3f0f-09fd-501c-9ccd-2c32855efa8b")}catch(e){}}();
 import { a as defineCommand, i as GLOBAL_FLAGS, r as ExitCode, t as toInvocation } from "./scrub-BN3gyBpR.js";
 import { n as toCompletion, t as toAnswer$1 } from "./completion-tree-rR_Rmeij.js";
 import { createRequire } from "node:module";
@@ -2939,9 +2939,10 @@ async function report(error, occasion, resolved) {
 	if (!isReportable(error)) return;
 	const telemetry = toTelemetry();
 	if (!telemetry.enabled) return;
+	const settings = resolved ?? toResolved();
+	if (!isSignedIn(settings) && occasion.invocation.command !== "login") return;
 	isAttempted = true;
 	try {
-		const settings = resolved ?? toResolved();
 		const status = settings ? await toStatus(settings) : void 0;
 		const secrets = [settings?.settings.apiKey, settings ? toCredential(settings.settings.issuerUrl)?.refreshToken : void 0];
 		return await (await import("./sentry-BM9Rvhhy.js")).send({
@@ -2955,6 +2956,16 @@ async function report(error, occasion, resolved) {
 		return;
 	}
 }
+/** isSignedIn reports whether this run holds an API key or a stored sign in. */
+function isSignedIn(resolved) {
+	if (!resolved) return false;
+	if (resolved.settings.apiKey) return true;
+	try {
+		return toCredential(resolved.settings.issuerUrl) !== void 0;
+	} catch {
+		return false;
+	}
+}
 /**
 * toResolved reads the settings again for a crash that happened before a command held them.
 * The global options are read from the command line, so the report describes the host the
@@ -6534,5 +6545,5 @@ async function toReported(error, isJSON, resolved, level = "error") {
 //#endregion
 export { toCli, toParseExitCode, toReported, wasReportAttempted };
 
-//# sourceMappingURL=program-CjKwnPeY.js.map
-//# debugId=5a591d32-0d31-5357-b447-e33420753351
+//# sourceMappingURL=program-d7dXAbE4.js.map
+//# debugId=18ff3f0f-09fd-501c-9ccd-2c32855efa8b

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@hardfin/cli",
-    "version": "0.1.0-dev.37",
+    "version": "0.1.1-dev.39",
     "description": "Command line interface for the Hardfin API",
     "license": "Apache-2.0",
     "author": "Hardfin, Inc.",