react-doctor 0.5.5 → 0.5.6

package/dist/cli.js

@@ -1,5 +1,5 @@
 
-!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]="06a2a696-2f0a-515a-8c24-d1a9b9898c6e")}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]="79c9f922-1713-5894-a0aa-142d0cd9c7ba")}catch(e){}}();
 import { createRequire } from "node:module";
 import * as NodeChildProcess from "node:child_process";
 import { execFile, execFileSync, spawn, spawnSync } from "node:child_process";
@@ -39740,13 +39740,24 @@ const checkSecurityScan = (rootDirectory, options = {}) => {
 		if (!shouldEnableRule(rule.requires, rule.tags, capabilities, ignoredTags, rule.disabledBy)) return [];
 		return [{
 			entry,
-			scan
+			scan,
+			committedFilesOnly: rule.committedFilesOnly === true
 		}];
 	});
 	if (enabledScanRules.length === 0) return [];
 	const diagnostics = [];
 	const seen = /* @__PURE__ */ new Set();
-	for (const file of collectSecurityScanFiles(rootDirectory)) for (const { entry, scan } of enabledScanRules) for (const finding of scan(file)) {
+	const gitIgnoredCache = /* @__PURE__ */ new Map();
+	const isFileGitIgnored = (file) => {
+		let status = gitIgnoredCache.get(file.absolutePath);
+		if (status === void 0) {
+			status = isPathGitIgnored(rootDirectory, file.absolutePath);
+			gitIgnoredCache.set(file.absolutePath, status);
+		}
+		return status === true;
+	};
+	for (const file of collectSecurityScanFiles(rootDirectory)) for (const { entry, scan, committedFilesOnly } of enabledScanRules) for (const finding of scan(file)) {
+		if (committedFilesOnly && isFileGitIgnored(file)) continue;
 		const diagnostic = buildSecurityScanDiagnostic(finding, entry, file.relativePath);
 		const key = `${diagnostic.rule}:${diagnostic.filePath}:${diagnostic.line}:${diagnostic.column}:${diagnostic.message}`;
 		if (seen.has(key)) continue;
@@ -42773,6 +42784,8 @@ const resolveConcreteVersion = (spec) => {
 	const trimmed = spec.trim();
 	if (trimmed.length === 0) return null;
 	if (trimmed.includes(":")) return null;
+	const range = import_semver.validRange(trimmed);
+	if (range === null || range === "*") return null;
 	return import_semver.minVersion(trimmed)?.version ?? null;
 };
 const locateDependencyKey = (packageJsonText, section, name) => {
@@ -43159,8 +43172,10 @@ const runInspect = (input, hooks = {}) => gen(function* () {
 	const lintFailureState = yield* get$2(lintFailure);
 	yield* afterLint(lintFailureState.didFail);
 	if (lintFailureState.didFail) yield* scanProgress.fail(formatLintFailText(lintFailureState.reasonTag, process.version));
+	const totalFileCount = lastReportedTotalFileCount || (lintIncludePaths?.length ?? project.sourceFileCount);
+	const scannedFilesLabel = `${totalFileCount} ${totalFileCount === 1 ? "file" : "files"}`;
 	const shouldRunDeadCode = input.runDeadCode && !isDiffMode && (showWarnings || deadCodeMaySurfaceWhenWarningsHidden(resolvedConfig.config));
-	const deadCodeCollected = lintFailureState.didFail || !shouldRunDeadCode ? [] : yield* scanProgress.update("Analyzing dead code...").pipe(andThen(runCollect(applyPerElementPipeline(deadCodeService.run({
+	const deadCodeCollected = lintFailureState.didFail || !shouldRunDeadCode ? [] : yield* scanProgress.update(`Scanned ${scannedFilesLabel}, analyzing dead code...`).pipe(andThen(runCollect(applyPerElementPipeline(deadCodeService.run({
 		rootDirectory: scanDirectory,
 		userConfig: resolvedConfig.config
 	}).pipe(catchTag("ReactDoctorError", (error) => unwrap(gen(function* () {
@@ -43173,10 +43188,9 @@ const runInspect = (input, hooks = {}) => gen(function* () {
 	const deadCodeFailureState = yield* get$2(deadCodeFailure);
 	const scanElapsedMilliseconds = Date.now() - scanStartTime;
 	const scanElapsedSeconds = (scanElapsedMilliseconds / 1e3).toFixed(1);
-	const totalFileCount = lastReportedTotalFileCount || (lintIncludePaths?.length ?? project.sourceFileCount);
 	if (!lintFailureState.didFail) if (deadCodeFailureState.didFail) yield* scanProgress.fail(DEAD_CODE_FAIL_TEXT);
 	else if (input.suppressScanSummary) yield* scanProgress.stop();
-	else yield* scanProgress.succeed(`Scanned ${totalFileCount} ${totalFileCount === 1 ? "file" : "files"} in ${scanElapsedSeconds}s${workerCountSuffix}`);
+	else yield* scanProgress.succeed(`Scanned ${scannedFilesLabel} in ${scanElapsedSeconds}s${workerCountSuffix}`);
 	yield* reporterService.finalize;
 	const finalDiagnostics = [
 		...envCollected,
@@ -43958,7 +43972,7 @@ const makeNoopConsole = () => ({
 });
 //#endregion
 //#region src/cli/utils/version.ts
-const VERSION = "0.5.5";
+const VERSION = "0.5.6";
 //#endregion
 //#region src/cli/utils/json-mode.ts
 let context = null;
@@ -44316,13 +44330,13 @@ const isDevVersion = (version) => version === "0.0.0" || version.includes("-");
 * uploads source-map artifacts under, so stack frames symbolicate. Honors the
 * standard `SENTRY_RELEASE` override.
 */
-const resolveSentryRelease = () => process.env.SENTRY_RELEASE || `react-doctor@0.5.5`;
+const resolveSentryRelease = () => process.env.SENTRY_RELEASE || `react-doctor@0.5.6`;
 /**
 * Deployment environment shown in Sentry's environment filter. Defaults to
 * `production` for tagged releases and `development` for dev/unbuilt versions,
 * overridable via the standard `SENTRY_ENVIRONMENT` env var.
 */
-const resolveSentryEnvironment = () => process.env.SENTRY_ENVIRONMENT || (isDevVersion("0.5.5") ? "development" : "production");
+const resolveSentryEnvironment = () => process.env.SENTRY_ENVIRONMENT || (isDevVersion("0.5.6") ? "development" : "production");
 /**
 * Performance-tracing sample rate in `[0, 1]`. Reads `SENTRY_TRACES_SAMPLE_RATE`
 * (set to `0` to disable tracing) and falls back to
@@ -53934,4 +53948,4 @@ Promise.resolve().then(() => assertNoRemovedFlags(process.argv)).then(() => prog
 export {};
 
 //# sourceMappingURL=cli.js.map
-//# debugId=06a2a696-2f0a-515a-8c24-d1a9b9898c6e
+//# debugId=79c9f922-1713-5894-a0aa-142d0cd9c7ba

package/dist/index.js

@@ -1,5 +1,5 @@
 
-!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]="547655b4-95a2-5599-be6e-d30cf62327d7")}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]="f55d58c3-c6f3-598b-bd99-d0abe85fd35e")}catch(e){}}();
 import { r as __toESM$1, t as __commonJSMin$1 } from "./chunk-N93fKeF6.js";
 import { createRequire } from "node:module";
 import * as NFS from "node:fs";
@@ -36520,13 +36520,24 @@ const checkSecurityScan = (rootDirectory, options = {}) => {
 		if (!shouldEnableRule(rule.requires, rule.tags, capabilities, ignoredTags, rule.disabledBy)) return [];
 		return [{
 			entry,
-			scan
+			scan,
+			committedFilesOnly: rule.committedFilesOnly === true
 		}];
 	});
 	if (enabledScanRules.length === 0) return [];
 	const diagnostics = [];
 	const seen = /* @__PURE__ */ new Set();
-	for (const file of collectSecurityScanFiles(rootDirectory)) for (const { entry, scan } of enabledScanRules) for (const finding of scan(file)) {
+	const gitIgnoredCache = /* @__PURE__ */ new Map();
+	const isFileGitIgnored = (file) => {
+		let status = gitIgnoredCache.get(file.absolutePath);
+		if (status === void 0) {
+			status = isPathGitIgnored(rootDirectory, file.absolutePath);
+			gitIgnoredCache.set(file.absolutePath, status);
+		}
+		return status === true;
+	};
+	for (const file of collectSecurityScanFiles(rootDirectory)) for (const { entry, scan, committedFilesOnly } of enabledScanRules) for (const finding of scan(file)) {
+		if (committedFilesOnly && isFileGitIgnored(file)) continue;
 		const diagnostic = buildSecurityScanDiagnostic(finding, entry, file.relativePath);
 		const key = `${diagnostic.rule}:${diagnostic.filePath}:${diagnostic.line}:${diagnostic.column}:${diagnostic.message}`;
 		if (seen.has(key)) continue;
@@ -39537,6 +39548,8 @@ const resolveConcreteVersion = (spec) => {
 	const trimmed = spec.trim();
 	if (trimmed.length === 0) return null;
 	if (trimmed.includes(":")) return null;
+	const range = import_semver.validRange(trimmed);
+	if (range === null || range === "*") return null;
 	return import_semver.minVersion(trimmed)?.version ?? null;
 };
 const locateDependencyKey = (packageJsonText, section, name) => {
@@ -39923,8 +39936,10 @@ const runInspect = (input, hooks = {}) => gen(function* () {
 	const lintFailureState = yield* get$2(lintFailure);
 	yield* afterLint(lintFailureState.didFail);
 	if (lintFailureState.didFail) yield* scanProgress.fail(formatLintFailText(lintFailureState.reasonTag, process.version));
+	const totalFileCount = lastReportedTotalFileCount || (lintIncludePaths?.length ?? project.sourceFileCount);
+	const scannedFilesLabel = `${totalFileCount} ${totalFileCount === 1 ? "file" : "files"}`;
 	const shouldRunDeadCode = input.runDeadCode && !isDiffMode && (showWarnings || deadCodeMaySurfaceWhenWarningsHidden(resolvedConfig.config));
-	const deadCodeCollected = lintFailureState.didFail || !shouldRunDeadCode ? [] : yield* scanProgress.update("Analyzing dead code...").pipe(andThen(runCollect(applyPerElementPipeline(deadCodeService.run({
+	const deadCodeCollected = lintFailureState.didFail || !shouldRunDeadCode ? [] : yield* scanProgress.update(`Scanned ${scannedFilesLabel}, analyzing dead code...`).pipe(andThen(runCollect(applyPerElementPipeline(deadCodeService.run({
 		rootDirectory: scanDirectory,
 		userConfig: resolvedConfig.config
 	}).pipe(catchTag("ReactDoctorError", (error) => unwrap(gen(function* () {
@@ -39937,10 +39952,9 @@ const runInspect = (input, hooks = {}) => gen(function* () {
 	const deadCodeFailureState = yield* get$2(deadCodeFailure);
 	const scanElapsedMilliseconds = Date.now() - scanStartTime;
 	const scanElapsedSeconds = (scanElapsedMilliseconds / 1e3).toFixed(1);
-	const totalFileCount = lastReportedTotalFileCount || (lintIncludePaths?.length ?? project.sourceFileCount);
 	if (!lintFailureState.didFail) if (deadCodeFailureState.didFail) yield* scanProgress.fail(DEAD_CODE_FAIL_TEXT);
 	else if (input.suppressScanSummary) yield* scanProgress.stop();
-	else yield* scanProgress.succeed(`Scanned ${totalFileCount} ${totalFileCount === 1 ? "file" : "files"} in ${scanElapsedSeconds}s${workerCountSuffix}`);
+	else yield* scanProgress.succeed(`Scanned ${scannedFilesLabel} in ${scanElapsedSeconds}s${workerCountSuffix}`);
 	yield* reporterService.finalize;
 	const finalDiagnostics = [
 		...envCollected,
@@ -40559,4 +40573,4 @@ const toJsonReport = (result, options) => buildJsonReport({
 export { AmbiguousProjectError, NoReactDependencyError, NotADirectoryError, PackageJsonNotFoundError, ProjectNotFoundError, ReactDoctorError, buildJsonReport, buildJsonReportError, clearCaches, defineConfig, diagnose, filterSourceFiles, getDiffInfo, isProjectDiscoveryError, isReactDoctorError, summarizeDiagnostics, toJsonReport };
 
 //# sourceMappingURL=index.js.map
-//# debugId=547655b4-95a2-5599-be6e-d30cf62327d7
+//# debugId=f55d58c3-c6f3-598b-bd99-d0abe85fd35e

package/dist/lsp.js

@@ -36506,13 +36506,24 @@ const checkSecurityScan = (rootDirectory, options = {}) => {
 		if (!shouldEnableRule(rule.requires, rule.tags, capabilities, ignoredTags, rule.disabledBy)) return [];
 		return [{
 			entry,
-			scan
+			scan,
+			committedFilesOnly: rule.committedFilesOnly === true
 		}];
 	});
 	if (enabledScanRules.length === 0) return [];
 	const diagnostics = [];
 	const seen = /* @__PURE__ */ new Set();
-	for (const file of collectSecurityScanFiles(rootDirectory)) for (const { entry, scan } of enabledScanRules) for (const finding of scan(file)) {
+	const gitIgnoredCache = /* @__PURE__ */ new Map();
+	const isFileGitIgnored = (file) => {
+		let status = gitIgnoredCache.get(file.absolutePath);
+		if (status === void 0) {
+			status = isPathGitIgnored(rootDirectory, file.absolutePath);
+			gitIgnoredCache.set(file.absolutePath, status);
+		}
+		return status === true;
+	};
+	for (const file of collectSecurityScanFiles(rootDirectory)) for (const { entry, scan, committedFilesOnly } of enabledScanRules) for (const finding of scan(file)) {
+		if (committedFilesOnly && isFileGitIgnored(file)) continue;
 		const diagnostic = buildSecurityScanDiagnostic(finding, entry, file.relativePath);
 		const key = `${diagnostic.rule}:${diagnostic.filePath}:${diagnostic.line}:${diagnostic.column}:${diagnostic.message}`;
 		if (seen.has(key)) continue;
@@ -39523,6 +39534,8 @@ const resolveConcreteVersion = (spec) => {
 	const trimmed = spec.trim();
 	if (trimmed.length === 0) return null;
 	if (trimmed.includes(":")) return null;
+	const range = import_semver.validRange(trimmed);
+	if (range === null || range === "*") return null;
 	return import_semver.minVersion(trimmed)?.version ?? null;
 };
 const locateDependencyKey = (packageJsonText, section, name) => {
@@ -39909,8 +39922,10 @@ const runInspect = (input, hooks = {}) => gen(function* () {
 	const lintFailureState = yield* get$2(lintFailure);
 	yield* afterLint(lintFailureState.didFail);
 	if (lintFailureState.didFail) yield* scanProgress.fail(formatLintFailText(lintFailureState.reasonTag, process.version));
+	const totalFileCount = lastReportedTotalFileCount || (lintIncludePaths?.length ?? project.sourceFileCount);
+	const scannedFilesLabel = `${totalFileCount} ${totalFileCount === 1 ? "file" : "files"}`;
 	const shouldRunDeadCode = input.runDeadCode && !isDiffMode && (showWarnings || deadCodeMaySurfaceWhenWarningsHidden(resolvedConfig.config));
-	const deadCodeCollected = lintFailureState.didFail || !shouldRunDeadCode ? [] : yield* scanProgress.update("Analyzing dead code...").pipe(andThen(runCollect(applyPerElementPipeline(deadCodeService.run({
+	const deadCodeCollected = lintFailureState.didFail || !shouldRunDeadCode ? [] : yield* scanProgress.update(`Scanned ${scannedFilesLabel}, analyzing dead code...`).pipe(andThen(runCollect(applyPerElementPipeline(deadCodeService.run({
 		rootDirectory: scanDirectory,
 		userConfig: resolvedConfig.config
 	}).pipe(catchTag("ReactDoctorError", (error) => unwrap(gen(function* () {
@@ -39923,10 +39938,9 @@ const runInspect = (input, hooks = {}) => gen(function* () {
 	const deadCodeFailureState = yield* get$2(deadCodeFailure);
 	const scanElapsedMilliseconds = Date.now() - scanStartTime;
 	const scanElapsedSeconds = (scanElapsedMilliseconds / 1e3).toFixed(1);
-	const totalFileCount = lastReportedTotalFileCount || (lintIncludePaths?.length ?? project.sourceFileCount);
 	if (!lintFailureState.didFail) if (deadCodeFailureState.didFail) yield* scanProgress.fail(DEAD_CODE_FAIL_TEXT);
 	else if (input.suppressScanSummary) yield* scanProgress.stop();
-	else yield* scanProgress.succeed(`Scanned ${totalFileCount} ${totalFileCount === 1 ? "file" : "files"} in ${scanElapsedSeconds}s${workerCountSuffix}`);
+	else yield* scanProgress.succeed(`Scanned ${scannedFilesLabel} in ${scanElapsedSeconds}s${workerCountSuffix}`);
 	yield* reporterService.finalize;
 	const finalDiagnostics = [
 		...envCollected,
@@ -40297,7 +40311,7 @@ const computeConfigFingerprint = (projectDirectory, version) => {
 /** Display name used in client-facing messages and progress titles. */
 const SERVER_DISPLAY_NAME = "React Doctor";
 /** Server version reported in `serverInfo`; injected at build, `dev` from source. */
-const SERVER_VERSION = "0.5.5";
+const SERVER_VERSION = "0.5.6";
 /** `Diagnostic.source` shown next to every published diagnostic. */
 const DIAGNOSTIC_SOURCE = "react-doctor";
 /**
@@ -42344,5 +42358,5 @@ const startLanguageServer = () => {
 };
 //#endregion
 export { startLanguageServer };
-!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]="98cc6fbf-9034-527e-835c-0b5664ef94b5")}catch(e){}}();
-//# debugId=98cc6fbf-9034-527e-835c-0b5664ef94b5
+!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]="8882c2be-9063-5954-a6a3-fc287e8c715f")}catch(e){}}();
+//# debugId=8882c2be-9063-5954-a6a3-fc287e8c715f

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.5.5",
+  "version": "0.5.6",
   "description": "Your agent writes bad React. This catches it",
   "keywords": [
     "accessibility",
@@ -64,7 +64,7 @@
     "vscode-languageserver": "^9.0.1",
     "vscode-languageserver-textdocument": "^1.0.12",
     "vscode-uri": "^3.1.0",
-    "oxlint-plugin-react-doctor": "0.5.5"
+    "oxlint-plugin-react-doctor": "0.5.6"
   },
   "devDependencies": {
     "@types/babel__code-frame": "^7.27.0",
@@ -72,9 +72,9 @@
     "@xterm/headless": "^6.0.0",
     "commander": "^14.0.3",
     "ora": "^9.4.0",
-    "@react-doctor/api": "0.5.5",
-    "@react-doctor/core": "0.5.5",
-    "@react-doctor/language-server": "0.5.5"
+    "@react-doctor/language-server": "0.5.6",
+    "@react-doctor/api": "0.5.6",
+    "@react-doctor/core": "0.5.6"
   },
   "engines": {
     "node": "^20.19.0 || >=22.13.0"