8000 feat(install): list the prerequisites and ask before installing them by adeelahmad · Pull Request #258 · adeelahmad/lense · GitHub
[go: up one dir, main page]

Skip to content

feat(install): list the prerequisites and ask before installing them - #258

Merged
adeelahmad merged 1 commit into
mainfrom
claude/project-thread-017a05
Oct 10, 2026
Merged

adeelahmad merged 1 commit into
mainfrom
claude/project-thread-017a05

Conversation

@adeelahmad
Copy link
Copy Markdown
Owner

Requested by Adeel · project thread

Description

Before: install.sh installed Docker, OrbStack or microsandbox on its own, without saying so first. On a Mac without Homebrew it stopped and told you to install Docker yourself. When the KVM module wasn't loaded, microsandbox wasn't offered at all.

After: at a terminal, it first lists what it's about to install and asks once:

Lens needs these, and will install them:
  - Docker Engine with Compose (from get.docker.com, with sudo)
Install them now? [Y/n]

Answering no installs nothing. With no terminal (CI, scripts) or with LENS_YES=1, it goes ahead as it did before.

It also installs more of what it needs by itself:

  • Homebrew, on a Mac that doesn't have it.
  • The KVM module, through modprobe, when the CPU supports virtualization but /dev/kvm is missing.
  • A missing Compose plugin, with apt or dnf.

Motivation and Context

Adeel asked whether the installer installs dependencies itself, asking and confirming in the terminal. The project's principle is that Lens installs its own prerequisites.

Steps to reproduce (if appropriate):

I tested this in a Debian container without Docker. The installer listed Docker, and answering n stopped it with "Nothing was installed". The install workflow's jobs (Docker and microsandbox) cover the no-terminal path.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change; ! in the title)

Checklist:

  • The title is a Conventional Commit.
  • My code follows the code style of this project (make hooks checks it on commit).
  • I added or updated tests.
  • My change requires documentation updates, and I have updated docs/ accordingly.
  • Users need more than the title to understand it, so I wrote it under ## Unreleased in CHANGELOG.md.

🤖 Generated with Claude Code

https://claude.ai/code/session_01VvAGWoQxiwkGkk374pvkSh


Generated by Claude Code

At a terminal, install.sh now says what it will install (Docker or OrbStack, Homebrew,
microsandbox, the KVM module and access, the Compose plugin) and asks once; without a
terminal, or with LENS_YES=1, it goes ahead as before. It also installs Homebrew on a Mac
that has none, loads the KVM module when /dev/kvm is missing, and installs a missing
Compose plugin with apt or dnf.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VvAGWoQxiwkGkk374pvkSh
@adeelahmad adeelahmad self-assigned this Oct 10, 2026
@adeelahmad
adeelahmad merged commit 317258c into main Oct 10, 2026
9 checks passed
@adeelahmad
adeelahmad deleted the claude/project-thread-017a05 branch October 10, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0