Skip to content
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 8 additions & 8 deletions overlays/default.nix
Original file line number Diff line number Diff line change
@@ -1,18 +1,18 @@
{ inputs }:
let
# Override clawdbot source to v2026.1.21
# Override clawdbot source to v2026.1.22
clawdbotSourceOverride = {
owner = "clawdbot";
repo = "clawdbot";
rev = "80c1edc3ff43b3bd3b7b545eed79f303d992f7dc";
hash = "sha256-IsTNC79KXKL7ByBh7zUmH6qXx0YFdiQ4a4TI40C53U8=";
rev = "c48751a99c1aacea1092927ff85a09ee40ac374d";
hash = "sha256-DJZulGqo2E7XvitR3kYKme9vqSfQyYs9I9fsBIpqQdQ=";
pnpmDepsHash = "sha256-tGzKcCiZNlWlKMNNFmxcFpIvO92G9myhM+OYaGea4hw=";
};
# Override clawdbot-app to v2026.1.21 (fixes broken app package)
# Override clawdbot-app to v2026.1.22 (fixes broken app package)
clawdbotAppOverride = {
version = "2026.1.21";
url = "https://github.com/clawdbot/clawdbot/releases/download/v2026.1.21/Clawdbot-2026.1.21.zip";
hash = "sha256-EhGRakuN0dhEkXvrOd21t79odf4T2jY7oKFRubLqGbI=";
version = "2026.1.22";
url = "https://github.com/clawdbot/clawdbot/releases/download/v2026.1.22/Clawdbot-2026.1.22.zip";
hash = "sha256-IXb+WNrSqH80mYOR62rRK+mJUhReJ+CZVPNFefgkcao=";
};
Comment on lines +3 to 16

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

To improve maintainability and reduce redundancy, you could define the clawdbot version in a single variable at the top-level let block. This would make future version bumps easier and less error-prone.

This change would require moving clawdbotVersion from the inner let block (line 25) to the top-level let block, and then using it for the version and URL.

Note: Applying this refactoring will require updating scripts/upgrade-overlays.sh to modify the single version variable instead of using multiple sed commands.

  clawdbotVersion = "2026.1.22";
  # Override clawdbot source to v2026.1.22
  clawdbotSourceOverride = {
    owner = "clawdbot";
    repo = "clawdbot";
    rev = "c48751a99c1aacea1092927ff85a09ee40ac374d";
    hash = "sha256-DJZulGqo2E7XvitR3kYKme9vqSfQyYs9I9fsBIpqQdQ=";
    pnpmDepsHash = "sha256-tGzKcCiZNlWlKMNNFmxcFpIvO92G9myhM+OYaGea4hw=";
  };
  # Override clawdbot-app to v2026.1.22 (fixes broken app package)
  clawdbotAppOverride = {
    version = clawdbotVersion;
    url = "https://github.com/clawdbot/clawdbot/releases/download/v${clawdbotVersion}/Clawdbot-${clawdbotVersion}.zip";
    hash = "sha256-IXb+WNrSqH80mYOR62rRK+mJUhReJ+CZVPNFefgkcao=";
  };

in
[
Expand All @@ -22,7 +22,7 @@ in
(
final: prev:
let
clawdbotVersion = "2026.1.21";
clawdbotVersion = "2026.1.22";

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Following the refactoring suggested for lines 3-16 to define clawdbotVersion in the outer scope, this line would become redundant and should be removed.

basePkgs = import "${inputs.nix-clawdbot}/nix/packages" {
pkgs = prev;
sourceInfo = clawdbotSourceOverride;
Expand Down
Loading