Skip to content

fix: preserve Skill marketplace navigation - #238

Closed
kapelame wants to merge 2 commits into
feat/skill-directory-catalogfrom
fix/skill-marketplace-navigation
Closed

fix: preserve Skill marketplace navigation#238
kapelame wants to merge 2 commits into
feat/skill-directory-catalogfrom
fix/skill-marketplace-navigation

Conversation

@kapelame

@kapelame kapelame commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • preserve the selected Skill/MCP type in the URL so switching back to MCP remains responsive
  • preserve the Workspace/Marketplace source when opening and returning from capability details
  • hide only legacy same-workspace Skill duplicates that match a built-in catalog item
  • add regression coverage for directory navigation and duplicate handling

Scope

This is a small frontend and E2E follow-up stacked on #236. It does not change backend APIs, database schema, migrations, or catalog data. After #236 merges, this PR can be retargeted to main.

Testing

  • make check
  • web typecheck and build
  • pnpm exec playwright test tests/e2e/mcp-directory.spec.ts (3 passed)

@kapelame

kapelame commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator Author

Closing this PR while the Skill marketplace navigation and legacy same-name handling are corrected and re-tested.

@kapelame kapelame closed this Aug 4, 2026
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.

1 participant