Skip to content

docs(repo): ADR-0004 の死んだリンクと README 依存表のずれを直す (#258 #257) - #262

Merged
shinyaoguri merged 1 commit into
mainfrom
docs/p3-readme-and-adr-links
Aug 15, 2026
Merged

docs(repo): ADR-0004 の死んだリンクと README 依存表のずれを直す (#258 #257)#262
shinyaoguri merged 1 commit into
mainfrom
docs/p3-readme-and-adr-links

Conversation

@shinyaoguri

Copy link
Copy Markdown
Owner

目的

2026-08 レビュー (#243) の残件のうち、docs の鮮度に関する低コスト指摘 2 件を解消する。

変更点

#258 — ADR-0004 が存在しないパスを参照している

docs/adr/0004-verifier-checkpoint-stance.md の References が packages/verify/src/core/VerificationEngine.ts を指していたが、このディレクトリごと存在しない。verify 側の現在の検証入口である packages/verify/src/workers/verificationWorker.ts へ張り替え、PR #252 以降 shared の verifyProofFile へ委譲している旨を添えた。

  • ADR-0004 の Decision 6 点は現行実装と整合していることを確認済み (root 再計算 / content replay / 未署名 cp の扱い / POSW_ITERATIONS === 10000 の必須検査 / isPureTyping 再計算 / 検証側に CHECKPOINT_INTERVAL の hard-code 無し)。リンクだけが古い
  • docs/ 配下 40 ファイルを走査して相対リンク切れはこの 1 件のみ。週次の鮮度チェック (chore: 鮮度チェックの検知結果 (自動) #261) が報告しているのも同一の 1 件なので、次回実行から消える

#257 — README の依存表が実態とずれている

packages/editor/README.md:

箇所 修正前 修正後 備考
実行環境表 (TypeScript) SWC typescript (ts.transpileModule) 完全な誤記。リポジトリに SWC は存在しない
monaco-editor ^0.55 ^0.56 #206 (dependabot) で上がった分
vite ^8.0 ^8.1 実態は ^8.1.5
dompurify (行なし) ^3.4 runtime dependency が表から欠落
marked (行なし) ^18.0 同上

packages/verify/README.md にも同種の vite の桁ずれ (^8.0^8.1) があったので揃えた。両 README の他の行は package.json と一致していることを全行確認済み。

「SWC」誤記を優先した理由: TS7 移行 (#208) を調べる人が実在しないビルドツールを探しに行くため、バージョン表記のずれより実害が大きい。

確認方法

  • npm run lint — green
  • 依存表の全行を packages/{editor,verify}/package.json と突き合わせ済み
  • ADR-0004 の新リンク先 packages/verify/src/workers/verificationWorker.ts の実在を確認済み

残課題 (本 PR の範囲外)

依存表のドリフトは dependabot が package.json だけを書き換える構造上、手作業同期では維持できない。freshness.yml に「README 依存表 vs package.json」の突合を足すのが根本対策だが、現状は表記規則 (patch を落とす / minor を落とす) がリポジトリ内でバラついており誤検知だらけになるため、規則の統一とあわせて別 Issue で扱う。

Closes #258
Closes #257

ADR-0004 の References が存在しないパス packages/verify/src/core/
VerificationEngine.ts を参照していた (ディレクトリごと無い)。verify 側の
現在の入口である verificationWorker.ts へ張り替え、PR #252 以降 shared の
verifyProofFile へ委譲している旨を添えた。docs 配下 40 ファイルを走査して
相対リンク切れはこの 1 件のみで、週次の鮮度チェック (#261) が報告している
のも同一の 1 件。

editor README の実行環境表が TypeScript のランタイムを「SWC」と書いていたが
これは完全な誤記で、リポジトリに SWC は存在しない。実装は typescript
パッケージの ts.transpileModule を直接呼んでいる。TS7 移行 (#208) の調査を
誤誘導するため、バージョン表記のずれより優先して直す。

依存表は dependabot が package.json だけを書き換えるため構造的にドリフト
する。monaco-editor ^0.55 → ^0.56 (#206 で上がった分)、vite ^8.0 → ^8.1 を
実態に合わせ、欠落していた dompurify / marked の 2 行を追加。verify README
にも同種の vite の桁ずれがあったので揃えた。

Closes #258
Closes #257

Assisted-by: Claude <noreply@anthropic.com>
@github-actions github-actions Bot added documentation Improvements or additions to documentation pkg:editor packages/editor pkg:verify packages/verify + verify-cli pkg:repo リポジトリ横断 (docs/CI/依存) labels Aug 15, 2026
@github-actions

Copy link
Copy Markdown

🚀 Preview Deployment

Name URL
Preview https://4874833e.typedcode.pages.dev

Deployed from commit 8fd1295
API: staging Workers

@shinyaoguri
shinyaoguri merged commit 30be258 into main Aug 15, 2026
8 checks passed
@shinyaoguri
shinyaoguri deleted the docs/p3-readme-and-adr-links branch August 15, 2026 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation pkg:editor packages/editor pkg:repo リポジトリ横断 (docs/CI/依存) pkg:verify packages/verify + verify-cli

Projects

None yet

1 participant