firefox: Add containersForce

[slotThe]

Description

Firefox, upon exit, creates the default containers.json file in place of the one that home-manager created. This leads to errors when switching to a new profile, as home-manager is careful with overwriting existing files. The added option toggles that behaviour.

Closes: #4989

---

I guess it would be best if this was inlined into the existing containers field, in a backwards compatible way; e.g.,

containers = {
  force = true;

  "name" = {
    id    = 1;
    color = "purple";
  };

but I haven't managed to make that work.

Checklist

• Change is backwards compatible.

• Code formatted with ./format.

• Code tested through nix-shell --pure tests -A run.all or nix develop --ignore-environment .#all using Flakes.

• Test cases updated/added. See example.

• Commit messages are formatted like

  {component}: {description}
  
  {long description}
  

  See CONTRIBUTING for more information and recent commit messages for examples.

Maintainer CC

@rycee @kira-bruneau

[DamienCassou]

Looks good to me.

[DamienCassou]

I guess it would be best if this was inlined into the existing containers field, in a backwards compatible way

I agree a profile.<name>.containers.force would be better but only if container definitions where not directly within profile.<name>.containers. Mixing both options and containers in the same attribute set is just confusing I think.

[teto]

since I've enabled this, firefox crashes on nixos-rebuild (at least the one in focus). Does anyone else see that ?

[DamienCassou]

since I've enabled this, firefox crashes on nixos-rebuild (at least the one in focus). Does anyone else see that ?

I've been using this for 2 weeks with no problem. I just tried again and it worked.

[slotThe]

I've also been using this ever since I created this PR, and across many nix-rebuilds I've never notices any weird things like that

[kanopeld]

working great for me, thanks! when to expect it in the master?

[DamienCassou]

Still using it and didn't notice anything.

[rycee]

Thanks! Merged to master now 🙂