fixed stuffs

Author: bajrangCoder

src/components/settingsPage.js

@@ -27,7 +27,7 @@ import searchBar from "./searchbar";
  * @property {string} [pageClassName] - Extra classes to apply to the page element
  * @property {string} [listClassName] - Extra classes to apply to the list element
  * @property {string} [defaultSearchGroup] - Default search result group label for this page
- * @property {boolean} [infoAsDescription] - Prefer item.info as subtitle instead of item.value
+ * @property {boolean} [infoAsDescription] - Override subtitle behavior; defaults to true when valueInTail is enabled
  * @property {boolean} [valueInTail] - Render item.value as a trailing control/value instead of subtitle
  * @property {boolean} [groupByDefault] - Wrap uncategorized settings in a grouped section shell
  * @property {"top"|"bottom"} [notePosition] - Render note before or after the settings list
@@ -237,6 +237,8 @@ function listItems($list, items, callback, options = {}) {
 	const renderedItems = [];
 	const $searchItems = [];
 	const $content = [];
+	const useInfoAsDescription =
+		options.infoAsDescription ?? Boolean(options.valueInTail);
 	/** @type {HTMLElement | null} */
 	let $currentSectionCard = null;
 	let currentCategory = null;
@@ -275,9 +277,9 @@ function listItems($list, items, callback, options = {}) {
 
 		let $checkbox, $valueText;
 		let $trailingValueText;
-		const subtitle = getSubtitleText(item, options);
+		const subtitle = getSubtitleText(item, useInfoAsDescription);
 		const showInfoAsSubtitle =
-			options.infoAsDescription || (item.value === undefined && item.info);
+			useInfoAsDescription || (item.value === undefined && item.info);
 		const searchGroup =
 			item.searchGroup || item.category || options.defaultSearchGroup;
 		const hasSubtitle =
@@ -476,8 +478,8 @@ function listItems($list, items, callback, options = {}) {
 	}
 }
 
-function getSubtitleText(item, options) {
-	if (options.infoAsDescription) {
+function getSubtitleText(item, useInfoAsDescription) {
+	if (useInfoAsDescription) {
 		return item.info;
 	}
 

src/components/settingsPage.scss

@@ -258,6 +258,7 @@ wc-page.detail-settings-page {
 			min-width: 0;
 			overflow: visible;
 			gap: 0.14rem;
+			padding-right: 0.35rem;
 
 			> .text {
 				display: flex;
@@ -290,14 +291,15 @@ wc-page.detail-settings-page {
 			flex-shrink: 0;
 			min-height: 1.65rem;
 			gap: 0.32rem;
-			margin-left: auto;
+			margin-left: 0.9rem;
 			align-self: center;
 		}
 	}
 
 	.detail-settings-list > .list-item.has-subtitle > .container,
 	.settings-section-card > .list-item.has-subtitle > .container {
 		padding-top: 0.12rem;
+		padding-right: 0.55rem;
 	}
 
 	.detail-settings-list > .list-item.compact > .container,

src/lib/acode.js

@@ -675,7 +675,6 @@ export default class Acode {
 				preserveOrder: true,
 				pageClassName: "detail-settings-page",
 				listClassName: "detail-settings-list",
-				infoAsDescription: true,
 				valueInTail: true,
 				groupByDefault: true,
 			},

src/settings/lspServerDetail.js

@@ -115,6 +115,48 @@ function formatInstallStatus(result) {
 	}
 }
 
+function formatStartupTimeoutValue(timeout) {
+	return typeof timeout === "number" ? `${timeout} ms` : "Default";
+}
+
+function sanitizeInstallMessage(message) {
+	const lines = String(message || "")
+		.split("\n")
+		.map((line) => line.trim())
+		.filter(Boolean)
+		.filter(
+			(line) =>
+				!/^proot warning:/i.test(line) &&
+				!line.includes(`"/proc/self/fd/0"`) &&
+				!line.includes(`"/proc/self/fd/1"`) &&
+				!line.includes(`"/proc/self/fd/2"`),
+		);
+
+	return lines.join(" ");
+}
+
+function formatInstallInfo(result) {
+	const cleanedMessage = sanitizeInstallMessage(result?.message);
+
+	switch (result?.status) {
+		case "present":
+			return result.version
+				? `Version ${result.version} is available.`
+				: "Language server is installed and ready.";
+		case "missing":
+			return "Language server is not installed in the terminal environment.";
+		case "failed":
+			return (
+				cleanedMessage || "Acode could not verify the installation status."
+			);
+		default:
+			return (
+				cleanedMessage ||
+				"Installation status could not be checked automatically."
+			);
+	}
+}
+
 function formatValue(value) {
 	if (value === undefined || value === null || value === "") return "";
 	let text = String(value);
@@ -154,9 +196,16 @@ function updateItemDisplay($list, itemsByKey, key, value, extras = {}) {
 	const $item = $list?.querySelector?.(`[data-key="${key}"]`);
 	if (!$item) return;
 
-	const $value = $item.querySelector(".value");
-	if ($value) {
-		$value.textContent = formatValue(item.value);
+	const $subtitle = $item.querySelector(".value");
+	if ($subtitle) {
+		$subtitle.textContent = $subtitle.classList.contains("setting-info")
+			? String(item.info || "")
+			: formatValue(item.value);
+	}
+
+	const $trailingValue = $item.querySelector(".setting-trailing-value");
+	if ($trailingValue) {
+		$trailingValue.textContent = formatValue(item.value);
 	}
 
 	const $checkbox = $item.querySelector(".input-checkbox");
@@ -200,38 +249,44 @@ function createItems(snapshot) {
 			text: "Enabled",
 			checkbox: snapshot.merged.enabled !== false,
 			info: "Enable or disable this language server",
+			category: "General",
 		},
 		{
 			key: "install_status",
 			text: "Installed",
 			value: formatInstallStatus(snapshot.installResult),
-			info:
-				snapshot.installResult.message ||
-				"Current installation state for this language server",
+			info: formatInstallInfo(snapshot.installResult),
+			category: "Installation",
+			chevron: true,
 		},
 		{
 			key: "install_server",
 			text: "Install / Repair",
 			info: "Install or repair this language server",
+			category: "Installation",
+			chevron: true,
 		},
 		{
 			key: "update_server",
 			text: "Update Server",
 			info: "Update this language server if an update flow exists",
+			category: "Installation",
+			chevron: true,
 		},
 		{
 			key: "uninstall_server",
 			text: "Uninstall Server",
 			info: "Remove installed packages or binaries for this server",
+			category: "Installation",
+			chevron: true,
 		},
 		{
 			key: "startup_timeout",
 			text: "Startup Timeout",
-			value:
-				typeof snapshot.merged.startupTimeout === "number"
-					? `${snapshot.merged.startupTimeout} ms`
-					: "Default (5000 ms)",
+			value: formatStartupTimeoutValue(snapshot.merged.startupTimeout),
 			info: "Configure how long Acode waits for the server to start",
+			category: "Advanced",
+			chevron: true,
 		},
 		{
 			key: "edit_init_options",
@@ -240,11 +295,15 @@ function createItems(snapshot) {
 				? "Configured"
 				: "Empty",
 			info: "Edit initialization options as JSON",
+			category: "Advanced",
+			chevron: true,
 		},
 		{
 			key: "view_init_options",
 			text: "View Initialization Options",
 			info: "View the effective initialization options as JSON",
+			category: "Advanced",
+			chevron: true,
 		},
 	];
 
@@ -254,6 +313,7 @@ function createItems(snapshot) {
 			text,
 			checkbox: snapshot.builtinExts[extKey] !== false,
 			info,
+			category: "Features",
 		});
 	});
 
@@ -275,9 +335,7 @@ async function refreshVisibleState($list, itemsByKey, serverId) {
 		"install_status",
 		formatInstallStatus(snapshot.installResult),
 		{
-			info:
-				snapshot.installResult.message ||
-				"Current installation state for this language server",
+			info: formatInstallInfo(snapshot.installResult),
 		},
 	);
 	updateItemDisplay($list, itemsByKey, "install_server", "");
@@ -287,9 +345,7 @@ async function refreshVisibleState($list, itemsByKey, serverId) {
 		$list,
 		itemsByKey,
 		"startup_timeout",
-		typeof snapshot.merged.startupTimeout === "number"
-			? `${snapshot.merged.startupTimeout} ms`
-			: "Default (5000 ms)",
+		formatStartupTimeoutValue(snapshot.merged.startupTimeout),
 	);
 	updateItemDisplay(
 		$list,
@@ -379,6 +435,9 @@ export default function lspServerDetail(serverId) {
 		undefined,
 		{
 			preserveOrder: true,
+			pageClassName: "detail-settings-page",
+			listClassName: "detail-settings-list",
+			valueInTail: true,
 		},
 	);
 
@@ -414,7 +473,8 @@ export default function lspServerDetail(serverId) {
 				const result = await checkServerInstallation(snapshot.merged);
 				const lines = [
 					`Status: ${formatInstallStatus(result)}`,
-					result.message ? `Details: ${result.message}` : null,
+					result.version ? `Version: ${result.version}` : null,
+					`Details: ${formatInstallInfo(result)}`,
 				].filter(Boolean);
 				alert("Installation Status", lines.join("<br>"));
 				break;

src/settings/lspSettings.js

@@ -124,6 +124,7 @@ export default function lspSettings() {
 			text: "Add Custom Server",
 			info: "Register a user-defined language server with install, update, and launch commands",
 			index: 0,
+			chevron: true,
 		},
 	];
 
@@ -140,6 +141,7 @@ export default function lspSettings() {
 			key: `server:${server.id}`,
 			text: server.label,
 			info: languagesList || undefined,
+			chevron: true,
 		});
 	}
 
@@ -149,6 +151,9 @@ export default function lspSettings() {
 
 	return settingsPage(title, items, callback, undefined, {
 		preserveOrder: true,
+		pageClassName: "detail-settings-page",
+		listClassName: "detail-settings-list",
+		groupByDefault: true,
 	});
 
 	async function callback(key) {