Skip to content

リポジトリ整備: 未使用依存の削除・CI追加・OGP修正・ドキュメント更新 - #53

Merged
aida0710 merged 7 commits into
masterfrom
agents/admiring-volta-q1un0d
Oct 5, 2026
Merged

aida0710 merged 7 commits into
masterfrom
agents/admiring-volta-q1un0d

Conversation

@aida0710

@aida0710 aida0710 commented Oct 5, 2026 •

Copy link
Copy Markdown
Owner

概要

バグ修正: 下層ページで OGP 画像・Twitter カードが消える

  • /projects などの下層ページで og:image・og:site_name・og:type と Twitter の summary_large_image カードが出力されていなかった
  • 原因: Next.js は openGraph / twitter をレイアウトとページの間でマージせず丸ごと置き換える。buildPageMetadata が title などだけを返していたため、ルートレイアウトの画像設定が消えていた
  • 修正: libs/i18n/metadata.ts で共通部分(画像・サイト名・type・カード種別)をまとめ、buildRootMetadata と buildPageMetadata の両方で使うようにした

依存・設定

  • 未使用の依存を削除: intl-messageformat, autoprefixer, @heroui/{card,chip,input,navbar,skeleton}
    • framer-motion と tailwind-variants はコードから直接 import されていないが、HeroUI の peer 依存なので残した(framer-motion は MobileMenu の motionProps 経由で使われている)
  • tsconfig.json の target を es5 → ES2017(TypeScript 6 で非推奨、7 で廃止予定)
  • typecheck スクリプト(tsc --noEmit)を追加

リポジトリ衛生

  • .idea/ を git 管理

claude added 4 commits October 5, 2026 10:17
- framer-motion / intl-messageformat / autoprefixer と未使用のHeroUIパッケージ
  (card/chip/input/navbar/skeleton) を削除
- target を es5 から ES2017 に更新 (TypeScript 7 での非推奨エラーを解消)
- typecheck スクリプト (tsc --noEmit) を追加

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TWUFfZADjGD1vLMxAqGdyU
- .idea をgit管理から外し .gitignore に .idea/ と .vscode/ を追加
- Biomeへ統一済みのため prettier/standardjs を指す .deepsource.toml を削除

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TWUFfZADjGD1vLMxAqGdyU
- README にセットアップ手順とコマンド一覧を追加
- CLAUDE.md: typecheck コマンドを追記、削除した framer-motion の注意書きを除去

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TWUFfZADjGD1vLMxAqGdyU
@vercel

vercel Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
profile Ready Ready Preview Oct 5, 2026 11:24am UTC

テンプレート (Next UI) 由来のコードが残っている可能性があるため、
MITの条件に従い元の著作権表記も併記している。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TWUFfZADjGD1vLMxAqGdyU
コードから直接importしていないが、HeroUIのpeer依存でMobileMenuの
motionProps経由で使われているため削除は誤りだった。CLAUDE.mdの
注意書きも復元し、削除してはいけない理由を追記した。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TWUFfZADjGD1vLMxAqGdyU
Next.jsはopenGraph/twitterをレイアウトとページでマージせず置き換えるため、
buildPageMetadataが返すメタデータでog:image・og:site_name・og:typeと
Twitterの大きい画像カードが失われていた。共通部分をまとめ、両方で使う。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TWUFfZADjGD1vLMxAqGdyU
@aida0710 aida0710 changed the title リポジトリ整備: 未使用依存の削除・CI追加・ドキュメント更新 リポジトリ整備: 未使用依存の削除・CI追加・OGP修正・ドキュメント更新 Oct 5, 2026
@aida0710
aida0710 merged commit f3ceb2a into master Oct 5, 2026
3 checks passed

This branch was successfully deployed

1 active deployment
Preview — 2fbebf0b Deployed Oct 5, 2026 by vercel[bot]
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.

2 participants