[claude] Add JSDoc types to all function parameters (#144) #168

Timmy · 2026-03-24T04:03:56Z

Timmy commented

2026-03-24 04:03:56 +00:00

Fixes #144

Adds JSDoc type annotations to all function parameters in app.js:

buildConstellationLines() — added @returns {THREE.LineSegments}
animate() — added @returns {void}
mousemove callback — @type {MouseEvent} on e
player-joined, player-left, chat-message callbacks — @type {CustomEvent} on event
querySelectorAll forEach callbacks — @type {HTMLElement} on el
neighbors.sort comparator — @type {{j: number, dist: number}} on a and b

All annotations use inline JSDoc syntax (/** @type {...} */) inside callback parameter lists, which is the standard pattern for unextracted arrow function parameters.

Fixes #144 Adds JSDoc type annotations to all function parameters in `app.js`: - `buildConstellationLines()` — added `@returns {THREE.LineSegments}` - `animate()` — added `@returns {void}` - `mousemove` callback — `@type {MouseEvent}` on `e` - `player-joined`, `player-left`, `chat-message` callbacks — `@type {CustomEvent}` on `event` - `querySelectorAll` forEach callbacks — `@type {HTMLElement}` on `el` - `neighbors.sort` comparator — `@type {{j: number, dist: number}}` on `a` and `b` All annotations use inline JSDoc syntax (`/** @type {...} */`) inside callback parameter lists, which is the standard pattern for unextracted arrow function parameters.

Timmy added 1 commit 2026-03-24 04:03:57 +00:00

chore: add JSDoc types to all function parameters in app.js

CI / validate (pull_request) Failing after 19s

Details

7600d414a2

- Added @returns JSDoc to buildConstellationLines() and animate()
- Added inline @type annotations to all callback parameters:
  - MouseEvent for mousemove handler
  - CustomEvent for player-joined, player-left, chat-message handlers
  - HTMLElement for querySelectorAll forEach callbacks
  - {j: number, dist: number} for neighbors.sort comparator

Fixes #144

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

claude referenced this pull request

2026-03-24 04:04:04 +00:00

Add JSDoc types to all function parameters in app.js #144

Timmy commented

2026-03-24 04:04:07 +00:00

🤖 Merge Bot: CI validation passed (HTML, JS syntax, JSON, size budget). Auto-merging.

🤖 **Merge Bot**: CI validation passed (HTML, JS syntax, JSON, size budget). Auto-merging.

Timmy merged commit 316ce63605 into main

2026-03-24 04:04:09 +00:00

Timmy referenced this issue from a commit

2026-03-24 04:04:09 +00:00

[claude] Add JSDoc types to all function parameters (#144) (#168)