chore(docs): 完了済み iOS shared-lib チケットを削除 (#381) - #390
Merged
Conversation
PR #381 の M1-M4 マイルストーン全完了に伴い、以下を削除: - docs/tickets/377-M{1,2,3,4}-*.md - docs/tickets/README.md (索引) 仕様書 docs/spec/ios-shared-lib.md からは削除チケットへのリンクを 除去し、各 M セクションの状態行を `done (PR #381, 2026-05-05)` に更新。 examples/dart/README.md の存在しない M5-20 チケットへの参照を文面に置換。 実装内容はコードと commit 97cac19 に記録済み。 永続仕様 docs/spec/ios-shared-lib.md と利用者ガイドは保持。 過去の同様の整理: 6f2772f (#346) / 7cc7e0e (HTS voice) / 204807a (phoneme timing)
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
完了済みの iOS shared-lib 作業チケット(docs/tickets 配下)を削除し、仕様書・README からの参照を整理して「完了状態」を仕様書側に集約するPR。
Changes:
docs/tickets/配下の M1–M4 チケットおよび索引 README を削除docs/spec/ios-shared-lib.mdから tickets/README 参照を除去し、各マイルストーンの状態表記をdone (PR #381, 2026-05-05)に統一examples/dart/README.mdの壊れたチケットリンクを削除し、文面に置換
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| examples/dart/README.md | 存在しない tickets へのリンクを除去し、Android AAR の位置づけを文面化 |
| docs/spec/ios-shared-lib.md | tickets に依存しない進捗記述へ移行し、各Mの状態を簡略化 |
| docs/tickets/README.md | チケット索引(SoT)を削除 |
| docs/tickets/377-M1-ort-fetch-fix.md | 完了済みチケットを削除 |
| docs/tickets/377-M2-xcframework.md | 完了済みチケットを削除 |
| docs/tickets/377-M3-docs-migration.md | 完了済みチケットを削除 |
| docs/tickets/377-M4-spm-package.md | 完了済みチケットを削除 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
PR #381 完了に伴い作業計画書としての §8 マイルストーン節 (M1-M4 スコープ・DoD・PR 単位・タイムライン) を削除し、永続的価値のある 設計判断 (§1-7) と仕様更新ルール (§Updating) のみ残す。 - docs/spec/ios-shared-lib.md: §8 全体 (~140 行) と冒頭 Status の 進捗注記を削除、§Updating からマイルストーン進捗反映の項目を削除、 §2.1 注記の `(M1 で対応)` `(M3 で利用者ガイドに明記)` を恒久表現に置換 - docs/guides/ios-integration.md: 仕様書参照テキストから 「milestone history」「plan A milestones」を除去 - examples/swift/README.md: 仕様書参照テキストから「milestone history (M1-M4)」を除去 - Package.swift: 「Mac Catalyst slices are M5 candidates」を 「not yet supported (see §6)」に置換 仕様書サイズ: 387 → 244 行 他箇所に残る M2/M4/M5 言及 (コードコメント、利用者ガイド、サンプル) は それぞれの文脈で意味が通る記述のため保守的に残す。
PR #390 の Copilot レビュー指摘 (コメント #7 + コメント #2-5 の精神) を反映: - docs/spec/ios-shared-lib.md: Status 行の `PR #381` を GitHub PR へのマークダウンリンクに変更し、実装根拠への 到達性を改善。 - examples/dart/README.md: 「Android AAR packaging is an M5 candidate.」を「未トラッキング、必要なら issue を立ててほしい」 と明記し、読者の次アクションを示す。 旧コミット f864f94 を対象としたコメント #1, #2-5, #6 は b8dc0ce で §8 マイルストーン節を全削除したため該当行が 存在せず Outdated 扱い。
Owner
Author
Copilot レビュー対応ご指摘ありがとうございます。コメントを精査して反映しました。 対応した指摘
Outdated 扱いとした指摘f864f94 を対象とした以下のコメントは、後続の b8dc0ce で §8 マイルストーン節 (~140 行) を全削除したため該当行が存在せず Outdated です:
|
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
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
PR #381 (iOS shared-lib パイプライン復旧 + xcframework 配布) のマージにより M1-M4 マイルストーンが全て完了したため、
docs/tickets/配下の作業チケットと仕様書 §8 マイルストーン節を削除し、関連参照を整理する。削除したチケットファイル (commit f864f94)
docs/tickets/377-M1-ort-fetch-fix.md(697 行)docs/tickets/377-M2-xcframework.md(907 行)docs/tickets/377-M3-docs-migration.md(768 行)docs/tickets/377-M4-spm-package.md(405 行)docs/tickets/README.md(チケット索引、77 行)仕様書 §8 マイルストーン節を削除 (commit b8dc0ce)
docs/spec/ios-shared-lib.mdから作業計画書としての §8 (M1-M4 スコープ・DoD・PR 単位・タイムライン、約 140 行) を削除し、永続的価値のある設計判断 (§1-7) と仕様更新ルール (§Updating) のみ残す。仕様書サイズ: 387 → 244 行。Status:行の進捗注記 (fix/ios-shared-lib-build-377 で M1→M2→M3 完了、M4 同ブランチ) を削除しImplemented (v1.13.0, PR #381)に簡潔化(M1 で対応)(M3 で利用者ガイドに明記)を恒久表現に置換外部ドキュメントの相互参照を整理
docs/guides/ios-integration.md: 仕様書リンクのテキストから「milestone history」「plan A milestones」を除去examples/swift/README.md: 仕様書リンクのテキストから「milestone history (M1-M4)」を除去examples/dart/README.md: 存在しないdocs/tickets/M5-20-android-aar.mdへの壊れたリンクを文面に置換Package.swift: 「Mac Catalyst slices are M5 candidates」を「not yet supported (see §6)」に置換保持したファイル (永続ドキュメント)
docs/spec/ios-shared-lib.md(仕様書 §1-7 + §Updating、Status: Implemented v1.13.0)docs/guides/ios-integration.md(利用者ガイド)examples/swift/README.md/examples/dart/README.md/examples/godot/README.md(利用者向けサンプル)過去の慣行との整合
マイルストーン完了後に作業チケット一式を削除する運用は過去 3 例で実施済み:
6f2772f— Issue C++ CLI: splitTextToSentences() does not consume CJK closing brackets after sentence terminators #346 (cpp CJK closing bracket) M1-M4 完了7cc7e0e— HTS voice 除去 M1-M4 完了204807a— phoneme timing 完了実装内容はコードと commit 97cac19 (PR #381) に記録されているため、作業チケットは git 履歴から復元可能。
Test plan
grep -r "377-M[1-4]\|tickets/377\|docs/tickets" docs/ src/ examples/で削除チケットへの参照が残っていないことを確認grep "§8\|fix/ios-shared-lib-build-377" docs/spec/ios-shared-lib.mdで仕様書内の自己参照が残っていないことを確認M2/M4/M5言及 (コードコメント、利用者ガイド、サンプル) はそれぞれの文脈で意味が通ることを確認Package.swift/ CI ワークフロー /CLAUDE.mdから仕様書本体への参照リンクが生きていることを確認