[SDK/Factories] Add Factory Pagination And Completion Options - #2431
[SDK/Factories] Add Factory Pagination And Completion Options#2431MRayermannMSFT wants to merge 4 commits into
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
55dffc6 to
79b0f32
比较
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Copilot review overview
Review tier: Balanced
Findings: None
What changed in this PR
Adds Node.js Agent Factories pagination metadata and runtime completion/phase controls.
Changes:
- Adds paginated
listRunsoverloads while preserving the array-returning API. - Forwards notification and phase logging options for runs and resumes.
- Documents and tests the new behavior.
| File | Description |
|---|---|
nodejs/src/factory.ts |
Defines pagination types and runtime options. |
nodejs/src/session.ts |
Implements pagination and option forwarding. |
nodejs/src/index.ts |
Exports pagination types. |
nodejs/src/extension.ts |
Exports pagination types for extensions. |
nodejs/docs/factories.md |
Documents the new API. |
nodejs/test/factory.test.ts |
Tests RPC forwarding and response shapes. |
nodejs/test/e2e/factory.e2e.test.ts |
Tests pagination and runtime controls end to end. |
nodejs/test/e2e/fixtures/factory-extension.mjs |
Adds a phased E2E factory fixture. |
CHANGELOG.md |
Records the feature additions. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
79b0f32 to
d38d8c0
比较
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
d38d8c0 to
894a4f8
比较
Cross-SDK Consistency Review ✅All 9 changed files in this PR are scoped to the Node.js SDK ( The Agent Factories API is an experimental, Node.js-only surface (clearly marked No cross-SDK consistency issues found.
|
What
The Node.js Agent Factories API adds paginated run listing with cursor and truncation metadata. Factory run and resume options now forward completion notifications and phase-name logging.
Why
SDK callers can inspect complete run history without raw RPC calls. This pull request is stacked on #2430 because that pull request provides the generated Copilot CLI 1.0.81 contract.