stacklok / toolhive-studio / 25056990066 / 1

70%

main: 70%

push

github

feat(skills): launch Skills as default landing route (#2130)

* feat(skills): always render Skills nav link and drop flag-specific copy

Removes the useFeatureFlag(SKILLS) check in the top nav so the Skills entry is unconditionally rendered (now the first item, matching its new role as the landing route), and drops the dead Skills-specific name/description branches in useExperimentalFeatures — the flag was never listed there (isExperimental: false) and the branches become unreachable once the key is removed in the next commit.

Adds a top-nav test asserting the Skills link renders without any flag setup.

* feat(skills): remove SKILLS feature flag

Skills is now GA. Drops the SKILLS key from the shared featureFlagKeys map and its options block in the main-process flag store. Existing 'feature_flag_skills' rows in SQLite become inert orphans — getFeatureFlag is only called for keys still present in featureFlagKeys, so no migration is needed.

Reroutes the four SKILLS-specific assertions in flags.test.ts onto featureFlagKeys.AGENTS (same shape: non-disabled, default false) so the SQLite read/write/default paths stay covered.

* feat(skills): make Skills the default landing route

Switches the '/' redirect from '/group/default' to '/skills'. The Skills route's validateSearch already defaults tab to 'installed' when the search param is missing, so first launch lands directly on the Installed tab without needing extra search params.

Adds renderer/src/routes/__tests__/index.redirect.test.tsx covering the new redirect using the same beforeLoad-driven router pattern as playground.index.test.tsx.

* test(e2e): navigate to MCP Servers before creating test groups

The shared Playwright fixture and the MCP Optimizer cleanup spec both clicked 'Add a group' as their first UI action, relying on the app landing on /group/default. Now that /skills is the default landing route, that button is no longer on screen and every group-creating spec timed out waiting for it.
... (continued)

4152 of 6828 branches covered (60.81%)

6322 of 9541 relevant lines covered (66.26%)

125.91 hits per line