Skip to content

refactor(wallet): remove notify API + NEXT_PUBLIC_USER_API_KEY - #13

Closed
phucbm wants to merge 10 commits into
mainfrom
preview
Closed

refactor(wallet): remove notify API + NEXT_PUBLIC_USER_API_KEY#13
phucbm wants to merge 10 commits into
mainfrom
preview

Conversation

@phucbm

@phucbm phucbm commented Jun 5, 2026

Copy link
Copy Markdown
Member

Summary

  • Drops deprecated user.openwallet.vn integration (notify service no longer maintained)
  • Removes NEXT_PUBLIC_USER_API_KEY / NEXT_PUBLIC_USER_API_URL — these were client-exposed security risk
  • Reminder toggles now local-only (IndexedDB), no remote sync
  • Removes test adapter button from notifications settings

Files deleted

  • lib/notify-api.ts — all remote reminder CRUD calls
  • lib/reminder-message.ts — message builder (only used by notify-api)

Test plan

  • Wallet app frozen behind WALLET_ENABLED gate — no regression risk on public site
  • TypeScript passes with no errors (pnpm tsc --noEmit)
  • Lint passes (pnpm lint)

🤖 Generated with Claude Code

phucbm and others added 10 commits June 4, 2026 14:56
GitHub-backed eval UI replaced by Langfuse for result storage/visualization.
- Groq → OpenRouter for judge (gpt-4o-mini default)
- Add custom checks for all A1-C4 + guard cases
- Add push:prompt script for Langfuse prompt sync
- Switch eval script to --env-file loader
- Add CT persona, language auto-switch
- Structured tool-call rules by intent category
- Grounding + hallucination guardrails
…DEL/EVAL_JUDGE_MODEL

Disambiguate prod vs eval env vars so they can be set independently.
AI_MODEL had dual use in prod + eval scripts — now CHAT_MODEL is prod-only,
CHAT_EVAL_MODEL/EVAL_JUDGE_MODEL are eval-only.
- Replace langfuse v3 with @langfuse/tracing + @langfuse/otel + @langfuse/client v5.4.1
- Push traces via OTel-based SDK (enables observation-level evaluators in Langfuse)
- Remove judgeResponse() — Langfuse 'card-advice-quality' evaluator handles judging
- Remove GROQ/judge model deps, clean up EvalResult type
- Add push:prompt script (lib/chat/system-prompt.ts → Langfuse)
- Migrate from Groq to OpenRouter for chat model
- Update evals.md with full architecture, gotchas, SDK version notes

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- generate sticky userId (adjective-animal-hex) in localStorage
- generate per-tab sessionId (UUID)
- pass both in request body → Langfuse userId/sessionId fields
- fix onFinish not firing on client disconnect via consumeStream()
- show userId chip in chat panel header
@vercel

vercel Bot commented Jun 5, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
openwallet Ready Ready Preview, Comment Jun 5, 2026 4:10am

@phucbm

phucbm commented Jun 5, 2026

Copy link
Copy Markdown
Member Author

Reopening with correct base branch (preview instead of main)

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