Skip to content

build(release): run only docs:typedoc in preversion#10

Merged
enixCode merged 1 commit into
mainfrom
build/preversion-typedoc-only
May 29, 2026
Merged

build(release): run only docs:typedoc in preversion#10
enixCode merged 1 commit into
mainfrom
build/preversion-typedoc-only

Conversation

@enixCode
Copy link
Copy Markdown
Owner

What

The preversion npm hook ran the full npm run docs (typedoc + VitePress build) before every npm version. That is wrong on two counts:

  • The VitePress build emits docs/.vitepress/dist, which is gitignored and never committed, and the published site is rebuilt + deployed by the Docs CI workflow on push to main. Building it during npm version is pure redundancy.
  • It breaks npm version on a OneDrive-synced working tree: the cloud-dehydrated .vitepress files fail to read/build, so the hook (and thus the release) aborts. The 0.15.0 release had to route around it with a manual bump.

Change

preversion now runs only docs:typedoc, which regenerates and stages the committed docs/api/ markdown (the hook's actual purpose). The site is still built and deployed by CI.

build with cc

- The preversion hook ran the full `npm run docs` (typedoc + VitePress build).
  The VitePress build emits a gitignored artifact (docs/.vitepress/dist) that
  is never committed and is rebuilt + deployed by the Docs CI workflow, so
  running it during `npm version` is pure redundancy.
- It also breaks `npm version` on a OneDrive-synced working tree, where the
  cloud-dehydrated .vitepress files fail to read/build.
- Keep only docs:typedoc, which regenerates and stages the committed docs/api
  markdown. The published site is still built by CI on push to main.

build with cc
@enixCode enixCode merged commit 5be4f71 into main May 29, 2026
4 checks passed
@enixCode enixCode deleted the build/preversion-typedoc-only branch May 29, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant