Skip to content

fix: scroll resize, 30s refetch, CI semver, note links (#bugfix)#28

Merged
BODMAT merged 5 commits into
developmentfrom
fix/scroll-refetch-ci-notes
Jun 3, 2026
Merged

fix: scroll resize, 30s refetch, CI semver, note links (#bugfix)#28
BODMAT merged 5 commits into
developmentfrom
fix/scroll-refetch-ci-notes

Conversation

@BODMAT

@BODMAT BODMAT commented Jun 3, 2026

Copy link
Copy Markdown
Owner

What

Resolves 4 bugs accumulated since v1.0.1: Lenis scroll did not update after lazy-loaded content, TanStack Query never auto-refetched extension data, CI zip had no version in its name, and URL events in the feed were not clickable links.

Changes

Code

  • dashboard/proxy.ts — renamed from middleware.ts (Next.js 16 convention); exported function renamed to proxy
  • dashboard/components/smooth-scroll.tsxResizeObserver on document.body (not documentElement) to call lenis.resize() automatically when lazy-loaded content changes page height
  • dashboard/app/dashboard/charts.tsx — added refetchInterval: 30_000
  • dashboard/app/dashboard/event-feed.tsx — added refetchInterval: 30_000
  • dashboard/app/dashboard/top-sessions.tsx — added refetchInterval: 30_000
  • dashboard/app/dashboard/music/music-client.tsx — added refetchInterval: 60_000
  • dashboard/app/dashboard/event-card.tsx — HTTP URLs render as clickable <a target="_blank">, note/internal URLs render as plain <span>

Tooling / config

  • .github/workflows/release-extension.yml — semver tag (v1.0.2), versioned zip name (worktrace-extension-v1.0.2.zip), gracefully skips if tag already exists
  • extension/package.json — version bumped 0.1.01.0.2
  • dashboard/.env.example — documented sslmode=verify-full for Neon connections

Docs / plan

  • CLAUDE.md — updated reference middleware.tsproxy.ts

Design decisions

  • ResizeObserver targets document.body instead of document.documentElement — Tailwind's h-full on <html> locks offsetHeight to viewport height; body actually grows with content
  • refetchInterval added only to data queries (events, stats, sessions, music) — report queries are on-demand and intentionally excluded
  • CI skips instead of failing on a duplicate tag — pushing to main without a version bump should not break the pipeline
  • isHttpUrl() check instead of inspecting event.type — more robust, decoupled from any future enum changes

Verification

  • npx tsc --noEmit (dashboard) — clean
  • npm run lint (dashboard) — clean
  • npm run typecheck (extension) — clean
  • Manual: /dashboard/music on a slow connection — scroll reaches the bottom after charts load
  • Manual: event feed updates automatically every 30 s without a page reload
  • Manual: URL events are clickable; note events are plain text

Out of scope (future PRs)

  • Three.js Clock deprecation warning — upstream bug in @react-three/fiber, waiting for a patch

Closes

  • n/a (bugfix batch)

@vercel

vercel Bot commented Jun 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
worktrace Ready Ready Preview, Comment Jun 3, 2026 10:12am

@BODMAT BODMAT merged commit 0216c0b into development Jun 3, 2026
4 checks passed
@BODMAT BODMAT deleted the fix/scroll-refetch-ci-notes branch June 3, 2026 10:13
BODMAT added a commit that referenced this pull request Jun 3, 2026
…semver, link events

fix: bug batch v1.0.2 - proxy rename, scroll resize, 30s refetch, CI semver, link events (#28)
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