jhheider / penknife / 29220478949

76%

Build Type

push

github

Committed by web-flow

Commit Message

fix: copy-after-fresh-push race + reuse a warm gist client (#41)

* fix: copy-after-fresh-push race + reuse a warm gist client

Pressing `c` on a not-yet-synced file (copy-URL right after a push)
copied the URL correctly but the status line reverted to a stale
"Pushed …" message, and separately every gist op paid a cold TLS
handshake because it built a fresh client per request.

- Track in-flight pushes in `pending_pushes`; `do_copy_url` queues the
  copy onto the running push instead of spawning a second, racing one,
  and `do_sync_up_for` debounces a duplicate non-force push (which
  would otherwise create two gists for one fresh file).
- Skip the intermediate "Pushed" status when a queued copy in the same
  handler is about to supersede it with "Copied".
- Reuse one warm `GistClient` (now `Clone`, sharing a reqwest
  connection pool) across all operations instead of a cold pool per
  request, keeping the API connection alive between pushes/pulls.
- Give penknife-gist and penknife-backend their own crates.io READMEs;
  hoist the app README onto the penknife crate.
- Bump penknife 0.2.1->0.2.2, penknife-gist/penknife-backend 0.2.0->0.2.1.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PykYTubrtTh4te2xg71JJS

* docs: drop em-dashes from the new crate READMEs (style check)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PykYTubrtTh4te2xg71JJS

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

70 of 84 new or added lines in 4 files covered. (83.33%)

334 existing lines in 4 files now uncovered.

7143 of 9378 relevant lines covered (76.17%)

26.97 hits per line