docs: ultracode audit — update stale content across 12 files#958
Merged
Conversation
Version pins: - README.md/en: @v0.42.0 → @v0.70.0 in Action examples - pages/tutorials: @v0.68.0 → @v0.70.0 version references - pages/guides: @v0.68.0 → @v0.70.0, actions/checkout@v6 → @v4 Content accuracy: - README.md/en: mark river review exec --plan as shipped (v0.68.0/#935) - README.md/en: update Epic 1/2 roadmap rows to reflect current state - README.en.md: align npm distribution note with README.md (PR #942 shipped) - pages/explanation/skills.md: remove unimplemented skill: prefix example - pages/explanation/river-architecture.en.md: add missing Mermaid diagrams (component, GitHub Actions sequence, local CLI sequence) matching JA version - pages/reference/skill-metadata.md: remove duplicate Phase type declaration Runbook: - docs/runbook/community-skill-eval.md: document agent-based eval path (nightly-eval.yml / skill-eval.yml / evaluate-all.mjs) alongside promptfoo Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
No changes to review compared to main. |
Contributor
There was a problem hiding this comment.
Code Review
このプルリクエストでは、CLIコマンドの安定化やnpm公開ワークフローの導入に伴うドキュメントの更新、Mermaidによるアーキテクチャ図の追加、統合評価パス(evaluate-all.mjsや各種CIワークフロー)に関する説明の追加などが行われています。レビューでは、一部のチュートリアルやガイド(getting-started.mdやrepo-wide-review.md)において、GitHub Actionsのバージョン指定が最新の@v0.70.0ではなく@v0.69.0になっている箇所が指摘されており、一貫性を保つための修正案が提示されています。
| with: | ||
| fetch-depth: 0 | ||
| - uses: s977043/river-reviewer/runners/github-action@v0.68.0 | ||
| - uses: s977043/river-reviewer/runners/github-action@v0.69.0 |
Contributor
There was a problem hiding this comment.
| fetch-depth: 0 # repo-wide context collector が周辺コミット履歴を読むため必須 | ||
| - name: Run River Reviewer (midstream) | ||
| uses: s977043/river-reviewer/runners/github-action@v0.68.0 | ||
| uses: s977043/river-reviewer/runners/github-action@v0.69.0 |
Contributor
There was a problem hiding this comment.
Contributor
PlanGate Review
PlanGate review decision: pass
ポリシー: critical=fail / major=fail-if-required (warn) / minor=comment-only / info=skipped — spec: |
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
ultracode ワークフロー(6エリア並列監査 → 検証 → 修正)による全ドキュメント棚卸し。
27件検出 → 20件確認 → 12ファイル修正。
変更内容
バージョンピン更新
README.md/README.en.md@v0.42.0→@v0.70.0pages/tutorials/getting-started.md/.en.md@v0.68.0→@v0.70.0pages/tutorials/integrating-with-github-actions.md@v0.68.0→@v0.70.0pages/guides/github-actions.md@v0.68.0→@v0.70.0、actions/checkout@v6→@v4pages/guides/repo-wide-review.md@v0.68.0→@v0.70.0、checkout@v6→@v4コンテンツ正確性
README.md/README.en.mdexec --plan実装済み記述、Epic 1/2 ロードマップ行更新、npm 配布状況統一pages/explanation/skills.mdskill:prefix 例を削除pages/explanation/river-architecture.en.mdpages/reference/skill-metadata.mdPhase型宣言を削除pages/tutorials/authoring-skills.mdnpm run skills:validateに変更Runbook 補完
docs/runbook/community-skill-eval.mdTest plan
npm test— 1113 tests pass🤖 Generated with Claude Code