pkg: prevent interactive dpkg prompts in automated builds

[igorpecovnik]

Summary

Fixes "end of file on stdin at conffile prompt" errors during automated builds by ensuring all apt-get and dpkg operations run non-interactively.

Problem

During automated builds, dpkg was prompting for conffile decisions (e.g., chromium master_preferences), causing the build to fail with "end of file on stdin at conffile prompt" errors.

Root Cause

The package management functions had inconsistent handling of DEBIAN_FRONTEND=noninteractive:

• Dialog mode had it explicitly set
• Read mode (used in automated builds) was missing it
• dpkg --configure fallbacks were missing it
• An export in desktops.sh wouldn't work in chroot environments

Solution

• Added DEBIAN_FRONTEND=noninteractive to all apt-get commands in read mode
• Added DEBIAN_FRONTEND=noninteractive to dpkg --configure fallbacks
• Removed redundant export from desktops module (now handled centrally)

Benefits

• ✅ Works in all contexts: chroot, subshells, automated builds
• ✅ No reliance on environment variable exports
• ✅ Centralized handling in package functions
• ✅ Prevents all interactive prompts during package operations

Test Plan

• Bash syntax check passes
• Changes verified in both files
• Test in automated build environment with chromium or similar package that has conffile prompts

[coderabbitai]

Walkthrough

This change relocates the DEBIAN_FRONTEND=noninteractive environment variable handling from a global export in the desktop installation module to being applied inline within package management functions. The global export in module_desktops.sh is removed and replaced with inline documentation explaining the change. The actual noninteractive behavior is now set directly in apt_operation_progress during read mode operations and in pkg_install/pkg_remove when handling dpkg --configure operations, maintaining functional equivalence while centralizing the setting at the point of execution.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main change: preventing interactive dpkg prompts during automated builds by applying DEBIAN_FRONTEND=noninteractive, which matches the core objective of the changeset.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description check	✅ Passed	The pull request description is directly related to the changeset, clearly explaining the problem (dpkg prompts during automated builds), root cause (inconsistent DEBIAN_FRONTEND handling), and solution (adding noninteractive flags to apt-get and dpkg operations).

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/pkg-noninteractive-prompts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}