diff --git a/README.md b/README.md index 1e6d711..48dec2c 100644 --- a/README.md +++ b/README.md @@ -42,6 +42,37 @@ you'll have to manually enable the service for each user (see below). #### Enable the service +##### Automatically for all users + +Instead of just +```nix +{ services.vscode-server.enable = true; } +``` + +use: +```nix +{ + services.vscode-server = { + enable = true; + enableForUsers.enable = true; + }; +} +``` + +This will use `tmpfiles` to setup the permanent symlink described below for each regular user. + +If you do not wish to enable it for all users, but only for a specific subset, the list of users this will be setup for can be overridden: + +```nix +{ + services.vscode-server.enableForUsers.users = [ "alice" "bob" ]; +} +``` + +Note that when disabling `services.vscode-server.enableForUsers.enable`, the file that was created in the user's `.config/systemd/user` will not be cleaned up, so you will have to clean it up manually. + +##### Manually for each user + And then enable them for the relevant users: ```bash @@ -79,6 +110,8 @@ ln -sfT /run/current-system/etc/systemd/user/auto-fix-vscode-server.service ~/.c ### Home Manager +#### Install as a tarball + Put this code into your [home-manager](https://github.com/nix-community/home-manager) configuration i.e. in `~/.config/nixpkgs/home.nix`: ```nix @@ -91,6 +124,25 @@ Put this code into your [home-manager](https://github.com/nix-community/home-man } ``` +#### Install as a flake + +```nix +{ + inputs.vscode-server.url = "github:nix-community/nixos-vscode-server"; + + outputs = { self, vscode-server, home-manager }: { + homeConfigurations.yourhostname = home-manager.lib.homeManagerConfiguration { + modules = [ + vscode-server.homeModules.default + ({ config, pkgs, ... }: { + services.vscode-server.enable = true; + }) + ]; + }; + }; +} +``` + ## Usage When using VS Code as released by Microsoft without any special needs, just enabling and starting the service should be enough to make things work. If you have some custom build or needs, there are a few options available that might help you out. @@ -140,14 +192,26 @@ This same list is also used to determine the `RPATH` when automatically patching ``` ### `installPath` -The installation path for VS Code server is configurable and the default can differ for alternative builds (e.g. oss and insider), so this option allows you to configure which installation path should be monitered and automatically fixed. +The installation path for VS Code server is configurable. You can specify either a single path as a string or multiple paths as a list. If you have multiple installations (e.g., stable, OSS, and insider builds), you can specify all of them to be monitored and automatically fixed. ```nix +# Single path (string) { - services.vscode-server.installPath = "$HOME/.vscode-server-oss"; + services.vscode-server.installPath = "$HOME/.vscode-server"; +} + +# Multiple paths (list) +{ + services.vscode-server.installPath = [ + "$HOME/.vscode-server" + "$HOME/.vscode-server-oss" + "$HOME/.vscode-server-insiders" + ]; } ``` +String values are automatically coerced to a single-element list for backwards compatibility. + ### `postPatch` The goal of this project is to make VS Code server work with NixOS, anything more is outside of the scope of the project, but if you want additional things to be done, you can use this hook to run a shell script after the patching is done. diff --git a/modules/vscode-server/default.nix b/modules/vscode-server/default.nix index 6e9b419..216f76f 100644 --- a/modules/vscode-server/default.nix +++ b/modules/vscode-server/default.nix @@ -1,10 +1,55 @@ -import ./module.nix ({ - name, - description, - serviceConfig, -}: { - systemd.user.services.${name} = { - inherit description serviceConfig; - wantedBy = [ "default.target" ]; - }; -}) +import ./module.nix ( + { name + , description + , serviceConfig + , lib + , config + , cfg + }: lib.mkMerge [ + { + systemd.user.services.${name} = { + inherit description serviceConfig; + wantedBy = [ "default.target" ]; + }; + } + (lib.mkIf cfg.enableForUsers.enable { + systemd.tmpfiles.settings = + let + forEachUser = ({ path, file }: builtins.listToAttrs + (builtins.map + (username: let user = config.users.users.${username}; in { + name = "${user.home}/${path}"; + value = file user.name; + }) + cfg.enableForUsers.users)); + homeDirectory = (path: forEachUser { + inherit path; + file = (username: { + "d" = { + user = username; + group = "users"; + mode = "0755"; + }; + }); + }); + in + { + # We need to create each of the folders before the next file otherwise parents get owned by root + "80-vscode-server-enable-for-users-create-config-folder" = homeDirectory ".config"; + "81-vscode-server-enable-for-users-create-systemd-folder" = homeDirectory ".config/systemd"; + "82-vscode-server-enable-for-users-create-systemd-user-folder" = homeDirectory ".config/systemd/user"; + "83-vscode-server-enable-for-users-enable-auto-fix-vscode-server-service" = forEachUser { + path = ".config/systemd/user/auto-fix-vscode-server.service"; + file = (username: { + "L+" = { + user = username; + group = "users"; + # This path is made available by `services.vscode-server.enable = true;` + argument = "/run/current-system/etc/systemd/user/auto-fix-vscode-server.service"; + }; + }); + }; + }; + }) + ] +) diff --git a/modules/vscode-server/home.nix b/modules/vscode-server/home.nix index 616145b..bd8d3ca 100644 --- a/modules/vscode-server/home.nix +++ b/modules/vscode-server/home.nix @@ -1,17 +1,26 @@ -import ./module.nix ({ - name, - description, - serviceConfig, -}: { - systemd.user.services.${name} = { - Unit = { - Description = description; - }; +import ./module.nix ( + { name + , description + , serviceConfig + , cfg + , ... + }: + { + systemd.user.services.${name} = { + Unit = { + Description = description; + }; - Service = serviceConfig; + Service = serviceConfig; - Install = { - WantedBy = [ "default.target" ]; + Install = { + WantedBy = [ "default.target" ]; + }; }; - }; -}) + + assertions = [{ + assertion = !cfg.enableForUsers.enable; + message = "enableForUsers.enable=true doesn't make sense when using nixos-vscode-server as a home-manager module"; + }]; + } +) diff --git a/modules/vscode-server/module.nix b/modules/vscode-server/module.nix index 4bf714b..3df329a 100644 --- a/modules/vscode-server/module.nix +++ b/modules/vscode-server/module.nix @@ -6,9 +6,9 @@ moduleConfig: { }: { options.services.vscode-server = let inherit (lib) mkEnableOption mkOption; - inherit (lib.types) lines listOf nullOr package str; + inherit (lib.types) lines listOf nullOr package str bool passwdEntry; in { - enable = mkEnableOption "VS Code Server"; + enable = mkEnableOption "VS Code Server autofix"; enableFHS = mkEnableOption "a FHS compatible environment"; @@ -33,11 +33,13 @@ moduleConfig: { }; installPath = mkOption { - type = str; - default = "$HOME/.vscode-server"; - example = "$HOME/.vscode-server-oss"; + type = lib.types.coercedTo str (x: [x]) (listOf str); + default = [ "$HOME/.vscode-server" ]; + example = [ "$HOME/.vscode-server" "$HOME/.vscode-server-oss" "$HOME/.vscode-server-insiders" ]; description = '' - The install path. + Path(s) where VS Code Server will be installed. + Accepts either a single path string or a list of paths. + String values are automatically coerced to a single-element list for backwards compatibility. ''; }; @@ -49,6 +51,35 @@ moduleConfig: { This can be used as a hook for custom further patching. ''; }; + + enableForUsers = { + enable = mkOption { + type = bool; + default = false; + example = true; + description = '' + Whether to enable the VS Code Server auto-fix service for each user. + + This only makes sense if auto-fix-vscode-server is installed as a NixOS module. + + This automatically sets up the service's symlinks for systemd in each users' home directory. + + By default this will set it up for all regular users, but that list can be overridden by setting `users`. + ''; + }; + + users = mkOption { + type = listOf (passwdEntry str); + default = builtins.attrNames (lib.attrsets.filterAttrs (username: userOptions: userOptions.isNormalUser) config.users.users); + defaultText = "builtins.attrNames (lib.filterAttrs (_: userOptions: userOptions.isNormalUser) config.users.users)"; + example = [ "alice" "bob" ]; + description = '' + List of users to enable the VS Code Server auto-fix service for. + + By default this will fallback to the list of "normal" users. + ''; + }; + }; }; config = let @@ -56,7 +87,7 @@ moduleConfig: { cfg = config.services.vscode-server; auto-fix-vscode-server = pkgs.callPackage ../../pkgs/auto-fix-vscode-server.nix - (removeAttrs cfg [ "enable" ]); + (removeAttrs cfg [ "enable" "enableForUsers" ]); in mkIf cfg.enable (mkMerge [ { @@ -71,9 +102,10 @@ moduleConfig: { # Unfortunately the monitor does not kill itself when it stops monitoring, # so rather than creating our own restart mechanism, we leverage systemd to do this for us. Restart = "always"; - RestartSec = 0; + RestartSec = 5; ExecStart = "${auto-fix-vscode-server}/bin/auto-fix-vscode-server"; }; + inherit config cfg lib; }) ]); } diff --git a/pkgs/auto-fix-vscode-server.nix b/pkgs/auto-fix-vscode-server.nix index 2649e40..a17bbf2 100644 --- a/pkgs/auto-fix-vscode-server.nix +++ b/pkgs/auto-fix-vscode-server.nix @@ -21,7 +21,7 @@ enableFHS ? false, nodejsPackage ? null, extraRuntimeDependencies ? [ ], - installPath ? "$HOME/.vscode-server", + installPath ? [ "$HOME/.vscode-server" ], postPatch ? "", }: let inherit (lib) makeBinPath makeLibraryPath optionalString; @@ -123,11 +123,12 @@ name = "auto-fix-vscode-server"; runtimeInputs = [ coreutils findutils inotify-tools ]; text = '' - bins_dir_1=${installPath}/bin - bins_dir_2=${installPath}/cli/servers + # Convert installPath list to an array + IFS=':' read -r -a installPaths <<< "${lib.concatStringsSep ":" installPath}" patch_bin () { local actual_dir="$1" + local current_install_path="$2" local patched_file="$actual_dir/.nixos-patched" if [[ -e $patched_file ]]; then @@ -139,9 +140,9 @@ old_patched_file="$(basename "$actual_dir")" if [[ $old_patched_file == "server" ]]; then old_patched_file="$(basename "$(dirname "$actual_dir")")" - old_patched_file="${installPath}/.''${old_patched_file%%.*}.patched" + old_patched_file="$current_install_path/.''${old_patched_file%%.*}.patched" else - old_patched_file="${installPath}/.''${old_patched_file%%-*}.patched" + old_patched_file="$current_install_path/.''${old_patched_file%%-*}.patched" fi if [[ -e $old_patched_file ]]; then echo "Migrating old nixos-vscode-server patch marker file to new location in $actual_dir." >&2 @@ -183,38 +184,58 @@ echo 0 > "$patched_file" } - mkdir -p "$bins_dir_1" "$bins_dir_2" - while read -rd ''' bin; do - if [[ $bin == "$bins_dir_2"* ]]; then + # Initialize arrays for bins_dirs_1 and bins_dirs_2 + bins_dirs_1=() + bins_dirs_2=() + + # Populate bins_dirs_1 and bins_dirs_2 based on installPaths + for current_install_path in "''${installPaths[@]}"; do + bins_dirs_1+=("$current_install_path/bin") + bins_dirs_2+=("$current_install_path/cli/servers") + done + + # Create directories and patch existing bins + for bins_dir_1 in "''${bins_dirs_1[@]}"; do + mkdir -p "$bins_dir_1" + while read -rd ''' bin; do + patch_bin "$bin" "$(dirname "$(dirname "$bin")")" + done < <(find "$bins_dir_1" -mindepth 1 -maxdepth 1 -type d -printf '%p\0') + done + for bins_dir_2 in "''${bins_dirs_2[@]}"; do + mkdir -p "$bins_dir_2" + while read -rd ''' bin; do bin="$bin/server" - fi - patch_bin "$bin" - done < <(find "$bins_dir_1" "$bins_dir_2" -mindepth 1 -maxdepth 1 -type d -printf '%p\0') - + patch_bin "$bin" "$(dirname "$(dirname "$bin")")" + done < <(find "$bins_dir_2" -mindepth 1 -maxdepth 1 -type d -printf '%p\0') + done + + # Watch for new installations while IFS=: read -r bins_dir bin event; do # A new version of the VS Code Server is being created. if [[ $event == 'CREATE,ISDIR' ]]; then actual_dir="$bins_dir$bin" - if [[ "$bins_dir" == "$bins_dir_2/" ]]; then + actual_install_path="$(dirname "$bins_dir")" + if [[ "$bins_dir" == */cli/servers/ ]]; then actual_dir="$actual_dir/server" # Hope that VSCode will not die if the directory exists when it tries to install, otherwise we'll need to # use a coproc to wait for the directory to be created without entering in a race, then watch for the node # file to be created (probably while also avoiding a race) # https://unix.stackexchange.com/a/185370 mkdir -p "$actual_dir" + actual_install_path="$(dirname "$(dirname "$bins_dir")")" fi echo "VS Code server is being installed in $actual_dir..." >&2 # Quickly create a node file, which will be removed when vscode installs its own version touch "$actual_dir/node" # Hope we don't race... inotifywait -qq -e DELETE_SELF "$actual_dir/node" - patch_bin "$actual_dir" + patch_bin "$actual_dir" "$actual_install_path" # The monitored directory is deleted, e.g. when "Uninstall VS Code Server from Host" has been run. elif [[ $event == DELETE_SELF ]]; then # See the comments above Restart in the service config. exit 0 fi - done < <(inotifywait -q -m -e CREATE,ISDIR -e DELETE_SELF --format '%w:%f:%e' "$bins_dir_1" "$bins_dir_2") + done < <(inotifywait -q -m -e CREATE,ISDIR -e DELETE_SELF --format '%w:%f:%e' "''${bins_dirs_1[@]}" "''${bins_dirs_2[@]}") ''; }; in