Merge dev to master

[devanfer02]

No description provided.

[Copilot]

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

[gemini-code-assist]

Code Review

This pull request introduces several enhancements to the CiteTrack codebase, including new scripts for KBBI data management, browser E2E testing runbooks, and improved configuration for API lookups. It also adds robust error handling and logging for third-party API calls, updates the documentation, and introduces a new React Doctor skill. My review identified two technical issues: a potential CSV parsing error due to line endings in the KBBI build script and a shell portability issue with ANSI-C quoting in the verification script. Both have been addressed with suggested fixes.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

If the fetched CSV file has CRLF (\\r\\n) line endings, lines[0] will end with \\r. Since parseCsvLine splits by ,, the last cell will contain \\r (e.g., 'submakna\\r'). This causes headerCells.indexOf('submakna') to return -1, throwing an error and crashing the script.

To prevent this, trim each cell in headerCells using .map(c => c.trim()) to ensure robust parsing regardless of line endings.

Suggested change

	const headerCells = parseCsvLine(lines[0])
	const headerCells = parseCsvLine(lines[0]).map((c) => c.trim())

[gemini-code-assist]

The bash-specific ANSI-C quoting $'\\t' is used to pass a tab character to psql. However, execSync runs commands using /bin/sh by default, which on many systems (like Ubuntu/Debian) is dash and does not support $'...' syntax. This will cause psql to receive the literal string $\\t or \\t as the field separator, causing parsing or execution failures.

To make this portable across all shells, use a portable JS tab escape \\t inside the template literal, which expands to a literal tab character in the shell command.

Suggested change

	const raw = execSync(
	`docker compose exec -T db psql -U postgres -d citetrack -tA -F $'\\t' -c "${sql}"`,
	{ encoding: 'utf8' },
	)
	const raw = execSync(
	`docker compose exec -T db psql -U postgres -d citetrack -tA -F '\\t' -c "${sql}"`,
	{ encoding: 'utf8' },
	)