deploy 0626#1213
Merged
Merged
Conversation
政治活動費と同じ5万円基準に統一。定数値・ドメインロジック・ UIラベル・コメント・テストを全て整合性をもって更新。 https://claude.ai/code/session_01DMPGfrNeAmEimm9tHf1q86
…old-vqSvP 経常経費のXMLエクスポート閾値を10万円から5万円に変更
YearSelectorはrouter.pushによるソフトナビゲーションのため、クライアント
コンポーネントのReportProfileFormはアンマウントされず、useStateの初期値
（最初にマウントした年度のid/formData）が保持されていた。結果、別の年度に
切り替えて保存しても、最初の年度のプロフィールがidベースでupdateされて
しまっていた。
key={financialYear}を付与することで、年度変更時にコンポーネントを
remountさせ、props（initialData）から正しくstateが再初期化されるようにする。
https://claude.ai/code/session_01EP5uiLH44LPCFmHX3AE19i
年度切り替え（YearSelector による router.push）が発生した際、 ReportProfileForm のクライアント state が新しい年度の initialData で 再初期化され、以下の性質を満たすことを検証する。 - 別年度に切り替えると前年度の入力値がフォームに残らない - ある年度で保存しても、他の年度のプロフィールを上書きしない - 年度を行き来しても、各年度の保存内容がそれぞれ独立して保持される https://claude.ai/code/session_01EP5uiLH44LPCFmHX3AE19i
…HOoKz Add key prop to ReportProfileForm for proper re-rendering
…gxAD Add security policy document (SECURITY.md)
MFクラウドのデータ整備に伴い、これまで普通預金として記帳されていた 取引が仮払金・立替金などの勘定科目に整理された結果、「無効な科目」 としてインポート対象外になる問題に対応する。 - BS_CATEGORIES に現金・仮払金・立替金（いずれも資産）を追加 - 現金は普通預金と同様に現金類（CASH_ACCOUNTS）として扱い、 PL科目との組み合わせを収入・支出として判定する - 仮払金・立替金とPL科目の組み合わせは非現金仕訳として取り込む Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PgeGvkDyYLoC7MnzpZVoxM
Playwright のブラウザ並列インストール衝突による CI フレーク解消のための再実行。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PgeGvkDyYLoC7MnzpZVoxM
admin・webapp の postinstall がともに `playwright install chromium` を 実行しており、pnpm install のたびに共有キャッシュ(~/.cache/ms-playwright) の __dirlock を奪い合って "Failed to install browsers" で CI が落ちていた。 ブラウザは E2E でしか使わないため、postinstall から pretest:e2e フックへ 移動する。これにより: - すべての pnpm install（CI 全ジョブ・ローカル）からブラウザDLの競合が消える - E2E 実行時のみブラウザを取得（ローカルは pretest フック、CI は e2e ジョブの 既存 `playwright install --with-deps` ステップで担保） - 非E2Eジョブの install が高速化 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PgeGvkDyYLoC7MnzpZVoxM
Support cash and temporary accounts in transaction type detection
政治資金収支報告書の「収入・支出項目の分類基準表」では「党費又は会費」は 「機関紙誌の発行その他の事業による収入」とは別の独立した収入の大分類である。 しかし表示用カテゴリ定義では「機関紙誌+その他事業収入」にまとめられており、 まる見え上でその他の事業収入として表示されていた。 党費・会費を独立した大分類「党費・会費」に変更する。単一勘定のため、 他の単一勘定カテゴリ（機関紙誌・借入金・交付金等）と同様に subcategory は持たせない。 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018n27dhvVTncXV57pF4we5z
Simplify account category mapping for membership fees
発生主義に基づき、PL科目と非CASH BS科目の組み合わせ仕訳を non_cash_journal ではなく income / expense として判定するよう変更。 クレジットカード払いや仮払い・仮受けを含む取引が webapp の サンキー図・トランザクション一覧に表示されるようになる。 既存データの更新は prisma/migrations/migrate-non-cash-journal.sql を Supabase SQL Editor で手動実行すること。 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
developブランチに既に仮払金が追加されていたため、 リベース後に重複が生じた。重複エントリを削除する。 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
developブランチで仮払金・立替金に関するテストが non_cash_journal を 期待する形で追加されていたため、今回の変更に合わせて expense に修正。 また、立替金も非CASH BS科目として SQLマイグレーションの対象に追加。 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…accounts-in-transaction-type feat: 非現金BS科目（未払金・仮払金・仮受金）の取引を収支として認識する
developブランチに既に仮払金が追加されていたため、 リベース後に重複が生じた。重複エントリを削除する。 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
developブランチで仮払金・立替金に関するテストが non_cash_journal を 期待する形で追加されていたため、今回の変更に合わせて expense に修正。 また、立替金も非CASH BS科目として SQLマイグレーションの対象に追加。 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
仮払精算（仮払金/選挙関係費など）のように非CASH BSが借方・ PL費用科目が貸方の仕訳が誤ってincomeと判定されていたバグを修正。 PL科目のtypeを参照してincome/expenseを分岐させる。 SQLマイグレーションにも同パターンのUPDATE条件を追加。 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…-classification fix: 非CASH BS/PL費用科目の仕訳をexpenseとして正しく認識する
借方=BS科目／貸方=PL費用の仕訳がexpense判定されると、サンキー集計の フォールバックで勘定科目名（仮払金など）がそのまま支出ノードに昇格して しまっていた。収入・支出のみを表すサンキー図にBS科目が出るのは不自然な ため、aggregateByCategory / aggregateByCategoryWithTag でBS科目を除外する。 未払費用の特別表示は adjustWithBalance 経由の別経路のため影響しない。 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
`in` 演算子やブラケットアクセスはプロトタイプチェーン上のプロパティ （toString など）も拾うため、DB由来の勘定科目名が誤って科目カテゴリと 判定されうる。BS/PL いずれの科目判定も Object.hasOwn ベースに統一する。 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…-sankey fix: サンキー図でBS科目（仮払金等）を収入・支出ノードから除外
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
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.
No description provided.