Conversation
…xamples Co-authored-by: blindzero <13959569+blindzero@users.noreply.github.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4c471b884a
ℹ️ 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
Updates the step reference documentation generator so generated step pages include required provider capabilities and richer usage guidance, addressing previously “Unknown” contracts/events and sparse inputs/examples.
Changes:
- Extend
tools/Generate-IdleStepReference.ps1to extract Required Capabilities from step metadata catalogs and render them in step pages. - Improve generated step pages with a 3-column Inputs table, a workflow-style configuration example, and (conditionally) “See Also” links.
- Regenerate 9 step reference pages under
docs/reference/steps/to match the new output format.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
| tools/Generate-IdleStepReference.ps1 | Generator now pulls RequiredCapabilities from step metadata and emits updated sections (Inputs/Example/See Also). |
| docs/reference/steps/step-trigger-directory-sync.md | Regenerated step page with required capabilities and new sections. |
| docs/reference/steps/step-move-identity.md | Regenerated step page with required capabilities, inputs table, and example. |
| docs/reference/steps/step-ensure-entitlement.md | Regenerated step page with required capabilities, inputs table, and example. |
| docs/reference/steps/step-ensure-attribute.md | Regenerated step page with required capabilities, inputs table, and example. |
| docs/reference/steps/step-enable-identity.md | Regenerated step page with required capabilities and new sections. |
| docs/reference/steps/step-emit-event.md | Regenerated step page with new inputs/example sections. |
| docs/reference/steps/step-disable-identity.md | Regenerated step page with required capabilities and new sections. |
| docs/reference/steps/step-delete-identity.md | Regenerated step page with required capabilities and new sections. |
| docs/reference/steps/step-create-identity.md | Regenerated step page with required capabilities, inputs table, and example. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@copilot all suggestions must be applied to the generator tool as the step mds themselves are generated |
…guation, consistent See Also Co-authored-by: blindzero <13959569+blindzero@users.noreply.github.com>
Applied all suggestions to the generator tool. Changes in commit 656729f include:
All 9 step reference pages regenerated with accurate information - DisableIdentity, EnableIdentity, DeleteIdentity now show IdentityKey; TriggerDirectorySync shows AuthSessionName and PolicyType; EmitEvent has "See Also" section. |
|
@copilot regenerate after the latest changes and review again |
Documentation already regenerated in commit 656729f. Re-ran generator to confirm - no changes needed, all pages are current. All PR review feedback addressed:
|
Step reference pages showed "Unknown" for contracts/events and lacked examples or input documentation. Generator now extracts capabilities from metadata catalogs, provides structured usage examples, and accurately detects required input keys.
Changes
Generator enhancements (
tools/Generate-IdleStepReference.ps1):IdLE.Identity.Create,IdLE.Entitlement.Grant)Example output improvement:
Before:
After:
All 9 step reference pages regenerated with accurate, consistent format. Steps that previously showed no inputs (DisableIdentity, EnableIdentity, DeleteIdentity, TriggerDirectorySync) now correctly document their required keys.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.