@mergesignal/shared 0.4.3 → 0.6.0

package/dist/actionsStepSummary.test.js

@@ -1,348 +1,53 @@
-import { describe, it, expect } from "vitest";
-import { ACTIONS_SUMMARY_DEFAULT_MAX_CHARS, ACTIONS_SUMMARY_DEFAULT_MAX_LINES, actionsSummaryDefaultFoldPrefix, buildActionsStepSummaryMarkdown, humanizeEngineSurfaceText, layerDriverSummary, layerRiskBandLabel, sortPRInsightsForDisplay, sortRecommendationsForDisplay, } from "./actionsStepSummary.js";
+import { describe, expect, it } from "vitest";
+import { assessmentFastifyRuntime } from "./fixtures/assessmentFixtures.js";
+import { buildActionsStepSummaryMarkdown } from "./actionsStepSummary.js";
 import { scanSurfaceCopyFlat } from "./scanSurfaceCopy.js";
 const copy = scanSurfaceCopyFlat();
-function makeInsight(overrides) {
-    return {
-        type: "usage_risk",
-        priority: "high",
-        confidence: "likely",
-        scope: "changed",
-        context: "ctx",
-        remediation: "fix",
-        ...overrides,
-    };
-}
-describe("layerRiskBandLabel", () => {
-    it("maps numeric layer scores to Low / Moderate / High", () => {
-        expect(layerRiskBandLabel(0)).toBe("Low");
-        expect(layerRiskBandLabel(19.9)).toBe("Low");
-        expect(layerRiskBandLabel(25)).toBe("Moderate");
-        expect(layerRiskBandLabel(50)).toBe("High");
-        expect(layerRiskBandLabel(null)).toBe("—");
-    });
-});
-describe("humanizeEngineSurfaceText", () => {
-    it("rewrites known graph metric tokens without adding new facts", () => {
-        expect(humanizeEngineSurfaceText("graph.depth")).toBe("Indirect dependency depth");
-        expect(humanizeEngineSurfaceText("graph.transitive volume")).toBe("Transitive dependency volume");
-        expect(humanizeEngineSurfaceText("graph.fan in max")).toBe("Many import paths converge on a few widely shared packages");
-    });
-    it("rewrites ecosystem surface and duplicate-path tokens", () => {
-        expect(humanizeEngineSurfaceText("Ecosystem.package surface")).toBe("Broad declared package footprint");
-        expect(humanizeEngineSurfaceText("Graph.duplicates")).toBe("Overlapping dependency paths to the same packages");
-    });
-    it("merges dependency depth phrasing into one canonical reviewer line", () => {
-        expect(humanizeEngineSurfaceText("dependency chain depth")).toBe("Indirect dependency depth");
-        expect(humanizeEngineSurfaceText("dependency depth")).toBe("Indirect dependency depth");
-    });
-    it("leaves ordinary explain titles mostly intact", () => {
-        expect(humanizeEngineSurfaceText("Duplicate majors on runtime boundary")).toBe("Duplicate majors on runtime boundary");
-    });
-});
-describe("sortPRInsightsForDisplay", () => {
-    it("orders by priority, confidence, then changed scope", () => {
-        const a = makeInsight({
-            message: "a",
-            priority: "medium",
-            confidence: "confirmed",
-            scope: "all",
-        });
-        const b = makeInsight({
-            message: "b",
-            priority: "high",
-            confidence: "likely",
-            scope: "changed",
-        });
-        const c = makeInsight({
-            message: "c",
-            priority: "high",
-            confidence: "confirmed",
-            scope: "all",
-        });
-        expect(sortPRInsightsForDisplay([a, b, c]).map((i) => i.message)).toEqual([
-            "c",
-            "b",
-            "a",
-        ]);
-    });
-});
-describe("sortRecommendationsForDisplay", () => {
-    it("orders by priorityScore then impact", () => {
-        const recs = [
-            {
-                id: "1",
-                title: "Low prio",
-                rationale: "",
-                impact: "low",
-                priorityScore: 10,
-            },
-            {
-                id: "2",
-                title: "High prio",
-                rationale: "",
-                impact: "high",
-                priorityScore: 90,
-            },
-        ];
-        expect(sortRecommendationsForDisplay(recs).map((r) => r.id)).toEqual([
-            "2",
-            "1",
-        ]);
-    });
-});
+const sampleResult = {
+    totalScore: 55,
+    layerScores: {
+        security: 10,
+        maintainability: 20,
+        ecosystem: 30,
+        upgradeImpact: 15,
+    },
+    findings: [],
+    generatedAt: "2026-01-01T00:00:00.000Z",
+    changedPackages: ["fastify"],
+    assessment: assessmentFastifyRuntime,
+    decision: {
+        recommendation: "needs_review",
+        confidence: "medium",
+        reasoning: [
+            "Changed package has confirmed usage on runtime application paths in this repository.",
+        ],
+    },
+};
 describe("buildActionsStepSummaryMarkdown", () => {
-    const trustedBase = {
-        totalScore: 42,
-        layerScores: {
-            security: 10,
-            maintainability: 50,
-            ecosystem: 30,
-            upgradeImpact: 40,
-        },
-        findings: [],
-        recommendations: [],
-        generatedAt: "2026-01-01T00:00:00.000Z",
-        methodologyVersion: "engine-test-fixture/v1",
-        decision: {
-            recommendation: "needs_review",
-            confidence: "high",
-            reasoning: ["Transitive dependency depth increased on hot paths."],
-        },
-        insights: [],
-    };
-    it("trusted: includes posture, risk index direction, and score breakdown", () => {
-        const md = buildActionsStepSummaryMarkdown({
-            result: trustedBase,
-            profile: "trusted",
-            copy,
-        });
-        expect(md).toMatch(/# MergeSignal dependency review — Needs review/);
-        expect(md).toMatch(/Risk index 42\/100/);
-        expect(md).toContain(copy["actions.riskIndexDirectionShort"]);
-        expect(md).toContain("| Layer | Score | Layer risk |");
-        expect(md).toContain("| Security | 10 | Low |");
-        expect(md).toMatch(/<details>/);
-        expect(md).not.toContain(copy["actions.layerScoreGlossary"]);
-    });
-    it("trusted: surfaces top insights and moves remainder to details", () => {
-        const insights = [
-            makeInsight({ message: "First critical", priority: "critical" }),
-            makeInsight({ message: "Second high", priority: "high" }),
-            makeInsight({ message: "Third medium", priority: "medium" }),
-            makeInsight({ message: "Fourth overflow", priority: "medium" }),
-        ];
-        const md = buildActionsStepSummaryMarkdown({
-            result: { ...trustedBase, insights },
-            profile: "trusted",
-            copy,
-        });
-        expect(md).toContain("First critical");
-        expect(md).toContain(copy["actions.moreInsightsDetailsSummary"]);
-        expect(md).toContain("Fourth overflow");
-    });
-    it("trusted: no decision uses posture-unavailable copy", () => {
-        const { decision: _d, ...rest } = trustedBase;
-        const md = buildActionsStepSummaryMarkdown({
-            result: rest,
-            profile: "trusted",
-            copy,
-        });
-        expect(md).toContain(copy["actions.mergePostureUnavailableShort"]);
-    });
-    it("development + stub shows demo banner and score breakdown", () => {
+    it("renders development profile with demo header", () => {
         const md = buildActionsStepSummaryMarkdown({
-            result: {
-                ...trustedBase,
-                methodologyVersion: "engine-stub/v2",
-                decision: {
-                    recommendation: "safe",
-                    confidence: "high",
-                    reasoning: [],
-                },
-            },
+            result: sampleResult,
             profile: "development",
             copy,
         });
-        expect(md).toContain(copy["actions.demoSummaryBanner"]);
-        expect(md).toContain("| Layer | Score | Layer risk |");
-        expect(md).toContain(copy["actions.layerScoreGlossary"]);
-    });
-    it("default-fold budget: heavy fixture stays within caps", () => {
-        const long = "x".repeat(400);
-        const insights = Array.from({ length: 12 }, (_, i) => makeInsight({
-            message: `${long} ${i}`,
-            priority: i === 0 ? "critical" : "medium",
-        }));
-        const recs = Array.from({ length: 8 }, (_, i) => ({
-            id: `r${i}`,
-            title: `Rec ${i}`,
-            rationale: long,
-            impact: "high",
-            priorityScore: 100 - i,
-        }));
-        const md = buildActionsStepSummaryMarkdown({
-            result: {
-                ...trustedBase,
-                insights,
-                recommendations: recs,
-                explain: {
-                    reasons: Array.from({ length: 10 }, (_, i) => ({
-                        id: `e${i}`,
-                        layer: "security",
-                        title: `Explain ${i}`,
-                        scoreImpact: 1,
-                    })),
-                },
-            },
-            profile: "trusted",
-            copy,
-        });
-        const prefix = actionsSummaryDefaultFoldPrefix(md);
-        expect(prefix.split("\n").length).toBeLessThanOrEqual(ACTIONS_SUMMARY_DEFAULT_MAX_LINES + 4);
-        expect(prefix.length).toBeLessThanOrEqual(ACTIONS_SUMMARY_DEFAULT_MAX_CHARS + 400);
-    });
-    it("includes scan-specific layer drivers from explain.reasons", () => {
-        const md = buildActionsStepSummaryMarkdown({
-            result: {
-                ...trustedBase,
-                layerScores: { ...trustedBase.layerScores, maintainability: 72 },
-                explain: {
-                    reasons: [
-                        {
-                            id: "x",
-                            layer: "maintainability",
-                            title: "Duplicate majors on runtime boundary",
-                            scoreImpact: 12,
-                        },
-                        {
-                            id: "y",
-                            layer: "maintainability",
-                            title: "Churn in devDependencies cluster",
-                            scoreImpact: 5,
-                        },
-                    ],
-                },
-            },
-            profile: "trusted",
-            copy,
-        });
-        expect(md).toContain(copy["actions.layerDriversHeading"]);
-        expect(md).toContain("Duplicate majors on runtime boundary");
-    });
-    it("layerDriverSummary returns null when no explain data for layer", () => {
-        expect(layerDriverSummary(trustedBase, "security", 120)).toBeNull();
-    });
-    it("generic recommendation titles lead with rationale in default fold", () => {
-        const md = buildActionsStepSummaryMarkdown({
-            result: {
-                ...trustedBase,
-                insights: [],
-                recommendations: [
-                    {
-                        id: "r1",
-                        title: "Reduce transitive dependency surface area",
-                        rationale: "This PR adds lodash twice at incompatible semver ranges pulled through different path aliases, widening the install tree.",
-                        impact: "high",
-                        priorityScore: 90,
-                    },
-                ],
-            },
-            profile: "trusted",
-            copy,
-        });
-        expect(md).toContain(copy["actions.recReviewLeadPrefix"]);
-        expect(md).toContain("incompatible semver");
-        expect(md).toContain("*Focus:*");
-        expect(md).toContain("Reduce transitive dependency surface area");
-    });
-    it("applies rec review prefix only to the first rationale-first recommendation", () => {
-        const md = buildActionsStepSummaryMarkdown({
-            result: {
-                ...trustedBase,
-                insights: [],
-                recommendations: [
-                    {
-                        id: "r1",
-                        title: "Reduce transitive dependency surface area",
-                        rationale: "This PR adds lodash twice at incompatible semver ranges pulled through different path aliases, widening the install tree.",
-                        impact: "high",
-                        priorityScore: 90,
-                    },
-                    {
-                        id: "r2",
-                        title: "Minimize dependency footprint in the service layer",
-                        rationale: "Several packages are pulled in only for types while runtime bundles still resolve overlapping majors, which increases upgrade coupling across services.",
-                        impact: "high",
-                        priorityScore: 85,
-                    },
-                ],
-            },
-            profile: "trusted",
-            copy,
-        });
-        const prefix = copy["actions.recReviewLeadPrefix"];
-        const first = md.indexOf(prefix);
-        const second = md.indexOf(prefix, first + 1);
-        expect(first).toBeGreaterThan(-1);
-        expect(second).toBe(-1);
+        expect(md).toContain("#");
+        expect(md.toLowerCase()).toContain("fastify");
     });
-    it("humanizes graph metric tokens in explain-driven layer drivers", () => {
+    it("renders trusted profile without demo header", () => {
         const md = buildActionsStepSummaryMarkdown({
-            result: {
-                ...trustedBase,
-                layerScores: { ...trustedBase.layerScores, maintainability: 72 },
-                explain: {
-                    reasons: [
-                        {
-                            id: "gdepth",
-                            layer: "maintainability",
-                            title: "graph.depth",
-                            scoreImpact: 12,
-                        },
-                    ],
-                },
-            },
+            result: sampleResult,
             profile: "trusted",
             copy,
         });
-        expect(md).toContain("Indirect dependency depth");
-        expect(md).not.toContain("graph.depth");
+        expect(md).not.toContain(copy["actions.demoSummaryBanner"] ?? "");
+        expect(md).toContain("MergeSignal");
     });
-    it("includes graph details when graphInsights present", () => {
+    it("returns failure message when assessment is missing", () => {
         const md = buildActionsStepSummaryMarkdown({
-            result: {
-                ...trustedBase,
-                graphInsights: {
-                    maxDepth: 7,
-                    nodes: 900,
-                    edges: 1200,
-                    deepest: [
-                        {
-                            kind: "deep",
-                            packageName: "lodash",
-                            direct: false,
-                            depth: 7,
-                            via: ["a", "b"],
-                        },
-                    ],
-                    hotspots: [
-                        {
-                            kind: "hotspot",
-                            packageName: "x",
-                            direct: true,
-                            depth: 1,
-                        },
-                    ],
-                    vulnerable: [],
-                },
-            },
+            result: { ...sampleResult, assessment: undefined },
             profile: "trusted",
             copy,
         });
-        expect(md).toContain(copy["actions.dependencyGraphDetailsSummary"]);
-        expect(md).toContain("moderately deep");
-        expect(md).toContain("fairly broad");
-        expect(md).toMatch(/> \*max chain length 7 · 900 resolved packages\*/);
+        expect(md).toContain("could not be completed");
     });
 });

package/dist/assessmentLabels.d.ts

@@ -0,0 +1,6 @@
+import type { MergeConcernKind } from "./assessmentSchema.js";
+/** Display labels for factor tokens (mirrors engine factor-registry labels). */
+export declare const ASSESSMENT_FACTOR_LABELS: Record<string, string>;
+export declare const MERGE_CONCERN_LABELS: Record<MergeConcernKind, string>;
+export declare function labelAssessmentFactor(token: string): string;
+//# sourceMappingURL=assessmentLabels.d.ts.map

package/dist/assessmentLabels.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"assessmentLabels.d.ts","sourceRoot":"","sources":["../src/assessmentLabels.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,uBAAuB,CAAC;AAE9D,gFAAgF;AAChF,eAAO,MAAM,wBAAwB,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAmB3D,CAAC;AAEF,eAAO,MAAM,oBAAoB,EAAE,MAAM,CAAC,gBAAgB,EAAE,MAAM,CAcjE,CAAC;AAEF,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAE3D"}

package/dist/assessmentLabels.js

@@ -0,0 +1,33 @@
+/** Display labels for factor tokens (mirrors engine factor-registry labels). */
+export const ASSESSMENT_FACTOR_LABELS = {
+    confirmed_runtime_usage: "Confirmed runtime usage",
+    queue_infrastructure: "Queue infrastructure",
+    http_framework_infrastructure: "HTTP framework infrastructure",
+    auth_infrastructure: "Authentication infrastructure",
+    moderate_reach: "Moderate repository reach",
+    high_reach: "High repository reach",
+    low_reach: "Low repository reach",
+    tooling_maintenance: "Tooling maintenance change",
+    test_infra_change: "Test infrastructure change",
+    security_advisory: "Security advisory",
+    breaking_or_major_change: "Breaking or major version change",
+    behavioral_change_risk: "Behavioral change risk",
+    ecosystem_unknown_package: "Unknown ecosystem package",
+    insufficient_collection_evidence: "Insufficient collection evidence",
+    graph_baseline_only: "Repository graph baseline only",
+    verification_required: "Verification focus required",
+    runtime_upgrade: "Runtime dependency upgrade",
+    runtime_patch: "Runtime dependency patch",
+};
+export const MERGE_CONCERN_LABELS = {
+    confirmed_runtime_usage: "Changed package has confirmed usage on runtime application paths in this repository.",
+    security_finding: "Security advisory or vulnerability finding affects changed dependencies.",
+    breaking_or_major: "Breaking change or major version bump warrants explicit review.",
+    behavioral_change: "Behavioral change risk detected on changed dependencies.",
+    ecosystem_risk: "Changed package has limited classification evidence; review dependency signals.",
+    insufficient_evidence: "Collection evidence is partial; confidence is reduced but review posture is preserved when runtime signals exist.",
+    graph_baseline_only: "Repository dependency graph baseline evaluated; no PR-specific merge concerns detected on changed packages.",
+};
+export function labelAssessmentFactor(token) {
+    return ASSESSMENT_FACTOR_LABELS[token] ?? token.replace(/_/g, " ");
+}

package/dist/assessmentProjection.d.ts

@@ -0,0 +1,12 @@
+import type { ScanResult } from "./types.js";
+import { type RepoIntelligence } from "./repoIntelligenceSchema.js";
+import type { Assessment, AssessmentPresentationPublic, ReachVisibility } from "./assessmentSchema.js";
+import type { PresentationStatus } from "./presentation/dto/types.js";
+export declare function collectVerificationFocus(result: ScanResult): string[];
+export declare function reachVisibilityLabel(visibility: ReachVisibility): string | undefined;
+export declare function formatAssessmentHeadline(assessment: Assessment, result: ScanResult, status: PresentationStatus): string;
+export declare function projectReasoningLines(result: ScanResult): string[];
+export declare function projectInsightLines(assessment: Assessment, result: ScanResult): string[];
+export declare function projectVerificationActions(assessment: Assessment, presentation: AssessmentPresentationPublic, result: ScanResult, max: number): string[];
+export declare function applicationAreaLabels(ri: RepoIntelligence | null): string[];
+//# sourceMappingURL=assessmentProjection.d.ts.map

package/dist/assessmentProjection.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"assessmentProjection.d.ts","sourceRoot":"","sources":["../src/assessmentProjection.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,YAAY,CAAC;AAC7C,OAAO,EAEL,KAAK,gBAAgB,EACtB,MAAM,6BAA6B,CAAC;AACrC,OAAO,KAAK,EACV,UAAU,EACV,4BAA4B,EAC5B,eAAe,EAChB,MAAM,uBAAuB,CAAC;AAO/B,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,6BAA6B,CAAC;AAEtE,wBAAgB,wBAAwB,CAAC,MAAM,EAAE,UAAU,GAAG,MAAM,EAAE,CAgBrE;AAED,wBAAgB,oBAAoB,CAClC,UAAU,EAAE,eAAe,GAC1B,MAAM,GAAG,SAAS,CAWpB;AAED,wBAAgB,wBAAwB,CACtC,UAAU,EAAE,UAAU,EACtB,MAAM,EAAE,UAAU,EAClB,MAAM,EAAE,kBAAkB,GACzB,MAAM,CAqDR;AAED,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,UAAU,GAAG,MAAM,EAAE,CAIlE;AAED,wBAAgB,mBAAmB,CACjC,UAAU,EAAE,UAAU,EACtB,MAAM,EAAE,UAAU,GACjB,MAAM,EAAE,CAaV;AAED,wBAAgB,0BAA0B,CACxC,UAAU,EAAE,UAAU,EACtB,YAAY,EAAE,4BAA4B,EAC1C,MAAM,EAAE,UAAU,EAClB,GAAG,EAAE,MAAM,GACV,MAAM,EAAE,CAIV;AAED,wBAAgB,qBAAqB,CAAC,EAAE,EAAE,gBAAgB,GAAG,IAAI,GAAG,MAAM,EAAE,CAI3E"}

package/dist/assessmentProjection.js

@@ -0,0 +1,100 @@
+import { safeParseRepoIntelligence, } from "./repoIntelligenceSchema.js";
+import { labelAssessmentFactor, MERGE_CONCERN_LABELS, } from "./assessmentLabels.js";
+import { scanSurfaceCopy } from "./scanSurfaceCopy.js";
+import { applyPostureVocabularyGuardHeadline } from "./presentation/intent/applyPostureVocabularyGuard.js";
+export function collectVerificationFocus(result) {
+    const parsed = safeParseRepoIntelligence(result.repoIntelligence);
+    if (!parsed.ok)
+        return [];
+    const changed = new Set(result.changedPackages ?? []);
+    const labels = [];
+    const seen = new Set();
+    for (const pkg of changed) {
+        const row = parsed.value.packages[pkg];
+        for (const focus of row?.verificationFocus ?? []) {
+            const t = focus.trim();
+            if (!t || seen.has(t))
+                continue;
+            seen.add(t);
+            labels.push(t);
+        }
+    }
+    return labels;
+}
+export function reachVisibilityLabel(visibility) {
+    switch (visibility) {
+        case "hidden":
+            return undefined;
+        case "contextual":
+            return scanSurfaceCopy.scanDetail.reachChip.limited;
+        case "prominent":
+            return scanSurfaceCopy.scanDetail.reachChip.moderate;
+        default:
+            return undefined;
+    }
+}
+export function formatAssessmentHeadline(assessment, result, status) {
+    const copy = scanSurfaceCopy.presentation;
+    const runtimePkg = assessment.concerns.find((c) => c.kind === "confirmed_runtime_usage")?.packages?.[0];
+    const primaryPkg = runtimePkg ??
+        result.changedPackages?.[0] ??
+        assessment.concerns.find((c) => c.packages?.length)?.packages?.[0] ??
+        "dependency";
+    if (assessment.primaryConcern === "insufficient_evidence") {
+        return applyPostureVocabularyGuardHeadline(copy.limitedContextHeadline, status, primaryPkg);
+    }
+    const changeClass = assessment.changeClasses[0];
+    if (assessment.posture === "safe" &&
+        (changeClass === "tooling_maintenance" || changeClass === "test_infra")) {
+        const headline = copy.toolingPatchHeadline.replace("{package}", primaryPkg);
+        return applyPostureVocabularyGuardHeadline(headline, status, primaryPkg);
+    }
+    if (assessment.primaryConcern === "confirmed_runtime_usage") {
+        const surface = assessment.factors.includes("queue_infrastructure")
+            ? "workers and queues"
+            : assessment.factors.includes("auth_infrastructure")
+                ? "authentication flows"
+                : "application code";
+        const headline = copy.runtimeUpgradeHeadline
+            .replace("{package}", primaryPkg)
+            .replace("{surface}", surface);
+        return applyPostureVocabularyGuardHeadline(headline, status, primaryPkg);
+    }
+    if (assessment.posture === "safe") {
+        return applyPostureVocabularyGuardHeadline(copy.safeNeutralHeadline.replace("{package}", primaryPkg), status, primaryPkg);
+    }
+    return applyPostureVocabularyGuardHeadline(copy.defaultUpgradeHeadline.replace("{package}", primaryPkg), status, primaryPkg);
+}
+export function projectReasoningLines(result) {
+    const fromDecision = result.decision?.reasoning?.filter(Boolean) ?? [];
+    if (fromDecision.length > 0)
+        return fromDecision.slice(0, 3);
+    return [];
+}
+export function projectInsightLines(assessment, result) {
+    const reasoning = projectReasoningLines(result);
+    if (reasoning.length > 0)
+        return reasoning;
+    const lines = [];
+    if (assessment.primaryConcern) {
+        lines.push(MERGE_CONCERN_LABELS[assessment.primaryConcern]);
+    }
+    for (const token of assessment.factors) {
+        const label = labelAssessmentFactor(token);
+        if (!lines.includes(label))
+            lines.push(label);
+    }
+    return lines.slice(0, 3);
+}
+export function projectVerificationActions(assessment, presentation, result, max) {
+    if (presentation.verificationIntensity === "none")
+        return [];
+    const focus = collectVerificationFocus(result);
+    return focus.slice(0, max);
+}
+export function applicationAreaLabels(ri) {
+    if (!ri)
+        return [];
+    const areas = ri.affectedAreas ?? ri.applicationAreas ?? [];
+    return areas.map((a) => a.label).filter(Boolean);
+}