Skip to content

chore(docs): 完了済み iOS shared-lib チケットを削除 (#381) - #390

Merged
ayutaz merged 3 commits into
devfrom
chore/cleanup-381-tickets
May 5, 2026
Merged

chore(docs): 完了済み iOS shared-lib チケットを削除 (#381)#390
ayutaz merged 3 commits into
devfrom
chore/cleanup-381-tickets

Conversation

@ayutaz

@ayutaz ayutaz commented May 5, 2026

Copy link
Copy Markdown
Owner

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) に簡潔化
  • §Updating からマイルストーン進捗反映の項目を削除
  • §2.1 注記の (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 例で実施済み:

実装内容はコードと 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 から仕様書本体への参照リンクが生きていることを確認

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)
Copilot AI review requested due to automatic review settings May 5, 2026 00:38

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

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.

Comment thread docs/spec/ios-shared-lib.md Outdated
Comment thread docs/spec/ios-shared-lib.md Outdated
Comment thread docs/spec/ios-shared-lib.md Outdated
Comment thread docs/spec/ios-shared-lib.md Outdated
Comment thread docs/spec/ios-shared-lib.md Outdated
Comment thread docs/spec/ios-shared-lib.md Outdated
Comment thread examples/dart/README.md Outdated
ayutaz added 2 commits May 5, 2026 09:50
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, #6b8dc0ce で §8 マイルストーン節を全削除したため該当行が
存在せず Outdated 扱い。
@ayutaz

ayutaz commented May 5, 2026

Copy link
Copy Markdown
Owner Author

Copilot レビュー対応

ご指摘ありがとうございます。コメントを精査して反映しました。

対応した指摘

  • コメント windowsのCI/CD対応 #7 (examples/dart/README.md:113) — 「Android AAR packaging is an M5 candidate.」だと追跡先が無く読者が行き止まりになる旨。→ 「未トラッキング、必要なら ayutaz/piper-plus/issues で issue を立ててほしい」と明記しました (commit d3e1173)。
  • コメント Fix/finetuning #2-5 の精神 (PR リンク化) — 該当行 (M1-M4 の状態行) は b8dc0ce で §8 マイルストーン節を全削除したため消滅していますが、仕様書冒頭の Status 行に残る PR #381 も同様にマークダウンリンクへ変更し、ナビゲーション性を確保しました (commit d3e1173)。

Outdated 扱いとした指摘

f864f94 を対象とした以下のコメントは、後続の b8dc0ce で §8 マイルストーン節 (~140 行) を全削除したため該当行が存在せず Outdated です:

  • コメント 日本語音素に対応した前処理および推論コード #1 (line 4 Status 整合性) — 旧 Status の M1→M2→M3 完了、M4 同ブランチに含むImplemented (v1.13.0, PR #381) に簡潔化したことで、進捗トラッキング指針 (M4 完了 = Implemented + SPM) を §8 ごと削除し、整合性問題自体が解消。
  • コメント Fix/finetuning #2-5 (line 248/272/306/333 各 M の状態行) — §8 削除で消滅。
  • コメント linuxのビルド #6 (line 251 チェックボックス整合性) — §8 削除で消滅。

@ayutaz
ayutaz merged commit 7bba0aa into dev May 5, 2026
14 checks passed
@ayutaz
ayutaz deleted the chore/cleanup-381-tickets branch May 5, 2026 01:19
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