feat: fj-dash TUI dashboard implementation #1

Merged

bryan merged 16 commits from get-started into main 2026-03-08 03:35:45 +00:00

Conversation 0 Commits 16 Files changed 10 +980 -201

bryan commented 2026-03-08 02:24:51 +00:00

Owner

Copy link

Summary

Transforms the skeleton ratatui app into a working Forgejo PR dashboard.

What it does

• Multi-instance support — configure multiple Forgejo instances via ~/.config/fj-dash/config.toml, each with its own API token and optional repo filter
• Tea CLI fallback — auto-discovers credentials from tea's config (XDG, macOS, legacy paths) when no fj-dash config exists
• Async data fetching — loads PRs concurrently across instances with loading/error states per-instance
• TUI navigation — j/k to navigate, Enter to open PR in browser, Tab to switch instances, r to refresh, q to quit
• PR table — shows number, repo, title, author, +/- diff stats, and last updated date
• rustls-tls — replaces native-tls/OpenSSL with rustls for simpler cross-platform builds

New modules

File	Purpose
api.rs	Forgejo API client — repo listing, PR fetching, response mapping
app.rs	Application state — instances, selection, navigation
config.rs	TOML config parsing with validation
event.rs	Async event loop — keyboard, tick, resize, data events
tea_config.rs	Tea CLI YAML config reader (fallback credentials)
ui.rs	Ratatui rendering — tabs, PR table, loading/error states

Config format

[[instance]]
name = "my-forgejo"
url = "https://forgejo.example.com"
token = "your-api-token"
repos = ["owner/repo1", "owner/repo2"]  # optional

## Summary Transforms the skeleton ratatui app into a working Forgejo PR dashboard. ### What it does - **Multi-instance support** — configure multiple Forgejo instances via `~/.config/fj-dash/config.toml`, each with its own API token and optional repo filter - **Tea CLI fallback** — auto-discovers credentials from tea's config (XDG, macOS, legacy paths) when no fj-dash config exists - **Async data fetching** — loads PRs concurrently across instances with loading/error states per-instance - **TUI navigation** — `j`/`k` to navigate, `Enter` to open PR in browser, `Tab` to switch instances, `r` to refresh, `q` to quit - **PR table** — shows number, repo, title, author, +/- diff stats, and last updated date - **rustls-tls** — replaces native-tls/OpenSSL with rustls for simpler cross-platform builds ### New modules | File | Purpose | |------|--------| | `api.rs` | Forgejo API client — repo listing, PR fetching, response mapping | | `app.rs` | Application state — instances, selection, navigation | | `config.rs` | TOML config parsing with validation | | `event.rs` | Async event loop — keyboard, tick, resize, data events | | `tea_config.rs` | Tea CLI YAML config reader (fallback credentials) | | `ui.rs` | Ratatui rendering — tabs, PR table, loading/error states | ### Config format ```toml [[instance]] name = "my-forgejo" url = "https://forgejo.example.com" token = "your-api-token" repos = ["owner/repo1", "owner/repo2"] # optional ```

bryan added 12 commits 2026-03-08 02:24:51 +00:00

chore: add flake.lock to gitignore 39bdc98288

feat: add TOML config parsing with unit tests 824b4d90c2

feat: add PR table rendering with fake data fe2eea3a3d

refactor: replace sync loop with async tokio event loop 6e059b24f0

feat: add Forgejo API client with unit tests fd627df952

feat: integrate API client with TUI, show real PR data ffca74d9b0

feat: add keyboard navigation and browser opening 1ed3cb6658

feat: add multi-instance tab switching c8565f48e4

feat: add optional tea CLI config reading 27fc85c062

feat: add error handling and edge case polish 1ba049bb63

fix: support macOS tea config path and wire tea fallback 46b9a6f042

chore: suppress dead_code warnings for future-use fields 1c082fc59f

bryan added 3 commits 2026-03-08 02:37:40 +00:00

fix: use rustls-tls instead of native-tls for forgejo-api ... f3864baf91

native-tls (macOS Secure Transport) fails with TLS error -9836
against the Forgejo instance. Switch to rustls via feature flag.

fix: prevent FetchError from being overwritten by PrsLoaded ... 66993439ad

Collect per-repo errors during fetch loop instead of sending
individual FetchError events. Only send FetchError when all repos
fail and no PRs were loaded.

feat: auto-discover repos when repos list is omitted from config ... 80d656630a

If an instance has no repos configured, uses the Forgejo API
(user_current_list_repos) to fetch all repos for the authenticated
user. Makes repos field optional in config with serde default.
Also allows tea CLI fallback to work without manual config.

bryan added 1 commit 2026-03-08 02:56:19 +00:00

refactor: address PR review findings ... 802d38ae8b

- Move PullRequest struct from ui.rs to app.rs to fix inverted dependency
- Replace process::exit(1) with anyhow::bail in Config::load()
- Use spawn_blocking for crossterm event polling instead of tokio::spawn
- Surface partial fetch failures via warnings field on InstanceData
- Return Vec<String> from Config::validate() instead of printing to stderr
- Remove duplicate table title block in draw_pr_table
- Filter out mirror and archived repos during auto-discovery
- Add PR number column to table display
- Remove dead code: to_instances(), unused fields, unnecessary allow(dead_code)

bryan merged commit 7565564267 into main 2026-03-08 03:35:45 +00:00

bryan referenced this pull request from a commit 2026-03-08 03:35:45 +00:00

feat: fj-dash TUI dashboard implementation (#1)

Commenting is not possible because the repository is archived.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

bryan/fj-dash-og!1

No description provided.