Improve deployment error handling and disable vim

[AronNovak]

Before the config change, deploy-to-nixos failed with:

       Failed assertions:
       - {option}`programs.vim.defaultEditor` requires {option}`programs.vim.enable` to be set to true.

NixOS 25.05 added this assertion. The IHP module sets programs.vim.defaultEditor = true, so we override it with mkForce false instead of enabling vim (which isn't needed for deployment).

[amitaibu]

- {option}programs.vim.defaultEditor requires {option}programs.vim.enable to be set to true.

I think we should ideally find how to disable vim entirely, as it shouldn't be needed for the app's deployment

[AronNovak]

@amitaibu Right, I simplified the solution based on this, it works as well.

[Copilot]

Pull request overview

Updates NixOS production configuration and CI deploy scripting to avoid NixOS 25.05 vim assertions and provide clearer deployment failure reporting.

Changes:

• Force programs.vim.defaultEditor = false in the production NixOS host config to override the IHP module’s default and satisfy the NixOS 25.05 assertion.
• Improve GitHub Actions deploy step failure handling by enabling strict shell behavior and emitting a GitHub Actions ::error:: annotation on failure.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
Config/nix/hosts/production/configuration.nix	Overrides programs.vim.defaultEditor to prevent the NixOS 25.05 assertion from breaking deployment.
.github/workflows/test.yml	Adds stricter shell settings and a clearer error annotation when deploy-to-nixos fails.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[mpscholten]

I think we should rather updates this one here: https://github.com/digitallyinduced/ihp/blob/master/NixSupport/nixosModules/appWithPostgres.nix#L23 because that's the actual source of vim here 👍

[amitaibu]

@copilot address #55 (comment)