Skip to content

feat: 独立チェックリスト集と図表索引を追加する#241

Merged
ootakazuhiko merged 2 commits into
mainfrom
codex/issue-240-required-ux
Jul 13, 2026
Merged

feat: 独立チェックリスト集と図表索引を追加する#241
ootakazuhiko merged 2 commits into
mainfrom
codex/issue-240-required-ux

Conversation

@ootakazuhiko

Copy link
Copy Markdown
Collaborator

概要(必須)

  • 変更内容: Profile B必須の独立チェックリスト集と図表索引を追加し、既存5 Mermaid図をJavaScript非依存・アクセシブルな自己完結SVGへ置換しました。公開正本 docs/** と安全な src/** ミラー、常設導線、UX flags、回帰契約を同期しています。

関連Issue/仕様(必須)

影響範囲(必須)

  • 対象章/ページ: /chapters/chapter01/, /chapters/chapter05/, /chapters/chapter07/, /chapters/chapter09/, /chapters/chapter12/, 2付録、トップ、navigation。
  • 影響: 既存Mermaid 5件のSVG化、独立モジュール追加、mobile横スクロール、QA契約追加。

受入基準(必須)

  • Issue の受入基準を満たす
  • 変更禁止領域に触れていない(または合意済み)
  • 章参照・内部リンク・アンカーが壊れていない

テスト/検証(必須)

  • 実行コマンドと結果:
  • 手動確認: desktopおよび390px相当を実ブラウザで確認。通常4図は横スクロール、narrow図は縮小表示。第5章判断分岐、第7章review flow、第9章継承矢印を目視確認済み。
  • 既知の既存債務: python3 scripts/validate_links.py docs は変更前から存在する外部/テンプレートURLを含む59件を報告します。今回追加した2 route・5 deep linkは専用契約で全件検証済みで、#235の依存関係warningとともに本PR非スコープです。

リスク/ロールバック(必須)

  • リスク: SVG内文字量が多いため、狭幅では縮小ではなく横スクロールを採用しています。
  • ロールバック手順: 本PRのmerge commitをrevertし、MermaidとUX flags/navigationを従前へ戻します。

Review 対応(必須)

  • review 本文・inline comment・suggestion を全件確認した
  • 修正した指摘には返信した
  • 修正しない指摘には理由を返信した
  • 未解決の review thread が 0 件である

QA(必須)

  • Book QA(Unicode / textlint(PRH) / 内部リンク・アンカー / Jekyll build / built-site smoke): PASS
    • 実行URL: PR checksで追記
  • 必須 status checks が green である

merge 後確認(原則必須)

  • main の checks が green である
  • release / deploy / Pages など公開先の smoke test を実施した
  • 完了証跡を関連 Issue に記録した

Pages確認(原則必須)

AI支援/エージェント利用の開示(任意)

  • AI/エージェント利用: 有
  • 利用範囲: 実装、契約checker、独立レビュー、視覚検証補助
  • 人間が最終確認した観点: Issue契約、変更所有権、レビュー対応、CI/公開証跡

補足

Copilot AI review requested due to automatic review settings July 13, 2026 10:39
@ootakazuhiko

Copy link
Copy Markdown
Collaborator Author

@codex review

Review exact head 7a94348aca277a3eebf0f17f1fb655d8d3e83130, including the checklist-pack contract, exact five-figure inventory, SVG semantics, docs/src parity, mobile behavior, and CI wiring.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

本PRは、Issue #240 の要件に沿って「独立チェックリスト集」と「図表索引」を追加し、既存5つの Mermaid 図を JavaScript 非依存でアクセシブルな自己完結 SVG 図版に置き換えることで、GitHub Pages 公開物(docs/**)の読者体験と回帰検証を強化する変更です。

Changes:

  • checklist-packfigure-index の2つの付録ページを docs/**src/** に追加し、トップ/ナビゲーションから到達可能に更新
  • 第1/5/7/9/12章の Mermaid 図を <figure id="..."> + SVG + caption へ置換し、deep link 可能な stable anchor を提供
  • UX flags と回帰契約(scripts/check-issue-240-ux.js)を追加し、Book QA / test:light に組み込み

Reviewed changes

Copilot reviewed 22 out of 22 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/chapters/chapter01/index.md Mermaid 図を stable anchor 付き SVG 図版へ置換
src/chapters/chapter05/index.md Mermaid 図を stable anchor 付き SVG 図版へ置換
src/chapters/chapter07/index.md Mermaid 図を stable anchor 付き SVG 図版へ置換
src/chapters/chapter09/index.md Mermaid 図を stable anchor 付き SVG 図版へ置換
src/chapters/chapter12/index.md Mermaid 図を stable anchor 付き SVG 図版へ置換
src/appendices/checklist-pack/index.md 独立チェックリスト集(source 側)を新規追加
src/appendices/figure-index/index.md 図表索引(source 側)を新規追加
docs/chapters/chapter01/index.md 公開本文の Mermaid 図を SVG 図版へ置換
docs/chapters/chapter05/index.md 公開本文の Mermaid 図を SVG 図版へ置換
docs/chapters/chapter07/index.md 公開本文の Mermaid 図を SVG 図版へ置換
docs/chapters/chapter09/index.md 公開本文の Mermaid 図を SVG 図版へ置換
docs/chapters/chapter12/index.md 公開本文の Mermaid 図を SVG 図版へ置換
docs/appendices/checklist-pack/index.md 独立チェックリスト集(公開側)を新規追加
docs/appendices/figure-index/index.md 図表索引(公開側)を新規追加
docs/index.md トップに新付録2ページへの導線を追加
docs/_data/navigation.yml 付録Gの後〜あとがき前に新付録2ページを追加
docs/assets/css/main.css モバイルで図版を横スクロール表示できる CSS を追加
book-config.json UX flags(checklistPack / figureIndex)を有効化
scripts/check-metadata-consistency.js 期待ナビゲーションに新付録2 route を追加
scripts/check-issue-240-ux.js Issue #240 の回帰契約(routes/anchors/parity/CSS/QA実行)を新規追加
package.json check:issue-240-ux を追加し test:light に組み込み
.github/workflows/book-qa.yml Book QA に Issue #240 回帰契約チェックを追加

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread scripts/check-issue-240-ux.js Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7a94348aca

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/assets/css/main.css
@ootakazuhiko

Copy link
Copy Markdown
Collaborator Author

@codex review

Review exact head d564d60af180989b136e477f2eb026acc44801ee. Prior review findings were addressed by preserving the mobile figure contract in templates/styles/main.css, checking both source/public stylesheets, and aggregating invalid JSON diagnostics without a stack trace. Please review the full PR at this head.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Chef's kiss.

Reviewed commit: d564d60af1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ootakazuhiko ootakazuhiko merged commit 5948d89 into main Jul 13, 2026
2 checks passed
@ootakazuhiko ootakazuhiko deleted the codex/issue-240-required-ux branch July 13, 2026 10:51
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.

feat: 独立チェックリスト集と図表索引を追加する

2 participants