Skip to content

fix(frontend): remove production API debug logs#4258

Merged
WillemJiang merged 4 commits into
bytedance:mainfrom
luojiyin1987:fix/remove-production-api-debug-logs
Jul 22, 2026
Merged

fix(frontend): remove production API debug logs#4258
WillemJiang merged 4 commits into
bytedance:mainfrom
luojiyin1987:fix/remove-production-api-debug-logs

Conversation

@luojiyin1987

Copy link
Copy Markdown
Contributor

Summary

  • remove the unconditional log of NEXT_PUBLIC_LANGGRAPH_BASE_URL
  • remove the API client creation log that prints the resolved endpoint

Why

These logs are executed in the browser client path and appear in production DevTools, where they disclose configured API endpoint details.

Fixes #4257

Validation

  • cd frontend && pnpm check

@luojiyin1987
luojiyin1987 marked this pull request as ready for review July 17, 2026 08:03
@github-actions github-actions Bot added area:frontend Next.js frontend under frontend/ needs-validation Touches front/back contract surface; needs real-path validation risk:medium Medium risk: regular code changes size/XS PR changes < 20 lines labels Jul 17, 2026
@WillemJiang
WillemJiang merged commit 6549fff into bytedance:main Jul 22, 2026
11 checks passed
yogyoho added a commit to yogyoho/eai-flow-main that referenced this pull request Jul 23, 2026
…UI hardening

- bytedance#4117 XSS hardening (isSafeHref, iframe sandbox, safeLocalStorage)
- bytedance#4354 streaming chunk batcher for large file tools
- bytedance#4294 LLM concurrency cap + burst-rate retry shedding
- bytedance#4355 E2B release serialization
- bytedance#4267 transient image context cleanup
- bytedance#4374 AI disclaimer (bytedance#4373 reasoning-effort default)
- bytedance#4337 /goal length validation + counter
- bytedance#4278/bytedance#4302/bytedance#4312 URL encoding fixes
- bytedance#4375 list_uploaded_files schema fix
- bytedance#4288 uploads markdown companion name claim
- bytedance#4258 remove frontend debug logs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:frontend Next.js frontend under frontend/ needs-validation Touches front/back contract surface; needs real-path validation risk:medium Medium risk: regular code changes size/XS PR changes < 20 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(frontend): remove production LangGraph client debug logs

2 participants