Skip to content

fix: content corrections across liveblocks + yjs skills#2

Open
aribn wants to merge 1 commit intoliveblocks:mainfrom
aribn:fix/skill-content-corrections
Open

fix: content corrections across liveblocks + yjs skills#2
aribn wants to merge 1 commit intoliveblocks:mainfrom
aribn:fix/skill-content-corrections

Conversation

@aribn
Copy link
Copy Markdown

@aribn aribn commented Apr 14, 2026

Thanks for publishing these skills! Looking forward to putting them to good use. While reading through, I came across a handful of small issues in the examples and prose worth flagging. All fixes verified against current main. Happy to split this up or drop anything you'd rather handle differently.

A few things that would trip up agents copying examples verbatim

  • liveblocks-best-practices/SKILL.md: the index entry reads primitive-components-parts (plural), but the file on disk is primitive-component-parts.md (singular). An agent following the index would hit a file-not-found.
  • suspense-vs-regular-hooks.md: two import lines use @/liveblocks/react with a stray leading slash. The rest of the file already uses the correct @liveblocks/react.
  • handling-connection-errors.md: the Other errors switch example declares const { roomId, threadId } across ~10 sibling cases without block scoping. Because all case clauses share one lexical scope, repeating those bindings throws SyntaxError: Identifier 'roomId' has already been declared at parse time. ROOM_CONNECTION_ERROR already has the right pattern with { }, so I applied the same shape to the rest.

A couple of markdown links that don't render

  • suspense-vs-regular-hooks.md: two instances of [`useThreads`][https://…] (reference-style link with a URL literal instead of a label — renders as plain text), plus one empty shortcut [`ClientSideSuspense`][] with no matching reference definition. Swapped both for the inline [text](url) form.

One small math thing

  • smoother-realtime-updates.md: "The minimum value is 16, once every 10ms, 60FPS." Since 1000 / 16 ≈ 62.5, the 16 and 60FPS are both right — just the "10ms" in the middle that's inconsistent. Changed to "once every 16ms".

Typos I noticed while reading through

  • liveblocks-best-practices/SKILL.md: extgensions, losing losing, missing : after `prevent-unsaved-changes-being-lost`, multiplayer,commenting, pdates, and one lowercase sentence start
  • ai-as-a-collaborator.md: colalborator
  • yjs-best-practices/SKILL.md: lazy laoded
  • smoother-realtime-updates.md: rooms will lots

Happy to split the correctness fixes from the typo cleanup into separate PRs if that's cleaner.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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