.flake8

@@ -1,8 +1,10 @@
 [flake8]
 count = True
-ignore = W191,W503,E704,E203
+# Several of the following could be autofixed or improved by running the code through psf/black
+ignore = E123,E126,E128,E203,E231,E261,E302,E402,E722,F541,W191,W292,W293,W503
 max-complexity = 40
-max-line-length = 160
+max-line-length = 236
 show-source = True
 statistics = True
-exclude = .git,__pycache__,build,docs,actions-runner
+builtins = _
+per-file-ignores = __init__.py:F401,F403,F405 simple_menu.py:C901,W503 guided.py:C901 network_configuration.py:F821

.github/FUNDING.yml

@@ -1,4 +0,0 @@
-# These are supported funding model platforms
-
-github: [archlinux]
-custom: ['https://archlinux.org/donate/']

.github/ISSUE_TEMPLATE/01_bug.yml

@@ -1,85 +0,0 @@
-name: bug report
-description: archinstall crashed or could not install properly?
-body:
-  - type: markdown
-    attributes:
-      value: >
-        Please read the ~5 known issues first:
-        https://archinstall.archlinux.page/help/known_issues.html
-
-  - type: markdown
-    attributes:
-      value: >
-        **NOTE: Always try the latest official ISO**
-
-  - type: input
-    id: iso
-    attributes:
-      label: Which ISO version are you using?
-      description: 'Always use the latest ISO version'
-      placeholder: '"2024-12-01" or "Dec 1:st"'
-    validations:
-      required: true
-
-  - type: textarea
-    id: bug-report
-    attributes:
-      label: The installation log
-      description: 'note: located at `/var/log/archinstall/install.log`'
-      placeholder: |
-        Hardware model detected: Dell Inc. Precision 7670; UEFI mode: True
-        Processor model detected: 12th Gen Intel(R) Core(TM) i7-12850HX
-        Memory statistics: 31111048 available out of 32545396 total installed
-        Disk states before installing: {'blockdevices': ... }
-        Testing connectivity to the Arch Linux mirrors ...
-        ...
-      render: json
-    validations:
-      required: true
-
-  - type: markdown
-    attributes:
-      value: >
-        **Note**: Assuming you have network connectivity,
-        you can easily upload the installation log and get a shareable URL by running:
-        `archinstall share-log`
-
-  - type: textarea
-    id: freeform
-    attributes:
-      label: describe the problem
-      description: >
-        Please describe your issue as best as you can.
-        And please consider personal preferences vs what the recommended
-        steps/values are in https://wiki.archlinux.org/title/Installation_guide
-        as we try to abide by them as best we can.
-      value: |
-        #### Description of the issue
-
-        I was installing on X hardware ...
-
-        Then X Y Z happened and archinstall crashed ...
-
-        #### Virtual machine config:
-
-        ```xml
-        <domain type="kvm">
-          <name>my-arch-machine</name>
-          ...
-          </devices>
-        </domain>
-        ```
-
-        ```console
-        /usr/bin/qemu-system-x86_64 -name guest=my-arch-machine,debug-threads=on -object ...
-        ```
-    validations:
-      required: true
-
-  - type: markdown
-    attributes:
-      value: >
-        **Note**: Feel free to modify the textarea above as you wish.
-        But it will grately help us in testing if we can generate the specific qemu command line,
-        for instance via:
-        `sudo virsh domxml-to-native qemu-argv --domain my-arch-machine`

.github/ISSUE_TEMPLATE/02_feature.yml

@@ -1,17 +0,0 @@
-name: feature request
-description: a new feature!
-body:
-  - type: markdown
-    attributes:
-      value: >
-        Please read our short mission statement before requesting more features:
-        https://github.com/archlinux/archinstall?tab=readme-ov-file#mission-statement
-
-  - type: textarea
-    id: freeform
-    attributes:
-      label: describe the request
-      description: >
-        Feel free to write any feature you think others might benefit from:
-    validations:
-      required: true

.github/workflows/bandit.yaml

@@ -1,12 +1,12 @@
 on: [ push, pull_request ]
 name: Bandit security checkup
 jobs:
-    bandit:
+    flake8:
         runs-on: ubuntu-latest
         container:
-            image: archlinux/archlinux:latest
+            image: archlinux:latest
         steps:
-            - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
+            - uses: actions/checkout@v3
             - run: pacman --noconfirm -Syu bandit
             - name: Security checkup with Bandit
               run: bandit -r archinstall || exit 0

.github/workflows/flake8.yaml

@@ -1,23 +1,14 @@
 on: [ push, pull_request ]
-name: flake8 linting
+name: flake8 linting (15 ignores)
 jobs:
     flake8:
         runs-on: ubuntu-latest
         container:
-            image: archlinux/archlinux:latest
+            image: archlinux:latest
         steps:
-            - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
+            - uses: actions/checkout@v3
-            - name: Prepare arch
+            - run: pacman --noconfirm -Syu python python-pip
-              run: |
-                pacman-key --init
-                pacman --noconfirm -Sy archlinux-keyring
-                pacman --noconfirm -Syyu
-                pacman --noconfirm -Sy python-pip python-pyparted pkgconfig gcc
             - run: pip install --break-system-packages --upgrade pip
-            # this will install the exact version of flake8 that is in the pyproject.toml file
+            - run: pip install --break-system-packages flake8
-            - name: Install archinstall dependencies
-              run: pip install --break-system-packages .[dev]
-            - run: python --version
-            - run: flake8 --version
             - name: Lint with flake8
               run: flake8

.github/workflows/github-pages.yml

@@ -1,41 +0,0 @@
-name: documentation
-
-on:
-  push:
-    paths:
-      - "docs/**"
-
-  pull_request:
-    paths:
-      - "docs/**"
-
-  workflow_dispatch:
-
-permissions:
-  contents: write
-
-jobs:
-  docs:
-    runs-on: ubuntu-latest
-    container:
-      image: archlinux/archlinux:latest
-      options: --privileged
-    steps:
-      - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
-      - uses: actions/setup-python@5fda3b95a4ea91299a34e894583c3862153e4b97 # v7
-      - name: Install pre-dependencies
-        run: |
-          pacman -Sy --noconfirm tree git python-pyparted python-setuptools python-sphinx python-sphinx_rtd_theme python-build python-installer python-wheel
-      - name: Sphinx build
-        run: |
-          sphinx-build docs _build
-      - name: Deploy to GitHub Pages
-        uses: peaceiris/actions-gh-pages@84c30a85c19949d7eee79c4ff27748b70285e453 # v4
-        if: ${{ github.event_name != 'pull_request' }}
-        with:
-          publish_branch: gh-pages
-          github_token: ${{ secrets.GITHUB_TOKEN }}
-          publish_dir: _build/
-          force_orphan: true
-          enable_jekyll: false # This is required to preserve _static (and thus the theme)
-          cname: archinstall.archlinux.page

.github/workflows/iso-build.yaml

@@ -23,17 +23,17 @@ jobs:
   build:
     runs-on: ubuntu-latest
     container:
-      image: archlinux/archlinux:latest
+      image: archlinux:latest
       options: --privileged
     steps:
-      - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
+      - uses: actions/checkout@v3
       - run: pwd
       - run: find .
       - run: cat /etc/os-release
       - run: pacman-key --init
       - run: pacman --noconfirm -Sy archlinux-keyring
-      - run: ./test_tooling/mkarchiso/build_iso.sh
+      - run: ./build_iso.sh
-      - uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7
+      - uses: actions/upload-artifact@v3
         with:
           name: Arch Live ISO
           path: /tmp/archlive/out/*.iso

.github/workflows/mypy.yaml

@@ -4,20 +4,15 @@ jobs:
     mypy:
         runs-on: ubuntu-latest
         container:
-            image: archlinux/archlinux:latest
+            image: archlinux:latest
         steps:
-            - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
+            - uses: actions/checkout@v3
-            - name: Prepare arch
+            - run: pacman --noconfirm -Syu python mypy python-pip
-              run: |
-                pacman-key --init
-                pacman --noconfirm -Sy archlinux-keyring
-                pacman --noconfirm -Syyu
-                pacman --noconfirm -Sy python-pip python-pyparted pkgconfig gcc
             - run: pip install --break-system-packages --upgrade pip
-            # this will install the exact version of mypy that is in the pyproject.toml file
+            - run: pip install --break-system-packages fastapi pydantic
-            - name: Install archinstall dependencies
-              run: pip install --break-system-packages .[dev]
             - run: python --version
             - run: mypy --version
+            # one day this will be enabled
+              # run: mypy --strict --module archinstall || exit 0
             - name: run mypy
               run: mypy --config-file pyproject.toml

.github/workflows/pylint.yaml

@@ -1,22 +0,0 @@
-on: [ push, pull_request ]
-name: Pylint linting
-jobs:
-    pylint:
-        runs-on: ubuntu-latest
-        container:
-            image: archlinux/archlinux:latest
-        steps:
-            - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
-            - name: Prepare arch
-              run: |
-                pacman-key --init
-                pacman --noconfirm -Sy archlinux-keyring
-                pacman --noconfirm -Syyu
-                pacman --noconfirm -Sy python-pip python-pyparted pkgconfig gcc
-            - run: pip install --break-system-packages --upgrade pip
-            - name: Install Pylint
-              run: pip install --break-system-packages .[dev]
-            - run: python --version
-            - run: pylint --version
-            - name: Lint with Pylint
-              run: pylint .

.github/workflows/pytest.yaml

@@ -4,18 +4,12 @@ jobs:
     pytest:
         runs-on: ubuntu-latest
         container:
-            image: archlinux/archlinux:latest
+            image: archlinux:latest
             options: --privileged
         steps:
-            - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
+            - uses: actions/checkout@v3
-            - name: Prepare arch
+            - run: pacman --noconfirm -Syu python python-pip qemu gcc
-              run: |
+            - run: python -m pip install --break-system-packages --upgrade pip
-                pacman-key --init
+            - run: pip install --break-system-packages pytest
-                pacman --noconfirm -Sy archlinux-keyring
-                pacman --noconfirm -Syyu
-                pacman --noconfirm -Sy python-pip python-pyparted pkgconfig gcc
-            - run: pip install --break-system-packages --upgrade pip
-            - name: Install archinstall dependencies
-              run: pip install --break-system-packages .[dev]
             - name: Test with pytest
-              run: pytest
+              run: python -m pytest || exit 0

.github/workflows/python-build.yml

@@ -8,32 +8,32 @@ jobs:
   deploy:
     runs-on: ubuntu-latest
     container:
-      image: archlinux/archlinux:latest
+      image: archlinux:latest
       options: --privileged
     steps:
-      - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
+      - uses: actions/checkout@v3
       - name: Prepare arch
         run: |
           pacman-key --init
           pacman --noconfirm -Sy archlinux-keyring
-          pacman --noconfirm -Syyu
+          pacman --noconfirm -Sy python-pip python-pyparted python-simple-term-menu pkgconfig gcc
-          pacman --noconfirm -Sy python-uv python-setuptools python-pip
+      - name: Install build dependencies
-          pacman --noconfirm -Sy python-pyparted python-pydantic python-textual
-      - name: Remove existing archinstall (if any)
-        run: 
-          uv pip uninstall archinstall --break-system-packages --system
-      - name: Build archinstall
-        run: uv build --no-build-isolation --wheel
-      - name: Install archinstall
         run: |
-          uv pip install dist/*.whl --break-system-packages --system --no-build --no-deps
+          python -m pip install --break-system-packages --upgrade pip
+          pip install --break-system-packages --upgrade build twine wheel setuptools installer
+          pip uninstall archinstall -y --break-system-packages
+      - name: Build archinstall
+        run: python -m build --wheel --no-isolation
+      - name: Install archinstall
+        run: python -m installer dist/*.whl
       - name: Run archinstall
         run: |
           python -V
           archinstall --script guided -v
+          archinstall --script swiss -v
           archinstall --script only_hd -v
           archinstall --script minimal -v
-      - uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7
+      - uses: actions/upload-artifact@v3
         with:
           name: archinstall
           path: dist/*

.github/workflows/python-publish.yml

@@ -1,4 +1,4 @@
-# This workflow will upload a Python Package when a release is created
+# This workflow will upload a Python Package using Twine when a release is created
 # For more information see: https://help.github.com/en/actions/language-and-framework-guides/using-python-with-github-actions#publishing-to-package-registries
 
 name: Upload archinstall to PyPi
@@ -11,23 +11,23 @@ jobs:
   deploy:
 
     runs-on: ubuntu-latest
-    permissions:
+
-      # IMPORTANT: this permission is mandatory for Trusted Publishing
-      id-token: write
-    container:
-      image: archlinux/archlinux:latest
-      options: --privileged
     steps:
-      - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
+      - uses: actions/checkout@v3
-      - name: Prepare arch
+      - name: Set up Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.x'
+      - name: Install dependencies
         run: |
-          pacman-key --init
+          python -m pip install --upgrade pip
-          pacman --noconfirm -Sy archlinux-keyring
+          pip install build twine
-          pacman --noconfirm -Syyu
-          pacman --noconfirm -Sy python python-uv python-setuptools python-pip python-pyparted python-pydantic python-textual
       - name: Build archinstall
         run: |
-          uv build --no-build-isolation --wheel
+          python -m build . --wheel
       - name: Publish archinstall to PyPi
+        env:
+          TWINE_USERNAME: ${{ secrets.PYPI_USERNAME }}
+          TWINE_PASSWORD: ${{ secrets.PYPI_PASSWORD }}
         run: |
-          uv publish --trusted-publishing always
+          twine upload dist/*

.github/workflows/ruff-format.yaml

@@ -1,9 +0,0 @@
-on: [ push, pull_request ]
-name: ruff check formatting
-jobs:
-  ruff_format_check:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
-      - uses: astral-sh/ruff-action@278981a28ce3188b1e39527901f38254bf3aac89  # v4.1.0
-      - run: ruff format --diff

.github/workflows/ruff-lint.yaml

@@ -1,8 +0,0 @@
-on: [ push, pull_request ]
-name: ruff check linting
-jobs:
-    ruff:
-        runs-on: ubuntu-latest
-        steps:
-            - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
-            - uses: astral-sh/ruff-action@278981a28ce3188b1e39527901f38254bf3aac89  # v4.1.0

.github/workflows/translation-check.yaml

@@ -1,22 +1,22 @@
-name: Translation validation
 on:
     push:
         paths:
-            - 'archinstall/**/*.py'
+        - 'archinstall/locales/**'
-            - 'archinstall/locales/**'
-            - '.github/workflows/translation-check.yaml'
     pull_request:
         paths:
-            - 'archinstall/**/*.py'
+        - 'archinstall/locales/**'
-            - 'archinstall/locales/**'
+name: translation file checks
-            - '.github/workflows/translation-check.yaml'
 jobs:
-    translations:
+    translation-check:
-        name: Validate translations
         runs-on: ubuntu-latest
+        container:
+            image: archlinux:latest
         steps:
-            - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
+            - uses: actions/checkout@v3
-            - name: Install gettext
+            - run: pacman --noconfirm -Syu python
-              run: sudo apt-get update && sudo apt-get install -y gettext
+            - run: cd archinstall/locales
-            - name: Run translation checks
+            - run: pwd
-              run: bash archinstall/locales/locales_generator.sh check
+            - run: ls -l
+            - run: bash locales_generator.sh
+            - run: git diff --name-only
+            - run: ls

.github/workflows/uki-build.yaml

@@ -1,40 +0,0 @@
-# This workflow will build an Arch Linux UKI file with the commit on it
-
-name: Build Arch UKI with ArchInstall Commit
-
-on:
-  push:
-    branches:
-      - master
-      - main # In case we adopt this convention in the future
-  pull_request:
-    paths-ignore:
-      - 'docs/**'
-      - '**.editorconfig'
-      - '**.gitignore'
-      - '**.md'
-      - 'LICENSE'
-      - 'PKGBUILD'
-  release:
-    types:
-      - created
-
-jobs:
-  build:
-    runs-on: ubuntu-latest
-    container:
-      image: archlinux/archlinux:latest
-      options: --privileged
-    steps:
-      - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
-      - run: pwd
-      - run: find .
-      - run: cat /etc/os-release
-      - run: pacman-key --init
-      - run: pacman --noconfirm -Sy archlinux-keyring
-      - run: pacman --noconfirm -Sy mkosi
-      - run: (cd test_tooling/mkosi/ && mkosi build -B)
-      - uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7
-        with:
-          name: Arch Live UKI
-          path: test_tooling/mkosi/mkosi.output/*.efi

.gitignore

@@ -21,11 +21,10 @@ SAFETY_LOCK
 **/**.network
 **/**.target
 **/**.qcow2
-**/**.log
-**/**.fd
 /test*.py
 **/archiso
 /guided.py
