From cb8b0582e9c31d8fcf6ed141d2208c58c0ac06f7 Mon Sep 17 00:00:00 2001 From: syui Date: Fri, 1 Aug 2025 21:25:52 +0900 Subject: [PATCH] rm log --- oauth/src/components/AuthButton.jsx | 3 ++- oauth/src/components/AvatarTest.jsx | 7 ++++--- oauth/src/components/ProfileForm.jsx | 3 ++- oauth/src/components/UserLookup.jsx | 3 ++- oauth/src/hooks/useAuth.js | 5 +++-- oauth/src/services/oauth.js | 13 +++++++------ 6 files changed, 20 insertions(+), 14 deletions(-) diff --git a/oauth/src/components/AuthButton.jsx b/oauth/src/components/AuthButton.jsx index eabbc61..33d0528 100644 --- a/oauth/src/components/AuthButton.jsx +++ b/oauth/src/components/AuthButton.jsx @@ -1,4 +1,5 @@ import React, { useState } from 'react' +import { logger } from '../utils/logger.js' export default function AuthButton({ user, onLogin, onLogout, loading }) { const [handleInput, setHandleInput] = useState('') @@ -12,7 +13,7 @@ export default function AuthButton({ user, onLogin, onLogout, loading }) { try { await onLogin(handleInput.trim()) } catch (error) { - console.error('Login failed:', error) + logger.error('Login failed:', error) alert('ログインに失敗しました: ' + error.message) } finally { setIsLoading(false) diff --git a/oauth/src/components/AvatarTest.jsx b/oauth/src/components/AvatarTest.jsx index a48cebc..bfa9403 100644 --- a/oauth/src/components/AvatarTest.jsx +++ b/oauth/src/components/AvatarTest.jsx @@ -1,6 +1,7 @@ import React, { useState, useEffect } from 'react' import Avatar, { AvatarWithCard, AvatarList } from './Avatar.jsx' import { getAvatar, batchFetchAvatars, prefetchAvatar } from '../utils/avatar.js' +import { logger } from '../utils/logger.js' /** * Test component to demonstrate avatar functionality @@ -63,7 +64,7 @@ export default function AvatarTest() { setTestResults(results) } catch (error) { - console.error('Test failed:', error) + logger.error('Test failed:', error) } finally { setLoading(false) } @@ -78,7 +79,7 @@ export default function AvatarTest() { batchResults: Object.fromEntries(avatarMap) })) } catch (error) { - console.error('Batch test failed:', error) + logger.error('Batch test failed:', error) } finally { setLoading(false) } @@ -94,7 +95,7 @@ export default function AvatarTest() { prefetchResult: cachedAvatar })) } catch (error) { - console.error('Prefetch test failed:', error) + logger.error('Prefetch test failed:', error) } finally { setLoading(false) } diff --git a/oauth/src/components/ProfileForm.jsx b/oauth/src/components/ProfileForm.jsx index d43466f..36e5d3a 100644 --- a/oauth/src/components/ProfileForm.jsx +++ b/oauth/src/components/ProfileForm.jsx @@ -1,6 +1,7 @@ import React, { useState } from 'react' import { atproto, collections } from '../api/atproto.js' import { env } from '../config/env.js' +import { logger } from '../utils/logger.js' const ProfileForm = ({ user, agent, apiConfig, onProfilePosted }) => { const [text, setText] = useState('') @@ -79,7 +80,7 @@ const ProfileForm = ({ user, agent, apiConfig, onProfilePosted }) => { } } catch (err) { - console.error('Failed to create profile:', err) + logger.error('Failed to create profile:', err) setError(err.message || 'プロフィールの作成に失敗しました') } finally { setPosting(false) diff --git a/oauth/src/components/UserLookup.jsx b/oauth/src/components/UserLookup.jsx index c9d4354..6125680 100644 --- a/oauth/src/components/UserLookup.jsx +++ b/oauth/src/components/UserLookup.jsx @@ -1,6 +1,7 @@ import React, { useState } from 'react' import { atproto } from '../api/atproto.js' import { getPdsFromHandle, getApiConfig } from '../utils/pds.js' +import { logger } from '../utils/logger.js' export default function UserLookup() { const [handleInput, setHandleInput] = useState('') @@ -26,7 +27,7 @@ export default function UserLookup() { config: apiConfig }) } catch (error) { - console.error('User lookup failed:', error) + logger.error('User lookup failed:', error) setUserInfo({ error: error.message }) } finally { setLoading(false) diff --git a/oauth/src/hooks/useAuth.js b/oauth/src/hooks/useAuth.js index df51dc1..45749e6 100644 --- a/oauth/src/hooks/useAuth.js +++ b/oauth/src/hooks/useAuth.js @@ -1,5 +1,6 @@ import { useState, useEffect } from 'react' import { OAuthService } from '../services/oauth.js' +import { logger } from '../utils/logger.js' const oauthService = new OAuthService() @@ -21,7 +22,7 @@ export function useAuth() { // If we're on callback page and authentication succeeded, notify parent if (window.location.pathname === '/oauth/callback') { - console.log('OAuth callback completed, notifying parent window') + logger.log('OAuth callback completed, notifying parent window') // Get referrer or use stored return URL const returnUrl = sessionStorage.getItem('oauth_return_url') || @@ -48,7 +49,7 @@ export function useAuth() { } } } catch (error) { - console.error('Auth initialization failed:', error) + logger.error('Auth initialization failed:', error) } finally { setLoading(false) } diff --git a/oauth/src/services/oauth.js b/oauth/src/services/oauth.js index ea5155a..b767197 100644 --- a/oauth/src/services/oauth.js +++ b/oauth/src/services/oauth.js @@ -2,6 +2,7 @@ import { BrowserOAuthClient } from '@atproto/oauth-client-browser' import { Agent } from '@atproto/api' import { env } from '../config/env.js' import { isSyuIsHandle } from '../utils/pds.js' +import { logger } from '../utils/logger.js' export class OAuthService { constructor() { @@ -44,7 +45,7 @@ export class OAuthService { // Try to restore session return await this.restoreSession() } catch (error) { - console.error('OAuth initialization failed:', error) + logger.error('OAuth initialization failed:', error) this.initPromise = null throw error } @@ -89,18 +90,18 @@ export class OAuthService { displayName = profile.data.displayName || null avatar = profile.data.avatar || null - console.log('Profile fetched from session:', { + logger.log('Profile fetched from session:', { did, handle, displayName, avatar: avatar ? 'present' : 'none' }) } catch (error) { - console.log('Failed to get profile from session:', error) + logger.log('Failed to get profile from session:', error) // Keep the basic info we have } } else if (did && did.includes('test-')) { - console.log('Skipping profile fetch for test DID:', did) + logger.log('Skipping profile fetch for test DID:', did) } this.sessionInfo = { @@ -140,7 +141,7 @@ export class OAuthService { } return null } catch (error) { - console.error('Auth check failed:', error) + logger.error('Auth check failed:', error) return null } } @@ -168,7 +169,7 @@ export class OAuthService { // Reload page window.location.reload() } catch (error) { - console.error('Logout failed:', error) + logger.error('Logout failed:', error) } }