react-doctor 0.2.14-dev.ac3ca1a → 0.2.14-dev.b612664

package/README.md

@@ -69,6 +69,8 @@ jobs:
 
 React Doctor scans the files changed in the pull request, emits inline annotations, blocks on error-level findings, and updates one sticky PR comment with the score and issue summary. The built-in GitHub token is used automatically; no secret or PAT is required. On forked PRs where GitHub withholds write permissions, the scan and annotations still run, but the sticky comment may be skipped.
 
+**Permissions:** set `permissions: { contents: read, pull-requests: write }` so React Doctor can read the pull request's changed files for a changed-files-only scan and post the sticky summary comment. If `pull-requests: read` is unavailable (for example on fork PRs or with a restricted default token), the action degrades gracefully to a full-project scan instead of failing.
+
 [Add GitHub Action →](https://github.com/marketplace/actions/react-doctor)
 
 ### 4. Configure rules in `react-doctor.config.json`

package/dist/{cli-logger-Cqq0L4Uo.js → cli-logger-BgVL1vBI.js}

@@ -6941,17 +6941,21 @@ var Reporter = class Reporter extends Context.Service()("react-doctor/Reporter")
 		});
 	}));
 };
-const parseScoreResult = (value) => {
-	if (typeof value !== "object" || value === null) return null;
-	if (!("score" in value) || !("label" in value)) return null;
-	const scoreValue = Reflect.get(value, "score");
-	const labelValue = Reflect.get(value, "label");
-	if (typeof scoreValue !== "number" || typeof labelValue !== "string") return null;
-	return {
-		score: scoreValue,
-		label: labelValue
-	};
-};
+const RulePrioritySchema = Schema.Struct({
+	priority: Schema.NullOr(Schema.Number),
+	tier: Schema.Literals([
+		"P0",
+		"P1",
+		"P2",
+		"P3"
+	])
+});
+const ScoreApiResponseSchema = Schema.Struct({
+	score: Schema.Number,
+	label: Schema.String,
+	rules: Schema.optional(Schema.Record(Schema.String, RulePrioritySchema))
+});
+const parseScoreResult = (value) => Option.getOrNull(Schema.decodeUnknownOption(ScoreApiResponseSchema)(value));
 const stripFilePaths = (diagnostics) => diagnostics.map(({ filePath: _filePath, ...rest }) => rest);
 const isAbortError = (error) => error instanceof Error && (error.name === "AbortError" || error.name === "TimeoutError");
 const describeFailure = (error) => {
@@ -7715,4 +7719,4 @@ const cliLogger = {
 //#endregion
 export { highlighter as A, discoverReactSubprojects as C, formatReactDoctorError as D, formatErrorChain as E, resolveScanTarget as F, restoreLegacyThrow as I, runInspect as L, isReactDoctorError as M, layerOtlp as N, getDiffInfo as O, listWorkspacePackages as P, toRelativePath as R, buildRulePromptUrl as S, filterSourceFiles as T, SKILL_NAME as _, DeadCode as a, buildJsonReport as b, Git as c, NodeResolver as d, OXLINT_NODE_REQUIREMENT as f, SHARE_BASE_URL as g, Reporter as h, Config as i, isMonorepoRoot as j, groupBy as k, LintPartialFailures as l, Project as m, cli_logger_exports as n, ENTERPRISE_CONTACT_URL as o, Progress as p, CANONICAL_GITHUB_URL as r, Files as s, cliLogger as t, Linter as u, Score as v, filterDiagnosticsForSurface as w, buildJsonReportError as x, StagedFiles as y };
 
-//# sourceMappingURL=cli-logger-Cqq0L4Uo.js.map
+//# sourceMappingURL=cli-logger-BgVL1vBI.js.map

package/dist/cli.js

@@ -1,5 +1,5 @@
 import { i as __toESM, n as __exportAll, r as __require, t as __commonJSMin } from "./rolldown-runtime-uZX_iqCz.js";
-import { A as highlighter, C as discoverReactSubprojects, D as formatReactDoctorError, E as formatErrorChain, F as resolveScanTarget, I as restoreLegacyThrow, L as runInspect, M as isReactDoctorError, N as layerOtlp, O as getDiffInfo, P as listWorkspacePackages, R as toRelativePath, S as buildRulePromptUrl, T as filterSourceFiles, _ as SKILL_NAME, a as DeadCode, b as buildJsonReport, c as Git, d as NodeResolver, f as OXLINT_NODE_REQUIREMENT, g as SHARE_BASE_URL, h as Reporter, i as Config, j as isMonorepoRoot, k as groupBy, l as LintPartialFailures, m as Project, o as ENTERPRISE_CONTACT_URL, p as Progress, r as CANONICAL_GITHUB_URL, s as Files, t as cliLogger, u as Linter, v as Score, w as filterDiagnosticsForSurface, x as buildJsonReportError, y as StagedFiles } from "./cli-logger-Cqq0L4Uo.js";
+import { A as highlighter, C as discoverReactSubprojects, D as formatReactDoctorError, E as formatErrorChain, F as resolveScanTarget, I as restoreLegacyThrow, L as runInspect, M as isReactDoctorError, N as layerOtlp, O as getDiffInfo, P as listWorkspacePackages, R as toRelativePath, S as buildRulePromptUrl, T as filterSourceFiles, _ as SKILL_NAME, a as DeadCode, b as buildJsonReport, c as Git, d as NodeResolver, f as OXLINT_NODE_REQUIREMENT, g as SHARE_BASE_URL, h as Reporter, i as Config, j as isMonorepoRoot, k as groupBy, l as LintPartialFailures, m as Project, o as ENTERPRISE_CONTACT_URL, p as Progress, r as CANONICAL_GITHUB_URL, s as Files, t as cliLogger, u as Linter, v as Score, w as filterDiagnosticsForSurface, x as buildJsonReportError, y as StagedFiles } from "./cli-logger-BgVL1vBI.js";
 import { createRequire } from "node:module";
 import { execFileSync, execSync } from "node:child_process";
 import path, { join } from "node:path";
@@ -6198,7 +6198,22 @@ const SEVERITY_ORDER = {
 	warning: 1
 };
 const colorizeBySeverity = (text, severity) => severity === "error" ? highlighter.error(text) : highlighter.warn(text);
-const sortByImportance = (diagnosticGroups) => diagnosticGroups.toSorted(([, diagnosticsA], [, diagnosticsB]) => {
+const buildRulePriorityMap = (scores) => {
+	const rulePriority = /* @__PURE__ */ new Map();
+	for (const score of scores) {
+		if (!score?.rules) continue;
+		for (const [ruleKey, info] of Object.entries(score.rules)) if (typeof info.priority === "number") rulePriority.set(ruleKey, info.priority);
+	}
+	return rulePriority;
+};
+const effectivePriority = (ruleKey, diagnostics, rulePriority) => {
+	const known = rulePriority?.get(ruleKey);
+	if (known !== void 0) return known;
+	return diagnostics[0].severity === "error" ? 55 : 35;
+};
+const sortByImportance = (diagnosticGroups, rulePriority) => diagnosticGroups.toSorted(([ruleKeyA, diagnosticsA], [ruleKeyB, diagnosticsB]) => {
+	const priorityDelta = effectivePriority(ruleKeyB, diagnosticsB, rulePriority) - effectivePriority(ruleKeyA, diagnosticsA, rulePriority);
+	if (priorityDelta !== 0) return priorityDelta;
 	const severityDelta = SEVERITY_ORDER[diagnosticsA[0].severity] - SEVERITY_ORDER[diagnosticsB[0].severity];
 	if (severityDelta !== 0) return severityDelta;
 	return diagnosticsB.length - diagnosticsA.length;
@@ -6235,14 +6250,20 @@ const buildCompactRuleGroupLine = (ruleKey, ruleDiagnostics, ruleNameColumnWidth
 	return `  ${icon} ${siteCountBadge.length > 0 ? colorizeBySeverity(padRuleNameToColumn(ruleKey, ruleNameColumnWidth), firstDiagnostic.severity) : colorizeBySeverity(ruleKey, firstDiagnostic.severity)}${siteCountBadge.length > 0 ? ` ${highlighter.gray(siteCountBadge)}` : ""}`;
 };
 const getWorstSeverity = (diagnostics) => diagnostics.some((diagnostic) => diagnostic.severity === "error") ? "error" : "warning";
-const buildCategoryDiagnosticGroups = (diagnostics) => {
+const categoryTopPriority = (categoryGroup, rulePriority) => {
+	const [topRuleKey, topDiagnostics] = categoryGroup.ruleGroups[0];
+	return effectivePriority(topRuleKey, topDiagnostics, rulePriority);
+};
+const buildCategoryDiagnosticGroups = (diagnostics, rulePriority) => {
 	return [...groupBy(diagnostics, (diagnostic) => diagnostic.category).entries()].map(([category, categoryDiagnostics]) => {
 		return {
 			category,
 			diagnostics: categoryDiagnostics,
-			ruleGroups: sortByImportance([...groupBy(categoryDiagnostics, (diagnostic) => `${diagnostic.plugin}/${diagnostic.rule}`).entries()])
+			ruleGroups: sortByImportance([...groupBy(categoryDiagnostics, (diagnostic) => `${diagnostic.plugin}/${diagnostic.rule}`).entries()], rulePriority)
 		};
 	}).toSorted((categoryGroupA, categoryGroupB) => {
+		const priorityDelta = categoryTopPriority(categoryGroupB, rulePriority) - categoryTopPriority(categoryGroupA, rulePriority);
+		if (priorityDelta !== 0) return priorityDelta;
 		const severityDelta = SEVERITY_ORDER[getWorstSeverity(categoryGroupA.diagnostics)] - SEVERITY_ORDER[getWorstSeverity(categoryGroupB.diagnostics)];
 		if (severityDelta !== 0) return severityDelta;
 		if (categoryGroupA.diagnostics.length !== categoryGroupB.diagnostics.length) return categoryGroupB.diagnostics.length - categoryGroupA.diagnostics.length;
@@ -6273,8 +6294,8 @@ const buildVerboseRuleGroupLines = (ruleKey, ruleDiagnostics, ruleNameColumnWidt
 	lines.push("");
 	return lines;
 };
-const buildDefaultDiagnosticsLines = (diagnostics) => {
-	const categoryGroups = buildCategoryDiagnosticGroups(diagnostics);
+const buildDefaultDiagnosticsLines = (diagnostics, rulePriority) => {
+	const categoryGroups = buildCategoryDiagnosticGroups(diagnostics, rulePriority);
 	const lines = [];
 	for (const categoryGroup of categoryGroups) lines.push(buildCompactCategoryLine(categoryGroup));
 	lines.push("");
@@ -6286,11 +6307,11 @@ const buildDefaultDiagnosticsLines = (diagnostics) => {
 * single Effect.forEach over Console.log so failures or fiber
 * interruption produce predictable partial output.
 */
-const printDiagnostics = (diagnostics, isVerbose, rootDirectory) => Effect.gen(function* () {
+const printDiagnostics = (diagnostics, isVerbose, rootDirectory, rulePriority) => Effect.gen(function* () {
 	let lines;
-	if (!isVerbose) lines = buildDefaultDiagnosticsLines(diagnostics);
+	if (!isVerbose) lines = buildDefaultDiagnosticsLines(diagnostics, rulePriority);
 	else {
-		const sortedRuleGroups = sortByImportance([...groupBy(diagnostics, (diagnostic) => `${diagnostic.plugin}/${diagnostic.rule}`).entries()]);
+		const sortedRuleGroups = sortByImportance([...groupBy(diagnostics, (diagnostic) => `${diagnostic.plugin}/${diagnostic.rule}`).entries()], rulePriority);
 		const ruleNameColumnWidth = computeRuleNameColumnWidth(sortedRuleGroups.map(([ruleKey]) => ruleKey));
 		lines = sortedRuleGroups.flatMap(([ruleKey, ruleDiagnostics]) => buildVerboseRuleGroupLines(ruleKey, ruleDiagnostics, ruleNameColumnWidth));
 	}
@@ -6686,7 +6707,7 @@ const resolveOxlintNodeEffect = (isLintEnabled, isQuiet) => Effect.gen(function*
 const resolveOxlintNode = (isLintEnabled, isQuiet) => Effect.runPromise(resolveOxlintNodeEffect(isLintEnabled, isQuiet).pipe(Effect.provide(NodeResolver.layerNode)));
 //#endregion
 //#region src/cli/utils/version.ts
-const VERSION = "0.2.14-dev.ac3ca1a";
+const VERSION = "0.2.14-dev.b612664";
 //#endregion
 //#region src/inspect.ts
 const silentConsole = makeNoopConsole();
@@ -6847,7 +6868,7 @@ const finalizeAndRender = (input) => Effect.gen(function* () {
 		return buildResult();
 	}
 	yield* Console.log("");
-	yield* printDiagnostics([...surfaceDiagnostics], options.verbose, directory);
+	yield* printDiagnostics([...surfaceDiagnostics], options.verbose, directory, buildRulePriorityMap([score]));
 	if (options.isNonInteractiveEnvironment && options.outputSurface !== "prComment") yield* printAgentGuidance();
 	if (demotedDiagnosticCount > 0) {
 		yield* Console.log(highlighter.gray(`  ${demotedDiagnosticCount} demoted from the ${options.outputSurface} surface (e.g. design cleanup) — run \`npx react-doctor@latest .\` locally for the full list.`));
@@ -7236,7 +7257,7 @@ const printMultiProjectSummary = (input) => Effect.gen(function* () {
 	const surfaceDiagnostics = filterDiagnosticsForSurface(completedScans.flatMap((scan) => scan.result.diagnostics), "cli", userConfig);
 	if (surfaceDiagnostics.length > 0) {
 		yield* Console.log("");
-		yield* printDiagnostics(surfaceDiagnostics, verbose, "");
+		yield* printDiagnostics(surfaceDiagnostics, verbose, "", buildRulePriorityMap(completedScans.map((scan) => scan.result.score)));
 	}
 	const aggregateScore = computeAggregateScore(completedScans);
 	const totalSourceFileCount = completedScans.reduce((sum, scan) => sum + scan.result.project.sourceFileCount, 0);
@@ -7507,7 +7528,7 @@ const warnSetupPromptFailure = async (options, error) => {
 		return;
 	}
 	try {
-		const { cliLogger } = await import("./cli-logger-Cqq0L4Uo.js").then((n) => n.n);
+		const { cliLogger } = await import("./cli-logger-BgVL1vBI.js").then((n) => n.n);
 		cliLogger.warn(message);
 	} catch {}
 };

package/dist/index.d.ts

@@ -349,9 +349,15 @@ interface ProjectInfo {
 }
 //#endregion
 //#region src/types/score.d.ts
+type RuleTier = "P0" | "P1" | "P2" | "P3";
+interface RulePriority {
+  readonly priority: number | null;
+  readonly tier: RuleTier;
+}
 interface ScoreResult {
   score: number;
   label: string;
+  readonly rules?: Readonly<Record<string, RulePriority>>;
 } //#endregion
 //#region src/types/diagnose.d.ts
 interface DiagnoseOptions {

package/dist/index.js

@@ -6970,17 +6970,21 @@ var Reporter = class Reporter extends Context.Service()("react-doctor/Reporter")
 		});
 	}));
 };
-const parseScoreResult = (value) => {
-	if (typeof value !== "object" || value === null) return null;
-	if (!("score" in value) || !("label" in value)) return null;
-	const scoreValue = Reflect.get(value, "score");
-	const labelValue = Reflect.get(value, "label");
-	if (typeof scoreValue !== "number" || typeof labelValue !== "string") return null;
-	return {
-		score: scoreValue,
-		label: labelValue
-	};
-};
+const RulePrioritySchema = Schema.Struct({
+	priority: Schema.NullOr(Schema.Number),
+	tier: Schema.Literals([
+		"P0",
+		"P1",
+		"P2",
+		"P3"
+	])
+});
+const ScoreApiResponseSchema = Schema.Struct({
+	score: Schema.Number,
+	label: Schema.String,
+	rules: Schema.optional(Schema.Record(Schema.String, RulePrioritySchema))
+});
+const parseScoreResult = (value) => Option.getOrNull(Schema.decodeUnknownOption(ScoreApiResponseSchema)(value));
 const stripFilePaths = (diagnostics) => diagnostics.map(({ filePath: _filePath, ...rest }) => rest);
 const isAbortError = (error) => error instanceof Error && (error.name === "AbortError" || error.name === "TimeoutError");
 const describeFailure = (error) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-doctor",
-  "version": "0.2.14-dev.ac3ca1a",
+  "version": "0.2.14-dev.b612664",
   "description": "Diagnose and fix React codebases for security, performance, correctness, accessibility, bundle-size, and architecture issues",
   "keywords": [
     "accessibility",
@@ -58,7 +58,7 @@
     "oxlint": "^1.66.0",
     "prompts": "^2.4.2",
     "typescript": ">=5.0.4 <7",
-    "oxlint-plugin-react-doctor": "0.2.14-dev.ac3ca1a"
+    "oxlint-plugin-react-doctor": "0.2.14-dev.b612664"
   },
   "devDependencies": {
     "@types/prompts": "^2.4.9",