Create nix module drumknotty which manages worblehat and dibbler for skrot. #129

Open
vegardbm wants to merge 7 commits from drumknotty into main
Owner

DrumknoTTY is a nix module that owns the screen session that skrot runs. Inside the screen session it spawns two windows and programs: dibbler and worblehat. This is essentially done to deploy worblehat. Both dibbler and worblehat each have a module that allows them to start in a screen session, but this essentially combines these two modules into one.

DrumknoTTY is a nix module that owns the screen session that skrot runs. Inside the screen session it spawns two windows and programs: dibbler and worblehat. This is essentially done to deploy worblehat. Both dibbler and worblehat each have a module that allows them to start in a screen session, but this essentially combines these two modules into one.
Author
Owner

This might also require updating some inputs such as worblehat.

This might also require updating some inputs such as worblehat.
oysteikt was assigned by vegardbm 2026-03-19 17:28:15 +01:00
oysteikt was unassigned by vegardbm 2026-03-19 17:28:23 +01:00
vegardbm requested review from Owners 2026-03-19 18:36:06 +01:00
Owner

Kan/vil du slå sammen noen commits her, typ fixupe de fix-commitene i en interactive rebase eller noe, sånn at det blir en håndfull gode commits når vi merger inn i main?

Kan/vil du slå sammen noen commits her, typ fixupe de fix-commitene i en interactive rebase eller noe, sånn at det blir en håndfull gode commits når vi merger inn i main?
Author
Owner

Jeg har allerede slått sammen en del av commitsa med en interactive rebase, men ikke så mange for det meste typos og slikt.

Jeg har allerede slått sammen en del av commitsa med en interactive rebase, men ikke så mange for det meste typos og slikt.
Author
Owner

Greien er at ting ikke egentlig funker i det hele tatt før siste commit her da, men jeg kan slå sammen sånn at man ender opp med commits som har en successful nix flake eval.

Greien er at ting ikke egentlig funker i det hele tatt før siste commit her da, men jeg kan slå sammen sånn at man ender opp med commits som har en successful nix flake eval.
vegardbm added 3 commits 2026-03-21 13:47:40 +01:00
flake.lock: bump dibbler and worblehat
All checks were successful
Eval nix flake / evals (push) Successful in 9m58s
Eval nix flake / evals (pull_request) Successful in 8m39s
083098d76e
vegardbm force-pushed drumknotty from b96c4407f1 to 083098d76e 2026-03-21 13:47:40 +01:00 Compare
Owner

Nice, ser bra ut nå! Er det denne konfigen som kjører live på skrot, eller er den ellers testet?
Vent gjerne på ordentlig oysteikt-review, men jeg ser ingen blinkende varsellys her ved gjennomlesing

Nice, ser bra ut nå! Er det denne konfigen som kjører live på skrot, eller er den ellers testet? Vent gjerne på ordentlig oysteikt-review, men jeg ser ingen blinkende varsellys her ved gjennomlesing
Author
Owner

Den kjører ikke live på skrot siden den rebygger seg tilbake til forrige versjon på natten, men jeg har spesifikt testet det på skrot ja.

Den kjører ikke live på skrot siden den rebygger seg tilbake til forrige versjon på natten, men jeg har spesifikt testet det på skrot ja.
Author
Owner

Jeg tror nok oysteikt har en del kommentarer på det her siden det i praksis dupliserer modulene til både dibbler og worblehat. Det er jo mulig å flytte kode og deuplisere, men det er et utgangspunkt som fungerer.

Jeg tror nok oysteikt har en del kommentarer på det her siden det i praksis dupliserer modulene til både dibbler og worblehat. Det er jo mulig å flytte kode og deuplisere, men det er et utgangspunkt som fungerer.
Author
Owner

Det er derimot ett problem, og det er at worblehat er default når den starter opp. Det er fordi screen alltid starter på vinduet som nyligst ble lagd, og jeg har ikke funnet ut en kommando som lar deg bestemme hvilket vindu man skal koble til først.

Det er derimot ett problem, og det er at worblehat er default når den starter opp. Det er fordi screen alltid starter på vinduet som nyligst ble lagd, og jeg har ikke funnet ut en kommando som lar deg bestemme hvilket vindu man skal koble til først.
vegardbm added 1 commit 2026-03-21 14:34:42 +01:00
attach with dibbler window selected
All checks were successful
Eval nix flake / evals (push) Successful in 10m1s
Eval nix flake / evals (pull_request) Successful in 10m15s
45a32fad73
Author
Owner

Nå skal det problemet være løst.

Nå skal det problemet være løst.
Author
Owner

Et annet problem er at screen bruker 0-indexing, noe som er upraktisk i forhold til keybinds. Det er fordi at man kan bytte mellom vinduer basert på index, og 0 og 1 er langt unna hverandre. Antakeligvis kan man konfigurere dette da.

Et annet problem er at screen bruker 0-indexing, noe som er upraktisk i forhold til keybinds. Det er fordi at man kan bytte mellom vinduer basert på index, og 0 og 1 er langt unna hverandre. Antakeligvis kan man konfigurere dette da.
All checks were successful
Eval nix flake / evals (push) Successful in 10m1s
Eval nix flake / evals (pull_request) Successful in 10m15s
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin drumknotty:drumknotty
git checkout drumknotty
Sign in to join this conversation.