Add sonar-project.properties for SonarQube configuration#4525
Merged
TinyKitten merged 1 commit intodevfrom Oct 28, 2025
Merged
Conversation
Contributor
統一の審視録歩み朕が統べる版図に新たなる勅令が加えられたり。SonarQube設定ファイルを新規に配置し、プロジェクト識別子、ソース設定、テスト設定、および除外対象を定むるものなり。 変更
審査の労力推定🎯 1 (些細なり) | ⏱️ ~3分
詩
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
sonar-project.properties (1)
1-4: プロジェクト識別情報の設定は端正なり朕の見立てるところ、プロジェクトキー、名前、版号の設定は秩序正しく整えられておる。しかれども、第4行目の
sonar.projectVersion=1.0.0は硬く固定されたる状態にて、package.json に記載せられた版号と乖離する可能性あり。朕の領国にては、版号は常に同期されるべき重要な記録なり。此の版号をpackage.jsonより自動的に取得するよう、SonarQube実行スクリプトにて調整することを勧奨する。
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
sonar-project.properties(1 hunks)
🔇 Additional comments (3)
sonar-project.properties (3)
15-16: 除外設定は周到なり朕の統治する領国にても、モックやテストファイルは本来の解析対象より除外するが吉なり。第16行目の除外設定は秩序よく整えられ、
src/__mocks__/**、src/__fixtures__/**、src/test/**、src/utils/test/**、src/setupTests.tsといった必要なる項目が網羅されておる。この設定は朕の要求する水準にかなうものなり。
6-9: 朕の検証完了せり。懸念事項は払拭されたり朕が命ぜし検証スクリプトの実行により、プロジェクト根部に tsconfig.json が確かに存在することが実証されたり。第9行目の
sonar.typescript.tsconfigPath=tsconfig.jsonの設定は有効にして、正当なるものなり。故に、当該設定に関する懸念事項は一切なく、修正を要さず。朕は此の設定の妥当性を認め、其のまま進めて然るべし。
11-13: ソナー設定は既に適切に構成されており、修正不要朕の検証より明らかになりたり。プロジェクト内のテストファイルは
src/components/HeaderE235.test.tsxsrc/hooks/useBounds.test.tsx等のごとく、.test.tsおよび.test.tsxの命名規則に統一されておるなり。
.spec.ts等の他の命名規則は一切用いられておらず、現行の sonar-project.properties 第13行目のパターンsrc/**/*.test.ts,src/**/*.test.tsxは実際のテストファイル分布と完全に一致せり。よって、パターンの補正は必要なく、当該設定は妥当なり。
TinyKitten
added a commit
that referenced
this pull request
Oct 29, 2025
* カナリア版Androidをself-hostedでデプロイする (#4518) * カナリア版Androidをself-hostedでデプロイする * bump android version * ANDROID_HOME明記 * 一旦毎回ビルドが走るようにする * org.gradle.jvmargs=-Xmx8g * Android版ストレージ系の権限削除 * READ_EXTERNAL_STORAGEしばき * expo-doctorで指摘されたパッケージのバージョン変更 * 一旦READ_EXTERNAL_STORAGE消すのを消す * android.permission.READ_EXTERNAL_STORAGEとandroid.permission.READ_MEDIA_IMAGESを明示的にremove * deploy yml更新 * 縦画面対応で不要となったファイルの削除 (#4523) * 縦画面対応で不要となったファイルの削除 * biome fix * 必要なファイルまで消してた * biome fix * pnpm lint --unsafe --fix * 縦画面対応に関わるバグの修正とGraphQL BFF移行 (#4521) * Refactor imports to use types from GraphQL instead of proto files; add GraphQL code generation configuration * Refactor code to use type imports consistently and improve type safety across components and hooks * Remove unused proto subproject reference * Refactor simulation mode tests to improve structure and add comprehensive mock setups * Refactor test mocks for useLocationStore to improve readability and consistency * Refactor mock setup for useCurrentLine in useSimulationMode tests for improved readability * Update type definition for line names in useTTSText to use 'undefined' instead of 'null' * Refactor various components and hooks to improve null handling and type safety - Updated `LineBoardYamanotePad` to handle optional line values. - Modified `SelectBoundModal` to default lineId to 0 instead of null. - Enhanced `useSimulationMode` to check for null values instead of undefined for latitude and longitude. - Adjusted `useTTSText` to ensure nameRoman is set to undefined if not present. - Filtered valid lines in `SelectLineScreen` to ensure line IDs are not null. - Added checks for train types in `SelectLineScreen` to ensure station IDs are valid. - Improved handling of train types in `TrainTypeSettingsScreen` to filter out null values. - Updated `groupStations` utility to ensure prefectureId is within valid range. - Added environment variable check in `codegen.ts` for GQL_API_URL. - Changed maybeValue type in `codegen.ts` to include null for better type safety. * Refactor TrainTypeSettingsScreen: Improve readability of train type mapping * Refactor type definitions and null handling across components and hooks for improved type safety * Refactor background color and border color assignments for improved readability * Add .d.ts to source extensions in metro.config.js and update pnpm-lock.yaml to include rxjs dependency * Refactor GraphQL query variables and condition checks for improved type safety and clarity * Refactor test data initialization in TrainTypeBoxJL and TrainTypeBoxJO: Replace undefined values with null for consistency * Refactor keyExtractor functions in StationList and StationSearchModal for improved uniqueness and clarity * Refactor StationSearchModal to improve cache management and handle query results more effectively * Refactor Transfers component to handle empty line symbols more effectively * Refactor Transfers component to prevent onPress execution for undefined stations * Refactor keyExtractor in TrainTypeList for improved uniqueness by including index as a fallback * Refactor latitude and longitude checks to use null comparison for consistency across hooks and components * Refactor useRefreshStation to use null comparison for nextStation for consistency * Refactor useStationList to improve null handling for lineId and stationId variables * Refactor useStationsCache to improve error handling for fetch operations * Refactor useTTSText to improve formatting of next station announcements * Refactor validateEnvUrls function to enhance error reporting for missing and empty API URL environment variables * Refactor handleSubmit to ensure a default limit for station search results * Refactor RouteSearchScreen to add guard clause for groupId presence before querying routes * Refactor RouteSearchScreen to add guard clause for selectedStation presence before querying stations by line ID * Refactor useSimulationMode tests to improve async handling and state verification * Refactor useSimulationMode tests to verify train speed profile generation only for stations with valid coordinates * Refactor useTTSText to validate stationNumberIndex before accessing stationNumbers * Refactor keyExtractor in SelectLineScreen to handle null or undefined line IDs * Refactor TypeChangeNotify to handle potential null values for trainType and currentLine properties * Refactor TrainTypeListModal keyExtractor to simplify ID extraction logic and ensure fallback to empty string. Enhance gqlClient cache configuration to safely merge incoming lines with existing values. * Refactor StationSearchModal to enhance uniqueness logic for station entries by incorporating prefecture information when groupId and id are unavailable. * Refactor useSimulationMode tests to utilize modern fake timers and improve mock restoration in afterEach. * Refactor SelectLineScreen to filter stations with valid coordinates and determine nearest station based on user location or first available station coordinates. * Refactor RouteSearchScreen keyExtractor to improve ID extraction logic with fallback mechanism using index and groupId or name. * Refactor EmptyResult component to handle loading and search states; remove RouteList and RouteListModal components; update StationSearchModal to use new EmptyResult props; modify TrainTypeListModal to improve line selection logic; enhance Apollo Client cache handling for TrainType; adjust GraphQL queries to streamline data fetching; update RouteSearchScreen to manage train type caching and improve search functionality. * Refactor TrainTypeListModal keyExtractor to improve ID extraction logic with fallback mechanism using groupId, id, or index; update RouteSearchScreen to parse and validate station result limit before fetching data. * Update README.md to reflect new Node.js and pnpm versions; enhance project structure details and available scripts; clarify GraphQL type regeneration requirements. * Refactor EmptyResult component to simplify condition for displaying search result message when not loading. * Add guard clause in handleTrainTypeSelected to prevent selection of train types without groupId * Increase JVM memory settings in gradle.properties to optimize build performance * Refactor SelectLineScreen to replace stationsRef with stationsCache for improved state management and data fetching logic. * ?? 0は不要 * 気が狂うほどバグを修正した * biome check ./src --unsafe --fix * 型エラー一掃 * loadingがたりん * ユニットテスト拡充と修正 * biome fix * すごいbiome fix * 型エラー修正 * 駅名検索後路線キャッシュをクリアする * batchMax拡張 * 路線選択フローのバグ修正、最適化 * エラー処理 * 必須バリデーション完了後にモーダルを開く * destinationStationIdはgroupIdではなくid * SelectBoundModalに片方の行先しか出ないことがあるバグを修正 * プリセットの駅一覧取得処理のバグを修正 * エラー処理をマシにした * クリーンアップ処理の欠落 * バグ修正 * 軽微な修正 * LineBoardのデグレ修正 * evict/gcをdepsにするとよくないらしい * 必要ないコードの削除 * 必要なのかわからんけど一応キャスト * 型安全 * useScaleのメモ化 * nullish送信絶対許さないマン * 堅牢化 * 初期化漏れ * Add repository guidelines for automation agents collaboration (#4524) * Add sonar-project.properties for SonarQube configuration (#4525) * 累積的UI改善とリファクタリング (#4526) * Add singleLine prop to Heading component for better text handling * Refactor LineBoard components for improved readability and maintainability - Extracted station numbering logic into a custom hook in LineBoardWest. - Introduced helper functions for rendering background and future bars in LineBoardToei. - Consolidated station progress calculations into a separate hook in LineBoardWest. - Simplified chevron visibility logic in LineBoard components. - Enhanced gradient rendering for station bars in LineBoardSaikyo and LineBoardToei. - Improved code organization by separating concerns into smaller components and hooks. * Enhance NowHeader component with dynamic background color and padding adjustments for improved styling on Android * Refactor SelectBoundModal and SelectBoundSettingListModal components to remove unused props and improve state management * Refactor TypeChangeNotify component to streamline styles and improve readability * Refactor NowHeader component to use type imports for ViewStyle and remove duplicate LED_THEME_BG_COLOR import * Refactor LineBoard components to improve empty array generation logic for better readability and maintainability * Refactor StationNameCell and LineBoardWest components to enhance station numbering logic by incorporating line parameter for improved data handling * bump android version
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
1 participant
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.
Summary by CodeRabbit
リリースノート