fix(docs): isolate TypeScript compiler API - #659
Merged
Conversation
|
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
commit: |
ScriptedAlchemy
force-pushed
the
fix/612-typescript-js-api
branch
from
September 6, 2026 01:09
a85e9f2 to
5117aa7
Compare
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
rspress.config.tsimports atypescript-6compiler-API aliasRefs #612.
Local merge gate
Branch HEAD
5117aa75ed4bc96faa440fddee17c65609033200contains currentorigin/main46e8e7d25a3f7c0db26c8d13c1bbdf978cdcdc50.pnpm build && pnpm typecheck && pnpm lint && pnpm test:unit— passed; linted 1,421 files; 283 test files, 4,229 tests, 0 failures (4,223 passed, 6 skipped)pnpm --filter @agent-bundle/docs typecheck— passed under native TypeScript 7.0.2pnpm docs:site:build— passed; 110 sitemap pages, 0 broken links, 28,127 anchors checked, TypeDoc API generated, twoslash hover markup renderedpnpm install --frozen-lockfile— passedpnpm why typescript -r— TypeDoc/twoslash JS-API consumers resolve 6.0.3; website/root native typecheckers resolve 7.0.2; no peer dependency issuesThe page count (110) and broken-link count (0) match #610's acceptance numbers.
Deslop
Deslop: GPT-5.6 Sol, 1 edit (removed a redundant package-extension layer from the final dependency wiring).
Self-review
Reviewer: Claude Fable 5.1 Thinking High.
.pnpmfile.cjswas initially untracked while its checksum was in the lockfile: fixed by staging and committing it with the lockfile.docs/effect-conventions.mdstill described a website-wide TypeScript 6 pin: fixed to document native 7 plus the isolated 6.0.3 compiler API.origin/main: no merge blockers; patch scope is exactly the seven intended files and the pnpmfile checksum matches the lockfile.No changeset: only private website and repository tooling/configuration changed (
skip-changeset).