Skip to content

fix(tui): keep Skills Manager scan toggle responsive - #4975

Merged
Hmbown merged 1 commit into
mainfrom
codex/v092-linux-skills-scan
Jul 29, 2026
Merged

fix(tui): keep Skills Manager scan toggle responsive#4975
Hmbown merged 1 commit into
mainfrom
codex/v092-linux-skills-scan

Conversation

@Hmbown

@Hmbown Hmbown commented Jul 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • reuse the already-audited owned skill rows when expanding to compatible scan mode
  • scan only newly eligible external roots, then recompute cross-root precedence and conflict state
  • prove the incremental result matches a fresh compatible scan

Release blocker

The exact v0.9.2 Linux CI candidate timed out waiting for scan=compatible because the key handler synchronously rehashed all 34 bundled owned skills on a cold filesystem. This keeps the key response bounded without weakening the PTY acceptance.

Verification

  • cargo fmt --all -- --check
  • focused incremental/fresh equivalence regression
  • isolated Skills Manager PTY acceptance
  • exact workspace clippy gate
  • dead-code budget 482/482
  • provider registry drift check

Closes #4976

Copilot AI review requested due to automatic review settings July 29, 2026 16:42
@cursor

cursor Bot commented Jul 29, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@Hmbown
Hmbown enabled auto-merge July 29, 2026 16:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Expand the current owned inventory by scanning only newly eligible compatible roots. This keeps the interactive toggle responsive on cold Linux filesystems while preserving the same precedence and conflict result as a fresh compatible audit.\n\nAdd an equivalence regression and retain the isolated PTY acceptance.

Signed-off-by: Hmbown <101357273+Hmbown@users.noreply.github.com>
@Hmbown
Hmbown force-pushed the codex/v092-linux-skills-scan branch from 2f16ecc to 520238f Compare July 29, 2026 16:43
@claude

claude Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Claude Code is working…

I'll analyze this and get back to you.

View job run

@claude

claude Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Claude encountered an error after 1s —— View job


I'll analyze this and get back to you.

@Hmbown
Hmbown merged commit ec08e6e into main Jul 29, 2026
33 of 36 checks passed
@Hmbown
Hmbown deleted the codex/v092-linux-skills-scan branch July 29, 2026 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Skills Manager compatible toggle times out on cold Linux filesystems

2 participants