fix: resolve winget background spinner parsing bug

Author: stkossman

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "winup-cli",
-	"version": "1.0.0",
+	"version": "1.0.1",
 	"description": "Minimalistic winget upgrade manager",
 	"main": "dist/cli/index.js",
 	"type": "module",

src/core/fetcher.ts

@@ -7,7 +7,9 @@ import { getUpgradeList } from './runner.js'
 export function fetchUpgrades(includeUnknown: boolean): PackageEntry[] {
 	try {
 		const rawText = getUpgradeList(includeUnknown)
+
 		const rawObjects = parseWingetOutput(rawText)
+
 		return normalizePackages(rawObjects)
 	} catch (error: any) {
 		Logger.error(`Failed to fetch updates: ${error.message}`)

src/core/normalizer.ts

@@ -6,14 +6,14 @@ export function normalizePackages(
 	const packageMap = new Map<string, PackageEntry>()
 
 	for (const raw of rawPackages) {
-		const id = raw['id']
-		const availableVersion = raw['available']
+		const id = raw.id
+		const availableVersion = raw.available
 
 		if (!id || !availableVersion) continue
 
-		const installedVersion = raw['version'] || ''
-		const source = raw['source'] || ''
-		const name = raw['name'] || id
+		const installedVersion = raw.version || ''
+		const source = raw.source || 'winget'
+		const name = raw.name || id
 
 		const isUnknownVersion =
 			installedVersion.toLowerCase().includes('unknown') ||

src/core/parser.ts

@@ -1,35 +1,41 @@
 export function parseWingetOutput(rawOutput: string): Record<string, string>[] {
-	const lines = rawOutput.split('\n').map(l => l.trimEnd())
-	const separatorIndex = lines.findIndex(line => line.startsWith('---'))
-	if (separatorIndex === -1 || separatorIndex === 0) {
+	const lines = rawOutput.split('\n').map(l => l.replace(/\r/g, '').trim())
+
+	const separatorIndex = lines.findIndex(line => /^-{10,}$/.test(line))
+	if (separatorIndex === -1) {
 		return []
 	}
-	const headerLine = lines[separatorIndex - 1]
+
 	const dataLines = lines.slice(separatorIndex + 1)
-	const columns: { name: string; start: number; end?: number }[] = []
-	const wordRegex = /\S+/g
-	let match: RegExpExecArray | null = wordRegex.exec(headerLine)
-	while (match !== null) {
-		columns.push({
-			name: match[0].toLowerCase(),
-			start: match.index,
-		})
-		match = wordRegex.exec(headerLine)
-	}
-	for (let i = 0; i < columns.length - 1; i++) {
-		columns[i].end = columns[i + 1].start
-	}
 	const results: Record<string, string>[] = []
+
 	for (const line of dataLines) {
-		if (!line.trim()) continue
-		const entry: Record<string, string> = {}
-		for (const col of columns) {
-			const text = col.end
-				? line.substring(col.start, col.end)
-				: line.substring(col.start)
-			entry[col.name] = text.trim()
+		const cleanLine = line.replace(/^[^a-zA-Z0-9(]+/, '').trim()
+		if (!cleanLine) continue
+
+		if (
+			/^\d+\s+upgrades/i.test(cleanLine) ||
+			cleanLine.includes('upgrades available')
+		)
+			continue
+		if (
+			cleanLine.startsWith('1 package') ||
+			cleanLine.includes('cannot be determined')
+		)
+			continue
+
+		const parts = cleanLine.split(/\s{2,}/)
+
+		if (parts.length >= 4) {
+			results.push({
+				name: parts[0],
+				id: parts[1],
+				version: parts[2],
+				available: parts[3],
+				source: parts[4] || 'winget',
+			})
 		}
-		results.push(entry)
 	}
+
 	return results
 }

src/core/runner.ts

@@ -8,7 +8,7 @@ interface ExecError {
 
 export function getUpgradeList(includeUnknown: boolean): string {
 	let command =
-		'winget upgrade --accept-source-agreements --accept-package-agreements'
+		'winget upgrade'
 	if (includeUnknown) {
 		command += ' --include-unknown'
 	}