Skip to content

Release v2026.4.1#155

Merged
sasagar merged 16 commits intomainfrom
dev
Apr 10, 2026
Merged

Release v2026.4.1#155
sasagar merged 16 commits intomainfrom
dev

Conversation

@sasagar
Copy link
Copy Markdown
Collaborator

@sasagar sasagar commented Apr 10, 2026

Summary

  • Upgrade waku to 1.0.0-alpha.7 (official Vite 8 support)
  • Remove vite/plugin-react overrides (no longer needed)
  • Fix custom emoji size in emoji picker (cva default size override)
  • Add IntersectionObserver-based infinite scroll for emoji categories
  • Fix NotificationStreamService test mock leakage on Bun 1.3.12
  • Upgrade GitHub Actions (checkout v6, setup-node v6)
  • Enable pino-pretty log formatting in production
  • Suppress noisy SSR error logs for 404 responses
  • Downgrade missing plugins directory log from ERROR to DEBUG
  • Add TSDoc across shared types, backend interfaces, and frontend components

Test plan

  • CI passes on all jobs (Unit Tests, SQLite, Lint, Type Check, Build)
  • Production tested on beta versions

sasagar added 16 commits April 10, 2026 22:02
- Upgrade waku from 1.0.0-alpha.6 to 1.0.0-alpha.7 (Vite 8 official support)
- Remove vite/plugin-react overrides (no longer needed with alpha.7)
- Clean up workaround comment in waku.config.ts
Custom emojis were rendered too small due to cva default size="md"
(h-10 px-4 py-2) overriding the className padding. Pass size={null}
to all emoji buttons and use w-7 h-7 for custom emoji images.
Replace scroll-to-bottom pagination with per-category IntersectionObserver
sentinels. Emojis now load seamlessly as the user scrolls within each
accordion section, instead of requiring the modal to reach the bottom.
Use Symbol.for-based global singleton to prevent dual-instance issues
when the module is loaded via different resolution paths (CJS/ESM).
Also remove .js extension from test import.
The mock.module for NotificationStreamService was leaking to other
test files in Bun 1.3.12. Provide a complete mock with all public
methods and remove .js extension from mock path to match imports.
- Remove mock.module for NotificationStreamService (caused leakage
  across test files in Bun 1.3.12 on Linux)
- Revert unnecessary Symbol.for singleton workaround
- Upgrade actions/checkout v4 -> v6, actions/setup-node v4 -> v6
  to resolve Node.js 20 deprecation warnings
… components

- shared: Add TSDoc to all exported types, interfaces, and validation constants
- backend: Replace Japanese comments with English TSDoc in IFileStorage,
  IUserRepository, di middleware, and OAuth routes
- frontend: Add @returns to SpinnerWithLabel
404 responses during OGP fetch are expected (e.g. deleted users/notes,
crawlers with bad URLs). Remove console.error to avoid polluting
production logs with stack traces for non-error conditions.
JSON logs are hard to read in journalctl. Enable pino-pretty in all
environments (colorize disabled in production to avoid ANSI codes).
Set LOG_PRETTY=false to opt out if JSON is preferred.
The plugins directory is optional. ENOENT when it doesn't exist is
expected behavior, not an error condition.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 10, 2026

Warning

Rate limit exceeded

@sasagar has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 22 minutes and 35 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 22 minutes and 35 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 563d7592-4a67-4e98-b9d6-91c047d26b45

📥 Commits

Reviewing files that changed from the base of the PR and between e02484b and 700c7bd.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (30)
  • .github/workflows/auto-tag.yml
  • .github/workflows/ci.yml
  • .github/workflows/docs.yml
  • .github/workflows/release.yml
  • package.json
  • packages/backend/package.json
  • packages/backend/src/interfaces/IFileStorage.ts
  • packages/backend/src/interfaces/repositories/IUserRepository.ts
  • packages/backend/src/lib/logger.ts
  • packages/backend/src/middleware/di.ts
  • packages/backend/src/plugins/loader.ts
  • packages/backend/src/routes/oauth.ts
  • packages/backend/src/services/NotificationStreamService.ts
  • packages/backend/src/tests/unit/NotificationService.test.ts
  • packages/backend/src/tests/unit/NotificationStreamService.test.ts
  • packages/frontend/package.json
  • packages/frontend/src/components/ui/EmojiPicker.tsx
  • packages/frontend/src/components/ui/Spinner.tsx
  • packages/frontend/src/pages/[username]/index.tsx
  • packages/frontend/src/pages/notes/[noteId].tsx
  • packages/frontend/waku.config.ts
  • packages/shared/package.json
  • packages/shared/src/constants/validation.ts
  • packages/shared/src/types/common.ts
  • packages/shared/src/types/file.ts
  • packages/shared/src/types/note.ts
  • packages/shared/src/types/reaction.ts
  • packages/shared/src/types/session.ts
  • packages/shared/src/types/uiSettings.ts
  • packages/shared/src/types/user.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sasagar sasagar merged commit d51f0da into main Apr 10, 2026
9 checks passed
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