Skip to content

fix(docs): isolate TypeScript compiler API - #659

Merged
ScriptedAlchemy merged 1 commit into
mainfrom
fix/612-typescript-js-api
Sep 6, 2026
Merged

fix(docs): isolate TypeScript compiler API#659
ScriptedAlchemy merged 1 commit into
mainfrom
fix/612-typescript-js-api

Conversation

@ScriptedAlchemy

@ScriptedAlchemy ScriptedAlchemy commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • run the website typecheck with native TypeScript 7.0.2 while rspress.config.ts imports a typescript-6 compiler-API alias
  • convert the TypeDoc/twoslash TypeScript peers into scoped dependencies pinned to 6.0.3, preventing them from resolving TypeScript 7
  • remove the temporary Dependabot major-version ignore and update the pin reference

Refs #612.

Local merge gate

Branch HEAD 5117aa75ed4bc96faa440fddee17c65609033200 contains current origin/main 46e8e7d25a3f7c0db26c8d13c1bbdf978cdcdc50.

  • 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.2
  • pnpm docs:site:build — passed; 110 sitemap pages, 0 broken links, 28,127 anchors checked, TypeDoc API generated, twoslash hover markup rendered
  • pnpm install --frozen-lockfile — passed
  • pnpm why typescript -r — TypeDoc/twoslash JS-API consumers resolve 6.0.3; website/root native typecheckers resolve 7.0.2; no peer dependency issues

The 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.cjs was initially untracked while its checksum was in the lockfile: fixed by staging and committing it with the lockfile.
  • docs/effect-conventions.md still described a website-wide TypeScript 6 pin: fixed to document native 7 plus the isolated 6.0.3 compiler API.
  • Pin drift risk: the reference now explicitly requires updating the alias and scoped overrides together.
  • Final pass on current 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).

@changeset-bot

changeset-bot Bot commented Sep 6, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 5117aa7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@ScriptedAlchemy ScriptedAlchemy added the skip-changeset PR changes a publishable package but ships no observable change; changeset not required label Sep 6, 2026
@pkg-pr-new

pkg-pr-new Bot commented Sep 6, 2026

Copy link
Copy Markdown
npm i https://pkg.pr.new/ScriptedAlchemy/agent-bundle@659
npm i https://pkg.pr.new/ScriptedAlchemy/agent-bundle/create-agent-bundle@659
npm i https://pkg.pr.new/ScriptedAlchemy/agent-bundle/rsc-markdown-stream@659
npm i https://pkg.pr.new/ScriptedAlchemy/agent-bundle/@agent-bundle/runtime@659

commit: a85e9f2

@ScriptedAlchemy
ScriptedAlchemy force-pushed the fix/612-typescript-js-api branch from a85e9f2 to 5117aa7 Compare September 6, 2026 01:09
@ScriptedAlchemy
ScriptedAlchemy merged commit 461a6dc into main Sep 6, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changeset PR changes a publishable package but ships no observable change; changeset not required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant