Skip to content

Add pnpm #996

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
goldflag wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add pnpm #996

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
98277d6
Add pnpm
goldflag May 14, 2026
c8201da
Update pnpm-lock.yaml to bump Next.js and related dependencies to ver…
goldflag May 16, 2026
78e0958
Add @smithy/node-http-handler and pino dependencies to package.json a…
goldflag May 16, 2026
ba6e3a6
Add d3-scale dependency to package.json and pnpm-lock.yaml
goldflag May 16, 2026
6dd3684
Update pnpm-lock.yaml and pnpm-workspace.yaml to include optional zod…
goldflag May 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions .dockerignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
.git
.github
.deepsec
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot May 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Description: Check if .deepsec directory or file exists

fd -H -t d -t f "^\.deepsec$" .

Repository: rybbit-io/rybbit

Length of output: 42

🏁 Script executed:

# Check if .deepsource exists (the suggested alternative)
fd -H -t d -t f "^\.deepsource$" .

Repository: rybbit-io/rybbit

Length of output: 42

🏁 Script executed:

# Search for any references to "deepsec" or "deepsource" in the codebase
rg -i "deepsec|deepsource" --type-list | head -20

Repository: rybbit-io/rybbit

Length of output: 450

🏁 Script executed:

# Check the git history of .dockerignore to understand context
git log --oneline -n 20 .dockerignore

Repository: rybbit-io/rybbit

Length of output: 78

🏁 Script executed:

# Search for any references to "deepsec" or "deepsource" in the codebase
rg -i "deepsec|deepsource"

Repository: rybbit-io/rybbit

Length of output: 42

🏁 Script executed:

# View the full .dockerignore file for context
cat .dockerignore

Repository: rybbit-io/rybbit

Length of output: 217

🏁 Script executed:

# Check full git history of .dockerignore
git log --all --oneline .dockerignore

Repository: rybbit-io/rybbit

Length of output: 78

Remove or clarify the .deepsec exclusion.

The .deepsec pattern doesn't exist in the repository and has no references in the codebase. This appears to be either a typo for .deepsource (a static analysis tool) or a leftover from a template. If intentional, document the purpose; otherwise, remove it.

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.dockerignore at line 3, The .dockerignore contains a .deepsec pattern that
isn't used; either remove the ".deepsec" entry or clarify it—if you meant the
Deepsource tool, replace it with ".deepsource" and add a comment explaining why
it's excluded, or add a README note in the repository describing the purpose of
".deepsec" to avoid confusion; locate the ".deepsec" line in .dockerignore and
apply the chosen change.

**/node_modules
**/.next
**/dist
**/.turbo
**/.cache
.env
.env.*
npm-debug.log
yarn-debug.log
yarn-error.log
.pnpm-debug.log
.DS_Store
7 changes: 1 addition & 6 deletions .github/dependabot.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,6 @@
version: 2
updates:
- package-ecosystem: "npm"
directory: "/client"
schedule:
interval: "daily"

- package-ecosystem: "npm"
directory: "/server"
directory: "/"
schedule:
interval: "daily"
16 changes: 12 additions & 4 deletions .github/workflows/translate-docs.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,16 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: 20
cache: pnpm

- name: Enable Corepack
run: corepack enable

- name: Install dependencies
run: cd docs && npm ci
run: pnpm install --frozen-lockfile --filter docs-v2...

- name: Extract messages
run: cd docs && node scripts/extract-messages.mjs
run: pnpm --filter docs-v2 extract

- name: Check for untranslated strings
id: changes
Expand Down Expand Up @@ -71,12 +75,16 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: 20
cache: pnpm

- name: Enable Corepack
run: corepack enable

- name: Install dependencies
run: cd docs && npm ci
run: pnpm install --frozen-lockfile --filter docs-v2...

- name: Extract messages
run: cd docs && node scripts/extract-messages.mjs
run: pnpm --filter docs-v2 extract

- name: Run Claude Code
uses: anthropics/claude-code-action@v1
Expand Down
16 changes: 12 additions & 4 deletions .github/workflows/translate.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,16 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: 20
cache: pnpm

- name: Enable Corepack
run: corepack enable

- name: Install dependencies
run: cd client && npm ci
run: pnpm install --frozen-lockfile --filter client...

- name: Extract messages
run: cd client && node scripts/extract-messages.mjs
run: pnpm --filter client extract

- name: Check for untranslated strings
id: changes
Expand Down Expand Up @@ -71,12 +75,16 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: 20
cache: pnpm

- name: Enable Corepack
run: corepack enable

- name: Install dependencies
run: cd client && npm ci
run: pnpm install --frozen-lockfile --filter client...

- name: Extract messages
run: cd client && node scripts/extract-messages.mjs
run: pnpm --filter client extract

