[docs][plugin sdk] Split host-hook recipes from workflow follow-ups#74853
[docs][plugin sdk] Split host-hook recipes from workflow follow-ups#74853100yenadmin wants to merge 7 commits intoopenclaw:mainfrom
Conversation
|
Codex review: needs maintainer review before merge. Summary Reproducibility: not applicable. this is a docs PR rather than a runtime bug. I source-checked the PR content against current main contracts and Gateway behavior instead of exercising a failing reproduction. Next step before merge Security Review detailsBest possible solution: Land the docs recipe split after the companion host-hook implementation stack is ready, keeping future APIs labeled and stable examples matched to current contracts. Do we have a high-confidence way to reproduce the issue? Not applicable; this is a docs PR rather than a runtime bug. I source-checked the PR content against current main contracts and Gateway behavior instead of exercising a failing reproduction. Is this the best way to solve the issue? Yes. A docs-only split is the maintainable review shape for this large recipe page, with the safe path being companion-stack merge ordering and explicit labels for APIs not yet on stable main. Acceptance criteria:
What I checked:
Likely related people:
Remaining risk / open question:
Codex review notes: model gpt-5.5, reasoning high; reviewed against 90a5b08fb799. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 57098df02f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
Adds a new Plugin SDK documentation page that collects host-hook “recipe” examples and wires it into the docs navigation + zh-CN glossary, enabling maintainers and plugin authors to reference composition patterns separately from the implementation PR stack.
Changes:
- Added
docs/plugins/host-hooks-examples.md(host-hook recipes/examples page). - Added the page to
docs/docs.jsonnavigation. - Added zh-CN glossary entries for the page title and sidebar label.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| docs/plugins/host-hooks-examples.md | New long-form host-hook recipes page with contracts, examples, diagrams, and archetype compositions. |
| docs/docs.json | Adds the new page to the Plugins section nav. |
| docs/.i18n/glossary.zh-CN.json | Adds zh-CN glossary translations for the new page title/sidebar strings. |
57098df to
268b469
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1c04613815
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6e66eb5cc1
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
16c0627 to
2261827
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 22618273b2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
3b44b32 to
2ce45c2
Compare
2ce45c2 to
07fe0d2
Compare
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Fixed in |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Summary
This PR splits the large host-hook recipe documentation out of the implementation stack so maintainers can review code/tests and docs separately. It carries the product-neutral recipe page, docs navigation entry, and zh-CN glossary entries that were previously part of #73384.
Why This Exists
The implementation PRs are already substantial, and the recipe page adds about 2.7k lines of prose/examples/diagrams by itself. Splitting it keeps the review surfaces smaller:
Scope
docs/plugins/host-hooks-examples.md.docs/docs.json.Stack Order
Review/merge order should be:
The docs intentionally reference APIs introduced by the implementation stack, so this PR should land after those implementation PRs.
Validation
pnpm docs:listRelated