Skip to content

Add the PTY service behind the embedded terminal - #23

Merged
cjimti merged 2 commits into
mainfrom
feat/2-pty-service
Aug 3, 2026
Merged

Add the PTY service behind the embedded terminal#23
cjimti merged 2 commits into
mainfrom
feat/2-pty-service

Resolve the shell against PATH before starting a session

2062a9d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 3, 2026 in 0s

No report found to compare against

View this Pull Request on Codecov

No report found to compare against

Details

Codecov Report

❌ Patch coverage is 94.54545% with 12 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@d4d8944). Learn more about missing BASE report.

Files with missing lines Patch % Lines
internal/pty/session.go 91.42% 6 Missing and 3 partials ⚠️
internal/pty/hangup_unix.go 75.00% 2 Missing and 1 partial ⚠️
Additional details and impacted files 
@@           Coverage Diff           @@
##             main      #23   +/-   ##
=======================================
  Coverage        ?   94.04%           
=======================================
  Files           ?        8           
  Lines           ?      252           
  Branches        ?        0           
=======================================
  Hits            ?      237           
  Misses          ?       11           
  Partials        ?        4

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
View more details on Codecov