Skip to content

fix(web): refresh skill and slash inventories - #1295

Merged
tiann merged 1 commit into
tiann:mainfrom
swear01:fix/issue-1294-inventory-refresh
Aug 2, 2026
Merged

fix(web): refresh skill and slash inventories#1295
tiann merged 1 commit into
tiann:mainfrom
swear01:fix/issue-1294-inventory-refresh

Conversation

@swear01

@swear01 swear01 commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • refresh skill and slash-command inventories every 30 seconds while a session is selected
  • refresh immediately when the web app regains focus
  • replace the permanent first-result cache so early fallback/empty inventories and filesystem changes become visible

Tests

  • bun typecheck
  • bun run test:web (204 files, 1760 tests)

Closes #1294
Part of #1284

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Findings

  • None.

Summary
Review mode: initial

  • No correctness, security, regression, data-loss, performance, or maintainability issue found in the added/modified lines.
  • Residual testing gap: no direct hook test covers the 30-second polling or focus-triggered refetch behavior in web/src/hooks/queries/useSkills.ts:45 and web/src/hooks/queries/useSlashCommands.ts:47.

Testing

  • Not run (automation): review security rules prohibit executing code from PR content; the GitHub test check was pending at review time.

HAPI Bot

@tiann
tiann merged commit 4683b0c into tiann:main Aug 2, 2026
2 checks passed
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.

fix(web): refresh cached skill and slash inventories

2 participants