Simplify package info fix: use rstrip and CSS wrap instead of env var plumbing

archinstall/lib/models/packages.py

@@ -1,5 +1,3 @@
-import os
-import textwrap
 from dataclasses import dataclass, field
 from enum import Enum
 from functools import cached_property
@@ -134,34 +132,14 @@ class AvailablePackage(BaseModel):
 	def longest_key(self) -> int:
 		return max(len(key) for key in self.model_dump().keys())
 
-	# return all package info line by line
 	def info(self) -> str:
-		# Preview pane occupies roughly half the terminal width when shown
-		# alongside the option list. Wrap each value to fit so long fields
-		# (Description, Optional Deps, etc.) do not produce horizontal scroll.
-		try:
-			cols = os.get_terminal_size().columns
-		except OSError:
-			cols = 80
-		preview_width = max(40, cols // 2 - 5)
-		indent = ' ' * (self.longest_key + 3)
-
-		lines: list[str] = []
+		output = ''
 		for key, value in self.model_dump().items():
-			key_label = key.replace('_', ' ').capitalize().ljust(self.longest_key)
-			prefix = f'{key_label} : '
-			lines.append(
-				textwrap.fill(
-					str(value),
-					width=preview_width,
-					initial_indent=prefix,
-					subsequent_indent=indent,
-					break_long_words=False,
-					break_on_hyphens=False,
-				),
-			)
+			key = key.replace('_', ' ').capitalize()
+			key = key.ljust(self.longest_key)
+			output += f'{key} : {value}\n'
 
-		return '\n'.join(lines) + '\n'
+		return output
 
 	@cached_property
 	def get_depends_on(self) -> list[str]:

archinstall/lib/packages/packages.py

@@ -9,18 +9,12 @@ from archinstall.lib.translationhandler import tr
 from archinstall.tui.ui.menu_item import MenuItem, MenuItemGroup
 from archinstall.tui.ui.result import ResultType
 
-# Force pacman to emit untruncated single-line field values. With the default
-# pty width of 80 columns, pacman wraps long fields like Description across
-# multiple lines, and our line-based parser ends up dropping the continuation
-# lines after they get pre-stripped of their leading whitespace. See #3580.
-_WIDE_PACMAN_ENV = {'COLUMNS': '500'}
-
 
 def installed_package(package: str) -> LocalPackage | None:
 	try:
 		package_info = []
 		for line in Pacman.run(f'-Q --info {package}'):
-			package_info.append(line.decode().strip())
+			package_info.append(line.decode().rstrip())
 
 		return _parse_package_output(package_info, LocalPackage)
 	except SysCallError:
@@ -58,8 +52,8 @@ def package_group_info(package: str) -> PackageGroup | None:
 def available_package(package: str) -> AvailablePackage | None:
 	try:
 		package_info: list[str] = []
-		for line in Pacman.run(f'-S --info {package}', environment_vars=_WIDE_PACMAN_ENV):
-			package_info.append(line.decode().strip())
+		for line in Pacman.run(f'-S --info {package}'):
+			package_info.append(line.decode().rstrip())
 
 		return _parse_package_output(package_info, AvailablePackage)
 	except SysCallError:
@@ -84,8 +78,8 @@ def list_available_packages(
 	except Exception as e:
 		debug(f'Failed to sync Arch Linux package database: {e}')
 
-	for line in Pacman.run('-S --info', environment_vars=_WIDE_PACMAN_ENV):
-		dec_line = line.decode().strip()
+	for line in Pacman.run('-S --info'):
+		dec_line = line.decode().rstrip()
 		current_package.append(dec_line)
 
 		if dec_line.startswith('Validated'):

archinstall/lib/pacman/pacman.py

@@ -18,11 +18,7 @@ class Pacman:
 		self.target = target
 
 	@staticmethod
-	def run(
-		args: str,
-		default_cmd: str = 'pacman',
-		environment_vars: dict[str, str] | None = None,
-	) -> SysCommand:
+	def run(args: str, default_cmd: str = 'pacman') -> SysCommand:
 		"""
 		A centralized function to call `pacman` from.
 		It also protects us from colliding with other running pacman sessions (if used locally).
@@ -41,7 +37,7 @@ class Pacman:
 				error(tr('Pre-existing pacman lock never exited. Please clean up any existing pacman sessions before using archinstall.'))
 				sys.exit(1)
 
-		return SysCommand(f'{default_cmd} {args}', environment_vars=environment_vars)
+		return SysCommand(f'{default_cmd} {args}')
 
 	def ask(self, error_message: str, bail_message: str, func: Callable, *args, **kwargs) -> None:  # type: ignore[no-untyped-def, type-arg]
 		while True:

archinstall/tui/ui/components.py

@@ -433,6 +433,11 @@ class SelectListScreen(BaseScreen[ValueT]):
 		color: white;
 		text-style: bold;
 	}
+
+	#preview_content {
+		width: 100%;
+		height: auto;
+	}
 	"""
 
 	def __init__(