Skip to content

fix: complete PR75 review fixes - #76

Merged
somasekimoto merged 3 commits into
feat/example-ui-monas-drivefrom
fix/pr75-review-followup
Sep 17, 2026
Merged

somasekimoto merged 3 commits into
feat/example-ui-monas-drivefrom
fix/pr75-review-followup

Conversation

@somasekimoto

@somasekimoto somasekimoto commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

Summary

PR75 のレビューで見つかった不具合を修正する追加PRです。PR75 マージ時に未pushだった修正を、このPRで反映します。base は PR75 と同じ feat/example-ui-monas-drive です。

Changes

  • 本文の更新順序を同じ DAG payload に保持し、policy-only 更新・再マージ・遅着した古い write による本文の巻き戻りを防止。
  • 分岐した DAG の operation export を実ノードとの構造対応に変更。兄弟枝を保持し、親から順に再同期。曖昧な対応は拒否。
  • Preview の同期表示を実際に表示している本文の version と比較。
  • 旧 node の revoke 伝播報告欠落を「伝播不要」ではなく「不明」と表示。
  • 旧複数 signing account を最後に作成された account へ移行。以前の復号鍵は保持し、古い signing key の復活を防止。
  • 実 replica と実画面経路の回帰テストを追加し、既存の設計書・テスト手順を更新。

Testing

  • cargo test --workspace: 870 passed / 0 failed / 4 ignored
  • cargo clippy --workspace --all-targets --profile test --no-deps -- --deny warnings: passed
  • cargo fmt --check, git diff --check: passed
  • cd example-ui && npm run build: passed
  • npm run test:regression: 12 passed(3回反復でも36 passed)
  • ローカル4-node libp2p mesh: create・即時同期・relay read の smoke 3 passed
  • 旧本文選択ロジックに戻すと実 replica の3件が失敗。未来の更新順序・policy clamp・UI 各修正も個別 mutation で回帰検出を確認。

Deployment / limitations

  • 保存・wire 形式変更あり: body_updated_at は必須。旧ストアの自動移行は未実装。全 node 同時更新と新ストアでのコンテンツ再作成、または別途移行対応が必要です。
  • データ削除・Terraform apply・hosted-node deploy は行っていません。main の Terraform 差分統合はこのPRに含めません。
  • 伝播が遅れた member に対する revoke 直後の write を禁止する変更ではありません。
  • Delete operation の export と曖昧な履歴は推測せず拒否します。
  • UI の backend は HTTP fixture。ブラウザから実 gateway・node を通す共有/revoke 全旅程は未実施です。

@somasekimoto
somasekimoto merged commit ce6b389 into feat/example-ui-monas-drive Sep 17, 2026
5 checks passed
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.

1 participant