- name: Run Claude Code
uses: anthropics/claude-code-action@v1
Expand Down
2 changes: 2 additions & 0 deletions .npmrc
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
auto-install-peers=true
strict-peer-dependencies=false
8 changes: 4 additions & 4 deletions client/CLAUDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@ This file provides guidance to Claude Code when working in the `/client` directo

## Commands

- `npm run dev` – Start dev server (Next.js + Turbopack, port 3002)
- `npm run build` – Production build
- `npm run lint` – ESLint
- `npm run format` – Prettier format
- `pnpm --filter client dev` – Start dev server (Next.js + Turbopack, port 3002)
- `pnpm --filter client build` – Production build
- `pnpm --filter client lint` – ESLint
- `pnpm --filter client format` – Prettier format
- `tsc --noEmit` – Type-check without emitting

## Stack
Expand Down
37 changes: 20 additions & 17 deletions client/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,25 +1,27 @@
FROM node:20-alpine AS base
ENV PNPM_HOME="/pnpm"
ENV PATH="$PNPM_HOME:$PATH"
RUN corepack enable

# Install dependencies only when needed
FROM base AS deps
RUN apk add --no-cache libc6-compat
WORKDIR /app

# Copy shared package first
COPY shared ./shared
WORKDIR /app/shared
RUN npm install && npm run build

# Install client dependencies
WORKDIR /app/client
COPY client/package.json client/package-lock.json* ./
RUN npm ci --legacy-peer-deps
# Install workspace dependencies
COPY package.json pnpm-lock.yaml pnpm-workspace.yaml .npmrc ./
COPY client/package.json ./client/package.json
COPY server/package.json ./server/package.json
COPY docs/package.json ./docs/package.json
COPY monitor-agent/package.json ./monitor-agent/package.json
COPY shared/package.json ./shared/package.json
RUN pnpm install --frozen-lockfile --filter client... --filter @rybbit/shared

# Rebuild the source code only when needed
FROM base AS builder
WORKDIR /app
COPY --from=deps /app/shared ./shared
COPY --from=deps /app/client/node_modules ./client/node_modules
COPY --from=deps /app ./
COPY shared/ ./shared
COPY client/ ./client

# Next.js collects completely anonymous telemetry data about general usage.
Expand All @@ -37,7 +39,8 @@ ENV NEXT_PUBLIC_TURNSTILE_SITE_KEY=${NEXT_PUBLIC_TURNSTILE_SITE_KEY}
ENV NEXT_PUBLIC_STRIPE_PUBLISHABLE_KEY=${NEXT_PUBLIC_STRIPE_PUBLISHABLE_KEY}

WORKDIR /app/client
RUN npm run build
RUN pnpm --filter @rybbit/shared build
RUN pnpm --filter client build

# Production image, copy all the files and run next
FROM base AS runner
Expand All @@ -49,16 +52,16 @@ ENV NEXT_TELEMETRY_DISABLED=1
RUN addgroup --system --gid 1001 nodejs
RUN adduser --system --uid 1001 nextjs

COPY --from=builder /app/client/public ./public
COPY --from=builder /app/client/public ./client/public

# Set the correct permission for prerender cache
RUN mkdir .next
RUN chown nextjs:nodejs .next
RUN mkdir -p client/.next
RUN chown nextjs:nodejs client/.next

# Automatically leverage output traces to reduce image size
# https://nextjs.org/docs/advanced-features/output-file-tracing
COPY --from=builder --chown=nextjs:nodejs /app/client/.next/standalone ./
COPY --from=builder --chown=nextjs:nodejs /app/client/.next/static ./.next/static
COPY --from=builder --chown=nextjs:nodejs /app/client/.next/static ./client/.next/static

USER nextjs

Expand All @@ -67,4 +70,4 @@ EXPOSE 3002
ENV PORT=3002
ENV HOSTNAME="0.0.0.0"

CMD ["node", "server.js"]
CMD ["node", "client/server.js"]
8 changes: 1 addition & 7 deletions client/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,7 @@ This is a [Next.js](https://nextjs.org) project bootstrapped with [`create-next-
First, run the development server:

```bash
npm run dev
# or
yarn dev
# or
pnpm dev
# or
bun dev
pnpm --filter client dev
```

Open [http://localhost:3000](http://localhost:3000) with your browser to see the result.
Expand Down
2 changes: 2 additions & 0 deletions client/next.config.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import type { NextConfig } from "next";
import createNextIntlPlugin from "next-intl/plugin";
import path from "node:path";

const withNextIntl = createNextIntlPlugin({
experimental: {
Expand All @@ -16,6 +17,7 @@ const withNextIntl = createNextIntlPlugin({

const nextConfig: NextConfig = {
output: "standalone",
outputFileTracingRoot: path.join(process.cwd(), ".."),
env: {
NEXT_PUBLIC_BACKEND_URL: process.env.NEXT_PUBLIC_BACKEND_URL,
NEXT_PUBLIC_DISABLE_SIGNUP: process.env.NEXT_PUBLIC_DISABLE_SIGNUP,
Expand Down
Loading
Loading