in the TUI overview, commits are showing weird json for the included files instead of the file information #60

Closed
opened 2026-08-03 20:56:21 +00:00 by hugo · 1 comment
Owner

when I go through activities and am on a commit, in the preview to the right JSON shows up where I would expect file information. the json has the file information inside, so it is generally looking at the right stuff, but it is not formatting it.

when I go through activities and am on a commit, in the preview to the right JSON shows up where I would expect file information. the json has the file information inside, so it is generally looking at the right stuff, but it is not formatting it.
hugo added this to the first feature complete release milestone 2026-08-03 20:56:21 +00:00
hugo added the bug label 2026-08-03 20:56:21 +00:00
Author
Owner

Implemented in 61e96cc.

  • Added a shared compatibility parser for Forgejo/Gitea commit activity payloads, including mixed-case Commits, HeadCommit, and Len fields.
  • Replaced raw JSON in the TUI Home preview with a Markdown-rendered commit count and readable rows containing each short hash, message, author, and timestamp.
  • Preserved comment and non-commit activity previews; opening the activity still routes to the head commit and its changed-file list.
  • Updated TESTING.md and added regression coverage at both the activity parser and TUI presentation boundaries.

Verification:

  • cargo fmt --all -- --check
  • RUSTFLAGS="-D warnings" cargo check --workspace --all-targets
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace (48 passed)
  • Live terminal check against the configured Forgejo server: selected the pushed commit activity, confirmed the formatted count/hash/message/author/timestamp with no JSON, then opened it and confirmed the changed-file list.

This is a TUI issue, so no iPhone release build or deployment was performed, as requested.

Implemented in `61e96cc`. - Added a shared compatibility parser for Forgejo/Gitea commit activity payloads, including mixed-case `Commits`, `HeadCommit`, and `Len` fields. - Replaced raw JSON in the TUI Home preview with a Markdown-rendered commit count and readable rows containing each short hash, message, author, and timestamp. - Preserved comment and non-commit activity previews; opening the activity still routes to the head commit and its changed-file list. - Updated `TESTING.md` and added regression coverage at both the activity parser and TUI presentation boundaries. Verification: - `cargo fmt --all -- --check` - `RUSTFLAGS="-D warnings" cargo check --workspace --all-targets` - `cargo clippy --workspace --all-targets -- -D warnings` - `cargo test --workspace` (48 passed) - Live terminal check against the configured Forgejo server: selected the pushed commit activity, confirmed the formatted count/hash/message/author/timestamp with no JSON, then opened it and confirmed the changed-file list. This is a TUI issue, so no iPhone release build or deployment was performed, as requested.
hugo closed this issue 2026-08-04 17:13:37 +00:00
Sign in to join this conversation.