Skip to content

Phase 0-1: CLAUDE.md / README.md を現構成に同期 - #69

Merged
champierre merged 1 commit into
mainfrom
feature/phase0-docs-sync
Jun 11, 2026
Merged

Phase 0-1: CLAUDE.md / README.md を現構成に同期#69
champierre merged 1 commit into
mainfrom
feature/phase0-docs-sync

Conversation

@champierre

Copy link
Copy Markdown
Contributor

リファクタリング計画 Phase 0(リポジトリ衛生)の作業項目 0-1 です。ドキュメントのみの変更で、コードの挙動には影響しません。

背景

CLAUDE.md の「アーキテクチャ」「Google OAuth 設定」「UI 状態管理」節が、初期のスタティック SPA 時代の記述のまま残っており、現在の実装と食い違っていました。古い情報が人や AI エージェントを誤誘導するため、以降のリファクタリングの前提として実態に揃えます。

主な修正

箇所 修正前(誤) 修正後(実態)
Client ID index.htmlapp.js にハードコード .env 管理 → GET /api/config で実行時取得
認可済みオリジン localhost:8081 localhost:3000(本番は対象ドメイン)
UI 状態管理 #loginSection / #profileSection 実在する #loginPrompt / #profileInfo / #appContent
ファイル構成 app.js のみ言及 js/modules/*.js(16 ファイル)+ バックエンド構成を明記
README セットアップ YOUR_GOOGLE_CLIENT_ID を HTML/JS に差し替え .envGOOGLE_CLIENT_ID に設定
README DB パス server/shifts.db data/shift.db

#loginPrompt 等の要素 ID と Client ID がハードコードされていないことは実コード(index.html, js/modules/config.js, auth.js, server/src/routes/config.ts)で確認済みです。

フロントエンドテストの慣習節には、Phase 1 で実コード import 方式へ移行する旨の注記を追加しました。

🤖 Generated with Claude Code

旧スタティックSPA時代の記述を現在のExpress+モジュール構成に更新。

- ファイル構成: js/modules/*.js とバックエンド構成を反映
- Google OAuth設定: Client IDはハードコードではなく .env 管理→
  GET /api/config で実行時取得する旨に修正(ポート8081の言及を削除)
- UI状態管理: 実在しない #loginSection/#profileSection を、実際の
  #loginPrompt/#profileInfo/#appContent に修正
- 認証フロー: loadConfig() からの設定取得ステップを追記
- フロントテスト慣習: Phase 1 で実コードimport方式へ移行予定の注記を追加
- README: 存在しない YOUR_GOOGLE_CLIENT_ID 差し替え手順を .env 設定に修正、
  DBパスを server/shifts.db → data/shift.db に訂正

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@champierre
champierre merged commit 6668223 into main Jun 11, 2026
4 checks passed
@champierre
champierre deleted the feature/phase0-docs-sync branch June 11, 2026 15:35
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