inup 1.6.5 → 1.6.7

package/dist/features/changelog/index.js

@@ -14,6 +14,6 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-__exportStar(require("./types/changelog.types"), exports);
+__exportStar(require("./types"), exports);
 __exportStar(require("./services/changelog-service"), exports);
 //# sourceMappingURL=index.js.map

package/dist/features/changelog/services/package-metadata-service.js

@@ -3,7 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.PackageMetadataService = void 0;
 const npm_registry_client_1 = require("../clients/npm-registry-client");
 const package_metadata_1 = require("../parsers/package-metadata");
-const inflight_1 = require("../../../services/http/inflight");
+const inflight_1 = require("../../../shared/http/inflight");
 class PackageMetadataService {
     npmRegistryClient;
     cache = new Map();
@@ -16,7 +16,11 @@ class PackageMetadataService {
         this.inFlight.clear();
     }
     getCached(packageName, version) {
-        for (const key of [this.getCacheKey(packageName, version), this.getCacheKey(packageName), packageName]) {
+        for (const key of [
+            this.getCacheKey(packageName, version),
+            this.getCacheKey(packageName),
+            packageName,
+        ]) {
             if (this.cache.has(key)) {
                 return this.cache.get(key) ?? null;
             }

package/dist/features/changelog/services/release-notes-service.js

@@ -39,7 +39,6 @@ const github_client_1 = require("../clients/github-client");
 const changelog_parser_1 = require("../parsers/changelog-parser");
 const github_release_html_parser_1 = require("../parsers/github-release-html-parser");
 const RELEASE_NOTES_FETCH_TIMEOUT_MS = 5000;
-const PREFER_GITHUB_RELEASE_PAGE = true;
 class ReleaseNotesService {
     metadataService;
     githubClient;
@@ -91,15 +90,12 @@ class ReleaseNotesService {
         return null;
     }
     getGitHubSources(repoUrl, version, signal) {
-        const directReleaseSources = PREFER_GITHUB_RELEASE_PAGE
-            ? [
-                () => this.fetchGitHubReleasePageNotes(repoUrl, version, signal),
-                () => this.fetchGitHubReleaseNotes(repoUrl, version, signal),
-            ]
-            : [
-                () => this.fetchGitHubReleaseNotes(repoUrl, version, signal),
-                () => this.fetchGitHubReleasePageNotes(repoUrl, version, signal),
-            ];
+        // Scraped release page first (richer content than the API body), then the
+        // release API by tag.
+        const directReleaseSources = [
+            () => this.fetchGitHubReleasePageNotes(repoUrl, version, signal),
+            () => this.fetchGitHubReleaseNotes(repoUrl, version, signal),
+        ];
         return [
             ...directReleaseSources,
             () => this.fetchGitHubReleaseListNotes(repoUrl, version, signal),

package/dist/features/changelog/types.js

@@ -0,0 +1,3 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+//# sourceMappingURL=types.js.map

package/dist/features/debug/index.js

@@ -14,8 +14,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-__exportStar(require("./types/debug.types"), exports);
-__exportStar(require("./services/performance-tracker"), exports);
-__exportStar(require("./services/perf-logger"), exports);
-__exportStar(require("./renderer/performance-modal"), exports);
+__exportStar(require("./types"), exports);
+__exportStar(require("./performance-tracker"), exports);
+__exportStar(require("./perf-logger"), exports);
 //# sourceMappingURL=index.js.map

package/dist/features/debug/{services/perf-logger.js → perf-logger.js}

@@ -6,7 +6,7 @@ exports.getPerfDir = getPerfDir;
 exports.writePerfLog = writePerfLog;
 const fs_1 = require("fs");
 const path_1 = require("path");
-const adaptive_controller_1 = require("../../../services/http/adaptive-controller");
+const adaptive_controller_1 = require("../../shared/http/adaptive-controller");
 /**
  * Performance debug logger.
  *

package/dist/features/debug/types.js

@@ -0,0 +1,3 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+//# sourceMappingURL=types.js.map

package/dist/features/headless/headless-runner.js

@@ -5,11 +5,10 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.HeadlessRunner = void 0;
 const chalk_1 = __importDefault(require("chalk"));
-const package_detector_1 = require("../../core/package-detector");
-const upgrader_1 = require("../../core/upgrader");
-const package_manager_detector_1 = require("../../services/package-manager-detector");
-const version_1 = require("../../ui/utils/version");
-const vulnerability_audit_1 = require("./vulnerability-audit");
+const upgrade_1 = require("../upgrade");
+const package_manager_1 = require("../../shared/package-manager");
+const versions_1 = require("../../shared/versions");
+const audit_1 = require("../audit");
 const report_1 = require("./report");
 const debug_1 = require("../../features/debug");
 /**
@@ -26,7 +25,7 @@ class HeadlessRunner {
     options;
     constructor(options) {
         this.options = options;
-        this.detector = new package_detector_1.PackageDetector(options);
+        this.detector = new upgrade_1.PackageDetector(options);
     }
     async run(options) {
         try {
@@ -52,7 +51,7 @@ class HeadlessRunner {
             }
             // Audit the current versions (one bulk request, best-effort) and cross-reference each
             // advisory against the upgrade targets, so the report says whether upgrading *fixes* it.
-            const vulnerabilities = await (0, vulnerability_audit_1.auditVulnerabilities)(outdated);
+            const vulnerabilities = await (0, audit_1.auditVulnerabilities)(outdated);
             // Build the report from the *pre-apply* outdated set: it describes what this run addressed.
             const report = (0, report_1.buildHeadlessReport)(packages, outdated, vulnerabilities);
             // --apply writes the bumps + lockfile. The scan above already honored .inuprc
@@ -89,7 +88,7 @@ class HeadlessRunner {
         const packageManager = this.resolvePackageManager();
         // With --json, run the upgrader quietly: its own progress + the install child's stdout go to
         // stderr, leaving stdout for the JSON document only.
-        const upgrader = new upgrader_1.PackageUpgrader(packageManager, { quiet: json });
+        const upgrader = new upgrade_1.PackageUpgrader(packageManager, { quiet: json });
         await upgrader.upgradePackages(choices, outdated);
     }
     /**
@@ -113,7 +112,7 @@ class HeadlessRunner {
                 continue;
             const targetVersionWithPrefix = saveExact
                 ? targetVersion
-                : (0, version_1.applyVersionPrefix)(pkg.currentVersion, targetVersion);
+                : (0, versions_1.applyVersionPrefix)(pkg.currentVersion, targetVersion);
             choices.push({
                 name: pkg.name,
                 packageJsonPath: pkg.packageJsonPath,
@@ -121,6 +120,7 @@ class HeadlessRunner {
                 upgradeType: useLatest ? 'latest' : 'range',
                 targetVersion: targetVersionWithPrefix,
                 currentVersionSpecifier: pkg.currentVersion,
+                catalog: pkg.catalog,
             });
         }
         return choices;
@@ -129,8 +129,8 @@ class HeadlessRunner {
     resolvePackageManager() {
         const cwd = this.options?.cwd || process.cwd();
         return this.options?.packageManager
-            ? package_manager_detector_1.PackageManagerDetector.getInfo(this.options.packageManager)
-            : package_manager_detector_1.PackageManagerDetector.detect(cwd);
+            ? package_manager_1.PackageManagerDetector.getInfo(this.options.packageManager)
+            : package_manager_1.PackageManagerDetector.detect(cwd);
     }
 }
 exports.HeadlessRunner = HeadlessRunner;

package/dist/features/headless/index.js

@@ -14,13 +14,10 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.renderPlainReport = exports.buildHeadlessReport = exports.upgradeClears = exports.auditVulnerabilities = exports.HeadlessRunner = void 0;
+exports.renderPlainReport = exports.buildHeadlessReport = exports.HeadlessRunner = void 0;
 __exportStar(require("./types"), exports);
 var headless_runner_1 = require("./headless-runner");
 Object.defineProperty(exports, "HeadlessRunner", { enumerable: true, get: function () { return headless_runner_1.HeadlessRunner; } });
-var vulnerability_audit_1 = require("./vulnerability-audit");
-Object.defineProperty(exports, "auditVulnerabilities", { enumerable: true, get: function () { return vulnerability_audit_1.auditVulnerabilities; } });
-Object.defineProperty(exports, "upgradeClears", { enumerable: true, get: function () { return vulnerability_audit_1.upgradeClears; } });
 var report_1 = require("./report");
 Object.defineProperty(exports, "buildHeadlessReport", { enumerable: true, get: function () { return report_1.buildHeadlessReport; } });
 Object.defineProperty(exports, "renderPlainReport", { enumerable: true, get: function () { return report_1.renderPlainReport; } });

package/dist/features/headless/report.js

@@ -23,6 +23,8 @@ function buildHeadlessReport(all, outdated, vulnerabilities) {
                 packageJsonPath: pkg.packageJsonPath,
                 hasMajorUpdate: pkg.hasMajorUpdate,
             };
+            if (pkg.catalog)
+                entry.catalog = pkg.catalog;
             if (pkg.deprecated)
                 entry.deprecated = pkg.deprecated;
             if (pkg.enginesNode)

package/dist/features/interactive/controllers/index.js

@@ -0,0 +1,6 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.PackageInfoModalController = void 0;
+var package_info_modal_controller_1 = require("./package-info-modal-controller");
+Object.defineProperty(exports, "PackageInfoModalController", { enumerable: true, get: function () { return package_info_modal_controller_1.PackageInfoModalController; } });
+//# sourceMappingURL=index.js.map

package/dist/{ui → features/interactive}/controllers/package-info-modal-controller.js

@@ -35,7 +35,7 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.PackageInfoModalController = void 0;
 const semver = __importStar(require("semver"));
-const services_1 = require("../../services");
+const changelog_1 = require("../../changelog");
 const RELEASE_NOTES_LOAD_DEBOUNCE_MS = 120;
 class PackageInfoModalController {
     abortController = null;
@@ -62,7 +62,7 @@ class PackageInfoModalController {
         this.cancel();
         const controller = new AbortController();
         this.abortController = controller;
-        const metadata = await services_1.changelogFetcher.fetchPackageMetadata(state.name, state.latestVersion, controller.signal);
+        const metadata = await changelog_1.changelogFetcher.fetchPackageMetadata(state.name, state.latestVersion, controller.signal);
         if (!metadata) {
             return null;
         }
@@ -142,7 +142,7 @@ class PackageInfoModalController {
         state.releaseNotesLoadingVersion = version;
         onLoaded(); // Re-render to show loading indicator
         try {
-            const notes = await services_1.changelogFetcher.fetchReleaseNotesForVersion(state.name, version, this.abortController?.signal);
+            const notes = await changelog_1.changelogFetcher.fetchReleaseNotesForVersion(state.name, version, this.abortController?.signal);
             loadedNotes.set(version, notes);
             return true;
         }

package/dist/features/interactive/index.js

@@ -0,0 +1,21 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.renderReadmeKeyTable = exports.PackageInfoModalController = exports.selectionKey = exports.createUpgradeChoices = exports.createPendingSelectionStates = exports.createSelectionStates = exports.runInteractiveSession = exports.StateManager = exports.UIRenderer = exports.ConfirmationInputHandler = exports.InputHandler = void 0;
+var input_handler_1 = require("./input-handler");
+Object.defineProperty(exports, "InputHandler", { enumerable: true, get: function () { return input_handler_1.InputHandler; } });
+Object.defineProperty(exports, "ConfirmationInputHandler", { enumerable: true, get: function () { return input_handler_1.ConfirmationInputHandler; } });
+var renderer_1 = require("./renderer");
+Object.defineProperty(exports, "UIRenderer", { enumerable: true, get: function () { return renderer_1.UIRenderer; } });
+var state_1 = require("./state");
+Object.defineProperty(exports, "StateManager", { enumerable: true, get: function () { return state_1.StateManager; } });
+var session_1 = require("./session");
+Object.defineProperty(exports, "runInteractiveSession", { enumerable: true, get: function () { return session_1.runInteractiveSession; } });
+Object.defineProperty(exports, "createSelectionStates", { enumerable: true, get: function () { return session_1.createSelectionStates; } });
+Object.defineProperty(exports, "createPendingSelectionStates", { enumerable: true, get: function () { return session_1.createPendingSelectionStates; } });
+Object.defineProperty(exports, "createUpgradeChoices", { enumerable: true, get: function () { return session_1.createUpgradeChoices; } });
+Object.defineProperty(exports, "selectionKey", { enumerable: true, get: function () { return session_1.selectionKey; } });
+var controllers_1 = require("./controllers");
+Object.defineProperty(exports, "PackageInfoModalController", { enumerable: true, get: function () { return controllers_1.PackageInfoModalController; } });
+var keymap_1 = require("./keymap");
+Object.defineProperty(exports, "renderReadmeKeyTable", { enumerable: true, get: function () { return keymap_1.renderReadmeKeyTable; } });
+//# sourceMappingURL=index.js.map

package/dist/{ui → features/interactive}/input-handler.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ConfirmationInputHandler = exports.InputHandler = void 0;
-const utils_1 = require("./utils");
+const terminal_1 = require("../../shared/terminal");
 const keymap_1 = require("./keymap");
 class InputHandler {
     stateManager;
@@ -235,7 +235,7 @@ class InputHandler {
         this.onAction({ type: 'resize', height });
     }
     cleanup() {
-        utils_1.CursorUtils.cleanup();
+        terminal_1.CursorUtils.cleanup();
     }
 }
 exports.InputHandler = InputHandler;
@@ -275,7 +275,7 @@ class ConfirmationInputHandler {
         }
     }
     cleanup() {
-        utils_1.CursorUtils.cleanup();
+        terminal_1.CursorUtils.cleanup();
     }
 }
 exports.ConfirmationInputHandler = ConfirmationInputHandler;

package/dist/{ui → features/interactive}/modal/layout.js

@@ -11,12 +11,12 @@ exports.getModalSectionRowCount = getModalSectionRowCount;
 exports.getModalFrameHeight = getModalFrameHeight;
 exports.renderModalFrame = renderModalFrame;
 const chalk_1 = __importDefault(require("chalk"));
-const utils_1 = require("../utils");
+const terminal_1 = require("../../../shared/terminal");
 function getModalWidth(terminalWidth, minWidth, maxWidth) {
     return Math.min(Math.max(minWidth, terminalWidth - 6), maxWidth);
 }
 function renderModalRow(padding, modalWidth, text) {
-    const rowLength = (0, utils_1.getVisualLength)(text);
+    const rowLength = (0, terminal_1.getVisualLength)(text);
     const rowPadding = Math.max(0, modalWidth - 4 - rowLength);
     return (' '.repeat(padding) +
         chalk_1.default.gray('│') +

package/dist/{ui/renderer/package-list.js → features/interactive/modal/package-info-sections/index.js}

@@ -14,6 +14,6 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-__exportStar(require("./package-list/rows"), exports);
-__exportStar(require("./package-list/interface"), exports);
-//# sourceMappingURL=package-list.js.map
+__exportStar(require("./sections"), exports);
+__exportStar(require("./release-notes"), exports);
+//# sourceMappingURL=index.js.map

package/dist/{ui → features/interactive}/modal/package-info-sections/sections.js

@@ -8,9 +8,9 @@ exports.buildPackageInfoSections = buildPackageInfoSections;
 const node_path_1 = __importDefault(require("node:path"));
 const chalk_1 = __importDefault(require("chalk"));
 const themes_colors_1 = require("../../themes-colors");
-const vulnerability_1 = require("../../presenters/vulnerability");
-const utils_1 = require("../../utils");
-const utils_2 = require("../../../utils");
+const audit_1 = require("../../../audit");
+const terminal_1 = require("../../../../shared/terminal");
+const engines_1 = require("../../../../shared/engines");
 const release_notes_1 = require("./release-notes");
 function formatNumber(num) {
     if (!num)
@@ -22,10 +22,15 @@ function formatNumber(num) {
     return num.toString();
 }
 function getUsedByPaths(state) {
+    // Catalog entries live in pnpm-workspace.yaml; the interesting "used by"
+    // answer is the member packages that reference the catalog entry.
+    if (state.catalog && state.catalogReferencedBy?.length) {
+        return state.catalogReferencedBy;
+    }
     return state.packageJsonPaths ?? [state.packageJsonPath];
 }
 function buildTabBarSuffix(activeTab, usedByCount) {
-    const styleFor = (tab) => tab === activeTab ? chalk_1.default.bold.underline : chalk_1.default.gray;
+    const styleFor = (tab) => (tab === activeTab ? chalk_1.default.bold.underline : chalk_1.default.gray);
     const usedByLabel = `Used by${usedByCount > 0 ? ` (${usedByCount})` : ''}`;
     return ('  ' +
         chalk_1.default.gray('[ ') +
@@ -40,14 +45,16 @@ function buildUsedBySections(state, modalWidth) {
     const contentWidth = Math.max(10, modalWidth - 4);
     const formatRelative = (absolutePath) => {
         const display = node_path_1.default.relative(cwd, absolutePath) || absolutePath;
-        return (0, utils_1.truncatePlainText)(display, contentWidth);
+        return (0, terminal_1.truncatePlainText)(display, contentWidth);
     };
-    return [
+    const sections = [
         {
             key: 'used-by-summary',
             rows: [
                 chalk_1.default.bold(`${paths.length} package.json file${paths.length === 1 ? '' : 's'} depend on ${state.name}`),
-                chalk_1.default.gray(`Type: ${state.type}`),
+                chalk_1.default.gray(state.catalog
+                    ? `Type: ${state.type} • via catalog:${state.catalog}`
+                    : `Type: ${state.type}`),
             ],
             required: true,
             behavior: 'pinned',
@@ -58,6 +65,25 @@ function buildUsedBySections(state, modalWidth) {
             behavior: 'body',
         },
     ];
+    // For catalog entries, also show what else the catalog pins — the whole
+    // point of a catalog is that these versions move together.
+    if (state.catalog && state.catalogEntries?.length) {
+        sections.push({
+            key: 'catalog-contents',
+            rows: [
+                chalk_1.default.bold(`Catalog "${state.catalog}"`) + chalk_1.default.gray(' — pnpm-workspace.yaml'),
+                ...state.catalogEntries.map((entry) => {
+                    const isCurrent = entry.name === state.name;
+                    const label = (0, terminal_1.truncatePlainText)(`${entry.name}: ${entry.range}`, contentWidth - 2);
+                    return isCurrent
+                        ? `${(0, themes_colors_1.getThemeColor)('packageName')('▸')} ${chalk_1.default.bold(label)}`
+                        : `${chalk_1.default.gray('•')} ${label}`;
+                }),
+            ],
+            behavior: 'body',
+        });
+    }
+    return sections;
 }
 function buildPackageInfoSections(state, modalWidth, activeTab) {
     const title = chalk_1.default.bold('Package: ') +
@@ -82,6 +108,11 @@ function buildPackageInfoSections(state, modalWidth, activeTab) {
         key: 'meta',
         rows: [
             `Current: ${currentVersion}  Target: ${targetVersion}`,
+            ...(state.catalog
+                ? [
+                    (0, themes_colors_1.getThemeColor)('textSecondary')(`Catalog: ${state.catalog} — range lives in pnpm-workspace.yaml`),
+                ]
+                : []),
             ...(state.weeklyDownloads !== undefined
                 ? [(0, themes_colors_1.getThemeColor)('primary')(`Downloads/week: ${formatNumber(state.weeklyDownloads)}`)]
                 : []),
@@ -95,13 +126,13 @@ function buildPackageInfoSections(state, modalWidth, activeTab) {
         // Wrap (don't truncate) so a deprecation URL stays whole and clickable —
         // truncation with "..." produces a dead link. `wrapPlainText` breaks on
         // spaces only, so the URL keeps its own intact line. No emoji marker:
-        // `getVisualLength` scores glyphs like ⚠ as width 2 while many terminals
-        // render them as width 1, which throws off the modal's border alignment.
-        for (const line of (0, utils_1.wrapPlainText)(`Deprecated: ${state.deprecated}`, warningContentWidth)) {
+        // terminal fonts disagree on whether such glyphs render one or two columns
+        // wide, which throws off the modal's border alignment.
+        for (const line of (0, terminal_1.wrapPlainText)(`Deprecated: ${state.deprecated}`, warningContentWidth)) {
             warningRows.push((0, themes_colors_1.getThemeColor)('warning')(line));
         }
     }
-    const engineWarning = (0, utils_2.checkNodeEngineCompatibility)(state.enginesNode);
+    const engineWarning = (0, engines_1.checkNodeEngineCompatibility)(state.enginesNode);
     if (engineWarning) {
         warningRows.push((0, themes_colors_1.getThemeColor)('warning')(`Hold: ${engineWarning}`));
     }
@@ -116,7 +147,7 @@ function buildPackageInfoSections(state, modalWidth, activeTab) {
         sections.push({
             key: 'homepage',
             rows: [
-                `Homepage: ${chalk_1.default.underline((0, themes_colors_1.getThemeColor)('primary')((0, utils_1.truncatePlainText)(state.homepage, modalWidth - 14)))}`,
+                `Homepage: ${chalk_1.default.underline((0, themes_colors_1.getThemeColor)('primary')((0, terminal_1.truncatePlainText)(state.homepage, modalWidth - 14)))}`,
             ],
             behavior: 'pinned',
         });
@@ -124,30 +155,30 @@ function buildPackageInfoSections(state, modalWidth, activeTab) {
     if (state.description) {
         sections.push({
             key: 'description',
-            rows: (0, utils_1.wrapPlainText)(state.description, modalWidth - 4)
+            rows: (0, terminal_1.wrapPlainText)(state.description, modalWidth - 4)
                 .slice(0, 4)
                 .map((line, index, rows) => index === rows.length - 1 && rows.length === 4
-                ? (0, utils_1.truncatePlainText)(line, modalWidth - 4)
+                ? (0, terminal_1.truncatePlainText)(line, modalWidth - 4)
                 : line),
             behavior: 'pinned',
         });
     }
     if (state.vulnerability && state.vulnerability.count > 0) {
-        const representative = (0, vulnerability_1.selectRepresentativeAdvisory)(state.vulnerability);
-        const severityColor = (0, vulnerability_1.getVulnerabilitySeverityColor)(state.vulnerability.highestSeverity);
+        const representative = (0, audit_1.selectRepresentativeAdvisory)(state.vulnerability);
+        const severityColor = (0, audit_1.getVulnerabilitySeverityColor)(state.vulnerability.highestSeverity);
         const vulnerabilityRows = [
             chalk_1.default.red.bold(`${state.vulnerability.count} known vulnerabilit${state.vulnerability.count === 1 ? 'y' : 'ies'} (${severityColor(state.vulnerability.highestSeverity.toUpperCase())})`),
         ];
         if (representative) {
             const severityLabel = ` ${severityColor(`[${representative.severity.toUpperCase()}]`)} `;
-            const availableTitleWidth = Math.max(0, modalWidth - 4 - (0, utils_1.getVisualLength)(severityLabel));
-            vulnerabilityRows.push(`${severityLabel}${(0, utils_1.truncatePlainText)(representative.title, availableTitleWidth)}`);
+            const availableTitleWidth = Math.max(0, modalWidth - 4 - (0, terminal_1.getVisualLength)(severityLabel));
+            vulnerabilityRows.push(`${severityLabel}${(0, terminal_1.truncatePlainText)(representative.title, availableTitleWidth)}`);
         }
         const detailsUrl = state.vulnerability.detailsUrl || representative?.url;
         if (detailsUrl) {
-            const linkPrefix = ` ${(0, vulnerability_1.getVulnerabilityLinkLabel)(detailsUrl)} `;
-            const availableLinkWidth = Math.max(0, modalWidth - 4 - (0, utils_1.getVisualLength)(linkPrefix));
-            vulnerabilityRows.push(`${linkPrefix}${chalk_1.default.underline((0, themes_colors_1.getThemeColor)('primary')((0, utils_1.truncatePlainText)(detailsUrl, availableLinkWidth)))}`);
+            const linkPrefix = ` ${(0, audit_1.getVulnerabilityLinkLabel)(detailsUrl)} `;
+            const availableLinkWidth = Math.max(0, modalWidth - 4 - (0, terminal_1.getVisualLength)(linkPrefix));
+            vulnerabilityRows.push(`${linkPrefix}${chalk_1.default.underline((0, themes_colors_1.getThemeColor)('primary')((0, terminal_1.truncatePlainText)(detailsUrl, availableLinkWidth)))}`);
         }
         if (state.vulnerability.count > 1) {
             vulnerabilityRows.push(chalk_1.default.gray(` ... and ${state.vulnerability.count - 1} more`));
@@ -163,7 +194,7 @@ function buildPackageInfoSections(state, modalWidth, activeTab) {
         sections.push({
             key: 'changelog',
             rows: [
-                `Changelog: ${chalk_1.default.underline((0, themes_colors_1.getThemeColor)('primary')((0, utils_1.truncatePlainText)(state.repository, modalWidth - 15)))}`,
+                `Changelog: ${chalk_1.default.underline((0, themes_colors_1.getThemeColor)('primary')((0, terminal_1.truncatePlainText)(state.repository, modalWidth - 15)))}`,
             ],
             behavior: 'pinned',
         });

package/dist/{ui → features/interactive}/modal/package-info-sections/text.js

@@ -8,7 +8,7 @@ exports.linkifyRepositoryReferences = linkifyRepositoryReferences;
 exports.linkifyMarkdownText = linkifyMarkdownText;
 exports.pushWrappedLines = pushWrappedLines;
 exports.isLowSignalTrailerLine = isLowSignalTrailerLine;
-const utils_1 = require("../../utils");
+const terminal_1 = require("../../../../shared/terminal");
 function formatTerminalLink(label, url) {
     return `]8;;${url}${label}]8;;`;
 }
@@ -48,16 +48,16 @@ function linkifyMarkdownText(text, repositoryUrl) {
     return linkifyRepositoryReferences(linkifyContributorMentions(text), repositoryUrl);
 }
 function pushWrappedLines(lines, text, width, firstPrefix, restPrefix = firstPrefix, style) {
-    const firstWidth = Math.max(1, width - (0, utils_1.getVisualLength)(firstPrefix));
-    const restWidth = Math.max(1, width - (0, utils_1.getVisualLength)(restPrefix));
-    const segments = (0, utils_1.wrapPlainText)(text, firstWidth);
+    const firstWidth = Math.max(1, width - (0, terminal_1.getVisualLength)(firstPrefix));
+    const restWidth = Math.max(1, width - (0, terminal_1.getVisualLength)(restPrefix));
+    const segments = (0, terminal_1.wrapPlainText)(text, firstWidth);
     if (segments.length === 0) {
         lines.push(firstPrefix.trimEnd());
         return;
     }
     lines.push(firstPrefix + (style ? style(segments[0]) : segments[0]));
     for (let i = 1; i < segments.length; i++) {
-        const wrappedSegments = (0, utils_1.wrapPlainText)(segments[i], restWidth);
+        const wrappedSegments = (0, terminal_1.wrapPlainText)(segments[i], restWidth);
         for (const segment of wrappedSegments) {
             lines.push(restPrefix + (style ? style(segment) : segment));
         }

package/dist/{ui → features/interactive}/modal/package-info.js

@@ -51,14 +51,22 @@ function renderPackageInfoModal(state, terminalWidth = 80, terminalHeight = 24,
     }
     const fixedModalHeight = maxHeight;
     const padding = Math.floor((terminalWidth - modalWidth) / 2);
-    const pinnedSections = allSections.filter((section) => (section.behavior ?? 'pinned') === 'pinned');
-    const bodySections = allSections.filter((section) => (section.behavior ?? 'pinned') !== 'pinned');
+    // Every section built by buildPackageInfoSections declares a behavior; the
+    // '?? pinned' fallbacks are type-level safety nets for hand-built sections.
+    const pinnedSections = allSections.filter(
+    /* v8 ignore next */
+    (section) => (section.behavior ?? 'pinned') === 'pinned');
+    const bodySections = allSections.filter(
+    /* v8 ignore next */
+    (section) => (section.behavior ?? 'pinned') !== 'pinned');
     const minBodyRows = 3;
-    const reservedBodyRows = minBodyRows + (bodySections.length > 0 ? 1 : 0);
+    // Reaching this point requires a 'body' section (hasScrollableBody above),
+    // so bodySections is never empty here; the ':0' arms are unreachable.
+    const reservedBodyRows = minBodyRows + (bodySections.length > 0 ? 1 : /* v8 ignore next */ 0);
     const maxPinnedHeight = Math.max(6, fixedModalHeight - reservedBodyRows);
     const fittedPinned = (0, layout_1.fitModalSections)(pinnedSections, maxPinnedHeight, trimOrder);
     const pinnedRowCount = (0, layout_1.getModalSectionRowCount)(fittedPinned);
-    const availableForBody = Math.max(minBodyRows, fixedModalHeight - 2 - pinnedRowCount - (bodySections.length > 0 ? 1 : 0));
+    const availableForBody = Math.max(minBodyRows, fixedModalHeight - 2 - pinnedRowCount - (bodySections.length > 0 ? 1 : /* v8 ignore next */ 0));
     const bodyRows = [];
     bodySections.forEach((section, index) => {
         if (index > 0) {
@@ -91,11 +99,17 @@ function renderPackageInfoModal(state, terminalWidth = 80, terminalHeight = 24,
             ? clampedOffset < maxScroll
                 ? chalk_1.default.gray(`Lines ${clampedOffset + 1}-${Math.min(clampedOffset + visibleBodyRows, totalScrollableRows)} of ${totalScrollableRows}`)
                 : chalk_1.default.gray('End of release notes')
-            : canGoNewer || canGoOlder
-                ? chalk_1.default.gray([canGoNewer ? '← newer version' : null, canGoOlder ? '→ older version' : null]
-                    .filter((hint) => Boolean(hint))
-                    .join('  ·  '))
-                : null;
+            : // A truthy hint footer shrinks visibleBodyRows, which forces
+                // maxScroll ≥ 1 whenever the scroll path is entered — so this arm can
+                // only resolve to the hints when maxScroll is 0, which cannot happen.
+                // Kept as a safety net for future layout changes.
+                /* v8 ignore start */
+                canGoNewer || canGoOlder
+                    ? chalk_1.default.gray([canGoNewer ? '← newer version' : null, canGoOlder ? '→ older version' : null]
+                        .filter((hint) => Boolean(hint))
+                        .join('  ·  '))
+                    : null;
+    /* v8 ignore stop */
     const visibleSlice = bodyRows.slice(clampedOffset, clampedOffset + visibleBodyRows);
     const lines = [];
     const topPadding = Math.max(0, Math.floor((terminalHeight - fixedModalHeight) / 2));
@@ -111,9 +125,9 @@ function renderPackageInfoModal(state, terminalWidth = 80, terminalHeight = 24,
             lines.push((0, layout_1.renderModalRow)(padding, modalWidth, row));
         }
     });
-    if (bodySections.length > 0) {
-        lines.push((0, layout_1.renderModalSeparator)(padding, modalWidth));
-    }
+    // bodySections is never empty on the scroll path (see above), so a
+    // separator between the pinned rows and the body is always drawn.
+    lines.push((0, layout_1.renderModalSeparator)(padding, modalWidth));
     let renderedScrollRows = 0;
     for (const entry of visibleSlice) {
         if (entry.row === '__SEPARATOR__') {
@@ -125,14 +139,18 @@ function renderPackageInfoModal(state, terminalWidth = 80, terminalHeight = 24,
         renderedScrollRows++;
     }
     const usedContentRows = pinnedRowCount +
-        (bodySections.length > 0 ? 1 : 0) +
+        (bodySections.length > 0 ? 1 : /* v8 ignore next */ 0) +
         renderedScrollRows +
         (resolvedFooterStatus ? 1 : 0);
     const totalContentSlots = fixedModalHeight - 2;
+    // The scroll path is only entered when content overflows the frame, so the
+    // window is always full and there are never filler rows to add.
+    /* v8 ignore start */
     const emptyRows = Math.max(0, totalContentSlots - usedContentRows);
     for (let i = 0; i < emptyRows; i++) {
         lines.push((0, layout_1.renderModalRow)(padding, modalWidth, ''));
     }
+    /* v8 ignore stop */
     if (resolvedFooterStatus) {
         lines.push((0, layout_1.renderModalRow)(padding, modalWidth, resolvedFooterStatus));
     }

package/dist/features/interactive/modal/types.js

@@ -0,0 +1,3 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+//# sourceMappingURL=types.js.map

package/dist/{ui → features/interactive}/presenters/health.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getHealthBadge = getHealthBadge;
-const utils_1 = require("../../utils");
+const engines_1 = require("../../../shared/engines");
 const themes_colors_1 = require("../themes-colors");
 /**
  * A compact badge flagging a package's health in the list:
@@ -16,7 +16,7 @@ function getHealthBadge(state) {
     if (state.deprecated) {
         return (0, themes_colors_1.getThemeColor)('warning')('[DEPR]');
     }
-    if ((0, utils_1.checkNodeEngineCompatibility)(state.enginesNode)) {
+    if ((0, engines_1.checkNodeEngineCompatibility)(state.enginesNode)) {
         return (0, themes_colors_1.getThemeColor)('warning')('[ENG]');
     }
     return '';

package/dist/{ui → features/interactive}/renderer/index.js

@@ -73,5 +73,4 @@ exports.UIRenderer = UIRenderer;
 // Re-export all functions for direct use if needed
 __exportStar(require("./package-list"), exports);
 __exportStar(require("./confirmation"), exports);
-__exportStar(require("../modal"), exports);
 //# sourceMappingURL=index.js.map

package/dist/{types.js → features/interactive/renderer/package-list/index.js}

@@ -14,7 +14,6 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-__exportStar(require("./types/domain"), exports);
-__exportStar(require("./types/ui"), exports);
-__exportStar(require("./types/streaming"), exports);
-//# sourceMappingURL=types.js.map
+__exportStar(require("./rows"), exports);
+__exportStar(require("./interface"), exports);
+//# sourceMappingURL=index.js.map