Skip to content

Add thv skill upgrade: re-resolve pinned skills to newer content#5896

Merged
samuv merged 4 commits into
mainfrom
skills-lock/05-upgrade
Jul 24, 2026
Merged

Add thv skill upgrade: re-resolve pinned skills to newer content#5896
samuv merged 4 commits into
mainfrom
skills-lock/05-upgrade

Fix forwarding notification context cancellation race

01bebac
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jul 24, 2026 in 0s

71.86% (-0.01%) compared to 52bc063

View this Pull Request on Codecov

71.86% (-0.01%) compared to 52bc063

Details

Codecov Report

❌ Patch coverage is 64.60674% with 63 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.86%. Comparing base (52bc063) to head (01bebac).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
pkg/skills/skillsvc/upgrade.go 63.76% 37 Missing and 13 partials ⚠️
pkg/skills/client/client.go 0.00% 12 Missing ⚠️
pkg/skills/skillsvc/pin.go 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5896      +/-   ##
==========================================
- Coverage   71.86%   71.86%   -0.01%     
==========================================
  Files         713      714       +1     
  Lines       73228    73401     +173     
==========================================
+ Hits        52624    52746     +122     
- Misses      16834    16876      +42     
- Partials     3770     3779       +9     

☔ 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.