+/install.log
 venv
 .venv
 .idea/**
@@ -36,12 +35,3 @@ venv
 /*.sig
 /*.json
 requirements.txt
-/.gitconfig
-/actions-runner
-/cmd_output.txt
-node_modules/
-uv.lock
-test_tooling/mkosi/mkosi.output/*image*
-test_tooling/mkosi/mkosi.cache/**
-test_tooling/mkosi/mkosi.tools/**
-test_tooling/mkosi/mkosi.tools.manifest

.gitlab-ci.yml

@@ -4,7 +4,7 @@
 # These jobs should leverage the same tag as that runner. If necessary, change the tag from 'docker' to the one it uses.
 # All jobs will be run in the official archlinux container image, so we will declare that here.
 
-image: archlinux/archlinux:latest
+image: archlinux:latest
 
 # This can be used to handle common actions. In this case, we do a pacman -Sy to make sure repos are ready to use.
 before_script:
@@ -36,7 +36,7 @@ flake8:
     - flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
 
 # We currently do not have unit tests implemented but this stage is written in anticipation of their future usage.
-# When a stage name is preceded with a '.' it's treated as "disabled" by GitLab and is not executed, so it's fine for it to be declared.
+# When a stage name is preceeded with a '.' it's treated as "disabled" by GitLab and is not executed, so it's fine for it to be declared.
 .pytest:
   stage: test
   tags:

.pre-commit-config.yaml

@@ -1,17 +1,19 @@
-default_stages: ['pre-commit']
+default_stages: ['commit']
 repos:
-  - repo: https://github.com/astral-sh/ruff-pre-commit
+  - repo: https://github.com/pycqa/autoflake
-    rev: v0.16.0
+    rev: v2.1.1
     hooks:
-        # fix unused imports and sort them
+      - id: autoflake
-      - id: ruff
+        args: [
-        args: ["--extend-select", "I", "--fix"]
+          '--in-place',
-        # format the code
+          '--remove-all-unused-imports',
-      - id: ruff-format
+          '--ignore-init-module-imports'
-        # run the linter
+        ]
-      - id: ruff
+        files: \.py$
+        require_serial: true
+        fail_fast: true
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v6.0.0
+    rev: v4.4.0
     hooks:
     # general hooks:
     - id: check-added-large-files  # Prevent giant files from being committed
@@ -21,35 +23,20 @@ repos:
     - id: check-yaml  # Attempts to load all yaml files to verify syntax
     - id: destroyed-symlinks  # Detects symlinks which are changed to regular files
     - id: detect-private-key  # Checks for the existence of private keys
+    - id: end-of-file-fixer  # Makes sure files end in a newline and only a newline
+    - id: trailing-whitespace  # Trims trailing whitespace
     # Python specific hooks:
     - id: check-ast  # Simply check whether files parse as valid python
     - id: check-docstring-first  # Checks for a common error of placing code before the docstring
   - repo: https://github.com/pycqa/flake8
-    rev: 7.3.0
+    rev: 6.0.0
     hooks:
     - id: flake8
       args: [--config=.flake8]
       fail_fast: true
   - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v2.3.0
+    rev: v1.1.1
     hooks:
     - id: mypy
-      args: [
+      args: [--config=pyproject.toml]
-        '--config-file=pyproject.toml'
-      ]
       fail_fast: true
-      additional_dependencies:
-        - pydantic
-        - pytest
-        - hypothesis
-        - cryptography
-        - textual
-  - repo: local
-    hooks:
-      - id: pylint
-        name: pylint
-        entry: pylint
-        language: system
-        types: [python]
-        fail_fast: true
-        require_serial: true

.pypirc

@@ -3,4 +3,4 @@ index-servers =
    pypi
 
 [pypi]
-repository = https://upload.pypi.org/legacy/
+repository = https://upload.pypi.org/legacy/

.readthedocs.yaml

@@ -1,15 +0,0 @@
-# .readthedocs.yml
-# Read the Docs configuration file
-# See https://docs.readthedocs.io/en/stable/config-file/v2.html for details
-
-version: 2
-
-sphinx:
-  builder: html
-  configuration: docs/conf.py
-  fail_on_warning: true
-
-build:
-  os: "ubuntu-22.04"
-  tools:
-    python: "3.12"

.readthedocs.yml

@@ -0,0 +1,22 @@
+# .readthedocs.yml
+# Read the Docs configuration file
+# See https://docs.readthedocs.io/en/stable/config-file/v2.html for details
+
+# Required
+version: 2
+
+# Build documentation in the docs/ directory with Sphinx
+sphinx:
+  configuration: docs/conf.py
+
+# Build documentation with MkDocs
+#mkdocs:
+#  configuration: mkdocs.yml
+
+# Optionally build your docs in additional formats such as PDF
+formats:
+  - pdf
+
+# Optionally set the version of Python and requirements required to build your docs
+python:
+  version: 3.8

CONTRIBUTING.md

@@ -16,7 +16,7 @@ Patch releases will be done against their own branches, branched from stable tag
 ## Discussions
 
 Currently, questions, bugs and suggestions should be reported through [GitHub issue tracker](https://github.com/archlinux/archinstall/issues).<br>
-For less formal discussions there is also an [archinstall Discord server](https://discord.gg/aDeMffrxNg).
+For less formal discussions there is also an [archinstall Discord server](https://discord.gg/cqXU88y).
 
 ## Coding convention
 
@@ -27,7 +27,9 @@ The exceptions to PEP8 are:
 * Archinstall uses [tabs instead of spaces](https://www.python.org/dev/peps/pep-0008/#tabs-or-spaces) simply to make it
   easier for non-IDE developers to navigate the code *(Tab display-width should be equal to 4 spaces)*. Exception to the
   rule are comments that need fine-tuned indentation for documentation purposes.
-* [Line length](https://www.python.org/dev/peps/pep-0008/#maximum-line-length) a maximum line length is enforced via flake8 with 160 characters
+* [Line length](https://www.python.org/dev/peps/pep-0008/#maximum-line-length) a maximum line length is enforced via flake8 with 236 characters
+* [Line breaks before/after binary operator](https://www.python.org/dev/peps/pep-0008/#should-a-line-break-before-or-after-a-binary-operator)
+  is not enforced, as long as the style of line breaks is consistent within the same code block.
 * Archinstall should always be saved with **Unix-formatted line endings** and no other platform-specific formats.
 * [String quotes](https://www.python.org/dev/peps/pep-0008/#string-quotes) follow PEP8, the exception being when
   creating formatted strings, double-quoted strings are *preferred* but not required on the outer edges *(
@@ -38,7 +40,7 @@ There might therefore be older code which does not follow the coding convention
 
 ## Git hooks
 
-`archinstall` ships pre-commit hooks that make it easier to run checks such as `mypy`, `ruff check`, and `flake8` locally.
+`archinstall` ships pre-commit hooks that make it easier to run check such as `mypy` and `flake8` locally.
 The checks are listed in `.pre-commit-config.yaml` and can be installed via
 ```
 pre-commit install
@@ -68,4 +70,15 @@ squash commits when the pull request is merged.
 Maintainer:
 * Anton Hvornum ([@Torxed](https://github.com/Torxed))
 
-[Contributors](https://github.com/archlinux/archinstall/graphs/contributors)
+At present the current contributors are (alphabetically):
+* Anton Hvornum ([@Torxed](https://github.com/Torxed))
+* Borislav Kosharov ([@nikibobi](https://github.com/nikibobi))
+* demostanis ([@demostanis](https://github.com/demostanis))
+* Dylan Taylor ([@dylanmtaylor](https://github.com/dylanmtaylor))
+* Giancarlo Razzolini (@[grazzolini](https://github.com/grazzolini))
+* Jerker Bengtsson ([@jaybent](https://github.com/jaybent))
+* j-james ([@j-james](https://github.com/j-james))
+* Ninchester ([@ninchester](https://github.com/ninchester))
+* nullrequest ([@advaithm](https://github.com/advaithm))
+* Philipp Schaffrath ([@phisch](https://github.com/phisch))
+* Varun Madiath ([@vamega](https://github.com/vamega))

PKGBUILD

@@ -1,53 +1,36 @@
 # Maintainer: David Runge <dvzrv@archlinux.org>
 # Maintainer: Giancarlo Razzolini <grazzolini@archlinux.org>
-# Maintainer: Anton Hvornum <torxed@archlinux.org>
 # Contributor: Anton Hvornum <anton@hvornum.se>
 # Contributor: demostanis worlds <demostanis@protonmail.com>
 
 pkgname=archinstall
-pkgver=4.4
+pkgver=2.6.0
 pkgrel=1
 pkgdesc="Just another guided/automated Arch Linux installer with a twist"
 arch=(any)
 url="https://github.com/archlinux/archinstall"
-license=(GPL-3.0-only)
+license=(GPL3)
 depends=(
   'arch-install-scripts'
   'btrfs-progs'
   'coreutils'
   'cryptsetup'
-  'dosfstools'
   'e2fsprogs'
-  'glibc'
   'kbd'
-  'libcrypt.so'
-  'libxcrypt'
   'pciutils'
   'procps-ng'
   'python'
-  'python-cryptography'
-  'python-pydantic'
   'python-pyparted'
-  'python-textual'
+  'python-simple-term-menu'
-  'python-markdown-it-py'
-  'python-linkify-it-py'
   'systemd'
   'util-linux'
-  'xfsprogs'
-  'lvm2'
-  'f2fs-tools'
-  'libfido2'
 )
 makedepends=(
-  'python-build'
-  'python-installer'
   'python-setuptools'
   'python-sphinx'
+  'python-build'
+  'python-installer'
   'python-wheel'
-  'python-sphinx_rtd_theme'
-  'python-pylint'
-  'python-pylint-pydantic'
-  'ruff'
 )
 optdepends=(
   'python-systemd: Adds journald logging'
@@ -56,17 +39,14 @@ provides=(python-archinstall archinstall)
 conflicts=(python-archinstall archinstall-git)
 replaces=(python-archinstall archinstall-git)
 source=(
-  $pkgname-$pkgver.tar.gz::$url/archive/refs/tags/$pkgver.tar.gz
+  $pkgname-$pkgver.tar.gz::$url/archive/refs/tags/v$pkgver.tar.gz
-  $pkgname-$pkgver.tar.gz.sig::$url/releases/download/$pkgver/$pkgname-$pkgver.tar.gz.sig
+  $pkgname-$pkgver.tar.gz.sig::$url/releases/download/v$pkgver/$pkgname-$pkgver.tar.gz.sig
 )
-sha512sums=()
+sha512sums=('64cb3593c5091b3885ad14ef073cfab31090b4f9bcb4405b18cf9b19adb5ca42255ba8891ec62e21f92d59872541ef6d94f186fb05c625822af63525441e08d9'
-b2sums=()
+            'SKIP')
-validpgpkeys=('8AA2213C8464C82D879C8127D4B58E897A929F2E') # torxed@archlinux.org
+b2sums=('9c0ec0871841804377ba8310dc744711adcec4eed7319a8d89d684af8e7b822bb9d47540b00f4d746a9fcd7b9ea1b9e07bac773e6c28fabc760e4df38b16748b'
-
+        'SKIP')
-check() {
+validpgpkeys=('256F73CEEFC6705C6BBAB20E5FBBB32941E3740A') # Anton Hvornum (Torxed) <anton@hvornum.se>
-  cd $pkgname-$pkgver
-  ruff check
-}
 
 pkgver() {
   cd $pkgname-$pkgver
@@ -74,6 +54,13 @@ pkgver() {
   awk '$1 ~ /^__version__/ {gsub("\"", ""); print $3}' archinstall/__init__.py
 }
 
+prepare() {
+  cd $pkgname-$pkgver
+
+  # use real directories for examples and profiles, as symlinks do not work
+  rm -fv $pkgname/{examples,profiles}
+}
+
 build() {
   cd $pkgname-$pkgver
 

README.md

@@ -6,148 +6,195 @@
 [![Lint Python and Find Syntax Errors](https://github.com/archlinux/archinstall/actions/workflows/flake8.yaml/badge.svg)](https://github.com/archlinux/archinstall/actions/workflows/flake8.yaml)
 
 Just another guided/automated [Arch Linux](https://wiki.archlinux.org/index.php/Arch_Linux) installer with a twist.
-The installer also doubles as a python library to install Arch Linux and manage services, packages, and other things inside the installed system *(Usually from a live medium or from an existing installation)*.
+The installer also doubles as a python library to install Arch Linux and manage services, packages and other things inside the installed system *(Usually from a live medium)*.
 
-* archinstall [discord](https://discord.gg/aDeMffrxNg) server
+* archinstall [discord](https://discord.gg/cqXU88y) server
-* archinstall [#archinstall:matrix.org](https://matrix.to/#/#archinstall:matrix.org) Matrix channel
+* archinstall [matrix.org](https://app.element.io/#/room/#archinstall:matrix.org) channel
-* archinstall [#archinstall@irc.libera.chat:6697](https://web.libera.chat/?channel=#archinstall)
+* archinstall [#archinstall@irc.libera.chat](irc://#archinstall@irc.libera.chat:6697)
-* archinstall [documentation](https://archinstall.archlinux.page/)
+* archinstall [documentation](https://archinstall.readthedocs.io/)
 
 # Installation & Usage
-> [!TIP]
-> In the ISO you are root by default. Use sudo if running from an existing system.
 
-```shell
+    $ sudo pacman -S archinstall
-pacman-key --init
-pacman -Sy archinstall
-archinstall
-```
 
-Alternative ways to install are `git clone` the repository (and is better since you get the latest code regardless of [build date](https://archlinux.org/packages/?sort=&q=archinstall)) or `pip install --upgrade archinstall`.
+Alternative ways to install are `git clone` the repository or `pip install --upgrade archinstall`.
-
-## Upgrade `archinstall` on live Arch ISO image
-
-Upgrading archinstall on the ISO needs to be done via a full system upgrade using 
-
-```shell
-pacman -Syu
-```
-
-When booting from a live USB, the space on the ramdisk is limited and may not be sufficient to allow running a re-installation or upgrade of the installer.
-In case one runs into this issue, any of the following can be used
-
-* Resize the root partition https://wiki.archlinux.org/title/Archiso#Adjusting_the_size_of_the_root_file_system
-* Specify the boot parameter copytoram=y (https://gitlab.archlinux.org/archlinux/mkinitcpio/mkinitcpio-archiso/-/blob/master/docs/README.bootparams#L26) which will copy the root filesystem to tmpfs
 
 ## Running the [guided](https://github.com/archlinux/archinstall/blob/master/archinstall/scripts/guided.py) installer
 
-Assuming you are on an Arch Linux live-ISO or installed via `pip`, `archinstall` will use the `guided` script by default
+Assuming you are on an Arch Linux live-ISO or installed via `pip`:
-```shell
-archinstall
-```
-similar goes for running the [guided](https://github.com/archlinux/archinstall/blob/master/archinstall/scripts/guided.py) installer using `git
 
-```shell
+    # archinstall
-git clone https://github.com/archlinux/archinstall
-cd archinstall
-python -m archinstall $@
-```
 
-To run alternative scripts using the `--script` parameter
+## Running the [guided](https://github.com/archlinux/archinstall/blob/master/archinstall/scripts/guided.py) installer using `git`
 
-```
+    # cd archinstall-git
-archinstall --script <name>
+    # cp archinstall/scripts/guided.py
-```
+    # python guided.py
 
 #### Advanced
-Some additional options that most users do not need are hidden behind the `--advanced` flag and all options/args can be consulted through `-h` or `--help`. 
+Some additional options that are not needed by most users are hidden behind the `--advanced` flag.
 
 ## Running from a declarative configuration file or URL
 
 `archinstall` can be run with a JSON configuration file. There are 2 different configuration files to consider,
 the `user_configuration.json` contains all general installation configuration, whereas the `user_credentials.json`
-contains the sensitive user configuration such as user password, root password, and encryption password.
+contains the sensitive user configuration such as user password, root password and encryption password.
 
 An example of the user configuration file can be found here
 [configuration file](https://github.com/archlinux/archinstall/blob/master/examples/config-sample.json)
-and an example of the credentials configuration here
+and example of the credentials configuration here
 [credentials file](https://github.com/archlinux/archinstall/blob/master/examples/creds-sample.json).
 
 **HINT:** The configuration files can be auto-generated by starting `archinstall`, configuring all desired menu
 points and then going to `Save configuration`.
 
 To load the configuration file into `archinstall` run the following command
-```shell
+```
 archinstall --config <path to user config file or URL> --creds <path to user credentials config file or URL>
 ```
 
-### Credentials configuration file encryption
-By default, all user account credentials are hashed with `yescrypt` and only the hash is stored in the saved `user_credentials.json` file.
-This is not possible for disk encryption password which needs to be stored in plaintext to be able to apply it.
-
-However, when selecting to save configuration files, `archinstall` will prompt for the option to encrypt the `user_credentials.json` file content.
-A prompt will require to enter a encryption password to encrypt the file. When providing an encrypted `user_configuration.json` as a argument with `--creds <user_credentials.json>`
-there are multiple ways to provide the decryption key:
-* Provide the decryption key via the command line argument `--creds-decryption-key <password>`
-* Store the encryption key in the environment variable `ARCHINSTALL_CREDS_DECRYPTION_KEY` which will be read automatically
-* If none of the above is provided a prompt will be shown to enter the decryption key manually
-
-
-# Help or Issues
-
-If you come across any issues, kindly submit your issue here on GitHub or post your query in the
-[discord](https://discord.gg/aDeMffrxNg) help channel.
-
-When submitting an issue, please:
-* Provide the stacktrace of the output if applicable
-* Attach the `/var/log/archinstall/install.log` to the issue ticket. This helps us help you!
-  * To upload the log from the ISO image and get a shareable URL, run<br>
-    ```shell
-    archinstall share-log
-    ```
-
-
 # Available Languages
 
 Archinstall is available in different languages which have been contributed and are maintained by the community.
-The language can be switched inside the installer (first menu entry). Bear in mind that not all languages provide
+Current translations are listed below and vary in the amount of translations per language
-full translations as we rely on contributors to do the translations. Each language has an indicator that shows
+```
-how much has been translated.
+English
+Arabic
+Brazilian Portuguese
+Czech
+Dutch
+Estonian
+French
+Georgian
+German
+Indonesian
+Italian
+Korean
+Modern Greek
+Polish
+Portuguese
+Russian
+Spanish
+Swedish
+Tamil
+Turkish
+Ukrainian
+Urdu
+```
 
 Any contributions to the translations are more than welcome,
 to get started please follow [the guide](https://github.com/archlinux/archinstall/blob/master/archinstall/locales/README.md)
 
-## Fonts
+# Help or Issues
-The ISO does not ship with all fonts needed for different languages.
-Fonts that use a different character set than Latin will not be displayed correctly. If those languages
-want to be selected then a proper font has to be set manually in the console.
 
-All available console fonts can be found in `/usr/share/kbd/consolefonts` and set with `setfont LatGrkCyr-8x16`.
+Submit an issue here on GitHub, or submit a post in the discord help channel.<br>
+When doing so, attach the `/var/log/archinstall/install.log` to the issue ticket. This helps us help you!
+
+# Mission Statement
+
+Archinstall promises to ship a [guided installer](https://github.com/archlinux/archinstall/blob/master/archinstall/scripts/guided.py) that follows
+the [Arch Principles](https://wiki.archlinux.org/index.php/Arch_Linux#Principles) as well as a library to manage services, packages and other Arch Linux aspects.
+
+The guided installer will provide user-friendly options along the way, but the keyword here is options, they are optional and will never be forced upon anyone.
+The guided installer itself is also optional to use if so desired and not forced upon anyone.
+
+---
+
+Archinstall has one fundamental function which is to be a flexible library to manage services, packages and other aspects inside the installed system.
+This library is in turn used by the provided guided installer but is also for anyone who wants to script their own installations.
+
+Therefore, Archinstall will try its best to not introduce any breaking changes except for major releases which may break backwards compatibility after notifying about such changes.
 
 
 # Scripting your own installation
 
 ## Scripting interactive installation
 
-For an example of a fully scripted, interactive installation please refer to the example
+There are some examples in the `examples/` directory that should serve as a starting point.
-[interactive_installation.py](https://github.com/archlinux/archinstall/blob/master/archinstall/scripts/guided.py)
 
+The following is a small example of how to script your own *interative* installation:
 
-> **To create your own ISO with this script in it:** Follow [ArchISO](https://wiki.archlinux.org/index.php/archiso)'s guide on creating your own ISO.
+```python
+from pathlib import Path
+
+from archinstall import Installer, ProfileConfiguration, profile_handler, User
+from archinstall.default_profiles.minimal import MinimalProfile
+from archinstall.lib.disk.device_model import FilesystemType
+from archinstall.lib.disk.encryption_menu import DiskEncryptionMenu
+from archinstall.lib.disk.filesystem import FilesystemHandler
+from archinstall.lib.interactions.disk_conf import select_disk_config
+
+fs_type = FilesystemType('ext4')
+
+# Select a device to use for the installation
+disk_config = select_disk_config()
+
+# Optional: ask for disk encryption configuration
+data_store = {}
+disk_encryption = DiskEncryptionMenu(disk_config.device_modifications, data_store).run()
+
+# initiate file handler with the disk config and the optional disk encryption config
+fs_handler = FilesystemHandler(disk_config, disk_encryption)
+
+# perform all file operations
+# WARNING: this will potentially format the filesystem and delete all data
+fs_handler.perform_filesystem_operations()
+
+mountpoint = Path('/tmp')
+
+with Installer(
+        mountpoint,
+        disk_config,
+        disk_encryption=disk_encryption,
+        kernels=['linux']
+) as installation:
+    installation.mount_ordered_layout()
+    installation.minimal_installation(hostname='minimal-arch')
+    installation.add_additional_packages(['nano', 'wget', 'git'])
+
+    # Optionally, install a profile of choice.
+    # In this case, we install a minimal profile that is empty
+    profile_config = ProfileConfiguration(MinimalProfile())
+    profile_handler.install_profile_config(installation, profile_config)
+
+    user = User('archinstall', 'password', True)
+    installation.create_users(user)
+```
+
+This installer will perform the following:
+
+* Prompt the user to configurate the disk partitioning
+* Prompt the user to setup disk encryption
+* Create a file handler instance for the configured disk and the optional disk encryption
+* Perform the disk operations (WARNING: this will potentially format the disks and erase all data)
+* Installs a basic instance of Arch Linux *(base base-devel linux linux-firmware btrfs-progs efibootmgr)*
+* Installs and configures a bootloader to partition 0 on uefi. On BIOS, it sets the root to partition 0.
+* Install additional packages *(nano, wget, git)*
+* Create a new user
+
+> **Creating your own ISO with this script on it:** Follow [ArchISO](https://wiki.archlinux.org/index.php/archiso)'s guide on how to create your own ISO.
 
 ## Script non-interactive automated installation
 
-For an example of a fully scripted, automated installation please refer to the example
+For an example of a fully scripted, automated installation please see the example
 [full_automated_installation.py](https://github.com/archlinux/archinstall/blob/master/examples/full_automated_installation.py)
 
+## Unattended installation based on MAC address
+
+Archinstall comes with an [unattended](https://github.com/archlinux/archinstall/blob/master/examples/mac_address_installation.py)
+example which will look for a matching profile for the machine it is being run on, based on any local MAC address.
+For instance, if the machine the code is executed on has the MAC address `52:54:00:12:34:56` it will look for a profile called
+[52-54-00-12-34-56.py](https://github.com/archlinux/archinstall/blob/master/archinstall/default_profiles/tailored.py).
+If it's found, the unattended installation will commence and source that profile as its installation procedure.
+
 # Profiles
 
-`archinstall` comes with a set of pre-configured profiles available for selection during the installation process.
+`archinstall` ships with a set of pre-defined profiles that can be chosen during the installation process.
 
 - [Desktop](https://github.com/archlinux/archinstall/tree/master/archinstall/default_profiles/desktops)
 - [Server](https://github.com/archlinux/archinstall/tree/master/archinstall/default_profiles/servers)
 
-The profiles' definitions and the packages they will install can be directly viewed in the menu, or
+The definitions of the profiles and what packages they will install can be seen directly in the menu or
 [default profiles](https://github.com/archlinux/archinstall/tree/master/archinstall/default_profiles)
 
 
@@ -155,20 +202,19 @@ The profiles' definitions and the packages they will install can be directly vie
 
 ## Using a Live ISO Image
 
-If you want to test a commit, branch, or bleeding edge release from the repository using the standard Arch Linux Live ISO image,
+If you want to test a commit, branch or bleeding edge release from the repository using the vanilla Arch Live ISO image,
-replace the archinstall version with a newer one and execute the subsequent steps defined below.
+you can replace the version of archinstall with a new version and run that with the steps described below:
 
 1. You need a working network connection
 2. Install the build requirements with `pacman -Sy; pacman -S git python-pip gcc pkgconf`
    *(note that this may or may not work depending on your RAM and current state of the squashfs maximum filesystem free space)*
-3. Uninstall the previous version of archinstall with `pip uninstall --break-system-packages archinstall`
+3. Uninstall the previous version of archinstall with `pip uninstall archinstall`
 4. Now clone the latest repository with `git clone https://github.com/archlinux/archinstall`
 5. Enter the repository with `cd archinstall`
    *At this stage, you can choose to check out a feature branch for instance with `git checkout v2.3.1-rc1`*
-6. To run the source code, there are 2 different options:
+6. Build the project and install it using `pip install --break-operating-system .`
-   - Run a specific branch version from source directly using `python -m archinstall`, in most cases this will work just fine, the
+
-      rare case it will not work is if the source has introduced any new dependencies that are not installed yet
+After this, running archinstall with `python -m archinstall` will run against whatever branch you chose in step 5.
-   - Installing the branch version with `pip install --break-system-packages .` and `archinstall`
 
 ## Without a Live ISO Image
 
@@ -176,10 +222,11 @@ To test this without a live ISO, the simplest approach is to use a local image a
 This can be done by installing `pacman -S arch-install-scripts util-linux` locally and doing the following:
 
     # truncate -s 20G testimage.img
-    # losetup --partscan --show ./testimage.img
+    # losetup -fP ./testimage.img
+    # losetup -a | grep "testimage.img" | awk -F ":" '{print $1}'
     # pip install --upgrade archinstall
     # python -m archinstall --script guided
-    # qemu-system-x86_64 -enable-kvm -machine q35,accel=kvm -device intel-iommu -cpu host -m 4096 -boot order=d -drive file=./testimage.img,format=raw -drive if=pflash,format=raw,readonly,file=/usr/share/edk2/x64/OVMF_CODE.4m.fd -drive if=pflash,format=raw,readonly,file=/usr/share/edk2/x64/OVMF_VARS.4m.fd 
+    # qemu-system-x86_64 -enable-kvm -machine q35,accel=kvm -device intel-iommu -cpu host -m 4096 -boot order=d -drive file=./testimage.img,format=raw -drive if=pflash,format=raw,readonly,file=/usr/share/ovmf/x64/OVMF_CODE.fd -drive if=pflash,format=raw,readonly,file=/usr/share/ovmf/x64/OVMF_VARS.fd
 
 This will create a *20 GB* `testimage.img` and create a loop device which we can use to format and install to.<br>
 `archinstall` is installed and executed in [guided mode](#docs-todo). Once the installation is complete, ~~you can use qemu/kvm to boot the test media.~~<br>
@@ -188,81 +235,6 @@ This will create a *20 GB* `testimage.img` and create a loop device which we can
 There's also a [Building and Testing](https://github.com/archlinux/archinstall/wiki/Building-and-Testing) guide.<br>
 It will go through everything from packaging, building and running *(with qemu)* the installer against a dev branch.
 
-## Boot an Arch ISO image in a VM
-
-You may want to boot an ISO image in a VM to test `archinstall` in there.
-
-* Download the latest [Arch ISO](https://archlinux.org/download/)
-* Use the the below command to boot the ISO in a VM
-
-```
-qemu-system-x86_64 -enable-kvm \
--machine q35,accel=kvm -device intel-iommu \
--cpu host -m 4096 -boot order=d \
--drive if=pflash,format=raw,readonly,file=/usr/share/edk2/x64/OVMF_CODE.4m.fd \
--drive if=pflash,format=raw,readonly,file=/usr/share/edk2/x64/OVMF_VARS.4m.fd \
--drive file=./archlinux-2025.12.01-x86_64.iso,format=raw
-```
-
-HINT: For espeakup support
-```
-qemu-system-x86_64 -enable-kvm \
--machine q35,accel=kvm -device intel-iommu \
--cpu host -m 4096 -boot order=d \
--drive if=pflash,format=raw,readonly,file=/usr/share/edk2/x64/OVMF_CODE.4m.fd \
--drive if=pflash,format=raw,readonly,file=/usr/share/edk2/x64/OVMF_VARS.4m.fd \
--drive file=./archlinux-2025.12.01-x86_64.iso,format=raw \
--device intel-hda -device hda-duplex,audiodev=snd0 \
--audiodev pa,id=snd0,server=/run/user/1000/pulse/native
-```
-
-
-# FAQ
-
-## AUR
-
-`archinstall` will not offer or bundle AUR helpers or AUR packages due to a current consensus. This is not any individual developers decision. The reasons and discussions for this stance on the topic can be found on our mailing list thread: [(optional) AUR helper in archinstall](https://lists.archlinux.org/archives/list/arch-dev-public@lists.archlinux.org/thread/VYOULH2GOJLFM2BXOFLWH3D754YXFPSL/).
-
-## Keyring out-of-date
-For a description of the problem see https://archinstall.archlinux.page/help/known_issues.html#keyring-is-out-of-date-2213 and discussion in issue https://github.com/archlinux/archinstall/issues/2213.
-
-For a quick fix the below command will install the latest keyrings
-
-```pacman -Sy archlinux-keyring```
-
-## How to dual boot with Windows
-
-To install Arch Linux alongside an existing Windows installation using  `archinstall`, follow these steps:
-
-1. Ensure some unallocated space is available for the Linux installation after the Windows installation.
-2. Boot into the ISO and run `archinstall`.
-3. Choose `Disk configuration` -> `Manual partitioning`.
-4. Select the disk on which Windows resides.
-5. Select `Create a new partition`.
-6. Choose a filesystem type.
-7. Determine the start and end sectors for the new partition location (values can be suffixed with various units).
-8. Assign the mountpoint `/` to the new partition.
-9. Assign the `Boot/ESP` partition the mountpoint `/boot` from the partitioning menu.
-10. Confirm your settings and exit to the main menu by choosing `Confirm and exit`.
-11. Modify any additional settings for your installation as necessary.
-12. Start the installation upon completion of setup.
-
-
-# Mission Statement
-
-Archinstall promises to ship a [guided installer](https://github.com/archlinux/archinstall/blob/master/archinstall/scripts/guided.py) that follows
-the [Arch Linux Principles](https://wiki.archlinux.org/index.php/Arch_Linux#Principles) as well as a library to manage services, packages, and other Arch Linux aspects.
-
-The guided installer ensures a user-friendly experience, offering optional selections throughout the process. Emphasizing its flexible nature, these options are never obligatory.
-In addition, the decision to use the guided installer remains entirely with the user, reflecting the Linux philosophy of providing full freedom and flexibility.
-
----
-
-Archinstall primarily functions as a flexible library for managing services, packages, and other elements within an Arch Linux system.
-This core library is the backbone for the guided installer that Archinstall provides. It is also designed to be used by those who wish to script their own custom installations.
-
-Therefore, Archinstall will try its best to not introduce any breaking changes except for major releases which may break backward compatibility after notifying about such changes.
-
 
 # Contributing
 

archinstall/__init__.py

@@ -1,3 +1,291 @@
-from archinstall.lib.plugins import plugin
+"""Arch Linux installer - guided, templates etc."""
+import importlib
+import os
+from argparse import ArgumentParser, Namespace
+from pathlib import Path
+from typing import TYPE_CHECKING, Any, Dict, Union
 
-__all__ = ['plugin']
+from .lib import disk
+from .lib import menu
+from .lib import models
+from .lib import packages
+from .lib import exceptions
+from .lib import luks
+from .lib import locale
+from .lib import mirrors
+from .lib import networking
+from .lib import profile
+from .lib import interactions
+from . import default_profiles
+
+from .lib.hardware import SysInfo, GfxDriver
+from .lib.installer import Installer, accessibility_tools_in_use
+from .lib.output import FormattedOutput, log, error, debug, warn, info
+from .lib.storage import storage
+from .lib.global_menu import GlobalMenu
+from .lib.boot import Boot
+from .lib.translationhandler import TranslationHandler, Language, DeferredTranslation
+from .lib.plugins import plugins, load_plugin
+from .lib.configuration import ConfigurationOutput
+
+from .lib.general import (
+	generate_password, locate_binary, clear_vt100_escape_codes,
+	JSON, UNSAFE_JSON, SysCommandWorker, SysCommand,
+	run_custom_user_commands, json_stream_to_structure, secret
+)
+
+
+if TYPE_CHECKING:
+	_: Any
+
+
+__version__ = "2.6.0"
+storage['__version__'] = __version__
+
+
+# add the custome _ as a builtin, it can now be used anywhere in the
+# project to mark strings as translatable with _('translate me')
+DeferredTranslation.install()
+
+# Log various information about hardware before starting the installation. This might assist in troubleshooting
+debug(f"Hardware model detected: {SysInfo.sys_vendor()} {SysInfo.product_name()}; UEFI mode: {SysInfo.has_uefi()}")
+debug(f"Processor model detected: {SysInfo.cpu_model()}")
+debug(f"Memory statistics: {SysInfo.mem_available()} available out of {SysInfo.mem_total()} total installed")
+debug(f"Virtualization detected: {SysInfo.virtualization()}; is VM: {SysInfo.is_vm()}")
+debug(f"Graphics devices detected: {SysInfo._graphics_devices().keys()}")
+
+# For support reasons, we'll log the disk layout pre installation to match against post-installation layout
+debug(f"Disk states before installing: {disk.disk_layouts()}")
+
+
+if os.getuid() != 0:
+	print(_("Archinstall requires root privileges to run. See --help for more."))
+	exit(1)
+
+
+parser = ArgumentParser()
+
+
+def define_arguments():
+	"""
+	Define which explicit arguments do we allow.
+	Refer to https://docs.python.org/3/library/argparse.html for documentation and
+			https://docs.python.org/3/howto/argparse.html for a tutorial
+	Remember that the property/entry name python assigns to the parameters is the first string defined as argument and
+	dashes inside it '-' are changed to '_'
+	"""
+	parser.add_argument("-v", "--version", action="version", version="%(prog)s " + __version__)
+	parser.add_argument("--config", nargs="?", help="JSON configuration file or URL")
+	parser.add_argument("--creds", nargs="?", help="JSON credentials configuration file")
+	parser.add_argument("--silent", action="store_true",
+						help="WARNING: Disables all prompts for input and confirmation. If no configuration is provided, this is ignored")
+	parser.add_argument("--dry-run", "--dry_run", action="store_true",
+						help="Generates a configuration file and then exits instead of performing an installation")
+	parser.add_argument("--script", default="guided", nargs="?", help="Script to run for installation", type=str)
+	parser.add_argument("--mount-point","--mount_point", nargs="?", type=str, help="Define an alternate mount point for installation")
+	parser.add_argument("--debug", action="store_true", default=False, help="Adds debug info into the log")
+	parser.add_argument("--offline", action="store_true", default=False, help="Disabled online upstream services such as package search and key-ring auto update.")
+	parser.add_argument("--no-pkg-lookups", action="store_true", default=False, help="Disabled package validation specifically prior to starting installation.")
+	parser.add_argument("--plugin", nargs="?", type=str)
+
+
+def parse_unspecified_argument_list(unknowns :list, multiple :bool = False, err :bool = False) -> dict:
+	"""We accept arguments not defined to the parser. (arguments "ad hoc").
+	Internally argparse return to us a list of words so we have to parse its contents, manually.
+	We accept following individual syntax for each argument
+		--argument value
+		--argument=value
+		--argument = value
+		--argument   (boolean as default)
+	the optional parameters to the function alter a bit its behaviour:
+	* multiple allows multivalued arguments, each value separated by whitespace. They're returned as a list
+	* error. If set any non correctly specified argument-value pair to raise an exception. Else, simply notifies the existence of a problem and continues processing.
+
+	To a certain extent, multiple and error are incompatible. In fact, the only error this routine can catch, as of now, is the event
+	argument value value ...
+	which isn't am error if multiple is specified
+	"""
+	tmp_list = unknowns[:]   # wastes a few bytes, but avoids any collateral effect of the destructive nature of the pop method()
+	config = {}
+	key = None
+	last_key = None
+	while tmp_list:
+		element = tmp_list.pop(0)			  # retrieve an element of the list
+		if element.startswith('--'):		   # is an argument ?
+			if '=' in element:				 # uses the arg=value syntax ?
+				key, value = [x.strip() for x in element[2:].split('=', 1)]
+				config[key] = value
+				last_key = key				 # for multiple handling
+				key = None					 # we have the kwy value pair we need
+			else:
+				key = element[2:]
+				config[key] = True   # every argument starts its lifecycle as boolean
+		else:
+			if element == '=':
+				continue
+			if key:
+				config[key] = element
+				last_key = key # multiple
+				key = None
+			else:
+				if multiple and last_key:
+					if isinstance(config[last_key],str):
+						config[last_key] = [config[last_key],element]
+					else:
+						config[last_key].append(element)
+				elif err:
+					raise ValueError(f"Entry {element} is not related to any argument")
+				else:
+					print(f" We ignore the entry {element} as it isn't related to any argument")
+	return config
+
+
+def cleanup_empty_args(args: Union[Namespace, Dict]) -> Dict:
+	"""
+	Takes arguments (dictionary or argparse Namespace) and removes any
+	None values. This ensures clean mergers during dict.update(args)
+	"""
+	if type(args) is Namespace:
+		args = vars(args)
+
+	clean_args = {}
+	for key, val in args.items():
+		if isinstance(val, dict):
+			val = cleanup_empty_args(val)
+
+		if val is not None:
+			clean_args[key] = val
+
+	return clean_args
+
+def get_arguments() -> Dict[str, Any]:
+	""" The handling of parameters from the command line
+	Is done on following steps:
+	0) we create a dict to store the arguments and their values
+	1) preprocess.
+		We take those arguments which use JSON files, and read them into the argument dict. So each first level entry becomes a argument on it's own right
+	2) Load.
+		We convert the predefined argument list directly into the dict via the vars() function. Non specified arguments are loaded with value None or false if they are booleans (action="store_true").
+		The name is chosen according to argparse conventions. See above (the first text is used as argument name, but underscore substitutes dash)
+		We then load all the undefined arguments. In this case the names are taken as written.
+		Important. This way explicit command line arguments take precedence over configuration files.
+	3) Amend
+		Change whatever is needed on the configuration dictionary (it could be done in post_process_arguments but  this ougth to be left to changes anywhere else in the code, not in the arguments dictionary
+	"""
+	config: Dict[str, Any] = {}
+	args, unknowns = parser.parse_known_args()
+	# preprocess the JSON files.
+	# TODO Expand the url access to the other JSON file arguments ?
+	if args.config is not None:
+		if not json_stream_to_structure('--config', args.config, config):
+			exit(1)
+
+	if args.creds is not None:
+		if not json_stream_to_structure('--creds', args.creds, config):
+			exit(1)
+
+	# load the parameters. first the known, then the unknowns
+	clean_args = cleanup_empty_args(args)
+	config.update(clean_args)
+	config.update(parse_unspecified_argument_list(unknowns))
+	# amend the parameters (check internal consistency)
+	# Installation can't be silent if config is not passed
+	if clean_args.get('config') is None:
+		config["silent"] = False
+	else:
+		config["silent"] = clean_args.get('silent')
+
+	# avoiding a compatibility issue
+	if 'dry-run' in config:
+		del config['dry-run']
+
+	return config
+
+
+def load_config():
+	"""
+	refine and set some arguments. Formerly at the scripts
+	"""
+	from .lib.models import NetworkConfiguration
+
+	arguments['locale_config'] = locale.LocaleConfiguration.parse_arg(arguments)
+
+	if (archinstall_lang := arguments.get('archinstall-language', None)) is not None:
+		arguments['archinstall-language'] = TranslationHandler().get_language_by_name(archinstall_lang)
+
+	if disk_config := arguments.get('disk_config', {}):
+		arguments['disk_config'] = disk.DiskLayoutConfiguration.parse_arg(disk_config)
+
+	if profile_config := arguments.get('profile_config', None):
+		arguments['profile_config'] = profile.ProfileConfiguration.parse_arg(profile_config)
+
+	if mirror_config := arguments.get('mirror_config', None):
+		arguments['mirror_config'] = mirrors.MirrorConfiguration.parse_args(mirror_config)
+
+	if arguments.get('servers', None) is not None:
+		storage['_selected_servers'] = arguments.get('servers', None)
+
+	if arguments.get('network_config', None) is not None:
+		config = NetworkConfiguration.parse_arg(arguments.get('network_config'))
+		arguments['network_config'] = config
+
+	if arguments.get('!users', None) is not None or arguments.get('!superusers', None) is not None:
+		users = arguments.get('!users', None)
+		superusers = arguments.get('!superusers', None)
+		arguments['!users'] = models.User.parse_arguments(users, superusers)
+
+	if arguments.get('bootloader', None) is not None:
+		arguments['bootloader'] = models.Bootloader.from_arg(arguments['bootloader'])
+
+	if arguments.get('audio_config', None) is not None:
+		arguments['audio_config'] = models.AudioConfiguration.parse_arg(arguments['audio_config'])
+
+	if arguments.get('disk_encryption', None) is not None and disk_config is not None:
+		password = arguments.get('encryption_password', '')
+		arguments['disk_encryption'] = disk.DiskEncryption.parse_arg(
+			arguments['disk_config'],
+			arguments['disk_encryption'],
+			password
+		)
+
+
+def post_process_arguments(arguments):
+	storage['arguments'] = arguments
+	if mountpoint := arguments.get('mount_point', None):
+		storage['MOUNT_POINT'] = Path(mountpoint)
+
+	if arguments.get('debug', False):
+		warn(f"Warning: --debug mode will write certain credentials to {storage['LOG_PATH']}/{storage['LOG_FILE']}!")
+
+	if arguments.get('plugin', None):
+		path = arguments['plugin']
+		load_plugin(path)
+
+	load_config()
+
+
+define_arguments()
+arguments: Dict[str, Any] = get_arguments()
+post_process_arguments(arguments)
+
+
+# @archinstall.plugin decorator hook to programmatically add
+# plugins in runtime. Useful in profiles_bck and other things.
+def plugin(f, *args, **kwargs):
+	plugins[f.__name__] = f
+
+
+def run_as_a_module():
+	"""
+	This can either be run as the compiled and installed application: python setup.py install
+	OR straight as a module: python -m archinstall
+	In any case we will be attempting to load the provided script to be run from the scripts/ folder
+	"""
+	script = arguments.get('script', None)
+
+	if script is None:
+		print('No script to run provided')
+
+	mod_name = f'archinstall.scripts.{script}'
+	# by loading the module we'll automatically run the script
+	importlib.import_module(mod_name)

archinstall/__main__.py

@@ -1,6 +1,19 @@
+import importlib
 import sys
+import pathlib
 
-from archinstall.main import main
+# Load .git version before the builtin version
+if pathlib.Path('./archinstall/__init__.py').absolute().exists():
+	spec = importlib.util.spec_from_file_location("archinstall", "./archinstall/__init__.py")
+
+	if spec is None or spec.loader is None:
+		raise ValueError('Could not retrieve spec from file: archinstall/__init__.py')
+
+	archinstall = importlib.util.module_from_spec(spec)
+	sys.modules["archinstall"] = archinstall
+	spec.loader.exec_module(archinstall)
+else:
+	import archinstall
 
 if __name__ == '__main__':
-	sys.exit(main())
+	archinstall.run_as_a_module()

archinstall/applications/audio.py

@@ -1,80 +0,0 @@
-from typing import TYPE_CHECKING
-
-from archinstall.lib.hardware import SysInfo
-from archinstall.lib.log import debug
-from archinstall.lib.models.application import Audio, AudioConfiguration
-from archinstall.lib.models.users import User
-
-if TYPE_CHECKING:
-	from archinstall.lib.installer import Installer
-
-
-class AudioApp:
-	@property
-	def pulseaudio_packages(self) -> list[str]:
-		return [
-			'pulseaudio',
-		]
-
-	@property
-	def pipewire_packages(self) -> list[str]:
-		return [
-			'pipewire',
-			'pipewire-alsa',
-			'pipewire-jack',
-			'pipewire-pulse',
-			'gst-plugin-pipewire',
-			'libpulse',
-			'wireplumber',
-		]
-
-	def _enable_pipewire(
-		self,
-		install_session: Installer,
-		users: list[User] | None = None,
-	) -> None:
-		if users is None:
-			return
-
-		for user in users:
-			# Create the full path for enabling the pipewire systemd items
-			service_dir = install_session.target / 'home' / user.username / '.config' / 'systemd' / 'user' / 'default.target.wants'
-			service_dir.mkdir(parents=True, exist_ok=True)
-
-			# Set ownership of the entire user catalogue
-			install_session.arch_chroot(f'chown -R {user.username}:{user.username} /home/{user.username}')
-
-			# symlink in the correct pipewire systemd items
-			install_session.arch_chroot(
-				f'ln -sf /usr/lib/systemd/user/pipewire-pulse.service /home/{user.username}/.config/systemd/user/default.target.wants/pipewire-pulse.service',
-				run_as=user.username,
-			)
-			install_session.arch_chroot(
-				f'ln -sf /usr/lib/systemd/user/pipewire-pulse.socket /home/{user.username}/.config/systemd/user/default.target.wants/pipewire-pulse.socket',
-				run_as=user.username,
-			)
-
-	def install(
-		self,
-		install_session: Installer,
-		audio_config: AudioConfiguration,
-		users: list[User] | None = None,
-	) -> None:
-		debug(f'Installing audio server: {audio_config.audio.value}')
-
-		if audio_config.audio == Audio.NO_AUDIO:
-			debug('No audio server selected, skipping installation.')
-			return
-
-		if SysInfo.requires_sof_fw():
-			install_session.add_additional_packages('sof-firmware')
-
-		if SysInfo.requires_alsa_fw():
-			install_session.add_additional_packages('alsa-firmware')
-
-		match audio_config.audio:
-			case Audio.PIPEWIRE:
-				install_session.add_additional_packages(self.pipewire_packages)
-				self._enable_pipewire(install_session, users)
-			case Audio.PULSEAUDIO:
-				install_session.add_additional_packages(self.pulseaudio_packages)

archinstall/applications/bluetooth.py

@@ -1,26 +0,0 @@
-from typing import TYPE_CHECKING
-
-from archinstall.lib.log import debug
-
-if TYPE_CHECKING:
-	from archinstall.lib.installer import Installer
-
-
-class BluetoothApp:
-	@property
-	def packages(self) -> list[str]:
-		return [
-			'bluez',
-			'bluez-utils',
-		]
-
-	@property
-	def services(self) -> list[str]:
-		return [
-			'bluetooth.service',
-		]
-
-	def install(self, install_session: Installer) -> None:
-		debug('Installing Bluetooth')
-		install_session.add_additional_packages(self.packages)
-		install_session.enable_service(self.services)

archinstall/applications/firewall.py

@@ -1,52 +0,0 @@
-from typing import TYPE_CHECKING
-
-from archinstall.lib.log import debug
-from archinstall.lib.models.application import Firewall, FirewallConfiguration
-
-if TYPE_CHECKING:
-	from archinstall.lib.installer import Installer
-
-
-class FirewallApp:
-	@property
-	def ufw_packages(self) -> list[str]:
-		return [
-			'ufw',
-		]
-
-	@property
-	def fwd_packages(self) -> list[str]:
-		return [
-			'firewalld',
-		]
-
-	@property
-	def ufw_services(self) -> list[str]:
-		return [
-			'ufw.service',
-		]
-
-	@property
-	def fwd_services(self) -> list[str]:
-		return [
-			'firewalld.service',
-		]
-
-	def install(
-		self,
-		install_session: Installer,
-		firewall_config: FirewallConfiguration,
-	) -> None:
-		debug(f'Installing firewall: {firewall_config.firewall.value}')
-
-		match firewall_config.firewall:
-			case Firewall.UFW:
-				install_session.add_additional_packages(self.ufw_packages)
-				install_session.enable_service(self.ufw_services)
-				# write default conf file to enabled
-				ufw_conf = install_session.target / 'etc/ufw/ufw.conf'
-				ufw_conf.write_text(ufw_conf.read_text().replace('ENABLED=no', 'ENABLED=yes'))
-
-			case Firewall.FWD:
-				install_session.add_additional_packages(self.fwd_packages)
-				install_session.enable_service(self.fwd_services)

archinstall/applications/fonts.py

@@ -1,14 +0,0 @@
-from typing import TYPE_CHECKING
-
-from archinstall.lib.log import debug
-from archinstall.lib.models.application import FontsConfiguration
-
-if TYPE_CHECKING:
-	from archinstall.lib.installer import Installer
-
-
-class FontsApp:
-	def install(self, install_session: Installer, fonts_config: FontsConfiguration) -> None:
-		packages = [f.value for f in fonts_config.fonts]
-		debug(f'Installing fonts: {packages}')
-		install_session.add_additional_packages(packages)

archinstall/applications/power_management.py

@@ -1,49 +0,0 @@
-from typing import TYPE_CHECKING
-
-from archinstall.lib.log import debug
-from archinstall.lib.models.application import PowerManagement, PowerManagementConfiguration
-
-if TYPE_CHECKING:
-	from archinstall.lib.installer import Installer
-
-
-class PowerManagementApp:
-	@property
-	def ppd_packages(self) -> list[str]:
-		return [
-			'power-profiles-daemon',
-		]
-
-	@property
-	def tuned_packages(self) -> list[str]:
-		return [
-			'tuned',
-			'tuned-ppd',
-		]
-
-	@property
-	def ppd_services(self) -> list[str]:
-		return [
-			'power-profiles-daemon.service',
-		]
-
-	@property
-	def tuned_services(self) -> list[str]:
-		return [
-			'tuned.service',
-		]
-
-	def install(
-		self,
-		install_session: Installer,
-		power_management_config: PowerManagementConfiguration,
-	) -> None:
-		debug(f'Installing power management daemon: {power_management_config.power_management.value}')
-
-		match power_management_config.power_management:
-			case PowerManagement.POWER_PROFILES_DAEMON:
-				install_session.add_additional_packages(self.ppd_packages)
-				install_session.enable_service(self.ppd_services)
-			case PowerManagement.TUNED:
-				install_session.add_additional_packages(self.tuned_packages)
-				install_session.enable_service(self.tuned_services)

archinstall/applications/print_service.py

@@ -1,23 +0,0 @@
-from typing import TYPE_CHECKING
-
-from archinstall.lib.log import debug
-
-if TYPE_CHECKING:
-	from archinstall.lib.installer import Installer
-
-
-class PrintServiceApp:
-	@property
-	def packages(self) -> list[str]:
-		return ['cups', 'system-config-printer', 'cups-pk-helper']
-
-	@property
-	def services(self) -> list[str]:
-		return [
-			'cups.service',
-		]
-
-	def install(self, install_session: Installer) -> None:
-		debug('Installing print service')
-		install_session.add_additional_packages(self.packages)
-		install_session.enable_service(self.services)

archinstall/default_profiles/applications/pipewire.py

@@ -0,0 +1,40 @@
+from typing import List, Union, Any, TYPE_CHECKING
+
+import archinstall
+
+from archinstall.default_profiles.profile import Profile, ProfileType
+from archinstall.lib.models import User
+
+if TYPE_CHECKING:
+	from archinstall.lib.installer import Installer
+	_: Any
+
+
+class PipewireProfile(Profile):
+	def __init__(self):
+		super().__init__('Pipewire', ProfileType.Application)
+
+	@property
+	def packages(self) -> List[str]:
+		return [
+			'pipewire',
+			'pipewire-alsa',
+			'pipewire-jack',
+			'pipewire-pulse',
+			'gst-plugin-pipewire',
+			'libpulse',
+			'wireplumber'
+		]
+
+	def _enable_pipewire_for_all(self, install_session: 'Installer'):
+		users: Union[User, List[User]] = archinstall.arguments.get('!users', [])
+		if not isinstance(users, list):
+			users = [users]
+
+		for user in users:
+			install_session.arch_chroot('systemctl enable --user pipewire-pulse.service', run_as=user.username)
+
+	def install(self, install_session: 'Installer'):
+		super().install(install_session)
+		install_session.add_additional_packages(self.packages)
+		self._enable_pipewire_for_all(install_session)

archinstall/default_profiles/custom.py

@@ -0,0 +1,218 @@
+# from typing import List, Dict, Optional, TYPE_CHECKING, Any
+#
+# from ..lib import menu
+# from archinstall.lib.output import log, FormattedOutput
+# from archinstall.lib.profile.profiles_handler import profile_handler
+# from archinstall.default_profiles.profile import Profile, ProfileType, SelectResult, ProfileInfo, TProfile
+#
+# if TYPE_CHECKING:
+# 	from archinstall.lib.installer import Installer
+# 	_: Any
+#
+#
+# class CustomProfileList(menu.ListManager):
+# 	def __init__(self, prompt: str, profiles: List[TProfile]):
+# 		self._actions = [
+# 			str(_('Add profile')),
+# 			str(_('Edit profile')),
+# 			str(_('Delete profile'))
+# 		]
+# 		super().__init__(prompt, profiles, [self._actions[0]], self._actions[1:])
+#
+# 	def reformat(self, data: List[TProfile]) -> Dict[str, Optional[TProfile]]:
+# 		table = FormattedOutput.as_table(data)
+# 		rows = table.split('\n')
+#
+# 		# these are the header rows of the table and do not map to any profile obviously
+# 		# we're adding 2 spaces as prefix because the menu selector '> ' will be put before
+# 		# the selectable rows so the header has to be aligned
+# 		display_data: Dict[str, Optional[TProfile]] = {f'  {rows[0]}': None, f'  {rows[1]}': None}
+#
+# 		for row, profile in zip(rows[2:], data):
+# 			row = row.replace('|', '\\|')
+# 			display_data[row] = profile
+#
+# 		return display_data
+#
+# 	def selected_action_display(self, profile: TProfile) -> str:
+# 		return profile.name
+#
+# 	def handle_action(
+# 		self,
+# 		action: str,
+# 		entry: Optional['CustomTypeProfile'],
+# 		data: List['CustomTypeProfile']
+# 	) -> List['CustomTypeProfile']:
+# 		if action == self._actions[0]:  # add
+# 			new_profile = self._add_profile()
+# 			if new_profile is not None:
+# 				# in case a profile with the same name as an existing profile
+# 				# was created we'll replace the existing one
+# 				data = [d for d in data if d.name != new_profile.name]
+# 				data += [new_profile]
+# 		elif entry is not None:
+# 			if action == self._actions[1]:  # edit
+# 				new_profile = self._add_profile(entry)
+# 				if new_profile is not None:
+# 					# we'll remove the original profile and add the modified version
+# 					data = [d for d in data if d.name != entry.name and d.name != new_profile.name]
+# 					data += [new_profile]
+# 			elif action == self._actions[2]:  # delete
+# 				data = [d for d in data if d != entry]
+#
+# 		return data
+#
+# 	def _is_new_profile_name(self, name: str) -> bool:
+# 		existing_profile = profile_handler.get_profile_by_name(name)
+# 		if existing_profile is not None and existing_profile.profile_type != ProfileType.CustomType:
+# 			return False
+# 		return True
+#
+# 	def _add_profile(self, editing: Optional['CustomTypeProfile'] = None) -> Optional['CustomTypeProfile']:
+# 		name_prompt = '\n\n' + str(_('Profile name: '))
+#
+# 		while True:
+# 			profile_name = menu.TextInput(name_prompt, editing.name if editing else '').run().strip()
+#
+# 			if not profile_name:
+# 				return None
+#
+# 			if not self._is_new_profile_name(profile_name):
+# 				error_prompt = str(_("The profile name you entered is already in use. Try again"))
+# 				print(error_prompt)
+# 			else:
+# 				break
+#
+# 		packages_prompt = str(_('Packages to be install with this profile (space separated, leave blank to skip): '))
+# 		edit_packages = ' '.join(editing.packages) if editing else ''
+# 		packages = menu.TextInput(packages_prompt, edit_packages).run().strip()
+#
+# 		services_prompt = str(_('Services to be enabled with this profile (space separated, leave blank to skip): '))
+# 		edit_services = ' '.join(editing.services) if editing else ''
+# 		services = menu.TextInput(services_prompt, edit_services).run().strip()
+#
+# 		choice = menu.Menu(
+# 			str(_('Should this profile be enabled for installation?')),
+# 			menu.Menu.yes_no(),
+# 			skip=False,
+# 			default_option=menu.Menu.no(),
+# 			clear_screen=False,
+# 			show_search_hint=False
+# 		).run()
+#
+# 		enable_profile = True if choice.value == menu.Menu.yes() else False
+#
+# 		profile = CustomTypeProfile(
+# 			profile_name,
+# 			enabled=enable_profile,
+# 			packages=packages.split(' '),
+# 			services=services.split(' ')
+# 		)
+#
+# 		return profile
+#
+#
+# # TODO
+# # Still needs some ironing out
+# class CustomProfile():
+# 	def __init__(self):
+# 		super().__init__(
+# 			'Custom',
+# 			ProfileType.Custom,
+# 			description=str(_('Create your own'))
+# 		)
+#
+# 	def json(self) -> Dict[str, Any]:
+# 		data: Dict[str, Any] = {'main': self.name, 'gfx_driver': self.gfx_driver, 'custom': []}
+#
+# 		for profile in self._current_selection:
+# 			data['custom'].append({
+# 				'name': profile.name,
+# 				'packages': profile.packages,
+# 				'services': profile.services,
+# 				'enabled': profile.custom_enabled
+# 			})
+#
+# 		return data
+#
+# 	def do_on_select(self) -> SelectResult:
+# 		custom_profile_list = CustomProfileList('', profile_handler.get_custom_profiles())
+# 		custom_profiles = custom_profile_list.run()
+#
+# 		# we'll first remove existing custom default_profiles with
+# 		# the same name and then add the new ones this
+# 		# will avoid errors of default_profiles with duplicate naming
+# 		profile_handler.remove_custom_profiles(custom_profiles)
+# 		profile_handler.add_custom_profiles(custom_profiles)
+#
+# 		self.set_current_selection(custom_profiles)
+#
+# 		if custom_profile_list.is_last_choice_cancel():
+# 			return SelectResult.SameSelection
+#
+# 		enabled_profiles = [p for p in self._current_selection if p.custom_enabled]
+# 		# in  case we only created inactive default_profiles we wanna store them but
+# 		# we want to reset the original setting
+# 		if not enabled_profiles:
+# 			return SelectResult.ResetCurrent
+#
+# 		return SelectResult.NewSelection
+#
+# 	def post_install(self, install_session: 'Installer'):
+# 		for profile in self._current_selection:
+# 			profile.post_install(install_session)
+#
+# 	def install(self, install_session: 'Installer'):
+# 		driver_packages = self.gfx_driver_packages()
+# 		install_session.add_additional_packages(driver_packages)
+#
+# 		for profile in self._current_selection:
+# 			if profile.custom_enabled:
+# 				log(f'Installing custom profile {profile.name}...')
+#
+# 				install_session.add_additional_packages(profile.packages)
+# 				install_session.enable_service(profile.services)
+#
+# 				profile.install(install_session)
+#
+# 	def info(self) -> Optional[ProfileInfo]:
+# 		enabled_profiles = [p for p in self._current_selection if p.custom_enabled]
+# 		if enabled_profiles:
+# 			details = ', '.join([p.name for p in enabled_profiles])
+# 			gfx_driver = self.gfx_driver
+# 			return ProfileInfo(self.name, details, gfx_driver)
+#
+# 		return None
+#
+# 	def reset(self):
+# 		for profile in self._current_selection:
+# 			profile.set_enabled(False)
+#
+# 		self.gfx_driver = None
+#
+#
+# class CustomTypeProfile(Profile):
+# 	def __init__(
+# 		self,
+# 		name: str,
+# 		enabled: bool = False,
+# 		packages: List[str] = [],
+# 		services: List[str] = []
+# 	):
+# 		super().__init__(
+# 			name,
+# 			ProfileType.CustomType,
+# 			packages=packages,
+# 			services=services,
+# 			support_gfx_driver=True
+# 		)
+#
+# 		self.custom_enabled = enabled
+#
+# 	def json(self) -> Dict[str, Any]:
+# 		return {
+# 			'name': self.name,
+# 			'packages': self.packages,
+# 			'services': self.services,
+# 			'enabled': self.custom_enabled
+# 		}

archinstall/default_profiles/desktop.py

@@ -1,44 +1,43 @@
-from typing import TYPE_CHECKING, Self, override
+from typing import Any, TYPE_CHECKING, List, Optional, Dict
 
-from archinstall.default_profiles.desktops.utils import provision_seat_access
+from archinstall.lib import menu
-from archinstall.default_profiles.profile import CustomSetting, DisplayServerType, GreeterType, Profile, ProfileType, SelectResult
+from archinstall.lib.output import info
-from archinstall.lib.log import info
-from archinstall.lib.menu.helpers import Selection
 from archinstall.lib.profile.profiles_handler import profile_handler
-from archinstall.tui.menu_item import MenuItem, MenuItemGroup
+from archinstall.default_profiles.profile import Profile, ProfileType, SelectResult, GreeterType
-from archinstall.tui.result import ResultType
 
 if TYPE_CHECKING:
 	from archinstall.lib.installer import Installer
-	from archinstall.lib.models.users import User
+	_: Any
 
 
 class DesktopProfile(Profile):
-	def __init__(self, current_selection: list[Self] = []) -> None:
+	def __init__(self, current_selection: List[Profile] = []):
 		super().__init__(
 			'Desktop',
 			ProfileType.Desktop,
+			description=str(_('Provides a selection of desktop environments and tiling window managers, e.g. gnome, kde, sway')),
 			current_selection=current_selection,
-			support_greeter=True,
+			support_greeter=True
 		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return [
 			'nano',
 			'vim',
 			'openssh',
 			'htop',
 			'wget',
+			'iwd',
+			'wireless_tools',
+			'wpa_supplicant',
 			'smartmontools',
-			'xdg-utils',
+			'xdg-utils'
 		]
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType | None:
+		combined_greeters: Dict[GreeterType, int] = {}
-		combined_greeters: dict[GreeterType, int] = {}
 		for profile in self.current_selection:
 			if profile.default_greeter_type:
 				combined_greeters.setdefault(profile.default_greeter_type, 0)
@@ -49,70 +48,40 @@ class DesktopProfile(Profile):
 
 		return None
 
-	async def _do_on_select_profiles(self) -> None:
+	def _do_on_select_profiles(self):
 		for profile in self.current_selection:
-			await profile.do_on_select()
+			profile.do_on_select()
 
-	@override
+	def do_on_select(self) -> SelectResult:
-	async def do_on_select(self) -> SelectResult:
+		choice = profile_handler.select_profile(
-		items = [
+			profile_handler.get_desktop_profiles(),
-			MenuItem(
+			self._current_selection,
-				p.name,
+			title=str(_('Select your desired desktop environment')),
-				value=p,
+			multi=True
-				preview_action=lambda x: x.value.preview_text() if x.value else None,
+		)
-			)
-			for p in profile_handler.get_desktop_profiles()
-		]
 
-		group = MenuItemGroup(items, sort_items=True, sort_case_sensitive=False)
+		match choice.type_:
-		group.set_selected_by_value(self.current_selection)
+			case menu.MenuSelectionType.Selection:
-
+				self.set_current_selection(choice.value)  # type: ignore
-		result = await Selection[Self](
+				self._do_on_select_profiles()
-			group,
-			multi=True,
-			allow_reset=True,
-			allow_skip=True,
-			preview_location='right',
-		).show()
-
-		match result.type_:
-			case ResultType.Selection:
-				self.current_selection = result.get_values()
-				await self._do_on_select_profiles()
 				return SelectResult.NewSelection
-			case ResultType.Skip:
+			case menu.MenuSelectionType.Skip:
 				return SelectResult.SameSelection
-			case ResultType.Reset:
+			case menu.MenuSelectionType.Reset:
 				return SelectResult.ResetCurrent
 
-	@override
+	def post_install(self, install_session: 'Installer'):
-	def post_install(self, install_session: Installer) -> None:
+		for profile in self._current_selection:
-		for profile in self.current_selection:
 			profile.post_install(install_session)
 
-	@override
+	def install(self, install_session: 'Installer'):
-	def provision(self, install_session: Installer, users: list[User]) -> None:
-		for profile in self.current_selection:
-			profile.provision(install_session, users)
-
-			if seat_access := profile.custom_settings.get(CustomSetting.SeatAccess):
-				provision_seat_access(install_session, users, seat_access)
-
-	@override
-	def install(self, install_session: Installer) -> None:
 		# Install common packages for all desktop environments
 		install_session.add_additional_packages(self.packages)
 
-		xorg_installed = False
+		for profile in self._current_selection:
-
-		for profile in self.current_selection:
 			info(f'Installing profile {profile.name}...')
 
 			install_session.add_additional_packages(profile.packages)
 			install_session.enable_service(profile.services)
 
-			if not xorg_installed and profile.display_server == DisplayServerType.Xorg:
-				install_session.add_additional_packages(['xorg-server', 'xorg-xinit'])
-				xorg_installed = True
-
 			profile.install(install_session)

archinstall/default_profiles/desktops/awesome.py

@@ -1,26 +1,23 @@
-from typing import TYPE_CHECKING, override
+from typing import List, Optional, Any, TYPE_CHECKING
 
-from archinstall.default_profiles.profile import DisplayServerType, Profile, ProfileType
+from archinstall.default_profiles.profile import ProfileType
+from archinstall.default_profiles.xorg import XorgProfile
 
 if TYPE_CHECKING:
 	from archinstall.lib.installer import Installer
+	_: Any
 
 
-class AwesomeProfile(Profile):
+class AwesomeProfile(XorgProfile):
-	def __init__(self) -> None:
+	def __init__(self):
-		super().__init__(
+		super().__init__('Awesome', ProfileType.WindowMgr, description='')
-			'Awesome',
-			ProfileType.WindowMgr,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Xorg,
-		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
+		return super().packages + [
-		return [
 			'awesome',
 			'alacritty',
+			'xorg-xinit',
 			'xorg-xrandr',
 			'xterm',
 			'feh',
@@ -31,37 +28,40 @@ class AwesomeProfile(Profile):
 			'xsel',
 		]
 
-	@override
+	def preview_text(self) -> Optional[str]:
-	def install(self, install_session: Installer) -> None:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()
+
+	def install(self, install_session: 'Installer'):
 		super().install(install_session)
 
 		# TODO: Copy a full configuration to ~/.config/awesome/rc.lua instead.
-		with open(f'{install_session.target}/etc/xdg/awesome/rc.lua') as fh:
+		with open(f"{install_session.target}/etc/xdg/awesome/rc.lua", 'r') as fh:
 			awesome_lua = fh.read()
 
 		# Replace xterm with alacritty for a smoother experience.
 		awesome_lua = awesome_lua.replace('"xterm"', '"alacritty"')
 
-		with open(f'{install_session.target}/etc/xdg/awesome/rc.lua', 'w') as fh:
+		with open(f"{install_session.target}/etc/xdg/awesome/rc.lua", 'w') as fh:
 			fh.write(awesome_lua)
 
 		# TODO: Configure the right-click-menu to contain the above packages that were installed. (as a user config)
 
 		# TODO: check if we selected a greeter,
 		# but for now, awesome is intended to run without one.
-		with open(f'{install_session.target}/etc/X11/xinit/xinitrc') as xinitrc:
+		with open(f"{install_session.target}/etc/X11/xinit/xinitrc", 'r') as xinitrc:
 			xinitrc_data = xinitrc.read()
 
 		for line in xinitrc_data.split('\n'):
-			if 'twm &' in line:
+			if "twm &" in line:
-				xinitrc_data = xinitrc_data.replace(line, f'# {line}')
+				xinitrc_data = xinitrc_data.replace(line, f"# {line}")
-			if 'xclock' in line:
+			if "xclock" in line:
-				xinitrc_data = xinitrc_data.replace(line, f'# {line}')
+				xinitrc_data = xinitrc_data.replace(line, f"# {line}")
-			if 'xterm' in line:
+			if "xterm" in line:
-				xinitrc_data = xinitrc_data.replace(line, f'# {line}')
+				xinitrc_data = xinitrc_data.replace(line, f"# {line}")
 
 		xinitrc_data += '\n'
 		xinitrc_data += 'exec awesome\n'
 
-		with open(f'{install_session.target}/etc/X11/xinit/xinitrc', 'w') as xinitrc:
+		with open(f"{install_session.target}/etc/X11/xinit/xinitrc", 'w') as xinitrc:
-			xinitrc.write(xinitrc_data)
+			xinitrc.write(xinitrc_data)

archinstall/default_profiles/desktops/bspwm.py

@@ -1,39 +1,52 @@
-from typing import override
+from typing import List, Optional, Any, TYPE_CHECKING
 
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
+from archinstall.default_profiles.profile import ProfileType, GreeterType
-from archinstall.lib.installer import Installer
+from archinstall.default_profiles.xorg import XorgProfile
-from archinstall.lib.models.users import User
+
+if TYPE_CHECKING:
+	_: Any
 
 
-class BspwmProfile(Profile):
+class BspwmProfile(XorgProfile):
-	def __init__(self) -> None:
+	def __init__(self):
-		super().__init__(
+		super().__init__('Bspwm', ProfileType.WindowMgr, description='')
-			'Bspwm',
-			ProfileType.WindowMgr,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Xorg,
-		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
+		# return super().packages + [
 		return [
 			'bspwm',
 			'sxhkd',
 			'dmenu',
 			'xdo',
-			'rxvt-unicode',
+			'rxvt-unicode'
 		]
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType:
 		return GreeterType.Lightdm
 
-	@override
+	def preview_text(self) -> Optional[str]:
-	def provision(self, install_session: Installer, users: list[User]) -> None:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
-		for user in users:
+		return text + '\n' + self.packages_text()
-			install_session.arch_chroot('mkdir -p ~/.config/bspwm ~/.config/sxhkd', run_as=user.username)
+
-			install_session.arch_chroot('cp /usr/share/doc/bspwm/examples/bspwmrc ~/.config/bspwm/', run_as=user.username)
+		# The wiki specified xinit, but we already use greeter?
-			install_session.arch_chroot('cp /usr/share/doc/bspwm/examples/sxhkdrc ~/.config/sxhkd/', run_as=user.username)
+		# https://wiki.archlinux.org/title/Bspwm#Starting
-			install_session.arch_chroot('chmod +x ~/.config/bspwm/bspwmrc', run_as=user.username)
+		#
+		# # TODO: check if we selected a greeter, else run this:
+		# with open(f"{install_session.target}/etc/X11/xinit/xinitrc", 'r') as xinitrc:
+		# 	xinitrc_data = xinitrc.read()
+
+		# for line in xinitrc_data.split('\n'):
+		# 	if "twm &" in line:
+		# 		xinitrc_data = xinitrc_data.replace(line, f"# {line}")
+		# 	if "xclock" in line:
+		# 		xinitrc_data = xinitrc_data.replace(line, f"# {line}")
+		# 	if "xterm" in line:
+		# 		xinitrc_data = xinitrc_data.replace(line, f"# {line}")
+
+		# xinitrc_data += '\n'
+		# xinitrc_data += 'exec bspwn\n'
+
+		# with open(f"{install_session.target}/etc/X11/xinit/xinitrc", 'w') as xinitrc:
+		# 	xinitrc.write(xinitrc_data)

archinstall/default_profiles/desktops/budgie.py

@@ -1,30 +1,30 @@
-from typing import override
+from typing import List, Optional, Any, TYPE_CHECKING
 
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	_: Any
 
 
-class BudgieProfile(Profile):
+class BudgieProfile(XorgProfile):
-	def __init__(self) -> None:
+	def __init__(self):
-		super().__init__(
+		super().__init__('Budgie', ProfileType.DesktopEnv, description='')
-			'Budgie',
-			ProfileType.DesktopEnv,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Wayland,
-		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return [
-			'materia-gtk-theme',
+			"arc-gtk-theme",
-			'budgie',
+			"budgie",
-			'mate-terminal',
+			"mate-terminal",
-			'nemo',
+			"nemo",
-			'nemo-fileroller',
+			"papirus-icon-theme"
-			'papirus-icon-theme',
 		]
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType:
+		return GreeterType.Lightdm
-		return GreeterType.LightdmSlick
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()

archinstall/default_profiles/desktops/cinnamon.py

@@ -1,33 +1,31 @@
-from typing import override
+from typing import Optional, List, Any, TYPE_CHECKING
 
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	_: Any
 
 
-class CinnamonProfile(Profile):
+class CinnamonProfile(XorgProfile):
-	def __init__(self) -> None:
+	def __init__(self):
-		super().__init__(
+		super().__init__('Cinnamon', ProfileType.DesktopEnv, description='')
-			'Cinnamon',
-			ProfileType.DesktopEnv,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Xorg,
-		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return [
-			'cinnamon',
+			"cinnamon",
-			'system-config-printer',
+			"system-config-printer",
-			'gnome-keyring',
+			"gnome-keyring",
-			'gnome-terminal',
+			"gnome-terminal",
-			'engrampa',
+			"blueberry",
-			'gnome-screenshot',
+			"metacity"
-			'gvfs-smb',
-			'xed',
-			'xdg-user-dirs-gtk',
 		]
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType:
 		return GreeterType.Lightdm
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()

archinstall/default_profiles/desktops/cosmic.py

@@ -1,26 +0,0 @@
-from typing import override
-
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
-
-
-class CosmicProfile(Profile):
-	def __init__(self) -> None:
-		super().__init__(
-			'Cosmic',
-			ProfileType.DesktopEnv,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Wayland,
-		)
-
-	@property
-	@override
-	def packages(self) -> list[str]:
-		return [
-			'cosmic',
-			'xdg-user-dirs',
-		]
-
-	@property
-	@override
-	def default_greeter_type(self) -> GreeterType:
-		return GreeterType.CosmicSession

archinstall/default_profiles/desktops/cutefish.py

@@ -0,0 +1,31 @@
+from typing import Optional, List, Any, TYPE_CHECKING
+
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	from archinstall.lib.installer import Installer
+	_: Any
+
+
+class CutefishProfile(XorgProfile):
+	def __init__(self):
+		super().__init__('Cutefish', ProfileType.DesktopEnv, description='')
+
+	@property
+	def packages(self) -> List[str]:
+		return [
+			"cutefish",
+			"noto-fonts"
+		]
+
+	@property
+	def default_greeter_type(self) -> Optional[GreeterType]:
+		return GreeterType.Sddm
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()
+
+	def install(self, install_session: 'Installer'):
+		super().install(install_session)

archinstall/default_profiles/desktops/deepin.py

@@ -1,27 +1,28 @@
-from typing import override
+from typing import List, Optional, Any, TYPE_CHECKING
 
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	_: Any
 
 
-class DeepinProfile(Profile):
+class DeepinProfile(XorgProfile):
-	def __init__(self) -> None:
+	def __init__(self):
-		super().__init__(
+		super().__init__('Deepin', ProfileType.DesktopEnv, description='')
-			'Deepin',
-			ProfileType.DesktopEnv,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Xorg,
-		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return [
-			'deepin',
+			"deepin",
-			'deepin-terminal',
+			"deepin-terminal",
-			'deepin-editor',
+			"deepin-editor"
 		]
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType:
 		return GreeterType.Lightdm
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()

archinstall/default_profiles/desktops/enlightenment.py

@@ -1,26 +1,27 @@
-from typing import override
+from typing import List, Optional, Any, TYPE_CHECKING
 
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	_: Any
 
 
-class EnlightenmentProfile(Profile):
+class EnlighenmentProfile(XorgProfile):
-	def __init__(self) -> None:
+	def __init__(self):
-		super().__init__(
+		super().__init__('Enlightenment', ProfileType.WindowMgr, description='')
-			'Enlightenment',
-			ProfileType.WindowMgr,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Xorg,
-		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return [
-			'enlightenment',
+			"enlightenment",
-			'terminology',
+			"terminology"
 		]
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType:
 		return GreeterType.Lightdm
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()

archinstall/default_profiles/desktops/gnome.py

@@ -1,26 +1,27 @@
-from typing import override
+from typing import List, Optional, Any, TYPE_CHECKING
 
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	_: Any
 
 
-class GnomeProfile(Profile):
+class GnomeProfile(XorgProfile):
-	def __init__(self) -> None:
+	def __init__(self):
-		super().__init__(
+		super().__init__('Gnome', ProfileType.DesktopEnv, description='')
-			'GNOME',
-			ProfileType.DesktopEnv,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Wayland,
-		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return [
 			'gnome',
-			'gnome-tweaks',
+			'gnome-tweaks'
 		]
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType:
 		return GreeterType.Gdm
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()

archinstall/default_profiles/desktops/hyperland.py

@@ -0,0 +1,31 @@
+from typing import List, Optional, Any, TYPE_CHECKING
+
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	_: Any
+
+
+class HyperlandProfile(XorgProfile):
+	def __init__(self):
+		super().__init__('Hyperland', ProfileType.DesktopEnv, description='')
+
+	@property
+	def packages(self) -> List[str]:
+		return [
+			"hyprland",
+			"dunst",
+			"xdg-desktop-portal-hyprland",
+			"kitty",
+			"qt5-wayland",
+			"qt6-wayland"
+		]
+
+	@property
+	def default_greeter_type(self) -> Optional[GreeterType]:
+		return GreeterType.Sddm
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()

archinstall/default_profiles/desktops/hyprland.py

@@ -1,52 +0,0 @@
-from typing import override
-
-from archinstall.default_profiles.desktops.utils import select_seat_access
-from archinstall.default_profiles.profile import CustomSetting, DisplayServerType, GreeterType, Profile, ProfileType
-
-
-class HyprlandProfile(Profile):
-	def __init__(self) -> None:
-		super().__init__(
-			'Hyprland',
-			ProfileType.DesktopEnv,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Wayland,
-		)
-
-		self.custom_settings = {CustomSetting.SeatAccess: None}
-
-	@property
-	@override
-	def packages(self) -> list[str]:
-		return [
-			'hyprland',
-			'dunst',
-			'kitty',
-			'uwsm',
-			'dolphin',
-			'wofi',
-			'xdg-desktop-portal-hyprland',
-			'qt5-wayland',
-			'qt6-wayland',
-			'polkit-kde-agent',
-			'grim',
-			'slurp',
-		]
-
-	@property
-	@override
-	def default_greeter_type(self) -> GreeterType:
-		return GreeterType.Sddm
-
-	@property
-	@override
-	def services(self) -> list[str]:
-		if pref := self.custom_settings.get(CustomSetting.SeatAccess, None):
-			return [pref]
-		return []
-
-	@override
-	async def do_on_select(self) -> None:
-		default = self.custom_settings.get(CustomSetting.SeatAccess, None)
-		seat_access = await select_seat_access(self.name, default)
-		self.custom_settings[CustomSetting.SeatAccess] = seat_access.value

archinstall/default_profiles/desktops/i3.py

@@ -1,33 +1,33 @@
-from typing import override
+from typing import Optional, List, Any, TYPE_CHECKING
 
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	_: Any
 
 
-class I3wmProfile(Profile):
+class I3wmProfile(XorgProfile):
-	def __init__(self) -> None:
+	def __init__(self):
-		super().__init__(
+		super().__init__('i3-wm', ProfileType.WindowMgr, description='')
-			'i3-wm',
-			ProfileType.WindowMgr,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Xorg,
-		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return [
 			'i3-wm',
 			'i3lock',
 			'i3status',
 			'i3blocks',
-			'xss-lock',
 			'xterm',
 			'lightdm-gtk-greeter',
 			'lightdm',
-			'dmenu',
+			'dmenu'
 		]
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType:
 		return GreeterType.Lightdm
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()

archinstall/default_profiles/desktops/kde.py

@@ -0,0 +1,32 @@
+from typing import List, Optional, Any, TYPE_CHECKING
+
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	_: Any
+
+
+class KdeProfile(XorgProfile):
+	def __init__(self):
+		super().__init__('Kde', ProfileType.DesktopEnv, description='')
+
+	@property
+	def packages(self) -> List[str]:
+		return [
+			"plasma-meta",
+			"konsole",
+			"kwrite",
+			"dolphin",
+			"ark",
+			"plasma-wayland-session",
+			"egl-wayland"
+		]
+
+	@property
+	def default_greeter_type(self) -> Optional[GreeterType]:
+		return GreeterType.Sddm
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()

archinstall/default_profiles/desktops/labwc.py

@@ -1,46 +0,0 @@
-from typing import override
-
-from archinstall.default_profiles.desktops.utils import select_seat_access
-from archinstall.default_profiles.profile import CustomSetting, DisplayServerType, GreeterType, Profile, ProfileType
-
-
-class LabwcProfile(Profile):
-	def __init__(self) -> None:
-		super().__init__(
-			'Labwc',
-			ProfileType.WindowMgr,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Wayland,
-		)
-
-		self.custom_settings = {CustomSetting.SeatAccess: None}
-
-	@property
-	@override
-	def packages(self) -> list[str]:
-		additional = []
-		if seat := self.custom_settings.get(CustomSetting.SeatAccess, None):
-			additional = [seat]
-
-		return [
-			'alacritty',
-			'labwc',
-		] + additional
-
-	@property
-	@override
-	def default_greeter_type(self) -> GreeterType:
-		return GreeterType.Lightdm
-
-	@property
-	@override
-	def services(self) -> list[str]:
-		if pref := self.custom_settings.get(CustomSetting.SeatAccess, None):
-			return [pref]
-		return []
-
-	@override
-	async def do_on_select(self) -> None:
-		default = self.custom_settings.get(CustomSetting.SeatAccess, None)
-		seat_access = await select_seat_access(self.name, default)
-		self.custom_settings[CustomSetting.SeatAccess] = seat_access.value

archinstall/default_profiles/desktops/lxqt.py

@@ -1,34 +1,35 @@
-from typing import override
+from typing import List, Optional, Any, TYPE_CHECKING
 
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	_: Any
 
 
-class LxqtProfile(Profile):
+class LxqtProfile(XorgProfile):
-	def __init__(self) -> None:
+	def __init__(self):
-		super().__init__(
+		super().__init__('Lxqt', ProfileType.DesktopEnv, description='')
-			'Lxqt',
-			ProfileType.DesktopEnv,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Xorg,
-		)
 
 	# NOTE: SDDM is the only officially supported greeter for LXQt, so unlike other DEs, lightdm is not used here.
 	# LXQt works with lightdm, but since this is not supported, we will not default to this.
 	# https://github.com/lxqt/lxqt/issues/795
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return [
-			'lxqt',
+			"lxqt",
-			'breeze-icons',
+			"breeze-icons",
-			'oxygen-icons',
+			"oxygen-icons",
-			'xdg-utils',
+			"xdg-utils",
-			'ttf-freefont',
+			"ttf-freefont",
-			'l3afpad',
+			"leafpad",
-			'slock',
+			"slock"
 		]
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType:
 		return GreeterType.Sddm
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()

archinstall/default_profiles/desktops/mate.py

@@ -1,26 +1,27 @@
-from typing import override
+from typing import List, Optional, Any, TYPE_CHECKING
 
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	_: Any
 
 
-class MateProfile(Profile):
+class MateProfile(XorgProfile):
-	def __init__(self) -> None:
+	def __init__(self):
-		super().__init__(
+		super().__init__('Mate', ProfileType.DesktopEnv, description='')
-			'Mate',
-			ProfileType.DesktopEnv,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Xorg,
-		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return [
-			'mate',
+			"mate",
-			'mate-extra',
+			"mate-extra"
 		]
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType:
 		return GreeterType.Lightdm
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()

archinstall/default_profiles/desktops/niri.py

@@ -1,54 +0,0 @@
-from typing import override
-
-from archinstall.default_profiles.desktops.utils import select_seat_access
-from archinstall.default_profiles.profile import CustomSetting, DisplayServerType, GreeterType, Profile, ProfileType
-
-
-class NiriProfile(Profile):
-	def __init__(self) -> None:
-		super().__init__(
-			'niri',
-			ProfileType.WindowMgr,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Wayland,
-		)
-
-		self.custom_settings = {CustomSetting.SeatAccess: None}
-
-	@property
-	@override
-	def packages(self) -> list[str]:
-		additional = []
-		if seat := self.custom_settings.get(CustomSetting.SeatAccess, None):
-			additional = [seat]
-
-		return [
-			'niri',
-			'alacritty',
-			'fuzzel',
-			'mako',
-			'xorg-xwayland',
-			'waybar',
-			'swaybg',
-			'swayidle',
-			'swaylock',
-			'xdg-desktop-portal-gnome',
-		] + additional
-
-	@property
-	@override
-	def default_greeter_type(self) -> GreeterType:
-		return GreeterType.Lightdm
-
-	@property
-	@override
-	def services(self) -> list[str]:
-		if pref := self.custom_settings.get(CustomSetting.SeatAccess, None):
-			return [pref]
-		return []
-
-	@override
-	async def do_on_select(self) -> None:
-		default = self.custom_settings.get(CustomSetting.SeatAccess, None)
-		seat_access = await select_seat_access(self.name, default)
-		self.custom_settings[CustomSetting.SeatAccess] = seat_access.value

archinstall/default_profiles/desktops/niri_dms.py

@@ -1,66 +0,0 @@
-import shutil
-from pathlib import Path
-from typing import TYPE_CHECKING, override
-
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
-
-if TYPE_CHECKING:
-	from archinstall.lib.installer import Installer
-	from archinstall.lib.models.users import User
-
-
-_TERMINAL = 'alacritty'
-_ASSETS_DIR = Path(__file__).parent / 'niri_dms_assets'
-
-
-class NiriDmsProfile(Profile):
-	def __init__(self) -> None:
-		super().__init__(
-			'niri - DankMaterialShell',
-			ProfileType.WindowMgr,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Wayland,
-		)
-
-	@property
-	@override
-	def packages(self) -> list[str]:
-		return [
-			'niri',
-			'dms-shell-niri',
-			'polkit',
-			'xdg-desktop-portal-gnome',
-			'xorg-xwayland',
-			'matugen',
-			'cava',
-			'kimageformats',
-			'cups-pk-helper',
-			'tuned-ppd',
-			_TERMINAL,
-		]
-
-	@property
-	@override
-	def default_greeter_type(self) -> GreeterType:
-		return GreeterType.GreetdDms
-
-	@override
-	def provision(self, install_session: Installer, users: list[User]) -> None:
-		binds = (_ASSETS_DIR / 'dms/binds.kdl').read_text().replace('{{TERMINAL_COMMAND}}', _TERMINAL)
-
-		for user in users:
-			home = install_session.target / 'home' / user.username
-			niri_dir = home / '.config/niri'
-			dms_dir = niri_dir / 'dms'
-			dms_dir.mkdir(parents=True, exist_ok=True)
-
-			shutil.copy(_ASSETS_DIR / 'niri.kdl', niri_dir / 'config.kdl')
-			for name in ('colors.kdl', 'layout.kdl', 'alttab.kdl', 'outputs.kdl', 'cursor.kdl'):
-				shutil.copy(_ASSETS_DIR / 'dms' / name, dms_dir / name)
-			(dms_dir / 'binds.kdl').write_text(binds)
-
-			niri_unit_dropin = home / '.config/systemd/user/niri.service.d'
-			niri_unit_dropin.mkdir(parents=True, exist_ok=True)
-			(niri_unit_dropin / 'dms.conf').write_text('[Unit]\nWants=dms.service\n')
-
-			install_session.arch_chroot(f'chown -R {user.username}:{user.username} /home/{user.username}/.config')

archinstall/default_profiles/desktops/niri_dms_assets/dms/alttab.kdl

@@ -1,10 +0,0 @@
-// ! DO NOT EDIT !
-// ! AUTO-GENERATED BY DMS !
-// ! CHANGES WILL BE OVERWRITTEN !
-// ! PLACE YOUR CUSTOM CONFIGURATION ELSEWHERE !
-
-recent-windows {
-    highlight {
-        corner-radius 12
-    }
-}

archinstall/default_profiles/desktops/niri_dms_assets/dms/binds.kdl

@@ -1,221 +0,0 @@
-binds {
-    // === System & Overview ===
-    Mod+D repeat=false { toggle-overview; }
-    Mod+Tab repeat=false { toggle-overview; }
-    Mod+Shift+Slash { show-hotkey-overlay; }
-
-    // === Application Launchers ===
-    Mod+T hotkey-overlay-title="Open Terminal" { spawn "{{TERMINAL_COMMAND}}"; }
-    Mod+Space hotkey-overlay-title="Application Launcher" {
-        spawn "dms" "ipc" "call" "spotlight" "toggle";
-    }
-    Mod+V hotkey-overlay-title="Clipboard Manager" {
-        spawn "dms" "ipc" "call" "clipboard" "toggle";
-    }
-    Mod+M hotkey-overlay-title="Task Manager" {
-        spawn "dms" "ipc" "call" "processlist" "focusOrToggle";
-    }
-
-    Super+X hotkey-overlay-title="Power Menu: Toggle" { spawn "dms" "ipc" "call" "powermenu" "toggle"; }
-    Mod+Comma hotkey-overlay-title="Settings" {
-        spawn "dms" "ipc" "call" "settings" "focusOrToggle";
-    }
-    Mod+Y hotkey-overlay-title="Browse Wallpapers" {
-        spawn "dms" "ipc" "call" "dankdash" "wallpaper";
-    }
-    Mod+N hotkey-overlay-title="Notification Center" { spawn "dms" "ipc" "call" "notifications" "toggle"; }
-    Mod+Shift+N hotkey-overlay-title="Notepad" { spawn "dms" "ipc" "call" "notepad" "toggle"; }
-
-    // === Security ===
-    Mod+Alt+L hotkey-overlay-title="Lock Screen" {
-        spawn "dms" "ipc" "call" "lock" "lock";
-    }
-    Mod+Shift+E { quit; }
-    Ctrl+Alt+Delete hotkey-overlay-title="Task Manager" {
-        spawn "dms" "ipc" "call" "processlist" "focusOrToggle";
-    }
-
-    // === Audio Controls ===
-    XF86AudioRaiseVolume allow-when-locked=true {
-        spawn "dms" "ipc" "call" "audio" "increment" "3";
-    }
-    XF86AudioLowerVolume allow-when-locked=true {
-        spawn "dms" "ipc" "call" "audio" "decrement" "3";
-    }
-    XF86AudioMute allow-when-locked=true {
-        spawn "dms" "ipc" "call" "audio" "mute";
-    }
-    XF86AudioMicMute allow-when-locked=true {
-        spawn "dms" "ipc" "call" "audio" "micmute";
-    }
-		XF86AudioPause allow-when-locked=true {
-		    spawn "dms" "ipc" "call" "mpris" "playPause";
-		}
-		XF86AudioPlay allow-when-locked=true {
-		    spawn "dms" "ipc" "call" "mpris" "playPause";
-		}
-		XF86AudioPrev allow-when-locked=true {
-		    spawn "dms" "ipc" "call" "mpris" "previous";
-		}
-		XF86AudioNext allow-when-locked=true {
-		    spawn "dms" "ipc" "call" "mpris" "next";
-		}
-        Ctrl+XF86AudioRaiseVolume allow-when-locked=true {
-            spawn "dms" "ipc" "call" "mpris" "increment" "3";
-        }
-        Ctrl+XF86AudioLowerVolume allow-when-locked=true {
-            spawn "dms" "ipc" "call" "mpris" "decrement" "3";
-        }
-
-    // === Brightness Controls ===
-    XF86MonBrightnessUp allow-when-locked=true {
-       spawn "dms" "ipc" "call" "brightness" "increment" "5" "";
-    }
-    XF86MonBrightnessDown allow-when-locked=true {
-       spawn "dms" "ipc" "call" "brightness" "decrement" "5" "";
-    }
-
-    // === Window Management ===
-    Mod+Q repeat=false { close-window; }
-    Mod+F { maximize-column; }
-    Mod+Shift+F { fullscreen-window; }
-    Mod+Shift+T { toggle-window-floating; }
-    Mod+Shift+V { switch-focus-between-floating-and-tiling; }
-    Mod+W { toggle-column-tabbed-display; }
-    Mod+Shift+W hotkey-overlay-title="Create window rule" { spawn "dms" "ipc" "call" "window-rules" "toggle"; }
-
-    // === Focus Navigation ===
-    Mod+Left  { focus-column-left; }
-    Mod+Down  { focus-window-down; }
-    Mod+Up    { focus-window-up; }
-    Mod+Right { focus-column-right; }
-    Mod+H     { focus-column-left; }
-    Mod+J     { focus-window-down; }
-    Mod+K     { focus-window-up; }
-    Mod+L     { focus-column-right; }
-
-    // === Window Movement ===
-    Mod+Shift+Left  { move-column-left; }
-    Mod+Shift+Down  { move-window-down; }
-    Mod+Shift+Up    { move-window-up; }
-    Mod+Shift+Right { move-column-right; }
-    Mod+Shift+H     { move-column-left; }
-    Mod+Shift+J     { move-window-down; }
-    Mod+Shift+K     { move-window-up; }
-    Mod+Shift+L     { move-column-right; }
-
-    // === Column Navigation ===
-    Mod+Home { focus-column-first; }
-    Mod+End  { focus-column-last; }
-    Mod+Ctrl+Home { move-column-to-first; }
-    Mod+Ctrl+End  { move-column-to-last; }
-
-    // === Monitor Navigation ===
-    Mod+Ctrl+Left  { focus-monitor-left; }
-    //Mod+Ctrl+Down  { focus-monitor-down; }
-    //Mod+Ctrl+Up    { focus-monitor-up; }
-    Mod+Ctrl+Right { focus-monitor-right; }
-    Mod+Ctrl+H     { focus-monitor-left; }
-    Mod+Ctrl+J     { focus-monitor-down; }
-    Mod+Ctrl+K     { focus-monitor-up; }
-    Mod+Ctrl+L     { focus-monitor-right; }
-
-    // === Move to Monitor ===
-    Mod+Shift+Ctrl+Left  { move-column-to-monitor-left; }
-    Mod+Shift+Ctrl+Down  { move-column-to-monitor-down; }
-    Mod+Shift+Ctrl+Up    { move-column-to-monitor-up; }
-    Mod+Shift+Ctrl+Right { move-column-to-monitor-right; }
-    Mod+Shift+Ctrl+H     { move-column-to-monitor-left; }
-    Mod+Shift+Ctrl+J     { move-column-to-monitor-down; }
-    Mod+Shift+Ctrl+K     { move-column-to-monitor-up; }
-    Mod+Shift+Ctrl+L     { move-column-to-monitor-right; }
-
-    // === Workspace Navigation ===
-    Mod+Page_Down      { focus-workspace-down; }
-    Mod+Page_Up        { focus-workspace-up; }
-    Mod+U              { focus-workspace-down; }
-    Mod+I              { focus-workspace-up; }
-    Mod+Ctrl+Down { move-column-to-workspace-down; }
-    Mod+Ctrl+Up   { move-column-to-workspace-up; }
-    Mod+Ctrl+U         { move-column-to-workspace-down; }
-    Mod+Ctrl+I         { move-column-to-workspace-up; }
-
-    // === Workspace Management ===
-    Ctrl+Shift+R hotkey-overlay-title="Rename Workspace" {
-        spawn "dms" "ipc" "call" "workspace-rename" "open";
-    }
-
-    // === Move Workspaces ===
-    Mod+Shift+Page_Down { move-workspace-down; }
-    Mod+Shift+Page_Up   { move-workspace-up; }
-    Mod+Shift+U         { move-workspace-down; }
-    Mod+Shift+I         { move-workspace-up; }
-
-    // === Mouse Wheel Navigation ===
-    Mod+WheelScrollDown      cooldown-ms=150 { focus-workspace-down; }
-    Mod+WheelScrollUp        cooldown-ms=150 { focus-workspace-up; }
-    Mod+Ctrl+WheelScrollDown cooldown-ms=150 { move-column-to-workspace-down; }
-    Mod+Ctrl+WheelScrollUp   cooldown-ms=150 { move-column-to-workspace-up; }
-
-    Mod+WheelScrollRight      { focus-column-right; }
-    Mod+WheelScrollLeft       { focus-column-left; }
-    Mod+Ctrl+WheelScrollRight { move-column-right; }
-    Mod+Ctrl+WheelScrollLeft  { move-column-left; }
-
-    Mod+Shift+WheelScrollDown      { focus-column-right; }
-    Mod+Shift+WheelScrollUp        { focus-column-left; }
-    Mod+Ctrl+Shift+WheelScrollDown { move-column-right; }
-    Mod+Ctrl+Shift+WheelScrollUp   { move-column-left; }
-
-    // === Numbered Workspaces ===
-    Mod+1 { focus-workspace 1; }
-    Mod+2 { focus-workspace 2; }
-    Mod+3 { focus-workspace 3; }
-    Mod+4 { focus-workspace 4; }
-    Mod+5 { focus-workspace 5; }
-    Mod+6 { focus-workspace 6; }
-    Mod+7 { focus-workspace 7; }
-    Mod+8 { focus-workspace 8; }
-    Mod+9 { focus-workspace 9; }
-
-    // === Move to Numbered Workspaces ===
-    Mod+Shift+1 { move-column-to-workspace 1; }
-    Mod+Shift+2 { move-column-to-workspace 2; }
-    Mod+Shift+3 { move-column-to-workspace 3; }
-    Mod+Shift+4 { move-column-to-workspace 4; }
-    Mod+Shift+5 { move-column-to-workspace 5; }
-    Mod+Shift+6 { move-column-to-workspace 6; }
-    Mod+Shift+7 { move-column-to-workspace 7; }
-    Mod+Shift+8 { move-column-to-workspace 8; }
-    Mod+Shift+9 { move-column-to-workspace 9; }
-
-    // === Column Management ===
-    Mod+BracketLeft  { consume-or-expel-window-left; }
-    Mod+BracketRight { consume-or-expel-window-right; }
-    Mod+Period { expel-window-from-column; }
-
-    // === Sizing & Layout ===
-    Mod+R { switch-preset-column-width; }
-    Mod+Shift+R { switch-preset-window-height; }
-    Mod+Ctrl+R { reset-window-height; }
-    Mod+Ctrl+F { expand-column-to-available-width; }
-    Mod+C { center-column; }
-    Mod+Ctrl+C { center-visible-columns; }
-
-    // === Manual Sizing ===
-    Mod+Minus { set-column-width "-10%"; }
-    Mod+Equal { set-column-width "+10%"; }
-    Mod+Shift+Minus { set-window-height "-10%"; }
-    Mod+Shift+Equal { set-window-height "+10%"; }
-
-    // === Screenshots ===
-    XF86Launch1 { screenshot; }
-    Ctrl+XF86Launch1 { screenshot-screen; }
-    Alt+XF86Launch1 { screenshot-window; }
-    Print { screenshot; }
-    Ctrl+Print { screenshot-screen; }
-    Alt+Print { screenshot-window; }
-    // === System Controls ===
-    Mod+Escape allow-inhibiting=false { toggle-keyboard-shortcuts-inhibit; }
-    Mod+Shift+P { power-off-monitors; }
-}

archinstall/default_profiles/desktops/niri_dms_assets/dms/colors.kdl

@@ -1,39 +0,0 @@
-// ! Auto-generated file. Do not edit directly.
-// Remove `include "dms/colors.kdl"` from your config to override.
-
-layout {
-    background-color "transparent"
-
-    focus-ring {
-        active-color   "#d0bcff"
-        inactive-color "#948f99"
-        urgent-color   "#f2b8b5"
-    }
-
-    border {
-        active-color   "#d0bcff"
-        inactive-color "#948f99"
-        urgent-color   "#f2b8b5"
-    }
-
-    shadow {
-        color "#00000070"
-    }
-
-    tab-indicator {
-        active-color   "#d0bcff"
-        inactive-color "#948f99"
-        urgent-color   "#f2b8b5"
-    }
-
-    insert-hint {
-        color "#d0bcff80"
-    }
-}
-
-recent-windows {
-    highlight {
-        active-color   "#4f378b"
-        urgent-color   "#f2b8b5"
-    }
-}

archinstall/default_profiles/desktops/niri_dms_assets/dms/cursor.kdl

@@ -1,6 +0,0 @@
-// Place cursor configuration here.
-// Example:
-// cursor {
-//     xcursor-theme "Adwaita"
-//     xcursor-size 24
-// }

archinstall/default_profiles/desktops/niri_dms_assets/dms/layout.kdl

@@ -1,22 +0,0 @@
-// ! DO NOT EDIT !
-// ! AUTO-GENERATED BY DMS !
-// ! CHANGES WILL BE OVERWRITTEN !
-// ! PLACE YOUR CUSTOM CONFIGURATION ELSEWHERE !
-
-layout {
-    gaps 4
-
-    border {
-        width 2
-    }
-
-    focus-ring {
-        width 2
-    }
-}
-window-rule {
-    geometry-corner-radius 12
-    clip-to-geometry true
-    tiled-state true
-    draw-border-with-background false
-}

archinstall/default_profiles/desktops/niri_dms_assets/dms/outputs.kdl

@@ -1,7 +0,0 @@
-// Place per-output configuration here.
-// Example:
-// output "DP-1" {
-//     mode "2560x1440@165"
-//     position x=0 y=0
-//     scale 1
-// }

archinstall/default_profiles/desktops/niri_dms_assets/niri.kdl

@@ -1,279 +0,0 @@
-// This config is in the KDL format: https://kdl.dev
-// "/-" comments out the following node.
-// Check the wiki for a full description of the configuration:
-// https://github.com/YaLTeR/niri/wiki/Configuration:-Introduction
-config-notification {
-    disable-failed
-}
-
-gestures {
-    hot-corners {
-        off
-    }
-}
-
-// Input device configuration.
-// Find the full list of options on the wiki:
-// https://github.com/YaLTeR/niri/wiki/Configuration:-Input
-input {
-    keyboard {
-        xkb {
-            // You can set rules, model, layout, variant and options.
-            // For more information, see xkeyboard-config(7).
-
-            // For example:
-            // layout "us,ru"
-            // options "grp:win_space_toggle,compose:ralt,ctrl:nocaps"
-
-            // If this section is empty, niri will fetch xkb settings
-            // from org.freedesktop.locale1. You can control these using
-            // localectl set-x11-keymap.
-        }
-
-        // Enable numlock on startup, omitting this setting disables it.
-        numlock
-    }
-
-    // Next sections include libinput settings.
-    // Omitting settings disables them, or leaves them at their default values.
-    // All commented-out settings here are examples, not defaults.
-    touchpad {
-        // off
-        tap
-        // dwt
-        // dwtp
-        // drag false
-        // drag-lock
-        natural-scroll
-        // accel-speed 0.2
-        // accel-profile "flat"
-        // scroll-method "two-finger"
-        // disabled-on-external-mouse
-    }
-
-    mouse {
-        // off
-        // natural-scroll
-        // accel-speed 0.2
-        // accel-profile "flat"
-        // scroll-method "no-scroll"
-    }
-
-    trackpoint {
-        // off
-        // natural-scroll
-        // accel-speed 0.2
-        // accel-profile "flat"
-        // scroll-method "on-button-down"
-        // scroll-button 273
-        // scroll-button-lock
-        // middle-emulation
-    }
-
-    // Uncomment this to make the mouse warp to the center of newly focused windows.
-    // warp-mouse-to-focus
-
-    // Focus windows and outputs automatically when moving the mouse into them.
-    // Setting max-scroll-amount="0%" makes it work only on windows already fully on screen.
-    // focus-follows-mouse max-scroll-amount="0%"
-}
-// You can configure outputs by their name, which you can find
-// by running `niri msg outputs` while inside a niri instance.
-// The built-in laptop monitor is usually called "eDP-1".
-// Find more information on the wiki:
-// https://github.com/YaLTeR/niri/wiki/Configuration:-Outputs
-// Remember to uncomment the node by removing "/-"!
-/-output "eDP-2" {
-    mode "2560x1600@239.998993"
-    position x=2560 y=0
-    variable-refresh-rate
-}
-// Settings that influence how windows are positioned and sized.
-// Find more information on the wiki:
-// https://github.com/YaLTeR/niri/wiki/Configuration:-Layout
-layout {
-    // Set gaps around windows in logical pixels.
-    background-color "transparent"
-    // When to center a column when changing focus, options are:
-    // - "never", default behavior, focusing an off-screen column will keep at the left
-    //   or right edge of the screen.
-    // - "always", the focused column will always be centered.
-    // - "on-overflow", focusing a column will center it if it doesn't fit
-    //   together with the previously focused column.
-    center-focused-column "never"
-    // You can customize the widths that "switch-preset-column-width" (Mod+R) toggles between.
-    preset-column-widths {
-        // Proportion sets the width as a fraction of the output width, taking gaps into account.
-        // For example, you can perfectly fit four windows sized "proportion 0.25" on an output.
-        // The default preset widths are 1/3, 1/2 and 2/3 of the output.
-        proportion 0.33333
-        proportion 0.5
-        proportion 0.66667
-        // Fixed sets the width in logical pixels exactly.
-        // fixed 1920
-    }
-    // You can also customize the heights that "switch-preset-window-height" (Mod+Shift+R) toggles between.
-    // preset-window-heights { }
-    // You can change the default width of the new windows.
-    default-column-width { proportion 0.5; }
-    // If you leave the brackets empty, the windows themselves will decide their initial width.
-    // default-column-width {}
-    // By default focus ring and border are rendered as a solid background rectangle
-    // behind windows. That is, they will show up through semitransparent windows.
-    // This is because windows using client-side decorations can have an arbitrary shape.
-    //
-    // If you don't like that, you should uncomment `prefer-no-csd` below.
-    // Niri will draw focus ring and border *around* windows that agree to omit their
-    // client-side decorations.
-    //
-    // Alternatively, you can override it with a window rule called
-    // `draw-border-with-background`.
-    border {
-        off
-        width 4
-        active-color   "#707070"      // Neutral gray
-        inactive-color "#d0d0d0"      // Light gray
-        urgent-color   "#cc4444"      // Softer red
-    }
-    shadow {
-        softness 30
-        spread 5
-        offset x=0 y=5
-        color "#0007"
-    }
-    struts {
-    }
-}
-layer-rule {
-    match namespace="^quickshell$"
-    place-within-backdrop true
-}
-overview {
-    workspace-shadow {
-        off
-    }
-}
-// Add lines like this to spawn processes at startup.
-// Note that running niri as a session supports xdg-desktop-autostart,
-// which may be more convenient to use.
-// See the binds section below for more spawn examples.
-// This line starts waybar, a commonly used bar for Wayland compositors.
-environment {
-  XDG_CURRENT_DESKTOP "niri"
-}
-hotkey-overlay {
-    skip-at-startup
-}
-prefer-no-csd
-screenshot-path "~/Pictures/Screenshots/Screenshot from %Y-%m-%d %H-%M-%S.png"
-animations {
-    workspace-switch {
-        spring damping-ratio=0.80 stiffness=523 epsilon=0.0001
-    }
-    window-open {
-        duration-ms 150
-        curve "ease-out-expo"
-    }
-    window-close {
-        duration-ms 150
-        curve "ease-out-quad"
-    }
-    horizontal-view-movement {
-        spring damping-ratio=0.85 stiffness=423 epsilon=0.0001
-    }
-    window-movement {
-        spring damping-ratio=0.75 stiffness=323 epsilon=0.0001
-    }
-    window-resize {
-        spring damping-ratio=0.85 stiffness=423 epsilon=0.0001
-    }
-    config-notification-open-close {
-        spring damping-ratio=0.65 stiffness=923 epsilon=0.001
-    }
-    screenshot-ui-open {
-        duration-ms 200
-        curve "ease-out-quad"
-    }
-    overview-open-close {
-        spring damping-ratio=0.85 stiffness=800 epsilon=0.0001
-    }
-}
-// Window rules let you adjust behavior for individual windows.
-// Find more information on the wiki:
-// https://github.com/YaLTeR/niri/wiki/Configuration:-Window-Rules
-// Work around WezTerm's initial configure bug
-// by setting an empty default-column-width.
-window-rule {
-    // This regular expression is intentionally made as specific as possible,
-    // since this is the default config, and we want no false positives.
-    // You can get away with just app-id="wezterm" if you want.
-    match app-id=r#"^org\.wezfurlong\.wezterm$"#
-    default-column-width {}
-}
-window-rule {
-    match app-id=r#"^org\.gnome\."#
-    draw-border-with-background false
-    geometry-corner-radius 12
-    clip-to-geometry true
-}
-window-rule {
-    match app-id=r#"^gnome-control-center$"#
-    match app-id=r#"^pavucontrol$"#
-    match app-id=r#"^nm-connection-editor$"#
-    default-column-width { proportion 0.5; }
-    open-floating false
-}
-window-rule {
-    match app-id=r#"^org\.gnome\.Calculator$"#
-    match app-id=r#"^gnome-calculator$"#
-    match app-id=r#"^galculator$"#
-    match app-id=r#"^blueman-manager$"#
-    match app-id=r#"^org\.gnome\.Nautilus$"#
-    match app-id=r#"^xdg-desktop-portal$"#
-    open-floating true
-}
-window-rule {
-    match app-id=r#"^steam$"# title=r#"^notificationtoasts_\d+_desktop$"#
-    default-floating-position x=10 y=10 relative-to="bottom-right"
-    open-focused false
-}
-window-rule {
-    match app-id=r#"^org\.wezfurlong\.wezterm$"#
-    match app-id="Alacritty"
-    match app-id="zen"
-    match app-id="com.mitchellh.ghostty"
-    match app-id="kitty"
-    draw-border-with-background false
-}
-window-rule {
-    match app-id=r#"firefox$"# title="^Picture-in-Picture$"
-    match app-id="zoom"
-    open-floating true
-}
-// Open dms windows as floating by default
-window-rule {
-    match app-id=r#"org.quickshell$"#
-    match app-id=r#"com.danklinux.dms$"#
-    open-floating true
-}
-debug {
-    honor-xdg-activation-with-invalid-serial
-}
-
-// Override to disable super+tab
-recent-windows {
-    binds {
-        Alt+Tab         { next-window scope="output"; }
-        Alt+Shift+Tab   { previous-window scope="output"; }
-        Alt+grave       { next-window filter="app-id"; }
-        Alt+Shift+grave { previous-window filter="app-id"; }
-    }
-}
-
-// Include dms files
-include "dms/colors.kdl"
-include "dms/layout.kdl"
-include "dms/alttab.kdl"
-include "dms/binds.kdl"
-include "dms/outputs.kdl"
-include "dms/cursor.kdl"

archinstall/default_profiles/desktops/plasma.py

@@ -1,120 +0,0 @@
-from enum import StrEnum
-from typing import override
-
-from archinstall.default_profiles.profile import CustomSetting, DisplayServerType, GreeterType, Profile, ProfileType
-from archinstall.lib.menu.helpers import Selection
-from archinstall.lib.packages.packages import available_package, package_group_info
-from archinstall.lib.translationhandler import tr
-from archinstall.tui.menu_item import MenuItem, MenuItemGroup
-from archinstall.tui.result import ResultType
-
-
-class PlasmaFlavor(StrEnum):
-	Meta = 'plasma-meta'
-	Plasma = 'plasma'
-	Desktop = 'plasma-desktop'
-
-	def show(self) -> str:
-		match self:
-			case PlasmaFlavor.Meta:
-				return f'{self.value} ({tr("Recommended")})'
-			case PlasmaFlavor.Plasma | PlasmaFlavor.Desktop:
-				return self.value
-
-	def package_details(self) -> str:
-		ty = ''
-		details = ''
-		desc = ''
-
-		match self:
-			case PlasmaFlavor.Meta:
-				ty = tr('Package')
-				desc = tr('Curated selection of KDE Plasma packages')
-				info = available_package(self.value)
-
-				if info is not None:
-					details = tr('Dependencies') + '\n'
-					details += '\n'.join(f'- {entry}' for entry in info.get_depends_on)
-			case PlasmaFlavor.Plasma:
-				ty = tr('Package group')
-				desc = tr('Extensive KDE Plasma installation')
-				group = package_group_info(self.value)
-
-				if group is not None:
-					details = tr('Packages in group') + '\n'
-					details += '\n'.join(f'- {entry}' for entry in group.packages)
-			case PlasmaFlavor.Desktop:
-				ty = tr('Package group')
-				desc = tr('Minimal KDE Plasma installation')
-				info = available_package(self.value)
-
-				if info is not None:
-					details = tr('Dependencies') + '\n'
-					details += '\n'.join(f'- {entry}' for entry in info.get_depends_on)
-
-		return f'{tr("Type")}: {ty}\n{tr("Description")}: {desc}\n\n{details}'
-
-	def packages(self) -> list[str]:
-		match self:
-			case PlasmaFlavor.Meta:
-				return ['plasma-meta']
-			case PlasmaFlavor.Plasma:
-				return ['plasma']
-			case PlasmaFlavor.Desktop:
-				return ['plasma-desktop']
-
-
-class PlasmaProfile(Profile):
-	def __init__(self) -> None:
-		super().__init__(
-			'KDE Plasma',
-			ProfileType.DesktopEnv,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Wayland,
-		)
-
-	@property
-	@override
-	def packages(self) -> list[str]:
-		flavor_str = self.custom_settings.get(CustomSetting.PlasmaFlavor)
-
-		if flavor_str is not None:
-			flavor = PlasmaFlavor(flavor_str)
-			return flavor.packages()
-		else:
-			return PlasmaFlavor.Meta.packages()  # use plasma-meta as the recommended default
-
-	@property
-	@override
-	def default_greeter_type(self) -> GreeterType:
-		return GreeterType.PlasmaLoginManager
-
-	async def _select_flavor(self) -> None:
-		header = tr('Select a flavor of KDE Plasma to install') + '\n'
-
-		items = [
-			MenuItem(
-				s.show(),
-				value=s,
-				preview_action=lambda x: x.value.package_details() if x.value else None,
-			)
-			for s in PlasmaFlavor
-		]
-		group = MenuItemGroup(items, sort_items=False)
-
-		default = self.custom_settings.get(CustomSetting.PlasmaFlavor, None)
-		group.set_default_by_value(default)
-
-		result = await Selection[PlasmaFlavor](
-			group,
-			header=header,
-			allow_skip=False,
-			preview_location='right',
-		).show()
-
-		if result.type_ == ResultType.Selection:
-			self.custom_settings[CustomSetting.PlasmaFlavor] = result.get_value().value
-
-	@override
-	async def do_on_select(self) -> None:
-		await self._select_flavor()

archinstall/default_profiles/desktops/qtile.py

@@ -1,26 +1,27 @@
-from typing import override
+from typing import Optional, List, Any, TYPE_CHECKING
 
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	_: Any
 
 
-class QtileProfile(Profile):
+class QtileProfile(XorgProfile):
-	def __init__(self) -> None:
+	def __init__(self):
-		super().__init__(
+		super().__init__('Qtile', ProfileType.WindowMgr, description='')
-			'Qtile',
-			ProfileType.WindowMgr,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Xorg,
-		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return [
 			'qtile',
-			'alacritty',
+			'alacritty'
 		]
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType:
 		return GreeterType.Lightdm
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()

archinstall/default_profiles/desktops/river.py

@@ -1,27 +0,0 @@
-from typing import override
-
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
-
-
-class RiverProfile(Profile):
-	def __init__(self) -> None:
-		super().__init__(
-			'River',
-			ProfileType.WindowMgr,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Wayland,
-		)
-
-	@property
-	@override
-	def packages(self) -> list[str]:
-		return [
-			'foot',
-			'xdg-desktop-portal-wlr',
-			'river',
-		]
-
-	@property
-	@override
-	def default_greeter_type(self) -> GreeterType:
-		return GreeterType.Lightdm

archinstall/default_profiles/desktops/sway.py

@@ -1,56 +1,81 @@
-from typing import override
+from enum import Enum
+from typing import List, Optional, TYPE_CHECKING, Any
 
-from archinstall.default_profiles.desktops.utils import select_seat_access
+from archinstall.default_profiles.profile import ProfileType, GreeterType
-from archinstall.default_profiles.profile import CustomSetting, DisplayServerType, GreeterType, Profile, ProfileType
+from archinstall.default_profiles.xorg import XorgProfile
+from archinstall.lib.menu import Menu
+
+if TYPE_CHECKING:
+	from archinstall.lib.installer import Installer
+	_: Any
 
 
-class SwayProfile(Profile):
+class SeatAccess(Enum):
-	def __init__(self) -> None:
+	seatd = 'seatd'
+	polkit = 'polkit'
+
+
+class SwayProfile(XorgProfile):
+	def __init__(self):
 		super().__init__(
 			'Sway',
 			ProfileType.WindowMgr,
-			support_gfx_driver=True,
+			description=''
-			display_server=DisplayServerType.Wayland,
 		)
 
-		self.custom_settings = {CustomSetting.SeatAccess: None}
+		self.custom_settings = {'seat_access': None}
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		additional = []
-		if seat := self.custom_settings.get(CustomSetting.SeatAccess, None):
+		if seat := self.custom_settings.get('seat_access', None):
 			additional = [seat]
 
 		return [
-			'sway',
+			"sway",
-			'swaybg',
+			"swaybg",
-			'swaylock',
+			"swaylock",
-			'swayidle',
+			"swayidle",
-			'waybar',
+			"waybar",
-			'wmenu',
+			"dmenu",
-			'brightnessctl',
+			"brightnessctl",
-			'grim',
+			"grim",
-			'slurp',
+			"slurp",
-			'pavucontrol',
+			"pavucontrol",
-			'foot',
+			"foot",
-			'xorg-xwayland',
+			"xorg-xwayland"
 		] + additional
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType:
 		return GreeterType.Lightdm
 
 	@property
-	@override
+	def services(self) -> List[str]:
-	def services(self) -> list[str]:
+		if pref := self.custom_settings.get('seat_access', None):
-		if pref := self.custom_settings.get(CustomSetting.SeatAccess, None):
 			return [pref]
 		return []
 
-	@override
+	def _ask_seat_access(self):
-	async def do_on_select(self) -> None:
+		# need to activate seat service and add to seat group
-		default = self.custom_settings.get(CustomSetting.SeatAccess, None)
+		title = str(_('Sway needs access to your seat (collection of hardware devices i.e. keyboard, mouse, etc)'))
-		seat_access = await select_seat_access(self.name, default)
+		title += str(_('\n\nChoose an option to give Sway access to your hardware'))
-		self.custom_settings[CustomSetting.SeatAccess] = seat_access.value
+
+		options = [e.value for e in SeatAccess]
+		default = None
+
+		if seat := self.custom_settings.get('seat_access', None):
+			default = seat
+
+		choice = Menu(title, options, skip=False, preset_values=default).run()
+		self.custom_settings['seat_access'] = choice.single_value
+
+	def do_on_select(self):
+		self._ask_seat_access()
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()
+
+	def install(self, install_session: 'Installer'):
+		super().install(install_session)

archinstall/default_profiles/desktops/utils.py

@@ -1,45 +0,0 @@
-from enum import Enum
-
-from archinstall.lib.installer import Installer
-from archinstall.lib.menu.helpers import Selection
-from archinstall.lib.models.users import User
-from archinstall.lib.translationhandler import tr
-from archinstall.tui.menu_item import MenuItem, MenuItemGroup
-from archinstall.tui.result import ResultType
-
-
-class SeatAccess(Enum):
-	seatd = 'seatd'
-	polkit = 'polkit'
-
-
-def provision_seat_access(
-	install_session: Installer,
-	users: list[User],
-	seat_access: str,
-) -> None:
-	if seat_access == SeatAccess.seatd.value:
-		for user in users:
-			install_session.arch_chroot(f'usermod -a -G seat {user.username}')
-
-
-async def select_seat_access(profile_name: str, default: str | None) -> SeatAccess:
-	header = tr('{} needs access to your seat').format(profile_name)
-	header += f' ({tr("collection of hardware devices i.e. keyboard, mouse")})' + '\n'
-	header += tr('Choose an option how to give {} access to your hardware').format(profile_name)
-
-	items = [MenuItem(s.value, value=s) for s in SeatAccess]
-	group = MenuItemGroup(items, sort_items=True)
-
-	group.set_default_by_value(default)
-
-	result = await Selection[SeatAccess](
-		group,
-		header=header,
-		allow_skip=False,
-	).show()
-
-	if result.type_ == ResultType.Selection:
-		return result.get_value()
-	else:
-		raise ValueError('Unexpected result type from seat access selection')

archinstall/default_profiles/desktops/xfce4.py

@@ -1,29 +1,30 @@
-from typing import override
+from typing import List, Optional, Any, TYPE_CHECKING
 
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
+from archinstall.default_profiles.profile import ProfileType, GreeterType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	_: Any
 
 
-class Xfce4Profile(Profile):
+class Xfce4Profile(XorgProfile):
-	def __init__(self) -> None:
+	def __init__(self):
-		super().__init__(
+		super().__init__('Xfce4', ProfileType.DesktopEnv, description='')
-			'Xfce4',
-			ProfileType.DesktopEnv,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Xorg,
-		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return [
-			'xfce4',
+			"xfce4",
-			'xfce4-goodies',
+			"xfce4-goodies",
-			'pavucontrol',
+			"pavucontrol",
-			'gvfs',
+			"gvfs",
-			'xarchiver',
+			"xarchiver"
 		]
 
 	@property
-	@override
+	def default_greeter_type(self) -> Optional[GreeterType]:
-	def default_greeter_type(self) -> GreeterType:
 		return GreeterType.Lightdm
+
+	def preview_text(self) -> Optional[str]:
+		text = str(_('Environment type: {}')).format(self.profile_type.value)
+		return text + '\n' + self.packages_text()

archinstall/default_profiles/desktops/xmonad.py

@@ -1,29 +0,0 @@
-from typing import override
-
-from archinstall.default_profiles.profile import DisplayServerType, GreeterType, Profile, ProfileType
-
-
-class XmonadProfile(Profile):
-	def __init__(self) -> None:
-		super().__init__(
-			'Xmonad',
-			ProfileType.WindowMgr,
-			support_gfx_driver=True,
-			display_server=DisplayServerType.Xorg,
-		)
-
-	@property
-	@override
-	def packages(self) -> list[str]:
-		return [
-			'xmonad',
-			'xmonad-contrib',
-			'xmonad-extras',
-			'xterm',
-			'dmenu',
-		]
-
-	@property
-	@override
-	def default_greeter_type(self) -> GreeterType:
-		return GreeterType.Lightdm

archinstall/default_profiles/minimal.py

@@ -1,9 +1,15 @@
+from typing import Any, TYPE_CHECKING
+
 from archinstall.default_profiles.profile import Profile, ProfileType
 
+if TYPE_CHECKING:
+	_: Any
+
 
 class MinimalProfile(Profile):
-	def __init__(self) -> None:
+	def __init__(self):
 		super().__init__(
 			'Minimal',
 			ProfileType.Minimal,
+			description=str(_('A very basic installation that allows you to customize Arch Linux as you see fit.'))
 		)

archinstall/default_profiles/profile.py

@@ -1,16 +1,16 @@
-from enum import Enum, StrEnum, auto
+from __future__ import annotations
-from typing import TYPE_CHECKING, Self
 
-from archinstall.lib.translationhandler import tr
+from enum import Enum, auto
+from typing import List, Optional, Any, Dict, TYPE_CHECKING, TypeVar
+
+from archinstall.lib.utils.util import format_cols
 
 if TYPE_CHECKING:
 	from archinstall.lib.installer import Installer
-	from archinstall.lib.models.users import User
+	_: Any
 
 
-class DisplayServerType(Enum):
+TProfile = TypeVar('TProfile', bound='Profile')
-	Xorg = 'Xorg'
-	Wayland = 'Wayland'
 
 
 class ProfileType(Enum):
@@ -26,18 +26,14 @@ class ProfileType(Enum):
 	DesktopEnv = 'Desktop Environment'
 	CustomType = 'CustomType'
 	# special things
+	Tailored = 'Tailored'
 	Application = 'Application'
 
 
 class GreeterType(Enum):
-	Lightdm = 'lightdm-gtk-greeter'
+	Lightdm = 'lightdm'
-	LightdmSlick = 'lightdm-slick-greeter'
 	Sddm = 'sddm'
 	Gdm = 'gdm'
-	Ly = 'ly'
-	CosmicSession = 'cosmic-greeter'
-	PlasmaLoginManager = 'plasma-login-manager'
-	GreetdDms = 'dms-greeter'
 
 
 class SelectResult(Enum):
@@ -46,34 +42,29 @@ class SelectResult(Enum):
 	ResetCurrent = auto()
 
 
-class CustomSetting(StrEnum):
-	SeatAccess = 'seat_access'
-	PlasmaFlavor = 'plasma_flavor'
-
-
 class Profile:
 	def __init__(
 		self,
 		name: str,
 		profile_type: ProfileType,
-		current_selection: list[Self] = [],
+		description: str = '',
-		packages: list[str] = [],
+		current_selection: List[TProfile] = [],
-		services: list[str] = [],
+		packages: List[str] = [],
+		services: List[str] = [],
 		support_gfx_driver: bool = False,
-		support_greeter: bool = False,
+		support_greeter: bool = False
-		display_server: DisplayServerType | None = None,
+	):
-	) -> None:
 		self.name = name
+		self.description = description
 		self.profile_type = profile_type
-		self.custom_settings: dict[CustomSetting, str | None] = {}
+		self.custom_settings: Dict[str, Any] = {}
 
 		self._support_gfx_driver = support_gfx_driver
 		self._support_greeter = support_greeter
-		self._display_server = display_server
 
-		# self.gfx_driver: str | None = None
+		# self.gfx_driver: Optional[str] = None
 
-		self.current_selection = current_selection
+		self._current_selection = current_selection
 		self._packages = packages
 		self._services = services
 
@@ -81,15 +72,19 @@ class Profile:
 		self.custom_enabled = False
 
 	@property
-	def packages(self) -> list[str]:
+	def current_selection(self) -> List[TProfile]:
+		return self._current_selection
+
+	@property
+	def packages(self) -> List[str]:
 		"""
 		Returns a list of packages that should be installed when
-		this profile is among the chosen ones
+		this profile is among the choosen ones
 		"""
 		return self._packages
 
 	@property
-	def services(self) -> list[str]:
+	def services(self) -> List[str]:
 		"""
 		Returns a list of services that should be enabled when
 		this profile is among the chosen ones
@@ -97,58 +92,54 @@ class Profile:
 		return self._services
 
 	@property
-	def default_greeter_type(self) -> GreeterType | None:
+	def default_greeter_type(self) -> Optional[GreeterType]:
 		"""
 		Setting a default greeter type for a desktop profile
 		"""
 		return None
 
-	def install(self, install_session: Installer) -> None:
+	def install(self, install_session: 'Installer'):
 		"""
 		Performs installation steps when this profile was selected
 		"""
 
-	def post_install(self, install_session: Installer) -> None:
+	def post_install(self, install_session: 'Installer'):
 		"""
 		Hook that will be called when the installation process is
 		finished and custom installation steps for specific default_profiles
 		are needed
 		"""
 
-	def provision(self, install_session: Installer, users: list[User]) -> None:
+	def json(self) -> Dict:
-		"""
-		Hook that will be called when the installation process is
-		finished and user configuration for specific default_profiles
-		is needed
-		"""
-
-	def json(self) -> dict[str, str]:
 		"""
 		Returns a json representation of the profile
 		"""
 		return {}
 
-	async def do_on_select(self) -> SelectResult | None:
+	def do_on_select(self) -> SelectResult:
 		"""
 		Hook that will be called when a profile is selected
 		"""
 		return SelectResult.NewSelection
 
-	def set_custom_settings(self, settings: dict[CustomSetting, str | None]) -> None:
+	def set_custom_settings(self, settings: Dict[str, Any]):
 		"""
 		Set the custom settings for the profile.
 		This is also called when the settings are parsed from the config
-		and can be overridden to perform further actions based on the profile
+		and can be overriden to perform further actions based on the profile
 		"""
 		self.custom_settings = settings
 
-	def current_selection_names(self) -> list[str]:
+	def current_selection_names(self) -> List[str]:
-		if self.current_selection:
+		if self._current_selection:
-			return [s.name for s in self.current_selection]
+			return [s.name for s in self._current_selection]
 		return []
 
-	def reset(self) -> None:
+	def reset(self):
-		self.current_selection = []
+		self.set_current_selection([])
+
+	def set_current_selection(self, current_selection: List[TProfile]):
+		self._current_selection = current_selection
 
 	def is_top_level_profile(self) -> bool:
 		top_levels = [ProfileType.Desktop, ProfileType.Server, ProfileType.Xorg, ProfileType.Minimal, ProfileType.Custom]
@@ -166,53 +157,34 @@ class Profile:
 	def is_xorg_type_profile(self) -> bool:
 		return self.profile_type == ProfileType.Xorg
 
+	def is_tailored(self) -> bool:
+		return self.profile_type == ProfileType.Tailored
+
 	def is_custom_type_profile(self) -> bool:
 		return self.profile_type == ProfileType.CustomType
 
 	def is_graphic_driver_supported(self) -> bool:
-		if not self.current_selection:
+		if not self._current_selection:
 			return self._support_gfx_driver
 		else:
-			if any([p._support_gfx_driver for p in self.current_selection]):
+			if any([p._support_gfx_driver for p in self._current_selection]):
 				return True
 			return False
 
 	def is_greeter_supported(self) -> bool:
 		return self._support_greeter
 
-	@property
+	def preview_text(self) -> Optional[str]:
-	def display_server(self) -> DisplayServerType | None:
-		return self._display_server
-
-	def preview_text(self) -> str:
 		"""
-		Override this method to provide a preview text for the profile
+		Used for preview text in profiles_bck. If a description is set for a
+		profile it will automatically display that one in the preivew.
+		If no preview or a different text should be displayed just
 		"""
-		if self.is_desktop_type_profile():
+		if self.description:
-			if self._display_server:
+			return self.description
-				text = tr('Environment type: {} {}').format(self._display_server.value, self.profile_type.value)
+		return None
-			else:
-				text = tr('Environment type: {}').format(self.profile_type.value)
-			if packages := self.packages_text():
-				text += f'\n{packages}'
-			return text
 
-		return self.packages_text()
+	def packages_text(self) -> str:
-
+		header = str(_('Installed packages'))
-	def packages_text(self, include_sub_packages: bool = False) -> str:
+		output = format_cols(self.packages, header)
-		packages = set()
+		return output
-
-		if self.packages:
-			packages = set(self.packages)
-
-		if include_sub_packages:
-			for sub_profile in self.current_selection:
-				if sub_profile.packages:
-					packages.update(sub_profile.packages)
-
-		text = tr('Installed packages') + ':\n'
-
-		for pkg in sorted(packages):
-			text += f'    - {pkg}\n'
-
-		return text

archinstall/default_profiles/server.py

@@ -1,74 +1,53 @@
-from typing import TYPE_CHECKING, Self, override
+from typing import Any, TYPE_CHECKING, List
 
-from archinstall.default_profiles.profile import Profile, ProfileType, SelectResult
+from archinstall.lib.output import info
-from archinstall.lib.log import info
+from archinstall.lib.menu import MenuSelectionType
-from archinstall.lib.menu.helpers import Selection
 from archinstall.lib.profile.profiles_handler import profile_handler
-from archinstall.tui.menu_item import MenuItem, MenuItemGroup
+from archinstall.default_profiles.profile import ProfileType, Profile, SelectResult, TProfile
-from archinstall.tui.result import ResultType
 
 if TYPE_CHECKING:
 	from archinstall.lib.installer import Installer
-	from archinstall.lib.models.users import User
+	_: Any
 
 
 class ServerProfile(Profile):
-	def __init__(self, current_value: list[Self] = []):
+	def __init__(self, current_value: List[TProfile] = []):
 		super().__init__(
 			'Server',
 			ProfileType.Server,
-			current_selection=current_value,
+			description=str(_('Provides a selection of various server packages to install and enable, e.g. httpd, nginx, mariadb')),
+			current_selection=current_value
 		)
 
-	@override
+	def do_on_select(self) -> SelectResult:
-	async def do_on_select(self) -> SelectResult:
+		available_servers = profile_handler.get_server_profiles()
-		items = [
-			MenuItem(
-				p.name,
-				value=p,
-				preview_action=lambda x: x.value.preview_text() if x.value else None,
-			)
-			for p in profile_handler.get_server_profiles()
-		]
 
-		group = MenuItemGroup(items, sort_items=True)
+		choice = profile_handler.select_profile(
-		group.set_selected_by_value(self.current_selection)
+			available_servers,
+			self._current_selection,
+			title=str(_('Choose which servers to install, if none then a minimal installation will be done')),
+			multi=True
+		)
 
-		result = await Selection[Self](
+		match choice.type_:
-			group,
+			case MenuSelectionType.Selection:
-			allow_reset=True,
+				self.set_current_selection(choice.value)  # type: ignore
-			allow_skip=True,
-			multi=True,
-			preview_location='right',
-		).show()
-
-		match result.type_:
-			case ResultType.Selection:
-				selections = result.get_values()
-				self.current_selection = selections
 				return SelectResult.NewSelection
-			case ResultType.Skip:
+			case MenuSelectionType.Skip:
 				return SelectResult.SameSelection
-			case ResultType.Reset:
+			case MenuSelectionType.Reset:
 				return SelectResult.ResetCurrent
 
-	@override
+	def post_install(self, install_session: 'Installer'):
-	def provision(self, install_session: Installer, users: list[User]) -> None:
+		for profile in self._current_selection:
-		for profile in self.current_selection:
-			profile.provision(install_session, users)
-
-	@override
-	def post_install(self, install_session: Installer) -> None:
-		for profile in self.current_selection:
 			profile.post_install(install_session)
 
-	@override
+	def install(self, install_session: 'Installer'):
-	def install(self, install_session: Installer) -> None:
 		server_info = self.current_selection_names()
 		details = ', '.join(server_info)
 		info(f'Now installing the selected servers: {details}')
 
-		for server in self.current_selection:
+		for server in self._current_selection:
 			info(f'Installing {server.name}...')
 			install_session.add_additional_packages(server.packages)
 			install_session.enable_service(server.services)

archinstall/default_profiles/servers/cockpit.py

@@ -1,21 +1,19 @@
-from typing import override
+from typing import List
 
 from archinstall.default_profiles.profile import Profile, ProfileType
 
 
 class CockpitProfile(Profile):
-	def __init__(self) -> None:
+	def __init__(self):
 		super().__init__(
 			'Cockpit',
-			ProfileType.ServerType,
+			ProfileType.ServerType
 		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return ['cockpit', 'udisks2', 'packagekit']
 
 	@property
-	@override
+	def services(self) -> List[str]:
-	def services(self) -> list[str]:
 		return ['cockpit.socket']

archinstall/default_profiles/servers/docker.py

@@ -1,30 +1,33 @@
-from typing import TYPE_CHECKING, override
+from typing import List, Union, TYPE_CHECKING
+
+import archinstall
 
 from archinstall.default_profiles.profile import Profile, ProfileType
+from archinstall.lib.models import User
 
 if TYPE_CHECKING:
 	from archinstall.lib.installer import Installer
-	from archinstall.lib.models.users import User
 
 
 class DockerProfile(Profile):
-	def __init__(self) -> None:
+	def __init__(self):
 		super().__init__(
 			'Docker',
-			ProfileType.ServerType,
+			ProfileType.ServerType
 		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return ['docker']
 
 	@property
-	@override
+	def services(self) -> List[str]:
-	def services(self) -> list[str]:
 		return ['docker']
 
-	@override
+	def post_install(self, install_session: 'Installer'):
-	def provision(self, install_session: Installer, users: list[User]) -> None:
+		users: Union[User, List[User]] = archinstall.arguments.get('!users', [])
+		if not isinstance(users, list):
+			users = [users]
+
 		for user in users:
 			install_session.arch_chroot(f'usermod -a -G docker {user.username}')

archinstall/default_profiles/servers/httpd.py

@@ -1,21 +1,19 @@
-from typing import override
+from typing import List
 
 from archinstall.default_profiles.profile import Profile, ProfileType
 
 
 class HttpdProfile(Profile):
-	def __init__(self) -> None:
+	def __init__(self):
 		super().__init__(
 			'httpd',
-			ProfileType.ServerType,
+			ProfileType.ServerType
 		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return ['apache']
 
 	@property
-	@override
+	def services(self) -> List[str]:
-	def services(self) -> list[str]:
 		return ['httpd']

archinstall/default_profiles/servers/lighttpd.py

@@ -1,21 +1,19 @@
-from typing import override
+from typing import List
 
 from archinstall.default_profiles.profile import Profile, ProfileType
 
 
 class LighttpdProfile(Profile):
-	def __init__(self) -> None:
+	def __init__(self):
 		super().__init__(
 			'Lighttpd',
-			ProfileType.ServerType,
+			ProfileType.ServerType
 		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return ['lighttpd']
 
 	@property
-	@override
+	def services(self) -> List[str]:
-	def services(self) -> list[str]:
 		return ['lighttpd']

archinstall/default_profiles/servers/mariadb.py

@@ -1,4 +1,4 @@
-from typing import TYPE_CHECKING, override
+from typing import List, TYPE_CHECKING
 
 from archinstall.default_profiles.profile import Profile, ProfileType
 
@@ -7,22 +7,19 @@ if TYPE_CHECKING:
 
 
 class MariadbProfile(Profile):
-	def __init__(self) -> None:
+	def __init__(self):
 		super().__init__(
 			'Mariadb',
-			ProfileType.ServerType,
+			ProfileType.ServerType
 		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return ['mariadb']
 
 	@property
-	@override
+	def services(self) -> List[str]:
-	def services(self) -> list[str]:
 		return ['mariadb']
 
-	@override
+	def post_install(self, install_session: 'Installer'):
-	def post_install(self, install_session: Installer) -> None:
 		install_session.arch_chroot('mariadb-install-db --user=mysql --basedir=/usr --datadir=/var/lib/mysql')

archinstall/default_profiles/servers/nginx.py

@@ -1,21 +1,19 @@
-from typing import override
+from typing import List
 
 from archinstall.default_profiles.profile import Profile, ProfileType
 
 
 class NginxProfile(Profile):
-	def __init__(self) -> None:
+	def __init__(self):
 		super().__init__(
 			'Nginx',
-			ProfileType.ServerType,
+			ProfileType.ServerType
 		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return ['nginx']
 
 	@property
-	@override
+	def services(self) -> List[str]:
-	def services(self) -> list[str]:
 		return ['nginx']

archinstall/default_profiles/servers/postgresql.py

@@ -1,4 +1,4 @@
-from typing import TYPE_CHECKING, override
+from typing import List, TYPE_CHECKING
 
 from archinstall.default_profiles.profile import Profile, ProfileType
 
@@ -7,22 +7,20 @@ if TYPE_CHECKING:
 
 
 class PostgresqlProfile(Profile):
-	def __init__(self) -> None:
+	def __init__(self):
 		super().__init__(
 			'Postgresql',
 			ProfileType.ServerType,
+			''
 		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return ['postgresql']
 
 	@property
-	@override
+	def services(self) -> List[str]:
-	def services(self) -> list[str]:
 		return ['postgresql']
 
-	@override
+	def post_install(self, install_session: 'Installer'):
-	def post_install(self, install_session: Installer) -> None:
+		install_session.arch_chroot("initdb -D /var/lib/postgres/data", run_as='postgres')
-		install_session.arch_chroot('initdb -D /var/lib/postgres/data', run_as='postgres')

archinstall/default_profiles/servers/sshd.py

@@ -1,21 +1,19 @@
-from typing import override
+from typing import List
 
 from archinstall.default_profiles.profile import Profile, ProfileType
 
 
 class SshdProfile(Profile):
-	def __init__(self) -> None:
+	def __init__(self):
 		super().__init__(
 			'sshd',
-			ProfileType.ServerType,
+			ProfileType.ServerType
 		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return ['openssh']
 
 	@property
-	@override
+	def services(self) -> List[str]:
-	def services(self) -> list[str]:
 		return ['sshd']

archinstall/default_profiles/servers/tomcat.py

@@ -1,21 +1,19 @@
-from typing import override
+from typing import List
 
 from archinstall.default_profiles.profile import Profile, ProfileType
 
 
 class TomcatProfile(Profile):
-	def __init__(self) -> None:
+	def __init__(self):
 		super().__init__(
 			'Tomcat',
-			ProfileType.ServerType,
+			ProfileType.ServerType
 		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return ['tomcat10']
 
 	@property
-	@override
+	def services(self) -> List[str]:
-	def services(self) -> list[str]:
 		return ['tomcat10']

archinstall/default_profiles/tailored.py

@@ -0,0 +1,21 @@
+from typing import List, Any, TYPE_CHECKING
+
+from archinstall.default_profiles.profile import ProfileType
+from archinstall.default_profiles.xorg import XorgProfile
+
+if TYPE_CHECKING:
+	from archinstall.lib.installer import Installer
+	_: Any
+
+
+class TailoredProfile(XorgProfile):
+	def __init__(self):
+		super().__init__('52-54-00-12-34-56', ProfileType.Tailored, description='')
+
+	@property
+	def packages(self) -> List[str]:
+		return ['nano', 'wget', 'git']
+
+	def install(self, install_session: 'Installer'):
+		super().install(install_session)
+		# do whatever you like here :)

archinstall/default_profiles/xorg.py

@@ -1,9 +1,9 @@
-from typing import TYPE_CHECKING, override
+from typing import Any, TYPE_CHECKING, List
 
-from archinstall.default_profiles.profile import DisplayServerType, Profile, ProfileType
+from archinstall.default_profiles.profile import Profile, ProfileType
 
 if TYPE_CHECKING:
-	from archinstall.lib.installer import Installer
+	_: Any
 
 
 class XorgProfile(Profile):
@@ -11,22 +11,17 @@ class XorgProfile(Profile):
 		self,
 		name: str = 'Xorg',
 		profile_type: ProfileType = ProfileType.Xorg,
+		description: str = str(_('Installs a minimal system as well as xorg and graphics drivers.')),
 	):
 		super().__init__(
 			name,
 			profile_type,
-			support_gfx_driver=True,
+			description=description,
-			display_server=DisplayServerType.Xorg,
+			support_gfx_driver=True
 		)
 
 	@property
-	@override
+	def packages(self) -> List[str]:
-	def packages(self) -> list[str]:
 		return [
-			'xorg-server',
+			'xorg-server'
-			'xorg-xinit',
 		]
-
-	@override
-	def install(self, install_session: Installer) -> None:
-		install_session.add_additional_packages(self.packages)

archinstall/lib/applications/application_handler.py

@@ -1,51 +0,0 @@
-from typing import TYPE_CHECKING
-
-from archinstall.applications.audio import AudioApp
-from archinstall.applications.bluetooth import BluetoothApp
-from archinstall.applications.firewall import FirewallApp
-from archinstall.applications.fonts import FontsApp
-from archinstall.applications.power_management import PowerManagementApp
-from archinstall.applications.print_service import PrintServiceApp
-from archinstall.lib.models import Audio
-from archinstall.lib.models.application import ApplicationConfiguration
-from archinstall.lib.models.users import User
-
-if TYPE_CHECKING:
-	from archinstall.lib.installer import Installer
-
-
-class ApplicationHandler:
-	def __init__(self) -> None:
-		pass
-
-	def install_applications(self, install_session: Installer, app_config: ApplicationConfiguration, users: list[User] | None = None) -> None:
-		if app_config.bluetooth_config and app_config.bluetooth_config.enabled:
-			BluetoothApp().install(install_session)
-
-		if app_config.audio_config and app_config.audio_config.audio != Audio.NO_AUDIO:
-			AudioApp().install(
-				install_session,
-				app_config.audio_config,
-				users,
-			)
-
-		if app_config.power_management_config:
-			PowerManagementApp().install(
-				install_session,
-				app_config.power_management_config,
-			)
-
-		if app_config.print_service_config and app_config.print_service_config.enabled:
-			PrintServiceApp().install(install_session)
-
-		if app_config.firewall_config:
-			FirewallApp().install(
-				install_session,
-				app_config.firewall_config,
-			)
-
-		if app_config.fonts_config:
-			FontsApp().install(
-				install_session,
-				app_config.fonts_config,
-			)

archinstall/lib/applications/application_menu.py

@@ -1,263 +0,0 @@
-from typing import override
-
-from archinstall.lib.hardware import SysInfo
-from archinstall.lib.menu.abstract_menu import AbstractSubMenu
-from archinstall.lib.menu.helpers import Confirmation, Selection
-from archinstall.lib.models.application import (
-	ApplicationConfiguration,
-	Audio,
-	AudioConfiguration,
-	BluetoothConfiguration,
-	Firewall,
-	FirewallConfiguration,
-	FontPackage,
-	FontsConfiguration,
-	PowerManagement,
-	PowerManagementConfiguration,
-	PrintServiceConfiguration,
-)
-from archinstall.lib.translationhandler import tr
-from archinstall.tui.menu_item import MenuItem, MenuItemGroup
-from archinstall.tui.result import ResultType
-
-
-class ApplicationMenu(AbstractSubMenu[ApplicationConfiguration]):
-	def __init__(
-		self,
-		preset: ApplicationConfiguration | None = None,
-	):
-		if preset:
-			self._app_config = preset
-		else:
-			self._app_config = ApplicationConfiguration()
-
-		menu_options = self._define_menu_options()
-		self._item_group = MenuItemGroup(menu_options, checkmarks=True)
-
-		super().__init__(
-			self._item_group,
-			config=self._app_config,
-			allow_reset=True,
-		)
-
-	@override
-	async def show(self) -> ApplicationConfiguration | None:
-		_ = await super().show()
-		return self._app_config
-
-	def _define_menu_options(self) -> list[MenuItem]:
-		return [
-			MenuItem(
-				text=tr('Bluetooth'),
-				action=select_bluetooth,
-				value=self._app_config.bluetooth_config,
-				preview_action=self._prev_bluetooth,
-				key='bluetooth_config',
-			),
-			MenuItem(
-				text=tr('Audio'),
-				action=select_audio,
-				preview_action=self._prev_audio,
-				key='audio_config',
-			),
-			MenuItem(
-				text=tr('Print service'),
-				action=select_print_service,
-				preview_action=self._prev_print_service,
-				key='print_service_config',
-			),
-			MenuItem(
-				text=tr('Power management'),
-				action=select_power_management,
-				preview_action=self._prev_power_management,
-				enabled=SysInfo.has_battery(),
-				key='power_management_config',
-			),
-			MenuItem(
-				text=tr('Firewall'),
-				action=select_firewall,
-				preview_action=self._prev_firewall,
-				key='firewall_config',
-			),
-			MenuItem(
-				text=tr('Additional fonts'),
-				action=select_fonts,
-				value=self._app_config.fonts_config,
-				preview_action=self._prev_fonts,
-				key='fonts_config',
-			),
-		]
-
-	def _prev_power_management(self, item: MenuItem) -> str | None:
-		if item.value is not None:
-			config: PowerManagementConfiguration = item.value
-			return f'{tr("Power management")}: {config.power_management.value}'
-		return None
-
-	def _prev_bluetooth(self, item: MenuItem) -> str | None:
-		if item.value is not None:
-			bluetooth_config: BluetoothConfiguration = item.value
-
-			output = f'{tr("Bluetooth")}: '
-			output += tr('Enabled') if bluetooth_config.enabled else tr('Disabled')
-			return output
-		return None
-
-	def _prev_audio(self, item: MenuItem) -> str | None:
-		if item.value is not None:
-			config: AudioConfiguration = item.value
-			return f'{tr("Audio")}: {config.audio.value}'
-		return None
-
-	def _prev_print_service(self, item: MenuItem) -> str | None:
-		if item.value is not None:
-			print_service_config: PrintServiceConfiguration = item.value
-
-			output = f'{tr("Print service")}: '
-			output += tr('Enabled') if print_service_config.enabled else tr('Disabled')
-			return output
-		return None
-
-	def _prev_firewall(self, item: MenuItem) -> str | None:
-		if item.value is not None:
-			config: FirewallConfiguration = item.value
-			return f'{tr("Firewall")}: {config.firewall.value}'
-		return None
-
-	def _prev_fonts(self, item: MenuItem) -> str | None:
-		if item.value is not None:
-			config: FontsConfiguration = item.value
-			packages = ', '.join(f.value for f in config.fonts)
-			return f'{tr("Additional fonts")}: {packages}'
-		return None
-
-
-async def select_power_management(preset: PowerManagementConfiguration | None = None) -> PowerManagementConfiguration | None:
-	group = MenuItemGroup.from_enum(PowerManagement)
-
-	if preset:
-		group.set_focus_by_value(preset.power_management)
-
-	result = await Selection[PowerManagement](
-		group,
-		allow_skip=True,
-		allow_reset=True,
-	).show()
-
-	match result.type_:
-		case ResultType.Skip:
-			return preset
-		case ResultType.Selection:
-			return PowerManagementConfiguration(power_management=result.get_value())
-		case ResultType.Reset:
-			return None
-
-
-async def select_bluetooth(preset: BluetoothConfiguration | None) -> BluetoothConfiguration | None:
-	header = tr('Would you like to configure Bluetooth?') + '\n'
-	preset_val = preset.enabled if preset else False
-
-	result = await Confirmation(
-		header=header,
-		allow_skip=True,
-		preset=preset_val,
-	).show()
-
-	match result.type_:
-		case ResultType.Selection:
-			return BluetoothConfiguration(result.get_value())
-		case ResultType.Skip:
-			return preset
-		case _:
-			raise ValueError('Unhandled result type')
-
-
-async def select_print_service(preset: PrintServiceConfiguration | None) -> PrintServiceConfiguration | None:
-	header = tr('Would you like to configure the print service?') + '\n'
-	preset_val = preset.enabled if preset else False
-
-	result = await Confirmation(
-		header=header,
-		allow_skip=True,
-		preset=preset_val,
-	).show()
-
-	match result.type_:
-		case ResultType.Selection:
-			result.get_value()
-			return PrintServiceConfiguration(result.get_value())
-		case ResultType.Skip:
-			return preset
-		case _:
-			raise ValueError('Unhandled result type')
-
-
-async def select_audio(preset: AudioConfiguration | None = None) -> AudioConfiguration | None:
-	items = [MenuItem(a.value, value=a) for a in Audio]
-	group = MenuItemGroup(items)
-
-	if preset:
-		group.set_focus_by_value(preset.audio)
-
-	result = await Selection[Audio](
-		group,
-		header=tr('Select audio configuration'),
-		allow_skip=True,
-	).show()
-
-	match result.type_:
-		case ResultType.Skip:
-			return preset
-		case ResultType.Selection:
-			return AudioConfiguration(audio=result.get_value())
-		case ResultType.Reset:
-			raise ValueError('Unhandled result type')
-
-
-async def select_firewall(preset: FirewallConfiguration | None = None) -> FirewallConfiguration | None:
-	group = MenuItemGroup.from_enum(Firewall)
-
-	if preset:
-		group.set_focus_by_value(preset.firewall)
-
-	result = await Selection[Firewall](
-		group,
-		allow_skip=True,
-		allow_reset=True,
-	).show()
-
-	match result.type_:
-		case ResultType.Skip:
-			return preset
-		case ResultType.Selection:
-			return FirewallConfiguration(firewall=result.get_value())
-		case ResultType.Reset:
-			return None
-
-
-async def select_fonts(preset: FontsConfiguration | None = None) -> FontsConfiguration | None:
-	items = [MenuItem(f'{f.value} ({f.description()})', value=f) for f in FontPackage]
-	group = MenuItemGroup(items)
-
-	if preset:
-		for f in preset.fonts:
-			group.set_selected_by_value(f)
-
-	result = await Selection[FontPackage](
-		group,
-		header=tr('Select font packages to install'),
-		allow_skip=True,
-		allow_reset=True,
-		multi=True,
-	).show()
-
-	match result.type_:
-		case ResultType.Skip:
-			return preset
-		case ResultType.Selection:
-			selected = result.get_values()
-			if selected:
-				return FontsConfiguration(fonts=selected)
-			return None
-		case ResultType.Reset:
-			return None

archinstall/lib/args.py

@@ -1,801 +0,0 @@
-import argparse
-import json
-import os
-import stat
-import sys
-import tempfile
-import urllib.error
-import urllib.parse
-from argparse import ArgumentParser, Namespace
-from dataclasses import dataclass, field
-from enum import Enum, StrEnum
-from pathlib import Path
-from typing import Any, Self
-from urllib.request import Request, urlopen
-
-from pydantic import TypeAdapter
-from pydantic.dataclasses import dataclass as p_dataclass
-
-from archinstall.lib.crypt import decrypt, encrypt
-from archinstall.lib.log import debug, error, logger, warn
-from archinstall.lib.menu.util import get_password
-from archinstall.lib.models.application import ApplicationConfiguration, ZramConfiguration
-from archinstall.lib.models.authentication import AuthenticationConfiguration
-from archinstall.lib.models.bootloader import Bootloader, BootloaderConfiguration
-from archinstall.lib.models.config import SubConfig
-from archinstall.lib.models.device import DiskEncryption, DiskLayoutConfiguration
-from archinstall.lib.models.locale import LocaleConfiguration
-from archinstall.lib.models.mirrors import MirrorConfiguration
-from archinstall.lib.models.network import NetworkConfiguration
-from archinstall.lib.models.package_types import DEFAULT_KERNEL
-from archinstall.lib.models.packages import Repository
-from archinstall.lib.models.pacman import PacmanConfiguration
-from archinstall.lib.models.profile import ProfileConfiguration
-from archinstall.lib.models.users import Password, User, UserSerialization
-from archinstall.lib.plugins import load_plugin
-from archinstall.lib.translationhandler import Language, tr, translation_handler
-from archinstall.lib.utils.format import as_key_value_pair
-from archinstall.lib.version import get_version
-from archinstall.tui.components import tui
-
-
-class SubCommand(Enum):
-	SHARE_LOG = 'share-log'
-
-
-@p_dataclass
-class Arguments:
-	config: Path | None = None
-	config_url: str | None = None
-	creds: Path | None = None
-	creds_url: str | None = None
-	creds_decryption_key: str | None = None
-	silent: bool = False
-	dry_run: bool = False
-	script: str | None = None
-	mountpoint: Path = Path('/mnt')
-	skip_ntp: bool = False
-	skip_wkd: bool = False
-	skip_boot: bool = False
-	debug: bool = False
-	offline: bool = False
-	no_pkg_lookups: bool = False
-	plugin: Path | None = None
-	plugin_url: str | None = None
-	skip_version_check: bool = False
-	skip_wifi_check: bool = False
-	advanced: bool = False
-	verbose: bool = False
-
-	command: SubCommand | None = None
-
-
-class ArchConfigType(StrEnum):
-	VERSION = 'version'
-	SCRIPT = 'script'
-	LOCALE_CONFIG = 'locale_config'
-	ARCHINSTALL_LANGUAGE = 'archinstall_language'
-	DISK_CONFIG = 'disk_config'
-	PROFILE_CONFIG = 'profile_config'
-	MIRROR_CONFIG = 'mirror_config'
-	NETWORK_CONFIG = 'network_config'
-	BOOTLOADER_CONFIG = 'bootloader_config'
-	APP_CONFIG = 'app_config'
-	AUTH_CONFIG = 'auth_config'
-	SWAP = 'swap'
-	USERS = 'users'
-	ROOT_ENC_PASSWORD = 'root_enc_password'
-	ENCRYPTION_PASSWORD = 'encryption_password'
-	HOSTNAME = 'hostname'
-	KERNELS = 'kernels'
-	NTP = 'ntp'
-	TIMEZONE = 'timezone'
-	SERVICES = 'services'
-	PACKAGES = 'packages'
-	PACMAN_CONFIG = 'pacman_config'
-	CUSTOM_COMMANDS = 'custom_commands'
-
-	def text(self) -> str:
-		match self:
-			case ArchConfigType.ARCHINSTALL_LANGUAGE:
-				return tr('ArchInstall Language')
-			case ArchConfigType.VERSION:
-				return tr('Version')
-			case ArchConfigType.SCRIPT:
-				return tr('Installation Script')
-			case ArchConfigType.LOCALE_CONFIG:
-				return tr('Locales')
-			case ArchConfigType.DISK_CONFIG:
-				return tr('Disk configuration')
-			case ArchConfigType.PROFILE_CONFIG:
-				return tr('Profile')
-			case ArchConfigType.MIRROR_CONFIG:
-				return tr('Mirrors and repositories')
-			case ArchConfigType.NETWORK_CONFIG:
-				return tr('Network')
-			case ArchConfigType.BOOTLOADER_CONFIG:
-				return tr('Bootloader')
-			case ArchConfigType.APP_CONFIG:
-				return tr('Application')
-			case ArchConfigType.AUTH_CONFIG:
-				return tr('Authentication')
-			case ArchConfigType.SWAP:
-				return tr('Swap')
-			case ArchConfigType.HOSTNAME:
-				return tr('Hostname')
-			case ArchConfigType.KERNELS:
-				return tr('Kernels')
-			case ArchConfigType.NTP:
-				return tr('Automatic time sync (NTP)')
-			case ArchConfigType.TIMEZONE:
-				return tr('Timezone')
-			case ArchConfigType.SERVICES:
-				return tr('Services')
-			case ArchConfigType.PACKAGES:
-				return tr('Additional packages')
-			case ArchConfigType.PACMAN_CONFIG:
-				return tr('Pacman')
-			case ArchConfigType.CUSTOM_COMMANDS:
-				return tr('Custom commands')
-			case ArchConfigType.USERS:
-				return tr('Users')
-			case ArchConfigType.ROOT_ENC_PASSWORD:
-				return tr('Root encrypted password')
-			case ArchConfigType.ENCRYPTION_PASSWORD:
-				return tr('Disk encryption password')
-
-
-USER_CONFIG_FILE: Path = Path('user_configuration.json')
-USER_CREDS_FILE: Path = Path('user_credentials.json')
-DEFAULT_SAVE_PATH = logger.directory
-
-
-@dataclass
-class ArchConfig:
-	version: str | None = None
-	script: str | None = None
-	locale_config: LocaleConfiguration | None = None
-	archinstall_language: Language = field(default_factory=lambda: translation_handler.get_language_by_abbr('en'))
-	disk_config: DiskLayoutConfiguration | None = None
-	profile_config: ProfileConfiguration | None = None
-	mirror_config: MirrorConfiguration | None = None
-	network_config: NetworkConfiguration | None = None
-	bootloader_config: BootloaderConfiguration | None = None
-	app_config: ApplicationConfiguration | None = None
-	auth_config: AuthenticationConfiguration | None = None
-	swap: ZramConfiguration | None = None
-	hostname: str = 'archlinux'
-	kernels: list[str] = field(default_factory=lambda: [DEFAULT_KERNEL.value])
-	ntp: bool = True
-	packages: list[str] = field(default_factory=list)
-	pacman_config: PacmanConfiguration = field(default_factory=PacmanConfiguration.default)
-	timezone: str = 'UTC'
-	services: list[str] = field(default_factory=list)
-	custom_commands: list[str] = field(default_factory=list)
-
-	def unsafe_config(self) -> dict[ArchConfigType, Any]:
-		config: dict[ArchConfigType, list[UserSerialization] | str | None] = {}
-
-		if self.auth_config:
-			if self.auth_config.users:
-				config[ArchConfigType.USERS] = [user.json() for user in self.auth_config.users]
-
-			if self.auth_config.root_enc_password:
-				config[ArchConfigType.ROOT_ENC_PASSWORD] = self.auth_config.root_enc_password.enc_password
-
-		if self.disk_config:
-			disk_encryption = self.disk_config.disk_encryption
-			if disk_encryption and disk_encryption.encryption_password:
-				config[ArchConfigType.ENCRYPTION_PASSWORD] = disk_encryption.encryption_password.plaintext
-
-		return config
-
-	def safe_config(self) -> dict[ArchConfigType, Any]:
-		base_config: dict[ArchConfigType, Any] = {
-			ArchConfigType.VERSION: self.version,
-			ArchConfigType.SCRIPT: self.script,
-			ArchConfigType.ARCHINSTALL_LANGUAGE: self.archinstall_language.json(),
-		}
-
-		base_config.update(self.plain_cfg())
-		sub_config = self.sub_cfg()
-
-		for config_type, value in sub_config.items():
-			if not hasattr(value, 'json'):
-				raise ValueError(f'Config value for {config_type} must implement json() method')
-			base_config[config_type] = value.json()
-
-		return base_config
-
-	def plain_cfg(self) -> dict[ArchConfigType, str | list[str] | bool]:
-		return {
-			ArchConfigType.HOSTNAME: self.hostname,
-			ArchConfigType.KERNELS: self.kernels,
-			ArchConfigType.NTP: self.ntp,
-			ArchConfigType.TIMEZONE: self.timezone,
-			ArchConfigType.SERVICES: self.services,
-			ArchConfigType.PACKAGES: self.packages,
-			ArchConfigType.CUSTOM_COMMANDS: self.custom_commands,
-		}
-
-	def sub_cfg(self) -> dict[ArchConfigType, SubConfig]:
-		cfg: dict[ArchConfigType, SubConfig] = {
-			ArchConfigType.PACMAN_CONFIG: self.pacman_config,
-		}
-
-		if self.mirror_config:
-			cfg[ArchConfigType.MIRROR_CONFIG] = self.mirror_config
-
-		if self.bootloader_config:
-			cfg[ArchConfigType.BOOTLOADER_CONFIG] = self.bootloader_config
-
-		if self.disk_config:
-			cfg[ArchConfigType.DISK_CONFIG] = self.disk_config
-
-		if self.swap:
-			cfg[ArchConfigType.SWAP] = self.swap
-
-		if self.auth_config:
-			cfg[ArchConfigType.AUTH_CONFIG] = self.auth_config
-
-		if self.locale_config:
-			cfg[ArchConfigType.LOCALE_CONFIG] = self.locale_config
-
-		if self.profile_config:
-			cfg[ArchConfigType.PROFILE_CONFIG] = self.profile_config
-
-		if self.network_config:
-			cfg[ArchConfigType.NETWORK_CONFIG] = self.network_config
-
-		if self.app_config:
-			cfg[ArchConfigType.APP_CONFIG] = self.app_config
-
-		return cfg
-
-	@classmethod
-	def from_config(cls, args_config: dict[str, Any], args: Arguments) -> Self:
-		arch_config = cls()
-
-		arch_config.locale_config = LocaleConfiguration.parse_arg(args_config)
-
-		if script := args_config.get('script', None):
-			arch_config.script = script
-
-		if archinstall_lang := args_config.get('archinstall-language', None):
-			arch_config.archinstall_language = translation_handler.get_language_by_name(archinstall_lang)
-			translation_handler.activate(arch_config.archinstall_language, set_font=False)
-
-		if disk_config := args_config.get('disk_config', {}):
-			enc_password = args_config.get('encryption_password', '')
-			password = Password(plaintext=enc_password) if enc_password else None
-			arch_config.disk_config = DiskLayoutConfiguration.parse_arg(disk_config, password)
-
-			# DEPRECATED
-			# backwards compatibility for main level disk_encryption entry
-			disk_encryption: DiskEncryption | None = None
-
-			if args_config.get('disk_encryption', None) is not None and arch_config.disk_config is not None:
-				disk_encryption = DiskEncryption.parse_arg(
-					arch_config.disk_config,
-					args_config['disk_encryption'],
-					Password(plaintext=args_config.get('encryption_password', '')),
-				)
-
-				if disk_encryption:
-					arch_config.disk_config.disk_encryption = disk_encryption
-
-		if profile_config := args_config.get('profile_config', None):
-			arch_config.profile_config = ProfileConfiguration.parse_arg(profile_config)
-
-		if mirror_config := args_config.get('mirror_config', None):
-			backwards_compatible_repo = []
-			if additional_repositories := args_config.get('additional-repositories', []):
-				backwards_compatible_repo = [Repository(r) for r in additional_repositories]
-
-			arch_config.mirror_config = MirrorConfiguration.parse_args(
-				mirror_config,
-				backwards_compatible_repo,
-			)
-
-		if net_config := args_config.get('network_config', None):
-			arch_config.network_config = NetworkConfiguration.parse_arg(net_config)
-
-		if bootloader_config_dict := args_config.get('bootloader_config', None):
-			arch_config.bootloader_config = BootloaderConfiguration.parse_arg(bootloader_config_dict, args.skip_boot)
-		# DEPRECATED: separate bootloader and uki fields (backward compatibility)
-		elif bootloader_str := args_config.get('bootloader', None):
-			bootloader = Bootloader.from_arg(bootloader_str, args.skip_boot)
-			uki = args_config.get('uki', False)
-			if uki and not bootloader.has_uki_support():
-				uki = False
-			arch_config.bootloader_config = BootloaderConfiguration(bootloader=bootloader, uki=uki, removable=True)
-
-		# deprecated: backwards compatibility
-		audio_config_args = args_config.get('audio_config', None)
-		app_config_args = args_config.get('app_config', None)
-
-		if audio_config_args is not None or app_config_args is not None:
-			arch_config.app_config = ApplicationConfiguration.parse_arg(app_config_args, audio_config_args)
-
-		if auth_config_args := args_config.get('auth_config', None):
-			arch_config.auth_config = AuthenticationConfiguration.parse_arg(auth_config_args)
-
-		if hostname := args_config.get('hostname', ''):
-			arch_config.hostname = hostname
-
-		if kernels := args_config.get('kernels', []):
-			arch_config.kernels = kernels
-
-		arch_config.ntp = args_config.get('ntp', True)
-
-		if packages := args_config.get('packages', []):
-			arch_config.packages = packages
-
-		if pacman_config := args_config.get('pacman_config', None):
-			arch_config.pacman_config = PacmanConfiguration.parse_arg(pacman_config)
-		elif parallel_downloads := args_config.get('parallel_downloads', 0):
-			arch_config.pacman_config = PacmanConfiguration(parallel_downloads=int(parallel_downloads))
-
-		swap_arg = args_config.get('swap')
-		if swap_arg is not None:
-			arch_config.swap = ZramConfiguration.parse_arg(swap_arg)
-
-		if timezone := args_config.get('timezone', 'UTC'):
-			arch_config.timezone = timezone
-
-		if services := args_config.get('services', []):
-			arch_config.services = services
-
-		# DEPRECATED: backwards compatibility
-		root_password = None
-		if root_password := args_config.get('!root-password', None):
-			root_password = Password(plaintext=root_password)
-
-		if enc_password := args_config.get('root_enc_password', None):
-			root_password = Password(enc_password=enc_password)
-
-		if root_password is not None:
-			if arch_config.auth_config is None:
-				arch_config.auth_config = AuthenticationConfiguration()
-			arch_config.auth_config.root_enc_password = root_password
-
-		# DEPRECATED: backwards compatibility
-		users: list[User] = []
-		if args_users := args_config.get('!users', None):
-			users = User.parse_arguments(args_users)
-
-		if args_users := args_config.get('users', None):
-			users = User.parse_arguments(args_users)
-
-		if users:
-			if arch_config.auth_config is None:
-				arch_config.auth_config = AuthenticationConfiguration()
-			arch_config.auth_config.users = users
-
-		if custom_commands := args_config.get('custom_commands', []):
-			arch_config.custom_commands = custom_commands
-
-		return arch_config
-
-	def user_config_to_json(self) -> str:
-		config = self.safe_config()
-
-		adapter = TypeAdapter(dict[ArchConfigType, Any])
-		python_dict = adapter.dump_python(config)
-		return json.dumps(python_dict, indent=4, sort_keys=True)
-
-	def user_credentials_to_json(self) -> str:
-		cfg = self.unsafe_config()
-
-		adapter = TypeAdapter(dict[ArchConfigType, Any])
-		python_dict = adapter.dump_python(cfg)
-		return json.dumps(python_dict, indent=4, sort_keys=True)
-
-	def write_debug(self) -> None:
-		debug(' -- Chosen configuration --')
-		debug(self.user_config_to_json())
-
-	def save(
-		self,
-		dest_path: Path | None = None,
-		creds: bool = False,
-		password: str | None = None,
-	) -> None:
-		save_path = dest_path or DEFAULT_SAVE_PATH
-
-		if not save_path.is_dir():
-			warn(
-				f'Destination directory {save_path} does not exist or is not a directory\n.',
-				'Configuration files can not be saved',
-			)
-			return
-
-		self.save_user_config(save_path)
-		if creds:
-			self.save_user_creds(save_path, password=password)
-
-	def save_user_config(self, dest_path: Path) -> None:
-		if not dest_path.is_dir():
-			error(f'Invalid path {dest_path}. User configuration could not be saved.')
-			return
-
-		target = dest_path / USER_CONFIG_FILE
-		data = self.user_config_to_json()
-		target.write_text(data)
-		target.chmod(stat.S_IRUSR | stat.S_IWUSR | stat.S_IRGRP)
-
-	def save_user_creds(
-		self,
-		dest_path: Path,
-		password: str | None = None,
-	) -> None:
-		if not dest_path.is_dir():
-			error(f'Invalid path {dest_path}. User credentials could not be saved.')
-			return
-
-		data = self.user_credentials_to_json()
-
-		if password:
-			data = encrypt(password, data)
-
-		target = dest_path / USER_CREDS_FILE
-		target.write_text(data)
-		target.chmod(stat.S_IRUSR | stat.S_IWUSR | stat.S_IRGRP)
-
-	def as_summary(self) -> str:
-		"""
-		Render a concise two-column summary of the current configuration.
-
-		Returns an empty string if nothing meaningful to show.
-		"""
-		cfg: dict[str, str | list[str] | bool] = {}
-
-		for key, value in self.plain_cfg().items():
-			cfg[key.text()] = value
-
-		for config_type, obj in self.sub_cfg().items():
-			if not hasattr(obj, 'summary'):
-				continue
-
-			summary = obj.summary()
-			if summary:
-				cfg[config_type.text()] = summary
-
-		simple_summary = as_key_value_pair(cfg, ignore_empty=True)
-
-		return simple_summary
-
-
-class ArchConfigHandler:
-	def __init__(self) -> None:
-		self._parser: ArgumentParser = self._define_arguments()
-		self._add_sub_parsers()
-
-		self._args: Arguments = self._parse_args()
-		config = self._parse_config()
-
-		try:
-			self._config = ArchConfig.from_config(config, self._args)
-			self._config.version = get_version()
-		except ValueError as err:
-			warn(str(err))
-			sys.exit(1)
-
-	@property
-	def config(self) -> ArchConfig:
-		return self._config
-
-	@property
-	def args(self) -> Arguments:
-		return self._args
-
-	def get_script(self) -> str:
-		if script := self.args.script:
-			return script
-
-		if script := self.config.script:
-			return script
-
-		return 'guided'
-
-	def print_help(self) -> None:
-		self._parser.print_help()
-
-	def _add_sub_parsers(self) -> None:
-		subparsers = self._parser.add_subparsers(dest='command', help='Available subcommands')
-		_ = subparsers.add_parser(SubCommand.SHARE_LOG.value, help='Upload log file to public server')
-
-	def _define_arguments(self) -> ArgumentParser:
-		parser = ArgumentParser(formatter_class=argparse.ArgumentDefaultsHelpFormatter)
-
-		parser.add_argument(
-			'-v',
-			'--version',
-			action='version',
-			default=False,
-			version='%(prog)s ' + get_version(),
-		)
-		parser.add_argument(
-			'--config',
-			type=Path,
-			nargs='?',
-			default=None,
-			help='JSON configuration file',
-		)
-		parser.add_argument(
-			'--config-url',
-			type=str,
-			nargs='?',
-			default=None,
-			help='Url to a JSON configuration file',
-		)
-		parser.add_argument(
-			'--creds',
-			type=Path,
-			nargs='?',
-			default=None,
-			help='JSON credentials configuration file',
-		)
-		parser.add_argument(
-			'--creds-url',
-			type=str,
-			nargs='?',
-			default=None,
-			help='Url to a JSON credentials configuration file',
-		)
-		parser.add_argument(
-			'--creds-decryption-key',
-			type=str,
-			nargs='?',
-			default=None,
-			help='Decryption key for credentials file',
-		)
-		parser.add_argument(
-			'--silent',
-			action='store_true',
-			default=False,
-			help='WARNING: Disables all prompts for input and confirmation. If no configuration is provided, this is ignored',
-		)
-		parser.add_argument(
-			'--dry-run',
-			'--dry_run',
-			action='store_true',
-			default=False,
-			help='Generates a configuration file and then exits instead of performing an installation',
-		)
-		parser.add_argument(
-			'--script',
-			nargs='?',
-			help='Script to run for installation',
-			type=str,
-		)
-		parser.add_argument(
-			'--mountpoint',
-			type=Path,
-			nargs='?',
-			default=Path('/mnt'),
-			help='Define an alternate mount point for installation',
-		)
-		parser.add_argument(
-			'--skip-ntp',
-			action='store_true',
-			help='Disables NTP checks during installation',
-			default=False,
-		)
-		parser.add_argument(
-			'--skip-wkd',
-			action='store_true',
-			help='Disables checking if archlinux keyring wkd sync is complete.',
-			default=False,
-		)
-		parser.add_argument(
-			'--skip-boot',
-			action='store_true',
-			help='Disables installation of a boot loader (note: only use this when problems arise with the boot loader step).',
-			default=False,
-		)
-		parser.add_argument(
-			'--debug',
-			action='store_true',
-			default=False,
-			help='Adds debug info into the log',
-		)
-		parser.add_argument(
-			'--offline',
-			action='store_true',
-			default=False,
-			help='Disabled online upstream services such as package search and key-ring auto update.',
-		)
-		parser.add_argument(
-			'--no-pkg-lookups',
-			action='store_true',
-			default=False,
-			help='Disabled package validation specifically prior to starting installation.',
-		)
-		parser.add_argument(
-			'--plugin',
-			nargs='?',
-			type=Path,
-			default=None,
-			help='File path to a plugin to load',
-		)
-		parser.add_argument(
-			'--plugin-url',
-			type=str,
-			nargs='?',
-			default=None,
-			help='Url to a plugin file to load',
-		)
-		parser.add_argument(
-			'--skip-version-check',
-			action='store_true',
-			default=False,
-			help='Skip the version check when running archinstall',
-		)
-		parser.add_argument(
-			'--skip-wifi-check',
-			action='store_true',
-			default=False,
-			help='Skip wifi check when running archinstall',
-		)
-		parser.add_argument(
-			'--advanced',
-			action='store_true',
-			default=False,
-			help='Enabled advanced options',
-		)
-		parser.add_argument(
-			'--verbose',
-			action='store_true',
-			default=False,
-			help='Enabled verbose options',
-		)
-		return parser
-
-	def _parse_args(self) -> Arguments:
-		argparse_args = vars(self._parser.parse_args())
-		args: Arguments = Arguments(**argparse_args)
-
-		# amend the parameters (check internal consistency)
-		# Installation can't be silent if config is not passed
-		if args.config is None and args.config_url is None:
-			args.silent = False
-
-		if args.debug:
-			warn(f'Warning: --debug mode will write certain credentials to {logger.path}!')
-
-		if args.plugin:
-			load_plugin(args.plugin)
-
-		if args.plugin_url:
-			plugin_data = self._fetch_from_url(args.plugin_url)
-			plugin_path = self._write_plugin_to_temp_file(plugin_data)
-			load_plugin(plugin_path)
-
-		if args.creds_decryption_key is None:
-			if os.environ.get('ARCHINSTALL_CREDS_DECRYPTION_KEY'):
-				args.creds_decryption_key = os.environ.get('ARCHINSTALL_CREDS_DECRYPTION_KEY')
-
-		return args
-
-	def _parse_config(self) -> dict[str, Any]:
-		config: dict[str, Any] = {}
-		config_data: str | None = None
-		creds_data: str | None = None
-
-		if self._args.config is not None:
-			config_data = self._read_file(self._args.config)
-		elif self._args.config_url is not None:
-			config_data = self._fetch_from_url(self._args.config_url)
-
-		if config_data is not None:
-			config.update(json.loads(config_data))
-
-		if self._args.creds is not None:
-			creds_data = self._read_file(self._args.creds)
-		elif self._args.creds_url is not None:
-			creds_data = self._fetch_from_url(self._args.creds_url)
-
-		if creds_data is not None:
-			json_data = self._process_creds_data(creds_data)
-			if json_data is not None:
-				config.update(json_data)
-
-		config = self._cleanup_config(config)
-
-		return config
-
-	def _process_creds_data(self, creds_data: str) -> dict[str, Any] | None:
-		if creds_data.startswith('$'):  # encrypted data
-			if self._args.creds_decryption_key is not None:
-				try:
-					creds_data = decrypt(creds_data, self._args.creds_decryption_key)
-					return json.loads(creds_data)
-				except ValueError as err:
-					if 'Invalid password' in str(err):
-						error(tr('Incorrect credentials file decryption password'))
-						sys.exit(1)
-					else:
-						debug(f'Error decrypting credentials file: {err}')
-						raise err from err
-			else:
-				header = tr('Enter credentials file decryption password')
-				wrong_pwd_text = tr('Incorrect password')
-				prompt = header
-
-				while True:
-					decryption_pwd: Password | None = tui.run(
-						lambda p=prompt: get_password(  # type: ignore[misc]
-							header=p,
-							allow_skip=False,
-							no_confirmation=True,
-						)
-					)
-
-					if not decryption_pwd:
-						return None
-
-					try:
-						creds_data = decrypt(creds_data, decryption_pwd.plaintext)
-						break
-					except ValueError as err:
-						if 'Invalid password' in str(err):
-							debug('Incorrect credentials file decryption password')
-							prompt = f'{header}' + f'\n\n{wrong_pwd_text}'
-						else:
-							debug(f'Error decrypting credentials file: {err}')
-							raise err from err
-
-		return json.loads(creds_data)
-
-	def _fetch_from_url(self, url: str) -> str:
-		if urllib.parse.urlparse(url).scheme:
-			try:
-				req = Request(url, headers={'User-Agent': 'ArchInstall'})
-				with urlopen(req) as resp:
-					return resp.read().decode('utf-8')
-			except urllib.error.HTTPError as err:
-				error(f'Could not fetch JSON from {url}: {err}')
-		else:
-			error('Not a valid url')
-
-		sys.exit(1)
-
-	def _write_plugin_to_temp_file(self, plugin_data: str) -> Path:
-		if not plugin_data.strip():
-			error('The downloaded plugin is empty')
-			sys.exit(1)
-
-		tmp_file = tempfile.NamedTemporaryFile(
-			mode='w',
-			suffix='.py',
-			prefix='archinstall_plugin_',
-			delete=False,
-		)
-
-		try:
-			with tmp_file as f:
-				f.write(plugin_data)
-		except OSError as err:
-			error(f'Could not write the downloaded plugin to a temporary file: {err}')
-			sys.exit(1)
-
-		return Path(tmp_file.name)
-
-	def _read_file(self, path: Path) -> str:
-		if not path.exists():
-			error(f'Could not find file {path}')
-			sys.exit(1)
-
-		return path.read_text()
-
-	def _cleanup_config(self, config: Namespace | dict[str, Any]) -> dict[str, Any]:
-		clean_args = {}
-		for key, val in config.items():
-			if isinstance(val, dict):
-				val = self._cleanup_config(val)
-
-			if val is not None:
-				clean_args[key] = val
-
-		return clean_args

archinstall/lib/authentication/authentication_handler.py

@@ -1,126 +0,0 @@
-import getpass
-from pathlib import Path
-from typing import TYPE_CHECKING
-
-from archinstall.lib.command import SysCommandWorker
-from archinstall.lib.log import debug, info
-from archinstall.lib.models.authentication import AuthenticationConfiguration, U2FLoginConfiguration, U2FLoginMethod
-from archinstall.lib.models.users import User
-from archinstall.lib.translationhandler import tr
-
-if TYPE_CHECKING:
-	from archinstall.lib.installer import Installer
-
-
-class AuthenticationHandler:
-	def setup_auth(
-		self,
-		install_session: Installer,
-		auth_config: AuthenticationConfiguration,
-		hostname: str,
-	) -> None:
-		if auth_config.u2f_config and auth_config.users is not None:
-			self._setup_u2f_login(install_session, auth_config.u2f_config, auth_config.users, hostname)
-
-	def _setup_u2f_login(self, install_session: Installer, u2f_config: U2FLoginConfiguration, users: list[User], hostname: str) -> None:
-		self._configure_u2f_mapping(install_session, u2f_config, users, hostname)
-		self._update_pam_config(install_session, u2f_config)
-
-	def _update_pam_config(
-		self,
-		install_session: Installer,
-		u2f_config: U2FLoginConfiguration,
-	) -> None:
-		match u2f_config.u2f_login_method:
-			case U2FLoginMethod.Passwordless:
-				config_entry = 'auth sufficient pam_u2f.so authfile=/etc/u2f_mappings cue'
-			case U2FLoginMethod.SecondFactor:
-				config_entry = 'auth required pam_u2f.so authfile=/etc/u2f_mappings cue'
-			case _:
-				raise ValueError(f'Unknown U2F login method: {u2f_config.u2f_login_method}')
-
-		debug(f'U2F PAM configuration: {config_entry}')
-		debug(f'Passwordless sudo enabled: {u2f_config.passwordless_sudo}')
-
-		sudo_config = install_session.target / 'etc/pam.d/sudo'
-		sys_login = install_session.target / 'etc/pam.d/system-login'
-
-		if u2f_config.passwordless_sudo:
-			self._add_u2f_entry(sudo_config, config_entry)
-
-		self._add_u2f_entry(sys_login, config_entry)
-
-	def _add_u2f_entry(self, file: Path, entry: str) -> None:
-		if not file.exists():
-			debug(f'File does not exist: {file}')
-			return
-
-		content = file.read_text().splitlines()
-
-		# remove any existing u2f auth entry
-		content = [line for line in content if 'pam_u2f.so' not in line]
-
-		# add the u2f auth entry as the first one after comments
-		for i, line in enumerate(content):
-			if not line.startswith('#'):
-				content.insert(i, entry)
-				break
-		else:
-			content.append(entry)
-
-		file.write_text('\n'.join(content) + '\n')
-
-	def _configure_u2f_mapping(
-		self,
-		install_session: Installer,
-		u2f_config: U2FLoginConfiguration,
-		users: list[User],
-		hostname: str,
-	) -> None:
-		debug(f'Setting up U2F login: {u2f_config.u2f_login_method.value}')
-
-		install_session.pacman.strap('pam-u2f')
-
-		print(tr('Setting up U2F login: {}').format(u2f_config.u2f_login_method.value))
-
-		# https://developers.yubico.com/pam-u2f/
-		u2f_auth_file = install_session.target / 'etc/u2f_mappings'
-		u2f_auth_file.touch()
-		existing_keys = u2f_auth_file.read_text()
-
-		registered_keys: list[str] = []
-
-		for user in users:
-			print('')
-			info(tr('Setting up U2F device for user: {}').format(user.username))
-			info(tr('You may need to enter the PIN and then touch your U2F device to register it'))
-
-			cmd = ' '.join(
-				['arch-chroot', '-S', str(install_session.target), 'pamu2fcfg', '-u', user.username, '-o', f'pam://{hostname}', '-i', f'pam://{hostname}']
-			)
-
-			debug(f'Enrolling U2F device: {cmd}')
-
-			worker = SysCommandWorker(cmd, peek_output=True)
-			pin_inputted = False
-
-			while worker.is_alive():
-				if pin_inputted is False:
-					if bytes('enter pin for', 'UTF-8') in worker._trace_log.lower():
-						worker.write(bytes(getpass.getpass(''), 'UTF-8'))
-						pin_inputted = True
-
-			output = worker.decode().strip().splitlines()
-			debug(f'Output from pamu2fcfg: {output}')
-
-			key = output[-1].strip()
-			registered_keys.append(key)
-
-		all_keys = '\n'.join(registered_keys)
-
-		if existing_keys:
-			existing_keys += f'\n{all_keys}'
-		else:
-			existing_keys = all_keys
-
-		u2f_auth_file.write_text(existing_keys)

archinstall/lib/authentication/authentication_menu.py

@@ -1,147 +0,0 @@
-from typing import override
-
-from archinstall.lib.disk.fido import Fido2
-from archinstall.lib.menu.abstract_menu import AbstractSubMenu
-from archinstall.lib.menu.helpers import Confirmation, Selection
-from archinstall.lib.menu.util import get_password
-from archinstall.lib.models.authentication import AuthenticationConfiguration, U2FLoginConfiguration, U2FLoginMethod
-from archinstall.lib.models.users import Password, User
-from archinstall.lib.translationhandler import tr
-from archinstall.lib.user.user_menu import select_users
-from archinstall.lib.utils.format import as_table
-from archinstall.tui.menu_item import MenuItem, MenuItemGroup
-from archinstall.tui.result import ResultType
-
-
-class AuthenticationMenu(AbstractSubMenu[AuthenticationConfiguration]):
-	def __init__(self, preset: AuthenticationConfiguration | None = None):
-		if preset:
-			self._auth_config = preset
-		else:
-			self._auth_config = AuthenticationConfiguration()
-
-		menu_options = self._define_menu_options()
-		self._item_group = MenuItemGroup(menu_options, checkmarks=True)
-
-		super().__init__(
-			self._item_group,
-			config=self._auth_config,
-			allow_reset=True,
-		)
-
-	@override
-	async def show(self) -> AuthenticationConfiguration | None:
-		return await super().show()
-
-	def _define_menu_options(self) -> list[MenuItem]:
-		return [
-			MenuItem(
-				text=tr('Root password'),
-				action=lambda x: select_root_password(),
-				preview_action=self._prev_root_pwd,
-				key='root_enc_password',
-			),
-			MenuItem(
-				text=tr('User account'),
-				action=self._create_user_account,
-				preview_action=self._prev_users,
-				key='users',
-			),
-			MenuItem(
-				text=tr('U2F login setup'),
-				action=select_u2f_login,
-				value=self._auth_config.u2f_config,
-				preview_action=self._prev_u2f_login,
-				key='u2f_config',
-			),
-		]
-
-	async def _create_user_account(self, preset: list[User] | None = None) -> list[User]:
-		preset = [] if preset is None else preset
-		users = await select_users(preset=preset)
-		return users
-
-	def _prev_users(self, item: MenuItem) -> str | None:
-		users: list[User] | None = item.value
-
-		if users:
-			return as_table(users)
-		return None
-
-	def _prev_root_pwd(self, item: MenuItem) -> str | None:
-		if item.value is not None:
-			password: Password = item.value
-			return f'{tr("Root password")}: {password.hidden()}'
-		return None
-
-	def _depends_on_u2f(self) -> bool:
-		devices = Fido2.get_fido2_devices()
-		if not devices:
-			return False
-		return True
-
-	def _prev_u2f_login(self, item: MenuItem) -> str | None:
-		if item.value is not None:
-			u2f_config: U2FLoginConfiguration = item.value
-
-			login_method = u2f_config.u2f_login_method.display_value()
-			output = tr('U2F login method: ') + login_method
-
-			output += '\n'
-			output += tr('Passwordless sudo: ') + (tr('Enabled') if u2f_config.passwordless_sudo else tr('Disabled'))
-
-			return output
-
-		devices = Fido2.get_fido2_devices()
-		if not devices:
-			return tr('No U2F devices found')
-
-		return None
-
-
-async def select_root_password() -> Password | None:
-	password = await get_password(header=tr('Enter root password'), allow_skip=True)
-	return password
-
-
-async def select_u2f_login(preset: U2FLoginConfiguration | None) -> U2FLoginConfiguration | None:
-	devices = Fido2.get_fido2_devices()
-	if not devices:
-		return None
-
-	items = []
-	for method in U2FLoginMethod:
-		items.append(MenuItem(method.display_value(), value=method))
-
-	group = MenuItemGroup(items)
-
-	if preset is not None:
-		group.set_selected_by_value(preset.u2f_login_method)
-
-	result = await Selection[U2FLoginMethod](
-		group,
-		allow_skip=True,
-		allow_reset=True,
-	).show()
-
-	match result.type_:
-		case ResultType.Selection:
-			u2f_method = result.get_value()
-			header = tr('Enable passwordless sudo?')
-
-			result_sudo = await Confirmation(
-				header=header,
-				allow_skip=True,
-				preset=False,
-			).show()
-
-			passwordless_sudo = result_sudo.item() == MenuItem.yes()
-
-			return U2FLoginConfiguration(
-				u2f_login_method=u2f_method,
-				passwordless_sudo=passwordless_sudo,
-			)
-		case ResultType.Skip:
-			return preset
-		case ResultType.Reset:
-			return None

archinstall/lib/boot.py

@@ -1,48 +1,37 @@
 import time
-from collections.abc import Iterator
+from typing import Iterator, Optional
-from pathlib import Path
+from .exceptions import SysCallError
-from types import TracebackType
+from .general import SysCommand, SysCommandWorker, locate_binary
-from typing import ClassVar, Self
+from .installer import Installer
-
+from .output import error
-from archinstall.lib.command import SysCommand, SysCommandWorker
+from .storage import storage
-from archinstall.lib.exceptions import SysCallError
-from archinstall.lib.log import error
 
 
 class Boot:
-	_active_boot: ClassVar[Self | None] = None
+	def __init__(self, installation: Installer):
-
+		self.instance = installation
-	def __init__(self, path: Path | str):
-		if isinstance(path, Path):
-			path = str(path)
-
-		self.path = path
 		self.container_name = 'archinstall'
-		self.session: SysCommandWorker | None = None
+		self.session: Optional[SysCommandWorker] = None
 		self.ready = False
 
-	def __enter__(self) -> Self:
+	def __enter__(self) -> 'Boot':
-		if Boot._active_boot and Boot._active_boot.path != self.path:
+		if (existing_session := storage.get('active_boot', None)) and existing_session.instance != self.instance:
-			raise KeyError('Archinstall only supports booting up one instance and another session is already active.')
+			raise KeyError("Archinstall only supports booting up one instance, and a active session is already active and it is not this one.")
 
-		if Boot._active_boot:
+		if existing_session:
-			self.session = Boot._active_boot.session
+			self.session = existing_session.session
-			self.ready = Boot._active_boot.ready
+			self.ready = existing_session.ready
 		else:
 			# '-P' or --console=pipe  could help us not having to do a bunch
 			# of os.write() calls, but instead use pipes (stdin, stdout and stderr) as usual.
-			self.session = SysCommandWorker(
+			self.session = SysCommandWorker([
-				[
+				'/usr/bin/systemd-nspawn',
-					'systemd-nspawn',
+				'-D', str(self.instance.target),
-					'-D',
+				'--timezone=off',
-					self.path,
+				'-b',
-					'--timezone=off',
+				'--no-pager',
-					'-b',
+				'--machine', self.container_name
-					'--no-pager',
+			])
-					'--machine',
-					self.container_name,
-				]
-			)
 
 		if not self.ready and self.session:
 			while self.session.is_alive():
@@ -50,21 +39,21 @@ class Boot:
 					self.ready = True
 					break
 
-		Boot._active_boot = self
+		storage['active_boot'] = self
 		return self
 
-	def __exit__(self, exc_type: type[BaseException] | None, exc_value: BaseException | None, traceback: TracebackType | None) -> None:
+	def __exit__(self, *args :str, **kwargs :str) -> None:
 		# b''.join(sys_command('sync')) # No need to, since the underlying fs() object will call sync.
 		# TODO: https://stackoverflow.com/questions/28157929/how-to-safely-handle-an-exception-inside-a-context-manager
 
-		if exc_type is not None:
+		if len(args) >= 2 and args[1]:
 			error(
-				str(exc_value),
+				args[1],
-				f'The error above occurred in a temporary boot-up of the installation {self.path!r}',
+				f"The error above occurred in a temporary boot-up of the installation {self.instance}"
 			)
 
 		shutdown = None
-		shutdown_exit_code: int | None = -1
+		shutdown_exit_code: Optional[int] = -1
 
 		try:
 			shutdown = SysCommand(f'systemd-run --machine={self.container_name} --pty shutdown now')
@@ -79,18 +68,19 @@ class Boot:
 			shutdown_exit_code = shutdown.exit_code
 
 		if self.session and (self.session.exit_code == 0 or shutdown_exit_code == 0):
-			Boot._active_boot = None
+			storage['active_boot'] = None
 		else:
 			session_exit_code = self.session.exit_code if self.session else -1
 
 			raise SysCallError(
-				f'Could not shut down temporary boot of {self.path!r}: {session_exit_code}/{shutdown_exit_code}',
+				f"Could not shut down temporary boot of {self.instance}: {session_exit_code}/{shutdown_exit_code}",
-				exit_code=next(filter(bool, [session_exit_code, shutdown_exit_code])),
+				exit_code=next(filter(bool, [session_exit_code, shutdown_exit_code]))
 			)
 
 	def __iter__(self) -> Iterator[bytes]:
 		if self.session:
-			yield from self.session
+			for value in self.session:
+				yield value
 
 	def __contains__(self, key: bytes) -> bool:
 		if self.session is None:
@@ -104,8 +94,18 @@ class Boot:
 
 		return self.session.is_alive()
 
-	def SysCommand(self, cmd: list[str], *args, **kwargs) -> SysCommand:  # type: ignore[no-untyped-def]
+	def SysCommand(self, cmd: list, *args, **kwargs) -> SysCommand:
-		return SysCommand(['systemd-run', f'--machine={self.container_name}', '--pty', *cmd], *args, **kwargs)
+		if cmd[0][0] != '/' and cmd[0][:2] != './':
+			# This check is also done in SysCommand & SysCommandWorker.
+			# However, that check is done for `machinectl` and not for our chroot command.
+			# So this wrapper for SysCommand will do this additionally.
 
-	def SysCommandWorker(self, cmd: list[str], *args, **kwargs) -> SysCommandWorker:  # type: ignore[no-untyped-def]
+			cmd[0] = locate_binary(cmd[0])
-		return SysCommandWorker(['systemd-run', f'--machine={self.container_name}', '--pty', *cmd], *args, **kwargs)
+
+		return SysCommand(["systemd-run", f"--machine={self.container_name}", "--pty", *cmd], *args, **kwargs)
+
+	def SysCommandWorker(self, cmd: list, *args, **kwargs) -> SysCommandWorker:
+		if cmd[0][0] != '/' and cmd[0][:2] != './':
+			cmd[0] = locate_binary(cmd[0])
+
+		return SysCommandWorker(["systemd-run", f"--machine={self.container_name}", "--pty", *cmd], *args, **kwargs)

archinstall/lib/bootloader/bootloader_menu.py

@@ -1,281 +0,0 @@
-import textwrap
-from typing import override
-
-from archinstall.lib.menu.abstract_menu import AbstractSubMenu
-from archinstall.lib.menu.helpers import Confirmation, Selection
-from archinstall.lib.models.bootloader import Bootloader, BootloaderConfiguration, PlymouthTheme
-from archinstall.lib.translationhandler import tr
-from archinstall.tui.menu_item import MenuItem, MenuItemGroup
-from archinstall.tui.result import ResultType
-
-
-class BootloaderMenu(AbstractSubMenu[BootloaderConfiguration]):
-	def __init__(
-		self,
-		bootloader_conf: BootloaderConfiguration,
-		uefi: bool,
-		skip_boot: bool = False,
-	):
-		self._bootloader_conf = bootloader_conf
-		self._skip_boot = skip_boot
-		self._uefi = uefi
-		menu_options = self._define_menu_options()
-
-		self._item_group = MenuItemGroup(menu_options, sort_items=False, checkmarks=True)
-		super().__init__(
-			self._item_group,
-			config=self._bootloader_conf,
-			allow_reset=False,
-		)
-
-	def _define_menu_options(self) -> list[MenuItem]:
-		bootloader = self._bootloader_conf.bootloader
-
-		# UKI availability
-		uki_enabled = self._uefi and bootloader.has_uki_support()
-		if not uki_enabled:
-			self._bootloader_conf.uki = False
-
-		# Removable availability
-		removable_enabled = self._uefi and bootloader.has_removable_support()
-		if not removable_enabled:
-			self._bootloader_conf.removable = False
-
-		return [
-			MenuItem(
-				text=tr('Bootloader'),
-				action=self._select_bootloader,
-				value=self._bootloader_conf.bootloader,
-				preview_action=self._prev_bootloader,
-				mandatory=True,
-				key='bootloader',
-			),
-			MenuItem(
-				text=tr('Unified kernel images'),
-				action=self._select_uki,
-				value=self._bootloader_conf.uki,
-				preview_action=self._prev_uki,
-				key='uki',
-				enabled=uki_enabled,
-			),
-			MenuItem(
-				text=tr('Install to removable location'),
-				action=self._select_removable,
-				value=self._bootloader_conf.removable,
-				preview_action=self._prev_removable,
-				key='removable',
-				enabled=removable_enabled,
-			),
-			MenuItem(
-				text=tr('Plymouth'),
-				action=self._select_plymouth,
-				value=self._bootloader_conf.plymouth,
-				preview_action=self._prev_plymouth,
-				key='plymouth',
-			),
-		]
-
-	def _prev_bootloader(self, item: MenuItem) -> str | None:
-		if item.value:
-			return f'{tr("Bootloader")}: {item.value.value}'
-		return None
-
-	def _prev_uki(self, item: MenuItem) -> str | None:
-		uki_text = f'{tr("Unified kernel images")}'
-		if item.value:
-			return f'{uki_text}: {tr("Enabled")}'
-		else:
-			return f'{uki_text}: {tr("Disabled")}'
-
-	def _prev_removable(self, item: MenuItem) -> str | None:
-		if item.value:
-			return tr('Will install to /EFI/BOOT/ (removable location, safe default)')
-		return tr('Will install to custom location with NVRAM entry')
-
-	def _prev_plymouth(self, item: MenuItem) -> str | None:
-		if item.value:
-			return f'{tr("Plymouth")}: {item.value.value}'
-		return None
-
-	@override
-	async def show(self) -> BootloaderConfiguration:
-		_ = await super().show()
-		return self._bootloader_conf
-
-	async def _select_bootloader(self, preset: Bootloader | None) -> Bootloader | None:
-		bootloader = await select_bootloader(preset, self._uefi, self._skip_boot)
-
-		if bootloader:
-			# Update UKI option based on bootloader
-			uki_item = self._menu_item_group.find_by_key('uki')
-			if not self._uefi or not bootloader.has_uki_support():
-				uki_item.enabled = False
-				uki_item.value = False
-				self._bootloader_conf.uki = False
-			else:
-				uki_item.enabled = True
-
-			# Update removable option based on bootloader
-			removable_item = self._menu_item_group.find_by_key('removable')
-			if not self._uefi or not bootloader.has_removable_support():
-				removable_item.enabled = False
-				removable_item.value = False
-				self._bootloader_conf.removable = False
-			else:
-				if not removable_item.enabled:
-					removable_item.value = True
-					self._bootloader_conf.removable = True
-				removable_item.enabled = True
-
-		return bootloader
-
-	async def _select_plymouth(self, preset: PlymouthTheme | None) -> PlymouthTheme | None:
-		# Plymouth is purely cosmetic and a frequent source of boot breakage
-		# (notably with the NVIDIA driver and disk encryption), so confirm before
-		# enabling it. When it is already enabled the user is only changing the
-		# theme, so the warning is skipped.
-		if preset is None:
-			prompt = (
-				'[ansi_bright_yellow]'
-				+ tr('Plymouth adds a cosmetic boot splash but can cause boot problems on some setups:')
-				+ '\n\n  • '
-				+ tr('black screen with the NVIDIA driver')
-				+ '\n  • '
-				+ tr('hidden password prompt with disk encryption (LUKS)')
-				+ '\n\n'
-				+ tr('Would you like to enable it?')
-				+ '[/]\n'
-			)
-
-			result = await Confirmation(header=prompt, allow_skip=True, preset=False).show()
-
-			match result.type_:
-				case ResultType.Skip:
-					return preset
-				case ResultType.Selection:
-					if not result.get_value():
-						return preset
-				case ResultType.Reset:
-					raise ValueError('Unhandled result type')
-
-		return await select_plymouth_theme(preset)
-
-	async def _select_uki(self, preset: bool) -> bool:
-		prompt = tr('Would you like to use unified kernel images?') + '\n'
-
-		result = await Confirmation(header=prompt, allow_skip=True, preset=preset).show()
-
-		match result.type_:
-			case ResultType.Skip:
-				return preset
-			case ResultType.Selection:
-				return result.item() == MenuItem.yes()
-			case ResultType.Reset:
-				raise ValueError('Unhandled result type')
-
-	async def _select_removable(self, preset: bool) -> bool:
-		prompt = (
-			tr('Would you like to install the bootloader to the default removable media search location?')
-			+ '\n\n'
-			+ tr('This installs the bootloader to /EFI/BOOT/BOOTX64.EFI (or similar) which is useful for:')
-			+ '\n\n  • '
-			+ tr('Firmware that does not properly support NVRAM boot entries like most MSI motherboards,')
-			+ '\n	 '
-			+ tr('most Apple Macs, many laptops...')
-			+ '\n  • '
-			+ tr('USB drives or other portable external media.')
-			+ '\n  • '
-			+ tr('Systems where you want the disk to be bootable on any computer.')
-			+ '\n\n'
-			+ tr(
-				textwrap.dedent(
-					"""\
-					If you do not know what this means, LEAVE THIS OPTION ENABLED, as it is the safe default.
-
-					It is suggested to disable this if none of the above apply, as it makes installing multiple
-					EFI bootloaders on the same disk easier, and it will not overwrite whatever bootloader
-					was previously installed at the default removable media search location, if any.
-
-					It may also make the installation more resilient in case of dual-booting with Windows,
-					as Windows is known to sometimes erase or replace the bootloader installed at the removable
-					location.
-					"""
-				)
-			)
-			+ '\n'
-		)
-
-		result = await Confirmation(
-			header=prompt,
-			allow_skip=True,
-			preset=preset,
-		).show()
-
-		match result.type_:
-			case ResultType.Skip:
-				return preset
-			case ResultType.Selection:
-				return result.get_value()
-			case ResultType.Reset:
-				raise ValueError('Unhandled result type')
-
-
-async def select_bootloader(
-	preset: Bootloader | None,
-	uefi: bool,
-	skip_boot: bool = False,
-) -> Bootloader | None:
-	options = []
-	hidden_options = []
-	header = tr('Select bootloader to install')
-
-	default = Bootloader.get_default(uefi, skip_boot)
-
-	if not skip_boot:
-		hidden_options += [Bootloader.NO_BOOTLOADER]
-
-	if not uefi:
-		options += [Bootloader.Grub, Bootloader.Limine]
-		header += '\n' + tr('UEFI is not detected and some options are disabled')
-	else:
-		options += [b for b in Bootloader if b not in hidden_options]
-
-	items = [MenuItem(o.value, value=o) for o in options]
-	group = MenuItemGroup(items)
-	group.set_default_by_value(default)
-	group.set_focus_by_value(preset)
-
-	result = await Selection[Bootloader](
-		group,
-		header=header,
-		allow_skip=True,
-	).show()
-
-	match result.type_:
-		case ResultType.Skip:
-			return preset
-		case ResultType.Selection:
-			return result.get_value()
-		case ResultType.Reset:
-			raise ValueError('Unhandled result type')
-
-
-async def select_plymouth_theme(preset: PlymouthTheme | None = None) -> PlymouthTheme | None:
-	items = [MenuItem(t.value, value=t) for t in PlymouthTheme]
-	group = MenuItemGroup(items, sort_items=False)
-	group.set_focus_by_value(preset)
-
-	result = await Selection[PlymouthTheme](
-		group,
-		header=tr('Select Plymouth theme'),
-		allow_reset=True,
-		allow_skip=True,
-	).show()
-
-	match result.type_:
-		case ResultType.Skip:
-			return preset
-		case ResultType.Reset:
-			return None
-		case ResultType.Selection:
-			return PlymouthTheme(result.get_value())

archinstall/lib/bootloader/utils.py

@@ -1,86 +0,0 @@
-from dataclasses import dataclass
-from enum import Enum, auto
-from pathlib import Path
-
-from archinstall.lib.hardware import SysInfo
-from archinstall.lib.models.bootloader import Bootloader, BootloaderConfiguration
-from archinstall.lib.models.device import DiskLayoutConfiguration
-
-
-class BootloaderValidationFailureKind(Enum):
-	LimineNonFatBoot = auto()
-	LimineLayout = auto()
-	BootloaderRequiresUefi = auto()
-	EfistubNonFatBoot = auto()
-
-
-@dataclass(frozen=True)
-class BootloaderValidationFailure:
-	kind: BootloaderValidationFailureKind
-	description: str
-
-
-def validate_bootloader_layout(
-	bootloader_config: BootloaderConfiguration | None,
-	disk_config: DiskLayoutConfiguration | None,
-) -> BootloaderValidationFailure | None:
-	"""Validate bootloader configuration against disk layout.
-
-	Returns a failure with a human-readable description if the configuration
-	would produce an unbootable system, or None if it is valid.
-	"""
-	if not (bootloader_config and disk_config):
-		return None
-
-	bootloader = bootloader_config.bootloader
-
-	if bootloader == Bootloader.NO_BOOTLOADER:
-		return None
-
-	if bootloader.is_uefi_only() and not SysInfo.has_uefi():
-		return BootloaderValidationFailure(
-			kind=BootloaderValidationFailureKind.BootloaderRequiresUefi,
-			description=f'{bootloader.value} requires a UEFI system.',
-		)
-
-	boot_part = next(
-		(p for m in disk_config.device_modifications if (p := m.get_boot_partition())),
-		None,
-	)
-
-	if bootloader == Bootloader.Efistub:
-		# The UEFI firmware reads the kernel directly from the boot partition,
-		# which must be FAT.
-		if boot_part and (boot_part.fs_type is None or not boot_part.fs_type.is_fat()):
-			return BootloaderValidationFailure(
-				kind=BootloaderValidationFailureKind.EfistubNonFatBoot,
-				description='Efistub does not support booting with a non-FAT boot partition.',
-			)
-
-	if bootloader == Bootloader.Limine:
-		# Limine reads its config and kernels from the boot partition, which
-		# must be FAT.
-		if boot_part and (boot_part.fs_type is None or not boot_part.fs_type.is_fat()):
-			return BootloaderValidationFailure(
-				kind=BootloaderValidationFailureKind.LimineNonFatBoot,
-				description='Limine does not support booting with a non-FAT boot partition.',
-			)
-
-		# When the ESP is the boot partition but mounted outside /boot and
-		# UKI is disabled, kernels end up on the root filesystem which
-		# Limine cannot access.
-		if not bootloader_config.uki:
-			efi_part = next(
-				(p for m in disk_config.device_modifications if (p := m.get_efi_partition())),
-				None,
-			)
-			if efi_part and efi_part == boot_part and efi_part.mountpoint != Path('/boot'):
-				return BootloaderValidationFailure(
-					kind=BootloaderValidationFailureKind.LimineLayout,
-					description=(
-						f'Limine requires kernels on a FAT partition. The ESP is mounted at {efi_part.mountpoint}, '
-						'enable UKI or add a separate /boot partition to install Limine.'
-					),
-				)
-
-	return None

archinstall/lib/command.py

@@ -1,384 +0,0 @@
-import os
-import shlex
-import stat
-import subprocess
-import sys
-import time
-from collections.abc import Iterator
-from select import EPOLLHUP, EPOLLIN, epoll
-from shutil import which
-from types import TracebackType
-from typing import Any, Self, override
-
-from archinstall.lib.exceptions import RequirementError, SysCallError
-from archinstall.lib.log import debug, error, logger
-from archinstall.lib.utils.encoding import clear_vt100_escape_codes
-
-
-class SysCommandWorker:
-	def __init__(
-		self,
-		cmd: str | list[str],
-		peek_output: bool | None = False,
-		environment_vars: dict[str, str] | None = None,
-		working_directory: str = './',
-		remove_vt100_escape_codes_from_lines: bool = True,
-	):
-		if isinstance(cmd, str):
-			cmd = shlex.split(cmd)
-
-		if cmd and not cmd[0].startswith(('/', './')):  # Path() does not work well
-			cmd[0] = locate_binary(cmd[0])
-
-		self.cmd = cmd
-		self.peek_output = peek_output
-		# define the standard locale for command outputs. For now the C ascii one. Can be overridden
-		self.environment_vars = {'LC_ALL': 'C'}
-		if environment_vars:
-			self.environment_vars.update(environment_vars)
-
-		self.working_directory = working_directory
-
-		self.exit_code: int | None = None
-		self._trace_log = b''
-		self._trace_log_pos = 0
-		self.poll_object = epoll()
-		self.child_fd: int | None = None
-		self.started = False
-		self.ended = False
-		self.remove_vt100_escape_codes_from_lines: bool = remove_vt100_escape_codes_from_lines
-
-	def __contains__(self, key: bytes) -> bool:
-		"""
-		Contains will also move the current buffert position forward.
-		This is to avoid re-checking the same data when looking for output.
-		"""
-		assert isinstance(key, bytes)
-
-		index = self._trace_log.find(key, self._trace_log_pos)
-		if index >= 0:
-			self._trace_log_pos += index + len(key)
-			return True
-
-		return False
-
-	def __iter__(self, *args: str, **kwargs: dict[str, Any]) -> Iterator[bytes]:
-		last_line = self._trace_log.rfind(b'\n')
-		lines = filter(None, self._trace_log[self._trace_log_pos : last_line].splitlines())
-		for line in lines:
-			if self.remove_vt100_escape_codes_from_lines:
-				line = clear_vt100_escape_codes(line)
-
-			yield line + b'\n'
-
-		self._trace_log_pos = last_line
-
-	@override
-	def __repr__(self) -> str:
-		self.make_sure_we_are_executing()
-		return str(self._trace_log)
-
-	@override
-	def __str__(self) -> str:
-		try:
-			return self._trace_log.decode('utf-8')
-		except UnicodeDecodeError:
-			return str(self._trace_log)
-
-	def __enter__(self) -> Self:
-		return self
-
-	def __exit__(self, exc_type: type[BaseException] | None, exc_value: BaseException | None, traceback: TracebackType | None) -> None:
-		# b''.join(sys_command('sync')) # No need to, since the underlying fs() object will call sync.
-		# TODO: https://stackoverflow.com/questions/28157929/how-to-safely-handle-an-exception-inside-a-context-manager
-
-		if self.child_fd:
-			try:
-				os.close(self.child_fd)
-			except Exception:
-				pass
-
-		if self.peek_output:
-			# To make sure any peaked output didn't leave us hanging
-			# on the same line we were on.
-			sys.stdout.write('\n')
-			sys.stdout.flush()
-
-		if exc_type is not None:
-			debug(str(exc_value))
-
-		if self.exit_code != 0:
-			raise SysCallError(
-				f'{self.cmd} exited with abnormal exit code [{self.exit_code}]: {str(self)[-500:]}',
-				self.exit_code,
-				worker_log=self._trace_log,
-			)
-
-	def is_alive(self) -> bool:
-		self.poll()
-
-		if self.started and not self.ended:
-			return True
-
-		return False
-
-	def write(self, data: bytes, line_ending: bool = True) -> int:
-		assert isinstance(data, bytes)  # TODO: Maybe we can support str as well and encode it
-
-		self.make_sure_we_are_executing()
-
-		if self.child_fd:
-			return os.write(self.child_fd, data + (b'\n' if line_ending else b''))
-
-		return 0
-
-	def make_sure_we_are_executing(self) -> bool:
-		if not self.started:
-			return self.execute()
-		return True
-
-	def tell(self) -> int:
-		self.make_sure_we_are_executing()
-		return self._trace_log_pos
-
-	def seek(self, pos: int) -> None:
-		self.make_sure_we_are_executing()
-		# Safety check to ensure 0 < pos < len(tracelog)
-		self._trace_log_pos = min(max(0, pos), len(self._trace_log))
-
-	def peak(self, output: str | bytes) -> bool:
-		if self.peek_output:
-			if isinstance(output, bytes):
-				try:
-					output = output.decode('UTF-8')
-				except UnicodeDecodeError:
-					return False
-
-			_cmd_output(output)
-
-			sys.stdout.write(output)
-			sys.stdout.flush()
-
-		return True
-
-	def poll(self) -> None:
-		self.make_sure_we_are_executing()
-
-		if self.child_fd:
-			got_output = False
-			for _fileno, _event in self.poll_object.poll(0.1):
-				try:
-					output = os.read(self.child_fd, 8192)
-					got_output = True
-					self.peak(output)
-					self._trace_log += output
-				except OSError:
-					self.ended = True
-					break
-
-			if self.ended or (not got_output and not _pid_exists(self.pid)):
-				self.ended = True
-				try:
-					wait_status = os.waitpid(self.pid, 0)[1]
-					self.exit_code = os.waitstatus_to_exitcode(wait_status)
-				except ChildProcessError:
-					try:
-						wait_status = os.waitpid(self.child_fd, 0)[1]
-						self.exit_code = os.waitstatus_to_exitcode(wait_status)
-					except ChildProcessError:
-						self.exit_code = 1
-
-	def execute(self) -> bool:
-		import pty
-
-		if (old_dir := os.getcwd()) != self.working_directory:
-			os.chdir(str(self.working_directory))
-
-		# Note: If for any reason, we get a Python exception between here
-		# and until os.close(), the traceback will get locked inside
-		# stdout of the child_fd object. `os.read(self.child_fd, 8192)` is the
-		# only way to get the traceback without losing it.
-
-		self.pid, self.child_fd = pty.fork()
-
-		# https://stackoverflow.com/questions/4022600/python-pty-fork-how-does-it-work
-		if not self.pid:
-			_cmd_history(self.cmd)
-
-			try:
-				os.execve(self.cmd[0], list(self.cmd), {**os.environ, **self.environment_vars})
-			except FileNotFoundError:
-				error(f'{self.cmd[0]} does not exist.')
-				self.exit_code = 1
-				return False
-		else:
-			# Only parent process moves back to the original working directory
-			os.chdir(old_dir)
-
-		self.started = True
-		self.poll_object.register(self.child_fd, EPOLLIN | EPOLLHUP)
-
-		return True
-
-	def decode(self, encoding: str = 'UTF-8') -> str:
-		return self._trace_log.decode(encoding)
-
-
-class SysCommand:
-	def __init__(
-		self,
-		cmd: str | list[str],
-		peek_output: bool | None = False,
-		environment_vars: dict[str, str] | None = None,
-		working_directory: str = './',
-		remove_vt100_escape_codes_from_lines: bool = True,
-	):
-		self.cmd = cmd
-		self.peek_output = peek_output
-		self.environment_vars = environment_vars
-		self.working_directory = working_directory
-		self.remove_vt100_escape_codes_from_lines = remove_vt100_escape_codes_from_lines
-
-		self.session: SysCommandWorker | None = None
-		self.create_session()
-
-	def __enter__(self) -> SysCommandWorker | None:
-		return self.session
-
-	def __exit__(self, exc_type: type[BaseException] | None, exc_value: BaseException | None, traceback: TracebackType | None) -> None:
-		# b''.join(sys_command('sync')) # No need to, since the underlying fs() object will call sync.
-		# TODO: https://stackoverflow.com/questions/28157929/how-to-safely-handle-an-exception-inside-a-context-manager
-
-		if exc_type is not None:
-			error(str(exc_value))
-
-	def __iter__(self, *args: list[Any], **kwargs: dict[str, Any]) -> Iterator[bytes]:
-		if self.session:
-			yield from self.session
-
-	def __getitem__(self, key: slice) -> bytes:
-		if not self.session:
-			raise KeyError('SysCommand() does not have an active session.')
-		elif type(key) is slice:
-			start = key.start or 0
-			end = key.stop or len(self.session._trace_log)
-
-			return self.session._trace_log[start:end]
-		else:
-			raise ValueError("SysCommand() doesn't have key & value pairs, only slices, SysCommand('ls')[:10] as an example.")
-
-	@override
-	def __repr__(self, *args: list[Any], **kwargs: dict[str, Any]) -> str:
-		return self.decode('UTF-8', errors='backslashreplace') or ''
-
-	def create_session(self) -> bool:
-		"""
-		Initiates a :ref:`SysCommandWorker` session in this class ``.session``.
-		It then proceeds to poll the process until it ends, after which it also
-		clears any printed output if ``.peek_output=True``.
-		"""
-		if self.session:
-			return True
-
-		with SysCommandWorker(
-			self.cmd,
-			peek_output=self.peek_output,
-			environment_vars=self.environment_vars,
-			remove_vt100_escape_codes_from_lines=self.remove_vt100_escape_codes_from_lines,
-			working_directory=self.working_directory,
-		) as session:
-			self.session = session
-
-			while not self.session.ended:
-				self.session.poll()
-
-		if self.peek_output:
-			sys.stdout.write('\n')
-			sys.stdout.flush()
-
-		return True
-
-	def decode(self, encoding: str = 'utf-8', errors: str = 'backslashreplace', strip: bool = True) -> str:
-		if not self.session:
-			raise ValueError('No session available to decode')
-
-		val = self.session._trace_log.decode(encoding, errors=errors)
-
-		if strip:
-			return val.strip()
-		return val
-
-	def output(self, remove_cr: bool = True) -> bytes:
-		if not self.session:
-			raise ValueError('No session available')
-
-		if remove_cr:
-			return self.session._trace_log.replace(b'\r\n', b'\n')
-
-		return self.session._trace_log
-
-	@property
-	def exit_code(self) -> int | None:
-		if self.session:
-			return self.session.exit_code
-		else:
-			return None
-
-	@property
-	def trace_log(self) -> bytes | None:
-		if self.session:
-			return self.session._trace_log
-		return None
-
-
-def run(
-	cmd: list[str],
-	input_data: bytes | None = None,
-) -> subprocess.CompletedProcess[bytes]:
-	_cmd_history(cmd)
-
-	return subprocess.run(
-		cmd,
-		input=input_data,
-		stdout=subprocess.PIPE,
-		stderr=subprocess.STDOUT,
-		check=True,
-	)
-
-
-def locate_binary(name: str) -> str:
-	if path := which(name):
-		return path
-	raise RequirementError(f'Binary {name} does not exist.')
-
-
-def _pid_exists(pid: int) -> bool:
-	try:
-		return any(subprocess.check_output(['ps', '--no-headers', '-o', 'pid', '-p', str(pid)]).strip())
-	except subprocess.CalledProcessError:
-		return False
-
-
-def _cmd_history(cmd: list[str]) -> None:
-	content = f'{time.time()} {cmd}\n'
-	_append_log('cmd_history.txt', content)
-
-
-def _cmd_output(output: str) -> None:
-	_append_log('cmd_output.txt', output)
-
-
-def _append_log(file: str, content: str) -> None:
-	path = logger.directory / file
-
-	change_perm = not path.exists()
-
-	try:
-		with path.open('a') as f:
-			f.write(content)
-
-		if change_perm:
-			path.chmod(stat.S_IRUSR | stat.S_IWUSR | stat.S_IRGRP)
-	except PermissionError, FileNotFoundError:
-		# If the file does not exist, ignore the error
-		pass

archinstall/lib/configuration.py

@@ -1,137 +1,181 @@
+import os
+import json
+import stat
 import readline
+from pathlib import Path
+from typing import Optional, Dict, Any, TYPE_CHECKING
 
-from archinstall.lib.args import USER_CONFIG_FILE, USER_CREDS_FILE, ArchConfig
+from .menu import Menu, MenuSelectionType
-from archinstall.lib.log import debug
+from .storage import storage
-from archinstall.lib.menu.helpers import Confirmation, Selection
+from .general import JSON, UNSAFE_JSON
-from archinstall.lib.menu.util import get_password, prompt_dir
+from .output import debug, info, warn
-from archinstall.lib.translationhandler import tr
+
-from archinstall.tui.menu_item import MenuItem, MenuItemGroup
+if TYPE_CHECKING:
-from archinstall.tui.result import ResultType
+	_: Any
 
 
-async def confirm_config(config: ArchConfig) -> bool:
+class ConfigurationOutput:
-	header = f'{tr("The specified configuration will be applied")}. '
+	def __init__(self, config: Dict):
-	header += tr('Would you like to continue?') + '\n'
+		"""
+		Configuration output handler to parse the existing configuration data structure and prepare for output on the
+		console and for saving it to configuration files
 
-	group = MenuItemGroup.yes_no()
+		:param config: A dictionary containing configurations (basically archinstall.arguments)
-	group.set_preview_for_all(lambda x: config.user_config_to_json())
+		:type config: Dict
+		"""
+		self._config = config
+		self._user_credentials: Dict[str, Any] = {}
+		self._user_config: Dict[str, Any] = {}
+		self._default_save_path = storage.get('LOG_PATH', Path('.'))
+		self._user_config_file = 'user_configuration.json'
+		self._user_creds_file = "user_credentials.json"
 
-	result = await Confirmation(
+		self._sensitive = ['!users', '!root-password']
-		group=group,
+		self._ignore = ['abort', 'install', 'config', 'creds', 'dry_run']
-		header=header,
-		allow_skip=False,
-		preset=True,
-		preview_location='bottom',
-		preview_header=tr('Configuration preview'),
-	).show()
 
-	if not result.get_value():
+		self._process_config()
-		return False
 
-	return True
+	@property
+	def user_credentials_file(self):
+		return self._user_creds_file
+
+	@property
+	def user_configuration_file(self):
+		return self._user_config_file
+
+	def _process_config(self):
+		for key, value in self._config.items():
+			if key in self._sensitive:
+				self._user_credentials[key] = value
+			elif key in self._ignore:
+				pass
+			else:
+				self._user_config[key] = value
+
+			# special handling for encryption password
+			if key == 'disk_encryption' and value:
+				self._user_credentials['encryption_password'] = value.encryption_password
+
+	def user_config_to_json(self) -> str:
+		return json.dumps({
+			'config_version': storage['__version__'],  # Tells us what version was used to generate the config
+			**self._user_config,  # __version__ will be overwritten by old version definition found in config
+			'version': storage['__version__']
+		}, indent=4, sort_keys=True, cls=JSON)
+
+	def user_credentials_to_json(self) -> Optional[str]:
+		if self._user_credentials:
+			return json.dumps(self._user_credentials, indent=4, sort_keys=True, cls=UNSAFE_JSON)
+		return None
+
+	def show(self):
+		print(_('\nThis is your chosen configuration:'))
+		debug(" -- Chosen configuration --")
+
+		info(self.user_config_to_json())
+		print()
+
+	def _is_valid_path(self, dest_path: Path) -> bool:
+		dest_path_ok = dest_path.exists() and dest_path.is_dir()
+		if not dest_path_ok:
+			warn(
+				f'Destination directory {dest_path.resolve()} does not exist or is not a directory\n.',
+				'Configuration files can not be saved'
+			)
+		return dest_path_ok
+
+	def save_user_config(self, dest_path: Path):
+		if self._is_valid_path(dest_path):
+			target = dest_path / self._user_config_file
+			target.write_text(self.user_config_to_json())
+			os.chmod(target, stat.S_IRUSR | stat.S_IWUSR | stat.S_IRGRP)
+
+	def save_user_creds(self, dest_path: Path):
+		if self._is_valid_path(dest_path):
+			if user_creds := self.user_credentials_to_json():
+				target = dest_path / self._user_creds_file
+				target.write_text(user_creds)
+				os.chmod(target, stat.S_IRUSR | stat.S_IWUSR | stat.S_IRGRP)
+
+	def save(self, dest_path: Optional[Path] = None):
+		dest_path = dest_path or self._default_save_path
+
+		if self._is_valid_path(dest_path):
+			self.save_user_config(dest_path)
+			self.save_user_creds(dest_path)
 
 
-async def save_config(config: ArchConfig) -> None:
+def save_config(config: Dict):
-	def preview(item: MenuItem) -> str | None:
+	def preview(selection: str):
-		match item.value:
+		match options[selection]:
-			case 'user_config':
+			case "user_config":
-				serialized = config.user_config_to_json()
+				serialized = config_output.user_config_to_json()
-				return f'{USER_CONFIG_FILE}\n{serialized}'
+				return f"{config_output.user_configuration_file}\n{serialized}"
-			case 'user_creds':
+			case "user_creds":
-				if maybe_serial := config.user_credentials_to_json():
+				if maybe_serial := config_output.user_credentials_to_json():
-					return f'{USER_CREDS_FILE}\n{maybe_serial}'
+					return f"{config_output.user_credentials_file}\n{maybe_serial}"
-				return tr('No configuration')
+				return str(_("No configuration"))
-			case 'all':
+			case "all":
-				output = [str(USER_CONFIG_FILE)]
+				output = [config_output.user_configuration_file]
-				config.user_credentials_to_json()
+				if config_output.user_credentials_to_json():
-				output.append(str(USER_CREDS_FILE))
+					output.append(config_output.user_credentials_file)
 				return '\n'.join(output)
 		return None
 
-	items = [
+	try:
-		MenuItem(
+		config_output = ConfigurationOutput(config)
-			tr('Save user configuration (including disk layout)'),
-			value='user_config',
-			preview_action=preview,
-		),
-		MenuItem(
-			tr('Save user credentials'),
-			value='user_creds',
-			preview_action=preview,
-		),
-		MenuItem(
-			tr('Save all'),
-			value='all',
-			preview_action=preview,
-		),
-	]
 
-	group = MenuItemGroup(items)
+		options = {
-	result = await Selection[str](
+			str(_("Save user configuration (including disk layout)")): "user_config",
-		group,
+			str(_("Save user credentials")): "user_creds",
-		allow_skip=True,
+			str(_("Save all")): "all",
-		preview_location='right',
+		}
-	).show()
 
-	match result.type_:
+		save_choice = Menu(
-		case ResultType.Skip:
+			_("Choose which configuration to save"),
+			list(options),
+			sort=False,
+			skip=True,
+			preview_size=0.75,
+			preview_command=preview,
+		).run()
+
+		if save_choice.type_ == MenuSelectionType.Skip:
 			return
-		case ResultType.Selection:
-			save_option = result.get_value()
-		case _:
-			raise ValueError('Unhandled return type')
 
-	readline.set_completer_delims('\t\n=')
+		readline.set_completer_delims("\t\n=")
-	readline.parse_and_bind('tab: complete')
+		readline.parse_and_bind("tab: complete")
+		while True:
+			path = input(
+				_(
+					"Enter a directory for the configuration(s) to be saved (tab completion enabled)\nSave directory: "
+				)
+			).strip(" ")
+			dest_path = Path(path)
+			if dest_path.exists() and dest_path.is_dir():
+				break
+			info(_("Not a valid directory: {}").format(dest_path), fg="red")
 
-	dest_path = await prompt_dir(
+		if not path:
-		tr('Enter a directory for the configuration(s) to be saved') + '\n',
+			return
-		allow_skip=True,
-	)
 
-	if not dest_path:
+		prompt = _(
+			"Do you want to save {} configuration file(s) in the following location?\n\n{}"
+		).format(options[str(save_choice.value)], dest_path.absolute())
+
+		save_confirmation = Menu(prompt, Menu.yes_no(), default_option=Menu.yes()).run()
+		if save_confirmation == Menu.no():
+			return
+
+		debug("Saving {} configuration files to {}".format(options[str(save_choice.value)], dest_path.absolute()))
+
+		match options[str(save_choice.value)]:
+			case "user_config":
+				config_output.save_user_config(dest_path)
+			case "user_creds":
+				config_output.save_user_creds(dest_path)
+			case "all":
+				config_output.save(dest_path)
+
+	except KeyboardInterrupt:
 		return
-
-	header = tr('Do you want to save the configuration file(s) to {}?').format(dest_path)
-
-	save_result = await Confirmation(
-		header=header,
-		allow_skip=False,
-		preset=True,
-	).show()
-
-	match save_result.type_:
-		case ResultType.Selection:
-			if not save_result.get_value():
-				return
-		case _:
-			return
-
-	debug(f'Saving configuration files to {dest_path.absolute()}')
-
-	header = tr('Do you want to encrypt the user_credentials.json file?')
-
-	enc_result = await Confirmation(
-		header=header,
-		allow_skip=False,
-		preset=False,
-	).show()
-
-	enc_password: str | None = None
-	if enc_result.type_ == ResultType.Selection:
-		if enc_result.get_value():
-			password = await get_password(
-				header=tr('Credentials file encryption password'),
-				allow_skip=True,
-			)
-
-			if password:
-				enc_password = password.plaintext
-
-	match save_option:
-		case 'user_config':
-			config.save_user_config(dest_path)
-		case 'user_creds':
-			config.save_user_creds(dest_path, password=enc_password)
-		case 'all':
-			config.save(dest_path, creds=True, password=enc_password)

archinstall/lib/crypt.py

@@ -1,125 +0,0 @@
-import base64
-import ctypes
-import os
-from pathlib import Path
-
-from cryptography.fernet import Fernet, InvalidToken
-from cryptography.hazmat.primitives.kdf.argon2 import Argon2id
-
-from archinstall.lib.log import debug
-
-libcrypt = ctypes.CDLL('libcrypt.so')
-
-libcrypt.crypt.argtypes = [ctypes.c_char_p, ctypes.c_char_p]
-libcrypt.crypt.restype = ctypes.c_char_p
-
-libcrypt.crypt_gensalt.argtypes = [ctypes.c_char_p, ctypes.c_ulong, ctypes.c_char_p, ctypes.c_int]
-libcrypt.crypt_gensalt.restype = ctypes.c_char_p
-
-LOGIN_DEFS = Path('/etc/login.defs')
-
-
-def _search_login_defs(key: str) -> str | None:
-	defs = LOGIN_DEFS.read_text()
-	for line in defs.split('\n'):
-		line = line.strip()
-
-		if line.startswith('#'):
-			continue
-
-		if line.startswith(key):
-			value = line.split(' ')[1]
-			return value
-
-	return None
-
-
-def crypt_gen_salt(prefix: str | bytes, rounds: int) -> bytes:
-	if isinstance(prefix, str):
-		prefix = prefix.encode('utf-8')
-
-	setting = libcrypt.crypt_gensalt(prefix, rounds, None, 0)
-
-	if setting is None:
-		raise ValueError(f'crypt_gensalt() returned NULL for prefix {prefix!r} and rounds {rounds}')
-
-	return setting
-
-
-def crypt_yescrypt(plaintext: str) -> str:
-	"""
-	By default chpasswd in Arch uses PAM to hash the password with crypt_yescrypt
-	the PAM code https://github.com/linux-pam/linux-pam/blob/master/modules/pam_unix/support.c
-	shows that the hashing rounds are determined from YESCRYPT_COST_FACTOR in /etc/login.defs
-	If no value was specified (or commented out) a default of 5 is chosen
-	"""
-	value = _search_login_defs('YESCRYPT_COST_FACTOR')
-	if value is not None:
-		rounds = int(value)
-		if rounds < 3:
-			rounds = 3
-		elif rounds > 11:
-			rounds = 11
-	else:
-		rounds = 5
-
-	debug(f'Creating yescrypt hash with rounds {rounds}')
-
-	enc_plaintext = plaintext.encode('utf-8')
-	salt = crypt_gen_salt('$y$', rounds)
-
-	crypt_hash = libcrypt.crypt(enc_plaintext, salt)
-
-	if crypt_hash is None:
-		raise ValueError('crypt() returned NULL')
-
-	return crypt_hash.decode('utf-8')
-
-
-def _get_fernet(salt: bytes, password: str) -> Fernet:
-	# https://cryptography.io/en/latest/hazmat/primitives/key-derivation-functions/#argon2id
-	kdf = Argon2id(
-		salt=salt,
-		length=32,
-		iterations=1,
-		lanes=4,
-		memory_cost=64 * 1024,
-		ad=None,
-		secret=None,
-	)
-
-	key = base64.urlsafe_b64encode(
-		kdf.derive(
-			password.encode('utf-8'),
-		),
-	)
-
-	return Fernet(key)
-
-
-def encrypt(password: str, data: str) -> str:
-	salt = os.urandom(16)
-	f = _get_fernet(salt, password)
-	token = f.encrypt(data.encode('utf-8'))
-
-	encoded_token = base64.urlsafe_b64encode(token).decode('utf-8')
-	encoded_salt = base64.urlsafe_b64encode(salt).decode('utf-8')
-
-	return f'$argon2id${encoded_salt}${encoded_token}'
-
-
-def decrypt(data: str, password: str) -> str:
-	_, algo, encoded_salt, encoded_token = data.split('$')
-	salt = base64.urlsafe_b64decode(encoded_salt)
-	token = base64.urlsafe_b64decode(encoded_token)
-
-	if algo != 'argon2id':
-		raise ValueError(f'Unsupported algorithm {algo!r}')
-
-	f = _get_fernet(salt, password)
-	try:
-		decrypted = f.decrypt(token)
-	except InvalidToken:
-		raise ValueError('Invalid password')
-
-	return decrypted.decode('utf-8')

archinstall/lib/disk/__init__.py

@@ -0,0 +1,40 @@
+from .device_handler import device_handler, disk_layouts
+from .fido import Fido2
+from .filesystem import FilesystemHandler
+from .subvolume_menu import SubvolumeMenu
+from .partitioning_menu import (
+	manual_partitioning,
+	PartitioningList
+)
+from .device_model import (
+	_DeviceInfo,
+	BDevice,
+	DiskLayoutType,
+	DiskLayoutConfiguration,
+	PartitionTable,
+	Unit,
+	Size,
+	SubvolumeModification,
+	DeviceGeometry,
+	PartitionType,
+	PartitionFlag,
+	FilesystemType,
+	ModificationStatus,
+	PartitionModification,
+	DeviceModification,
+	EncryptionType,
+	DiskEncryption,
+	Fido2Device,
+	LsblkInfo,
+	CleanType,
+	get_lsblk_info,
+	get_all_lsblk_info,
+	get_lsblk_by_mountpoint
+)
+from .encryption_menu import (
+	select_encryption_type,
+	select_encrypted_password,
+	select_hsm,
+	select_partitions_to_encrypt,
+	DiskEncryptionMenu,
+)

archinstall/lib/disk/device_handler.py

@@ -1,114 +1,82 @@
-import logging
+from __future__ import annotations
+
+import json
 import os
+import time
+import logging
 from pathlib import Path
+from typing import List, Dict, Any, Optional, TYPE_CHECKING
 
-from parted import Device, Disk, DiskException, FileSystem, Geometry, IOException, Partition, PartitionException, freshDisk, getAllDevices, getDevice, newDisk
+from parted import (  # type: ignore
-
+	Disk, Geometry, FileSystem,
-from archinstall.lib.command import SysCommand
+	PartitionException, DiskLabelException,
-from archinstall.lib.disk.luks import Luks2, unlock_luks2_dev
+	getAllDevices, freshDisk, Partition, Device
-from archinstall.lib.disk.utils import (
-	find_lsblk_info,
-	get_all_lsblk_info,
-	get_lsblk_info,
-	mount,
-	udev_sync,
-	umount,
 )
-from archinstall.lib.exceptions import DiskError, SysCallError, UnknownFilesystemFormat
+
-from archinstall.lib.log import debug, error, info, log
+from .device_model import (
-from archinstall.lib.models.device import (
+	DeviceModification, PartitionModification,
-	DEFAULT_ITER_TIME,
+	BDevice, _DeviceInfo, _PartitionInfo,
-	BDevice,
+	FilesystemType, Unit, PartitionTable,
-	BtrfsMountOption,
+	ModificationStatus, get_lsblk_info, LsblkInfo,
-	DeviceModification,
+	_BtrfsSubvolumeInfo, get_all_lsblk_info, DiskEncryption
-	DiskEncryption,
-	FilesystemType,
-	LsblkInfo,
-	ModificationStatus,
-	PartitionFlag,
-	PartitionGUID,
-	PartitionModification,
-	PartitionTable,
-	SubvolumeModification,
-	Unit,
-	_BtrfsSubvolumeInfo,
-	_DeviceInfo,
-	_PartitionInfo,
 )
-from archinstall.lib.models.users import Password
+
-from archinstall.lib.pathnames import ARCHISO_MOUNTPOINT
+from ..exceptions import DiskError, UnknownFilesystemFormat
+from ..general import SysCommand, SysCallError, JSON
+from ..luks import Luks2
+from ..output import debug, error, info, warn, log
+from ..utils.util import is_subpath
+
+if TYPE_CHECKING:
+	_: Any
 
 
-class DeviceHandler:
+class DeviceHandler(object):
 	_TMP_BTRFS_MOUNT = Path('/mnt/arch_btrfs')
 
-	def __init__(self) -> None:
+	def __init__(self):
-		self._devices: dict[Path, BDevice] = {}
+		self._devices: Dict[Path, BDevice] = {}
-		self._partition_table = PartitionTable.default()
 		self.load_devices()
 
 	@property
-	def devices(self) -> list[BDevice]:
+	def devices(self) -> List[BDevice]:
 		return list(self._devices.values())
 
-	@property
+	def load_devices(self):
-	def partition_table(self) -> PartitionTable:
-		return self._partition_table
-
-	def load_devices(self) -> None:
 		block_devices = {}
 
-		udev_sync()
+		for device in getAllDevices():
-		all_lsblk_info = get_all_lsblk_info()
+			if get_lsblk_info(device.path).type == 'rom':
-		devices = getAllDevices()
-		devices.extend(self.get_loop_devices())
-
-		for device in devices:
-			dev_lsblk_info = find_lsblk_info(device.path, all_lsblk_info)
-
-			if not dev_lsblk_info:
-				debug(f'Device lsblk info not found: {device.path}')
-				continue
-
-			if dev_lsblk_info.type == 'rom':
-				continue
-
-			# exclude archiso loop device
-			if dev_lsblk_info.mountpoint == ARCHISO_MOUNTPOINT:
 				continue
 
 			try:
-				if dev_lsblk_info.pttype:
+				disk = Disk(device)
-					disk = newDisk(device)
+			except DiskLabelException as err:
+				if 'unrecognised disk label' in getattr(error, 'message', str(err)):
+					disk = freshDisk(device, PartitionTable.GPT.value)
 				else:
-					disk = freshDisk(device, self.partition_table.value)
+					debug(f'Unable to get disk from device: {device}')
-			except DiskException as err:
+					continue
-				debug(f'Unable to get disk from {device.path}: {err}')
-				continue
 
 			device_info = _DeviceInfo.from_disk(disk)
 			partition_infos = []
 
 			for partition in disk.partitions:
-				lsblk_info = find_lsblk_info(partition.path, dev_lsblk_info.children)
+				lsblk_info = get_lsblk_info(partition.path)
-
-				if not lsblk_info:
-					debug(f'Partition lsblk info not found: {partition.path}')
-					continue
-
 				fs_type = self._determine_fs_type(partition, lsblk_info)
 				subvol_infos = []
 
-				if fs_type == FilesystemType.BTRFS:
+				if fs_type == FilesystemType.Btrfs:
-					subvol_infos = self.get_btrfs_info(partition.path, lsblk_info)
+					subvol_infos = self.get_btrfs_info(partition.path)
 
 				partition_infos.append(
 					_PartitionInfo.from_partition(
 						partition,
-						lsblk_info,
 						fs_type,
-						subvol_infos,
+						lsblk_info.partuuid,
-					),
+						lsblk_info.mountpoints,
+						subvol_infos
+					)
 				)
 
 			block_device = BDevice(disk, device_info, partition_infos)
@@ -116,39 +84,13 @@ class DeviceHandler:
 
 		self._devices = block_devices
 
-	@staticmethod
-	def get_loop_devices() -> list[Device]:
-		devices = []
-
-		try:
-			loop_devices = SysCommand(['losetup', '-a'])
-		except SysCallError as err:
-			debug(f'Failed to get loop devices: {err}')
-		else:
-			for ld_info in str(loop_devices).splitlines():
-				try:
-					loop_device_path, _ = ld_info.split(':', maxsplit=1)
-				except ValueError:
-					continue
-
-				try:
-					loop_device = getDevice(loop_device_path)
-				except IOException as err:
-					debug(f'Failed to get loop device: {err}')
-				else:
-					devices.append(loop_device)
-
-		return devices
-
 	def _determine_fs_type(
 		self,
 		partition: Partition,
-		lsblk_info: LsblkInfo | None = None,
+		lsblk_info: Optional[LsblkInfo] = None
-	) -> FilesystemType | None:
+	) -> Optional[FilesystemType]:
 		try:
 			if partition.fileSystem:
-				if partition.fileSystem.type == FilesystemType.LINUX_SWAP.parted_value:
-					return FilesystemType.LINUX_SWAP
 				return FilesystemType(partition.fileSystem.type)
 			elif lsblk_info is not None:
 				return FilesystemType(lsblk_info.fstype) if lsblk_info.fstype else None
@@ -158,191 +100,199 @@ class DeviceHandler:
 
 		return None
 
-	def get_device(self, path: Path) -> BDevice | None:
+	def get_device(self, path: Path) -> Optional[BDevice]:
 		return self._devices.get(path, None)
 
-	def get_device_by_partition_path(self, partition_path: Path) -> BDevice | None:
+	def get_device_by_partition_path(self, partition_path: Path) -> Optional[BDevice]:
 		partition = self.find_partition(partition_path)
 		if partition:
 			device: Device = partition.disk.device
 			return self.get_device(Path(device.path))
 		return None
 
-	def find_partition(self, path: Path) -> _PartitionInfo | None:
+	def find_partition(self, path: Path) -> Optional[_PartitionInfo]:
 		for device in self._devices.values():
 			part = next(filter(lambda x: str(x.path) == str(path), device.partition_infos), None)
 			if part is not None:
 				return part
 		return None
 
-	def get_uuid_for_path(self, path: Path) -> str | None:
+	def get_uuid_for_path(self, path: Path) -> Optional[str]:
 		partition = self.find_partition(path)
 		return partition.partuuid if partition else None
 
-	def get_btrfs_info(
+	def get_btrfs_info(self, dev_path: Path) -> List[_BtrfsSubvolumeInfo]:
-		self,
+		lsblk_info = get_lsblk_info(dev_path)
-		dev_path: Path,
+		subvol_infos: List[_BtrfsSubvolumeInfo] = []
-		lsblk_info: LsblkInfo | None = None,
-	) -> list[_BtrfsSubvolumeInfo]:
-		if not lsblk_info:
-			lsblk_info = get_lsblk_info(dev_path)
-
-		subvol_infos: list[_BtrfsSubvolumeInfo] = []
 
 		if not lsblk_info.mountpoint:
-			mount(dev_path, self._TMP_BTRFS_MOUNT, create_target_mountpoint=True)
+			self.mount(dev_path, self._TMP_BTRFS_MOUNT, create_target_mountpoint=True)
 			mountpoint = self._TMP_BTRFS_MOUNT
 		else:
 			# when multiple subvolumes are mounted then the lsblk output may look like
-			# "mountpoint": "/mnt/archinstall/var/log"
+			# "mountpoint": "/mnt/archinstall/.snapshots"
-			# "mountpoints": ["/mnt/archinstall/var/log", "/mnt/archinstall/home", ..]
+			# "mountpoints": ["/mnt/archinstall/.snapshots", "/mnt/archinstall/home", ..]
 			# so we'll determine the minimum common path and assume that's the root
-			try:
+			path_strings = [str(m) for m in lsblk_info.mountpoints]
-				common_path = os.path.commonpath(lsblk_info.mountpoints)
+			common_prefix = os.path.commonprefix(path_strings)
-			except ValueError:
+			mountpoint = Path(common_prefix)
-				return subvol_infos
-
-			mountpoint = Path(common_path)
 
 		try:
-			result = SysCommand(f'btrfs subvolume list {mountpoint}').decode()
+			result = SysCommand(f'btrfs subvolume list {mountpoint}')
 		except SysCallError as err:
 			debug(f'Failed to read btrfs subvolume information: {err}')
 			return subvol_infos
 
-		# It is assumed that lsblk will contain the fields as
+		try:
-		# "mountpoints": ["/mnt/archinstall/log", "/mnt/archinstall/home", "/mnt/archinstall", ...]
+			if decoded := result.decode('utf-8'):
-		# "fsroots": ["/@log", "/@home", "/@"...]
+				# ID 256 gen 16 top level 5 path @
-		# we'll thereby map the fsroot, which are the mounted filesystem roots
+				for line in decoded.splitlines():
-		# to the corresponding mountpoints
+					# expected output format:
-		btrfs_subvol_info = dict(zip(lsblk_info.fsroots, lsblk_info.mountpoints))
+					# ID 257 gen 8 top level 5 path @home
-
+					name = Path(line.split(' ')[-1])
-		# ID 256 gen 16 top level 5 path @
+					sub_vol_mountpoint = lsblk_info.btrfs_subvol_info.get(name, None)
-		for line in result.splitlines():
+					subvol_infos.append(_BtrfsSubvolumeInfo(name, sub_vol_mountpoint))
-			# expected output format:
+		except json.decoder.JSONDecodeError as err:
-			# ID 257 gen 8 top level 5 path @home
+			error(f"Could not decode lsblk JSON: {result}")
-			name = Path(line.split(' ')[-1])
+			raise err
-			sub_vol_mountpoint = btrfs_subvol_info.get('/' / name, None)
-			subvol_infos.append(_BtrfsSubvolumeInfo(name, sub_vol_mountpoint))
 
 		if not lsblk_info.mountpoint:
-			umount(dev_path)
+			self.umount(dev_path)
 
 		return subvol_infos
 
-	def format(
+	def _perform_formatting(
 		self,
 		fs_type: FilesystemType,
 		path: Path,
-		additional_parted_options: list[str] = [],
+		additional_parted_options: List[str] = []
-	) -> None:
+	):
-		mkfs_type = fs_type.value
-		command = None
 		options = []
+		command = ''
 
 		match fs_type:
-			case FilesystemType.BTRFS | FilesystemType.XFS:
+			case FilesystemType.Btrfs:
-				# Force overwrite
+				options += ['-f']
-				options.append('-f')
+				command += 'mkfs.btrfs'
-			case FilesystemType.F2FS:
+			case FilesystemType.Fat16:
-				options.append('-f')
+				options += ['-F16']
-				options.extend(('-O', 'extra_attr'))
+				command += 'mkfs.fat'
-			case FilesystemType.EXT2 | FilesystemType.EXT3 | FilesystemType.EXT4:
+			case FilesystemType.Fat32:
-				# Force create
+				options += ['-F32']
-				options.append('-F')
+				command += 'mkfs.fat'
-			case _ if fs_type.is_fat():
+			case FilesystemType.Ext2:
-				mkfs_type = 'fat'
+				options += ['-F']
-				# Set FAT size
+				command += 'mkfs.ext2'
-				options.extend(('-F', fs_type.value.removeprefix(mkfs_type)))
+			case FilesystemType.Ext3:
-			case FilesystemType.LINUX_SWAP:
+				options += ['-F']
-				command = 'mkswap'
+				command += 'mkfs.ext3'
+			case FilesystemType.Ext4:
+				options += ['-F']
+				command += 'mkfs.ext4'
+			case FilesystemType.Xfs:
+				options += ['-f']
+				command += 'mkfs.xfs'
+			case FilesystemType.F2fs:
+				options += ['-f']
+				command += 'mkfs.f2fs'
+			case FilesystemType.Ntfs:
+				options += ['-f', '-Q']
+				command += 'mkfs.ntfs'
+			case FilesystemType.Reiserfs:
+				command += 'mkfs.reiserfs'
 			case _:
 				raise UnknownFilesystemFormat(f'Filetype "{fs_type.value}" is not supported')
 
-		if not command:
+		options += additional_parted_options
-			command = f'mkfs.{mkfs_type}'
+		options_str = ' '.join(options)
 
-		cmd = [command, *options, *additional_parted_options, str(path)]
+		info(f'Formatting filesystem: /usr/bin/{command} {options_str} {path}')
-
-		debug('Formatting filesystem:', ' '.join(cmd))
 
 		try:
-			SysCommand(cmd)
+			SysCommand(f"/usr/bin/{command} {options_str} {path}")
 		except SysCallError as err:
 			msg = f'Could not format {path} with {fs_type.value}: {err.message}'
 			error(msg)
 			raise DiskError(msg) from err
 
-	def encrypt(
+	def _perform_enc_formatting(
 		self,
 		dev_path: Path,
-		mapper_name: str | None,
+		mapper_name: Optional[str],
-		enc_password: Password | None,
-		lock_after_create: bool = True,
-		iter_time: int = DEFAULT_ITER_TIME,
-	) -> Luks2:
-		luks_handler = Luks2(
-			dev_path,
-			mapper_name=mapper_name,
-			password=enc_password,
-		)
-
-		key_file = luks_handler.encrypt(iter_time=iter_time)
-
-		udev_sync()
-
-		luks_handler.unlock(key_file=key_file)
-
-		if not luks_handler.mapper_dev:
-			raise DiskError('Failed to unlock luks device')
-
-		if lock_after_create:
-			debug(f'luks2 locking device: {dev_path}')
-			luks_handler.lock()
-
-		return luks_handler
-
-	def format_encrypted(
-		self,
-		dev_path: Path,
-		mapper_name: str | None,
 		fs_type: FilesystemType,
-		enc_conf: DiskEncryption,
+		enc_conf: DiskEncryption
-	) -> None:
+	):
-		if not enc_conf.encryption_password:
-			raise ValueError('No encryption password provided')
-
 		luks_handler = Luks2(
 			dev_path,
 			mapper_name=mapper_name,
-			password=enc_conf.encryption_password,
+			password=enc_conf.encryption_password
 		)
 
-		key_file = luks_handler.encrypt(iter_time=enc_conf.iter_time)
+		key_file = luks_handler.encrypt()
-
-		udev_sync()
 
+		debug(f'Unlocking luks2 device: {dev_path}')
 		luks_handler.unlock(key_file=key_file)
 
 		if not luks_handler.mapper_dev:
 			raise DiskError('Failed to unlock luks device')
 
 		info(f'luks2 formatting mapper dev: {luks_handler.mapper_dev}')
-		self.format(fs_type, luks_handler.mapper_dev)
+		self._perform_formatting(fs_type, luks_handler.mapper_dev)
 
 		info(f'luks2 locking device: {dev_path}')
 		luks_handler.lock()
 
-	def _setup_partition(
+	def _validate(self, device_mod: DeviceModification):
+		checks = {
+			# verify that all partitions have a path set (which implies that they have been created)
+			lambda x: x.dev_path is None: ValueError('When formatting, all partitions must have a path set'),
+			# crypto luks is not a valid file system type
+			lambda x: x.fs_type is FilesystemType.Crypto_luks: ValueError('Crypto luks cannot be set as a filesystem type'),
+			# file system type must be set
+			lambda x: x.fs_type is None: ValueError('File system type must be set for modification')
+		}
+
+		for check, exc in checks.items():
+			found = next(filter(check, device_mod.partitions), None)
+			if found is not None:
+				raise exc
+
+	def format(
+		self,
+		device_mod: DeviceModification,
+		enc_conf: Optional['DiskEncryption'] = None
+	):
+		"""
+		Format can be given an overriding path, for instance /dev/null to test
+		the formatting functionality and in essence the support for the given filesystem.
+		"""
+		self._validate(device_mod)
+
+		# make sure all devices are unmounted
+		self._umount_all_existing(device_mod)
+
+		for part_mod in device_mod.partitions:
+			# partition will be encrypted
+			if enc_conf is not None and part_mod in enc_conf.partitions:
+				self._perform_enc_formatting(
+					part_mod.safe_dev_path,
+					part_mod.mapper_name,
+					part_mod.safe_fs_type,
+					enc_conf
+				)
+			else:
+				self._perform_formatting(part_mod.safe_fs_type, part_mod.safe_dev_path)
+
+	def _perform_partitioning(
 		self,
 		part_mod: PartitionModification,
 		block_device: BDevice,
 		disk: Disk,
-		requires_delete: bool,
+		requires_delete: bool
-	) -> None:
+	):
 		# when we require a delete and the partition to be (re)created
 		# already exists then we have to delete it first
-		if requires_delete and part_mod.status in [ModificationStatus.MODIFY, ModificationStatus.DELETE]:
+		if requires_delete and part_mod.status in [ModificationStatus.Modify, ModificationStatus.Delete]:
 			info(f'Delete existing partition: {part_mod.safe_dev_path}')
 			part_info = self.find_partition(part_mod.safe_dev_path)
 
@@ -350,238 +300,282 @@ class DeviceHandler:
 				raise DiskError(f'No partition for dev path found: {part_mod.safe_dev_path}')
 
 			disk.deletePartition(part_info.partition)
+			disk.commit()
 
-		if part_mod.status == ModificationStatus.DELETE:
+		if part_mod.status == ModificationStatus.Delete:
 			return
 
 		start_sector = part_mod.start.convert(
 			Unit.sectors,
-			block_device.device_info.sector_size,
+			block_device.device_info.sector_size
 		)
 
 		length_sector = part_mod.length.convert(
 			Unit.sectors,
-			block_device.device_info.sector_size,
+			block_device.device_info.sector_size
 		)
 
 		geometry = Geometry(
 			device=block_device.disk.device,
 			start=start_sector.value,
-			length=length_sector.value,
+			length=length_sector.value
 		)
 
-		fs_value = part_mod.safe_fs_type.parted_value
+		filesystem = FileSystem(type=part_mod.safe_fs_type.value, geometry=geometry)
-		filesystem = FileSystem(type=fs_value, geometry=geometry)
 
 		partition = Partition(
 			disk=disk,
 			type=part_mod.type.get_partition_code(),
 			fs=filesystem,
-			geometry=geometry,
+			geometry=geometry
 		)
 
 		for flag in part_mod.flags:
-			partition.setFlag(flag.flag_id)
+			partition.setFlag(flag.value)
 
 		debug(f'\tType: {part_mod.type.value}')
-		debug(f'\tFilesystem: {fs_value}')
+		debug(f'\tFilesystem: {part_mod.safe_fs_type.value}')
 		debug(f'\tGeometry: {start_sector.value} start sector, {length_sector.value} length')
 
 		try:
 			disk.addPartition(partition=partition, constraint=disk.device.optimalAlignedConstraint)
+			disk.commit()
+
+			# the creation will take a bit of time
+			time.sleep(3)
+
+			# the partition has a real path now as it was created
+			part_mod.dev_path = Path(partition.path)
+
+			lsblk_info = self._fetch_partuuid(part_mod.dev_path)
+
+			part_mod.partuuid = lsblk_info.partuuid
+			part_mod.uuid = lsblk_info.uuid
 		except PartitionException as ex:
 			raise DiskError(f'Unable to add partition, most likely due to overlapping sectors: {ex}') from ex
 
-		if disk.type == PartitionTable.GPT.value:
+	def _fetch_partuuid(self, path: Path) -> LsblkInfo:
-			if part_mod.is_root():
+		attempts = 3
-				partition.type_uuid = PartitionGUID.LINUX_ROOT_X86_64.bytes
+		lsblk_info: Optional[LsblkInfo] = None
-			elif PartitionFlag.LINUX_HOME not in part_mod.flags and part_mod.is_home():
-				partition.setFlag(PartitionFlag.LINUX_HOME.flag_id)
 
-		# the partition has a path now that it has been added
+		self.partprobe(path)
-		part_mod.dev_path = Path(partition.path)
+		for attempt_nr in range(attempts):
+			time.sleep(attempt_nr + 1)
+			lsblk_info = get_lsblk_info(path)
 
-	def fetch_part_info(self, path: Path) -> LsblkInfo:
+			if lsblk_info.partuuid:
-		lsblk_info = get_lsblk_info(path)
+				break
 
-		if not lsblk_info.partn:
+			self.partprobe(path)
-			debug(f'Unable to determine new partition number: {path}\n{lsblk_info}')
-			raise DiskError(f'Unable to determine new partition number: {path}')
 
-		if not lsblk_info.partuuid:
+		if not lsblk_info or not lsblk_info.partuuid:
 			debug(f'Unable to determine new partition uuid: {path}\n{lsblk_info}')
 			raise DiskError(f'Unable to determine new partition uuid: {path}')
 
-		if not lsblk_info.uuid:
+		debug(f'partuuid found: {lsblk_info.json()}')
-			debug(f'Unable to determine new uuid: {path}\n{lsblk_info}')
-			raise DiskError(f'Unable to determine new uuid: {path}')
-
-		debug(f'partition information found: {lsblk_info.model_dump_json()}')
 
 		return lsblk_info
 
-	def create_lvm_btrfs_subvolumes(
-		self,
-		path: Path,
-		btrfs_subvols: list[SubvolumeModification],
-		mount_options: list[str],
-	) -> None:
-		info(f'Creating subvolumes: {path}')
-
-		mount(path, self._TMP_BTRFS_MOUNT, create_target_mountpoint=True)
-
-		for sub_vol in sorted(btrfs_subvols, key=lambda x: x.name):
-			debug(f'Creating subvolume: {sub_vol.name}')
-
-			subvol_path = self._TMP_BTRFS_MOUNT / sub_vol.name
-
-			SysCommand(f'btrfs subvolume create -p {subvol_path}')
-
-			if BtrfsMountOption.nodatacow.value in mount_options:
-				try:
-					SysCommand(f'chattr +C {subvol_path}')
-				except SysCallError as err:
-					raise DiskError(f'Could not set nodatacow attribute at {subvol_path}: {err}')
-
-			if BtrfsMountOption.compress.value in mount_options:
-				try:
-					SysCommand(f'chattr +c {subvol_path}')
-				except SysCallError as err:
-					raise DiskError(f'Could not set compress attribute at {subvol_path}: {err}')
-
-		umount(path)
-
 	def create_btrfs_volumes(
 		self,
 		part_mod: PartitionModification,
-		enc_conf: DiskEncryption | None = None,
+		enc_conf: Optional['DiskEncryption'] = None
-	) -> None:
+	):
 		info(f'Creating subvolumes: {part_mod.safe_dev_path}')
 
+		luks_handler = None
+
 		# unlock the partition first if it's encrypted
 		if enc_conf is not None and part_mod in enc_conf.partitions:
 			if not part_mod.mapper_name:
 				raise ValueError('No device path specified for modification')
 
-			luks_handler = unlock_luks2_dev(
+			luks_handler = self.unlock_luks2_dev(
 				part_mod.safe_dev_path,
 				part_mod.mapper_name,
-				enc_conf.encryption_password,
+				enc_conf.encryption_password
 			)
 
 			if not luks_handler.mapper_dev:
 				raise DiskError('Failed to unlock luks device')
 
-			dev_path = luks_handler.mapper_dev
+			self.mount(luks_handler.mapper_dev, self._TMP_BTRFS_MOUNT, create_target_mountpoint=True)
 		else:
-			luks_handler = None
+			self.mount(part_mod.safe_dev_path, self._TMP_BTRFS_MOUNT, create_target_mountpoint=True)
-			dev_path = part_mod.safe_dev_path
 
-		mount(
+		for sub_vol in part_mod.btrfs_subvols:
-			dev_path,
-			self._TMP_BTRFS_MOUNT,
-			create_target_mountpoint=True,
-			options=part_mod.mount_options,
-		)
-
-		for sub_vol in sorted(part_mod.btrfs_subvols, key=lambda x: x.name):
 			debug(f'Creating subvolume: {sub_vol.name}')
 
-			subvol_path = self._TMP_BTRFS_MOUNT / sub_vol.name
+			if luks_handler is not None:
+				subvol_path = self._TMP_BTRFS_MOUNT / sub_vol.name
+			else:
+				subvol_path = self._TMP_BTRFS_MOUNT / sub_vol.name
 
-			SysCommand(f'btrfs subvolume create -p {subvol_path}')
+			SysCommand(f"btrfs subvolume create {subvol_path}")
 
-		umount(dev_path)
+			if sub_vol.nodatacow:
+				try:
+					SysCommand(f'chattr +C {subvol_path}')
+				except SysCallError as err:
+					raise DiskError(f'Could not set nodatacow attribute at {subvol_path}: {err}')
+
+			if sub_vol.compress:
+				try:
+					SysCommand(f'chattr +c {subvol_path}')
+				except SysCallError as err:
+					raise DiskError(f'Could not set compress attribute at {subvol_path}: {err}')
 
 		if luks_handler is not None and luks_handler.mapper_dev is not None:
+			self.umount(luks_handler.mapper_dev)
 			luks_handler.lock()
+		else:
+			self.umount(part_mod.safe_dev_path)
 
-	def umount_all_existing(self, device_path: Path) -> None:
+	def unlock_luks2_dev(self, dev_path: Path, mapper_name: str, enc_password: str) -> Luks2:
-		debug(f'Unmounting all existing partitions: {device_path}')
+		luks_handler = Luks2(dev_path, mapper_name=mapper_name, password=enc_password)
 
-		existing_partitions = self._devices[device_path].partition_infos
+		if not luks_handler.is_unlocked():
+			luks_handler.unlock()
+
+		if not luks_handler.is_unlocked():
+			raise DiskError(f'Failed to unlock luks2 device: {dev_path}')
+
+		return luks_handler
+
+	def _umount_all_existing(self, modification: DeviceModification):
+		info(f'Unmounting all partitions: {modification.device_path}')
+
+		existing_partitions = self._devices[modification.device_path].partition_infos
 
 		for partition in existing_partitions:
 			debug(f'Unmounting: {partition.path}')
 
 			# un-mount for existing encrypted partitions
-			if partition.fs_type == FilesystemType.CRYPTO_LUKS:
+			if partition.fs_type == FilesystemType.Crypto_luks:
 				Luks2(partition.path).lock()
 			else:
-				umount(partition.path, recursive=True)
+				self.umount(partition.path, recursive=True)
 
 	def partition(
 		self,
 		modification: DeviceModification,
-		partition_table: PartitionTable | None = None,
+		partition_table: Optional[PartitionTable] = None
-	) -> None:
+	):
 		"""
 		Create a partition table on the block device and create all partitions.
 		"""
-		partition_table = partition_table or self.partition_table
+		if modification.wipe:
+			if partition_table is None:
+				raise ValueError('Modification is marked as wipe but no partitioning table was provided')
+
+			if partition_table.MBR and len(modification.partitions) > 3:
+				raise DiskError('Too many partitions on disk, MBR disks can only have 3 primary partitions')
+
+		# make sure all devices are unmounted
+		self._umount_all_existing(modification)
 
 		# WARNING: the entire device will be wiped and all data lost
 		if modification.wipe:
-			if partition_table.is_mbr() and len(modification.partitions) > 3:
-				raise DiskError('Too many partitions on disk, MBR disks can only have 3 primary partitions')
-
 			self.wipe_dev(modification.device)
-			disk = freshDisk(modification.device.disk.device, partition_table.value)
+			part_table = partition_table.value if partition_table else None
+			disk = freshDisk(modification.device.disk.device, part_table)
 		else:
 			info(f'Use existing device: {modification.device_path}')
 			disk = modification.device.disk
 
 		info(f'Creating partitions: {modification.device_path}')
 
-		# don't touch existing partitions
+		# TODO sort by delete first
-		filtered_part = [p for p in modification.partitions if not p.exists()]
+
+		for part_mod in modification.partitions:
+			# don't touch existing partitions
+			if part_mod.exists():
+				continue
 
-		for part_mod in filtered_part:
 			# if the entire disk got nuked then we don't have to delete
 			# any existing partitions anymore because they're all gone already
 			requires_delete = modification.wipe is False
-			self._setup_partition(part_mod, modification.device, disk, requires_delete=requires_delete)
+			self._perform_partitioning(part_mod, modification.device, disk, requires_delete=requires_delete)
 
-		disk.commit()
+		self.partprobe(modification.device.device_info.path)
 
-		# Wipe filesystem/LVM signatures from newly created partitions
+	def mount(
-		# to prevent "signature detected" errors
+		self,
-		for part_mod in filtered_part:
+		dev_path: Path,
-			if part_mod.dev_path:
+		target_mountpoint: Path,
-				debug(f'Wiping signatures from: {part_mod.dev_path}')
+		mount_fs: Optional[str] = None,
-				SysCommand(f'wipefs --all {part_mod.dev_path}')
+		create_target_mountpoint: bool = True,
+		options: List[str] = []
+	):
+		if create_target_mountpoint and not target_mountpoint.exists():
+			target_mountpoint.mkdir(parents=True, exist_ok=True)
 
-		# Sync with udev after wiping signatures
+		if not target_mountpoint.exists():
-		if filtered_part:
+			raise ValueError('Target mountpoint does not exist')
-			udev_sync()
 
-	def detect_pre_mounted_mods(self, base_mountpoint: Path) -> list[DeviceModification]:
+		lsblk_info = get_lsblk_info(dev_path)
-		part_mods: dict[Path, list[PartitionModification]] = {}
+		if target_mountpoint in lsblk_info.mountpoints:
+			info(f'Device already mounted at {target_mountpoint}')
+			return
+
+		str_options = ','.join(options)
+		str_options = f'-o {str_options}' if str_options else ''
+
+		mount_fs = f'-t {mount_fs}' if mount_fs else ''
+
+		command = f'mount {mount_fs} {str_options} {dev_path} {target_mountpoint}'
+
+		debug(f'Mounting {dev_path}: command')
+
+		try:
+			SysCommand(command)
+		except SysCallError as err:
+			raise DiskError(f'Could not mount {dev_path}: {command}\n{err.message}')
+
+	def umount(self, mountpoint: Path, recursive: bool = False):
+		try:
+			lsblk_info = get_lsblk_info(mountpoint)
+		except SysCallError as ex:
+			# this could happen if before partitioning the device contained 3 partitions
+			# and after partitioning only 2 partitions were created, then the modifications object
+			# will have a reference to /dev/sX3 which is being tried to umount here now
+			if 'not a block device' in ex.message:
+				return
+			raise ex
+
+		if len(lsblk_info.mountpoints) > 0:
+			debug(f'Partition {mountpoint} is currently mounted at: {[str(m) for m in lsblk_info.mountpoints]}')
+
+			for mountpoint in lsblk_info.mountpoints:
+				debug(f'Unmounting mountpoint: {mountpoint}')
+
+				command = 'umount'
+
+				if recursive:
+					command += ' -R'
+
+				SysCommand(f'{command} {mountpoint}')
+
+	def detect_pre_mounted_mods(self, base_mountpoint: Path) -> List[DeviceModification]:
+		part_mods: Dict[Path, List[PartitionModification]] = {}
 
 		for device in self.devices:
 			for part_info in device.partition_infos:
 				for mountpoint in part_info.mountpoints:
-					if mountpoint.is_relative_to(base_mountpoint):
+					if is_subpath(mountpoint, base_mountpoint):
 						path = Path(part_info.disk.device.path)
 						part_mods.setdefault(path, [])
-						part_mod = PartitionModification.from_existing_partition(part_info)
+						part_mods[path].append(PartitionModification.from_existing_partition(part_info))
-						if part_mod.mountpoint:
-							part_mod.mountpoint = mountpoint.root / mountpoint.relative_to(base_mountpoint)
-						else:
-							for subvol in part_mod.btrfs_subvols:
-								if sm := subvol.mountpoint:
-									subvol.mountpoint = sm.root / sm.relative_to(base_mountpoint)
-						part_mods[path].append(part_mod)
 						break
 
-		device_mods: list[DeviceModification] = []
+		device_mods: List[DeviceModification] = []
 		for device_path, mods in part_mods.items():
 			device_mod = DeviceModification(self._devices[device_path], False, mods)
 			device_mods.append(device_mod)
 
 		return device_mods
 
-	def partprobe(self, path: Path | None = None) -> None:
+	def partprobe(self, path: Optional[Path] = None):
 		if path is not None:
 			command = f'partprobe {path}'
 		else:
@@ -592,35 +586,42 @@ class DeviceHandler:
 			SysCommand(command)
 		except SysCallError as err:
 			if 'have been written, but we have been unable to inform the kernel of the change' in str(err):
-				log(f'Partprobe was not able to inform the kernel of the new disk state (ignoring error): {err}', fg='gray', level=logging.INFO)
+				log(f"Partprobe was not able to inform the kernel of the new disk state (ignoring error): {err}", fg="gray", level=logging.INFO)
 			else:
 				error(f'"{command}" failed to run (continuing anyway): {err}')
 
-	def _wipe(self, dev_path: Path) -> None:
+	def _wipe(self, dev_path: Path):
 		"""
 		Wipe a device (partition or otherwise) of meta-data, be it file system, LVM, etc.
-		@param dev_path:	Device path of the partition to be wiped.
+		@param dev_path:    Device path of the partition to be wiped.
-		@type dev_path:		str
+		@type dev_path:     str
 		"""
 		with open(dev_path, 'wb') as p:
 			p.write(bytearray(1024))
 
-	def wipe_dev(self, block_device: BDevice) -> None:
+	def wipe_dev(self, block_device: BDevice):
 		"""
 		Wipe the block device of meta-data, be it file system, LVM, etc.
 		This is not intended to be secure, but rather to ensure that
 		auto-discovery tools don't recognize anything here.
 		"""
 		info(f'Wiping partitions and metadata: {block_device.device_info.path}')
-
 		for partition in block_device.partition_infos:
-			luks = Luks2(partition.path)
-			if luks.isLuks():
-				luks.erase()
-
 			self._wipe(partition.path)
 
 		self._wipe(block_device.device_info.path)
 
 
 device_handler = DeviceHandler()
+
+
+def disk_layouts() -> str:
+	try:
+		lsblk_info = get_all_lsblk_info()
+		return json.dumps(lsblk_info, indent=4, sort_keys=True, cls=JSON)
+	except SysCallError as err:
+		warn(f"Could not return disk layouts: {err}")
+		return ''
+	except json.decoder.JSONDecodeError as err:
+		warn(f"Could not return disk layouts: {err}")
+		return ''