Skip to content

Commit 2a58c93

Browse files
chore(deps): remove unused jaeger dep + upgrade @supabase/ssr 0.10.2 (#79)
* chore(deps): remove unused @opentelemetry/exporter-jaeger devDep Package was never imported in source code. Removing it eliminates dead weight and closes the stale Dependabot PR #74 cleanly. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * chore(deps): upgrade @supabase/ssr 0.9.0 → 0.10.2 Minor version bump with full lockfile regeneration. Dependabot PR #77 had the same lockfile sync issue as #74 — both failed npm ci due to missing transitive deps in the PR's lockfile. Closes #77 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * ci: upgrade to Node 24 + npm 11; remove hardcoded JWT fallbacks - node-version 22 → 24 (Node 24 LTS, latest stable) - Add 'npm install -g npm@11' in all 3 jobs to match local dev environment (npm 10 vs npm 11 generate incompatible lockfiles — deduplication differs) - Remove hardcoded Supabase demo JWT fallbacks (CWE-321); fail explicitly instead if supabase status does not return keys (supabase CLI ≥ v1.23 always returns keys after supabase start succeeds) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * chore: format CLAUDE.md with prettier CLAUDE.md was previously gitignored; now tracked, prettier check in CI catches formatting issues. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> --------- Co-authored-by: Emiya Kiritsugu <emiyakiritsugu3@users.noreply.github.com> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent d53e7e4 commit 2a58c93

4 files changed

Lines changed: 25 additions & 330 deletions

File tree

.github/workflows/ci.yml

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,12 @@ jobs:
2121
- name: Setup Node.js
2222
uses: actions/setup-node@v4
2323
with:
24-
node-version: '22'
24+
node-version: '24'
2525
cache: 'npm'
2626

27+
- name: Upgrade npm
28+
run: npm install -g npm@11
29+
2730
- name: Install dependencies
2831
run: npm ci
2932

@@ -51,9 +54,12 @@ jobs:
5154
- name: Setup Node.js
5255
uses: actions/setup-node@v4
5356
with:
54-
node-version: '22'
57+
node-version: '24'
5558
cache: 'npm'
5659

60+
- name: Upgrade npm
61+
run: npm install -g npm@11
62+
5763
- name: Install dependencies
5864
run: npm ci
5965

@@ -83,9 +89,12 @@ jobs:
8389
- name: Setup Node.js
8490
uses: actions/setup-node@v4
8591
with:
86-
node-version: '22'
92+
node-version: '24'
8793
cache: 'npm'
8894

95+
- name: Upgrade npm
96+
run: npm install -g npm@11
97+
8998
- name: Install dependencies
9099
run: npm ci
91100

@@ -102,18 +111,13 @@ jobs:
102111

103112
- name: Export Supabase keys
104113
run: |
105-
# Try --output env first (Supabase CLI v1.23+); strip surrounding quotes
106114
STATUS_ENV=$(supabase status --output env 2>/dev/null || echo "")
107115
ANON_KEY=$(echo "$STATUS_ENV" | grep "^ANON_KEY=" | cut -d= -f2- | sed "s/^[\"']//; s/[\"']$//")
108116
SERVICE_KEY=$(echo "$STATUS_ENV" | grep "^SERVICE_ROLE_KEY=" | cut -d= -f2- | sed "s/^[\"']//; s/[\"']$//")
109-
echo "Extracted anon key length: ${#ANON_KEY}"
110-
echo "Extracted service key length: ${#SERVICE_KEY}"
111-
# Fallback: well-known defaults when config.toml has no explicit jwt_secret
112-
if [ -z "$ANON_KEY" ]; then
113-
ANON_KEY="eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZS1kZW1vIiwicm9sZSI6ImFub24iLCJleHAiOjE5ODM4MTI5OTZ9.CRFA0NiK7W9odiD2okw387wrjgFjtl7gx5LwhwlM6ic"
114-
fi
115-
if [ -z "$SERVICE_KEY" ]; then
116-
SERVICE_KEY="eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZS1kZW1vIiwicm9sZSI6InNlcnZpY2Vfcm9sZSIsImV4cCI6MTk4MzgxMjk5Nn0.EGIM96RAZx35lJzdJsyH-qQwv8Hj04zWl196z2-SBc0"
117+
if [ -z "$ANON_KEY" ] || [ -z "$SERVICE_KEY" ]; then
118+
echo "ERROR: Could not extract Supabase keys from 'supabase status --output env'"
119+
supabase status
120+
exit 1
117121
fi
118122
echo "SUPABASE_LOCAL_ANON_KEY=$ANON_KEY" >> $GITHUB_ENV
119123
echo "SUPABASE_LOCAL_SERVICE_ROLE_KEY=$SERVICE_KEY" >> $GITHUB_ENV

CLAUDE.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
Auto-generated from all feature plans. Last updated: 2026-04-19
44

55
## Active Technologies
6+
67
- TypeScript 5 (strict mode) + Playwright 1.59, Next.js 15 App Router, Supabase local CLI (feat/006-it4-instrutor-e2e)
78
- PostgreSQL via Prisma (local Supabase, ports 54321/54322) (feat/006-it4-instrutor-e2e)
89

@@ -40,13 +41,12 @@ All 4 gates must pass before merging.
4041
TypeScript 5 (strict mode, `useUnknownInCatchVariables` enabled): Follow standard conventions
4142

4243
## Recent Changes
44+
4345
- feat/006-it4-instrutor-e2e: Added TypeScript 5 (strict mode) + Playwright 1.59, Next.js 15 App Router, Supabase local CLI
4446

4547
- 004-elite-workflow-setup: Added TypeScript 5 (strict mode, `useUnknownInCatchVariables`) + Node.js 20 + Next.js 15 App Router, Prisma 7, Supabase SSR, Zod 3, Genkit 1.31
4648
- fix/telemetry-and-e2e-stability: Modernized Sentry to v10 + Next.js 15, implemented Privacy-First Replay, linked Supabase User IDs to observability, and repaired ESLint flat config.
4749

48-
49-
5050
<!-- MANUAL ADDITIONS START -->
5151

5252
## AI Session Protocol

0 commit comments

Comments
 (0)