@produtype/core 0.45.0 → 0.46.0

package/dist/report/buildReport.js

@@ -287,6 +287,15 @@ function buildReport(analysis, options) {
     const maturityLevel = overallScore === null
         ? 'inconclusive'
         : (0, score_1.computeMaturity)(overallScore, coverage, openCriticals);
+    /**
+     * The observed half goes with it.
+     *
+     * "Overall score: not scored" printed directly above "Observed score: 100/100" —
+     * the report refusing to characterise a repository and then praising it in the next
+     * line. The observed number is the same arithmetic over the same three checks: if
+     * it cannot support a reading it cannot support half of one either.
+     */
+    const reportedObservedScore = inconclusive ? null : observedScore;
     const findingsByCategory = Object.fromEntries(types_1.CATEGORIES.map((c) => [c, []]));
     for (const f of findings)
         findingsByCategory[f.category].push(f);
@@ -334,7 +343,7 @@ function buildReport(analysis, options) {
         categoryScores,
         profile: productProfile,
         maturity: maturityLevel,
-        observedScore,
+        observedScore: reportedObservedScore,
         overallScore,
         inconclusive,
         inconclusiveReasons,
@@ -365,7 +374,7 @@ function buildReport(analysis, options) {
     return {
         projectPath: analysis.projectPath,
         generatedAt: new Date().toISOString(),
-        observedScore,
+        observedScore: reportedObservedScore,
         expectedCapabilityScore: expectationScore,
         overallScore,
         maturityLevel,

package/dist/report/categoryScores.d.ts

@@ -2,7 +2,14 @@ import type { Category, Finding } from './types';
 export interface CategoryScore {
     category: Category;
     /** 0-100. 100 means nothing actionable was found in this category. */
-    score: number;
+    /**
+     * `null` where nothing in the analysis touched this category.
+     *
+     * `notAssessed` has been beside it for a while and the number went on saying 100.
+     * On a report with no overall score at all, twelve categories still read 100 of 100
+     * — a repository the analyzer could not read, presented as perfect in twelve areas.
+     */
+    score: number | null;
     findingCount: number;
     /** Checks that ran and found what they were looking for. */
     verifiedCount: number;

package/dist/report/categoryScores.js

@@ -103,9 +103,10 @@ function buildCategoryScores(findings) {
          */
         const worstOpen = actionable.reduce((worst, finding) => (SEVERITY_WEIGHT[finding.severity] > SEVERITY_WEIGHT[worst] ? finding.severity : worst), 'info');
         const ceiling = SEVERITY_CEILING[worstOpen];
+        const notAssessed = categoryFindings.every((finding) => finding.status === 'unknown');
         return {
             category,
-            score: Math.max(0, Math.min(100, proportional, ceiling)),
+            score: notAssessed ? null : Math.max(0, Math.min(100, proportional, ceiling)),
             findingCount: actionable.length,
             /** Checks in this category that ran and found what they were looking for. */
             verifiedCount: assessed.filter((finding) => finding.status === 'passed').length,
@@ -122,7 +123,7 @@ function buildCategoryScores(findings) {
              * A Django school platform with no payments anywhere was reported as having
              * "no issues found in taking payments", as a strength.
              */
-            notAssessed: categoryFindings.every((finding) => finding.status === 'unknown'),
+            notAssessed,
         };
     });
 }
@@ -130,6 +131,6 @@ function buildCategoryScores(findings) {
 function weakestCategories(scores, limit = 3) {
     return [...scores]
         .filter((entry) => !entry.notAssessed && entry.findingCount > 0)
-        .sort((left, right) => left.score - right.score || right.criticalCount - left.criticalCount)
+        .sort((left, right) => (left.score ?? 100) - (right.score ?? 100) || right.criticalCount - left.criticalCount)
         .slice(0, limit);
 }

package/dist/report/executiveSummary.d.ts

@@ -20,7 +20,7 @@ export declare function buildExecutiveSummary(args: {
     categoryScores: CategoryScore[];
     profile: ProductExpectationResult | undefined;
     maturity: MaturityLevel;
-    observedScore: number;
+    observedScore: number | null;
     overallScore: number | null;
     inconclusive: boolean;
     /** Why, when the report could not form a reading. Named in the verdict. */

package/dist/report/executiveSummary.js

@@ -158,7 +158,7 @@ function buildStrengths(categoryScores, findings) {
     const passedCategories = new Set(passed.map((finding) => finding.category));
     const strongCategories = categoryScores
         .filter((entry) => !entry.notAssessed
-        && entry.score >= 80
+        && (entry.score ?? 0) >= 80
         && entry.findingCount === 0
         && passedCategories.has(entry.category));
     /**

package/dist/report/markdownReport.js

@@ -40,8 +40,10 @@ function profileSummary(report) {
         `- Selected profile: ${profile.selectedProfile}`,
         `- Profile: ${profile.profileTitle}`,
         `- Description: ${profile.profileDescription}`,
-        `- Observed score: ${report.observedScore}/100`,
-        `- Expected capability score: ${report.expectedCapabilityScore}/100`,
+        report.observedScore === null ? '- Observed score: not scored' : `- Observed score: ${report.observedScore}/100`,
+        report.expectedCapabilityScore === undefined
+            ? '- Expected capability score: not measured'
+            : `- Expected capability score: ${report.expectedCapabilityScore}/100`,
         report.overallScore === null ? '- Final score: not scored' : `- Final score: ${report.overallScore}/100`,
         '- Capability summary:',
         `- Required: ${requiredPresent} present / ${requiredMissing} missing / ${requiredPartial} partial`,
@@ -134,7 +136,7 @@ function categoryScoreSection(report) {
          */
         '| Area | Score | Verified | Open findings | Critical |',
         '| --- | --- | --- | --- | --- |',
-        ...assessed.map((entry) => `| ${entry.category} | ${entry.score}/100 | ${entry.verifiedCount} of ${entry.assessedCount} | ${entry.findingCount} | ${entry.criticalCount} |`),
+        ...assessed.map((entry) => `| ${entry.category} | ${entry.score === null ? 'not scored' : `${entry.score}/100`} | ${entry.verifiedCount} of ${entry.assessedCount} | ${entry.findingCount} | ${entry.criticalCount} |`),
         '',
     ];
 }
@@ -222,7 +224,7 @@ function renderMarkdown(report) {
         '## Score',
         '',
         report.overallScore === null ? '- Overall score: not scored' : `- Overall score: ${report.overallScore}/100`,
-        `- Observed score: ${report.observedScore}/100`,
+        report.observedScore === null ? '- Observed score: not scored' : `- Observed score: ${report.observedScore}/100`,
         ...(report.expectedCapabilityScore !== undefined ? [`- Expected capability score: ${report.expectedCapabilityScore}/100`] : []),
         `- Maturity level: ${report.maturityLevel}${report.inconclusive ? ' (inconclusive)' : ''}`,
         /**
@@ -238,7 +240,7 @@ function renderMarkdown(report) {
             : []),
         ...(report.inconclusive
             ? [
-                '- Assessment: INCONCLUSIVE — the score is capped because the project could not be recognized:',
+                '- Assessment: INCONCLUSIVE — this report has no score, for these reasons:',
                 ...report.inconclusiveReasons.map((r) => `  - ${r}`),
             ]
             : []),

package/dist/report/types.d.ts

@@ -88,7 +88,8 @@ export interface ReportDiagnostics {
 export interface ProductionReadinessReport {
     projectPath: string;
     generatedAt: string;
-    observedScore: number;
+    /** `null` where the reading is inconclusive, for the same reason `overallScore` is. */
+    observedScore: number | null;
     expectedCapabilityScore?: number;
     /**
      * `null` where the reading is inconclusive, which is the same rule this product

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@produtype/core",
-  "version": "0.45.0",
+  "version": "0.46.0",
   "description": "Deterministic CLI and library that analyzes a web application repository and reports how far it is from production-ready for the kind of product it is meant to be.",
   "license": "MIT",
   "bin": {