Codex/unify locale resolution#316
Open
n1n9-jp wants to merge 207 commits into
Open
Conversation
- Create api/github-proxy.ts for CORS-enabled GitHub proxy - Configure vercel.json for Vercel deployment - Supports REACT_APP_GITHUB_PROXY environment variable 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Update outputDirectory to packages/gephi-lite/build to match monorepo layout 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Env variables will be set directly in Vercel dashboard instead of using Secrets references 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Use npx to properly resolve preconstruct command during npm install in Vercel environment 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Use workspace-specific npm commands to avoid slow monorepo build 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Use npm ci with --ignore-scripts to avoid preconstruct delay during install phase 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Include npm install and preconstruct build in buildCommand to ensure proper dependency resolution 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
…logy version conflict
…ure consistent builds
… not Preconstruct
…es, not gephi-lite
…etween local and Vercel builds
provider フィールドが undefined の場合、JSON.parse が '"undefined" is not valid JSON' というエラーを起こしていました。 修正内容: - user/index.tsx: provider が存在しない場合、localStorage に含めない - AuthInit.tsx: provider が存在しない場合のチェックを追加 これにより、provider が undefined でも安全にデシリアライズできるようになります。 Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
「Loaded Project を読み込みました」という不自然なメッセージを 「プロジェクトを読み込みました」に修正しました。 背景: - dataviz-tool-header API の onProjectLoad では projectData のみが渡される - メタデータ（プロジェクト名など）が available でないため、 ダミー値の「Loaded Project」を表示していた 修正内容: - Initialize.tsx: 読込メッセージを簡潔に変更 - ToolHeaderConfig.tsx: * onProjectLoad メッセージを簡潔に * onProjectLoad エラーメッセージも日本語に統一 Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
…-tool-header-hooks [codex] Refactor tool header helpers and attribute hooks
Collaborator
|
Hi, First of all, thank you for your PR and for your interest in this project. Unfortunately, your PR doesn't include a description. Could you explain its goal and what it is intended to achieve? This PR is also extremely large (+65,169 / -8,937 lines across 210 commits), which makes it impossible for the team to review effectively. Could you please split it into multiple smaller PRs, each focused on a single feature or change? It would also help if each commit addressed a single logical change. A few additional comments:
Cheers. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull request type
Check the type of change your PR introduces:
What is the current behavior?
Issue Number: N/A
What is the new behavior?
Other information