Skip to content

chore(skill): sync workleap-squide skill with docs - #702

Open
claude[bot] wants to merge 1 commit into
mainfrom
agent/skill-sync-20260827-174938-a66a04a
Open

chore(skill): sync workleap-squide skill with docs#702
claude[bot] wants to merge 1 commit into
mainfrom
agent/skill-sync-20260827-174938-a66a04a

Conversation

@claude

@claude claude Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Summary

Reviewed every in-scope page under docs/ against agent-skills/workleap-squide/. The skill was already in sync on nearly everything; three items needed correcting.

Fixed a wrong package in an import (references/integrations.md)

The "Apply a Backend Preferred Language Setting" example imported useChangeLanguage from @squide/firefly. It is only exported from @squide/i18next (docs/reference/i18next/useChangeLanguage.md, packages/i18next/src/index.ts), and @squide/firefly does not re-export @squide/i18next, so the snippet would not have compiled. The skill's two other useChangeLanguage snippets already imported it correctly. Note the same bad import exists upstream in docs/integrations/setup-i18next.md.

Documented the navigation item key fallback (references/hooks-api.md)

docs/reference/routing/useRenderedNavigationItems.md documents that when an item declares no $id, the key handed to renderItem defaults to a value computed from index and level, which cannot keep the menu from flickering across deferred registration updates. The skill described the key parameter's type but never said what it holds.

Documented $visibility on a nested route (references/runtime-api.md)

docs/reference/runtime/FireflyRuntime.md and docs/essentials/register-routes.md both cover declaring a nested route's visibility through the $visibility option, alongside registerPublicRoute for a root public route. The skill listed $visibility as a route property but had no example.

Bumped metadata.version to 1.27. Validation passed 21/21 on coverage with no remaining skill-vs-docs discrepancies.

Not changed — upstream docs drift

references/patterns.md documents ReadonlySessionLocalStorage from @squide/fakes, faithfully mirroring docs/reference/fakes/ReadonlySessionLocalStorage.md. That symbol no longer exists: packages/fakes/src/index.ts now exports LocalStorageSessionAccessor (same constructor option and getSession()). Left the skill alone since the docs are its source of truth — the docs should be updated first, and the next sync will carry the change into the skill.

🤖 Generated with Claude Code

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.

0 participants