anomalyco/opencode
1
0
Fork 0
mirror of https://github.com/anomalyco/opencode.git synced 2026-07-22 02:05:26 -04:00
Code Issues 5.9k Packages Projects Releases 19 Wiki Activity

nix-hashes workflow doesn't watch nix/scripts changes #8898

New Issue
Closed
opened 2026-02-16 18:11:07 -05:00 by yindo · 2 comments
yindo commented 2026-02-16 18:11:07 -05:00
Owner
Copy Link
Copy Source

Originally created by @jerome-benoit on GitHub (Feb 9, 2026).

Originally assigned to: @rekram1-node on GitHub.

The nix-hashes workflow only triggers on changes to bun.lock, package.json, packages/*/package.json, flake.lock, and the workflow file itself.

However, nix/node_modules.nix includes nix/scripts/*.ts and patches/* in the derivation. When these files change, the output hash changes, but the workflow doesn't re-run.

This causes hash mismatches when building locally after changes like #12694.

Fix:

  • Add nix/scripts/**, nix/node_modules.nix, and patches/** to the workflow paths trigger
  • Remove non-existent install/ from fileset
Originally created by @jerome-benoit on GitHub (Feb 9, 2026). Originally assigned to: @rekram1-node on GitHub. The `nix-hashes` workflow only triggers on changes to `bun.lock`, `package.json`, `packages/*/package.json`, `flake.lock`, and the workflow file itself. However, `nix/node_modules.nix` includes `nix/scripts/*.ts` and `patches/*` in the derivation. When these files change, the output hash changes, but the workflow doesn't re-run. This causes hash mismatches when building locally after changes like #12694. **Fix**: - Add `nix/scripts/**`, `nix/node_modules.nix`, and `patches/**` to the workflow paths trigger - Remove non-existent `install/` from fileset
yindo added the nix label 2026-02-16 18:11:07 -05:00
yindo closed this issue 2026-02-16 18:11:07 -05:00
yindo commented 2026-02-16 18:11:07 -05:00
Author
Owner
Copy Link
Copy Source

@gigamonster256 commented on GitHub (Feb 9, 2026):

the desktop build needs the install script

Cannot build '/nix/store/jrkk83ywjk3pvvsr46lzxc3f7x0a0gnn-opencode-desktop-1.1.53-5be1202-dirty.drv'.
Reason: builder failed with exit code 1.
Output paths:
 /nix/store/bgwaaavb52cmpn76dw6qs1sijvwp4s1x-opencode-desktop-1.1.53-5be1202-dirty
Last 25 log lines:
> error: couldn't read `src/../../../../install`: No such file or directory (os error 2)
>   --> src/cli.rs:56:30
>    |
> 56 | const INSTALL_SCRIPT: &str = include_str!("../../../../install");
>    |                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>
> warning: unused import: `process::Command`
>   --> src/lib.rs:25:5
>    |
> 25 |     process::Command,
>    |     ^^^^^^^^^^^^^^^^
>    |
>    = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
>
> warning: unused variable: `app_name`
>    --> src/lib.rs:237:20
>     |
> 237 | fn check_linux_app(app_name: &str) -> bool {
>     |                    ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_app_name`
>     |
>     = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
>
> warning: `opencode-desktop` (lib) generated 2 warnings
> error: could not compile `opencode-desktop` (lib) due to 1 previous error; 2 warnings emitted
>        Error failed to build app: failed to build app
@gigamonster256 commented on GitHub (Feb 9, 2026): the desktop build needs the install script ```bash Cannot build '/nix/store/jrkk83ywjk3pvvsr46lzxc3f7x0a0gnn-opencode-desktop-1.1.53-5be1202-dirty.drv'. Reason: builder failed with exit code 1. Output paths: /nix/store/bgwaaavb52cmpn76dw6qs1sijvwp4s1x-opencode-desktop-1.1.53-5be1202-dirty Last 25 log lines: > error: couldn't read `src/../../../../install`: No such file or directory (os error 2) > --> src/cli.rs:56:30 > | > 56 | const INSTALL_SCRIPT: &str = include_str!("../../../../install"); > | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > warning: unused import: `process::Command` > --> src/lib.rs:25:5 > | > 25 | process::Command, > | ^^^^^^^^^^^^^^^^ > | > = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default > > warning: unused variable: `app_name` > --> src/lib.rs:237:20 > | > 237 | fn check_linux_app(app_name: &str) -> bool { > | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_app_name` > | > = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default > > warning: `opencode-desktop` (lib) generated 2 warnings > error: could not compile `opencode-desktop` (lib) due to 1 previous error; 2 warnings emitted > Error failed to build app: failed to build app ```
yindo commented 2026-02-16 18:11:07 -05:00
Author
Owner
Copy Link
Copy Source

@gigamonster256 commented on GitHub (Feb 9, 2026):

Remove non-existent install/ from fileset

it's not a non-existend directory, its the install script file

@gigamonster256 commented on GitHub (Feb 9, 2026): > Remove non-existent install/ from fileset it's not a non-existend directory, its the install script file
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
acp
bad
bug
discussion
docs
documentation
enhancement
extension
good first issue
growth
help-wanted
mcp
model-problem
needs:compliance
nix
opentui
perf
pull-request
Mirrored from GitHub Pull Request
 tui
web
windows
zen
No Label nix
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#8898
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?