plac

Unofficial Roon clients

Commits at 9eb4f42c08b788abe88ead92b34617a0b6f6e64d

  1. 9eb4f42c gtk-adwaita: Show server name in window title Shota FUJI authored at Shota FUJI comitted at
  2. 40089fa6 gtk-adwaita: Proper style for playback UI Shota FUJI authored at Shota FUJI comitted at
  3. aae8758c gtk-adwaita: Seek bar Shota FUJI authored at Shota FUJI comitted at
  4. f814958a core: Seek change event Shota FUJI authored at Shota FUJI comitted at
  5. 915553e7 core/gtk-adwaita: Now playing info Shota FUJI authored at Shota FUJI comitted at
  6. cfcffb49 gtk-adwaita: Skip forward and backward buttons Shota FUJI authored at Shota FUJI comitted at
  7. a089a9dd gtk-adwaita: Rename Plac/Discovery.vala to Plac.vala The file contains non-Discovery code. Shota FUJI authored at Shota FUJI comitted at
  8. 848c1b69 gtk-adwaita: Play and pause control Shota FUJI authored at Shota FUJI comitted at
  9. 3ac492f2 core: Playback control API Shota FUJI authored at Shota FUJI comitted at
  10. 87e78be8 core: Parse allowed actions for a zone Required for playback controls. Shota FUJI authored at Shota FUJI comitted at
  11. d6154cfd core: Fix GLib error log terminates application GLib's error level log terminates application, even though the function is logging one and nothing says error is fatal to application. This bullshit design is the root cause of app crashes when server closed a connection. Shota FUJI authored at Shota FUJI comitted at
  12. 588336be gtk-adwaita: Display connection error Shota FUJI authored at Shota FUJI comitted at
  13. e0f587b9 macos: Minimal readonly toolbar (zone display) Shota FUJI authored at Shota FUJI comitted at
  14. 325ef897 macos: Init schema Why have the project been working for now without schema? idk. Shota FUJI authored at Shota FUJI comitted at
  15. 2040425f macos: Fix missing argument (compile error) Shota FUJI authored at Shota FUJI comitted at
  16. cd03a5ae gtk-adwaita: Save and restore connection Shota FUJI authored at Shota FUJI comitted at
  17. fa78ce00 gtk-adwaita: GSettings for connection state Shota FUJI authored at Shota FUJI comitted at
  18. d587253c cli: Playback command For testing token restoration. Shota FUJI authored at Shota FUJI comitted at
  19. 644556c3 core: Use saved token Shota FUJI authored at Shota FUJI comitted at
  20. 7120e027 core: Fix connection struct not released without manual ref Shota FUJI authored at Shota FUJI comitted at
  21. d530b378 cli: Connect command This patch also fixes find API not working on search hit. Shota FUJI authored at Shota FUJI comitted at
  22. 8edfd166 core: Server find API Shota FUJI authored at Shota FUJI comitted at
  23. 7c6f0d40 macos: UI for server discovery Shota FUJI authored at Shota FUJI comitted at
  24. f0f5f054 cli: Remove memcheck option Most of the time I run cli, it's for checking memory leaks. There is no usecase for disabling memcheck option. Shota FUJI authored at Shota FUJI comitted at
  25. 58c2a848 cli: Switch to new core I once considered removing "cli" together, but it's very useful at debugging memory leaks and such. Shota FUJI authored at Shota FUJI comitted at
  26. 047caaf8 core: Switch to Atomic Reference Counting I don't know how to trigger access error on non-atomic variant though. Shota FUJI authored at Shota FUJI comitted at
  27. c0364945 core: Workaround for Debug build crash on macOS Shota FUJI authored at Shota FUJI comitted at
  28. 3a4c2e2a macos: Use new core Shota FUJI authored at Shota FUJI comitted at
  29. 63e467c0 gtk-adwaita/core: Extract core logic to dedicated package I found out this logic for Gtk-Adwaita can be used naturally in Swift. Shota FUJI authored at Shota FUJI comitted at
  30. 357690b0 Rename "core" to "old_core" I'd like to rewrite it from scratch but existing usage prevents that. Shota FUJI authored at Shota FUJI comitted at
  31. f8d353fc gtk-adwaita: per app core instead of shared core Writing a library that is comfortable to use both in Vala and in Swift is really challenging. That makes library pretty C-ish, I mean barebone manual memory management which is hard to use, design and implement. By having dedicated Zig modules and extracting shared logic as a Zig module instead of C library, development speed and application quality will be improved thanks to more ergnomic API and natural memory management (reference counting). This commit does not include that "shared logic" part, because I have no idea how macos/ part will go on. Although the number of allocation massively increased due to heavy copy, chance of memory leak or invalid writes (double-free, use-after-free) is significantly lower compared to my previous shitty implementation. Shota FUJI authored at Shota FUJI comitted at
  32. 6d1b49ca Migrate from devbox to Nix Flake Turns out, versioned package manager powered by Nix, won't work. Many libraries, especially runtime dependencies, suffers from duplicated packages and application crashes with cryptic error messages. Shota FUJI authored at Shota FUJI comitted at
  33. 422c9306 gtk-adwaita: Style zone list like menu When navigating using keyboard, closing popover does not reset selection state thus UI goes incorrect state where inactive row can be displayed as active. However, I'm not going to spend times on details: functionality first. Shota FUJI authored at Shota FUJI comitted at
  34. 3b7f83e8 gtk-adwaita: Zone selection Now user can switch zone to show in UI. Shota FUJI authored at Shota FUJI comitted at
  35. 2fa4ec35 cli: Ignore Valgrind core dump Shota FUJI authored at Shota FUJI comitted at
  36. 2bbd18f1 core: Subscribe to zone updates Shota FUJI authored at Shota FUJI comitted at
  37. 1e9c2f8b core: Cancel pending response handler on deinit Shota FUJI authored at Shota FUJI comitted at
  38. 4c605385 gtk-adwaita: Fix closing on server selector doesn't exit application Shota FUJI authored at Shota FUJI comitted at
  39. 25cd076d core: Use thread pool so every threads will be closed Unterminated threads were noise on Valgrind. Shota FUJI authored at Shota FUJI comitted at
  40. a6de2c9e core: Fix size of struct containing `usize` being incorrect Shota FUJI authored at Shota FUJI comitted at
  41. 36158e3d gtk-adwaita: List zones and show playback state of the first one UI is shitty but it at least demonstrates zone fetching functionality. Shota FUJI authored at Shota FUJI comitted at
  42. 31f27d67 gtk-adwaita: Split widgets into separate files Shota FUJI authored at Shota FUJI comitted at
  43. 8cc196cb gtk-adwaita: Automatic lookup for resources and Vala source Shota FUJI authored at Shota FUJI comitted at
  44. b9dfb81f gtk-adwaita: Split connecting state to separate UI file Too much nesting. Shota FUJI authored at Shota FUJI comitted at
  45. 63ba1ec7 gtk-adwaita: Properly style connecting UI Shota FUJI authored at Shota FUJI comitted at
  46. e9531b48 core/gtk-adwaita: Export mutex API The initial plan was to complete every actions inside callbacks. However, most UI libraries/frameworks requires main thread for accessing UI. That means, state could be changed during the UI operation because it's outside the callback's mutex locking. Shota FUJI authored at Shota FUJI comitted at
  47. 9ed3f8f9 gtk-adwaita: Access GTK only from main thread App was flaky due to off-thread accesses. Shota FUJI authored at Shota FUJI comitted at
  48. 1aa6eb97 core: Disarm function should take context/user data too Since functions are static and what makes "function for this context" is a context pointer, disarm functions too have to take that. Shota FUJI authored at Shota FUJI comitted at
  49. 588603fc core: Remove legacy manual API Shota FUJI authored at Shota FUJI comitted at
  50. 4b14946c Do not use swift-format@5.8 I thought it works okay even for Swift 6 source code given Swift 6 announcement post and various posts do not list syntax changes. They seem to be "docs does not matter, we have migration tool!" people. Somehow swift-format@5.8 rejects valid function call (in my case, it was struct initializer) and if I commented the relavant section, it messed code so badly that I had to "fix" every unnecessary stupid newlines. Until nixpkgs or Swift team fixes build pipeline, I have to manually invoke the latest swift-format via XCode. fuck. Shota FUJI authored at Shota FUJI comitted at
  51. 496f08e9 macos: Migrate to new C API Shota FUJI authored at Shota FUJI comitted at
  52. 8a172f02 gtk-adwaita: Save and restore connection It's extremely flaky due to thread unsafety... I have to organize callbacks sure. Shota FUJI authored at Shota FUJI comitted at
  53. 9435612b core: Save connection state to a file Previously I thought each apps having platform specific save logic would be better than implementing it in the core. However, Vala has no simple way to do that (had to import various GLib things). Apparently every platform I plan to implement uses file system for state files, thus implementing in the core is safe as long as it takes full file path. Shota FUJI authored at Shota FUJI comitted at
  54. 3f169255 core: Avoid implicit pass-by-reference conversion Shota FUJI authored at Shota FUJI comitted at
  55. 5dc2caa9 core: Comment on why `scan_result` is not deinit-ed I thought it's confusing during refactoring. Shota FUJI authored at Shota FUJI comitted at
  56. 6755b30b core: Adhere init/deinit to guideline This makes "who manages this memory" more obvious. Shota FUJI authored at Shota FUJI comitted at
  57. 8b399348 core: Docs for code style This is solo development project but I'm already confused due to inconsistency. As there is no general Zig guidelines, I had to make a one for this project. For brevity, this commit does not touch Zig source code. This merely creates a guideline. Shota FUJI authored at Shota FUJI comitted at
  58. 96637ddb gtk-adwaita: Migrate to the new Core API This commit includes changes to Core: debug logging and fixing array of structure to array of pointer for structure. Although client code got simpler, I'm not sure accessing GTK from callback function is okay. For now, every callbacks are invoked from spawned thread and this means accessing GTK from non-UI thread. Additional async-ing in Vala side might be necessary? Shota FUJI authored at Shota FUJI comitted at
  59. 43a74204 core: Migrate to callback design Forcing callers to take care of threads turned out a bad idea. That requires core to be extremely simple, durable, and thread-safe. By making majority of core's functionality threaded, clients do not have to take a look inside to see whether a field is thread-safe or not. Shota FUJI authored at Shota FUJI comitted at
  60. e0277f46 core: Fix memory leaks Shota FUJI authored at Shota FUJI comitted at
  61. b2b447f1 cli: Fix memory leak Shota FUJI authored at Shota FUJI comitted at
  62. 00258b0e core: Remove obsolete TODO comment Shota FUJI authored at Shota FUJI comitted at
  63. 5c7effd2 core: Release unhandled response bytes on deinit Shota FUJI authored at Shota FUJI comitted at
  64. f3fa46a4 core: Return result code and token from connect method Shota FUJI authored at Shota FUJI comitted at
  65. ab47cb29 cli: Escape server name and version in TSV output Shota FUJI authored at Shota FUJI comitted at
  66. 49d86a77 cli: Create exit code for stdout write error Shota FUJI authored at Shota FUJI comitted at
  67. 6ac67e0a cli: Create exit code for OOM Shota FUJI authored at Shota FUJI comitted at
  68. ebb7ce2e gtk-adwaita: Fix runtime library error on macOS libadwaita depends on gtk4 but devbox cannot resolve to same package as the root one, so there are duplicated "gtk4" package therefore symbol duplication error happens. I have no idea why this is not a problem on Linux, though. Shota FUJI authored at Shota FUJI comitted at
  69. ed3b9ab8 gtk-adwaita: Init core app I have no idea what to do next. Flux-like state-action API? Simple struct with state properties and callback functions like GObject? Completely lost. Shota FUJI authored at Shota FUJI comitted at
  70. c7269fdd core: Setup debug logs Shota FUJI authored at Shota FUJI comitted at
  71. 31fe7e9b core: Use proper logging Shota FUJI authored at Shota FUJI comitted at
  72. 8d7910f5 cli: Use proper logging Shota FUJI authored at Shota FUJI comitted at
  73. 248e9669 core/cli: Get list of available zones Shota FUJI authored at Shota FUJI comitted at
  74. 4f914150 core/cli: Register extension to Roon Code is messy. Logs should be proper logs not debug one. Few places leaks memory. WebSocket read timeout should be configurable. CLI should also store IP address and TCP port. Error handling is nonexistent. However, it works (TM). And if I optimize or tidy I/F up at this point, there is high chance of the resulted code would be difficult to use. Tidying I/F by seeing only one implementation (in this case, CLI) brings other client hard time integrating. This project's main focus is native GUI clients, not CLI. Thus, I should refactor/optimize at the time of writing GTK or SwiftUI code. Let it evolve from worse state. Shota FUJI authored at Shota FUJI comitted at
  75. 190d510d core: Move server discovery code to "server/" dir I could not put playback related functionality to "lib.h" safely. Shota FUJI authored at Shota FUJI comitted at
  76. f5785163 gtk-adwaita: Loading state for server discovery window Shota FUJI authored at Shota FUJI comitted at
  77. 9bebfb66 gtk-adwaita: Loading screen for server discovery Shota FUJI authored at Shota FUJI comitted at
  78. e97495e7 gtk-adwaita: Generic error dialog class To eliminate the duplication. Shota FUJI authored at Shota FUJI comitted at
  79. 0c6835fc gtk-adwaita: Improve writing and styles of error dialogs Shota FUJI authored at Shota FUJI comitted at
  80. dc5a9a93 gtk-adwaita: Server discovery error handling Loading is still work-in-progress, but error handling is okay. There must be better way to disconnect a signal (error_detail_hid) though. Shota FUJI authored at Shota FUJI comitted at
  81. 96c37943 gtk-adwaita: Use Gnome UI components Shota FUJI authored at Shota FUJI comitted at
  82. 9423b7c4 gtk-adwaita: Rename from "gtk" I'm not going to do "pure GTK", as the primary reason to use GTK is creating native looking GUI. If I freely build styles using GTK components, it would then look alien due to web-by styles. Shota FUJI authored at Shota FUJI comitted at
  83. fdd1d11a core: Find Roon Server by unique ID Except the very initial setup, clients have previously connected Server's unique ID. There is virtually no reason to perform enumerate servers on a network in that case. Shota FUJI authored at Shota FUJI comitted at
  84. 0255a14a core: Fix memory leaks Thanks, valgrind. You're awesome. Shota FUJI authored at Shota FUJI comitted at
  85. 29e4e7f8 cli: Fix memory leak Shota FUJI authored at Shota FUJI comitted at
  86. 762ad37d cli: Fix failing to build Shota FUJI authored at Shota FUJI comitted at
  87. 488e6c8c core/cli: Drop Zig API The primary purpose of cli is to test C API. However, Zig's ideal API design and one of C API is drastically different (esp. allocation), and that makes designing C API difficult. This simplifies the process and hopefully brings better C API with less memory mis-managements. Shota FUJI authored at Shota FUJI comitted at
  88. 38f47e6c core: Fix memory access violation on macos app I forgot C enum is platform int. No idea why this wasn't a problem on Vala, though. Shota FUJI authored at Shota FUJI comitted at
  89. 88a3effa macos: Display Roon Servers on network Shota FUJI authored at Shota FUJI comitted at
  90. 627429da core/macos: Import core from macos https://mitchellh.com/writing/zig-and-swiftui This blog post hugely helped me writing build scripts... I could not do this without that blog post. Shota FUJI authored at Shota FUJI comitted at
  91. e92280ed Remove vala-language-server from tool installation It requires Meson (no way) or Clang's `compile_commands.json` to understand dependencies. While the latter looks simple, I'm not willing to maintain duplicated compile command. If bug or unconvenience or frustration or whatever reaches certain threshold, I'll reconsider re-introducing the LS. Shota FUJI authored at Shota FUJI comitted at
  92. 319cb3a3 macos: macOS app development setup Shota FUJI authored at Shota FUJI comitted at
  93. 7a323e0e gtk: Display scanned Roon Servers No empty state, no error handling, though. Shota FUJI authored at Shota FUJI comitted at
  94. ff30d4c7 core: Fix build fails on macOS Interestingly, Zig smartly assigns `u31` as the type of `usec`. ```zig @compileLog(@typeName(@TypeOf(usec))); ``` yields, ``` @as(*const [3:0]u8, "u31") ``` Shota FUJI authored at Shota FUJI comitted at
  95. 647bb5b2 Don't install Valgrind on macOS It's marked as broken on darwin, despite search.nixos.org lists darwin platforms at "Platforms" section. Shota FUJI authored at Shota FUJI comitted at
  96. 9b785fff core/gtk: C API and use it from Vala Shota FUJI authored at Shota FUJI comitted at
  97. a079c8cd core: Simpler memory management for "Server" Although this increases memory allocation, API is simple and easier to build C API. Shota FUJI authored at Shota FUJI comitted at
  98. eefc813d gtk: Build GTK app using Zig build system It's convenient, consistent, and reliable. Goodbye, system CC's bullshit warnings. Shota FUJI authored at Shota FUJI comitted at
  99. 62d9a8b8 cli: JSONL output for "server ls" ``` ./zig-out/bin/plac server ls -f jsonl | jq ``` Shota FUJI authored at Shota FUJI comitted at
  100. 9fc8db70 core: Set HTTP port to ip_addr field Having separate HTTP port and keeping UDP source port is wasteful and not user friendly. Shota FUJI authored at Shota FUJI comitted at
  101. 9498d876 cli: Improve memory debuggability Now DebugAllocator can detect memory leaks. Shota FUJI authored at Shota FUJI comitted at
  102. 995d035e core: Fix server list not deduped due to double free ``` valgrind --leak-check=full --show-leak-kinds=all --track-origins=yes --num-callers=15 -s ./zig-out/bin/plac server ls -c 10 ``` When a same Roon Server responds more than one time, servers hashmap gets corrupted entry due to having a free-d key. Shota FUJI authored at Shota FUJI comitted at
  103. ce738568 cli: Replace "calloc" build option with "memcheck" option For more enriched debugging. Valgrind is super useful but Zig's DebugAllocator displays more friendly message. It does not catch all errors so Valgrind is still necessary. Shota FUJI authored at Shota FUJI comitted at
  104. eac279a7 cli: Fix main not executing defer statements Shota FUJI authored at Shota FUJI comitted at
  105. 482b779c Add valgrind debugging Shota FUJI authored at Shota FUJI comitted at
  106. eec7b07a core/cli: Roon Server discovery It works (TM) quality though. Shota FUJI authored at Shota FUJI comitted at
  107. b4642ced cli: register "server list" command Not doing anything useful for now, though. Shota FUJI authored at Shota FUJI comitted at
  108. 96b2faf4 cli: Basic CLI setup Shota FUJI authored at Shota FUJI comitted at
  109. 88ae701c cli: Zig project setup Shota FUJI authored at Shota FUJI comitted at
  110. 621870cd gtk: Fix build error on macOS ``` error: Failed to execute child process “pkg-config” (No such file or directory) ``` Shota FUJI authored at Shota FUJI comitted at
  111. 5260c3be gtk: GTK4 application project setup I still have no idea how "libadwaita" alone supplies required dependencies such as pango and glib. Without that, even adding those to "packages" section, ld complains files are missing. Ghostty has "libadwaita" and "gtk4" as dependencies for GTK4 so I simply did the same. Shota FUJI authored at Shota FUJI comitted at
  112. f11a7f77 Manage tools using Devbox rather than mise-en-place/asdf Vala, a programming language often(?) used in GTK world, is missing from mise/asdf registries. I've been feeling itches while using mise/asdf due to lack of packages. For example, REUSE tool, which I use for most of my recent projects, does not exist in their registries. I choose Devbox among the three options: * devenv * Nix Flake (without helper tools) * Devbox ## devenv This is the first tool I tried. Their website looks good, features sounds good, and sample Nix file looks okay. Their getting started guide starts with "devenv init", which creates not only their config file but touches other toolings' files (`.gitignore` and `.envrc`). "init" command without manual setup step, especially touching others' files is big sign of shitty tooling design, but I continued hoping other parts would be "okay". Unfortunately they weren't. CLI "search" command emits hundres of warnings, and outputs table does not check terminal width so it badly wraps and border characters gets in a way that the output is nearly unreadable. When I ran its shell command, it paused the execution with a warning message about binary cache. So devenv by default uses creator's own paid SaaS? I don't think just using devenv requires paid subscription, but unclear writing and CLI tries to push Cachix service into users' Nix config is no-no for me. The tool seems to be a thin wrapper around Nix: it inherits some of upstream's bad designs (e.g. absolutely worst error trace, "nix develop" invokes "bash" regardless of the current shell). Their website advertise "Version support." but very few selected packages have that. Everything else relies on Nix's "use latest or die" versioning schema. Considering devenv's "Language support" is just import helpers for nixpkgs, this "Version support." feels like a false advertising. Half-baked YAML/Nix architecture prevents users from using overlays or other flakes with "follows" input to reduce duplications. And using Flake requires running Flake command ("nix develop --no-pure-eval") instead of their CLI, which completely dismissed the value of their tool for me. Also reading their docs are uncomfortable given the tool is not good quality one: > Many experienced Nix users prefer to use Nix flakes, although devenv > is considered a superior interface since it's way simpler, but lacks > integration with existing tooling. Superior, huh? There are still features not present in bare Nix/Flake, but I find those scripts/tasks/tests not useful. If every developer has access to same toolchains, why not simply write script? (JS or Python or Go or whatever) Overall, I see no benefits over bare Nix/Flake or mise/asdf. ## Nix Flake Nix the language is ugly and readability is mediocre (I maintain my machines using Home Manager and still hard time *parsing* program.) It's not good at handling toolchain versions and I don't believe people not using Nix regularly can understand/write even *okay* quality code. Combined with its worst of the worst debuggability, I mean error trace, this is not something I'd like to maintain alongside application development. But most importantly, they still runs "bash" on "nix develop" even a user uses zsh or fish. Hard pass. ## Devbox Actually this was not in my radar initially, but found during figuring best way to write Flake for monorepo. It's not a kind of software I like: docs are next to a cloud offering, docs site hosted on Vercel, corporate site calling nonsense-text- generators-having-high-chance-of-makes-sense-response "AI" without defining intelligence or artifical one, allowing JSONC for ".json" file extension (fuck you M$ for starting this fucking tradition) and not supporting ".jsonc" file extension. But the tool itself is good actually. * Config file is JSON rather than YAML. * Config schema is concise and straightforward. * CLI works great. No wrapping tables. * CLI is fast. * Structure of their docs website is *great*. I should steal from that. * Supports versions, really. It's hard to describe... UX for Devbox is similar to or better than mise-en-place. Simplicity? No bullshit API design? I don't know. I find this tool to be simpler to use and easier to understand compared to other tools, even including non-Nix ones such as asdf, nvm, and mise. I may encounter problems rooted to Devbox in a future. I may get frustrated due to mistyping "debvoc" or something. But for now, I believe Devbox is the best suited for this project (and my other projects too.) Shota FUJI authored at Shota FUJI comitted at
  113. e7e581ad core: Zig project setup Shota FUJI authored at Shota FUJI comitted at
  114. e2a91fa9 README.md and Markdown authoring setup Shota FUJI authored at Shota FUJI comitted at
  115. 4b704338 Create a license file I don't care someone using this for profit or whatever. This project started because the status quo is terrible. Shota FUJI authored at Shota FUJI comitted at