chore: build-essentials => build-essential

[rick-a-lane-ii]

build-essentials => build-essential

[coderabbitai]

Walkthrough

Adjusted Debian package name in install script from "build-essentials" to "build-essential", preserving dpkg-based detection, accumulation of missing tools, and apt-get installation flow (using sudo when available). No other behavioral or public-interface changes.

Changes

Cohort / File(s)	Summary
Debian dependency package name fix script/install_dev_dependencies.sh	Changed the Debian package checked/installed from build-essentials to build-essential; detection via dpkg, missing-tools accumulation, and apt-get install logic (with sudo fallback) remain unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Pay attention to the exact package string and any other similar typos in nearby checks or comments.
• Verify behavior on Debian-based systems where build-essential is typically provided as a meta-package.

Poem

I hop through deps with careful tread,
From plural bits to single thread—essential instead!
Apt wakes up, sudo lends a paw,
My script hums tidy, without a flaw.
Tiny change, big cheer—carrots for all! 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The PR title "chore: build-essentials => build-essential" directly and clearly summarizes the main change in the changeset. The raw summary confirms that the primary alteration is changing the package name from "build-essentials" to "build-essential" in the script/install_dev_dependencies.sh file. The title is concise, specific, and uses clear notation to indicate the package name correction without unnecessary noise or vagueness.
Description Check	✅ Passed	The PR description "build-essentials => build-essential" is directly related to the changeset. It mirrors the title and clearly indicates the package name change that is the core modification in this PR. While minimal in detail, the description is not vague or generic—it specifically identifies what is being changed and what it is being changed to, which satisfies the requirement that it be related to the changeset.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4a777cd and bf73062.

📒 Files selected for processing (1)

• script/install_dev_dependencies.sh (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• script/install_dev_dependencies.sh

---

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.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (3)

script/install_dev_dependencies.sh (3)

14-24: Optional: drop redundant make in Debian deps.

build-essential already pulls in make; you can remove the explicit make entry to reduce noise (safe either way).

 debian_deps=(
-    "make"
     "build-essential"
     "direnv"
     "unzip"
     "wget"
     "curl"
     "npm"
     "jq"
     "coreutils"
 )

---

94-97: Harden Debian package detection and silence output.

dpkg -l "$dep" can return 0 for non-installed but known packages (e.g., rc state) and is noisy. Use dpkg -s with redirection.

-            if ! dpkg -l "$dep" ; then
+            if ! dpkg -s "$dep" >/dev/null 2>&1 ; then
                 tools="$tools $dep"
                 status="(not installed)"
             fi

---

86-116: Fail fast on non-Debian Linux instead of silently continuing.

If dpkg is absent, the script does nothing and exits 0. Emit a clear message and non-zero exit.

 elif [[ "$OSTYPE" == "linux-gnu"* ]]; then
     if is_command dpkg; then
         echo "Detected Debian based system"
         tools=
         # ...
-    fi
+    else
+        echo "Detected Linux but not a Debian-based system (dpkg not found). Please install dependencies manually." >&2
+        exit 1
+    fi

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6dbe971 and 4a777cd.

📒 Files selected for processing (1)

• script/install_dev_dependencies.sh (1 hunks)

🔇 Additional comments (1)

script/install_dev_dependencies.sh (1)

16-16: Correct package rename to build-essential.

LGTM; this fixes the Debian meta-package name.

[rick-a-lane-ii]

Rebased against latest changes to main.