Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
8 changes: 4 additions & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
- uses: actions/checkout@v6

- name: Setup Node.js
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: '20'
cache: 'npm'
Expand Down Expand Up @@ -54,7 +54,7 @@ jobs:
- uses: actions/checkout@v6

- name: Setup Node.js
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: '20'
cache: 'npm'
Expand Down Expand Up @@ -91,7 +91,7 @@ jobs:
- uses: actions/checkout@v6

- name: Setup Node.js
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: '20'
cache: 'npm'
Expand Down Expand Up @@ -121,7 +121,7 @@ jobs:
- uses: actions/checkout@v6

- name: Setup Node.js
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: '20'
cache: 'npm'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jobs:
token: ${{ secrets.GITHUB_TOKEN }}

- name: Use Node.js 20.x
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: 20.x

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/validate-mocks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
- uses: actions/checkout@v6

- name: Setup Node.js
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: '20'
cache: 'npm'
Expand Down
18 changes: 9 additions & 9 deletions client/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,25 +11,25 @@
"@fsf/shared": "*",
"d3": "^7.9.0",
"leaflet": "^1.9.4",
"lucide-react": "^0.575.0",
"portos-ai-toolkit": "^0.1.0",
"lucide-react": "^0.577.0",
"portos-ai-toolkit": "^0.8.0",
Copy link

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

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

portos-ai-toolkit is upgraded here alongside React 19, but the toolkit’s published peer range in the lockfile is still React 18. Given the client imports portos-ai-toolkit/client, this is likely to result in multiple React versions in the dependency tree and runtime hook errors. Please ensure the toolkit version used has React 19-compatible peer ranges before merging.

Suggested change
"portos-ai-toolkit": "^0.8.0",
"portos-ai-toolkit": "^1.1.0",

Copilot uses AI. Check for mistakes.
"react": "^18.3.1",
"react-dom": "^18.3.1",
"react-hot-toast": "^2.6.0",
"react-leaflet": "^4.2.1",
"react-router-dom": "^7.1.1",
"react-router-dom": "^7.13.1",
Comment on lines 18 to +20
Copy link

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

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

Upgrading to react-router-dom@7.13.1 (and Tailwind 4’s tooling) brings an effective minimum Node.js version of 20 (per package engines). Since the repo currently doesn’t declare an engines.node constraint, developers on Node 18 may see install/build failures. Consider adding an explicit Node >=20 engine requirement (and aligning docs) as part of this upgrade.

Copilot uses AI. Check for mistakes.
"socket.io-client": "^4.8.3",
"zustand": "^5.0.2"
"zustand": "^5.0.11"
},
"devDependencies": {
"@testing-library/react": "^16.3.2",
"@tailwindcss/vite": "^4.2.1",
"@types/d3": "^7.4.3",
"@types/leaflet": "^1.9.21",
"@types/react": "^18.3.18",
"@types/react-dom": "^18.3.5",
"@types/react": "^18.3.28",
"@types/react-dom": "^18.3.7",
"@vitejs/plugin-react": "^5.1.4",
"autoprefixer": "^10.4.20",
"postcss": "^8.4.49",
"tailwindcss": "^3.4.17",
"tailwindcss": "^4.2.1",
"typescript": "^5.7.2",
"vite": "^7.3.1"
}
Expand Down
6 changes: 0 additions & 6 deletions client/postcss.config.js

This file was deleted.

10 changes: 5 additions & 5 deletions client/src/index.css
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
@tailwind base;
@tailwind components;
@tailwind utilities;
@import "tailwindcss";
@config "../tailwind.config.js";

/* Scan portos-ai-toolkit for Tailwind classes */
@source "../../node_modules/portos-ai-toolkit/src/client/**/*.{js,ts,jsx,tsx}";

/* Light theme (default) */
:root {
Expand Down Expand Up @@ -171,7 +173,6 @@ input:focus, select:focus, textarea:focus {
}

/* Custom theme utilities - Override Tailwind's hardcoded values with CSS variables */
/* These MUST come after @tailwind utilities to override properly */
.text-app-text {
color: var(--color-app-text) !important;
}
Expand Down Expand Up @@ -223,7 +224,6 @@ input:focus, select:focus, textarea:focus {
}

/* Override hardcoded text-white in portos-ai-toolkit for light mode */
/* The library uses text-white for headings/text which is invisible on light backgrounds */
:root:not(.dark) h1.text-white,
:root:not(.dark) h2.text-white,
:root:not(.dark) h3.text-white,
Expand Down
15 changes: 0 additions & 15 deletions client/tailwind.config.js
Original file line number Diff line number Diff line change
@@ -1,21 +1,6 @@
/** @type {import('tailwindcss').Config} */
export default {
content: [
"./index.html",
"./src/**/*.{js,ts,jsx,tsx}",
"./node_modules/portos-ai-toolkit/src/client/**/*.{js,ts,jsx,tsx}",
],
darkMode: 'class',
safelist: [
'text-app-text',
'text-app-text-secondary',
'text-app-text-muted',
'text-app-text-subtle',
'bg-app-text',
'bg-app-text-muted',
'border-app-text',
'border-app-text-muted',
],
theme: {
extend: {
screens: {
Expand Down
3 changes: 2 additions & 1 deletion client/vite.config.ts
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
import { defineConfig } from 'vite';
import react from '@vitejs/plugin-react';
import tailwindcss from '@tailwindcss/vite';

export default defineConfig({
plugins: [react()],
plugins: [react(), tailwindcss()],
server: {
port: 6373,
proxy: {
Expand Down
Loading
Loading