[codex] Enable API Extractor#33
Merged
Merged
Conversation
nick-pape
force-pushed
the
codex/enable-api-extractor
branch
from
June 10, 2026 14:05
4898271 to
dfc8032
Compare
nick-pape
marked this pull request as ready for review
June 10, 2026 16:33
There was a problem hiding this comment.
Pull request overview
This PR enables API Extractor across the monorepo via the shared Heft rig, standardizes a shared API Extractor base configuration that writes API reports to common/reviews, and updates packages to provide package-level TSDoc / public API documentation with checked-in API report baselines.
Changes:
- Adds an
api-extractorHeft task (via@rushstack/heft-api-extractor-plugin) and a sharedapi-extractor-base.jsonrig config. - Adds per-package
config/api-extractor.jsonfiles extending the shared base config. - Adds package-level
@packageDocumentation(and other public-surface TSDoc) plus checked-incommon/reviews/*.public.api.mdbaselines.
Reviewed changes
Copilot reviewed 67 out of 68 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| rigs/heft-rig/profiles/default/config/heft.json | Adds the api-extractor task to the shared Heft build phase. |
| rigs/heft-rig/profiles/default/config/api-extractor-base.json | Introduces shared API Extractor baseline settings/output locations. |
| rigs/heft-rig/package.json | Adds Heft API Extractor plugin + API Extractor dependencies to the rig. |
| packages/workspace-nodepod/src/runtime.ts | Adds/exports public types and TSDoc for the Nodepod workspace runtime surface. |
| packages/workspace-nodepod/src/index.ts | Adds package-level @packageDocumentation. |
| packages/workspace-nodepod/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/web-agent/src/tool-provider.ts | Adds TSDoc for the web MCP tool provider public surface. |
| packages/web-agent/src/index.ts | Adds package-level @packageDocumentation. |
| packages/web-agent/src/dependencies.ts | Adds TSDoc for browser agent composition helpers and options. |
| packages/web-agent/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/tools-mcp-node/src/index.ts | Adds package docs, exports config types, and documents Node MCP provider API. |
| packages/tools-mcp-node/src/config.ts | Adds TSDoc for the Node MCP configuration types. |
| packages/tools-mcp-node/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/session-memory/src/index.ts | Adds package docs and TSDoc for the in-memory session manager API. |
| packages/session-memory/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/session-log/src/session-store.ts | Adds TSDoc for JSONL session store API. |
| packages/session-log/src/index.ts | Adds package-level @packageDocumentation. |
| packages/session-log/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/session-local-storage/src/index.ts | Adds package docs and TSDoc for the localStorage-backed session manager API. |
| packages/session-local-storage/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/session-file-system/src/index.ts | Adds package docs and TSDoc for the filesystem-backed session manager API. |
| packages/session-file-system/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/mcp-workspace/src/index.ts | Adds package-level @packageDocumentation for the CLI entrypoint. |
| packages/mcp-workspace/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/mcp-workspace-webcontainer/src/runtime.ts | Adds TSDoc for the WebContainer workspace runtime adapter API. |
| packages/mcp-workspace-webcontainer/src/index.ts | Adds package-level @packageDocumentation. |
| packages/mcp-workspace-webcontainer/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/mcp-workspace-browser/src/sidecar.ts | Adds TSDoc for the browser sidecar contracts and helpers. |
| packages/mcp-workspace-browser/src/runtime.ts | Adds TSDoc for browser workspace runtime contracts and helpers. |
| packages/mcp-workspace-browser/src/index.ts | Adds package-level @packageDocumentation. |
| packages/mcp-workspace-browser/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/context-builder/src/token-estimator.ts | Adds TSDoc for token estimation utilities. |
| packages/context-builder/src/prune-events.ts | Adds TSDoc for pruning options/results/types. |
| packages/context-builder/src/index.ts | Adds package-level @packageDocumentation. |
| packages/context-builder/src/build-context.ts | Adds comprehensive TSDoc for context building/compaction APIs and types. |
| packages/context-builder/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/common/src/session-events.ts | Adds TSDoc for session event model/public contracts. |
| packages/common/src/index.ts | Adds package-level @packageDocumentation. |
| packages/common/src/context-hints.ts | Adds TSDoc for context hint metadata/types/utilities. |
| packages/common/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/agent-core/src/session-cleanup.ts | Adds TSDoc for session cleanup contracts and helpers. |
| packages/agent-core/src/prompt-errors.ts | Adds TSDoc for prompt error normalization/sanitization APIs. |
| packages/agent-core/src/prompt-content.ts | Adds TSDoc for prompt content/text extraction helpers. |
| packages/agent-core/src/interfaces.ts | Adds TSDoc for core agent interfaces and public types. |
| packages/agent-core/src/index.ts | Adds package-level @packageDocumentation. |
| packages/agent-core/src/ai-sdk-model-turn-runner.ts | Adds TSDoc for the AI SDK model turn runner public surface. |
| packages/agent-core/src/agent.ts | Adds TSDoc for the main ACP agent class methods. |
| packages/agent-core/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/acp-host-log/src/index.ts | Adds package-level @packageDocumentation for the host CLI entrypoint. |
| packages/acp-host-log/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| packages/acp-agent/src/index.ts | Adds package-level @packageDocumentation for the agent CLI entrypoint. |
| packages/acp-agent/config/api-extractor.json | Enables API Extractor for the package via rig base config. |
| common/reviews/workspace-nodepod.public.api.md | Checks in regenerated API report baseline for @fledgling/workspace-nodepod. |
| common/reviews/web-agent.public.api.md | Checks in regenerated API report baseline for @fledgling/web-agent. |
| common/reviews/tools-mcp-node.public.api.md | Checks in regenerated API report baseline for @fledgling/tools-mcp-node. |
| common/reviews/session-memory.public.api.md | Checks in regenerated API report baseline for @fledgling/session-memory. |
| common/reviews/session-log.public.api.md | Checks in regenerated API report baseline for @fledgling/session-log. |
| common/reviews/session-local-storage.public.api.md | Checks in regenerated API report baseline for @fledgling/session-local-storage. |
| common/reviews/session-file-system.public.api.md | Checks in regenerated API report baseline for @fledgling/session-file-system. |
| common/reviews/mcp-workspace.public.api.md | Checks in regenerated API report baseline for @fledgling/mcp-workspace. |
| common/reviews/mcp-workspace-webcontainer.public.api.md | Checks in regenerated API report baseline for @fledgling/mcp-workspace-webcontainer. |
| common/reviews/mcp-workspace-browser.public.api.md | Checks in regenerated API report baseline for @fledgling/mcp-workspace-browser. |
| common/reviews/context-builder.public.api.md | Checks in regenerated API report baseline for @fledgling/context-builder. |
| common/reviews/common.public.api.md | Checks in regenerated API report baseline for @fledgling/common. |
| common/reviews/agent-core.public.api.md | Checks in regenerated API report baseline for @fledgling/agent-core. |
| common/reviews/acp-host-log.public.api.md | Checks in regenerated API report baseline for @fledgling/acp-host-log. |
| common/reviews/acp-agent.public.api.md | Checks in regenerated API report baseline for @fledgling/acp-agent. |
| common/config/rush/pnpm-lock.yaml | Updates lockfile for new API Extractor + Heft plugin dependencies. |
Files not reviewed (1)
- common/config/rush/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
common/reviews.@fledgling/tools-mcp-nodeand@fledgling/workspace-nodepod.Validation
node common\scripts\install-run-rush.js updatenode common\scripts\install-run-rush.js buildrg -n "\(undocumented\)|Warning:|No @packageDocumentation" common\reviewsreturned no matches