• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

dcdpr / jp / 24159695622
67%

Build:
DEFAULT BRANCH: main
Ran 08 Apr 2026 09:33PM UTC
Jobs 1
Files 257
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

08 Apr 2026 09:29PM UTC coverage: 64.396% (+0.006%) from 64.39%
24159695622

push

github

web-flow
fix(workspace): Prevent premature session-mapping deletion (#524)

Introduce a `Liveness` enum (`Alive`, `Dead`, `Unknown`) to replace the
previous boolean return from session-process checks. This allows the
cleanup logic to distinguish three states instead of collapsing `Alive`
and `Unknown` into a single `false`.

The old logic treated `Getsid`/`Hwnd` sessions and `Env` sessions the
same way when process liveness was inconclusive: it would fall through
to a conversation-existence check against the *in-memory* workspace
index. This had two bugs. First, a `Getsid`/`Hwnd` session whose process
is still alive could be deleted if its conversations hadn't yet been
indexed by the current process. Second, the in-memory index is a
snapshot taken at startup, so any conversation created by a concurrent
`jp` process after that point was invisible, making that session look
orphaned.

The new rules are:

- **Getsid/Hwnd** — if the process is alive, the session survives
unconditionally. Only delete when the process is confirmed dead.
- **Env** — liveness is unknowable; fall back to a conversation-
existence check, but now re-scan conversation IDs from disk rather than
relying on the stale in-memory index.

The `is_session_process_dead` helper is renamed to
`is_session_process_liveness`, and `is_pid_dead`/`is_hwnd_dead` become
`pid_liveness`/`hwnd_liveness` to reflect that they now return a
`Liveness` value rather than a boolean.

Signed-off-by: Jean Mertz <git@jeanmertz.com>

22 of 30 new or added lines in 1 file covered. (73.33%)

1 existing line in 1 file now uncovered.

20165 of 31314 relevant lines covered (64.4%)

349.46 hits per line

Uncovered Changes

Lines Coverage ∆ File
8
77.34
0.92% crates/jp_workspace/src/session_mapping.rs

Coverage Regressions

Lines Coverage ∆ File
1
77.34
0.92% crates/jp_workspace/src/session_mapping.rs
Jobs
ID Job ID Ran Files Coverage
1 24159695622.1 08 Apr 2026 09:33PM UTC 257
64.4
GitHub Action Run
Source Files on build 24159695622
  • Tree
  • List 257
  • Changed 2
  • Source Changed 2
  • Coverage Changed 2
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #24159695622
  • 628f64f8 on github
  • Prev Build on main (#24158693542)
  • Next Build on main (#24172309547)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc